Packager returned unexpected code 500

Hello! This time I'm trying to build a standalone app on Linux (Lubuntu 15.04), after running exp build:android and following the instructions in the docs, I get the following error:
? Would you like to upload a keystore or have us generate one for you?
If you don't know what this means, let us handle it! :)
 false
[exp] Starting build process...
[exp] Publishing...
|[exp] Packager returned unexpected code 500
Error: Packager returned unexpected code 500
    at XDLError.Error (native)
    at new XDLError (/Users/Jesse/Exponent/newuniverse/universe/dev/xdl/gulp/src/XDLError.js:12:46)
    at _callee4$ (/Users/Jesse/Exponent/newuniverse/universe/dev/xdl/gulp/src/Project.js:83:11)
    at tryCatch (/usr/local/lib/node_modules/exp/node_modules/regenerator-runtime/runtime.js:62:40)
    at GeneratorFunctionPrototype.invoke [as _invoke] (/usr/local/lib/node_modules/exp/node_modules/regenerator-runtime/runtime.js:336:22)
    at GeneratorFunctionPrototype.prototype.(anonymous function) [as next] (/usr/local/lib/node_modules/exp/node_modules/regenerator-runtime/runtime.js:95:21)
    at step (/usr/local/lib/node_modules/exp/node_modules/babel-runtime/helpers/asyncToGenerator.js:17:30)
    at /usr/local/lib/node_modules/exp/node_modules/babel-runtime/helpers/asyncToGenerator.js:28:20
    at run (/usr/local/lib/node_modules/exp/node_modules/core-js/library/modules/es6.promise.js:87:22)
    at /usr/local/lib/node_modules/exp/node_modules/core-js/library/modules/es6.promise.js:100:28
    at flush (/usr/local/lib/node_modules/exp/node_modules/core-js/library/modules/_microtask.js:18:9)
    at _combinedTickCallback (internal/process/next_tick.js:67:7)
    at process._tickDomainCallback (internal/process/next_tick.js:122:9)

Any tips about what's going on? Thanks in advance!

Looks like your project isn't building. Was it working when you opened it on your phone?

Yep, just tested again from XDE and it's working!

Anything useful in exp logs?

Found something!
exp-serve/test:16a1deb4-0 (err): Error<EMAIL_ADDRESS>could not be found, because 'node_modules/react-native/Libraries/CustomComponents/NavigationExperimental/assets' is not a subdirectory of any of the roots

Maybe something went wrong when I installed the dependencies?

Might be easier to join our slack to figure this out https://slack.exponentjs.com. you sure the same project is working in xde?

I * think * I'm sure. I'll triple check things anyways!
I'll close this issue since it's clearly my problem, and I'll join you guys on slack tomorrow (I can't today!)
Thanks for the help!

Might come in handy for someone having this issue too:
react-native link
on my project resolved the issue

react-native link might accidentally fix this issue but it's not the right fix. It sounds like there could be an issue with the packager. I'd recommend clearing your node_modules and re-installing (if you use Yarn this step is much more likely to be reliable), run watchman watch-del-all, and restart the packager (click Restart in XDE for example).


feat: adds codecov coverage badge

Codecov Report

Merging #6 (69e9957) into main (551df88) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main       #6   +/-   ##
=======================================
  Coverage   87.67%   87.67%
=======================================
  Files           4        4
  Lines          73       73
=======================================
  Hits           64       64
  Misses          9        9

Continue to review full report at Codecov.

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


Support TLS connections to Docker

Respect DOCKER_TLS_VERIFY and DOCKER_CERT_PATH
environment variables. I believe that boot2docker
sets these up by default.

There is nothing wrong with this, but ... you shouldn't run logspout on a different host and if it's on the same host, is TLS necessary?

Fair points.
Consider this to be support for development, not production. I bumped into this while playing with logspout. It was easier to add support in logspout than to fight with boot2docker to figure out how to turn off TLS. I suspect I'm not the only one.

Oh, so running logspout on os x hitting the boot2docker VM? Did you have a reason not to use the dev container? (make dev)

Yes, that. No reason, I just didn't notice the dev container.
However:
$ make dev
flag provided but not defined: -f

Usage: docker build [OPTIONS] PATH | URL | -

Build a new image from the source code at PATH

  --force-rm=false     Always remove intermediate containers, even after unsuccessful builds
  --no-cache=false     Do not use cache when building the image
  -q, --quiet=false    Suppress the verbose output generated by the containers
  --rm=true            Remove intermediate containers after a successful build
  -t, --tag=""         Repository name (and optionally a tag) to be applied to the resulting image in case of success
make: *** [dev] Error 2

Hmm, maybe I have an old Docker client version. Updating...

Updated. make dev works.
I think I'd still rather have go run logspout.go work, though. Up to you.

Try using make dev for a bit and let me know if you still think that.

Aye aye. Will let this idle and report back. Please consider it abandoned if I've not said anything either way by June 1, 2015. (I may not spend more time on logspout immediately.)

+1 to this. Our devs need it for docker-machine.

Closing for now since the use case is dev-related and you can always use make dev instead.


⚠ PrivateHD has degraded performance

In 7177f8d, PrivateHD ($PHD) experienced degraded performance:

HTTP code: 200
Response time: 3292 ms

Resolved: PrivateHD performance has improved in 5d5327a.


Unable to use Netty native transports in modularized Java 9 projects

Due to automatic name generation in Java 9 it is impossible to use Netty native transports - Epoll, KQueue in modularized Java 9 projects. For example, automatic module name for the epoll transport would be :
netty.transport.native.epoll
And native is the reserved keyword which doesn't allow the module to be exported to any other module. I created also question in SO in case there any workarounds.
However, there is actually very simple fix without moving the whole netty to java 9 that netty can apply right now.
We can add to manifest next line and it will solve the issue for Java 9. I checked it already and it works.
<Automatic-Module-Name>netty.transport.epoll</Automatic-Module-Name>
WDYT?

@doom369 ptal https://github.com/netty/netty/issues/7218

@johnou thanks. I saw that. My PR just fixes the concrete issue with epoll and kqueue. I think there is no need right now to give precise names for other modules until netty will provide module-info.java files.

@doom369 -> actually, there is a need to give them now: http://blog.joda.org/2017/05/java-se-9-jpms-automatic-modules.html
I'll remove the native word from my PR.

Wasn't this fixed recently?

@Scottmitch yep.. good catch


issue #11126 - Add sample code to TextFormField.

https://github.com/flutter/flutter/issues/11126

@branflake2267 Are you planning on applying the suggested fixes or should we take the PR over? (Either way is fine, just checking so we don't step on your toes.)

@branflake2267 I'm going to close this PR for now, but please feel free to resubmit it if you get a chance to work on it again. If you don't we'll get to it eventually.

Oops, I had lost this in my noise. Sounds good. :)

@jslavitz Can we close this now that your PR has been submitted?


⚠ Minecraft has degraded performance

In 9dca4b2, Minecraft (https://www.minecraft.net) experienced degraded performance:

HTTP code: 200
Response time: 1054 ms

Resolved: Minecraft performance has improved in 3e8eaaf.


AE Stale Status

A stale status from AE indicates that the transaction didn't make it to oracle.
We should treat these the same as error or rejected and resubmit with a new consumerTrackingId

@jSylvestre Should the same transaction be updated and resubmitted? Or should it be marked as cancelled, and a new copy submitted?

@sprucely I think updated and resubmitted. That way all the info is together in the timeline.
So, closing this one.


⚠ Bofrost.de has degraded performance

In 81e2938, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1882 ms

Resolved: Bofrost.de performance has improved in d4f3dac after 17 minutes.


ConfigMgr Site Server Details - Not Populated Automatically, Cannot Input Manually

I have installed 6.4.8 on primary site server.
DAT does not automatically detect/populate the ConfigMgr site server details, but also the ConfigMgr Settings\Site Server Settings | Package Options tab is not clickable.
It only becomes clickable after trying and failing to connect to ConfigMgr.
PS We are looking to implement this tool into our production environment: 1 CAS, 4 primaries, ~100k clients. Happy to help with testing.

Did you ever get tis issue resolved? I'm having the same issue on a new ConfigMgr build.


Laravel 5.2 User model?

Hi Everyone,
i use this package with Laravel 5.2
I added lines in to the User.php
<?php

namespace App;

use Illuminate\Foundation\Auth\User as Authenticatable;
use Bican\Roles\Traits\HasRoleAndPermission;
use Bican\Roles\Contracts\HasRoleAndPermission as HasRoleAndPermissionContract;

class User extends Authenticatable implements HasRoleAndPermissionContract
{
    use HasRoleAndPermission;

    protected $fillable = [
        "name", "email", "password",
    ];

    protected $hidden = [
        "password", "remember_token",
    ];

}

and on controller for if question..
      $user = \App\User::find(Auth::id());
       if ($user->is('admin')) { // you can pass an id or slug
        return redirect('login');
      }

but not running. Where is the error?
Thanks

Same here! If I call $user->getRoles() I get null but $user->roles works properly.

korayzorluoglu I've had the same problem. In controllers and in models I couldn't identify role by role name, so I had to do it trought slug or id.
if ($user->is(1)) { // pass role id          return redirect('login');       }
if ($user->is('admin')) { // pass slug, but make sure that it's not the same with the role name         return redirect('login');       }

I'm on laravel 5.1 and  I'm able to get roles of Authenticated user but can't get roles of any other user... i was able to do so by getRoles() method and roles() returns null. I any buddy found solution plz comment

Try something like this:

$users = User::all();
$foreach($users as $user){
$user->getRoles();
}

If this is what you need.

maybe this issue is related to #183 ?


Animation bug caused from inputs having same ID

What vue-mdc-adapter Version are you using?
"vue-mdc-adapter": "0.15.3"
What browser(s) is this bug affecting?
Chrome, Safari, and Firefox are all affected.
What OS are you using?
MacOS High Sierra 10.13.5
What are the steps to reproduce the bug?
Create a page with a mdc-textfield and mdc-checkbox on it. Visit the page and select the checkbox by clicking its label.
What is the expected behavior?
I would expect that the animations for the textfield and checkbox would be independent of each other, but because the inputs share the same ID (and the labels share the same for), selecting the checkbox label triggers animation on the textfield.

@rrgahan
if think this is a dup of #440 which has been fixed in v0.16.xx
would you mind checking ?

i'm closing the issue will reopen if you still have issues


Add potential new fields for discussion.

Howdy @joshreisner and @tintinnabulate, I hope you are both staying safe.
I've had to move quickly to accommodate changes in Philly, and I'm sure you're both probably doing the same. I'm trying to think long-term. A few things have become apparent:

We needed a way for meetings to be marked suspended / temp closed.
We needed a way for meetings to become online-only, even if temporarily.
Meetings may want to keep things like a backup online URL for emergencies.
Meetings needed a dial-in option for people without smart phones / computers.

To this end, it seemed to make sense to introduce a status field, rather than overloading meeting types. To avoid confusion with Open and Closed meeting types, we're using Active and Inactive, and have added Suspended (language adopted by GSO and NY Intergroup) and Online Only.
We're also suggesting adding or changing these fields:
video_conference_url - keep as is.
video_conference_dial_in - text field for dial-in info to the Zoom/BlueJeans/Hangouts/Skype.
payment_venmo - rename the field so we can have multiple payment types in alpha order.
payment_paypal - a bunch of our groups use PayPal (for old folks) and Venmo (for young folks).
This will allow us to tag meetings as Online Only - for now - and return them to be Active once the emergency passes, while allowing them to retain their online meeting information. Some meetings will want to continue streaming, while others will keep it "in case of emergency."
I'm not saying this is the end-all, be-all way to do things, but this is where I ended up when scrambling to get things working, and it seems to be working well so far. We've pre-pended meeting titles with ONLINE ONLY or SUSPENDED to make it abundantly clear. You can see it here:
https://www.aasepia.org/?day=any&search=online
What do y'all think?

Hey @FlipperPA as I mentioned in the other PR, a) I need to take some time to step back from all this volunteer work, so i can't approve changes right now, sorry. b) conference_phone would seem to be better than conference_dial_in.
Also, we have a new meeting code TC which was added last week to this spec, to the Meeting Guide app, and to the plugin, so I'd prefer to use that going forward rather than adding a redundant field for status.
Finally, venmo is already in use and can't be changed.
Let's sync up a little when things settle down, in the meantime I'm going to need some downtime from this and other service work.


Start db container using rabbitmq image from ecr

https://github.com/Shippable/pm/issues/10020

this is done, closing


Implement eth_sign

The eth_sign interface hasn't been implemented.

Don't add the secp256k1 dependency.
Have the eth_sign function check whether secp256k1 is available and if so, execute the signing.  If not, return a user friendly error message instructing them to install the dependency.

This feature would be really handy as more and more contracts rely on off-chain signing.
0x-based token exchanges are a good example here.


Scripts not functioning in battle (Nox)

Describe the bug
Scripts simply dont read or press any buttons once in a battle, battle script works up until that point and other modes work fine, yet as soon as it actually enters a battle the autoskill does nothing
Video
Video with debug
https://streamable.com/dsb6z4
Device Info (please fill at-least 'Device model'):
I'm not sure what I should put here since its Nox, the instance is Android 7 and the resolution in Nox's settings is 1920x1080 but was not working at 1600x900 as well
I've read through the troubleshooting and the emulator page, and the only similar issues I've seen have been on real phones and not Nox. The autoskill was also working fine a few days ago before not working suddenly last night

You didn't mention about Root, so maybe that got turned OFF.

I thought I did mention that, I guess I forgot. I double checked both root settings in the app and in nox's settings and even turned them off and back on again and it didn't fix the issue

Closing since #743 is a bigger problem now.


fixed "handling input" section of docs index

Fixes relative URLs of Remote / Keyboard section
Adds missing "Handling Input" parent menu and its content (mouse, touch)
Changes "Remote Control Interaction" menu item name to "Remote / Keyboard".

Pushed another commit to fix a link in the TypeScript section of the index file.


Bad path ref parameter breaks the linter

When lint-openapi is called with a YAML file which contains a path parameter, and the referenced parameter has an invalid definition (no "in" property), it causes lint-openapi to fail with:
[Error] There was a problem with a validator.
Cannot read property 'toLowerCase' of undefined

TypeError: Cannot read property 'toLowerCase' of undefined
    at /usr/local/lib/node_modules/ibm-openapi-validator/src/plugins/validation/2and3/semantic-validators/paths-ibm.js:58:37
    at Array.filter (<anonymous>)
    at /usr/local/lib/node_modules/ibm-openapi-validator/src/plugins/validation/2and3/semantic-validators/paths-ibm.js:58:12
    at Array.forEach (<anonymous>)
    at Object.module.exports.validate (/usr/local/lib/node_modules/ibm-openapi-validator/src/plugins/validation/2and3/semantic-validators/paths-ibm.js:36:13)
    at /usr/local/lib/node_modules/ibm-openapi-validator/src/cli-validator/utils/validator.js:119:51
    at Array.forEach (<anonymous>)
    at validateSwagger (/usr/local/lib/node_modules/ibm-openapi-validator/src/cli-validator/utils/validator.js:118:41)
    at processInput (/usr/local/lib/node_modules/ibm-openapi-validator/src/cli-validator/runValidator.js:274:17)

Sample YAML file:
openapi: 3.0.3

info:
  version: 0.0.1
  title: Example Title
  description: |
    A brief description of the API.
    It can be multiple lines.

servers:
  - url: https://api.example.com

paths:
  '/persons/{name}':
    parameters:
      - $ref: "#/components/headers/Broken-Header"
    get:
      summary: 'Get People by Name'
      description: "Test API"
      responses:
        '200':
          description: Successful response
          content:
            application/json:
              schema:
                type: object
                properties:
                  persons:
                    type: array
                    items:
                      $ref: '#/components/schemas/Person'

components:
  schemas:
    Person:
      title: Person
      type: object
      properties:
        name:
          description: "Person"

  headers:
    "Broken-Header":
      description: "This ref breaks the linter"
      schema:
        type: string
      required: true

This can be worked around by setting paths-ibm.js:58 to include a defined check: .filter(param => param.in && param.in.toLowerCase() === 'path') but since "in" is required by spec it may make more sense to include specific validation logic rather than simply ignoring it.

Sorry, I forgot to include that this was tested on:
$ lint-openapi --version
0.46.1

That's a good catch, thanks for the issue. I thought we did flag parameters without an in value, so I'll have to check on this.

:tada: This issue has been resolved in version 0.46.2 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[WIP] feat: Import code for role

Enhancement: Import code
Reason:
Result:
Issue Tracker Tickets (Jira or BZ if any):

[citest]

[citest]

[citest]

@richm do you know why the tests are skipped after [citest]?

@richm do you know why the tests are skipped after [citest]?

Yes - the platforms and galaxy_tags in meta/main.yml need to be fixed - see https://github.com/linux-system-roles/aide/pull/3/files#r1832783551

I don't know what ansible-lint is complaining about - just ignore it for now

[citest]

Looks like Testing Farm is having some issues right now

[citest]

[citest]

[citest]

[citest]

[citest]

[citest]

[citest]

[citest]

Resolves: https://github.com/orgs/linux-system-roles/discussions/103


Move example sketches to appropriately named folder

The Arduino IDE requires the sketch folder name to match the filename of the primary sketch file. This change causes the example sketch to be accessible via the Arduino IDE's File > Examples > LIBRARYNAME menu after the library is installed.
Reference:
https://github.com/arduino/Arduino/wiki/Arduino-IDE-1.5:-Library-specification#library-examples

Great suggestions - thanks for your contribution!


Error when trying to specify unit for measurement

I can't figure out how to specify a unit for a measurement. Here's my current code:
essentialHypertension <- cs(
  descendants(320128),
  exclude(descendants(3012888)),
  name = "Essential hypertension"
)

sbp <- cs(3004249, name = "SBP")
dbp <- cs(3012888, name = "DBP")
mmHg <- cs(8876, name = "mmHg")

hypertensionCohort <- cohort(
  entry = entry(
    conditionOccurrence(essentialHypertension),
    measurement(sbp, valueAsNumber(gte(130)), unit(mmHg)),
    measurement(dbp, valueAsNumber(gte(80)), unit(mmHg))
  ),
  exit = exit(
    endStrategy = observationExit()
  )
)

which will throw this error:
Error in h(simpleError(msg, call)) :
  error in evaluating the argument 'x' in selecting a method for function 'as.data.frame': argument "name" is missing, with no default

Thanks!
I'm struggling a bit to figure our what type of values can be used as arguments. Maybe there's a better way to document this?
Anyway, maybe this example could be added to the examples vignette? (Which I find very useful)


Unable to install gRPC 1.16.0

I would like to install gRPC on Ubuntu 20.04 because that's the last version compatible with FlatBuffers.
When I run
git clone --recurse-submodules -b v1.16.0 https://github.com/grpc/grpc
cd ./grpc
git submodule update --init
mkdir cmake/build
cd ./cmake/build/
cmake -DgRPC_INSTALL=ON -DgRPC_BUILD_TESTS=OFF -DgRPC_PROTOBUF_PROVIDER=package -DgRPC_ZLIB_PROVIDER=package -DgRPC_CARES_PROVIDER=package -DgRPC_SSL_PROVIDER=package -DCMAKE_BUILD_TYPE=Release ../..

I receive the following error:
_PROVIDER=package -DgRPC_SSL_PROVIDER=package -DCMAKE_BUILD_TYPE=Release ../..
-- The C compiler identification is GNU 9.3.0
-- The CXX compiler identification is GNU 9.3.0
-- Check for working C compiler: /usr/bin/cc
-- Check for working C compiler: /usr/bin/cc -- works
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Detecting C compile features
-- Detecting C compile features - done
-- Check for working CXX compiler: /usr/bin/c++
-- Check for working CXX compiler: /usr/bin/c++ -- works
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Detecting CXX compile features
-- Detecting CXX compile features - done
CMake Error at CMakeLists.txt:138 (include):
  include could not find load file:

    cmake/zlib.cmake

CMake Error at CMakeLists.txt:139 (include):
  include could not find load file:

    cmake/cares.cmake

CMake Error at CMakeLists.txt:140 (include):
  include could not find load file:

    cmake/protobuf.cmake

CMake Error at CMakeLists.txt:141 (include):
  include could not find load file:

    cmake/ssl.cmake

CMake Error at CMakeLists.txt:142 (include):
  include could not find load file:

    cmake/gflags.cmake

CMake Error at CMakeLists.txt:143 (include):
  include could not find load file:

    cmake/benchmark.cmake

CMake Error at CMakeLists.txt:144 (include):
  include could not find load file:

    cmake/address_sorting.cmake

CMake Error at CMakeLists.txt:145 (include):
  include could not find load file:

    cmake/nanopb.cmake

CMake Error: File /home/armegon/grpc/cmake/gRPCConfig.cmake.in does not exist.
CMake Error at CMakeLists.txt:18443 (configure_file):
  configure_file Problem configuring file

CMake Error: File /home/armegon/grpc/cmake/gRPCConfigVersion.cmake.in does not exist.
CMake Error at CMakeLists.txt:18443 (configure_file):
  configure_file Problem configuring file

-- Configuring incomplete, errors occurred!
See also "/home/armegon/grpc/cmake/build/CMakeFiles/CMakeOutput.log".

Why am I receiving this error ? How can I fix and install it with Cmake?

Hi, please redirect questions to wider communities like stack-overflow.


parser error when { in new line

If I write idl like this:
struct  Example
{
    1: int a,
    2: int b,
}

a error return when parsing.
otherwise, I write like this:
struct Example {
    1: int a,
    2: int b,
}

It will be ok.
How about support both two style ?

this has been fixed in master


Include Oauth2GenericAuthenticator OOTB in the product

Add Oauth2GenericAuthenticator as an authenticator inside plugins OOTB with Product IS.
Related issue: https://github.com/wso2/product-is/issues/14166

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/2690484888

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/2690484888
Status: failure

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/2694523962

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/2694523962
Status: success


Avoid unconditionally unwrapping the Result - UI Stack System

Objective

Fixes #11572

Solution

Avoid unconditionally unwrapping the Result in the ui_stack_system function.

Ah thank you for fixing this. So On deeper digging I found that this error is caused because despawning a child does not automatically remove the child from its Parent, and so UI stack gets re-populated with removed entity because parent still thinks they have a child. I think I better fix would be to remove children from parents on despawn.

I think I better fix would be to remove children from parents on despawn.

IMO we should do both. I'm happy with this PR as a first step, and then that as a more complex and controversial follow-up fix.


feat: register & get  CustomClaimType, allow CustomClaim

JIRA Link
https://polymesh.atlassian.net/browse/DA-908
Changelog / Description
Allows registering and getting customClaimType and issuing CustomClaim
Checklist -

[x] New Feature ?
[x] Updated swagger annotation (if API structure is changed) ?
[x] Unit Test (if possible) ?
[x] Updated the Readme.md (if required) ?

:tada: This PR is included in version 4.0.0-alpha.5 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


APEXCORE-741 Upgrade netlet dependency to 1.3.1

Looks good to me. I will merge it if there are no objections.

@vrozov can you take a look at the following exceptions in the malhar CI log and see if theses should be suppressed?
java.io.IOException: Broken pipe
	at sun.nio.ch.FileDispatcherImpl.write0(Native Method)
	at sun.nio.ch.SocketDispatcher.write(SocketDispatcher.java:47)
	at sun.nio.ch.IOUtil.writeFromNativeBuffer(IOUtil.java:93)
	at sun.nio.ch.IOUtil.write(IOUtil.java:51)
	at sun.nio.ch.SocketChannelImpl.write(SocketChannelImpl.java:487)
	at com.datatorrent.netlet.WriteOnlyClient.channelWrite(WriteOnlyClient.java:187)
	at com.datatorrent.netlet.WriteOnlyLengthPrependerClient.write(WriteOnlyLengthPrependerClient.java:75)
	at com.datatorrent.netlet.Listener$DisconnectingListener.write(Listener.java:241)
	at com.datatorrent.netlet.DefaultEventLoop.handleSelectedKey(DefaultEventLoop.java:380)
	at com.datatorrent.netlet.OptimizedEventLoop$SelectedSelectionKeySet.forEach(OptimizedEventLoop.java:59)
	at com.datatorrent.netlet.OptimizedEventLoop.runEventLoop(OptimizedEventLoop.java:192)
	at com.datatorrent.netlet.OptimizedEventLoop.runEventLoop(OptimizedEventLoop.java:157)
	at com.datatorrent.netlet.DefaultEventLoop.run(DefaultEventLoop.java:171)
	at java.lang.Thread.run(Thread.java:745)

@tweise Do you suggest to suppress them in netlet, apex or malhar unit tests?

I'm looking for your recommendation. If it can be handled in core then it would avoid potentially repeated work downstream.

I don't think that core can blindly suppress the exception, it can't distinguish actually broken connection from a connection that was terminated while tearing down. To be able to handle the case on the apex side, it would be necessary to revisit buffer server client disconnect protocol.

Looks like the discussion wouldn't have an impact on this particular PR, will merge it.

Can you rebase

Done


Where may I update the dismatched habitat_sim?

🐛 Bug
I'm using habitat-lab 0.2.5 and home-robot-sim ==0.1.0, found version dismatch:
NameError: pbr_image_based_lighting is not found on habitat_sim but is found on habitat_lab config.
                    It's also not in the list of keys to ignore: {'gpu_gpu'}
                    Did you make a typo in the config?
                    If not the version of Habitat Sim may not be compatible with Habitat Lab version: {'gpu_device_id': 0, 'gpu_gpu': False, 'allow_sliding': False, 'frustum_culling': True, 'enable_physics': True, 'physics_config_file': './data/default.physics_config.json', 'leave_context_with_background_renderer': False, 'enable_gfx_replay_save': False, 'pbr_image_based_lighting': True}

Exception ignored in: <function Simulator.__del__ at 0x2b8d28d498b0>
Traceback (most recent call last):
  File "/home/planner/anaconda3/envs/habitat_latest/lib/python3.9/site-packages/habitat_sim-0.3.1-py3.9-linux-x86_64.egg/habitat_sim/simulator.py", line 528, in __del__
    self.close(destroy=True)
  File "/home/planner/anaconda3/envs/habitat_latest/lib/python3.9/site-packages/habitat_sim-0.3.1-py3.9-linux-x86_64.egg/habitat_sim/simulator.py", line 131, in close
    if not self._initialized:
AttributeError: 'OVMMSim' object has no attribute '_initialized'

It seems like you installed habitat_sim 0.3.1. This is not currently supported. Please install habitat_sim==0.2.5 when working with habitat-lab 0.2.5

I see, the version of habitat-lab and habitat-sim should be exactly the same.

Use this conda install habitat-sim==0.2.5 -c conda-forge -c aihabitat to install specified version of habitat-sim!

https://github.com/facebookresearch/habitat-sim


feat(#141): fakehub.node_id()

In this pull, I've introduced fakehub.node_id() method that prints MD5 hash for fakehub instance based on datetime, when it was started.
closes #141
History:

feat(#141): node_id
feat(#141): node_id
feat(#141): node_id() in fakehub
feat(#141): register with fakehub
feat(#141): clean for cargo fmt

PR-Codex overview
This PR introduces a new node_id module, implements a NodeId struct with as_string method for MD5 hashing, and updates related functions in fakehub and user modules.
Detailed summary

Added node_id module for MD5 hashing
Implemented NodeId struct with as_string method
Updated fakehub and user modules to use NodeId for hashing

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

@rultor merge

@rultor merge

@h1alexbel OK, I'll try to merge now. You can check the progress of the merge here

@rultor merge

@h1alexbel Done! FYI, the full log is here (took me 53min)


Buttons disappear in Gangsters Organized Crime

Great work on all the fixes for Gangsters Organized Crime.  This updated v0.3.0 definitely works better than any other wrapper for this game.  However, there is still one issue left.
If you start the game and immediately enter the Options menu then all the default selected buttons are invisible.  However, once you click a different button then the these buttons become visible again.  See GIF below:

Fixed in v0.3.1.


[video_player] Update texture on seekTo

Description
@cbenhagen I basically copied your PR: https://github.com/flutter/plugins/pull/2397,  and opened a new one here with changes after the big 0.10.10 update.  It's still a very simple change.  I see that they want XC tests, but I need some clarification on how that is expected to work.
@cyanglaz  I see the PR you reference on the other PR, (https://github.com/flutter/plugins/pull/2445) but I don't understand how to run those tests.  I commented on it with the issues I am seeing.  If I could understand how I am expected to install the mock dependency and run the test, I could add the tests to this PR.  Test added!
What this fixes
Explicitly mark texture frame available on seekTo. This is necessary since flutter/engine@6a90b7f has changed the external texture data flow from a producer-consumer model. Now ios_external_texture_gl caches previous opengl texture, if no new frame are available, it does not execute the copyPixelBuffer method, just uses cached opengl texture to draw.
Related Issues
Fixes flutter/flutter#43567
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[x] I read the [Contributor Guide] and followed the process outlined there for submitting PRs.
[x] My PR includes unit or integration tests for all changed/updated/fixed behaviors (See [Contributor Guide]).
[x] All existing and new tests are passing.
[x] I updated/added relevant documentation (doc comments with ///).
[x] The analyzer (flutter analyze) does not report any problems on my PR.
[x] I read and followed the [Flutter Style Guide].
[x] The title of the PR starts with the name of the plugin surrounded by square brackets, e.g. [shared_preferences]
[x] I updated pubspec.yaml with an appropriate new version according to the [pub versioning philosophy].
[x] I updated CHANGELOG.md to add a description of the change.
[x] I signed the [CLA].
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Does your PR require plugin users to manually update their apps to accommodate your change?

[ ] Yes, this is a breaking change (please indicate a breaking change in CHANGELOG.md and increment major revision).
[x] No, this is not a breaking change.

Hi @KevinTheGray
can you resolve the merge conflicts? I am looking forward to having this issue fixed. :)

@dthuering thanks for the heads up! resolved it.
If you want, you can also give this PR a thumbs up and the original issue here a thumbs up: https://github.com/flutter/flutter/issues/43567
I believe that's part of how they prioritize which ones to review and merge in

I’m also eagerly awaiting this PR.

What's the status on this? This PR would be really important for my app :)

What's the status on this? This PR would be really important for my app :)

@KevinTheGray Does https://github.com/flutter/plugins/blob/master/CONTRIBUTING.md#setting-up-xcuitests and https://github.com/flutter/plugins/blob/master/CONTRIBUTING.md#xctests-ios address your question about running tests? If not, can you clarify what exactly you're looking for help with?

@stuartmorgan most likely! I'll give those a look and follow up if not.  Sorry, have just been keeping this branch up to date but haven't dug into the testing.  Hopefully I can have something soon.  Thanks

@stuartmorgan so I currently have two problems understanding how these tests are to be written.  Taking the webview package as an example
1.) The tests aren't passing for me in webview when I run pod lib lint --allow-warnings
I changed to the iOS directory and get the following error when I run that command:
pod lib lint --allow-warnings

 -> webview_flutter (0.0.1)
    - ERROR | [iOS] xcodebuild: Returned an unsuccessful exit code. You can use `--verbose` for more information.
    - NOTE  | xcodebuild:  note: Using new build system
    - NOTE  | xcodebuild:  note: Building targets in parallel
    - NOTE  | xcodebuild:  note: Using codesigning identity override: -
    - NOTE  | xcodebuild:  note: Planning build
    - NOTE  | xcodebuild:  note: Constructing build description
    - NOTE  | [iOS] xcodebuild:  warning: The iOS Simulator deployment target 'IPHONEOS_DEPLOYMENT_TARGET' is set to 8.0, but the range of supported deployment target versions is 9.0 to 14.2.99. (in target 'App' from project 'App')
    - NOTE  | [iOS] xcodebuild:  warning: Skipping code signing because the target does not have an Info.plist file and one is not being generated automatically. (in target 'App' from project 'App')
    - NOTE  | [iOS] xcodebuild:  warning: The iOS Simulator deployment target 'IPHONEOS_DEPLOYMENT_TARGET' is set to 8.0, but the range of supported deployment target versions is 9.0 to 14.2.99. (in target 'Pods-App' from project 'Pods')
    - NOTE  | [iOS] xcodebuild:  ld: warning: ignoring file /Users/KG/Library/Developer/Xcode/DerivedData/App-grtmcjsybkgxiofopejgpjphnagi/Build/Products/Release-iphonesimulator/webview_flutter/webview_flutter.framework/webview_flutter, building for iOS Simulator-arm64 but attempting to link with file built for iOS Simulator-x86_64
    - NOTE  | [iOS] xcodebuild:  clang: error: linker command failed with exit code 1 (use -v to see invocation)
    - NOTE  | [iOS] xcodebuild:  ld: warning: ignoring file /Users/KG/Library/Developer/Xcode/DerivedData/App-grtmcjsybkgxiofopejgpjphnagi/Build/Products/Release-iphonesimulator/webview_flutter/webview_flutter.framework/webview_flutter, building for iOS Simulator-i386 but attempting to link with file built for iOS Simulator-x86_64
    - NOTE  | [iOS] xcodebuild:  ld: warning: ignoring file Flutter/Flutter.framework/Flutter, missing required architecture i386 in file Flutter/Flutter.framework/Flutter (3 slices)
    - NOTE  | xcodebuild:  warning: The iOS Simulator deployment target 'IPHONEOS_DEPLOYMENT_TARGET' is set to 8.0, but the range of supported deployment target versions is 9.0 to 14.2.99. (in target 'Flutter' from project 'Pods')
    - NOTE  | xcodebuild:  warning: The iOS Simulator deployment target 'IPHONEOS_DEPLOYMENT_TARGET' is set to 8.0, but the range of supported deployment target versions is 9.0 to 14.2.99. (in target 'webview_flutter' from project 'Pods')
    - NOTE  | [iOS] xcodebuild:  warning: The iOS Simulator deployment target 'IPHONEOS_DEPLOYMENT_TARGET' is set to 8.0, but the range of supported deployment target versions is 9.0 to 14.2.99. (in target 'OCMock' from project 'Pods')
    - NOTE  | [iOS] xcodebuild:  warning: The iOS Simulator deployment target 'IPHONEOS_DEPLOYMENT_TARGET' is set to 8.0, but the range of supported deployment target versions is 9.0 to 14.2.99. (in target 'webview_flutter-Unit-Tests' from project 'Pods')

I can't get the imports or autocomplete to work in the test files.  If I open the example project and open FLTWKNavigationDelegateTests.m or FLTWebviewTests.m, @import Flutter; fails saying that it can't be found.  Is there a specific way I'm supposed to open a file/project to get autocomplete to work?

This is my flutter doctor:
[✓] Flutter (Channel master, 1.27.0-9.0.pre.126, on Mac OS X 10.15.4 19E287 darwin-x64, locale en-US)
    • Flutter version 1.27.0-9.0.pre.126 at /Users/KG/Developer/Flutter/flutter
    • Framework revision e81e0b850c (3 days ago), 2021-02-27 19:36:02 -0500
    • Engine revision 5c14c528cc
    • Dart version 2.13.0 (build 2.13.0-81.0.dev)

[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.3)
    • Android SDK at /Users/KG/Library/Android/sdk
    • Platform android-29, build-tools 29.0.3
    • ANDROID_HOME = /Users/KG/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6915495)
    • All Android licenses accepted.

[!] Xcode - develop for iOS and macOS
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.2, Build version 12B45b
    ! CocoaPods 1.9.3 out of date (1.10.0 is recommended).
        CocoaPods is used to retrieve the iOS and macOS platform side's plugin code that responds to your plugin usage
        on the Dart side.
        Without CocoaPods, plugins will not work on iOS or macOS.
        For more info, see https://flutter.dev/platform-plugins
      To upgrade see https://guides.cocoapods.org/using/getting-started.html#installation for instructions.

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 4.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6915495)

[✓] IntelliJ IDEA Community Edition (version 2018.2.5)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin version 31.3.3
    • Dart plugin version 182.5215

[✓] VS Code (version 1.53.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.20.0

Thanks guys for your work! Do you have any updates? It's been more than a year since this and other similar PRs where created. We urgently need this for production, otherwise we can't launch.

@ldelafuente I wouldn't hold my breath on this. The regression was introduced in August 2019 already and PR's with a fix exist since December 2019. The team is slowly digging through the backlog of plugin PR's and the situation is getting better every month. If you need this today I would recommend to maintain your own fork of the plugin with the fix or just use the branch of this PR in a dependency override.

@KevinTheGray There's now more documentation about plugin tests at https://github.com/flutter/flutter/wiki/Plugin-Tests, and basic test harnesses have been backfilled in the iOS implementations of the plugins (see here for video_player) which should make adding new tests much easier.

@stuartmorgan Thanks! I'll merge ad take a look.  Hope to have something soon

@stuartmorgan  Sorry for the delay, but the test that was originally requested is now in. The requirements were as follows:
We need some test, how about an XCTest that mocks the texture registry, invokes handleMethodCall with a "seekTo" and asserts that textureFrameAvailable was invoked?

I've mocked the registry and invoked seekTo directly on the pigeon Api (the plugin has changed a lot since then and no longer uses methodCalls).  Then I verified that textureFrameWasAvailable

@stuartmorgan I think this should be as you suggested now.  Let me know if I didn't get it right, happy to make any changes!

I know we're still waiting on a few tweaks to this PR before it can be merged, but can some other people chime in to mention whether this PR fixes the seek behavior in their apps?

LGTM from my side too. We have been using a version of this PR for quite a while in production. CC @guymclean


Node 14 - ganache-core - Error: Callback was already called.

OpenZeppelin Test Environment fails on node 14 with ganache-core error: Error: Callback was already called
node --version v14.4.0
npm --version 6.14.5
npx oz --version 2.8.2

To reproduce follow Learn guides tutorial using node 14 (https://docs.openzeppelin.com/learn/writing-automated-tests#writing-unit-tests)
$ npm test

><EMAIL_ADDRESS>test /home/abcoathup/projects/forum/testenv
> mocha --exit --recursive test

  Box

/home/abcoathup/projects/forum/testenv/node_modules/ganache-core/node_modules/merkle-patricia-tree/node_modules/async/lib/async.js:358
                callback(err);
                ^
Error: Callback was already called.
    at /home/abcoathup/projects/forum/testenv/node_modules/ganache-core/node_modules/merkle-patricia-tree/node_modules/async/lib/async.js:43:36
    at WriteStream.<anonymous> (/home/abcoathup/projects/forum/testenv/node_modules/ganache-core/node_modules/merkle-patricia-tree/node_modules/async/lib/async.js:358:17)
    at WriteStream.emit (events.js:315:20)
    at WriteStream.destroy (/home/abcoathup/projects/forum/testenv/node_modules/ganache-core/node_modules/level-ws/level-ws.js:140:8)
    at finish (_stream_writable.js:658:14)
    at processTicksAndRejections (internal/process/task_queues.js:84:21)

Reported in the community forum: https://forum.openzeppelin.com/t/testing-with-node-14-ganache-core-errors-with-callback-was-already-called/3068
Workaround is to use an earlier version of node such as node 10 or node 12.

Open issue: https://github.com/trufflesuite/ganache-core/issues/568

Install the beta version of ganache-core works for me (from the issue trufflesuite/ganache#568).
npm install ganache-core@beta -E


Releasing DBFlow4

DBFlow Version:4.0.0-beta
Issue Kind (Bug, Question, Feature):
Any reason why DBFlow 4.0 is still beta?  Can it be pushed to jCenter?
Description:

I believe this can be an answer is 4.0 beta stable enough for production? I didn't upgrade for that reason.


Version history page in helm mode fixes

What this PR does / why we need it:
Changes are forHelm managed mode only:

Installed revisions are sorted by revision number in descending order, similar to helm history output. Pending updates are still listed in semver order, but separately from installed revisions (some UI work still pending)
Sequence label changed to Revision
Pending updates no linger have sequence/revision number displayed
Various fixes to update cache.

Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:

Steps to reproduce

Does this PR introduce a user-facing change?

* Fixes versions ordering on the [version history](/enterprise/updating-apps#update-an-application-in-the-admin-console) page in [Helm managed mode (Alpha)](/vendor/helm-install).

Does this PR require documentation?

NONE

@ajp-io yep, that comment has a wrong word.  The picture is how it's intended to be to mimic Helm.  I reverse it.


Avoid concurrent initdb calls for a timeline

Takes a lock before running initdband check initdb target dir for existence, to avoid concurrent runs.
Currently, related lock manipulations are in tenant.rs, so the timeline creation code needs to be moved there from timelines.rs

Had discussed it more offline, and looks like https://github.com/neondatabase/neon/pull/2451#discussion_r972033214 approach with directory creation races is indeed possible and looks better than current, proposed in this PR and https://github.com/neondatabase/neon/pull/2451#discussion_r972023678 alternatives to me, so closing this and trying the new one instead.


NilClass error when using self.updateable fields

To make sure that I'm using self.createable fields and self.updateable fields in the right way, is the intention that you can use these to add as well as limited attributes that can be written? Could you, for example not an attribute in the attributes macro-style method but instead include it in the above mentioned and still allow it to be written.
When calling a create using this code:
  attributes :request_status

  has_many :time_off_request_components, class_name: 'TimeOffRequestComponent'

  def request_status
    @model.current_state
  end

  def self.updateble_fields(context)
   super +  [:time_off_request_components]
  end

  def self.createable_fields(context)
   super +  [:time_off_request_components, :comment]
  end

The error I get is:
TypeError:
no implicit conversion of nil into Hash
It has no useful backtrace but stepping through with the debugger suggests that it is coming from jsonapi/resource.rb @ line 424. @_attributes[attr] can't be merged with default_attribute_options because it is nil.
If this is from using these methods incorrectly, maybe you could let me know if there is a clean way to accomplish what I'm trying to do using the existing API. As mentioned in #129 and #133, I'm trying to do a complex post that involves sending data under the time_off_request_components key nested inside the primary object. I then want this nested object to be serialized in the default side-loaded way.
Calling @request.include << 'time_off_request_components' in a controller before_filter gets it right when it is not whitelisted in the resource, but when it is listed I can't seem to get the side-loaded format to serialize. Should I be whitelisting and explicitly calling a serializer from inside the resource from a reader-override method, or is there a way to get around whitelisting that lets the default include functionality work?

As the code is written you can't add additional fields into the self.createable_fields and self.updateble_fields fields functions, only remove them. The problem is @_attributes[attr] returns nil for the added fields, which breaks things as you can see.
What is the use case for adding a field that isn't an attribute on the resource?

As I was describing in the other issue threads that I referenced above, I have been trying to find the best way to accept a complicated set of post params that don't map onto a single model and then return the saved data in the side-loaded forward under 'includes'. I realize it sounds contrived, but we have decided to diverge from the spec by accepting a nested post/put until the JSONAPI spec authors include a more clear alternative in 1.1. [https://github.com/json-api/json-api/issues/205]
What I end up getting is that the associated resources are returned from the POST both nested under the main data object as well as included at the top level, where I really only want them at the top level.
I've managed to use a clean enough hack as an after filter in the controller that deletes the unwanted nested params. It seems removing the nested resource (:time_off_request_components) from the fetchable fields meant that they couldn't be loaded as a side-loaded included resource either.

I'm having a bit of trouble wrapping my head around this. What I can say is the resource does not need to be a one-to-one mapping to a model. It seems that should allow you to create the a structure that at least returns the data properly side loaded, though without seeing the code I can't be sure.

Ya thanks for the quick reply. I've got workable hack figured out so if this seems confounding feel free to consider it closed. I'll attempt to give it one more go though in case you're the curious type and this is bothering you.

I want to accept a parameter key that has the same name as a has_many association defined on the builder, so I can manually create the record it as a nested resource. The key needs to be listed as an attribute so that an error is not thrown

Listing the parameter as an attributes means that, when being serialized, the resource expects either an attribute on the model or a custom method to match this name, instead of inferring the resource to include from the association (I presume, because this works seamlessly for other simpler resources).

Writing a custom method that manually calls the appropriate serializer gives me both nested and side loaded data.

model_name 'TimeOffRequest'

  attributes :id,
             :request_status,
             :time_off_request_components,
             :comment

  has_many :time_off_request_components, class_name: 'TimeOffRequestComponent'

  def time_off_request_components
    serializer = JSONAPI::ResourceSerializer.new(TimeOffRequestComponentResource)
    component_resources = @model.time_off_request_components.map { |component| TimeOffRequestComponentResource.new(component) }
    component_resources.each { |component_resource| serializer.serialize_to_hash(component_resource) }
  end


gRPC server missing some required information

This doesn't seem to run using the commands in the README.md.
$ docker-compose up
[+] Running 2/0
 ⠿ Container warhammer-grpc-1  Created                                                                                0.0s
 ⠿ Container warhammer-db-1    Created                                                                                0.0s
Attaching to warhammer-db-1, warhammer-grpc-1
warhammer-db-1    |
warhammer-db-1    | PostgreSQL Database directory appears to contain a database; Skipping initialization
warhammer-db-1    |
warhammer-grpc-1  | 10:17PM FTL No $CHANNEL_ID given. role=gRPC
warhammer-db-1    | 2022-01-26 22:17:46.738 UTC [1] LOG:  starting PostgreSQL 14.1 (Debian 14.1-1.pgdg110+1) on x86_64-pc-linux-gnu, compiled by gcc (Debian 10.2.1-6) 10.2.1 20210110, 64-bit
warhammer-db-1    | 2022-01-26 22:17:46.738 UTC [1] LOG:  listening on IPv4 address "<IP_ADDRESS>", port 5432
warhammer-db-1    | 2022-01-26 22:17:46.738 UTC [1] LOG:  listening on IPv6 address "::", port 5432
warhammer-db-1    | 2022-01-26 22:17:46.740 UTC [1] LOG:  listening on Unix socket "/var/run/postgresql/.s.PGSQL.5432"
warhammer-db-1    | 2022-01-26 22:17:46.791 UTC [26] LOG:  database system was shut down at 2022-01-25 01:13:45 UTC
warhammer-db-1    | 2022-01-26 22:17:46.800 UTC [1] LOG:  database system is ready to accept connections
warhammer-grpc-1 exited with code 1

What other info is required to get the server up and running?

Hey @xansec! Thanks for opening this issue. Sorry it's taken me a few days to get back to you. Looks like I was initially intending to make this able to work as a Discord bot but... honestly... not super sure why I did that! So that CHANNEL_ID was it looking for a discord channel to use the bot with.
I've removed that functionality so it's a straight gRPC warhammer service now and it will spin up just fine with the commands in the README 😄
When you're ready to try the fix, just run it with docker-compose up --build and it will build with that fresh image.

The fix seemed to work. Thanks!


feat: can limit search and replace to selected files only

Previously the search and replace operation would be applied to all files in the directory where the operation was started. This commit adds a feature to limit the search and replace operation to only the files that were visually selected (with v or <space> by default in yazi) in the yazi view.

Demo: I select some files using <space>, and then press <c-g> to open the default search and replace integration (grug-far.nvim). The search is limited to the selected files only.

https://github.com/user-attachments/assets/5ce6feac-c355-443c-b9f3-b1f514eb1583
This allows fine tuning of the search and replace operation to only the files that are relevant to the user.

@MagicDuck I was wondering if you could review this integration and let me know if there are some things I could do to improve it. Thanks, and enjoy your weekend! 🙂

Looks kosher to me! 😄
I am fascinated by your cypress tests though. Never knew cypress can be used for this kind of stuff. Does it do snapshots as well?

Nice, thanks 👍🏻
A small technical explanation: The testing setup is something I created for this project specifically. It's a node.js application that runs a terminal app (in this case obviously Neovim) and lets the frontend see what happens as well as send keypresses and mouse clicks to control the app.
Cypress doesn't seem to support text snapshots (images are supported I think), but some other test runner might. I looked into the snapshot thing yesterday after seeing some of your changes in grug-far.nvim. it looks really nice for testing Neovim stuff.

That’s cool! Yeah, mini.test is pretty good, but not polished on the same level as something like jest if you are used to that so you have to write a bunch of boilerplate and helper functions. Sometimes I wish we just had busted + text snapshots + headless child neovim instances.


🏇 Personal Profile: Adding Deletions Tests and Updating Migrations

In this PR you can find the several updates:

Added deletion tests for the personal profile model.

Updated the data migration so the names of the resumes + and profile pictures will include the user's name for organizational purposes. Also added a specific path for profile pictures to be saved.

Adding the option for a personal profile to not have a birth date so it's not a requirement.

LGTM
I don't think it's mandatory but you could squash some of the commits together in order to make this PR clearer.

I am working on the bug mentioned here: https://github.com/redhat-beyond/JobSeeker/issues/68 .
Therefore my tries are failing PyTest for the meantime, please disregard and I will have this PR ready for review once fixed.

As I discussed in #68, I succeeded in deleting the test files that are created when the pytest runs.
The "leftover" files are created from a function in the migrations. That is a bug to fix but it is not related to the tests as I suppose. Therefore I think this PR can be merged, and another PR, hopefully with a fix to that specific bug, will be created separately.

In this PR you can find the several updates:

If it is several updates, it should be several PRs, sorry @ShirleyFichman

In this PR you can find the several updates:

If it is several updates, it should be several PRs, sorry @ShirleyFichman

OK, closing this PR and making new ones.


Progress bars

See issue #171.
Here I added progress bars for many functions in analysis. The implementation for the simulation phase is trivial and is demonstrated in the first tutorial. In analysis, many of the for loops were changed to have progress bars. I'm not sure how to include the rich package as a dependency of the pAPRika package... I tried to add it to the paprika-dev conda-env but I assume there's more that needs to be done to make sure the dependency is added.

Looks like the tests are failing due to multiple-live-display errors. To my knowledge, the track function can only be run for one progress bar at a time. I wonder if this is an error that is encountered because the tests are trying to run the functions in analysis.py in a different way than linear way they would be called during normal use. I didn't encounter any issues running multiple sequential progress bars in my experimenting but if the tests suggest otherwise it should be looked into.

Is this because we print out things inside one of the track loops? Do you get the same error if you run it locally?

Is this because we print out things inside one of the track loops? Do you get the same error if you run it locally?

I made the incorrect assumption that each function in the analysis.py file was independent; I added progress bars to functions like run_mbar assuming they would be only called directly by the user, but they are called inside compute_free_energy. The code now has one separate progress bar for each phase of the analysis in compute_free_energy, which is the only function in analysis.py with a progress bar.

Codecov Report

Merging #172 (cc78024) into master (5376f44) will decrease coverage by 0.02%.
The diff coverage is 100.00%.

This might be a silly questions, but is it possible to run the repository's tests locally?

@jaketanderson yes, you can run the tests locally using pytest. For example, to run the test for analysis modules you run pytest test_analysis.py, or if you want to run a specific test in the test file: pytest test_analysis.py::test_reference_state_work.


'yoda_style' => false not working

The  'yoda_style' => false rule seems to behave like  'yoda_style' => null.
 'yoda_style' => true works corretly.
The PHP version you are using ($ php -v):

PHP 7.3.8 (cli) (built: Aug  2 2019 05:16:32) ( NTS )
Copyright (c) 1997-2018 The PHP Group
Zend Engine v3.3.8, Copyright (c) 1998-2018 Zend Technologies

PHP CS Fixer version you are using ($ php-cs-fixer -V):

PHP CS Fixer 2.15.1 Europe Round by Fabien Potencier and Dariusz Ruminski

The command you use to run PHP CS Fixer:
php vendor/bin/php-cs-fixer fix --dry-run --diff --config=.php_cs
The configuration file you are using, if any:
<?php

return PhpCsFixer\Config::create()
    ->setUsingCache(false)
    ->setRules([
        'yoda_style' => false,
    ])
    ->setFinder(
        PhpCsFixer\Finder::create()
            ->in(['src'])
    )
;

If applicable, please provide minimum samples of PHP code (as plain text, not screenshots):

before running PHP CS Fixer (no changes):

<?php

$var = true;

if (true === $var) {
	echo 'true';
} else {
	echo 'false';
}

with unexpected changes applied when running PHP CS Fixer:

Nothing

with the changes you expected instead:

<?php

$var = true;

if ($var === true) {
	echo 'true';
} else {
	echo 'false';
}

This is solved by using this configuration:
<?php

return PhpCsFixer\Config::create()
    ->setUsingCache(false)
    ->setRules([
        'yoda_style' => [
            'identical' => false,
        ],
    ])
    ->setFinder(
        PhpCsFixer\Finder::create()
            ->in(['src'])
    );

But I think  'yoda_style' => false should work

But I think 'yoda_style' => false should work

And it works, as for any fixer - and setting any fixer to false disable it.
To achieve your expected changes you need to do this:
        'yoda_style' => [
            'equal' => false,
            'identical' => false,
            'less_and_greater' => false,
        ],

Closing as it was more misunderstanding how to configure the rule and not bug in it.


how do I join the Slack channel?

it's asking me to sign in. would you invite me?

Here is the invite link: https://slack.testcontainers.org/
I will fixed link in the docs.


Added BasicWrappedGraph

which implements the interface methods of this.rawGraph.addVertex() .. etc. etc... so you can simply override instead of having so much boiler plate.

Done.


bug tests4 conda forge

Main changes:

Minor changes to make error messages more informative and to add some comments in tofu/tests/tests01_geom/tests03_core.py
Corrected a typo in setup.py that was preventing the proper directory (tests/tests01_geom/tests03_core_data/) to be included in the sdist package, leading to missing data for tests to run correctly

Issues:
Fixes, in devel, issue #409

Codecov Report

Merging #410 into devel will not change coverage.
The diff coverage is 58.69%.

@@           Coverage Diff           @@
##            devel     #410   +/-   ##
=======================================
  Coverage   40.40%   40.40%
=======================================
  Files          74       74
  Lines       24785    24785
=======================================
  Hits        10014    10014
  Misses      14771    14771

Impacted Files
Coverage Δ

tofu/geom/_core.py
62.51% <47.05%> (-0.03%)
:arrow_down:

tofu/tests/tests01_geom/tests03_core.py
90.95% <91.66%> (+0.23%)
:arrow_up:

Continue to review full report at Codecov.

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

Also, I can confirm that the tests run locally now. @Didou09, I would merge the PR, if I could, but I don't have the rights 😓
Can you do it?


How to use axios in endpoints for fetching data

Hello,
I would like to have an endpoint that fetches data via XHR. I have installed axios and it didn't work since it's not an ESM.
I found a Stack Overflow answer that suggested to install @bundled-es-modules/axios, which I did.
Now there is the following exception:
ReferenceError: XMLHttpRequest is not defined
    at dispatchXhrRequest (/node_modules/@bundled-es-modules/axios/axios.js:977:20)
    at new Promise (<anonymous>)
    at xhrAdapter (/node_modules/@bundled-es-modules/axios/axios.js:969:11)
    at dispatchRequest (/node_modules/@bundled-es-modules/axios/axios.js:774:11)
    at async get (/home/me/Projects/Svelte/svelte-kit-test/src/routes/marvel/characters.js:6:17)
    at async render_route (file:///home/me/Projects/Svelte/svelte-kit-test/node_modules/@sveltejs/kit/dist/ssr.js:2512:20)
    at async ssr (file:///home/me/Projects/Svelte/svelte-kit-test/node_modules/@sveltejs/kit/dist/ssr.js:2593:9)
    at async Immediate.<anonymous> (file:///home/me/Projects/Svelte/svelte-kit-test/node_modules/@sveltejs/kit/dist/chunks/index2.js:3286:23)

It seems like axios "thinks" that is runs in the browser and not in Node.
Any help would be appreciated.

Sounds like its the SSR throwing the error with not window object available. Make sure you check for window before using the endpoint.
Either way, I don't think this belongs here as an issue.

Thank you for the response.
I have opened this as a "help", not as an "issue".
I couldn't find a relevant example in the docs. I thought that "endpoint" runs in node.

Our help template suggests to use Discord or Stack Overflow for questions
I've never used axios so wouldn't be able to tell you what the issue is since it's not related to SvelteKit


Edit field different size than table column

Expected behavior and actual behavior:
Expected behavior: when clicking on a cell to edit textual data, the edit field is the same width/size as the original cell
Actual behavior:
edit field is different size (usually smaller) than the original cell
Steps to reproduce the problem:
Enter text data into any field on a row in order to make the table stretch
Once you see sufficient table stretching, click out, then click back into the cell to edit.
You'll now see the edit field is the same size as the original cell before stretching, and the table "snaps" back to accommodate this.
Environment (component version number, Browser (including version), operating system, hardware, etc):
Component Version Number: 5.0.4
Browser: Google Chrome Ver. 61.0.3163.100 64-bit
OS: "OS X El Capitan" (10.11.6)
Hardware: Macbook Pro - Retina 15" - 2.5Ghz Intel Core i7

Can you log against the data-table repository please? This is progress-bar :)
https://github.com/PredixDev/px-data-table/issues

Man, I’m so sorry. I’ll take care of that!
--
Victor Bashorun
Software Engineer, Specialist
PCCE SWAT Team
victor.bashorun<EMAIL_ADDRESS>From: Randy Askin<EMAIL_ADDRESS>Reply-To: PredixDev/px-progress-bar<EMAIL_ADDRESS>Date: Tuesday, October 31, 2017 at 10:14 AM
To: PredixDev/px-progress-bar<EMAIL_ADDRESS>Cc: "Bashorun, Victor (GE Digital)"<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: EXT: Re: [PredixDev/px-progress-bar] Edit field different size than table column (#7)
Can you log against the data-table repository please? This is progress-bar :)
https://github.com/PredixDev/px-data-table/issues
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/PredixDev/px-progress-bar/issues/7#issuecomment-340834896, or mute the threadhttps://github.com/notifications/unsubscribe-auth/Afsg9JcLVEc2rfIQRK_OYhJ_MwEtp0KKks5sx1VggaJpZM4QM7-c.