Add multi-arch support (amd64 and arm64)

Please hold off merging this, trying to see if I can speed it up a bit more ...

Hi @ppalaga I just made some minor tweaks, startup time on my laptop for the container is approx 2 seconds now, as opposed to approx 10 seconds.  Please feel free to let me know in case you have any feedback!

Great work, thanks @shumonsharif !

It is deployed on quay.io now https://github.com/l2x6/calculator-ws/actions/runs/2991250564


No null safety for tappable cells

Tappable cells throw an exception when tapped if the onCellTap callback is not provided.
Logs:-
The following NoSuchMethodError was thrown while handling a
gesture:
                    The method 'call' was called on null.
                    Receiver: null
                    Tried calling: call()

                    When the exception was thrown, this was the stack:
                    #0      _FluttableState.detectOnCellTap
                    (package:qurbaniapp/Cidgets/fluttable.dart:475:21)
                    #1      _FluttableState.relevantCellDetector.<anonymous
closure>
                    (package:qurbaniapp/Cidgets/fluttable.dart:442:17)
                    #2      _InkResponseState._handleTap
                    (package:flutter/src/material/ink_well.dart:991:20)
                    #3      GestureRecognizer.invokeCallback
                    (package:flutter/src/gestures/recognizer.dart:182:24)
                    #4      TapGestureRecognizer.handleTapUp
                    (package:flutter/src/gestures/tap.dart:607:11)
                    #5      BaseTapGestureRecognizer._checkUp
                    (package:flutter/src/gestures/tap.dart:296:5)
                    #6      BaseTapGestureRecognizer.acceptGesture
                    (package:flutter/src/gestures/tap.dart:267:7)
                    #7      GestureArenaManager.sweep
                    (package:flutter/src/gestures/arena.dart:157:27)
                    #8      GestureBinding.handleEvent
                    (package:flutter/src/gestures/binding.dart:385:20)
                    #9      GestureBinding.dispatchEvent
                    (package:flutter/src/gestures/binding.dart:361:22)
                    #10     RendererBinding.dispatchEvent
                    (package:flutter/src/rendering/binding.dart:278:11)
                    #11     GestureBinding._handlePointerEventImmediately
                    (package:flutter/src/gestures/binding.dart:316:7)
                    #12     GestureBinding.handlePointerEvent
                    (package:flutter/src/gestures/binding.dart:280:5)
                    #13     GestureBinding._flushPointerEventQueue
                    (package:flutter/src/gestures/binding.dart:238:7)
                    #14     GestureBinding._handlePointerDataPacket
                    (package:flutter/src/gestures/binding.dart:221:7)
                    #18     _invoke1 (dart:ui/hooks.dart:180:10)
                    #19     PlatformDispatcher._dispatchPointerDataPacket
                    (dart:ui/platform_dispatcher.dart:276:7)
                    #20     _dispatchPointerDataPacket
(dart:ui/hooks.dart:96:31)
                    (elided 3 frames from dart:async)

                    Handler: "onTap"
                    Recognizer:
                      TapGestureRecognizer#65a4a

I think it is set to a required property. Will be fixed.

Added null checks. https://github.com/ZainUlMustafa/Fluttable/tree/tap-able-cells
Will be released as a main feature in v1.2.3


Class Modification from commit breaks Sitemap

https://github.com/siramsay/perihelion/blob/23c4f2b23e8a06f820ba54a537893e4ba3a6c338/deploy/src/perihelion/php/model/page.model.php#L5
From this commit
https://github.com/siramsay/perihelion/commit/8398c3ee458b45e64f539d2811c056d4456cac6c

This was Fixed with with https://github.com/chishiki/perihelion/commit/e3b95f7220a188272618fe1a6b66f5f0a195db87
So will close.


Create ci.yml

Q
A

Branch?
main

Bug fix?
yes

New feature?
yes

Deprecations?
no

Tickets
Fix #...

License
MIT

Let's wait on this until we replace PHPunit with Pest.


Add snap to grid/ticks functionality

Possibly we can put an array of sizes to which the container will snap ['100px', '45%']

This probably won't be implemented by core maintainers but if someone opens a PR we'll gladly accept.


MoM - Monsters being set to Spirits when Physical (and visa versa)

Description of Problem or Question
I have a lot of feedback on my scenarios where generic monsters that are physical are presenting "Spiritual" attack dialogues.
For example, a deepone is attacked and it says "Bullets are of no use here, you drop your weapon and plead with the spirit, test Influence".
I've also seen the opposite where I can physically attack Ghosts, here is a screenshot example of this:

Here is the Spawn settings:

I have not set any Traits in the Spawn, so I would assume that the Base Monsters trait (a Ghost being "Spirit" would be selected), however it does not seem to be working this way. It would be a lot of work to go through all of my spawns and manually select the monsters trait (assuming that will even fix it).
Also its not intuitive to the authors to have to manually set this, since its not a mandatory field.
I've created a small scenario, that can be used to chain spawn monsters and attempt to recreate the issue.
Its very sporadic, so I suspect that monsters that don't have pool traits selected may be randomly pulling events from the entire attack pool? But I may be wrong.
Valkyrie Version
2.1.5
Attach Save, Quest or Scenario
MonsterTest.zip

Here's an example from the same test scenario, this deep one as an unarmed attack is showing up as a spirit:

The next unarmed attack at the same foe is showing correctly as a physical attack:

Same monster, trying to shoot it - this little phasing Deep One is a pest!

(Another Physical Monster, getting a Spiritual Attack)

I think we can consider it as a bug

Yeh, it's a needle in a haystack kind of bug.

I got this bug today, while playing.
This one definitely needs a fix, it's killing all the strategy related to the different attack type.

Thanks a lot for the demo scenario, it helped me a lot.
The traits of the character was simply not checked so you could get a test for any kind of creatures.
(The type of attack was checked though.)
I'm now checking that the attack type correspond to at least one the traits of the monster.
This may lead to some strange behaviour if some monster are "ghost" and "humanoid" for example, but I suppose this should not the case.
This fix will changed the gameplay quite a lot, for the best hopefully.

The base monsters should have default traits, which can be overridden by the manual trait selection.
Great work finding this out BenRQ.
@NPBruce could you add some context to this bug, was it actually intentional and can you see any issues with Ben's proposed change?


code did not work and download made me get all the files.  And still did not work

code did not work and download made me get all the files.  And still did not work
[Enter feedback here]

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

ID: 0156ffc3-47ac-d5da-fbc2-9ff5018b071b
Version Independent ID: 615db388-a244-41f7-21b5-a4085d9ed479
Content: Model validation in ASP.NET Core MVC
Content Source: aspnetcore/mvc/models/validation.md
Product: aspnet-core
Technology: aspnetcore-mvc
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

What code didn't work?


Background visibility

It would be great if this supported a style option when rightclicking the window:

Personally my UI is all transparent:

and this would be a big feature for me if possible.
Thanks,
John

8752463 title and background can now be transparent.


Programatically trigger showing of the picker

I haven't found way to show picker programatically, is this possible?

see https://github.com/fragaria/angular-daterangepicker/issues/106#issuecomment-152369027


Fix draw effects

Will fix #270

Codecov Report

Merging #281 (1585502) into master (d31d17e) will increase coverage by 0.03%.
The diff coverage is 100.00%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##           master     #281      +/-   ##
==========================================
+ Coverage   95.51%   95.54%   +0.03%
==========================================
  Files         176      176
  Lines        4057     4066       +9
==========================================
+ Hits         3875     3885      +10
+ Misses        182      181       -1

Impacted Files
Coverage Δ

impl/jamtemplate/common/bar.hpp
100.00% <ø> (ø)

impl/jamtemplate/common/graphics/drawable_impl.hpp
100.00% <ø> (ø)

...jamtemplate/common/graphics/drawable_interface.hpp
100.00% <ø> (ø)

impl/jamtemplate/common/graphics/flicker_impl.cpp
100.00% <ø> (ø)

impl/jamtemplate/common/animation.cpp
94.66% <100.00%> (+0.19%)
:arrow_up:

impl/jamtemplate/common/bar.cpp
100.00% <100.00%> (ø)

impl/jamtemplate/common/graphics/drawable_impl.cpp
97.29% <100.00%> (+0.74%)
:arrow_up:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Remove the need to set the MY_IP env var during OH hub setup.

Teach the ./hub/makefile how to introspect the public IP address of the machine you're running on and use that introspected IP address during generation of the config.json file. If the introspection is not successful, make will fail. Don't forget to update the doc.
Issue #8 should be done before this one.

Normally, I use curl -s httpbin.org/ip | jq -r .origin to automatically get the external IP address. Since this approach depends on curl and jq commands, would you think, this would be a fairly platform agnostic solution? If yes, I can take a shot.

Normally, I use curl -s httpbin.org/ip | jq -r .origin to automatically get the external IP address. Since this approach depends on curl and jq commands, would you think, this would be a fairly platform agnostic solution? If yes, I can take a shot.

Yes, both curl and jq are required and installed, so both would be available to retrieve the external IP address.

Joseph "Joe" Pearson
Technology Strategist
IBM Cloud
+1<PHONE_NUMBER> Office
<EMAIL_ADDRESS>

----- Original message -----From: mrchainhead<EMAIL_ADDRESS>edgexfoundry-holding/open-horizon-integration<EMAIL_ADDRESS>Subscribed<EMAIL_ADDRESS>[EXTERNAL] Re: [edgexfoundry-holding/open-horizon-integration] Remove the need to set the MY_IP env var during OH hub setup. (#10)Date: Mon, Feb 24, 2020 6:20 AM
Normally, I use curl -s httpbin.org/ip | jq -r .origin to automatically get the external IP address. Since this approach depends on curl and jq commands, would you think, this would be a fairly platform agnostic solution? If yes, I can take a shot.
—You are receiving this because you are subscribed to this thread.Reply to this email directly, view it on GitHub, or unsubscribe.

However, on my test VM, it has a local subnet IP address of 172.* and an external IP of 9.* and yet running curl -s httpbin.org/ip | jq -r .origin  shows my IP address as 129.*, which I'm guessing is my NAT'ted IP address.

IP address for ens3:    172.x.x.x
IP address for ens7:    9.x.x.x
curl -s httpbin.org/ip | jq -r .origin129.x.x.x

So that's not going to work for me.

Joseph "Joe" Pearson
Technology Strategist
IBM Cloud
+1<PHONE_NUMBER> Office
<EMAIL_ADDRESS>

----- Original message -----From: Joseph Pearson/Atlanta/IBMTo: reply@reply.github.comCc:Subject: Re: [EXTERNAL] Re: [edgexfoundry-holding/open-horizon-integration] Remove the need to set the MY_IP env var during OH hub setup. (#10)Date: Mon, Feb 24, 2020 10:02 AM
Yes, both curl and jq are required and installed, so both would be available to retrieve the external IP address.

Joseph "Joe" Pearson
Technology Strategist
IBM Cloud
+1<PHONE_NUMBER> Office
<EMAIL_ADDRESS>

----- Original message -----From: mrchainhead<EMAIL_ADDRESS>edgexfoundry-holding/open-horizon-integration<EMAIL_ADDRESS>Subscribed<EMAIL_ADDRESS>[EXTERNAL] Re: [edgexfoundry-holding/open-horizon-integration] Remove the need to set the MY_IP env var during OH hub setup. (#10)Date: Mon, Feb 24, 2020 6:20 AM
Normally, I use curl -s httpbin.org/ip | jq -r .origin to automatically get the external IP address. Since this approach depends on curl and jq commands, would you think, this would be a fairly platform agnostic solution? If yes, I can take a shot.
—You are receiving this because you are subscribed to this thread.Reply to this email directly, view it on GitHub, or unsubscribe.

PR #12 is now merged, so we might be able to close this issue now.

When I ran the command to set MY_IP on an AWS EC2 instance, there is no output. I noticed that the awk { print $2 } does not return a string that qualifies for the subsequent egrep.
ubuntu@ip-172-31-20-227:~/horizon-edgex$ ifconfig | egrep 'inet ' | awk '{ print $2 }'
<IP_ADDRESS>
<IP_ADDRESS>
<IP_ADDRESS>


Include files in device list

Based on experience, the devices.conf file can get very long, very fast. Support should be added to include configuration files. Perhaps even support script execution to generate the config on the fly. Purposed syntax:
@/path/to/file.conf
@/path/to/file2.conf

@!/path/to/script.sh

The output of any script execution will be used as part of the config file.

A few thoughts on implementation:

Read the root file storing it into memory. If a normal include is encountered, recursively open, read, and store returning a []byte to append to the parent and continue. If a script is encountered, execute it and append the result. Ultimately, a complete version of the inventory file would be in memory and can then be parsed as usual. Maybe store in a temporary file while building so as not to fill memory, but text shouldn't take too much.

Read the root file, process while reading as normal but with the main for loop in a separate function that can be called recursively for includes. Scripts would again be called and their output processed as a byte slice.

The first would be easiest to implement, but the second may be more efficient.

Including from a script is not yet implemented.


global variables ?

I'm trying to convert some CSS files to glamor syntax.
Currently I use CSS3 variables to define some common rules and calc to adapt thoses variables to my components context
#variables.css
:root {
--gutter: 1rem;
}

#myComponent.css
.myComponent {
margin:calc(var(--gutter) / 2)
}

I'm struggling to find a good way to translate that in glamor API.
So far I do things like that :
#variables.js
export const gutter = 1;

#myComponent.js`
import {gutter} from 'variables.js';
const style = css({
  margin:(gutter * 2) + 'rem'
});

But I'm not satisfied with the +'rem' part and the lack of unit in gutter definition.
Am I doing it wrong ? is there a better way ?

@sylvainbannier I am not sure this is better, but I usually do all that in my variables file (assuming that you reusing that margin a lot:
#variables.js
const GUTTER_VAL = 1
const MARGIN_VAL = GUTTER_VAL * 2

export const margin = `${MARGIN_VAL} rem`

#myComponent.js
import { margin } from 'variables.js'
import { style } from glamor

const styles = style({
  myComponentStyle: {
    margin
  }
})

there's a helper for 'vars' aided by react - https://github.com/threepointone/glamor/blob/master/docs/vars.md
I have a full fledged example under examples/tweet (an implementation of https://github.com/styled-components/comparison)
now, I don't have a good answer for the rem issue. Open to ideas; wondering what a plugin for the same would look like.

vars seems well suited for discrete values, but in my case I need to compute thoses values then append the unit.
Perhaps I don't get it but it seems that in https://github.com/threepointone/glamor/tree/master/examples/tweet there all sizes are statically defined (ex: https://github.com/threepointone/glamor/blob/master/examples/tweet/app.js#L33).

yeah I see the problem, haven't thought of a clean enough solution though.

maybe a calc() helper?
native calc has pretty wide support, maybe something like margin: calc(${vars.gutter} /2) would do the trick?

That what I used to do. But calc support is not as wide as I'd like :
http://caniuse.com/#search=calc (around 80%). That's why I was trying to replace it with js variables.

so do you suppose a calc() helper would solve your problem?
margin: calc(`${vars.gutter} /2`)

just an idea, of course. open to implementations/ PRs as well :)

Yes I think so. Except for UC browser but who cares :) ?
thanks all.


Helm upgrade fails upgrade ConfigMap

I keep having issues with ConfigMaps not getting updated on helm upgrade. It's not a permanent issue so I can't give an exact scenario how to reproduce. I'll try to give as much info as possible. When I'm saying it's not a permanent issue I mean it doesn't always happen when I'm changing ConfigMaps but when it does happen I can run the upgrade any number of times and the value won't change.
What I'm trying to do
I have a ConfigMap that looks like this. I'm trying to change the value of APP_DOMAIN from test.mydomain.com to develop.europa.mydomain.com.
kubectl describe cm jupiter-config
Name:		jupiter-config
Namespace:	develop
Labels:		<none>
Annotations:	kubectl.kubernetes.io/last-applied-configuration={"apiVersion":"v1","data":{"API_URL":"http://node-services","APP_DOMAIN":"test.mydomain.com","LOG_LEVEL":"info",...

Data
====
SHOW_ADS:
----
true
API_URL:
----
http://node-services
APP_DOMAIN:
----
test.mydomain.com
HTTP_DEBUG:
----
true
LOG_LEVEL:
----
info

I run a helm upgrade to change this value to develop.europa.mydomain.com. You can see the debug output from Helm. The upgrade completes successfully but the ConfigMap stays the same.
helm version output
Client: &version.Version{SemVer:"v2.4.1", GitCommit:"46d9ea82e2c925186e1fc620a8320ce1314cbb02", GitTreeState:"clean"}
Server: &version.Version{SemVer:"v2.4.1", GitCommit:"46d9ea82e2c925186e1fc620a8320ce1314cbb02", GitTreeState:"clean"}

helm upgrade output
REVISION: 61
RELEASED: Tue May 23 15:11:34 2017
CHART: jupiter-0.1.0
USER-SUPPLIED VALUES:
cluster: europa
config:
  API_URL: http://node-services
  APP_DOMAIN: develop.europa.mydomain.com
  HTTP_DEBUG: "true"
  LOG_LEVEL: info
  SHOW_ADS: "true"
resources:
  limits:
    cpu: 500m
    memory: 500Mi
  requests:
    cpu: 100m
    memory: 100Mi
scaling:
  maxReplicas: 4
  minReplicas: 2
version: 1

COMPUTED VALUES:
cluster: europa
config:
  API_URL: http://node-services
  APP_DOMAIN: develop.europa.mydomain.com
  HTTP_DEBUG: "true"
  LOG_LEVEL: info
  SHOW_ADS: "true"
resources:
  limits:
    cpu: 500m
    memory: 500Mi
  requests:
    cpu: 100m
    memory: 100Mi
scaling:
  maxReplicas: 4
  minReplicas: 2
version: 1

HOOKS:
MANIFEST:

---
# Source: jupiter/templates/config.yaml
kind: ConfigMap
apiVersion: v1
metadata:
  name: jupiter-config
data:
  API_URL: http://node-services
  APP_DOMAIN: develop.europa.mydomain.com
  HTTP_DEBUG: "true"
  LOG_LEVEL: info
  SHOW_ADS: "true"

It looks like the configmap was modified with kubectl.  Helm will diff on the last release rather than what is running in the cluster.

Yes that is probably the case. Thanks for the quick answer!

I had the same issue caused few 1 hour to figure out that helm failed to update configmap.
I think this is quite confusing to users, and once the configmap modified with kubectl, it won't update except 1) either update helm configmap values, 2) or delete the configmap and use helm to redeploy.
So could you add a flag, or recognize the --force flag, to not check with last release, but do check with the running one in cluster?


add license field

Summary

This addition is with reference to #385, to add licensing information to the buildpack.toml file. I used this BP commit as reference. I have PRs ready for all the other implementation BPs for Go, so if this looks alright I can PR everything else too.
Use Cases

As the issue states, this would allow users to see licensing information through their app image's BoM.
Checklist

[x] I have viewed, signed, and submitted the Contributor License Agreement.
[x] I have linked issue(s) that this PR should close using keywords or the Github UI (See docs)
[x] I have added an integration test, if necessary.
[x] I have reviewed the styleguide for guidance on my code quality.
[x] I'm happy with the commit history on this PR (I have rebased/squashed as needed).

I played around with this a bit and it appears that jam is dropping this metadata when it packages up the buildpack. I've opened this PR to make sure we keep these new fields when we package the buildpack up.

A version of jam has been released that now supports the licenses field. Are there any other blockers on this PR @ryanmoran?

@ryanmoran are there any further blockers on this PR (or the analogous ones across the go implementation repos?)

@ryanmoran are there any further blockers on this PR (or the analogous ones across the go implementation repos?)

I don't think so, but I haven't yet pulled this branch with the new version of jam to test it out.

After rebasing the PR so it's using jam v0.13.0, I can confirm that the buildpack.toml that's packaged into the buildpack.tgz includes:
[buildpack]
  homepage = "https://github.com/paketo-buildpacks/go"
  id = "paketo-buildpacks/go"
  name = "Paketo Go Buildpack"
  version = "9.9.9"

  [[buildpack.licenses]]
    type = "Apache-2.0"
    uri = "https://github.com/paketo-buildpacks/go/blob/main/LICENSE"


Improve IsRequiredSpecified performance

The IsRequiredSpecified method isn't slow by itself but with how frequently it can be called, it kinda still is. For example in the code I'm working on, the total time within IsRequiredSpecified is 3 seconds:

When you look at the memory allocations too, it gets kinda crazy:

The problem here is that calling GetCustomAttributesData is expensive. How expensive depends on the member itself but here is the results for a basic benchmark:

Now, while the original code specifically checked if the user specified a Required value at all, this new code only checks that it isn't the default value (the default value is that it is not required). The original code also has a mistake here as it treats setting the Required value to anything means it is required which isn't true (as you can set it to Required.Default which is to not require something).
Technically DisallowNull, according to the Newtonsoft docs says "The property is not required..." but I'm not really wanting to modify that type of behaviour. It is a lot more clear cut that Required.Default means it isn't required.

I guess with the subtle behaviour change, the method name doesn't really line up 100% anymore however that really only matters if someone wanted [JsonProperty(Required = Required.Default)] to actually make the property required (which it shouldn't).

Thanks for contributing - if you'd like to continue with this pull request, please rebase against the default branch to pick up our new CI.


Mipmap level not working with mod (1.16)

Mipmap slider doesn't change anything, and it seems to be stuck on 0 makign the game look pretty gritty, this goes away when I remove the mod

It's a really weird issue


Custom element creation is confusing

I'm submitting a bug report

Library Version:
1.10.2

Please tell us about your environment:

Operating System:
Windows 10

Node Version:
10.15.3

NPM Version:
6.4.1

JSPM OR Webpack AND Version
webpack 4.27.0

Browser:
Chrome 77

Language:
TypeScript 3.1.2

Current behavior:
I created a repro for our problem:
https://github.com/code-chris/aurelia-templating-issue
We define child-views as custom-elements (without CustomElement suffix or annotation). In all of our applications this works as expected (using with  tag or global registration).
Our current problem is, that the child-views are not initialized correctly (shown in the repro with the CommentItem and SolutionItem views.)
In the repro this only works with the customElement-Attribute. In one of our projects where this issue appears now, it works correctly if the Attribute is located on the SolutionItem, the DefaultItem seems not to need this...
I also had the issue that both item-views have switched their VM-Implementation if the Attribute were not set. So The SolutionItem.html was bound against the DefaultItem.ts and the other way round...
Expected/desired behavior:
It works as normal without the need of custom elements

With a very quick glance at the code, I'm guessing you miss a metadata-generating decorator in derived classes. This is a requirement for bindable - inheritance. Could you please help try and confirm it?

What i meant is: in either CommentItem or SolutionItem, decorate it with @bindable, or @customElement, just one is enough, so it has the right metadata. This is a legacy issue that we traverse to the root metadata holding class definition, because there was no intention to support inheritance at first. We can fix it, but the potential breakage is unknown.

Yes
class A {} // no issue
class B extends A {} // using class A metadata without its own metadata holder
class C extends A { @bindable c } // using class C metadata

okay, thanks for your quick response.
Then I think we can close this issue. It was only a bit weird 😃


Add support for whitelisting search paths

I usually have a directory containing all kinds of git repos so whitelisting repos helps a lot.

Thanks @rootmos, really nice one :)
Here is another idea, if you would like to extend on to it. How about we make it accept the filename from the args? For example if somebody has multiple roles e.g. you work as a freelancer as well as a fulltime employee, you could create multiple standup files specifying the projects for each and then you could pass the file path to the standup command to show the logs for that e.g. one could have two files called .freelance-standup and .fulltime-standup with the contents as below
freelance-project-a
freelance-project-b

and
office-project-a
office-project-b

and then pass it to git standup as
git standup -f .freelance-standup
git standup -f .fulltime-standup

What do you think?
But anyways, nice addition. Thanks :)


Access control for chaincode deployment?

Description
Is there a way to limit who can deploy chaincode to the network? In my scenario any client can submit a smart contract and I'd like more control over that aspect.

@iiiLiQuiDiii currently there is no ACL support for chaincode lifecycle. Its on the roadmap.

Thank you for the info.


Slow query

I have 8 records of 1 kind and I select them all. Response time is 500-2000 ms, is it expected?
Code example https://github.com/GoogleCloudPlatform/google-cloud-node/issues/2284
I also tried to use REST HTTP API to get 1 entity by Kind + ID and response time is 500 - 2000 ms
What am I doing wrong?

Hey, it seems like your results came from the authentication overhead from the first request (per the discussion on that issue. I'm closing this pre-emptively. If you continue to encounter performance issues and you're sure that they're not related to the environment or Node library, feel free to reopen.


<IP_ADDRESS> not working on NodeMCU 32S

When I did ur project everything was good, I flashed everything into the NodeMCU 32S, And when I try to access 192.
168.4.1, The browser keeps on loading. I used ESP-IDF

silly question but are you connected to the Management AP right?

What was the problem?

I did not connect to ManagmentAP my bad cos i did not have the usb dongle on my pc


x/tools/gopls: improve completion results when they depend on code after the current line in the same file

What version of Go are you using (go version)?
1.15.6 / gopls 0.6.0
Does this issue reproduce with the latest release?
yes
What operating system and processor architecture are you using (go env)?
linux/amd64
Description
A problem I frequently encounter using gopls is that if the completion target depends on code defined in the same file but after the cursor position, the completion is non-functional unless the expression is extremely simple.
Here are some concrete examples using vscode.
If the type S is defined before the current function and I start typing a range statement, completion works fine:

If the type S is defined after the current function and I start typing a range statement, completion is not useful (it suggests some irrelevant packages):

An if expression has the same problem:

But if I only type a simple s., completion works:

I understand that the problem of trying to parse partial (i.e., broken) code is, in the general case, impossible. But doing a passable job in the typical cases seems important. ISTM that a heuristic as simple as "try parsing the code after deleting the line that the cursor is on" would cover the examples I showed here and most of the instances I notice day to day.
I looked for an existing issue as best I could with GH issue search but didn't find anything.

try parsing the code after deleting the line that the cursor is on

The problem with that approach is that we need to parse the current line to know we are completing s. (i.e. to look for fields on the S struct).
See https://go-review.googlesource.com/c/tools/+/216482/ for a previous relevant fix. Adding phantom curlies (i.e. if s. {}) and re-parsing is one way to make things work. Perhaps fixMissingCurlies can be tweaked in some way to fix this case as well.

try parsing the code after deleting the line that the cursor is on

The problem with that approach is that we need to parse the current line to know we are completing s. (i.e. to look for fields on the S struct).
See https://go-review.googlesource.com/c/tools/+/216482/ for a previous relevant fix. Adding phantom curlies (i.e. if s. {}) and re-parsing is one way to make things work. Perhaps fixMissingCurlies can be tweaked in some way to fix this case as well.


Question: how to get current S.M.A.R.T. status

I please want to know if i always have to recall "SmartDrive.GetDrives" if i want to get the current values of SMART attributes?
Is it possible to get SMART only for a special drive? Calling GetDrives takes some seconds on my machine to get ALL drives (i have 20 drives and some network drives mapped wich dont have SMART).
Thanks.

It should be possible I think to filter the list of drives returned.
Checkout the WmiController class where the drives are being listed.
Currently all drives are returned using this query.
@"SELECT * FROM Win32_DiskDrive"
So I believe this query could be extended with a WHERE filter.
Another option would be to checkout the Win32_DiskDrive API docs and see if is possible to filter out network drives as I suspect/guess those drives are responsible for creating a delay when returning smart data.


When class membership probabilities are not provided by the user, the should be computed from the training data.

When --dictionary-format classes is used, i.e. the dictionary does not contain class membership probabilities, compute them from the word unigram probabilities in the training data.

This has been implemented.


Azure Search: Update sample service name

This checklist is used to make sure that common issues in a pull request are addressed. This will expedite the process of getting your pull request merged and avoid extra work on your part to fix issues discovered during the review process.
PR information

[X ] The title of the PR is clear and informative.
[ X] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For information on cleaning up the commits in your pull request, see this page.
[X ] Except for special cases involving multiple contributors, the PR is started from a fork of the main repository, not a branch.
[X ] If applicable, the PR references the bug/issue that it fixes.
[ X] Swagger files are correctly named (e.g. the api-version in the path should match the api-version in the spec).

Quality of Swagger

[ X] I have read the contribution guidelines.
[X ] My spec meets the review criteria:

[ X] The spec conforms to the Swagger 2.0 specification.
[ X] Validation errors from the Linter extension for VS Code have all been fixed for this spec. (Note: for large, previously checked in specs, there will likely be many errors shown. Please contact our team so we can set a timeframe for fixing these errors if your PR is not going to address them).
[X ] The spec follows the patterns described in the Swagger good patterns document unless the service API makes this impossible.

This change is

Hi @seansaleh, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

It looks like you're working at Microsoft (seasa). If you're full-time, we DON'T require a contribution license agreement.

If you are a vendor, DO please sign the electronic contribution license agreement. It will take 2 minutes and there's no faxing! https://cla.azure.com.

TTYL, AZPRBOT;

FYI @brjohnstmsft

LGTM

Merged

No modification for Python

No modification for NodeJS

No modification for Ruby


[#152800384] replace sendgrid with mailup

this cannot work in production until the orded is committed, but it's fine fot test environment.

Affected stories

🌟 #152800384: Migrare il servizio di invio email al provider MailUp

  Generated by :no_entry_sign: dangerJS


PipelineResourceResult Renamed and Moving to V1

In v1beta1 PipelineResourceResult is both a struct and a result type that was used to be in the resource related codebase. Now that we are removing pipelineResources, either renaming or a new design is required to be moved to V1.
There are 3 resultTypes of PipelineResourceResult:

pipelineResourceResultType will be removed
InternalTektonResultType
TaskRunResultType

This would need to be brought to v1 in some way:
The usage of PipelineResourceResult with respective types:

Git-init  image usage of Termination package for pod termination message via termination.WriteMessage - possibly blocker for TEP074 pipelineresources deprecation
Pod package status https://github.com/tektoncd/pipeline/blob/5b702f970d198c9c1d73f2636a8f30fc42faeeed/pkg/pod/status.go#L258 used as TaskRunResult.
For the spire package spire mock test etc as v1beta1.TaskRunResultType, learned from @jagathprakash  that this could be removed
Entrypointer package https://github.com/tektoncd/pipeline/blob/f732985ec30817703e6eeaa6914cc77a2b8adf3c/pkg/entrypoint/entrypointer.go as in InternalTektonResultType
Termination package

Goal of this issue is:

[ ] Decouple pipelineResourceResult
[ ] Remove in v1beta1 along with pipelineResouces
[ ] Introduce new struct in v1 along with resourceResult

[ ] the resourceResult is in v1beta1 but not moved to v1 so we have to introduce a new field

One question here is about the pkg/pod/status when will we move it to v1

/assign

How PipelineResourceResult has been introduced and removed, this is tracked to find the actual purpose of this struct:

First PR introducing PipelineResourceResult

https://github.com/tektoncd/pipeline/commit/3fe258464db1cc295a2658d7bc599841b76833d1#diff-5589d22b1568e49f5351ae729c0c0214014a7d88bd82c8b7ae31eb7ea9d41cb8:
So it was first introduced only fo add a field in the taskRun status to include the image name and
digest -> resourcesResult
Then its modification to add more generic functions
https://github.com/tektoncd/pipeline/commit/e01fe60d0f7ab31e2182835f9b9891f2c72700b9#diff-5589d22b1568e49f5351ae729c0c0214014a7d88bd82c8b7ae31eb7ea9d41cb8
Then made more generic to include other results
https://github.com/tektoncd/pipeline/pull/2011

/kind misc

Here are a couple of questions to be answered before moving forward after the offline conversation with:

In which package shall we move the PipelineResourceResult Struct to since resource_types is going to be remoed?

to not to introduce a new package, so to pkg/status
to include in pkg/apis/v1beta1/result_types since it is related to results

whether we could introduce a generics for result to be refactored for func WriteMessage and func ParseMessage

In which package shall we move the PipelineResourceResult Struct to since resource_types is going to be remoed?

no strong feelings - if the main use for this struct is writing to termination message maybe pkg/termination?

Shall we keep this Struct or refactor it to be more generic?

not sure what a more generic solution would be so a struct sounds find

what shall we rename the Struct to?

I like TektonTerminationMessage since this struct encapsulates the different kinds of things we write to the termination message

In which package shall we move the PipelineResourceResult

I think if we still use this struct in v1beta1.taskrun, we may need to keep it in the api?

I think the ResourceName can be safely deleted. I am not finding any usage left after we delete the pipeline resources related code.
The ResultType field of PipelineResourceResult is mainly used to identify whether a particular object is a taskResult (TaskRunResultType) or some other metadata (InternalTektonResultType)  from the termination message, such as, startedAt, exitCode, etc.
https://github.com/tektoncd/pipeline/blob/90f33a63497e8c07fa799f8cab5138756a075299/pkg/pod/status.go#L268
https://github.com/tektoncd/pipeline/blob/90f33a63497e8c07fa799f8cab5138756a075299/pkg/pod/status.go#L290

One question regarding pkg/pod/status: when will we move it to v1? I am assuming by the time when v1 is swapped as storage version?

Yes, pkg/pod/status.go will use v1.* instead of v1beta1.* eventually 🙃

I think if we still use this struct in v1beta1.taskrun, we may need to keep it in the api?

Yes, I am thinking the same, it will go under api.
The PipelineResourceResult can be stripped down to:
type PipelineResourceResult struct {
	Key          string     `json:"key"`
	Value        string     `json:"value"`
	ResultType   ResultType `json:"type,omitempty"`
}

There is a possibility of deleting ResultType from this struct if we set the Key to its actual value StartedAt and ExitCode either while parsing the termination message ParseMessage rather than wait until setting it in:
https://github.com/tektoncd/pipeline/blob/90f33a63497e8c07fa799f8cab5138756a075299/pkg/pod/status.go#L323
or while reading from a termination message in extractStartedAtTimeFromResults and extractExitCodeFromResults, once the Key is set to startedAt or ExitCode, instead of checking ResultType, check the Key.

whether we could introduce a generics for result to be refactored for func WriteMessage and func ParseMessage
Please elaborate on using generics. I couldn't envision using generics here but will be great if we could.

Thanks all for the inputs and guidance on this @afrittoli @pritidesai @dibyom @lbernick @Yongxuanzhang ,
There are two concerns related with the migration of PipelineResourceResult and taskrun.status.resourcesresult:

git-init image used to write PipelineResourceResult to taskrun.status.resourcesresult, so we need to first migrate it off from using PipelineResourceResult and then tombstoning the resourcesresult field
taskrun.status.resourcesresult filed has not been explicitly declared to be deprecated previously for users even though it would not function much without pipelineResources

Therefore, the following steps are planned to move forward targeting a cleanup for easier storage swap:

Migrating git-init image off from populating resourcesname (thanks to @lbernick
at https://github.com/tektoncd/pipeline/pull/6310)
Create a new Struct Result renamed from the PipelineResourceResult struct that is used by termination package; whilst we keep the old PipelineResourceResult struct for backward compatibility.
Refactor the usage of PipeilneResourceResult to be on actual TaskRunResult for git-init image and for pod status etc.

PipelineResourceResult is used as a means to read the container status and make it available to the taskRun status, for example, startedAt, exitCode, taskResults, failure message, etc.

This struct introduced ResultType to support identifying internal usage v/s pipeline resources.
Now, since pipeline resources are being deleted, please evaluate to see if ResourceName and ResultType are needed or not. Thanks!

A followup PR for #6434 will move the renamed struct to a proper space since resource_types is no longer used after the removal of all PipelineResources.

So far we have renamed and going to tombstone ResourcesResult + move the renamed RunResult to a different package.
One question remaining is that whether we are going to move this struct to v1 or a un-versioned package otherwise this would be left in v1beta1. 🤔


pwarp being able to be set in vanha

pwarps being able to be set is somthing very bad as someone could just set up a pwarp to a legend quest start or a checkpoint if they have proper money

Considering legend quest starting locations will be listed in the wiki, and that there are no "checkpoints" as such other than linear point-to-point objectives that there's no use teleporting to, I don't particularly view this as an issue. Thanks for the report!


Add paragraph mode to String#each_line in mrblib

#4511 demonstrated an infinite loop in String#each_line when  …
given an empty string separator. In MRI, an empty separator places
String#each_line in paragraph mode, where the String is separated on
successive runs of newlines. In paragraph mode, the String
"abc\n\n\ndef\nxyz" is split into ["abc\n\n\n", "def\nxyz"].
This commit makes the String#each_line implementation as close to
ruby/spec compliant as possible given the limitations of mruby core.
With this patch, the following specs fail for String#each_line:

uses $/ as the separator when none is given (can be fixed by
aliasing and redefining the method to use $/ as the default value
of separator in mruby-io)
when no block is given returned Enumerator size should return nil
(Enumerator#size is not supported on mruby)
tries to convert the separator to a string using to_str (String#to_str
is not implemented on mruby)

This patch has similar memory consumption compared to the prior
implementation and is takes 4x the time the prior implementation takes to
execute:
/usr/bin/time -l ./bin/mruby -e '("aaa\n\nbbbbb\n\n\n\n\ncccc" * 100000).each_line("\n") { }';

cc @dearblue
Fixes GH-4511.

The second argument seems to be unnecessary.

@shuujii the second argument is specified by CRuby. I've included it for compatibility.

My latest commit improves performance of the base case to be within a factor of 2 of the existing implementation.

Because the second argument isn't used, it is better to remove it or implement correctly, I think.
BTW, I think unless block is faster than unless block_given? (ref. ef3bbd2e).

@lopopolo, there are two things I noticed in order to optimize the speed.

Pass position and length instead of passing range to get substrings.
Avoid range objects being generated each time in the loop.
Assign a method call whose value does not change in a loop to a variable outside the loop.
Because == is also a method, branch by putting a boolean value into the variable.

Because the second argument isn't used, it is better to remove it or implement correctly, I think.

I've removed this in the latest patch.

The latest implementation is faster than the existing implementation while remaining as memory efficient. To test performance I ran this command:
/usr/bin/time -l ./bin/mruby -e '1000.times { ("aaa\n\nbbbbb\n\n\n\n\ncccc" * 10000).each_line("\n") { } }';

Existing mrblib implementation
mruby@master
       17.91 real        17.84 user         0.06 sys
  54132736  maximum resident set size
         0  average shared memory size
         0  average unshared data size
         0  average unshared stack size
     45916  page reclaims
         0  page faults
         0  swaps
         0  block input operations
         0  block output operations
         0  messages sent
         0  messages received
         0  signals received
         0  voluntary context switches
      1472  involuntary context switches

This PR at 8ea45b8
mruby@string-each-line-paragraph-mode-8ea45b86
       17.91 real        17.84 user         0.05 sys
  54140928  maximum resident set size
         0  average shared memory size
         0  average unshared data size
         0  average unshared stack size
     45915  page reclaims
         0  page faults
         0  swaps
         0  block input operations
         0  block output operations
         0  messages sent
         0  messages received
         0  signals received
         0  voluntary context switches
      1343  involuntary context switches

This PR at 6b92acf3
mruby@string-each-line-paragraph-mode-6b92acf3
       17.40 real        17.34 user         0.05 sys
  54132736  maximum resident set size
         0  average shared memory size
         0  average unshared data size
         0  average unshared stack size
     45914  page reclaims
         0  page faults
         0  swaps
         0  block input operations
         0  block output operations
         0  messages sent
         0  messages received
         0  signals received
         0  voluntary context switches
      1219  involuntary context switches

@shuujii and @dearblue thank you for helping me iterate on this PR. I think I've addressed all of the feedback which resulted in an implementation that is faster than the existing one while still being mostly spec-compliant. 🎉

It's a great :+1:
I realized I could replace it with a common loop.
example:
if separator.empty?
  separator = "\n\n"
  paragraph_mode = true
end

while (pointer = string.index(separator, start))
  pointer += sep_len
  if paragraph_mode
    pointer += 1 while string[pointer] == "\n"
  end
  if should_yield_subclass_instances
    ...snip...

I'm worried about whether it fits your spec and the speed down, but I think the mruby bytecode size will be smaller.
Becomes like the dispensing of small information, I'm sorry.

@dearblue thanks for the optimization! That does end up cutting down the code size quite a bit and it does still pass ruby-spec. 🎉
I think this patch is ready for final review.


Getting Started with eksctl

https://karpenter.sh/v0.18.0/getting-started/getting-started-with-eksctl/
It didn't work in my environment.
helm upgrade --install karpenter oci://public.ecr.aws/karpenter/karpenter --version ${KARPENTER_VERSION} --namespace karpenter --create-namespace \   --set serviceAccount.annotations."eks\.amazonaws\.com/role-arn"=${KARPENTER_IAM_ROLE_ARN} \   --set clusterName=${CLUSTER_NAME} \   --set clusterEndpoint=${CLUSTER_ENDPOINT} \   --set aws.defaultInstanceProfile=KarpenterNodeInstanceProfile-${CLUSTER_NAME} \   --wait # for the defaulting webhook to install before creating a Provisioner
I met error when I run above is below.
`Error: "helm upgrade" requires 2 arguments
Usage:  helm upgrade [RELEASE] [CHART] [flags]
`
But it worked so in helm command misunderstood comment out in this getting started.
Could you delete comment out?
helm upgrade --install karpenter oci://public.ecr.aws/karpenter/karpenter --version ${KARPENTER_VERSION} --namespace karpenter --create-namespace \   --set serviceAccount.annotations."eks\.amazonaws\.com/role-arn"=${KARPENTER_IAM_ROLE_ARN} \   --set clusterName=${CLUSTER_NAME} \   --set clusterEndpoint=${CLUSTER_ENDPOINT} \   --set aws.defaultInstanceProfile=KarpenterNodeInstanceProfile-${CLUSTER_NAME} \   --wait

Sorry, I'm not seeing the issue. I copied/pasted the command from the getting started guide and the installation of v0.18.0 worked for me.

Could it be possible you ran this in a shell that didn't have those environment variables set? Maybe try resetting them and re-running the command?


Support for auth() in @default annotation

Is your feature request related to a problem? Please describe.
If i already set the UserId when creating the EnhancedPrisma it would be nice not have to pass an additonal argument around for the user or session to ensure a field is populated with the right value.
Describe the solution you'd like
model Post {
    id String @id
    title String @length(1, 100)
    author User @relation(fields: [authorId], references: [id])
+    authorId String @default(auth().id)

    @@allow('update', author == auth() && future().author == author)
}

Additional features that would be nice might be using other fields related to the user.
model Post {
    id String @id
    title String @length(1, 100)
    author User @relation(fields: [authorId], references: [id])
    authorId String @default(auth().id)

+   tenantId String  @default(auth().tenantId)

    @@allow('update', author == auth() && future().author == author)
}

Yeah i think it would be fine and expected to have the runtime error, unless the author was Optional.


Fix "FIXME" issue in src/library/Abstract/AdhesiveHLR.hs

FIXME found
https://codeclimate.com/github/Verites/verigraph/src/library/Abstract/AdhesiveHLR.hs#issue_5915ed1aabd07f00010000b6

File moved


Beware of cyclic objects...

Try to marshal (i.e. send to native code) a cyclic object, verify there is no stack overflow.

Does not work. This:
console.log process

Currently produces this message:
Error: user error: object

While it's supposed to dump the contents of the process object, as when console.log(process) is run in Node.

The user error is actually a JavaScript exception caused by the stack overflow.


When app is in foreground, how to push the notification to notification bar.

When app is in use and notification comes, is there any way to send that notification to notification bar of the device. Is there any way to notify the user

In the onNotificationOpen method you should call what happens when user receives notification, for example "alert(notification.data)"
This way you can notify the user while using the app.

It works for me (for iOS 10 only):
#if defined(__IPHONE_10_0) && __IPHONE_OS_VERSION_MAX_ALLOWED >= __IPHONE_10_0

(void)userNotificationCenter:(UNUserNotificationCenter *)center
willPresentNotification:(UNNotification *)notification
withCompletionHandler:(void (^)(UNNotificationPresentationOptions))completionHandler {
...
completionHandler(UNNotificationPresentationOptionSound | UNNotificationPresentationOptionAlert | UNNotificationPresentationOptionBadge);
}

And if anybody wants to handle clicks on notification in foreground, change application:didReceiveRemoteNotification:fetchCompletionHandler method setting YES for 'tap':

(void)application:(UIApplication *)application didReceiveRemoteNotification:(NSDictionary *)userInfo
fetchCompletionHandler:(void (^)(UIBackgroundFetchResult))completionHandler {
...
[mutableUserInfo setValue:@(YES) forKey:@"tap"];
...
}

@Ioama @StasKalishenko  @harshvardhan12 - did any of you ever get notifications working on Android when app is open and in foreground?  I got it working on iOS but not on Android.
If app is closed or open in the background - I got it working on both Android & iOS.  But if open and in foreground I only have it working on iOS - not on Android.
In fact, on Android, when a message arrives while app is open in foreground nothing happens - I can't even get a console.log to fire in the success callback (like Ioama's example above) - but works just fine in iOS.
I am on Android 6x...wondering if that has anything to do with it?  Oh, and I am literally just using native Javascript - as defined in the plugin specs.
Here is my code:
  window.FirebasePlugin.onNotificationOpen(function(payload) {
    alert(payload.data) ;  // open/foreground - this should fire when but does not
    console.log("here we are") ;
    // the below all works just fine if app is closed or open/background.
    // if there is a payload it will be in payload object
    if (payload.action == 1) { // email verification confirmation
      setDB("user_emailVerify",payload.user_emailVerify) ;
    } else if (payload.action == 2) {
      alert(payload.dataTitle+ "\n" +payload.dataMsg) ;
    }
  }, function(error) {
      console.error(error);
  }) ;

closing for now, reopen if still an issue with the latest version of the plugin

cordova - how to show firebase notification when the app is in foreground?
Is the a method I can use to trigger notifications while application is open,
It works perfect when the application is closed.

on AppDelegate+FCMPlugin.m
use UNNotificationPresentationOptionAlert
// Change this to your preferred presentation option
completionHandler(UNNotificationPresentationOptionAlert);


docs: update to use setGlobalConfig

Ref #129.
Changes

Update docs to use setGlobalConfig instead of _opts directly

Type

[ ] New Module
[ ] Other New Feature
[ ] Validation Fix
[ ] Other Bugfix
[x] Docs
[ ] Chore/other

Amazing, thanks, @PythonCreator27!

:tada: This PR is included in version 1.11.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Support Parcelable types for Android

This came up on the mailing list, and sounds like it might be very useful for a lot of people:
https://groups.google.com/forum/#!topic/jsonschema2pojo-users/IX0g58VAS58
More details:
http://developer.android.com/reference/android/os/Parcelable.html

I've just rebased the parcelable branch against master and added a proper 'parcelable' config option. If anyone would like to test the parcelable support, just build a snapshot from this branch:
https://github.com/joelittlejohn/jsonschema2pojo/tree/parcelable
Then activate the parcelable support like:
<parcelable>true</parcelable>

@jsuksl @premnirmal @LOG-TAG if you can give this a test and provide some feedback, this would be very helpful. I don't really know if the current implementation approach will work or not.

@joelittlejohn The latest https://github.com/joelittlejohn/jsonschema2pojo/tree/parcelable works fine. (BTW,  I see jsonschema2pojo-maven-plugin build fails on parcelable branch)

@joelittlejohn https://github.com/joelittlejohn/jsonschema2pojo/tree/parcelable works fine. BTW, I see jsonschema2pojo-maven-plugin build fails.

Thanks @jsukl! Could you give me some details of how you tested this? What kind of object did you parcel and what environment did you use to test this?
Also, I don't see any problems with the build when I run mvn clean verify on the parcelable branch. What build failure did you see?

I used jsonschema2pojo built from https://github.com/joelittlejohn/jsonschema2pojo/tree/parcelable to convert json schema files(30+) to POJO classes which contains information for UI components. few of the POJO classes are used as a parameter value( "http://developer.android.com/reference/android/content/Intent.html#putExtra(java.lang.String, android.os.Parcelable)" )  and can be written to and restored from Parcel successfully.
I encountered the errors below when I run  mvn clean verify
[INFO] Reactor Summary:
[INFO]
[INFO] jsonschema2pojo ................................... SUCCESS [53.950s]
[INFO] jsonschema2pojo-core .............................. SUCCESS [20.446s]
[INFO] jsonschema2pojo-ant ............................... SUCCESS [0.758s]
[INFO] jsonschema2pojo-cli ............................... SUCCESS [4.502s]
[INFO] jsonschema2pojo-gradle-plugin ..................... SUCCESS [6.990s]
[INFO] jsonschema2pojo-maven-plugin ...................... FAILURE [5.623s]
[INFO] jsonschema2pojo-integration-tests ................. SKIPPED
[INFO] ------------------------------------------------------------------------
[INFO] BUILD FAILURE
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 1:34.933s
[INFO] Finished at: Tue Mar 10 10:38:07 JST 2015
[INFO] Final Memory: 34M/81M
[INFO] ------------------------------------------------------------------------
[ERROR] Failed to execute goal org.apache.maven.plugins:maven-compiler-plugin:2.3.2:testCompile (default-testCompile) on project jsonschema2pojo-maven-plugin: Compilation failure: Compilation failure:
[ERROR] /Users/jsuk/Documents/play_ground/jsonschema2pojo/jsonschema2pojo-maven-plugin/src/test/java/org/jsonschema2pojo/maven/MatchPatternsFileFilterTest.java:[49,8] cannot find symbol
[ERROR] symbol  : method assertThat(java.lang.String,java.util.List<java.io.File>,org.hamcrest.Matcher<java.lang.Iterable<java.lang.Object>>)
[ERROR] location: class org.jsonschema2pojo.maven.MatchPatternsFileFilterTest
[ERROR] /Users/jsuk/Documents/play_ground/jsonschema2pojo/jsonschema2pojo-maven-plugin/src/test/java/org/jsonschema2pojo/maven/MatchPatternsFileFilterTest.java:[67,8] cannot find symbol
[ERROR] symbol  : method assertThat(java.lang.String,java.util.List<java.io.File>,org.hamcrest.Matcher<java.lang.Iterable<java.lang.Object>>)
[ERROR] location: class org.jsonschema2pojo.maven.MatchPatternsFileFilterTest
[ERROR] -> [Help 1]
[ERROR]
[ERROR] To see the full stack trace of the errors, re-run Maven with the -e switch.
[ERROR] Re-run Maven using the -X switch to enable full debug logging.
[ERROR]
[ERROR] For more information about the errors and possible solutions, please read the following articles:
[ERROR] [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/MojoFailureException
[ERROR]
[ERROR] After correcting the problems, you can resume the build with the command
[ERROR]   mvn <goals> -rf :jsonschema2pojo-maven-plugin

Hmm, I think the assertThat call is confusing the compiler as the varargs are making it ambiguous. I don't see this error when I run the build though. Could you let me know your OS, JDK version and Maven version?

Here you are
Apache Maven 3.1.1<PHONE_NUMBER>757ffbf44136acec0402957f723d9a; 2013-09-18 00:22:22+0900)
Maven home: /usr/local/Cellar/maven/3.1.1/libexec
Java version: 1.6.0_65, vendor: Apple Inc.
Java home: /System/Library/Java/JavaVirtualMachines/1.6.0.jdk/Contents/Home
Default locale: en_US, platform encoding: MacRoman
OS name: "mac os x", version: "10.10.2", arch: "x86_64", family: "mac"

The errors have been resolved after letting maven use JDK 1.7
[INFO]
[INFO] --- maven-failsafe-plugin:2.7.2:verify (verify) @ jsonschema2pojo-integration-tests ---
[INFO] Failsafe report directory: /Users/jsuk/Documents/play_ground/jsonschema2pojo/jsonschema2pojo-integration-tests/target/failsafe-reports
[INFO] ------------------------------------------------------------------------
[INFO] Reactor Summary:
[INFO]
[INFO] jsonschema2pojo ................................... SUCCESS [27.635s]
[INFO] jsonschema2pojo-core .............................. SUCCESS [5.232s]
[INFO] jsonschema2pojo-ant ............................... SUCCESS [0.860s]
[INFO] jsonschema2pojo-cli ............................... SUCCESS [3.106s]
[INFO] jsonschema2pojo-gradle-plugin ..................... SUCCESS [6.553s]
[INFO] jsonschema2pojo-maven-plugin ...................... SUCCESS [20.631s]
[INFO] jsonschema2pojo-integration-tests ................. SUCCESS [33.722s]
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 1:38.148s
[INFO] Finished at: Wed Mar 11 00:20:56 JST 2015
[INFO] Final Memory: 91M/545M
[INFO] ------------------------------------------------------------------------
Mac-b8e8560fa544:jsonschema2pojo jsuk$ mvn -v
Apache Maven 3.1.1<PHONE_NUMBER>757ffbf44136acec0402957f723d9a; 2013-09-18 00:22:22+0900)
Maven home: /usr/local/Cellar/maven/3.1.1/libexec
Java version: 1.7.0_45, vendor: Oracle Corporation
Java home: /Library/Java/JavaVirtualMachines/jdk1.7.0_45.jdk/Contents/Home/jre
Default locale: en_US, platform encoding: UTF-8
OS name: "mac os x", version: "10.10.2", arch: "x86_64", family: "mac"

I've included 'experimental' support for this in master. People can give this a try and provide some feedback if they have problems.

thanks!


Option to disable tree felling?

I'm trying to balance felling trees, and I used the mod Felling to add an enchant which makes it available. The trees from your mod get completely destroyed even when punched with bare hands. Could we get an option to turn off tree felling from your mod? Or some sort of an alternative? It's kind of OP right now imho.

The tree felling is how the mod is designed. If you don't like that, get a different mod.


[Bug] Radar plot using plotly not displaying

Running model-comparison.ipynb I realized the radar plot is not displaying.
I fixed by adding the following code:
import plotly.io as pio
pio.renderers.default = 'notebook'

Thank you for reporting this bug.
I tried the model-comparison.ipynb notebook on my dev env, with the fix that you proposed, I still face the same issue.
Can you help explain more?

@jcassiojr can you please provide update on this PR whether the issue has been fixed or not?


http (no -s) or nabu casa connectivity

Hi,
I just wanted to ask if maybe something has changed over last two years in regard to being able to connect via nabu casa or a local HTTP?
I'd love to use the app, but no way Im gonna do local https, too many complications :(

I'm really sorry, but since I never got feedback from Fitbit, I stopped developing the app. I understand the complications about HTTP myself.

Thanks for the reply, hoped I'll get it somehow to work, but no luck :(


chore: light mode for ContainerConnectionStatusInfo

it will be handled by https://github.com/containers/podman-desktop-extension-ai-lab/pull/1317


Docker installation problem

Hi
I am following the instructions to install hydrachain using docker. Whilst building using docker-compose build. I am having problems with not being able to find a suitable distribution for devp2p. It seems to be throwing what seems to be a dependency error of pyethapp.
...

Searching for devp2p>=0.7.1
Reading https://pypi.python.org/simple/devp2p/
No local packages or download links found for devp2p>=0.7.1
error: Could not find suitable distribution for Requirement.parse('devp2p>=0.7.1')
Service 'node' failed to build: The command '/bin/sh -c python setup.py install' returned a non-zero code: 1

Hi, I am facing this issue, too.
Do you have any suggestions how I could resolve the issue?
System: Virtual machine: Ubunut 64 Bit (15.10).
Exact error message:
Installed /usr/local/lib/python2.7/site-packages/ipython-4.1.2-py2.7.egg
Searching for devp2p>=0.7.1
Reading https://pypi.python.org/simple/devp2p/
No local packages or download links found for devp2p>=0.7.1
error: Could not find suitable distribution for Requirement.parse('devp2p>=0.7.1')
ERROR: Service 'node' failed to build: The command '/bin/sh -c python setup.py install' returned a non-zero code: 1
~~~

Hello, I'm struggling with the same error on ubuntu 15.10 64bit, with docker 1.11 and docker-compose 1.5.2. Has anyone found a solution yet?

Wish i could help but couldn't find solution to it

ditto on:
user@user:~/hydrachain$ docker-compose --version
docker-compose version 1.6.0rc1, build d2556a1
user@user:~/hydrachain$ docker version
Client:
 Version:      1.10.3
 API version:  1.22
 Go version:   go1.5.3
 Git commit:   20f81dd
 Built:        Thu Mar 10 15:54:52 2016
 OS/Arch:      linux/amd64

Server:
 Version:      1.10.3
 API version:  1.22
 Go version:   go1.5.3
 Git commit:   20f81dd
 Built:        Thu Mar 10 15:54:52 2016
 OS/Arch:      linux/amd64

In case anyone wants to workaround this, I've found that adding:
RUN pip install devp2p

before
RUN python setup.py install

in hydrachain/node/Dockerfile makes the issue go away.

We'll have to re-build the docker example because of changes in docker itself. This is being tracked in #68