text:
File /css/app.css not defined in asset manifest.

Hey, thanks for contributing to Koel! To save time for both of us, please make sure these checkboxes are checked before submitting the issue:

[-] You have read and followed closely the Wiki, Upgrade Guide, as well as Troubleshooting
[-] The issue has not been reported before
[-] This is not a "how to install on Windows" or "why is my npm messed up" question
[-] You're a cool person

All checked? Now also make sure your issue

[-] Is associated with a version. Or better yet, a commit.
[-] Is as detailed as possible (ahem... OS, browser, steps to reproduce, maybe?)
[-] Includes the error output if it's a bug/error report ("Whoops!" is not very helpful, you know)
[-] Is in English, 因为我不说中文。

I get this error after I have cloned koel version v3.7.2, ran composer install, npm install and yarn build.
yarn build shows this errors too

Please, what can I do?

@phanan
Node - v10.0.0
NPM - v6.0.1
Yarn - v1.6.0

@AdebsAlert @MichaIng Super helpful info, thanks! I've noticed some (unrelated) problems with the latest combination of yarn/Node as well. May I ask you guys to give it a try with an older version of Node (like 8.x or 7.x) using something like nvm or n and let me know if the problem persists?

@phanan

With version 8 indeed the error does not appear any more, but I face different ones then:

root@DietPi:~# npm install -g n
/usr/local/bin/n -> /usr/local/lib/node_modules/n/bin/n
+ n@2.1.11
added 1 package from 4 contributors in 2.118s
root@DietPi:~# n 8

     install : node-v8.11.2
       mkdir : /usr/local/n/versions/node/8.11.2
       fetch : https://nodejs.org/dist/v8.11.2/node-v8.11.2-linux-x64.tar.gz
######################################################################################################################################################### 100.0%
   installed : v8.11.2

root@DietPi:~# cd /var/www/koel/
root@DietPi:/var/www/koel# php artisan koel:init
Attempting to install or upgrade Koel.
Remember, you can always install/upgrade manually following the guide here:
📙  https://koel.phanan.net/docs

App key exists -- skipping
JWT secret exists -- skipping
Migrating database
Data seeded -- skipping
Compiling front-end stuff
yarn install v1.6.0
[1/4] Resolving packages...
[2/4] Fetching packages...
info fsevents@1.1.2: The platform "linux" is incompatible with this module.
info "fsevents@1.1.2" is an optional dependency and failed compatibility check. Excluding it from installation.
[3/4] Linking dependencies...
[4/4] Building fresh packages...
success Saved lockfile.
$ yarn production
yarn run v1.6.0
$ cross-env NODE_ENV=production node_modules/webpack/bin/webpack.js --progress --hide-modules --config=node_modules/laravel-mix/setup/webpack.config.js
 95% emitting ERROR  Failed to compile with 47 errors01:33:30

 error  in ./resources/assets/js/components/main-wrapper/main-content/artist.vue

Module build failed:
      @mixin control($playBtnWidth, $fontSize, $textIndent) {
            ^
      Mixins may not be defined within control directives or other mixins.
      in /var/www/koel/resources/assets/sass/partials/_mixins.scss (line 79, column 14)

 @ ./node_modules/vue-style-loader!./node_modules/css-loader!./node_modules/vue-loader/lib/style-compiler?{"vue":true,"id":"data-v-1ce16e40","scoped":true,"hasInlineConfig":true}!./node_modules/sass-loader/lib/loader.js!./node_modules/vue-loader/lib/selector.js?type=styles&index=0!./resources/assets/js/components/main-wrapper/main-content/artist.vue 4:14-378
 @ ./resources/assets/js/components/main-wrapper/main-content/artist.vue
 @ ./node_modules/babel-loader/lib?{"cacheDirectory":true,"presets":[["es2015",{"modules":false}]]}!./node_modules/vue-loader/lib/selector.js?type=script&index=0!./resources/assets/js/components/main-wrapper/main-content/index.vue
 @ ./resources/assets/js/components/main-wrapper/main-content/index.vue
 @ ./node_modules/babel-loader/lib?{"cacheDirectory":true,"presets":[["es2015",{"modules":false}]]}!./node_modules/vue-loader/lib/selector.js?type=script&index=0!./resources/assets/js/components/main-wrapper/index.vue
 @ ./resources/assets/js/components/main-wrapper/index.vue
 @ ./node_modules/babel-loader/lib?{"cacheDirectory":true,"presets":[["es2015",{"modules":false}]]}!./node_modules/vue-loader/lib/selector.js?type=script&index=0!./resources/assets/js/app.vue
 @ ./resources/assets/js/app.vue
 @ ./resources/assets/js/app.js
 @ multi ./resources/assets/js/app.js ./resources/assets/sass/app.scss ./resources/assets/sass/remote.scss

 error  in ./resources/assets/sass/app.scss
....

Several similar module build errors appear, I guess the 47 mentioned ones.
See also my edit (€) above about node-sass, as also this error contains sass.

Have you checked this? https://github.com/akveo/nebular/issues/339
2018-05-20 1:43 GMT+02:00 MichaIng notifications@github.com:

@phanan https://github.com/phanan

With version 8 indeed the error does not appear any more, but I face
different ones then:

root@DietPi:~# npm install -g n
/usr/local/bin/n -> /usr/local/lib/node_modules/n/bin/n

n@2.1.11
added 1 package from 4 contributors in 2.118s
root@DietPi:~# n 8
install : node-v8.11.2
mkdir : /usr/local/n/versions/node/8.11.2
fetch : https://nodejs.org/dist/v8.11.2/node-v8.11.2-linux-x64.tar.gz
######################################################################################################################################################### 100.0%
installed : v8.11.2

root@DietPi:~# cd /var/www/koel/
root@DietPi:/var/www/koel# php artisan koel:init
Attempting to install or upgrade Koel.
Remember, you can always install/upgrade manually following the guide here:
📙  https://koel.phanan.net/docs
App key exists -- skipping
JWT secret exists -- skipping
Migrating database
Data seeded -- skipping
Compiling front-end stuff
yarn install v1.6.0
[1/4] Resolving packages...
[2/4] Fetching packages...
info fsevents@1.1.2: The platform "linux" is incompatible with this module.
info "fsevents@1.1.2" is an optional dependency and failed compatibility check. Excluding it from installation.
[3/4] Linking dependencies...
[4/4] Building fresh packages...
success Saved lockfile.
$ yarn production
yarn run v1.6.0
$ cross-env NODE_ENV=production node_modules/webpack/bin/webpack.js --progress --hide-modules --config=node_modules/laravel-mix/setup/webpack.config.js
95% emitting ERROR  Failed to compile with 47 errors01:33:30
error  in ./resources/assets/js/components/main-wrapper/main-content/artist.vue
Module build failed:
@mixin control($playBtnWidth, $fontSize, $textIndent) {
^
Mixins may not be defined within control directives or other mixins.
in /var/www/koel/resources/assets/sass/partials/_mixins.scss (line 79, column 14)
@ ./node_modules/vue-style-loader!./node_modules/css-loader!./node_modules/vue-loader/lib/style-compiler?{"vue":true,"id":"data-v-1ce16e40","scoped":true,"hasInlineConfig":true}!./node_modules/sass-loader/lib/loader.js!./node_modules/vue-loader/lib/selector.js?type=styles&index=0!./resources/assets/js/components/main-wrapper/main-content/artist.vue 4:14-378
@ ./resources/assets/js/components/main-wrapper/main-content/artist.vue
@ ./node_modules/babel-loader/lib?{"cacheDirectory":true,"presets":[["es2015",{"modules":false}]]}!./node_modules/vue-loader/lib/selector.js?type=script&index=0!./resources/assets/js/components/main-wrapper/main-content/index.vue
@ ./resources/assets/js/components/main-wrapper/main-content/index.vue
@ ./node_modules/babel-loader/lib?{"cacheDirectory":true,"presets":[["es2015",{"modules":false}]]}!./node_modules/vue-loader/lib/selector.js?type=script&index=0!./resources/assets/js/components/main-wrapper/index.vue
@ ./resources/assets/js/components/main-wrapper/index.vue
@ ./node_modules/babel-loader/lib?{"cacheDirectory":true,"presets":[["es2015",{"modules":false}]]}!./node_modules/vue-loader/lib/selector.js?type=script&index=0!./resources/assets/js/app.vue
@ ./resources/assets/js/app.vue
@ ./resources/assets/js/app.js
@ multi ./resources/assets/js/app.js ./resources/assets/sass/app.scss ./resources/assets/sass/remote.scss
error  in ./resources/assets/sass/app.scss
....

Several similar module build errors appear, I guess the 47 mentioned
ones.
See also my edit (€) above about node-sass, as also this error
contains sass.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/phanan/koel/issues/748#issuecomment-390440779, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AHrt0l2BjEEqGtkwpvIBrihnHamncBynks5t0K4fgaJpZM4UC4Il
.

@phanan
Same with errors after npm i node-sass@4.8.1, if this is what you meant?

I mean what you described (which I’ve not time to test yet) looks exactly
that of the linked issue, so maybe explore in that direction :)
On Sun, 20 May 2018 at 02:19, MichaIng notifications@github.com wrote:

@phanan https://github.com/phanan
Same with errors after npm i node-sass@4.8.1, if this is what you meant?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/phanan/koel/issues/748#issuecomment-390443379, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AHrt0sXvxG0i93RMO9NxIohc9PN7gNlEks5t0LaCgaJpZM4UC4Il
.

@phanan
The issue you linked looks similar but not exactly the same than mine. Different line numbers and functions names and also the workaround didn't work and it should be already fixed as well.
BUT: Tried it with Node 8 on the other system and it works! Koel web UI opens without issue.
I then removed the Koel build environment on the other system and started from scratch, including a version downgrade to Node v8.11.2 before running php artisan koel:init and it worked without issues as well!
So the issue is indeed an incompatibility in combination with Node 10. I refer to https://github.com/phanan/koel/issues/748#issuecomment-390433548 bottom €, as at least with those node-sass versions outside and inside laravel-mix something goes wrong and the link clearly states that node-sass 4.9 is needed in combination with Node 10.
@AdebsAlert
Long story short, try:
npm install -g n
n 8
php artisan koel:init

That’s some amazing debugging right there, thanks! I’ll take a deeper look
when I have time. In short though, Node 10 is really a pain to work with :(
On Sun, 20 May 2018 at 14:30, MichaIng notifications@github.com wrote:

@phanan https://github.com/phanan
The issue you linked looks similar but not exactly the same than mine.
Different line numbers and functions names and also the workaround didn't
work and it should be already fixed as well.
BUT: Tried it with Node 8 on the other system and it works! Koel web UI
opens without issue.
I then removed the Koel build environment on the other system and started
from scratch, including a version downgrade to Node v8.11.2 before running php
artisan koel:init and it worked without issues as well!
So the issue is indeed an incompatibility in combination with Node 10. I
refer to #748 (comment)
https://github.com/phanan/koel/issues/748#issuecomment-390433548 bottom
€, as at least with those node-sass versions outside and inside
laravel-mix something goes wrong and the link clearly states that node-sass
4.9 is needed in combination with Node 10.
@AdebsAlert https://github.com/AdebsAlert
Long story short, try:
npm install -g n
n 8
php artisan koel:init
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/phanan/koel/issues/748#issuecomment-390477430, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AHrt0hx12maGLqro1PQhMA2gfCjFYj1kks5t0WHngaJpZM4UC4Il
.

Thank you @MichaIng it works!
Thank you @phanan for an amazing tool.

authors: AdebsAlert MichaIng phanan

repo: phanan/koel


text:
Call.method issue or documentation issue.

Documentation on Phantasma.io site states that Call.method can be used to call functions inside of our smart contract. Upon further testing and research it appears that it is not implemented.
Module.cs line 336:

libDecl.AddMethod("method", MethodImplementationType.Custom, VarType.Generic(0), new[] { new MethodParameter("method", VarKind.Method), new MethodParameter("...", VarKind.Any) });
Its a custom method; however there is no callback.

ModuleCallExpressons.cs line 217:

`case MethodImplementationType.Custom:
              if (this.method.PreCallback == null && this.method.PostCallback == null)
              {
                  output.AppendLine(this, $"LOAD r0 \"{this.method.Library.Name}.{this.method.Name} not implemented\"");
                  output.AppendLine(this, $"THROW r0");
              }

              break;`

-States that if there is no callback then the method is not implemented

Also, upon further investigation, the naming convention "Call.method" is confusing the compiler to believe type "method" is being called. Changing the name to "Call.routine" fixed this issue.

Yes, I understand I can use this.MyMethod(myArgs), but I wanted to test out different libraries that was built in TOMB. My issue was with calling the Call library  In particular:

Call.contract
Call.method

I was suggesting that the naming convention be changed since 'contract' is a keyword and 'method' is a type. The compiler reads it as such. My suggestion was renaming it:
Example

Call.method => Call.routine
Call.contract => Call.cntrct

** I did not know that this.myMethod(myArgs) was more optimal so thank you for teaching me that.

Thank you! I will close the issue.

source: gharchive/issue

repo: phantasma-io/TOMB

license: MIT

license_type: permissive

license_source: github-api


text:
Document Issue Closing Process

The following process should be documented in the “Reviewing/Closing an Issue” section on the git vignette.
Issues are not closed until they are released. If no work is required for an issue, it should be moved to “Sprint Done” without closing it.

We should probably announce this on the dev slack channel.  I've been closing the issue when my Pull Request is moved into devel.

@bms63 , yes, good idea. I have posted it on slack.

Let's make sure to include this in our next sprint.

Should we make a "sprint" to get all the links fixed, documents updated, website polished before the release?  Like, make sure all that stuff is in this sprint before release?

authors: bms63 bundfussr thomas-neitmann

repo: pharmaverse/admiral


text:
Closes #2378 add test for get_imputation_target_date()

Thank you for your Pull Request! We have developed this task checklist from the Development Process Guide to help with the final steps of the process. Completing the below tasks helps to ensure our reviewers can maximize their time on your code as well as making sure the admiral codebase remains robust and consistent.
Please check off each taskbox as an acknowledgment that you completed the task or check off that it is not relevant to your Pull Request. This checklist is part of the Github Action workflows and the Pull Request will not be merged into the main branch until you have checked off each task.

[ ] Place Closes #<insert_issue_number> into the beginning of your Pull Request Title (Use Edit button in top-right if you need to update)
[ ] Code is formatted according to the tidyverse style guide. Run styler::style_file() to style R and Rmd files
[ ] Updated relevant unit tests or have written new unit tests, which should consider realistic data scenarios and edge cases, e.g. empty datasets, errors, boundary cases etc. - See Unit Test Guide
[ ] If you removed/replaced any function and/or function parameters, did you fully follow the deprecation guidance?
[ ] Review the Cheat Sheet. Make any required updates to it by editing the file inst/cheatsheet/admiral_cheatsheet.pptx and re-upload a PDF version of it to the same folder.
[ ] Update to all relevant roxygen headers and examples, including keywords and families. Refer to the categorization of functions to tag appropriate keyword/family.
[ ] Run devtools::document() so all .Rd files in the man folder and the NAMESPACE file in the project root are updated appropriately
[ ] Address any updates needed for vignettes and/or templates
[ ] Update NEWS.md under the header # admiral (development version) if the changes pertain to a user-facing function (i.e. it has an @export tag) or documentation aimed at users (rather than developers)
[ ] Build admiral site pkgdown::build_site() and check that all affected examples are displayed correctly and that all new functions occur on the "Reference" page.
[ ] Address or fix all lintr warnings and errors - lintr::lint_package()
[ ] Run R CMD check locally and address all errors and warnings - devtools::check()
[ ] Link the issue in the Development Section on the right hand side.
[ ] Address all merge conflicts and resolve appropriately
[ ] Pat yourself on the back for a job well done! Much love to your accomplishment!

@bms63 @millerg23 I was mistaken in that the internal functions weren't being loaded in the test suite, I probably had somethin wrong with my session, nothing needs to be done to address this, just regular ol' additional tests

I assume failing checks are around ERROR message updates. Happy to Approve this, once checks running clean.

@zdz2101 @millerg23 is this one good to merge?

@zdz2101 @millerg23 is this one good to merge?

Should be once cicd goes through!

source: gharchive/pull-request


text:
Problems with includes, mixins and partials

When gulp render jade show arror if the file are using includes, mixins or partials, how I fix this... what's the correct setting?

Please direct support questions to stack overflow

authors: phated scarnago

repo: phated/gulp-jade

license: mit

license_source: bigquery


text:
Update bitflags to 0.9.1

Update bitflags from 0.7.0 to 0.9.1.
Ref: #344

Thanks a lot!

authors: crawfxrd phil-opp


text:
Add a badge to the doc

Also changed the URL to the doc so that users can reach it easily.

Thanks!

authors: phil-opp toku-sa-n

repo: phil-opp/linked-list-allocator


text:
Prevent overflow in src/memory_map.rs

This commit makes it so that pointers in src/memory_map.rs are casted only to other pointers and all arithmatic is done with the pointer offset() method.

Sorry about that. I didn't test it the first time because my project is in a state of disarray. I tried it with blog_os this time and it seems to work fine.

No worries :). Thanks!


text:
Hide inaccessible members and functions

Currently, racer completes private struct members and functions cross module.

@phildawes just to be sure, everything should be done in matchers.rs ?
With local variable and pub keyword ?

Hi @tafia,
I think for field the problem is in nameres::search_struct_fields() and ast::parse_struct_fields(). It should be using the 'visibility' attribute in the parsed ast 'NamedField' type.

But this would require getting the Visibility from the Item in case the field visibility is inherited. How would one do that?

Has this been deemed a wontfix or is it just not of high importance?
(racer 1.2.10 fwiw)

Would be nice if this 5 year old issue got fixed since internal functions shouldn't be shown when referencing some other crate.

authors: 0xd4d Phrohdoh Stebalien llogiq phildawes tafia

repo: phildawes/racer


text:
Unable to set specific list width for each individual horizontal DragAndDropLists

Im using the listWidth to set the width of each indivual horizontal list. This sets all the lists to the same width.
Is there a method of doing something like this?
listWidth: list.isEmpty ? widthWhenEmpty : widthWhenNotEmpty
So that all empty lists have a different listWidth.
Thanks to any responder!

use contentsWhenEmpty : Container( width //Set your width when the list is empty )

repo: philip-brink/DragAndDropLists


text:
PageMe: a tool for students to quickly a) create a web page, and b) make it easy for others to comment and help improve it

Ratings:

Agreement: 8
Neutral: 1

I suggested "PageMe" as a name -- please change / update if you've got a better idea!

repo: phillipadsmith/getting-down-with-github


text:
What kind of "!" prefixed expressions cannot decompose?

Can you elaborate on this?

REQUIRE_FALSE( expression ) and
CHECK_FALSE( expression )

Evaluates the expression and records the logical NOT of the result. If an exception is thrown it is > caught, reported, and counted as a failure. (these forms exist as a workaround for the fact that ! prefixed expressions cannot be decomposed).

Example:

REQUIRE_FALSE( thisReturnsFalse() );

But all of the checks in this example pass:
#define CATCH_CONFIG_MAIN
#include "catch.hpp"

bool func(){
  return false;
}

TEST_CASE("test"){

    CHECK(!func());
    CHECK_FALSE(func());
    CHECK(!false);
    CHECK((((!false))));
}

Is this documentation still correct?
Can you provide some examples where "!" prefixed expressions cannot be decomposed?

Did you try CHECK(!true)?

CHECK(!true);  does not pass.  This seems to decompose correctly as well.

Expressions that involve other operators, e.g.: REQUIRE( ! world < hello );
#define CATCH_CONFIG_MAIN
#include "catch.hpp"

TEST_CASE( "! expr", "[show]" )
{
    std::string hello("hello");
    std::string world("world");

    REQUIRE( ! true );
    REQUIRE(   hello < world );
    REQUIRE( ! world < hello );
}

// g++ -Wall -I../single_include -o not_expr.exe not_expr.cpp && not_expr.exe

Gives:
In file included from not_expr.cpp:2:0:
not_expr.cpp: In function 'void ____C_A_T_C_H____T_E_S_T____8()':
not_expr.cpp:15:14: error: no match for 'operator!' (operand type is 'std::__cxx11::string {aka std::__cxx11::basic_string<char>}')
     REQUIRE( ! world < hello );
              ^
not_expr.cpp:15:14: note: candidate: operator!(bool) <built-in>
not_expr.cpp:15:14: note:   no known conversion for argument 1 from 'std::__cxx11::string {aka std::__cxx11::basic_string<char>}' to 'bool'
not_expr.cpp:15:14: error: no match for 'operator!' (operand type is 'std::__cxx11::string {aka std::__cxx11::basic_string<char>}')
     REQUIRE( ! world < hello );
              ^
not_expr.cpp:15:14: note: candidate: operator!(bool) <built-in>
not_expr.cpp:15:14: note:   no known conversion for argument 1 from 'std::__cxx11::string {aka std::__cxx11::basic_string<char>}' to 'bool'
not_expr.cpp:15:22: warning: logical not is only applied to the left hand side of comparison [-Wlogical-not-parentheses]
     REQUIRE( ! world < hello );
                      ^

So, maybe:
these forms exist as a workaround for the fact that ! prefixed expressions cannot be decomposed if they also involve other operators.

Notice: edited previous comment.

That's not a valid expression one would expect to work in normal C++ either.
Doing just:
std::string hello ="hello";
std::string world = "world";
if(! hello < world){
}

gives a similar error since string does not support the logical negation operator.

Oops, you're right.
The reason is that writing the expression in parenthesis, like REQUIRE( ! ( hello < world ) ) defeats expression decomposition.

Reworked example:
#define CATCH_CONFIG_MAIN
#include "catch.hpp"

TEST_CASE( "! expr", "[show]" )
{
    int a = 1;
    std::string hello("hello");
    std::string world("world");

    CHECK( ! true );  // Ok

    CHECK( ! --a );   // Ok

    CHECK(     hello < world );     // Ok
    CHECK( ! ( world < hello ) );   // No expression decomposition
}

// g++ -Wall -I../single_include -o not_expr.exe not_expr.cpp && not_expr.exe --reporter compact --success

Result
prompt>not_expr.exe --reporter compact --success
not_expr.cpp:10: failed: ! true for: false
not_expr.cpp:12: passed: ! --a for: true
not_expr.cpp:14: passed: hello < world for: "hello" < "world"
not_expr.cpp:15: passed: ! ( world < hello ) for: true
Failed 1 test case, failed 1 assertion.

Oh!  'Decomposition' refers to showing the expression in the output.
I was confused into thinking it had something to do with how the actual expression got evaluated, and whether the result was a passing/failing test.
So the only real danger of using an expression prefixed with "!", is that may appear less clear in the diagnostic output.  CHECK_FALSE exists to provide a more cosmetic output, but it's not required to get the expression to evaluate correctly.  Thanks, that makes more sense now.

authors: kirbyfan64 luxe martinmoene

repo: philsquared/Catch


text:
CATCH REQUIRE() macro isn't threadsafe

Hi,
I'm using CATCH over at https://github.com/ned14/ISO_POSIX_standards_stuff. One unit test is creating 32 threads and thrashing a proposed POSIX permit object, but it turns out that REQUIRE() uses static storage and therefore isn't threadsafe.
Could you perhaps add an optional magic macro at the top which turns on the C++11 thread_local specifier for CATCH static data, at least those static data REQUIRE() uses? Intel's TBB and Microsoft's PPL know how to propagate CATCH exceptions out of parallel regions on C++11 compilers, so this actually should "fix" CATCH for multithreaded use.
Thanks,
Niall

For those needing a brutal hack job of a partially thread safe CATCH and are reading this issue, you may find my fork at https://github.com/ned14/Catch-ThreadSafe of use. I make no claims as to its reliability, but it works for me.

Thread safety is planned for Catch2, but current Catch will be most likely left thread-unsafe.
I am closing this as part of backlog cleanup.

@philsquared So, it's been several months, if not years, since this was posted. I see re: @horenmar 's post earlier this year re: Catch v2. This is now in development. Any plans when this will be released? I've got a client/server C++11 threading situation where this would be very welcome. Thank you!

@ned14 Would you mind providing a synopsis of the things you addressed to achieve at least a hacked v1.5 thread safety? Thank you...

@ned14 Would you mind providing a synopsis of the things you addressed to achieve at least a hacked v1.5 thread safety? Thank you...

That was a long time ago I am afraid. I remember hacking a mutex into the iostreams redirector which took some trouble to figure out. But I basically "giant lock"ed it.
I no longer use my hacked edition of CATCH as I needed a unit test framework which ran well with exceptions and RTTI disabled as well as under multiple threads, so I ended up writing my own even lighter weight test framework than CATCH which still outputs JUnit XML and I use that exclusively now: https://github.com/ned14/quickcpplib/blob/master/include/boost/test/unit_test.hpp. Yes, it's Boost.Test drop in compatible, and it's ridiculously simple, perhaps too simple for some folk. But it has near zero compile time overhead, which fixes the almost 30 second compile times I saw with CATCH.
Niall

@ned14 I take to mean does not assume a Boost dependency. I happen to be here getting my knees wet with Catch after struggling to heavy lift Boost, BTW. I haven't noticed compile time issues (yet); that's not to say I would the further I get into it. I may have a look at that, however. Thank you.

Oh no, the whole point of that quickcpplib file is to eliminate the Boost dependency.
Compile time issues only appear if you're doing hefty metaprogramming and feeding those expressions into CATCH. CATCH runs further metaprogramming on such expressions you see, and compile times start breaking past twenty seconds quickly enough.

@ned14 My interest is piqued. How do you handle "matcher" scenarios? Or "generator" scenarios? Can you "match" exceptions when they are handled? i.e. expectations of an exception, with a match on its user-defined state?

authors: horenmar mwpowellhtx ned14


text:
feat: add Hash support for class and style attributes

Description :book:
This pull request adds support for passing Hash in class and style attributes in a way that feels idiomatic in Ruby and achieves composability.
Background 📜
When writing Phlex components, things usually start simple:
div(id: "hero", class: "section")

but when you need a conditional class, things get a bit awkward:
div(id: "hero", **classes("section", ("active" if active)))

This proposal is meant to provide an alternative which is easier to understand, and easier to write:
div(id: "hero", class: ["section", active:])

Examples 🧪
class
A component such as:
def active = true

def inactive = false

def view_template
  div(class: ["dropdown", inactive:, active:])
end

would render:

style
A component such as:
def view_template
  div(style: {font_size: "16px", opacity: 0})
end

would render:

Precedents
This feels like the Ruby equivalent of what frameworks like Vue provide in their template system:

Class and Style Bindings

I really like the look of this!

@bradgessler this makes your idea in #720 even more useful.

This syntax has never really jived with my expectations—I think it's too clever.
For example, what does this return?
def active = true

def inactive = false

def priority = %[low medium high].sample

def view_template
  active = false
  inactive = true
  div(class: ["dropdown", inactive:, active:, priority:])
end

I find it surprising that it binds to the value of a method and shows the class, or not, if its true or false.
Instead I'd propose methods that make it clearer what these things are doing. Here's an idea if method is used:
def active = true

def inactive = false

def priority = %[low medium high].sample

def view_template
  # This would inflect on the Method class. Token is Method#name and if the return
  # value is a string, it would populate with that string. If it's boolean it would return
  # Method#name token if it's true. Otherwise nothing if its empty.
  div(class: ["dropdown", method(:active), method(:inactive), method(:priority))]
end

If I'm reading this code it's much clearer to me, "Oh, this is the :active method". I can even copy and paste the code into my debugger to look at it.
There could probably be some Phlex helper methods that dress this up a bit more, but I think at a minimum using the already existing method method would improve readability and prevent this API from being painted into a corner.

+1 for style Hash support. We've used a custom implementation at Clearscope for some time now.
I tend to agree with @bradgessler's sentiments around class. We've always used arrays of strings and have never been tempted to do more:
def view_template
  div(class: ["btn", ("active" if active?)])
end

def active? = true

# OR

def view_template
  div(class: ["btn", active_class])
end

def active_class = "active" if condition?

@bradgessler I think the example wasn’t clear because it used the same class name as conditional. It’s not calling the key as a method. I think this might be a better example:
div(class: ["rounded", "bg-blue color-white font-bold" => @active])

Read => above as “if”.

Quick note: this is similar to the token_list Rails helper (aliased class_names) which is in my experience quite easy to work with.

@bradgessler I think the example wasn’t clear because it used the same class name as conditional. It’s not calling the key as a method. I think this might be a better example:
div(class: ["rounded", "bg-blue color-white font-bold" => @active])

Read => above as “if”.

Oh I see, so the "magic method" stuff is gone since the tokens helper is removed? What you propose is much more sane and less surprising since the behavior is congruent to how it works for HTML attributes.

authors: ElMassimo Spone bradgessler hangsu joeldrapper

repo: phlex-ruby/phlex


text:
Semantic: naming of Phly\Conduit\Middleware

Hi there, love what you are doing here.
I just wanted to discuss the naming of Phly\Conduit\Middleware. When I first read the readme, I was totally confused by the fact that a middleware is a callable, but at the same time a middleware is an instance of Phly\Conduit\Middleware. And worse: you add middlewares on a Middleware by calling pipe()…
So correct me if I'm wrong, but Middleware is actually a sort of "middleware builder", or rather "middleware root" which just so happen to be a callable (which means it's a middleware) right? So would it make sense to name that class differently, maybe with a more specific name to avoid confusion? Or did I miss something?

The Middleware class exists to allow building a stack of middleware... and, because it defines __invoke with the expected arguments, is middleware itself. This means you can have a Middleware instance as the root of your application:
use Phly\Conduit\Middleware
use Phly\Http\Response;
use Phly\Http\Server;
use Phly\Http\ServerRequest;

// Using Server:
$app = new Middleware();
$server = Server::createServer($app);
$server->listen();

// or as a callable:
$app = new Middleware();
$response = $app(ServerRequestFactory::fromGlobals(), new Response());
// emit the response somehow...

Alternately, it can compose/aggregate other middleware, which may or not be instances of Middleware:
use Phly\Conduit\Middleware

$app = new Middleware();

$contact = new Middleware();
$contact->pipe('/', function ($req, $res, $next) { });
$contact->pipe('/process', function ($req, $res, $next) { });
$contact->pipe('/thank-you', function ($req, $res, $next) { });
$app->pipe('/contact', $contact);

It's not a middleware builder, but more of an aggregator, with a little extra logic thrown in to ensure that a "final handler" (for handling error conditions) is present if no $next callable is provided.
To be honest, I'm not exactly sure what to call it. I settled on Middleware, as it can function in that role, but I also see your point that it may be confusing (developers may thing they must extend that class to implement middleware).
I'm open to naming suggestions -- but not Builder. :)

Yep not "builder" indeed. Why not MiddlewareStack as you said it yourself ;) Or MiddlewareAggregator?
I understand the definition of a middleware is "a callable that takes X & Y parameters" but since it's not possible to define contracts for callables, would it make sense to define a Middleware interface?
I'm guessing most actual implementation of middlewares will be classes (e.g. all StackPHP "middlewares" are classes, especially since all of it is installed with Composer), so an interface would definitely be useful. The interface would be used to provide a strong contract of Middleware implementations, but wouldn't be required of course (since any callable can be a middleware).

MiddlewareStack fits nicely, and is one name I considered -- the name mirrors the design (well, kind of -- technically it's a queue). My primary concern is I don't want it to be confused with StackPHP (which implements lambda-based middleware, vs. message-passing/functional, which Conduit is impelementing). I may go with MiddlewareQueue.
Regarding a middleware interface, I've resisted the idea until now. None of Rack, WSGI, or Connect define one; they operate on functions/callables only, and rely on the language to raise errors if the function cannot accept the arguments.
That said, when I talked to @simensen about middleware this past fall, one idea we came up with was having a site or project that would define a number of different middleware styles. (StackPHP started down this trail, but it hasn't really gone anywhere.) Defining an interface here might be a first step towards that -- a project could indicate they implement a particular style of middleware by implementing the interface. It could also lead to some generic "middleware translation" code, that translates from one style to another, allowing you to drop in a wrapper around your own middleware that's compliant with the consuming system.
I'm going to slate these changes for the 0.11.0 release. Thanks for the conversation, @mnapoli !

@weierophinney are we going to have a chance to talk in person again soon? or maybe a video chat? it would be nice to sync up again and talk more about middlewares and the future of stack. stack is currently positioned as only working w/ symfony's http kernel interface but i'm planning on talking more about the future of stack and psr http message during my #phpbnl15 talk.
to be blunt, i don't see much future in stack being tied to http kernel interface beyond symfony 2. the tools we've built there should continue to work for as long as symfony 2 is around. if the existing tools work w/ symfony 3 out of the box then that will be supported as well. however, i don't see anyone spending more time in continuing to develop stack as a symfony-only project from this point forward.
what i'd like to see is trying to leverage stack as a thing that people may already be associating with middlewares and build out a hub for "php middlewares." perhaps part of that would be writing "stack psr http adapters for symfony" (to continue the symfony legacy) but who knows what will happen there. if psr http message is ready early enough i still have hope that symfony might be able to implement it directly (or provide official adapters) in time for symfony 3's launch at the end of the year.
one can hope, right? :)

@mnapoli There's actually two interfaces for middleware with the paradigm Connect and Conduit use: normal middleware, which you described, and "error" middleware, which has a different signature:
function ($error, $request, $response, $next)

Unfortunately, the arguments are not equivalent, which raises a bit of a concern -- you can pipe() either to the middleware queue, and Next takes care of determining whether or not they are valid for the current context.
I'm playing around with ideas right now, and will have a PR for you to look at soon.

are we going to have a chance to talk in person again soon? or maybe a video chat?

Will you be at SunshinePHP? Otherwise, we can do video chat anytime after your return from Benelux.

what i'd like to see is trying to leverage stack as a thing that people may already be associating with middlewares and build out a hub for "php middlewares."

I really like this idea. There's a ton of middleware out there, between the StackPHP and Slim communities, and soon to be flooded with stuff from the Laravel community (and, if all goes as planned, the ZF community). If they can all use PSR-7, or bridges from PSR-7 to the specific HTTP message types can be built, I think we'll see some very cool cross-pollination.

Please see #29

authors: mnapoli simensen weierophinney

repo: phly/conduit


text:
[Android] deviceId persists between uninstalls, but is invalid after an uninstall.

Expected Behaviour

Application registers with GCM, gets a deviceId, which can be used to push notifications.
User uninstalls Application at some point, and GCM invalidates the deviceId, and the deviceId is discarded from the device's storage, since it's no longer valid (see https://developers.google.com/cloud-messaging/registration )
User reinstalls Application, which registers anew with GCM, and gets a new deviceId, which can be used to push notifications.

Actual Behaviour

Application registers with GCM, gets a deviceId, which can be used to push notifications.
deviceId is stored in Shared Preferences, which persist between uninstalls.
User uninstalls Application at some point, and GCM invalidates the deviceId, but Shared Preferences (along with the now-invalid deviceId) persist.
User reinstalls Application, the Phonegap Push Plugin sees the old invalid deviceId, so it does not re-register with GCM. Device is unable to receive notifications.

Reproduce Scenario (including but not limited to)

An Android user has installed the app at one point, uninstalled it, and after some time re-installs the app.

Steps to Reproduce

Install an application which registers with Google Cloud Messaging onto an Android device.
Uninstall the application after it registers with GCM.
Allow some time for the deviceId to be invalidated (Google notes that it's not an immediate step; however, this time period is certainly long enough for a user who uninstalled the app to later decide to re-install).
Re-install the application, and see that the old, invalid deviceId is returned and a new GCM registration is not requested.

Platform and Version (eg. Android 5.0 or iOS 9.2.1)
Android, all (to my knowledge -- observed on 7.0 and 6.x)
(Android) What device vendor (e.g. Samsung, HTC, Sony...)
All; observed on Nexus 6P and Nexus 6.
Cordova CLI version and cordova platform version
cordova@6.2.0
cordova-android@5.0.0

Plugin version
phonegap-plugin-push 1.8.2 "PushPlugin"

Sample Push Data Payload
Sample Code that illustrates the problem
Logs taken while reproducing problem

@spencerwi thanks for the excellent bug report.

No problem! Just running into this issue in my employer's app, and hoping to minimize/correct its impact on users.

I think Shared Preferences is not persistent between uninstalls.
Please correct me if I am wrong.

Any news in this issue? I can't send push notifications to users that have reinstalled the app.

@malwatte Ah, okay. So the backups are what's causing the cache to persist across uninstalls? I can't seem to figure out why, but somehow the same deviceId is being shortcut-fetched from cache (no web request, same deviceId returned) even after I uninstall and reinstall.

@spencerwi I had faced similar issue relevant to Shared Preferences. And it was not about this plugin. I assumed cause for this issue is the same.
I am using this plugin as well and I have not tested uninstall and installed in a Android M device. Please let me know if setting android:allowBackup="false" worked for you.
Thanks.

@malwatte Thanks! I'll give that a shot to see if that helps.

@spencerwi
Tested in a Galaxy S5 (Android 6.0.1).  App gets new deviceId after uninstall and install. No issue. Did not even set android:allowBackup="false".
Hmmm. Weird.
Steps I followed

Installed the app and got the deviceId
Sent a push. Push received
Uninstalled the app.
Sent a push. Got "NotRegistered" error from GCM
Installed the app again. A new deviceId received
Sent a push. Push received

Maybe it's a device/version-specific thing; I tested on a Nexus 6p running Android 7.0.

I tested App reinstall on 3 devices:

Motorola Moto X 2014 with Android 6.0: Push stop working
Samsung Galaxy S6 Edge with Android 6.0.1: Push stop working
Samsung Galaxy S5 Neo with Android 6.0.1: Push has not stop working

I made a hook to add the android:allowBackup="false" in the AndroidManifest application tag and will test on production this week.

Any updates?

I can confirm that after adding android:allowBackup="false" as explained above, the two devices that had problems receiving notifications after a reinstall are now receiving them again.

@allanpoppe ,
Good to hear that it fixed. Appreciate if you can share the hook.
Thanks

@spencerwi @malwatte okay, as part of 1.8.4 I removed the code that persists the registration ID to shared preferences. It will request a token from GCM each time the init method is called now instead of checking for a cached version.
You won't need to turn off backup in your app with that version of the plugin.

authors: allanpoppe macdonst malwatte spencerwi

repo: phonegap/phonegap-plugin-push


text:
force-start issue on current master, with android

If the app is closed, and you send a notification, and if the screen is turned on, then it notifies and then it opens the app to the foreground.  Ok.
However, if the screen is turned OFF, and the app is not running, then you send a notification, it notifies and then starts the app in the foreground (with the screen still turned off), so all future notifications will NOT sound a notification because the app is acting like it s running in the foreground even though the screen is off.  If you turn on the screen, go into the app and then go back to the homescreen, then the app is background and it works again.  The on 'notify' still works, just no system notification shows up.
This is a problem because after the first notification starts the app, you will never get more notifications until you go to the app and then back out because it thinks the app is in the foreground after that first notification starts the app while the screen is turned off.
Android 6.0
phonegap-plugin-push 1.9.2 from npm and also from current github master

Sorry to be a jerk but I'm now closing issues that don't follow the issue submission guidelines. This plugin takes a lot of work to maintain and I can't afford the time required to solicit the required info from everyone.
I'm closing this issue but please feel free to re-open the issue if you provide the requested details from the new issue submission guidelines.
https://github.com/phonegap/phonegap-plugin-push/blob/master/.github/ISSUE_TEMPLATE.md

authors: macdonst uj


text:
Windows notifications (WP 8.1) - Unable to connect to WNS from client mobile app

From my windows mobile application, I want to implement the windows notifications.
I just created a simple basic cordova project generated by Visual Studio 2015 RC. I added only two plugins : cordova-plugin-device and phonegap-plugin-push of course.
My mobile application is based on :

Cordova (version 4.0.0)
Cordova Windows (version 4.1.0)
Deploy on Windows Phone 8.1

To implement this feature, I'm using the cordova plugin "phonegap-plugin-push" (version 1.5.2).
When my mobile application is launched for the first time, I ask to my application to connect to the notification client platform. The goal is this notification client platform ask to WNS to return a new notification channel (URI form) to my application.
When I use the method "registration" from PushNotification class :
    var push = PushNotification.init({
        "windows": {}
    });

    push.on('registration', function(data) {
        console.log("registration event");
        console.log("URI : ", data.registrationId);
        console.log(JSON.stringify(data));
    });

    push.on('notification', function(data) {
        console.log("notification event");
        console.log(JSON.stringify(data));

        push.finish(function () {
            console.log('finish successfully called');
        });
    });

    push.on('error', function(e) {
        console.log("Push error : ", e);
    });

I get this error :

Push error : The notification client platform is unable to connect to the cloud
notification service

I thought the notification client platform is currently under maintenance. I found a file where are IP address ranges used by WNS. But when I ping differents IP address ranges and it works.
Is there anyone who already face this problem ? Or is there anyone who have an idea about how to fix it ?

@charpin: Do you get an error code in return? These are the common errors encountered during connecting with the WNS. Can you verify the data service is active on your device?
https://msdn.microsoft.com/en-us/library/windows/apps/windows.networking.pushnotifications.pushnotificationchannelmanager.createpushnotificationchannelforapplicationasync

I don't remember the error code in return. I think it was this error code : 0x800704C6
But thanks to you, I found the problem. I had not checked on my Windows Phone that data service is disabled. I thought by default that data service was activated. So, I activated the data service from my Windows Phone and the push notification works again.

authors: charpin rakatyal


text:
Non-anlytic correction calculation issue

Hi There,
I have an issue when I use phonopy-vasp-born, the error message is below:
phonopy-vasp-born
Traceback (most recent call last):
File "/home/tl236/Apps/anaconda3/envs/phonopy/bin/phonopy-vasp-born", line 10, in
sys.exit(run())
^^^^^
File "/home/tl236/Apps/anaconda3/envs/phonopy/lib/python3.12/site-packages/phonopy/scripts/phonopy_vasp_born.py", line 154, in run
borns, epsilon, atom_indices = get_born_vasprunxml(
^^^^^^^^^^^^^^^^^^^^
File "/home/tl236/Apps/anaconda3/envs/phonopy/lib/python3.12/site-packages/phonopy/interface/vasp.py", line 495, in get_born_vasprunxml
return elaborate_borns_and_epsilon(
^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/home/tl236/Apps/anaconda3/envs/phonopy/lib/python3.12/site-packages/phonopy/structure/symmetry.py", line 540, in elaborate_borns_and_epsilon
assert len(borns) == len(ucell), "num_atom %d != len(borns) %d" % (
^^^^^^^^^^^^^^^^^^^^^^^^
AssertionError: num_atom 64 != len(borns) 0
As you noticed that I use 64 atom supercell for VASP calculation. I saw an option --pa in the reference page to resolve the supercell issue for NAC, could you provide with more detailed command to resolve this issue? Thanks a lot!
Best Regards,
Tingguang

Please send your question to the phonopy mailing list.

authors: atztogo tingguangli

repo: phonopy/phonopy


text:
Chrome may suspend the Web Audio context absent a user gesture

An AudioContext must be created or resumed after the document received a user gesture to enable audio playback.

I get this warning in Chrome Canary (M65) on, eg, http://phaser.io/examples/v2/audio/fade-in (iframe), and on my localhost (no iframe) when I open a game tab in the background and then focus it. But I don't get it when doing the same on https://samme.github.io/phaser-examples-mirror/audio/fade in.html. Not sure why.
The audio context is created but with a suspended state. Playback appears to go normally but there's no audio output.
A workaround would be
if (game.sound.usingWebAudio &&
    game.sound.context.state === 'suspended')
{
  game.input.onTap.addOnce(game.sound.context.resume, game.sound.context);
}

The AudioContext was not allowed to start. It must be [resumed] (or created) after a user gesture on the page. https://goo.gl/7K7WLu

https://samme.github.io/phaser-examples-mirror/audio/fade in.html

Phaser can try to resume the audio context via mouseout, which is wrong and will fail.

repo: photonstorm/phaser-ce


text:
Support for trackpad multitouch TUIO

This is a great feature when developing html5 touch apps/games on the desktop, emulating multi touch with your trackpad.
However there seems to be some problem with phaser compatibility. It doesn't register multi-touch events inside the phaser canvas with hammer.js. It works fine on the iPad however. And the tiou demo works also fine in firefox on the mac with the touchpad. Any ideas?
http://www.html5rocks.com/en/mobile/touch/#toc-touchdev
https://www.youtube.com/watch?v=6wL-pLX9Kq0

Phaser supports multi-touch natively, we just don't enable the extra Pointers because most games don't need them, but it's a single call to do so. I would imagine throwing hammer into the mix would seriously confuse it. Also try this against dev branch, we made a recent change in there that allowed Chrome touch emulation to work, may support this too.

Hello Photon,
Thanks for the info, my understanding was that phaser does not recognize multi-touch gestures ?
I was using hammer mainly so it would detect gestures and i wouldnt have to go through the process of doing this myself.
http://www.html5gamedevs.com/topic/8762-zoom-outin-camera-as-seen-in-angry-birds/
Maybe i should strip out the gesture recognition from hammer js, and use it with the phaser inputs?

Hello Photon,
Thanks for the info, my understanding was that phaser does not recognize multi-touch gestures ?
I was using hammer mainly so it would detect gestures and i wouldnt have to go through the process of doing this myself.
http://www.html5gamedevs.com/topic/8762-zoom-outin-camera-as-seen-in-angry-birds/
Maybe i should strip out the gesture recognition from hammer js, and use it with the phaser inputs?

Ok, i have fixed the integration with Hammer, you can see it working here on iPad / iPhone
http://www.tabloro.com/tables/Testin/play
You can drag 1-4 tiles simultaneously with 1-4 fingers which is handled by phaser.
If you use 2 fingers on the table you can pan/zoom with pinch, which is handled by hammer.js.

So what do I need to do? :) Is there a PR coming with a fix in it, or did you just manage to get it to work anyway?

Was more fyi :)
Hammer js works fine alongside phaser if you let them handle separate events.
The TUIO trackpad multitouch demo from paul still doesn't work in phaser. (It's also outdated so it works only in Firefox at this time.) Maybe i will look into this a bit more to see if this can be made to work with phaser, it would be convenient for testing touch games.
http://www.paulirish.com/demo/multi

To be honest I'd rather find a way to not need Hammer at all. Gestures plugin anyone? :)

authors: fyyyyy photonstorm

repo: photonstorm/phaser


text:
game.time.slowMotion should lerp

Love what you're doing with this framework. Noticed that slowMotion makes the game feel like it's lagging (see here). Would be cool if the positions of game objects lerped between the previous and current when slowMotion is used so you still get a nice 60fps feel.

When using slowMotion you should increase desiredFps such that
60 === game.time.desiredFps / game.time.slowMotion

Seems like a non-obvious api. Perhaps this should be in the documentation for game.time.slowMotion ?

I have hunch the lag (#2631) was a bug introduced by some other change in the game loop, and this is a non-obvious workaround. :)

Thank you for taking the time to open this issue. However, official support for Phaser 2 has now ended as we focus on preparing Phaser 3 for beta release. In a bid to keep the project tidy, and avoid confusion, we are closing all v2 related issues.
This does not mean your issue won't be looked-at. In November 2016 the Phaser CE (Community Edition) project began. The aim of Phaser CE was to allow the community to continue v2 development. Since then it has gained significant traction, with many updates and releases.
If you are still interested in having your issue investigated we strongly recommend testing the latest Phaser CE release. And only if the problem persists, opening an issue on the Phaser CE repo.

authors: nickaversano photonstorm samme


text:
tilemap layer render bug

bandicam 2017-04-20 15-43-59-392.zip
         game.camera.bounds = null;
         this.zoomgrp = game.add.group();
         this.map = game.add.tilemap();
         this.layer = this.map.create('minemap', 100, 100, 16, 16);
         this.zoomgrp.add(this.layer);

         ....

         this.zoomgrp.scale.set(2);

and drag mouse (move camera position)
please view video.

Thank you for taking the time to open this issue. However, official support for Phaser 2 has now ended as we focus on preparing Phaser 3 for beta release. In a bid to keep the project tidy, and avoid confusion, we are closing all v2 related issues.
This does not mean your issue won't be looked-at. In November 2016 the Phaser CE (Community Edition) project began. The aim of Phaser CE was to allow the community to continue v2 development. Since then it has gained significant traction, with many updates and releases.
If you are still interested in having your issue investigated we strongly recommend testing the latest Phaser CE release. And only if the problem persists, opening an issue on the Phaser CE repo.

authors: photonstorm pocketoy


text:
UpdateList for 3.17.0

remove: function (child)
{
    // I think here should be _list.indexOf(child)? or perhaps check both just like the add method does?
    // like this: if (this._list.indexOf(child) !== -1 && this._pendingRemoval.indexOf(child) === -1)
    // {
    //     this._pendingRemoval.push(child);
    // }

    var index = this._pendingRemoval.indexOf(child);

    // or if use _pendingRemoval to check duplications, here should be === -1?
    if (index !== -1)
    {
        this._pendingRemoval.push(child);
    }

    return child;
}

please read the comments, so is this a mistake or I misunderstood something?
because I'm not quite clear of the purpose, so can't make a PR, sorry.

I would say this is a bug. I have found the same, because of performance problems, after my game is running for a while. And in my game there are many objects created and removed again.
Can somebody please fix this? This is also in 3.18.1.

@jcyuan Can you change the title to something more explaining, like "UpdateList not working", so it is better visible. And changing the description, to have everything from the create issue template, so it is processed, because if the template is not used, the issue will be ignored. Because I don't want to open a duplicate issue.

@CipSoft-Components — Can you explain the bug, i.e. what's the defective behavior and how it can be produced?

@samme Explanation:
Defective behavior: The UpdateList will not remove objects, when its remove method is called. When I understand the code correctly, the remove method only adds the element to a "ready to remove" list, which will be removed soon (next update step). But the method first looks up, if the element is already in this "ready to remove" list, to not add it multiple times. The problem now is, that the condition to check this, is wrong, and always evaluates to false, so that the element never is added to the "ready to remove" list.
How to reproduce:
Create some GameObjects, destroy them with <GameObject>.destroy() (which should remove them from the UpdateList), and then look into the elements in the UpdateList after some updates occured, The objects are remaining in the list, and also its "ready to remove" array (_pendingRemoval) is empty.
Main problem from that behavior: When you have a long running scene, with many created and destroyed objects, the performance goes down extremely after some time.

@CipSoft-Components sure, will do.
@photonstorm this should be a coding mistake for fixing #4365 which was reported by me too.
this is the existing code since 3.1.7.0:
https://github.com/photonstorm/phaser/blob/34bf26592e922ca2ef53d76753dfa5eccf97ac13/src/gameobjects/UpdateList.js#L217-L227
as you can see, if my object is in active statu, it can't be removed at all!
really easy to fix (I patched for my own project locally)
way1: just like the way the add method does:
if (this._list.indexOf(child) !== -1 && this._pendingRemoval.indexOf(child) === -1) {
        this._pendingRemoval.push(child);
}

way2: change the condition
modify https://github.com/photonstorm/phaser/blob/34bf26592e922ca2ef53d76753dfa5eccf97ac13/src/gameobjects/UpdateList.js#L221
to
 if (index === -1)

but personally way 1 is better and recommended.

I'm curious why people not report this problem as this topic is already here for a little while...
guys don't remove things in their game? :rofl:

I think it depends on the games. Perhaps, many of the games don't have long running scenes, with very much unique objects, which they destroy manually. If a scene is shutdown, everything will be cleared up. And if you have many nearly identically objects you use groups, and don't destroy so many objects manually.

yep, it does make sense, my game is a bit heavy too, so found this problem before.

Thank you for submitting this issue. We have fixed this and the fix has been pushed to the master branch. It will be part of the next release. If you get time to build and test it for yourself we would appreciate that.

authors: CipSoft-Components jcyuan photonstorm samme


text:
Debug Colliding tiles throw an error: t.translate is not a function.

Version

Phaser Version: 3.18.0 and 3.18.1

Operating system: Windows 7

Browser: No browser related

Description
When trying to debug tiles collisi;on with renderDebug() throw the error: t.translate is not a function.

Example Test Code
When I look for examples to check if the way to debug has changed, I saw that the example fails too, so it works as an test code:
https://phaser.io/examples/v3/view/game-objects/tilemap/static/debug-colliding-tiles
Additional Information
The weird thing is that it works with: 3.17.0 and with "Development build" version.
The error happen in the renderDebug function,
graphics.translate(layer.tilemapLayer.x, layer.tilemapLayer.y);
here:
`var RenderDebug = function (graphics, styleConfig, layer)
{
if (styleConfig === undefined) { styleConfig = {}; }
// Default colors without needlessly creating Color objects
var tileColor = (styleConfig.tileColor !== undefined) ? styleConfig.tileColor : defaultTileColor;
var collidingTileColor = (styleConfig.collidingTileColor !== undefined) ? styleConfig.collidingTileColor : defaultCollidingTileColor;
var faceColor = (styleConfig.faceColor !== undefined) ? styleConfig.faceColor : defaultFaceColor;

var tiles = GetTilesWithin(0, 0, layer.width, layer.height, null, layer);

graphics.translate(layer.tilemapLayer.x, layer.tilemapLayer.y);
graphics.scale(layer.tilemapLayer.scaleX, layer.tilemapLayer.scaleY);`

Thank you for spending time reporting this issue. However, we've already fixed this in the master branch of Phaser and it will be part of the next official release. You can track what's already been fixed by searching closed issues, or by looking at the Change Log.

authors: Kailcor photonstorm


text:
Fix TimeStep#wake() when running

This PR

Fixes a bug?

Currently, if you call TimeStep#wake() while the loop is running it calls TimeStep#sleep() and then continues waking, which produces an extra loop step.
I changed it so that if you call TimeStep#wake() while the loop is running, nothing happens.

👍

authors: photonstorm samme


text:
Update Documentation

Hi there! I just installed this promising library, but the documentation seems to be outdated. Nor Gettext\Scanner\PhpScanner class anymore, for example. Are you planning on updating the documentation?

Sorry, Just noticed that I just hadn't required the gettext/php-scanner package. Closing.

repo: php-gettext/Gettext


text:
Minor fixes that address comments made in #66

Ref #66

thanks

Thank you for merging

authors: Nyholm dbu

repo: php-http/discovery


text:
FEATURE: Add testcase

Context
With this test case, I'm reproducing the problem that I'm facing. If the request matcher is ignoring some parts of the request making them the same, those requests should increment the same key in the $indexTable.
What has been done

Failing test explaining the problem that I have

How to test

Run the test

Notes
If you are open for accepting it, I could submit a fix for this issue.

Hi @dam1r89
thx 4 contributing! I think i understood the problem.
What do you think about changing or implementing a new request matching api to return something like a hash (same as Request::getHash() does before) to still allow using of an index-table instead persisting and matching all requests again and again when a new arrives?
I'm asking because on long running processes this might get a problem.
What do you think?

At the moment, I am occupied with other tasks and cannot say when I will have the time to review this.
I  am curious about the scale of the tests you referenced - are they limited to hundreds of requests?
It may not be necessary to pursue optimization unless it becomes a significant issue, as the majority of tests will likely not generate a large volume of requests.

Hey @higidi, any update on this? Just to let you know, we are currently using the version with this patch without any problems on a fairly large code base.
I wanted to check in to see if you would be open to accepting another PR with an additional feature that I need.

repo: php-vcr/php-vcr


text:
🛑 phpMyFAQ Demo is down

In 9c791b5, phpMyFAQ Demo (https://demo.phpmyfaq.de) was down:

HTTP code: 0
Response time: 0 ms

Resolved: phpMyFAQ Demo is back up in cb9177f.


text:
🛑 phpMyFAQ API is down

In 5bee3c7, phpMyFAQ API (https://api.phpmyfaq.de/versions) was down:

HTTP code: 0
Response time: 0 ms

Resolved: phpMyFAQ API is back up in c971b01.


text:
🛑 phpMyFAQ API is down

In d275351, phpMyFAQ API (https://api.phpmyfaq.de/versions) was down:

HTTP code: 0
Response time: 0 ms

Resolved: phpMyFAQ API is back up in a3915d9.


text:
🛑 phpMyFAQ API is down

In 9ba7c90, phpMyFAQ API (https://api.phpmyfaq.de/versions) was down:

HTTP code: 0
Response time: 0 ms

Resolved: phpMyFAQ API is back up in 255202a.


text:
🛑 phpMyFAQ Demo is down

In f937ee4, phpMyFAQ Demo (https://demo.phpmyfaq.de) was down:

HTTP code: 0
Response time: 0 ms

Resolved: phpMyFAQ Demo is back up in d175de9.


text:
🛑 phpMyFAQ Demo is down

In fcd277b, phpMyFAQ Demo (https://demo.phpmyfaq.de) was down:

HTTP code: 0
Response time: 0 ms

Resolved: phpMyFAQ Demo is back up in 1c2f821 after 4 minutes.


text:
🛑 phpBB.com is down

In ec7ab7f, phpBB.com (https://www.phpbb.com) was down:

HTTP code: 503
Response time: 262 ms

Resolved: phpBB.com is back up in 4f20d2a after 29 minutes.

repo: phpbb/status-site


text:
[Share] Can't switch PHP version from 5 to 7 with Apache2 on Ubuntu

Action

Install PHP7 by phpbrew succesfully (It might required at least 1GB Memory during the installation compiler process; fail installation on Digital Ocean 512MB enviroment)
Restart Apache 2.0

Output
The Apache would still using PHP5
Expected Result
It should display PHP7
Platform
OS: Ubuntu 14.04
Running PHP: 5.0
Installing PHP: 7.0-dev
Solution
sudo apt-get install libapache2-mod-php7.0
sudo a2dismod php5.0 ; sudo a2enmod php7.0 ; sudo service apache2 restart

It's the solution and it's not the issue.

repo: phpbrew/phpbrew


text:
json parser

not just .inis

At first I thought json_decode would be enough, but there is actually a really nice, fully tested OO library  here for JSON related things: https://bitbucket.org/hampel/json.git
Consider using for parser.

repo: phpgt/config


text:
Read source code from StdInput

It would be very useful to validate the code in a Git pre-commit hook.
Something like this:
git show :foo.php | phpmd ...
I tried the following, but the exit code was 0 without any output with a PHP file which contains several errors.
git show :foo.php | bin/phpmd php://stdin text naming
The problem is that the \PDepend\Engine::createFileIterator() iterates over the $this->files which contains a file path like this php://stdin, but that file path is added to the $fileIterator like this:
$fileIterator->append(new Iterator(new \GlobIterator($file), $this->fileFilter));,
and later the \PDepend\Input\Iterator::getFullPath() is called, which calls the ::getRealPath() method of the current item, which is an instance of \SplFileObject, and it always returns false.
In short this is the problem:
$item = new \SplFileObject('php://stdin');
$path = $item->getRealPath(); // $path === false

pdepend/pdepend#361

Note than the  alias - can be used instead of php://stdin:
echo ... | bin/phpmd - text naming

And the support of php://stdin should be fixed in the next release of PDepend.

authors: Sweetchuck kylekatarnls

repo: phpmd/phpmd


text:
Process ends when all tokens are consumed

To quote from the BPMN specification: "To continue discussing how flow proceeds throughout the Process, an End Event consumes a token that had been generated from a Start Event within the same level of Process. If parallel Sequence Flows targets the End Event, then the tokens will be consumed as they arrive. All the tokens that were generated within the Process MUST be consumed by an End Event before the Process has been completed." (page 245 from "Business Process Model and Notation (BPMN), v2.0.2")
Basically we have to change the workflow so that when "end" is called we consume the tokens as they arrive. When the last one is consumed we could set the "endDate" as a property on the workflow and also have a "closed" property on the Workflow class.
At the moment "isClosed" is actually checking if all remaining tokens are "End" events.
What do you think @iteman ?
I could push a PR to solve this.

Another quote from the BPMN 2.0 specification (page 426):
"Each Start Event that occurs creates a token on its outgoing Sequence Flows, which is followed as described by the
semantics of the other Process elements.
A Process instance is completed, if and only if the following three conditions hold:

If the instance was created through an instantiating Parallel Gateway, then all subsequent Events (of that
Gateway) MUST have occurred.
There is no token remaining within the Process instance.
No Activity of the Process is still active.
For a Process instance to become completed, all tokens in that instance MUST reach an end node, i.e., a node without
outgoing Sequence Flows. A token reaching an End Event triggers the behavior associated with the Event type, e.g.,
the associated Message is sent for a Message End Event, the associated Signal is sent for a Signal End Event, and
so on. If a token reaches a Terminate End Event, the entire Process is abnormally terminated."

Maybe we should consider implementing the Terminate End Event too.

Fixed in https://github.com/phpmentors-jp/workflower/pull/75

repo: phpmentors-jp/workflower


text:
[Request] Auto accept Y on --use-local-branch-name

Currently cli is used in CI and we would like to have auto accept questions like:
Should we create a new branch in Phrase with the same name and push to it? [y/N]:

The command is used like so and there is no way to skip interactive mode
./phrase_linux_amd64 push --use-local-branch-name -w

You have currently checked out the branch 'feature/translations-using-branches'.
There currently is no branch in Phrase with this name.

Should we create a new branch in Phrase with the same name and push to it? [y/N]:

Thanks for the suggestion. That sounds like a quite useful feature. I can't promise yet when we'll add this but I'll keep you posted here when I have any news to share.

As a workaround in Bickucket piple step currently I'm using like so:
echo "Y" | ./phrase_linux_amd64 push --use-local-branch-name -w | grep --line-buffered -Po "Upload Id: \K\w+" | xargs -I{} echo ./phrase_linux_amd64 uploads cleanup -y --branch ${BITBUCKET_BRANCH} --id {}

just a reference for others

We have decided to close all GitHub issues and, instead, handle them through our Customer Support. We think this shift will allow us to offer proper prioritization of issues as well as a more appropriate response time that would be expected from Phrase products.
We apologize for any inconveniences this may cause. If you believe your issue is still relevant or has a high priority, please contact support@phrase.com, and we can take more direct action toward finding a solution.

authors: FDiskas egze theSoenke

repo: phrase/phrase-cli


text:
Adding WebViews to IndexedStack and Hybrid Composition not working

[x] I have read the Get Started - Installation section
[x] I have read and done the Get Started - Setup Android section
[x] I have read and done the Get Started - Setup iOS section
[x] I have already searched for the same problem

Environment

Technology
Version

Flutter version
2.0.4

Plugin version
5.3.2

Android version
11

iOS version
------

Xcode version
------

Device information:
Description
Hi everyone, I'm having this problem with Stacked Webviews (IndexedStack) and useHybridComposition: true (I need this option enabled since the keyboard has many problems if im not using hybrid composition).
I have an IndexedStack that contains an array of views that contain an InAppWebView inside, when using hybridComposition and switching/adding new webviews (adding new views to the array) the webview stops working and shows blank screens.
This does not happen when using hybridComposition: false
Expected behavior:
useHybridComposition: false

Current behavior:
useHybridComposition: true

Steps to reproduce
I've built a simple app to reproduce this behaviour.
import 'package:flutter/material.dart';
import 'package:flutter_inappwebview/flutter_inappwebview.dart';

void main() {
  runApp(MyApp());
}

class MyApp extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      theme: ThemeData(
        primarySwatch: Colors.blue,
      ),
      home: MyHomePage(title: 'Flutter Demo Home Page'),
    );
  }
}

class MyHomePage extends StatefulWidget {
  MyHomePage({Key key, this.title}) : super(key: key);

  final String title;

  @override
  _MyHomePageState createState() => _MyHomePageState();
}

class _MyHomePageState extends State<MyHomePage> {
  int _index = 0;
  final bool useHybrid = true; //change to false and this works fine
  final GlobalKey webViewKey = GlobalKey();
  final GlobalKey webViewKey1 = GlobalKey();

  InAppWebViewController webViewController;

  List<InAppWebView> webviews = <InAppWebView>[];

  @override
  void initState() {
    super.initState();

    webviews = [InAppWebView(
      key: webViewKey,
      initialUrlRequest: URLRequest(url: Uri.parse("")),
      initialOptions: InAppWebViewGroupOptions(
        android: AndroidInAppWebViewOptions(
          useHybridComposition: useHybrid
        )
      ),
      onWebViewCreated: (InAppWebViewController webViewController) {
        webViewController.loadUrl(urlRequest: URLRequest(url: Uri.parse("https://inappwebview.dev/")));
      },
    )];
  }

  void _switchWebview() {
    if (_index == 0) {
      setState(() {
        if (webviews.length == 1) {
          List<InAppWebView> clone = <InAppWebView>[]..addAll(webviews);
          clone.add(InAppWebView(
            key: webViewKey1,
            initialUrlRequest: URLRequest(url: Uri.parse("https://google.com/")),
            initialOptions: InAppWebViewGroupOptions(
              android: AndroidInAppWebViewOptions(
                useHybridComposition: useHybrid
              )
            ),
          ));

          webviews = clone;
        }

        _index = 1;
      });
    } else {
      setState(() {
        _index = 0;
      });
    }
  }

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        title: Text(widget.title),
      ),
      body: Container(
        child: SafeArea(bottom: false, right: true, child: IndexedStack(
          index: _index,
          children: webviews
        ))
      ),
      floatingActionButton: FloatingActionButton(
        onPressed: _switchWebview,
        tooltip: 'Switch',
        child: Icon(Icons.switch_left),
      ),
    );
  }
}

I don't see any errors on the console that could guide me in any directions.

Thanks in advance! Any suggestions or help is welcomed.

Probably, it's related to https://github.com/pichillilorenzo/flutter_inappwebview/issues/704.
If you try Flutter version 1.27.0-4.0.pre or lower, does it work?

Probably, it's related to #704.
If you try Flutter version 1.27.0-4.0.pre or lower, does it work?

Hi Lorenzo, thanks for the quick reply.
I tried switching back to 1.27.0-4.0.pre but sadly this doesn't seem to fix the issue :(

I am also facing the same issue :-(

authors: PawanFyers juuliaans pichillilorenzo


text:
Consider adding disk quota per competitor user

It seems prudent to add a disk quota to each competitor account user even though everyone has been nice.
P.S. Some of our own "problem users" (users that are created due to problems) may actually need disk space. So we should not impose disk quota on them.

Someone thought it'd be helpful to fill up 2018 server with 300GB of generated files.

https://bugs.launchpad.net/ubuntu/+source/linux-aws/+bug/1773172 hmm...
https://www.howtoforge.com/community/threads/ubuntu-18-04-aws-ec2-disk-quotas.79445/ removing aws tuning sounds iffy

b403a06b4dbb35a069c929dca166e12af44f7e3f automates the application of quota per user, assuming disk quota setup already

authors: hiliang-cmu maverickwoo

repo: picoCTF/picoCTF


text:
dotted line if tooltip is added

Describe the issue
Current Behavior

                {%
                    include 'flaticons/bright.svg' %}
                    <input name="terms" class="theme-toggle" id="theme-toggle" type="checkbox" role="switch"
                        checked />{%
                    include 'flaticons/moon.svg' %}

produces a dotted line below the element.
data-tooltip="Tooltip" within the <input> element destroys formatting completely.
Expected Behavior
Only tooltip added

Are you using the newest version of Pico here? Cause https://github.com/picocss/pico/blob/6dc6489e69cc1f22daa938ebd38c02400032428c/scss/components/_tooltip.scss#L12-L16 prevents the dotted line on inputs in the current release of Pico here.

Yes, I'm using the latest version (2.0.6). Following your hint I added style="border-bottom: none;" to the element locally as a patch.

I was able to duplicate your issue, and found a fix which I included in my fork of PicoCSS.
The main change for your issue is to update that line I pointed out previously to the follow:
&:not(:has(a, button, input, [role="button"])):not(a, button, input, [role="button"]) {

authors: Kugeleis Yohn

repo: picocss/pico


text:
Q: Source of MHC proteins

Hi TCRmodel2 developers,
Apologies for the slightly off-topic question: may I ask what the source of the MHC I sequences (e.g. GSH..TLQ for HLA-A*23:01) on the website is?
Many thanks,
Andreas

Hi @andreas-wilm ,
We got the MHC I sequences from IMGT database:
https://www.ebi.ac.uk/ipd/imgt/hla/download/
They also have a github repo for convenient downloading of their files:
https://github.com/ANHIG/IMGTHLA
Regarding the specific gene you are referring to (HLA-A*23:01), you can find it here:
https://raw.githubusercontent.com/ANHIG/IMGTHLA/3540/hla_prot.fasta
Best,
Rui

No problem, Andreas, happy to help! We extracted the alpha 1 and 2 domains of Class I MHC using a hidden Markov model built from a multiple sequence alignment containing alpha 1 and 2 domains of Class I MHC sequences. You can refer to the trim_mhc function in seq_utils.py to see how the processing is performed.
Best,
Rui

Oh, it's all in ./scripts! Wonderful. Thank you very much!

authors: andreas-wilm rui-yin


text:
Launched App Do Not Open Window On Tap

Describe the bug
When an app is launched but without any open windows, Pock will switch to the app on tap but will not open the primary window for the app. This is expected behavior from a dock action.
How to reproduce
Steps to reproduce the behavior:
1. Add Chrome to the dock and Pock
2. Launch Chrome (or any apps that do not quit when the last window is closed)
3. Close all Chrome windows but leave Chrome running
4. Use another application (have another app as focus)
5. Click on the Chrome icon in Pock touch bar
6. Chrome will be the app in focus in the menu, but no windows will open

Expected behavior
What should happen in step 6 above is that a new Chrome window should open. This is the default behavior if the same thing happens for an app and the app icon is clicked on in the dock.
Versions:

macOS version: 10.14.5
Pock version:  0.5.3

I have the same issue. When the app is on but no windows active, tapping this app on Touch Bar shows nothing. Just like using (command + tab) to switch to a app without any windows, it seems that it don't creative a new window.

Same issue here. I can reproduce with:

Slack
Calendar
and pretty much any app that can be open without displaying any window.

This is a bug introduced with the initial multi-window support that is currently partially implemented.
I hope to publish a fully-functional release this weekend! 🤞

Any progress on this one? Seems to be happening for me, too. Would love to replace my on-screen Dock with Pock moving forward! Thank you for the awesome project.
macOS 10.14.6
Pock 0.5.3

Bug

Bug

would love to see a progress here, this affects all windows if i have closed them :)

This is a bug introduced with the initial multi-window support that is currently partially implemented.
I hope to publish a fully-functional release this weekend! 🤞

I'm working on an update that is coming soon that will introduce multi-window support.

authors: benoitzohar elimuhub josephsiu lucien-guoheng pigigaldi robertdistefano tofixx

repo: pigigaldi/Pock


text:
Able to access folders via finder widget even when screen is locked.

Description
I am able to navigate my folder system and open files via the touchbar when my computer is locked. This is a major security concern, I can work around it by disabling file navigation with pock, but would like to use that feature still.
How to reproduce
Enable the finder widget and then lock PC
Expected behavior
File system should not be browsable and opening files should definitely not be possible when the screen is locked.
Versions:

macOS version: 10.14.6
Pock version:  0.6.1

Additional context
Pock Settings
General

Hide Control Strip = True
Lock at login = True
Check for updates once a day = False
Dock Widget
Notification badge refresh rate = Instantly
Open app expose = More Than one Window
Hide Finder = False
Show only running apps = False
Hide persistent items = False
Hide trash = True
Opens Finder and directories inside Pock = True

I would suggest generally disable pock when the mac is locked. Just show the default macOS touch bar when locked.


text:
🛑 Stripe Test Endpoint URL is down

In 519ea8b, Stripe Test Endpoint URL ($STRIPETESTAPI) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Stripe Test Endpoint URL is back up in f06c747 after 11 minutes.

repo: pikfarm/PikfarmStatus


text:
🛑 Stripe Endpoint URL is down

In 6897cd5, Stripe Endpoint URL ($STRIPEAPI) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Stripe Endpoint URL is back up in dbe36bd after 36 minutes.


text:
🛑 Blog URL is down

In 01f0d4e, Blog URL (http://blog.pik.farm) was down:

HTTP code: 429
Response time: 495 ms

Resolved: Blog URL is back up in 16ba674.


text:
remove constant columns from the design matrix

Those columns are useless for prediction, and completely collinear, redundant with bias term.
Make sure to keep the indices, so one can reconstruct the original form.

Hi Memming,
Sorry, I'm not sure I understand the comment -- was this a "note to self"?
-J
On Fri, Jan 23, 2015 at 2:19 PM, Il Memming Park notifications@github.com
wrote:

Those columns are useless for prediction, and completely co-linear,
redundant with bias term.
Make sure to keep the indices, so one can reconstruct the original form.
--
Reply to this email directly or view it on GitHub
https://github.com/pillowlab/neuroGLM/issues/1.

This is not a message to you. This is an issue tracker on github. I guess
you are subscribed to receive it as email.
On Fri, Jan 23, 2015 at 2:23 PM, Jonathan Pillow notifications@github.com
wrote:

Hi Memming,
Sorry, I'm not sure I understand the comment -- was this a "note to self"?
-J
On Fri, Jan 23, 2015 at 2:19 PM, Il Memming Park <notifications@github.com

wrote:

Those columns are useless for prediction, and completely co-linear,
redundant with bias term.
Make sure to keep the indices, so one can reconstruct the original form.
--
Reply to this email directly or view it on GitHub
https://github.com/pillowlab/neuroGLM/issues/1.

--
Reply to this email directly or view it on GitHub
https://github.com/pillowlab/neuroGLM/issues/1#issuecomment-71249472.

--
Assistant Professor
Department of Neurobiology and Behavior
Stony Brook University
http://memming.wordpress.com | memming@Twitter

authors: jpillow memming

repo: pillowlab/neuroGLM


text:
Just a fancy Conway's Game of Life example

I just wanted to give you guys this running example if you are interested.

I had to include the board_info.* source to be able to use the c implementation directly

What were you using board_info for? The library uses rpihw.c internally to determine board particulars: https://github.com/pimoroni/unicorn-hat/blob/master/library/rpi-ws281x/lib/rpihw.c

I've manually committed game_of_life.py with a reference to this PR. Thank you. Beautiful choice of colours and excellent execution. It's rather mesmerizing!

repo: pimoroni/unicorn-hat


text:
TLS 1.0 not supported by stripe

I'm using  python==2.7.11, pinax-stripe==3.2.1 with Django==1.9.6 and getting the following error when running: python manage.py sync_plans
stripe.error.AuthenticationError: Request req_8x3dWCHUrqcqGs: Stripe no longer supports API requests made with TLS 1.0. Please initiate HTTPS connections with TLS 1.2 or later. You can learn more about this at https://stripe.com/blog/upgrading-tls.

This seems more an issue of the python-stripe package.

Solved it after installing this libraries:
pip install urllib3
pip install pyopenssl
pip install ndg-httpsclient
pip install pyasn1

In case anyone else is having the same problem, I had to uninstall and reinstall the requests module after installing the 4 modules @gusreyes01 listed above.

Got same issue today with no longer supports TLS 1.0. Just updated mine Python (windows) from 2.7.2 to 2.7.12 and it works well now.

@gusreyes01 @irothschild @johndark thanks for the information.  not sure what the solution is yet to make it just work for folks but at least this thread is a start for people to fix it until we figure out what the solution is.

@gusreyes01 Thanks! That worked for me. I thought I was going to have to do much more based on the url that Strip provides.
Now I have to go see what all those packages do. I assume they forward all the cc data to you. :-)

@gusreyes01 Worked for me!! Thanks!

How about stripe.net? Not working even after upgraded to 8.0.0

@gusreyes01 LIFE SAVER! thanks

We should summarize this workaround/scenario in the documentation somewhere before closing this.

I updated to python 3.6.3.
pip3.6 is broke but pip3 works.
I did the following:
$ pip3 install urllib3
$ pip3 install pyopenssl
$ pip3 install ndg-httpsclient
Still getting "TLS 1.0 not supported by stripe"

authors: Jony-Y gusreyes01 irothschild johndark leehinde mesas998 paltman rguisando studiorooster

repo: pinax/pinax-stripe


text:
ucp logs: popup error prompt when bad request

UCP #113
add popup when request error.

Thanks for the contribution!

Need to handle the case that there is network failure, which does not have error?.response?.data?.message field.

Also need to handle task errors in the search progress page:

(You can simulate a task error by closing a TiDB and search logs over that TiDB.)

@breeswish hi, mentor, I extract a util function to handle search log page request error with message prompt.

@Deardrops @breeswish Right now prompt function receive a message param to show reasonable error message to user, PTAL

Here also need popup error prompt:
https://github.com/pingcap-incubator/tidb-dashboard/blob/318fbc0f575843679db7a182a14b12f1602aa271/ui/src/apps/searchLogs/components/SearchProgress.tsx#L232
https://github.com/pingcap-incubator/tidb-dashboard/blob/318fbc0f575843679db7a182a14b12f1602aa271/ui/src/apps/searchLogs/components/SearchProgress.tsx#L252
https://github.com/pingcap-incubator/tidb-dashboard/blob/master/ui/src/apps/searchLogs/components/SearchHistory.tsx#L73

@breeswish @Deardrops hi mentors, I move reqWithErrPrompt to packages/dashboard_clients, and add more error message, PTAL.

LGTM, @breeswish PTAL

/test

looks like tiup download failed.

Cleaning up outdated PRs. Feel free to reopen if there are new updates.

authors: Deardrops SSebo breeswish

repo: pingcap-incubator/tidb-dashboard


text:
add basic readme

@queenypingcap @siddontang PTAL

any doc to install rust protoc and rust grpc?

authors: BusyJay siddontang

repo: pingcap/grpc-rs


text:
[Explorer] Generate SQL failed for question 35f094f4-40e2-4251-a9d0-a342cddbff6b: show me all the top contributors  of opentelemetry in the main repositories aggregatee in the last 60 days, show their username and name and company

Hi, I have some problems with the question 35f094f4-40e2-4251-a9d0-a342cddbff6b show me all the top contributors  of opentelemetry in the main repositories aggregatee in the last 60 days, show their username and name and company (errorType = error-answer-generate):

Error message
Failed to generate answer, please try again later.

gharchive_dev> select * from explorer_question_feedbacks where question_id = UUID_TO_BIN('35f094f4-40e2-4251-a9d0-a342cddbff6b')\G
***************************[ 1. row ]***************************
id               | 390455
user_id          | 0
question_id      | 0x35f094f440e24251a9d0a342cddbff6b
satisfied        | 0
feedback_type    | error-answer-generate
feedback_content | {"message":"aborted"}
created_at       | 2023-02-08 14:31:20
1 row in set
Time: 0.210s

May be fixed in https://github.com/pingcap/ossinsight/pull/1284

repo: pingcap/ossinsight


text:
Added better node position logging

What problem does this PR solve?
Maybe #910 -> How do I get the position of an AST node.
The previous solution logged the starting offset of the original text,
this pull request added more detailed position information to ast.Node
What is changed and how it works?
Moved the struct Pos from parser/lexer.go into parser/ast/ast.go
Added 2 methods
func (ast.node) SetOriginTextPos(ast.Pos)
func (ast.node) OriginTextPos() ast.Pos

to the interface ast.Node
Check List
Tests

Unit test

Code changes

Has exported function/method change

Side effects

Possible performance regression
Breaking backward compatibility

Related changes
N/A

 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.Yifan Luo seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

Hi @TeemoKill, would you like to explain why we need this feature? Did you encounter any problems?

 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.Yifan Luo seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

authors: CLAassistant TeemoKill tangenta

repo: pingcap/parser


text:
*:added the doc 'development-workflow.md'

What problem does this PR solve?
The document can help contributor how to do development workflow.
What is changed and how it works?
I just added a document and update the document CONTRIBUTING.md.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.2 out of 3 committers have signed the CLA.:white_check_mark: nolouch:white_check_mark: elitecodegroovy:x: superadminsuperadmin seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

please close the PR if superadmin account blocks the license/cla

authors: CLAassistant elitecodegroovy

repo: pingcap/pd


text:
Implement dialect "tidb+pymysql"/"tidb+mysqldb" for sqlalchemy

What's changed

Implement TiDBDialect_mysqldb/TiDBDialect_pymysql for connection string "tidb+mysqldb"/"tidb+pymysql"

These two dialect classes for tidb overwrite the ddl_compiler by TiDBDDLCompiler so we can generate the correct SQL for:

altering tiflash replica
add vector index

Introduce class VectorIndex and TiFlashReplica so that we can define them using the sqlalchemy framework
Introduce function get_declarative_base so that users can utilize it to define a table model with VectorIndex. See the example for details.

Local test result
>  TEST_TIDB_HOST=127.0.0.1 TEST_TIDB_PORT=8042 tox -e py39
py39: install_package> python -I -m pip install --force-reinstall --no-deps /Users/jayson/Projects/tidb-vector-python/.tox/.tmp/package/12/tidb_vector-0.0.0.tar.gz
py39: commands[0]> pytest tests
=================================================================================================================================== test session starts ===================================================================================================================================
platform darwin -- Python 3.9.6, pytest-8.3.3, pluggy-1.5.0
cachedir: .tox/py39/.pytest_cache
rootdir: /Users/jayson/Projects/tidb-vector-python
configfile: pyproject.toml
collected 40 items

tests/integrations/test_utils.py .                                                                                [  2%]
tests/integrations/test_vector_client.py ..........                                                                                [ 27%]
tests/peewee/test_peewee.py .............                                                                                [ 60%]
tests/sqlalchemy/test_sqlalchemy.py ................                                                                                [100%]

==================================================================================================================================== warnings summary =====================================================================================================================================
tests/sqlalchemy/test_sqlalchemy.py::TestSQLAlchemy::test_insert_get_record
  /Users/jayson/Projects/tidb-vector-python/tests/sqlalchemy/test_sqlalchemy.py:57: SAWarning: Dialect tidb:pymysql will not make use of SQL compilation caching as it does not set the 'supports_statement_cache' attribute to ``True``.  This can have significant performance implications including some performance degradations in comparison to prior SQLAlchemy versions.  Dialect maintainers should seek to set this attribute to True after appropriate development and testing for SQLAlchemy 1.4 caching support.   Alternatively, this attribute may be set to False which will disable this warning. (Background on this warning at: https://sqlalche.me/e/20/cprf)
    session.query(Item1Model).delete()

-- Docs: https://docs.pytest.org/en/stable/how-to/capture-warnings.html
============================================================================================================================= 40 passed, 1 warning in 55.69s ==============================================================================================================================
.pkg: _exit> python /Users/jayson/Projects/tidb-vector-python/lib/python3.13/site-packages/pyproject_api/_backend.py True poetry.core.masonry.api
  py39: OK (58.66=setup[2.77]+cmd[55.89] seconds)
  congratulations :) (58.72 seconds)

Inspired from

https://github.com/xzkostyan/clickhouse-sqlalchemy
https://github.com/cockroachdb/sqlalchemy-cockroachdb

/cc @wd0517 @Mini256  @breezewish

Great job!
However, I think this should be merged to https://github.com/pingcap/sqlalchemy-tidb. We also need to maintain the sqlalchemy-tidb repository again and ensure it works for both sqlalchemy 1.4 and 2.0. Any suggestions?
BTW, a simplified and MySQL-compatible syntax for creating vector indexe would be more preferable.

Only suite for sqlalchemy~2.0. Closing it because it take too much maintenance cost

repo: pingcap/tidb-vector-python