Array | Uncaught exception - Invalid field access

Uncaught exception - Invalid field access
I'm working on a game engine. When working in the camera concept, I came across a problem; "Uncaught exception - Invalid field access" it won't let me make use of the array of GameObjects 'activeObjects' during runtime. At first I thought it was because I was accessing it from another class, but I tried to trace the length of the array from the class Game, but it won't work either way.
How can I fix it?
Here's Main.hx:
package;

import openfl.display.Sprite;

import openfl.events.Event;

import Game;

class Main extends Sprite {

	var inited:Bool;

	public static var game:Game;

	function resize(e) {

		if (!inited) init();

	}

	function init() {

		if (inited) return;

		inited = true;

		game = new Game();
		game.initGame(true, 16);
		addChild(game);

	}

	public function new() {

		super();

		addEventListener(Event.ADDED_TO_STAGE, added);

	}

	function added(e) {

		removeEventListener(Event.ADDED_TO_STAGE, added);

		stage.addEventListener(Event.RESIZE, resize);

		init();

	}

}

And Game.hx:
package;

import openfl.Lib;

import openfl.events.Event;

import openfl.display.Sprite;
import openfl.display.StageDisplayState;

class Game extends Sprite {

    public var activeObjects:Array<GameObject>;

    public var worldScale:Int;

    var mainCamera:Camera;

    public function new() {

        super();

    }

    public function initGame(fullScreen:Bool, _worldScale:Int):Void {

        if (fullScreen) Lib.current.stage.displayState = StageDisplayState.FULL_SCREEN_INTERACTIVE;

        worldScale = _worldScale;

        addEventListener(Event.ENTER_FRAME, Update);

        new Player();
        mainCamera = new Camera(16, 9);

    }

    function Update(event:Event):Void {

        GetDeltaTime();

        //I get invalid field access when using either of these:
        trace(Main.game.activeObjects.length);
        trace(this.activeObjects.length);
        trace(activeObjects.length);

        //It also doesn't work if I try to access anything of the
        //variable, even if I try to use a function of the array

    }

    var deltaTime:Float = 0;
    var timeElapsed:Int = 0;
    var lastFrameTime:Int = 0;
    function GetDeltaTime() {

        timeElapsed = Lib.getTimer();
        deltaTime = (timeElapsed - lastFrameTime)/1000;
        lastFrameTime = timeElapsed;

        return deltaTime;

    }

}

Haxe version: 4.2.4
Lime version: 7.9.0
OpenFL version: 9.1.0
Target: neko

You need to initialize activeObjects array.
public var activeObjects:Array<GameObject>;

to
public var activeObjects:Array<GameObject> = [];

Here is fix

ohhhh, thanks!
I didn't know you had to initialize arrays!
thank you for the quick answer


Add ci argument to fail on badge changes

What?
I've added a new argument ci which will throw an error if the old and new badges don't match. Along with updating the README.md with a usage guide.
Why?
There are scenarios such as working on large codebases where it isn't always practical to run all the tests and produce coverage on each commit via githooks.
By providing this option for teams, they can include this in their CI for simple validation and peace of mind it's being updated.
Testing?
I wasn't too sure how to go about adding testing for this as couldn't see an example of another method being tested where getArgumentValue() was being stubbed or calling a test with an argument.
Would be very keen to add some in with help 😃

I discovered this package at work today and instantly fell in love, I've set it up for my codebase and ran into this challenge where it isn't feasible to run all the tests on each commit so thought it easiest to put a PR with this functionality.
Looking forward to hearing your thoughts on it and hope others may be able to benefit from it 🔥

@troypoulter thank you very much for oppening this pull request. Very interesting proposal! :D Could I ask you one more thing? Have you added this new case into tests folder?

@troypoulter Thank you again, man, I am looking into it right now. As for the Github actions, please do feel free to do so hahaha I would very much appreaciate it! 🍻

@all-contributors please add @troypoulter for code, ideas and test!

@troypoulter please do check for version 1.5.0!
again, thank you very very much and I hope you like it (your own doing haha)


Clarify the rules for using styled #RGB values

Document that an unqouted # character has special meaning to the Elvish
parser in the context of passing #RGB values to the styled command.

 Is there any ambiguity that requires the # prefix? Why not simply test whether the string (minus any fg- or bg- prefix) consists of six hexadecimal characters?

I also realized the fg- color name prefix wasn't documented so added a bullet item for it as well.

Didn't see your comments previously - I feel #rrggbb is so common elsewhere that it's worth doing the same in Elvish even if that requires quoting.
I don't like omitting the # because some hexdecimal sequences can sound like words which can be quite confusing, decafe for example.


mavros: 2.2.0-1 in 'rolling/distribution.yaml' [bloom]

Increasing version of package(s) in repository mavros to 2.2.0-1:

upstream repository: https://github.com/mavlink/mavros.git
release repository: https://github.com/ros2-gbp/mavros-release.git
distro file: rolling/distribution.yaml
bloom version: 0.11.1
previous version for package: 2.1.0-2

libmavconn
* Merge pull request #1720 <https://github.com/mavlink/mavros/issues/1720> from SylvainPastor/fix/libmavconn/udp/deadlocks
  libmavconn: fix UDP deadlocks
* libmavconn: fix UDP deadlock
  Same problems as for the TCP:
  - #1682 <https://github.com/mavlink/mavros/issues/1682>: fix std::system_error when tcp interface loses connection
  - #1679 <https://github.com/mavlink/mavros/issues/1679>: fix deadlock when call close()
* Contributors: Sylvain Pastor, Vladimir Ermakov

mavros
* extras: fix build
* Merge branch 'master' into ros2
  * master:
  mount_control.cpp: detect MOUNT_ORIENTATION stale messages
  ESCTelemetryItem.msg: correct RPM units
  apm_config.yaml: add mount configuration
  sys_status.cpp fix free memory for values > 64KiB
  uncrustify cellular_status.cpp
  Add CellularStatus plugin and message
  *_config.yaml: document usage of multiple batteries diagnostics
  sys_status.cpp: fix compilation
  sys_status.cpp: support diagnostics on up-to 10 batteries
  sys_status.cpp: do not use harcoded constants
  sys_status.cpp: Timeout on MEMINFO and HWSTATUS mavlink messages and publish on the diagnostics
  sys_status.cpp: fix enabling of mem_diag and hwst_diag
  sys_status.cpp: Do not use battery1 voltage as voltage for all other batteries (bugfix).
  sys_status.cpp: ignore sys_status mavlink messages from gimbals
  mount_control.cpp: use mount_nh for params to keep similarities with other plugins set diag settings before add()
  sys_status.cpp: remove deprecated BATTERY2 mavlink message support
  Mount control plugin: add configurable diagnostics
  Bugfix: increment_f had no value asigned when input LaserScan was bigger than obstacle.distances.size()
  Bugfix: wrong interpolation when the reduction ratio (scale_factor) is not integer.
  Disable startup_px4_usb_quirk in px4_config.yaml
* cmake: style fix
* cmake: downgrade to C++17 as 20 breaks something in rclcpp
* pylib: fix flake8 for checkid
* cmake: hide -std=c++2a
* Merge pull request #1752 <https://github.com/mavlink/mavros/issues/1752> from alehed/fix/make_compatible_with_mavlink_pr_666
  Make compatible with pymavlink type annotations PR
* Make compatible with pymavlink type annotations PR
  In that PR, the attribute name is changed to msgname due to conflicts
  with message instance variables.
* Merge pull request #1744 <https://github.com/mavlink/mavros/issues/1744> from amilcarlucas/pr_gimbal_diagnostics_fixes
  mount_control.cpp: detect MOUNT_ORIENTATION stale messages
* mount_control.cpp: detect MOUNT_ORIENTATION stale messages
  correct MountConfigure response success
  correct constructor initialization order
  some gimbals send negated/inverted angle measurements, correct that to obey the MAVLink frame convention using run-time parameters
* Update global_position.py
* Merge pull request #1745 <https://github.com/mavlink/mavros/issues/1745> from antonyramsy/ros2
  fix subscribe_raw_salellites typo
* fix subscribe_raw_salellites typo
  subscribe_raw_salellites -> subscribe_raw_satellites
* Merge pull request #1743 <https://github.com/mavlink/mavros/issues/1743> from amilcarlucas/pr_apm_config
  apm_config.yaml: add mount configuration
* apm_config.yaml: add mount configuration
* Merge pull request #1732 <https://github.com/mavlink/mavros/issues/1732> from amilcarlucas/pr-meminfo-fix
  MEMINFO fixes
* sys_status.cpp fix free memory for values > 64KiB
* Merge pull request #1716 <https://github.com/mavlink/mavros/issues/1716> from amilcarlucas/avoid-harcoded-values
  sys_status.cpp: do not use harcoded constants
* Merge pull request #1711 <https://github.com/mavlink/mavros/issues/1711> from amilcarlucas/diagnose-up-to-n-batteries
  Diagnose up-to 10 batteries
* *_config.yaml: document usage of multiple batteries diagnostics
* sys_status.cpp: fix compilation
* sys_status.cpp: support diagnostics on up-to 10 batteries
  Uses as many battery monitors as the user specified in min_voltage parameter.
  Add myself as a contributor, this is not my first patch to this file
* Merge pull request #1712 <https://github.com/mavlink/mavros/issues/1712> from amilcarlucas/fix-disabled-diagnostics
  sys_status.cpp: fix enabling of mem_diag and hwst_diag
* sys_status.cpp: do not use harcoded constants
* sys_status.cpp: Timeout on MEMINFO and HWSTATUS mavlink messages and publish on the diagnostics
  Use atomic variable to prevent potential threading problems
* sys_status.cpp: fix enabling of mem_diag and hwst_diag
* Merge pull request #1704 <https://github.com/mavlink/mavros/issues/1704> from amilcarlucas/correct-bat-voltages
  sys_status.cpp: Do not use battery1 voltage for all batteries.
* sys_status.cpp: Do not use battery1 voltage as voltage for all other batteries (bugfix).
  Support both cell and total voltages above 65V
  Support up-to 14S batteries
  If available, add cell voltage information to the battery diagnostic
* Merge pull request #1707 <https://github.com/mavlink/mavros/issues/1707> from amilcarlucas/ignore-gimbal-sys-status
  sys_status.cpp: ignore sys_status mavlink messages from gimbals
* sys_status.cpp: ignore sys_status mavlink messages from gimbals
* Merge pull request #1703 <https://github.com/mavlink/mavros/issues/1703> from amilcarlucas/remove-deprecated-battery2
  sys_status.cpp: remove deprecated BATTERY2 mavlink message support
* sys_status.cpp: remove deprecated BATTERY2 mavlink message support
* Merge pull request #1696 <https://github.com/mavlink/mavros/issues/1696> from okalachev/patch-2
  Disable startup_px4_usb_quirk in px4_config.yaml
* Disable startup_px4_usb_quirk in px4_config.yaml
* Contributors: Alexander Hedges, Dr.-Ing. Amilcar do Carmo Lucas, Karthik Desai, Oleg Kalachev, Vladimir Ermakov, antonyramsy

mavros_extras
* extras: fix build
* extras: fix build
* Merge branch 'master' into ros2
  * master:
  mount_control.cpp: detect MOUNT_ORIENTATION stale messages
  ESCTelemetryItem.msg: correct RPM units
  apm_config.yaml: add mount configuration
  sys_status.cpp fix free memory for values > 64KiB
  uncrustify cellular_status.cpp
  Add CellularStatus plugin and message
  *_config.yaml: document usage of multiple batteries diagnostics
  sys_status.cpp: fix compilation
  sys_status.cpp: support diagnostics on up-to 10 batteries
  sys_status.cpp: do not use harcoded constants
  sys_status.cpp: Timeout on MEMINFO and HWSTATUS mavlink messages and publish on the diagnostics
  sys_status.cpp: fix enabling of mem_diag and hwst_diag
  sys_status.cpp: Do not use battery1 voltage as voltage for all other batteries (bugfix).
  sys_status.cpp: ignore sys_status mavlink messages from gimbals
  mount_control.cpp: use mount_nh for params to keep similarities with other plugins set diag settings before add()
  sys_status.cpp: remove deprecated BATTERY2 mavlink message support
  Mount control plugin: add configurable diagnostics
  Bugfix: increment_f had no value asigned when input LaserScan was bigger than obstacle.distances.size()
  Bugfix: wrong interpolation when the reduction ratio (scale_factor) is not integer.
  Disable startup_px4_usb_quirk in px4_config.yaml
* cmake: style fix
* cmake: downgrade to C++17 as 20 breaks something in rclcpp
* cmake: hide -std=c++2a
* Merge pull request #1744 <https://github.com/mavlink/mavros/issues/1744> from amilcarlucas/pr_gimbal_diagnostics_fixes
  mount_control.cpp: detect MOUNT_ORIENTATION stale messages
* extras: fix cog re to extract plugin name
* mount_control.cpp: detect MOUNT_ORIENTATION stale messages
  correct MountConfigure response success
  correct constructor initialization order
  some gimbals send negated/inverted angle measurements, correct that to obey the MAVLink frame convention using run-time parameters
* Merge pull request #1735 <https://github.com/mavlink/mavros/issues/1735> from clydemcqueen/fix_1734
  Fix crash in vision_pose plugin
* Remove unrelated log message
* Initialize last_transform_stamp with RCL_ROS_TIME, fixes #1734 <https://github.com/mavlink/mavros/issues/1734>
* Merge pull request #1727 <https://github.com/mavlink/mavros/issues/1727> from BV-OpenSource/pr-cellular-status
  Pr cellular status
* uncrustify cellular_status.cpp
* Add CellularStatus plugin and message
* Merge pull request #1702 <https://github.com/mavlink/mavros/issues/1702> from amilcarlucas/mount_diagnostics
  Mount control plugin: add configurable diagnostics
* mount_control.cpp: use mount_nh for params to keep similarities with other plugins
  set diag settings before add()
* Mount control plugin: add configurable diagnostics
* Merge pull request #1700 <https://github.com/mavlink/mavros/issues/1700> from oroelipas/fix-obstacle-distance
  Fix obstacle distance
* Bugfix: increment_f had no value asigned when input LaserScan was bigger than obstacle.distances.size()
* Bugfix: wrong interpolation when the reduction ratio (scale_factor) is not integer.
* Contributors: Clyde McQueen, Dr.-Ing. Amilcar do Carmo Lucas, Rui Mendes, Vladimir Ermakov, oroel

mavros_msgs
* Merge branch 'master' into ros2
  * master:
  mount_control.cpp: detect MOUNT_ORIENTATION stale messages
  ESCTelemetryItem.msg: correct RPM units
  apm_config.yaml: add mount configuration
  sys_status.cpp fix free memory for values > 64KiB
  uncrustify cellular_status.cpp
  Add CellularStatus plugin and message
  *_config.yaml: document usage of multiple batteries diagnostics
  sys_status.cpp: fix compilation
  sys_status.cpp: support diagnostics on up-to 10 batteries
  sys_status.cpp: do not use harcoded constants
  sys_status.cpp: Timeout on MEMINFO and HWSTATUS mavlink messages and publish on the diagnostics
  sys_status.cpp: fix enabling of mem_diag and hwst_diag
  sys_status.cpp: Do not use battery1 voltage as voltage for all other batteries (bugfix).
  sys_status.cpp: ignore sys_status mavlink messages from gimbals
  mount_control.cpp: use mount_nh for params to keep similarities with other plugins set diag settings before add()
  sys_status.cpp: remove deprecated BATTERY2 mavlink message support
  Mount control plugin: add configurable diagnostics
  Bugfix: increment_f had no value asigned when input LaserScan was bigger than obstacle.distances.size()
  Bugfix: wrong interpolation when the reduction ratio (scale_factor) is not integer.
  Disable startup_px4_usb_quirk in px4_config.yaml
* msgs: support humble
* Merge pull request #1742 <https://github.com/mavlink/mavros/issues/1742> from amilcarlucas/correct_rpm_units
  ESCTelemetryItem.msg: correct RPM units
* ESCTelemetryItem.msg: correct RPM units
* Merge pull request #1727 <https://github.com/mavlink/mavros/issues/1727> from BV-OpenSource/pr-cellular-status
  Pr cellular status
* Add CellularStatus plugin and message
* Contributors: Dr.-Ing. Amilcar do Carmo Lucas, Rui Mendes, Vladimir Ermakov

Holding for Rolling sync


Add documentations on source code directory

Currently, several guide documentations are on the Github issues.
However, Github issue is not adequate to accommodate those guides.
It is good to move those from Github issue to /docs directory in source code.

With PR #10, I added documentations on source code directory.
Therefore I close this issue.


Improve error reporting when two packages in a monorepo use conflicting versions

using https://github.com/invertase/dart_custom_lint/issues/2 instead


LG not start when pc run.

Hi,
I try my best,  but porgram not start when I turn on PC.
turn off seems work well, but only that.

Hi @devNostyle
I assume you have followed the setup guide in the readme, but can you please confirm you have done this:

Ensure that the TV can be woken via the network. For the CX line of displays this is accomplished by navigating to Settings (cog button on remote)->All Settings->Connection->Mobile Connection Management->TV On with Mobile, and then enable 'Turn On via Wi-Fi'. For C1 and C2 it's All Settings->General->Devices->External Devices->TV On With Mobile->Turn on via Wi-Fi

Then have a look at these troubleshooting steps:

When connecting the TV via Wi-Fi it seems some users must enable "Quickstart+" and disable "HDD Eco mode" to avoid the NIC becoming inactive. (physical network cable does not seem to need this)
Try reconfiguring the device and use one of the other wake-on-lan network options, primarily use option three, using a subnet mask.
Ensure the network is not dropping WOL-broadcasts.
The MAC-address configuration for the device in the application is erroneous.

If you can't get it to work by following the above please enable logging in the global options and upload the full log to pastebin and paste the link here and I will have a look.

ehh, sry
Ensure that the TV can be woken via the network. For the CX line of displays this is accomplished by navigating to Settings (cog button on remote)->All Settings->Connection->Mobile Connection Management->TV On with Mobile, and then enable 'Turn On via Wi-Fi'. For C1 and C2 it's All Settings->General->Devices->External Devices->TV On With Mobile->Turn on via Wi-Fi
FIXED my issue, all work great now !


Errors from unit-tests using conda-forge environment with pandas 2.0.0

Automated tests have started failing since the conda-forge environment started picking up Pandas version 2.0.0 (released 3rd April 2023). This major release has removed some deprecated functionality and requires fixing.
An example of the failures is here: https://github.com/MoseleyS/improver/actions/runs/4730944769/jobs/8395284604
Of the three failures, two are easy to fix (replace df.append() with df.concat()), the third is not as the new version appears to have identified an out-of-range index which it then is unable to df.drop().
Acceptance criteria:

All tests pass in all environments

I would suggest pinning pandas right now and creating a new issue on supporting pandas 2.0

improver PR: https://github.com/metoppv/improver/pull/1899


Disable peek state

How can I disable the peek state? I'm using MenuSheetView inside a NestedScrollView and a CoordinatorLayout and it only has 2 times but it always appears first in peek state, and I can't scroll it up to reveal the whole menu.

It sounds like you're using wrap content for your scrollview, which isn't going to expand past the peek state since the number of items isn't enough to justify anything else. Wrap your scrollview in a frame layout and set match_parent for its layout params

Actually the NestedScrollView has match_parent for its layout params, I'm using it inside a ViewPager with the ToolBar set to scroll|enterAlways so when it's scrolled up and hidden, the bottomSheet appears in expanded state, if I scroll the page down to reveal the toolBar, then the bottomSheet appears in the peek state and I'm not able to expand it by scrolling

Er... Ok that's hard to picture. Do you have a sample you can point me to?


Improve Custom Resources UX

fixes #1393
We now have:

Name CRDs in the sidebar is now Custom Resources

The list view associated with the sidebar entry has now a link to Resources and Definitions:

Clicking on a Resource in the view above, will show the list view for those resources (if the CR spec has additional columns, these are shown too)

Clicking on an object in the Resource list view will show that object's details view

This details view now has a Definition value, allowing to see the CRD association

In the CRD details view (which existed before), now we have a new Resource value, allowing to access the Resource list view

@joaquimrocha maybe update the comment with the new screenshot where link for definition is aligned on right

great, thanks @joaquimrocha !
Some quick feedback on the UX:

would be great to surface the current CR in the navigation as to allow easy browsing to other CRs of the same type

on the left colum: CRD -> Cronjob -> my-new-cronjob-object
on the top of screen (next to the back button)

the object detail view should display the fields in the additional columns as well as an option to display the full details (spec and status)

Hi @gberche-orange .
Adding the kind of CRs as entries in the sidebar may mean showing dozens of them and this would make the sidebar look awkward and make navigation difficult. Having them as a 3rd level, and have the group be the sub-entries of the root-level (Custom Resources > org.mygroup.name > MyKind) would scale better but there are other UX issues as I mentioned. Like what to do when you collapse the sidebar.

the object detail view should display the fields in the additional columns as well as an option to display the full details (spec and status)

We do show the additional columns as extra items in that table. This particular CronTab just doesn't have any.
I have to double check about the status.


There is an area that does not work on a 21:9 wide monitor.

There is an area that does not work on the wide monitor shown in the title.
If Crevice is restarted, it will work, but will not work again.

Hi@Teady999 ,
thank you for the report. Would you mind if  I ask more details?

actual display size (e.g. 1280x1080)
display scaling (e.g. 1.0, 1.25. 1.5)
the number of monitors and relative positions of each of them (if you have multiple monitors)
how long do you need to reproduce the issue?
where the area are you meaning for not working?


can't import ~/.mitmproxy/mitmproxy-ca-cert.pem to firefox

I'm trying to import cert in firefox 23.0
doing everything like here: http://mitmproxy.org/doc/certinstall/firefox.html
in section "Click "View Certificates", "Import", and select the certificate file:"
after selecting file mitmproxy-ca-cert.pem nothing happens
file choose dialog disappears and I see list of CA's again, mitmproxy not there
tried also to import into "Your certificates" there - but here I'm choosing .p12 file instead of .pem, dialog asks password and after empty password gives error:
"Failed to decode the file.  Either it is not in PKCS #12 format, has been corrupted, or the password you entered was incorrect."
all this on os x 10.8.4
also tried on windows - same problem
uploaded pem file here https://www.dropbox.com/s/v68aiymbe7voiqm/mitmproxy-ca-cert.pem
what am I doing wrong?

When you visit mitm.it, you will see four symbols for Android, Windows, iOS and * (Other). I can't import the Windows one to Firefox, the solution is using the fourth (* Other), just click it, tick all the checkbox and click OK

@dsultanr 2017 year - I have the same issue. How to solve it? mitmproxy installed via brew

@gerchicov-bp Did you try clicking the "Other" option?

@ujjwal96 the problem is need to install .p12. How if it requires password and you don't write it anywhere?

@gerchicov-bp I had the same problem today. And I solved it by choosing the .pem file from the "...Users<Username>.mitmproxy" folder in the firefox Authorities-Tab. After that I had a mitmproxy entry in the Authorities list and HTTPS connections worked.

@lzj78 First of all my goal is mobile safari, not firefox. But .p12 requires password even on mac.
I chose all .pem files from this directory (via browser on device) so I hadn't to install .p12 file and the last .pem file (but as I understand I don't need it). Result - safari requests approval for website's certificate in 50% cases and denies access to website at all in other 50% cases. So I didn't even try the requests in my own app.
And finally mitmproxy generates these files including .p12 but doesn't post a password or doesn't say how to get it (if it is pc-dependent). WTF?

Hey, I'm getting same behaviour described by the original post, no error message from Firefox though.
MacOS 10.13 Firefox 60.02
the proxy works on Chrome/Chrome on Android


systemd_unit not respecting sensitive property

Description
systemd_unit resource appears not to be respecting the senstive property as the documentation suggests it should.
It looks like the created file resource doesn't pass on the supplied sensitive to the underlying file resource, but just a guess:
https://github.com/chef/chef/blob/9326d7d9f55fa62e9fafd1e6d4e3874adaa288df/lib/chef/provider/systemd_unit.rb#L235-L243
Documentation says it should support this (under the 'common' resource properties):
https://docs.chef.io/resources/systemd_unit/
Chef Version
14.5.33 (but the current GitHub code looks like it suffers the issue, too)
Platform Version
$ uname -a
Linux localhost 3.10.0-1062.12.1.el7.x86_64 #1 SMP Tue Feb 4 23:02:59 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

Replication Case
Create a systemd_unit resource, see that it outputs the /etc/systemd/system/<name>.service diffs in the verbose log.
Client Output
see above
Stacktrace
n/a

Thanks for the report @jessepeterson. This fix will go out in Chef Infra Client 16.0


[Question] - Test framework

I see that MSTest is the test framework. This framework is really out of date in comparison to its competitors in many ways. What do you think about NUnit or XUnit? Asking because I have some plans to increase code coverage

I think xUnit is the way to go and that has support for .NET core too, (maybe nUnit does too?)
The only reason why MSTest was used is that I've had problems with the VS testrunner for xUnit from time to time.

As far as I know, NUnit has also support for .NET Core.  Still, guys from MS are using xUnit as a default test framework for .NET Core projects.
I would prefer XUnit because I like the the concept of test framework without teardown, setup etc.
If you agree, we can close this issue, I will handle PR

@lukasz-pyrzyk do it

XUnit support have been merged. closing


Add a retry when reading time from the modem

Sometimes the modem has not received the current time from the network before the firmware tries to read it, so the modem returns its default value of 01/01/2015.
There should be a retry with delay to increase the chances of getting a valid time from the modem.
We know any time before 2023 is invalid because the firmware wasn't in production until then.

Went back to using NTP so no need for this.


Skipped frames PlaceAutoCompleteFragment when typing

Hi everyone,
I use a PlaceAutoCompleteFragment to overlay the search bar over my map which works fine but when I type it takes a few seconds before the character shows up and the debug kernel shows that it's skipping 30-40 frames with almost every character I type. Also using a PlaceAutoCompleteActivity results in the same problem.
I tried placing the fragment in a separate activity to check if it was because of the map but the problem remained. Even stranger is that I have another app that utilizes the PlaceAutoCompleteFragment in the exact same way as this scenario but that one works fine and does not introduce any delay and skipped frames.
I tried finding the answer online but it seems like nobody has this problem. I'm using the newest v8:0.9.0 version and downgrading to v7 doesn't resolve it either. Does anyone know why this happens and how to fix it?
Kind regards,
Luuk

Apparently the problem was caused by debugging, when I open the app when it is not connected to Android Studio for debugging there are no frame skips and delays.


geoJsonFlatten is not a function error

In a react app we grab the bounds of a geojson using this lib:
import * as getExtent from '@mapbox/geojson-extent'
...
this.bounds = getExtent(this.json)
...
This works fine in 0.3.2 but breaks in 1.0.0 with an error: "geojsonFlatten is not a function". In the extent() function in geojson-coords.

I'm noticing this in v0.4.0 as well.

Any plans to fix this?

Any plans to fix this?

I looked into this a bit. It looks like the geojson-flatten lib which is a dependency of @mapbox/geojson-flatten is now exporting a Module, rather than a function.
If you change this line in @mapbox/geojsonFlatten/index.js from
var normalized = geojsonFlatten(geojsonNormalize(_))

to
var normalized = geojsonFlatten.default(geojsonNormalize(_))

things work as expected

I looked into this a bit. It looks like the geojson-flatten lib which is a dependency of @mapbox/geojson-flatten is now exporting a Module, rather than a function.
If you change this line in @mapbox/geojsonFlatten/index.js from
var normalized = geojsonFlatten(geojsonNormalize(_))

to
var normalized = geojsonFlatten.default(geojsonNormalize(_))

things work as expected

Should be fixed by https://github.com/mapbox/geojson-coords/pull/2 if someone can merge that...

Should be fixed by https://github.com/mapbox/geojson-coords/pull/2 if someone can merge that...

Can someone please merge this fix!?!?

Still occurs in @mapbox/geojson-extent 1.0.0
I replaced the package with my own utility function:
const getBbox = (coordinates) => {
  const [firstX, firstY] = coordinates[0];
  const initialBbox = { w: firstX, s: firstY, e: firstX, n: firstY };

  return coordinates.reduce(
    ({ w, s, e, n }, [x, y]) => ({
      w: Math.min(w, x),
      s: Math.min(s, y),
      e: Math.max(e, x),
      n: Math.max(n, y),
    }),
    initialBbox,
  );
};

Any update on when this might be fixed?


Allow Doctrine DBAL 3

Subject

I am targeting this branch, because {reason}.

Closes #{put_issue_number_here}.
Changelog

### Added
- Added some `Class::newMethod()` to do great stuff.

### Changed

### Deprecated

### Removed

### Fixed

### Security

Not sure with any of this changes, just tried to follow deprecation messages. This bundle blocks the doctrine orm admin bundle. @phansys or @VincentLanglet do you know why test fail?


MuiAutocomplete component- getOptionLabel and aria-label needs to be different, could not able to achieve it

In the MuiAutocomplete component,

On inspecting one of the selected Option, for example: "Male" then for accessibility it should display aria-label as "Remove Male" by keeping the Option label text as "Male".

Tried updating code in the
renderInput={params =>
as
'role': 'button',
'aria-label': 'remove' + params.inputProps.value,
but not working.
Looking for a solution.
Search keywords:

You can use the chip slotProps slotProps.chip:
<Autocomplete
   ...
   slotProps={{ chip: { 'aria-label': `Remove ${option}` } }}
/>;

See the API documentation for more details.

the provided solution didn't work. May be because of the version issue.
The below code worked perfectly.
renderTags={(values, props) => {
return (
<>
{values.map((value, i) => (
<Chip className=" MuiAutocomplete-tag MuiChip-sizeSmall"
label={value.label}
aria-label={Remove ${value.label}}
onDelete={e => props({ index: i }).onDelete(e)}
/>
))}
</>
);
}}


Fix crash when closing multiple splits from the tab context menu

Summary of the Pull Request
Fix for #9021
Turns out that what was happening is that the parent pane's Closed event was being caught by the tab, and parent panes always have nullopt as their id. So now the Pane::Id() call always returns an optional, allowing us to check if it has a value before we access it.
PR Checklist

[x] Closes #9021

Validation Steps Performed
No more crash

You'll need to merge main to get the spellbot happy.

(the crash is still fixed with my recommended change, right? I'd hate to just make your whole PR a no-op lol)

(the crash is still fixed with my recommended change, right?

Wait shouldn't we be checking if pane->Id().has_value() first in both of these cases? I know that's weird and unexpected for the _activePane to not have an Id, but I'd rather be defensive

So I felt like it was guaranteed that the active pane and the pane being split are leaves, but yeah better be safe - added those checks! Also once the pane refactor comes in we won't have these kind of issues anymore (no comment on the 99 other issues we get though)


⚠ Site ANTS has degraded performance

In 3d0450f, Site ANTS (https://ants.gouv.fr) experienced degraded performance:

HTTP code: 200
Response time: 26168 ms

Resolved: Site ANTS performance has improved in 9305c0f after 6 minutes.


icon generation breaks if directory contains a space

Paths supplied to convert are not escaped or quoted, which causes issues if the path contains a space
Failed to call 'convert res/icon/universal/logo.png -resize 1024x1024 platforms/ios/Sample Project/Images.xcassets/AppIcon.appiconset/ios-marketing-1024x1024-1x.png. Error is 'Command failed: convert res/icon/universal/logo.png -resize 1024x1024 platforms/ios/Sample Project/Images.xcassets/AppIcon.appiconset/ios-marketing-1024x1024-1x.png
convert: unable to open image 'platforms/ios/Sample': No such file or directory @ error/blob.c/OpenBlob/3109.
convert: no decode delegate for this image format `' @ error/constitute.c/ReadImage/509.
convert: unable to open image 'Project/Images.xcassets/AppIcon.appiconset/ios-marketing-1024x1024-1x.png': No such file or directory @ error/blob.c/OpenBlob/3109.
convert: WriteBlob Failed `Project/Images.xcassets/AppIcon.appiconset/ios-marketing-1024x1024-1x.png' @ error/png.c/MagickPNGErrorHandler/1711.

See pull request #45

Fixed in v0.6.3 - thanks for the help @sgdesmet!


Refactor how-to docs

What this PR does / why we need it:
Fixes https://issues.redhat.com/browse/MULTIARCH-2569
Which issue(s) this PR fixes (optional, use fixes #<issue_number>(, fixes #<issue_number>, ...) format, where issue_number might be a GitHub issue, or a Jira story:
Fixes #
Checklist

[x] Subject and description added to both, commit and PR.
[ ] Relevant issues have been referenced.
[ ] This change includes docs.
[ ] This change includes unit tests.

/cc @mkumatag

/cc @alishaIBM

/retest-required

/approve

/lgtm


Installation of MA integration fails if add-on isn't already started

What version of Music Assistant has the issue?
2.0.4
What version of the Home Assistant Integration have you got installed?
2024.5.1
Have you tried everything in the Troubleshooting FAQ and reviewed the Open and Closed Issues and Discussions to resolve this yourself?

[X] Yes

The problem
The MA integration fails to install unless the add-on is already started
How to reproduce

uninstall MA - both the add-on and integration . I did this yesterday on 6/10
restart HA (I'm using HAOS) .
install MA from settings / add-ons / add-on store / MA
wait for it to be fully installed, but don't start it
install the MA integration from settings / devices and services / add integration
The "select connection method" dialog appears, with the "Use the official Music assistant server add-on" pre-checked

in the Conversation agent drop-down, there are two options "None" and "Home Assistant". Select Home Assistant.
Hit submit
a "Starting add-on" dialog shows. Wait as instructed
After a while, it fails with "Failed to start the Music Assistant Server add-on".
. settings / add-ons / MA / log shows many entries from the prior installation dated 6/10 . It also shows new entries from today. Log is attached
ma.txt
settings / add-ons / MA shows that the add-on actually started successfully
in HA, go to settings, devices and services / add integration , and add the MA integration once again (same as step 5)
A different dialog appears, "Discovered Music Assistant Server"
select Home Assistant as the conversation agent
Click submit
It succeeds
repeat steps 1 through 4
start the add-on manually
reinstall the integration, as in step 5
the dialog from step 14 appears, rather than the one from step 6
repeat steps 16, 17 and 18

Music Providers
N/A, install issue
Player Providers
N/A, install issue
Full log output
[
ma.txt
](url)
Additional information
Obviously, there is an easy workaround - manually start the add-on - if you are aware of it.
However, there was clearly an attempt to start the add-on automatically when installing the integration, but the integration fails to detect that the add-on started.
What version of Home Assistant Core are your running
2024.6.2
What type of installation are you running?
Home Assistant OS
On what type of hardware are you running?
Windows

I will leave this open for now but it will be overtaken by events as control of the addon by the integration is going to be removed

Makes complete sense - wish I had known about that upcoming change before wasting my time filing this bug.

This one can be resolved now because we decided to remove the addon manager from the integration.


Title & message in AlertController.init(view: ...)

Hey @KyoheiG3,
how can I have the Title + message view still when I use the AlertController.init(view: ...) initializer.
It's seems that you replace the 'basic view' with the view passed as parameter, is there any way to keep both views?
Thanks

Hi @akabab
It cannot keep title with your view. But, I think that's good idea.
Thank you.

@KyoheiG3 yeah actually I'd like that to be possible, any idea how to realise it easily ? thanks

It is the easiest that add Title on your view. Or I'm going to try to add View between Contents and Buttons if I had a lot of time to do.

You can also use configContentView closure. It can add View on the Alert.

@akabab It's supported in #32 on greater than iOS8.

@KyoheiG3 Thanks for the support, is it in a release on cocoapod?

@akabab Yes, you can use it on cocoapods.


Booming Blade / Green Flame Blade not working

Describe the bug
Both Booming Blade and Green Flame Bade aren't working.
I don't know much about the macro stuff in spells, but it looks like it is getting a copies the weapon and applies an effect and then you roll that. My guess is that the 3.0 5e system actually checks now that a weapon is in the character's inventory before you can attack with it.
Also looking at the callstack, the error is being generated in item.mjs in _onChatCardAction in this block which seems to support this:
      // Get the Item from stored flag data or by the item ID on the Actor
      const storedData = message.getFlag("dnd5e", "itemData");
      const item = storedData ? new this(storedData, {parent: actor}) : actor.items.get(card.dataset.itemId);
      if ( !item ) {
        ui.notifications.error(game.i18n.format("DND5E.ActionWarningNoItem", {
          item: card.dataset.itemId, name: actor.name
        }));
        return null;
      }

To Reproduce
Steps to reproduce the behavior:

Have a caster with either/or Green Flame Blade/Booming Blade
Cast either spell, pick one of your equipped weapons.
See that it ended up in the chat log
Click the Attack button
See a red error banner saying "The requested item no longer exists on Actor "

Expected behavior
A melee attack roll to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Environment:

Browser Chrome
Foundry Version 11.315
5e System Version 3.0.4
Module Version 4.0.16
Does this behaviour occur with other modules disabled: Yes

Additional context
Add any other context about the problem here.
An effected character you can import - https://www.dndbeyond.com/characters/112215903
Console Error - I cleared the console then triggered the error again
requesteItemNoLongerExists.log

It's working here. What versions of midi and DAE do you have installed?
Have you tried with just DDB Importer, Midi (and it's required modules)? No others?

With just DDB-I + Midi (and its required modules) => I just got a template placer when casting GFB (no asking which weapon)
When I enabled enough to be allowed to check the DDB-I box for "Generate Automated Effects for spells" => I get the same error as my initial post.
Versions of things:
MidiQOL - 11.4.10 (I see there is a 11.4.11 from yesterday with a big "Breaking" on it, will try but tested this as-is)
DFred's - 6.0.3
DAE - 11.3.14

I was playing around with MidiQOL settings, and found if I enabled "Auto Roll Attack" the macro works for Booming Blade. If that is how you are testing that could be the difference?

Yep, that's it, I think I have fixed this in the next version, v5.0.1. Please can you check and let me know.

Confirmed both Booming/Green Flame blade are working after reimporting the character.


lwjgl64 error

When trying to run/debug the client, this error occurs:
Exception in thread "Client thread" java.lang.UnsatisfiedLinkError: no lwjgl64 in java.library.path 	at java.lang.ClassLoader.loadLibrary(Unknown Source) 	at java.lang.Runtime.loadLibrary0(Unknown Source) 	at java.lang.System.loadLibrary(Unknown Source) 	at org.lwjgl.Sys$1.run(Sys.java:72) 	at java.security.AccessController.doPrivileged(Native Method) 	at org.lwjgl.Sys.doLoadLibrary(Sys.java:66) 	at org.lwjgl.Sys.loadLibrary(Sys.java:87) 	at org.lwjgl.Sys.<clinit>(Sys.java:117) 	at net.minecraft.client.Minecraft.getSystemTime(Minecraft.java:2602) 	at net.minecraft.util.Timer.<init>(Timer.java:59) 	at net.minecraft.client.Minecraft.<init>(Minecraft.java:231) 	at net.minecraft.client.main.Main.main(Main.java:108) 	at Start.main(Start.java:7)
I saw the wiki and checked the fix for this, but the working directory is already in "MCPLOC/jars", so I don't know what to do. If that can help, my IDE is Eclipse.

Be sure to add your 1.8.8-natives to the IntelliJ project structure as a
dependency.  The error is saying that it cannot locate Minecraft natives,
specifically lwjgl.
On Mon, Jan 10, 2022 at 6:03 AM waw @.***> wrote:

When trying to run/debug the client, this error occurs:
Exception in thread "Client thread" java.lang.UnsatisfiedLinkError: no
lwjgl64 in java.library.path at java.lang.ClassLoader.loadLibrary(Unknown
Source) at java.lang.Runtime.loadLibrary0(Unknown Source) at
java.lang.System.loadLibrary(Unknown Source) at
org.lwjgl.Sys$1.run(Sys.java:72) at
java.security.AccessController.doPrivileged(Native Method) at
org.lwjgl.Sys.doLoadLibrary(Sys.java:66) at
org.lwjgl.Sys.loadLibrary(Sys.java:87) at
org.lwjgl.Sys.(Sys.java:117) at
net.minecraft.client.Minecraft.getSystemTime(Minecraft.java:2602) at
net.minecraft.util.Timer.(Timer.java:59) at
net.minecraft.client.Minecraft.(Minecraft.java:231) at
net.minecraft.client.main.Main.main(Main.java:108) at
Start.main(Start.java:7)
I saw the wiki and checked the fix for this, but the working directory is
already in "MCPLOC/jars", so I don't know what to do. If that can help, my
IDE is Eclipse.
—
Reply to this email directly, view it on GitHub
https://github.com/OxideWaveLength/Minecraft-Hack-BaseClient/issues/98,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEPATXZUP5LSJSEYF3G5YGDUVLKLHANCNFSM5LTTG67Q
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Sorry, can you maybe tell me how to do that on Eclipse?

Oh, I forgot.

I fixed the problem.
 You can find this on GitHub easily. For my case, the batch file did not installed everything correctly.


Fix jerkiness in message list

Now the composeMenu doesn't overshoot before coming back to the intended width, and thus prevents text or placeholders from resizing due to this.
https://chat.zulip.org/user_uploads/2/e0/LAA4D-H2hTQ500VnsDgO4Vz5/easing.mp4
Fixes #2572

Good find, thanks @agzuniverse ! I appreciate how simple this code fix is. :-) Merged.


Can't install the newest version (v18.4.2) in Chromebook

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version: 18.4.2
OS Version: Chromebook Debian 11.8

Steps to Reproduce:

The old version 18.4.1 can be installed, but when I upgrade to 18.4.2 by code_1.84.2-1699528352_amd64.deb, it showed the following errors:
The following packages have unmet dependencies:
code: Depends: libasound2 (>= 1.0.17) but it is not installable
Depends: libatk-bridge2.0-0 (>= 2.5.3) but it is not installable
Depends: libatk1.0-0 (>= 2.2.0) but it is not installable
Depends: libatspi2.0-0 (>= 2.9.90) but it is not installable
Depends: libc6 (>= 2.14) but it is not installable
Depends: libc6 (>= 2.16) but it is not installable
Depends: libc6 (>= 2.17) but it is not installable
Depends: libc6 (>= 2.2.5) but it is not installable
Depends: libcairo2 (>= 1.6.0) but it is not installable
Depends: libcurl3-gnutls but it is not installable or
libcurl3-nss but it is not installable or
libcurl4 but it is not installable or
libcurl3 but it is not installable
Depends: libdbus-1-3 (>= 1.5.12) but it is not installable
Depends: libdrm2 (>= 2.4.75) but it is not installable
Depends: libexpat1 (>= 2.0.1) but it is not installable
Depends: libgbm1 (>= 17.1.0~rc2) but it is not installable
Depends: libglib2.0-0 (>= 2.37.3) but it is not installable
Depends: libgssapi-krb5-2 but it is not installable
Depends: libgtk-3-0 (>= 3.9.10) but it is not installable
Depends: libgtk-3-0 (>= 3.9.10) but it is not installable or
libgtk-4-1 but it is not installable
Depends: libkrb5-3 but it is not installable
Depends: libnspr4 (>= 2:4.9-2~) but it is not installable
Depends: libnss3 (>= 2:3.30) but it is not installable
Depends: libnss3 (>= 3.26) but it is not installable
Depends: libpango-1.0-0 (>= 1.14.0) but it is not installable
Depends: libx11-6 but it is not installable
Depends: libx11-6 (>= 2:<IP_ADDRESS>) but it is not installable
Depends: libxcb1 (>= 1.9.2) but it is not installable
Depends: libxcomposite1 (>= 1:0.4.4-1) but it is not installable
Depends: libxdamage1 (>= 1:1.1) but it is not installable
Depends: libxext6 but it is not installable
Depends: libxfixes3 but it is not installable
Depends: libxkbcommon0 (>= 0.4.1) but it is not installable
Depends: libxkbfile1 but it is not installable
Depends: libxrandr2 but it is not installable
Recommends: libvulkan1 but it is not installable
Do I need to upgrade something else?
Thanks.

Chromebooks are not a supported OS.


[JENKINS-46096] Avoid display at bottom of page

Fixes JENKINS-46096

@svanoort Please review this oneliner, which reverts https://github.com/jenkinsci/pipeline-stage-view-plugin/pull/35

@ssbarnea Thanks for the contribution, but I think a simple revert won't do it -- the idea behind #35 was solid if there was enough screen real-estate -- I think it needs a more comprehensive fix so the layout reflows correctly.
For review of the CSS change, we should also have a before-and-after screenshots to show the bug and the result of the new layout.   Thanks!

Hi guys, just wondering if there was any work being done on this? Having the stage view overflowing on the warnings/tests report is a real pain... I think I'd personally prefer to have the stage view lower in the screen than having a broken UI. Yes it's not pretty to the eyes but at least all widgets/components would work compared to what we have right now.
What do you guys think?

This is now hopefully obsolete, thanks to #72


one-to-many-through: use correct fkey

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


Added location to drop down

Added location translation

I think this pr is on the wrong branch

It's fine. I'll merge locally.

ok


Datasets & Dataset Collections need time filter option

For some datasets (and dataset collections), all or most records are temporally scoped (have meaningful start or start/end), so the ds_status and ds_collection_build pages now have a switch option to display the time filter slider on their pub pages. A simple boolean: show time filter, yes or no. The entire vis_parameters control would be confusing in those cases. I imagine flipping the switch writes a vis_parameters value to that objects record (initially NULL) with an ajax call, and flipping it back NULLs it again. The vis_parameters object would just need the settings that enable a slider. Sequence is not a factor for these. Did not draft a function b/c it gets into vis_paramets, b/c that is @docuracy territory.

Partial Overlap/Duplication: https://github.com/WorldHistoricalGazetteer/whg3/issues/159#issue-2224657561
SG: Digging into the configuration of temporal controls, I've just realised that tableFunctions.js is configured such that only place collections will ever have columns for start or end properties (or indeed seq). Is that how it should be? Do we not need to show such properties for datasets and dataset collections?
KG: About time controls…this hasn’t been thought out fully ... all Place records have at least a start value, but in the great majority of datasets, all of its places have the same year as a publication date of a source — not meaningful in the same way as ‘existed at’ or ‘attested at’ dates, and certainly not like the start/end in the annotations of a place collection. this leads me to think that a configuration option of a dataset should be something like a ‘my dates matter’ toggle - which would lead to a time slider appearing. For a handful of datasets, the time filter widget would be very good if not essential. I don’t think a player option makes sense.
maybe this needs a few more example datasets to work with, which I can do next week.

[ ] Temporal control options on builder pages (sequencer, player-start-date, player-end-date)

[ ] Datasets (no options at present: no change proposed, display should use default of slider for both start- and end-dates)
[ ] Dataset Collections (no options at present: no change proposed, display should use default of slider for both start- and end-dates)
[ ] Place Collections (all 3 options at present: no change proposed)

[ ] Temporal control configuration/test

[ ] http://localhost:8001/datasets/10/places (both showing)
[ ] http://localhost:8001/collections/14/browse_ds  (both showing)
[ ] http://localhost:8001/collections/13/browse_pl (date slider only)
[ ] http://localhost:8001/collections/11/browse_pl (sequencer only)

[ ] Upgrade ds_collection_build and ds_metadata.html to use base_webpack
[ ] Drop VisualisationControl into both
[ ] Configure VisualisationControl in both

I very much do not want the full vis control in these two pages. Time? Y/N is all they need.

I see I put the Display temporal filter" switch in both ds_status and ds_metadata. I intended to remove it from ds_metadata, and have that choice be in ds_status alone, with that space in ds_metadata used for the file information instead.
I do not believe the entire VisualisationControl is necessary, for either ds_collection_build or ds_status, only a switch. If the data has meaningful dates, the switch writes a vis_parameters value that cause a time filter and start/end columns to appear.

KG: "...if someone chooses to display a time slider for a dataset or dataset collection, because their dates are meaningful (i.e. related to the place's existence, not the pub date of the source), then start and end columns should be available. So the toggle switch label should reflect that, and 'show slider' should in fact enable start/end columns, not only a slider".

My take is that there is one configuration of a vis_parameters object that applies 'show slider' and 'show start/end'. Therefore it is a binary. If there is meaningful time, then add a a vis_parameters object to the record. If there isn't, don't. So the use choice is y/n and the interface needs to reflect that.
I am assuming that when a dataset or dataset collection pub page loads, it looks for the value in vis_parameters, if there is one. In that case, there is a single configuration of that field that applies, and the user should not be presented with the full complexity.
It should not be a choice to say time slider yes but start/end no. Too many choices.


Use fixed version nightly-2023-06-27 for cargo udeps

This should avoid random breakage with newer nightly versions

The previously failing build of cargo-udeps was not actually executed in https://github.com/rp-rs/rp-hal/actions/runs/5493763368/jobs/10012154038?pr=645 because it used a cached version.
See https://github.com/jannic/rp-hal/actions/runs/5493762713/jobs/10012153213 for a successful run, and https://github.com/jannic/rp-hal/actions/runs/5493635352/jobs/10011948303 for a previously failing one.

Wait - I'm not sure this is enough. Once a new version of cargo-udeps is released, it will probably rely on newer nightly features. So we need to specify a certain version of cargo-udeps, as well.


minDate and maxDate mixed up

When we use two datetimepickers on same page, minDate and maxDate are mixed up.
You can test it on your demo, in the DateRange Example : https://logbon72.github.io/angular-material-datetimepicker/

Select a start date.
Reopen the start date : all is ok
Open the end date : we can't select date before start date, all is ok
Select the end date.
Reopen the start date : we can't select a date before than start date we previously selected
Reopen the end date : we can't select a date after than end date we previously selected

As a workaround you have to set undefined attribute min-date or max-date.

@fabienbranchel can you test in my fork?

@hexadecy thank you, seems to work ;-)


Rewrite PythonInterpreter.

What is this PR for?
I've been testing the python interpreter and I found at least 4 major issues in the current python interpreter.

not working streaming output.

https://issues.apache.org/jira/browse/ZEPPELIN-2225

printed "..." when there is indent in the python code.

https://issues.apache.org/jira/browse/ZEPPELIN-1929

very slow output of matplotlib

https://issues.apache.org/jira/browse/ZEPPELIN-1894
https://issues.apache.org/jira/browse/ZEPPELIN-1360

Unexpected output of matplotlib.

https://issues.apache.org/jira/browse/ZEPPELIN-2107

so I changed python interpreter to use py4j  based on pyspark interpreter and would be fixed above issues.
and I am going to recreate conda, docker for python interpreter ASAP.
What type of PR is it?
Bug Fix | Hot Fix | Refactoring
How should this be tested?

not working streaming output.

import time
for x in range(0, 5):
    print x
    time.sleep(1)

printed "..." when there is indent in the python code.

def fn():
    print("hi")

fn()

very slow output of matplotlib.

import matplotlib
import sys
import matplotlib.pyplot as plt
plt.plot([1,2,3])

Unexpected output of matplotlib.

import matplotlib.pyplot as plt
import matplotlib as mpl

# Make a figure and axes with dimensions as desired.
fig = plt.figure(figsize=(8, 3))
ax1 = fig.add_axes([0.05, 0.80, 0.9, 0.15])
ax2 = fig.add_axes([0.05, 0.475, 0.9, 0.15])
ax3 = fig.add_axes([0.05, 0.15, 0.9, 0.15])

# Set the colormap and norm to correspond to the data for which
# the colorbar will be used.
cmap = mpl.cm.cool
norm = mpl.colors.Normalize(vmin=5, vmax=10)

# ColorbarBase derives from ScalarMappable and puts a colorbar
# in a specified axes, so it has everything needed for a
# standalone colorbar.  There are many more kwargs, but the
# following gives a basic continuous colorbar with ticks
# and labels.
cb1 = mpl.colorbar.ColorbarBase(ax1, cmap=cmap,
                                norm=norm,
                                orientation='horizontal')
cb1.set_label('Some Units')

# The second example illustrates the use of a ListedColormap, a
# BoundaryNorm, and extended ends to show the "over" and "under"
# value colors.
cmap = mpl.colors.ListedColormap(['r', 'g', 'b', 'c'])
cmap.set_over('0.25')
cmap.set_under('0.75')

# If a ListedColormap is used, the length of the bounds array must be
# one greater than the length of the color list.  The bounds must be
# monotonically increasing.
bounds = [1, 2, 4, 7, 8]
norm = mpl.colors.BoundaryNorm(bounds, cmap.N)
cb2 = mpl.colorbar.ColorbarBase(ax2, cmap=cmap,
                                norm=norm,
                                # to use 'extend', you must
                                # specify two extra boundaries:
                                boundaries=[0] + bounds + [13],
                                extend='both',
                                ticks=bounds,  # optional
                                spacing='proportional',
                                orientation='horizontal')
cb2.set_label('Discrete intervals, some other units')

# The third example illustrates the use of custom length colorbar
# extensions, used on a colorbar with discrete intervals.
cmap = mpl.colors.ListedColormap([[0., .4, 1.], [0., .8, 1.],
                                  [1., .8, 0.], [1., .4, 0.]])
cmap.set_over((1., 0., 0.))
cmap.set_under((0., 0., 1.))

bounds = [-1., -.5, 0., .5, 1.]
norm = mpl.colors.BoundaryNorm(bounds, cmap.N)
cb3 = mpl.colorbar.ColorbarBase(ax3, cmap=cmap,
                                norm=norm,
                                boundaries=[-10] + bounds + [10],
                                extend='both',
                                # Make the length of each extension
                                # the same as the length of the
                                # interior colors:
                                extendfrac='auto',
                                ticks=bounds,
                                spacing='uniform',
                                orientation='horizontal')
cb3.set_label('Custom extension lengths, some other units')

plt.show()

Questions:

Does the licenses files need update? no
Is there breaking changes for older versions? no
Does this needs documentation? no

@astroshim Have you checked #1495 which also rewrite the python interpreter ? And strongly +1 to rewrite python interpreter. And it would be better to consider extensibility for python interpreter. So that other places can reuse it such as pyspark interpreter.

@zjffdu Thank you for your opinion!
Of course i checked #1495 and I totally agree of extensibility for python interpreter.
but As you can see, current python interpreter has lots of issues and i think these issues are really serious so I wanted it to fix them on current python interpreter but I couldn't.
It would be better if #1495 is done ASAP but I thought we also need to think about zeppelin users while implementing #1495.
Actually i think it would be better to rewrite python interpreter as python codes for simplicity.

I just added conda interpreter.

What a nice fix! Let me test it out and give you a feedback!

@Leemoonsoo thanks for reviewing.. Let me fix the issues you commented and add PandasSql.

@Leemoonsoo fixed the issues you commented and added PandasSql.
and also added conda, pandasql interpreter test on PR description.

Tested %python.conda and %python.pandasql and it works well.
Could you restore PythonDockerInterpreter.java and PythonDockerInterpreterTest.java for %python.docker as well?
To make it work, i think we need to let docker container mount directory that py4j exists using -v option, and then set PYTHONPATH env variable to load py4j library using -e option.
So this line
setPythonCommand("docker run -i --rm " + image + " python -iu");

can be updated to something like
String py4jDir = ZEPPELIN_HOME + "/interpreter/python";
String py4jFile = "py4j-{version}.zip";   // read directory py4jDir and get file starts with py4j
setPythonCommand("docker run -i --rm -v " + py4jDir + ":/zeppelin_python -e /zeppelin_python/" + py4jFile + " " + image + " python -iu");

I tested with the first two scripts and worked well.

@astroshim made a pullrequest to this branch, to bringing docker support back . Please take a look and merge if you think it's okay.


Compatibility with ARM alignment requirements

Fixes #36

Fixes #36

Hey, thanks a lot for this library!
I'd argue that it's also a problem in x86 / x64 / many architectures and should not be resolved only for ARM.
The C11 standard states (section 6.2.8):

6.2.8

Complete object types have alignment requirements which place restrictions on the addresses at which objects of that type may be allocated. An alignment is an implementation-defined integer value representing the number of bytes between successive addresses at which a given object can be allocated. An object type imposes an alignment requirement on every object of that type: stricter alignment can be requested using the _Alignas keyword.

[...]
<IP_ADDRESS>
[...]

A pointer to an object or incomplete type may be converted to a pointer to a different object or incomplete type. If the resulting pointer is not correctly aligned for the pointed-to type, the behavior is undefined. [...]

The alignment of uint16_t and uint32_t in x86_64 can be verified in godbolt for both clang and gcc: https://godbolt.org/z/se7cEM
Which means that unaligned load / store in x64 will cause undefined behavior and may cause problems (for some it happened)

Hey, thanks a lot for this library!
I'd argue that it's also a problem in x86 / x64 / many architectures and should not be resolved only for ARM.
The C11 standard states (section 6.2.8):

6.2.8

Complete object types have alignment requirements which place restrictions on the addresses at which objects of that type may be allocated. An alignment is an implementation-defined integer value representing the number of bytes between successive addresses at which a given object can be allocated. An object type imposes an alignment requirement on every object of that type: stricter alignment can be requested using the _Alignas keyword.

[...]
<IP_ADDRESS>
[...]

A pointer to an object or incomplete type may be converted to a pointer to a different object or incomplete type. If the resulting pointer is not correctly aligned for the pointed-to type, the behavior is undefined. [...]

The alignment of uint16_t and uint32_t in x86_64 can be verified in godbolt for both clang and gcc: https://godbolt.org/z/se7cEM
Which means that unaligned load / store in x64 will cause undefined behavior and may cause problems (for some it happened)

Yeah, I doubt that SSE will be used in this case though, but maybe it's better to be safe than sorry :)

Yeah, I doubt that SSE will be used in this case though, but maybe it's better to be safe than sorry :)


clarification - I enter the secret and url

I successfully build project - run it on imac / x86
get the window

enter my email and bogus password - click register
nothing....
logs don't show anything
does this need service level secret - or anonymous key is fine?

23-03-21 13:25:59.905728+1100 Examples[29716:7817175] [API] cannot add handler to 3 from 3 - dropping
2023-03-21 13:26:03.767122+1100 Examples[29716:7817175] Warning: The selected font is a system font, but the macOS font panel will not show system fonts to the user. When configuring a text view to display rich text, you should only use fonts that are available for selection (see NSFontManager.availableFonts). A fallback font (Helvetica) will be shown to the user instead.
2023-03-21 13:26:06.212996+1100 Examples[29716:7817175] +[CATransaction synchronize] called within transaction
2023-03-21 13:26:10.488359+1100 Examples[29716:7817175] [API] cannot add handler to 3 from 3 - dropping
2023-03-21 13:26:20.799889+1100 Examples[29716:7817175] [API] cannot add handler to 3 from 3 - dropping
2023-03-21 13:26:25.257187+1100 Examples[29716:7817175] [API] cannot add handler to 3 from 3 - dropping
2023-03-21 13:26:29.102149+1100 Examples[29716:7817175] [API] cannot add handler to 3 from 3 - dropping
2023-03-21 13:29:25.971556+1100 Examples[29716:7817506] [connection] nw_read_request_report [C1] Receive failed with error "Socket is not connected"
2023-03-21 13:29:25.972588+1100 Examples[29716:7817506] [connection] nw_read_request_report [C1] Receive failed with error "Socket is not connected"
2023-03-21 13:29:25.972772+1100 Examples[29716:7817506] [connection] nw_read_request_report [C1] Receive failed with error "Socket is not connected"
2023-03-21 13:32:19.933275+1100 Examples[29716:7817175] [API] cannot add handler to 3 from 3 - dropping
2023-03-21 14:14:29.462854+1100 Examples[29716:7817175] [API] cannot add handler to 3 from 3 - dropping
2023-03-21 14:14:41.022752+1100 Examples[29716:7817175] [API] cannot add handler to 3 from 3 - dropping
2023-03-21 14:17:31.114376+1100 Examples[29716:7866026] [connection] nw_read_request_report [C3] Receive failed with error "Socket is not connected"
2023-03-21 14:17:31.114712+1100 Examples[29716:7866026] [connection] nw_read_request_report [C3] Receive failed with error "Socket is not connected"
2023-03-21 14:17:31.115106+1100 Examples[29716:7866026] [connection] nw_read_request_report [C3] Receive failed with error "Socket is not connected"
2023-03-21 14:17:44.189923+1100 Examples[29716:7817175] [API] cannot add handler to 3 from 3 - dropping
2023-03-21 14:30:23.232544+1100 Examples[29716:7817175] [API] cannot add handler to 3 from 3 - dropping
2023-03-21 14:30:49.026283+1100 Examples[29716:7817175] [API] cannot add handler to 3 from 3 - dropping

same problem on device - I hit the signup many times - it then shows error - For Security purposes - ....

it's not a misunderstanding of using values.
Using my org it just doesn't work.
let SUPABASE_URL = URL(string: "https://pqxcxltwoifmxcmhghzf.supabase.co/auth/v1")!
let SUPABASE_API_KEY = "eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJzdXBhYmFzZSIsInJlZiI6InBxeGN4bHR3b2lmbXhjbWhnaHpmIiwicm9sZSI6ImFub24iLCJpYXQiOjE2NjAxODczNDQsImV4cCI6MTk3NTc2MzM0NH0.NiufAQmZ3Oy7eP7wNWF-tvH-e2D-UIz-vPLpLAyDMow"


Could not process ':PlugClean' when I set PowerShell as Vim's shell

I'm a programming beginner, after I completed my .vimrc file, I found the command ':PlugClean' cannot be used correctly.
The output is
Error detected while processing function <SNR>9_clean[13]..<SNR>9_git_validate[36]..<SNR>9_system[30]..function <SNR>9_clean[13]..<SNR>9_git_validate[36]..<SNR>9_system:
line   24:
E484: Can't open file C:/Users/ZHANGC~1/AppData/Local/Temp/VBVFDC.tmp
Error detected while processing function <SNR>9_clean[13]..<SNR>9_git_validate:
line   36:
E171: Missing :endif
Error detected while processing function <SNR>9_clean:
line   13:
E171: Missing :endif

After I pressed ENTER, vim-plug showed it was searching, but this task can never be completed.
Using the elimination method, I found that the reason was I added the following command to my .vimrc file to set PowerShell as Vim's shell.
set shell=pwsh

The version of PowerShell is 7.1.1, but the same with Windows Powershell pre-installed on my computer.

:version
VIM - Vi IMproved 8.2 (2019 Dec 12, compiled Jan 19 2021 23:03:47)
MS-Windows 64-bit console version
Included patches: 1-2377
Compiled by appveyor@APPVYR-WIN
Huge version without GUI.   Read the documentation for shell option via :h 'shell'.

set shell=pwsh is insufficient to setup Vim to use powershell. Read the documentation for shell option via :h 'shell'.

This command works, even though I don't enter the full path of PowerShell. My main purpose is use ':terminal' to open integrated terminal with PowerShell, and I can achieve this by the command. I know another way to achieve is set a keyboard shortcut to ':terminal pwsh', but may be It showed vim-plug can not run properly when Vim's shell is PowerShell?

This command works, even though I don't enter the full path of PowerShell. My main purpose is use ':terminal' to open integrated terminal with PowerShell, and I can achieve this by the command. I know another way to achieve is set a keyboard shortcut to ':terminal pwsh', but may be It showed vim-plug can not run properly when Vim's shell is PowerShell?

Setting shell option is insufficient to use powershell. For reference, inspect SetShell() function in https://github.com/janlazo/dotvim8/blob/master/shared.vim.

Setting shell option is insufficient to use powershell. For reference, inspect SetShell() function in https://github.com/janlazo/dotvim8/blob/master/shared.vim.

I can't understand because my level is not enough...now I think may be I should use a keyboard shortcut instead of change Vim's shell...😢

I can't understand because my level is not enough...now I think may be I should use a keyboard shortcut instead of change Vim's shell...😢

And what makes me feel strange is if I use :terminal pwsh, The function of Ctrl+W Ctrl+C is change terminal to normal mode instead of close it. But this is not under the topic.

And what makes me feel strange is if I use :terminal pwsh, The function of Ctrl+W Ctrl+C is change terminal to normal mode instead of close it. But this is not under the topic.

I have the same problem. Plugclean can't work. Is there a solution

I have the same problem. Plugclean can't work. Is there a solution

My solution is set back to default shell, and use powershell with a keyboard shortcut binding.

thank

I think the problem is that vim-plug does not recognize pwsh  as a powershell variant.
https://github.com/junegunn/vim-plug/blob/cffcfe150bda75177920530c5f1911b52f70a065/plug.vim#L506-L508


Add a local graph view to the Starting Material edit page

The local graph view is currently shown in Samples, Cells, and Collections, but not Starting Materials

This is a good idea (maybe one place that this graph view is actually very useful) but we might need to revise our default settings for the item graph, since adding "lithium" will blow up quite quickly once it gets further out than one node. See also the bug I ran into when creating a manager account which seems to display all nodes in the local item graph (#629)


fix CallAsync bug

fix CallAsync bug

Thanks for the fix.
Now that I am going to be using the library in one of my projects maybe ill get a full bug sweep, such as hackathons


raw strings in rust are context-sensitive

The grammar docs note:

Note: Raw strings start with an r, followed by n # and a quotation mark ". They may contain any characters or escapes (except the end sequence). A raw string ends with a quotation mark ("), followed by n #.
I don't know any sane way to write this out in EBNF, if you do, let me know.

It can't be written in EBNF because the syntax of EBNF is for expressing context-free grammars, while raw strings are context-sensitive:
https://github.com/rust-lang/rust/blob/master/src/grammar/raw-string-literal-ambiguity.md
(As a general rule, context-free languages can only compare two numbers destructively, so comparing 3 numbers doesn't work)

Thanks @Gankro, fixed this in #171.


HasOneThrough relationship is... wrong?

Laravel Version: 9.0.0
PHP Version: n/a
Database Driver & Version: n/a

Description:
The "has-one-through" relationship defines a one-to-one relationship with another model, as per the docs. However, the actual example given (according to the DB schema at least) is, wrong, and backwards.
The DB schema defines a Mechanic, ala the following - fine.
mechanics
    id - integer
    name - string

And then it defines a Car, which is associated to a single mechanic via mechanic_id ala:
cars
    id - integer
    model - string
    mechanic_id - integer

This would represent the 'many' side of a 1:many relationship, i.e. many cars can be associated to a single mechanic, which should yield the following possible relationships:

$mechanic->cars (via HasMany)
$car->mechanic (via BelongsTo)

And it's the same with Owner. The Owner is associated to a single car via car_id:
owners
    id - integer
    name - string
    car_id - integer

Which actually means multiple owners could be related to a single Car, giving rise to the following relationships:

$car->owners (via HasMany)
$owner->car (via BelongsTo)

Problem 1: The schema is backwards
The schema is clearly counter to the intent.
If it was the intention for cars.mechanic_id to be a unique field (not mentioned or referenced, and counter-intuitive), then it'd make more sense to define mechanic.car_id to associate the Mechanic to a single Car, so via the schema it's obvious which way the 1:* relationship goes.
And same with the Owner. If Car is meant to have only one owner, then cars.owner_id should be defined instead of owners.car_id.
i.e. the schema ought to be the following if the nature of the relationships the example is attempting to demonstrate is to be defined properly:
mechanics
    id - integer
    name - string
    car_id - integer

cars
    id - integer
    model - string
    owner_id - integer

owners
    id - integer
    name - string

Problem 2: The 'has-one-through' relationship is wrong
The documentation presumes that there's a "has-one-through" relationship from the mechanic to the owner, i.e. a mechanic is related through to a single owner. And that's simply not the case according to the schema (unless you accept the incorrect implementation of the foreign fields with uniqueness indexes).
According to the schema, a Mechanic is associated to multiple Cars, and each Car is associated to multiple Owners.
Which means, each Owner can have a 'has-one-through' relationship to the Mechanic (Owner has one Car, which has one Mechanic), but not the other way around.
What should it be?
According to the incorrect schema, Owners have a 'has-one-through' relationship through to a Mechanic, not vice-versa. Owners are associated to a particular Car, which is then associated to a particular Mechanic. Therefore, if we know the Owner, we can determine the Mechanic, but not the other way around.
So what's the problem?
It literally cannot be implemented with the 'HasOneThrough' relationship.
If we try to implement the following:
class Owner extends Model
{
    public function mechanic() {
        return $this->hasOneThrough(Mechanic::class, Car::class);
    }
}

It won't work because it expects the backwards DB design.
We need to manipulate the key field names to tweak the resulting SQL to get it to work properly, like this:
class Owner extends Model
{
    public function mechanic() {
        $mechanic = new Mechanic;
        $car = new Car;

        return $this->hasOneThrough(
            $mechanic,
            $car,
            $car->primaryKey,
            $mechanic->primaryKey,
            $car->getForeignKey(),
            $mechanic->getForeignKey()
        );
    }
}

This appears to return the correct result, but shouldn't require this much manipulation to achieve
TLDR
The documentation presumes mechanics are associated to one owner, whereas the schema defines it the other way around. The presumption is incorrect, and the code is engineered around this incorrect assumption.
HasOneThrough is broken, because it primarily works for a broken/incorrect schema implementation, rather than a correctly normalised schema, and therefore does not do (by default) what it's supposed to.

Heya, appreciate the lengthy post but if there's an issue with the documentation please just send in a PR to correct them.


Anyway to trigger wheel calibration or modify wheel center ?

When you first plug in the wheel it runs a calibration.
Is there a command to trigger the calibration process or adjust the center point left/right manually without unplugging /replugging the wheel.
I will check out the data exchanged via usb analyzer, but am wondering if you already came across some information while developing this.

Hi there. ^_^
Not sure if you can adjust the center point.
You should be able to re-run the calibration process by taking a copy of the init code from inside https://github.com/ForestMist/logitech-g27/blob/master/code/index.js
So maybe something like this...
var g = require('logitech-g27')

g.connect(function(err) {
    // do something here or wait awhile

    // calibrate again
    // G27 Racing Wheel init from https://www.lfs.net/forum/post/1587144#post1587144
    g.relayOS([0xf8, 0x0a, 0x00, 0x00, 0x00, 0x00, 0x00])
    g.relayOS([0xf8, 0x09, 0x04, 0x01, 0x00, 0x00, 0x00])
})

I don't have my wheel hooked up to my computer right now so the above code may need tweaking.


feat: add multi-column to/in combobox

It would be very nice if existing Combobox support multi column in dropdown list. It is very common in applications where the field for selecting for example a customer need to provide more details than a name, (e.g. to also include: phone, email, social number e.t.c. ). The same happens with products, more columns are needed so as to identify and select the correct product. Attached is a sample screenshot of a multicolumn combobox I used in a windows desktop application, that makes more clear.

Please contact me for any details. Thanks!

You could use OptionTemplate to customize each items. And to define your table cells.

Thanks Denis, I didn't know that way. I will try it.
Once more... congratulations for this beautiful library.


Tutorial: Record and stream inference metadata with video

I'm trying to implement this tutorial with Visual Studio Code AVA extension:
https://docs.microsoft.com/en-us/azure/azure-video-analyzer/video-analyzer-docs/record-stream-inference-data-with-video
In which you show this schematic:

So I tried to implement it, but 2 errors ocurred: (There are no matching media types between nodes 'objectTrackingProcessor' and 'videoSink')

I read in this document that Video Sink has limitations: "Must be immediately downstream from RTSP source or signal gate processor."
https://docs.microsoft.com/en-us/azure/azure-video-analyzer/video-analyzer-docs/quotas-limitations
So it is not possible to link Video Sink and Object Tracker like you said?

There is no video from object tracker, it can inference or metadata.  if you remove video between object tracker & video sink this will work..

you right! But I can not watch the video.

Do you have any idea?

Could you share more detail on where you see this issue, this is based on https://docs.microsoft.com/en-us/azure/azure-video-analyzer/video-analyzer-docs/record-stream-inference-data-with-video ??
Thanks
-Kishor
From: Paulo Ilhéu @.>
Sent: Monday, October 4, 2021 6:54 AM
To: Azure/video-analyzer @.>
Cc: Nandakishor Basavanthappa @.>; Comment @.>
Subject: Re: [Azure/video-analyzer] Tutorial: Record and stream inference metadata with video (#58)
you right! But I can not watch the video.
[image]https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fuser-images.githubusercontent.com%2F8053606%2F135863772-59b718a5-3045-4580-90a9-93a58167183b.png&data=04|01|nandab%40microsoft.com|7ca0fcaf50ab420fec7808d9873e709f|72f988bf86f141af91ab2d7cd011db47|1|0|637689524521821203|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=GyFG%2BZ8j8i4wknmJWQk1%2B6uK4nWNpY031vLbtAp%2BARM%3D&reserved=0
Do you have any idea?

You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FAzure%2Fvideo-analyzer%2Fissues%2F58%23issuecomment-933509698&data=04|01|nandab%40microsoft.com|7ca0fcaf50ab420fec7808d9873e709f|72f988bf86f141af91ab2d7cd011db47|1|0|637689524521821203|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=SMZ9bflFFh52CQ3E9YAyBDva3g%2BbEwoL34N%2FdM29YBI%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAJOMSERI7GOSZRQ47WK7MH3UFGWYDANCNFSM5FEW2DAA&data=04|01|nandab%40microsoft.com|7ca0fcaf50ab420fec7808d9873e709f|72f988bf86f141af91ab2d7cd011db47|1|0|637689524521831152|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=IAYwuAi0Fpx%2FrTaQ4h46eQSd2uivByJ02jo8iSSgcv8%3D&reserved=0.
Triage notifications on the go with GitHub Mobile for iOShttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fapps.apple.com%2Fapp%2Fapple-store%2Fid1477376905%3Fct%3Dnotification-email%26mt%3D8%26pt%3D524675&data=04|01|nandab%40microsoft.com|7ca0fcaf50ab420fec7808d9873e709f|72f988bf86f141af91ab2d7cd011db47|1|0|637689524521831152|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=zbjIzawU0t0ku0ckZK0zqM4Sofm%2Bv915hWsdAiExso0%3D&reserved=0 or Androidhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fplay.google.com%2Fstore%2Fapps%2Fdetails%3Fid%3Dcom.github.android%26referrer%3Dutm_campaign%253Dnotification-email%2526utm_medium%253Demail%2526utm_source%253Dgithub&data=04|01|nandab%40microsoft.com|7ca0fcaf50ab420fec7808d9873e709f|72f988bf86f141af91ab2d7cd011db47|1|0|637689524521841111|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=nHj89X18HNmNPuqxrx73OFq5VttG2F4Tdd5u%2FYW2i2k%3D&reserved=0.

also could you remove video in your object tracker to video sinc output type ?

there is no video from object tracker refer to following snippet

Closing this issue. Thank you @KishorIoT .


Expire sessions

This comes on top of #257.
Aside from what the subject says (and what this needs), this PR pulls apart initial tasks and setting up periodic tasks to make race conditions less likely (see #265).

Ugh... somethings wonky here:
duffy=> select expires_at from sessions;
         expires_at
----------------------------
 2081-04-27 10:30:23.296186
(1 row)

duffy=>

While that's fixed, the workaround to the race condition doesn't work: with the standard PersistentScheduler, scheduled tasks can only be set up very early, i.e. before the workers are up – and the workaround is to run the background tasks once before installing them being run periodically. Whoops.


Room users trade user context

I present to you
My pointless and probably not useful, feature :)

I think you forgot to push some local changes, I don't see what you posted here

I think you forgot to push some local changes, I don't see what you posted here

COORRECT, i will push it

good job


Update sbt-scoverage

Codecov Report

Merging #75 into master will increase coverage by 4.23%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master      #75      +/-   ##
==========================================
+ Coverage   86.82%   91.05%   +4.23%
==========================================
  Files          11        8       -3
  Lines         129      123       -6
  Branches       10        7       -3
==========================================
  Hits          112      112
+ Misses         17       11       -6

Impacted Files
Coverage Δ

...rc/main/scala/io/circe/jackson/JacksonCompat.scala

...rc/main/scala/io/circe/jackson/JacksonCompat.scala

...rc/main/scala/io/circe/jackson/JacksonCompat.scala

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 fea70c9...61e9afa. Read the comment docs.


Getting error ValueError: need more than 1 value to unpack

I've been using rtd for some time now with no issues, but now getting a failure on 1 page which is causing the build to fail.  I am using platyPS to convert PowerShell help to markdown.  The error I'm receiving is

File "/home/docs/checkouts/readthedocs.org/user_builds/venafitppps/envs/latest/local/lib/python2.7/site-packages/markdown/serializers.py", line 228, in add_qname
uri, tag = qname[1:].split("}", 1)
ValueError: need more than 1 value to unpack

The file it is failing on can be found at GitHub.

Read the Docs project URL: https://venafitppps.readthedocs.io/en/latest/
Build URL (if applicable): https://readthedocs.org/projects/venafitppps/builds/7749272/

I'm not doing anything different (famous last words) in the other files so wondering if this is a bug.
Thanks in advance for the help and let me know if there's any other info I can provide.

Does this work locally for you?

If this was working for you a time ago, it may be related to a mkdocs update, rtd uses mkdocs==0.17.3. We recommend to pin your mkdocs version in a requirements file, so you don't have these problems.

Thanks, @stsewd.  It was working up until 2 weeks ago.  0.17.3 was released in March so I didn't think that was it.
The process looks to fail on a curly brace so I did some troubleshooting and found that it doesn't like the following line
{@{Name=Given Name; Value=Greg}, @{Name=Surname; Value=Brownstein}}
Does anything strike you as being problematic?

I'm not really familiar with that syntax, I think the missing } should be located in another place (maybe above that line)

Also, are you having this problem locally? If so, you should try solving that first, this issue tracker is for rtd related problems only.


feat(pixels): add my new pixel

Checklist

[ x] I ran npm test locally and it passed without errors.

[ x] I only edited the _data/pixels.json file.

[ x] I entered the username in the pixels.json that I'm also using to create this pull request.
[ x] I acknowledge that all my contributions will be made under the project's license.

hi! thanks for adding my pixel to the project!


extra space removed on gettingstarted page.fix #72

Extra horizontal space removed from Getting Started page.
Screenshot:

@imjog thank you so much for raising a pull request on kitura.io!
We need a little bit of admin your end though - in order for us to accept pull-requests to the Swift@IBM repositories, the contributor must first complete a Contributor License Agreement (CLA).  Please see our CLA repository https://github.com/IBM-Swift/CLA for more information.  I realise that this is not in the kitura.io contributing guidelines so I will address that oversight.  Once you've submitted your form (I suggest emailing the form as a pdf) you can then contribute to any of the Swift@IBM repositories.
I did test out the style="width:100%" option but it doesn't play well on non-mobile screens as it forces the image to be displayed at a width so great that the image becomes blurry. I think style="max-width:100%" might work on both mobile and desktop?

@helenmasters thank you for reviewing PR!
I will complete Contributor License Agreement (CLA) tomorrow. I am looking forward to contributing more to Swift@IBM.
Yes, style="max-width:100%" will work fine for both mobile and desktop. I really did'nt consider blur issue. I will make one more commit on this PR.

@imjog did you complete and send a CLA form to<EMAIL_ADDRESS>as I would like to merge your pull request if you have done so.
Many thanks Helen

Hi @helenmasters I sent a CLA form to<EMAIL_ADDRESS>Sorry for delay. I was busy with my exams.
Please merge my PR. Looking forward to contribute more.
Thanks :)

@helenmasters thank you :)


JavaReflection: Dynamic check loading

JenkinsLintAction class is the root class. But does it make sense to instance new Checks explicitly or it is worth to use Reflection and load all those classes under org.jenkins.ci.plugins.jenkinslint.check?
Let's refactor this plugin and enable this feature.

I'm not sure whether using a third party library might help to avoid explicitly defining those checks...


Made isClient setter internal to remove an internal setter method.

Sorry,  I made the same change in master not realizing this was here.  Thanks.


Warning with webpack

WARNING in ../~/node-fetch/~/encoding/lib/encoding.js
Critical dependencies:
9:12-34 the request of a dependency is an expression
 @ ../~/node-fetch/~/encoding/lib/encoding.js 9:12-34

You are looking for https://www.npmjs.com/package/isomorphic-fetch
It uses node-fetch on server-side and switches to github fetch polyfill on client.

Great Thanks!


Error while try to execute 'make notebook' in Docker

It seems I have a problem with nvidia-docker. I am trying to execute make notebook on my HP notebook but I get the following error:
Successfully tagged keras:latest
GPU=0 nvidia-docker run -it -v /home/user/keras:/src/workspace -v "/home/user/Data":/data --net=host --env KERAS_BACKEND=tensorflow keras
/bin/sh: 1: nvidia-docker: not found
Makefile:25: recipe for target 'notebook' failed
make: *** [notebook] Error 127

What I did:

checked out the project from git: git clone https://github.com/keras-team/keras.git
cd docker
make notebook

Any idea what is wrong here?

Can replicate the same issue, the scripts are still using a deprecated commands: https://github.com/NVIDIA/nvidia-docker#usage


if E3 == E3:  <-- Error!

The output of /version on my server is:

[23:55:02 INFO]: This server is running Paper version git-Paper-118 (MC: 1.15.2) (Implementing API version 1.15.2-R0.1-SNAPSHOT)
[23:55:02 INFO]: Checking version, please wait...
[23:55:03 INFO]: Previous version: git-Paper-105 (MC: 1.15.2)
[23:55:03 INFO]: You are running the latest version

The output of /version denizen on my server is:

[23:55:32 INFO]: Denizen version 1.1.3-SNAPSHOT (build 4621-DEV)
[23:55:32 INFO]: Author: The DenizenScript Team

(If relevant) A link to a paste of a script demonstrating the issue:
Debug (Line 86)
https://one.denizenscript.com/paste/65560
The issue is:
I was comparing two tags in an if statement: - if <tag1> == <tag2>: and was getting console errors, so I tried narrating instead. - narrate "<tag1> == <tag2>" This correctly output "E3 == E3" in chat. I did a little digging and discovered that, for some strange reason, even a direct comparison of - if E3 == E3: throws an error. See my debug above for a test of exactly that.

Post your actual test script please


Go: inconsistent outline for variables defined in for loops

Testing #2389
Shows in in the outline but not want:

Doesn't show any variables in the outline:

It's not the idea to show locals in outline. However, in the first sample the loop defines a struct-type (nameless) and outline only shows one of its properties (in ignores want)

@tanhakabir I can actually not reproduce. Can you give this another try with<EMAIL_ADDRESS>or later? Also, please the repo/files you tried this with

This is the repo I was working with: https://github.dev/golang/example
Testing with 0.0.36 now


Terminal window opens but no prompt is shown on Windows 10

Issue Type: Bug
I'm running VScode in a docker container on Windows 10.
When I click view terminal, the window for the terminal opens but no prompt is shown.
VS Code version: Code 1.45.1 (5763d909d5f12fe19f215cbfdd29a91c0fa9208a, 2020-05-14T08:27:35.169Z)
OS version: Windows_NT x64 10.0.17763

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-8650U CPU @ 1.90GHz (8 x 2112)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: disabled_offprotected_video_decode: enabledrasterization: enabledskia_renderer: disabled_off_okvideo_decode: enabledviz_display_compositor: enabled_onviz_hit_test_surface_layer: disabled_off_okwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
15.91GB (1.63GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions (4)

Extension
Author (truncated)
Version

vscode-eslint
dba
2.1.5

vscode-docker
ms-
1.2.0

python
ms-
2020.5.80290

remote-wsl
ms-
0.44.2

Please provide a screenshot illustrating the problem. See this wiki page for more info on how to report actionable terminal issues.
/label "needs more info" integrated-terminal

Hello Daniel
I've attached a screenshot
On Mon, Jun 8, 2020 at 12:23 PM Daniel Imms<EMAIL_ADDRESS>wrote:

Please provide a screenshot illustrating the problem. See this wiki page
https://github.com/microsoft/vscode/wiki/Terminal-Issues for more info
on how to report actionable terminal issues.
/label "needs more info" integrated-terminal
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/microsoft/vscode/issues/98491#issuecomment-640826411,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACE23D347NRYZ36G2ENSZGDRVU3CZANCNFSM4NJHLCEA
.

@ReddyKilowatt image attachments don't work via email, you'll need to go to github.com to attach it.


[adapter-venice] Introduce new VeniceDB adapter

Summary:

introduce a new Adapter to read data from VeniceDB (LinkedIn OSS)

Notes:

this is the first step to unlock testing, we will add more ops in the future
unfortunately we had to depend on the DataStax for of Venice and so we need to add a third party repository (the DataStax public repository)
unfortunately the Venice client depends on some OSS libraries from LinkedIn, so I had to also add the LinkedIn OSS repository

LGTM


Fix NT.HealthCheck table access in sometimes illegal way

This was causing (among other things perhaps?) the hourglass trigger appends to not work after a mod restart, because NT.HealthCheck could not be set to an empty table at init (global store not yet ready?), which the next line then attempts to access and fails, throwing a lua error.

hold on merging for a little, but approved


Reversing words in a String-leetcode

AIM
Want to add solution the "Reversing words in  a String".
Kindly assign me this problem I want to add solution to this problem in C++.
Link To the problem:-https://leetcode.com/problems/reverse-words-in-a-string/
DETAILS
Given an input string s, reverse the order of the words.
A word is defined as a sequence of non-space characters. The words in s will be separated by at least one space.
Return a string of the words in reverse order concatenated by a single space.
Example 1:
Input: s = "the sky is blue"
Output: "blue is sky the"
Example 2:
Input: s = "  hello world  "
Output: "world hello"
Explanation: Your reversed string should not contain leading or trailing spaces.
Programming language

[ ] C
[ ] C++
[ ] Java
[ ] Python

Please assign me this issue under LGM-SOC'21

Kindly assign me this issue under LGMSOC'21

I would like to code in Python. Kindly assign me this issue under LGM SoC'21

i want to work on this issue under java language
please assign me

i want to work on this issue under cpp please assign it to me @Amit366


Completely silent updates

An option for completely silent and automatic VS Code updates. Instead of needing to click a bunch of popups in the bottom right corner.

For the first popup that says "update is available", automatically download and install the update.
For the second popup that says "restart visual studio code to apply the latest update", automatically assume "later". When the user restarts VS Code, the update will install.
Don't pop up the release notes in a new tab after updating.

When opening a new window, prevent these popups as well, which tend to pop up again every time you open a new window.
Trigger this behavior by selecting Settings -> Update: Mode > silent and automatic.

duplicate of #9539 found by https://github.com/microsoft/vscode/issues?q=silent+updates+is%3Aopen

/duplicate - thanks @matkoniecz


Use Vault in CI scripts

:rocket: deployment of gp-lookup succeeded (http://gp-lookup-pr-42.dev.beta.nhschoices.net)

:rocket: deployment of gp-lookup succeeded (http://gp-lookup-pr-42.dev.beta.nhschoices.net)


🐛 fix: Optional email service proper handling

Description:
As per the docs, email service should be optionally provided.
But the code requires it.
If we do not provide it, an exception is raised due to missing environment variables
If we provide it with incorrect values (such as the values in the example .env we get error TypeError: Converting circular structure to JSON when we try to login using google oauth.
This error, when traced, is due to the below request error sent to api.convertkit.com
data: { error: 'Authorization Failed', message: 'API Key not valid' }

This is how the request URL looks like<EMAIL_ADDRESS>so an error is expected to happen here.
This PR handles the aforementioned case
Adding Feature(s):

Make email service optional.

Fixing Issue(s):

Error during oauth sign in due to incorrect email environment variables

Changes

Thanks for being proactive about creating a solution for this.
I'd like to get away from using local variables like _error and _optional and instead use a validation library like zod to do this for us. Please leave this as-is for now. I will follow up with a follow-up commit that does that.

reopend here


CLIMATE-952 - Update ocw.data_processor.write_netcdf to compatible wUpdate ocw.data_processor.write_netcdf to compatible with xarray

write_netcdf and write_netcdf_multiple_datasets_with_subregions have been updated
now the time dimension is unlimited
x and y are names for the dimensions for longitudinal and latitudinal coordinates

@agoodm Could you review and merge this PR?

LGTM


How to get a localStorage value

I have an angular app and with bubblewrap tool I generated an android application and it's working fine
I want  to access localStorage in my android code to get certain keys values
How can I achieve this ?

Duplicate of #