Use Elmish for game state management (basic sample)

Finally I got some time to have a look at this :) First I tried to update #27 (actually this PR is branched from that) but I realized that #27, while sharing the state between the React and Pixi parts (so, for example, dragons can be added by clicking the button or the sprite), Elmish is used to manage the React part and it's complicated to share the update function with Pixi so at the end I opted for integrating Elmish with the basic sample.
Please have a look at src/basic/App.fs, it should be structured more or less as a typical Elmish application. For that I use the src/basic/Elmish.Animation.fs helper, which is an adaptation of the Elmish.Worker in the FableConf game workshop. Note that I've used an additional tick function to solve the issue mentioned here. Actually, I noticed with this the PIXI ticker is not actually necessary so I removed it.
Advantages of this approach:

Reduce the cognitive load: Devs can create games using the Elmish model.
Reuse Elmish helpers: In theory we can use other Elmish helpers. I say "in theory" because I had problems with the debugger (apparently Pixi objects don't serialize well) and HMR (it works but only the first time) so they may still need some work.

But there also disadvantages:

We still need the tick function: So devs have to handle two update functions. I tried to make dispatch only collect the messages and then make tick the actual update function, which would receive the collection of messages (aka events) happened in this frame together with the delta. However, this was not possible the way Elmish works. We would have to modify it for this.
Pixi sprites are not a good fit for an immutable model: There's mutation every where so it's difficult to keep state snapshots after every update. And restricting state mutations to only the update and tick functions can only be enforced by the programmer discipline, not the compiler.

So basically we need to decide if we want to a) adapt Elmish as is for games, b) create something similar but more fit to games, or c) leave the Elmish way definitely. What do you think? @whitetigle @MangelMaxime

First of all thanks for taking time to make these searches on that difficult Elmish topic @alfonsogarciacaro 👍
I need to review you code. Hopefully I will post my thoughts a little bit later this week-end! 😄

Hi!
In order to see what at stake here, I have gathered some information relative to a simple game architecture and what we have today.
State machines
Through the years I've always been using state machines.
A state is essentially a model that evolves through time until reaching its END_OF_STATE condition then triggering the change to a brand new state.
  let mutable gameState = Init

  // our render loop, every frame
  app.ticker.add (fun delta ->
    gameState<-
      match screen with

      | Init -> // prepare model then move to start
         let model = ...
         NextScreen (Start model)

      | Start model -> // launch start anim, often some title tween
          let model, reachedEnd = Start.Update model
          if reachedEnd then
             Start.Cleanup() // one call to remove all unused sprites
             NextScreen (Game model)

      | NextScreen nextScreen -> nextScreen // wait one frame to go to the next state

      | GameOver model ->
         let model, done = GameOverState.Update model
        if done then
             GameOver.Cleanup()
             NextScreen Init

      | Game model ->
          let model, reachedEnd = GameState.Update model
          if reachedEnd then
             GameState.Cleanup() // one call to remove all unused sprites
             NextScreen GameOver

    ) |> ignore

This sample could easily be refactored to have central cleanup system and GoToNextState  mechanism.
Then in every Update, there should be an internal local state machine.
So we would end with

General game model: the model all states would share, responsible to hold such things as numbers of players, configuration, high scores, etc...

Local state model: a model local to the current state. For instance: sprite list, current score, remaining lives, etc..

a way to know when to switch from one state to the other.

Events
Events plague all game developments with callbacks... that may call other callbacks to bubble up to an event management system wtht will tell what to do.
Here, the Dispatch mechanism in Elmish is just great. We can respond to events that come from the remotest sprite in a complex layer hierarchy. So that's great!
But then the information dispatched needs to be analysed thanks to a context, which would often be our local model. For instance, if sprite A collided with Sprite B, we would dispatch a CollisionBetween A B localModel. In our localModel we would have some sprite list which we would update consequently and maybe the reaching of END_OF_STATE (for instance Sprite A was our space ship and it just died)
Then, we would let the main loop or an upper management system handle this END_OF_STATE message. For instance, it would lead to the cleanup of the current state, local model and start of a new Game Over state to display a neat GameOver animation.
Elmish
Now regarding Elmish, I think that dispatching messages is the most important thing. But as you said, we are working in an always mutable land that does not need to wait for user input or model input to update its rendering. I mean, unlike your previous Canvas sample, here with pixi (and also with paperjs and maybe any game lib) I think it may be best to let the rendering happen and just control the logic and analyse the dispatched messages.
Strengthening
But like OpenFrameworks or Processing (setup, draw) I think we should enforce some mechanism to make our FSM more robust. It whould not be complicated because what we always end up using:

some sprite list which may be a custom list or just the container holding the sprites
some container list which we could also name layers since we have this notion of stacking up containers on top of each other.

Sprite licecycle
Also a sprite needs to be taken care of by removing it from its ancestor. There are two ways to do that:

either we have a reference to the container and do container.removeChild sprite

or

we do this right from the sprite like this mySprite.parent.removeChild mySprite. In case of generic sprites, this has the huge benefit of not having to use a custom list. We use the sole pixe container system.

Conclusion
if I read you well, and if I recollect my memories from previous experiences, I think using Elmish is not a good fit unless we animate everything using event based animation systems like Animatejs which allows us to have some [start-during-endfAnimation callbacks and can be easily fit into an Elmish architecture.
I do have this kind of projects. I think Secret of Monkey Island could fit well in an Elmish Architecture.
But to make a Bullet Hell shmup like like the Touhou series... well, I think we may just find try to dig further into a GloriousStateMachine 😉
I don't think I have said all. But I'm a little bit short on time so let's say here are my starters 😉

Thanks a lot for your feedback @whitetigle! So I guess the best option is to build our own framework, inspired by Elmish but more adapted to a 60-FPS game :+1:
State Machines
I absolutely agree with this. Actually one of the drawbacks of Elmish for me is that, because everything works by composition you cannot define isolated state machines. I'm a big fan of the proposal of Tomas Petricek to model state machines with async. This would allow us to represent the top flow very nicely. For example:
type Result = Win | Lose
type MenuOption = StartGame | ShowCredits
type Screen = Menu | GamePlay of int | GameOver | GameCompleted | Credits

let rec startGame(): Async<unit> = async {
    let! option = showScreen Menu
    match option with
    | StartGame ->
        let mutable level = 1
        while level > 0 && level < MAX_LEVEL do
            let! res = showScreen (GamePlay level)
            match res with
            | Win  -> level <- level + 1
            | Lose -> level <- 0
                      do! showScreen GameOver
        if level = MAX_LEVEL then
            do! showScreen GameCompleted
    | ShowCredits ->
        do! showScreen Credits
    return! startGame()
}

Events
I also agree here that a dispatch model à la Elmish is easier than a subscriber model. My only doubt is if the events have to be dealt with immediately or one by one, or rather just collected and be dealt with later in the update that in a game is going to be run every frame anyways. I have the feeling that collecting events and running update only once per frame is both easier for devs an also more performant. What do you think @whitetigle? As a reference, in the FableConf Game workshop there was a mixed model as user input (arrows) were dealt with one by one, while collisions (as  provided by the physics woker) were handled all at once every frame.
State
Besides different levels of State (global and local) I wonder if the state should be coupled with the renderer or not. Basically, whether Pixi sprites, etc, should go into the state or not. Having only simple types in the state can have advantages: it's easier to serialize and thus makes time travel debugging and exchanges with a worker also easier, separates rendering from logic. On the other hand, it can be more complicated to the dev, who will have to update the Pixi objects in the render function (unless we can create something like Virtual DOM for Pixi).

Thanks @alfonsogarciacaro 👏
State Machines: ok for Async. Could you just prepare some sample so that I can better see what it involves?
Events: I think you're right. So how could we apply that to a Drag and drop model with pixi Interaction Events like in this sample?
State: Like I said, I think the state should be decoupled from the renderer. It should be somehow agnostic.

@whitetigle I've updated the dragging sample. The result is much more verbose than the original and probably many things can be improved but I hope it helps you get a better idea. It's a very simple state machine with two states: dragging and waiting for dragging to start, each one with a different update function.
The logic (State.fs) is the decoupled from the view (Renderer.fs) module. The main function is an asynchronous loop which sends a dispatch function to the Renderer to collect the events and sends the collected events to the appropriate update function according to the state. Note for example, that we don't need to add a dragging flag to all the dragons because this info is in the state machine.

Hey that's very interesting!
I need to make my mind around it though so I will try to update another sample.
Thanks @alfonsogarciacaro 👍

Ok. I started a very secret ㊙ project involving your proposal. It's here
It's my little contribution to the advent calendar.
So there's no real events like your take on the drag and drop sample. There's just a render function that dispatch messages. But still it's clearly interesting to dispatch messages.
It's running at 60fps on my laptop and there are some particles 😉

I think @alfonsogarciacaro is creating a bank of image and gif since several months and is now using them :)
Other than that, great work to both or you on this Elmish + Pixi related stuff. Don't have much time to help on this subject ATM.

I think @alfonsogarciacaro is creating a bank of image and gif since several months and is now using them :)

😆😆😆

Other than that, great work to both or you on this Elmish + Pixi related stuff. Don't have much time to help on this subject ATM.

No problem at all. I also have zounds of things to do. Anyway thanks @MangelMaxime . In 2018 we'll Hink the world 😉 🍷  at Strasbourg 😁


Matching image snapshot fails with cy.screenshot() failed because it requires a DOM element

Discussed in https://github.com/simonsmith/cypress-image-snapshot/discussions/40

Originally posted by Fastova November 24, 2023
Hi guys, thank you for your great tool!
I'm facing a problem currently, which I can't overgo. Could you please advise, if I'm missing smth?
I"ve installed a plugin,
configured it and added a command to cypress.config
import { addMatchImageSnapshotPlugin } from '@simonsmith/cypress-image-snapshot/plugin';

addMatchImageSnapshotPlugin(on);

Added types for the plugin in ts.config:
@simonsmith/cypress-image-snapshot/types
In my test the command is available, it runs fine:
cy.matchImageSnapshot('snapshot', {
        failureThreshold: 0.4,
        blur: 10,
        capture: 'viewport',
        diffFilenameExtension: '.wrong',
      });

However it fails with no DOM element:
get #el-id
wrap<div#el-id>
taskMatching image snapshot, Object{12}
screenshotstatistic-card, Object{12}

CypressError
cy.screenshot() failed because it requires a DOM element.

The subject received was:
> null

The previous command that ran was:
> cy.task()

All 3 subject validations failed on this subject.

Am I doing smth wrong?
Regular cy.screenshot() works and saves the screenshot.

@Fastova I'm afraid I can't reproduce this. I've added tests to this repo to validate that it can work with and without a subject and it seems fine:
https://github.com/simonsmith/cypress-image-snapshot/blob/master/cypress/e2e/matchImageSnapshot.cy.ts#L6-L14
Perhaps you can provide a valid reproduction and then I can reopen the issue


Fix Crystal build on Linux: PWD => pwd

Crystal build uses PWD instead of pwd, which fails on case-sensitive operating systems and filesystems, e.g. Linux

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

thanks!


Can play with EmbyTheater？

Description
Can play with EmbyTheater？
Desired Behavior
Can play with EmbyTheater？
Additional Context
No response
Code of Conduct

[X] I agree to follow Overseerr's Code of Conduct

Unfortunately not possible


Add Genesis G80 to special panda safety mode

Description Upon activating OP in 2018 Genesis G80, screen immediately shows "Take Control Immediately, Controls Mismatch" and disengages. This PR is to add the G80 to special panda safety mode to resolve this issue.
Verification I was able to verify by driving my G80 on my fork and engage OP without encountering the Controls Mismatch screen. I am unable to provide the exact route because I am unable to connect my device with Comma Connect (scanning QR code doesn't do anything), so I don't have access to useradmin.
Route
Route: 6b301bf83f10aa90|2020-11-22--16-45-07

Can you provide a route from this branch that we can use for our tests? Currently, there is no test route for the Genesis G80.

Sure -- is there a way to get the route directly from the device without using https://my.comma.ai?

No, the route will need to be fully uploaded. You can check the upload status in your my.comma.ai/useradmin page.

Yeah so I wasn't able to connect my device to my account (scanning the QR code in the app doesn't do anything) so my useradmin page doesn't have any devices, not sure if there's another way to connect them?

Was able to get account linked. Here's the route ID 6b301bf83f10aa90|2020-11-22--16-45-07


Fix Relative Source with {x:Type} bindings

What is the current behavior?
This works:
<Button Content="Action!" VerticalAlignment="Bottom" Width="120"
    Command="{Binding ViewModel.Action, RelativeSource={RelativeSource AncestorType=views:MainView}}" />

What is the updated/expected behavior with this PR?
This also works now:
<Button Content="Action!" VerticalAlignment="Bottom" Width="120"
    Command="{Binding ViewModel.Action, RelativeSource={RelativeSource AncestorType={x:Type views:MainView}}}" />

Fixed issues
Fixes #10486

What is the diff between "this works" and "this also now works"?

What is the diff between "this works" and "this also now works"?

{x:Type} markup extension wasn't working in this context.

Ok now I see thanks 🙏

You can test this PR using the following package version. 11.0.999-cibuild0030842-beta. (feed url: https://pkgs.dev.azure.com/AvaloniaUI/AvaloniaUI/_packaging/avalonia-all/nuget/v3/index.json) [PRBUILDID]


Fix a small spelling mistake in docs.

Signed-off-by: remzi<EMAIL_ADDRESS>Which issue does this PR close?
None.
Rationale for this change
Just to fix a small spelling mistake in docs.
Are there any user-facing changes?
The document will be changed.
https://docs.rs/arrow/9.1.0/arrow/compute/kernels/window/fn.shift.html

Codecov Report

Merging #1406 (05f4e4a) into master (e40bc91) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1406   +/-   ##
=======================================
  Coverage   83.13%   83.13%
=======================================
  Files         182      182
  Lines       53327    53327
=======================================
  Hits        44334    44334
  Misses       8993     8993

Impacted Files
Coverage Δ

arrow/src/compute/kernels/window.rs
100.00% <ø> (ø)

parquet_derive/src/parquet_field.rs
65.98% <0.00%> (-0.23%)
:arrow_down:

parquet/src/encodings/encoding.rs
93.71% <0.00%> (+0.19%)
:arrow_up:

Continue to review full report at Codecov.

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


Enable auto-imports by default

I think it is a reasonable feature to enable now

Going block this until the performance problems in #749 are resolved


[chore] Add multiple source docs

Implementation details

Do 1
Do 2

Please ensure the following requirements are met before submitting a pull request:

[ ] The pull request is targeted against the correct target branch
[ ] The pull request is linked to an issue with appropriate discussion and an accepted design OR is linked to a spec that describes the work.
[ ] The pull request includes a description of the implementation/work done in detail.
[ ] The pull request includes any appropriate unit/integration tests
[ ] You have added a relevant changelog entry to CHANGELOG.md
[ ] You have re-reviewed the files affected by the pull request (e.g. using the Files changed tab in the GitHub PR explorer)

@colmazia  please fix test-stable as well.

please follow the same format as bothan-api. https://github.com/rust-lang/rfcs/blob/master/text/1574-more-api-documentation-conventions.md#appendix-a-full-conventions-text

Could you give me an example of what went wrong here, and how to correct it?

bc I try to do the same as the bothan-binance


feat: make kosmosctl support --context  and support e2e test for kosmosctl

What type of PR is this?
/kind feature
What does this PR do?

Which issue(s) does this PR fix?

Fixes #326
Special notes for your reviewer:

Does this PR introduce a user-facing change?

/cc @ONE7live @wuyingjun-lucky


IGNITE-5814 Remove non Ignite classes from marshaller exclusions

The ticket is resolved, closing PR. Feel free to reopen if it is needed.


Internet Archive Integration

[ ] Integrate bookmarks with Internet Archive. Once the bookmark is added, an Internet Archive Link is added.
[ ] Availability API: https://archive.org/help/wayback_api.php
[ ] Saving Page API: https://docs.google.com/document/d/1Nsv52MvSjbLb2PCpHlat0gkzw0EvtSgpKHu4mk0MnrA/edit

I now use a Linkding (https://github.com/sissbruecker/linkding) instance so this is no longer relevant.


Do not override kubelet hostname if cloud_provider is used

Starting with Kubernetes v1.8.4, kubelet ignores the AWS cloud
provider string and uses the override hostname, which fails
Node admission checks.
Fixes #2094

ci check this


Health Probe Source IP

In Azure Load Balancer, it is documented what the source IP Address is: https://docs.microsoft.com/en-us/azure/load-balancer/load-balancer-custom-probe-overview#probesource.
For Application Gateway, I don't see anything similar as to what Source IP for probing is.  Any chance this can be documented?

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

ID: 05a592c5-5d16-e44d-d6c7-5d0b2c0833c4
Version Independent ID: dd9d601c-5d4b-f1bc-01df-7e1d6b57d0da
Content: Health monitoring overview for Azure Application Gateway
Content Source: articles/application-gateway/application-gateway-probe-overview.md
Service: application-gateway
GitHub Login: @vhorne
Microsoft Alias: victorh

@ElanShudnow Thanks for the feedback! I have assigned the issue to the content author to evaluate and update as appropriate.


<EMAIL_ADDRESS>fails with an Error: getaddrinfo ENOTFOUND

Is this a bug report?
Yes

Did you try recovering your dependencies?

Not relevant.
Which terms did you search for in User Guide?

(Write your answer here if relevant.)
Environment

node -v: v9.7.1
npm -v: v5.6.0
yarn --version (if you use Yarn):
npm ls react-scripts (if you haven’t ejected):

Then, specify:

Operating system:
MacOSX Sierra 10.12.6
MacOSX Sierra 10.13.3
Browser and version (if relevant):

Steps to Reproduce

(Write your steps here:)

npm run start # fresh installation

Expected Behavior

If it's an error caused by the system configuration, it probably should suggest something meaningful.
Actual Behavior

This issue has been reported on a bunch of different repositories:
https://github.com/halfzebra/create-elm-app/issues/239
https://github.com/node-modules/detect-port/issues/29
The original maintainer @fengmk2 says it has been fixed in the original detect-port package.
Reproducible Demo

Not available.

A temporary solution:
Set the HOST env variable. e.g. HOST=localhost
And, optionally, also set the PORT=5000 or whatever port you want to use.

I couldn't get the above solution to work, so I just manually edited node_modules/detect-port-alt/lib/detect-port.js and removed the checks around return callback as shown below.
listen(port, host, (err, realPort) => {
// ignore random listening
return callback(err, realPort);

And what's the difference between detect-port-alt and detect-port? Why not send a PR to dectect-port? If you think it is necessary to fork detect-port so please let us know the repo address then I can make a PR for you.


Need to change http to https or disable it

At the moment, we are getting error because of http request instead of https when requesting http://schemas.datapackages.org/registry.json:
Mixed Content: The page was loaded over HTTPS, but requested an insecure resource 'http://schemas.datapackages.org/registry.json'. This request has been blocked; the content must be served over HTTPS.

At the moment, datapackage-test library uses data from following source: http://schemas.datapackages.org/registry.json
And in this source, there are 3 schema paths, which have http URLs.
Also, when we try manually change http to https, there is certificate problems on https://schemas.datapackages.org/registry.json

Thanks!
@dumyan
Registry https: https://schemas.frictionlessdata.io/registry.csv

@roll should just be able to do references as //schemas.frictionlessdata.io/registry.csv from now on.

Will be resolved by https://github.com/frictionlessdata/schemas/pull/67 (I will update builtin datapackage schemas)

FIXED in 0397fb83b0841fb2b85b51ce6f3f78b32bffa368


Add configuration option for customizing error condition

Currently, this project uses a hard-coded failure condition based on the response' HTTP STATUS code as follows:
if (statusCode < 200 || statusCode >= 300) {
  return cb(new Error('Fetching ' + JSON.stringify(options) + ' failed with HTTP status code ' + statusCode));
}

This causes the grunt task runner to stop executing tasks post-error  if not run with the -f or --force flag, and prevents the current executing method from completing its implementation. This is problematic when doing something like the following:
curl:
  geodatasource_cities_auth:
    src:
      url: 'http://www.geodatasource.com/login'
      method: 'POST'
      formData:
        password: 'password'
        emailAddress<EMAIL_ADDRESS>      jar: true
    dest: '/dev/null'
  geodatasource_cities_free:
    src:
      url: 'http://www.geodatasource.com/download?id=8'
      jar: true
    dest: 'seed/geodatasource_cities_free.zip'

Where the geodatasource_cities_auth task returns an HTTP STATUS 302; a success condition in this scenario.
Please add some kind of configuration option to ignore warnings/errors or specify permissible status codes outside of the 200 block.

Can you explain your use case further? I don't fully grasp why we want to support a 302 request -- this task is usually intended to download content.

Sure. In this particular use case, I'm making two requests. The first is an authentication request, POSTing my credentials to the server. I'm using the plugin's ability to take request options to specify jar: true to cache the resulting authentication cookies. Subsequently, in the second request, I also specify jar: true to use the cookies set in the previous request, which then permits me to access the resource I'm attempting to download.

I think that usage is bending the purpose of this tool a bit. With the requirement of sharing global state, it feels like that should be handle by a custom script or a library better fit for making successive requests. Going to decline this request for now =/


Version on npm outdated

According to the npm page for react-native-drawer, it is at version 2.3.0. August 25th, 2016 was the last time that the version was incremented. There have been several commits since then. In particular, the callback functionality on open() and close() is what I need. Could you please re-publish? Thanks in advance.

Has this project been abandoned? Need help maintaining it?

Ping on this. Would be amazing if the community got access and could help push new versions


[Synapse] Update Azure.Analytics.Synapse.Artifacts to eea7c0141c0b7ad…

…9f66f8ba06560b549c6b3b014

Should I bump all synapse libs in lockstep, even if it only changed for one?

Teams discussion suggests I should bump these all in sync. Adding that.


npm install fails on Xenial

Hey @Zarkopafilis, I suspect that my node is probably too old (I really should replace my worn out Mint 18 with Kubuntu Bionic) (also, it's my first time doing anything with Node at all). At first, npm install failed when installing chromedriver with a complaint that the executable node is unavailable, which I fixed by installing the APT package nodejs-legacy ("legacy" in the name seems like a red flag). Then this happened:
pavel@pavel-workstation $ npm install chromedriver

><EMAIL_ADDRESS>install frontend/node_modules/chromedriver
> node install.js

frontend/node_modules/chromedriver/lib/chromedriver.js:5
  let port = 9515;
  ^^^

SyntaxError: Block-scoped declarations (let, const, function, class) not yet supported outside strict mode
    at exports.runInThisContext (vm.js:53:16)
    at Module._compile (module.js:374:25)
    at Object.Module._extensions..js (module.js:417:10)
    at Module.load (module.js:344:32)
    at Function.Module._load (module.js:301:12)
    at Module.require (module.js:354:17)
    at require (internal/module.js:12:17)
    at Object.<anonymous> (frontend/node_modules/chromedriver/install.js:5:16)
    at Module._compile (module.js:410:26)
    at Object.Module._extensions..js (module.js:417:10)
npm WARN optional Skipping failed optional dependency /chokidar/fsevents:
npm WARN notsup Not compatible with your operating system or architecture<EMAIL_ADDRESS>npm WARN optional Skipping failed optional dependency /sane/fsevents:
npm WARN notsup Not compatible with your operating system or architecture<EMAIL_ADDRESS>npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
npm ERR! Linux 4.4.0-21-generic
npm ERR! argv "/usr/bin/nodejs" "/usr/bin/npm" "install" "chromedriver"
npm ERR! node v4.2.6
npm ERR! npm  v3.5.2
npm ERR! code ELIFECYCLE

npm ERR<EMAIL_ADDRESS>install: `node install.js`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>install script 'node install.js'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the chromedriver package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     node install.js
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs chromedriver
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls chromedriver
npm ERR! There is likely additional logging output above.

npm ERR! Please include the following file with any support request:
npm ERR!     frontend/npm-debug.log

I can PM my npm-debug.log if neccessary.
(I know that there are a pull requests and forum threads waiting for my attention, I will get to those first thing tomorrow; the last few days were spent between stupid meetings and finishing up the PyDSDL parser)

The particular chromedriver dependency is only used for the e2e tests.
It works on my mac. Just to be sure, can you generate a new empty hello project with
the vue cli? Include unit tests with jest and e2e tests with nightwatch. If this fails too, I'll set up a VM and work this out.

Okay, I see what the problem is now:
You are using Node v4.2.6, but this version of vue-cli requires Node >=8.9.
Please upgrade your Node version.

I got rid of the node which I installed through APT and installed an official snap package v11.x, everything seems to be working well now. I think we should specify in the README that the minimum required node version is v11.


saveas feature for cordova app

This change is

Reviewed 8 of 8 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


TS flavored generated code should support TS tests

Summary
converting existing jest file to ts causes test failures
Steps To Reproduce:

get tests passing as js on an existing repo
convert app's js files to ts
yarn test:unit

Expected result
selecting TS for project yields

test in ts
properly configured jest.config for ts testing

Actual result
Jest encountered an unexpected token
This usually means that you are trying to import a file which Jest cannot parse, e.g. it's not plain JavaScript.

By default, if Jest sees a Babel config, it will use that to transform your files, ignoring "node_modules".

Here's what you can do:
 • To have some of your "node_modules" files transformed, you can specify a custom "transformIgnorePatterns" in your config.
 • If you need a custom transformation specify a "transform" option in your config.
 • If you simply want to mock your non-JS modules (e.g. binary assets) you can stub them out with the "moduleNameMapper" config option.

You'll find more details and examples of these config options in the docs:
https://jestjs.io/docs/en/configuration.html

First #92 needs to be fixed to make that happen.
And then there's the test file matching. This needs to be added to the Jest config.
testMatch: ['**/__tests__/**/?(*.)(spec|test).ts']
Either I add it "manually" to a projects jest.config.js file. Or we add it as default to @lwc/jest-preset in the PR for #92.
Thoughts @trevor-bliss?

I think it'd be fine to update the testMatch in the preset to match against test files that end in js or ts files.


text and icon issue

Can i insert text and any type of icon into it

can you describe your demand more clear ? or show me an example ?

i have a project where i'm using this plugin. my front end view like > a input filed with border radius and the text into it > Form date + icon To date + icon. where user click this input filed then popup your calendar. the problem is i can not insert any text like (Form date + icon To date + icon).

If we provide a input slot for you, may it ok?
https://github.com/ly525/material-vue-daterange-picker/blob/master/src/components/Picker.vue#L4

if you have any question，please let me know


Fixes to support MySQL 8 due to specific reserved keywords

Description
Describe the changes made and why they were made.
Ignore if these details are present on the associated Apache Fineract JIRA ticket.
Checklist
Please make sure these boxes are checked before submitting your pull request - thanks!

[x] Write the commit message as per https://github.com/apache/fineract/#pull-requests

[x] Acknowledge that we will not review PRs that are not passing the build ("green") - it is your responsibility to get a proposed PR to pass the build, not primarily the project's maintainers.

[x] Create/update unit or integration tests for verifying the changes made.

[x] Follow coding conventions at https://cwiki.apache.org/confluence/display/FINERACT/Coding+Conventions.

[x] Add required Swagger annotation and update API documentation at fineract-provider/src/main/resources/static/legacy-docs/apiLive.htm with details of any API changes

[x] Submission is not a "code dump".  (Large changes can be made "in repository" via a branch.  Ask on the developer mailing list for guidance, if required.)

FYI our guidelines for code reviews are at https://cwiki.apache.org/confluence/display/FINERACT/Code+Review+Guide.

@galovics I think the easiest approach would be to merge first #2313, because that solves very likely those 2 build errors.


TestWatchEventType: WaitGroup is reused before previous Wait has returned

via semaphore (https://semaphoreci.com/coreos/etcd/branches/master/builds/995)
panic: sync: WaitGroup is reused before previous Wait has returned

goroutine 57085 [running]:
panic(0xf5aa20, 0xc420937640)
	/usr/local/golang/1.7/go/src/runtime/panic.go:500 +0x1ae
sync.(*WaitGroup).Wait(0xc4281f9ae8)
	/usr/local/golang/1.7/go/src/sync/waitgroup.go:133 +0x13e
github.com/coreos/etcd/etcdserver.(*EtcdServer).run.func1(0xc4281f98c0, 0x1568880, 0xc42214e8a0)
	/home/runner/workspace/src/github.com/coreos/etcd/gopath/src/github.com/coreos/etcd/etcdserver/server.go:580 +0x82
github.com/coreos/etcd/etcdserver.(*EtcdServer).run(0xc4281f98c0)
	/home/runner/workspace/src/github.com/coreos/etcd/gopath/src/github.com/coreos/etcd/etcdserver/server.go:645 +0xad7
created by github.com/coreos/etcd/etcdserver.(*EtcdServer).start
	/home/runner/workspace/src/github.com/coreos/etcd/gopath/src/github.com/coreos/etcd/etcdserver/server.go:503 +0x5aa

@heyitsanthony @xiang90 Go doc says
	// This goroutine has set counter to 0 when waiters > 0.
	// Now there can't be concurrent mutations of state:
	// - Adds must not happen concurrently with Wait,
	// - Wait does not increment waiters if it sees counter == 0.

We are calling Wait in defer statement, while concurrent Add call (from goAttach) happens.


Ford: fix gear parsing

CANDefine converts all of the signal defined values to uppercase/snake case, so our map from signal to gears should use uppercase too.
Also added Manumatic as an extra drive gear for Ford.

Just rebased due to conflicts with master


KenLM with unicode hack (decreased LM size)

👋 Hello! Maybe it would be useful for the community. It's about how to decrease your model size in several times.
🔨 I've worked with different ctc decoders and used one from the NeMo repo. Specially for it I've trained the KenLM with this scripts. NeMo ctc_decoder don't support the BPE tokenization so they developed a trick.

Consider the token ids as as a unicode characters ([char(i) + TOKEN_OFFSET for i in range(tokenizer.vocab_size)) to imitate the simple alphabet.
Train KenLM at the character level with this characters.
After rescoring shift symbols from beam back ([ord(c) - TOKEN_OFFSET for c in transcription]).
Decode ids with tokenizer.

🎯 My the 6-gram word level model has 1.2 Gb and unicoded 6-gram token-based model has 300Mb and performs better both on speed and quality.
I've managed to get it working with pyctcdecoder. It's another hack here -- I'm passing alphabet with "_" symbol before all the unicode chars so the decoder treat them as a separate words and do the proper rescoring.
🔭 On the other hand, the support of hotwords has been broken with trick. Maybe you'll  have some thoughts on this, because this trick is pretty useful.

Hey, thanks. we are not officially supporting this re-mapping, since it effectively turns a word level LM into a wordpiece LM, which means less context (which is where the size reduction mostly comes from). Feel free to have a look into using pyctc with the remapping, but unless there is an overwhelmingly good reason to support it, we probably shouldn't write code for that in the main repo.

Ok, I've got it.


if we had a server with big diskspace, function build_vol_hash_table cost to much to build the vol_hash_table for CacheVol

if the Volume  space is 50TB, the rtable_size will be 50 1000 * 1000 / 8=6250000,it cost too much memory and cpu usage( see the code in build_vol_hash_table listed below), maybe we need to optimize this build_vol_hash_table function.*
rtable_pair *rtable = (rtable_pair *)ats_malloc(sizeof(rtable_pair) * rtable_size);
 int rindex = 0;
 for (int i = 0; i < num_vols; i++)
   for (int j = 0; j < (int)rtable_entries[i]; j++) {
     rtable[rindex].rval = next_rand(&rnd[i]);
     rtable[rindex].idx = i;
     rindex++;
   }

The size of the cache index should be vol space / average object size * 10 bytes.
What is your average object size set to (proxy.config.cache.min_average_object_size)?

If this is still an issue please reopen.


How to get OpenShift version out of the ClusterDeployment?

In UHC, we're looking at the ClusterDeployment's status.clusterVersionStatus.current.version, hoping that this would give us the version of OpenShift deployed on the cluster, but the values we're seeing are something like 0.0.1-2018-12-08-172651.
I assume that this is the version of the Cluster Version Operator, and not OpenShift itself (as this is OpenShift 4, not 0.0.1).
Is there a canonical way to convert this string into an OpenShift version? Is this string expected to always look like this, or to turn into the user facing 4.0.x style number when OpenShift 4 is GA?
Thanks.

cc: @jhernand

This comes from the ClusterVersion object in-cluster, CC @smarterclayton, will this eventually be a more user friendly 4.0.x value? (as opposed to what it is today, 0.0.1-...)

My cluster this morning is reporting 4.0.0-0.alpha-2019-01-03-031244. Good enough @elad661 ?

Sure, this sounds great, thanks!


bug: error with the latest dev version AttributeError: 'list' object has no attribute 'dtype'

Got error with the latest dev version:
fig, ax = pplt.subplot()
ax.plot(np.arange(10))

Traceback
---------------------------------------------------------------------------
AttributeError                            Traceback (most recent call last)
/tmp/ipykernel_109695/309448141.py in <module>
      1 fig, ax = pplt.subplot()
----> 2 ax.plot(np.arange(10))

~/local/conda/envs/nemo/lib/python3.8/site-packages/proplot/internals/inputs.py in _preprocess_or_redirect(self, *args, **kwargs)
    292
    293                 # Call main function
--> 294                 return func(self, *args, **kwargs)  # call unbound method
    295
    296         return _preprocess_or_redirect

~/local/conda/envs/nemo/lib/python3.8/site-packages/proplot/axes/plot.py in plot(self, *args, **kwargs)
   2899         """
   2900         kwargs = _parse_vert(default_vert=True, **kwargs)
-> 2901         return self._apply_plot(*args, **kwargs)
   2902
   2903     @inputs._preprocess_or_redirect('y', 'x', allow_extra=True)

~/local/conda/envs/nemo/lib/python3.8/site-packages/proplot/axes/plot.py in _apply_plot(self, vert, *pairs, **kwargs)
   2858             ys, kw = inputs._dist_reduce(ys, **kw)
   2859             guide_kw = _pop_params(kw, self._update_guide)  # after standardize
-> 2860             for _, n, x, y, kw in self._iter_arg_cols(xs, ys, **kw):
   2861                 kw = self._parse_cycle(n, **kw)
   2862                 *eb, kw = self._add_error_bars(x, y, vert=vert, default_barstds=True, **kw)  # noqa: E501

~/local/conda/envs/nemo/lib/python3.8/site-packages/proplot/axes/plot.py in _iter_arg_cols(self, label, labels, values, *args, **kwargs)
   4216             labels = [
   4217                 str(_not_none(label, ''))
-> 4218                 for label in inputs._to_numpy_array(labels)
   4219             ]
   4220         else:

~/local/conda/envs/nemo/lib/python3.8/site-packages/proplot/internals/inputs.py in _to_numpy_array(data, strip_units)
    131     elif isinstance(data, (DataFrame, Series, Index)):
    132         data = data.values
--> 133     if data.dtype == bool:
    134         data = data.view(np.uint8)
    135     if Quantity is not ndarray and isinstance(data, Quantity):

AttributeError: 'list' object has no attribute 'dtype'

debug
> ~/local/conda/envs/nemo/lib/python3.8/site-packages/proplot/internals/inputs.py(133)_to_numpy_array()
    131     elif isinstance(data, (DataFrame, Series, Index)):
    132         data = data.values
--> 133     if data.dtype == bool:
    134         data = data.view(np.uint8)
    135     if Quantity is not ndarray and isinstance(data, Quantity):

ipdb> p data
[None]

The error seems related to this PR https://github.com/proplot-dev/proplot/pull/410.

It works if I removed the two lines introduced in PR https://github.com/proplot-dev/proplot/pull/410
Not sure if it is related to my configuration. Let's see if where are others who met this problem as well.

Yes you're right, sorry about that. It is from that PR since I didn't test a wide variety of inputs in my quick fix. I'll try and add something to avoid the bad dtype check, but I'm open to people's ideas about robustly checking for it

Thanks for the report -- not your fault @scottstanie. That was a very sloppy approval on my part. Should have been more careful in the review, and travis CI should have caught it when you created the PR anyway.  Seems proplot is out of credits again, and the CI webhook is not functioning (red triangle in screenshot); have just contacted Travis to replenish credits.

Eventually I need to switch to github actions, as travis has unfortunately become very unreliable / hostile to open source projects. I am also planning to improve the CI testing -- currently I simply have travis CI compile the documentation examples, but I have a full testing suite based on pytest-mpl uncommitted on my machine that I am planning to add to the CI workflow this spring. PRs / community contributions should then be much simpler / safer.


Item Reorder Gesture problem in RTL

Ionic version:
[ ] 1.x (For Ionic 1.x issues, please use https://github.com/ionic-team/ionic-v1)
[x] 2.x
[ ] 3.x
I'm submitting a ...
[x] bug report
[ ] feature request
Current behavior:
When you try to do drag and drop to reorder a list in RTL the list cannot reorder.
Expected behavior:
You can to drag and drop on list.
Steps to reproduce:
Try to reorder a list in RTL.
https://ionicframework.com/docs/api/components/item/ItemReorder/
Ionic info:
Cordova CLI: 6.5.0
Ionic Framework Version: 2.0.1
Ionic CLI Version: 2.2.1
Ionic App Lib Version: 2.2.0
Ionic App Scripts Version: 1.0.0
ios-deploy version: 1.9.1
ios-sim version: Not installed
OS: macOS Sierra
Node Version: v6.9.4
Xcode version: Xcode 8.3.3 Build version 8E3004b

Thanks for opening an issue with us. @AmitMY would you mind taking a look at this?

Sure.
The thing is, you are using v2.x. We fixed RTL in version 3.4, so I recommend you upgrade to latest (it is really simple, and there are guides online like this one - I recommend first upgrading to 3.0.0, and then to 3.6.0 (will be easier)

Note that it is really recommended to have a version >=3.4 if you are using RTL, as there are plenty of stuff that won't work, and you can't fix just from scss.

Another thing, is that you can't control RTL from the browser. meaning, dir="rtl" is plenty, but is not enough. You also need to do this.platform.setDir('rtl')
Full explanation on RTL is here


Suggestion: Open interpreter integration

Is this even possible with containers?

Yes,
I was thinking about integrating it just today. It should be possible with a caveat of a subfolder-only file access down from the $PWD, I need to learn more about it's internals and specifically how the context is constructed, though

Just pushed initial version of the integration, writing some docs now. The (yet) undocumented CLI interface is here

released in v0.0.17

Closing as the integration was made


[QUESTION] 官方的限流组件hyperf/rate-limit似乎必须用在微服务的环境中，如果只是普通的api接口项目，能否直接使用此组件

Before you submit an issue, please be sure to search through existing issues as well as search through the documentation

[] I've searched all existing issues
[] I've read all relevant documentation I could find

Describe your question

可以用。


HN Algolia as source

Support the HN search https://hn.algolia.com/ as a source; to monitor popular posts with certain keywords (that often tend to strategically get buried away from the front page).

Could you demonstrate how to make Elixir and Elm works with Firebase / Parse ?
Consider HN has its api powered by Firebase:
https://www.firebase.com/blog/2014-10-07-hacker-news-api-is-firebase.html

@thelittlesaul I'm not entirely sure what you are asking, but SlowNews is already using the Firebase API for reading top HN posts.


Enabling the elasticsearch feature makes Zebra panic

Starting Zebra with the elasticsearch feature enabled, for example, by
cargo run --release --features elasticsearch

causes the following panic:
The application panicked (crashed).
Message:  ES Request should never fail: Error { kind: Http(reqwest::Error { kind: Request, url: Url { scheme: "https", cannot_be_a_base: false, username: "", password: None, host: Some(Domain("localhost")), port: Some(9200), path: "/zcash_mainnet/_bulk", query: None, fragment: None }, source: hyper::Error(Connect, ConnectError("tcp connect error", Os { code: 111, kind: ConnectionRefused, message: "Connection refused" })) }) }
Location: zebra-state/src/service/finalized_state.rs:507

Metadata:
version: 1.6.0
Zcash network: Mainnet
running state version: 25.3.0
initial disk state version: 25.3.0
features: default,elasticsearch,getblocktemplate_rpcs,howudoin,indicatif,progress_bar,release_max_level_info
branch: main
git commit: 58bfe97
commit timestamp: 2024-02-23T23:28:44.000000000Z
target triple: x86_64-unknown-linux-gnu
rust compiler: 1.76.0
rust release date: 2024-02-04
optimization level: 3
debug checks: false

...
[1]    8829 IOT instruction (core dumped)  cargo r --features elasticsearch --release

As Arya pointed out, the panic is likely due to an unavailable Kibana server. However, this causes further problems because, for example, running Zebra with --all-features implicitly panics as well.

I think this is not that bad. If you are running Zebra with the elasticsearch feature enabled and there is no elasticsearch database running the node should stop. I think we might want to replace the panic with an error and proper shutdown.

Another option can be something like:

check for the database server if the feature is enabled, log an error and keep a flag if needed.
if the feature is enabled but the flag is false ignore elasticsearch code.

I guess we deal with something similar when we enable metrics with the prometheus feature, but Prometheus is not available. Zebra can handle that without panicking.

Zebra can handle that without panicking.

Or stopping. I just checked that, and it just continues running.


Custom action Swift example

It'd be nice to have a simple example of how to do a custom action using Swift.

Fixed by https://github.com/google/EarlGrey/issues/145


Emotes in messages.

What do you mean by that when you say that hanss?

@Bottersnike, hanss hasn't commented on this and I'm concerned for your health

A.) I worked out what he meant by checking in #suggestions.
B.) Why?

you're seemingly talking to nobody

No one's active at the moment, and I don't have my laptop anymore, so I can't do any dev in the evenings :(

Oh, it was just a suggestion by Milo to put some more emotes

Yeah. I've thrown in some, but I'll leave this open because there are plenty that can still be added.

people can suggest these in new issues when they find ideas


Landing page and Menu

@ificiana @gods-gift I've merged in the PR #14.
A couple notes:

Merge the myver into __main__.py so the menu shows up when you start the game.
Fix some of the bugs on the different clicks

A question, do we need to move the inventory and game into views as well?


Implement JpqlSerializer for JpqlExists

Backgrounds

In Kotlin JDSL 3.0.0, the Query objects no longer know how to draw themselves as SQL String.
The Query object is drawn in JPQL String by JpqlSerializer.

Requirements

Please implement  JpqlSerializer that draws JpqlExists as a SQL String.
Please implement a unit test for the implemented class.

Contribution

Please check if there is an assignee in this issue. And if it don't have an assignee, please change the assignee to yourself.
Please check the commit convention through the commitlint.config.js file. The commit  convention also described in the PR template.
When you use intellij, please set the code style through the .editorconfig file.
Please create the PR for develop/3.0.0 as the base branch.
If you have any questions, please leave comment in Kotlin-JDSL Discord Channel in LINE Open Source

I'd like to work on this issue.

This issue is closed by #423


Could Address models like BillingAddress ShippingAddress inherit from Address ?

In shopify_api/lib/shopify_api/resources/

I think I would be hesitant about doing this, since they aren't really modelled in this way in the actual API. Do you have a specific use case for this in mind?

Those models seems to be modelled as same as Address for example billing_address / shipping_address  in https://help.shopify.com/api/reference/order or addresses / default_address (I have seen a ShopifyAPI::Customer::DefaultAddress) in https://help.shopify.com/api/reference/customer and also https://help.shopify.com/api/reference/customeraddress got the same fields
It would be easier then to implement a generic processing of whatever address object we have on our side.


[api-minor] Let Catalog.getAllPageDicts return an empty  dictionary when loading the first /Page fails (issue 15590)

Please refer to the individual commit messages for additional details.
Note that while the first patch feels fairly uncontroversial, the second one I'm a little less "happy" about myself since it'll lead to us displaying nothing but an empty page in some corrupt documents (that we previously just failed to open).

/botio-linux preview

/botio test

/botio test


Added ability for getting position by mentioning

Fixes #9. Needs to be updated when the bot is switched over to ids.

Forgot proof again:


Fix TypeError from trying to call a boolean

user.is_authenticated is a boolean value in Django 2.0, this causes a TypeError when trying to load the page with adminlte2 templates.
Example end of stack trace for loading the admin site:
File "/usr/lib/python3.6/site-packages/django_adminlte/templatetags/adminlte_helpers.py" in avatar_url
  20.         hash=md5(user.email.encode('utf-8')).hexdigest() if user.is_authenticated() else '',

Exception Type: TypeError at /admin/
Exception Value: 'bool' object is not callable

Thanks for the PR @Diftraku, it is much appreciated. In the end I added a compat.py file with is_authenticated() which detects the appropriate course of action.

I've just released 0.2.2 containing this fix


C++ course by hitesh choudhary in hindi

What does this PR do?
Add C++ course in Hindi language
For resources
Description
Why is this valuable (or not)?
How do we know it's really free?
available for free on YouTube
For book lists, is it a book? For course lists, is it a course? etc.
it is a course
Checklist:

[x] Read our contributing guidelines.
[x] Search for duplicates.
[x] Include author(s) and platform where appropriate.
[x] Put lists in alphabetical order, correct spacing.
[x] Add needed indications (PDF, access notes, under construction).
[x] Used an informative name for this pull request.

Follow-up

Check the status of GitHub Actions and resolve any reported warnings!

Please accept it as i have commit your suggestion


Job not executing until app reopen

It is happening to me on some devices, that once I schedule the job for a specific time and go to the mobile home screen without closing the app, the job does not run on the established date and when entering the app it starts to run . on other devices that do not happen. Any ideas on why this behavior?

Yes, see https://developer.android.com/training/monitoring-device-state/doze-standby

Also https://dontkillmyapp.com


A nice way to obtain predictions?

Hello! First of all, thank you for open sourcing StarSpace, it is a great project.
I was wondering, whether there exists an elegant way of obtaining row-level predictions during testing.  Now, when new instances arrive, I would like to compute, apart from HITS score, also e.g., F1 or AUC (for binary classification).
 This outputs HITS scores, yet I am wondering whether it is possible to obtain e.g., top k labels for a given instance.
I also found the following tool:
make query_predict ./query_predict <model> k [basedocs]
which does exactly what I want, apart from the fact that it requires user input (instead of e.g., file input). Am I simply missing something?
So, to summarize, given e.g., an instance comprised of
w_1 w_2 w_53 ...
I would like to obtain, using a trained model, [__label__something ... ] vector of predictions.
Thank you!

Currently solved with
./query_predict tmp/storedModel 1  < ./tmp/test_data.txt


Why doesn't {{#with}} set context when it's nested?

Description:
I remember (but I'm unsure) that we could use {{#with foo}} to set the context to foo. It still works the same way, but not with nested structures. Why is that?
Versions affected:
1.4
Platforms affected:
Browser
Reproduction:
Ractive({
  el: 'body',
  append: true,
  data: {
    foo: {
    }
  },
  template: `
    <!-- context becomes foo -->
    {{#with foo as f}}
      <input value="{{f.hello}}" />
      {{#with f.bar as b}}
      	<input value="{{b.yay}}" />
      {{/with}}
    {{/with}}
  `,
  onrender: function(){
    console.log(JSON.stringify(this.get()))
  }
})

This code prints the expected output: {"foo":{"bar":{"yay":""},"hello":""}}
However, this simplified version produces an undesired result:
Ractive({
  el: 'body',
  append: true,
  data: {
    foo: {
    }
  },
  template: `
    <!-- context becomes foo -->
    {{#with foo}}
      <input value="{{hello}}" />
      {{#with bar}}
      	<input value="{{yay}}" />
      {{/with}}
    {{/with}}
  `,
  onrender: function(){
    console.log(JSON.stringify(this.get()))
  }
})```

Outputs: `{"foo":{"hello":""}}`

Is this behavior expected?

This is correct according to the tests, even though I agree that it doesn't look right and I've fought with it before. The problem is overloading and historical implementation detail in that originally with was just a plain mustache that's an internal if-with under the covers. To avoid the issue, foo.bar has to be in place in order to render the {{#with bar}} section, which will cause the yay twoway binding to update the model with the empty string.

Do you think this desired behavior may be a feature request?

I've definitely got it on my list for Ractive 2.0.


Spelling Mistakes and Incorrect use of Grammar

Summary

Ticket Link

Hello @ArmanChand,
Thanks for your pull request! A Core Committer will review your pull request soon. For code contributions, you can learn more about the review process here.

@cwarnermm , kindly merge this PR


[flutterfire_cli]: CLI creates plugin references wrongly in Android

Is there an existing issue for this?

[X] I have searched the existing issues.

Which plugins are affected?
Other
Which platforms are affected?
Android
Description
I have executed:
flutterfire configure \
--project="my-project" \
--platforms="ios,android,web" \
--ios-build-config=Debug-production \
--ios-out=/ios/Runner/GoogleService-Info.plist \
--android-out=/android/app/google-services.json \
--ios-bundle-id="id.myproject" \
--android-package-name="id.myproject" \
--out=/lib/firebase/firebase_options_production.dart \
--overwrite-firebase-options \
--yes

And my android/app/build.gradle file was modified as:
plugins {
    id "com.android.application"
// START: FlutterFire Configuration
apply plugin: 'com.google.gms.google-services'
// END: FlutterFire Configuration
    // TODO: Check if this problem still applied to release builds:
    // There is an issue with sentry plugin version >=4.3.0 when making a release build.
    // Check issue at https://github.com/getsentry/sentry-react-native/issues/3644
    id "io.sentry.android.gradle" version "4.10.0"
}

Which generates this compiler error:

Only method calls to 'id' and 'alias' are available in 'plugins' block

I got this error because I tried to run an android build and it generated this error:

class org.codehaus.groovy.ast.expr.TupleExpression cannot be cast to class org.codehaus.groovy.ast.expr.ArgumentListExpression (org.codehaus.groovy.ast.expr.TupleExpression and org.codehaus.groovy.ast.expr.ArgumentListExpression are in unnamed module of loader org.gradle.internal.classloader.VisitableURLClassLoader @29774679)
class org.codehaus.groovy.ast.expr.TupleExpression cannot be cast to class org.codehaus.groovy.ast.expr.ArgumentListExpression (org.codehaus.groovy.ast.expr.TupleExpression and org.codehaus.groovy.ast.expr.ArgumentListExpression are in unnamed module of loader org.gradle.internal.classloader.VisitableURLClassLoader @29774679)
Gradle's dependency cache may be corrupt (this sometimes occurs after a network connection timeout.)
Re-download dependencies and sync project (requires network)
The state of a Gradle build process (daemon) may be corrupt. Stopping all Gradle daemons may solve this problem.
Stop Gradle build processes (requires restart)
Your project may be using a third-party plugin which is not compatible with the other plugins in the project or the version of Gradle requested by the project.
In the case of corrupt Gradle processes, you can also try closing the IDE and then killing all Java processes.

Additional context and comments
No response

Hello @feinstein, please report this issue in this repository: https://github.com/invertase/flutterfire_cli

Can you transfer?


feat(cis zones): CIS Zones pagination update

PR summary

Fixes: <! -- link to issue -->
PR Checklist
Please make sure that your PR fulfills the following requirements:

[ ] The commit message follows the Angular Commit Message Guidelines.
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

PR Type

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] New tests
[ ] Build/CI related changes
[ ] Documentation content changes
[ ] Other (please describe)

What is the current behavior?

What is the new behavior?

Does this PR introduce a breaking change?

[ ] Yes
[ ] No

Other information

@ajay-malhotra1 DirectLink test case is getting failed. Can you please check and let me know?
Thanks,

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


Public PR for ASE 2403 release notes.

If this PR needs approval, or if the commits need to be pulled into the private repo, contact @ManikaDhiman. Thanks.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

Closed the PR as it has been converted to private PR and merged.


Inconsistency in /eth/v1/node/peers

The /eth/v1/node/peers returns a meta element that contains a count item.  The schema for this item declares the value to be a number, which is inconsistent with the common format of providing numeric values as quoted strings.
A quick tour of the main beacon nodes shows the following for this value:

lighthouse: returns the value unquoted
nimbus: returns the value quoted
prysm: does not return the meta element
teku: returns the value unquoted

Options that I can think of:

patch the spec so that the v1 endpoint returns a quoted value
raise bugs with the relevant beacon nodes to ensure this value is returned unquoted
deprecate the v1 endpoint and create a v2 endpoint with a quoted value

retaining the meta element
discarding the meta element (nowhere else do we return a value like this that can easily be calculated from the other returned information)
moving the count value to the top level returned object where all other metadata lives

My inclination is for option 3 suboption 2, but I would be interested in hearing if anyone has a reason for this meta element to exist, and the count value within it.

i wouldn't be against a v2 without the count - its pretty trivial to get the count from an array...
I guess that translates to 3 suboption 2 being my preference.

The meta element might have been a good idea if used consistently across all APIs but as it is right now, any top-level property should be considered metadata (execution_optimistic, finalized, etc.) and everything else (the actual data) must be part of data element.
As this API is an outlier and does not follow this pattern, I would also prefer option 3 suboption 2.

+1 to burn the meta

No one likes the meta element, but there's no appetite either for the effort to remove it

See discussion in https://github.com/ethereum/beacon-APIs/pull/384#issuecomment-1850805471

According to @mcdee review of clients seems that support of the meta element is poor. If no-one has complained should mean that no consumer is dependent on it.
To address this one-off oddity we could do a breaking change on route v1 and mark the meta property as optional. Then only Nimbus needs to fix the route to be compliant, which @arnetheduck is already keen on doing.
Thoughts?


Spotify plugin not working

I'm receiving the following error for Spotify plugin, Spotify plugin is not working
PlayError: https://open.spotify.com/track/5nTtCOCds6I0PHMNtqelas
message is not a Discord.Message.

PlayError: https://open.spotify.com/artist/5a2w2tgpLwv26BYJf2qYwu
Cannot read property 'map' of undefined

PlayError: https://open.spotify.com/album/4tDBsfbHRJ9OdcMO9bmnai
Cannot read property 'send' of null

I'm using the example bot play command

The latest version of spotify plugin is 0.3.0

Ok, thank you

It's working fine after updating version,
Thank you so much 🙂


Android M issues. Due to new intent/permissions

I've been having a lot of issues getting the scanner working on Marshmallow, and it all seems to be due to the use on Intent rather than requesting Permissions.
The people working on the cordova-plugin-camera project hit the same issue. https://github.com/apache/cordova-plugin-camera/pull/142
Is this something that can be easily resolved?

:+1:
I'm getting "Sorry, the Android camera encountered a problem..." The logs say:
02-09 14:32:35.589   198   822 W ServiceManager: Permission failure: android.permission.CAMERA from uid=10178 pid=13777
02-09 14:32:35.589   198   822 E CameraService: Permission Denial: can't use the camera pid=13777, uid=10178
02-09 14:32:35.589 13777 13777 W CameraBase: An error occurred while connecting to camera: 0

02-09 14:32:35.590 13777 13777 W CaptureActivity: Unexpected error initializing camera
02-09 14:32:35.590 13777 13777 W CaptureActivity: java.lang.RuntimeException: Fail to connect to camera service
...

I believe this is the same bug as https://github.com/phonegap/phonegap-plugin-barcodescanner/issues/139

@poshaughnessy that is indeed the same issue. Fixed using the same temporary fix. Wondering what we can do to provide a more bulletproof fix for the longterm.

I've resolved this by using the fork from https://github.com/phonegap/phonegap-plugin-barcodescanner/pull/143
cordova plugin add https://github.com/kbytesys/phonegap-plugin-barcodescanner.git#android-zxing-integration

I seem to have no issues with the barcode scanner when I used Visual Studio and Cordova to use it. Once I use Ionic and ngCordova, the plugin does not seem to work, maybe that can provide some help?

Using that fork (https://github.com/kbytesys/phonegap-plugin-barcodescanner.git#android-zxing-integration) will require the installation of another Play Store app that scans barcodes. Whenever the scan function is called, it will instead open the secondary application. Is there no other way to have a lightweight QR code scanner, instead of a pluging that requires another app to work?

I rewrited the plugin and resolved this problem.
cordova plugin add https://github.com/jrontend/phonegap-plugin-barcodescanner

@JrontEnd +1

@JrontEnd does this work with ngCordova?

@SammyIsra I don't have the environment of ngCordova,but you can try it.

@JrontEnd I just did, it worked nicely (I edited the last message too late)
Are there any settings on the reader?

@SammyIsra Nothing,the permission is requested automatically.

@JrontEnd so there is no way to avoid the "beep" the reader makes?
Thank you for your contribution, I had forgotten about this plugin!

@SammyIsra Haha, I am glad to see the plugin helpful to you.

@Gp2mv3 I created a PR ,but not accepted.

@guardezi
I have a good news.The pull request is merged, you can add the plugin of latest version in phonegap repository.

@JrontEnd thanks your plugin resolved the issue.

Is there any plugin available without installing another supporting app in device?

Hi, i have found an issue if you have this pb (10/2016 !).
Don't use cordova plugin add phonegap-plugin-barcodescanner -> 4.0.2 !!!
but cordova plugin add https://github.com/jrontend/phonegap-plugin-barcodescanner -> 4.1.0

I am using ionic with ngcordova. By using,
cordova plugin add https://github.com/jrontend/phonegap-plugin-barcodescanner
this plugin for me barcode scanner is opening. but i am not able get the scan it...
any solution???

Use this one :
Remove the old
ionic plugin add https://github.com/Telerik-Verified-Plugins/BarcodeScanner https://github.com/Telerik-Verified-Plugins/BarcodeScanner —save
ionic platform remove ios
ionic platform remove android
ionic add platform ios
ionic add platform android
ionic state save

Le 25 oct. 2016 à 06:45, Smohanapriya<EMAIL_ADDRESS>a écrit :
I am using ionic with ngcordova. By using,
cordova plugin add https://github.com/jrontend/phonegap-plugin-barcodescanner
this plugin for me barcode scanner is opening. but i am not able get the scan it...
any solution???
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub https://github.com/phonegap/phonegap-plugin-barcodescanner/issues/167#issuecomment-255935341, or mute the thread https://github.com/notifications/unsubscribe-auth/ASvttabp44A77ltKv5vJYEJF7_M1WfvYks5q3YlPgaJpZM4HWXcD.

This plugin,
ionic plugin add https://github.com/Telerik-Verified-Plugins/BarcodeScanner
also not working for me. I am using android marshmallow...
<uses-sdk android:minSdkVersion="16" android:targetSdkVersion="23" />

Im using nougat ... But can you explain with more details and your code ?
The camera is oppening or not ?
Le 26 oct. 2016 06:46, "Smohanapriya"<EMAIL_ADDRESS>a écrit :

This plugin,
ionic plugin add https://github.com/Telerik-Verified-Plugins/
BarcodeScanner
also not working for me. I am using android marshmallow...

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/phonegap/phonegap-plugin-barcodescanner/issues/167#issuecomment-256248448,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ASvttRxVhW-8-rHYE9v1Z5H4uguqdbsUks5q3ts3gaJpZM4HWXcD
.


Deloitte - wrong job title for the "Recruitment and Employer Branding Manager" position

ENVIRONMENT: production
URL: https://peviitor.ro/
Browser: Safari
Device: Laptop
OS: macOS 14 Sonoma
STEPS TO REPRODUCE:

Open URL in browser
Type "Deloitte" in the search bar
Click on "Cauta"
Right-click and select "Inspect"
Copy the Request cURL: https://api.peviitor.ro/v3/search/?q=deloitte&country=România&page=10
Open Postman and create a new entry with GET request
Click on "Send"

EXPECTED RESULT:
The GET request should return a response with a status of 200 OK and information related to job openings retrieved from the API response should match the information related to job openings listed on the official website of Deloitte and on the peviitor.ro website.
ACTUAL RESULT:
The job title from the API response doesn't reflect the job type from the Deloitte website.

fixed


refactor: move getSyncableElements to CollabWrapper

Moved getSyncableElements util to CollabWrapper. Not sure why we were exporting it from the package since it's an implementation detail tightly coupled to excalidraw.com collab implementation.

[ ] changelog

These are elements including deleted but excluding invisibly small elements, I think we should still export it since it's util which host might want to use and maybe rename it.
Since we have been exporting and its in the readme as well so there is a high chance the host will be using it so removing it suddenly also is not a good idea unless we have a strong reason to remove it.

We can expose isInvisiblySmallElement for sure, but there's no point exposing something like getSyncableElements.
We can note in the changelog that if anyone's been using it, they can reimplement it by doing elements.map((el) => el.isDeleted || !isInvisiblySmallElement(el)). That's super low-cost migration.

We can expose isInvisiblySmallElement for sure, but there's no point exposing something like getSyncableElements.
We can note in the changelog that if anyone's been using it, they can reimplement it by doing elements.map((el) => el.isDeleted || !isInvisiblySmallElement(el)). That's super low-cost migration.

yes, getSyncableElements is something that doesn't make sense to expose, hence I suggested if we can rename it but not sure what to rename it though.
let's mark it as a breaking change and expose isInvisiblySmallElement + add the migration in the changelog as you mentioned


Remove unused of parameter

Removes an unused parameter.

So, @casperboone, should we close this PR?

@gandreadis The plan is to suppress the warning, but I can't get the suppression to work :(

@gandreadis @casperboone Done! I managed to suppress the warning :)

Codecov Report

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

@@            Coverage Diff            @@
##             master     #292   +/-   ##
=========================================
  Coverage     80.49%   80.49%
  Complexity      529      529
=========================================
  Files            68       68
  Lines          1579     1579
  Branches        250      250
=========================================
  Hits           1271     1271
  Misses          210      210
  Partials         98       98

Impacted Files
Coverage Δ
Complexity Δ

...ne/githubinteractor/githubapi/JsonObjectBuilder.kt
100% <ø> (ø)
3 <0> (ø)
:arrow_down:

Continue to review full report at Codecov.

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


chore(main): release 1.0.2

:robot: I have created a release beep boop
1.0.2 (2024-12-28)
Performance Improvements

Update workspacePathFormat and change release action (d2e92fc)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

v1.0.2
:sunflower:


As a user, I want to edit my logmessage

Because users are not anonymous, they are linked to just one message. Therefore, they can be restricted to only edit their own message. Which makes it possible to allow editing for authors.

Functionality is finished.


Ability to execute custom SQL before debugging

Is your feature request related to a problem? Please describe.
I use PGTT https://github.com/darold/pgtt extension, and it require load it's module after new session started:
LOAD '$libdir/plugins/pgtt';
https://github.com/darold/pgtt#use-of-the-extension
When I start debugging, I should load this module before debugging.
I try use "Evaluate expression" window, but it, seems, not worked (PostgreSQL 14.1), show nothing.
Describe the solution you'd like
Allow execute LOAD 'module' in the variables
Describe alternatives you've considered
Make separate function for initialize and load module. Execute function once in Variables block.
Additional context

We have to implements a configuration for the plugin, there is others parameters we can put on it.
Your suggestion to run a statement on the debugging session can be implemented in two ways.

Plugin level, with a configuration entry
Debug configuration level

Evaluation is not yet implemented

Thank you!
Now I can alse avoid this problem: create other test procedure with initialization and call of debugging routine.

Since 1.1.1 you can set it in the config panel. Because there are two session, the calling session and the debugger session, there are two parameters.
In the next future parameters(and timeouts) will be moved at project level.
If you encounter other issues with the debugger and it's related to queries (for function search or data grabbing), you can also try to customise it in the config panel.
I will write a documentation for that.


Problems Installing Cnapy

Traceback (most recent call last): File "D:\Usuarios\iprivitera.conda\envs\cnapy-1.1.8\Scripts\cnapy-script.py", line 10, in sys.exit(main_cnapy()) File "D:\Usuarios\iprivitera.conda\envs\cnapy-1.1.8\lib\site-packages\cnapy_main_.py", line 34, in main_cnapy Application() File "D:\Usuarios\iprivitera.conda\envs\cnapy-1.1.8\lib\site-packages\cnapy\application.py", line 76, in init self.window = MainWindow(self.appdata) File "D:\Usuarios\iprivitera.conda\envs\cnapy-1.1.8\lib\site-packages\cnapy\gui_elements\main_window.py", line 68, in init self.central_widget = CentralWidget(self) File "D:\Usuarios\iprivitera.conda\envs\cnapy-1.1.8\lib\site-packages\cnapy\gui_elements\central_widget.py", line 72, in init self.reaction_list = ReactionList(self) File "D:\Usuarios\iprivitera.conda\envs\cnapy-1.1.8\lib\site-packages\cnapy\gui_elements\reactions_list.py", line 128, in init for r in self.appdata.project.cobra_py_model.reactions: AttributeError: 'ProjectData' object has no attribute 'cobra_py_model'
Hello!
I tried to install Cnapy in my job computer, wich has Windows, I used Anaconda to download the cnapy packages with the commands put in the cnapy installation page, but when I thaught that was alla ok when i tried to run the program appears the message above.

Hello! We don't directly recognize the problem :3 But it appears that you tried to install an older version (1.1.8) instead of the most recent one (1.1.9), so that it woud help us if you would try to install version 1.1.9 and see if the same error occurs?

I also have a similar error with the newest version 1.1.9. I am trying an installation on Win10 and used both the  "INSTALL_CNAPY.bat" and the conda install procedure. My error message is:
Traceback (most recent call last): File "C:\Users\liebalu.conda\envs\cnapy-1.1.9\Scripts\cnapy-script.py", line 10, in sys.exit(main_cnapy()) File "C:\Users\liebalu.conda\envs\cnapy-1.1.9\lib\site-packages\cnapy_main_.py", line 34, in main_cnapy Application() File "C:\Users\liebalu.conda\envs\cnapy-1.1.9\lib\site-packages\cnapy\application.py", line 76, in init self.window = MainWindow(self.appdata) File "C:\Users\liebalu.conda\envs\cnapy-1.1.9\lib\site-packages\cnapy\gui_elements\main_window.py", line 68, in init self.central_widget = CentralWidget(self) File "C:\Users\liebalu.conda\envs\cnapy-1.1.9\lib\site-packages\cnapy\gui_elements\central_widget.py", line 72, in init self.reaction_list = ReactionList(self) File "C:\Users\liebalu.conda\envs\cnapy-1.1.9\lib\site-packages\cnapy\gui_elements\reactions_list.py", line 137, in init for r in self.appdata.project.cobra_py_model.reactions: AttributeError: 'ProjectData' object has no attribute 'cobra_py_model'

Thank you @uliebal for the additional report! Finally, we also had a colleague with the same error xD, which can be solved through a different Gurobi error handling routine. Therefore, I'd like to ask if you (and @Ivano2021 :-) also get Gurobi errors before this final error message?

Hello! Sorry for the delay! Clearly I saw that Cnapy works better in MacOS, in Windows is different, even in my MacOS I had to change the solver, instead of Gurobi, I installed GLPK. Beacuse I had the option once I was in the app,  but in Windows was impossible (I work with Windows in my job, but I have a Macbook. Maybe If I install GLPK as a solver for Cnapy can work, but I don´t know how to do it. Thanks.

@Ivano2021 There's no problem with the delay :-) I'm glad to hear that it's connected to the Gurobi issue our colleague had, that's what the new commit (and the next release) will solve, so that CNApy should also run for you under Windows :D

I also had the Gurobi error message, so I am looking forward to the next release.

@uliebal @Ivano2021  Our newest release 1.1.10 is now ready, including a fix which should solve this issue:
https://github.com/cnapy-org/CNApy/releases/tag/v1.1.10

As the problem does not seem to persist anymore, I'll close this issue. But if there are any new problems (or even the same D:), feel free to open a new issue :-)


How to create multiple limits for the same limiter

I want to limit either 1 login per second, or 100 logins per hour for example, but [100, 'h'] is unsatisfactory, because it allows the user to send all 100 of their requests in a single second if they wanted to, which could overload the server.
Hence, I also want both checks - also, if this library is capable of doing that, please could you add this as one of the primary examples on the README.md?

Maybe it can be added as an array of rates for the built-in limiters. I have to experiment a bit.

Added now in 0.6.0, specify the rates as an array and it should work. Can you confirm that it does?

Closing this, as the tests are passing.


Tree shaking: unused code remains in productive app

Which @angular/* package(s) are the source of the bug?
Don't known / other
Is this a regression?
No
Description
Hi,
I tried to upgrade from angular 15 to 16.0.3. The application works fine without the "optimization" flag set to true in the angular.json file. But when set to true, some code remains in the main....js that causes an error, when the page gets loaded.
The console log states "main.bc110eb6fa951c13.js:1 Uncaught ReferenceError: hd is not defined".
Here is the part of the code, that causes this issue:
class EP extends hd { ... }
After some investigation (and also turning off the optimization flag), I found out that this code originally looked like that:
class ReferenceExpr extends ExpressionBase { ... }
So the ExpressionBase class is missing. Which is also fine, I guess, since it is not used in the code and hence was removed by tree shaking. Also the ReferenceExpr class is, as far as I can tell, not used in the code. When I search for a usage of the class "EP" in the optimised code, there is no location where I could find it.
Version 15.2.4 worked without any issues. Did I miss anything in the upgrade process, or is it a bug in the build optimisation?
Please provide a link to a minimal reproduction of the bug
No response
Please provide the exception or error you saw
main.1dde1c799c90e486.js:1 Uncaught ReferenceError: hd is not defined
    at 83 (main.1dde1c799c90e486.js:1:290279)
    at a (runtime.de42e30e1a22c589.js:1:143)
    at main.1dde1c799c90e486.js:1:4227714
    at n (runtime.de42e30e1a22c589.js:1:951)
    at main.1dde1c799c90e486.js:1:55

Please provide the environment you discovered this bug in (run ng version)
Angular CLI: 16.0.3
Node: 18.12.1
Package Manager: npm 8.19.3
OS: darwin arm64

Angular: 16.0.3
... animations, cli, common, compiler, compiler-cli, core, forms
... language-service, platform-browser, platform-browser-dynamic
... router

Package                         Version
---------------------------------------------------------
@angular-devkit/architect       0.1600.3
@angular-devkit/build-angular   16.0.3
@angular-devkit/core            16.0.3
@angular-devkit/schematics      16.0.3
@schematics/angular             16.0.3
rxjs                            7.8.1
typescript                      5.0.4

Anything else?
No response

I think I found the culprit. There was a legacy import statement import '@angular/compiler'; in the main.ts file.
After removing it, the app works fine. Should this still be treated as a bug? Or is anyway bad practice to import the compiler in the main.ts file?

You should not be importing import '@angular/compiler'; directly. The compiler is needed for JIT support and which does not support advanced levels of build-optimisation.

Yes, there was actually some code that used the JIT compiler in an older version of the application. But his feature was removed quite a while ago and removing this import was just forgotten. So I think this issue can be treated as solved :).

For me, it was removing these:
import { JitCompilerFactory } from "@angular/platform-browser-dynamic"; import { Compiler, CompilerFactory, COMPILER_OPTIONS } from "@angular/core";
It was used during Angular 4 days. Forgot to remove them


Extract Input parameters from “mxnet” model

I have saved the model using
mx.model.save(model = fit_dl, prefix = "model", iteration = 10)
and loaded later
fit <- mx.model.load(prefix = "model", iteration = 10)
Now, using object fit, I want to extract the input features (column names of train data). How to do that

Hi, @shivonkar
Just as I have said in the email, I don't think this is possible since we never store this info in the model.
You can see the content of your model as below.
> fit
$symbol
C++ object <0x7d6c790> of class 'MXSymbol' <0x5cdeba0>

$arg.params
$arg.params$fc1_weight
             [,1]         [,2]         [,3]
[1,]  0.001403935  0.006410724 -0.005822465
[2,] -0.009231491  0.012077687  0.002701177
[3,] -0.001227969 -0.019062255 -0.006773810
[4,]  0.002685482 -0.001257635  0.009905992

$arg.params$fc1_bias
[1] 0.000000e+00 7.581989e-05 0.000000e+00

$arg.params$fc3_weight
             [,1]         [,2]
[1,]  0.003062166 -0.001712628
[2,]  0.014468276 -0.013505054
[3,] -0.004934168 -0.000506050

$arg.params$fc3_bias
[1] -0.0001711422  0.0001711314

$aux.params
list()

attr(,"class")
[1] "MXFeedForwardModel"


Re-implement the shoutout command from Vulcan

Closes #88

@all-contributors please add @jwalter for code and tests


[Idea] GIF instead of screenshot

I think this would make the addon look a lot cooler 😀

Nope, sorry. You already have a GIF 😅


Improve Helm charts by adding some missing configuration possibilities

It adds some missing configuration options in Helm charts. It also makes Kiali chart more alike to the rest of charts, so it would be possible also for Kiali provide service type and annotation, like with other charts.
[ x ] Configuration Infrastructure
[ ] Docs
[ x ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure

Hi @voroniys. Thanks for your PR.
I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
Once the patch is verified, the new status will be reflected by the ok-to-test label.
I understand the commands that are listed here.

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

@googlebot I signed it!

Note the helm charts are deprecated... all these changes won't take effect when installing via istioctl.
Note also that soon (once the monorepo work is done with the installer stuff), we are going to incorporate the Kiali Operator into istioctl so these kinds of things won't be needed since the operator will handle this.

FYI: istioctl work is here: https://github.com/istio/installer/pull/556 and https://github.com/istio/operator/pull/622 but those are on hold until the monorepo work is done and we'll collapse all that work into a new PR to go into the new istio/istio installer codebase. But that is where this kind of kiali installer config in this PR should go (I think the Kiali CR will handle all of the config this PR wants to add)

FYI: istioctl work is here: istio/installer#556 and istio/operator#622 but those are on hold until the monorepo work is done and we'll collapse all that work into a new PR to go into the new istio/istio installer codebase. But that is where this kind of kiali installer config in this PR should go (I think the Kiali CR will handle all of the config this PR wants to add)

@jmazzitelli I'd encourage you to submit this work to master of istio/istio. the monorepo merge has been done for a while. The PRs you proposed look good on a cursory review. They need to hit the monorepo though as installer and operator repos have been deprecated.

@sdake the work for the monorepo was partially already done here: https://github.com/istio/istio/pull/20131
But two things:

That PR isn't complete because I don't understand all the knobs/dials I have to touch to get the "make gen" to work. I would need some assistance from someone who knows the intricacies of how the manifest data needs to be modified and auto-code generated to get it to completion.

It is my understanding from attending the last few environment WG meetings that all third-party components such as Prometheus, Grafana, Kiali, etc. are going to be entirely removed from istioctl installer/operator. So I'm not clear on whether I should proceed with https://github.com/istio/istio/pull/20131 anyway - it seems like I should not since third-party components are getting/will be removed.

@sdake the work for the monorepo was partially already done here: #20131
But two things:

That PR isn't complete because I don't understand all the knobs/dials I have to touch to get the "make gen" to work. I would need some assistance from someone who knows the intricacies of how the manifest data needs to be modified and auto-code generated to get it to completion.
It is my understanding from attending the last few environment WG meetings that all third-party components such as Prometheus, Grafana, Kiali, etc. are going to be entirely removed from istioctl installer/operator. So I'm not clear on whether I should proceed with #20131 anyway - it seems like I should not since third-party components are getting/will be removed.

I can help you with your PR to integrate the Kiali operator. This work SHOULD still be relevant for 1.6.
Any PRs that remove addons before there is a clear integration path and  migration forward will be reverted. I am not sure who said they would be removed, but they do not hold the consensus view. I don't expect this work to finish quickly.
cc/ @ostromart @howardjohn


qt4: enable gtk2 by default

Motivation for this change
Analogous to Qt 5 currently. I split GNOME dependencies (we don't need them for most cases). Qt 4 applications look so much better now on XFCE!
Things done

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

[x] NixOS
[ ] macOS
[ ] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

It's perhaps not clear whether to have qt4 depend on gtk2 by default (during runtime).
BTW, gtk2 doesn't need schemas to display file dialogs, I think, so I expect we won't need any wrapping.

Perhaps /cc @ttuegel, too.

@vcunat Indeed, we don't need any wrapping. My point was that Qt 5 currently depends on GTK 3 for theming (sad but necessary), so we may as well have Qt 4 depend on GTK 2. I don't think there is any other way to make this work D:

Yes, I see no other easy and correct way.  We don't yet have a good solution for plugin-like dependencies.

I'll merge this in few days unless anyone objects.


How to increment badge when app in background?

Hi,
How would we auto increment the badge count on iOS when the app is not in foreground?
Tried the following without success:
      Notifications.events().registerNotificationReceivedBackground((notif, completion) => {
        Notifications.ios.getBadgeCount((count) => Notifications.ios.setBadgeCount(count + 1));
        completion({ alert: true, sound: true, badge: true });
      });

@julien9999, if it's about local notification so you can add badge to the params like this (should be added to docs):
Notifications.postLocalNotification({
    ...
    badge: {{new badge value}},
});

If it's about remote notification so you can add the value to the object is being send to APNS as said in apple docs

Hey Daniel, thanks for your reply. it’s about remote notifications.
I would like to get the current badge count and increment. Is it possible?

@julien9999, unfortunately we can't count on registerNotificationReceivedBackground due to iOS restrictions as stated here:

However, the system does not automatically launch your app if the user has force-quit it. In that situation, the user must relaunch your app or restart the device before the system attempts to launch your app automatically again.


How to set Margin and Page breaks

Hi,
Tanks for wonderful plugin.
We are using the plugin in a project. We want to know way to set margins and page breaks.
Thanks & Regards,
kmshelke

Hello @kmshelke the approach for styling print pages is a little different than a normal page. You should check this guide.


KBC-2761 migrate Tickets to JSON - fix type hint

Jira: KBC-2761
Before asking for review make sure that:
Checklist

[x] New client method(s) has tests
[x] Apiary file is updated

Release

[x] I gave the PR a proper label:

major (BC break)
minor (new feature)
patch (backwards compatible fix)
no release (just test changes)

Spadly vsechny testy v ComponentsEventsTest:
15:06:31-UTC - common                             -	1) Keboola\Test\Common\ComponentsEventsTest::testConfigurationChange with data set "defaultBranch" ('defaultBranch')
15:06:31-UTC - common                             -	Failed asserting that two arrays are identical.
15:06:31-UTC - common                             -	--- Expected
15:06:31-UTC - common                             -	+++ Actual
15:06:31-UTC - common                             -	@@ @@
15:06:31-UTC - common                             -	 Array &0 (
15:06:31-UTC - common                             -	     'component' => 'wr-db'
15:06:31-UTC - common                             -	-    'configurationId' => 21068662
15:06:31-UTC - common                             -	+    'configurationId' => '21068662'
15:06:31-UTC - common                             -	     'name' => 'component-events-test'
15:06:31-UTC - common                             -	     'version' => 2
15:06:31-UTC - common                             -	 )
15:06:31-UTC - common                             -
15:06:31-UTC - common                             -	/code/tests/Utils/EventTesterUtils.php:128
15:06:31-UTC - common                             -	/code/tests/Common/ComponentsEventsTest.php:103
15:06:31-UTC - common                             -	/code/vendor/keboola/phpunit-retry-annotations/src/RetryTrait.php:32
15:06:31-UTC - common                             -	phpvfscomposer:///code/vendor/phpunit/phpunit/phpunit:97

Opravil jsem typy a testy uz prochazeji:


Converted BurnState Visualizer

Part of #8995

First pull request and C# endeavour ever, thank you for the comments this simple refactor is helpful for figuring out what's going on.
I'll do a couple more of these soon as this one's correct!

Fixed, oversight with naming and indentation.
( DataField("camelCase"), public var PascalCase. )
Still acclimatizing.
Will make sure it tests successful in future cases before review.
Going to look into GenericVisualizerSystem before next review.


The config's scroll_history should be able to recover from errors

The scroll_history being the wrong type (like -1) will lead to the complete config failing to parse.
This should be changed so only the scroll_history field falls back to the default, instead of the complete config.

This should be fixed with #1157.


My friend is having a rendering bug, and it might be lighting based.

Describe the bug
Hello, my friend is using your modpack. He is on mac, and is getting a lot of weird rendering glitches on some of the blocks on our server. Only he sees it, and we don't know what the problem is.
https://imgur.com/a/NGI4Sqn
Modpack version
3.2.3
Launcher
Minecraft Launcher (vanilla)
Other things

[ ] I updated FO to a newer version (mention, what version to what version)
[X] I removed or added some mods (mention, which ones)

Additional context
I added minihud, tweakeroo, and litematica.

3.2.3 is very old. Please update to the latest version.
Remove the extra mods to make sure they don't cause it
I noticed you are using "fancy grass" on LambdaBetterGrass. While that shouldn't be an issue in itself, it's worth trying if the "fast" option has the same issue.

Due to lack of response, I assume you got it fixed.


[Rest Server] Cannot start job container by nvidia-docker

Currently, rest server cannot start job container by nvidia-docker. If nvidia-docker is installed on the host, job container cannot use gpu correctly.

maybe we can detect if nvidia-docker is installed, if yes, run by nvidia-docker


Support incremental download for Artifact Engine.

Artifact engine should support an option to download artifacts incrementally, only files that have changed should be downloaded. If there is no change in files from the previous build to new build then we should skip downloading those files. This will lead to faster download times.

https://github.com/Microsoft/vsts-rm-extensions/pull/446


Update request: Update Foliate to the GTK4 branch.

Package name: foliate
Latest released version: 2.6.4

Current version on the unstable channel: 2.6.4
Current version on the stable/release channel: 2.6.4

[x] Checked the nixpkgs pull requests

Seeing as the GTK3 branch (master) is pretty EOL and the app hasn't been updated for more than a year, I think it's a good idea to update to the latest commit of the GTK4 branch since that's the branch that being worked on.
Notify maintainers
@onny

Note for maintainers: Please tag this issue in your PR.

the gtk4 branch is now stable


feat: upgrade to version of yargs that introduces extends feature

config files can now be extended using a key extends, which points to a configuration file to inherit from.
fixes #503

@bcoe should we just go to v7.0.2 and merge this? :)

@JaKXz we need to do a little bit of refactoring to because all the top-level options will be global now; I think to keep the CLI identical we should just set global: false for these.
there might be a couple more tweaks required.

closed in favor of: https://github.com/istanbuljs/nyc/pull/541
@JaKXz would love help testing (npm i nyc@next if/when you get two seconds).


Added a 404_NotFound page

Hi I have added a 404 page, issue #134. It is made using react and styled majorly using css. I request you to merge this PR.

Hi @Diptenusarkar could you please add hacktoberfest accepted tag to this PR


[BUG] ECR module missing removal_policy configuration

This module has the removal policy as a method parameter in stack.py, but it is not configurable via manifests and app.py.
It is hard-coded to RETAIN...
Need to update:

app.py
README
tests/test_app.py

@dgraeber started fixing this one and later noticed you've fixed it already.


python3Packages.python-jenkins: 1.3.0 -> 1.4.0

Motivation for this change
Updates to the most recent version of python-jenkins. It was
originally broken during the auto-update in b4588c6a0f2aba6da8cf52c2aef52a4fda6bdb89.
The tests need to be skipped as it doesn't build without the PyPI
tarball as source which doesn't provide the tests.
Things done

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

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

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

The tests need to be skipped as it doesn't build without the PyPI
tarball as source which doesn't provide the tests.

So using the source at https://git.openstack.org/cgit/openstack/python-jenkins doesn't work?

I used the GitHub version (https://github.com/openstack/python-jenkins/) using fetchgit with leaveDotGit = true and it broke with an exception that such a tarball doesn't work for building and a PyPI tarball is recommended.

@worldofpeace anything else to add? :)

@worldofpeace I retried and managed to get the tests running :)

Noticed that you didn't pull in all the deps test-requirements.txt which I suppose is fine.
Though I did see some warnings when the tests were running
WARNING:root:JENKINS_API_EXTRA_HEADERS adds these HTTP headers: ['X-Auth: 123', 'X-Key: 234']
.WARNING:root:JENKINS_API_EXTRA_HEADERS adds these HTTP headers: ['X-Auth: 123', 'X-Key: 234']
.....................building False
msg test
revision 66
user unknown
building False
msg test
revision 66
user unknown
building False
msg test
revision 66
user unknown
building False
msg test
revision 66
user unknown

Noticed that you didn't pull in all the deps test-requirements.txt which I suppose is fine.

Yeah. Upstream put several development tools into test_requires (like pre-commit) that aren't packagd in nixpkgs and may be helpful in development, but aren't necessary to get the test suite running.

Though I did see some warnings when the tests were running

The code yields warnings if extra headers for Jenkins requests are set. If I understand the code right, this happens in some testcases and is actually fine.