OOP school library: decorate a class

Create a class Nameable.

Implement a method called correct_name that will raise a NotImplementedError.

Turn Person class to Nameable

That Person class inherits from Nameable
That this class has a method correct_name implemented. It should simply return the name attribute.

Prepare base Decorator

that it inherits from Nameable.
In the constructor assign a nameable object from params to an instance variable.
Implement the correct_name method that returns the result of the correct_name method of the @nameable.

Prepare CapitalizeDecorator and TrimmerDecorator
CapitalizeDecorator:

Create a class that inherits from the base Decorator class.
Implement a method correct_name that capitalizes the output of @nameable.correct_name.

TrimmerDecorator:

Create a class that inherits from the base Decorator class.
Implement a method correct_name that makes sure that the output of @nameable.correct_name has a maximum of 10 characters. If it's longer it should trim the word.

brother please help me i face a problem on day 4 and i spent 2 days on it but didn't fix it please help me I will be very grateful to you it's about  (execution permission).Sent from my Verizon, Samsung Galaxy smartphone
-------- Original message --------From: David Bassey @.> Date: 1/19/23  10:21 AM  (GMT+05:00) To: RashidMenhas/oop-school-library @.> Cc: Rashid Menhas @.>, Mention @.> Subject: Re: [RashidMenhas/oop-school-library] OOP school library: decorate a class (PR #2)
@Davidosky007 requested changes on this pull request.
Hi @RashidMenhas 👋,
Good job so far!
There are some issues that you still need to work on to go to the next project but you are almost there!
Highlights
No linter errors 👍
Good looking readme file 👍
Required Changes ♻
Check the comments under the review.
Optional suggestions
Every comment with the [OPTIONAL] prefix is not crucial enough to stop the approval of this PR. However, I strongly recommend you take them into account as they can make your code better.
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me @davidsoky007 in your question so I can receive the notification.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.
As described in the Code reviews limits policy you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.
In capitalizedecorator.rb:

@@ -0,0 +1,12 @@

+require 'BaseDecorator'
Kindly import the right file properly as the ./basedecorator.rb is in lowercase
In person.rb:

@@ -0,0 +1,30 @@

+require 'nameable'

+class Person < Nameable

attr_reader :id

attr_accessor :name, :age

def initialize(age, name = 'unknown', parent_permission: true)

super

Kindly add super( ) as I get an argument error as `nameable is not expecting any argument Kindly read more on the super keyword
In basedecorator.rb:

@@ -0,0 +1,12 @@

+require 'nameable'

+class BaseDecorator < Nameable

def initialize(nameable)

super

Kindly add super( ) as I get an argument error as `nameable is not expecting any argument Kindly read more on the super keyword
In basedecorator.rb:

@@ -0,0 +1,12 @@

+require 'nameable'

+class BaseDecorator < Nameable

def initialize(nameable)

super

This is what i get after fixing the issue
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: @.***>
[
{
@.***": "http://schema.org",
@.***": "EmailMessage",
"potentialAction": {
@.***": "ViewAction",
"target": "https://github.com/RashidMenhas/oop-school-library/pull/2#pullrequestreview-1261126907",
"url": "https://github.com/RashidMenhas/oop-school-library/pull/2#pullrequestreview-1261126907",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
@.***": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]


cursor surrounding lines setting is not respected in a new build

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

VS Code Version: 1.92.2
OS Version: Windows_NT x64 10.0.22631

Version: 1.92.2 (system setup)
Commit: fee1edb8d6d72a0ddff41e5f71a671c23ed924b9
Date: 2024-08-14T17:29:30.058Z
Electron: 30.1.2
ElectronBuildId: 9870757
Chromium: 124.0.6367.243
Node.js: 20.14.0
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.22631
Steps to Reproduce:

Make sure your settings for cursor surrounding lines is 0 (or 1)
Open any long document
Try to put cursor on the topmost line
you cannot - it is auto scrolled to leave 5 lines above the cursor
it's pretty pesky

This is due to sticky scrolling being enabled by default. The editor reserves 5 lines to the top for sticky scrolling.

Duplicate. Closed in favor of https://github.com/microsoft/vscode/issues/227750


fix: Added skip setting for 1ES PT tag in PR pipeline

Details
1ES PT adds tags to pipeline runs using the access token provided by ADO. In the case of pipeline runs against forked repos of a GitHub repo, the access token does not have Edit build quality permissions and hence cannot add tags. To unblock those pipelines, skipped tagging by using the option SkipBuildTagsForGitHubPullRequests.
Verified that the only change in the pipeline run is that it do not add "1ES.PT.Unofficial" tag on the Run. In place of that it adds below warning message.
##[warning]1ES PT Warning: Skipping build tags as this build is a pull request form GitHub which does not have permissions to add tags.
Motivation

Context

Pull request checklist

[x] PR title respects Conventional Commits (starts with fix:, feat:, etc, and is suitable for user-facing release notes)
[na] PR contains no breaking changes, OR description of both PR and final merge commit starts with BREAKING CHANGE:
[na] (if applicable) Addresses issue: #0000
[na] Added relevant unit tests for your changes
[x] Ran yarn precheckin
[na] Verified code coverage for the changes made

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Add ''Are you sure you want to delete this file from your computer? It was on your hard drive before you uploaded it to the network.' Warning

If I upload a file to the network and later want to remove my claim I'm asked if I want to remove the file from my computer.
If I'm the one who uploaded the file it makes sense that I should be reminded clicking 'remove this file from my computer' I'm trashing a file that was on my computer before I ever opened the LBRY app.

hi, james, how exactly did you generate this warning? can you please provide a screenshot and steps to generated this warning? so that we can be of assistance. Because from what you saying, i do not get that kind of warning when i am revoke my claim. i included a screenshot of the message i get.

Thanks


"Repeater" question type: implement bare bones

The Repeater Question type will be used to ask applicants for an arbitrary number of repeated entities (children, household members, jobs, cars, etc.). Admins will be able to create Repeated Questions for each Repeater Question (#612), and add these to program blocks. Applicants will be asked Repeated Questions for each repeated entity the applicant lists (#613).

In the future it’d be great to import the text typed into these fields into questions in the repeated blocks -- for instance, applicants type their children’s first names into the multi-text question, and those first names auto-fill into the Name questions when it comes time to fill in details about each child. But that adds a lot of complexity to the UAT admin experience (they have to drill deep into the data model to show what field the text inputs correlate with), so it’s better to leave off for v1.
(Also, there are edge cases where two household members have the same name, so auto-filling information might get messy.)
Make it possible for UAT admins to create, and for applicants to fill out, questions of this type. Answers should be saved to a database. Just implement the sub-fields; configuration options, validation criteria, and custom functionality aren't necessary yet.
See details at https://github.com/seattle-
uat/universal-application-tool/wiki/Question-type-requirements-notes
See more details at
https://docs.google.com/document/d/1jkbFPzG_whZ1AqZVmxmkGuoqR5wRdG817COX5usPAWo/edit#heading=h.vkhepsgmicsf

See this wiki page to learn how to add a new question type to the codebase!

In progress, moving to M5.


Feature Request: Add support for auth partitions with route domains

Please add the ability to create auth partitions which then have default route domains.  This would be an enhancement to the way https://github.com/F5Networks/f5-declarative-onboarding/issues/10 was requesting route domain usage.
The workflow we are trying to replicate is
Provider A VLAN - Configure untagged on interface 1.1
tmsh create net vlan vlan$PROVIDER_A_VLAN_TAG$ interfaces add { 1.1 { } } tag $PROVIDER_A_VLAN_TAG$
Provider B VLAN - Configure untagged on interface 1.2
tmsh create net vlan vlan$PROVIDER_B_VLAN_TAG$ interfaces add { 1.2 { } } tag $PROVIDER_B_VLAN_TAG$
Create route domain and assign VLANs to it
tmsh create net route-domain $ROUTE_DOMAIN_NAME$ id $ROUTE_DOMAIN_ID$ vlans add { vlan$PROVIDER_A_VLAN_TAG$ vlan$PROVIDER_B_VLAN_TAG$ }
Create admin partition and set $ROUTE_DOMAIN_NAME$ as the default RD in that partition
tmsh create auth partition $PARITION_NAME$ default-route-domain $ROUTE_DOMAIN_ID$ description "$ROUTE_DOMAIN_DESCRIPTION$"
Create self-IPs on necessary VLANs
Provider A
tmsh create net self /$PARTITION_NAME$/vlan$PROVIDER_A_VLAN_TAG$-v4 address $PROVIDER_A_IPV4_SELF$/$PROVIDER_A_IPV4_PREFIX$ traffic-group traffic-group-local-only vlan vlan$PROVIDER_A_VLAN_TAG$
tmsh create net self /$PARTITION_NAME$/vlan$PROVIDER_A_VLAN_TAG$-v6 address $PROVIDER_A_IPV6_SELF$/$PROVIDER_A_IPV6_PREFIX$ traffic-group traffic-group-local-only vlan vlan$PROVIDER_A_VLAN_TAG$

The ability to create auth partitions with a specified default route domain is already available in AS3. Can you use AS3 for that part of the configuration?

Closing for inactivity. If you still have an issue, please reopen this issue with answers to the previous question.

+1 for this request. I have the same issue that @steveh565 outined above. In AWS I have 2 devices deployed to different AZ's, and I now need to cluster them with DO. Because they are in different AZ's, I need them to have different default routes, so I need to create a specially configured LOCAL_ONLY partition that is not part of cluster device-group. So I need this to be available via DO or AS3, otherwise I'm forced to use bash commands which is not ideal.

I have re-opened the question via a new issue #141


Automated cherry pick of #3390: disk list add manual snapshot count

Cherry pick of #3390 on release/2.11.
#3390: disk list add manual snapshot count

Codecov Report

:exclamation: No coverage uploaded for pull request base (release/2.11@9a88408). Click here to learn what that means.
The diff coverage is 0%.

@@              Coverage Diff               @@
##             release/2.11   #3395   +/-   ##
==============================================
  Coverage                ?   7.13%
==============================================
  Files                   ?     476
  Lines                   ?   77286
  Branches                ?       0
==============================================
  Hits                    ?    5516
  Misses                  ?   71307
  Partials                ?     463

Flag
Coverage Δ

#aFlag
7.13% <0%> (?)

Impacted Files
Coverage Δ

pkg/compute/models/disks.go
0.64% <0%> (ø)

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 9a88408...545df80. Read the comment docs.

/lgtm

/approve


Add common use cases

Description

Add common Use Cases

Related Issue(s)

Checklist

[ ] I have read the contribution guidelines
[ ] I have considered the performance impact of these changes
[ ] Suitable unit/system level tests have been added and they pass
[ ] Documentation has been updated

@MarianRaphael Can you take a look at my comment above?


EufySecurity    ERROR        Not connected can't continue!

What happened?
What can I do to fix it?
EufySecurity     Finished initializing Eufy Security Platform
EufySecurity     ERROR        Not connected can't continue!

Hello,
Can you enable debug and post the log output ?

8/22/2021, 12:55:32 PM Home[2021-08-22T16:55:32.440Z]  INFO: ef/2744 on v0.3.36: Finished initializing Eufy Security Platform
8/22/2021, 12:55:32 PM Home[2021-08-22T16:55:32.553Z] DEBUG: ef/2744 on v0.3.36: discoveringDevices
8/22/2021, 12:55:34 PM Home[2021-08-22T16:55:34.056Z] DEBUG: ef/2744 on v0.3.36: EufyClient connected false
8/22/2021, 12:55:34 PM Home[2021-08-22T16:55:34.057Z] ERROR: ef/2744 on v0.3.36: Not connected can't continue!
8/22/2021, 12:55:35 PM Home WARNING unhandled rejection: Error: Request failed: https://mysecurity.eufylife.com/api/v1/passport/login -> 100028 - Failed to request.

Can you update to 1.0.0-rc2 and do the Start over the wizard process within the plugin config ?

it's on 1.0.0-rc2 and I'm using Hoobs

I did a factory reset to the Hoobs and installed 1.0.0-rc2 version and put the username and password but still
8/22/2021, 3:16:27 PMEufy Security Bridge starting
8/22/2021, 3:16:32 PMEufy Security BridgeInitializing HAP-NodeJS v0.9.4...
8/22/2021, 3:16:53 PMEufy Security BridgeLoaded plugin 'homebridge-eufy-security'
8/22/2021, 3:16:53 PMEufy Security BridgeRegistering platform 'homebridge-eufy-security.EufySecurity'
8/22/2021, 3:16:53 PMEufy Security BridgeLoading 1 platforms...
8/22/2021, 3:16:53 PMEufy Security BridgeEufySecurityFinished initializing Eufy Security Platform
8/22/2021, 3:16:54 PMEufy Security BridgePreparing Advertiser for 'Eufy Security Bridge F892' using bonjour-hap backend!
8/22/2021, 3:16:54 PMEufy Security BridgeStarting to advertise 'Eufy Security Bridge F892' using bonjour-hap backend!
8/22/2021, 3:16:54 PMEufy Security BridgeBridge is running on port 51826.
8/22/2021, 3:16:54 PMEufy Security BridgeEufySecurityERRORNot connected can't continue!
8/22/2021, 3:17:20 PMEufy Security BridgeConfiguration change
8/22/2021, 3:17:20 PMeufysecuritybridge stopping
8/22/2021, 3:17:25 PMEufy Security Bridge starting
8/22/2021, 3:17:30 PMEufy Security BridgeInitializing HAP-NodeJS v0.9.4...
8/22/2021, 3:17:34 PMEufy Security BridgeLoaded plugin 'homebridge-eufy-security'
8/22/2021, 3:17:34 PMEufy Security BridgeRegistering platform 'homebridge-eufy-security.EufySecurity'
8/22/2021, 3:17:34 PMEufy Security BridgeLoading 1 platforms...
8/22/2021, 3:17:34 PMEufy Security Bridge[2021-08-22T19:17:34.567Z]  INFO: ef/12217 on v1.0.0-rc.2: Eufy Security Plugin: enableDetailedLogging on
8/22/2021, 3:17:34 PMEufy Security Bridge[2021-08-22T19:17:34.592Z]  INFO: ef/12217 on v1.0.0-rc.2: Finished initializing Eufy Security Platform
8/22/2021, 3:17:34 PMEufy Security Bridge[2021-08-22T19:17:34.597Z] DEBUG: ef/12217 on v1.0.0-rc.2: discoveringDevices
8/22/2021, 3:17:34 PMEufy Security BridgePreparing Advertiser for 'Eufy Security Bridge F892' using bonjour-hap backend!
8/22/2021, 3:17:34 PMEufy Security BridgeStarting to advertise 'Eufy Security Bridge F892' using bonjour-hap backend!
8/22/2021, 3:17:34 PMEufy Security BridgeBridge is running on port 51826.
8/22/2021, 3:17:35 PMEufy Security Bridge[2021-08-22T19:17:35.224Z] DEBUG: ef/12217 on v1.0.0-rc.2: EufyClient connected false
8/22/2021, 3:17:35 PMEufy Security Bridge[2021-08-22T19:17:35.226Z] ERROR: ef/12217 on v1.0.0-rc.2: Not connected can't continue!

8/22/2021, 3:53:40 PM Home [2021-08-22T19:53:40.659Z] DEBUG: ef/14550 on v1.0.0-rc.2: discoveringDevices
8/22/2021, 3:53:40 PM Home [2021-08-22T19:53:40.676Z] DEBUG: ef/14550 on v1.0.0-rc.2: Authenticate and get an access token { token: null, tokenExpiration: null }
8/22/2021, 3:53:40 PM Home[2021-08-22T19:53:40.683Z] DEBUG: ef/14550 on v1.0.0-rc.2:
8/22/2021, 3:53:40 PM Home    Request: {
8/22/2021, 3:53:40 PM Home method: 'post',
8/22/2021, 3:53:40 PM Home endpoint: 'passport/login',
8/22/2021, 3:53:40 PM Home baseUrl: 'https://mysecurity.eufylife.com/api/v1',
8/22/2021, 3:53:40 PM Home token: null,
8/22/2021, 3:53:40 PM Home data: { email<EMAIL_ADDRESS>password: 'XXX' },
8/22/2021, 3:53:40 PM Home headers: {
8/22/2021, 3:53:40 PM Home app_version: 'v2.8.0_887',
8/22/2021, 3:53:40 PM Home os_type: 'android',
8/22/2021, 3:53:40 PM Home os_version: '30',
8/22/2021, 3:53:40 PM Home phone_model: 'EUFYCLIENT',
8/22/2021, 3:53:40 PM Home country: 'US',
8/22/2021, 3:53:40 PM Home language: 'en',
8/22/2021, 3:53:40 PM Home openudid: '327a9b5618b0bc3',
8/22/2021, 3:53:40 PM Home uid: '',
8/22/2021, 3:53:40 PM Home net_type: 'wifi',
8/22/2021, 3:53:40 PM Home mnc: '02',
8/22/2021, 3:53:40 PM Home mcc: '262',
8/22/2021, 3:53:40 PM Home sn: '6b2514e00ac2',
8/22/2021, 3:53:40 PM Home Model_type: 'PHONE',
8/22/2021, 3:53:40 PM Home timezone: 'GMT-04:00'
8/22/2021, 3:53:40 PM Home }
8/22/2021, 3:53:40 PM Home }
8/22/2021, 3:53:40 PM Home
For what it seems it can't get the token so it can log in

I uninstalled it, installed it again, did the username and password on the script and now it got it connected but still its not showing the cameras and I'm getting the warning at the bottom. I only get that warning with Eufy if I uninstall it it doesn't show it.
8/23/2021, 2:03:02 AM Home EufySecurity Finished initializing Eufy Security Platform
8/23/2021, 2:03:02 AM Home EufySecurity discoveringDevices
8/23/2021, 2:03:03 AM Home Setup URI 'X-HM://0023ISYWY06IT'
8/23/2021, 2:03:03 AM Home EufySecurityEufyClient connected true
8/23/2021, 2:03:03 AM Home EufySecurityPollingInterval: 10
8/23/2021, 2:03:03 AM Home EufySecurityRefresh data from cloud and schedule next refresh.
8/23/2021, 2:03:03 AM Home Starting to advertise 'Home 0D7C' using bonjour-hap backend!
8/23/2021, 2:03:03 AM Home Bridge is running on port 51826.
8/23/2021, 2:03:08 AM Home EufySecurityFound 2 stations.
8/23/2021, 2:03:08 AM Home EufySecurityFound Station Serial Number of my device Home Base STATION { value: '<IP_ADDRESS>', timestamp:<PHONE_NUMBER>000 }
8/23/2021, 2:03:08 AM Home WARNING unhandled rejection: TypeError: Cannot read property 'indexOf' of undefined

Can you do that again but enable debug straigh away ?

8/23/2021, 3:36:13 AMHome[2021-08-23T07:36:13.416Z]  INFO: ef/3091 on v1.0.0-rc.2: Eufy Security Plugin: enableDetailedLogging on
8/23/2021, 3:36:13 AMHome[2021-08-23T07:36:13.438Z]  INFO: ef/3091 on v1.0.0-rc.2: Finished initializing Eufy Security Platform
8/23/2021, 3:36:13 AMHome[2021-08-23T07:36:13.537Z] DEBUG: ef/3091 on v1.0.0-rc.2: discoveringDevices
8/23/2021, 3:36:13 AMHome[2021-08-23T07:36:13.913Z] DEBUG: ef/3091 on v1.0.0-rc.2: EufyClient connected true
8/23/2021, 3:36:13 AMHome[2021-08-23T07:36:13.915Z] DEBUG: ef/3091 on v1.0.0-rc.2: PollingInterval: 10
8/23/2021, 3:36:13 AMHome[2021-08-23T07:36:13.916Z] DEBUG: ef/3091 on v1.0.0-rc.2: Refresh data from cloud and schedule next
8/23/2021, 3:36:32 AMHome[2021-08-23T07:36:32.861Z] DEBUG: ef/3091 on v1.0.0-rc.2: Found 2 stations.
8/23/2021, 3:36:32 AMHome[2021-08-23T07:36:32.880Z] DEBUG: ef/3091 on v1.0.0-rc.2: Found Station T8010P1320450AE4 Home Base STATION { value: '<IP_ADDRESS>', timestamp:<PHONE_NUMBER>000 }
8/23/2021, 3:36:32 AMHomeWARNINGunhandled rejection: TypeError: Cannot read property 'indexOf' of undefined
8/23/2021, 3:36:43 AMHome[2021-08-23T07:36:43.985Z] DEBUG: ef/3091 on v1.0.0-rc.2: Push Connected!

I put the default settings in Pollong intervals, ignoring devices and ignoring stating, and it shows them but it doesn't open them
8/23/2021, 3:52:52 AMHomeEufySecurityRestoring existing accessory from cache: Home Base
8/23/2021, 3:52:52 AMHomeEufySecurityWARNINGBackyard looks station but it's not could imply some errors Type: 3
8/23/2021, 3:52:52 AMHomeEufySecurityRestoring existing accessory from cache: Backyard
8/23/2021, 3:52:52 AMHomeEufySecurityRestoring existing accessory from cache: Front of the House
8/23/2021, 3:52:52 AMHomeEufySecurityWARNINGBackyard Looks like not compatible with hasBattery
8/23/2021, 3:52:52 AMHomeEufySecurityRestoring existing accessory from cache: Backyard
8/23/2021, 3:55:56 AMHome[Backyard] The image snapshot handler for the given accessory is slow to respond! See https://git.io/JtMGR for more info.
8/23/2021, 3:56:02 AMHome[Front of the House] The image snapshot handler for the given accessory is slow to respond! See https://git.io/JtMGR for more info.
8/23/2021, 3:56:04 AMHomeEufySecurityStarting video stream: 1280 x 720, 30 fps, 299 kbps Backyard
8/23/2021, 3:56:16 AMHomeEufySecurityDevice appears to be inactive. Stopping stream. Backyard
8/23/2021, 3:56:16 AMHomeEufySecurityStopped video stream. Backyard

can you share your device to me ?
<EMAIL_ADDRESS>

I just send you the invite

Now it's working but it takes long time to load
Also the logs are showing this:
8/23/2021, 4:17:39 AM Eufy ERROR [Eufy A2B7@Backyard@Enabled@On] Unhandled error thrown inside read handler for characteristic: Cannot read property 'value' of undefined
8/23/2021, 4:17:39 AM Eufy ERROR [Eufy A2B7@Backyard@Motion@On] Unhandled error thrown inside read handler for characteristic: Cannot read property 'value' of undefined

log.txt

Was able to reproduce, look like lib issue need more time to identify where and push a PR to bropat

Ok at least you could find what us happening

erf sorry Can I ask you to reshare to me ? I'm not getting the same issue. Looks like FLASHLIGHT CAMERA related.

Can you test on rc3 ?

It takes a long time to stream the cameras, sometimes it works and sometimes it just shows No Response this camera is not responding message in HomeKit

Eufy[2021-08-24T02:09:08.776Z] DEBUG: ef/28139 on v1.0.0-rc.3: FFmpeg exited with code: null and signal: SIGKILL (Expected) Front of the House true

log.txt


ractive.el vs ractive.target

Description:
"target" seems to be preferred to "el" in the object literal to Initialize a Ractive instance.  But is there a .target property on the created Ractive instance, or only a .el property?  If only a .el property, this feels unbalanced.
Versions affected:
Platforms affected:
Reproduction:
Code:

It would make sense to have a target property to match the init option. Out of curiosity, what do you use ractive.el for?

We are burying a ractive instance inside a custom element.  Sometimes we
need access to the element for its properties and attributes.
On Mon, May 29, 2017, 10:24 PM Chris Reeves<EMAIL_ADDRESS>wrote:

It would make sense to have a target property to match the init option.
Out of curiosity, what do you use ractive.el for?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/ractivejs/ractive/issues/2977#issuecomment-304725828,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACAWhwDx8-LifBxIsw7-gzOQXcIplhveks5r-ylngaJpZM4NoH9r
.


Reverting for now so we can release.


Fix table

@yoichinak : Thanks for your contribution! The author(s) have been notified to review your proposed change.


The addon is not compatible with knobs @storybook/addon-knobs anymore

Describe the bug
Hi there, to be honest, I don't know who to address this issue: info addon or knobs 🤷♀ .  But there is a bug: If info is activated for a story - knobs work only once and then not updating anymore.
To Reproduce
Steps to reproduce the behavior:

Go to https://storybook-addon-vue-info.netlify.com/?path=/story/compatibilities-storybook-addon-knobs--panel
Click on the "Knobs" tab
Type something in the "Label" input.
See that the first time you write something - the data in the story will update, but then, if you will type something else - you will not see that changes are applying

Expected behavior
Stories should react on knobs changing each time
Versions
I have the same issue on my local storybook, and I have these versions there:

storybook-addon-vue-info: 1.3.2
@storybook/vue: 5.2.6

Additional context
Please don't mind me if it is a "knobs" issue, just want you to know these addons are not compatible anymore.
btw Info addon is amazing!

Probably https://github.com/storybookjs/storybook/issues/8165 (Duplicate of #116).

@angelika94
TL;DR
This is an addon-knobs' bug, and could be fixed at<EMAIL_ADDRESS>Workarounds:

Upgrading Storybook to v5.3.0-alpha (i don't know when the stable release comes BTW).
Downgrading to<EMAIL_ADDRESS>Using addon-docs, which uses this addon's loader (docgen loader) under the hood.

I'm not sure whether the patch will be included in the future v5.2.x releases :confused:
The easiest way I think is upgrading to v5.3.0, however, it is still in an alpha stage.


Structured & packaged & some imports bugs fixed

Hi Philippe,
I made a slight refactoring of the package to make it installable and useful for other projects.
To install, just need python setup.py install. You can also add some description (e.g. your contacts, citing etc.) and add it to PyPi - I guess it will be very useful for text summarization researchers & developers :).

Thanks a lot, Akim! I'll look it over in the next week and will try to merge and make it into a pip package! Thanks so much for your help with this :)

Hey Akim,
Thanks again for the work.  I'd love to know what you think of the release and if it all works on your end!
Philippe

Hi Philippe,
Sure, we actively use your model in our research, I will launch several experiments this week using your release and will return with a feedback!
Akim


gluegun Dependency Error on Fresh Install

Greetings,
Just attempted to try solidarity for the first time and hit a dependency issue.
Repro steps (on a fresh project):
yarn add -D solidarity
yarn solidarity
Error is:
$ "./node_modules/.bin/solidarity"
module.js:529
    throw err;
    ^

Error: Cannot find module './node_modules/solidarity/bin/../node_modules/gluegun/sniff'
    at Function.Module._resolveFilename (module.js:527:15)
    at Function.Module._load (module.js:476:23)
    at Module.require (module.js:568:17)
    at require (internal/module.js:11:18)
    at Object.<anonymous> (./node_modules/solidarity/bin/solidarity:3:38)
    at Module._compile (module.js:624:30)
    at Object.Module._extensions..js (module.js:635:10)
    at Module.load (module.js:545:32)
    at tryModuleLoad (module.js:508:12)
    at Function.Module._load (module.js:500:3)
error Command failed with exit code 1.

That's pretty interesting!  Can you verify what version of gluegun got pulled in?

@ElliotChong - fixed!  Thanks so much for the catch.  Be sure to use latest from npm.


Add a firstBy aggregator

Any reason for using the name firstBy instead of minBy? I think the latter is more precise, not to mention that it is consistent with the name on Aggregator. Also, I would add a corresponding maxBy method for completeness.

@dkristian, @triggerNZ suggested firstBy, but I agree minBy will be more consistent. Adding a maxBy is also a good idea.

:+1: to the latest changes


 This command can only be run *outside* an existing app

An alias would be nice too.

Fixed in 2cf7cfc - supports -v and --version


Can't Use --predict as Last Option While Executing submit.py

While executing submit.py, --help shows that -p (--predict) can be used as the last option before entering the model name. However, doing so causes an invalid choice error.

The --help message is automatically generated as part of argparse. I can look further into the options for each argument to see if there is a specific configuration of options that allows us to have a positional argument anywhere.


waybar/custom tooltips

What has happened to tooltips in custom modules?
I have a weather app which returns 2 strings, 'text', and 'tooltip' which used to display on hover, but now 'text' is displayed instead?
I have downgraded waybar to previous version.

OK, solved this - I had tooltip-format statements in my custom modules. Removing those has fixed this issue.
However, how do I apply pango markup eg fonts to my tooltips if I can't use a tooltip-format?

It's possible, but need to mind the levels of escaping. The below works for me.
I think this was only fairly recently fixed with a453ea3c70195803e9962a97f9243c0c78d4ecdc. That should be part of 0.10.3 and later.
#!/bin/sh

TEXT="";
TOOLTIP="DND is enabled";

if [ -e _testxyz ];then
  TEXT="";
  TOOLTIP="Notifications are enabled";
  printf '{"text": "%s", "tooltip": "%s"}
' "$TEXT" "$TOOLTIP"
fi

printf '{}'

{
    "layer": "top", // Waybar at top layer
    "position": "top", // Waybar position (top|bottom|left|right)
    "height" : 20,
    "modules-center": ["custom/test-output"],

    "custom/test-output": {
      "format": "{} icon",
      "exec": "./test-output2.sh",
      "return-type":"json",
      "hide-empty-text": true,
      "interval": 1,
    },
}


[Bibdata] Traject::Indexer::ConfigLoadError: Error loading configuration file /opt/marc_liberation/releases/20200417185535/marc_to_solr/lib/traject_config.rb:29 Faraday::SSLError:SSL_connect returned=1 errno=0 state=error: certificate verify failed (certificate has expired)

Backtrace
line 7 of [PROJECT_ROOT]/config/initializers/traject_initializer.rb: setup_indexer
line 29 of [PROJECT_ROOT]/config/initializers/traject_initializer.rb: <top (required)>
line 7 of [PROJECT_ROOT]/config/environment.rb: <top (required)>

View full backtrace and more info at honeybadger.io

This was triggered when: https://github.com/pulibrary/marc_liberation/blob/c57963655aaf7bbef95a4b50b30a41882dcc8bed/config/schedule.rb#L90.
Similar error when: https://github.com/pulibrary/marc_liberation/blob/c57963655aaf7bbef95a4b50b30a41882dcc8bed/config/schedule.rb#L45
I checked the certificates for catalog-staging, catalog, bibdata, bibdata-staging, lib-solr8-prod and lib-solr8-staging they all seem to be valid with an expiration date on 2021 and catalog on 2022.

resolved by https://github.com/pulibrary/princeton_ansible/pull/1547


Implement all basic clustermgt commands in client

Currently there is a ClusterMgt class that implements some nice functions to deal with a cluster. However the basic commands is not implemented in the client class. All cluster mgt methods should be implemented in the base client and the ClusterMgt class can be used to provide a better use and interface to deal with the cluster.

+1

// , :+1: This will make the code that uses this less confusing.
Until resolution, should we use a different library (redis-py) for non-cluster commands that deal with a given Redis instance directly?

@v6 If you are thinking about the support that has a PR by iandy has open, yes i would suggest using that for now. I am basing my implementation on that work but with fixes for how it will work when running in a proper cluster. Because in a cluster client, you do not connection to a single node and runs the commands, but they are global and needs targeting inside some of the commands. There is need for development when switching between the 2 implementations. The only other solution would be to implement all commands inside the ClusterMgmt class that we already have here because it works in a similar fashion. I will continue with the raw commands and see how usefull the ClusterMgmtclass is in the future and maybe remove it completly or maybe i will make that one the choice. I am not sure yet.


Add performance tests to presubmit

Maybe adding them now is of no use, but at least building them would be useful. The motivation is that #2440 seems to have broken the tests.
/area test-and-release
/kind feature
/kind good-first-issue
cc @jessiezcc @srinivashegde86

Sorry, wrong repository


Broadcast message

Hi,
can I broadcast message to all subscribers?

have you seen this FAQ?

As I asked here: http://stackoverflow.com/q/38686587/301198
You can (with the limitations of the API FAQ) but you have to store yourself all the chat ids you ever received and then just send messages to all those chats.

Tanx Cristian

Tanx Cristian


Chip: Remove special palette variables

Description

Controversial, but I don't like how MudChip is one of the only components that has its own color properties, especially when it's only for the default chip. I was inspired to revert that after seeing #8529.
If someone customizes their palette the typical way, their chips will look out of place because it deviates from the global style.
Open to other ideas including this PR being denied if it's not the right move.

Reverts most of https://github.com/MudBlazor/MudBlazor/pull/7406/commits/b681da1f80962678505c002cbcd8ac25e7c80990
Resolves https://github.com/MudBlazor/MudBlazor/issues/8529

How Has This Been Tested?

visually
Types of changes

[X] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[X] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Can't see the selected chips:
https://github.com/MudBlazor/MudBlazor/assets/7112040/f84d2c2f-1ff4-4ef9-93f5-5d4e7ffef07f
After:
https://github.com/MudBlazor/MudBlazor/assets/7112040/4c8ee8c1-c09c-4404-ae60-1e8435a318bb
https://github.com/MudBlazor/MudBlazor/assets/7112040/e8721024-884f-4d71-88a5-1649988de3ab
Checklist

[X] The PR is submitted to the correct branch (dev).
[X] My code follows the code style of this project.
[ ] I've added relevant tests.

Requires #8598 to be merged first

@Garderoben we need your input here

Thanks!
Added to v7.0.0 Migration Guide #8447


[DNM] Testing tempest / tobiko secrets

Depends-On: https://github.com/openstack-k8s-operators/ci-framework/pull/1271

No longer needed.


helm-install-traefik fails with the error "exec /usr/bin/entry: exec format error"

Environmental Info:
K3s Version:
k3s version v1.26.4+k3s1 (8d0255af)
go version go1.20.4

Node(s) CPU architecture, OS, and Version:
Linux crls-kube-pi2 6.1.21 #1-NixOS SMP Tue Jan  1 00:00:00 UTC 1980 aarch64 GNU/Linux

Cluster Configuration:
1 server, 3 agents
Describe the bug:
helm-install-traefik fails with the error "exec /usr/bin/entry: exec format error"
Steps To Reproduce:

Upgrade k3s from 1.25.3+k3s1 to 1.26.4+k3s1

It seems like I somehow ended up with the wrong arch for rancher/klipper-helm:v0.7.7-build20230403, but I am not entirely certain, and I don't know how to verify that - k3s crictl seems to give fairly limited information about images.

What model of Pi are you running this on? I'm unable to reproduce this - tests look fine on our aarch64 CI runners, and it runs fine on my personal Pi4b.
sysadm@pi02:~$ uname -a
Linux pi02.lan.khaus 5.19.0-1016-raspi #23-Ubuntu SMP PREEMPT Fri Mar 31 13:54:29 UTC 2023 aarch64 aarch64 aarch64 GNU/Linux

sysadm@pi02:~$ sudo ctr run --rm --tty docker.io/rancher/klipper-helm:v0.7.7-build20230403 klipper-helm
if [[ ${KUBERNETES_SERVICE_HOST} =~ .*:.* ]]; then
	echo "KUBERNETES_SERVICE_HOST is using IPv6"
	CHART="${CHART//%\{KUBERNETES_API\}%/[${KUBERNETES_SERVICE_HOST}]:${KUBERNETES_SERVICE_PORT}}"
else
	CHART="${CHART//%\{KUBERNETES_API\}%/${KUBERNETES_SERVICE_HOST}:${KUBERNETES_SERVICE_PORT}}"
fi

Are you perhaps using a local registry mirror, and have pushed a single arch-specific image it, instead of the multi-arch manifest list?

What model of Pi are you running this on? I'm unable to reproduce this - tests look fine on our aarch64 CI runners, and it runs fine on my personal Pi4b

I'm also running this on rpi-4b!
Still here's the output I get:
> sudo k3s ctr --address "/run/containerd/containerd.sock" run --rm --tty docker.io/rancher/klipper-helm:v0.7.7-build20230403 klipper-helm
exec /usr/bin/entry: exec format error

Are you perhaps using a local registry mirror, and have pushed a single arch-specific image it, instead of the multi-arch manifest list?

Not unless that's the default configuration, which I guess is not the case and I don't remember to have changed anything in that area.
The image does seem to get pulled very quickly after it has been deleted, but since its size is pretty small, I can't really tell whether that indicates anything.
I noticed that the sha256 digest of docker.io/rancher/klipper-helm:v0.7.7-build20230403 doesn't seem to match anything that's published here - https://registry.hub.docker.com/r/rancher/klipper-helm/tags?page=1&name=v0.7.7-build20230403
> sudo k3s ctr --address "/run/containerd/containerd.sock" images ls
REF                                                                                TYPE                                                      DIGEST                                                                  SIZE      PLATFORMS                                                                    LABELS
docker.io/rancher/klipper-helm:v0.7.7-build20230403                                                    application/vnd.docker.distribution.manifest.list.v2+json sha256:3cd71ccc3cce5010865d9ab3548788a3a8cf5e9cfc5b48cc54b5b815675dd121 80.5 MiB  linux/amd64,linux/arm,linux/arm64,linux/s390x                                io.cri-containerd.image=managed
docker.io/rancher/klipper-helm@sha256:3cd71ccc3cce5010865d9ab3548788a3a8cf5e9cfc5b48cc54b5b815675dd121 application/vnd.docker.distribution.manifest.list.v2+json sha256:3cd71ccc3cce5010865d9ab3548788a3a8cf5e9cfc5b48cc54b5b815675dd121 80.5 MiB  linux/amd64,linux/arm,linux/arm64,linux/s390x                                io.cri-containerd.image=managed
registry.k8s.io/pause:3.8                                                                              application/vnd.docker.distribution.manifest.list.v2+json sha256:9001185023633d17a2f98ff69b6ff2615b8ea02a825adffa40422f51dfdcde9d 262.1 KiB linux/amd64,linux/arm/v7,linux/arm64,linux/ppc64le,linux/s390x,windows/amd64 io.cri-containerd.image=managed
registry.k8s.io/pause@sha256:9001185023633d17a2f98ff69b6ff2615b8ea02a825adffa40422f51dfdcde9d          application/vnd.docker.distribution.manifest.list.v2+json sha256:9001185023633d17a2f98ff69b6ff2615b8ea02a825adffa40422f51dfdcde9d 262.1 KiB linux/amd64,linux/arm/v7,linux/arm64,linux/ppc64le,linux/s390x,windows/amd64 io.cri-containerd.image=managed
sha256:48957d5c01a77280be19ca9f4ba00eaed44a6b5b60028d4879adcfa9b361b976                                application/vnd.docker.distribution.manifest.list.v2+json sha256:3cd71ccc3cce5010865d9ab3548788a3a8cf5e9cfc5b48cc54b5b815675dd121 80.5 MiB  linux/amd64,linux/arm,linux/arm64,linux/s390x                                io.cri-containerd.image=managed
sha256:4e42fb3c9d90ed7895bc04a9d96fe3102a65b521f485cc5a4f3dd818afef9cef                                application/vnd.docker.distribution.manifest.list.v2+json sha256:9001185023633d17a2f98ff69b6ff2615b8ea02a825adffa40422f51dfdcde9d 262.1 KiB linux/amd64,linux/arm/v7,linux/arm64,linux/ppc64le,linux/s390x,windows/amd64 io.cri-containerd.image=managed

I wonder if perhaps the NixOS kernel is missing something? Are you able to reproduce this on PiOS, Debian, or Ubuntu?
That's the correct image.
sysadm@pi02:~$ sudo ctr images ls name==docker.io/rancher/klipper-helm:v0.7.7-build20230403
REF                                                 TYPE                                                      DIGEST                                                                  SIZE     PLATFORMS                                     LABELS
docker.io/rancher/klipper-helm:v0.7.7-build20230403 application/vnd.docker.distribution.manifest.list.v2+json sha256:3cd71ccc3cce5010865d9ab3548788a3a8cf5e9cfc5b48cc54b5b815675dd121 80.5 MiB linux/amd64,linux/arm,linux/arm64,linux/s390x io.cri-containerd.image=managed

I wonder if perhaps the NixOS kernel is missing something?

I'm actually using the rpi-6.1 kernel with that NixOS installation, so I don't think that should be a problem.

Are you able to reproduce this on PiOS, Debian, or Ubuntu?

I'm pretty sure that I wouldn't be able to reproduce with a fresh install on any OS, including NixOS. I think I messed something up in between the version upgrade, had to uninstall traefik and now am stuck with its reinstallation.
That said, I would like to avoid nuking and salvage this setup somehow...

That's the correct image

Good to know, thanks!

Did you perhaps grab the arm32 version of k3s instead of arm64 when upgrading?

Did you perhaps grab the arm32 version of k3s instead of arm64 when upgrading?

I checked and it doesn't look like it
> file /nix/store/h5k23rfvh7fkriwlhgbvxjkjpkh9g51x-k3s-1.26.4+k3s1/bin/.k3s-wrapped
/nix/store/h5k23rfvh7fkriwlhgbvxjkjpkh9g51x-k3s-1.26.4+k3s1/bin/.k3s-wrapped: ELF 64-bit LSB executable, ARM aarch64, version 1 (SYSV), statically linked, Go BuildID=FWGNi6WUigIz1MTM0SE5/cgURd-dZEsas9R6vgzDz/QB1xHE3_UIcqalpzKEG3/vemFrxowUkDWLFkKFTKb, stripped

I went ahead and mounted the container to check what binaries it had, and everything was aarch64, although the /usr/bin/entry was empty (?)...
> file  /tmp/containerfiles/bin/busybox
/tmp/containerfiles/bin/busybox: ELF 64-bit LSB pie executable, ARM aarch64, version 1 (SYSV), dynamically linked, interpreter /lib/ld-musl-aarch64.so.1, stripped

> file  /tmp/containerfiles/usr/bin/entry
/tmp/containerfiles/usr/bin/entry: empty

Eventually, I got it fixed by just deleting all images via pulling the image manually, deleting all images, and allowing k3s to pull it again.
sudo k3s ctr --address "/run/containerd/containerd.sock" -n k8s.io i rm $(sudo k3s ctr --address "/run/containerd/containerd.sock" -n k8s.io i ls -q)
sudo k3s ctr --address "/run/containerd/containerd.sock" content fetch docker.io/rancher/klipper-helm:v0.7.7-build20230403

This doesn't really answer how I got into that initial state, but since it works, I am happy to close this issue! Thanks for the support @brandond !

Hmm that's odd. Perhaps some undetected corruption in the containerd image store?

Hmm that's odd. Perhaps some undetected corruption in the containerd image store or overlayfs?

Perhaps. I don't think I'll be able to verify that anymore though :(

Especially on a pi, that is the most likely explanation.

related issue: https://github.com/containerd/containerd/issues/5854


x/review/git-codereview: provide a way to configure gerrit URL without codereview.cfg file in the repository

Please answer these questions before submitting your issue. Thanks!

What version of Go are you using (go version)?

go version go1.5.3 linux/amd64

What operating system and processor architecture are you using (go env)?

GOARCH="amd64"
GOBIN=""
GOEXE=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOOS="linux"
GOPATH="/home/cmdeoliv/go"
GORACE=""
GOROOT="/usr/lib/golang"
GOTOOLDIR="/usr/lib/golang/pkg/tool/linux_amd64"
GO15VENDOREXPERIMENT=""
CC="gcc"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0"
CXX="g++"
CGO_ENABLED="1"

What did you do?
If possible, provide a recipe for reproducing the error.
A complete runnable program is good.
A link on play.golang.org is best.

Trying to use git-codereview with a Gerrit repository that doesn't have codereview.cfg configuration file. It is not always possible to add the file to upstream repository, but git-codereview features that depend on Gerrit could still be used.

What did you expect to see?

git-codereview either provide a way for me to configure the gerrit URL that doesn't depend on the codereview.cfg file being in the repository.

What did you see instead?

git-codereview doesn't identify my repository as Gerrit, then disable features that depend on Gerrit.

Does CL 24001 help with this?

Closing; I think CL 24001 is a good fix.


Issue 1119 - Show contact names and email adresses

Hi,
has suggested the issue https://github.com/k9mail/k-9/issues/1119 I add an option to show both real name and email address.

The build failed because you broke a related test. Also it would be good to add some. What happens if mShowCorrespondentNames is false but mShowCorrespondentNamesAndAddresses is true.
To me it seems like it would be better changing mShowCorrespondentNames from a boolean to a three-value enum.

Sorry for this mistake, how can I run the unit tests on my own ? I tried to build the project with the "gradlew.bat testDebug" and "gradlew.bat assembleDebug" commands but it seems that all tests passed...

Try gradlew.bat clean testDebug.

@philipwhiuk is right. The appearance of correspondents should be configured using only one setting. Both in code and in the user interface.

I add the three value enum and I fix the unit test problem as expected.
However I still have a problem. When the user choose between the three option the change is real but the interface is not refresh (I have to return out of the parameters to refresh the interface).
Can you see were I'm wrong ?

I've thought a bit more about it and don't like the user experience of this approach. See #1119 for more details.


Add whisper notebook

I'm not able to run on Ubuntu 20.04 with Python 3.8 using latest 2022.3 pre-release. This is the error I see after running cell 6:

Any ideas? Here is full warning+error message: https://gist.github.com/ryanloney/71d9929aa615148d060ec2cd5c696b99

@ryanloney which pytorch version do you use? Looks like it does not support dict-like input format for onnx export.

@adrianboguszewski, thank you for suggestions, updated. Please take a look

@eaidova I'm really excited for this notebook! Please see my suggested edits.
Btw I'm still having trouble running on Windows and Ubuntu. I will wait for new instructions.

@ryanloney thank you for your comments, I applied suggested changes, please take a look. regarding issue with launching, I added proper pytorch version to requirements installation for this notebook, now it should work.

@eaidova I'm really excited for this notebook! Please see my suggested edits.
Btw I'm still having trouble running on Windows and Ubuntu. I will wait for new instructions.

@ryanloney thank you for your comments, I applied suggested changes, please take a look. regarding issue with launching, I added proper pytorch version to requirements installation for this notebook, now it should work.

Thank you. I will test again today. I have been creating clean environment each time for testing.


Fix links to hpc.uit.no since they will disappear

git grep -l hpc.uit.no

Most links are gone. The application list has a placeholder in applications/sw_list.rst, which is not used o referenced to at the moment.

Excellent. Thanks. I close this issue since it is solved. We will reintroduce
the software list later.


Fix import in ggplotd/stat.d

This is currently blocking [1]. A few explanations: due to a compiler bug, privately imported symbols would be visible from other modules if they were selectively imported. As you can see, the module dstats.random does not define the symbol normalPDF [2], however it does import dstats.distrib which does define it [4].
After merging this, please push a new tag so that buildkite cand use the latest version of ggplotd.
Thanks!
[1] https://github.com/dlang/dmd/pull/9393
[2] https://github.com/DlangScience/dstats/blob/master/source/dstats/random.d
[3] https://github.com/DlangScience/dstats/blob/master/source/dstats/random.d#L120
[4] https://github.com/DlangScience/dstats/blob/master/source/dstats/distrib.d#L901

Ping

Thanks!


POM Dependency Clean Up

updated based on results from: "mvn dependency:analyze"
Minor item from GSOC commit.
Ordered dependencies to improve maintenance.

This is my attempt to bring the changes in https://github.com/apache/vxquery/pull/162 into master. Something went wrong in the REST project and now the tests fail due to a missing class. Any ideas?

These changes seem good since there are no dependencies problems.


updated devops notes

Motivation
It's confusing whether to click or not to click on the DevOps notes points. Re phrased them to make it clear.

bors merge


rewrote Makefile, restructured build dirs and simplified .gitignore

Looks good to me!
I would remove the test_ from build/runtest onnxBackendSuite test_$OPERATOR; \ in the makefile, so that the operator to test has the exact name as in the onnx test backend and in tests.c (i.e. test_and_bcast4v4d) so it can be copy-pasted. But its a matter of taste.
Thanks!


installer: add graphical GNOME iso

Adds a new iso for the release with GNOME 3. Renames graphical to graphical-kde.
Screenshot on virtualbox:

cc @edolstra @domenkozar

Sweet!
Observation: You're enabling both displayManager.slim and displayManager.gdm. Is that intentional?

I don't think we should do this. Building more ISOs puts quite a load on Hydra (especially the graphical ISOs, which are very large) and on the channel update script (since it needs to upload the ISOs to EC2). One graphical ISO is enough IMHO.

@edolstra ok what about including the nix but not in the release?

How big is the closure?

Could we enable it only for stable releases?

The result closure of the iso build is 5.0GB. The iso itself is 1.2GB.
From hydra charts, the result closure of the kde iso build is about 4GB, and the iso is 960MB.

Well, what's the goal? AFAIK other distros don't offer a different ISO for each desktop environment they support. The only reason for the installation ISO is to, well, install. After that the user can install any desktop environment they want.
NixOS releases are already 4.4 GB per channel update and I regularly need to throw away old (stable!) releases to prevent the disks from filling up.

It's a showcase. Also doesn't the installer benefit from /nix/store of the iso? That would mean less downloads if installing gnome. Anyway, is it possible to include the nix without building on hydra?

Looking at the "major Linux distributions" (minus FreeBSD):

Linux Mint has 3 variants: http://www.linuxmint.com/download.php
Ubuntu has lots of Flavors: https://wiki.ubuntu.com/UbuntuFlavors
Debian has lots of Live Iso's: http://cdimage.debian.org/debian-cd/current-live/amd64/bt-hybrid/
Mageia has KDE and GNOME: http://www.mageia.org/en/downloads/
Fedora offers ISO's for every desktop: https://spins.fedoraproject.org/
OpenSUSE has Live KDE and Live GNOME: http://software.opensuse.org/132/en
Arch doesn't provide a graphical installer, but has lots of derivatives that do
CentOS has GNOME and KDE: http://mirror.lug.udel.edu/pub/centos/7/isos/x86_64/
PCLinuxOS provides KDE, LXDE, and MATE: http://www.pclinuxos.com/get-pclinuxos/
Slackware does only KDE: http://www.slackware.com/getslack/torrents.php

So, 8 out of 10 distros do offer a different ISO for each desktop environment they support.

I think there's a common concept of live iso that people can try out a bit to get some feel about the distro, i.e. installation seems not the only purpose for those images. IMHO, the isos don't need to be updated so often, especially the big ones, as the installation is network-bounded anyway, and it's enough to keep just 2-3 latest of each channel (especially when we have the -small variants as well).

I would also like to see a Gnome based ISO. Great idea @lethalman

Let's merge this for 16.03 but not build it on hydra by default.

Now that the unstable release passed, I will work on this and merge it.

To build it: nix-build -A gnome3.installerIso. Didn't find a better place under nixos/ for now.

:beers:


Contact me form added to the portfolio project

The contact form section was added to the project and style as close to the template as possible
While executing the project, the following objectives were adhered to:

Name field accepts up to 30 chars.
Email field accepts only emails.
Message field accepts up to 500 chars.
All fields have placeholder text.
All fields have the correct type.
A user cannot submit the form if one or more of the fields is empty.
Data collection by using the Formspree service.
Flexbox is used In order to lay out the elements on the page.
Button interactions implemented as per the template(enable, hover, etc.).

Images added to the background of the 'Hero' and 'Contact' as per the Figma template

The extra space at the bottom of the contact section has been removed and the project is more reflective of the Figma template


Can't specify nullable state for generated columns

Environment details

Programming language: Python
OS: Linux
Language runtime version: Python3.9
Package version: sqlalchemy-spanner-1.0.0

Steps to reproduce

Run the following snippet:

from sqlalchemy import (
  Column,
  Computed,
  MetaData,
  String,
  Table,
)

metadata = MetaData(bind=engine)

singers = Table(
    "Singers",
    metadata,
    Column("SingerId", String(36), primary_key=True, nullable=False),
    Column("FirstName", String(200)),
    Column("LastName", String(200), nullable=False),
    Column("FullName", String(400), Computed("COALESCE(FirstName || ' ', '') || LastName"), nullable=False),
)

metadata.create_all(engine)

I get the following error:

    raise exceptions.from_grpc_error(exc) from exc
google.api_core.exceptions.InvalidArgument: 400 Error parsing Spanner DDL statement: CREATE TABLE `Singers` (\n\t`SingerId` STRING(36) NOT NULL, \n\t`FirstName` STRING(200), \n\t`LastName` STRING(200) NOT NULL, \n\t`FullName` STRING(400) AS (COALESCE(FirstName || \' \', \'\') || LastName) STORED NOT NULL\n) PRIMARY KEY (SingerId) : Syntax error on line 5, column 80: Expecting \')\' but found \'NOT\'

It seems that we can't specify the nullable condition for generated columns.

@skuruppu, hm-m. I suspect the syntax is not supported by Spanner. According to the docs, it's impossible to directly write into a generated column. So, everything is up to the expression - there we're setting if it can contain NULLs, and there is no need in a separate nullable parameter. Should we throw an exception with explanation in case nullable was used with a generated column?

When we were working on EntityFramework, we were able to set this property. See this data model for example.
Just to be sure, I ran the above DDL using the Cloud Console and it worked.

@skuruppu, oh, I see. The NOT NULL part is inserted in front of the computed part. Okay, let me see how we can fix it.


sending the .ino Code via Email

To make it easy to upload the Code again send the code/id via Email. Sometimes it's necessary to upload the code again

implemented in last update


Update - Cybersixgill Actionable Alerts

Change(s):

Updated Data connector to accept Organization ID
Updated playbooks to accept Organization ID

Reason for Change(s):

Added Multi-tenancy support

Testing Completed:

Yes

Checked that the validations are passing and have addressed any issues that are present:

Yes

Hi @rahul0216 / @manishkumar1991, please check and provide your feedback on this, thanks!

Hi @rahul0216 / @manishkumar1991 please provide update on this, thanks!

Hi @rahul0216 / @manishkumar1991 please share your feedback, thanks!

@v-vdixit, please get the connector updates reviewed by Anki or Mahesh.

Hi @mkchiliveri Please check the connector updates in this and provide your feedback, thanks!

Hi @mkchiliveri please provide your updates on this, thanks!

Hi @anki-narravula Please provide your feedback on this, thanks!

Hi @anki-narravula please provide your feedback on this, thanks!

Hi @anki-narravula please provide feedback on this, thanks!

Hi @anki-narravula please provide update on this, thanks!

Hi @anki-narravula please provide feedback on this, thanks!

Hi @anki-narravula we are waiting for your feedback on this, thanks!

Hi we are looking into this, will update soon, thanks!

Hi @syed-loginsoft, could you please add screenshots demonstrating successful working of the function app, thanks!

Hi @syed-loginsoft we are waiting for your update on this, thanks!

Hi @syed-loginsoft, could you please add screenshots demonstrating successful working of the function app, thanks!

Hi @syed-loginsoft we are still waiting for your update on this

Hi @syed-loginsoft we have checked all other content in the PR it looks fine, waiting for you to add screenshots demonstrating successful working of connector, thanks!

I will share the screenshots today

Hi @syed-loginsoft please fix the failing arm-ttk validation by replacing the apiVersions dated "2021-05-01" at all error locations with "2022-02-01", more details of this error can be found here, thanks.

Hi @syed-loginsoft waiting for your update on this, thanks!

Requested screenshot


Golang ioutil.ReadAll(resp.Body) results in EOF errors which is caused by TCP window full

When I use http.get to download a file from website as follow:
resp, err := http.Get(url)
if err != nil {
log.Fatal(err)
}
defer resp.Body.Close()
io, err_read := ioutil.ReadAll(resp.Body)
if err_read != nil {
log.Fatal(err_read)
}
An EOF error will happen as follow:
2015/07/22 17:31:49 Get http://xxxxxxxxxxxxxxxxxDE=1?749da030: EOF
exit status 1
I captured the packets between client and server and found tcp window full in client.
"Expert Info (Warn/Sequence): TCP transmission window is now completely full"
There will be so many tcp re-transmission happened  that the server close the connection.
Who could help to look into this issue?

This doesn't sound like a Go question. You can try asking on golang-nuts@ though and see if anybody has networking advice for you.


Include original source in the annotation

Resolves #31

I think this should be in the Rules parser and not the sheet parser. What we're looking for is the actual styling rule source. For example, if you have the following class:
"big-red" do
  color(.red)
  size(.big)
end

these aren't actual SwiftUI modifiers, just to make the example. We'd want the AST annotation for each rule to include just the original source of that single rule. I don't believe there is much value is annotating the class matcher itself.

Closing in favor of https://github.com/liveview-native/liveview-client-swiftui/pull/1191


E2E Tests for Hotkeys User Preferences menu

What shows if there are no hotkey definitions?
What shows if there are enough to overflow the container?
What happens if there are duplicate definitions?
Restore to default?
Create additional test cases

Hi @dannyrb , I need clarification on the following questions:

Can we have a hotkey combination with more than 2 keys or 2 is the limit? Ex: ctrl+shift+E
Is it mandatory to insert a hotkey for a function or the field can be left blank on User Preferences modal?
What are all the "invalid" hotkey combination? I was able to identify:

ctrl+z
ctrl+x
ctrl+c
ctrl+v
ctrl+b
ctrl+n
ctrl+a
ctrl+d
ctrl+f
ctrl+g
ctrl+h
ctrl+j
ctrl+l
ctrl+w
ctrl+r
ctrl+t
ctrl+o
ctrl+p
Do we have others?

Can we have a hotkey combination with more than 2 keys or 2 is the limit? Ex: ctrl+shift+E

We have the ability to support as many key combinations as we would like. Including sequences (Like ctrl+k then ctrl+c). We're using this library to listen for a respond to keys: https://github.com/ccampbell/mousetrap
That being said, I think the "record keys" logic in the User Preferences panel may be currently limited to two keys.

Is it mandatory to insert a hotkey for a function or the field can be left blank on User Preferences modal?

We should allow functions/commands to be blank/empty.

What are all the "invalid" hotkey combination? I was able to identify:

"Built-in" hotkeys (like undo with ctrl+z) can't be registered the "normal" way with mousetrap. We should determine the best way to handle these cases, as it may not be good practice to simply allow users to override them:

https://craig.is/killing/mice (Stopping the default behavior)


Remove default inline styles

Is there any possibility to remove default inline styling to avoid style overwritings with custom ones?

closing because of https://github.com/ReactTooltip/react-tooltip/issues/811
Thanks @NemeZZiZZ


Missing "Not-In" Firestore Query Operator

Is your feature request related to a problem? Please describe.
This request has been created under the "Feature Request" category, but should be considered as "Missing Functionality" and/or a "Bug Report"; there appears to be a limitation with the Google Cloud Firestore PHP library whereby the documented not-in query comparison operator is missing.
Describe the solution you'd like
Ideally, it'd be great to have the missing operator introduced to bring the PHP library in-line and up to date with the global documentation for Firestore. All other operators are usable, and while this problem does have workaround fixes, the fact that the functionality is documented does result in considerable debug time to realise that it is merely the library missing it.
Additional context
The operator is documented in the global documentation here but is not present or usable within the library unlike all other operators (including the newly introduced !=), and as such, returns an error when attempting to be used. The missing operator can be easily seen as being absent from the list of operators found here where only the in variant is included.

Hi @metallicgloss,
I'll work on adding the convenience mapping to the Query class, but in the meantime, you can use Not In via the underlying protobuf type, like this:
$collection->where(
    'field',
    \Google\Cloud\Firestore\V1\StructuredQuery\FieldFilter\Operator::NOT_IN,
    ["value1", "value1"]
);

Perfect, thank you @jdpedrie for that pointer! I'll use that as a temporary workaround until you've been able to complete the mapping in the Query class. Thanks!


freefilesync: add disable update patch

Description of changes
This patch from Debian removes update functionality. After this, we won't see notification window that newer version is available. Also, this removes option in "Help" - "Check for update now".

More about changes in this patch you can find here: https://sources.debian.org/data/main/f/freefilesync/13.3-1/debian/patches/ffs_no_check_updates.patch
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Thanks a lot for the patches!

Result of nixpkgs-review pr 286477 run on aarch64-linux 1

1 package built:

freefilesync


Count non expiring tokens when determining if the limit is reached

Fixes #280
Thanks to @pablomm for the code in the ticket

@morty @pablomm Thanks for the contribution! That's a nice catch!
I have one concern with this change though. Once we start considering non-expiring tokens, it is possible that users end up getting locked out of their accounts after issuing a certain number of tokens without logging out properly.
We either need this behind settings flag or a way for new log-ins to remove the oldest unused token.
Do you think my concern makes sense?

HI @giovannicimolin and thanks to @morty for turning the fix into a pull request.
In my case, I use these non-expiring tokens for users that uses my platform through an API. They have access to a control panel (which they access with another authentication system) where they can create, view and delete their api tokens (knox tokens which may be non-expiring). So logging out does not make sense in my use case.
The problem you comment, wouldn't it also happen if someone creates too many tokens within the expiration time?

To get to this situation two settings (TOKEN_TTL and TOKEN_LIMIT_PER_USER) would have had to be changed from their default values. That's a deliberate choice. For TOKEN_LIMIT_PER_USER to be ignored without warning if TOKEN_TTL  is None seems dangerous. Maybe a warning in the documentation for TOKEN_LIMIT_PER_USER saying that when the limit is reached users won't be able to get another one until one of their tokens has timed out and if they are non-expiring then will be locked out until they can use another method to delete a token?
My client is using this with a very small number of end users where they could handle support requests to delete tokens if required. I could see this wouldn't be possible where there are a very large number of end users.
If LogoutAllView allowed other authentication methods like LoginView then the user could clean out their tokens with username and password. But it looks like this might have been explored before based on the warning in the docs for the LogoutAllView.

@morty @giovannicimolin I believe the change made here makes full sense, this is clearly a bug that needs to be fixed.
I would add a note to the documentation (and probably a comment in the code) that clearly explains the consequence of having dangling non-expiring tokens around.
Using non-expiring tokens should be also discouraged as it's a bad practice anyway.


fix: remove encoding step before feeding request to signer

Issue #

https://github.com/aws-amplify/amplify-swift/issues/2542
Description

⚠ This change alters the assertions in RESTRequestUtilsTests to match the behavior necessary to prepare requests for signing in the Swift SDK. Requesting extra scrutiny around these changes.
API (REST) with IAM based auth signs requests using the AWS SDK for Swift's signing implementation. This implementation differs from the previous AWS SDK for iOS implementation in that it expects unencoded query parameters. Prior to this fix, we were encoding callsite provided query parameters before feeding the request to the signer, which resulted in InvalidSignatureExceptions due to improper signing. This change removes that (now) unnecessary step and explicitly removes encoding of any query parameters prior to request signing.
General Checklist

[ ] ~Added new tests to cover change, if needed~ (existing tests now passing)
[x] Build succeeds with all target using Swift Package Manager
[x] All unit tests pass
[x] All integration tests pass
[x] Security oriented best practices and standards are followed (e.g. using input sanitization, principle of least privilege, etc)
[ ] ~Documentation update for the change if required~
[x] PR title conforms to conventional commit style
[x] If breaking change, documentation/changelog update with migration instructions

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Kicked off API integration tests against this PR branch --> https://github.com/aws-amplify/amplify-swift/actions/runs/3903956017


.typewriter_skip() doesn't appear to reset speed when changing page

In other words, using the skip function makes all text in one dialogue appear instantly.
This is obviously an issue if you have dialogue with multiple pages, or if you want to write out the same dialogue multiple times.

Thanks for making this bug report, I'll look at it as soon as I'm able to.

Fixed v7.1.1pre001 - sent you a message on Discord to confirm the fix


KAFKA-7158: Add unit test for window store range queries

While debugging the reported issue, I found that our current unit test lacks coverage to actually expose the underlying root cause.
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

Cherry-picked to 2.0 after merging https://github.com/apache/kafka/pull/5474.


Change the cli from argparse to typer

ref:
https://github.com/osl-incubator/envers/blob/main/src/envers/cli.py

cc @jhonnye0

contributing guide: https://osl-incubator.github.io/sugar/contributing/

https://github.com/OpenScienceLabs/opensciencelabs.github.io/pull/130/files


Fix Curriculum lessons container overflow

Fixes the overflow by applying overflow: hidden;
Resolves #824

Setting overflow: hidden breaks scroll on mobile (or any device with small screen size).
It's supposed to be like this: initially you get lesson list with arrow prompting you to scroll right (or swipe right  on mobile). If you scroll you see discord bar, announcement and additional resources.

If you hide overflow it won't be possible to get the second screen. Check vercel preview and compare it to live version.

@Ulisseus You're right. It'll break the mobile swipe feature.
It seems the scrollbar only appears on Brave. I've tried Chrome and Firefox.


fix(pci.projet.user): update error translation keys

fix(pci.projet.user): update error translation keys
Description of the Change
a9aa9511 — fix(pci.projet.user): update error translation keys

It seems that there is a typo in your commit message projet > project.
Will be solved with a Squash and merge action.


grouped all websockets-related properties under provider.websocket na…

10963

 Quality gate: Passed
14 Issues

 14 Code issues

Details


Add a administration tool to inspect and control memcached through a UI

I love this project -- I find it very useful for drupal development.
I am not sure if you are taking requests by recently I need to debug an issue i was having with memcached. I connected a debug container which gave me a debug  web site so that I could visually see what was going wrong.
with the patch applied visit
http://localhost:8006/configure.php add
drupal_memcached_1 to the server list and every thing works as expected
http://localhost:8006/index.php is the landing page for the full site.
Anyway I hope you consider this a useful thing to do.

Hi @martinfrances107,
Thank you for the great effort! Only one thing. Won't you mind if I rename phpMemcachedAdmin to memcached-ui?

That sound like a good idea.


update @types/vis to latest version

Please consider updating dependency to vis types to the latest version  (4.21.9 )
as it allows to support more options for Nodes (e.g level).

thanks for hint.


[flow-builder] Access flow versions from builder

Is your feature request related to a problem? Please describe.
When on a flow it's really annoying to get to the screen showing all flow verisons
Describe the solution you'd like
a button when in a flow to take me to all versions
I'm assuming this isn't difficult to implement being that the DefinitionId is right on the flow

That was quick, awesome!

Yes it does open the same page as 'View Details and Versions' link.
It will be available in beta probably within two weeks


Content Bot - add channel to claim URL hyperlink name / URL

If content is posted under a channel, it should show the URL as lbry://@jiggytom#232131231231231/claimname
actual example from recent content post:
lbry://@humblegoddess#7d00b2935e6f508bf5e2cbef609571f96495bedb/twitchstream91120171

@nikooo777 is aware of this one and it requires some type of decoding of claim data.
From Jack:
jack [2:56 PM]
not without building lbryschema into lbrycrd to decode them
[2:56]
lbrycrd returns protobuf claims as some kind of escaped unicode (edited)
jack
[2:58 PM]
it probably wouldnt be terribly difficult to include lbryschema.go for decoding/validating them to json text (edited)

I have a feeling chainquery could work some magic for the claimbot @tiger5226

yeah chainquery can provide the decoded json data of a claim or channel. For example @fillerino

@tzarebczan The example you posted doesn't work, it just redirects me to the channel. Is this correct?

Fixed in #210


Introducing casting to pointers support.

Overview
Fix #90

Please preserve this line to notify @danielfireman (lead of this repository)

This change is

cc/ @joeberth

Coverage increased (+0.06%) to 92.927% when pulling 237a132360f3ecafd5d6a81002e96b56799692d0 on castPointer into 126e37f6486662939b12059700086ff5b5caf799 on master.


WebExtensions can be installed in Thunderbird (they just currently don't work)

I've managed to successfully install two of my WebExtensions in Thunderbird, and the options page does show up correctly (but doesn't actually change anything).
The issue is that Thunderbird doesn't have any of the required WebExtension APIs implemented, and as such the WebExtensions don't work at all.
But because of the fact that they are recognized as WebExtensions and the description and options page show up correctly, I suspect that Thunderbird will eventually implement the required WebExtension APIs.

The extensions in question:
Firefox:
ExE Boss’s Extension Suite:

Firefox about:about Button
View Image in New Tab

Thunderbird:
ExE Boss’s Extension Suite:

Firefox about:about Button
View Image in New Tab

P.S.: I'm setting up this issue to track when Thunderbird implements the required WebExtension APIs, and to inform the developers of Markdown Here of this.

I'm going to close this. I'm 99% sure that Thunderbird will never get WebExtensions. If it seems otherwise in  the future, this can be re-opened.

Thunderbird will get WebExtensions in a near future:
https://bugzilla.mozilla.org/show_bug.cgi?id=1396172
https://wiki.mozilla.org/Thunderbird/Add-ons_Guide_57


bypass lock for IOContext

This "fixes" the error that Pluto raises (with ImageIO only) when trying to display an image, e.g., img = rand(RGB{N0f8}, 4, 4):
      From worker 2:	Errors encountered while saving nothing.
      From worker 2:	All errors:
      From worker 2:	===========================================
      From worker 2:	MethodError: no method matching lock(::PNGFiles.var"#13#14"{Int64, Int64, Int64, Nothing, IOContext{IOBuffer}, Matrix{ColorTypes.RGB{FixedPointNumbers.N0f8}}, Ptr{Nothing}, Ptr{Nothing}}, ::IOContext{IOBuffer})
      From worker 2:	Closest candidates are:
      From worker 2:	  lock(::Any, !Matched::Base.GenericCondition) at condition.jl:78
      From worker 2:	  lock(::Any, !Matched::Base.AbstractLock) at lock.jl:184
      From worker 2:	  lock(::Any, !Matched::WeakKeyDict) at weakkeydict.jl:76
      From worker 2:	===========================================
      From worker 2:	ArgumentError: Package QuartzImageIO not found in current path:
      From worker 2:	- Run `import Pkg; Pkg.add("QuartzImageIO")` to install the QuartzImageIO package.
      From worker 2:
      From worker 2:	===========================================
      From worker 2:	ArgumentError: Package ImageMagick not found in current path:
      From worker 2:	- Run `import Pkg; Pkg.add("ImageMagick")` to install the ImageMagick package.
      From worker 2:
      From worker 2:	===========================================
      From worker 2:	MethodError: no method matching save(::FileIO.Stream{FileIO.DataFormat{:PNG}, IOContext{IOBuffer}}, ::Matrix{ColorTypes.RGB{FixedPointNumbers.N0f8}}; mapi=ImageShow.var"#14#16"())
      From worker 2:	Closest candidates are:
      From worker 2:	  save(!Matched::FileIO.File{FileIO.DataFormat{:PNG}}, ::Any) at /Users/jc/.julia/packages/FileIO/TyKdX/src/mimesave.jl:5 got unsupported keyword argument "mapi"
      From worker 2:	  save(!Matched::FileIO.File{FileIO.DataFormat{:SVG}}, ::Any) at /Users/jc/.julia/packages/FileIO/TyKdX/src/mimesave.jl:15 got unsupported keyword argument "mapi"
      From worker 2:	  save(!Matched::FileIO.File{FileIO.DataFormat{:PDF}}, ::Any) at /Users/jc/.julia/packages/FileIO/TyKdX/src/mimesave.jl:25 got unsupported keyword argument "mapi"
      From worker 2:	  ...
      From worker 2:	===========================================
      From worker 2:
      From worker 2:	Fatal error:

with this method, the image is now displayed appropriately in Pluto.
cc: @fonsp

Thanks! 💗


Browser usage

Hi!
I'm not experienced with this type of import, but should I be able to use "plainjs" in the browser just like that? 🤔
<html>
<head>
  <script type="module">
    import { Parser } from<EMAIL_ADDRESS>    import { StreamParser } from<EMAIL_ADDRESS>  </script>
</head>
...
</html>

(test made through a basic "test.html" file and opened in the latest Chrome)
It's not working. At the console, we can see the error:

caught TypeError: Failed to resolve module specifier "lodash.get". Relative references must start with either "/", "./", or "../".

I've tried creating an "importmap" before the import to solve this issue:
<script type="importmap">
  {
    "imports": {
      "lodash.get"            : "https://cdn.jsdelivr.net/npm/lodash.get/index.js",
      "@json2csv/formatters/" : "https://cdn.jsdelivr.net/npm/@json2csv/formatters/src/",
      "@streamparser/json"  <EMAIL_ADDRESS>    }
  }
</script>
<script type="module">...

But each remap goes to a new failure. After these three, the error ends being:

caught SyntaxError: The requested module 'lodash.get' does not provide an export named 'default' (at BaseParser.js:1:8)

Any recommendations?
Thanks.

Sorry for the super-slow response time.
You can see how I do it on the docs page:
https://github.com/juanjoDiaz/json2csv/blob/2d827e99d849e3afbbfcde4fa25f9789aab25c83/docs/index.html#L172-L177
The key thing is that, for this to work you need to pull from GitHub and not from npm as npm won't have all the needed modules connected in a way that works for the browser.

Oh, that's okay... Late answer yet. haha
Many thanks! Didn't have the eyes to get that – and by some reason I didn't see the "try it live", which is very cool!


Fix some typos.

One resulted in one of the Driver's execute methods always returning
true (PASS).
One was just a typo on a printed message.

lgtm


Crash when loading some image with version 3.10.3

I want to show wallet passes in my APP with image effect but some passes may cause crash.
***Terminating app due to uncaught exception 'NSInvaildArgumentException', reason: '~[CIContext initWithOptions:]: unrecognized selector sent to instance 0x17020c0a0'

It's a bug of iOS 8. I am not quite willing to fix it by introducing an Objective-C layer.
I guess it could be a good chance to upgrade the deploy target to iOS 9 now... :[

Yes, I found it on iOS 8.3 at the beginning, and also on iOS 10.3.2 later. Both target is 8.2. This is MrBrown App and I will consider it. Thanks a lot 😊
On 2017年7月10日 +0800 PM5:41, Wei Wang<EMAIL_ADDRESS>wrote:

It's a bug of iOS 8. I am not quite willing to fix it by introducing an Objective-C layer.
I guess it could be a good chance to upgrade the deploy target to iOS 9 now... :[
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.


Add support for JsonPath for ready state path

@ash2k Have 2 tests failing, seems to be the incorrect test with orphans?
2017/06/14 16:34:42 [WORKER][default/bundle-attribute] Failed to rebuild bundle (retriable: false): object tpr.atlassian.com/v1, Kind=Sleeper "sleeper1" is not owned by the Bundle
2017/06/14 16:34:43 [Sleeper] default/sleeper1 was added/updated. Setting Status to "Sleeping" and falling asleep for 1 seconds... ZZzzzz
2017/06/14 16:34:44 [Sleeper] Hello, Infravators! Updating default/sleeper1 Status to "Awake!"
--- FAIL: TestTprAttribute (4.57s)
	utils_for_test.go:228: Creating a new object default/bundle-attribute of kind Bundle
        Error Trace:    utils_for_test.go:239
			tpr_attribute_test.go:75
			utils_for_test.go:221
			tpr_attribute_test.go:50
	Error:  	Received unexpected error "context deadline exceeded"

	utils_for_test.go:77: Deleting bundle bundle-attribute
	utils_for_test.go:92: Deleting resource "sleeper1"

2017/06/14 16:34:49 [WORKER][default/bundle1] Checking resource "config2"
2017/06/14 16:34:49 [WORKER][default/bundle1] Failed to rebuild bundle (retriable: false): object /v1, Kind=ConfigMap "config2" is not owned by the Bundle
--- FAIL: TestUpdate (4.58s)
	utils_for_test.go:228: Creating a new object default/bundle1 of kind Bundle
        Error Trace:    utils_for_test.go:239
			update_test.go:171
			utils_for_test.go:221
			update_test.go:142
	Error:  	Received unexpected error "context deadline exceeded"

	utils_for_test.go:77: Deleting bundle bundle1
	utils_for_test.go:92: Deleting resource "config2"
	utils_for_test.go:92: Deleting resource "sleeper2"

@ash2k checked on the latest master, have the same tests failing


Support for method sendInvoice

The sendInvoice method supports inline keyboard, so it would be nice to have ability sending invoices via the library. I guess it can be implemented by introducing new interface InvoiceBody (aside TextBody, MediaBody and etc.).

added in 955dec829b1839166eb3507c97e24d5b466a39c8 which is part of v6.3.0.
Havnt tested it but as the Typings are good I assume it should be working. If it doesnt feel free to report back.


Workflow data export GET request 404

mentioning @rbruhn
Regarding requests to https://panoptes.zooniverse.org/api/workflows/11705/classifications_export, but not workflow specific.
Workflow data export GET request is returning a 404 "No classifications_export exists for workflow #11705", when similar request from within PFE Lab/Data Exports/Request New Workflow Classification Export returns expected response with 200.
Workflow data export POST request (with same headers and auth) returns expected 200 response noting export metadata.state: "creating".
I believe auth is from user biospex, confirmed collaborator on project.
Related scripts, working historically, unchanged recently:

this is the main class for calling the API
which extends this class
these are the jobs that run to create (POST) then check (GET) the csv files

If it makes sense I could try testing locally on Postman or similar, but I might need a little help with setup to request on production (vs staging, which I don't think requires same level of auth). Is there any existing related documentation?
Long shot, but maybe related to #3157?

Long shot, but maybe related to #3157?

Doubtful, i think we need more information about the request formats / HTTP traffic to rule this out though.

Workflow data export GET request is returning a 404 "No classifications_export exists for workflow #11705", when similar request from within PFE Lab/Data Exports/Request New Workflow Classification Export returns expected response with 200.

The API will return a 404 if the requesting user doesn't have the correct access privileges vs saying unauthorized (we chose to do this based on github API).
https://github.com/iDigBio/Biospex/blob/5c80758265d4b03aa55bdeacefec33dae584d242/app/Services/Api/NfnApi.php#L34
Noting that the calling code uses a OAuth client to get API tokens, which user owns that OAuth application, they will be the user that is checked in authorization.
Ideally we'd get some debugging HTTP traffic to help with this (avoiding posting the auth token)
Another thing to try would be via the python cli tool to see if it works for the user they are expecting it to?
https://github.com/zooniverse/panoptes-cli#generate-and-download-a-classifications-export
panoptes workflow download-classifications --help

If you need me to do something, let me know.

The API will return a 404 if the requesting user doesn't have the correct access privileges vs saying unauthorized (we chose to do this based on github API).

I believe this issue started when workflows started having their own project id. Previously, all Biospex workflows had the project id 1558. Since NfN switched to each workflow being a project, Michael has had to add the Biospex user to each workflow he creates.

Is https://www.zooniverse.org/projects/md68135/notes-from-nature-plants-of-arkansas https://www.zooniverse.org/lab/9275 the correct project? Biospex is a collab on that project.
Can you try using the python client instructions to access the download? I believe if you set the PANOPTES_CLIENT_ID and PANOPTES_CLIENT_SECRET as environment variables that cli tool will pick this up and use it. However if not you can use the biospex username / password and the panoptes configure cmd to setup the cli tool to connect to the API.

There are several projects Biospex is collab on. It's not just one project this issue has effected. It also occurs on the project id 1558 workflows we use to use. For example, workflow id 8673 under project id 1558.
I will try the Python thing. I'm not a Python programmer but will see what I can do.

There are several projects Biospex is collab on. It's not just one project this issue has effected. It also occurs on the project id 1558 workflows we use to use. For example, workflow id 8673 under project id 1558.

Understood, i'm trying to pick up if there is an issue with client creds oauth flow token (can't really see one but it could be) or a straight up authorization issue via the API.

I will try the Python thing. I'm not a Python programmer but will see what I can do.

It's a command line tool so you can use it from bash etc, hence the suggestion.

Ok... so these were the results.
First tried using the API credentials. I added the enviro variables like this...
export PANOPTES_CLIENT_ID=ourid
export PANOPTES_CLIENT_SECRET=oursecret

Not sure if that is the correct way or not... then ran
 panoptes project download --generate 11705  11705.csv

Received error:
Traceback (most recent call last):
  File "/home/vagrant/.local/bin/panoptes", line 11, in <module>
    sys.exit(cli())
  File "/home/vagrant/.local/lib/python2.7/site-packages/click/core.py", line 722, in __call__
    return self.main(*args, **kwargs)
  File "/home/vagrant/.local/lib/python2.7/site-packages/click/core.py", line 697, in main
    rv = self.invoke(ctx)
  File "/home/vagrant/.local/lib/python2.7/site-packages/click/core.py", line 1066, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/home/vagrant/.local/lib/python2.7/site-packages/click/core.py", line 1066, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/home/vagrant/.local/lib/python2.7/site-packages/click/core.py", line 895, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/home/vagrant/.local/lib/python2.7/site-packages/click/core.py", line 535, in invoke
    return callback(*args, **kwargs)
  File "/home/vagrant/.local/lib/python2.7/site-packages/panoptes_cli/commands/project.py", line 186, in download
    project = Project.find(project_id)
  File "/home/vagrant/.local/lib/python2.7/site-packages/panoptes_client/project.py", line 57, in find
    return cls.where(id=id, slug=slug).next()
  File "/home/vagrant/.local/lib/python2.7/site-packages/panoptes_client/panoptes.py", line 672, in where
    return cls.paginated_results(*cls.http_get(_id, params=kwargs))
  File "/home/vagrant/.local/lib/python2.7/site-packages/panoptes_client/panoptes.py", line 623, in http_get
    **kwargs
  File "/home/vagrant/.local/lib/python2.7/site-packages/panoptes_client/panoptes.py", line 320, in get
    retry=retry,
  File "/home/vagrant/.local/lib/python2.7/site-packages/panoptes_client/panoptes.py", line 281, in json_request
    json_response['errors']
panoptes_client.panoptes.PanoptesAPIException: Could not find project with id='11705'

I then used panoptes configure and added the user/pass we use to log into the data-exports dashboard. I ran the same command as above:
 $: panoptes project download --generate 11705  11705.csv

Traceback (most recent call last):
  File "/home/vagrant/.local/bin/panoptes", line 11, in <module>
    sys.exit(cli())
  File "/home/vagrant/.local/lib/python2.7/site-packages/click/core.py", line 722, in __call__
    return self.main(*args, **kwargs)
  File "/home/vagrant/.local/lib/python2.7/site-packages/click/core.py", line 697, in main
    rv = self.invoke(ctx)
  File "/home/vagrant/.local/lib/python2.7/site-packages/click/core.py", line 1066, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/home/vagrant/.local/lib/python2.7/site-packages/click/core.py", line 1066, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/home/vagrant/.local/lib/python2.7/site-packages/click/core.py", line 895, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/home/vagrant/.local/lib/python2.7/site-packages/click/core.py", line 535, in invoke
    return callback(*args, **kwargs)
  File "/home/vagrant/.local/lib/python2.7/site-packages/panoptes_cli/commands/project.py", line 186, in download
    project = Project.find(project_id)
  File "/home/vagrant/.local/lib/python2.7/site-packages/panoptes_client/project.py", line 57, in find
    return cls.where(id=id, slug=slug).next()
  File "/home/vagrant/.local/lib/python2.7/site-packages/panoptes_client/panoptes.py", line 672, in where
    return cls.paginated_results(*cls.http_get(_id, params=kwargs))
  File "/home/vagrant/.local/lib/python2.7/site-packages/panoptes_client/panoptes.py", line 623, in http_get
    **kwargs
  File "/home/vagrant/.local/lib/python2.7/site-packages/panoptes_client/panoptes.py", line 320, in get
    retry=retry,
  File "/home/vagrant/.local/lib/python2.7/site-packages/panoptes_client/panoptes.py", line 281, in json_request
    json_response['errors']
panoptes_client.panoptes.PanoptesAPIException: Could not find project with id='11705'

What is the OAuth application CLIENT_ID? (no need for the secret)

I then used panoptes configure and added the user/pass we use to log into the data-exports dashboard. I ran the same command as above:

And just checking that it's the user with login biospex

CLIENT_ID=0fb75d7490ffe9050e12d3b75d943e544fec08cf34cc47051dd61e35863f5bd0
Yes, the user "biospex" we use to log into https://www.zooniverse.org/lab/1558/data-exports or other data-exports.

The first one worked using the "panoptes configure" option. I tried clearing the user/pass in the configure by using "space" and now I can't access the configure options. Gives me an error. How do I clear the configure so I can test the OAuth credentials?
Traceback (most recent call last):
  File "/home/vagrant/.local/bin/panoptes", line 11, in <module>
    sys.exit(cli())
  File "/home/vagrant/.local/lib/python2.7/site-packages/click/core.py", line 722, in __call__
    return self.main(*args, **kwargs)
  File "/home/vagrant/.local/lib/python2.7/site-packages/click/core.py", line 697, in main
    rv = self.invoke(ctx)
  File "/home/vagrant/.local/lib/python2.7/site-packages/click/core.py", line 1063, in invoke
    Command.invoke(self, ctx)
  File "/home/vagrant/.local/lib/python2.7/site-packages/click/core.py", line 895, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/home/vagrant/.local/lib/python2.7/site-packages/click/core.py", line 535, in invoke
    return callback(*args, **kwargs)
  File "/home/vagrant/.local/lib/python2.7/site-packages/click/decorators.py", line 17, in new_func
    return f(get_current_context(), *args, **kwargs)
  File "/home/vagrant/.local/lib/python2.7/site-packages/panoptes_cli/scripts/panoptes.py", line 47, in cli
    admin=admin,
  File "/home/vagrant/.local/lib/python2.7/site-packages/panoptes_client/panoptes.py", line 108, in connect
    cls._local.panoptes_client = cls(*args, **kwargs)
  File "/home/vagrant/.local/lib/python2.7/site-packages/panoptes_client/panoptes.py", line 140, in __init__
    self.login()
  File "/home/vagrant/.local/lib/python2.7/site-packages/panoptes_client/panoptes.py", line 501, in login
    response.json().get('error', 'Login failed')
panoptes_client.panoptes.PanoptesAPIException: Invalid email or password.

The OAuth credentials seem to work as well.
So I guess the next question would be why it doesn't work when calling via Postman or our website.

So if the OAuth credentials are working correctly via our python client then it must be a malformed request for the authentication in your code. What does the Authorization header on your GET HTTP requests look like? Are they being set correctly with the token from the oauth client credentials flow? I think that's the place to look here.
Postman i've had varying success using, last time i got it working and documented at https://zooniverse.github.io/Panoptes/#example-using-postman. However the interface has apparently changed though i'd be interested to know if you can get it working.

The way I use to do it with Postman and our code:
I set the auth url and POST the client_id and client_secret. I receive the access_token in the response. That is saved in the Bearer Token.
In the GET request for classifications, I use Bearer Token in the Authorization, the content and accept headers have the appropriate api and json info.

I did a debug on the panoptes client. I set the Postman urls to the same that are used in the panoptes client. So everything works up until "GET /api/workflows/11705/classifications_export"
vagrant@homestead:~/.panoptes$ PANOPTES_DEBUG=true panoptes workflow download-classifications 11705 11705.csv
DEBUG:urllib3.connectionpool:Starting new HTTPS connection (1): www.zooniverse.org:443
DEBUG:urllib3.connectionpool:https://www.zooniverse.org:443 "POST /oauth/token HTTP/1.1" 200 944
DEBUG:redo:attempt 1/5
DEBUG:urllib3.connectionpool:https://www.zooniverse.org:443 "GET /api/workflows/11705 HTTP/1.1" 200 None
DEBUG:redo:attempt 1/5
DEBUG:urllib3.connectionpool:https://www.zooniverse.org:443 "GET /api/workflows/11705/classifications_export HTTP/1.1" 200 1499
DEBUG:urllib3.connectionpool:Starting new HTTPS connection (1): zooniverse-static.s3.amazonaws.com:443
---connects to amazon and downloads the file.

Hi @rbruhn so for clarity, the current state of play is that you can use the panoptes python client and get access to the files? However you cannot using postman or your custom php code?

Yes, that is correct. Remember too, my code and Postman has been working for a couple years now. It wasn't until recently that it stopped.

Can you export the postman scripts (removing any access tokens etc) and send to<EMAIL_ADDRESS>or post here is you are happy with them being public?
I'd really like to debug the API calls, it is strange that postman and your php code stopped working. I can't think of any API changes that could impact on these data flows but it is a simple explanation for the issue.

Not sure how to export them, but it allowed me to create a link to share
https://www.getpostman.com/collections/a2065b8582089a762c51


Update basht package

This PR updates the version of basht to 0.1.0

@cf-london Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.

@cf-london Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.


livestream video image is blurry

My Setup

Operating System: ubuntu 18.04 LTS
App Version: 2.17.9
Installation type: debian package as shipped at https://github.com/RocketChat/Rocket.Chat.Electron/releases

[x] I have tested with the latest version
[x] I can simulate the issue easily

Description
See below a screenshot of a livestream on the electron app (left) and the same stream on firefox (right)

Current Behavior
Image is blurry with the Electron app
Expected Behavior
Same image quality on the Electron app than for any browser.

Please test if the problem persists with our latest release. I will be closing the issue but if there is any problem yet, we will open it again.


Process page breaks

Page breaks are shown like
<p:r><docx properties like font, .../><w:br type="page"/></p:r>

When converting this to HTML, this is replaced by  in the function:
 // Transform w:br to h:br.
 if (element.Name == W.br || element.Name == W.cr)
  {
       return ProcessBreak(element);
  }

Which causes the page break to not appear.
I have changed this so it returns a div style="page-break-before:always;"  in the following function.
private static object ProcessPageBreak(XElement element)
    {
        XElement div = new XElement(Xhtml.div);
        div.SetAttributeValue(NoNamespace.style, "page-break-before:always;");
        XElement span = null;

        return new object[]
        {
            div,
            new XEntity("#x200e"),
            span
     };
    }

Which causes a correct page break.
PS. The page break (docx) is detected in a child element. So i have to set the page break ( html) in it's parent. I detect it with the following function:
 private static bool HasAPageBreak(XElement element)
    {
        return element.Descendants().Any(dl => dl.Name == W.br && dl.Attribute(W.type).Value == "page");
    }

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

I added an issue here: https://github.com/OfficeDev/Open-Xml-PowerTools/issues/248


Add first batch of binder getting started assets

Adds getting started assets, which will be included during binder deployment.

Uses the assets from https://github.com/elyra-ai/elyra/pull/613 and addresses @akchinSTC's review comments.
These assets will be automatically  incorporated during the Binder build by

cloning this repository
copying the assets to the appropriate destination location


l1 contracts paused & upgraded & multi-signature

gnosis safe
paused
contract upgraded

L1StandardBridge Upgrade Task : https://github.com/mantlenetworkio/mantle/pull/179


Tags Sidebar Steals Focus

The tags sidebar steals focus (when you create a tag) even when you have it closed.
https://user-images.githubusercontent.com/7171118/234714747-3429b5cd-5280-4e2b-aed4-53e93fba3286.mov
Note: When nothing is happening in the video I'm mashing my keyboard.
I believe that if we actually pop open the sidebar when tags are created, it would probably solve #3478, though not exactly as they requested.

This has been resolved. The dialog accepts the tag text and the tags list no longer steels focus.


Different scales of images have different execution effects. How to obtain more detailed results on a high-resolution image

When I upload a high-resolution image, the segmentation result is not accurate at the contour

But when I upload a slice, the segmentation results become accurate.How can I achieve the same results？

I'd find small objects on the main image and then take a crop of that part of the image 1k x 1k (the image encoder size) to get a more refined result for those objects.

So do I still need to manually search for small objects in the main image, and how to merge these finer contours back into the original main image

You can try to tune some parameters in the SamAutomaticMaskGenerator, such as pred_iou_thresh and stalility_score_thresh.And there are some improvement when I set lower value at them.