useServiceAccountAuth causing infinite loop

why I try to use
const { GoogleSpreadsheet } = require('google-spreadsheet');
const doc = new GoogleSpreadshee(id);
doc.userServiceAccountAuth(require("./creds.json"));

In my electron app, it just freezes completely and uses 100% of my CPU.
creds.json
{
  "type": "service_account",
  "project_id": "string",
  "private_key_id": "string",
  "private_key": "string",
  "client_email": "string",
  "client_id": "string",
  "auth_uri": "https://accounts.google.com/o/oauth2/auth",
  "token_uri": "https://oauth2.googleapis.com/token",
  "auth_provider_x509_cert_url": "https://www.googleapis.com/oauth2/v1/certs",
  "client_x509_cert_url": "string"
}

package-lock.json entry
    "google-spreadsheet": {
      "version": "3.1.15",
      "resolved": "https://registry.npmjs.org/google-spreadsheet/-/google-spreadsheet-3.1.15.tgz",
      "integrity": "sha512-S5477f3Gf3Mz6AXgCw7dbaYnzu5aHou1AX4sDqrGboQWnAytkxqJGKQiXN+zzRTTcYzSTJCe0g7KqCPZO9xiOw==",
      "requires": {
        "axios": "^0.21.1",
        "google-auth-library": "^6.1.3",
        "lodash": "^4.17.20"
      },
      "dependencies": {
        "lodash": {
          "version": "4.17.21",
          "resolved": "https://registry.npmjs.org/lodash/-/lodash-4.17.21.tgz",
          "integrity": "sha512-v2kDEe57lecTulaDIuNTPy3Ry4gLGJ6Z1O3vE1krgXZNrsQ+LFTGHVxVjcXPs17LhbZVGedAJv8XZ1tvj5FvSg=="
        }
      }
    },

you need await, either by using async/await or promise.then
You should then be able to catch any errors and see what's going on.
Also, try going through the instructions again and make sure your service account creds have been generated correctly

closing - feel free to reopen with more info if still a problem


[nrf toup][nrfconnect] Added BT reassembly config to BT SMP samples

This PR adds BT reassembly config to SMP DFU samples as it is recommended

none
Note: This comment is automatically posted and updated by the Contribs GitHub Action.

Maybe it's better to first add it to upstream and then pull in as [nrf fromtree.]

Maybe it's better to first add it to upstream and then pull in as [nrf fromtree.]

@markaj-nordic I don't know what are pluses and minuses of this approach so I can change it if you guys agree on that.

Maybe it's better to first add it to upstream and then pull in as [nrf fromtree.]

@markaj-nordic I don't know what are pluses and minuses of this approach so I can change it if you guys agree on that.

For me the rule of thumb for [toup] is the need of urgent patch/fix in our SDK. This change is just a minor improvement, so I'd suggest to go upmerge first. On the other hand, it's just our platform being affected, so I maybe I'm too picky.


Texture problem with detached DOM elements

When you create a texture from a DOM element, for example a canvas, if this element is not attached to the document, the texture will not work.
The reason is texture size is read from the element's offsetWidth/offsetHeight properties, and these properties are always 0 for DOM detached elements; hence texture size will be (0,0).
			boolean isImagePowerOfTwo = Mathematics.isPowerOfTwo( image.getOffsetWidth() )
					&& Mathematics.isPowerOfTwo( image.getOffsetHeight() );

(NB: I pasted these few lines of code but same problem in others methods.)
Three.js read width/height properties instead, which is nice because you must fill these properties when you create a canvas to use it as a texture; thus you dont need to attach the canvas to the document.
One possible solution would be to first read width/height properties, then having a fallback on offsetWidth/Height (I can provide a patch if you wish and if you agree on this solution).

Here is my texture generation code:
    private Texture createTextTexture(double size, double margin, String text, Color color, Color backColor) {
        if (text == null)
            return null;
        Canvas canvas = Canvas.createIfSupported();
        Context2d context = canvas.getContext2d();
        double width = size + margin;
        double height = size + margin;
        canvas.setCoordinateSpaceWidth((int) width);
        canvas.setCoordinateSpaceHeight((int) height);
        context.setFont(size + "pt Arial");
        context.setFillStyle("#" + getHexString(backColor));
        context.fillRect(0, 0, width, height);
        context.setTextAlign("center");
        context.setTextBaseline("middle");
        context.setFillStyle("#" + getHexString(color));
        context.fillText(text, width / 2, height / 2);

        // Without the fix, this line is mandatory, otherwise the texture wont display
//        RootPanel.get().add(canvas, -10000, 0);

        Texture texture = new Texture(canvas.getElement());
        texture.setNeedsUpdate(true);
        return texture;
    }

Example usage:
createTextTexture(10, 20, "0", new Color(0xaaaaaa), new Color(0x202020));

I think it would be a nice addition, as rendering text is easy with canvas and well supported by browsers, but not everybody will have the idea to use it.
There are a few choices to consider:

Should it handle multiline text (this is not builtin in canvas but can be achieved with canvas' measureText())
Should the texture size be computed automatically based on font size and margin, as a power of 2
I'm a WebGL beginner and I wouldn't be able to tell what are the most common use cases for text textures, but the class should be easy to extends to cover custom use cases.


Make imports fall back on inserting into document unless ending in .less

so if the import does not end in .less and cannot be read as a file we should fallback on inserting it as an @import.
This came up on the mailing list when someone wanted to import
@import "http://fonts.googleapis.com/css?family=Oswald";
the work-around is to do...
@import "http://fonts.googleapis.com/css?family=Oswald&ignore=.css";
But this fixes this issue. See the unit tests added for more detail.

Done


Document the pypi-dependencies feature

Docs we need:

[ ] The version spec that we are using.
[ ] CLI
[ ] What is still missing, git dependencies and such. To avoid a stream of questions.

A way to disable the ALPHA warning?:
 WARN pixi::project: ALPHA feature enabled!

It looks like your project contains `[pypi-dependencies]`. This feature is currently still in an ALPHA state!

You may encounter bugs or weird behavior. Please report any and all issues you encounter on our github repository:

	https://github.com/prefix-dev/pixi.

I think we are progressing quite rapidly with rip. We could think about making the warning smaller, and make it disappear all the way at the end of the year.
Or we could store on disk that we have already shown the warning and only show it once.

Yeah one more big release and it should be gone. The adjacent milestone to this issue describes the steps until we will let go of the warning.


407 error when downloading via proxy agent

Describe the bug and how to reproduce it
I have a proxy agent that I can call getBasicInfo() with (which I had to upgrade udici to 6.21.0). It works as expected.
But when I try to download a video, it will await for about 5 or 6 seconds and then throw an error with http status code 407.
In my proxy dashboard, I see my bandwidth use going up, so it seems like it is successful at some point in the process.
Here is my stack trace:
Uncaught Exception: MinigetError: Status code: 407
    at ClientRequest.<anonymous> (/home/thing/service/node_modules/@distube/ytdl-core/node_modules/miniget/src/index.ts:255:19)
    at Object.onceWrapper (node:events:633:26)
    at ClientRequest.emit (node:events:518:28)
    at HTTPParser.parserOnIncomingClient (node:_http_client:702:27)
    at HTTPParser.parserOnHeadersComplete (node:_http_common:118:17)
    at Socket.socketOnData (node:_http_client:544:22)
    at Socket.emit (node:events:518:28)
    at Readable.read (node:internal/streams/readable:782:10)
    at Socket.read (node:net:777:39)
    at flow (node:internal/streams/readable:1283:53) {
      statusCode: 407
}

Environment

@distube/ytdl-core version: 4.15.6
Node.js version: 20
Operating system: linux/mac

I've verified that downloading works via the proxy I am using when I download via youtube-dl-exec, so it looks like the problem is related to something that occurs in this repo or with one of its dependencies.


UtilityAIArea3DVisibilitySensor never finds unoccluded areas

This might be my misunderstanding, but based on the documentation, if do_occlusion_test == true and the collision_mask is set to something that could be blocking a line of sight to an area (ex. setting it to 1 for scenery), then I should see unoccluded results in unoccluded_areas.
In my scene, I have these settings set as defined above, and have an area that is being correctly detected as overlapping a couple of meters away from the origin point and not being blocked by anything. When num_entities_found is checked, no entities are found. I don't know CPP or how Godot-CPP is supposed to work, but this if statement seems potentially wrong? Based on my understanding, I would expect this:
Dictionary results = dss->intersect_ray( params );
if( !results.is_empty() ) { <-- this currently is checking if results _is_ empty
    _unoccluded_areas.push_back(area);
    _squared_distances_to_unoccluded_areas.push_back(distance_squared);
    if( _closest_unoccluded_area_index == -1 ) {
        _closest_unoccluded_area_index = found_unoccluded_areas;
        closest_unoccluded_area_distance = distance_squared;
    } else if( closest_unoccluded_area_distance > distance_squared ) {
        _closest_unoccluded_area_index = found_unoccluded_areas;
        closest_unoccluded_area_distance = distance_squared;
    }
    ++found_unoccluded_areas;
    ++_num_entities_found;
}

since the body of the statement adds the area to the unoccluded areas and increments them. I saw the same logic in the 2D visibility sensor as well.

Looking at it more closely, I was wrong about the !results.is_empty() call, because the occluding collision mask is what is being tested with the raycast.

Hi, thanks for creating an issue about this question and commenting your conclusion, as it may help others in a similar situation. Good luck with your project!


New GDG Logo (2019)

Can we get an updated version of the tool that includes the new GDG logo unveiled march 2019?

You can find here new logo; https://twitter.com/gdg/status/1111701428228313093?s=20

Right now I do not have time to update it. But PRs are welcome!


Header and hero mobile first

Completed the Styling for the first two section of my mobile first portfolio

I tried to fix the error raised by the linter by putting a space at the expected place, but the moment I save the prettier plugin in VSCode returned the line to the initial state

Hi Favour,
Thanks for your feedback, I have tried to add a title and summary, but I do not see any way to do that

Completed styling the header and the hero section

updated the HTML by adding appropriate tags and classes
styled the page
Fix the error thrown by the linter

Can I have two nav tags in the header as suggested by the reviewer


Example of ES6 promises for Angular-style code

Here's an addition to the examples directory with a writeup.

This would be awesome as a blog post. If you have a good place to post a tutorial, I'd love to just link to that instead of bloating the examples with something much more thorough and opinionated than the other examples.

Ok, sound good. Here’s the full explanation:
https://github.com/pauleveritt/sap-then-assert https://github.com/pauleveritt/sap-then-assert
It’s a longer version of what I hoped was a non-bloated minimum case in your docs. :)
—Paul

On Aug 17, 2015, at 2:16 PM, Ben Drucker<EMAIL_ADDRESS>wrote:
This would be awesome as a blog post. If you have a good place to post a tutorial, I'd love to just link to that instead of bloating the examples with something much more thorough and opinionated than the other examples.
—
Reply to this email directly or view it on GitHub https://github.com/bendrucker/sinon-as-promised/pull/28#issuecomment-131914040.

:+1: If you're cool doing it as a blog post that'd be my preference. It's good info for a very specific subset of use cases and I think it'll go unnoticed in an examples folder.


Fix appveyor.

Had an error wherein the right node version never got installed... Also add node10.

Coverage remained the same at 94.616% when pulling c7cd8d9c6452be30b3b0a51f2b983ab6bff08efd on fix-appveyor into 5f0348a28b328acd2c96b2a5b734ec5a592a22b1 on master.

Confirmed - node6 installs node6
Installing node 6.14.2 (x86)...


Add variable parsing to actions & feedbacks

accept & parse variables for most action options
accept & parse variables for feedback event id
Use Node 22
Minor refactors
Use InstanceStatus enum for status updates.

Still needs to be tested against a unit.

So you do not have an Elemental to test this on?

We have multiple, but they are normally managed by a conductor so can only test this when there is a reasonable downtime in production.


Add New Resource: NHAC

New Resource Request
We received an email to add "Nursing Home Abuse Center (NHAC)" to our health resources page.
https://www.nursinghomeabusecenter.com/
https://techqueria.org/resources/health/
Email
I came across Techqueria while searching for organizations with the common goal of supporting the LGBT community.
Our main focus is spreading awareness to LGBT seniors being mistreated and often abused in nursing homes and other support facilities.
There are approximately 3 million LGBT seniors over the age of 65, and this number is projected to double by 2030.
And sadly, as more LGBT community members come out, there are more abuse cases being reported especially within the senior community.
Of particular concern is housing discrimination, both in terms of access to housing and the treatment they receive once there.
Access to safe and equal-opportunity housing is a fundamental right, but in practice, LGBT members face larger obstacles in finding comfortable living situations.
As the Director of Advocacy at the Nursing Home Abuse Center (NHAC) my goal is to ensure that our message reaches the people who need to see it the most.
I’m asking you for help with this.
By adding us to your resource page here - https://techqueria.org/resources/health/ it would go a long way towards achieving our goal of spreading awareness about these issues.
I truly appreciate your support.

✅ By submitting this issue, I have verified the following

[x] Checked to see if the issue has already been discussed before 🤔
[x] Made sure enough details were provided if proposing a new feature 🔍

Hi @fvcproductions! Was this issue solved in the unmerged PR you submitted 3 days ago? If not, I'd like to get it done. 👍

@enriquenov Yeah I'm still working on it, PR should be merged in soon once I fix the formatting issues 😄


Outputting A Format Of "%U%n" To List All PR URLs Doesn't Work

Command attempted:
hub pr list -f "%U%n"
What happened:
It outputs:
myusername"https://github.com/myusername/myrepo/pull/1250
https://github.com/myusername/myrepo/pull/1249
https://github.com/myusername/myrepo/pull/1248
https://github.com/myusername/myrepo/pull/1234
https://github.com/myusername/myrepo/pull/1116

It should output:
https://github.com/myusername/myrepo/pull/1250
https://github.com/myusername/myrepo/pull/1249
https://github.com/myusername/myrepo/pull/1248
https://github.com/myusername/myrepo/pull/1234
https://github.com/myusername/myrepo/pull/1116

For what it's worth, the %U flag does work on occasion.  For example, if I do:
hub pr list -f "%>(50)%U%n"
What I get is:
  https://github.com/myusername/myrepo/pull/1250
  https://github.com/myusername/myrepo/pull/1249
  https://github.com/myusername/myrepo/pull/1248
  https://github.com/myusername/myrepo/pull/1234
  https://github.com/myusername/myrepo/pull/1116

More info:
git version 2.22.0
hub version 2.14.2

Hi thank you for reporting! So it looks like the unwanted piece of output is this on the first line:
myusername"

And you're saying that this doesn't happen all the time, i.e. the same format string sometimes works? Or do you have to change the format string for this to work

The same format string never works.  Yes the unwanted string is myusername".  When I said the %U sometimes works, I was pointing out that in other configurations it sometimes works.  In the %U%n configuration, it never works.

@jfelchner Can you reproduce this in other shells, or when you use single quotes?
hub pr list -f `%U%n`

I'm wondering whether your shell interprets %U somehwo


Inconsistent defaults for lower keyword in flapack.pyf.src

In the flapack.pyf.src there are many routines with upper/lower keywords to decide which part of the matrix is used for symmetric matrices.
However, while the largest part of the routines default to lower == 'U', some are defaulting to lower == 'L':

sytf2 (https://github.com/scipy/scipy/blob/master/scipy/linalg/flapack.pyf.src#L660)
sytrf (https://github.com/scipy/scipy/blob/master/scipy/linalg/flapack.pyf.src#L680)
hetrf (https://github.com/scipy/scipy/blob/master/scipy/linalg/flapack.pyf.src#L884)

I think this is a mistake and should be corrected.
I will be happy to provide a PR if you agree to the change.

Breaks backward compatibility. I think trying to change this is more
pain than profit, so probably not worth it.

sytrf and hetrf are for LDL decomposition and by definition their naming the default is lower. That's why.

Speaking of which I think I should recycle the code in #6831 since we are not going past LAPACK 3.5.0 anytime soon and by the time Python 2 support is gone,  newer routines with new keywords can be added without any compatibility problems.

@ilayn thanks for the clarification. However, I do not see the argument. sytrf can compute LDL or UDU.
In my opinion should a routines defaults not be decided by what it is intended for when adding it. It should adhere to some kind of standard across the package. And currently all routines are defaulting the upper part.
@pv the routines have just been added in #7851, so it hasn't been released yet.

There are no standards or defaults in LAPACK routines by their design only orders. Since these are low level and it goes by the name unanimously LDLt decompositions instead of Bunch/Kaufmann the natural choice is better than a consistency that doesn't exist. Note that these are the wrapper defaults not the LAPACK  defaults so there is no choice involved.

Yes, there are no standards in LAPACK, however, the scipy.linalg.lapack is a package and it does implement a standard. I know that these are low level, but if I see 10 routines having lower=False by default, then I would expect it to be the default for all routines.
Consider I wan'to to compare the factorizations from a Cholesky and a LDL decomposition, then I have to change the arguments of one of them, but not the other:
  a_po = scipy.linalg.potrf(A, lower=True)
  a_sy = scipy.linalg.sytrf(A)

Yes but note that you can't use the upper part of the array and obtain an LDLt factorization. That keyword defines both the array side AND the factorization structure. It is a shortcoming of the LAPACK design which I've discussed some time ago on LAPACK board that seemingly doesn't exist anymore.

I agree. But that is beyond my argument. sytrf may also be used in other instances where LDLt is not wanted.
So again, I think the argument of defaulting to upper is that all other routines default to upper. This choice, I agree, is arbitrary. But at least it is a choice.

I also agree with that but what I mean is that this is also your choice. Consistency among only one of the arguments across all wraps does not exist. So I think this what you would like to see instead of consistency. Because sy/hetrf is pretty standard for indefinite factorization and you expect to have the lower as the default (because that's how matlab/octave conditioned people). Many users won't even know the difference since 99% of the cases users supply the symmetric array. And the ones who have the triangular array will check the signature anyways.

But we are in the low-level part of linear algebra routines. So people won't just use it. They have to dig and implement get_lapack_funcs etc. to even call it.
If this is a commonly encountered problem, then it is probably better to add a def ldl routine which returns postprocessed sytrf(..,lower=True).
Something like this:
def ldl(a, overwrite_a=False, check_finite=True):
    if check_finite:
        a1 = asarray_chkfinite(a)
    else:
        a1 = asarray(a)
    if len(a1.shape) != 2:
        raise ValueError('expected matrix')
    overwrite_a = overwrite_a or (_datacopied(a1, a))
    sytrf = get_lapack_funcs('sytrf', (a1,))
    LDLt, ipiv, info = sytrf(a1, overwrite_a=overwrite_a)
    if info < 0:
        raise ValueError('illegal value in %d-th argument of '
                                            'internal lu.getrf' % -info)
    if info > 0:
        warn("Diagonal number %d is exactly zero. Singular matrix." % info,
                    RuntimeWarning)
    ipiv -= 1
    L = np.tril(LDLt, -1)
    D = np.diag(LDLt)
    return L, D, ipiv

@pv what do you think? Also, as I mentioned, no backwards compatibility broken, nothing has been released.

Also, sytrf is also used for solving systems where either upper or lower parts are used. There, there are no defaults. So why not be consisent? ;)
PS. Thanks for the discussion (albeit we disagree ;) )

No problem at all, I think these are quite productive discussions. The API is probably the most important part of the packages. I'm just trying to keep the intuitiveness or user-friendliness (whatever that means) of the routines because LAPACK already makes life difficult with its compact storage schemes. Thus, usage of these wrappers are probably pretty rare. That's why I assumed the position that each of them should make sense for their own right. Otherwise there will be yet another set of standards for the user to learn on top of LAPACK but depends on what we aim for.
I already prepared the ldl part (though it takes surprisingly much more effort to get L and D out from the array than your proposition, see #6831with ignoring the rook parts). I'll send a PR with the rook parts removed.

;) Thanks! I think so too.
My position is that the flapack sources are low level and that any users would need to read the documentation to proceed using them. None of the default choices makes sense, per see. They are arbitrary. However, they should be consistent.
As such I don't think we should impose decisions on what they are intended for, regardless of popularity. They are baseline implementation that adheres to an internal decision on defaults. Any usage of these functions that are popular should correspondingly be created as a python method (great you are looking at a ldl). And hence the problem becomes that of the implementor of the ldl method, and will never be a problem of the user.

If the routines haven't been in any release yet, then I don't have a
preference, though would be nicer to be consistent. Ultimately, people
are able to read the documentation and these low-level routines in any
case require care to use correctly.
But it's getting late for 1.0.0, there's about two days time left, and I
don't think we'll want to delay the release schedule for this so best to
hurry up if you agree.

OK then, @zerothi do you have time to make the PR ready soon? Because I can make it to this evening the whole wrapper and tests (they are ready anyways) and I'd like to base it on your change then to avoid commit clashes.

Great @ilayn I can make it now.
@pv I am not clear on how you do these things? Are you merging 1.0.x branch into master, or are you keeping them strictly separate?
If so, let me know and I will create a PR for 1.0.x also. See #7929 for a PR on master

As a main rule, everything goes to master, any commits to be backported
to maintenance branches are picked by the release manager.

thanks for the clarification.

Anything left to do here, or closed by gh-7929?

Yes this is fixed in #7929


Use $this->t as default for translatable fields

Fixes #24
This PR changes the default translatable value style from t() `$this->t('').
This also prevents DrupalPractice.Objects.GlobalFunction.GlobalFunction warnings from showing when linting with best practises.
Users will be required to import \Drupal\Core\StringTranslation\StringTranslationTrait, if it does not already exist in the class.
#17 suggest you would like to kept the extension compatable only with supported Drupal versions and best practises.

Thanks for the issue & solution!
The main concernI have with this approach is it requires the code you're writing to be in a class. While that may be your general use case, I know for a fact it is not mine. The majority of the time I'm putting in new elements and would use these snippets I'm doing so in a hook, where $this would be undefined.
I'd also be concerned about novice Drupalers accepting the default and then struggling when $this->t() causes errors.
Let me ping some mentors in the community and get their thoughts on this.

@andy-blum thanks for the detailed explaination.
I know of your long time contributions to Drupal (and DDEV) so I knew there was must have been a good reason. I'm fairly new to Drupal and tend to work in module development.

I'd also be concerned about novice Drupalers accepting the default and then struggling when $this->t() causes errors.

This is definately something to be concerned about. Drupal can be overwhelming for novices and we certainly don't want to make it harder.
When I was first writing up the issue, I thought it would be a good case for a configuration option: set your prefered style. I'm not sure VSCode has a hook feature for snippets though.
I'm more than happy to go in what ever direction your experience suggest will work best! Thanks for just considering it. 😄

My overly ambitious long-term goals would be to find a way to tap into a language server like PHP Intelephense to be able to create snippets on the fly.
If that were possible, we could definitely push for $this->t() when in classes, as well as pull in hooks/elements/services defined in contrib/custom code.

I think I'm with @andy-blum on this one - switching to $this->t() will probably confuse newer users when they're writing hooks (including preprocess functions).
I like it the way it is right now, as it defaults to a solution (t()) that works in areas where novice developers typically spend time - in hooks and preprocess functions.
As is discussed above, it would be great if this could depend on the context (in/not in a class) and adjust on the fly. But, in lieu of that capability (currently), I think sticking with just t() is prudent.
-mike

Thanks again @tyler36 for the recommendation. For now, I'm opting to leave this as-is.

Thank you for taking the time consider it.


Add BufferingPriceEstimator

In order to enable our frontend to get faster price estimates #1659 got merged. The idea is to send 2 price estimation requests at the same time (one for fast prices, one for optimal prices) and let the frontend display the prices as they come in.
This will lead to many duplicated identical price estimation requests which can result in us getting rate limited by external services more often.
The new BufferingPriceEstimator wraps around any Box<dyn PriceEstimating> and keeps track of all the requests which are currently in flight. This does not happen on a batch basis but on a Query basis. When it gets asked to estimate a batch of queries it only actually calls the wrapped dyn PriceEstimating for the queries which are not already in flight.
This is accomplished by creating a Shared future for each Query which simply awaits the original batch future and returns the single result which corresponds to the Query. We store WeakShared version of each one and await the Shared future instead of the underlying batch request to avoid invalidating the WeakShared immediately.
This has also the effect that even if a price estimator decides that it no longer wants to make progress on a price estimation (which happens in the RacingCompetitionPriceEstimator) price estimations which depend on that batch can still poll that batch to completion on their own.
To not waste any memory the BufferingPriceEstimator collects garbage (i.e. completed futures or one which nobody holds on to anymore) after each batch call.
Note that this PR on its own does nothing. To actually be effective the creation of the price estimators in the orderbook main, needs to be refactored such that the fast and optimal fee calculator use the same instance of each estimator wrapped in a BufferingPriceEstimator. This will happen in a follow up PR.
Test Plan
Added a unit test which for garbage collection and finishing a request which got dropped by the original initiator.

I'm now using a single critical section.
Also I now store WeakShared instead of Shared. In order to not make the code too awkward because of it, I decided to not await the single batch estimation request but await each individual request (which is just an adapter around the batch request). Otherwise I would have to keep the individual queries alive artificially in order to not make the stored WeakShared immediately unusable.
Every WeakShared which can not be upgraded or already finished is now considered garbage.
One issue I thought about is the timing of garbage collection. Garbage collection can basically happen at any time if you consider that multiple requests can happen in parallel. With that in mind there is really no "right" time to collect garbage (also we now have the lifetimes of the WeakShared futures in order). That means we might as well collect garbage during the big critical section to avoid wasting time acquiring the lock one more time.
What do you think about that idea?

That means we might as well collect garbage during the big critical section to avoid wasting time acquiring the lock one more time.

Yes I thought about this too. If the code stays the way it is now where we go through all futures in collect_garbage then I agree. On the other hand if we do something like the drop guard checking only those queries that were used then there would be a small difference.

Codecov Report

Merging #1677 (b323cb1) into main (214ab96) will increase coverage by 0.18%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main    #1677      +/-   ##
==========================================
+ Coverage   65.57%   65.76%   +0.18%
==========================================
  Files         176      178       +2
  Lines       35013    35303     +290
==========================================
+ Hits        22961    23218     +257
- Misses      12052    12085      +33


Replace audio timestamp system with fft audio analysis.

Replaced the code for the audio timestamps with the Fast Fourier Transform, using the spectrum analyzer effect.
Performance impact is medium, for me it adds between 0.22 and 0.88 ms per frame, with the frame taking 18.69 ms total and most other function calls between 0.03-0.17 ms.
This should also make it much easier to add new songs, as all that is needed is to preload the file and add it to the array.
Potential optimization with disabling the FFT when the setting is set to 0.

I'll check this out in a few hours. If it works well I might add more songs when I renew my Epidemicsound subscription

LGTM


Fix function name to be compatible with bash

Script function name is not valid for bash, replacing "-" for "_"
Closes #446

This PR is changing indentation across the whole file. Is there any reason for it?

Applying shfmt over it via pre-commit :)
Pablo Iranzo Gómez
GnuPG: 0x5BD8E1E4
Web: https://iranzo.io  ||  https://t.me/redken_bot
El jue, 11 nov 2021 a las 12:56, Ricardo Noriega De Soto (<
@.***>) escribió:

This PR is changing indentation across the whole file. Is there any reason
for it?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/redhat-et/microshift/pull/445#issuecomment-966243140,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AACMJD2YVWURQGZXITELICTULOVQRANCNFSM5H2ES2EQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Applying shfmt over it via pre-commit :) -- Pablo Iranzo Gómez GnuPG: 0x5BD8E1E4 Web: https://iranzo.io || https://t.me/redken_bot El jue, 11 nov 2021 a las 12:56, Ricardo Noriega De Soto (< @.***>) escribió:
…
This PR is changing indentation across the whole file. Is there any reason for it? — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#445 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AACMJD2YVWURQGZXITELICTULOVQRANCNFSM5H2ES2EQ . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

We try to keep consistency of doing one thing per PR, and the indentation changes were not stated in the description. That's why I'm asking... :-)

/assign oglok

/ok-to-test

Script function name is not valid for bash, replacing "-" for "_" and shfmt -i 4 -w over the file

Is this causing an error that may be respective of your version of bash? Or was this just reported by shfmt as being non-idiomatic of bash function names?  AFAICT, bash has no issues with hyphenated function names, and doesn't mention any special character other than $ as being off limits.

Going to close as we will be deprecating install.sh in the near future


impl(bigquery): Update json parsing for time fields

This PR fixes https://github.com/googleapis/google-cloud-cpp/issues/12268
Currently all time fields in the response are expected to be numbers for parsing into chrono fields, but in actual they are being sent as strings (or numbers within quotes).
This PR modifies the to_json and `from_json1 methods for chrono fields to accomodate both strings and numbers as it is not guaranteed that all time fields would be sent as strings in the future as well.

This change is

/gcbrun

My apologies! I didn't see the comments. I'll send a separate PR for it!

google/cloud/bigquery/v2/minimal/internal/json_utils.cc line 42 at r1 (raw file):
Previously, jsrinnn wrote…
The intention was two fold

To keep the changes to minimal

Its not guaranteed all time and numbers would always be sent as strings. Hence tried to accomodate both.

I understand you need to accept both numbers and strings.  The question is more about the ToJson() functions.  Seems like you only need to support one outgoing format, and might as well make it a string.  Put another way: other than in tests, it seems that is_number will have a fixed value?

google/cloud/bigquery/v2/minimal/internal/json_utils.cc line 42 at r1 (raw file):
Previously, jsrinnn wrote…
Thats right. At this point, ToJson() will be string for production code and number for just tests.
To clarify, are you saying

Remove is_number parameter and make ToJson() outgoing format as only string?

Modify all tests to output string (instead of number)?

I guess I was trying to avoid  (2) :-).

Yes, that is what I was saying.  I don't want to create unnecessary work.  I am sure you have more urgent things to do.


csound-manual: init at 6.12.0

Motivation for this change
These generated html files can be used in conjunction with CsoundQt.
Things done

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

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

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

@GrahamcOfBorg build csound-manual


fish: fix build with Darwin sandbox enabled

Description of changes
fish’s test suite has numerous failures when run on a Darwin machine with the sandbox enabled. The sandbox profile allows the test suite to run successfully.
Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[x] x86_64-darwin
[x] aarch64-darwin

[x] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

I didn't know sandboxProfile worked with sandbox = true (as opposed to relaxed)!

I don’ think it does. I’m using sandbox = relaxed. I guess I’ve gotten lazy about updating the message to reflect that I’m using relaxed instead of true. 😅

I said that because it worked fine with my configuration, where it is true... odd.

I said that because it worked fine with my configuration, where it is true... odd.

Huh. That is odd.


Experimental: styled prompt

This is only for Scale accounts:

Demonstrating how we can use various "styles" of response using custom prompts
All prompts need to be specifically defined in the target account, and only available to Scale customers
Selection of the prompt style occurs in the options drawer component.

Will not merge


getToken return empty string

Hi!
first of all thanks for this plugin!
the getToken method is returning an empty sting but the plugin is working fine with notifications. Also onTokenRefresh is not working (it is never fired).
So notifications are working but token methods are not working.
Utilities.logOnScreen simply print on device screen the message (only for my comfort)
Thank you in advance!
Cordova 6.4.0
IDE NetBeans 8.2
OS: Xubuntu 16.04
Tested on Virtual Devices:

android 4.0.3
android 6.0
Tested on Real device:
Huawuei P8 lite android 6.0

`
onDeviceReady: function () {
this.receivedEvent('deviceready');
window.FirebasePlugin.onTokenRefresh(function (token) {
    Utilities.logOnScreen('onTokenRefresh ' + token);
}, function (error) {
    Utilities.logOnScreen('onTokenRefresh: ' + error);
});

jQuery("#getToken").click(function () {
    window.FirebasePlugin.getToken(function (token) {
	Utilities.logOnScreen('getToken ' + token);
	console.log("token: " + token);
    }, function (error) {
	Utilities.logOnScreen('getToken error: ' + error);
    });
});

window.FirebasePlugin.onNotificationOpen(function (notification) {
    Utilities.logOnScreen('wasTapped ' + JSON.stringify(notification));
}, function (error) {
    Utilities.logOnScreen('else ' + JSON.stringify(error));

});

}

`

I did have the same problem,
mi solution was to fix the config.xml of the proyect changing the name of the <widget id="same proyect name at firebase">, rm the platform and validate that the google-services.json was not empty at the platform/android/

@shepax Hi, I did not understand your answer, you can put the code fragment where the change is made. Thank you very much for your help

This was the solution
Download your Firebase configuration files, GoogleService-Info.plist for ios and google-services.json for android, and place them in the root folder of your cordova project:

My Project/
platforms/
plugins/
www/
config.xml
google-services.json       <--
GoogleService-Info.plist   <--

I need to add those two files to the project and with that they return the tokens correctly

I am still having this issue. Updating google-services.json didn't work. @robertarnesson ?

@VesperDev  sorry for the bad exp,
When you build a project at firebase you have to define the iOS bundle ID or Android bundle ID, that bundle have to be the same that you have defined at the config.xml  of the ionic or cordoba project
<widget id="bundle id">

Still had the issue . please help

Hi,
I have the same issue. I placed the fcm config file in root folder and my widget id is same.
still  its getting empty token.

I am having the same issue, yesterday I was receiving the token correctly but today I am getting empty string.  maybe the server is down because it is not possible to access to my project in firebase console

anyone fix this? I have exactly same problem - and the bundle id and package name I have in Firebase is definitely equal to the value of widget id!

The problem is still there... any update on this issue !?

Another solution is here: https://github.com/arnesson/cordova-plugin-firebase/issues/558
To the author: A PR and a fix is URGENT now... ;-)

closing this for now. the more recent problems are not related. see #558 for android 7 issues


Switch to Maintained Matrix Lib

Should switch to using matrix-nio instead of matrix-python-sdk. 😄

(hopefully) fixed in 4131690fe59!


pressing "save and add another" in page edit causes error

For example http://comicframework.org/admin/comicmodels/page/40/
press save and add another -> server error.

Fixed in pages app


Adds new class, GunsClass

Holds all guns so It's easier to access for later on

Pull request needs to be synced and re-updated.


High CPU on the cadvisor container

Hi,
I have installed this package and it is looking great. I do see a high CPU usage of cadvisor :

Is this normal?
It is running on a RP4 with 4Gb on a 64Bit Bookworm OS.

Hello,
Thank you very much for highlighting this issue. After incorporating the new parameters into the cAdvisor command, the CPU usage has returned to normal.
Can you confirm ?

Hello,
Thank you very much for highlighting this issue. After incorporating the new parameters into the cAdvisor command, the CPU usage has returned to normal.
Can you confirm ?

Hi,
Thanks for this. I have made the change and it helped.


nixos/filesystems: skip fsck for bind mounts

Description of changes
Without this change, configurations like
fileSystems."/path/to/bindMountedDirectory" = {
  device = "/path/to/originalDirectory";
  options = [ "bind" ];
};

will lead to a warning message in dmesg:
systemd-fstab-generator: Checking was requested for "/path/to/originalDirectory", but it is not a device.

This happens because the generated /etc/fstab entry contains a non-zero fsck pass number, which doesn't make sense for a bind mount.
Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

@viraptor @ajs124 can you please review again?


Issue with DUF model.

When I run as follows:
python run.py --model=srcnn --dataset=MYDATA
python run.py --model=rcan --dataset=MYDATA
python run.py --model=cdn --dataset=MYDATA

There are no any problems.
But, When I run,
python run.py --model=duf --dataset=MYDATA
Traceback:
| 2019-06-01 17:14:55 | Epoch: 1/50 | LR: 0.0001 |
  0%|                                                                                | 0/200 [00:00<?, ?batch/s]
Traceback (most recent call last):
  File "run.py", line 46, in <module>
    tf.app.run(main)
  File "/home/ryluo/anaconda3/envs/tf12/lib/python3.6/site-packages/tensorflow/python/platform/app.py", line 125, in run
    _sys.exit(main(argv))
  File "run.py", line 42, in main
    return Run.run(*args[0][1:], **additional_functions)
  File "/home/ryluo/Desktop/VideoSuperResolution-master/VSR/Tools/Run.py", line 322, in run
    t.fit([train_loader, val_loader], train_config)
  File "/home/ryluo/Desktop/VideoSuperResolution-master/VSR/Framework/Trainer.py", line 356, in fit
    self.fn_train_each_epoch()
  File "/home/ryluo/Desktop/VideoSuperResolution-master/VSR/Framework/Trainer.py", line 261, in fn_train_each_epoch
    self.fn_train_each_step(label, feature, name, post)
  File "/home/ryluo/Desktop/VideoSuperResolution-master/VSR/Framework/Trainer.py", line 282, in fn_train_each_step
    epochs=v.epoch)
  File "/home/ryluo/Desktop/VideoSuperResolution-master/VSR/Framework/SuperResolution.py", line 173, in train_batch
    list(self.train_metric.values()) + loss, feed_dict=self.feed_dict)
  File "/home/ryluo/anaconda3/envs/tf12/lib/python3.6/site-packages/tensorflow/python/client/session.py", line 929, in run
    run_metadata_ptr)
  File "/home/ryluo/anaconda3/envs/tf12/lib/python3.6/site-packages/tensorflow/python/client/session.py", line 1128, in _run
    str(subfeed_t.get_shape())))
ValueError: Cannot feed value of shape (16, 12, 12, 3) for Tensor 'input/lr:0', which has shape '(?, ?, ?, ?, 3)'

What's problem when I run above code.
Thanks, ruyi.

DUF is a video model, hence you need to feed video-format data to train it.

As you say, I feed video-format data to train DUF model.
But the terminal was killed automatically as follow
2019-06-02 12:53:00.177487: I tensorflow/core/platform/cpu_feature_guard.cc:141] Your CPU supports instructions that this TensorFlow binary was not compiled to use: AVX2 FMA
2019-06-02 12:53:00.654478: I tensorflow/stream_executor/cuda/cuda_gpu_executor.cc:964] successful NUMA node read from SysFS had negative value (-1), but there must be at least one NUMA node, so returning NUMA node zero
2019-06-02 12:53:00.655355: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1432] Found device 0 with properties:
name: GeForce GTX 1050 Ti major: 6 minor: 1 memoryClockRate(GHz): 1.62
pciBusID: 0000:01:00.0
totalMemory: 3.95GiB freeMemory: 3.89GiB
2019-06-02 12:53:00.655396: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1511] Adding visible gpu devices: 0
2019-06-02 12:53:11.173973: I tensorflow/core/common_runtime/gpu/gpu_device.cc:982] Device interconnect StreamExecutor with strength 1 edge matrix:
2019-06-02 12:53:11.174024: I tensorflow/core/common_runtime/gpu/gpu_device.cc:988]      0
2019-06-02 12:53:11.174038: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1001] 0:   N
2019-06-02 12:53:11.174419: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1115] Created TensorFlow device (/job:localhost/replica:0/task:0/device:GPU:0 with 3623 MB memory) -> physical GPU (device: 0, name: GeForce GTX 1050 Ti, pci bus id: 0000:01:00.0, compute capability: 6.1)
INFO:tensorflow:Fitting: DUF
Killed

My data-format
  MYDATA-V:
    train: hr_train
    val: hr_val/*.yuv     # You can still write absolute path patterns like this,
    test: [test_data*.yuv]  # or as array like this
    param:  # For MCL-V is raw data, these param must be specified
      mode: YV12
      parser: raw
      width: 1920
      height: 1080

(VSR) ryluo@Legion-Y7000:~/Desktop/VideoSuperResolution-master/VSRTorch$ python train.py frvsr --dataset MYDATA-v --cuda
2019-06-02 13:03:49,949 INFO: @LoSealL. Video related ops, nets...
2019-06-02 13:04:14,750 INFO: Total params: 2810661
2019-06-02 13:04:15,187 WARNING: trying to restore state for optimizer adam, but failed.
2019-06-02 13:04:15,188 INFO: Fitting: [FRVSR]
Killed
(VSR) ryluo@Legion-Y7000:~/Desktop/VideoSuperResolution-master/VSRTorch$ python train.py rbpn --cuda --dataset MYDATA-v
2019-06-02 13:06:47,903 INFO: LICENSE: RBPN is implemented by M. Haris, et. al. @alterzero
2019-06-02 13:06:47,903 WARNING: I use unsupervised flownet to estimate optical flow, rather than pyflow module.
2019-06-02 13:06:47,925 INFO: @LoSealL. Video related ops, nets...
2019-06-02 13:07:13,728 INFO: Total params: 14517449
2019-06-02 13:07:14,174 WARNING: trying to restore state for optimizer adam, but failed.
2019-06-02 13:07:14,174 INFO: Fitting: [RBPN]
Killed
(VSR) ryluo@Legion-Y7000:~/Desktop/VideoSuperResolution-master/VSRTorch$ python train.py sofvsr --cuda --dataset MYDATA-v
2019-06-02 13:10:22,983 INFO: LICENSE: SOF-VSR is implemented by Longguan Wang. @LongguanWang https://github.com/LongguangWang/SOF-VSR.
2019-06-02 13:10:49,892 INFO: Total params: 1639676
2019-06-02 13:10:50,344 WARNING: trying to restore state for optimizer opt, but failed.
2019-06-02 13:10:50,344 INFO: Fitting: [SOF]
Killed
(VSR) ryluo@Legion-Y7000:~/Desktop/VideoSuperResolution-master/VSRTorch$ python train.py vespcn --cuda --dataset MYDATA-v
2019-06-02 13:14:08,969 INFO: LICENSE: VESPCN is proposed at CVPR2017 by Twitter. Implemented by myself @LoSealL.
2019-06-02 13:14:09,005 INFO: @LoSealL. Video related ops, nets...
2019-06-02 13:14:33,185 INFO: Total params: 878787
2019-06-02 13:14:33,629 WARNING: trying to restore state for optimizer opt, but failed.
2019-06-02 13:14:33,629 INFO: Fitting: [VESPCN]
Killed

It seems that the same peoblem has not been solved to video model.

You need to add --memory_limit=<numbers> to avoid OOM killer. <number> can be 1GB, 100MB for instances. This is a CPU memory limitation.

That‘s OK
Thanks.

When I ran python run.py --model=duf --dataset=mydata --memory_limit=50MB in my laptop for debug model, I have found the model can regularly train before it finish one epoch. But after that, it's time to test model, the terminal is dead。
And When I run above code in LAB server,  it can pass through about 10 epochs no any issues,  then the server's terminal also is dead. And even the server is dead.
I set the epoch_num, val_num, and memory_limit, but there are still have these problems.
Another problem is about data format.
I found when I  run VSR model, the data format seems like that
  MCL-V:  # http://mcl.usc.edu/mcl-v-database/, small set videos
    train: MCL-V
    val: MCL-V/test/*.yuv     # You can still write absolute path patterns like this,
    test: [MCL-V/test/*.yuv]  # or as array like this
    param:  # For MCL-V is raw data, these param must be specified
      mode: YV12
      parser: raw
      width: 1920
      height: 1080

But, If I have the lr and hr video, how can I set the data format like follow:
  SIDD:
    train: "NTIRE19/Denoise/train/drsr_patches/**/*.png"
    train_pair: "NTIRE19/Denoise/train/noisy_patches/**/*.png"
    val: "NTIRE19/Denoise/train/gt_patches/**/*.png"
    val_pair: "NTIRE19/Denoise/train/noisy_patches/**/*.png"
    infer: ["NTIRE19/Denoise/validation_corrected"]
    param:
      parser: custom_pairs

I will check that. Before this, what's your total mem, gpu mem and your training yaml file?
For hr and lr videos, you can use
MYDATA:
    train: "path/video/hr"
    train_pair: "path/video/lr"
    param:
      parser: custom_pairs

Under hr and lr, are a list of folders containing video frames (png files). If you want to use RAW format like that, no parser currently support it, so you have to make things around.

My total memory is 62.8GB, and gpu memory is 12GB.
For lr and hr videos, you mean that a folder contains a video frames?

And I also found the used mem is increasing with the epoch increasing.

I change the data format to video frames, and the model can train successfully in Lab server.
But, when I use that data format, the psrn is inf sometimes during testing model, as follow.
| 2019-06-08 12:14:46 | Epoch: 25/10000 | LR: 0.0001 |
100%|####################################################| 200/200 [01:49<00:00,  1.94batch/s, loss=00.00004]
| Epoch average loss = 0.000063 |
Test: 100%|##################################################################| 10/10 [00:02<00:00,  3.31it/s]
mse: 0.000212, psnr: 42.779514,
| 2019-06-08 12:16:42 | Epoch: 26/10000 | LR: 0.0001 |
100%|####################################################| 200/200 [01:48<00:00,  1.98batch/s, loss=00.00002]
| Epoch average loss = 0.000083 |
Test: 100%|##################################################################| 10/10 [00:02<00:00,  3.40it/s]
mse: 0.000236, psnr: inf,
| 2019-06-08 12:18:36 | Epoch: 27/10000 | LR: 0.0001 |
100%|####################################################| 200/200 [01:52<00:00,  1.96batch/s, loss=00.00008]
| Epoch average loss = 0.000109 |

I have another interesting problem.
When I use raw video data and set the val_num=10 in VSR.Tools.Run.py  to train DUF, that's means we use 10 videos to val? And when use video frames to do that, it means we use 10 frames to val?

@RuyiLuo This is some bug in raw parser. To quickly workaround:

Write your own parser in VSR/DataLoader/Parser
Use Tools/Raw2Image.py to convert YUV into png files, you can feed model by png files. Which means a folder including pngs representing a video clip.

val_num=10 means 10 iterations run on validation loader. Each iteration gets a random cropped batch.


Print warning when kretprobe's fail to fire

Sometimes kretprobe's do not fire, depending on how many threads are currently "inside" the traced function, and the maxactive setting.  The kernel tracks how many firings are missed.  It would be helpful if bpftrace would print some sort of warning when these events are dropped.
https://www.kernel.org/doc/Documentation/kprobes.txt :

While the probed function is executing, its return address is
stored in an object of type kretprobe_instance.  Before calling
register_kretprobe(), the user sets the maxactive field of the
kretprobe struct to specify how many instances of the specified
function can be probed simultaneously.  register_kretprobe()
pre-allocates the indicated number of kretprobe_instance objects.
For example, if the function is non-recursive and is called with a
spinlock held, maxactive = 1 should be enough.  If the function is
non-recursive and can never relinquish the CPU (e.g., via a semaphore
or preemption), NR_CPUS should be enough.  If maxactive <= 0, it is
set to a default value.  If CONFIG_PREEMPT is enabled, the default
is max(10, 2*NR_CPUS).  Otherwise, the default is NR_CPUS.
It's not a disaster if you set maxactive too low; you'll just miss
some probes.  In the kretprobe struct, the nmissed field is set to
zero when the return probe is registered, and is incremented every
time the probed function is entered but there is no kretprobe_instance
object available for establishing the return probe.

Relatedly, is there a way to specify maxactive?  If not, that would be a nice feature to have.  It's important to set this correctly if you are using bpftrace to measure performance of heavily multi-threaded workloads.

Example output today.  Workload is random reads from 4x as many threads as there are CPU's.  The kretprobe fires 1/4 as many times as the kprobe.
$ sudo bpftrace -e 'kretprobe:zpl_iter_read,kprobe:zpl_iter_read{@[probe]=count();}' -c 'sleep 10'
Attaching 2 probes...

@[kretprobe:zpl_iter_read]: 263207
@[kprobe:zpl_iter_read]: 1037071

I'd like for bpftrace to additionally output something like
kretprobe:zpl_read_iter failed to fire 750,000 times due to insufficient "maxactive".
The "maxactive" setting can be increased by <insert instructions here>.

I took a look at the code. It seems the code path is:
perf_kprobe_event_init
  perf_kprobe_init
    create_local_trace_kprobe
      max_active=0
      __register_trace_kprobe

https://github.com/torvalds/linux/blob/master/kernel/trace/trace_kprobe.c#L1405 is where maxactive is hardcoded to 0.
Since bpftrace uses the perf API, it doesn't look like there's currently a way to specify maxactive. This seems somewhat important to have as part of the API so I'll ask around and see what kind of kernel change is appropriate.
As far as overflow reporting goes, I don't immediately see how it's reported to perf right now (if at all). Will need to study the code later.

cc @liu-song-6 who introduced the original API to see whether API can be extended to include maxactive in perf_event_attr for kretprobe.

Thanks @yonghong-song for the tag.
As @danobi mentioned, currently, there is no way to specify maxactive via perf_kprobe_event_init(). We can probably support it with a few bits from perf_event_attr.config. I guess 8 bits (max of 255) should be sufficient.
There is also an alternative without kernel change here: use the old text-based kprobe API. Of course, with this approach, we lose benefits of the new kprobe API.

256 doesn’t seem too high to me. If 8 bits is the most the api can spare, is there also a way to report overflow count through the perf api?
Happy to write any kernel patches or start mailing list discussions if you’re busy.

We have 31 bits left in perf_event_attr.config, so that's not a problem.
The change should be straightforward. I will get something for test soon.

@danobi Could you please try https://github.com/liu-song-6/linux/tree/maxactive ?
To specify max_active, use the highest 12 bits of perf_event_attr.config. (0 to 4095).

It's great to see progress on being able to specify maxactive, thank you!
Have y'all given any thought to how bpftrace could retrieve nmissed, and print it with a warning if it's nonzero?  That way the user could realize that they might want to increase maxactive.

@ahrens , yeah I have the same concern. I'm worried that b/c # tasks is more or less unrestricted, any limit on max_active is an incomplete solution. I think it would be better if we could report nmissed. @liu-song-6 , does that sound possible?

Maybe we can start with /sys/kernel/debug/tracing/kprobe_profile ?

@liu-song-6 does this (/sys/kernel/debug/tracing/kprobe_profile) work with FD-based kprobe?

@yonghong-song good point... It doesn't work with fd-based kprobe. And we don't want it to work with fd-based kprobes.
I think we will need new ioctl on the fd for this use case.

This seems a viable solution. Thanks!

@danobi would you like to work on the kernel patch to add ioctl for nmissed?

@liu-song-6 sure. I will come bug you about that tomorrow.

I put the patches up on the mailing list. It's also hosted here: https://github.com/danobi/linux/tree/perf_kprobe_ioctl

I've posted the second attempt<EMAIL_ADDRESS>

@danobi Any update on this?

Ah sorry, notification must have fallen into the void.
I sent a couple versions but then lost track of what PeterZ was talking about. Unfortunately ran out of time to figure it out. I believe this is the most up to date version here: https://github.com/danobi/linux/commits/perf_read_format_missed if anyone has time to pick it up.


Configure MaxWorkers, NumWorkers and ServiceAccount params

Configure MaxWorkers, NumWorkers and ServiceAccount params for dataflow jobs via SMT.
Context: This pr does not cover the end to end suppor for dataflow params, this is a stopgap to unblock zee because they need these params for migration. The detailed support will be added as a followup.
Tested:

Dataflow form: NumWorkers, Maxworkers only accepting positive integers. Tool tips working correctly.
Non-sharded Flow: NumWorkers, MaxWorkers and ServiceAccountEmail set correctly.  Without configuring, defaults set correctly.
Sharded Flow:  NumWorkers, MaxWorkers and ServiceAccountEmail set correctly.

@manitgupta created b/300068690 to track unit testing for dataflow params.

@manitgupta since this is a stopgap, not adding other dataflow params right now. The follow up excercise is coming up with a proper way (maybe a new UI box) for dataflow params, cover all the relevant params and handl ethe prechecks etc.


ODHack: Add deposits to comparison tables - zklend

"Add deposits to the table with loan states. ZkLend allows its users not to use some of their deposits as collateral, while other protocols automatically count every deposit as collateral. Distinguish between regular collateral and deposits which are not labeled as collateral.
Steps:

The current pipeline which computes the current states of loans contains the deposit variable. When saving the data to the database (loan states table), the deposits are omitted and only collateral is saved. Save deposits under a newly created deposit column."

Definition of Done
The code functions well and is documented. The data is being saved in a database.

Hi, can i jump on this issue

Can I hop on this?

Hello! I would like to help with this one !

I'd like to work on this issue. This is my first time contributing to an open source issue. I think this may take me 3-5 days. I graduated last year, I studied data science and math. I have experience with Python and Cairo. I'm currently completing the Starknet Basecamp, with the intention of becoming a seasoned Cairo dev.

Hi, assigning @nicosanc , because they provided a good description of the approach and @danielcdz is already assigned to another issue.
@nicosanc Let me know if everything is clear and whether you'll be working on the issue. If you have any questions, please ask here. What is you TG handler please? 🙏🏼
Consider joining our TG group.
See also our contributor guidelines.

Thank you for the opportunity Lukas! My TG is @nicolander.
I'll be starting on this issue immediately. If I have questions I will post them here. Thanks!

Congrats on completing this @nicosanc !


DUPLO-3635: Support for cluster auto-scaler in asg_profile.

Change description

Description here

Type of change

[ ] Bug fix (fixes an issue)
[ ] New feature (adds functionality)

Related issues

Fix #1

Checklists
Development

[ ] Pull requests may not be submitted to the master branch (use develop instead) - or they will be closed.
[ ] Lint rules pass locally
[ ] Application changes have been tested thoroughly
[ ] Automated tests covering modified code pass

Security

[ ] Security impact of change has been considered
[ ] Code follows company security practices and guidelines

Code review

[ ] Pull request has a descriptive title and context useful to a reviewer.
[ ] "Ready for review" label attached and reviewers assigned
[ ] Changes have been reviewed by at least one other contributor
[ ] Pull request linked to task tracker where applicable

Task linked: DUPLO-3635 Support for cluster auto-scaler in asg_profile


suggestion to add link for a good post about baobab

this blog post really help me understand how to use baobab,
maybe it will be valuable for others to mention it in the readme?
http://christianalfoni.github.io/javascript/2015/02/06/plant-a-baobab-tree-in-your-flux-application.html

@christianalfoni, is that ok for you if I link towards your article in the documentation?

Of course, it would be an honor ;-)
Btw, I am writing up a new article now which will also get a pull request. I have figured out something really awesome. It is a way to use contexts of React JS with Baobab to build isomorphic apps.
Using it on a project I am working on https://github.com/christianalfoni/EmptyBox.
I hope to get the article out today, really excited about it :-)

:+1: I just came here to post this same suggestion.

v1 just landed @christianalfoni. Should we rewrite the blog post a little to account for the changes?

Writing a new article :-) Will insert link to new article in old one when it is done.


Add Pistons

I was trying to made a monostable circuit in the integrated circuits but there are no pistons.

There are a few reasons i did't add pistons:

Integrated circuits shouldn't have moving parts. Imo that makes sense when comparing it to irl ICs.
Pistons need block entities, and integrated circuits currently only work with  blockstates (mimicking vanilla)

However i don't think that limits them too much, monostables are still possible with normal components.


Update test case of reading nested unsigned parquet file

Update test case of reading nested unsigned parquet file after cudf fixed issue 9240
This is related to https://github.com/rapidsai/cudf/issues/9240
Signed-off-by: Chong Gao<EMAIL_ADDRESS>

build

build

Updated.


Improve collision masks API

Currently collision masks just raw bits. It would be great to have a macro as in heron for enums to convert members to bits. This will allow to have a lot of convenient methods in InteractionGroups. In heron it named CollisionLayers:
So basically my suggestion is to make masks API similar to heron.

On second thought maybe it's better to use an external crate for it, like https://github.com/bitflags/bitflags


Fix org visibility bug when git cloning

Should fix #6491, ~but if someone could add some tests are better. I'm so sleepy so have to go to bed now.~
Integration tests added.

Codecov Report

Merging #6743 into master will decrease coverage by 0.01%.
The diff coverage is 42.85%.

@@            Coverage Diff             @@
##           master    #6743      +/-   ##
==========================================
- Coverage      41%   40.98%   -0.02%
==========================================
  Files         421      421
  Lines       57915    57913       -2
==========================================
- Hits        23746    23734      -12
- Misses      31010    31020      +10
  Partials     3159     3159

Impacted Files
Coverage Δ

modules/context/repo.go
57.34% <ø> (+0.37%)
:arrow_up:

routers/repo/view.go
42.02% <ø> (-0.59%)
:arrow_down:

routers/api/v1/api.go
68.68% <ø> (ø)
:arrow_up:

models/repo_permission.go
71.49% <42.85%> (-0.94%)
:arrow_down:

models/unit.go
45.71% <0%> (-5.72%)
:arrow_down:

models/repo_indexer.go
66.52% <0%> (-1.28%)
:arrow_down:

models/repo_list.go
66.84% <0%> (-1.06%)
:arrow_down:

models/repo.go
47.79% <0%> (-0.18%)
: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 6a58832...7d771e4. Read the comment docs.

It's ready to review.

Drone fails

@lafriks @zeripath done.

Please backport


Please upload 0.4.2 download tarball to PyPI

Hi, I am maintaining python-ghp-import package in Arch Linux [community] repo.
I found that you have a 0.4.2 release two month ago, and it is released on PyPI.
But pypi/ghp-import/0.4.2 don't have a download file. The latest dowloadable release on PyPI is pypi/ghp-import/0.4.1.
So please upload 0.4.2 tarball to PyPI.
Thank you!

I suspect this means it was released and then deleted. PyPI wont allow it to be re-uploaded.

0.5.0 is tagged and on pypi.


Setup Keycloak group

Title of ticket:
Description
Set up our users and the groups for those users within keycloak
Dependencies
A related task will be developing the permissions model (ie: different users will have different permissions levels)
DOD

[ ] Keycloak group for product team
[ ] Keycloak group for intake

Groups have been created. Users are not known at this time and will be added at a later date


AZ_BATCH_APP_PACKAGE on Linux

Assume the application package name is "Agent-linux-x64" and the version is "<IP_ADDRESS>". Following the documentation on this page the environment variable on a Linux machine would be: AZ_BATCH_APP_PACKAGE_Agent-linux-x64_<IP_ADDRESS>
The result of my test shows that the environment name is actually: AZ_BATCH_APP_PACKAGE_agent_linux_x64_1_1_46_0
The name of the application package is lower case and . are replaced by _

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

ID: fe76a04d-fca3-b64a-9fdf-b12de26d3d19
Version Independent ID: 9dbb15f3-8665-595d-2322-fb8cdfc0c6e3
Content: Task runtime environment variables - Azure Batch - Azure Batch
Content Source: articles/batch/batch-compute-node-environment-variables.md
Service: batch
GitHub Login: @LauraBrenner
Microsoft Alias: labrenne

@holietenk
Thanks for the feedback! I have assigned the issue to content author to check and update the document as appropriate..

@LauraBrenner
Can you please check and add your comments on this doc update request as applicable.

Added the info to the doc and publishing today.

Some typos are in the text.
I would like to suggest the following text:

A prefix of all the app package environment variables. For example, if Application "Foo" version "1.1" is installed onto a pool, the environment variable is AZ_BATCH_APP_PACKAGE_Foo#1.1 on Windows and AZ_BATCH_APP_PACKAGE_foo_1_1 on Linux, using underscores and lower case. The value of the variable points to the location that the package was downloaded (a folder). When using the default version of the app package, use the AZ_BATCH_APP_PACKAGE environment variable without the version numbers.

I've made the fix and republished

#please-close


Exited (1) ```

I've been unable to reproduce this bug.
I can't see from the stack trace provided that the error is originating from flutter_typeahead. If you are sure this problem is caused by the package, kindly file an issue on the package repo
Can you kindly show your flutter doctor result

Hi @danvick , im using Dart 2.9.3 and Flutter 1.20.1 on Windows 10.
I can use TypeAheadField or TypeAheadFormField directly using flutter_typeahead package.
But my other fields are using this flutter_form_builder package. That means, i'ts better to use FormBuilderTypeAhead by this package instead using flutter_typeahead directly.
The problem is, I'm not able to use FormBuilderTypeAhead even for a simple use. I've tried this example (of course adjusted to my need) and it didn't work. Then i tried even for a very simple, it's still not working. I tried to hot restart, also still not working.
However, thanks a lot for creating & maintaining the plugin :)
Here is the debug messages:

════════ Exception caught by widgets library ═══════════════════════════════════
The following assertion was thrown building RespondentPicker(dirty, dependencies: [MediaQuery, _InheritedTheme, _LocalizationsScope-[GlobalKey#c0e5b]], state: _RespondentPickerState#2520c):
'package:flutter_form_builder/src/fields/form_builder_typeahead.dart': Failed assertion: line 80 pos 16: 'T == String || selectionToTextTransformer != null': is not true.

The relevant error-causing widget was
    RespondentPicker
lib\…\survey\survey_detail_screen.dart:146
When the exception was thrown, this was the stack
#2      new FormBuilderTypeAhead
package:flutter_form_builder/…/fields/form_builder_typeahead.dart:80
#3      _RespondentPickerState.build
package:survey/…/respondent/respondent_picker.dart:62
#4      StatefulElement.build
package:flutter/…/widgets/framework.dart:4663
#5      ComponentElement.performRebuild
package:flutter/…/widgets/framework.dart:4546
#6      StatefulElement.performRebuild
package:flutter/…/widgets/framework.dart:4719
...
════════════════════════════════════════════════════════════════════════════════
══╡ EXCEPTION CAUGHT BY WIDGETS LIBRARY ╞═══════════════════════════════════════════════════════════
The following assertion was thrown building RespondentPicker(dirty, dependencies: [MediaQuery,
_InheritedTheme, _LocalizationsScope-[GlobalKey#c0e5b]], state: _RespondentPickerState#2520c):
'package:flutter_form_builder/src/fields/form_builder_typeahead.dart': Failed assertion: line 80 pos
16: 'T == String || selectionToTextTransformer != null': is not true.
The relevant error-causing widget was:
  RespondentPicker
lib\…\survey\survey_detail_screen.dart:146
When the exception was thrown, this was the stack:
#2      new FormBuilderTypeAhead
package:flutter_form_builder/…/fields/form_builder_typeahead.dart:80
#3      _RespondentPickerState.build
package:survey/…/respondent/respondent_picker.dart:62
#4      StatefulElement.build
package:flutter/…/widgets/framework.dart:4663
#5      ComponentElement.performRebuild
package:flutter/…/widgets/framework.dart:4546
#6      StatefulElement.performRebuild
package:flutter/…/widgets/framework.dart:4719
#7      Element.rebuild
package:flutter/…/widgets/framework.dart:4262
#8      ComponentElement._firstBuild
package:flutter/…/widgets/framework.dart:4525
#9      StatefulElement._firstBuild
package:flutter/…/widgets/framework.dart:4710
#10     ComponentElement.mount
package:flutter/…/widgets/framework.dart:4520
...


Make DocBlock insertion configurable, or at least disable'able

Right now, I can't find how to configure or at least turn off the intrusive DocBlock formatting functionality native to VS Code.
It inserts docblocks misaligned, breaking interoperability with editors that do not support random indentation within the same source file.

Could you provide an example? Is this with a specific language?

This is with both JS and PHP.

Could you explain what the problem is? What should it look like instead?

It should have same indent through all lines. Not random 4-to-5-with-intent-for-7.

I don't understand. Please share a screenshot of what you expect it to look like

What you don't understand in "random line indentation"?
What I expect is this at the very least. Or no useless stars in between comment delimiters.

Duplicate of #21042
This is configured per language. Open a separate issue for php

WHERE it is configured per language?


Kong with external Postgre

Hi there,
I've tried to connect Kong to external DB (say it's in RDS Postgre).
The scenario:

the only way to connect to DB via DB config read from .env file
no need to provision new container internal DB as a proxy from local to external, just connect to external DB defined in .env
no external_hosts connection

Any helps, please!
Thank you.
Fran

error logs:
kong_1      | 2017-04-03T08:39:16.230398696Z Error: /usr/local/share/lua/5.1/kong/cmd/start.lua:21: [postgres error] could not get current migrations: [postgres error] connection refused

Maybe too late for you, but I just had the same problem. The issue was that in my case Postgres was being started immediately before Kong and it needed a few seconds to become available. So, make sure your Postgres is running and that the connection parameters are correct.

root@slave3:~# sudo kong start -c /etc/kong/kong.conf
ulimit is currently set to "1024". For better performance set it to at least "4096" using "ulimit -n"
Error: /usr/local/share/lua/5.1/kong/cmd/start.lua:21: [postgres error] could not get current migrations: [postgres error] connection refused

You should set password in progres and kong. There's a lack of env_var KONG_PG_PASSWORD in official user guide.
Here is my compose file. It works.
version: '2'
services:
  kong-database:
    image: postgres:9.4
    container_name: kong-database
    ports:
      - 5432:5432
    environment:
      POSTGRES_DB: kong
      POSTGRES_USER: kong
      POSTGRES_PASSWORD: your_pwd

  kong:
    image: kong:latest
    restart: always
    links:
      - kong-database:kong-database
    ports:
      - 8000:8000
      - 8443:8443
      - 8001:8001
      - 7946:7946
      - 7946:7946/udp
    environment:
      - KONG_DATABASE=postgres
      - KONG_PG_HOST=kong-database
      - KONG_PG_PASSWORD=your_pwd

Same problem, and @lomori reply gave a hint.
You can run docker logs -f --tail="50" kong-database to see it takes around 10 seconds for postgres (around 40 for cassandra) to finish boot.
restart: always should solve this, or a very unreliable sleep 30

@abdulgaffarabu
How do you fix  your issue finally ？
i meet the same issue with you

I am having the same issue even after postgres has started. I am using the helm charts to deploy kong on kubernetes
The error that I am getting is given below:
prefix directory /usr/local/kong not found, trying to create it
nginx: [error] init_by_lua error: /usr/local/share/lua/5.1/kong/init.lua:169: [postgres error] could not retrieve server_version: connection refused
stack traceback:
[C]: in function 'error'
/usr/local/share/lua/5.1/kong/init.lua:169: in function 'init'
init_by_lua:3: in main chunk

I faced the same issue.
This error is returned from init function of dao. There is checking db version with query "SHOW server_version;" if you use postgres.
When above query returns version correctly,  you should no get that.
it's in my case what it's from miss setting security group.

same issue here

I faced the same issue.
This error is returned from init function of dao. There is checking db version with query "SHOW server_version;" if you use postgres.
When above query returns version correctly, you should no get that.
it's in my case what it's from miss setting security group.

same here. It's EVER SO INFURIATING

Same here, their user guide is wrong, not working.

Same here, their user guide is wrong, not working.

you can use statefulset to create postgres

my postgre is running,  problem is kong is crashing with the command provided in user guide

https://github.com/Kong/kong-dist-kubernetes/issues/60

Ok looks like they have updated the manual during the night, now it's working if we respect following:
-e "KONG_ADMIN_LISTEN=<IP_ADDRESS>:8001"
-e "KONG_ADMIN_LISTEN_SSL= <IP_ADDRESS>:8444" \
and not -e "KONG_ADMIN_LISTEN=<IP_ADDRESS>:8001, <IP_ADDRESS>:8444 ssl" \

Getting the same error :
2019/02/06 16:44:50 [verbose] Kong: 1.0.2 2019/02/06 16:44:50 [debug] ngx_lua: 10013 2019/02/06 16:44:50 [debug] nginx: 1013006 2019/02/06 16:44:50 [debug] Lua: LuaJIT 2.1.0-beta3 2019/02/06 16:44:50 [verbose] reading config file at /etc/kong/kong.conf 2019/02/06 16:44:50 [debug] reading environment variables 2019/02/06 16:44:50 [debug] admin_access_log = "logs/admin_access.log" 2019/02/06 16:44:50 [debug] admin_error_log = "logs/error.log" 2019/02/06 16:44:50 [debug] admin_listen = {"<IP_ADDRESS>:8001","<IP_ADDRESS>:8444 ssl"} 2019/02/06 16:44:50 [debug] anonymous_reports = true 2019/02/06 16:44:50 [debug] cassandra_consistency = "ONE" 2019/02/06 16:44:50 [debug] cassandra_contact_points = {"<IP_ADDRESS>"} 2019/02/06 16:44:50 [debug] cassandra_data_centers = {"dc1:2","dc2:3"} 2019/02/06 16:44:50 [debug] cassandra_keyspace = "kong" 2019/02/06 16:44:50 [debug] cassandra_lb_policy = "RequestRoundRobin" 2019/02/06 16:44:50 [debug] cassandra_port = 9042 2019/02/06 16:44:50 [debug] cassandra_repl_factor = 1 2019/02/06 16:44:50 [debug] cassandra_repl_strategy = "SimpleStrategy" 2019/02/06 16:44:50 [debug] cassandra_schema_consensus_timeout = 10000 2019/02/06 16:44:50 [debug] cassandra_ssl = false 2019/02/06 16:44:50 [debug] cassandra_ssl_verify = false 2019/02/06 16:44:50 [debug] cassandra_timeout = 5000 2019/02/06 16:44:50 [debug] cassandra_username = "kong" 2019/02/06 16:44:50 [debug] client_body_buffer_size = "8k" 2019/02/06 16:44:50 [debug] client_max_body_size = "0" 2019/02/06 16:44:50 [debug] client_ssl = false 2019/02/06 16:44:50 [debug] database = "postgres" 2019/02/06 16:44:50 [debug] db_cache_ttl = 0 2019/02/06 16:44:50 [debug] db_resurrect_ttl = 30 2019/02/06 16:44:50 [debug] db_update_frequency = 5 2019/02/06 16:44:50 [debug] db_update_propagation = 0 2019/02/06 16:44:50 [debug] dns_error_ttl = 1 2019/02/06 16:44:50 [debug] dns_hostsfile = "/etc/hosts" 2019/02/06 16:44:50 [debug] dns_no_sync = false 2019/02/06 16:44:50 [debug] dns_not_found_ttl = 30 2019/02/06 16:44:50 [debug] dns_order = {"LAST","SRV","A","CNAME"} 2019/02/06 16:44:50 [debug] dns_resolver = {} 2019/02/06 16:44:50 [debug] dns_stale_ttl = 4 2019/02/06 16:44:50 [debug] error_default_type = "text/plain" 2019/02/06 16:44:50 [debug] headers = {"server_tokens","latency_tokens"} 2019/02/06 16:44:50 [debug] log_level = "notice" 2019/02/06 16:44:50 [debug] lua_package_cpath = "" 2019/02/06 16:44:50 [debug] lua_package_path = "./?.lua;./?/init.lua;" 2019/02/06 16:44:50 [debug] lua_socket_pool_size = 30 2019/02/06 16:44:50 [debug] lua_ssl_verify_depth = 1 2019/02/06 16:44:50 [debug] mem_cache_size = "128m" 2019/02/06 16:44:50 [debug] nginx_admin_directives = {} 2019/02/06 16:44:50 [debug] nginx_daemon = "on" 2019/02/06 16:44:50 [debug] nginx_http_directives = {} 2019/02/06 16:44:50 [debug] nginx_optimizations = true 2019/02/06 16:44:50 [debug] nginx_proxy_directives = {} 2019/02/06 16:44:50 [debug] nginx_user = "nobody nobody" 2019/02/06 16:44:50 [debug] nginx_worker_processes = "auto" 2019/02/06 16:44:50 [debug] origins = {} 2019/02/06 16:44:50 [debug] pg_database = "kong" 2019/02/06 16:44:50 [debug] pg_host = "rds.amazonaws.com" 2019/02/06 16:44:50 [debug] pg_password = "******" 2019/02/06 16:44:50 [debug] pg_port = 5432 2019/02/06 16:44:50 [debug] pg_ssl = true 2019/02/06 16:44:50 [debug] pg_ssl_verify = false 2019/02/06 16:44:50 [debug] pg_timeout = 5000 2019/02/06 16:44:50 [debug] pg_user = "kong" 2019/02/06 16:44:50 [debug] plugins = {"bundled"} 2019/02/06 16:44:50 [debug] prefix = "/usr/local/kong/" 2019/02/06 16:44:50 [debug] proxy_access_log = "logs/access.log" 2019/02/06 16:44:50 [debug] proxy_error_log = "logs/error.log" 2019/02/06 16:44:50 [debug] proxy_listen = {"<IP_ADDRESS>:8000","<IP_ADDRESS>:8443 ssl"} 2019/02/06 16:44:50 [debug] real_ip_header = "X-Real-IP" 2019/02/06 16:44:50 [debug] real_ip_recursive = "off" 2019/02/06 16:44:50 [debug] ssl_cipher_suite = "modern" 2019/02/06 16:44:50 [debug] ssl_ciphers = "ECDHE-ECDSA-AES256-GCM-SHA384:ECDHE-RSA-AES256-GCM-SHA384:ECDHE-ECDSA-CHACHA20-POLY1305:ECDHE-RSA-CHACHA20-POLY1305:ECDHE-ECDSA-AES128-GCM-SHA256:ECDHE-RSA-AES128-GCM-SHA256:ECDHE-ECDSA-AES256-SHA384:ECDHE-RSA-AES256-SHA384:ECDHE-ECDSA-AES128-SHA256:ECDHE-RSA-AES128-SHA256" 2019/02/06 16:44:50 [debug] stream_listen = {"off"} 2019/02/06 16:44:50 [debug] trusted_ips = {} 2019/02/06 16:44:50 [debug] upstream_keepalive = 60 2019/02/06 16:44:50 [verbose] prefix in use: /usr/local/kong Error: /usr/local/share/lua/5.1/kong/cmd/start.lua:30: [PostgreSQL error] failed to retrieve server_version_num: timeout stack traceback:         [C]: in function 'assert'         /usr/local/share/lua/5.1/kong/cmd/start.lua:30: in function 'cmd_exec'         /usr/local/share/lua/5.1/kong/cmd/init.lua:87: in function </usr/local/share/lua/5.1/kong/cmd/init.lua:87>         [C]: in function 'xpcall'         /usr/local/share/lua/5.1/kong/cmd/init.lua:87: in function </usr/local/share/lua/5.1/kong/cmd/init.lua:44>         /usr/local/bin/kong:7: in function 'file_gen'         init_worker_by_lua:49: in function <init_worker_by_lua:47>         [C]: in function 'xpcall'         init_worker_by_lua:56: in function <init_worker_by_lua:54>

Getting the same:
[error] 1#0: init_by_lua error: /usr/local/share/lua/5.1/kong/init.lua:277: [PostgreSQL error] failed to retrieve server_version_num: connection refused stack traceback: 	[C]: in function 'assert' 	/usr/local/share/lua/5.1/kong/init.lua:277: in function 'init' 	init_by_lua:3: in main chunk nginx: [error] init_by_lua error: /usr/local/share/lua/5.1/kong/init.lua:277: [PostgreSQL error] failed to retrieve server_version_num: connection refused stack traceback: 	[C]: in function 'assert' 	/usr/local/share/lua/5.1/kong/init.lua:277: in function 'init' 	init_by_lua:3: in main chunk

You can get this running by reverse engineering the example docker-compose.yml or the https://docs.konghq.com should have everything one requires
.env
KONG_ADMIN_ACCESS_LOG=/dev/stdout
KONG_ADMIN_ERROR_LOG=/dev/stderr
KONG_ADMIN_LISTEN=<IP_ADDRESS>:8001
KONG_DATABASE=postgres
KONG_PG_DATABASE=kong
KONG_PG_HOST=<IP_ADDRESS>
KONG_PG_PASSWORD=secret
KONG_PG_USER=postgres
KONG_PROXY_ACCESS_LOG=/dev/stdout
KONG_PROXY_ERROR_LOG=/dev/stderr

docker run --name kong-postgres -e POSTGRES_DB=kong -e POSTGRES_PASSWORD=secret -d -p 5432:5432 postgres
docker run -t --rm --network=host --env-file=.env kong kong migrations bootstrap
docker run -d --rm --network=host --env-file=.env kong

All Kong configuration settings can be set with KONG_ environment variables docs and the postgres specific configuration values are here
To test if you've got docker / network / ENV variables setup I do the following:
docker run -it --rm --network=host --env-file=.env kong:1.0-centos /bin/bash
psql -h $KONG_PG_HOST -U $KONG_PG_USER

If the Kong container can't connect to your external postgres Kong will also not be able to.

  I've added. but still has the error like above.

@rakehell1986 could you create a new issue?
First step in debugging is this error is typically your Kong container can't reach your postgresql host exec into the Kong container and check you can psql to the postgresql server

Same problem. I solve the problem by configuring my external pg database accepts the remote login.
You should modify two files, pg_hba.conf and postgresql.conf.

pg_hba.conf
add the record in this file host all all <IP_ADDRESS>/0 trust
postgresql.conf
modify the  listen_addresses as *   listen_addresses = '*'
Then restart the postgres db.
You can find these configure files' path in your database by

postgres=# show config_file;


feat(dev-portal): node content component

Fixes https://github.com/stoplightio/platform-internal/issues/5645
Adds handler to fetch node content
Adds React hook for fetching node content

Can see example of this stuff working in our dev portal PR env: https://dev-portal-fuvmsk8zx-stoplight.vercel.app/docs/platform/ZG9jOjQ0-welcome-to-the-stoplight-docs

I'll abstain from review on this one and leave it up to the engineers.


[Errors] error recovery with interface with implements clause

interface I implements A {    }

Expected:
!!! Interface declaration cannot have 'implements' clause.
Actual:
!!! '{' expected.!!! ';' expected.!!! ';' expected.!!! Cannot find name 'implements'.!!! Cannot find name 'A'.

Seems to be fixed in master
Test result here: https://github.com/Microsoft/TypeScript/blob/master/tests/baselines/reference/interfaceWithImplements1.errors.txt

thanks @DickvdBrink


[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: duderino
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: linsun
If they are not already assigned, you can assign the PR to them by writing /assign @linsun in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: douglas-reid, duderino
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: linsun
If they are not already assigned, you can assign the PR to them by writing /assign @linsun in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/lgtm
/approve

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: douglas-reid, duderino, louiscryan
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: linsun
If they are not already assigned, you can assign the PR to them by writing /assign @linsun in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@duderino: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-multicluster-e2e.sh
bedd8fe538edb445c91c4ade59c4a2b14e741334
link
/test istio-pilot-multicluster-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@duderino: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-multicluster-e2e.sh
bedd8fe538edb445c91c4ade59c4a2b14e741334
link
/test istio-pilot-multicluster-e2e

prow/istio-integ-k8s-tests.sh
bedd8fe538edb445c91c4ade59c4a2b14e741334
link
/test istio-integ-k8s-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


Update overview-architecture.md

Spelling error corrected

@mswantek68 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@mswantek68 Hello, sorry for the delayed response. The word is spelled correctly. Shard refers to splitting the data into the the hash distributions. Thank you for your contribution and we hope to hear from you again.

#reassign:julieMSFT

#please-close


Syntax colorating and typings stops working since 1.6.0

Hello. I have this problem since version 1.6.0. I have no more syntax coloring in vscode and typescript type of component seems to be of type any.

In version 1.4.4, everything was working fine

I can't reproduce this problem, please provide minimal reproduction.

Same issue. Prop types are shown as any when hovering over them. Also not getting all Vue warnings/errors in Visual Code, e.g. when a required prop is missing for a child component. No problem in 1.4.4.

@johnsoncodehk, here is a reproduction repo https://github.com/vidal7/volar-issue-2742

I have the same issue, and tried downgrading version by version. Apparently the issue does not exist in 1.5.2 and only started appearing in 1.5.3 onwards
VSCode version - 1.76.2

I reproduced it and this happens when using TS 4.8.

@johnsoncodehk you are right.
TS 4.8.3 is not working
TS 4.9.5 is not working
TS 5.0.4 is working

Wow!  Thank you for fixing it so fast!


new architecture not supported

Summary
When turning my react native project from old to new architecture, rendering the component that contains the MapView is returning these errors in my metro:
Error: Unsupported top level event type "topOnMapReady" dispatched
Error: Unsupported top level event type "topOnMapLoaded" dispatched
Reproducible sample code
import MapView, { Marker } from "react-native-maps";

<MapView
          style={{flex:1}
          showsUserLocation={true}
          initialRegion={{
            latitude: 33.32265,
            longitude: 44.350832,
            latitudeDelta: 0.0922,
            longitudeDelta: 0.0421,
          }}
>
    <Marker
            coordinate={{
              latitude: 33.32265,
              longitude: 44.350832,
            }}
            title={"Test Marker"}
     />
</MapView>

Steps to reproduce
Add the sample code to whatever component and run the component.
Expected result
Expected to run normally like when it was running on old architecture, and not having these errors.
Actual result
Error: Unsupported top level event type "topOnMapReady" dispatched
Error: Unsupported top level event type "topOnMapLoaded" dispatched

React Native Maps Version
1.15.4
What platforms are you seeing the problem on?
Android
React Native Version
0.72.1
What version of Expo are you using?
Not using Expo
Device(s)
Samsung Galaxy A54 (android 14)
Additional information
Please to fix it asap as its holding back development.

update to latest react-native first and try again


Bug report

VS Code Version: 1.71.0
Azure Account Extension Version: v0.11.1
OS Version: Windows 10 Home 21H2, OS Build 19044.1889

Steps to Reproduce:

Sign In to Azure Account
Have a Static Web App resource in Azure
Try to open the resource in the subscription tree in  VS Code
I get an error Internal error: Expected value to be neither null nor undefined: parent

Hi @meehocz, are you still facing this issue?

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


mergeInput option (default:true) to keep input splitted in outDir by …

let's say i have:
src/app1/i18n.csv
src/app2/i18n.csv

and want in outDir:
dist/app1/i18n.csv
dist/app2/i18n.csv

now possible with option:
createContext({
  outDir: 'dist',
  mergeInput: false,
}).scanConvert()

Hi @iznotek,
Thanks a lot for contributing!,
I think the feature will be better named as "preserveStructure" for less confusion and to align with the ecosystem's naming.
What do you think?

I have merged the commit to main but refactored & improved it into preserveStructure with many modes.


Error during ovpn-admin installation through docker

Hi, I have windows 10 and installed ubuntu 22 on wsl2. I am getting error when trying to install ovpn-admin through docker    I have attached the screenshot for the same. Please help here to fix.

Hello, we do not support windows/wsl at the current time.


Refactor: code formatting

Use "@id" may be more clear about FAB "layout_anchor" for existing view element.
If someone remove "plant_detail_scrollview" view element and not change FAB, then compiler would report error to pointer out the reason for developer.

I signed it!

Thank you @williamwue!


Configuration slash command

Is your feature request related to a problem? Please describe.
Add new slash command for configuring guild database information.
Need to be able to set:

bseddies channel
bseddies role
bseddies category
wordle channel

Realted to #79 and #178

Added config stuff for:

wordle
valorant
salary
threads

Still need to do BSEddies stuff.

Added ability to turn off revolution event


NIP-29 Implementation: Group Chat Events Support

This PR implements support for NIP-29 (Group Chat Events) in the relay.
Changes

Add support for group chat event kinds (39000-39004)
Implement GroupEventValidator with comprehensive validation rules
Add configuration options for group settings
Add documentation for NIP-29 implementation
Update README.md to reflect new features

New Features

Group Creation (39000)
Group Metadata (39001)
Group Messages (39002)
Group Member Approval (39003)
Group Invites (39004)

Configuration
Added new environment variables for group settings:

GROUP_MAX_MEMBERS
GROUP_MAX_METADATA_LENGTH
GROUP_MAX_MESSAGE_LENGTH
GROUP_MESSAGE_RATE_LIMIT
GROUP_MESSAGE_RATE_PERIOD

Testing

Added unit tests for GroupEventValidator
Tested with real-world group chat clients
Verified all event types work as expected

Documentation

Added detailed docs/nip-29.md
Updated README.md with NIP-29 status
Added configuration documentation

Checklist

[x] Implementation follows NIP-29 specification
[x] Added unit tests
[x] Updated documentation
[x] Tested with real clients
[x] Added configuration options

It looks like it can be roughly divided into three PRs: Documentation, Security Improvements, and NIP-29 🤔

Looking forward to more small PRs.


Chainlink has incorrect balance

Chainlink uses a slightly different erc20 transfer signature which breaks how many tokens it thinks were transferred.
Compare
https://ethtx.info/mainnet/0xbaf08c360fc8f6098adbe2c7c3612106432123397dd2d5b84d042c1786261207/

https://etherscan.io/tx/0xbaf08c360fc8f6098adbe2c7c3612106432123397dd2d5b84d042c1786261207

Potential Solution
Add in an additional signature to look for.
In erc20.py add
erc20_modified_transfer_event = EventSemantics(     signature="0xe19260aff97b920c7df27010903aeb9c8d2be5d310a2c67824cf3f15396e4c16",     anonymous=False,     name="Transfer",     parameters=[         ParameterSemantics(             parameter_name="src", parameter_type="address", indexed=True         ),         ParameterSemantics(             parameter_name="dst", parameter_type="address", indexed=True         ),         ParameterSemantics(             parameter_name="value", parameter_type="uint256", indexed=False         ),         ParameterSemantics(             parameter_name="data", parameter_type="bytes", indexed=False,dynamic=True         ),     ], )
and
ERC20_MODIFIED_EVENTS = {     erc20_modified_transfer_event.signature: erc20_modified_transfer_event,     erc20_approval_event.signature: erc20_approval_event, }
And then in repository.py modify line 214
from
if all(erc20_event in events for erc20_event in ERC20_EVENTS) and all(
to
if (all(erc20_event in events for erc20_event in ERC20_EVENTS) or all(erc20_event in events for erc20_event in ERC20_MODIFIED_EVENTS)) and all(
This will pull the correct balance:

I don't know if this is where you actually want to put it but it does solve the issue. I can submit a pull request if you'd like.

Pull request https://github.com/EthTx/ethtx/pull/196


Add ADSBHub support.

Look into adding the ability to install support for ADSBHub if feasible.

Added in commit 981af38c95ac82d043c5a562ed9ef98859556ba2.