Running manim via Docker can't find output directory

I've successfully installed docker to run manim but I get the following error message.
$docker run --rm -v "$PWD/files":/app/files manim example_scenes.py WarpSquare
Traceback (most recent call last):
  File "extract_scene.py", line 14, in <module>
    from helpers import *
  File "/app/helpers.py", line 13, in <module>
    from constants import *
  File "/app/constants.py", line 91, in <module>
    """)
Exception:
        Redefine MEDIA_DIR in constants.py to point to
        a valid directory where movies and images will
        be written

I did change the MEDIA_DIR directory in constants.py to a folder that exists and I've even run os.path.exists(MEDIA_DIR) in separate Python file and it does return True.

I've worked around this issue like below because I couldn't be brought to changing one line of code and waiting for another image build:
mkdir out
docker run --rm -v `pwd`/out:"/root/Dropbox (3Blue1Brown)/3Blue1Brown Team Folder" manim example_scenes.py WarpSquare

Now it has this to say:
Writing to /root/Dropbox (3Blue1Brown)/3Blue1Brown Team Folder/animations/example_scenes/1080p60/WarpSquareTemp.mp4
Animation 0: ApplyPointwiseFunctionSquareToSquare: 100%|████████████████████████████████████████████████████████████████████| 180/180 [00:04<00:00, 39.91it/s]
Played a total of 1 animations
sh: 1: play: not found

and the video is a 4 second long black screen. You'll find it in ./out/animations/example_scenes/1080p60/WarpSquareTemp.mp4

The black screen problem when running with docker seems to be common.  It looks like the PNGs don't get generated correctly.  I've tested the ffmpeg with a video transcode and it works.  I've also manually built the docker image with a debian base and the issue persists.
See https://github.com/3b1b/manim/issues/170 https://github.com/3b1b/manim/issues/119

It still says  Writing to /app/example_scenes/1440p60/WarpSquareTemp.mp4
ishandutta2007@MacBook-Pro:~/Documents/Projects/manim$ mkdir out

mkdir: out: File exists
ishandutta2007@MacBook-Pro:~/Documents/Projects/manim$ docker run --rm -v `pwd`/files:/app/files -v `pwd`/out:"/root/Dropbox (3Blue1Brown)/3Blue1Brown Team Folder" manim example_scenes.py WarpSquare

Animation 0: ApplyPointwiseFunctionSquareToSquare: 100%|##########| 180/180 [00:16<00:00, 10.77it/s]
sh: 1: play: not found
Writing to /app/example_scenes/1440p60/WarpSquareTemp.mp4
Played a total of 1 animations
ishandutta2007@MacBook-Pro:~/Documents/Projects/manim$ ls out
ishandutta2007@MacBook-Pro:~/Documents/Projects/manim$

Closing, as this issue is addressed in #296.


Amendments for test and cover commands

Add nyc as a dev dependency.
Modify test and cover commands to use glob-run so that
glob expansion works on Windows.
Add .coveralls.yml to .gitignore.

thanks 👍

I fixed the bug in test-runner which was preventing it expanding glob patterns on Windows, therefore glob-run is no longer needed here - it is now removed on the master branch. Please update your local branch.
Also, please update your npm to the latest version as the package-lock you pushed contained entirely fixed semver paths (i.e. the ^ was missing) which is an old bug - you need to update npm.


Responding to "events"

Hi, first of all I was thinking about that problem for a few days and tried different approaches but none was really satisfying.
Real life example. Im having systems that are responsible for:
-collision
-calculating velocity/gravity
-animating object
-rendering object
-logic (searching for enemies, regenerating hp etc)
For example, player is falling down from a platform so he gains velocity, because of time and gravity variables that are responsible for calculating it. Then he suddenly hits bottom platform, and polygons are separated but this is the point where im having a problem - right now velocity components MUST be informed that time of fall should be set to 0 and forces should be set to 0 as well. Also animation component must be requested to discontinue animating current display object and instead to show "hit" or "fell" animation. Last but not least - logic component must know that player hit ground so it will be possible once again for him to use "jump" action.
How can I achieve all of that without sensless coupling and injecting components into nodes/systems that should not ever have access to it. I thought about some "state" component but how should it work? how should i notify about continous state for example "falling" and about one-shot type of state "just fell."
Thats only really big architecture problem so far for me with xember.
Best Regards,
Pawel.

Hi Pawel,
Thanks for your question. It's a tricky issue, for sure. There are several approaches that you can take.
Firstly, you could inject a signal or event dispatcher into your system, then when an event occurs dispatch that, and bind the event to a Command. I don't believe that a fundamental division between entity systems and MVCS(RobotLegs) exists, so I'm an advocate of using the best of both approaches, depending upon the circumstances. Clearly, since you think you have an "event" use case, it feels like it could be best-served being handled in a Command.
The downside to this approach is that your command gets instantiated, executed and destroyed. That's pretty costly if you have a high-performance game (please note though that this is rarely an issue unless you're contemplating 100s of collisions per frame, and there are probably other performance bottle-necks that should concern you well before this one). If though it is a problem, then you could add a CollisionComponent to the entity that would then handle all collisions through a CollisionSystem. Those components could be object-pooled...
The real answer though Pawel is that there are as many choices as you can think of. There is no One Pattern To Rule Them All unfortunately. I would need to play with your project, understand the way you like to wire things up and how you think, as well as understand the project requirements, before I could say "in this situation I would do X". Hope this helps.
Alec


introduce IntervalTimer and use it instead of threading.Timer

threading.Timer only runs the function once, and then stops the thread.
To get an interval timer, we used to start a new thread when the original
thread expired. This is wasteful, and in the worst case can lead to
runoff creation of timer threads (#450).
Instead, the IntervalTimer will run in an endless loop, executing the
given function every n seconds.

closed in 5337c43


PBS design updates

Change layout for some components to match the design.
Add css ids to make it easier to style components.

looks good to me, nice job @bogdan-cornianu !


BouncyListView

Add bouncy support for ListView

won't be worked on anymore, please use BouncyRecyclerView for now

won't be worked on anymore, please use BouncyRecyclerView for now


Add pattern to ValueBlock to only allow number input.

Adds a "'pattern=[0-9]+" attribute to the value field so only numbers are allows as input.

@Parbhat  You are of course correct, will close this. It's by design that the numbers type allow commas and periods in number like 100.000.00. Will open a new issue to dela with that. Thanks!


Generalize the positioning and sizing concept for windows

A generalized concept / mechanism for positioning and sizing for the module windows should be discussed and implemented.
Currently the positioning is done with different technologies (CSS vs. initPos in config). This should be cleaned up.
Should also solve the problem with the minimizable option set on a floating window, which will sometimes cause the window to shrink in width.
Taken from the idea of @fschmenger in #213.

A first quick fix could be to support more positioning options like bottom, right, etc. in initPos and remove the corresponding css. Sizing properties are already defined in the module card base and just have to be used by the individual modules. #219 will also support configuration of these properties via app-conf.json.
On the long run I think you should come up with a more sophisticated concept, which is invariant for different screen sizes and scales well for mobile devices.
It's not on my personal list of features to implement, as I'm moving towards sidebars. However feel free to get back to me if you need some feedback.


Can't use existing IAM role in spotty

In more managed environments, you don't always have the flexibility to create your own IAM role versus use the one that is assigned to you. Spotty should have the ability to use an already existing IAM role if it has the right permissions.
Opening this as an issue so I can keep track of it, I'll be submitting a pull request next week likely.

At the moment Spotty creates an IAM role and an Instance Profile that wraps that role to use it with an EC2 instance. I assume that in some environments users also cannot create their own Instance Profile. Maybe then better to add functionality to use an existing Instance Profile?
Let's say it will be the instanceProfileArn parameter, and if it's specified then Spotty will use it instead of creating one. What do you think?
Also, I have a question about your environment: do you have permissions to create your own bucket? Or is it an issue as well? Because Spotty creates a bucket that is being used to sync the project and download files.

Hello, I also have issues with using Spotty in a managed AWS environment and would love the ability to provide a predefined IAM Role and Instance Profile as a configuration option. Is this feature in the pipeline and what would you estimate the level of effort required to implement?
To answer your S3 bucket question: yes I can create S3 buckets in my particular environment. In more restrictive environments where wholesale bucket creation is not an option I imagine that admins could grant users permissions to create buckets under some ARN naming restrictions. An admin may allow a user to create a private bucket conforming to some naming convention as long as the bucket is private, for example.

Apologies, I did actually end up modifying the spotty code to allow this, but I don't remember what I did and I've since lost the code. The fix wasn't too difficult if I remember correctly.

@tsdalton Actually, it's straight forward. I cannot promise that I'll release this feature in the following days as I'm a little busy with another project, but maybe I'll get some spare time.
For now, you can just clone the repo and replace the call that creates the Instance Profile stack with your Instance Profile ARN here:
https://github.com/apls777/spotty/blob/e67cae8afec893ec32c80a2f158a9fe3566b144c/spotty/providers/aws/deployment/instance_deployment.py#L73
But if you'll also create the instanceProfileArn parameter in the configuration file and will use it when it's specified, feel free to open a PR :).

@apls777 Thanks I have indeed updated my fork to surface the instanceProfileArn parameter to the config file so I'll open up a pull request when I get a chance to clean things up and merge-ready, probably towards the end of the week.

@apls777 sorry it took me so long, got sidetracked. I've opened a pull request, please review it at your leisure!

Thank you @tsdalton, this functionality now released with Spotty 1.3.0.


Improved detection for connections to miner pools

Signed-off-by: beryxz<EMAIL_ADDRESS>What type of PR is this?
/kind feature
/kind rule-update
Any specific area of the project related to this PR?
/area rules
/area proposals
What this PR does / why we need it:
While testing I noticed that the Detect outbound connections to common miner pool ports rule didn't work against "xmrig". The reason is that the sendto syscall is only used with the DNS server as shown:
1336 xmrig (1980:12) > socket domain=2(AF_INET) type=526338 proto=0
1337 xmrig (1980:12) < socket fd=14(<4>)
1338 xmrig (1980:12) > bind fd=14(<4>)
1339 xmrig (1980:12) < bind res=0 addr=<IP_ADDRESS>:0
1340 xmrig (1980:12) > sendto fd=14(<4><IP_ADDRESS>:0) size=38 tuple=<IP_ADDRESS>:38156-><IP_ADDRESS>:53
1341 xmrig (1980:12) < sendto res=38 data=.$...........xmr-eu1.nanopool.org.....
1342 xmrig (1980:12) > sendto fd=14(<4u><IP_ADDRESS>:38156-><IP_ADDRESS>:53) size=38 tuple=<IP_ADDRESS>:38156-><IP_ADDRESS>:53
1343 xmrig (1980:12) < sendto res=38 data=.$...........xmr-eu1.nanopool.org.....
1344 xmrig (1980:12) > sendto fd=14(<4u><IP_ADDRESS>:38156-><IP_ADDRESS>:53) size=38 tuple=<IP_ADDRESS>:38156-><IP_ADDRESS>:53
1345 xmrig (1980:12) < sendto res=38 data=.............xmr-eu1.nanopool.org.....
1346 xmrig (1980:12) > sendto fd=14(<4u><IP_ADDRESS>:38156-><IP_ADDRESS>:53) size=38 tuple=<IP_ADDRESS>:38156-><IP_ADDRESS>:53
1347 xmrig (1980:12) < sendto res=38 data=.............xmr-eu1.nanopool.org.....

After retrieving the IP of the pool, write is used in place of send, I don't know exactly why, but they are equivalent after all.
1364 xmrig (1978:<NA>) > socket domain=2(AF_INET) type=526337 proto=0
1365 xmrig (1978:<NA>) < socket fd=14(<4>)
1366 xmrig (1978:<NA>) > setsockopt
1367 xmrig (1978:<NA>) < setsockopt res=0 fd=14(<4>) level=2(SOL_TCP) optname=0(UNKNOWN) val=.... optlen=4
1368 xmrig (1978:<NA>) > setsockopt
1369 xmrig (1978:<NA>) < setsockopt res=0 fd=14(<4>) level=1(SOL_SOCKET) optname=9(SO_KEEPALIVE) val=1 optlen=4
1370 xmrig (1978:<NA>) > setsockopt
1371 xmrig (1978:<NA>) < setsockopt res=0 fd=14(<4>) level=2(SOL_TCP) optname=0(UNKNOWN) val=<... optlen=4
1372 xmrig (1978:<NA>) > setsockopt
1373 xmrig (1978:<NA>) < setsockopt res=0 fd=14(<4>) level=2(SOL_TCP) optname=0(UNKNOWN) val=.... optlen=4
1374 xmrig (1978:<NA>) > setsockopt
1375 xmrig (1978:<NA>) < setsockopt res=0 fd=14(<4>) level=2(SOL_TCP) optname=0(UNKNOWN) val=.... optlen=4
1376 xmrig (1978:<NA>) > connect fd=14(<4>)
1377 xmrig (1978:<NA>) < connect res=-115(EINPROGRESS) tuple=<IP_ADDRESS>:55946-><IP_ADDRESS>:14433
1382 xmrig (1978:<NA>) > getsockopt
1383 xmrig (1978:<NA>) < getsockopt res=0 fd=14(<4t><IP_ADDRESS>:55946-><IP_ADDRESS>:14433) level=1(SOL_SOCKET) optname=4(SO_ERROR) val=0 optlen=4
1404 xmrig (1978:<NA>) > write fd=14(<4t><IP_ADDRESS>:55946-><IP_ADDRESS>:14433) size=293
1405 xmrig (1978:<NA>) < write res=293 data=...
1412 xmrig (1978:<NA>) > mmap addr=0 length=266240 prot=3(PROT_READ|PROT_WRITE) flags=10(MAP_PRIVATE|MAP_ANONYMOUS) fd=-1(EPERM) offset=0
1413 xmrig (1978:<NA>) < mmap res=7F8860FBC000 vm_size=41524 vm_rss=4684 vm_swap=0
1414 xmrig (1978:<NA>) > read fd=14(<4t><IP_ADDRESS>:55946-><IP_ADDRESS>:14433) size=65536
1415 xmrig (1978:<NA>) < read res=1738 data=...

As getsockopt, read, and write are not supported by Falco, the only other option seems to be the connect syscall. With evt.type = connect and evt.dir = < the rule can also detect xmrig thorugh the same fd.sip.name field.
{
    "output": "11:51:51.410171274: Critical Outbound connection to IP/Port flagged by https://cryptoioc.ch (command=xmrig --donate-level 5 -o xmr-eu1.nanopool.org:14433 -u WALLET_ADDRESS --tls --coin monero port=14433 ip=<IP_ADDRESS> container=xmrig-test (id=8add3bb1eab0) image=ubuntu)",
    "priority": "Critical",
    "rule": "Detect outbound connections to common miner pool ports",
    "source": "syscall",
    "tags": [ "mitre_execution", "network" ],
    "time": "2022-05-23T09:51:51.410171274Z",
    "output_fields": {
        "container.id": "8add3bb1eab0",
        "container.image.repository": "ubuntu",
        "container.name": "xmrig-test",
        "evt.time":<PHONE_NUMBER>410171274,
        "fd.rip": "<IP_ADDRESS>",
        "fd.rport": 14433,
        "proc.cmdline": "xmrig --donate-level 5 -o xmr-eu1.nanopool.org:14433 -u WALLET_ADDRESS --tls --coin monero"
    }
}

Which issue(s) this PR fixes:

The default Detect outbound connections to common miner pool ports rules doesn't detect xmrig connecting to nanopool.

Special notes for your reviewer:
If needed I can provide further details.
Does this PR introduce a user-facing change?:
rule(macro net_miner_pool): additional syscall for detection

Welcome @beryxz! It looks like this is your first PR to falcosecurity/falco 🎉

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: beryxz
To complete the pull request process, please assign mstemm after the PR has been reviewed.
You can assign the PR to them by writing /assign @mstemm in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

rules/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/milestone 0.32.0

Closing and reopening to disenchant tide :roll_eyes:
/close

@leogr: Reopened this PR.

In response to this:

/reopen

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

LGTM label has been added.  Git tree hash: 62fd54b18a94314b892a1fa19a8a2fed354a4382

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: beryxz, jasondellaluce
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

rules/OWNERS [jasondellaluce]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/close


Add support to decode slices from the environment.

This change allow to use slices of all supported primitive types in configuration structs. Delimiter for splitting environment variable is comma. This change is backward compatible with previous releases.

@joeshaw Hi, will you have a minute to take a look? Thanks.

Looks good. I'd like your opinion on trimming spaces from the argument list.  Although it means you might not get exactly what you wanted in strings, I think the benefits for other datatypes outweighs the few cases in which you'd want leading or trailing spaces.

Really good point, I didn't think about this simple scenario. Fixed and squashed.

Thanks a lot!


Freezing the header panels on scrolling

Hi,
I am using ui.grid in my project.
I have three panel left-panel, top-panel, center-panel. Each panel contains separate grids. I want to perform the following :

When I scroll center-panel vertically then left-panel and center-panel should be scrolled and top-panel should be stable at the same position.
When I perform scroll center-panel horizontally then top-panel and center panel should be scrolled and left-panel should be stable at the same position.
I have gone through the zebkits documents but I am not able to find the a way to do the same.
How can achieve this behaviour ?

Hi,
Is there any way to achieve the behavior mentioned issue? Please comment.

As far as I understand you should synchronize scrolling between different grid components. One of a possible solution is override "catchScrolled(psx, psy)" method in center grid component and initiate scrolling from the overridden method for other two grids. Something like the following:
var centralGrid = new zebkit.ui.grid.Grid(model, [
    function catchScrolled(psx, psy) {
        this.$super(psx, psy);
        //  compute scrolled horizontal and vertical offset
        var dx = this.scrollManager.getSX() - psx,
              dy = this.scrollManager.getSY() - psy;
        if (dx !== 0) {
              leftGrid.scrollManager.scrollXTo(leftGrid.scrollManager.getSX() + dx);
        }
        if (dy !== 0) {
              topGrid.scrollManager.scrollYTo(topGrid.scrollManager.getSY() + dy);
        }
    }
]);

Thanks you! It works.
I did it with panels. I have added panel to scrollpan and made little change
leftGrid.scrollObj.scrollManager.scrollXTo(leftGrid.scrollObj.scrollManager.getSX() + dx);


Cookie expires field is missing in Android

@brentvatne Is there any way to get expires field in the cookie object in Android? currently, I am not receiving that field in Android but it works fine in iOS. I really appreciate any help you can provide.

@Drashti-Gabani Unfortunately Android does not provide the raw cookie data so the expiry is not available
https://github.com/react-native-cookies/cookies/issues/70#issuecomment-738948521


Adding Null Route Breaks FRR Diagnotics IPv4 Routes

This has been broken for a while but it's taken me a little time to narrow it down.
To generate the Routing -> Diagnostics -> General -> IPv4 Routes screen, there is a ruby script at /usr/local/opnsense/scripts/quagga/quagga.rb which runs vtysh and issues the command show ip route. It then tries to parse it using a regex. If you have a null route configured in the OS the format that vtysh gives that route in doesn't match the parser and ruby bombs out with:
Exception 'NoMethodError' at /usr/local/opnsense/scripts/quagga/quagga.rb:121 - undefined method 'split' for nil:NilClass
Normal routes in the text output look like this (anonymised):
O>* <IP_ADDRESS>/24 [110/730] via <IP_ADDRESS>, vmx1, 00:19:02
O>* <IP_ADDRESS>/24 [110/720] via <IP_ADDRESS>, vmx1, 00:19:02
O>* <IP_ADDRESS>/24 [110/540] via <IP_ADDRESS>, vmx1, 00:19:02
vtysh displays a null route like so:
K>* <IP_ADDRESS>/24 [0/0] unreachable (blackhole), 00:19:14
If the quagga.rb script encounters such a line in vtysh's output it outputs:
{"general_routes":"error"}
And you get no routing table displayed in the web interface. A quick fix is to comment out line 121 which stops the web page from displaying the code (although it's still there in the JSON output).
Tested on OPNsense 19.7.6-amd64

The following regex parses the vtysh output correctly, but because it adds another match group the parameters need tweaking on line 121
entry_regex = /(\S+?)\s+?(\S+?)(?: \[(\d+)\/(\d+)\])? (?:via (\S+?),|is ([^,]+?),|(unreachable)) ([^,\n]+)(?:, (\S+))?/
Change line 121 to
code, network, ad, metric, via, direct, unreach, interface, time = line.scan(entry_regex).first
There is probably a nicer, neater way of doing this that will look better in the UI.

Realised this probably belongs in opnsense/plugins not opnsense/core - is it possible to move it, or do i need to close/reopen?


Stopped working with phpunit 6.3

The same exact config with 5.7.21 works fine but with 6.3 returns:
PHPUnit 6.3.0 by Sebastian Bergmann and contributors.

Could not use "LimeDeck\Testing\Printer" as printer.

For what I see they removed the long class names and everything is now namespaced but I don't know if this has anything to do with this problem. Phpunit is not giving any details...

I've made a pull request (#7) that makes the class compatible with PHPUnit 6. Since everything is namespaced it will definitely break compatibility with earlier versions so if this pull request is accepted, it might be wise to tag it accordingly.
Let me know if there is anything else I can do.

This has been resolved in 3.0.0.
PHPUnit < 6.0 will still work with ^2.0.0
Thanks

Looks like it's not working with PHPUnit 7


can't find the card

Can someone help me with examples of the card?
Is this a manual card ? because i am not seeing it in card even though integration is there.

@nicoolaro this is an example configuration with the Shelly 3EM
type: custom:energy-overview-card
a_current: sensor.shellyem3_yourid_channel_a_current
b_current: sensor.shellyem3_yourid_channel_b_current
c_current: sensor.shellyem3_yourid_channel_c_current
a_voltage: sensor.shellyem3_yourid_channel_a_voltage
b_voltage: sensor.shellyem3_yourid_channel_b_voltage
c_voltage: sensor.shellyem3_yourid_channel_c_voltage
a_power: sensor.shellyem3_yourid_channel_a_power
b_power: sensor.shellyem3_yourid_channel_b_power
c_power: sensor.shellyem3_yourid_channel_c_power
a_power_factor: sensor.shellyem3_yourid_channel_a_power_factor
b_power_factor: sensor.shellyem3_yourid_channel_b_power_factor
c_power_factor: sensor.shellyem3_yourid_channel_c_power_factor

perfect thank you


Fix numerics for activation recompute

This PR is a numerical fix to ensure that the results are bitwise identical when running TE (w/o FP8) with and without activation recompute.

Add missing warmup for kernel with no bias.
Fix incorrect input shape during jit warmup.

/te-ci


Question: why allow_other is need for auto_mount

Hi, I've been playing around with fuser, specially the auto_unmount option which I'm interested in. I'm not sure though why allow_other is automatically added to the options when auto unumount is set:
https://github.com/cberner/fuser/blob/39d4177e809c7ee3b6757136fee8a28d5f41f040/src/session.rs#L81
If it goes through fusermount, why allowing access to "others" is required? I tested it without that option, it works as far I can tell, but I'm probably missing something there...
Thanks!
Env: 18.04.1-Ubuntu, Linux 5.4.0, fuse 2.9.7

duplicate of https://github.com/cberner/fuser/issues/230, sorry for the noise...


Add SERVICE_TRACKING_HEARTBEAT ENV

The env from this PR needs to be added to OPS repositories
https://github.com/ITISFoundation/osparc-simcore/pull/4570

done, has a default value


VS2015 returns JsErrorScriptException when compilerOptions.watch is true

In Visual Studio 2015 create TypeScript HTML app.
Add .ts file to the project
Add tsconfig.json to the project and set "watch": true
{
  "compileOnSave" : true,
  "compilerOptions": {
    "watch": true,
    "noImplicitAny": false,
    "noEmitOnError": true,
    "removeComments": false,
    "sourceMap": true,
    "target": "es5"
  },
  "exclude": [
    "node_modules",
    "wwwroot"
  ]
}

Build the project. Build should fail with an error:

VSTSC : Script failed with error : 'JsErrorScriptException (0x30001)'.

I'm using:
VS2015 Enterprise Version 14.0.24720.00 Update 1
Typescript <IP_ADDRESS>
WebEssentials 2015.1, build 1.0.209
Resharper 2015.2, build 103.0.20150818.200216 (I've tried to suspend it in order to eliminate influence)

im getting the same error all things updated though

I also still have this error with
VS2015 Enterprise Version 14.0.24720.00 Update 1
Typescript <IP_ADDRESS>

The issue has been fixed in master. it is available in the nightly nuget package, and will be available in the nextrelease. TS 1.8.* does not have that.
Either way, the fix is to show a valid error message instead of crashing. but --watch is still an invalid argument to pass to tsc.exe.

@mhegazy Thank you very much for quick answer. I understand the first part of your answer, that this bug is only fixed in the upcomming 2.0 version.  But how can I compile the *.ts sources to *.js sources (in Visual Studio 2015) by saving the *.ts?

set "compileOnSave": true but note this is not in the "compilerOptions". i.e
{
  "compileOnSave": true,
  "compilerOptions": { ...}
  "exclude": []
}

see https://github.com/Microsoft/TypeScript/wiki/tsconfig.json#compileonsave for more details.

@mhegazy
Thank you! It works <3

Using compileOnSave worked for me when using Visual Studio w/ Update 2. But i've recently installed Update 3 and it will no longer compile on save. My setup is as follows...
VS TypeScript Extension: <IP_ADDRESS>
Environment Variable Path: C:\Program Files (x86)\Microsoft SDKs\TypeScript\1.8
tsc -v: 1.8.34
I've also made the manual fix for compiling RxJs extensions, https://github.com/Microsoft/TypeScript/issues/8518#issuecomment-223156931.

I have the same error on my VS2015 update 3, RC1.
Adding "compileOnSave" didn't work for me.
I also have the "Web-Essentials 2015.3" extension.
In my folder path: "C:\Program Files (x86)\Microsoft SDKs\TypeScript"
I have 2 folders: "1.0", "1.8".
However, when checking for version from cmd in the project foder I got:

tsc -v
Version <IP_ADDRESS>

THIS post solved my problem:
Updating the PATH variable of Win10 to be "1.8" instead of "1.0",
and using "compileOnSave" instead of "watch".
Now it's works.

I have VS 2015 Update 3 on Win10, fixed the 1.0/1.8 issue in the PATH variable. I've tried compileOnSave, the settings in options for compile on save, and watch gives me a build error as well as not compiling on save either.
Works fine when I build

Turns out when I added a Gulp task to delete all .js and .js.map files in the Typescript outDir folder it causes Typescript to generate the refreshed .js and .js.map files automatically. These files have been added to the .tfignore file so they are not in TFS.  Not sure why this is happening.

If you are getting a JsErrorScriptException (0x30001) in VS then almost certainly you missed mhegazy's point that "compileOnSave": true, does NOT go inside the "compilerOptions": {}, but on the same level as it.
This is mine:
{
  "compileOnSave": true,
  "compilerOptions": {
    "emitDecoratorMetadata": true,
    "experimentalDecorators": true,
    "inlineSourceMap": true,
    "inlineSources": true,
    "module": "system",
    "moduleResolution": "node",
    "noEmitOnError": true,
    "noImplicitAny": false,
    "outDir": "./wwwroot/js",
    "removeComments": false,
    "target": "es5"
  },
  "exclude": [
    "node_modules",
    "wwwroot"
  ]
}


Create a walkthrough for Auth on Ionic/Cordova

This isn't necessarily the purview of AngularFire2, but it does affect enough of our user base that we could relieve a lot of pain by providing a best practice and walkthrough. See #243 for discussion and pain points, and summary of solutions.

Hi @katowulf  Added a quick tutorial as #691 . If it looks good and this is what is expected, happy to convert it into a doc.
Thoughts please ?

Yeah, let's do it. Create a .md file in docs/ and let's add a link this to the README where we list the guide docs.

Hi @katowulf created a PR #693

@katowulf Updated the document to reflect native support using Ionic Native. Although there is a room for improvement for this document (which I'll try to work later), it should provide a good start as of now. And I've closed the issue.


<urlopen error [Errno -2] Name or service not known> Error

I am testing the Ethernet speed (using a ENC28J60 module) on a Raspberry pi zero. When I run speedtest-cli, it gives me this error:
Could not retrieve speedtest.net configuration: <urlopen error [Errno -2] Name or service not known>
Any idea of what might cause this issue? Thanks

You cannot get to the speedtest.net server. Can you go anywhere else? Can you do a wget https://speedtest.net? It could be a simple matter of resolving DNS or your interface is not reaching the internet.

Yeah looks like it's a network issue on my end. Thanks


Provide proper upgrade guidance in admin depending on installation method

This is a todo based on @gsalkin's comments in #1673 — basically we should properly message upgrade procedure in WP Admin based on how Timber was installed (Composer vs. WP.org)

I would modify requirements for timber-library 2.0 to also require Twig 2.x. Currently timber-library still calls for the deprecated version of Twig. Now that most installations support PHP 7.x, I would move users to have that installed.
Creates better parity between the Github version and WordPress plugin.

A quick note: Bumping the minimum Twig version to 2.x would probably break compatibility with WPML. WPML still uses Twig 1.32.0 (for compatibility with PHP5) and also loads Twig through Composer. Currently, the only solution I found to make WPML work with Timber is to install Timber as a plugin (related: #1064). But then, WPML’s Twig version will be used instead of Timber’s, which is not future-proof either.
I’m all for parity. I’m not saying we shouldn’t upgrade the minimum requirements (I’m not a big fan of WPML myself 🙊), but if we do it, we probably need to mark Timber as incompatible with WPML.

Gahhhhhh!!!!!! Twig!!!!!

I guess this can be closed. We won’t release 2.x as a plugin, but only as a Composer package. See #2005.


Please do an episode on Reactive Programming

Channel
This is for "C++Weekly"
Topics

The programming paradigm itself;

Push vs Pull when processing asynchronous data
Observers vs Enumerators
Callbacks vs coroutines
ReactiveX's Observer/Observable or project Reactor's Mono/Flux. (which is admittedly Java, but still a good reference)

A look at alternative libraries/frameworks

Length
I think this warrants more than just 5 minutes, if only because I don't see much content about this subject in combination with C++.

It is very hard topic starting from the definition itself. And there are at least 2 flavors:

Rx - that works exclusively with streams and ads "completed" and "failed" as a separate value channels
FRP (Functional Reactive Programming) that originated from functional languages and consist of 2 primitives (which naming is inconsistent between libraries) - behavior and event stream.

For the first there are at least official implementation and some alternatives under development.

https://github.com/CODIANZ/another-rxcpp
https://github.com/victimsnino/ReactivePlusPlus

For the second I can't find any mature solution used for lots of people. All projects are either under development or abandoned.

(sodium-cxx)[https://github.com/SodiumFRP/sodium-cxx] - abandoned port of Java reactive library sodium. From writers of "Functional Reactive Programming" book.
(cpp.react)[https://github.com/snakster/cpp.react/tree/legacy1] - abandoned port of Scala.React for which authors wrote paper "Deprecating the Observer Pattern with Scala.React". It has great ideas and documentation if you look at legacy1 branch only that is stable state before unfinished rewrite attempt in master.
(µReact)[https://github.com/YarikTH/ureact] my port and rethinking of cpp.react with modern syntax and modern standards. Greatly suffers from lack of documentation (only doxygen comments in header and tests) and real usage experience.
(urp)[https://github.com/joaquintides/usingstdcpp2019] - experimental 900 lines implementation made for cppcon presentation. Not ready for production and have never intended for.

It's kinda funny that author of FRP is not happy both with the name itself and with "so called FRP libraries".
https://github.com/conal/talk-2015-essence-and-origins-of-frp
While his attempts to describe what he meant by FRP are so overflown with complex math terminology that mere mortal programmers can't comprehend it including myself.


Can @types/electron be removed as a devDependency?

I'm pulling ngx-electron into my project as a dependency, which has a dependency on @types/electron, which appears to have a dependency on electron. This causes my Angular app's node_modules to have a complete electron distribution in it, which it never uses. However, when I then use electron-builder, it dutifully pulls it in. According to this (https://www.npmjs.com/package/@types/electron), there is no reason to include @types/electron anymore. Should it be removed entirely?

For grins, I did try what I'm suggesting. I forked the repo and removed @types/electron from package.json entirely. Now it won't build, because the namespace Electron doesn't exist. I imagine there must be a way to do this without bringing in the entire electron package as an angular dependency. I just can't think of what that is.

@walkingriver electron provides it's own types since two weeks I think. They've deleted @types/electron as mentioned on npmjs.
I'm currently refactoring ngx-elecrton to work again correctly.

Let me know if there's anything I can do to assist.

thanks for the offer. I've just published 1.0.3 on npmjs perhaps can you try it in your scenario / build. I've tested and verified functionality with @angular/cli based samples

@walkingriver thanks for the input I'm closing this due to the currently working 'workaround' 😄 and the upcoming release mentioned in #11


Migration: Socket improvements

Add read lock to WebsocketIO.Read() function as the underlying gorilla websocket package only allows 1 reader at a time.
Use gorilla websocket's WriteMessage() function in WebsocketIO.Write() function as this benefits from a fastpath mode.
Set timeouts in migration push mode to mirror those setup in pull mode.
Fixes hang in zfs send (and potentially other storage drivers) due to Go waiting for socket to become writable sometimes when migration fails on target (even though websocket is closed).

jenkins: test this please

Good to go.


Build fails if cmake is run outside of the source directory

When invoked from a new subdirectory within the source tree (e.g. build/), cmake writes the autogenerated files (tcmuhandler-generated.c, tcmuhandler-generated.h and version.h) into the current directory. The subsequent build then fails, because the autogenerated files aren't located during compilation:

tcmu-runner (master)> mkdir -p build
tcmu-runner (master)> cd !$
cd build
tcmu-runner/build (master)> ls
tcmu-runner/build (master)> cmake -Dwith-glfs=false ../
-- The C compiler identification is GNU 4.8.5
-- 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
-- Found PkgConfig: /usr/bin/pkg-config (found version "0.28")
-- checking for module 'gio-unix-2.0'
--   found gio-unix-2.0, version 2.44.1
-- checking for module 'libkmod'
--   found libkmod, version 17
-- Found ZLIB: /usr/lib64/libz.so (found version "1.2.8")
-- Configuring done
-- Generating done
-- Build files have been written to: build
tcmu-runner/build (master)> make
[  3%] Generating tcmuhandler-generated.c, tcmuhandler-generated.h
Scanning dependencies of target tcmu
[  7%] Building C object CMakeFiles/tcmu.dir/api.c.o
[ 11%] Building C object CMakeFiles/tcmu.dir/libtcmu.c.o
[ 15%] Building C object CMakeFiles/tcmu.dir/libtcmu-register.c.o
libtcmu-register.c:24:35: fatal error: tcmuhandler-generated.h: No such file or directory
 #include "tcmuhandler-generated.h"
                                   ^
compilation terminated.
CMakeFiles/tcmu.dir/build.make:117: recipe for target 'CMakeFiles/tcmu.dir/libtcmu-register.c.o' failed
make[2]: *** [CMakeFiles/tcmu.dir/libtcmu-register.c.o] Error 1
CMakeFiles/Makefile2:215: recipe for target 'CMakeFiles/tcmu.dir/all' failed
make[1]: *** [CMakeFiles/tcmu.dir/all] Error 2
Makefile:127: recipe for target 'all' failed
make: *** [all] Error 2

The obvious workaround for this is to:

# mv tcmuhandler-generated.c  tcmuhandler-generated.h  version.h $tcmu-runner_src_dir

...before running make.

Closing - thanks for the merge Andy!


Problem installing in MacOS

I am trying to run some simple G4 examples to compare with https://github.com/JuliaHEP/Geant4.jl but I am having problems installing the extension on MacOS.

I tried with a X86 MacBook using the pre-built wheel but I get the following error after a successful pip3 install geant4-pybind :

>>> from geant4_pybind import *
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
ImportError: dlopen(/Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/site-packages/geant4_pybind.cpython-39-darwin.so, 2): Symbol not found: __ZTTNSt3__114basic_ofstreamIcNS_11char_traitsIcEEEE
  Referenced from: /Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/site-packages/geant4_pybind.cpython-39-darwin.so
  Expected in: /usr/lib/libc++.1.dylib
 in /Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/site-packages/geant4_pybind.cpython-39-darwin.so

I have also tried on a M1 MacMini, for which the wheel needs to be build, and I get the following error after having build correctly the dynamic library extension.

...
[490/490] Linking CXX shared module /private/var/folders/zz/gs4fh_9d5lx4k_q51gd7kvyr0000gn/T/pip-install-3_079jq7/geant4-pybind_d1557da9e313418685b29199fe58baf5/build/lib.macosx-13.4-arm64-cpython-39/geant4_pybind.cpython-39-darwin.so
Traceback (most recent call last):
  File "/private/var/folders/zz/gs4fh_9d5lx4k_q51gd7kvyr0000gn/T/pip-install-3_079jq7/geant4-pybind_d1557da9e313418685b29199fe58baf5/doc/generate_stubs.py", line 14, in <module>
    FunctionSignature.ignore_invalid_signature = True
NameError: name 'FunctionSignature' is not defined
Traceback (most recent call last):
  File "/Users/Shared/cvmfs/sft.cern.ch/lcg/views/LCG_104/arm64-mac13-clang140-opt/lib/python3.9/site-packages/pip/_vendor/pep517/in_process/_in_process.py", line 363, in <module>
    main()
  File "/Users/Shared/cvmfs/sft.cern.ch/lcg/views/LCG_104/arm64-mac13-clang140-opt/lib/python3.9/site-packages/pip/_vendor/pep517/in_process/_in_process.py", line 345, in main
...

Do you know how to fix these installation issues? Thanks in advance.

There was an update in one of the tools used. I pushed a hotfix 3dac008 to use a previous version of that tool, which should allow you to build it.
The import error seems to be identical with #13. I can't say much about this as I'm not an expert on Apple products and I also don't have access to any Apple machines apart from the github CI runner (which seems to work fine).

Thanks. But I am not sure how to use the modified pyproject.toml with pip3 install geant4-pybind

It is fixed. I just had to clone with recursive submodules. Thanks again.
In case you are interested I run some very simple benchmarks.

With B2a example with protons @ 3 GeV
On my Mac-mini with the M1 processor

| B2a (C++) | B2a.jl | B2a.py
-- | -- | -- | --
events = 1 | 0.9 s | 6 s | 1 s
events =100k | 106 s | 79 s | 176 s
events =100k (MT) | 23 s | 22 s | --
The reason why the python is slower is understandable because the user actions and sensitive detectors are implemented in Python and , it is many factors slower than C++ or Julia. If more work is done in these user actions, as it will be the case on more realistic applications, the performance will deteriorate  even further. I do to have an explanation why Julia should be faster than C++ in the serial case.  I was expecting within a few percent the same.

The OSX import error should be fixed too with b89bb3afd11672f903f0e59e3db9557bdd023116 by building for an older OSX version.


"Open in Editor" button doesn't work with visual scripts

Godot Master, Linux
In the inspector, the "Open Editor" under the "scripts" sections works fine for a gdscript, but doesn't do anything if the script is a Visual Script. The icon in the scene tree dock works as it should.
Steps to reproduce:

create a visual script for a node in the tree
go back to the scene, and check the inspector for that node
scroll to the script section and open it if necessary
click the "open in editor" button, nothing happens

Reopening as #20112 was reverted.


Support find-references

The find-references feature doesn't seem to work right out of the box. What's needed to make this feature work?

Any news on the pieces that we would need for this @ndmitchell? When you say the set of roots would just the list of files that are loaded in ghcide be enough?

@meditans We are making progress towards this issue certainly. One issue to overcome is that you really don't want to keep a lot of every information about every module in memory as it causes too much memory usage so creating an out of memory cache is important.

Thanks @mpickering, I just read the weekly update and went back to check this issue. You're doing an amazing job in helping the ide infrastructure moving forward!

Maybe related with "goto reference": https://github.com/haskell/haskell-language-server/issues/211
It seems the commits referencing this issue are not related.

This should be fixed by #898

@meditans as this repo is about to be archived we are gonna include this one in https://github.com/haskell/ghcide/pull/898


smeshing

I cannot click on get smeshing. Nothing is happening. I attached a log to see if this node is fully functional.
Can you let me know
screen shot and log.zip
also getting java error when running all benchmakrs see attached

### Environment

Smapp version: 0.xx.x | develop
Node version: v0.x.xx
OS: Windows 10 | macOS High Sierra (10.13) | Ubuntu 20.04

Describe the bug
A clear and concise description of what the bug is.
Steps to reproduce
Try to narrow down your scenario to a minimal working/failing example. That is, if you have a big program causing a problem, start with deleting parts not relevant to the issue and observing the result: is the problem still there? Repeat until you get the most straightforward sequence of steps to reproduce the problem without any noise surrounding it.

Open this
Press that
Click here and there

Expected behavior
What should happen?
Actual behavior
What’s happening now?
Attach screenshots or GIF’s of what’s going on if it makes the issue more illustrative.
Logs
Attach app-logs.txt and spacemesh-log.txt

If you can find out the part of the logs that relates to the problem, attach them or paste in code-block.
How to fix
If you have an idea of how the bug can be fixed, provide a short description.

@Hattrix3 why did you close it? I have the same issue


Update schedule schema to include API sources

Update schedule schema to include API key permission (yes/no) and where users can go to find the API key auth steps. This can match the realtime schema authentication information.

Closing since this issue has been merged into #130 .


[FEATURE REQUEST] Skip "Link This Account" popup when Import to account

Describe the solution you'd like
To skip the "Import Transactions" popup when the account is not linked and show directly the "Choose File" window to select a QIF/CSV file.
Additional context
My banks can't be linked to YNAB so this popup is useless and annoying for me.

We use Trello to track Feature Requests so that users can vote on what they'd like to see next. This is crucial for us since there are so many requests and so little time amongst those of us who work on the Toolkit.
Your feature request has been added here and will need to be manually sorted.
This issue can now be closed.

@joshmadewell Can you close the Feature Request on the Trello, it's not needed anymore after the last YNAB update.


update license attribute

specifying the type and URL is deprecated:
https://docs.npmjs.com/files/package.json#license
http://npm1k.org/

Any reason not to accept this fix? Would be nice to resolve license warnings about this package. /cc @broofa

:bow: ty


Node at Path block does not update when reaming a node

What Happened
If you drag a Node to block code, then rename it, it breaks your code.
How to Reproduce

Add a Node to your Block Code, e.g. in the Remove Node block
Change the referenced node name in the sidebar
Go back to the Block Code

Block Coding Plugin Version
No response
Godot Engine Version
No response
Operating System
No response
Logs or command-line output
No response
Anything else?
No response

Sadly, this is a known issue. Godot doesn't provide a good way to track changes to part of the scene tree, so nothing is handled. For instance, if you add a node, it won't be added to the drop down list. Rename would be tricky as you'd have to catch the event and then find an the block code nodes and update potentially affected nodes.


Fixes for MXNet 1.8

Issue #, if available: #1374
Tests against mxnet 1.8 cannot be enabled because the pypi package for windows is not available. However this PR passes tests on macos and linux when using mxnet 1.8.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

There were issues when doing inference on SageMaker with mxnet 1.7. Does this problem still persist with mxnet 1.8?

I'm not sure, we would have to try: this fix only concerns failing tests. What kind of issues are there on SM with 1.7?

There wasn't a particular error message. Inference jobs where not running, basically (i.e. running forever).

I see, that doesn't sound right. I think the best thing to do is to try out an mxnet 1.8-based container to see whether it was mxnet 1.7 causing the issue. From the tests I ran on mxnet 1.8 for the present PR, everything was green except the one I went fixing


I encounter a Late Field initialization error when using Socket-IO

This is the error I am encountering on calling find query on feathers Js server from flutter app both running on local machine
I/flutter ( 7821): │ ⛔ Unexpected error ::: LateInitializationError: Field 'scketio' has not been initialized.
The error comes from this code below;
This code from my provider;
  Future<APIResponse<List<ActivityI>>> getActivities() async {
    List<ActivityI>? messages;
    String? error;
    try {
      Map<String, dynamic> response = await socketIOClient.scketio.find(
        serviceName: "activities",
        query: {},
      );
      logger.i(response.toString());
      messages = List<Map<String, dynamic>>.from(response["data"])
          .map((map) => ActivityI.fromMap(map))
          .toList();
    } on FeatherJsError catch (e) {
      logger.e("FeatherJsError error ::: Type => ${e.type} ::: Message => ${e.message}");
      error = "Unexpected FeatherJsError occurred, please retry!";
    } catch (e) {
      logger.e("Unexpected error ::: ${e.toString()}");
      error = "Unexpected error occurred, please retry!";
    }
    return APIResponse(errorMessage: error, data: messages);
  }

// Below is how I have initialized my socket-io client for the flutter app;

FlutterFeathersjs socketIOClient = FlutterFeathersjs();

void main() {
  WidgetsFlutterBinding.ensureInitialized();

  socket.Socket io = socket.io(API.baseUrl, <String, dynamic>{
    'transports': ['websocket'],
  });
  socketIOClient.configure(FlutterFeathersjs.socketioClient(io));

  runApp(MultiProvider(providers: [
    ChangeNotifierProvider(create: (_) => UserProvider()),
    ChangeNotifierProvider(create: (_) => ThemeProvider()),
    ChangeNotifierProvider(create: (_) => ActivityProvider()),

  ], child: const MyApp()));
}

//Currently using flutter_feathersjs: 4.1.5
//                       socket_io_client: ^2.0.1

//  This is my code calling on getActivities from ActivityProvider

 fetchActivities() async {
    context.read<ActivityProvider>().getActivities().then(
      (response) {
        isLoading = false;
        if (response.errorMessage == null) {
          setState(() {
            activities = response.data!;
          });
        } else {
          setState(() {
            error = response.errorMessage;
          });
        }
      },
    );
  }

  @override
  void initState() {
    isLoading = true;
    fetchActivities();
    super.initState();
  }

Currently running debug app on memu emulator

You may probably mis initiate the flutter feathers js client.
I dont realy understand  your code:

Where does come this code ?

 @override
  void initState() {
    isLoading = true;
    fetchActivities();
    super.initState();
  }

Is it in your main() function . If it's, why does you provide a such method where according to me, state are relative to widget ?

More informations can help because from now, I cannot reproduce the given issue

Where is  fetchActivities method defined ?

The code above call this this code below which is from activities_provider.dart
class ActivityProvider with ChangeNotifier{
//this is the method on init state in main.dart that is being called by fetchActivities
Future<APIResponse<List>> getActivities() async {
List? messages;
String? error;
try {
Map<String, dynamic> response = await socketIOClient.scketio.find(
serviceName: "activities",
query: {},
);
logger.i(response.toString());
messages = List<Map<String, dynamic>>.from(response["data"])
.map((map) => ActivityI.fromMap(map))
.toList();
} on FeatherJsError catch (e) {
logger.e("FeatherJsError error ::: Type => ${e.type} ::: Message => ${e.message}");
error = "Unexpected FeatherJsError occurred, please retry!";
} catch (e) {
logger.e("Unexpected error ::: ${e.toString()}");
error = "Unexpected error occurred, please retry!";
}
return APIResponse(errorMessage: error, data: messages);
}
}
I am accessing getActivities() method using provider
fetchActivities() async {
context.read().getActivities().then(
(response) {
isLoading = false;
if (response.errorMessage == null) {
setState(() {
activities = response.data!;
});
} else {
setState(() {
error = response.errorMessage;
});
}
},
);
}

Can you please, format  correctly your code (snippets) in your previous comments. It's hard to look in up to your code.
Just use
```dart
// Put your code here.
```

dart
//activity provider
class ActivityProvider with ChangeNotifier{
//this is the method on init state in main.dart that is being called by fetchActivities

Future<APIResponse<List<ActivityI>>> getActivities() async {
    List<ActivityI>? messages;
    String? error;
    try {
      Map<String, dynamic> response = await socketIOClient.scketio.find(
        serviceName: "activities",
        query: {},
      );
      logger.i(response.toString());
      messages = List<Map<String, dynamic>>.from(response["data"])
          .map((map) => ActivityI.fromMap(map))
          .toList();
    } on FeatherJsError catch (e) {
      logger.e("FeatherJsError error ::: Type => ${e.type} ::: Message => ${e.message}");
      error = "Unexpected FeatherJsError occurred, please retry!";
    } catch (e) {
      logger.e("Unexpected error ::: ${e.toString()}");
      error = "Unexpected error occurred, please retry!";
    }
    return APIResponse(errorMessage: error, data: messages);
  }
}
//socket-io initialization  method in main.dart
FlutterFeathersjs socketIOClient = FlutterFeathersjs();

void main() {
WidgetsFlutterBinding.ensureInitialized();

socket.Socket io = socket.io(API.baseUrl, <String, dynamic>{
'transports': ['websocket'],
"forceNew": true
});
socketIOClient.configure(FlutterFeathersjs.socketioClient(io));

//then in activities.dart page am calling fetchActitivites() on page init
fetchActivities() async {
    context.read<ActivityProvider>().getActivities().then(
      (response) {
        isLoading = false;
        if (response.errorMessage == null) {
          setState(() {
            activities = response.data!;
          });
        } else {
          setState(() {
            error = response.errorMessage;
          });
        }
      },
    );
  }


Bug or ficha

That's called "feature", and no, this is not the one :)
That "+" is an indication of a bonus draw, you can't have it "stacked" in any event possible.
P.S.
May I introduce you to the SmoothFont?

Actually you can have multiple bonus draws... I just wasn't expecting that many. The fact that you go that many is in fact a bug since most of the time you will use it up in your next draw.


Clicking logo should restore application to a startup state

i.e. clear the search, remove the selection and go back to the favourites tab.

I am not sure anymore if this is a good idea. Since we are doing the tearout the design of clicking to get the initial state seems bad. Thoughts? Close?

Hmm. Yeah until someone wants it we don't need it. Close.


Evaluation of Datastore Caching changes

Summary
Reconsideration of the data cache mechanism added to the datastore layer.

Report on the differences between current lbm-sdk and cosmos-sdk.

Aim of the change to lbm-sdk style implementation.
The actual effect of the lbm-sdk style implementation.
Consideration of the impact of this change on future cosmos-sdk merge task (e.g., how many changes are needed).

Decision whether to keep the lbm-sdk implementation or revert to the cosmos-sdk implementation.
If revert to a cosmos-sdk implementation, plan and progress of completing that work.

This task is to sort out useful features in order to reduce the differences between lbm-sdk and cosmos-sdk.
Parent: #549

For Admin Use

[x] Not duplicate issue
[x] Appropriate labels applied
[x] Appropriate contributors tagged
[x] Contributor assigned/self-assigned

Datastore Caching
This is part of the work to reduce the difference between lbm-sdk and cosmos-sdk.
Unlike cosmos-sdk, lbm-sdk performs datastore caching through cache manager in iavl.
It is necessary to determine the validity of the function and, if it is determined that it is not necessary, roll back it for ease of management in the future.
Process
I proceeded to remove the cache manager of iavl. Performance improvements were made in the repository fork cosmos/iavl and tendermint/tm-db. However, there was a problem that the race test failed due to the changed part.
It was necessary to find out how much performance improvement was made in line/iavl and line/tm-db compared to cosmos/iavl and tendermint/tm-db.
Method
A benchmark was performed by executing BankKeeper.SendCoins() for both random and fixed addresses.
package pfm

import (
	"encoding/json"
	"math/rand"
	"strconv"
	"testing"

	abci "github.com/line/ostracon/abci/types"
	"github.com/line/ostracon/libs/log"
	ocproto "github.com/line/ostracon/proto/ostracon/types"
	dbm "github.com/tendermint/tm-db"
	"github.com/stretchr/testify/require"

	"github.com/line/lbm-sdk/simapp"
	sdk "github.com/line/lbm-sdk/types"
)

func BenchmarkBaseAccount(b *testing.B) {
	encCfg := simapp.MakeTestEncodingConfig()
	app := simapp.NewSimApp(log.NewNopLogger(), dbm.NewMemDB(), nil, true, map[int64]bool{}, simapp.DefaultNodeHome, 0, encCfg, simapp.EmptyAppOptions{}, nil)
	genState := simapp.NewDefaultGenesisState(encCfg.Marshaler)
	stateBytes, err := json.MarshalIndent(genState, "", " ")
	require.NoError(b, err)
	app.InitChain(abci.RequestInitChain{
		Validators:      []abci.ValidatorUpdate{},
		ConsensusParams: simapp.DefaultConsensusParams,
		AppStateBytes:   stateBytes,
	})

	ctx := app.NewContext(false, ocproto.Header{})

	addrs := []sdk.AccAddress{}
	addrCount := 100000
	coin := sdk.NewCoins(sdk.NewCoin("atom", sdk.NewInt(1000000000)))
	for ii := 0; ii < addrCount; ii++ {
		addr := sdk.AccAddress("addr" + strconv.Itoa(ii))
		acc := app.AccountKeeper.NewAccountWithAddress(ctx, addr)
		app.AccountKeeper.SetAccount(ctx, acc)
		simapp.FundAccount(app, ctx, addr, coin)
		addrs = append(addrs, addr)
	}

	sendAmount := sdk.NewCoins(sdk.NewCoin("atom", sdk.NewInt(1)))
	b.Run("Testing/IAVLCache", func(b *testing.B) {
		for ii := 0; ii < b.N; ii++ {
			from := rand.Intn(addrCount)
			to := rand.Intn(addrCount)
			// if fixed case:
			// from := 0
			// to := 1

			err = app.BankKeeper.SendCoins(ctx, addrs[from], addrs[to], sendAmount)
		}
	})
	require.NoError(b, err)
}

Result
The following table is the performance of BankKeeper.SendCoins() run with random address and fixed address for the 100,000 accounts. It can be seen that the case of BankKeeper.SendCoins() with cosmos/iavl and tendermint/tm-db is a little faster than that with line/iavl and line/tm-db.

Case
Random Addr
Fixed Addr

line/iavl, line/tm-db
16,661 ns/op
11,948 ns/op

cosmos/iavl, tendermint/tm-db
16,316 ns/op
11,845 ns/op

Conclusion
I can't think of any reason to keep the our fork versions(line/iavl and line/tm-db) of cosmos/iavl and tendermint/tm-db.


Initialization of  Default Protocol class

Description
Initialization of the Default Protocol class.The default protocol will contain all the primtive operations.In a session if a given protocol does not implement a specific operation.It would be reverted to the Default Protocol class.
Affected Dependencies
Since Falcon is 3PC,Adding DefaultProtocol would make RSTensor framework agnostic
How has this been tested?
Added tests for the same.
Checklist

[x] I have followed the Contribution Guidelines and Code of Conduct
[x] I have commented my code following the OpenMined Styleguide
[x] I have labeled this PR with the relevant Type labels
[x] My changes are covered by tests

We can close and come back to the PR later?


Load Fonts is to slow

Load Fonts is to slow ，Maybe it can be removed or changed to local.
url：
https://fonts.googleapis.com/css?family=Lato:400,700,400italic,700italic&subset=latin

If not changed to local and served with the site they can block the page. Also China blocks some Google domains which could cause 10s page loads depending on browser and location—even outside CN in AsiaPac.

Related #4837


[Feature] Improve Chat Picker

Example: I'm trying to make an alert for the "Your cannon has broken!" message. I assume that it is a Notification so I make that alert type, I use the message picker tool to verify that it is of that type from chat history, then I pick it. Turns out, that's actually a Game Message not a Notification so the alert doesn't work.
It'd be great if the message picker feature only showed messages which were relevant for the current alert type that you had configured so it wouldn't give you the option to pick something that ultimately wouldn't work.

Well, it's a chat message picker, not a notification message picker. There's not a way to tell if a game message came from a notification, so filtering isn't possible. Regardless, if you see a message in your chat, then that's a game message. So you can/should just use the game message alert. The use case for using the chat message picker on a notification fired alert is if you know it's a notification.
Aside from tracking every notification message fired (potentially viable, but not high priority in my mind), there's nothing that can be done. The alternative is not having the picker at all on notification fired alerts. Better than nothing!

Regardless, if you see a message in your chat, then that's a game message. So you can/should just use the game message alert

If you have the RuneLite "Game message notifications" setting enabled (which I do), then RuneLite Notifications show up in the chatbox and are indistinguishable from Runescape messages. I was hoping that the message picker would help identify which was which since it's nearly impossible to tell without guess and checking.
So I think if someone has that setting enabled, it's incorrect both ways. If someone is picking a Game Message, it'll show RuneLite Notification Messages in the message picker. If someone is picking a Notification, it'll show Game Messages in the message picker. I think it'll only work correctly for Game Messages if that setting is disabled.

Aside from tracking every notification message fired

Funny enough, that's what I assumed it was doing currently. But I guess it's actually just reading from the chatbox whenever you click the message picker rather than keeping it's own in-memory copy of the messages pre-categorized.
That'd also enable really useful things like the Player Chat Message which has the new Chat Type filter, if you select Chat Type friends and then click the message picker, it should only show you messages from friends, etc.

If you have a game message alert, and are sending notifications as game messages, it will work. I think that's what you're saying wouldn't work but correct me if I'm misunderstanding you.

Funny enough, that's what I assumed it was doing currently. But I guess it's actually just reading from the chatbox whenever you click the message picker rather than keeping it's own in-memory copy of the messages pre-categorized.

I just checked and actually I am storing chat messages myself, in an evicting queue of 20. So it keeps the 20 last messages. It was previously just reading the chat box, but it would have been more complicated to sort and such. So adding another tracker for notifications would be fine.

That'd also enable really useful things like the Player Chat Message which has the new Chat Type filter, if you select Chat Type friends and then click the message picker, it should only show you messages from friends, etc.

I really like that, that's a good idea.

If you have a game message alert, and are sending notifications as game messages, it will work. I think that's what you're saying wouldn't work but correct me if I'm misunderstanding you.

Ah I didn't realize that. What I had done was used the Notification Fired alert type. So I guess Notification Fired is actually a subset of Game Message? I guess I wonder why there's a difference then, that's confusing haha. TIL thank you!

 It's not a common use case, that's why I have it at the bottom of the list, and why I recommend against using it unless you know what you're doing.

Ah okay. So Notification Fired will work regardless of that RuneLite setting, but Game Message may or may not work depending on the RuneLite setting. So I guess yeah, if it's an alert based on a RuneLite notification it should use the Notification Fired alert type to make sure that it isn't broken by an external setting.
Thank you for explaining. 😄

Ya, and messages that are from runescape itself will only work with game message.

Fixed in f6bd7802849e95dc88a48a884b5d28c84a92b0c4 and dfff8d2f2dbdb1477642a004a265fcc92f547693


[java] Support for recursive merge of additional spec parameters

Method setAdditionalSpecParameters can only patch top-level spec fields. It would be convenient if it could also patch spec fields recursively.

Just to note it somewhere, it would also probably be nice to have something like setDockerImage in MapSpec/ReduceSpec builders.


Error: Uncaught, unspecified "error" event. (null)

row.update() causes the following error:
Error: Uncaught, unspecified "error" event. (null)
    at Parser.emit (events.js:163:17)
    at Parser.exports.Parser.Parser.parseString (/Users/hellouser/google-spreadsheet/node_modules/xml2js/lib/xml2js.js:512:16)
    at Parser.parseString (/Users/hellouser/google-spreadsheet/node_modules/xml2js/lib/xml2js.js:7:59)
    at Request._callback (/Users/hellouser/google-spreadsheet/node_modules/google-spreadsheet/index.js:164:24)
    at Request.self.callback (/Users/hellouser/google-spreadsheet/node_modules/request/request.js:186:22)
    at emitTwo (events.js:106:13)
    at Request.emit (events.js:191:7)
    at Request.<anonymous> (/Users/hellouser/google-spreadsheet/node_modules/request/request.js:1081:10)
    at emitOne (events.js:96:13)
    at Request.emit (events.js:188:7)
    at IncomingMessage.<anonymous> (/Users/hellouser/google-spreadsheet/node_modules/request/request.js:1001:12)
    at IncomingMessage.g (events.js:291:16)
    at emitNone (events.js:91:20)
    at IncomingMessage.emit (events.js:185:7)
    at endReadableNT (_stream_readable.js:974:12)
    at _combinedTickCallback (internal/process/next_tick.js:74:11)
    at process._tickCallback (internal/process/next_tick.js:98:9)

Funny thing: if I start the program using devtool instead of node, there is no error and every thing works fine.

hm, what version of node were you using?

I've just a published a new version of this module which is a complete rewrite using Google's V4 Sheets API (v3 is being deprecated on March 3, 2020).
Docs available here - https://theoephraim.github.io/node-google-spreadsheet
I do believe this issue should be fixed in the new version.
Please test it out and if you find that there is still a problem, reopen the issue.
Thanks! 😄


az ad app create command doesn't work. "password" argument is not recognized

Registering new app using this command no longer work ("Create an identity resource" section, "Multi-tenant" tab)
az ad app create --display-name "<name>" --password "<password>" --available-to-other-tenants

Output after trying to run this command:
This command or command group has been migrated to Microsoft Graph API. Please carefully review all breaking changes introduced during this migration: https://docs.microsoft.com/cli/azure/microsoft-graph-migration
unrecognized arguments: --password [password here] --available-to-other-tenants

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

ID: 096d934c-2856-e868-efeb-6622e2a5eafb
Version Independent ID: 8bf74926-6a35-5753-0a1a-6663ba5965b2
Content: Publish your bot to Azure - Bot Service
Content Source: articles/bot-builder-deploy-az-cli.md
Service: bot-service
GitHub Login: @JonathanFingold
Microsoft Alias: iawilt

Hi @piotrek-k,
As mentioned in the documentation linked in the error message, Azure CLI version 2.37.0 introduces breaking changes in the functionality of various commands due to the required migration to Microsoft Graph from the deprecated AAD Graph.
At the moment, older versions of Azure CLI should support those older commands, and we are working on getting documentation updated. Essentially, this is expected behavior, and the document should help explain which commands to use instead.

Instructions will indicate that (for now) Azure CLI v2.36 should be used for C#, JavaScript, and Python bots.

It should also tell people how to downgrade or install the correct out-dated version


docs(ui/dialog): use the setup script for example and docs

Checklist

[ ] Fix linting errors
[ ] Tests have been added / updated (or snapshots)

Change information

Codecov Report

Merging #458 (7279e54) into docs/script-setup (7da02f0) will not change coverage.
The diff coverage is n/a.

@@                Coverage Diff                 @@
##           docs/script-setup     #458   +/-   ##
==================================================
  Coverage              96.42%   96.42%
==================================================
  Files                    217      217
  Lines                   5148     5148
  Branches                 944      944
==================================================
  Hits                    4964     4964
  Misses                   184      184

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 7da02f0...7279e54. Read the comment docs.


Clear Steps

Can you update readme with steps to run this code with come examples.

No, sorry. This repository is deprecated.


Projects available on different computers

It is very frustrating when I connect to my Bolt account on a different computer and the chats/projects are not available. These are only available on the computer I initiated the chat/project with.

@ChatGPTBuddy a full fix for this is coming very soon, but a simple workaround can be found in #39 in the meantime


[Bug] When using Spring Boot version 3.2.2, addSink is unable to retrieve beans

Search before asking

[X] I searched in the issues and found nothing similar.

Flink version
1.18.1
Flink CDC version
3.0.1
Database and its version
8
Minimal reproduce step
Get Spring Bean
What did you expect to see?
If it is confirmed that there is no problem with this method, the bean cannot be obtained in the Flink CDC job
SpringUtils.getBean(EsProperties.class)

What did you see instead?
Failed to obtain Bean, it seems that all static variables cannot be obtained, so not only Spring's beans
Anything else?
No response
Are you willing to submit a PR?

[X] I'm willing to submit a PR!

The issue of integrating with Spring is not something that needs to be considered on the FlinkCDC framework.

The issue of integrating with Spring is not something that needs to be considered on the FlinkCDC framework.

ok


Multiple tiles of the same devices appearing on Homekit

Hi there,
I have recently added a ZemiSmart Homekit Zigbee Hub to my Smartlife app, so that I can pair zigbee devices to Homekit. However, the zigbee devices are appearing on Homekit through the ZemiSmart Homekit Zigbee Hub and Homebridge at the same time. I am now having 2 sets of same tiles for each devices. I tried to hide ZemiSmart Homekit Zigbee Hub from my Homekit with the configuration as shown below, but the zigbee devices are still appearing through Homebridge. Is there anyway to overcome this? Thank you.
{
"platform": "TuyaWebPlatform",
"name": "TuyaWebPlatform",
"scenes": true,
"options": {
"platform": "smart_life",
"username"<EMAIL_ADDRESS>"password": "XXXXXXXX",
"countryCode": "##"
},
"hiddenAccessories": [
"bf5e8a081eecf24a951xbd"
]
}

Hello,
I'm gonna need some more info from you.
Zigbee devices should not be exposed through my plugin so I don't understand how adding a zigbee hub could create duplicate devices?
Could you post a screenshot of your control app (probably the iOS home app) to show what you mean?
Please all post the result of the steps under "How to check whether the api this library uses can control your device".
These results will show if my plugin is indeed exposing these devices (and if so how they are exposing them and probably why)


Add simple GC for view array types

Which issue does this PR close?

Closes #5513 .
Rationale for this change
This PR is largely based on excellent work from @ClSlaid in #5720
I simplified the GC process and the test cases, and now they are dead simple (but dumb)!
Note that this pr does not check whether the original array is already compact (and thus avoids unnecessary copies). Once we get this in, we can continue the compact check discussion as in #5720

What changes are included in this PR?

Are there any user-facing changes?

I overhauled the test case, and it should be more straightforward now. Also added a simple benchmark -- in case we want to make a more performant version :)


Chrome doesn't start or pop up

I'm wondering why Chrome application doesn't show when I start testing. The test do work though. When I change setting to Firefox, it will show the Firefox application.
Is this a normal behavior?

Please read the contributing guidelines; these types of questions are ill-suited for GitHub. Thanks!


PSO constraint optimization

Recently, I'm using pso to solve the single objective function. This optimization is an integer programming with constraints, but it doesn't seem to have any effect in the running process. This is my code.At present, there are about 89 unequal constraints and 73 equal constraints.
problem = MyProblem(data.m_time[i],data.num_person[i],data.a_num[i],data.b_num[i],data.c_num[i],
data.d_num[i],data.e_num[i],data.person_dict[i],data.ability[i])
algorithm = PSO(pop_size=96,sampling=IntegerRandomSampling())
res = minimize(problem,
               algorithm,
               ('n_gen',2000),
               seed=1,
               verbose=False)
print("Best solution found: \nX = %s\nF = %s" % (res.X, res.F))

The PSO algorithm is designed to work in a continuous space. Also, the basic version of the algorithm does not support constraints.
I recommend using a GA in your case: https://pymoo.org/customization/discrete.html


New tab search bar doesn't work

Operating system: Ubuntu 17.10 x86_64, Kernel 4.13.0-37
This is:

[x] a bug
[ ] general feedback

The search bar on the new tab / start page does nothing on enter or on click.
Input on the bar throws an uncaught error:

Uncaught (in promise) Error: SQLITE_ERROR: duplicate column name: pinOrder
at onUpdateSearchQuery (main.js:1010)
at onInputSearch (main.js:998)

Hitting enter throws this error:

Uncaught TypeError: Cannot read property 'targetUrl' of undefined
at onInputSearch (main.js:978)
onInputSearch @ main.js:978
setTimeout (async)
delay @ main.js:1248
yo.update.yo @ main.js:1125

I screwed up the rollout of the last DB update. Basically a fresh install doesnt set the schema version to latest, so an incremental DB update is being applied that doesnt need to be (an ALTER TABLE which adds the pinOrder column).
Latest master solves this but I'm going to need to deploy a new version soon to deal with this

I have to add that Beaker browser for Windows (downloaded on 10/04/2018 from Beaker Browser website) doesn't have the next and previous buttons in the search bar.

@ideocrat which version of Beaker are you using? Can you share a screenshot?

Ah, we’re no longer actively developing that version. We have a prerelease
for a new version available at:
https://github.com/beakerbrowser/beaker/releases/tag/0.8.0-prerelease.5
I recommend giving it a spin!
On Thu, Apr 12, 2018 at 01:24 ideocrat<EMAIL_ADDRESS>wrote:

[image: capture]
https://user-images.githubusercontent.com/29451306/38664864-7e62540a-3e43-11e8-88c2-83a901939466.PNG
I downloaded the windows version that is on the beaker browser website. It
is the version 0.7.11. I'm not sure if I can find a newer version on github.
Anyway in the screenshot there are no up and down arrows next to the
search bar. In the scroll bar to the right you can see where your search
results are but you can't jump immediately between them by pressing on
arrows.
The second screenshot shows the arrows next to the search bar.
Thanks!
[image: other browser]
https://user-images.githubusercontent.com/29451306/38665020-fbfbca4a-3e43-11e8-9736-f1f0861da393.PNG
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/beakerbrowser/beaker/issues/934#issuecomment-380720257,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AHO8QUlPilwEn8PR8QvdGoWlvKY0S5Ycks5tnw8lgaJpZM4TDj6p
.


Bug in the generation of files in the build

Hello, I have a problem with the generation of files in the build: npm run build.
The file index.spa.html have three tag script with path js file into folder /_nuxt but this folder not exist, ie:
<script src="/_nuxt/manifest.xxx.js"/>
<script src="/_nuxt/common.xxx.js"/>
<script src="/_nuxt/app.xxx.js"/>

The JS files manifest, common and app is at the folder root (dist).
Info:
Nuxt 1.0.0-rc3
Node v6.11.1
SO Windows 10

Hi @juliandavidmr . For SPA deployment you have to deploy dist folder at the root of your project (~/dist) not one in the .nuxt dir. (Please upgrade your nuxt to rc6 please for a better SPA experience too)


[Bug] Update from v0.20 to v0.21 fails with existing mariadb object (OLM)

Describe the bug

Is the bug related to Galera?
No
Expected behaviour
The new operator csv does not install successfully because the old CRD has an incompatible field type spec.image.  It changed from an object to a string.
Steps to reproduce the bug
Create a mariadb object (note: spec.image is required) with operator version 0.20 and update to 0.21.
Can I just remove all mariadb objects and recreate them after the update is completed?  Does it delete the running databases?

The problem is, that I can't update the manifests, because the old CRDs are still installed.  I guess that the new CRDs can't be installed because the old objects don't confirm to the new CRD.  Looks like a catch-22.
Maybe this does not happen with the operator being installed with helm, but with OLM it does not work.

Yes, I was lucky I didn't have to care about downtime. Deleted the custom objects and ran
kubectl replace -f https://raw.githubusercontent.com/mariadb-operator/mariadb-operator/main/deploy/crds/crds.yaml

After a while, Flux managed to reconcile. Using Helm under the hood didn't help.
I guess this is the price of using alpha APIs.

I wonder, whether I'll have to dump and restore the DB.  I don't mind some downtime, but I'd rather not lose the mysql data dir.

For me the persistent volume claim stayed in place and got picked up again by the same MariaDB (with the same name and same namespace). But make a backup if it's production data. YMMV.

Had the same problem. I guess the CRD-Schema did not change so Flux doesnt know it should replace the CRDs. I yeeted all CRDs and the deployments. Than let Flux reconcile and everything came back up. I use local-storage as a PVC. Dont know if this helped. I would have had a backup in case this wouldnt have worked. Best luck to you. I'm on this operator for over a year now. It is breaking frequently.

Hey there, sorry to hear that you are experienced problems with the migration. Unfortunately we are still in alpha version and shaping the API of our CRDs, so sometimes we have to introduce breaking changes. As mentioned in this thread, here it is the migration guide:

https://github.com/mariadb-operator/mariadb-operator/blob/main/docs/UPGRADE_v0.0.21.md

Maybe this does not happen with the operator being installed with helm, but with OLM it does not work.

I will give it a try but at the very end, our helm based OLM operator installs our helm chart programatically, so there should be a way to achieve this. Probably you will need to downscale the OLM operator itself to avoid conflicts while updating the Helm release manually. Have you tried this out?

I guess that the new CRDs can't be installed because the old objects don't confirm to the new CRD

No. This won't be a problem. CRD installations don't depend on the current objects. When you install a need CRD version, the old fields get wiped out and then you perform the migrations as indicated in the migration guide to conform to the new CRD version.

For me the persistent volume claim stayed in place and got picked up again by the same MariaDB

Yes! As long as you don't delete the PVCs, you won't loose the mysql data dir, as they will be reused by the new MariaDB instance.

Hello there,
I've something closed to this .
I've also try to upgrade with OLM.
I've followed all breaking change guide ... but I got a really short timing.
I think it works but I've deleted mutatingwebhookconfiguration and validatingwebhookconfiguration
Do you know how can i restore a nice and clean situation ?
Best regards


Device not found!

Hello,
I've tried to use this library with my DS18B20 sensor and a NodeMCU v1.
The circuit is very simple, the DS18B20 GND>GND, Data & Vin > 4K7 Resistor, Data>D4, Vin>3v3.
I use the full code of the Examples/Simple. There is an address, which is not documented:
uint8_t address[] = {40, 250, 31, 218, 4, 0, 0, 52};
Since I use this example, I've got the following exception:
14:17:44.292 -> Device not found!
The device is not recognized on the given address, and there is not any clue in both of your code, or in the documentation. What can I do? How can I figure out the address of my sensor?
Thanks in advance,
Montyx

Hi @montyx99
That address is hardcoded from one of my own devices, if I remember correctly. Please try the other example, "Multiple", which will automatically detect any devices present on the bus and print their addresses (which you can then use in the first example).


MM-35518 - Add CodeQL check

Summary
Added step to run automated CodeQL checks. Once this file is merged, the codeql checks (code scanning checks) will automatically be performed on following conditions:

new pull requests to master branch
scheduled to run every week once (it automatically pulls the latest codeql checks added, if any)

Ticket Link
Fixes https://mattermost.atlassian.net/browse/MM-35518

@srkgupta, is this still a work in progress?

FYI @crspeller,
The CodeQL checks were previously reverted in this PR. I am re-enabling them. The Security team will be notified of any new CodeQL alerts and will take appropriate actions, i.e.

raise an alert in the corresponding PR (or)
dismiss it as false positive/won't fix.

As of now, there are no new alerts generated.

@srkgupta, is this still a work in progress?

@agarciamontoro Sorry, I completely missed at re-looking this PR once again. Thanks for bringing this to my attention. It's complete and ready for review.

/update

/update-branch

@agarciamontoro as of now, if there are any alerts generated on a PR, it will not block the PR from being merged. Its always advised for the owner/reviewer to glance the alert and verify if the alert is valid and requires some code changes (or) if its handled through some other restrictions in place and is a false positive.
The Security team will be anyways be notified of any new CodeQL alerts and will take appropriate actions, i.e.

raise an alert in the corresponding PR as a comment.
dismiss it as false positive/won't fix.

Even if the PR gets merged with an open CodeQL alert (due to emergency fixes / by mistake), the security team will create a separate ticket and follow up and ensure that the issue is fixed if its a valid issue.

Makes sense. Thanks for the explanations, @srkgupta :)


Add Linode driver

This is still a work in progress. I've manually tested parts of it but not a full end to end use.
Just wanted to get this up here to see that it is all going in the right direction in case I miss something.
Will close https://github.com/metacloud/molecule/issues/1450.

Looking good so far....  I suggest looking at #1436 and how the model was implemented along with tests.  Docker and vagrant are the only drivers which really implement an API from molecule to create.yml.  Might be good to define that as part of this.

I suggest looking at #1436 and how the model was implemented along with tests. Docker and vagrant are the only drivers which really implement an API from molecule to create.yml. Might be good to define that as part of this.

OK, I've added the model support, I think. There are now unit tests in place to guard for a proper schema.
(.venv) ➜  molecule (add-linode-driver) ✔ py.test test/unit/model/v2/ -k linode
=============================================================== test session starts ===============================================================
platform linux2 -- Python 2.7.13, pytest-3.0.7, py-1.6.0, pluggy-0.4.0
rootdir: /home/decentral1se/hobby/molecule, inifile: pytest.ini
plugins: testinfra-1.14.1, verbose-parametrize-1.3.0, mock-1.10.0, helpers-namespace-2017.11.11, cov-2.6.0
collected 59 items

test/unit/model/v2/test_platforms_section.py .......

=============================================================== 52 tests deselected ===============================================================
===================================================== 7 passed, 52 deselected in 0.16 seconds =====================================================
(.venv) ➜  molecule (add-linode-driver) ✔

Unfortunately, I've found a fairly major bug while E2E testing this which stops this feature from actually being any use at all. That is reported at https://github.com/ansible/ansible/issues/45403 where the bug stops the Linode module destroying the instances it creates :face_with_thermometer:
I'll be heading back over that way to fix that bug. Hopefully, I can get back here in a reasonable time.

OK, false alarm on https://github.com/ansible/ansible/issues/45403, it was just a docs issue. I'll explain the short of it, since it matters here: when a linode is created a linode_id is generated and that is needed to specify that linode on deletion. So, specifying linode_id in the model doesn't make sense because it won't match the generated value, so I am going to try and adapt my destroy.yml to match what is in the ec2/destroy.yml using the molecule_instance_config lookup.

OK, @retr0h, this is ready for a review!
I see I just missed the latest release but perhaps this is better off on the devel branch for now ...
So, the manual testing I've done for this:

molecule init role -r foobar -d linode
molecule test
add 3 instances and run molecule test
molecule create && molecule login --host instance1
molecule login

It seems to be in working order!

Related: https://github.com/metacloud/molecule/issues/1479.

@retr0h, can I clarify if you're leaning towards rejecting this PR or accepting it?

@lwm Linode is probably going to be the last.  It's something I have been meaning to do for quite some time.

@lwm Nicely implemented!  I know adding drivers is a pain.

@lwm Can you please run tox -e format.  Looks like formatting is off.

This is missing a bunch of unit tests.  I'm going through it now on my checkout.  Hold off on submitting any additional work.

Sorry for the delay on this @lwm -- when I add the unit tests for the linode driver, I am getting all kinds of other problems, which means I stumbled across a bug in how I test drivers.  I have been busy the last couple weeks and apologize for not looking into this sooner.

No worries! Thanks for the update. Happy to have helped smoke out a new bug :)

Just in case others look at this, Linode & Molecule both have Working Groups within IRC
#ansible-linode https://github.com/ansible/community/tree/master/group-linode
#molecule-users

I'm going to hold off from merging this.  As you know Ansible is adopting Molecule, therefore I do not want to merge anything that may impact future architectural decisions, as I will no longer be leading the development of this project.

Once the repo has been moved under Ansible we can look at getting this merged.
In the short term we welcome testing and feedback on this PR.
Thanks for your understanding.

All good @retr0h! Thanks for getting us here. Solid software if I ever saw it.
@gundalow, any ETA on when this might get moved + merged?

Adding this to the v2.20 milestone

@webknjaz, can you drop me a line when #1558 is merged and I can make start on updating? Some tips would be nice on what I should focus on. Thanks.

@lwm please take into account that #1558 brutally disables a bunch of tests. Any ideas of how to improve that?

Ref #1561

@webknjaz. I've:

[x] Added missing Linode driver unit tests
[x] Taken a stab at adding the Linode tests under tox correctly

I do no know how to add 'scenario' tests using the resources folder. I don't even know what they do really and I am wondering if some mock Linode server is required. Do they run a full molecule run for real?!

@lwm yes, that's the problem. They run a full molecule run with real drivers. We usually have dead simple servers HTTP-responding with pre-recorded payload, like: https://github.com/ansible/foreman-test-container/blob/master/foreman_stub.py#L26-L46

OK, well, what is the plan? I don't think I have time to implement the structure for all this but if someone lays something out, I can follow along and then implement responses for my driver. Please do let me know how I can move this forward.
I'm not blaming anyone, it's just timing, but I'm waiting a heck of a long time for this to be merged.

Hey @lwm, it looks like you forgot to set up GPG key at https://github.com/settings/keys ;)

Thanks for the review @webknjaz! I've addressed comments and setup my GPG key.

OK, ready for another review.

These fixtures may be useful for the stub server: https://github.com/linode/linode_api4-python/tree/2.0.0/test

@webknjaz, this PR won't be merged until I implement a full scenario test suite for the driver?
Could we merge this first and then I could work on that afterwards? The change set will be big.

I guess we could merge it w/o stub. But have you tried running functional tests locally? with smth like tox -- -k linode? I saw you added params to tests, it should work if you have linode client set up locally.

All the unit tests pass along with style checks. I've put more place holders in now, so that I can run the functional tests. A number of them are failing. The functional tests actually run the scenarios, which is a pain.

shipit

OK, tox -- -k linode is passing now (for all Pythons I have installed locally). All functional tests are skipped because I don't install the linode extras via Tox but they do get to the point where they contact the server. I will try to get some time to implement the stub server (but could do with some guidance and surely others will need this too). Can we merge this now?

@lwm this needs bits of clean up, but overall is good.
N.B. I didn't mark every place with repeating comments.

Taken another pass! Thanks.

tox -e py27-ansible27-unit -- -k linode passes. When run with the rest, a lot of unrelated tests fail. I have dropped an hour into this but I can't track down whatever ordering or sequencing is causing this. 59610a8428dab7622b39cb6a782a41c78fd3a298 was the key commit to make the Linode related unit tests pass again. Some pytest fixture black magic.

Anyone got any clue as to what this might be, I am all ears.

@lwm plz try rebasing

@webknjaz, when running tox -e pyXX-ansible27-unit -- -k test_driver, we see that directly after the test_drivers[linode], we are getting failures (see Travis CI logs here). When I then make the following diff to the test_drivers test:
diff --git a/test/unit/cookiecutter/test_molecule.py b/test/unit/cookiecutter/test_molecule.py
index 937001cc..58ae5291 100644
--- a/test/unit/cookiecutter/test_molecule.py
+++ b/test/unit/cookiecutter/test_molecule.py
@@ -96,7 +96,6 @@ def test_vagrant_driver(temp_dir, _molecule_file, _role_directory,
     ('docker'),
     ('ec2'),
     ('gce'),
-    ('linode'),
     ('lxc'),
     ('lxd'),
     ('openstack'),

Then, if I run tox -e py27-ansible27-unit -- -k test_driver it passes.
This is one set of failures (there are more but I think the root of it is in the validation). I think it might have something to do with:
https://github.com/ansible/molecule/blob/467cb2bf3518af81bd0808a70facd9923f1f449d/molecule/model/schema_v2.py#L943
since the following failing tests seem to run schema_v2.validate(data) with the linode schema merged into the validation schema and not the schema for the driver of the test (LXC in the first failing test). It's like the tests are not tearing down correctly.
I ran out of time again but this bug is somewhere related to the schema V2 validation or in the tests setup/tear down. Any ideas?

Thanks for the explanation. No ideas so far but I believe you're probably right about broken teardown: I've fixed numerous problems with it already after the acquisition and I expect that there may be more issues left unnoticed.
We'll need to tackle schema validation closer now.

Do we need something like https://github.com/ansible/ansible/blob/devel/test/sanity/validate-modules/schema.py to validate the he schema?

@gundalow there's schema validation built-in Molecule already:  https://github.com/ansible/molecule/blob/master/molecule/model/schema_v2.py

https://github.com/ansible/molecule/pull/1639 might get me moving forward ...


Promise.all array types incorrect in ES6

TypeScript Version:
1.8.10 (appears to be fixed in 1.9.0-dev.20160422 by removal of the final Promise.all fall through type)
Code
const foo: string[] = []

const value = Promise.all(foo.map(async (id) => {
  return Promise.resolve(id)
}))

function acceptsPromises (p: Promise<string[]>) {}

acceptsPromises(value)

{
  "compilerOptions": {
    "target": "es6"
  }
}

Expected behavior:
Promise.all<TAll> should unwrap the array of promises to Promise<string[]>, instead the structures are incompatible under ES6 and you get Promise<Promise<string>[]>.
Actual behavior:
index.ts(9,17): error TS2345: Argument of type 'Promise<Promise<string>[]>' is not assignable to parameter of type 'Promise<string[]>'.
  Type 'Promise<string>[]' is not assignable to type 'string[]'.
    Type 'Promise<string>' is not assignable to type 'string'.

looks like  dupe of https://github.com/Microsoft/TypeScript/issues/7566

@vladima Thanks for that, I swear I tried to find it - using the wrong keywords I suppose. Cheers!


license issue

Hi,
i'm working on a project for my classroom. I started from a p5 preview plugin and use come of your code in order to make it work better (console.log in the debug window was the problem that make me find this plugin not useful because I want to use it with my students at school). I look at you repo in order to check the license and find that you haven't one, so now I have to delete my project and I can'r deliver it to my students. It'n not a production code but perhaps the community will take some advantages if I can publish it.
I think you will choose a license soon, so I can deliver my project, otherwise I will remove every code form your repo and redo that part in some other way.
good evening from Italy

Glad to hear this is useful to you, Andrea!
Shame that using the developer tools doesn't suit your use case.
I've added the license file, thanks for the feedback!


Adding some recipes for Mythic Metals compatibility

For #92
Added compatibility recipes for Mythic Metals and moved Alloy Forgery compatibility recipes into datagen.

Great!
Btw could you add "close #92" in the commit message so it automatically closes that issue?

This is looking great, thanks for all the worr that went into this. When merging i'll adjust the message to close the issue with this. One more thing before merging: Is there a way to only include those mobs in datagen, and maybe turn them on and off when actually running the dev environment? I don't want to have all  compat mods active when working on the mod, and having to manually enable them and reload gradle before each datagen also doesnt seem right?

I'll see if I can add the dependencies in a datagen run configuration.

That was easy. I tried adding a dependencies block inside of configureDataGeneration and the datagen is still able to run. It also makes it more obvious what the dependencies are for and prevents anything else from accidentally relying on them.

Thanks, this is looking cleaner already. I'll go ahead and merge it


Bump Dependencies #337

Bumps dependencies for ${REPOSITORY}.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.jenkins-is-staging seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.