builtin semver 2.0.0 support

This PR adds builtin support for parsing and ordering semver strings:
parse_semver function:
parses semver strings according to semver regex
Taking the example valid semver strings this would look something like this:
cat /tmp/test.json | jaq
[
  "0.0.4",
  "1.2.3",
  "10.20.30",
  "1.1.2-prerelease+meta",
  "1.1.2+meta",
  "1.1.2+meta-valid",
  "1.0.0-alpha",
  "1.0.0-beta",
  "1.0.0-alpha.beta",
  "1.0.0-alpha.beta.1",
  "1.0.0-alpha.1",
  "1.0.0-alpha0.valid",
  "1.0.0-alpha.0valid",
  "1.0.0-alpha-a.b-c-somethinglong+build.1-aef.1-its-okay",
  "1.0.0-rc.1+build.1",
  "2.0.0-rc.1+build.123",
  "1.2.3-beta",
  "10.2.3-DEV-SNAPSHOT",
  "1.2.3-SNAPSHOT-123",
  "1.0.0",
  "2.0.0",
  "1.1.7",
  "2.0.0+build.1848",
  "2.0.1-alpha.1227",
  "1.0.0-alpha+beta",
  "1.2.3----RC-SNAPSHOT.12.9.1--.12+788",
  "1.2.3----R-S.12.9.1--.12+meta",
  "1.2.3----RC-SNAPSHOT.12.9.1--.12",
  "1.0.0+0.build.1-rc.10000aaa-kk-0.1",
  "99999999999999999999999.999999999999999999.99999999999999999",
  "1.0.0-0A.is.legal",
  "3.0.0-rc.1",
  "3.0.0-rc.2",
  "3.0.0-rc.3",
  "3.0.0"
]

cat /tmp/test.json | jaq 'map(. | parse_semver)' | [.[14]]
[
  {
    "major": "1",
    "minor": "0",
    "patch": "0",
    "prerelease": "rc.1",
    "buildmetadata": "build.1"
  }
]

semver_ord function:
use parse_semver to create a nested array sortable by sort_by function, this is done
by transforming the version into the following structure:
[ [.major, .minor, .patch], [ .prerelease_name, .prerelease_version_number] ]

if there is no prerelease then the instead of the second array there will be {}.
buildmetadata is ignored for ordering
to construct the second array another regex is used which can be found here. It is what i came up with, if you think it should be adjusted feel free to play with it ;)
the effect of this is that versions get orderd by their [major, minor,  patch] first. prereleases are ordered lexicographically first and then according to their subversion

applying the ordering to the above list of semver strings would look like this:
cat /tmp/test.json | ./target/debug/jaq 'sort_by(. | semver_ord) | reverse'
[
  "99999999999999999999999.999999999999999999.99999999999999999",
  "10.20.30",
  "10.2.3-DEV-SNAPSHOT",
  "3.0.0",
  "3.0.0-rc.3",
  "3.0.0-rc.2",
  "3.0.0-rc.1",
  "2.0.1-alpha.1227",
  "2.0.0+build.1848",
  "2.0.0",
  "2.0.0-rc.1+build.123",
  "1.2.3",
  "1.2.3-beta",
  "1.2.3-SNAPSHOT-123",
  "1.2.3----RC-SNAPSHOT.12.9.1--.12",
  "1.2.3----RC-SNAPSHOT.12.9.1--.12+788",
  "1.2.3----R-S.12.9.1--.12+meta",
  "1.1.7",
  "1.1.2+meta-valid",
  "1.1.2+meta",
  "1.1.2-prerelease+meta",
  "1.0.0+0.build.1-rc.10000aaa-kk-0.1",
  "1.0.0",
  "1.0.0-rc.1+build.1",
  "1.0.0-beta",
  "1.0.0-alpha0.valid",
  "1.0.0-alpha.beta",
  "1.0.0-alpha.beta.1",
  "1.0.0-alpha.0valid",
  "1.0.0-alpha-a.b-c-somethinglong+build.1-aef.1-its-okay",
  "1.0.0-alpha+beta",
  "1.0.0-alpha",
  "1.0.0-alpha.1",
  "1.0.0-0A.is.legal",
  "0.0.4"
]

If you put the module in a repo you could add a link to it at https://github.com/fiatjaf/awesome-jq?tab=readme-ov-file#libraries-and-tools-for-jq-itself. There are some jq module repositories but don't know how much they are used

Hi guys, I experimented doing the same thing with the original jq. But I have found that the regex implementation of the original is not working correctly. There are already open issues for this, and they have been open for a while. So that is why i turned to jaq
So doing this is just not possible with the original 'jq' until they fix their regex parsing.
I get your policy of not adding thing not present in the original, sadly this will mean that I have to maintain my own fork for my purposes, to have this working easily in CI environments. And that people how would like to use have to do extra work to benefit from the work I put into this.

@ju6ge i see, you could you point me to the regex issue?

Ok, the error seem to be about (?P<name>) named capture groups or? but i don't see how they differ from just (?<name>)? if i change parse_semver to use (?<name>) it seems to work better:
$ jq -n -L . 'include "semver"; "1.2.3-4+meta" | parse_semver'
{
  "major": "1",
  "minor": "2",
  "patch": "3",
  "prerelease": "4",
  "buildmetadata": "meta"
}

@wader thanks for the pointer … so with some adjustment I could make it work for jq \o/


Fix(eos_designs): network services vlan interfaces ospf authentication message-digest

Change Summary
Fix missing "ip ospf authentication message-digest" configuration on vlan interfaces when "authentication: message-digest" is defined
adding
Related Issue(s)
Fixes #2726
Component(s) name
`arista.avd.eos_designs
Proposed changes
add "vlan_interface_config["ospf_authentication"] = ospf_authentication" at line 148 or vlan_interfaces.py python module
Checklist
User Checklist

N/A

Repository Checklist

[x] My code has been rebased from devel before I start
[x] I have read the CONTRIBUTING document.
[ ] My change requires a change to the documentation and documentation have been updated accordingly.
[x] I have updated molecule CI testing accordingly. (check the box if not applicable)

I will augment a molecule test to cover this scenario

I added the molecule test


Fix a header.

Remove an unwanted character.

@alexandair : Thanks for your contribution! The author, @cynthn, has been notified to review your proposed change.

@alexandair thanks for the fix! #sign-off


[Question] Syntax problem?

What is the syntax of the code?
[self addSubview:({

        UIScrollView *scrollView = [[UIScrollView alloc] initWithFrame:CGRectMake(0, 0, scrollW, scrollH)];

        [self addSubviewToScrollView:scrollView];

        scrollView.delegate                       = self;
        scrollView.scrollsToTop                   = NO;
        scrollView.pagingEnabled                  = YES;
        scrollView.showsHorizontalScrollIndicator = NO;
        scrollView.contentOffset                  = CGPointMake(scrollW, 0);
        scrollView.contentSize                    = CGSizeMake((self.count + 2) * scrollW, 0);

        self.scrollView = scrollView;
    })];

C++ Lambda syntax? If remove the self.scrollView = scrollView; compiler to the error. The return value is not automatically speculated?

In fact, I don't really know its syntax name, I also think it's a lambda syntax. And we have to return a right value in this func:)

For me, self.scrollView = scrollView; as a return value of the syntax is unfamiliar, can provide information see more? : ]

scrollView;
发自我的 iPhone

在 2016年11月13日，下午5:34，Lavare<EMAIL_ADDRESS>写道：
For me, self.scrollView = scrollView; as a return value of the syntax is unfamiliar, can provide information see more? : ]
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.


Fatal error: Cannot use 'Object' as class name as it is reserved

What steps will reproduce the problem?
install php 7.2.x
composer create-project --prefer-dist yiisoft/yii-app-advanced adv
What is the expected result?
open browser and see welcome yii2 page.
What do you get instead?
Fatal error: Cannot use 'Object' as class name as it is reserved in /usr/local/var/www/adv/vendor/yiisoft/yii2/base/Object.php on line 77
Additional info

Q
A

Yii version
2.0.12

PHP version
7.2.0beta3

Operating system
Darwin macOS 10.12.6

already fixed by commit https://github.com/yiisoft/yii2/commit/69673c0c94679ba3f38829cf44cb001d9186621d.
see issue https://github.com/yiisoft/yii2/pull/12699

Got the same issue, does this mean 2.0.12 dev will now have the object renamed to base object and all we have to do is upgrade to yii 2.0.12 dev?

Yes. That's the case when we can't do it in any other way.

noted. will wait for the release. in the meantime, I am asking the server
admin to down grade to 7.0.23. I prefer not to mess with the core
On Sat, Sep 30, 2017 at 6:19 PM, Alexander Makarov<EMAIL_ADDRESS>
wrote:

Yes. That's the case when we can't do it in any other way.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/yiisoft/yii2/issues/14823#issuecomment-333299125, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AQYmd_0drh3lWxSQbNBK5tAgmuvLd_njks5snhW_gaJpZM4PYUly
.

--
--
Lionel Yeo
Chartered Alternative Investment Analyst SM
Partner And Developer
Wesvault
PaperlessCloud
Website: www.wesvault.com
paperlesscloud.wesvault.com
Main-SG:  +65 6521 2851
HP         :   +65 9667 4603
Bronze Sponsor
Seapex 2014
www.seapex.org

I've upgraded to PHP 7.2.2 and also get error in my app when try to login:
yii\base\ErrorException: Cannot use 'Object' as class name as it is reserved in \vendor\yiisoft\yii2\base\Object.php:28
Stack trace:
#0 [internal function]: yii\base\ErrorHandler->handleFatalError()
#1 {main}

Yii version is <IP_ADDRESS>

You have somewhere code, that uses class Object

@alexander-ronin
If your application is based on yii2-app-basic change the requirement in your composer.json to  "yiisoft/yii2": "~2.0.14". Read https://github.com/yiisoft/yii2/blob/master/framework/UPGRADE.md.
Change Object to BaseObject everywhere in your application.
Probably this will fix the issue.

@schmasterz no, I have Advanced version.

Change Object to BaseObject everywhere in your application.

I didn't found any declared Object in my app except vendor/yiisoft/yii2/base/Object.php.
The only help is downgrade PHP version to 7.0.27

I think Yii2 needs to push an update where the rename object to base object. This issue has been outstanding since Last Jan.

@auzadventure Can you link the issue?
Actually the rename took part from 2.0.12 to 2.0.13 - https://github.com/yiisoft/yii2/blob/master/framework/UPGRADE.md#upgrade-from-yii-2012

@SilverFire Sorry, you was right. I've found usage of Object in mdmsoft/yii2-admin extension.
It's strange situation: As I see, @mdmunir replaced Object usage in his extension, but it doesn't want to update.
I've tried to clear composer cache, removed vendor folder, reinstall mdmsoft/yii2-admin, but useless. Anyway I get version of this extension without last commit.

@alexander-ronin Try requireing it as dev-master#e9cc1d8adc as 2.8-p1 in composer.json`...

@schmunk42 yes, you're saved me!
All errors are gone, thanks a lot =)

Fatal error: Cannot use 'Object' as class name as it is reserved in G:\xampp\htdocs\humhub-1.2.4\humhub-1.2.4\protected\vendor\yiisoft\yii2\base\Object.php on line 77
Please respond to this issue ??

@kahoul See https://github.com/yiisoft/yii2/issues/14823#issuecomment-369930685 or don't use PHP 7.2

You should configure your PHP installation so that it meets the minimum requirements of Yii. Most importantly, you should have PHP 5.4 or above.

https://www.yiiframework.com/doc/guide/2.0/en/start-installation

I was having this problem in the yii2-admin module. I solved this, editing the files that was calling Object instead of BaseObject.
The files are:

vendor\mdmsoft\yii2-admin\components\Configs.php
vendor\mdmsoft\yii2-admin\models\Assignment.php
vendor\mdmsoft\yii2-admin\models\Route.php

Rename folder from Object.php to BaseObject.php
and on vile also change class Object to class BaseObject

@alievinfo
Good to know. Thanks!

http://prntscr.com/jm0l0k
What is the expected result?
open browser and see welcome yii2 page.
What do you get instead?
Fatal error: Cannot use 'Object' as class name as it is reserved in yii\ipaddress\project\vendor\yiisoft\yii2\base\Object.php on line 77
can any one help me, solve for this error.

@meddev001 Please read through this issue, especially my comment from 26 Mar.

Thanks but I use humhub 1.2.7. mdmsoft folder does not exist at all...
Any way to solve it for humhub?

Why can't we have a patch merge update to Yii2 so that we can use PHP 7.1+ the issue is not complicated, Object is now a protected word in 7.1+ so it can be renamed to BaseObject, but the community needs an official patch to this. I've posted when it was 7.0.29 and decided to just stay at 7.0.

We would like this as well.

Why can't we have a patch merge update to Yii2 so that we can use PHP 7.1+

Yii 2 is compatible with PHP 7.2 and above since 2.0.13 (see https://github.com/yiisoft/yii2/blob/master/framework/UPGRADE.md#upgrade-from-yii-2012)

This issue should be locked. Compatibility with PHP 7.2 was fixed almost a year ago. If someone still experiences this bug, this means that:

He's using outdated Yii and should upgrade.
He's using extension which uses deprecated Object class - this is extension fault and it should be upgraded or fixed to use BaseObject instead of Object.
He's using Object directly in application code - this is application fault and it should be fixed to use BaseObject instead of Object.

None of this can be fixed in Yii itself.


Is this paper accepted?

Is this paper accepted at any conference/journal?

Hi. At this time, the paper has not been accepted to any conference or journal. I will (likely) update the repo if the paper's status changes.


add snippets for polymerfire

It would be pretty nice to have snippets for polymerfire as well to speed up development with firebase + polymer.
Specifically for :
firebase-auth
firebase-app
firebase-document
firebase-query
firebase-storage (upcoming)
Anyway, thanks a lot for this awesome package!

Closing per #47.


optimize window drag

Hi all! I've just bought Mad Productivity on itch.io, and I really like this so far.
I've noticed that dragging the window is extremely slow and on Windows the lag is pretty bad.
Here's a quick fix to make it smooth. Handling OS operations directly to the window is pretty resource-intensive in Godot as I've already experienced with some UI customization projects.
Here's how I'm handling it in one app of mine, thanks to the team I've achieved a smooth lag-less dragging on most of desktop OS (windows, ubuntu and macOS), and I think that this implementation would be really useful for Mad Productivity UX.
I've also tried to fixed the resizing on dragging while the window is maximized, to keep the aspect ratio between the mouse position and the window's one.
Before (release mode, in debug mode the lag is even worse)
https://user-images.githubusercontent.com/45271396/141540473-3744d490-d46c-4c83-ac69-2c28c2d724b1.mp4
After (debug mode and recording the screen drops some frames, in release mode performance should be improved!)
https://user-images.githubusercontent.com/45271396/141540503-7e7cc41a-9f12-47ed-ab60-372b754cc738.mp4
Hope you find it useful.
Cheers!

Compared to the previous implementation this is indeed much faster and smoother! Thanks! The only downside is that it doesn't limit the movement according to the window dimensions and can therefore allow the user to move the window slightly too far off screen.

@lentsius-bark  It's a common behaviour for all windows, what the OS will automatically prevent is to move the cursor outside of the screen boundaries while dragging, so it is really hard by default to loose your window. Anyway here's a quick fix to specifically achieve what you wanted.
Notice that some calculations need to be done, since it's really hard to know what boundaries to use, since you don't know what's the min position and max size to stay in because users may have multiple screens with different positions
For example I've got two screens of different sizes, the main one is on the center (1920x1080) and the secondary screen is on my left (2400x1350), but I could have the main one on the left, and the secondary in the center, or eventually a third one.
I've added an rough calculation on the boundaries, and it works fine for now

@fenix-hub I gave this a thought and you're right, that is the normal behaviour for windows. I realized I had implemented the limit because the previous dragging hid the mouse and used event.relative to move it hence the user could move it way out.
Due to the improved window drag that shouldn't be necessary now. Hence I propose merging the previous commit without window limits, what say you? :) Again thanks for the contribution!

@lentsius-bark Reverted to #fcdda6c :)

Thanks!


Superfluid - Corner case fixes + spec improvements

Closes: #XXX
Description

For contributor use:

[ ] Targeted PR against correct branch (see CONTRIBUTING.md)
[ ] Updated relevant documentation (docs/) or specification (x/<module>/spec/)
[ ] Added a relevant changelog entry to the Unreleased section in CHANGELOG.md
[ ] Re-reviewed Files changed in the Github PR explorer

Codecov Report

Merging #921 (1df2fcc) into main (320cb4a) will increase coverage by 0.65%.
The diff coverage is 85.71%.

@@            Coverage Diff             @@
##             main     #921      +/-   ##
==========================================
+ Coverage   20.32%   20.97%   +0.65%
==========================================
  Files         192      192
  Lines       24904    25220     +316
==========================================
+ Hits         5061     5291     +230
- Misses      18917    18969      +52
- Partials      926      960      +34

Impacted Files
Coverage Δ

x/superfluid/keeper/stake.go
58.79% <85.71%> (+10.86%)
:arrow_up:

x/lockup/keeper/utils.go
90.19% <0.00%> (-5.36%)
:arrow_down:

x/superfluid/keeper/slash.go
100.00% <0.00%> (ø)

x/lockup/keeper/msg_server.go
24.50% <0.00%> (+0.18%)
:arrow_up:

x/incentives/keeper/distribute.go
62.66% <0.00%> (+1.61%)
:arrow_up:

x/lockup/keeper/synthetic_lock.go
75.62% <0.00%> (+2.04%)
:arrow_up:

x/lockup/keeper/invariants.go
38.46% <0.00%> (+2.09%)
:arrow_up:

x/superfluid/keeper/twap_price.go
85.71% <0.00%> (+4.76%)
:arrow_up:

x/lockup/keeper/lock.go
58.19% <0.00%> (+6.49%)
:arrow_up:

Continue to review full report at Codecov.

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

Can you fix the conflicts, then go ahead and merge?


Sia "v3" will not parse correctly

I am using a Scantronic (Eaton/Cooper) alarm and am getting the following error in HA generated by pssialarm
2020-09-29 20:44:45 DEBUG (MainThread) [pysiaalarm.aio.sia_server] Incoming line: 1908002B"SIA-DCS"0000L0#3080[#3080|Nti19:44/id1/RP] 2020-09-29 20:44:45 DEBUG (MainThread) [pysiaalarm.base_sia_server] Parsed event: Content: #3080|Nti19:44/id1/RP], Zone: , Code: Nt, Message: i19:44/id1/RP, Concerns: None, Type: None, Description: None, Account: 3080, Receiver: None, Prefix: L0, Timestamp: None, Length: 002B, Sequence: 0000, CRC: 1908, Calc CRC: 1908, Message type: None, Encrypted Content: None, Full Message: "SIA-DCS"0000L0#3080[#3080|Nti19:44/id1/RP].. 2020-09-29 20:44:45 WARNING (MainThread) [pysiaalarm.base_sia_server] Code not found, replying with DUH to account: 3080
As described in HA forum here
https://community.home-assistant.io/t/unknown-event-code-handling-with-sia-integration/230885/3?u=jamesarbrown
The alarm has direct ARC reporting as follows

The report types available are: Fast Format, Contact ID, SIA 1, SIA 2, Scancom SIA 3, Extended SIA 3 and Extended SIA3 v2.

Where SIA2 does work with pysialarm, but the issue for me is to enable the Secureconnect Eaton cloud, the cloud takes over the ARC reporting and that only offers the SIA3, ex SIA 3 and SIA 3. The above is with SC SIA3. Later I will try others.
I dont know if it helps but here is the Excerpt from the Eaton engineering guide about the SIA types

SIA 1, SIA 2, SIA 3 and Extended SIA 3
When using the SIA formats the control unit transmits data from the event log to the ARC.
The four SIA formats differ in the amount of data transmitted with each message:
Type Format
SIA1: #AAAAAA|NCCcc
SIA2: #AAAAAA|Nidnnn/rinn/CCcc
SIA3: #AAAAAA|Ntihh:mm/idnnn/rinn/CCcc
#AAAAAA|AS
Extended SIA
3:
#AAAAAA|Ntihh:mm/idnnn/rinn/CCcc/AS
Where:
AAAAAA 6-digit programmable account code (e.g. 123456).
“N” New Event (always N).
“ti”hh:mm/ time (e.g. ti10:23/).
“id”nnn/ user number, if applicable; otherwise not sent (e.g.
id123/ or id6/).
“ri”nn/ partition no. (e.g. ri12/ or ri3).
CC event code (e.g. FA = Fire Alarm).
cc zone number, if applicable; otherwise not sent
(e.g. 23 or 5).
“A”S text description of event, usually the log event
description.
(The control unit sends those characters shown between “ and “ above literally as they
appear in the text.)
For example, if there is a fire alarm on zone 2 of partition 4 at 10:15 (partition 4 account
number is 10), the message would be:
SIA1: #000010|NFA2
SIA2: #000010|N/ri4/FA2
SIA3: #000010|Nti10:15/ri4/FA2 #000010|AFire Zone 2
Extended SIA3: #000010|Nti10:15/ri4/FA2/AFire Zone 2
Extended SIA3 V2
Some versions of the software that works in SIA receivers do not always recognise the “/”
text delimiter. This can cause problems with spurious “Mains Fail” messages appearing to
the ARC when the control unit sets, unsets or goes into or out or installer mode.
In version 2 of Extended SIA3 the “/” delimiter has been replaced by a “|”. So, for example,
the string:
#000010|Nti10:15/ri4/FA2/AFire Zone 2
becomes:
#000010|Nti10:15|ri4|FA2|AFire Zone 2
If you have experienced problems with spurious “Mains Fail” messages try using this
Extended SIA3 V2

Hi @jamesarbrown thanks for this, I didn't know there were already multiple extended versions of the format, what we could do is make that configurable in the underlying package and then a setting in the HA component, would just mean using a different parsing REGEX here. So should be doable, but requires some testing and fiddling!

Hi @eavanvalkenburg, if you wanted to put that in a testing branch I can try, test and tweak as necessary. I may try and mess with the regex, but at the moment have many projects on the go

I Have the same problem with SIA3 with textblock, my SIA looks like this"SIA-DCS"1994R000001L010000#123455[#123455|Nri1/id1/OP^Skalskyddsdetektorer Ben^][MA4580F942489]_18:24:42,11-09-2020

I've created a new pre-release that should be able to do all this, please have a look! It's version 2.0.9b2

I've created a new pre-release that should be able to do all this, please have a look! It's version 2.0.9b2

Hey @benilsson and @jamesarbrown have you guys had a chance to look at the latest pre-release?

Hey @benilsson and @jamesarbrown have you guys had a chance to look at the latest pre-release?

I seem to be unable to recall how i installed SIA the first time... memory says it was a community integration, but can no longer find.
Running a ha docker image
Will keep trying

I seem to be unable to recall how i installed SIA the first time... memory says it was a community integration, but can no longer find.
Running a ha docker image
Will keep trying

Yeah its in the HACS store!

Yeah its in the HACS store!

Sorry, had to update HA, seems HACS had gone semi offline. But not seeing the beta you refer

Sorry, had to update HA, seems HACS had gone semi offline. But not seeing the beta you refer

I was referring to the version of the underlying package, but that is included in the beta you see there

I was referring to the version of the underlying package, but that is included in the beta you see there

I have it running, just not had chance to trigger an alarm to test. Ultimately would like to add more codes, such as armed and unarmed and these would make testing much easier

@jamesarbrown I've just released a new version of the component which works very differently, it uses the eventbus in HA to tell the built-in component that a message has arrived, each of the components then checks if it should respond to that message, which is really nice, because it also allow you to create automations with triggers from those same events!

Presume this is 4beta1
Sorry, I cant get it to work. Eaton seem to have gone for either no encryption or 128bits, whether that is the issue not sure. but what ever, their server connects and complains there is an issue, so its not getting an expected response.
from HA
2021-02-14 20:02:15 DEBUG (MainThread) [pysiaalarm.aio.sia_server] Incoming line: 1AB000B5"*SIA-DCS"0000L0#123[72217fb5939734a6195309fc60a94be1f9ce5df9ddcd1372ebd5be1930415850a05aeb5417f10756e
ef165d7acbce2fdbf49b82c5edfea164285eb69b255a987b89ce90c90ea2e38d46d528886d4878b
2021-02-14 20:02:15 DEBUG (MainThread) [pysiaalarm.base_sia_server] Parsed event: Content: None, Zone (ri): None, Code: None, Message: , Concerns: None, Type: None, Description: None, Account:
 123, Receiver: None, Prefix: L0, Timestamp: None, Length: 00B5, Sequence: 0000, CRC: 1AB0, Calc CRC: 1AB0, Message type: None, Encrypted Content: 72217fb5939734a6195309fc60a94be1f9ce5df9ddcd1
372ebd5be1930415850a05aeb5417f10756eef165d7acbce2fdbf49b82c5edfea164285eb69b255a987b89ce90c90ea2e38d46d528886d4878b, Full Message: "*SIA-DCS"0000L0#123[72217fb5939734a6195309fc60a94be1f9ce5df9
ddcd1372ebd5be1930415850a05aeb5417f10756eef165d7acbce2fdbf49b82c5edfea164285eb69b255a987b89ce90c90ea2e38d46d528886d4878b..
2021-02-14 20:02:15 WARNING (MainThread) [pysiaalarm.base_sia_server] Code not found, replying with DUH to account: 123

@jamesarbrown  just published a new beta (3.0.0b1) which has many improvements, please give it a try!

To bad i cant get it to work i would love to see events of my alaam panel.
Logger: pysiaalarm.base_server
Source: /usr/local/lib/python3.8/site-packages/pysiaalarm/base_server.py:100
First occurred: 16:18:41 (47 occurrences)
Last logged: 16:18:46
Last line could not be parsed as a SIAEvent or OHEvent, line was: F#123456
tried different SIA levels but same problem

yeah, that is a much to short message for anything SIA (including other versions) so not sure what happens with that @bjkoole

yeah, that is a much to short message for anything SIA (including other versions) so not sure what happens with that @bjkoole

II’m a installer of alarmsystems  and connect these systems almost daily and able to see the messages in the reciever at the ARC and pretty sure its a complete SIA3. I had my panel set up with a ARC and it works fine so dont know what could be wrong but pretty sure the SIA code is good.

So you're saying you get more than just F#123456 when using another system?

So you're saying you get more than just F#123456 when using another system?
  I will try it with a different ethernet module later this week to see how that goes. Im also looking for a complete solution to integrate my panel in HA so i can also see output states but there is not that much to find.

So you're saying you get more than just F#123456 when using another system?

btw I think its a poll message cause when I change the IP adres in my alarm panel The systems gives a connection fail. When i change it back to my HA ip the connection fail message restores.

@eavanvalkenburg how do I change in SIA integration where it read the code? It says I must change in the pysiaalarm but where I find it?

@eavanvalkenburg how do I change in SIA integration where it read the code? It says I must change in the pysiaalarm but where I find it?

@benilsson I'm not sure what you mean?

@eavanvalkenburg
my sia looks like this
"SIA-DCS"1994R000001L010000#123455[#123455|Nri1/id1/OP^Skalskyddsdetektorer Ben^][MA4580F942489]_18:24:42,11-09-2020
It means that the integration reads that the code is "id" and not "OP".

@jamesarbrown @benilsson @bjkoole are you guys still having issues? The latest HA version has the latest version of the package so please test with that and let me know, otherwise lets close this issue!

Hey,
I am having issues with the eaton/Scantronic I-on40H.
I am using SIA 2 from the Scantronic  and the settings below.

Any advice on how to test if i am getting communication would be much appreciated.
Edd

Haven't heard back, so closing this one.


CWE-151222: ispw/git cli logging improvements

https://agile.compuware.com/browse/CWE-151222

wrong pull request. original project has been fored into jenkinsci.


issues about high DPI resolution

hello, it seems that in my PC (4K monitor), there are some issues for the high DPI resolution.

Hello,
Thank you for your interest in pyDRTtools. Can you please share other specifications of your computer e.g. operating system etc?

Hi, thank you for your assistance. I have deleted the old DRT environment and recompiled it. But the issue still happens.

My email address is<EMAIL_ADDRESS>

Hello. Thank you for your interest in pyDRTtools. The GUI has been rescaled to handle high DPI resolution, and we have updated the GitHub accordingly. Could you please test this and let us know if everything works fine. Thank you.


Completion api injecting restart text after every prompt, no option to switch it off like in playground

I'm not sure this is an issue with the python api per se.
When using the completion api to generate the next line of a poem using davinci, the api always starts the text with "\n\n". This can't be filtered with the logit parameter as the phrase "\n\n" is actually 4 tokens according to the tokenizer: [59, 77, 59, 77]
What is the correct workaround?
Is there any plan to add the option "Inject restart text" like in the playground? The restart text adds 4 tokens to each request, which adds up fast...

I'm going to close this due to inactivity. Feel free to reopen.


RSpec failure when testing RecursiveOpenStruct.new(nil)

When testing RecursiveOpenStruct (ROS) using RSpec in a Rails project, my test against initializing ROS with nil is failing and reports a 'no method error for nil class' when sending a key to the ROS instance.
I am wrapping calls to RecursiveOpenStruct in a method that makes the call:
RecursiveOpenStruct.new(hash, recurse_over_arrays: true)
I tested this with 2 different versions of RecursiveOpenStruct and received 2 different outcomes:
first using the current 0.6.4 version gem build, I instantiated the RecursiveOpenStruct class passing nil and recurse_over_arrays: true.
RecursiveOpenStruct.new(nil, recurse_over_arrays: true)
This test fails
Using the older version 0.5.0 of the gem and the same initialization, I have no issues with the tests failing.
When I pry the session in the failing test, I see that RecursiveOpenStruct.new(nil, recurse_over_arrays: true) returns a RecursiveOpenStruct instance, however when I send a key to the ROS instance that does not exist, then the "NoMethodError: undefined method []' for nil:NilClass" appears.
Pry session debug output:
[1] pry(#RSpec::ExampleGroups::ResponseDeserializer)> described_class.recursive_open_struct(nil).client_id
NoMethodError: undefined method []' for nil:NilClass
from .../.rvm/rubies/ruby-2.2.2/lib/ruby/2.2.0/ostruct.rb:185:in method_missing'
It appears that somewhere in the method_missing calls, the ROS object gets lost, perhaps because the instance does not correctly maintain the @table object?
Files and additional test output are below.
Test Error:
`From: ../spec/lib/serializers/response_deserializer_spec.rb @ line 79 :
74:   it "handles nil" do
77:     nil_ostruct = described_class.recursive_open_struct(nil)
78:     binding.pry
79:     expect(nil_ostruct.client_id).to be_nil
80:     # expect{nil_ostruct.client_id}.to_not raise_error

`
[1] pry(#RSpec::ExampleGroups::ResponseDeserializer)> nil_ostruct
=> #RecursiveOpenStruct:0x3fc71154d338
[2] pry(#RSpec::ExampleGroups::ResponseDeserializer)> nil_ostruct.client
NoMethodError: undefined method []' for nil:NilClass
from .../.rvm/rubies/ruby-2.2.2/lib/ruby/2.2.0/ostruct.rb:185:in method_missing'
More console output, notice that it is not possible to assign a new key-value pair to the ROS instance that was initialized with nil.
[10] pry(main)> b=RecursiveOpenStruct.new(nil)
=> #RecursiveOpenStruct:0x3ff34cd4f2b8
[11] pry(main)> b.test
NoMethodError: undefined method []' for nil:NilClass
from .../.rvm/rubies/ruby-2.2.2/lib/ruby/2.2.0/ostruct.rb:185:in method_missing'
[12] pry(main)> b.test=0
NoMethodError: undefined method has_key?' for nil:NilClass
from .../.rvm/gems/ruby-2.2.2/gems/recursive-open-struct-0.6.4/lib/recursive_open_struct.rb:93:in get_key_from_table'
[13] pry(main)> b
=> #RecursiveOpenStruct:0x3ff34cd4f2b8
[6] pry(main)> a=OpenStruct.new(nil)
=> #
[7] pry(main)> a.test
=> nil
[8] pry(main)> a.test = 0
=> 0
[9] pry(main)> a
=> #
`
class ResponseDeserializer
class << self
def recursive_open_struct(json)
  hash = json.kind_of?(String) ? from_json_string(json) : json
  hash.kind_of?(Array) ? recurse_array(hash) : recurse(hash)
end

private

def recurse_array(array)
  new_array = []
  array.each do |el|
    new_array << recurse(el)
  end
  new_array
end

def recurse(hash)
  RecursiveOpenStruct.new(hash, recurse_over_arrays: true)
end

def from_json_string(json)
  JSON.parse(json) unless json.nil?
end

end
end
`

OpenStruct.new(nil).foo # => nil

RecursiveOpenStruct.new(nil).foo # => NoMethodError: undefined method `[]' for nil:NilClass

0.6 reworked the internals of the ROS class a bit in order to add recursion support to subscript notation and to make it possible to choose whether the input hash is mutated or duplicated, so I'm not too surprised that un-spec'd OpenStruct functionality broke. This should be an easy fix though.

@ckoch-ncsa could you look over my commit and see if that solves your problem? If so, I will cut a patch release in the next day or so.

Looks great and works as expected now. Thanks for fixing this so quickly.

Just released v0.6.5 to rubygems.


RMF for NP beta

Task/Issue URL: https://app.asana.com/0/1203137811378537/1205187503800387/f
Description
RMF for NetP beta closure
See asana task
Steps to test this PR
See Asana task

Current dependencies on/for this PR:

develop

PR #3391

PR #3407   👈

This comment was auto-generated by Graphite.


ja: Make docs/concepts/workloads/controllers/ttlafterfinished.md follow v1.17 of the original text

This is a Feature Request

What would you like to be added
Update docs/concepts/workloads/controllers/ttlafterfinished.md to follow v1.17 of the original (Englist) text.
Why is this needed
content/ja/docs/concepts/workloads/controllers/ttlafterfinished.md is outdated.
Comments
/language ja
/good-first-issue
File to update
https://github.com/kubernetes/website/blob/dev-1.17-ja.2/content/ja/docs/concepts/workloads/controllers/ttlafterfinished.md
Original
https://github.com/kubernetes/website/blob/release-1.17/content/en/docs/concepts/workloads/controllers/ttlafterfinished.md
diff between translated and v1.17
https://gist.github.com/b52b1778c59e006c3d36d60b9f0f13c7
Note
Currently, we use dev-1.17-ja.2 branch. Please open a PR targeting the branch.
ref
How To Contribute
翻訳スタイルガイド
If you have a question, feel free to ask us at slack.k8s.io #kubernetes-docs-ja channel.

/assign

/close


Update DC/OS UI package to 1.10.0-rc.8 (1.10)

High Level Description
Update the DC/OS UI package to include the latest fixes and improvements.
What features does this change enable? What bugs does this change fix? High-Level description of how things changed.
Related Issues

DCOS_OSS-1582: Fix unreachableStrategy validation

Checklist for all PR's

[ ] Included a test which will fail if code is reverted but test is not. If there is no test please explain here:
[x] Read the DC/OS contributing guidelines
[x] Followed relevant code rules Rules for Packages and Systemd

Checklist for component/package updates:
If you are changing components or packages in DC/OS (e.g. you are bumping the sha or ref of anything underneath packages), then in addition to the above please also include:

[x] Change log from the last version integrated (this should be a link to commits for easy verification and review): example
[x] Test Results: Unit, Integration, System
[x] Code Coverage (if available): Cobertura Report

PLEASE FILL IN THE TEMPLATE ABOVE / DO NOT REMOVE ANY SECTIONS ABOVE THIS LINE
Instructions and Review Process
What is the review process and when will my changes land?
All PRs require 2 approvals using github's pull request reviews.
Reviewers should be:

Developers who understand the code being modified.
Developers responsible for code that interacts with or depends on the code being modified.

It is best to proactively ask for 2 reviews by @mentioning the candidate reviewers in the PR comments area. The responsibility is on the developer submitting the PR to follow-up with reviewers and make sure a PR is reviewed in a timely manner. Once a PR has 2 ship-it's, no red reviews, and all tests are green it will be included in the next train.

@mesosphere-mergebot bump-ee

@mesosphere-mergebot label [Ready For Review]

@orsenthil 1.10 or 1.10.0?

dcos:1.10 is correct. However, there is something wrong with pkgpanda here:
fatal: ambiguous argument 'v1.10.0-rc.8^{commit}': unknown revision or path not in the working tree.

teamcity/create-release-pr  had a failure.

[Building package dcos-ui variant <default>] pkgpanda.build.BuildError: Validation error when fetching sources for package: Unable to find sha1 of ref_origin v1.10.0-rc.8: Unable to find ref 'v1.10.0-rc.8' in '/teamcity/work/19ad0dff41587ee6/packages/cache/packages/dcos-ui/dcos-ui/cache.git': Command '['git', '--git-dir', '/teamcity/work/19ad0dff41587ee6/packages/cache/packages/dcos-ui/dcos-ui/cache.git', 'rev-parse', 'v1.10.0-rc.8^{commit}']' returned non-zero exit status 128
[15:33:21]

@mesosphere-mergebot bump-ee

@mesosphere-mergebot label [Ship It]


Added new Quotes

Added new quote @AnirudhPanda

I have modified the Hashmap now we can search in O(1) to find the sent Quotes

Hey @DADDY-DOUBLESHOT
A quick suggestion
Create an issue for the same (using hashmap thing) and then create a PR only to solve that issue!
This will be helpful for us too and it is a good practice to do one thing at a time

Thank you @AnirudhPanda  for mentioning those details about my PR . I will  create those issue and perform those tasks ASAP , PS this is my first PR thanks for your humble guidance :)


lscert | Add support for setting server and port values from URL pattern provided as positional argument

Currently the lscert tool is used to check a FQDN and port like so:

lscert --server FQDN --port 443

explicitly specify port

lscert --server FQDN

use default port of 443

This feature would allow you to specify the FQDN and port like so:

lscert https://www.example.com

map www.example.com to the server flag
map 443 (default value) to port flag

lscert https://www.example.com:443/

map www.example.com to the server flag
map 443 (explicit value) to port flag

lscert www.example.com

map www.example.com to the server flag
map 443 (default value) to port flag

lscert https://www.example.com:443/ --port 8443

map www.example.com to the server flag
map 443 (value from URL) to port flag

the positional argument is parsed after the --port flag is already processed
the positional argument overwrites any value already provided via the --port flag

This behavior would only be applied if the server flag is not specified.
See also GH-381.

This feature would allow you to specify the FQDN and port like so:
...

lscert https://www.example.com:443/ --port 8443

map www.example.com to the server flag
map 443 (value from URL) to port flag

the positional argument is parsed after the --port flag is already processed
the positional argument overwrites any value already provided via the --port flag

Turns out that the standard library flags package requires that positional arguments come after flags. See GH-385 for further details.


install-docker 17.03.sh shell script is not working on 18.04 Ubuntu

Version: Master 2.2 (2/8/19) and this might occur in 1.6 as well (we should check there)
What kind of request is this (question/bug/enhancement/feature request):
Bug
Steps to reproduce (least amount of steps as possible):

Attempt to provision a cluster by selecting v17.03.x for docker engine

Alternatively just try to execute the following on a fresh VM:
curl https://releases.rancher.com/install-docker/17.03.sh | sh

Result:
The script will not successfully install docker. It will appear to be operating normally and then eventually print the following before exiting
E: Version '' for 'docker-ce' was not found
Other details that may be helpful:
It appears as though this version may have been removed from the repo.
Exit code is 100
Here is entire output from my terminal with exit code check at end:
term_output.txt

17.03 was never released for Bionic Beaver, see https://github.com/rancher/rancher/issues/13222#issuecomment-394307237

gotcha. Then we should update the shell script to provide notice to the user that 17.03 is not available on 18.04 ub and while it might not be practical it would be nice to see this in the UI too if someone tries to create a cluster with 17.03 on 18.04 ub

Confirmed 17.03sh working on 16.04 ub; closing in favor of Seb's issue #13222 and I will open a separate issue with enhancement request for the script so it at least notifies the user that 17.03 may not be / is not available on 18.04 ub.

Enhancement request: #17936


Split out the jupyter notebook integration

And add syntax highlighting, following the ideas from fugue

Codecov Report

Merging #152 (4075828) into main (d576109) will decrease coverage by 0.10%.
The diff coverage is 100.00%.

:exclamation: Current head 4075828 differs from pull request most recent head e140ef3. Consider uploading reports for the commit e140ef3 to get more accurate results

@@             Coverage Diff             @@
##              main     #152      +/-   ##
===========================================
- Coverage   100.00%   99.89%   -0.11%
===========================================
  Files           52       53       +1
  Lines         1953     1958       +5
  Branches       266      266
===========================================
+ Hits          1953     1956       +3
- Misses           0        2       +2

Impacted Files
Coverage Δ

dask_sql/context.py
100.00% <100.00%> (ø)

dask_sql/integrations/ipython.py
100.00% <100.00%> (ø)

dask_sql/input_utils/hive.py
90.00% <0.00%> (-10.00%)
:arrow_down:

dask_sql/utils.py
100.00% <0.00%> (ø)

dask_sql/physical/rel/custom/predict.py
100.00% <0.00%> (ø)

Continue to review full report at Codecov.

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


v6.0.0-rc: unable to execute a command on a remote host

Steps to reproduce
I have a VM running, and I'm trying to execute a command remotely with the following command (from a bash shell). It sometimes crashes (see #5562), and when it does not, nothing is executed on the remote host.
    pwsh -Command - <<EOF
\$Username = '$VM_USER'
\$Password = '$VM_PASSWORD'
\$SecurePassword = ConvertTo-SecureString -AsPlainText \$Password -Force
\$Cred = New-Object System.Management.Automation.PSCredential -ArgumentList \$Username,\$SecurePassword
Invoke-Command -ComputerName $VM_IP -Credential \$Cred -Authentication Basic -Command { $VM_COMMAND }
EOF

Expected behavior
The command executes on the remote host
Actual behavior
The execution fails, and by running tcpdump at the same time on the interface that connects the host and the virtual machine, I can tell that absolutely no network communication is attempted, not even opening a TCP connection. Here is the run with strace monitoring the socket and connect calls. As an attachement you'll find a run with strace monitoring everything.
strace.log
socket(PF_LOCAL, SOCK_DGRAM|SOCK_CLOEXEC, 0) = 40
connect(40, {sa_family=AF_LOCAL, sun_path="/dev/log"}, 110) = -1 ENOENT (No such file or directory)
socket(PF_LOCAL, SOCK_DGRAM|SOCK_CLOEXEC, 0) = 56
connect(56, {sa_family=AF_LOCAL, sun_path="/dev/log"}, 110) = -1 ENOENT (No such file or directory)
socket(PF_LOCAL, SOCK_DGRAM|SOCK_CLOEXEC, 0) = 95
connect(95, {sa_family=AF_LOCAL, sun_path="/dev/log"}, 110) = -1 ENOENT (No such file or directory)
Invoke-Command -ComputerName <IP_ADDRESS> -Credential System.Management.Automation.PSCredential -Authentication Basic -Command { Date }
[<IP_ADDRESS>] Connecting to remote server <IP_ADDRESS> failed with the following error message : MI_RESULT_ACCESS_DENIED For more information, see the about_Remote_Troubleshooting Help topic.
+ CategoryInfo          : OpenError: (<IP_ADDRESS>:String) [], PSRemotingTransportException
+ FullyQualifiedErrorId : 2,PSSessionStateBroken
+++ exited with 1 +++

By running tcpdump at the same time on the interface connecting the VM, I can tell that no
Environment data
> $PSVersionTable
Name                           Value
----                           -----
PSVersion                      6.0.0-rc
PSEdition                      Core
GitCommitId                    v6.0.0-rc
OS                             Linux 4.9.0-0.bpo.4-amd64 #1 SMP Debian 4.9.51-1~bpo8+1 (2017-10-17)
Platform                       Unix
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0...}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

Please note that this works fine with at least 6.0.0-beta.9 and 6.0.0-beta.8-1.
If there is any additional steps I can do to help you find the problem, please let me know and i'll try to do them.

I have the same problem.
Using powershell 6.0.0.-rc
I have it working from my Cent OS to Windows 2012 R2 but only with OpenSSH and PowerShell 6.0.0.rc on my Windows 2012 R2 server.
Invoke-command -Computername does not work on powershell 6.0.0-rc
Same as Enter-PSSession -Computername
Only way I could get it to work is with
invoke-command -hostname
enter-PSSEssion -hostname
but those methods have limited paramaters.   no -Credential no -Configurationname  etc
When can we have WINRM working?


DISCOVERY: Clarify the UUID TEI TYPE - align with standards

https://datatracker.ietf.org/doc/html/rfc9562 defines many versions of UUID.
We need to either specify ONE version we use in the TEI UUID type or allow for multiple types.
Multiple types could look like:
tei:uuid:carloselectriccycles.com:uuidv3:234234-234-234-234-3
Not sure if there would be any benefit from supporting multiple UUID types from the API/TEI point of view. Are we aware of any usage out there that we need to support?

I think that each producer can decide its own UUID type and even change UUID type in time. The only requirement I see is that the identifier must be unique, which means that tei:carloselectriccyccles.com:turbo_bike is a very poor choice of an identifier.
Note that UUIDs already contain the version number in their digits, so just using uuid:<something> identifiers should be fine. On the other hand the uuid: namespace should always be included in the identifier.


docs: improve Introduction docs

@karsa-mistmere I totally agree with you! I'll keep this in mind for future docs improvements.

In the new site #1275 I also updated this text. I tried to split it up a bit to create some sections in text that improve the overview of the page.
Live version: https://lucide-docs.vercel.app/guide/
Maybe we can combine these changes?
This is the text:
What is Lucide?
Lucide is an open-source icon library that provides 1000+ vector (svg) files for displaying icons and symbols in digital and non-digital projects. The library aims to make it easier for designers and developers to incorporate icons into their projects by providing several official packages to make it easier to use these icons in your project.
Available Icons
Lucide contains icons with different variants and states, allowing users to choose the most suitable icon for their needs. And if a desired icon isn't available yet, users can open a design request, and the Lucide community contributors will help provide new icons. With more icons to choose from, users have more options to work with in their projects.
Complete Set of Icons
As new applications with specific features arise, Lucide aims to provide a complete set of icons for every project. The community follows a set of design rules when designing new icons. These rules maintain standards for the icons, such as recognizability, consistency in style, and readability at all sizes. While creativity is valued in new icons, recognizable design conventions are important to ensure that the icons are easily identifiable by users.
Code Optimization
In addition to design, code is also important. Assets like icons can significantly increase bandwidth usage in web projects. With the growing internet, Lucide has a responsibility to keep their assets as small as possible. To achieve this, Lucide uses SVG compression and specific code architecture for tree-shaking abilities. After tree-shaking, you only ship the icons you used, which helps to keep software distribution size to a minimum.
Official Packages
Lucide's official packages are designed to work on different platforms, making it easier for users to integrate icons into their projects. The packages are available for various technologies, including Web (Vanilla), React, React Native, Vue, Vue 3, Svelte,Preact, Solid, Angular, NodeJS and Flutter.
Community
If you have any questions about Lucide, feel free to reach out to the community. You can find them on GitHub and Discord.


Multi tenant oauth1

I'm using your library to wrap the xero api. So far so good. I can get 1 legged oauth1 working like so:
const oas = require('./openAPI/Xero_accounting_2.0.0_swagger.json');
        let oauth =
            {oauth:{ consumer_key: user.consumerKey,
                    token: user.consumerKey,
                    private_key: user.consumerSecret,
                    signature_method : 'RSA-SHA1'
                }};
        let {schema} = await OASGraph.createGraphQlSchema(oas,{requestOptions: oauth});

This is fine, you get support for free from request.
But if I want to change the oauth settings I have to recreate the whole schema. I can't use tokenJSONpath as oauth1 doesn't work that way. Perhaps the best way would be to allow request to be wrapped by some form of middleware so any form of auth (or auto retry for rate limiters etc) could be supported.
(yes oauth1 is old, tell it to xero)

I've set my fork of oasgraph to load the request options from the request context. I've also set it to save the response headers to the context. It seems like a minimal method for solving this issue and helping with request diagnostics/rate limits.
I'll do a PR in due course.

Ah great! So sorry for the lack of response! This slipped under my radar. It seems like you found a workaround but let us know if you have any trouble. We would highly appreciate a PR so if you are willing, please go for it! 😄


base 29e1c5，test p0

Proposed changes
Issue Number: close #xxx

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall


iOS Callback Not Fired for DocumentPicker.show

Any ideas?
RN 0.50.0
document picker ^2.1.0

same here
const pickerShow = () => new Promise((resolve, reject) => {         DocumentPicker.show(           (options || {             filetype: [DocumentPickerUtil.allFiles()],           } as any),           (error, res) => {             if (error) {               reject(error);               return;             }             resolve(res);           },         );       });       const file = yield call(pickerShow);       console.log(file);
Don't see file variable in console

iOS doesn't fire the callback if you don't select a file. Android returns an error. This difference in behavior caused me confusion. The whole point of RN is to provide packages that behave the same way from the Javascript perspective. I'll close this issue.


The data grid will not be focused if uncheck the top order column against ADLS Gen2/blob/file/queue/table explorers

Storage Explorer Version: 1.32.0-dev (95)
Build Number: 20231025.1
Branch: rel/1.32.0
Platform/OS: Windows 10/Linux Ubuntu 20.04/MacOS Sonoma 14.0(Apple M1 Pro)
Architecture: x64/x64/arm64
How Found: AD-hoc testing
Regression From: Not a regression
Steps to Reproduce

Expand one storage account -> Blob Containers.
Create a blob container -> Upload one blob.
Right click the 'Name' column and select 'Customize Columns...'.
Uncheck 'Name' column -> Click 'OK'.
Reopen the blob container.
Check whether the data grid is focused.

Expected Experience
The data grid is focused.

Actual Experience
The data grid is not focused.

Additional Context
This issue doesn't reproduce if move one of the visible columns to top.

@v-xianya Step 5 says to reopen the container. Did you close the container after clicking OK? If that's true, I can't reproduce. Or are you referring to the fact that focus is lost right after the dialog closes?

@craxal This issue reproduces whether the original container explorer is closed or not.
To reproduce this issue, we must uncheck the top visible column on the 'Column Options' dialog.
If select one column that is not the top visible one, this issue will not be reproduced.
Here is the record:

@craxal Verified this issue on the main build 20240130.3.
Fixed for file shares/queues/tables.
Reproduce for ADLS Gen2 /regular blob containers. So, we are going to reopen this issue.

Changes have now been merged for blobs.

Verified this issue on the main build 20240130.14. Fixed for blobs. So, we are going to close this issue.


Concurrency bug: argument passed into goroutine is not evaluated first before being passed by value

When the time quantum is sufficiently large (say 30 instructions), the variable being passed as an argument in the creation of a new goroutine may be modified by the calling goroutine as it is not evaluated at the time when the new goroutine is created.
Since context is not switched yet as the time quantum is not reached, the calling goroutine can continue running before the newly created goroutine can run. The calling goroutine modifies the value of the variable(s) being passed into the goroutine's argument(s), resulting in the new goroutine using the arguments of possibly wrong values, despite it being supposed to be "passed by value", which means that the argument is not evaluated yet.
Example:
package main
import "fmt"
func add(a int){
    fmt.Println(a);
}

func main() {
   for i := 0; i < 10; i++ {
      go add(i);
   }
   fmt.Println("Done");
}

should print 0\n1\n2\n3\n4\n5\n6\n7\n8\n9\nDone\n
However, it prints 2\n2\n2\n5\n5\n7\n7\n9\n9\n9\nDone\n as the loop incremented i before the goroutine can run, so the goroutine created when i = 0,1 was only run when i = 2 in the loop, so it prints 2 instead of 0 or 1.

Go statements should create a "Go (argCount)" instead of "Call (argCount)" instruction, so the current implementation is actually wrong


test: add runtime testing for PHP

The runtime testing variant for the PHP generator. It uses standard PHPUnit dependency with a composer dependency manager.
The GHA uses a matrix with PHP 8.1 and 8.2 (the latest two)
I noticed that PHPUnit ALWAYS outputs something therefore the npm run test:runtime:java command will always fail. PHPUnit has a no-output possibility, but then it will not report an error.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 92.203%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
5247

Relevant Lines:
5549

💛  - Coveralls

/rtm


First page infinite scroll /page/2

Hi Outdooricon,
Your solution for the infnite scroll is wonderful!! We were looking for this for months and it works great!
There are just two things and maybe you can help me out?

The home URL (first page) has /page/2 in the URL. How can set change this to just the home page?
/page/XX can I translate that to eg pagina?

Thank you in advance.
Best,
Mark

Infinite Scroll v3 is out!. It's an all new re-write, so take a look at how to upgrade from v2. With the brand new release, this contribution is no longer applicable.
Thank you for your interest and effort in Infinite Scroll. Sorry I had to pass on this one. 🌈🐻


Featured Suggestion: comp screenshot attached

I need to set spacing on each slick item.
Sample Comp: http://prntscr.com/axbhfa
wrapper width is 1258px
Currently the slick can only be setup in a way where all slick item are same width and close together --> http://prntscr.com/axbhrt
Is there a way to add spacing on each slick item.
Thanks in advanced.
====================================================================
[  paste your jsfiddle link here  ]
use this jsfiddle to reproduce your bug:
http://jsfiddle.net/simeydotme/fmo50w7n/
we will likely close your issue without it.
====================================================================
Steps to reproduce the problem

...
...

====================================================================
What is the expected behaviour?
...
====================================================================
What is observed behaviour?
...
====================================================================
More Details

Which browsers/versions does it happen on?
Which jQuery/Slick version are you using?
Did this work before?

Hey @manuelparejas just add padding on the slick slide, I recommend a nested div too. Good luck.
Closing: support request.


Plugin distribution

Hello,
I created a plugin that forwards spam to signal-spam.fr: https://github.com/Rudloff/rainloop-signalspam
What is the best way to distribute it?
Do you accept externals plugins on repository.rainloop.net? Or should I create my own repository?

@Rudloff How did you start your development? Which one of the existing plugins was the basis to start from? Unfortunately, your plugin is hardly documented. :-(

I mainly did it by digging into Rainloop's code. (Please open an issue on my repo if you think something specific should be documented.)
I agree it would be nice to have better documentation about Rainloop's plugin APIs.


Document details of the Enhancement - how it treats Lists, transients, scalar types etc

Add to the website documentation on the details of how the enhancement works.

General overview of what the enhancement does
How are transients treated versus @Transient
How are @OneToMany and @ManyToMany lists/sets treated
How are embedded beans handled
How are constructors handled (when no default constructor etc)

I think this is now covered effectively in:

http://ebean-orm.github.io/docs/mapping/list
http://ebean-orm.github.io/docs/mapping/constructors


[Feature Request] Directional Terrain

On a few cases I've wanted difficult terrain to only apply in one direction - namely a literal uphill battle where it took double movement to go one direction, or a separate battle where the water had a current that made one direction even worse difficult terrain. Not entirely sure how best it could be implemented, though. There's a lot of angles at which one can enter a tile.
Perhaps slightly further than that, the ideal would be somehow setting things up so, like... This image would be plausible.

Yes, would "slope" difficult terrain would be great! Maybe could add some metadata as to the directions in which it applies.


chkcrontab command takes nonexistent file without complaints and exits okay (exitcode=0)

As seen below, randomly typing a nonexistent filename chkcrontab would take it in fine with no complaints and gives an okay exist code (0).
$ ls -l
total 16
-rw-r--r--  1 ye  staff   120 Nov  7 18:04 Dockerfile
-rw-r--r--  1 ye  staff  2145 Nov  8 11:02 test-crontab
$ chkcrontab ./8e5f63ef-915b-4616-ad90-31a1a590922d
Checking correctness of ./8e5f63ef-915b-4616-ad90-31a1a590922d
$ echo $?
0
$ chkcrontab
ERROR: No crontab file was specified.
$ echo $?
1

It should probably exit with an error exitcode(1) and print out an error message just as if I didn't provide a file or similar. "ERROR: ./8e5f63ef-915b-4616-ad90-31a1a590922d doesn't exist."

Valid.  Added in b8ed6c0. Sorry for the delay. Had some personal issues which kept me from this.

(this will be in 1.7)

@lyda you are simply adding a log line, the exit code hasn't been changed if file doesn't exist, has it?

If it's a warning it changes the exit code:
/ # chkcrontab kitten
Checking correctness of kitten
W: File "kitten" does not exist.
W: There were 1 warnings.
See http://goo.gl/7XS9q for more info.
/ # echo $?
1
/ # chkcrontab /dev/null
Checking correctness of /dev/null
/ # echo $?
0

I did push out a new version so if you upgrade you can confirm it works as desired. If not let me know.


feat(profile): implement update profile picture

Fixes #222
Describe the changes you have made in this PR -
Screenshots of the changes (If any) -
Note: Please check Allow edits from maintainers. if you would like us to assist in the PR.

@aman-singh7 It's very important for us to keep minimal number third party packages as much as possible, so keep that in mind while implementing new features.

@aman-singh7 It's very important for us to keep minimal number third party packages as much as possible, so keep that in mind while implementing new features.

Yup, I have added only two dependencies. One is to pick the image and another one to crop that picked one.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

47 of 98 (47.96%)  changed or added relevant lines in 8 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.4%) to 70.142%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

lib/ui/views/profile/profile_view.dart
6
7
85.71%

lib/utils/api_utils.dart
13
16
81.25%

lib/services/API/users_api.dart
5
9
55.56%

lib/viewmodels/profile/edit_profile_viewmodel.dart
2
16
12.5%

lib/ui/views/profile/edit_profile_view.dart
17
46
36.96%

Totals

Change from base Build<PHONE_NUMBER>:
-0.4%

Covered Lines:
3956

Relevant Lines:
5640

💛  - Coveralls

@aman-singh7 Can you also send screenshot/screen recording here to see how the update UI and UX looking like ?

@aman-singh7 Can you also send screenshot/screen recording here to see how the update UI and UX looking like ?

Screenshots are mentioned in my biweekly blog.
And here is the screen recording of the flow:
https://user-images.githubusercontent.com/77198905/176388175-e240f6b7-75d5-4ea2-90d5-a399a452c99a.mp4


Cause server / client sync on block texture change

I've been noticing this on my MP server.
This also should fix #21
The tile entity wasn't being marked dirty when textures or glowstone were added, causing lighting and texture desync.

Fantastic! Thanks for the fix


Change the tailwind-olugin display in tree.file = 'path' view

Clear and concise description of the problem
When using tree.file: 'path' in the config:
import { defineConfig } from 'histoire'
import { HstVue } from '@histoire/plugin-vue'
import { HstNuxt } from '@histoire/plugin-nuxt'

export default defineConfig({
  plugins: [
    HstVue(),
    HstNuxt(),
  ],
  tree: {
    file: 'path',
  },
})

The tailwind plugin shows up under node_modules:

At first this raised the alarm bells for me and I thought that histoire crawled all of node_modules, after ignoring node_modules via storyIgnored: ['node_modules'], I learned that this is not the case, instead the plugin is just displayed that way.
Suggested solution
Use a plugins/ top level element to display plugins in path mode.
Alternative
No response
Additional context
Thanks for the library, it's great and a really fresh breeze compared to configuring, maintaining and working with storybook (as of sb 6.5).
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that requests the same feature to avoid creating a duplicate.

Thanks for the library, it's great and a really fresh breeze compared to configuring, maintaining and working with storybook (as of sb 6.5).

Thanks! 🤗


Differences with Perceiver Preprint

I think the implementation in this repo differs from the Perceiver preprint description in two significant ways:

The figure in the preprint and text indicate that the cross attention block is followed by a 'Latent Transformer'. The text indicates that the latent transformer has 6 blocks when training on ImageNet. This repo , to be fair, most re-implementations I have seen for Keras or JAx make the same mistake and use a single latent block, not a Latent Transformer with several blocks.
The signature of the forward method in this implementation cannot support training with multi-modality inputs. Multi-modality is when you want to train with video, audio and image for instance as inputs to the same model. The signature does not support multi-modality because each modality has a different number of dimensions, and positional encoding must be applied to each modality independently. This is not possible when accepting a single x tensor as input to forward.
I have implemented fixes for these two issues in this fork: https://github.com/fac2003/perceiver-multi-modality-pytorch

I tried contributing to this repo by starting to add tests, but the PR was rejected, so I won't bother to submit a PR for these fixes.

@fac2003 (1) the latent transformer is taken care of by this keyword argument https://github.com/lucidrains/perceiver-pytorch/blob/main/perceiver_pytorch/perceiver_pytorch.py#L149 - a transformer is just layers of self-attention, and by setting this to be greater than 1, you get the latent transformer
(2) what kind of modalities are you hoping to support? you can actually encode the data yourself and then just turn this flag to False https://github.com/lucidrains/perceiver-pytorch/blob/main/perceiver_pytorch/perceiver_pytorch.py#L148
I want to keep the library simple and avoid having different encodings for too many modalities. Images is enough for now


Fix compliance link

issue: #277

fix translations to allow link in alert

/gcbrun

Temporary Url for review: https://business-ar-dev--pr-278-18eelwpy.web.app


Check for previous/existing GitHub issues/module proposals

[X] I have checked for previous/existing GitHub issues/module proposals

Check this module doesn't already exist in another project that is being evolved into AVM

[X] I have checked for that this module doesn't already exist in CARML or TFVM

Terraform or Bicep?
Bicep
Module Classification?
Resource Module
Module Name
avm/res/network/traffic-manager-profile
Module Details
Migrate existing module from CARML
Do you want to be the owner of this module?
Yes
Module Owner's GitHub Username
lsnoddy

Hi @lsnoddy,
Thanks for requesting/proposing to be an AVM module owner!
We just want to confirm you agree to the below pages that define what module ownership means:

Team Definitions & RACI
Shared Specification (Bicep & Terraform)
Module Support

Any questions or clarifications needed, let us know!
If you agree, please just reply to this issue with the exact sentence below (as this helps with our automation 👍):
"I CONFIRM I WISH TO OWN THIS AVM MODULE AND UNDERSTAND THE REQUIREMENTS AND DEFINITION OF A MODULE OWNER"
Thanks,
The AVM Core Team

NOTE:

This Module Proposal issue MUST remain open until the module is fully developed, tested and published to the relevant registry. Do NOT close the issue before the successful publication is confirmed!
Once the module is fully developed, tested and published to the relevant registry, and the Module Proposal issue was closed, it MUST remain closed.

I CONFIRM I WISH TO OWN THIS AVM MODULE AND UNDERSTAND THE REQUIREMENTS AND DEFINITION OF A MODULE OWNER

Hi @lsnoddy,
Thanks for confirming that you wish to own this AVM module and understand the related requirements and responsibilities!
We just want to ask you to double check a few important things before you start the development.
Please use the following values explicitly as provided in the module index page:

For your module:

ModuleName - to name your module
TelemetryIdPrefix - to be used in your module's telemetry

For your module's repository:

Repository name and folder path are defined in RepoURL
Create the GitHub teams defined in the ModuleOwnersGHTeam and ModuleContributorsGHTeam columns

You can now start the development of this module! ✅ Happy coding! 🎉
Please respond to this comment, once your module is ready to be published! 🙏
Any further questions or clarifications needed, let us know!
Thanks,
The AVM Core Team

NOTE:

This Module Proposal issue MUST remain open until the module is fully developed, tested and published to the relevant registry. Do NOT close the issue before the successful publication is confirmed!
Once the module is fully developed, tested and published to the relevant registry, and the Module Proposal issue was closed, it MUST remain closed.

Please also note that since this is a migrate from CARML scenario, @prjelesi might be able to help you with the next steps. Please feel free to contact him internally.

@lsnoddy,
Due to a recent change in the naming convention for AVM Bicep modules, this module is getting an updated name.
The title and description of this GH issue has been updated to reflect the name change. Please make sure you adhere to the new naming convention when publishing your module!
Please see the new naming convention in:

RMNFR1 for resource modules
PMNFR1 for pattern modules

Please note that the Telemetry ID and name of GitHub Teams haven't changed.
If you feel uncertain about any details, please check the Bicep modules indexes.

@lsnoddy, I've just noticed that this PR has already been merged. Next time, please drop a note here for faster processing! Thanks! 😉

[x] Module published.
[x] GitHub teams are available and assigned to their parents.
[x] CODEOWNERS file updated.

Thank you for your work! I'm closing this issue now.


Thanks

you should use socks5 client

windows dose not have native support of Socks5, so we need convert it to https proxy using privoxy


No response from proxy when RAML loaded from file://

Here is my raml file:
#%RAML 0.8
---
title: Test

protocols: [ HTTP ]

mediaType: application/json

/bar:
  description: Collection of available foo.
  get:
    description: Retrieve a list of foo.
    responses:
      200:
        body:
          application/json:
            schema: |
              {
                "type": "string",
                "required": "true"
              }

I have a server running on port 25001. I start the proxy using this command:
java -jar ./dev_deps/node_modules/raml-tester/bin/raml-tester-standalone.jar -thttp://localhost:25001 -rfile:///<path>/test_api.raml -s./ -p25099

If I do a curl request directly against the server, things work. Specifically, I see the request and response in my server logs and then curl prints the result:
$ curl -vk http://localhost:25001/bar
* About to connect() to localhost port 25001 (#0)
*   Trying ::1... Connection refused
*   Trying <IP_ADDRESS>... connected
* Connected to localhost (<IP_ADDRESS>) port 25001 (#0)
> GET /bar HTTP/1.1
> User-Agent: curl/7.19.7 (x86_64-redhat-linux-gnu) libcurl/7.19.7 NSS/<IP_ADDRESS> zlib/1.2.3 libidn/1.18 libssh2/1.4.2
> Host: localhost:25001
> Accept: */*
>
< HTTP/1.1 200 OK
< X-Powered-By: Express
< Content-Type: application/json
< X-Response-Time: 1.084ms
< Date: Wed, 08 Jul 2015 02:03:27 GMT
< Connection: keep-alive
< Transfer-Encoding: chunked
<
* Connection #0 to host localhost left intact
* Closing connection #0

When I try to connect to the proxy however, I still see the request hitting the server and it responding (in the server log), but curl never sees a response from the server:
$ curl -vk http://localhost:25099/bar
* About to connect() to localhost port 25099 (#0)
*   Trying ::1... connected
* Connected to localhost (::1) port 25099 (#0)
> GET /bar HTTP/1.1
> User-Agent: curl/7.19.7 (x86_64-redhat-linux-gnu) libcurl/7.19.7 NSS/<IP_ADDRESS> zlib/1.2.3 libidn/1.18 libssh2/1.4.2
> Host: localhost:25099
> Accept: */*
>

After waiting more than 2 minutes, I gave up. Is this a bug or am I doing something wrong?

I use version 0.8.1 extensively to validation integration and production APIs and can testify that it works great.
The version brought by the node module is pretty old (0.0.4) so get the latest JAR from Maven central and use it instead: http://search.maven.org/#browse|1760247101
This said version 0.0.4 was already pretty much functional so it must be something else. What does the proxy output on startup and when the test request hits it?

Ok, I've updated to the version provided above and I see the same results.
Here's the proxy command and the entire output:
$ java -jar ./raml-tester-standalone-0.8.1.jar -thttp://localhost:25001 -rfile:///<pth>/test_api.raml -s./ -p25099
[main] INFO guru.nidi.ramlproxy.report.Reporter - Reporting in TEXT format into: .
[main] INFO guru.nidi.ramlproxy.jetty.JettyProxyServlet - Proxy started

I've found how to reproduce your issue: it seems that if the RAML file is loaded with file:/// the proxy doesn't work. I load my RAML files over HTTP so I've never had the problem (I expose RAML files over HTTP so I can dynamically generate a value for baseUri as explained here).
I've tried to force the base URI with -b hoping that it could clear up the issue but to no avail so @nidi3 I think this is a legit bug.
@bpytlik Could you please append "when RAML loaded from file://" to this issue's title?

I'll have a look at it tomorrow.
Thanks @ddossot for the help!

Just tried it and it works on my machine :disappointed:
To be sure:

When you load the RAML using http://, it works fine.
When you load the RAML using file://, the proxy never responds.
Right?

But in #23 the proxy responds? What's the difference in the setup?

Ok, I think the problem is that the RAML doesn't contain a 'baseUri'.
It's a bug that I'm going to fix.
Meanwhile can you try and add a baseUri and see if it helps?

fixed on trunk

I'm having the same issue. I just installed the latest raml-tester (0.0.4) from npm and get the same output as @bpytlik.


Group prod dependencies for Ruby

We don't usually get many regressions when updating updater dependencies, so this may be fine and allow us to stay more up to date.
Unfortunately since I introduced the root Gemfile & Gemfile.lock, Dependabot PRs for updater/ require manual amendment because both Gemfile and updater/Gemfile use dependencies in dependabot-common. However, those get currently bumped in updater/Gemfile.lock, but not in the root Gemfile.lock, and CI complains about that. So one needs to run bundle lock and amend Dependabot PRs with the updates that creates.
This is a limitation of Dependabot that will be fixed once multi-directory version-updates are supported.

Just to clarify, I'm in favor of trying this! Just wanted to explain in my other comment that updater PRs currently require a bit of extra work and are not mergeable as is.

Can this be automated with a dedicated workflow?

This is a limitation of Dependabot that will be fixed once multi-directory version-updates are supported.

Is this no longer a blocker now that we've shipped https://github.blog/changelog/2024-06-25-simplified-dependabot-yml-configuration-with-multi-directory-key-directories-and-wildcard-glob-support/ ?

Yep, I think it should be better now with multi-directory updates 👍.


The previously-deprecated -i switch to pg_dump is no longer supported in PostgreSQL 9.5.

rake db:structure:dump from a Rails 3 app fails with

pg_dump: invalid option -- i

This is used in lib/arjdbc/tasks/databases3.rake line 115 (version 1.3.20).
This switch was deprecated in PostgreSQL 8.4 and was finally removed in 9.5.

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

#783 Should fix this(barring anything I'm missing).

resolved at https://github.com/jruby/activerecord-jdbc-adapter/pull/783


Customizable claimlist command

Request Description
The server I'm playing on only has claimslist enabled for players instead of claimlist, so it would be nice to have the command customizable through the config menu

Do you require this for a version pre-1.21?

yes, on 1.20.6

Added in v2.0.0 and for 1.20.6 in v0.1.1


Copy button appears on startup out of nowhere

Tested versions
Reproducible in 4.3. I haven't seen it in 4.2 but it could be there too. More testing is needed but I want to report this first.
System information
Godot v4.3.stable - Fedora Linux 40 (KDE Plasma) - Wayland - GLES3 (Compatibility) - Mesa Intel(R) UHD Graphics 620 (KBL GT2) - Intel(R) Core(TM) i7-8650U CPU @ 1.90GHz (8 Threads)
Issue description
I open a project from the editor and a copy button appears on top of everything. Including other windows.
Steps to reproduce
Just open a project from the editor. I have a few addons, like dialogic, notes_tab and phantom camera if that helps.
Minimal reproduction project (MRP)
N/A

Seems like it was a problem with either the project or the addons. Closing...


[3.6] Guard  AqlValues properly in ModificationExecutor

Backport of #11116

http://jenkins.arangodb.biz:8080/job/arangodb-matrix-pr/8619/

The Jenkins has a known issue on Windows only.


Lots of changes. Please read comments

Added a command to run query under cursor position as long as it's surrounded by blank lines #277
Added a command to describe the object-word (table only currently) under the cursor position on any editor
Created another dialect for oracle where only the current schema objects are retrieved. This makes it a lot faster. Also optimized and recreated from scratch all the sqls for retrieving metadata.

Fixed the commands position in package json.
Also added first release of bind parameter support for oracle.
I had a problem with the api.d.ts file had to rename it to .ts for some reason

@vrokolos thanks!
I'll review it tonight after work! Reach you back if needed some changes

I added a new command GetDDL that will output the DDL script for the current object on a new tab. That includes tables, functions, packages and everything else in oracle. Only did it for oraclelocal though. Someone needs to implement it for other dialects.
Fixed some bugs on the metadata queries and made them faster for oraclelocal.
The previewlimit now works for all queries (except the metadata ones) on the oracle dialect. Not just the showrecords one

Completely replaced ReactTable with SlickGrid => ALOT faster results table.
Automatically add parameter default values when query is run without parameter values so that user can change them.
Had problem with oracle date type on bind parameters so replace the type with string temporarily.
DescribeCurrent command now shows table columns, data and DDL script on separate tabs.
Store bind parameter values on global cache so that they are automatically used next time the same parameter name is found.
Fixed bugs!

Adding issues affected by this pull request:
#265 #261 #277

Awesome!

@vrokolos I'm starting to review and test it right now. I'll extends your changes to other dialects too
Great job!

@vrokolos is there any other option for slickgrid? the project seems to be abandoned. That can affect the maintenance and compatibility of this project. Also I saw we are not supporting vscode color scheme to match user's color scheme.
I was planning to switch to one of this libs, but I didn't have time to do it yet:

https://adazzle.github.io/react-data-grid/docs/examples/simple-grid
https://devexpress.github.io/devextreme-reactive/react/grid/
https://www.ag-grid.com/example.php#/

I want to test both first.

Not sure but I've been using slickgrid a lot and had no problems with it but yes you are right about the color theme. Maybe add an option about what grid to use since it is easily doable until people get to use all of them and see how it goes from there

Hey @vrokolos ,
I tested "Run current query" and it works fine. :+1:
I have some problems with "Describe Current Object".  Can you please explain what is "current" in this case? Is it a selection or a line parse...
I tried on a simple table (no complex types) and I get "ORA-01036: illegal variable name/number".

I think I fixed that in latest commits. Will retest later
On Mon, Jul 22, 2019, 11:23 Mickey Pearce<EMAIL_ADDRESS>wrote:

Hey @vrokolos https://github.com/vrokolos ,
I tested "Run current query" and it works fine. 👍
I have some problems with "Describe Current Object". Can you please
explain what is "current" in this case? Is it a selection or a line parse...
I tried on a simple table (no complex types) and I get "ORA-01036: illegal
variable name/number".
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/mtxr/vscode-sqltools/pull/293?email_source=notifications&email_token=AACD4HR6XX2CCRZ2XHODXCDQAVU73A5CNFSM4IDRLSRKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD2PFD2A#issuecomment-513692136,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AACD4HUXD7RM62IV2FVECPTQAVU73ANCNFSM4IDRLSRA
.

Added a command to run query under cursor position as long as it's surrounded by blank lines #277
Added a command to describe the object-word (table only currently) under the cursor position on any editor
Created another dialect for oracle where only the current schema objects are retrieved. This makes it a lot faster. Also optimized and recreated from scratch all the sqls for retrieving metadata.

Could it be merged with an existing dialect so they wouldn't be two?

One dialect. Need property local set to true for eingle scheme

The error probably occurs because you're not using local=true and the old queries need changing

Is "local" missing in package.json?

Here's the thing. On the old multi-schema queries ":table" is used to reference the table but "table" is a reserved word on oracle and you can't use it for a variable name so I changed it to something else on the single-schema queries I think ":thetable". Now we're using bindable parameters and not replaceable ones and it throws an exception on ":table". Need to change the old queries.
The "local" option I put it in temporarily since @mtxr  should name it as he chooses :)

@vrokolos I started to replace react-table as you suggest by dx-react-grid. Slickgrid looks fine, but it lacks of some features that we currently have and lacks also of some features I'm looking forward to add.
I did some benchmarks with that libraries and slickgrid and dx-react-grid is awesome.
Don't worry that I'll adapt this to your PR as soon as I finish doing theme changes.
You can see a preview of the benchmark here:
https://github.com/mtxr/vscode-sqltools/pull/297#issue-299732714

If it works ok then great!
Is it virtualised like slickgrid? You don't need paged results in slickgrid since it automatically creates virtual pages and it can handle millions of records without any loss of speed.
Was searching for one myself. Could use this in my projects if it can do that

@vrokolos yes it does. We can do both. I'll add an option to disable pagination and use virtual scrolling but by default I'll keep as is. Even with slickgrid I got some slowness using it when scrolling very fast. So I'll test it better. But having an option is not a big deal

I've added react-grid to master branch. Can you update your branch? Please, consider every change i've made on ui pakage

I'm on vacation for the next three weeks so it will take some time

@vrokolos can we talk someday next week?
I see many changes all together and this is making it hard to test. There are some stuff that's not working well here when I'm testing with other dialects, so I would like to get more information about it to try to use the same strategy you did for other dialects.
What do you think?


Add authorization in request header

Is it possible to configure the request header and add authorization token before it is sent?

Thanks,

Thanks,
I have a token that can change from one call to another, so I can't just pass  a string, i want to pass it something dynamic, maybe a function.
I understand that loadPath can get a function, can customHeaders get a function as well?
Or is there any way to change this middleware's config after the init phase?

You should be able to modify backend.options, something like:
i18next.services.backendConnector.backend.options.customHeaders = { .... }

Great, thanks!

This still isn't ideal, could it be made to accept a headers function also? Trying to make this as declarative as possible..


[benchmark][LoadGenerator] Process both read/write load requests

Summary:
This commit stack introduces the new features to make Benchmarker to be able to handle generic LoadRequest. It spawns a thread to handle read requests' responses.
Test Plan:
Added a smoke test.
Also tested by adding read requests in both ring and pairwise generators, use a histogram counter to record the size of the response items.
$ ./target/release/ruben -f faucet_for_ruben -s temp_config -g 100 -n 32 -c 4 -e 6 \
-m localhost:45345 -t pairwise -q 3 -k 40
$ ./target/release/ruben -f faucet_for_ruben -s temp_config -g 100 -n 32 -c 4 -e 6 \
-m localhost:45345 -t ring -q 3 -k 40

From offline discussion:
I suggest separating the added functionality (i.e., full support of LoadRequest, without conversion to SubmitTransactionRequest) from the example use that introduces another argument and changes the existing transaction generators.

About renaming TXNs => load requests:

I renamed TXNs to load requests almost everywhere, since load covers both read load and TXN loads, including both code and comments.
The module is also renamed to load_generator.rs.
Except for the case that the load request can only be TXNs. For example, RingTransferTxnGenerator/PairwiseTransferTxnGenerator are generating only TXNs, so TXN still suite them. Another examples include wait_txns_committed and counters for TXNs, since they are TXN specific.
In ruben_opt.rs, the argument is still TransactionPattern because load are only from TXN generator for now, but it will be changed to LoadPattern sooner or later.

Remove the -q parameter and its corresponding feature, leave existing txn generator unchanged.

Rebase to upstream/master.

I don't want us to invent new terminology, and "load request" shouldn't become a thing (since we do not have requests that are free and not generating any load).

OK. I have to admit I am very bad at naming things.
Just to confirm, @fbandersnatch

we should change LoadGenerator and load_generator into RequestGenerator and request_generator.rs?
and struct LoadRequest should be changed to struct Request?

It seems like "load/loads" terminology used here is interchangeable with "requests". Can we simply use requests everywhere, so no one needs to learn new terms?

I will sweep the code to replace load with request in comments.
Also I will avoid using different words when referring to TXN.

LoadGenerator is a fine name. But as part of your name changes in the code + comments you passed through and replaced the word "transaction" or "request" with the word "load", and I don't think that it was a good change -- it's a new non standard name, sometimes used in combination with other names to make it just confusing.

LoadGenerator is a fine name. But as part of your name changes in the code + comments you passed through and replaced the word "transaction" or "request" with the word "load", and I don't think that it was a good change -- it's a new non standard name, sometimes used in combination with other names to make it just confusing.

Ok. I removed "load(s)" from comments/code, except for "load_generator.rs" and "LoadGenerator".
Also rebased to upstream/master.


dist bundle should contain other jars in some common location

-do 'ant dist'
-unzip target/testng-$version.zip
or download the distribution zip from testng.org website and unzip it
-find 'testng-$version-bsh-noguice.jar'
=> for distributed version it is in $TESTNG_HOME/Users/cedric/java/testng/target/other-jars, for own build it can be ie: $TESTNG_HOME/space/sources/testng/target/other-jars
where TESTNG_HOME is a folder where testng has been unzipped.
With this patch other jars are packaged into testng-$version.zip!/testng-$version/other-jars instead

first commit fixes mentioned issue, second one adds ability to run JUnit 4 tests by TestNG and updates junit dependency from v. 3 to version 4


[NFC] Refactor getRenameDecl to use the request evaluator

Move getRenameDecl from PrintAsObjC into its own request so that
other components can re-use this functionality.

@swift-ci please test


How to escape # in query

Hey there,
how would I escape # (the hashtag sign) in a query, since it's usually used as placeholder? Tried various amounts of backslashes but no luck.
Best,
Daniel

I think you have configure QueryFactoy
Can you provide a query example with hastag?


Update to Bootswatch 4?

With MkDocs 1.1, the built-in mkdocs theme will use Bootswatch 4. We'll either need to update the themes here or stop deriving from the mkdocs theme. However, one complication is that the Amelia and Readable themes don't seem to exist in Bootswatch 4. What should we do with those? Just delete them? Trying to update the theme CSS seems like a lot of work...
@waylan Any thoughts? I did a proof of concept with one theme and it seemed fairly straightforward, so as long as we're ok with killing Amelia and Readable, this shouldn't be too bad. Plus, we'd be able to (relatively) easily add all the new Bootswatch themes to this project...

I suppose another option would be for Amelia and Readable to stop inheriting from the mkdocs theme, though it might be better to just get rid of them to reduce the maintenance burden...

I see two options:

Drop support for the themes which Bootswatch dropped.
Keep the old themes.

I'm inclined to do the latter. We can just keep the old CSS and JS in those themes and document that they don't support newer Bootswatch stuff.

Another option might be to make a mkdocs-bootswatch3 package that just makes a copy of the old mkdocs theme and inherits from that. In addition to providing a long-term home for Amelia and Readable, this could be useful for people who can't use Bootstrap 4 for whatever reason (my understanding is that it breaks multi-level dropdowns in the navbar). It'd also provide a somewhat-nice solution to https://github.com/mkdocs/mkdocs/issues/1865, since any third-party theme out there inheriting from the mkdocs theme could instead inherit from mkdocs-classic or whatever we call it.

That seems like a reasonable alternative. However, I do not have the time to work on it. If you want to take it on, go ahead. When you get it set up, let me know and we can see about transferring it to a new repo under the mkdocs organization here on GitHub.

@waylan I'll try and get this set up in the coming week or two. It should be relatively straightforward, since with #55 merged, everything in this repo is fully-compatible with MkDocs 1.0.x as far as I can tell. I think it's just a matter of duplicating this repo, making a few bookkeeping changes, and copying the 1.0.x mkdocs theme in.

Ok, this is pretty much ready: https://github.com/jimporter/mkdocs-bootswatch/tree/bootswatch-4
I'll file a PR once #57 lands and I get the chance to clone this repo to make mkdocs-bootswatch-classic.

@waylan Ok, here's the repo for the "classic" versions of the themes: https://github.com/jimporter/mkdocs-bootswatch-classic
I haven't done anything with mkdocs-bootstrap, but I think I'll update it to Bootstrap 4 and then put the "classic" version in mkdocs-bootswatch-classic for simplicity...

Resolved in #58


feat(e2e): adding playwright e2e tests for /learn page

Checklist:

[x] I have read and followed the contribution guidelines.
[x] I have read and followed the how to open a pull request guide.
[x] My pull request targets the main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine, or GitPod.

Closes #XXXXX

Hi @raisedadead :wave:
This PR contains a simple rewrite of e2e tests in playwright, of the existing cypress tests for /learn page.

Hi @raisedadead @naomi-lgbt 👋
The login functionality is a bigger feature as it will require a global job setup for login. I suggest we roll with this much progress first and track the remaining in an issue, if you guys see that fit.

@raisedadead @naomi-lgbt
I have added an issue to track this request of yours. Rest assured, setting up a login job for playwright is next task in our to-do list.

Are we able to include the test that asserts the quote appears correctly for an authenticated user?

I've added a test for this on this PR https://github.com/freeCodeCamp/freeCodeCamp/pull/52449
Do take a look :)


don't try to start containers

and remove some unused commands/variables.
The rationale is that these containers need validator keys uploaded to
them before they can start and this is managed elsewhere.

Because it makes no sense to start those containers without knowing if the external data they need is there or not.

This is how they're stopped: https://github.com/status-im/nim-beacon-chain/blob/578ea2013a7c95a7a2e78c3b25b9a1818fdf24b7/docker/manage_testnet_hosts.nims#L135
This is how they're started: https://github.com/status-im/nim-beacon-chain/blob/578ea2013a7c95a7a2e78c3b25b9a1818fdf24b7/docker/manage_testnet_hosts.nims#L112
Ansible tasks don't come into play at all.

No, I understand why you want them stopped. I just don't understand why you are removing functionality from the role. You should parametrize it in such a way that it does what you want, rather than removing functionality that can be useful in the future.

That functionality never worked, because containers started without a bootstrap address and validator keys are useless.
If we'll need it in the future, we'll add it in the future.


CI: Cancel previous builds

This will cancel a previous job on the same branch if a new one is started. #544
Should free up some computing power.
Basics

[x] I added a line to /doc/CHANGELOG.md
[x] The PR is rebased with current master.

[ ] References to issues, e.g. close #X, are in the commit messages.
[ ] The buildserver is happy.

Checklist

[x] I have installed and I am using pre-commit hooks
[x] I fully described what my PR does in the documentation
(not in the PR description)
[x] I fixed all affected documentation
[x] I fixed all affected decisions
[ ] I added unit tests for my code
[ ] I added code comments, logging, and assertions as appropriate
[ ] I translated all strings visible to the user
[ ] I mentioned every code or binary not directly written or done by me in reuse syntax
[ ] I created left-over issues for things that are still to be done
[ ] Code is conforming to our Architecture
[ ] Code is conforming to our Guidelines
(exceptions are documented)
[ ] Code is consistent to our Design Decisions

Review

[ ] I've tested the code
[ ] I've read through the whole code
[ ] Examples are well chosen and understandable

Ok so the skipping works now, the issue is, how to handle the missing stashes when deploying PR.

The skipping works, the problem now is, how should we deal with the missing stashes in stage PR deploy.

What are the options? If a job was canceled, it also shouldn't be deployed.

Lets say you just want to change something in the mdbook and also have a new deploy. Or you change something in the frontend and you dont want to rebuild the backend, because its still the same. So we need a way to reuse already build backends. Maybe this is the place for the container registry.
With a container registry, we could always refer to the :latest from master and use that for deployment.

Yes, you need a way to compare with the latest version that was actually built.

Lets do the skip stage in a separate PR, so we can get the cancel previous build going for development.

As we can see, this works and is ready to be merged 🐱

Yeah, great that you also updated docu! :heart:


inconsistant molybdate parents

Hi,
Lauren Brinkac, a TIGR microbial annotator, alerted me
to this problem with the molybdate terms:
The process term "molybdate ion transport" (GO:0015689)
has the parent "cation transport" while the
corresponding function term "molybdate ion transporter
activity" (GO:0015098) has "anion transporter" as a
parent.
The definitions for these terms both indicate that
molybdate is an anion which is our thinking here at
TIGR as well.
Therefore, I wish to request that the process term
"molybdate ion transport" (GO:0015689) be a child of
"anion transport" (GO:0006820) and not "cation transport".
Thanks,
Michelle
Reported by: mlgwinn
Original Ticket: "geneontology/ontology-requests/1454":https://sourceforge.net/p/geneontology/ontology-requests/1454

thanks www.movied.org


refactor: have Sort* and Type* generate fresh level names directly

This is a slight performance improvement because it specializes levelMVarToParam to the case of a single level metavariable.

Note: still need to benchmark to substantiate this being a performance improvement!

!bench

At best, the benchmarking says it's a slight improvement, but they seem inconclusive. It's not worth looking into this anymore for now.


New version: xavidop.cxcli 1.84.2

[ ] Have you signed the Contributor License Agreement?
[ ] Have you checked that there aren't other open pull requests for the same manifest update/change?
[ ] This PR only modifies one (1) manifest
[ ] Have you validated your manifest locally with winget validate --manifest <path>?
[ ] Have you tested your manifest locally with winget install --manifest <path>?
[ ] Does your manifest conform to the 1.4 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Automated with GoReleaser
Microsoft Reviewers: codeflow:open?pullrequest=https://github.com/microsoft/winget-pkgs/pull/113082&drop=dogfoodAlpha

msftbot/validationCompleted


Google Play Console marked Mixpanel SDK as possible advertising

Hi,
Google Play Console marked Mixpanel SDK as possible advertising. Do you use any Ad ID within the lib? Could we omit this issue somehow?

Also see #708

Dup #708


Is the reverse possible with this? SMIL to Bodymovin or Lottie?

Hi, is the reverse possible with this? SMIL to Bodymovin or Lottie?

hi, unfortunately that is not possible


Call/invoke responses next to function cards

Rearrange the UX of the "Interact" UI section to align with Voyager.

@rjnrohit currently the interface is like that

Could you add more information on how you want it to be ?

@rjnrohit currently the interface is like that

Could you add more information on how you want it to be ?

You may add a response box just below the interactive form box in the interact section when a call has been made to any function and when it has received the response