text:
Few introduction lines

HAd some spare time, started drafting the introduction.

Thanks

source: gharchive/pull-request

authors: gsantoni kif

license_type: permissive

license_source: github-api


text:
Unable to set preserveAspectRatio

I need to change the SVG's preserveAspectRatio because of an IE issue.As it is now, I have to do pbElem.svg.setAttribute("preserveAspectRatio", "xMinYMin"); I can't set it via CSS.
Can we please get it added as a property option? Like var pbElem = new ProgressBar.Line('.selector', { preserveAspectRatio: "xMinYMin"});? Same for the other SVG properties

As it is now, I have to do pbElem.svg.setAttribute("preserveAspectRatio", "xMinYMin");

Is there something wrong with the pbElem.svg.setAttribute("preserveAspectRatio", "xMinYMin")? I have made a decision to offer references to the underlying svg and path elements so that you can call their APIs. I could forwards some options directly to the svg, but then I'd have to list all available svg properties and maintain the list. I'd prefer not to.

If more people want this, I can reconsider. For now, please use the setAttribute() method like you already did.

source: gharchive/issue

authors: ZachSaucier kimmobrunfeldt

license: mit

license_source: bigquery


text:
Initialize TemplatePath when ScriptableObject asset created

As far as I know, ScriptableObject.OnEnable() is called when Unity detect some scripts created or changed in Unity Editor env.
Also called when an asset of the ScriptableObject is selected first time after starting up Unity Editor.
Because of that behavior, TemplatePath I assign to my custom templates folder is always reset to QuickSheet/GDataPlugin/Templates described in gif animation.
So I have replaced OnEnable() with Awake().

Cool! Thank you for the patch.

authors: kanonji kimsama

repo: kimsama/Unity-QuickSheet


text:
Examples/snippets demonstrating the integrations

I'd like to keep a track of the libraries that can be integrated to scikit-fem with ease. How about having a single example which simply demonstrates the possible integrations? Or should it be a collection of snippets instead? If the latter, how is it tested in pytest?

I think a collection; the integrations serve diverse ends.
For pytest, I've seen this idiom:
netCDF4 = pytest.importorskip("netCDF4")

https://github.com/nschloe/meshio/blob/3a380fc1100827ed38b8f38960087a066c4aab26/test/test_exodus.py#L6

So basically in practice we could have have something like docs/integrations/pygmsh.py, docs/integrations/dmsh.py, and so on, make sure that these are collected by pytest, and then maybe have docs/snippets.rst which includes all these in the Sphinx documentation?

Yes, I think so. That directory structure should make it easy to find the case of interest.
Is there a catch with trying to import pygmsh from inside a file.called pygmsh.py or from another file in a directory with that file?

Yeah, let's call them mesh_pygmsh.py, mesh_dmsh.py, solver_pacopy.py, and so on, i.e. try to have a 'category' which describes the type of integration?

Here are some integrations that come to mind:

[ ] pygmsh
[ ] pacopy
[ ] dmsh
[ ] pyamgcl
[ ] sksparse
[ ] optimesh
[ ] pyfvm (moving mesh to and from, e.g., for iterative coupling)
[ ] vedo (actually I should continue on #361 instead)

I'm sure there are others. Feel free to add to the list.

Some additional urls for consideration:

https://github.com/andrenarchy/krypy
https://pymesh.readthedocs.io/en/latest/api_sparse_solver.html
https://github.com/nschloe/pygalmesh
https://github.com/nschloe/meshzoo
https://github.com/pyvista/pyvista
https://pypi.org/project/petsc4py/

There's a start on PETSc in #236 .
After that recent question about pressure boundary conditions for the Stokes equation #466 , which referred to a FEniCS example using MUMPS, I started looking at PyMumps.
nschloe has a new package picking up from krypy: Krylov.
I've been looking at pyMOR; besides it being useful for scikit-fem applications, I think scikit-fem.could be useful for it. It has bindings to FEniCS, deal.ii, and NGSolve, and its own little pythonic FEM implementation but that's rather limited.
I like the idea of coupling to a finite volume solver. For finite differences, there's scikit-fdiff. A boundary element code could be really useful too.

I had been that thinking optimesh might be useful with the new MeshTri.init_circle #475 in examples #476 but actually it doesn't seem to be required #477 .

I think that in near future I might be reimplementing one specific
algorithm from optimesh that I'm very fond of and also using in
https://github.com/kinnala/adaptmesh.

For external reasons but possibly of interest here, I've posted an example of integration with OpenFOAM.

https://github.com/gdmcbain/OpenFOAM-9/tree/d4afc0eb4e000d82ecd81dd72dd5da3c9c808e03/tutorials/basic/potentialFoam/cylinderSkFem

It generates a quasi-two-dimensional hexahedral mesh using OpenFOAM's blockMesh as in the OpenFOAM tutorial basic/potentialFoam/cylinder, one hexahedron thick in the z-direction, which is how two dimensions is done in OpenFOAM.  This is then parsed in a small general Python submodule to create a skfem.MeshQuad for the domain with .boundaries for the inlet.  The inlet boundary is also saved as a trace mesh (#530 #687) and the discrete finite volume inlet velocity boundary condition computed in scikit-fem and written for potentialFoam.  The same problem is then solved in both codes.
The creation of the skfem.Mesh for domain and boundary and their writing and rereading was greatly simplified following the recent improvements to skfem.io.meshio #680.

authors: gdmcbain kinnala

repo: kinnala/scikit-fem


text:
Update zio-prelude to 1.0.0-RC33

About this PR
📦 Updates dev.zio:zio-prelude from 1.0.0-RC31 to 1.0.0-RC33
📜 GitHub Release Notes - Version Diff
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "dev.zio", artifactId = "zio-prelude" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "dev.zio", artifactId = "zio-prelude" }
}]

labels: library-update, early-semver-pre-release, semver-spec-pre-release, commit-count:1

Superseded by #777.

repo: kinoplan/utils


text:
Further clarify behavior of complete

Fixes #213

Codecov Report

Merging #219 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #219   +/-   ##
=======================================
  Coverage   87.97%   87.97%
=======================================
  Files           5        5
  Lines         632      632
=======================================
  Hits          556      556
  Misses         76       76

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 76c547d...59d285a. Read the comment docs.

authors: codecov-io evanunderscore

repo: kislyuk/argcomplete


text:
Update .equals? predicate to account for lookups that return 0

I had an issue when using displayIndex with the .equals? predicate where it would fail when the ctx._LookUpStack returned 0. This was due to the equality cascade used to set a default.
(ctx._LookUpStack( arg1 ) || ctx.get( arg1 ) || arg1) would evaluate to 0 || 0 || 'displayIndex' resulting in a final equality test of 'displayIndex' === 0, which is false. The || cascade treats 0 as false even though in this case it is the value we want to set the arg1 to.
This fix checks if the lookup and get are truthy on their own or if they are numeric (which 0 is). If any of that is true, the argument is set to it. Otherwise, left as the original argument passed in.
There might be a better way -- I just dug in deep enough to get this working in my case. It doesn't seem to break anything, but again, there may be test cases I'm unaware of.

First of all, thanks for finding this. It looks right to me. There's probably some refactoring that could be done to reduce code duplication. Something that lets you do this type of thing:
if ( !isNumeric( arg1 ) ) {
    arg1 = getLookupArg( ctx, arg1 );
}

// Repeat for arg2

// Return

Where getLookupArg is a method at the same level as isNumeric and looks something like this:
getLookupArg = function ( ctx, arg ) {
    var get = ctx.get( arg ),
        lookup = ctx._LookUpStack( arg ),
        lookupIsNumeric = isNumeric( lookup ),
        getIsNumeric = isNumeric( get );

    if ( lookup || lookupIsNumeric ) {
        arg = lookup;

    } else if ( get || getIsNumeric ) {
        arg = get;
    }

    return arg;
}

Good call. Refactored.

Looks good man, thanks for catching this bug :)

Merged and published. Since the server version didn't change, an npm install -g node-squarespace-server as opposed to an update will pull the latest dependency in for you :D

Great. Thanks!

authors: kitajchuk paulfarning

repo: kitajchuk/node-squarespace-jsont


text:
JSX type definitions in conlfict with react

Prerequisites

[X] I have written a descriptive issue title
[X] I have searched existing issues to ensure the bug has not already been reported

Versions
Tested with 3.0.10
Description
I have installed @kitajs/html as an inderect dependency of @elysiajs/html. Now I get Typescript errors in my project in random JSX properties, for example
<meta charSet="UTF-8" />
//    ^^^^^^^
//           Property 'charSet' does not exist on type 'HtmlMetaTag'. Did you mean 'charset'?ts(2322)

React's types defined this attribute to be written in camel case, ie. charSet while @kitajs/html overrides this with the all lowercase charset. Other examples are event handlers like onClick or onKeyPress
To me this is unexpected und unpractical because all my other projects use React's definitions as lingua franca
Steps to Reproduce
Setup a project with Next js and use this component
const Button = () => <button onClick={() => {}}>click</click>

You will see no type Errors. Install @kitajs/html and you will get type errors ot onClick
Expected Behavior
KitaJs matches, re-exports or expects React's TS Definitions as a peer dependency instead of providing divergent types

React's types defined this attribute to be written in camel case, ie. charSet while @kitajs/html overrides this with the all lowercase charset. Other examples are event handlers like onClick or onKeyPress

This is not react, this project does not have anything in common with React without by both using JSX. charSet html property is lowercase, so kitajs/html repescts this value. Kitajs/html tags renders to string, so onClick and all others functions CANNOT be used here. Kitajs/Html is a template engine.

Typescript spec uses the JSX namespace to correctly define its types. React and Kitajs/Html cannot be defined in the global namespace, you can import kitahtml in each file and use a jsx pragma comment to use it.

Besides that, are you having problems when generating html strings with kitajs/html?

I understand that this project has nothing in common, so to speak, with react, other that both work with JSX. What I am intending here is that even though this project is not related to react, react has established defininitions (a lingua franca) for camel case JSX attributes. Now when someone uses this project with TypeScript, they will have use all-lowercase attributes, while in all their other projects, they will use camel-case attributes. To me, switching between styles, is usually a cognitive pain.
In my case, I am neither using react nor am I using your project directly. I am acutally using ElysiaJS which, for it's html plugin uses your project internally. My issue here is, once I install the html plugin, your projects JSX TypeScript definitions, will override the JSX TypeScript definitions I had before and most of my .tsx files become errorneous, only because this is a dependency of a dependency that I use.

So whether or not Kitajs renders onClick or not, it will break the types for it

And I am pretty sure there are "official" types that can be installed somehere or somehow, personally, I believe it would make sense to let each project decide which version of that to use, because surely in a project there are always other dependencies relying on JSX and their types.

And I am pretty sure there are "official" types that can be installed somehere or somehow

Typescript does not provides JSX type definitions, it uses the JSX namespace and expects it to be defined by something in each project, in your case is being double declared by React and Kitajs/Html.

surely in a project there are always other dependencies relying on JSX and their types.

Nope, in fact this is rare enough they didn't think of adding support for multiple JSX definitions. Either you use one declared globally, or a imported one per file.

So whether or not Kitajs renders onClick or not, it will break the types for it

It must complain about the onClick type, as it is not a property kitajs html can handle (functions in general).

Anyways, without a reproducible example, I cannot help you with your specific use case. Your setup is misconfigured and interpreting both React and Kitajs/Html globally, you should fix it.
we cannot fix this problem in our side. Feel free to open another issue if you have other problems 😊

ject internally. My issue here is, once I install the html plugin, your projects JSX TypeScript definitions, will override the JSX TypeScript definitions I had before and most of my .tsx files become errorneous, only because this is a dependency of a dependency that I use.

Did you find a solution? Spend all day on this. Tried a monorepo and seperate packages for UI and API, as soon as I import App from Elysia all types break in the React project - even if they are separate packages in a monorepo

Here is a repro @arthurfiorette - just comment and uncomment this line: https://github.com/dominictobias/kitajs-breaks-react-types/blob/main/ui/src/App.tsx#L6

This line below loads the @api into the type definitions.
https://github.com/dominictobias/kitajs-breaks-react-types/blob/4d388224c977ee7106698aadd6e55061320f5817/ui/src/App.tsx#L6
And here, inside your @api, loads kita JSX into type definitions.
https://github.com/dominictobias/kitajs-breaks-react-types/blob/4d388224c977ee7106698aadd6e55061320f5817/api/src/index.ts#L5
This is making both JSX type definitions available at runtime. You need to figure a way to not load both of them.
This is not a kitajs issue. Feel free to reopen if more help is needed.

repo: kitajs/html

license: MIT


text:
Unable to download grafana/grafana image? (HTTP 404)

I'm on OSX 10.10.4.
I have installed Kitematic (Beta - version 0.6.0) via Homebrew cask.
I'm trying to install the grafana/grafana image and I get the the following error:

Is there any way to get more details about what's going on?
As far as I'm aware, the grafana/grafana image does exist on Docker Hub:
https://registry.hub.docker.com/u/grafana/grafana/

Thanks for the issue!
Is the VM up and running? Often this error will show if the Docker engine cannot be reached.

Very strange - it seems to have been an intermittent thing, because when I tried 30 minutes later, it seemed to work fine...
Sorry I can't reproduce it =(. Happy if you want to close the ticket, and we can re-open if it recurs?

@victorhooi Closing this issue for now. Please do reopen it if you see it again!
Thanks!

authors: jeffdm mchiang0610 victorhooi

repo: kitematic/kitematic


text:
🛑 HASS UK Home Website is down

In e678c55, HASS UK Home Website ($HASS_UK_HOME_DOMAIN) was down:

HTTP code: 503
Response time: 1051 ms

Resolved: HASS UK Home Website is back up in 2c63fa1.


text:
Maybe fix curlString

Please try this @NikkyAI
Description

Type of change
Check all that apply

[x] Bug fix (a non-breaking change which fixes an issue)
[ ] New feature (a non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Refactoring (a change which changes the current internal or external interface)
[ ] This change requires a documentation update

Checklist:

[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation, if necessary
[x] My changes generate no new compiler warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] Inspect the bytecode viewer, including reasoning why

it seems to work now, i will add some regression tests later to verify it too

Some how, Travis CI removed sdkmanager.

authors: NikkyAI SleeplessByte iNoles

repo: kittinunf/fuel


text:
Some documentation

PreS: I'v not found any space to ask some questions so sorry to do that here ;(
Hi, i would like to know if there is any documentation available, who (obviously) group all Hexi functions etc.. cause its really painful searching ways to do new things without docu.
And is this project abandoned ? Cause i would like to add some features :)
Thx !

Hi,
Check out core.js and read the comments. 😄
You'll see it's just high-level glue code around Pixi, and a bunch of specialized modules for making Pixi easier to work with and giving it a nicer API - nothing special.
We generally add features to Hexi by creating stand-alone modules that should work with Pixi.
Take a look at Bump.js, Charm.js and Dust.js for examples on how to do that.
But there's pretty much a feature freeze at the moment until the current code base is 100% bug free.
If you can do any bug squashing, please help :)
We'd like to get this to a stable 0.1.0 release, but that's going to take a lot more testing.
And, there's quite a bit more code coming down the pipeline that I need to merge from my local computer (a bunch of tile based game utilities, include tile based collision, isometric maps and and A-Start implementation).
I would definitely like to get the help of collaborators in future, but at the moment the code is not really setup for that to happen easily.
Thank you for asking and please let me know if you have any questions for how I can help 😄

Hi @kittykatattack
Thanks for building Hexi!
I have been having fun with it, check out https://github.com/somosazucar/python-web which I built inspired by @artyprog 's https://github.com/artyprog/GFK approach.
However I am new to building larger stuff in Javascript and Canvas/WebGL/PIXIJS.
I would like to slowly (or rapidly) build an IDE - like thing for learning and producing expressive games. So I am learning as I go along - I am interested in writing some concise entry level learning materials - likely initially in Spanish. If I manage to do so, I would share them with a Creative Commons License.

Do you expect Hexi's API to change significantly? (Should I wait?)

Please, if you can, review my approach at an Object Orientated architecture at https://github.com/somosazucar/python-web/blob/master/src/main.py  . I am coming from building Python games with http://platipy.readthedocs.io/en/latest/pong.html#sprites . I think it should be possible to simplify it.

Do you have a suggestion for implementing Physics?

I am interesting in trying the new Isometric code - so far I managed to load and display a map, learning to place sprites and pan camera.

Great to hear that you're active and hoping you find inspiration.
@icarito

Ok, I'll do that !
For "improvements" I was speaking about little things like:
g.pointer.tap(x, y) =>...
Just little things that together makes hexi more agile :)

Hi @icarito!
I just got back from a 3 week holiday in Peru and loved it!
Next time I'm in Peru I'll bring my sitar and we can jam! 😄
Thanks so much for your comments, I'll try my best to answer.

Do you expect Hexi's API to change significantly? (Should I wait?)

I expect Hexi's high-level API to remain very, very stable. More than anything else, Hexi is an attempt to build a great (and simple!) API for making a wide variety of games. Its current API is the result on many, many years of refinement and testing, and so I'm pretty confident it's not going to change. Hexi actually shares its API with its sister game engine, Ga. Even though their underlying code base is radically different, their high level code is almost completely interchangeable. Hexi is an API more than anything else, and the most important thing for me is that its API is as stable as possible so that it's a solid foundation for people to build things on.
Some of the lower-level code certainly will change, but because it's not user-facing, it won't break the high-level API

Do you have a suggestion for implementing Physics?

There are many excellent Physics libraries that should work fine with Hexi. Physics.js, and P2.js are a few that come to mind.

I am interesting in trying the new Isometric code - so far I managed to load and display a map, learning to place sprites and pan camera.

I have a stand-alone module for this, including a system for integration with Tiled Editor, and will gradually merge this into Hexi's source over the next few months.

Please, if you can, review my approach at an Object Orientated architecture at https://github.com/somosazucar/python-web/blob/master/src/main.py . I am coming from building Python games with http://platipy.readthedocs.io/en/latest/pong.html#sprites . I think it should be possible to simplify it.

I'm afraid I may not be the best person to help you with this as I now do almost all my professional work using Elm (www.elmlang.org), which is a functional programming language.

For "improvements" I was speaking about little things like:
g.pointer.tap(x, y) =>

Oh, I see what you mean 😄
That makes a lot of sense.
Maybe you could add some of those things as a feature request?
The next time I plunge back in to Hexi's source I'll see how easy it might be to add.
This is a slow moving project (which I think is a strength!) but Hexi will be here for the long term :)
It's my baby! 😄

Hi Carito,
Here is what I have done when I used RapydScript
http://salvatore.pythonanywhere.com/RapydBox/default/editor     (load the samples)
http://salvatore.pythonanywhere.com/RapydScript
http://salvatore.pythonanywhere.com/RapydRay
http://salvatore.pythonanywhere.com/RapydGlow
It would indeed nice to have such and editor for Hexi and share  the code...
Regards

Hi @artyprog and @kittykatattack !
Greetings again from the rainforest!
So after a long time finally I have something to show you :-)
I'm barely getting to the point of adding Hexi to Jappy Activity, the Python (Rapydscript) editor!
Here's a very rough first approach to a platformer. I've never done one, and I'm wondering about simple Physics.
Notice it's live-collaborative!
@kittykatattack sorry to have missed to reply to that one point, please, do bring your sitar, we must absolutely jam.
All the best!

@icarito That's wonderful, thanks so much for sharing it !!!

authors: Akronae artyprog icarito kittykatattack

repo: kittykatattack/hexi


text:
Animation and weak references

I was having a bit of an issue with Animations and weak references. I couldn't just put "try: ... except: ReferenceError" around my code because the weak reference issues were being triggered by Kivy animation code. I could change the lifespan of the widgets involved, but I have currently just put the try: except ReferenceError: in the animation class. Should this be put into the core?
In kivy/animation.py:
 def _update(self, dt):
      ...
           try:
                # apply progression on widget
                for key, values in anim['properties'].items():
                    a, b = values
                    value = calculate(a, b, t)
                    setattr(widget, key, value)

                self.dispatch('on_progress', widget, progress)

                # time to stop ?
                if progress >= 1.:
                    self.stop(widget)

            except ReferenceError:
                # TODO: Log
                continue

Encountered this in kivy 2.3. Trying to reliably reproduce

repo: kivy/kivy


text:
How to prevent screen capture in kivy?

How to prevent screen capture or disable screenshot in kivy that protect content in App?
Platform: Windows or Android

I don't believe we have a solution for that, and that seems outside of the projects scope.

authors: toaoman tshirtman


text:
Removed python 2 support (behaviors)

Maintainer merge checklist

[x] Title is descriptive/clear for inclusion in release notes.
[ ] Applied a Component: xxx label.
[ ] Applied the api-deprecation or api-break label.
[ ] Applied the release-highlight label to be highlighted in release notes.
[ ] Added to the milestone version it was merged into.
[ ] Unittests are included in PR.
[ ] Properly documented, including versionadded, versionchanged as needed.

Thank you for your contribution and working on this!
However, before you add additional commits and spend more time on this, we should pause and discuss whether we will merge it. In general it's a good idea to ask or touch base with maintainers before working on large changes to a project because it's possible it may not be accepted, which isn't a great feeling after you spent a lot of effort on it. So it's generally better to first ask whether changes would be accepted, and if agreed, then start working on them.
Now specifically about these changes. We generally don't merge these kind of changes, what people call stylistic changes. We generally only merge changes that either fix a bug or adds a feature. Only very rarely do we merge PRs that only contain stylistic changes - changes that don't fix a bug or add a feature.
The reason is that there are so many different styles to choose from, and styles change over time. So, if were to accept PRs that only change the code style, it would never end. Because styles always change and everyone has different preferences. E.g. the style of super (super(...)) you're replacing - that is not deprecated or in frowned upon, it's just an older style. But it's perfectly legal in Python3. So while it looks old, there's no reason to change it. The same for class X(object).
An additional thing is that any change, no matter how simple introduces a possibility of a bug. So when merging a bug fix or new feature this possible cost of introducing a bug is nothing compared to the cost of not applying the bug fix/feature. But when merging a stylistic change, the cost of the possible bug is tends to outweigh other consideration so hence we don't merge them for fear of causing a bug.
Finally, such changes tend to touch many files, which is discouraged for PRs because it makes review harder and is much more likely to introduce bugs.
So overall I'd be against merging this PR. But I do very much appreciate your desire to contribute and improve Kivy.

Yeah! I can understand now about stylistic changes, anyway I will try to avoid these kind of PRs

Ohhhh..... I'm a big fan of clean code
https://www.martinfowler.com/tags/clean code.html.
This PR would help to make Kivy's code cleaner, shorter and easier to
understand/read.
IMHO, rejecting stylistic changes, in general, is not a good idea for an
open-source project like Kivy.
Would you accept it, if there were a separate PR for every single
file/module? (I actually working on some code clean-ups for Kivy's layout
classes, but if you not accepting it I'll give up with these kinds of
contributions in the future)...
Am So., 2. Mai 2021 um 07:49 Uhr schrieb Kulothungan U.G <
@.***>:

Closed #7510 https://github.com/kivy/kivy/pull/7510.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/kivy/kivy/pull/7510#event-4674435351, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA4U63QVBISOTFZKF7Q47PDTLTYY3ANCNFSM437CIDKQ
.

authors: AndiEcker Kulothungan16 matham


text:
Analyser dependency 5.2.0

Hello
is it possible to publish a new version to pub.dev with analyzer 5.2.0 instead of : ">=4.0.0 <5.0.0"
Thank you

Same question here.
As it's already pull request available https://github.com/kiwi-bop/intl_generator/pull/17, any chance that it going to be merged in the near future?
Thank you.

Done on 0.4.0 :)

authors: ilyachange jaumard redDwarf03


text:
Add backup feature like as in Soul Browser.

It would be really good to backup our settings to device and restore it, right?
Like this

Duplicate of #122

Duplicate of #122

What makes you think that this issue is "duplicate"?

Because it's the same feature request.


text:
List of FAQ doesn't reflect the booking status on mobile

When accessing MMB with simple token on mobile device, e.g. /en/content/manage/7388934/29b3de42-aa94-4b4d-8d28-d7d38b063332, fetched list of FAQs is wrong.
Pls add cypress test as well.

this is not only happening with simple token, this happens even if you're fully logged in, you only see the correct faqs once the booking is loaded, in mobile the booking is loaded after the user clicks in the booking tab -> related issue https://github.com/kiwicom/smart-faq/issues/544

ha, thanks for explaining

closing in favor of #544

authors: jalbertsr jaroslav-kubicek

repo: kiwicom/smart-faq


text:
Le tableau des scores ne fonctionne pas 🥇

Describe the bug
Impossible de charger les scores (donc impossible de voir si le score a été enregistré non plus)
To Reproduce
Steps to reproduce the behavior:

Ouvrir l'app

Faire une partie

Cliquer sur "Submit"

Ça tourne dans le vide ...

Ouvrir l'app

Cliquer sur "Scores"

Ça tourne dans le vide ...

Expected behavior
Pouvoir envoyer son meilleur score et voir les scores des autres🥈
Screenshots

Smartphone (please complete the following information):

Device: iPhone 13 mini
OS: iOS 15.1.1

normal faut avoir un server qui tourne derrière!

Yes je sais, j'ai viré le serveur, fausse manip' 😅 Je le note juste là histoire de pas oublier dans 2 semaines 😌
Je me disais, il faudrait peut être mettre le code du serveur dans le même repo 🧐
On peut PUSH seulement un folder sur Heroku si le serveur n'est pas à la racine 📁

authors: AdrKacz kiwily

repo: kiwily/game-jam-merge


text:
🛑 Gateway (3bb) is down

In 52b2a22, Gateway (3bb) (https://check-3bb.gongtham.net/index.php) was down:

HTTP code: 530
Response time: 280 ms

Resolved: Gateway (3bb) is back up in 6fad1d7 after 9 minutes.

repo: kizzna/uptime


text:
🛑 Thainess is down

In 9bf6fc9, Thainess (https://thainess.net) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Thainess is back up in a78b191.


text:
Bugfixes/consistency improvements for multiline stuff

Movement::{BeginningOfLine,EndOfLine,etc} now actually go to beginning/end of the line.
Movement::BeginningOfBuffer, etc was added to get the original functionality back.
Fix some off-by-one bugs in the line movement code.
Make it easier to test the multiline functionality by adding a matching bracket validator, and using it in the examples.

zsh seems to escape only '\n':
echo "\test\
t"
echo "\test\\
t"

But zsh is buggy if line ends with a backslash:
$ echo test\\
$ # a space is appended in history

That part is just so that if you have a multiline history item, after saving/loading it doesn't come back as individual lines.

With bash, Ctrl-A/Ctrl-E|Ctrl-U/Ctrl-K moves|kills to begin/end of buffer even with multiline input.
Maybe we should keep the current behaviour ?

It's a mix with what does what -- fish-shell and... every text editor supporting these have it go to the beginning of the line.
Also, as an anecdote: I use these very commonly and find going to the beginning of the buffer very frustrating (as it behaves differently than text editors).

Ok.

Anything left for this PR to get merged (except resolving conflicts of course)? We already use the code from the branch, and are pretty happy with it.

Sorry, I was mistaken that I'd have time last weekend, but I have nothing else planned for this weekend/tomorrow afternoon and will rebase it.

This should be good now.

Thanks.

authors: gwenn tailhook thomcc

repo: kkawakam/rustyline


text:
ffmpeg._run.Error: ffmpeg error (see stderr output for detail)

I failed to exprt pcm file when use the following code:
ffmpeg.input('song.mp3', allowed_media_types='audio').output('song.pcm', format='s16le', acodec='pcm_s16le', ac=1, ar='16k').run()
I am a newer in learning ffmpeg-python and can't solve this problem for a long time.I can use these two codes to reliseze my goal like:
import os os.system('ffmpeg -i song.mp3 -acodec pcm_s16le -f s16le -ac 1 -ar 16000 -y  song.pcm> log 2>&1')
however,I just wanna to use methods in the first style because I felt very frustrated for spending too much time on it ...

I reinstalled ffMPEG many times, I also spent a lot of time to solve this problem, who solved it, please comment in the following, thanks

@msx-123 did you get the solution? I am too stuck at same point. Please respond if issue is resolved.

Have you solved the problem yet?

repo: kkroening/ffmpeg-python


text:
Write test data in a table format agnostic manner

Hi,
This PR adds the HiveShell#insertInto method which allows you to fluently add test data to Hive tables without having to know the table's storage format. In the back end it uses HCatWriter to insert the test data. See the updated README or InsertIntoTableIntegrationTest for some working examples.
This provides an alternative to the HiveShell#getResourceOutputStream. Not sure how you'd want to run with this. Should that method be deprecated?

Awesome news guys! First glance it looks very tidy with high coding standards. We're trying to fit a team code review for this PR as soon as possible (hopefully this week) and then we'll get back with comments (if any!).
Regards
/Pelle

I agree with your comments and have made the changes. Look ok?

I think it's nice to have a static factory method for readability and to contain the construction of the TableDataBuilder and TableDataInserter but I also don't like InsertIntoTable.newInstance. I'm thinking of renaming to TableData.insertInto. Thoughts?

@Teabot said:

I think the intention here was to focus on keeping the HiveRunner API easy to learn and use. Having different entry points to different functional elements requires the user to have a broader knowledge of the classes in the library. A single framework façade that collects together related functionality allows a user to simple hit Ctrl+Space (or equivalent) to see what is available. I think here of the Mocktio framework which hangs almost everything off of a single Mockito class. This arrangement has allowed me to effectively use a very complex library by keeping just few class names in my head.

With the suggested change the HiveShell instance must now explicitly be wired into the InsertIntoTable method by the user. This is unfortunate as from a user perspective it again adds more complexity to the API. I'm not sure why as a user writing Hive tests I should have to worry about wiring up what are effectively the internals of the test framework I'm using.

I see that @nahguam has made the change, but can we take a moment to see if there are any other options available that would keep the API nice an simple (we should aim for trivial!) as I believe this is a key feature for a tool that's used to write tests. We should eliminate boiler plate and repetition, and everything we ask the user to write should have some relevance to their test, which the hiveShell arg does not IMHO.

My concern was that the fact that the table had to be created before the data insert could happen was a bit different from how HiveRunner originally was intended to be used with its pre and post start states.
HiveRunners pre state was intended to be used for all setup including setting up data and the post step was meant for verification only. Since the insertIntoTable functionality requires the table to be created this shifts much more of the setup to the post start phase of HiveRunner.
However, it's probably time to rethink the whole API of HiveRunner and do a full compatibility breaking overhaul anyway. It might be that we should remove the pre/post start phases. They where an enabler for the annotations mainly, and I'm leaning towards removing most of the annotations in the overhaul.
@teabot comment about simplicity is a good argument (although it might come on the expense of unbalancing the API). After writing this response (and sorting my thoughts in the process) I'm not sure where to lean really. I'd say @Teabot comment is more valid so lets stick it back into the HiveShell the way you originally built it.
@nahguam: Sorry for leading you astray...

In order of preference, I would choose:

HiveShell#insertInto(String, String)
TableData#insertInto(String, String, HiveConf) (HiveConf instead of HiveShell)
@HiveTableData(String, String)

..happy to go with the consensus.

Looks good. I'll merge it!

authors: PelleUllberg nahguam

repo: klarna/HiveRunner


text:
App won't start - "A JavaScript error occurred in the main process"

Repro steps
Download .dmg from release, mount, drag AO to applications folder, open AO app
Error Message

Environment
macOS 10.13.2

Thank you for taking the time to report the issue : )

authors: jaredreich klauscfhq

repo: klauscfhq/ao


text:
Special characters in filenames

There is a similar open issue here but it was announced that it's fixed.
My issue is when looking at a commit, name for file "sündmused.php" isn't displayed at all. I mean it takes a while to work out from the changes what file it was that changed.
I tried both the last release and latest code from master.

Basically it's an issue with the git show command called in src/Git/Respository.php function getCommit.
The output on my server is a mess. The reason no filename at all is shown is that the filename in the output has quotations around it, where ASCII file names do not. There is a regex there parsing the output that doesn't account for this.
When I fix that I see the name but it's still a mess, with "ü" being replaced with "\303\274".
After some digging I found git allows to disable that odd feature with git config --global core.quotepath off. It was an effort to make this execute under www-data but worth it. After that everything works fine.

repo: klaussilveira/gitlist


text:
TouchableBaseSpan as base span which can be extended for custom spans

Resolves #26

Looks good to me, thanks


text:
What units do I get for page dimensions from pdfium-cli info

The few documents I have tried seems to output dimensions in millimeters, i.e. approx. 210 x 297 for A4.
Will this be true for all PDF?
Or is it possible that a PDF can be created with other dpi than 1/72?
When identifying page sizes with Poppler I used to multiply by 72 and divide by 25.4, but this calculation already seems to be performed in pdfium info

Oops, sorry.
I misread the output, disregard the above flase information.
I see now that Width and Height in the Pages array is expressed in "points".
My only outstanding question is not a Pdfium question, but someone might know:

Is it possible that a PDF can be created with other dpi than 1/72?

Is it possible that a PDF can be created with other dpi than 1/72?
As far as I know, no. The value pdfium returns is always points and have not seen any other value that might indicate some other DPI.

What I'm reading is that it might be different since PDF 1.6, which is configured by the UserUnit on a page object. This value will be in multiples of 1/72. I can't find "UserUnit" in the source code of pdfium anywhere, so I doubt if we can get that information in pdfium right now.

I have looked around and it looks like almost nobody is using this, and that the implementation in Adobe itself is even kinda weird, so no idea if you should take this into account.
However, there was an open issue report for pdfium about it, so I added my comment to it: https://bugs.chromium.org/p/pdfium/issues/detail?id=734
If it's added in pdfium, I'll make sure it gets added to go-pdfium and pdfium-cli.

Thanks for looking into this. I'm guessing that I would have encountered this while using Poppler as well, but I haven't. My guess is that it's extremely rare that UserUnit is used.

repo: klippa-app/pdfium-cli


text:
revisit StageName for Expose + Cloudfront

Similar to https://github.com/klothoplatform/klotho/issues/652, we currently hard-code the stage name to "stage", because edge expansion happens before node configure (and the stage name only gets set in configure).

Probably best to change the factory.ts to use the stage's name property in IaC (ts) instead of the resource property (go).

I could definitely be missing something, but I think this requires a just a tad more than I originally thought it might. From 4b833b7:

Unfortunately, the deployment doesn't have a stage at pulumi up time, so the origin path becomes "/UNSPECIFIED_STAGE" (literally that text).
I think to fix this, we would need to either:

set the Deployment's stageName; or
explicitly create a State

In either case, we would no longer be relying on the implicit stage creation-and-naming.

Neither of those is hard, but I did want to pause and make sure they're the right path forward.
I think the current code does amount to explicitly using the stageName, based on what's implicitly created — which is not great! So maybe explicitly setting the stageName is the way to go.

explicitly create a State

I thought we did explicitly create one. Or was that only in HTTP gateway?

🤦 You're right, we do.

I think I got it now. Pulumi's weird typing threw me off: they have a circular dependency between Deployment and Stage. I can see why they did it, but at 830am on a Monday it got me.

Cool, got it working.
Since this came from #647, which hasn't been approved/merged yet, I just threw it into that existing PR. I'll keep this open for now just in case we end up rolling that part out before merging.

authors: gordon-klotho yuval-klotho

repo: klothoplatform/klotho


text:
Can't push to my application with default files.

I just installed balenaOS and successfully installed balenaSound and wanted to install Pi-hole to so I followed the tutorial from balena like I did with balenaSound but I get an error and I'm not able to install it due to it. Error log: https://pastebin.com/SPNRV4hw

Can you confirm what device you're running, and which architecture?
The Dockerfiles in this repo are meant for rpi-3 32-bit and rpi-zero.
Can you try renaming Dockerfile.armv7hf to Dockerfile in both the pihole and dnscrypt-proxy folders and let me know if that changes anything?

Changing it from Dockerfile.armv7hf to Dockerfile fixed everything. Tanks a lot for the quick reply too!

Can you confirm what device you're running, and which architecture, so I can support it better in the future?

repo: klutchell/balena-pihole


text:
Make SECURITY.md consistent

We're missing these across a lot of Knative repos, this is copied/improved from the 3 that existed.

/approve
/lgtm

authors: davidhadas evankanderson

repo: knative-extensions/reconciler-test


text:
Tests for broker broker non existing external topic

Signed-off-by: Matthias Wessendorf mwessend@redhat.com

Proposed Changes

two more tests for non-existing external topic

/assign @creydr

repo: knative-sandbox/eventing-kafka-broker


text:
[main] Upgrade to latest dependencies

Cron -knative-prow-robot
/cc knative/operations-wg-leads
/assign knative/operations-wg-leads
Produced by: knative-extensions/knobots/actions/update-deps

/test build-tests

/test build-tests
/lgtm

authors: houshengbo knative-automation matzew

repo: knative/operator


text:
Expose Go memstats as metrics.

This exposes metrics for the properties available through here so that we can get some better visibility into things like GC pressure.
Here's an example of Prometheus showing HeapAlloc for our controller process:

cc @Harwayne @mdemirhan

repo: knative/pkg


text:
Format markdown

Produced via:
prettier --write --prose-wrap=always $(find -name '*.md' | grep -v vendor | grep -v .github)
/assign k4leung4

/approve
/lgtm

authors: houshengbo mattmoor

repo: knative/serving-operator


text:
VSCODE further support for win and linux based on ide.talon

Replacing #109 , this PR extends ide.talon for vscode in windows and linux,   some comments are inline .
A key issue is I have reverted misc/git.talon  as cannot get tag:terminal working in vscode, cmd.exe or pwsh.exe  I just don't fully understand the use of tag: and its got late in day, maybe you can identify a quick fix.
A second issue is the Mac support, I want to just copy the Linux for start but need a tester, I can probably reactivate a MAC virtual machine but may take me a  couple of days

probably preferable to alias the commands in the generic vscode. i think there's a lot of value in keeping the grammar consistent across OS', unless something's really, really specific
open containing folder | show in explorer | show in finder: dostuff()

Further reduction of duplication with ide.talon,  added some more common debug support,
Outstanding:  ( should do's before merge)

create a `vscode.mac.talon` based on `vscode.linux.talon`

some code folding support

editor find/replace support

@knausj85 can you please look at and comment on in particular
a594837 ide - New Commands and Actions to support find & replace
97172ef git.talon   - I think we should change in misc/git.talon  the tag:terminal to (or add if multiple tags are supported) the tag:git
also in 97172ef is there any reason we can't use tag: ide in vscode.*.talon   it appears that you cannot use app: and tag: simultaneous in a .talon

[ ] I will now see if I can do a quick port of vscode.linux.talon to mac

I can test out the Mac stuff tomorrow. I'll poke around the PR at the same time.

also in 97172ef is there any reason we can't use tag: ide in vscode.*.talon it appears that you cannot use app: and tag: simultaneous in a .talon

Yeah, this seems like a good time to do so. You'd just to declare the tag in ide.py, and update the jetbrains/vscode.talons to assert the tag

Thanks for the review, looks all sensible, I will get into it tomorrow

I'm going to take this as-is and take care of these minor things tonight - I'm switching to VSCode myself.
I really appreciate this contribution, and your patience.

Most of these are resolved on my branch, plus some extra support added fyi. Not sure about bulk enabling terminal commands.


text:
How should I change the material for my armor?

The armor I created in the editor will only show up in the item bar inside the game, is there any way to show it on my body as well?

You need to change its Worn texture. Unfortunately, this only works for players with Optifine. Before changing the worn texture, you need to create a Worn armor texture (see Edit item set -> Textures -> Worn armor textures).

Closing this due to inactivity. Feel free to reopen this when needed.

authors: HansOffice knokko

repo: knokko/custom-items-gradle


text:
Added static files lint CI

Added a lint CI workflow for static files i.e. JS, HTML, CSS...
This CI uses packages defined in package.json located at "edison/linters/for-static-files".

Please read the README file at the location stated above.

Please note that package-lock.json is an auto-generated file so it shouldn't be reviewed.

I'll be happy to hear other path suggestions in which to put the files at "edison/linters/for-static-files" (Those files can be used also by us, besides the CI, so putting them inside .github folder doesn't seem to be the right location).

In a future PR I will format the code of our project with those linters, so if the CI starts yelling please ignore :)

If you don't intend for this pr to be merged until the lint update, please specify the update pr or change this pr to a WIP/draft... Otherwise, please post the linted files PR so We'll be able to accept this pr when the CI passes (Even though the reason for the failure is clear, merging It will fail other prs CI, and we don't want that).

repo: knowabli/edison


text:
Bug on CategoryStyler generating Area type charts

Description
According to the charts offered by the library, when generating one of the Area type, the expected result does not match the one generated.
The issue is reproducible in:
Module Version: xchart.3.8.1
OS Version etc: Ubuntu 20.04.4 LTS
The steps to reproduce are:
Define a new CategoryChart
Set the CategorySeriesRenderStyle as an Area type.
final CategoryChart chart = new CategoryChartBuilder().width(chartWrapper.chartDimensions.get(1))
              .height(chartWrapper.chartDimensions.get(0)).build();
...
chart.getStyler().setPlotGridLinesColor(Color.GRAY);
chart.getStyler().setYAxisDecimalPattern(PATTERN);
chart.getStyler().setDefaultSeriesRenderStyle(CategorySeries.CategorySeriesRenderStyle.Area);

The expected result is:

Show a chart with the area below the boundary colored
The actual results is:

It does not generate any error other than the visible one, a solution found is presented in the Plot_Category class as indicated by the attached .txt
xchart_3_8_1.txt
I've searched for any related issues and avoided creating a duplicate issue, thanks.

Instead of attaching a patch file, I suggest you create a Pull Request toward the repository.

i fixed this in a different way

authors: mccartney mmotta-zagile timmolter

repo: knowm/XChart


text:
Different instances

Could you add a toggle or something to make it also look at killproofs when in different instances?

With the current implementation that is fully based on arcdps i also have the limitations of arcdps. So that is not possible currently. I will try to implement such a feature in the future :)


text:
Fixed placement of commas

, You are welcome

REOPENED

repo: knuthy/coinpayments-payments


text:
Hangs on "empty return"

In the ^C places I've pressed return but it hung. I expected it was going to use the value shown in brackets.
It does work when I type in the value.
macOS 10.15.6

Thanks for the report :)
Technically it doesn't hang, the prompt was just waiting for a non-empty input, though I can definitely see how that can be confused with hanging. I've change the ui of those prompts to make it hopefully more apparent what is happening and also added error messages when you want to continue with an empty input, instead of doing nothing.
When you got time, please checkout 0.3.0 https://github.com/knutwalker/git-drive/releases/tag/0.3.0

Very nice -- this makes it a lot clearer! Happy to close.

authors: Mats-SX knutwalker

repo: knutwalker/git-drive


text:
Move to KoaJS org?

this is a great module. want to move it to the koajs github org so that it could have more maintainers and seem more official? let me know

Jon

@jonathanong-houzz @inca  I'm the maintainer now (admittedly need to spend more time here) whereas @dlau  is the owner. I don't have the permissions to move this into the koa org but I would be in total support of it.

@MarkHerhold  thanks for pinging me. Would love to help transfer it over.
I don't have permissions to create repositories in the koajs org, what's the general process here? @jonathanong-houzz

@dlau transfer to me @jonathanong and I'll transfer to koa and add you all as collaborators. you have to be an owner of the org to transfer.
mind the username - I accidentally commented with my work GitHub account above 😂

@jonathanong  got it. I've initiated the transfer, please let me know if I can help in any way.

done!

@dlau @MarkHerhold also invited you to the org, please accept

@jonathanong accepted, thanks!

@dlau did you add members of the koa team to npm for publishing or transfer ownership?
@jonathanong (assuming answer is no) who should those people/that team be?

@MarkHerhold Thanks for following up. I did not update any npm permissions yet
@jonathanong let us know who to add
thanks again everyone for your help, I think this module will do great under the koajs org's stewardship :-)

authors: MarkHerhold dlau jonathanong jonathanong-houzz

repo: koajs/koa-body


text:
why?

Wrong in the first step of training, why? How did the blogger achieve it? Thank you

put_your_model_name, means that the directory that you want to save the file produced by the running code!

you need to create the dir names 'logs' and 'models' firstly

authors: Zhang-haotian vinthony wkflyerman

repo: kobiso/CBAM-tensorflow


text:
Update [[/attack]] enricher to concatenate the ability modifier and proficiency bonus when rendered

Right now, if you use that enricher, it renders as +4+4 (ability mod and PB), which does not look good in the text.  It would be better if it simply displays as a single value by default, e.g., +8.  If content creators want to break out the attack bonuses, perhaps a value can be added to the enricher to render the bonus values separately, e.g. [[/attack format=long]].
~Submitted by Clayton Thomson

This is intended, unfortunately Foundry's built in ability to simplify rolls is limited so it will need some custom handling.

authors: arbron secondworldpublishing

repo: koboldpress/black-flag


text:
Single jar

Need to get rid of several jar in release.
A single jar which detects an operating system and dispatches to an appropriate launcher would be good.

Fixed.
Revs. a1e069164610f675fc5896a29b175ca9269ae118, 5deafe3ffcd3b815617cad1105e6b740a3b73861

repo: kocherovms/metracer


text:
Model-viewer updates

bump new model viewer  version
use current 3d image preview as poster #1245
placement should be in front of you to the ground, 1m before
expecting safari fixes

model-viewer v1.9.1

We can close this since we use model-viewer version 1.10.1 right now

placement should be in front of you to the ground, 1m before

Is this fixed?

authors: roiLeo vikiival yangwao

repo: kodadot/nft-gallery


text:
feat: hover popup for collection item

Thank you for your contribution to the KodaDot NFT gallery.
👇 _ Let's make a quick check before the contribution.
PR type

[ ] Bugfix
[x] Feature
[ ] Refactoring

What's new?

[x] PR closes #3653
[ ] Requires deployment <rubick/magick_issue>
[ ] <brief_description_of_what_I've_added>

Before submitting Pull Request, please make sure:

[ ] My contribution builds clean without any errors or warnings
[ ] I've merged recent default branch -- main and I've no conflicts
[ ] I've tried to respect high code quality standards
[ ] I've didn't break any original functionality
[ ] I've posted a screenshot of demonstrated change in this PR

Optional

[ ] I've tested it at </rmrk/collection/26902bc2f7c20c546a-1FVG7>
[ ] I've tested PR on mobile and everything seems works
[ ] I found edge cases
[ ] I've written some unit tests 🧪

Had issue bounty label?

[x] Fill up your KSM address: Payout

Community participation

[ ] Are you at KodaDot Discord?

Screenshot

[x] My fix has changed something on UI; a screenshot is best to understand changes for others.

seems I see some volume here  https://deploy-preview-3677--koda-nuxt.netlify.app/bsx/collection/1437651165

https://deploy-preview-3677--koda-nuxt.netlify.app/bsx/collection/1235404021

My profile: offers made:
First issue: I'm not sure if that's possible, but it would be nice if expired offers would be cancelled automatically.
Another issue: after clicking on cancel and cancelling one offer - it did not disappear automatically (dissapeared after refresh); it stayed like this: (look at the first picture - NFT Spodee - you can see that red X is "clicked").

Last issue: I was playing with the offers I made (cancelling them), but sometimes they did not disappear (I had to cancel them twice) I got notifications like this - and I don't understand them - so maybe make them user-friendly?

pay 100

😍 Perfect, I’ve sent the payout
💵 $100 @ 64.91 USD/KSM ~ 1.541 $KSM
🧗 DaWoVNNhBTdqJ5iPN3UcaLeamkPpJeGWdrs8LhvX9TQbadc
🔗 0x73287438a18e244969b32cebc069cecf199a5569ca1ded69993d093fc10244b1
🪅 Let’s grab another issue and get rewarded!
🪄 github.com/kodadot/nft-gallery/issues

Work well for me

authors: JustLuuuu KngZhi jrafaelangarita kkukelka yangwao


text:
#4910 new input max min

Thank you for your contribution to the KodaDot NFT gallery.
👇 _ Let's make a quick check before the contribution.
PR Type

[ ] Bugfix
[ ] Feature
[x] Refactoring

Context

[x] Partially Closes #4910
[ ] Requires deployment <>

Before submitting pull request, please make sure:

[x] My contribution builds clean without any errors or warnings
[x] I've merged recent default branch -- main and I've no conflicts
[x] I've tried to respect high code quality standards
[x] I've didn't break any original functionality
[ ] I've posted a screenshot of demonstrated change in this PR

Optional

[ ] I've tested it at </bsx/collection>
[ ] I've tested PR on mobile
[ ] I've written unit tests 🧪
[ ] I've found edge cases

Had issue bounty label?

[x] Fill up your KSM address:
Payout

Community participation

[x] Are you at KodaDot Discord?

Screenshot 📸

[x] My fix has changed something on UI; a screenshot is best to understand changes for others.

dark mode miss arrow 👀

Explorer input max min

change the color of icon in dark mode.
Let's add a new translation key for it.
https://github.com/kodadot/nft-gallery/pull/4924/files/272c608265fc9bddd9f77aa44ac184bd36bce352#diff-243018571f28605fceba0ba9db964d3c1a8b5e9a00152d26f383eb09bd5bd071R39
https://github.com/kodadot/nft-gallery/pull/4924/files/272c608265fc9bddd9f77aa44ac184bd36bce352#diff-243018571f28605fceba0ba9db964d3c1a8b5e9a00152d26f383eb09bd5bd071R30

pay 20 usd
continue in

#4930

😍 Perfect, I’ve sent the payout
💵 $20 @ 36.35 USD/KSM ~ 0.55 $KSM
🧗 EzGc4s9PgCPx1YnF3fqzhLzVHpHMTL4LWPScwpDrR8JKgSU
🔗 0x4e1982ba3b81a3de510605a73701d51c97c2faf5e5ad32db33e782a249e8b058
🪅 Let’s grab another issue and get rewarded!
🪄 github.com/kodadot/nft-gallery/issues


text:
🛑 stt is down

In 6501266, stt (https://squid.subsquid.io/speck/v/v1/graphql) was down:

HTTP code: 504
Response time: 3160 ms

Resolved: stt is back up in 44ebf5b.

repo: kodadot/uptime


text:
[FUNCTION] PlayAnimationOnce

A function that plays an animation ONLY once.

I didn't need this function. I added a parameter to the "PlayAnimation" function so the user can choose if they want to play it once or not.


text:
Add a button to reset headers

Describe your suggested feature
After opening a webview, the headers setting of Client.get() may change, which sometimes can cause the extension to not work. A button to reset the headers can be added on the extension detail page.
Other details
No response
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.

Can you give some uses cases

https://github.com/kodjodevf/mangayomi-extensions/blob/main/javascript%2Fmanga%2Fsrc%2Fzh%2Fcopymanga.js#L182-L184
In the above code, the User-Agent for a desktop environment is set through headers. However, if a webview is opened on an Android platform later, the User-Agent will be overwritten to be mobile.

Oh I see. I will see what to do for this

Fixed!

authors: Ftbom kodjodevf

repo: kodjodevf/mangayomi


text:
chunkc set global_desktop_offset_top doesn't seem to work

When I set the global desktop offset it seems to be applied to different screens.. but once I swap to that screen I can't change it?
It also doesn't seem to affect the current screen, is this a bug or does another command need to be called afterwards?

This is an init setting only and changes are not reflect during runtime. Originally, this was the reason for it being a "set" command, and not like the others. This convention has not been followed consistently though. Which is something that should be fixed sooner rather than later.

Is it noted anywhere that not only are they init-only, but all the global_desktop_offset_* settings need to be done before chunkc core::load tiling.so? That bit me.

https://koekeishiya.github.io/chunkwm/docs/userguide.html

Plugin settings should be set before the command to load said plugin.

In the configuration section.

Ah, good to know others won't be messing up! Watch your 6 for a new issue in a sec.

authors: cormacrelf dominicglenn koekeishiya

repo: koekeishiya/chunkwm


text:
Feature: 增加图片筛选

Sources
Danbooru, Gelbooru, Konachan.com (NSFW), Konachan.net (SFW), Lolibooru, Lolicon, Pixiv, Safebooru, Yande.re
Describe the problem related to the feature request
图源中未筛选，图片质量层次不齐
Describe the solution you'd like
增加图片筛选（若支持），优化pixiv图片筛选。
Describe alternatives you've considered
No response
Additional context
No response

So the filter feature could be implemented as an option or a configuration, which one is better?

Configuration. Option would be too complex to use.

I think add default configuration and search option will fully meet the demand, but maybe a bit complex to use. Just configuration first?

So there are multiple ways to filter an image:
Booru-series sites

[ ] Tag
[ ] Author
[ ] Rating (Safe, Questionable, Explicit, etc)
[ ] Furry, Guro, BL, etc

Pixiv

[ ] Is AI Generation
[ ] R18 / R18G
[ ] Tag
[ ] Author
[ ] Bookmarks count

Lolicon

[ ] Is AI Generation
[x] R18 (differ to Pixiv's R18 standard)
[ ] Tag
[ ] Author

好复杂哦

I would implement the basic support fur this in the configurations then release the v1.0.2 version.

repo: koishijs/koishi-plugin-booru


text:
Mouse over doesnt render description (just empty pop up) Free code camp

Describe the bug

To Reproduce
https://www.freecodecamp.org/learn/front-end-libraries/bootstrap/use-responsive-design-with-bootstrap-fluid-containers

Fixed in v4.7

authors: ediardo iphutch

repo: kommandr/browser-extension


text:
增加 link 标签的 crossorigin tag

有些 link 标签也会链选一些外部的资源，所以也需要设置跨域标签，可以在src 中增加下述代码
htmlPluginData.assetTags.styles.forEach(item => (item.attributes.crossorigin = this.options.crossorigin))`

有些 link 标签也会链选一些外部的资源，所以也需要设置跨域标签，可以在src 中增加下述代码
htmlPluginData.assetTags.styles.forEach(item => (item.attributes.crossorigin = this.options.crossorigin))`

请问具体的配置方法

authors: baifengbxl clevergod

repo: komomoo/crossorigin-webpack-plugin


text:
Custom folder with field-name as filename for Render PHP File

You have multiple options for choosing a folder on the dev branch, but the one I'm missing:
Allow for a folder to be specified, say below templates/ but then automatically use the fieldname as filename by default.
For example:
templates/fields/fieldname.php
modules/MyModule/fields/fieldname.php
So one would only need to specify the folder/path but not the file name for each field. This could even be a base module setting...
Hope this was clear.

I'm not sure about this one. I'll need to have a think.

authors: kongondo noelboss

repo: kongondo/FieldtypeRuntimeMarkup


text:
Update awssdk:auth, awssdk:core, ... to 2.17.127

Updates

software.amazon.awssdk:auth
software.amazon.awssdk:core
software.amazon.awssdk:regions

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

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

labels: library-update, early-semver-patch, semver-spec-patch, commit-count:1

Superseded by #29.


text:
Notify grid nodes when node information is updated

Notify all connected grid nodes when node information (currently only labels) are changed. Changing node information might have effect on how network and other stuff is configured.

@jnummelin ping

LGTM

authors: jakolehm jnummelin

repo: kontena/kontena


text:
Fix commands for windows

#21

also, there were some changes in the rulesets that's causing CI to fail. we need to add two more labels on one of the rules that gets triggered. Here's a diff:
diff --git a/test-data/analysis-output.yaml b/test-data/analysis-output.yaml
index c9f0c89..7d24d64 100644
--- a/test-data/analysis-output.yaml
+++ b/test-data/analysis-output.yaml
@@ -137,6 +137,8 @@
       category: mandatory
       labels:
       - konveyor.io/target=cloud-readiness
+      - konveyor.io/target
+      - konveyor.io/source
       - discovery
       incidents:
       - uri: file:///opt/input/source/target/classes/persistence.properties

authors: eemcmullan pranavgaikwad

repo: konveyor-ecosystem/kantra


text:
Add tacle-diva into the tackle tool list

Add tackle-diva into the tackle tool list.

LGTM

authors: YasuKatsuno apinnick

repo: konveyor/tackle-documentation


text:
Can you write new tiddlys from the webpage?

Hello!
I was wondering how this works. Does this code allow you to write a new entry and save it so github from the website? Or what are the steps you take to add entries? Thanks

Hi @juilie
This is a workflow to create Tiddlywiki from Node.JS! So, you can edit a single tiddler and you will see the updated index.html is there.
New way to do this is GitHub Actions.

authors: juilie kookma

repo: kookma/Tiddlywiki-Travis-CI


text:
[proposal] Report node allocatable RDT ctrl groups

What is your proposal:
The koordlet needs to report the node allocatable of the Resctrl/RDT ctrl groups, so the koordlet modules and other components can be aware of the resctrl capacity and achieve guaranteed allocation.
Why is this needed:
#756, #1798.
The number of the machine's Resctrl/RDT groups is limited and normally much less than the pod number of a node. To ensure the resctrl groups are available to set up for pods, the node allocatable of the resctrl groups should be collected and reported. Furthermore, we can schedule guaranteed resctrl groups for pods with capacity awareness and cooperate with the QoS strategy which may reserve node-level groups.
Is there a suggested solution, if so, please add it:

/assign @kangclzjc

repo: koordinator-sh/koordinator


text:
[リファクタ] PrivacyPolicyContentの改修

Overview

Composableの分割と一部Previewを追加した

JaCoCo Code Coverage 74.78% :white_check_mark:

Class
Covered
Meta
Status

  Generated by :no_entry_sign: Danger

authors: KSNDBOT kosenda

repo: kosenda/hiragana-converter


text:
Alarming error output sql

Apologies, I am probably doing something horribly wrong. Inserts are failing with
 code: 'ER_PARSE_ERROR',
  errno: 1064,
  sqlState: '42000',
  sqlMessage: "You have an error in your SQL syntax; check the manual that corresponds to your MariaDB server version for the right syntax to use near '' at line 1",

Confused, I pasted the code into an SQL editor, and found that syntax highlighting was also broken. It breaks
'
Omie,

We aren\\'t in contact
                        ^^
                        here

And the problem becomes obvious! Single quotes are being escaped like \\', when really they ought to be like ''.
I'm unclear where this escaping code is even coming from. I did a search for escape and found nothing. Interestingly, Connection.escape(value: any): string is also escaping in a faulty way:
'
Omie,

We aren\'t in contact

So it's pretty unclear to me if this issue is coming from kysely or mysql2. I was hoping you could provide some insight.
Some other things

Regardless of the source of the problem, it's very concerning. Will a deliberate, more sophisticated attack be handled?
I can't escape manually, I tried and it just results in exponential \\\\\\\\\

You should never ever ever ever ever concatenate user input to SQL strings! Instead you should pass all user input as parameters. Then, there's no need to escape anything, as the value is never added to the SQL string. Could you share the code that's causing that and I can point out how it should be written?

The code is simple
    await db
        .insertInto('ltc')
        .values([message: "
Omie,

We aren't in contact"], [message: "
Very cute. Nice try
"])
        .onDuplicateKeyUpdate({})
        .execute()

There's absolutely no way that's causing the bug you reported, unless you are doing something crazy with plugins or using a third-party dialect.

Here's one of the tens of tests I have for inserts and the SQL that's produced
      const query = ctx.db.insertInto('person').values({
        first_name: 'Foo',
        last_name: 'Barson',
        gender: 'other',
      })

      testSql(query, dialect, {
        postgres: {
          sql: 'insert into "person" ("first_name", "last_name", "gender") values ($1, $2, $3)',
          parameters: ['Foo', 'Barson', 'other'],
        },
        mysql: {
          sql: 'insert into `person` (`first_name`, `last_name`, `gender`) values (?, ?, ?)',
          parameters: ['Foo', 'Barson', 'other'],
        },
        sqlite: {
          sql: 'insert into "person" ("first_name", "last_name", "gender") values (?, ?, ?)',
          parameters: ['Foo', 'Barson', 'other'],
        },
      })

As you can see, the values are not concatenated to ths SQL.

Actually it could happen if you are doing that in a migration? Is that migration code?

Don't know what to tell you. I can't reproduce that.

Sure, I want to get to the bottom of this. One moment

Actually that empty onDuplicateKeyUpdate is causing the query to fail but it has nothing to do with the input.

Oh interesting. Is it possible that when the query fails, it just dumps an odd output?

The code

#!/usr/bin/env ts-node

import { Generated, Kysely, MysqlDialect } from "kysely"
import { createPool } from 'mysql2'

const letters = [
    {
        "Id": 13,
        "letterMessage": "
Omie,

We aren't in contact - that's. Recently, I haven't. I. If, all.

One hope, right?
",
        "letterPostDate": "/Date(1650851725000)/",
    },
    {
        "Id": 12,
        "letterMessage": "
Nothing special here. A few periods.",
        "letterPostDate": "/Date(1234567890000)/",
    }
]

interface LtcBase {
    letterMessage: string;
}

interface LtcTable extends LtcBase {
    id: Generated<number>
    letterPostDate: Date;
}

interface Database {
    test: LtcTable
}

let pool = createPool({
    database: "your",
    host: "localhost",
    user: "creds",
    password: "here"
})

const db = new Kysely<Database>({
    dialect: new MysqlDialect({
        pool: pool
    })
})

await db.schema
    .createTable('test')
    .addColumn('id', 'integer', (col) => col.autoIncrement().primaryKey())
    .addColumn('letterPostDate', 'datetime')
    .addColumn('letterMessage', 'text')
    .execute()

// @ts-ignore
const patchedJson: LtcTable[] = letters.map(letter => {
    delete Object.assign(letter, { ["id"]: letter["Id"] })["Id"]
    // @ts-ignore
    letter.letterPostDate = new Date(Number(letter.letterPostDate.split('/Date(')[1].split(')/')[0]))
    return letter
})

await db
    .insertInto('test')
    .values(patchedJson)
    .onDuplicateKeyUpdate({})
    .execute()

console.log('done')

You will need to add your credentials but otherwise should work fine.
Some observations

Without the .onDuplicateKeyUpdate({}) the query does indeed work fine 🎉 🎉
The error is very concerning with the \\'

  code: 'ER_PARSE_ERROR',
  errno: 1064,
  sqlState: '42000',
  sqlMessage: "You have an error in your SQL syntax; check the manual that corresponds to your MariaDB server version for the right syntax to use near '' at line 1",
  sql: "insert into `test` (`letterMessage`, `letterPostDate`, `id`) values ('
Omie,

We aren\\'t in contact - that\\'s. Recently, I haven\\'t. I. If, all.

One hope, right?
', '2022-04-24 21:55:25.000', 13), ('
Nothing special here. A few periods.', '2009-02-13 18:31:30.000', 12) on duplicate key update "
}

Somehow this incorrect query worked 80 times before erroring (the for loop in the actual code posted above successfully iterated 80 times)

I think mariadb is just printing the whole thing as SQL for some odd reason, but the error is from the invalid SQL caused by the wrong use of onDuplicateKeyUpdate.
Feel free to add console.logs to the kysely code to see what kysely sends to the mysql2 driver. Kysely is definitely not doing something that stupid. And neither is mysql2.
The error sqlMessage has nothing to do with the ' characters in the values.

Here's the line where the query is sent to mysql2. You can edit that line in your node_modules to see what's happening.

Thank you! Sorry for that whole roller coaster. Is there some way I could buy you a coffee?

There's no way or need to do that. Thanks for the offer though 😊

authors: boehs koskimas

repo: koskimas/kysely


text:
InsertResult insertID type uses bigInt instead of number

Why the types uses bigInt | undefined instead of number | bigInt | undefined ? isn't it too restricting because bigInt type is never going to happen early on?

Kysely wraps the integer in a bigint.

@koskimas Sorry for pinging you, but isn't it better if we only wrap it with BigInt if the count is greater than 2^53 which is the official threshold recommendation from MDN ? this could also improve performance I think

It would be a bad API if the value changed based on the size of the value. Performance-wise this is insignificant. Inserting a row takes some milliseconds and even if you remove all waiting time, tens or hundreds of microseconds of CPU time. Converting a number to a Bigint takes nanoseconds. So the insert itself takes tens or hundreds of thousands of times longer. At best you could speed things up by 0.001% to 0.01% by making the API more difficult to use. Not a good tradeoff.

Ah okay thanks for the explanation!

@koskimas Sorry for reviving this issue but I have found while looking for bigint issue I am having before creating a new issue.
kysely-codegen generates number type for auto increment id's in planetscale. I can manually go and edit all the typings but it doesn't really feel like a correct approach. Do you think it's an issue on codegen or it works as expected?

@koskimas Ohh I am sorry I think that's not a project you are involved but a 3rd party 😅

This issue is about the InsertResult type that's returned if you don't return anything from an insert query using returning. The InsertResult is not related to the table/column types you provide to Kysely. It's always the same.

authors: harisraharjo koskimas suchcodemuchwow


text:
feat: add taint for node

What type of PR is this?

/kind feature
What does this PR do?

Which issue(s) does this PR fix?

Fixes #
Special notes for your reviewer:

Does this PR introduce a user-facing change?

NONE

/lgtm
/approve

/hold

/unhold

/unhold

authors: OrangeBao duanmengkk

repo: kosmos-io/kosmos


text:
Removing of deprecated builders suffixed by OrThrow

Description
Like discussed in the idea #104, we would like to remove deprecated builders suffixed by OrThrow.
Dependencies
This issue is blocked by the following ones:

[x] #120

Checklist

[ ] Remove deprecated builders with a hidden level.
[ ] Test.
[ ] Refactor.
[ ] Check the generated documentation.
[ ] Check and update the public API binaries.
[ ] Update the Unreleased section in changelog.

These type converters suffixed by OrThrow will be removed in the issue #258.

repo: kotools/types


text:
what's the difference between load_stock_weight, load_bert_weight and load_albert_weight?

As the name suggested, load_bert_weight and load_albert_weight are used to load bert and albert weights, But what's load_stock_weight mean?

@fancyerii - load_stock_weights is a synonym and the old/deprecated name for load_bert_weights. Both of them are the same method. I have initially used the name load_stock_weights but only later realized that it is not a good name (it is still there for compatibility, and I will remove it at some point).

authors: fancyerii kpe


text:
Use aot-test-generators to generate test files

This makes it possible to use Mocha tests out of the box.

I'll release tonight.

authors: kratiahuja t-sauer

repo: kratiahuja/broccoli-tslinter


text:
JWT Authentication

Implement JWT authentication in middlewares/auth.js

Hey, I would like to work on this

repo: kratosbase/kratos


text:
The custom token format is incorrect

I am using the bellow snippet to generate custom tokens for clients who are using my Flutter mobile app.
$privateKey 	= json_decode(file_get_contents( storage_path()  . '/app/google-services.json'), true)['private_key'];
$generator 	    = CustomTokenGenerator::withClientEmailAndPrivateKey($clientEmail, $privateKey);
$token 		    = $generator->createCustomToken('uid', ['first_claim' => 'first_value' /* ... */]);
return json_encode(["status_code" => "200", "token" => $token->toString() ]);

When I am reusing the $token->toString() in my   FirebaseAuth.instance.signInWithCustomToken ( token: '${httpResponse['token']}') I get firebase error  “FirebaseAuthInvalidCredentialsException: The custom token format is incorrect.” .
According to firebase group it is a generic error message for if the maximum expiration time is more than one hour
https://groups.google.com/forum/#!msg/firebase-talk/tYRfde4nwBY/yAn5I_spBQAJ . Does the CustomTokenGenerator::withClientEmailAndPrivateKey take any maximum expiration time parameter?
Also I wonder if the $token->toString() is the correct value which I should return from my server to my mobile app ?
Thank you for you troubles.

Hi @ronlinet and thanks for reaching out!
Custom tokens have a TTL of one hour (3600 seconds) by default and the duration is added to the current time, so my assumption is that the time in your environment is ever so slightly ahead of the time on Google's servers.
As an example: if it's 12:00 on Google's Servers, but 12:01 in your environment, the token will be set to expire at 13:01. Since it's one minute earlier for Google, they say "wait, that's 61 minutes, but we only allow a maximum TTL of 60 minutes, so... no!"
I probably didn't need to explain this to you, I just added the explanation for others that might stumble upon this issue as well.
If you want to try out using a shorter TTL to see if this helps, you can use a third parameter on the createCustomToken() method with a custom TTL:
$uid = 'uid';
$claims = [/* ... */];
$ttl = 1800; // or 'PT30M'

$token = $generator->createCustomToken($uid, $claims, $ttl);

But I would still recommend to check if the time in your environment is in sync with the world 🙈

The token string is the correct value, if you use it to sign-in with on the client-side, for example in the Web Client SDK as described here: https://firebase.google.com/docs/auth/web/custom-auth#authenticate-with-firebase

Hi @jeromegamez thank your for you qick feedback.
To put you more in context...
My firestore database rule is set as following
allow read : if request.auth != null;
My web server server is acting as an authorization portal.
The clients are already authenticated with authorisation headers in my website.
All I need is to blindely delivering the tokens to any http query that reaches the bellow php code.
$privateKey     = json_decode(file_get_contents( storage_path()  . '/app/google-services.json'), true)['private_key'];
$generator      = CustomTokenGenerator::withClientEmailAndPrivateKey($clientEmail, $privateKey);
token           = $generator->createCustomToken('uid', ['first_claim' => 'first_value' /* ... */], 'PT30M');

return json_encode(["status_code" => "200", "token" => $token->toString() ]);

As you see I have tried to set time for less then one minute but no chance the mobile app is throwing an invalid format execption.
Unhandled Exception: PlatformException(ERROR_INVALID_CUSTOM_TOKEN, The custom token format is incorrect. Please check the documentation., null)
I have dumped the $token->toString()  what may be wrong tiwh format here ?
eyJ0eXAiOiJKV1QiLCJhbGciOiJSUzI1NiJ9.eyJpc3MiOiJzdHVkZW50QHByb2QuY29tIiwic3ViIjoic3R1ZGVudEBwcm9kLmNvbSIsImF1ZCI6Imh0dHBzOlwvXC9pZGVudGl0eXRvb2xraXQuZ29vZ2xlYXBpcy5jb21cL2dvb2dsZS5pZGVudGl0eS5pZGVudGl0eXRvb2xraXQudjEuSWRlbnRpdHlUb29sa2l0IiwiaWF0IjoxNTg2MjE1NzU1LCJleHAiOjE1ODYyMTc1NTUsInVpZCI6InVpZCIsImNsYWltcyI6eyJmaXJzdF9jbGFpbSI6ImZpcnN0X3ZhbHVlIn19.J8uN3ybYYqvRkMFVE3dCSmFVnibJ7ZLCA_noFhZaGGBX_DBB9RFXjMOyVaSOEUxy_fYQB8sgf6LMNNKxyJlCcUS9AiQ2mILuFsuyL5TLhCHgn6AG3vJn_w4dZM8xH_l-iyskfVMA0fTWTisTgLoLMDzqZsE8e4tXdpYjZGFnLBn7IvY1KFJVWTOlkHlzh63r5gfh08VFFMuMHvIBxWs7ijVOFgEnRY5G54OgnXIEMl1D_CgaGW_lV88Z3k9mJwh9Por5S3FLQwE9X7ZzjLDz9QIk3QKnSdN0OioxMkH56i9f77Rjw5v3Dg1LUOV3Vepr2BHhuBU05BDpwi_dopvIdw

PS: I will see about the time server and google API time.

I noticed that you don‘t seem retrieve the client email from the JSON file as you do with the private key - is this really the case? If so, could you make sure that the client email you‘re using matches the value in the JSON file?
Beside that I currently have no idea what might be the issue, but I‘ll make sure to try it myself as well as soon as I‘m back at my computer. 🤞

I just tested this with my project with the help of kreait/firebase-php and it works for me™. So a possibly mismatching client email is the only possible issue left that I could think of.
I just also noticed that you seem to be using Laravel/Lumen - have you considered using https://github.com/kreait/laravel-firebase or is it too heavy for your use case?

Casting the token object to string is the solution in my case.
@jeromegamez thank you for your time, much appreciated.
Working snippet :
use Kreait\Firebase\ServiceAccount;
use Kreait\Firebase\Factory;
$serviceAccount = ServiceAccount::fromJsonFile( storage_path()  . '/app/google-services.json');
$firebase = (new Factory)
                        ->withServiceAccount($serviceAccount)
                        ->create();
$token = $firebase->getAuth()->createCustomToken($request->header('PHP_AUTH_USER'));
return json_encode(["status_code" => "200", "token" => (string) $token ]);

Aaaah! My mistake, you‘re right! I forgot that lcobucci/jwt has different outputs of toString() vs casting the object 🙈, and I apparently didn‘t fully test the same procedure that you used 😅. Kudos for getting to a solution, and thanks for sharing! 🌺

authors: jeromegamez ronlinet

repo: kreait/firebase-tokens-php


text:
Provide a title for the chart in the chart options?

Is there a way to add a title to the chart?

Currently, it's not provided via options. I think it can be added, if it will be demanded by community.

When this feature will be available?

+1

authors: Raghuramsoori fsaez-ix krispo sisirap


text:
Gulp pipeline baby!

Check it out. It's pretty sweet IMHO.
cc/ @MiniGod @benediktvaldez @kristjanmik

Oh baby. This is ready for review! :revolving_hearts:

Maybe I should add documentation for this...

BOOOOOM!
Somebody review me plz!

I think I resolved both your questions @kristjanmik.

repo: kristjanmik/apis


text:
swth: do not ignore keyword arguments after positional argument

Previously, swth go --biber would silently use biblatex. Now it works!

I don't get it, can you give an example?

On second thought, No, swth go is not meant to take options.
It really should take info from the .swth file

Before, all keyword arguments after the command were silently ignored. While swth --dry-run go did a dry run as expected, swth go --dry-run did an actual run. It's just about typing convenience. (Personally, I think --dry-run is simpler than modifying the .swth file for a single invocation, but that's just an opinion ...)
It's the same for swth create and others. Unless you want to eliminate all keyword arguments in this script, this patch will make sure that they are not ignored just because they appear after instead of before the command. Nothing else. :-)

I understand.
But I disagree. The options ought to come before the command, because all arguments after the command are intended to be passed through.
So
swth latex -interaction=nonstopmode

or
swth --dry-run latex -interaction=nonstopmode

ought to work.
It still does make only little sense for swth go

The options ought to come before the command, because all arguments after the command are intended to be passed through.

Ah, now I see! Thanks for the explanation! Do you think it would make sense to alter
https://github.com/krono/swathesis/blob/08dfb005c9f983c20225fddf6d968bdcb4e22641/swth#L15
like
USAGE="Usage: $PROGRAM [OPTIONS] COMMAND [COMMAND_OPTIONS]

to avoid future confusion in naive script users like me? :)

Well if you care that much.
swth is merely a tiny helper from times long before I knew arara or latexmk.
So keeping this in mind, it is just a tiny tool :)

authors: LinqLover krono

repo: krono/swathesis


text:
Feature/fri wrench control

Created a wrench forward controller, a controller for setting the cartesian impedance values, modified the robot application to accept cartesian impedance control mode. Not tested on real robot yet.

PR can be merged after:

comments are resolved
tested on real robot
CI passing
documentation extended with new parameters

I will approve after testing on real hardware

authors: Svastits krmihaly


text:
Binlog-Event is to big - How to skip a binlog event?

Operating System: Linux
PHP Version: 7.2.8
php-mysql-replication Version: v5.0.4
mysql version:  Percona 5.7.18

We have a binlog event that is larger than 1 GB and therefore receive the error message log event entry exceeded max_allowed_packet; Increase max_allowed_packet on master; the first event 'mysql-bin.021114' at 2147481564, the last event read from '/var/log/mysql/mysql-bin.021114' at 123, the last byte read from '/var/log/mysql/mysql-bin.021114' at 2147481583..
Unfortunately, the values for max_allowed_packet or slave_max_allowed_packet can not be set larger than 1 GB.
How can we solve the problem or skip the event?

We found out the following additional information and corrections:
The problem only occurs

Because the binlog file on the database master is 9 GB in size. The individual events are small and the binlog file has become so large because a very large insert / update query has led to many thousands of individual row events.
If we do not start at the beginning of the binlog file

A work-around could therefore be that always beginning at the beginning of the binlog file and all events are ignored until reaching the relevant position.

Hi, @check24-axel-nitzsche I'm now sure 100% if this will work for you but to skip this event you must use for configuration binLogFileName and binLogPosition. So start from example  bin-001 position 1 then wait until it crash on that big event on position 100 and then restart from position 101. I will try to replicate and add look for permanent solution in free time.

I ran into this same error today. I found that the max_allowed_packet error can occur for other reasons, including resuming from the wrong spot in a binlog.
My cause today:
I had been setting the eventsOnly option, and not including the ROTATE event. As the code currently stands, its critical that this event always be included, because otherwise the binlog filename will not be updated here: https://github.com/krowinski/php-mysql-replication/blob/f6033e7e4dbe971802a8879e4f85589a0df3a519/src/MySQLReplication/Event/RotateEvent.php#L26
@krowinski The rotate is a pretty important event to handle internally. Should the user really be able to inadvertently disable the functionality?

Hi @c4l3b tx for reporting this, indeed its a bug I fixed that and also added all mandatory events before checking if you can skip them.

repo: krowinski/php-mysql-replication


text:
Deployments managed by InferenceService are not updating status at all!

/kind bug
What steps did you take and what happened:
kubectl rollout status deployment/my-service-transformer-default -n my-namespace
It keeps hanging with this response:
Waiting for deployment spec update to be observed...
This deployment was created by Inference Service. I'm using RawDeployment
What did you expect to happen:
It should tell me the status of the deployment. That it has successfully finished or something like that. This command works for any other deployment on my cluster not managed by InferenceService Controller. But for deployments created by InferenceService it just hang.
Anything else you would like to add:
I just feel there is a bigger and more general problem here. From two days investigating this issue, I just feel InferenceService is not really updating the status of deployments and something seems to be fundamentally wrong with these deployments. Even command like:
"kubectl rollout restart deployment/my-service-transformer-default -n my-namespace" does absolutely nothing. Not even restarting the pods!!
Environment:

Istio Version:
Knative Version: Not used (I'm using RawDeployment)
KFServing Version: KServe 0.8
Kubeflow version: N/A (use KServe standalone)
Kfdef:[k8s_istio/istio_dex/gcp_basic_auth/gcp_iap/aws/aws_cognito/ibm]
Minikube version: N/A
Kubernetes version: (use kubectl version): 1.23
OS (e.g. from /etc/os-release): 12.4

@abdulvirta the deployment should be managed by high level resource inference service, you should never manipulate the deployment directly. This is fundamentally how KServe works, if you want to check the deployment status you can check the  inference service status instead.

yeah this won't work. The conditions on inference services are not updated during a rollout. TransformerReady and PredictorReady will always have True even when a rollout is in progress. What I want is some kind of condition to wait on until a new deployment/rollout finished successfully. Like what you ideally would do with this command:
kubectl rollout status deployment my-deployment
The scenario is:

kubectl apply ....
Wait until the new changes have fully manifested (meaning new pods containing the new changes are up and running, and old pods are terminated)
Do validation for the deployment.

I don't see any possible way to do this. Because when I make a new deployment, InferenceService is not keeping track on that. And the underlying deployments resources doesn't seem to get updated so doing  kubectl rollout status deployment ... command will just hang forever

@abdulvirta the ready condition is not always true, it transits to false when you make a new deployment  and marks true until the new deployment is ready.

hmmm this is not what I'm observing for sure for whatever reason

authors: abdulvirta yuzisun

repo: kserve/kserve


text:
fix: Fix fq_names metrics parameter

Motivation
fq_names is a boolean sub-parameters that can be used to alter the metrics exposed by model-mesh. Unfortunately however its value is inverted, so setting it to "true" actually disables fully qualified method names (which is already the default).
Modification
Change to interpret anything other than false as enabling (default is still disabled).
Result
Metrics configuration behaves as expected.

Makes sense to me! Thanks Nick.
/lgtm

authors: njhill rafvasq

repo: kserve/modelmesh


text:
[BE/FEAT] 팀빌딩을 생성합니다.

📌 관련 이슈
closed #244
✨ PR 내용
팀빌딩을 생성합니다.
팀빌딩 생성 시 active 한 팀빌딩은 1개로 제한되어야 하는 제약사항은 별도의 PR을 통해 구현합니다.
주의 사항
브랜치 방향 꼭 확인하세요!!!!

통과하지 않는 테스트 13개는 별도의 이슈를 통해 해결합니다.
따라서 해당 PR은 merge 합니다.

repo: kssumin/black-company-be


text:
🛑 Nexus is down

In a6cf41c, Nexus (http://repo.chelsea.kt.co.kr) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Nexus is back up in f56219a.

repo: kt-chelsea/status


text:
🛑 Chelsea is down

In 9d320c0, Chelsea (http://chelsea.kt.co.kr) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Chelsea is back up in fc168fb.


text:
Merge pull request #1 from ktaranov/master

Merge changes to my fork
Fixes # .
Changes proposed in this pull request:

How to test this code:

Has been tested on (remove any that don't apply):

Case-sensitive SQL Server instance
SQL Server 2008
SQL Server 2008 R2
SQL Server 2012
SQL Server 2014
SQL Server 2016
Amazon RDS
Azure SQL DB

merging to my repro

repo: ktaranov/sqlserver-kit


text:
Sbt interactive session halts in case of jmh error

I run interactive sbt session and start jmh benchmark that contains some errors.
The session halts on error.
Is it possible to keep session running?
sbt
jmh:run
[info] Done packaging.
@TearDown annotation is placed within the class not having @State annotation. This has no behavioral effect, and prohibited.
[org.openjdk.jmh.generators.reflection.RFMethodInfo@4234de75]

Sample code to reproduce:
import java.util.concurrent.TimeUnit
import org.openjdk.jmh.annotations._

object TestBenchmark {
@State(Scope.Benchmark)
class BenchmarkState {
@Param(Array("25", "50", "100", "1000"))
var elementsCount: Int = 0
var items: Seq[Int] = _
}
@Setup
def setup(state: BenchmarkState): Unit = {
state.items = (0 until state.elementsCount)
}
}
@Warmup(iterations = 5, time = 1)
@Measurement(iterations = 5, time = 1)
@BenchmarkMode(Array(Mode.AverageTime))
@OutputTimeUnit(value = TimeUnit.MICROSECONDS)
class TestBenchmark {
import TestBenchmark._
@Benchmark
def benchmarkSize(state: BenchmarkState): Unit = {
state.items.size
}
}

@f1yegor It seems no one would write like this way. There are some examples that you could see how to write with JMH. ListBenchmark.scala from Scala's repository or TestBenchmark.scal from this repository.

repo: ktoso/sbt-jmh


text:
Create daily docker image build

What would you like to be added:
Run the docker image build for kubedl daily
Why is this needed:

this is done

repo: kubedl-io/kubedl


text:
Propagate configuration/route conditions to KFService

/kind feature
This is to fix the TODOs left in the kfservice_status.go to propagate configuration and route conditions to KFService.
https://github.com/kubeflow/kfserving/blob/master/pkg/apis/serving/v1alpha1/kfservice_status.go#L51

/assign @yuzisun

repo: kubeflow/kfserving


text:
[request] Add devstats in our website

We have a devstats deployment for visualizing various metrics for the kubeflow community: http://devstats.kubeflow.org
I think we could add it in the website, maybe together with #185

/area docs

/assign
/close
I'm going to close this issue, because the devstats dashboard is no longer working. Please do re-open the issue if there's work to be done in the docs.

authors: gaocegege sarahmaddox

repo: kubeflow/website


text:
[Feature] Remove specific label/annotations for configmap/secret

Contextualization
When I use kubed and argo-cd together, kubed will sync labels such as argocd.argoproj.io/instance which will make argo-cd classify the configmap/secret as a part of application and mark app as "OutOfSync".
Solution
Use annotation/labels like argocd.argoproj.io/remove-label: labelA, labelB and argocd.argoproj.io/remove-annotation: AnnoA, AnnoB to remove the label/annotation from the annotation/labels.
Any help will be highly appreciated :)

Any ideas/workarounds?

repo: kubeops/config-syncer


text:
fix: Adjust order for Log and output err when get NatRule faild.

[x] Make sure you have followed Kube-OVN Code Style.

What type of this PR

Bug fixes

Which issue(s) this PR fixes:
Fixes #1732

Thanks! @xujunjie-cover

authors: oilbeater xujunjie-cover

repo: kubeovn/kube-ovn


text:
fix: can not update LSP into another LS

Pull Request

[ ] Make sure you have followed Kube-OVN Code Style.

What type of this PR
Examples of user facing changes:

Bug fixes

update 端口 ip的时候，需要校验下子网和分配 ip 的子网是一致的,。
ovn 在某一个子网更新使用为另一个子网的 ip 是无效的，
这样会导致 ip 对应的 LSP 无法删除，如果一直使用这个 ip ，在无法自动修复，只能手动清理 LSP。
先添加一种逻辑： 当子网不一致时，需要先清理旧的 LSP。

Which issue(s) this PR fixes
Fixes #(issue-number)

Pull Request Test Coverage Report for Build 12157439322
Details

6 of 9 (66.67%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.01%) to 18.691%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pkg/ovs/ovn-nb-logical_switch_port.go
4
7
57.14%

Totals

Change from base Build 12153017747:
0.01%

Covered Lines:
8689

Relevant Lines:
46488

💛  - Coveralls

之前的问题是因为 删除 LSP 应该要使用 lsp-del 的调用，lsp-add 应该没有效果

这样会导致 ip 对应的 LSP 无法删除

为什么无法删除？是 ovn 禁止删除还是 Kube-OVN libovsdb 里某个地方的代码禁止了这种删除操作？这个 PR 修复的是哪种故障场景？

这样会导致 ip 对应的 LSP 无法删除

为什么无法删除？是 ovn 禁止删除还是 Kube-OVN libovsdb 里某个地方的代码禁止了这种删除操作？这个 PR 修复的是哪种故障场景？

是因为子网被修改了，就跳过删除 lsp 了

场景1:
配置多网卡时，没有指定子网，这时会自动分配到 ovn-default，然后如果再指定子网，会更新 LSP 到指定的子网，然后 这个 LSP 就删除不掉了

确认是旧代码的问题

authors: bobz965 coveralls zhangzujian


text:
feat: add container logs max size to the kubeone API

What this PR does / why we need it:
This allow the user to set container logs maximum size on kubeone and defaults the container logs max size to 100Mi
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #1587
Special notes for your reviewer:
Does this PR introduce a user-facing change?:
YES

Ability to change container log maximum size. Currently defaults to 100Mi

/retest

/retest

/retest

Should/will this configuration be propagated to the initial machine deployment for workers? @nerdeveloper @xmudrii

@ahmedwaleedmalik Yes, we should. There's a PR in the machine-controller repo as well: https://github.com/kubermatic/machine-controller/pull/1173

authors: ahmedwaleedmalik kron4eg nerdeveloper xmudrii

repo: kubermatic/kubeone


text:
Remove hostpath

Signed-off-by: Humble Chirammal <hchiramm@redhat.com>

@msau42 Sure, I have removed makefile from the PR. PTAL

Can you also modify the top level README to say that this repo is deprecated. And point to the new replacement repos?
I also need to turn off travis for this repo.

/retest
I've disabled travis on this repo

repo: kubernetes-csi/drivers


text:
Is OWNERS stale?

They aren't exactly the same thing, but only @aaronlevy is listed in OWNERS while first-parent commits seem to come from a few people:
$ git glog --first-parent --format='%aN' | sort | uniq -c | sort -n
      1 * Chance Zibolski
      1 * Kenan Karamehmedovic
      1 * Levi Blackstone
      1 * Quentin Machu
      2 * Abhinav Dahiya
      2 * Kristian Klausen
      2 * rithu leena john
      3 * Brandon Philips
      3 * Lucas Servén
     11 * Cole Mickens
     13 * Xiang Li
     14 * Hongchao Deng
     19 * Eric Chiang
     23 * Ryan Phillips
     24 * Derek Parker
     27 * Yifan Gu
     28 * Patrick Baxter
     48 * Dalton Hubble
     71 * Diego Pontoriero
    229 * Aaron Levy

Some of those entries may be due to backwards merges or squashed merges (e.g. @chancez's 94e50ffe is a squashed merge or similar of #94.  But at least some of the higher-count folks seem to wield approver power.

We should update it. Myself @rphillips and @colemickens have been the primary maintainers for a bit.

authors: ericchiang wking

repo: kubernetes-incubator/bootkube