Reorganize makefile

Description

generate target for every testcase from filename
simplify targets to improve readability

# create k3d cluster
make cluster

# install cert-manager & kubewarden
make install

# run test by filename or test group
make basic-end-to-end-tests.bats upgrade.bats
make tests

# install fresh kw on new cluster
make reinstall

# remove k3d cluster
make clean

The changes seems fine to me. But the upgrade tests is failing when I try to run it locally. The command that I'm using to run it is:
make cluster install upgrade.bats

Furthermore, can you please update the README file with the basic target and how to use them? The file is out of date and we should keep it updated.

My guess is the issue is in this line:
upgrade.bats:: clean cluster

After I've aborted the command I could not find any cluster locally. Thus, I think that the cluster is deleted before running the tests. That's make sense if you see the error messages:
pod/kubewarden-controller-5c489689c5-n6vx7 condition met
k3d cluster delete kubewarden-testing
INFO[0000] Deleting cluster 'kubewarden-testing'
INFO[0002] Deleting 2 attached volumes...
WARN[0003] Failed to delete volume 'k3d-kubewarden-testing-images' of cluster 'kubewarden-testing': failed to find volume 'k3d-kubewarden-testing-images': Error: No such volume: k3d-kubewarden-testing-images -> Try to delete it manually
INFO[0003] Removing cluster details from default kubeconfig...
INFO[0003] Removing standalone kubeconfig file (if there is one)...
INFO[0003] Successfully deleted cluster kubewarden-testing!
./scripts/generate_resources_dir.sh /home/jvanz/suse/kubewarden-end-to-end-tests/resources policies.kubewarden.io/v1 v1
RESOURCES_DIR=/home/jvanz/suse/kubewarden-end-to-end-tests/resources/resources_v1 TIMEOUT=5m KUBEWARDEN_CRDS_CHART_OLD_VERSION=1.0.0 KUBEWARDEN_DEFAULTS_CHART_OLD_VERSION=1.1.6 KUBEWARDEN_CONTROLLER_CHART_OLD_VERSION=1.1.0 KUBEWARDEN_CRD
S_CHART_VERSION=1.1.0 KUBEWARDEN_DEFAULTS_CHART_VERSION=1.1.7 KUBEWARDEN_CONTROLLER_CHART_VERSION=1.1.1 KUBEWARDEN_CONTROLLER_CHART_RELEASE=kubewarden-controller KUBEWARDEN_CHARTS_LOCATION=kubewarden CONTROLLER_CHART=kubewarden/kubewarde
n-controller CLUSTER_CONTEXT=k3d-kubewarden-testing  NAMESPACE=kubewarden bats --print-output-on-failure  /home/jvanz/suse/kubewarden-end-to-end-tests/tests/upgrade.bats
upgrade.bats
 ✗ [CRD upgrade] Install old Kubewarden
   (from function `wait_pods' in file tests/common.bash, line 34,
    from function `setup' in test file tests/upgrade.bats, line 5)
     `wait_pods' failed
   /home/jvanz/suse/kubewarden-end-to-end-tests/tests/common.bash: line 3: bats_require_minimum_version: command not found
   error: context "k3d-kubewarden-testing" does not exist
   Fail
   error: context "k3d-kubewarden-testing" does not exist
   Fail
   error: context "k3d-kubewarden-testing" does not exist
   Fail
   error: context "k3d-kubewarden-testing" does not exist
   Fail
   error: context "k3d-kubewarden-testing" does not exist
   Fail
   error: context "k3d-kubewarden-testing" does not exist
   Fail
   Godot: pods not running

/home/jvanz/suse/kubewarden-end-to-end-tests/tests/common.bash: line 3: bats_require_minimum_version: command not found

What version of bats do you use? You need at least 1.5.0
For normal tests you run
make clean   # to remove previous cluster
make cluster # to create k3d cluster
make install # to install kubewarden

# or you can group 3 steps above into
make reinstall

# then run tests
make test1.bats test2.bats

Upgrade test is different, it needs to install kubewarden itself because it starts with version -1.
I added dependencies to makefile, so you run just
# makefile will clean, create cluster, install cert-manager and run upgrade test
make upgrade.bats

Currently you need to hack KUBEWARDEN_DEFAULTS_CHART_OLD_VERSION to jq -r ".[0].version" - because .[1].version is broken
I will fix README, thanks for review

I updated README and fixed upgrade.test. I also removed prepare target because it only installed k3d - I don't think it's useful to have target just for that. Can you review again pls?

@kravciak for some reason when I run the command make upgrade.bats it get stuck. The command never finished. This is my environment:
The last commit:
commit 55f511ee0eb3647d403d12bce291f0fb1002316f (HEAD -> martin-main, kravciak/main)
Author: Martin Kravec<EMAIL_ADDRESS>Date:   Mon Sep 5 14:45:40 2022 +0200

    Fix Makefile description in README

 README.md | 60 ++++++++++++++++++++++++++++++++++--------------------------
 1 file changed, 34 insertions(+), 26 deletions(-)

The change applied in the Makefile:
$ git --no-pager diff
diff --git a/Makefile b/Makefile
index f1d1ebe..5f1e43f 100644
--- a/Makefile
+++ b/Makefile
@@ -22,7 +22,7 @@ KUBEWARDEN_HELM_REPO_URL ?= https://charts.kubewarden.io
 KUBEWARDEN_CHARTS_LOCATION ?= kubewarden

 KUBEWARDEN_CONTROLLER_CHART_VERSION ?= $(shell helm search repo $(KUBEWARDEN_HELM_REPO_NAME)/$(KUBEWARDEN_CONTROLLER_CHART_RELEASE) --versions -o json --devel | jq -r ".[0].version")
-KUBEWARDEN_CONTROLLER_CHART_OLD_VERSION ?= $(shell helm search repo $(KUBEWARDEN_HELM_REPO_NAME)/$(KUBEWARDEN_CONTROLLER_CHART_RELEASE) --versions -o json --devel | jq -r ".[1].version")
+KUBEWARDEN_CONTROLLER_CHART_OLD_VERSION ?= $(shell helm search repo $(KUBEWARDEN_HELM_REPO_NAME)/$(KUBEWARDEN_CONTROLLER_CHART_RELEASE) --versions -o json --devel | jq -r ".[0].version")
 KUBEWARDEN_CONTROLLER_CHART_RELEASE ?= kubewarden-controller
 KUBEWARDEN_CRDS_CHART_VERSION ?= $(shell helm search repo $(KUBEWARDEN_HELM_REPO_NAME)/$(KUBEWARDEN_CRDS_CHART_RELEASE) --versions -o json --devel | jq -r ".[0].version")
 KUBEWARDEN_CRDS_CHART_OLD_VERSION ?= $(shell helm search repo $(KUBEWARDEN_HELM_REPO_NAME)/$(KUBEWARDEN_CRDS_CHART_RELEASE) --versions -o json --devel | jq -r ".[1].version")****

Bats version:
$ bats --version
Bats 1.6.0

Some errors from the policy server logs:
2022-09-05T18:35:22.071026Z ERROR policy_server::worker_pool: cannot spawn worker error="[ghcr.io/kubewarden/policies/capabilities-psp:v0.1.9: settings of policy ghcr.io/kubewarden/policies/capabilities-psp:v0.1.9 are invalid: Some(\
"Error invoking settings validation callback: GuestCallFailure(\\\"Guest call failed: Error decoding validation payload {\\\\\\\"allowed_capabilities\\\\\\\":null,\\\\\\\"required_drop_capabilities\\\\\\\":[\\\\\\\"ALL\\\\\\\"],\\\\\
\\"default_add_capabilities\\\\\\\":null}: Error(\\\\\\\"invalid type: null, expected a sequence\\\\\\\", line: 1, column: 28)\\\")\")]"                                                                                │

2022-09-05T18:35:22.071054Z ERROR policy_server::worker_pool: cannot spawn worker error="[ghcr.io/kubewarden/policies/capabilities-psp:v0.1.9: settings of policy ghcr.io/kubewarden/policies/capabilities-psp:v0.1.9 are invalid: Some(\
"Error invoking settings validation callback: GuestCallFailure(\\\"Guest call failed: Error decoding validation payload {\\\\\\\"allowed_capabilities\\\\\\\":null,\\\\\\\"required_drop_capabilities\\\\\\\":[\\\\\\\"ALL\\\\\\\"],\\\\\
\\"default_add_capabilities\\\\\\\":null}: Error(\\\\\\\"invalid type: null, expected a sequence\\\\\\\", line: 1, column: 28)\\\")\")]"

@jvanz your git diff is wrong - change KUBEWARDEN_DEFAULTS_CHART_OLD_VERSION, not KUBEWARDEN_CONTROLLER_CHART_OLD_VERSION
-KUBEWARDEN_DEFAULTS_CHART_OLD_VERSION ?= $(shell helm search repo $(KUBEWARDEN_HELM_REPO_NAME)/$(KUBEWARDEN_DEFAULTS_CHART_RELEASE) --versions -o json --devel | jq -r ".[1].version")
+KUBEWARDEN_DEFAULTS_CHART_OLD_VERSION ?= $(shell helm search repo $(KUBEWARDEN_HELM_REPO_NAME)/$(KUBEWARDEN_DEFAULTS_CHART_RELEASE) --versions -o json --devel | jq -r ".[0].version")

@jvanz your git diff is wrong - change KUBEWARDEN_DEFAULTS_CHART_OLD_VERSION, not KUBEWARDEN_CONTROLLER_CHART_OLD_VERSION
-KUBEWARDEN_DEFAULTS_CHART_OLD_VERSION ?= $(shell helm search repo $(KUBEWARDEN_HELM_REPO_NAME)/$(KUBEWARDEN_DEFAULTS_CHART_RELEASE) --versions -o json --devel | jq -r ".[1].version")
+KUBEWARDEN_DEFAULTS_CHART_OLD_VERSION ?= $(shell helm search repo $(KUBEWARDEN_HELM_REPO_NAME)/$(KUBEWARDEN_DEFAULTS_CHART_RELEASE) --versions -o json --devel | jq -r ".[0].version")

:man_facepalming:


improve the calculation method

make sure that the program can directly calculate expressions like: 1+3 * 2 +4 and give the correct result

Got it, thanks.


OAuth2: There is no cleanup for orphaned auth codes/tokens/refreshTokens

Can be solved by implementing expiration for codes/tokens/refreshTokens


Freezes: Limit the maximum return row count.

What type of PR is this? (check all applicable)

[ ] Refactor
[ ] Feature
[ ] Bug Fix
[ ] New Query Runner (Data Source)
[ ] New Alert Destination
[ ] Other

Description
Related Tickets & Documents
Mobile & Desktop Screenshots/Recordings (if there are UI changes)

 Redash dashboards loading slow/overloading browser


Support smart filters for headers

already implemented, tests added in https://github.com/provectus/kafka-ui/pull/2763 PR


Stick Hero Game Using Javascript

🛠 Fixes Issue (Number)
Fixes #837
👨💻 Changes proposed and Brief Description
Implemented Stick Hero Game using Javascript.
Styled using CSS
✅ Check List (Check all the applicable boxes)

[x] My code doesn't break any part of the project
[x] This PR does not contain plagiarized content.
[x] My Addition/Changes works properly and matches the overall repo pattern.
[x] The title of my pull request is a short description of the requested changes.

📷 Screenshots

Review Requested by @swapnilsparsh
Kindly add labels: GSSoC and Level 3 . Thank you!


Siri Problem

Hi,
When I use Siri I receive this response:
"Ok, the light is masculine"
On homebridge log I don't see any error.

Italian Siri?  Seems to be an issue with Siri, see https://github.com/nfarina/homebridge/issues/1228.


[Feature][Issue Tracking] Identify scrum and non-scrum board

Search before asking

[X] I had searched in the issues and found no similar feature requirement.

Description

I propose that we can add a field into "board" object to identify the type of different boards.
Also, when I pick up a data scope of Jira connection or some other tools within blueprint on Config UI, I want to know what kind of boards they are as well.

Use case
In practice, a Scrum board and a Normal board represent two different agile patterns, and when analyzing delivery performance, we will use different data aggregate method.
E.g.

Since a scrum board normally have sprints to collect data, we will gather, aggregate, and compare issue data by each sprints.
When a team applies Kanban agile practice, they will pick up issue from backlog periodically, like every fortnight. Therefore, gathering, aggregating, and comparing issue date by fortnight is a natural choice.

Related issues
No response
Are you willing to submit a PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Plan of attack
collect board.type and store

@thenicetgp would you please take this one?

Willing to take this one


[W7.7][F09-B1] Immunity Book

Your forked repo is supposed to be named "main" and the branch should be "master".
You should be able to access repo at the following URL
https://github.com/CS2103AUG2017-{team-ID}/main/

Do not close this PR as we expect to track changes through the coming weeks.

Hello, we just change the name of our repo to main and the default working branch to master. However, this PR is from the old "main" branch which we are not developing on anymore. Should we create a new pull request from master to this repo or would you like us to keep moving our commits in the future from master to main so that this PR is constantly updated?


Release 0.0.9?

It could be nice if the DDE stuff was available

Sorry, I forgot to create the release 😅
Fixed by now, and the artifacts are available. I've also pushed a new DocBleach-Web version


Truncation one more time

In this PR I refactored the logic for truncation to no longer rely on a watch event that's triggered when data changes. Instead it will trigger the truncation logic when the height of the table row changes. This also handles the case to reset the truncation logic if the content of the table changes from a page size change or more faceting criteria is added.
This is more performant and works more flawlessly for text cases. There are mixed results for images and iframes which can render after the digest cycle has finished, increasing the row height without triggering the sensor (which checks during the next digest cycle which doesn't occur).

I found an inconsistency in the behavior of this logic that might help us debug this further.
On load, the ellipsis logic works flawlessly and consistently for text columns. But if you go to the next page, and get back to the first page, it won't trigger the ellipsis logic at all. This could mean that the Angularjs is not calling the link function of the ellipsis directive for the newly drawn rows.

@RFSH  I made changes to the reference modified listener to also check and set the overflows. This appears to address the cases when the user changes the page size, changes a facet, or using next/previous and the overflows don't update automatically.


Bumped dependencies and added some new rules

Description
Bumped dependencies and added some new rules

Codecov Report
Attention: 5 lines in your changes are missing coverage. Please review.

Comparison is base (61c7516) 46.41% compared to head (89a9efb) 46.42%.

Files
Patch %
Lines

internal/scanners/dec/rules.go
0.00%
5 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #179      +/-   ##
==========================================
+ Coverage   46.41%   46.42%   +0.01%
==========================================
  Files          72       72
  Lines        3456     3466      +10
==========================================
+ Hits         1604     1609       +5
- Misses       1839     1844       +5
  Partials       13       13

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


"your settings are incorrect"

I have a blue notification box in the top left of my screen that i cant get rid of.
It reads "Your settings are incorrect. Please have marks in the correct boxes if you are running on steam / with dx11
You are running: Explorer.exe
Your settings expect: PathOfExile_x64.exe"

its not a mercury trade


Decide how to implement difficulty selection

So I see that @cugone is beginning to work on the difficulty implementation. Thank you for your efforts! It's very much appreciated especially while I'm recovering from illness.

So the point of this topic is to decide exactly how to select difficulty. The 'planet select' screen is sort of a stub screen at this point. The difficulty selection that's been presented is very much like the original game, something I'd hoped to avoid.
What I'm envisioning is a game setup screen that present all of these options as a sort of full-screen UI. I'll finish fleshing this out later today...

Yeah, I'm finding the new popup slows down the process of creating a new game.
I think it might flow smoother if the difficult selection and the planet selection were done on the same screen. Potentially you could skip selecting a difficulty setting, and just have it use a default value, making the selection optional. That way you still only need one click (on the planet type) to start a new game.
I suppose the initial default should be "easy". If the player changes the difficulty, perhaps it can be remembered for future games.
As for display layout, maybe place a horizontal list of difficulty settings above the list of planets. Since clicking the planet starts the game, it should be below any optional settings.

I agree, moving the difficulty options to the PlanetSelectionState would be better.
The differences between each difficulty aren't totally explained even in the original. The only thing I've been able to determine is the frequency of disasters. Beginner having none and each difficulty afterwards increasing the frequency.

Maybe we can eventually add some sort of mouse-over popup that summarizes what the difficulty setting changes.

I thought I posted this already but anyway, here goes.
The idea here is to make this as visual as possible with plenty of details, the sort of thing that modern games do. Attached is an archive with a very basic visual layout that I'm considering. Note that color scheme isn't set in stone -- I've been using the black/green as a sort of nod to the original (see the fullscreen UI's) but it's definitely not set in stone.
I didn't note what the images are exactly but basically the first set is some sort of icon that indicates visually a difficulty level (I'm no artist so I leave that up to someone better suited to that to offer ideas), the next set I think is for just the planets? I'm not sure a visual for stars is really necessary.
The final one shows the planet selection rotating.
I just realized now that I forgot about a site map choice but ultimately I'd like to offer a number of site maps for planets (minimum of one, maximum of 10) with the user being able to expand into multiple sites and create routes between them. That's a later part of development though but assume that after planet choice the user then gets to choose a site map for landing.
game-setup-demo.zip

Side note -- difficulty for OPHD will affect:

Initial population count (what comes down from the ship)
Initial resource count (what comes down from the ship)
Disaster frequency
Intensity of disasters
Choice of stars
Choice of Planets
Mine Counts
Mine Yields
Population growth
Effect of morale on factory production and research
Decreased positive morale effects and increased negative morale effects
How critically the population responds to your decisions (related to the above)
Structure costs
Recycling values of razed structures and landers
Research times
Benefits of research (e.g., if base beneficial effect of a completed research topic is 10% reduction in cost of structures for normal difficulty, beginner would be 12%, normal 10%, hard 7.5% and impossible 5%... arbitrary numbers but illustrates the thinking here)

Probably a few other factors but basically difficulty here can be potentially punishing especially on 'nightmare mode'.
The current state of the game is very close to how I want 'difficult' to be in terms of resource production. I wanted to get a sort of balance right for 'hard' that's not impossible but is also kinda hard to get right. Normal modes would offer more initial resources

Direct posting the images from the zip file Leeor attached for ease of reference:

Looks like the "Star Select" and the second "Planet Select" screens are identical. I assume only one of those was intended.

To add my thoughts to that, it might be nice if we could reduce clicking. Instead of having next buttons, clicking large images representing the difficulty or planet could proceed to the next screen. Details information about the options can be displayed on mouse over. Having a back button is fine. The "Star Select" screen might need to be reworked to support such a flow, as none of the options seem to represent a final proceed to the next screen type of selection.
For the "Star Select" screen, it seems there are a few options. Maybe that one can be re-worked to be a bit more graphical. In particular, the radio buttons can be replaced by sequences of clickable images. Same for the check box, which can be replaced by two images, and function much like the radio buttons converted to graphical form.

Actually, I wonder if it would be possible to present everything on one screen. That would eliminate the forward/back navigation. Instead, if space is limited, we might have various areas enlarge on mouse over, so we can see more detail about a given set of parameters.
I imagine the three sections to be stacked, with various toggle icons for the options. Initially the difficulty select and star select options could be small, and represent default options, while the planet select is large. If the user moves the mouse over the difficulty select section, that area could enlarge, while the previously enlarged section shrinks a bit. That would help draw the user's attention to a given area. If the mouse is not over any particular area, I think it should revert to making the planet select area large. That would be the section where making a selection would close the screen and proceed to start the game.

I assume "Star Select" is similar to the original game. You select a star system first (not knowing what planets are available), then a planet within it.

Hmm, that creates some interesting opportunities for animations. When changing the star, we may need to change the planets. Rather than just have them abruptly change, perhaps we can animate the change. Maybe have the planets fly off, and new ones fly in. Or perhaps the planets can be illustrated as some kind of solar system, and changing the star make the view fly to another solar system.
I think the details about each selection should all appear on mouse over.

It depends on the amount of text. For small amounts, sure show it on hover. But for large amounts you want it to stay up when it's clicked so the user can read it.

To be clear, I'm still talking about having a permanent display area for the text, not a popup tooltip text display. I was thinking it should work the same as the current planet select screen in OPHD. On mouse over, the planet spins, and descriptive text is displayed at the bottom of the screen.

The star select screen isn't too much like the original game -- it doesn't have an effect on the availability of planets but it does affect other things like resource availability, light output, stellar storm frequency, etc.
The 'text area' at the bottom of the screens is intended to be that 'hover text' area -- there won't be a box drawn around it, that's just there to visualize the layout of the components.
As a note, I'm entirely open to suggestions here especially in terms of Stellar choice and exactly what variables star choice will have on the simulation.

Hmm, that creates some interesting opportunities for animations. When changing the star, we may need to change the planets. Rather than just have them abruptly change, perhaps we can animate the change. Maybe have the planets fly off, and new ones fly in. Or perhaps the planets can be illustrated as some kind of solar system, and changing the star make the view fly to another solar system.

That's the general idea. I have a tweening engine I was experimenting with way back when. Would ultimately be part of the polishing stage but yeah, having things slide around and animated would be slick :D

Reread some of @DanRStevens's comments and wanted to go over some of it.

To add my thoughts to that, it might be nice if we could reduce clicking. Instead of having next buttons, clicking large images representing the difficulty or planet could proceed to the next screen. Details information about the options can be displayed on mouse over. Having a back button is fine. The "Star Select" screen might need to be reworked to support such a flow, as none of the options seem to represent a final proceed to the next screen type of selection.

I generally like this idea. It's better than what originally put together.

For the "Star Select" screen, it seems there are a few options. Maybe that one can be re-worked to be a bit more graphical. In particular, the radio buttons can be replaced by sequences of clickable images. Same for the check box, which can be replaced by two images, and function much like the radio buttons converted to graphical form.

I like this idea, though I would want to be sure that it's as intuitive as possible without having to have too much layer stacking.

Actually, I wonder if it would be possible to present everything on one screen. That would eliminate the forward/back navigation. Instead, if space is limited, we might have various areas enlarge on mouse over, so we can see more detail about a given set of parameters.

That could be done using 'panels' that expand when clicked on, almost like clicking on a card to see what's on it. I wouldn't want everything to be all on one screen at once as it can be really intimidating and I imagine the screen could get very busy. I like the idea of modern EX designs where smaller chunks of information are presented to the user at a time.

I imagine the three sections to be stacked, with various toggle icons for the options. Initially the difficulty select and star select options could be small, and represent default options, while the planet select is large. If the user moves the mouse over the difficulty select section, that area could enlarge, while the previously enlarged section shrinks a bit. That would help draw the user's attention to a given area. If the mouse is not over any particular area, I think it should revert to making the planet select area large. That would be the section where making a selection would close the screen and proceed to start the game.

This sounds a lot like my card metaphor. Could expand on this one a bit.


Hang after deploying to Android

I opened the SmartHotel.Clients.sln file in VSMac, selected the Android platform and Debug|ARM configuration. When I try to Run/Debug in the emulator, after the app is deployed the launch screen appears in the emulator but it hangs from there.

VSMac info
=== Visual Studio Community 2017 for Mac (Preview) ===

Version 7.5 Preview (7.5 build 933)
Installation UUID: b2e09fb1-8867-4094-8886-a5bc5fe7571b
Runtime:
	Mono <IP_ADDRESS> (2017-12/dac60775616) (64-bit)
	GTK+ 2.24.23 (Raleigh theme)
	Xamarin.Mac <IP_ADDRESS> (master / 0c7c49a6)

	Package version: 510000179

=== NuGet ===

Version: <IP_ADDRESS>45

=== .NET Core ===

Runtime: /usr/local/share/dotnet/dotnet
Runtime Versions:
	2.0.5
	2.0.3
SDK: /usr/local/share/dotnet/sdk/2.1.4/Sdks
SDK Versions:
	2.1.4
	2.0.3
MSBuild SDKs: /Library/Frameworks/Mono.framework/Versions/5.10.0/lib/mono/msbuild/15.0/bin/Sdks

=== Xamarin.Profiler ===

Version: 1.6.2
Location: /Applications/Xamarin Profiler.app/Contents/MacOS/Xamarin Profiler

=== Xamarin.Android ===

Version: <IP_ADDRESS> (Visual Studio Community)
Android SDK: /Users/sayedhashimi/Library/Developer/Xamarin/android-sdk-macosx
	Supported Android versions:
		6.0 (API level 23)
		7.1 (API level 25)
		8.0 (API level 26)
		8.1 (API level 27)

SDK Tools Version: 26.1.1
SDK Platform Tools Version: 26.0.2
SDK Build Tools Version: 26.0.3

Java SDK: /usr
java version "1.8.0_131"
Java(TM) SE Runtime Environment (build 1.8.0_131-b11)
Java HotSpot(TM) 64-Bit Server VM (build 25.131-b11, mixed mode)

Android Designer EPL code available here:
https://github.com/xamarin/AndroidDesigner.EPL

=== Apple Developer Tools ===

Xcode 9.2 (13772)
Build 9C40b

=== Xamarin.Mac ===

Version: <IP_ADDRESS> (Visual Studio Community)

=== Xamarin.iOS ===

Version: <IP_ADDRESS> (Visual Studio Community)
Hash: 4d8c4e32
Branch: d15-7
Build date: 2018-03-07 18:39:46-0500

=== Xamarin Inspector ===

Version: 1.4.0
Hash: b3f92f9
Branch: master
Build date: Fri, 19 Jan 2018 22:00:34 GMT
Client compatibility: 1

=== Build Information ===

Release ID: 705000933
Git revision: ace1e107c88477db1abe5e807809650f064112c4
Build date: 2018-03-19 17:42:41-04
Xamarin addins: a01263bdbba2589b1f6e42ef085bd31f6e3bfcf4
Build lane: monodevelop-lion-d15-7

=== Operating System ===

Mac OS X 10.13.3
Darwin 17.4.0 Darwin Kernel Version 17.4.0
    Sun Dec 17 09:19:54 PST 2017
    root:xnu-4570.41.2~1/RELEASE_X86_64 x86_64

=== Enabled user installed addins ===

Internet of Things (IoT) development (Preview) 7.1

We have fix this with a latest update.


Document title should be set based on AppSpec.clientAppName

We currently set the document title based on the appName in the webpack config via the HtmlWebpackPlugin, but in the case where an application has multiple sub-apps, this is not usually what you want.  The admin app is a good example where it would make more sense for the title to be "MyApp Admin" instead of just "MyApp".
Not sure if there is a way for us to retrieve that from the AppSpec during build time, we may need to set it at runtime, though that is not really ideal since you would see the default appName from the webpack config for a second until our init process has started and the AppSpec has been parsed ...

Yes - agree.
Note we already adjust the title on the fly to append non-prod environment name:
https://github.com/xh/hoist-react/blob/96d4cc95c05e854217ee7e983278b550dbcb671c/svc/EnvironmentService.js#L61
I would vote for moving this logic to the overall init sequence - probably here:
https://github.com/xh/hoist-react/blob/96d4cc95c05e854217ee7e983278b550dbcb671c/core/XH.js#L509
and include the change to use clientAppName. Note this is the real resolution for https://github.com/xh/toolbox/issues/252.

Better to not set a name then in the webpack config?  Or set a name of the empty string, trusting the javascript to set the name?


Animation stops moving when page isn't done loading

Once the genome API is done loading:

The loading symbol freezes (obviously you can't tell, but notice the console on the right)

The animation needs to consider that all aspects of the interface must be done loading before stopping spinning.

This is related to #710 and part of 5.7. I'm closing this ticket.


Title: Hover effect on service section

Is your feature request related to a problem? Please describe.
Service section is so simple, it can be more enhanced .
Describe the solution you'd like.
We can add a hover effect by using simple css and js code.
Describe alternatives you've considered.
We can also add another feactures on hovering on services.
Add any other context or screenshots about the feature request here.
No response
Record

[X] I agree to follow this project's Code of Conduct
[X] I have checked the existing issues
[X] I'm a GSSoC'23 contributor
[X] I want to work on this issue

i want to improve the hovering part of mail and phone icon. please assign me this work under gssoc'23

Already assigned before


SPKI-based construction

Per the spec, it would be great if one could construct public keys from DER or PEM encoded SPKI values with the RSASSA-PSS OID. Would you be open to adding such a feature?

The Zig and C implementations can already export public keys as SPKI values with the RSASSA-PSS OID.
So yes, I'd definitely be open to adding SPKI support in the Rust implementation as well. And importation in other implementations would be nice to have, too!


Editing operations: Elements of array parameters are displayed with incorrect height when pressing Reset and then Cancel

Steps to reproduce:

Add a custom procedure containing an array parameter, e.g. gimp-paintbrush.
Set the size of the array to a non-zero value, e.g. 6.
Close the dialog.
Edit the procedure (open the dialog again).
Press Reset.
Press Cancel.
Edit the procedure again.

Result: Each element in the array has a height of 1 pixel.
Workaround: Edit the procedure and press Cancel again.

Fixed in 4.0.


support vips avif speed

This PR try to add avif speed for imagor.
It's said imagor doesn't support avif speed, while similar image cdns like thumbor and imgproxy do support.
In a test with wrk, imagor is faster than thumbor and imgproxy, and with a smallest Stdev.
Please note, libvips has changed to parameter from speed to effort in 2021, however, speed still works.
ref 1: https://www.libvips.org/API/current/VipsForeignSave.html#vips-heifsave-buffer
ref 2: https://github.com/libvips/libvips/blob/master/libvips/foreign/heifsave.c
(libvips would convert effort to speed again for heif's api. :-)
BTW, heif support different speed for different codec:
aom 0-9, https://github.com/strukturag/libheif/blob/master/libheif/plugins/encoder_aom.cc
rav1e 0-10, https://github.com/strukturag/libheif/blob/master/libheif/plugins/encoder_rav1e.cc
svt 0-13, https://github.com/strukturag/libheif/blob/master/libheif/plugins/encoder_svt.cc

Thanks for the PR! Would be great if you can test a test assertion for WithAvifSpeed at
https://github.com/cshum/imagor/blob/master/vips/option_test.go

Thanks for merge.

Before the next version release, this is currently available under the Docker build of develop branch:
https://github.com/cshum/imagor/pkgs/container/imagor/255863182?tag=develop
ghcr.io/cshum/imagor@sha256:cd5469eaf7541eee310e9dc9ca90490a931781673cbc8875ddea3a1fcc0e9081


Fix vertical alignment of Transfer amount model

Fixes #
https://github.com/busyorg/busy/issues/1905
Changes
Summary of changes you made.

It moves model from top alignment to center alignment

Test plan
Explain how to test changes you made.

Go to http://localhost:3000/@husaini/transfers
Click on 'Transfer' button

Demo (optional)
Before is available in issue: https://github.com/busyorg/busy/issues/1905
After:
Mobile

Desktop

@Sekhmet can you please look at this? It has been lingering around for 8 days now. I had to fix conflicts. It is probably best to update and merge it or close this pull request. Both ways are fine by me.


LinearLayout Support - loading skeleton messes recycler view orders in parent view.

First of all, Thank you for your fast bugfixes. I appreciate that.
I'll try to help you with reporting bugs and as soon as I can, I will contribute on the lib.
About this issue, take this layout hierarchy:
<androidx.core.widget.NestedScrollView
            android:layout_width="match_parent"
            android:layout_height="match_parent">

            <LinearLayout
                android:layout_width="match_parent"
                android:layout_height="wrap_content"
                android:orientation="vertical">

                <androidx.recyclerview.widget.RecyclerView
                    android:id="@+id/rv_first"
                    android:layout_width="match_parent"
                    android:layout_height="wrap_content" />

                <androidx.recyclerview.widget.RecyclerView
                    android:id="@+id/rv_second"
                    android:layout_width="match_parent"
                    android:layout_height="wrap_content" />

            </LinearLayout>

 </androidx.core.widget.NestedScrollView>

So when I try to call .loadSkeleton() function on both of the RecyclerViews their order changes. so on loading state the first RV shows at bottom and the second one shows at the top.

I found the problem. The issue is coming from LinearLayout. I've changed the container to ConstraintLayout and problem fixed. Any idea why LinearLayout has problem with this lib?

Hi!, @TurKurT656. Sorry for late reply.
LinearLayouts are not yet supported!!. You can use another ViewGroup. For example ConstraintLayout, just like you said.
I'll be working in this new feature.
Thanks a lot!  ✌ ✌

Any updates for this issue? Using FlexboxLayout is also messes the recycler view orders. Also, I have to convert all recycler view parents to another ViewGroup only to temporarily fix this issue.


add uofcanada

the university official website URL:
https://www.upei.ca/cairocampus

a URL of a page on the official website where a long-term: https://www.upei.ca/programs/computer-science

a URL of a page or some other proof showing that the university recognizes the domain which you are submitting as an official email domain for the enrolled students:
http://upei.uofcanada.edu.eg/SelfService/Home.aspx

-email domain:
@uofcanada.edu.eg

@magdi-amer Pull request merged. Thank you!


Enhancement: Embedded Subtitles

2.9.13 Release
Most of my library is in MKV format and comes with embedded subtitles, is it possible to monitor already existing subtitles in such a format?

We should already detect them (listed on mediaInformation tab), and should add them already into the NFO. IMHO.
What do you mean with "monitor" ?!

Ohh my mistake, It does seem to be picking them up. Jumped the gun on bringin' my perceived issue to your attention.
I do however have some files that now have multiple subtitles (One local thanks to TMM scraping, and the original embedded ones).
I'll remove the extras and let you know if it persists.
though another idea would be converting file types and automatically embedding subtitles for MKV, I don't see that being too trivial of a implementation though. so, for now, I'll continue using MKVMerge

Well, it would be not hard... but we decided to not touch/alter the user's video file.
Mainly to keep the file hash consistent, to not trigger any sync scripts...


Update ModalDropdown.js

The top value is not valid value which is return is NAN.
Error:
Invariant Violation: [757,"RCTView",91,{"position":"absolute","height":"auto","borderWidth":1,"borderColor":4291022286,"borderRadius":2,"backgroundColor":4294967295,"justifyContent":"center","borderTopWidth":0,"width":372.6,"marginTop":17.92,"shadowColor":4278190080,"shadowOffset":{"width":0,"height":2},"shadowOpacity":0.25,"shadowRadius":3.84,"elevation":5,"top":"<>","left":33}] is not usable as a native method argument

I got this issue too
I'm try to install these version

v0.7.0
"react-native-modal-dropdown": "git+https://github.com/siemiatj/react-native-modal-dropdown.git",
but, still has an error
please, help me

https://gyazo.com/bb5be5715bc81cd25d8d20557d4de434

This error occued when you define the dropDown's height value and the value is not numeric.
You can located the error at .../node_modules/react-native-modal-dropdown/components/ModalDropdown.js  line 264.
To resolve this problem, just given a numeric to the dropDown Component.
My English is so weak. Hope you can understand.hah


contracts fix store-call test path

missed file rename from https://github.com/paritytech/substrate/pull/13565/files#diff-6370b6056b1f65146c88f1b9f291c2d3a64bc5f59c93b1905bad95a09d959f7b

bot merge force

bot merge

bot merge force


Show the token when starting a notebook from the REPL

Would it be possible to show the token of a new Jupyter instance when doing notebook()?
When starting through a ssh tunnel a browser can't be automatically started and my current workaround is:
ssh -L 8888:localhost:8888 HOST
> julia
julia> using IJulia
julia> notebook(detached=true)
julia> run(`$(IJulia.notebook_cmd[1]) notebook list`)
# Get the token used by the instance bound to localhost:8888
# and visit localhost:8888 in your browser and past the token their

notebook could also start the notebook with token authentication disabled...

Yes that would be lovely, but not easy see https://github.com/jupyter/notebook/issues/2254
You apparently need to either set a password or set a configuration option to set the token to be empty.

Considering the way Jupyter is used in IJulia/Julia, this seems like a reasonable suggestion: "Would it be possible to show the token of a new Jupyter instance when doing notebook()?"
You can also consider setting up a password, along the lines of: https://discourse.julialang.org/t/ijulia-password-token-requirement/4974 (Careful, that "howto" is not completely accurate!)
@stevengj : You might be sharing the same machine with multiple users in a cluster environment. You would not want other users to be able to execute code with your user's privileges. Tokens prevent that.

notebook_cmd is no longer working

notebook_cmd is no longer working

This would be useful! As a workaround, I'm now using the following shameful hack with IJulia 1.23.2:
using IJulia
jupyter = IJulia.find_jupyter_subcommand("")[1]
run(`$(jupyter) notebook list`)

This would be useful! As a workaround, I'm now using the following shameful hack with IJulia 1.23.2:
using IJulia
jupyter = IJulia.find_jupyter_subcommand("")[1]
run(`$(jupyter) notebook list`)

I wanted to note that this helped me troubleshoot a problem during a clean install of conda and jupyter via IJulia.  Previous to the clean install I had been tinkering around with hosting jupyter on a remote server and one of the server links was still open and connected to the default url and ports.  As a result this caused a connection failure when trying to run the newly installed jupyter via IJulia.  Long story short, I was able to use this (without installing jupyter by some other means) to see all the notebooks lingering in the background that had failed to close down properly and were conflicting with this

This would be useful! As a workaround, I'm now using the following shameful hack with IJulia 1.23.2:
using IJulia
jupyter = IJulia.find_jupyter_subcommand("")[1]
run(`$(jupyter) notebook list`)

Thanks for this.  I am brand new to WSL and this is this only way I could get the notebook running in the Windows browser.  It's been a few years, but has there been anything implemented more permanent?  Or a better way to pass the token?

@uajdgiz I am also facing this issue.
 Not to say I wouldn't appreciate this feature being updated to eliminate the need for a token or print it out on launch, but yeah--it feels like there should be a better way.
The best thing I could think of was just running a Jupyterlab docker container with Julia built in. I found a helpful guide here to introduce the concept, and there's more documentation on Jupyter's documentation on doing this. This feels like the right way to handle this.
Unfortunately for me, I need Julia 1.6.7, but their docker images only go back to 1.9.1 or something--so I wasn't able to personally take this route without making my own image. For now (and for the sake of time) I'll be sticking to the "shameful hack" lol.


Clarification Needed on the Specificity of test_dataset

Hello,
I am currently working with the project and have a question regarding the test_dataset used within. Could you please clarify whether the test_dataset needs to be domain-specific, particularly tailored to the RAG domain, or if a generic labeled dataset is suitable for this purpose?

For evaluating Retrieval-Augmented Generation (RAG) models like the ones you’re working with, the choice of a validation dataset can significantly influence how well the model’s performance generalizes across different types of data and use cases.
If your RAG model is intended to be used in a specific domain (like medical, legal, or technical documents), it would be beneficial to use a domain-specific validation dataset. This approach helps ensure that the model performs well on the type of content it will encounter in its expected environment.
However, if the model is intended for more general use, a generic labeled dataset could suffice. This kind of dataset helps evaluate the model’s ability to handle a broad range of topics and types of queries.
In your case, it be beneficial to use a domain-specific validation dataset tailored to the RAG domain to accurately evaluate your RAG model using ARES.


[SwipeRefresh] Update doc page

Also renamed SwipeRefreshCustomSample to SwipeRefreshTweakedIndicatorSample, and added a new SwipeRefreshCustomIndicatorSample.
Thanks to @ricknout for providing some demo videos!
Closes #324

Changes LGTM, but the videos don't seem to be working when I view the markdown on GitHub on the cb/swiperefresh branch 🤔

I'm not surprised, this is using all sorts of markdown extensions which GitHub doesn't support.
I should try and figure out a way to build the site and deploy it somewhere, similar to what we do for snapshots.


loading loop

App is in a loading loop while nothing being displayed.
Win10
Version: 2.5.0

Fixed in 2.6.0

Thanks, but it still doesn't work for me with version 2.6.0

Thanks for the reply. Deleting "%APPDATA%/twitch-desktop" fixed the issue :)
If I may, I have three improvements you probably have planned anyway.

ffz support (chat doesn't load when enabling BetterTTV btw)
smaller and more tiles for streams/categories when increasing window size
minimize streaming window when looking for other streams/favorites

Thank you for feedback, very appreciated. Will be reviewing that during the coming days :) .


Chemical Notation Support

Hey, thank you for the plugin, I have yet to find anything like nabla.nvim. I am trying to use Vim to take scientific notes, which often contain chemical and mathematical formulas.
Though, the plugin is quite primitive (naturally), so it doesn't really help me to take notes in Vim. See the image below where I am getting an error.

This is how it is supposed to be rendered:

I have to add that the italic letters are of no significance, what's important is formulas like H_{12} to work.

The parser will need to be changed in order to support chemical formulas. This will take some time but it's probably feasible. I will see what I can do when I have some spare time.
How would you like to formula to be rendered in ASCII? so that I have a reference.

The parser will need to be changed in order to support chemical formulas. This will take some time but it's probably feasible. I will see what I can do when I have some spare time.
How would you like to formula to be rendered in ASCII? so that I have a reference.

Something like this?
C₅H₁₂(l) + 8O₂(g) → 5CO₂ + 6H₂O(l)

Something like this?
C₅H₁₂(l) + 8O₂(g) → 5CO₂ + 6H₂O(l)

Something like this?
C₅H₁₂(l) + 8O₂(g) → 5CO₂(g) + 6H₂O(l)

Exactly like that!

Something like this?
C₅H₁₂(l) + 8O₂(g) → 5CO₂(g) + 6H₂O(l)

Exactly like that!

Could you test if it's working for you?
I added basic support for chemical formulas in the last commit. It was much easier than a thought (well the code is also not that pretty).
You do not need to use the Latex { } for subscript because it's already supported as the parser parses the number as a whole.
Try like this:
C_5H_12(l)+8O_2(g)->5CO_2(g)+H_2O(l)

Try some other formulas to see if it works overall.

Could you test if it's working for you?
I added basic support for chemical formulas in the last commit. It was much easier than a thought (well the code is also not that pretty).
You do not need to use the Latex { } for subscript because it's already supported as the parser parses the number as a whole.
Try like this:
C_5H_12(l)+8O_2(g)->5CO_2(g)+H_2O(l)

Try some other formulas to see if it works overall.

The formula I provided above looks like this:

The arrow is not rendered properly but it could be an issue on my end.

Here's another 'formula', this is how it should look like:

And this is how it looks:

I also tried to put spaces between the characters but I got the following error: E5105: Error while calling lua chunk: ...pData\Local\nvim\plugged\nabla.nvim\lua\nabla\parser.lua:591: attempt to call field 'priority' (a nil value).

There are also formulas like this, for which I use \cdot:

Though, obviously it won't recognize \cdot and the arrow renders like a square, (unidentified character) anyway, even when copy-pasted. I tried to delete the arrow but still I get nabla error!:

For Chemistry, I think curly brackets, { }, are necessary because you will get an error if you want to add a plus or a minus sign in the superscript after a number, as shown in the image below.

When you need to have both a superscript and a subscript on a character, it places the superscript on the subscript. If you try to type the subscript first, you get an error.
Compare this:

To this:

I also look into replicating this in Vim:

But I get an error: E5105: Error while calling lua chunk: ...ppData\Local\nvim\plugged\nabla.nvim\lua\nabla\ascii.lua:212: attempt to concatenate a nil value.
I also get that blank square character whenever I try to make a letter subscript. Probably a problem on my end.

Fractions in sub/superscripts would be quite useful as well.

Symbols like approximately equal, ≈, would be a neat addition.

To produce the following, I am converted theta to θ in a separate line, then moved it next to sin. I copy-pasted ≈ and moved λ / D manually from a different set of lines.

I can't test integer yet as it doesn't render properly on this system for some reason. Anyway, I do realize many of the things I included in this comment might be irrelevant or simply not suitable for Neovim but I just wanted to let it all out, just in case it benefits someone out there.

The formula I provided above looks like this:

The arrow is not rendered properly but it could be an issue on my end.

Here's another 'formula', this is how it should look like:

And this is how it looks:

I also tried to put spaces between the characters but I got the following error: E5105: Error while calling lua chunk: ...pData\Local\nvim\plugged\nabla.nvim\lua\nabla\parser.lua:591: attempt to call field 'priority' (a nil value).

There are also formulas like this, for which I use \cdot:

Though, obviously it won't recognize \cdot and the arrow renders like a square, (unidentified character) anyway, even when copy-pasted. I tried to delete the arrow but still I get nabla error!:

For Chemistry, I think curly brackets, { }, are necessary because you will get an error if you want to add a plus or a minus sign in the superscript after a number, as shown in the image below.

When you need to have both a superscript and a subscript on a character, it places the superscript on the subscript. If you try to type the subscript first, you get an error.
Compare this:

To this:

I also look into replicating this in Vim:

But I get an error: E5105: Error while calling lua chunk: ...ppData\Local\nvim\plugged\nabla.nvim\lua\nabla\ascii.lua:212: attempt to concatenate a nil value.
I also get that blank square character whenever I try to make a letter subscript. Probably a problem on my end.

Fractions in sub/superscripts would be quite useful as well.

Symbols like approximately equal, ≈, would be a neat addition.

To produce the following, I am converted theta to θ in a separate line, then moved it next to sin. I copy-pasted ≈ and moved λ / D manually from a different set of lines.

I can't test integer yet as it doesn't render properly on this system for some reason. Anyway, I do realize many of the things I included in this comment might be irrelevant or simply not suitable for Neovim but I just wanted to let it all out, just in case it benefits someone out there.

The arrow is not rendered properly but it could be an issue on my end.

The font seems to be issue. Changing it will fix the problem. I use Fantasque Sans which supports special Unicode. There are plenty of others.

Though, obviously it won't recognize \cdot and the arrow renders like a square, (unidentified character) anyway, even when copy-pasted. I tried to delete the arrow but still I get nabla error!:

Yes, if you pass Latex commands, the parser will error because it won't be able to recognize the syntax.

For Chemistry, I think curly brackets, { }, are necessary because you will get an error if you want to add a plus or a minus sign in the superscript after a number, as shown in the image below.

After thinking for a bit, I think it might be interesting to switch to a standard LaTeX parser instead of this math notations I'm using currently. I came up with a simple design document for nabla.nvim which you could check out .
Could write what you need and approve? (it should be publicly editable)

Fractions in sub/superscripts would be quite useful as well.

Already possible, see https://github.com/jbyuki/nabla.nvim/blob/03e3a55375fac32f2c060d10095d8a44bf491326/test/input.txt#L37

To produce the following, I converted theta to θ in a separate line, then moved it next to sin. I copy-pasted ≈ and moved λ / D manually from a different set of lines.

Also possible, though the approximatively equal is different (but I could change it):
sin(theta) ~= lambda/D

I can't test integer yet as it doesn't render properly on this system for some reason. Anyway, I do realize many of the things I included in this comment might be irrelevant or simply not suitable for Neovim but I just wanted to let it all out, just in case it benefits someone out there.

The rendering of integral is not so pretty but should be functional with the correct font. Something like:
https://github.com/jbyuki/nabla.nvim/blob/03e3a55375fac32f2c060d10095d8a44bf491326/test/input.txt#L27
The design document is to have something so that we could use as a reference. Who knows, maybe I will maybe never implement it. But at least, we would have something written. It will be essentially the specifications of such a plugin.

The arrow is not rendered properly but it could be an issue on my end.

The font seems to be issue. Changing it will fix the problem. I use Fantasque Sans which supports special Unicode. There are plenty of others.

Though, obviously it won't recognize \cdot and the arrow renders like a square, (unidentified character) anyway, even when copy-pasted. I tried to delete the arrow but still I get nabla error!:

Yes, if you pass Latex commands, the parser will error because it won't be able to recognize the syntax.

For Chemistry, I think curly brackets, { }, are necessary because you will get an error if you want to add a plus or a minus sign in the superscript after a number, as shown in the image below.

After thinking for a bit, I think it might be interesting to switch to a standard LaTeX parser instead of this math notations I'm using currently. I came up with a simple design document for nabla.nvim which you could check out .
Could write what you need and approve? (it should be publicly editable)

Fractions in sub/superscripts would be quite useful as well.

Already possible, see https://github.com/jbyuki/nabla.nvim/blob/03e3a55375fac32f2c060d10095d8a44bf491326/test/input.txt#L37

To produce the following, I converted theta to θ in a separate line, then moved it next to sin. I copy-pasted ≈ and moved λ / D manually from a different set of lines.

Also possible, though the approximatively equal is different (but I could change it):
sin(theta) ~= lambda/D

I can't test integer yet as it doesn't render properly on this system for some reason. Anyway, I do realize many of the things I included in this comment might be irrelevant or simply not suitable for Neovim but I just wanted to let it all out, just in case it benefits someone out there.

The rendering of integral is not so pretty but should be functional with the correct font. Something like:
https://github.com/jbyuki/nabla.nvim/blob/03e3a55375fac32f2c060d10095d8a44bf491326/test/input.txt#L27
The design document is to have something so that we could use as a reference. Who knows, maybe I will maybe never implement it. But at least, we would have something written. It will be essentially the specifications of such a plugin.

Yes, if you pass Latex commands, the parser will error because it won't be able to recognize the syntax.
I see but how do I go about inserting a middle dot, then? Unicode has a middle dot character.

sin(theta) ~= lambda/D
This works but what I meant was sintheta without parantheses. Though, it makes no difference, so I'm guessing there is no point in wasting time on this one. The approximately equal sign is fine as well but when there is a tilde, ~, in the line, I get a nil instead of the rendered equation. See the image below.

https://github.com/jbyuki/nabla.nvim/blob/03e3a55375fac32f2c060d10095d8a44bf491326/test/input.txt#L27
Integral seems to be working pretty well, also, thank you for bearing with me when I'm sleepy, only now I notice I called integral an integer.

As for fractions, I was thinking about something like this ⅕ but I don't think it is possible.

Yes, if you pass Latex commands, the parser will error because it won't be able to recognize the syntax.
I see but how do I go about inserting a middle dot, then? Unicode has a middle dot character.

sin(theta) ~= lambda/D
This works but what I meant was sintheta without parantheses. Though, it makes no difference, so I'm guessing there is no point in wasting time on this one. The approximately equal sign is fine as well but when there is a tilde, ~, in the line, I get a nil instead of the rendered equation. See the image below.

https://github.com/jbyuki/nabla.nvim/blob/03e3a55375fac32f2c060d10095d8a44bf491326/test/input.txt#L27
Integral seems to be working pretty well, also, thank you for bearing with me when I'm sleepy, only now I notice I called integral an integer.

As for fractions, I was thinking about something like this ⅕ but I don't think it is possible.

I see but how do I go about inserting a middle dot, then? Unicode has a middle dot character.

This works but what I meant was sintheta without parantheses. Though, it makes no difference, so I'm guessing there is no point in wasting time on this one. The approximately equal sign is fine as well but when there is a tilde, ~, in the line, I get a nil instead of the rendered equation. See the image below.

I see then the LaTeX syntax which is more flexible would be the solution here. I will try to implement one when I have some spare time.
Also please look at the design document if you wish to have something as a feature. It will provide me with written reference.

As for fractions, I was thinking about something like this ⅕ but I don't think it is possible.

I wasn't aware of Unicode for fractions, there are probably implementable.

I see but how do I go about inserting a middle dot, then? Unicode has a middle dot character.

This works but what I meant was sintheta without parantheses. Though, it makes no difference, so I'm guessing there is no point in wasting time on this one. The approximately equal sign is fine as well but when there is a tilde, ~, in the line, I get a nil instead of the rendered equation. See the image below.

I see then the LaTeX syntax which is more flexible would be the solution here. I will try to implement one when I have some spare time.
Also please look at the design document if you wish to have something as a feature. It will provide me with written reference.

As for fractions, I was thinking about something like this ⅕ but I don't think it is possible.

I wasn't aware of Unicode for fractions, there are probably implementable.

Do you have programming capabilities? You might help me with the process.

Do you have programming capabilities? You might help me with the process.

Do you have programming capabilities? You might help me with the process.

I don't have any experience in programming, I don't think I can help although I wish to.

Do you have programming capabilities? You might help me with the process.

I don't have any experience in programming, I don't think I can help although I wish to.

The recently added LaTeX syntax made it much better, thank you. I am now able to do \cdot and write sub/superscripts using curly brackets.
I would also like to point out the spaces between characters. As seen in the image below, I didn't put any spaces between characters.

However, when it renders, there is a space between each element of the equation:

This is how it should be like:

The recently added LaTeX syntax made it much better, thank you. I am now able to do \cdot and write sub/superscripts using curly brackets.
I would also like to point out the spaces between characters. As seen in the image below, I didn't put any spaces between characters.

However, when it renders, there is a space between each element of the equation:

This is how it should be like:

I will continue adding support for LaTeX syntax such as matrices.
Is the rendering now looking good for most of the chemical formulas?

I will continue adding support for LaTeX syntax such as matrices.
Is the rendering now looking good for most of the chemical formulas?

I will continue adding support for LaTeX syntax such as matrices.
Is the rendering now looking good for most of the chemical formulas?

It indeed looks pretty neat now, including letters with both superscripts and subscripts.
One thing I noticed though, probably after the latest commit, is that I can add superscripts or subscripts to letters but they disappear when added to numbers.

I will continue adding support for LaTeX syntax such as matrices.
Is the rendering now looking good for most of the chemical formulas?

It indeed looks pretty neat now, including letters with both superscripts and subscripts.
One thing I noticed though, probably after the latest commit, is that I can add superscripts or subscripts to letters but they disappear when added to numbers.

It indeed looks pretty neat now, including letters with both superscripts and subscripts.

Great, now with LaTeX syntax, it should be much easier to type exotic formulas.

One thing I noticed though, probably after the latest commit, is that I can add superscripts or subscripts to letters but they disappear when added to numbers.

Right, I need to add superscript/subscript to number expressions. I will fix it in the next commit.

It indeed looks pretty neat now, including letters with both superscripts and subscripts.

Great, now with LaTeX syntax, it should be much easier to type exotic formulas.

One thing I noticed though, probably after the latest commit, is that I can add superscripts or subscripts to letters but they disappear when added to numbers.

Right, I need to add superscript/subscript to number expressions. I will fix it in the next commit.


lab search fails with less than 3 characters

Best explained with an example:

[prarit@prarit rhel-8]$ lab mr list tg
[prarit@prarit rhel-8]$ lab mr list tg3
!325 tg3: improve PCI VPD access
[prarit@prarit rhel-8]$

I'll look into it.


Selenoid not finding chrome image

Browsers.json file

{
	"chrome": {
        "default": "latest",
        "versions": {
            "latest": {
                "image": "selenoid/chrome:latest",
                "port": "4444",
                "path": "/"
            }
        }
    }
}

Docker image ls

REPOSITORY             TAG              IMAGE ID       CREATED        SIZE
selenoid/chrome        latest           6c08b8cfe30e   3 weeks ago    1.14GB
aerokube/selenoid      latest           fdc2ce7d1eb8   3 months ago   16.2MB

Docker compose file
version: '3'
services:
  selenoid:
    image: "aerokube/selenoid"
    network_mode: bridge
    ports:
      - "4000:4444"
    volumes:
      - ".:/etc/selenoid/" # assumed current dir contains browsers.json
      - "/var/run/docker.sock:/var/run/docker.sock"
    command: ["-conf", "/etc/selenoid/browsers.json","-log-output-dir", "/opt/selenoid/logs"]

http://localhost:8086/status
{"state":{"total":5,"used":0,"queued":0,"pending":0,"browsers":{"MicrosoftEdge":{"latest":{}},"chrome":{"latest":{}},"firefox":{"latest":{}}},"videos":null},"origin":"http://selenoid:4444","browsers":{"MicrosoftEdge":0,"chrome":0,"firefox":0},"sessions":{},"version":"1.10.4[2021-10-10_08:53:47AM]","errors":[]}
I am running this using docker vm created by lima. When I run the tests it says the below error
 selenium.common.exceptions.SessionNotCreatedException: Message: create container: Error response from daemon: No such image: selenoid/chrome:latest
Even though I have selenoid/chrome:latest, selenoid is not able to connect to it and throws not found error. But it works with docker desktop. What config , I am missing here?

@vishnuramvs29 you always have to pull images on every host with Selenoid.

@vishnuramvs29 you always have to pull images on every host with Selenoid.

@vania-pooh I am pulling images everytime. As shown above, I have pulled chrome image and docker ls also shows that.

@vishnuramvs29 Selenoid is just asking to start an exact image using Docker API. Your issue is somewhere on Docker level. This response No such image: selenoid/chrome:latest is from Docker daemon API.

Docker daemon API

@vania-pooh Thanks vania. But not sure how to debug this docker issue.

@vishnuramvs29 make sure docker run --rm -it --privileged selenoid/chrome:latest works for you.

@vishnuramvs29 make sure docker run --rm -it --privileged selenoid/chrome:latest works for you.

2022/02/23 18:40:01 [INIT] [Listening on :7070]
20
Waiting X server...
Waiting X server...
Logging to: /dev/null
Waiting X server...
Waiting X server...
Waiting X server...
Starting ChromeDriver 98.0.4758.48 (d869ab3eda60629b9fabbd4e30c0f833466c83fd-refs/branch-heads/4758@{#415}) on port 4444
All remote connections are allowed. Use an allowlist instead!
Please see https://chromedriver.chromium.org/security-considerations for suggestions on keeping ChromeDriver safe.
ChromeDriver was started successfully.

@vishnuramvs29 then check whether you have DOCKER_HOST environment variable set in your operating system. If it's pointing to another host, then Selenoid will send requests there.

@vishnuramvs29 then check whether you have DOCKER_HOST environment variable set in your operating system. If it's pointing to another host, then Selenoid will send requests there.

DOCKER_HOST is set correctly. Because of that only , I am able to execute the above queries. I am able to see the docker images and selenium is also successfully connecting to Selenoid. only selenoid is not finding the chrome image.

@vishnuramvs29 try to set the same value for Selenoid container.

@vishnuramvs29 try to set the same value for Selenoid container.

Yeah, I have set the same value for selenoid container

@sdt-bkruk in Moon configuration is static. So if you are getting this, then something in browsers.json has changed.


App\Exceptions\TeamSpeak\MaxConnectionAttemptsExceededException in app/Console/Commands/TeamSpeakDaemon.php:128

Error in VATSIM-UK/Core (Laravel)
App\Exceptions\TeamSpeak\MaxConnectionAttemptsExceededException in app/Console/Commands/TeamSpeakDaemon.php:128
Max connection attempts exceeded
View on Bugsnag
Stacktrace
app/Console/Commands/TeamSpeakDaemon.php:128 - App\Console\Commands\TeamSpeakDaemon::establishConnection
app/Console/Commands/TeamSpeakDaemon.php:148 - App\Console\Commands\TeamSpeakDaemon::establishConnection
app/Console/Commands/TeamSpeakDaemon.php:148 - App\Console\Commands\TeamSpeakDaemon::establishConnection
app/Console/Commands/TeamSpeakDaemon.php:148 - App\Console\Commands\TeamSpeakDaemon::establishConnection
app/Console/Commands/TeamSpeakDaemon.php:148 - App\Console\Commands\TeamSpeakDaemon::establishConnection
app/Console/Commands/TeamSpeakDaemon.php:148 - App\Console\Commands\TeamSpeakDaemon::establishConnection
app/Console/Commands/TeamSpeakDaemon.php:46 - App\Console\Commands\TeamSpeakDaemon::handle
app/Console/Commands/TeamSpeakCommand.php:41 - App\Console\Commands\TeamSpeakCommand::run

View full stacktrace

An error linked to this issue has been marked as fixed in Bugsnag
App\Exceptions\TeamSpeak\MaxConnectionAttemptsExceededException in app/Console/Commands/TeamSpeakDaemon.php:128


Let's hold seminar for git usage.

We are already using git(with github) but many contributors doesn't good at the git command.
So, let's hold seminar for git usage. Any volunteer?

@romandev, I'm closing this issue, you can re-open it if you want. :D


wip: fix(test/libsinsp_e2e): fixed libsinsp_e2e tests for more stability

What type of PR is this?
/kind bug
Any specific area of the project related to this PR?
/area driver-modern-bpf
/area libsinsp
/area tests
Does this PR require a change in the driver versions?
What this PR does / why we need it:
This also fixes a couple of small issues found while testing, namely:

a bug in how cgroups were pushed in the modern bpf driver
a new cgroup layout for libvirt-lxc container engine

Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Will leave a self-review before removing wip.
Also, i will trigger the CI many times to see if we are really stable.
Does this PR introduce a user-facing change?:

NONE

First run :green_circle: https://github.com/falcosecurity/libs/actions/runs/11070690824/job/30761583727?pr=2085

Tests with new fixes:

https://github.com/falcosecurity/libs/actions/runs/11102504115/job/30843233834?pr=2085 :green_circle:

Latest fixes:

https://github.com/falcosecurity/libs/actions/runs/11106378445/job/30854576531?pr=2085 :green_circle:

Arm64 modern-ebpf tests failed again with latest code, even if there were no drops :/
Will give it a look tomorrow.

I don't think we can do much better :)

removing wip!

Love this! What an amazing job! LGTM! 🚀


[Antctl]: Support commands to query AntreaPolicy types (CNP/ANP)

Describe the problem/challenge you have
Antctl does not support retrieving internal objects corresponding to CNP
antctl get networkpolicy should support retrieving policies and objects related to CNP.

Thanks for bringing it up. I will work on it.

@GraysonWu please see issue description in https://github.com/vmware-tanzu/antrea/issues/1262 for some hints. Thanks for taking it up!

Hi @tnqn @abhiraut @jianjuns, a question to bring up with you after some offline discussions with @GraysonWu today. After https://github.com/vmware-tanzu/antrea/pull/1258 is merged (the entire PR looks like would be merged after v0.10 from the description), we still have the issue where antctl get netpol ns/name being ambiguous if there is a ANP/K8s NP created by the same ns/name. Do we want to solve this problem by adding a type field to the query for example, or display the both internal NPs if it comes to this situation? What's your preference and thoughts on this?

I feel we should be consistent with kubectl, and uses different resource names. Do you see any issue with that?

The tricky part is the resource antctl trying to get is the internal NP, regardless of whether it was created for K8s NP/ANP/CNP. So antctl get netpol/antctl get cnp/antctl get anp does not make sense to me, because in here netpol maps to the internal NP as opposed to the K8s NP. Following the discussion with @tnqn, I think my proposal would be:

Add a new command line option antctl get netpol [name] [-n namespace] [-t k8s|cnp|anp] [-o yaml]
where -t stand for type. User can specify whether they want to examine the internal NP created for a specific
Network Policy type this way.
If the -t flag is not set, antctl get netpol [name] [-n namespace] could potentially return a list instead of a single internalNP. If there are K8s/ANP both matching ns/name, antctl would output both in the case.

Does the proposal make sense?

I still prefer to keep the K8s resource names, for a CLI tool for end users. But like to here what others think.

I prefer Dyanngg's proposal.
I think If we keep K8s resource name, I'm not sure if users will feel that's a duplicate function with kubectl.
Since K8sNP/ANP/CNP has been transferred to internal NP, we should treat them as the same 'class' only with a different attribute which is type/reference. We can add a column to the output called type. The [-t k8s|cnp|anp] option is just like a filter applied on type.
What do you guys think?

Personally I think it makes sense to keep the subcommand consistent with the API since we already reuse a single controlplane NetworkPolicy resource to represent all kinds of NetworkPolicy. I guess it would be a little ambiguous to have 3 subcommands for a single API and code would be somewhat inconsistent with other get subcommand.
From the usage's perspective, I feel antctl is mostly used by networking admin/antrea developer for troubleshooting purpose, in which situation namespace and policy type are less important. For instance, it's more convenient to check all policies that are applied to a provided appliedToGroup with a single networkpolicy subcommand.
The proposal that adds a type column and option makes sense to me.

For getting a single type of policies, I think different resource names are easier for users to remember.
But I get your point that we might like to support returning all types of policies in one command (probably when policy type is not specified).

Fixed by #1301

Fixed by #1301


[ASWB] Fix android_instrumentation_test test button on Kotlin test targets

Checklist

[x] I have filed an issue about this change and discussed potential changes with the maintainers.
[ ] I have received the approval from the maintainers to make this change.
[x] This is not a stylistic, refactoring, or cleanup change.

Please note that the maintainers will not be reviewing this change until all checkboxes are ticked. See
the Contributions section in the README for more
details.
Discussion thread for this change
Issue number: https://github.com/bazelbuild/intellij/issues/2318
Description of this change
IDE Test Run configuration button doesn't work if android instrumentation test is written in kotlin.

The root cause is that KotlinTestContextProvider only handles kotlin tests, not android tests, and AndroidTestContextProvider only handles java android tests, but not kotlin tests. In this PR, I added support for AndroidTestContextProvider to create AndroidKtTestContext when kotlin elements are detected. I also created an abstracted class AndroidTestContextBase for sharing coding between java and kotlin test context.
After the fix, I verified the test button works for  android_instrumentation_test

@chaoren @vsiva ptal

@chaoren @vsiva any updates here?

@chaoren @vsiva can you take a look?

@alice-ks can you help find an owner for this? Thanks in advance!

Hi there! Thank you for contributing to the IntelliJ repository. We appreciate your time and effort. We're doing a clean up of old PRs and will be closing this one since it seems to have stalled. Please feel free to reopen/let us know if you’re still interested in pursuing this or if you'd like to discuss anything further. We’ll respond as soon as we have the bandwidth/resources to do so.

correct this fix is still needed to support kotlin android instrumentation test targets

@sgowroji can you mark this as not stale?

not stale - cc @mai93 @tpasternak


BUG: get_measurement() time vector

when periodic saving is turned on, time vector from get_measurement() should not alwys start from 0. It should reflect what data in time is stored in the data and video buffer.

implemented


Add Chris to Contributors Table

He messaged me on Twitter pointing out the spacebar issue so I'm adding him to the table.

@all-contributors add @SQUADBCHRIS for bug


Renaming to an Empty name throws exception / Progress Bar stays displayed

Trying to rename assets to an empty name should probably show the Warning dialog, tooltip, and refuse to rename, similar to how it works when making duplicate names.

Fixed in develop-1.1.0


Remove the layout shift when using widgets

One reason for layout shift is the React key that includes the term:
https://github.com/OM-MediaWorks/shacl-form/blob/master/lib/components/field/FieldWrapper.tsx#L180
Each widget should then be hard resetting values. So instead of defaultValue we should be using value and for uHTML we should use .value instead of value.

I was able to do this for most things. There is still some shift happening for BlankNodeEditor.


refactor reference forecasts in metrics

https://github.com/SolarArbiter/solarforecastarbiter_dashboard/issues/214 discussed the need to implement reference forecast metrics on a per observation/forecast basis rather than a single reference forecast for all observation/forecasts. Most relevant part is quoted below:

Looking again at the core calculator and preprocessing code, I'm a little concerned that we might need to change things so that the reference forecast is validated along with the forecast and observations. This would make it easier to supply a unique reference forecast for each forecast/observation pair in core, which could be used or not in the dashboard/api.
I can imagine that people might want to do something like "compare forecast skill metrics for each test forecast at each SURFRAD site". You end up with a graph that shows forecast skill like any other metric. Perhaps the labels are twice as long for clarity about what's being shown.
I had forgotten that the data model already supports one reference forecast per fx/obs pair:

https://github.com/SolarArbiter/solarforecastarbiter-core/blob/7c933afe97307a35057a9eeae1bc92e60d129d7b/solarforecastarbiter/datamodel.py#L922

https://github.com/SolarArbiter/solarforecastarbiter-core/blob/7c933afe97307a35057a9eeae1bc92e60d129d7b/solarforecastarbiter/datamodel.py#L951

https://github.com/SolarArbiter/solarforecastarbiter-core/blob/7c933afe97307a35057a9eeae1bc92e60d129d7b/solarforecastarbiter/datamodel.py#L1156

It appears we need to do the following in core:

[x] reports.get_data_for_report needs to get data for reference forecasts just like any other forecast.
[x] Change preprocessing.process_forecast_observations to handle the reference forecast data and assign it to ProcessedForecastObservation.reference_forecast_values. It appears that ProcessedForecastObservation.reference_forecast_values is not currently used anywhere in the code.
[ ] Remove the ref_pair/ref_fx_obs kwargs from calculator.calculate_metrics,  calculator.calculate_deterministic_metrics, and calculate_probabilistic_metrics. Also event metrics pending #361. Modify these functions to look up the right data from the processed_fx_obs instead.
[ ] Add reference forecast information to reports/templates/metrics_meta_table.html, obsfx_table.html, or a new table. If we put it in the metrics_meta_table, the new column would probably just contain the name (with link) or None. The obsfx_table would make sense to be clear about the resampling/alignment (or lack thereof). But I don't know if we can cram more into that table.

https://github.com/SolarArbiter/solarforecastarbiter_dashboard/pull/220 implements the dashboard front end for this, including a new reference_forecast key under object_pairs in the report's json specification.

Add reference forecast information to reports/templates/metrics_meta_table.html, obsfx_table.html, or a new table. If we put it in the metrics_meta_table, the new column would probably just contain the name (with link) or None. The obsfx_table would make sense to be clear about the resampling/alignment (or lack thereof). But I don't know if we can cram more into that table.

Do we really need the interval label/length for the observation and forecast in the obsfx_table when we provide the links to the metadata? It would free up space for a reference forecast column. Alternatively move the interval information to another table


Test mappings work properly after schema update

After schema update there might be issues with mappings. Cases to test:

[ ] New field added to schema
[ ] Field removed from schema
[ ] Field type changed in schema
[ ] New subschema added
[ ] Subschema removed (renamed)

This is related to #2095 ?

@vechocho No, it's not. This issues is about the Frictionless Data Package update, not DwC.


The future of Object.merge

Object.merge has had a bit of a complicated history. Although it seems straightforward enough to merge properties from one object into another, there are a number of different ways to do it, and things that can only be handled during the merge operation itself. Date.create has had a similar problem, and beginning in 2.0.0 an options object will handle some of the smaller edge cases when creating dates. I'm not a huge fan of options objects as it's hard to remember the names of options -- having them all over Sugar would be horrible. However, once you have enough arguments it becomes a similar problem to remember their order, and it's also a hassle to write out (false, false, true) just to get to the option you want. Object.merge is becoming like this. There are 2 options now ("deep" and "resolve") and I would like to add 2 more. I'll list them out here (abbreviating here and there for space):
Options
Deep Merging
This should be pretty obvious -- just recursively merging if properties themselves are objects. This is currently the 3rd argument and is false by default. Proposal is to stay false by default and become deep on the options object.
var o1 = { name: { first: 'Darth' } };
var o2 = { name: { last: 'Vader' } };
merge(o1, o2); ->  { name: { last: 'Vader' } }
merge(o1, o2, true); ->  { name: { first: 'Darth', last: 'Vader' } }
merge(o1, o2, { deep: true }); // with options

Resolve
When a conflict arises in a merge, which property should win? Right now the default is true, which means the source object's property wins -- in other words what you would expect from a standard merge. If this argument is false then the target's property wins, or what you would expect from a defaults method (side proposal: create a defaults method as a shortcut to this behavior). However this argument can also be passed a function:
var o1 = { name: 'Darth'  };
var o2 = { name: 'Vader' };
var fn = function (key, targetVal, sourceVal) {
  return targetVal + ' ' + sourceVal;
}
merge(o1, o2); ->  { name: 'Vader' }
merge(o1, o2, false, false); ->  { name: 'Darth' }
merge(o1, o2, false, fn); ->  { name: 'Darth Vader' }
merge(o1, o2, { resolve: fn }); // with options

The resolver function is intended to allow fine grained control over merge conflicts. However, right now it has some confusing behavior that can almost be considered buggy. First, deep merges will continue to step in to an object returned by the resolver function. This is confusing because the sprit of the resolver function is to allow the user to override standard merge behavior, so anything returned by the function should become the final value. Conversely, right now the resolver function will always use the return value of the function, even if that is undefined. I would like to change this so that the final value of the resolver function be used if defined. If undefined, then Object.merge will handle the merge in the default way. This will make it more intuitive for users to be able to step in and take control of merges for certain properties only, make it more aligned with the rest of Object.merge (as undefined properties are not merged normally), and also be in line with Lodash (not top priority, but nice). So (I'm squashing both behaviors here into one example):
Current Behavior:
var o1 = { user_count: 5, foo: 'bar', obj: { inner: 1 } };
var o2 = { user_count: 5, foo: 'baz', obj: { inner: 2 } };
var fn = function (key, targetVal, sourceVal) {
  if (key === 'user_count') {
    return targetVal + sourceVal;
  }  else if (key === 'obj') {
    // New object, something else entirely
    return { a: 'b' }
  }
}
// "foo" wasn't handled so it becomes undefined
// "obj" was passed a new object, but since it is a deep merge the
// result is traversed into. Since "inner" isn't handled it becomes undefined.
merge(o1, o2, true, fn); ->  { user_count: 10, foo: undefined, obj: { inner: undefined } }

Proposed behavior:
// "foo" wasn't handled so it gets handled in the default way
// "obj" was passed a new object, so this is respected and becomes the final value
merge(o1, o2, { deep: true, resolve: fn }); ->  { user_count: 10, foo: 'baz', obj: { a: 'b' } }

Non-enumerable Properties
Object.merge currently only operates on enumerable properties. This should be correct for most use cases, however there are times when merging non-enumerable properties is desirable as well. Proposal is to add this as an option.
Current behavior:
var o1 = { foo: 'bar' };
Object.defineProperty(obj, 'hiddenFoo', {
  value: 'hiddenBar',
  enumerable: false
});
var o2 = merge({}, o1);
console.log(o1.foo); // 'bar'
console.log(o1.hiddenFoo; // 'hiddenBar'
console.log(o2.foo); // 'bar'
console.log(o2.hiddenFoo; // undefined

Proposed behavior:
var o2 = merge({}, o1, { hidden: true });
console.log(o1.foo); // 'bar'
console.log(o1.hiddenFoo; // 'hiddenBar'
console.log(o2.foo); // 'bar'
console.log(o2.hiddenFoo; // 'hiddenBar'

As for the naming, I am proposing hidden because non-enumerable is difficult to write out, and enumerable: false does not mean the same thing. If there are other ideas for naming that would make this clearer, I'm open to them.
Merging Property Descriptors
Currently, only properties with an associated "value" are accessed and merged. As with non-enumerable properties, I think that this is fine as a default. However, ES5 introduced accessors in the form of property descriptors, and I think it's time that Sugar finally supported this as an option.
Current behavior:
var data = { label: 'foo' };
var o1 = {
  get label() {
    return data.label;
  },
  set label(value) {
    data.label = value;
  }
}
var o2 = merge({}, o1);
console.log(o1.label, o2.label);  // foo foo
data.label = 'bar';
console.log(o1.label, o2.label);  // bar foo (link created by setter is broken)

Proposed behavior:
merge({}, o2, { descriptor: true });  // produces a faithful merge of all accessors

As for naming descriptor seems to make sense, although I'm not sure about singular vs. plural. Also open to other naming options as well. Note that property descriptors are not supported in IE8 and below, so this option will do nothing in those environments. However, accessors can't be used in the first place, so if you need to support <=IE8 you won't need this option anyway.
Summary
To summarize the proposals here:

Using an options object instead of listed arguments
Adding an option for non-enumerable properties (off by default)
Adding an option for property descriptors (off by default)
Changing the resolve function to not go deep (respect the final value if returned)
Changing the resolve function to do a default merge if the final value is undefined
Adding a method called Object.defaults that is a shortcut to resolve: false.

Read carefully, looks really great. Both flexible and backward-compatible. Thanks!

So glad you're happy... I know you've been through a lot :D
What do you think about the Object.defaults idea? Would you use it?

What do you think about the Object.defaults idea? Would you use it?

I understand why did you name it .defaults, but it would be bit confusing for those who will read  code. Is there any short word in English, that means filling in the gaps ot smth like this? Maybe just Object.fill?

I understand why did you name it .defaults, but it would be bit confusing for those who will read code.

Well that's interesting. I'm usually the one going for more readable naming. However recently I feel I've been too haphazard about it. I'm all for readable, but if there is a name out there that really is more commonly accepted and guessable, then I think it should win. I feel that defaults is like that, but I may be wrong. Looking at it I can mostly only find Underscore/Lodash that uses it.
But hold on a second. Actually I also wanted to bring up another idea. Merging multiple objects together is also a common operation, so I was thinking that there also may be a need for Object.mergeAll or possibly another option on merge telling it to combine all array objects together.

What would you think of a mergeAll?
If we added that, it would probably remove the need for defaults. Ordering the objects a certain way would decide which one wins (this is more jQuery style)

I'm usually the one going for more readable naming

Indeed, Sugar naming is very nice (.has is especially cute :). But as for .defaults – it‘s not only my opinion, that‘s why I‘ve taken a little pause. I‘ve asked 3 guys about it, showing em code. Only one gave partially valid answer. It‘s quite weak argument, but it shows realities.
As for me, .mergeAll looks definitely much more clear, despite of camelCase.

Only one gave partially valid answer. It‘s quite weak argument, but it shows realities.

No, I think it's great.. I especially want real opinions. But are you saying that only one liked defaults, and two didn't like it?

As for me, .mergeAll looks definitely much more clear, despite of camelCase.

I agree... and I also agree about the camelCase, like to keep it one word when possible but unfortunately it's not always possible.
So here's another question (I never stop) ... so if you had Object.mergeAll([a,b,c,d,e]) then should it merge everything into a or into a new object?

Seems .mergeAll is an array method, not object. So [a,b,c].mergeAll(), not Object.mergeAll([a,b,c]). As for me, Object.mergeAll() should take plain objects as arguments.

should it merge everything into a

Yes, first element acts as accumulator. If one wants new object, let him write [{}, b, c].mergeAll();

So [a,b,c].mergeAll(), not Object.mergeAll([a,b,c])

Ah, well I wasn't expecting that. That's definitely not a direction I'd like to take this. Merging objects together is something that should be a responsibility of the Object class/package and I definitely want to keep it there for various reasons. Also, I don't think it makes sense for a method to become an array method just because it accepts arrays.

Yes, first element acts as accumulator. If one wants new object, let him write [{}, b, c].mergeAll();

I tend to agree with this though.

I don't think it makes sense for a method to become an array method just because it accepts arrays

Can‘t understand this, it surely is array method ) You don‘t write Object.map() or Object.reduce() for arrays – and.mergeAll is, actually, kinda reduce.

it surely is array method

Well it is in the sense that it could be mapped onto an array, but those other methods are performing operations on arrays, whereas this is all about performing operations on objects, which is the main difference. Of course you're right that reduce is a bit different, but it's kind of the odd-man on the Array class and it isn't doing anything more important than a series of callbacks with an accumulator. Object.mergeAll is doing merge operations on the objects themselves. The array is completely unimportant to the implementation, it's just a holder. For example it could take enumerated arguments just as easily (Object.mergeAll(a, b, c)), it makes no difference, really.
For another example, Date.UTC accepts a list of parameters but that doesn't mean you would make Array#createDate simply because it's passed a list. The point is about operating on dates.

.mergeAll([obj1, obj2, objn]) is definitely fold of a list, that is denoted as explicit array – by proposed syntax.
If you proposed Object.mergeAll( obj1, obj2, objn ) – I would never say it‘s an array method, despite it‘s also fold.
Date.UTC example is not relevent in this case – it does not take array, it takes list of args.
Although, since Sugar is your lib, and I like it as is – I‘m stopping here, just to avoid ping-pong. Do what you think is reasonable )

For me, list of args or array are mostly an implementation detail and in any case not relevant to the question of where a method should exist. Object.mergeAll, if it's added at all, will definitely exist on Object because it is all about merging objects together. This is for both because it's conceptually more obvious, but also for technical reasons as the code required for this lives in the object package and it can't exist in both.

Suggestion for resolver function:
var fn = function (key, targetVal, sourceVal, undefined) { // note the extra parameter
  if (key === 'user_count') {
    return targetVal + sourceVal;
  }  else if (key === 'obj') {
    // New object, something else entirely
    return { a: 'b' }
  } else if (key === 'obj2') {
  return undefined; // assigns the value ``undefined``
  }
//return void 0; // ignores the property; statement may be left out
}

Internally, the resolver function could be called with a 4th parameter, a private sugar object, so that if the result of the resolver function strictly equals to it, then the property value will be undefined, rather than ignored. Of course, if the Object.merge user doesn't want this behavior, they would simply pass a function with length 3.
This would allow fine-grained control.over the behavior of undefined.

That's an interesting idea. Note that I'm also planning on adding the target and source objects to the arguments to give more control. Another possibility is that instead of undefined, it could be the inverse and the passed object could be a way to tell it to continue the merge, and undefined returned would actually be undefined. I like your idea better though.
However all that said, I think I'm ok with Sugar taking a hard stance that undefined properties are equivalent to non-existent properties, and that this use case does not have to be directly handled. null exists instead when something needs to be defined, but no value. If this use case absolutely has to be supported then it can be worked around by directly setting undefined on the target argument once that is passed.

OK this one is now finally locked down. First, there will be 2 new methods, mergeAll and defaults. If mergeAll sees an array for source it simply treats it as a list of objects to be merged into target instead of merging properties of the array itself (0, 1, etc). defaults is the same as mergeAll but sets the resolve option to false, meaning the properties of target will win if defined. All 3 methods are on the Object class and are static methods, however are also mapped onto extended objects (Object.extended()). Options for all 3 methods are the same. I'll let the shiny new docs describe:
deep:        If `true` deep properties are merged recursively.
             (Default is `false`)

resolve:     Determines which property wins in the case of conflicts.
             If `true`, <source> wins. If `false`, <target> wins. If a
             function is passed, its return value will decide the result.
             Any non-undefined return value will resolve the conflict
             for that property (will not continue if `deep`). Returning
             `undefined` will do nothing (no merge). Finally, returning
             the global object `Sugar` will allow Sugar to handle the
             merge as normal. Resolve functions are passed the arguments:
             `key`, `targetValue`, `sourceValue`, `target`, `source`.
             (Default is `true`)

hidden:      If `true`, non-enumerable properties will be merged as well.
             (Default is `false`)

descriptor:  If `true`, properties will be merged by property descriptor.
             Use this option to merge getters or setters, or to preserve
             properties like `enumerable`, `configurable`, etc.
             (Default is `false`)

@Slayer95 Thanks for your input here. In the end I went with a slightly modified form of your idea. In the end the motivation was the fact that there was no way to bail out of merging a property in a resolve function as returning undefined meant "merge as normal". I feel that returning the global Sugar object is much more intuitive as it is telling the result to "let Sugar handle the merge".

@andrewplummer You are welcome! Looking forward to 2.0 :)


ArraySlice.flatMap and ArraySlice.reduce test cases

Adds ArraySlice test cases copied from Array.reduce and Array.flatMap.
ArraySlice.flatMap had to be adjusted slightly to flatten an array of arrays, per Mozilla's test cases. That change is compatible with the existing ArraySlice.flatMap test. Are we OK with that difference in behavior between ArraySlice.flatMap and Array.flatMap? Is aligning with Array.flatMap desirable?

Is aligning with Array.flatMap desirable?

👍 I think it makes sense to align to how the standard Array flatMap implementation works.

Looks like the Mozilla code is permitted without license as it is public domain, so from legal perspective this is fine in terms of using those tests (just mentioning this so that others looking at PR are aware):

No licensing notice is necessary, but if you need one, you can use: "Any copyright is dedicated to the Public Domain. http://creativecommons.org/publicdomain/zero/1.0/".

I'm approving the changes, I'm going to hold off merging right now as it would be a breaking change to the interface (if there is anyone whom is relying on the existing behaviour).
I'll need to figure out which release this will end up in and the release coordination (since minim, fury and the adapters are currently tied to very specific versions).

in the spirit of aligning to Array.flatMap, should i drop the original source line and update the existing flatMap test?

This should end up in v0.21.0


keep nodes toggle state and optims

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Add scaling-bitcoin-with-the-erlay-protocol to bitcoin-magazine/bitcoin-explained

This PR adds Scaling Bitcoin With The Erlay Protocol transcript to the bitcoin-magazine/bitcoin-explained directory.

Remarkable work once again, @davidgumberg! Your consistent and high-quality editing truly shines through in this submission as well. Your dedication to refining transcripts is admirable. I added some minor fixups that you can see here. Additionally, the feedback points I previously shared for your first review also apply to this submission.
Looking forward to more of your impressive reviews! Your consistent high-quality work sets a fantastic example for all submissions.


G2Loader.py Test improvement for Data Source

If the DATA_SOURCE does exist, I would like to have the results of "Test" mode (e.g ./G2Loader.py -f NPI_Orgs_LasVegas.csv/?DATA_SOURCE=NPI_ORGANIZATION,file=csv -T) return that the data source is needed or will be in created (default mode).
This will help when typos are made, right now if I input records with a data source of CUSTOMER and when I come back another time to add an additional data under that source and run test before, it all looks good. Even though I may have defined the new data set as a source of CUSTOMERS. Now because of a simple typo and G2Loader.py not pointing it out, I have two different data sources even though I really only wanted one.

See #71


failing test in travis

I'm investigating failing builds in travis, and noticed https://travis-ci.org/sous-chefs/nrpe/builds/262477223
Most tests are passing, but for UNIT_AND_LINT=1:
source install on ubuntu 16.04
  templates systemd unit file (FAILED - 1)
Failures:
  1) source install on ubuntu 16.04 templates systemd unit file
     Failure/Error: expect(chef_run).to render_file('/lib/systemd/system/nrpe.service').with_content('/usr/sbin/nrpe')

       expected Chef run to render "/lib/systemd/system/nrpe.service" matching:

       /usr/sbin/nrpe

       but got:

We should probably switch this over to using the systemd_unit resource in 12.11+. It works a lot better than the current templating setup


Changing font etc?

Hi,
Is there a supported way to change fonts and generally load custom css without having to edit the theme?

never mind, found this from reading source:
[params]
   [params.globalFrontmatter]
      localCss = ["css/custom.css"]

Then create custom.css in assets/css


docker: use Digest instead of string

Hello @maiamcc, @jazzdan,
Please review the following commits I made in branch nicks/ref:
64bdcea9cbbffd98c9117bcfad7a079886c14350 (2018-08-10 14:02:35 -0400)
docker: use Digest instead of string

this was integrated in my pr