Outdated information/link for/in KtonDAO docs

Hello, I hope I'm reporting this issue in the appropriate place and manner.
https://docs.darwinia.network/community/orgs/ktondao/
"On the Darwinia chains, users can deposit their RING for up to 36 months"
It is 12 months since some weeks ago.
Also, this link https://collator-staking.darwinia.network/ is the old, v1 staking website. :+1:

@boundless-forest

Nice catch, I'll update this later

Done. updated https://docs.darwinia.network/community/orgs/ktondao/

Done. updated https://docs.darwinia.network/community/orgs/ktondao/

@boundless-forest
Hi, i think in the updated doc it is still the v1 staking website.
v2 Ring staking website is https://collator-staking.ringdao.com/


[Snyk] Security upgrade cryptography from 41.0.1 to 41.0.3

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to fix one or more vulnerable packages in the `pip` dependencies of this project.
Changes included in this PR

Changes to the following files to upgrade the vulnerable dependencies to a fixed version:

requirements.txt

Vulnerabilities that will be fixed
By pinning:

Severity
Priority Score (*)
Issue
Upgrade
Breaking Change
Exploit Maturity

471/1000  Why? Recently disclosed, Has a fix available, CVSS 3.7
Insufficient Verification of Data Authenticity SNYK-PYTHON-CRYPTOGRAPHY-5813745
cryptography:  41.0.1 -> 41.0.3
No
No Known Exploit

551/1000  Why? Recently disclosed, Has a fix available, CVSS 5.3
Denial of Service (DoS) SNYK-PYTHON-CRYPTOGRAPHY-5813746
cryptography:  41.0.1 -> 41.0.3
No
No Known Exploit

471/1000  Why? Recently disclosed, Has a fix available, CVSS 3.7
Denial of Service (DoS) SNYK-PYTHON-CRYPTOGRAPHY-5813750
cryptography:  41.0.1 -> 41.0.3
No
No Known Exploit

(*) Note that the real score may have changed since the PR was raised.
Some vulnerabilities couldn't be fully fixed and so Snyk will still find them when the project is tested again. This may be because the vulnerability existed within more than one direct dependency, but not all of the affected dependencies could be upgraded.
Check the changes in this PR to ensure they won't cause issues with your project.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
🛠 Adjust project settings
📚 Read more about Snyk's upgrade and patch logic

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Denial of Service (DoS)

Checkmarx One – Scan Summary & Details – d71f7d2c-7480-4d46-adc8-e1ebd5d46591
New Issues

Severity
Issue
Source File / Package
Checkmarx Insight

CVE-2023-36053
Python-Django-4.1.9
Vulnerable Package

CSRF
/dojo/finding/views.py: 481
Attack Vector

CSRF
/dojo/finding/views.py: 478
Attack Vector

CSRF
/dojo/test/views.py: 83
Attack Vector

CSRF
/dojo/test/views.py: 81
Attack Vector

CSRF
/dojo/finding/views.py: 478
Attack Vector

CSRF
/dojo/finding/views.py: 623
Attack Vector

CSRF
/dojo/test/views.py: 379
Attack Vector

CSRF
/dojo/jira_link/views.py: 359
Attack Vector

CSRF
/dojo/finding/views.py: 942
Attack Vector

CSRF
/dojo/test/views.py: 280
Attack Vector

CSRF
/dojo/engagement/views.py: 218
Attack Vector

CVE-2007-2379
Npm-jquery-3.7.0
Vulnerable Package

CVE-2014-6071
Npm-jquery-3.7.0
Vulnerable Package

Privacy_Violation
/dojo/benchmark/views.py: 58
Attack Vector

Log_Forging
/dojo/test/views.py: 83
Attack Vector

Log_Forging
/dojo/finding/views.py: 478
Attack Vector

Log_Forging
/dojo/finding/views.py: 481
Attack Vector

Log_Forging
/dojo/test/views.py: 81
Attack Vector

Log_Forging
/dojo/test/views.py: 83
Attack Vector

Log_Forging
/dojo/finding/views.py: 478
Attack Vector

Log_Forging
/dojo/finding/views.py: 481
Attack Vector

Log_Forging
/dojo/test/views.py: 81
Attack Vector

Fixed Issues

Severity
Issue
Source File / Package
Checkmarx Insight

Reflected_XSS_All_Clients
/dojo/finding/views.py: 481
Attack Vector

Reflected_XSS_All_Clients
/dojo/test/views.py: 83
Attack Vector

CSRF
/dojo/finding/views.py: 481
Attack Vector

CSRF
/dojo/finding/views.py: 478
Attack Vector

CSRF
/dojo/finding/views.py: 481
Attack Vector

CSRF
/dojo/finding/views.py: 478
Attack Vector

CSRF
/dojo/finding/views.py: 623
Attack Vector

CSRF
/dojo/jira_link/views.py: 359
Attack Vector

CSRF
/dojo/finding/views.py: 942
Attack Vector

CSRF
/dojo/finding/views.py: 478
Attack Vector

CSRF
/dojo/finding/views.py: 481
Attack Vector

CSRF
/dojo/finding/views.py: 478
Attack Vector

CSRF
/dojo/test/views.py: 81
Attack Vector

CSRF
/dojo/test/views.py: 81
Attack Vector

CSRF
/dojo/test/views.py: 379
Attack Vector

CSRF
/dojo/test/views.py: 280
Attack Vector

CSRF
/dojo/engagement/views.py: 218
Attack Vector

CSRF
/dojo/test/views.py: 81
Attack Vector

Filtering_Sensitive_Logs
/dojo/models.py: 633
Attack Vector

Filtering_Sensitive_Logs
/dojo/finding/views.py: 2712
Attack Vector

Filtering_Sensitive_Logs
/dojo/finding/views.py: 2712
Attack Vector

Filtering_Sensitive_Logs
/dojo/finding/views.py: 2712
Attack Vector

Filtering_Sensitive_Logs
/dojo/finding/views.py: 2712
Attack Vector

Privacy_Violation
/dojo/finding/views.py: 2712
Attack Vector

Privacy_Violation
/dojo/finding/views.py: 2712
Attack Vector

Privacy_Violation
/dojo/finding/views.py: 2712
Attack Vector

Privacy_Violation
/dojo/finding/views.py: 2712
Attack Vector

Privacy_Violation
/dojo/finding/views.py: 2712
Attack Vector

Privacy_Violation
/dojo/finding/views.py: 2712
Attack Vector

Privacy_Violation
/dojo/finding/views.py: 2712
Attack Vector

Privacy_Violation
/dojo/finding/views.py: 2712
Attack Vector

Privacy_Violation
/dojo/finding/views.py: 2712
Attack Vector

Privacy_Violation
/dojo/finding/views.py: 2712
Attack Vector

Log_Forging
/dojo/test/views.py: 661
Attack Vector

Log_Forging
/dojo/engagement/views.py: 616
Attack Vector

Log_Forging
/dojo/test/views.py: 661
Attack Vector

Log_Forging
/dojo/jira_link/views.py: 235
Attack Vector

Log_Forging
/dojo/jira_link/views.py: 235
Attack Vector


/setgame enhancement

What should be added?
The /setgame command should use the search categories helix endpoint prior to setting the game: https://dev.twitch.tv/docs/api/reference#search-categories
Why should it be added?
Searching for the game in twitch's db allows for a much more "loose" way of setting a game. Easiest example I can think of is "the divison 2". It's not the full game name and it isn't capitalized at all. However twitch's category search endpoint will allow for a much more "loose" search for a game. Refer below:

We can easily use the ID from the resulting search to set the game. This will result in 2 API requests being made, but a far better experience for the user. Under the scenario where multiple games are returned from the endpoint, we should use the first entry.

This is a pretty decent first issue for someone wanting to contribute to Chatterino2.
It consists of two parts:

Update src/providers/twitch/api/Helix.{hpp,cpp} to support the Search Categories endpoint.
Update src/controllers/commands/CommandController.cpp to use the newly implemented Search Categories endpoint instead of fetchGames

Done this already, doing PR atm

Closed by #2609


lighthouse: fix validator service not setting arguments properly

This fixes lighthouse validator not respecting metrics and additional arguments NixOS configuration options.
Description of changes

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] 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
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.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
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

So... I see that this issue is still present on nixpkgs master, and this causes no large rebuild at all.  Therefore the right process would be:

open this as a pull request against master
ideally get relevant people to review, though in this case it seems a no-brainer
apply backport release-22.11 label to the PR (at any time), so that a backport PR is automatically created after merging the master PR

Hello @vcunat, this is my first PR to nixpkgs and NixOS ecosystem in general, while I tried following the CONTRIBUTING, I'm still quite confused about some steps, such as which branch should I merge into.
Do I understand correctly that you want me to re-make this PR against master branch instead?

Yes, that will be best.

Close due to being superseded.


Fix paddings

Fix behaviour when 'padding' option applied only if 'position' option placed after

Would be nice if this were merged, pretty annoying that a position has to be defined. Pretty unexpected.


FileMetadata object should not have chunks as member

chunks of a file is dependent on the recipient - the one who asks for the file wrt it's own configured buffer size. Thus, it doesn't make sense to keep it part of the FileMetadata.

Fixed in commit b7939120a07099e91ced5d6396bd9eb8c3fc4444.


feat: add fastify adapter

still need to write some tests and an example/template, but using this puppy on a side project: https://shredder.mcan.sh
Signed-off-by: Logan McAnsh<EMAIL_ADDRESS>

Thank you for signing the Contributor License Agreement. Let's get this merged! 🥳

hm, thought i fixed sending a comment to an open PR if they've signed already...

This is awesome.
But we've got to figure out a better way to support a bunch of adapters without all of them being part of the remix source code. Let's figure this out after the break. I made a linear issue.

yeah i felt the same when i opened it haha 👍

What is the status of fastify integration?
I am fine in using custom solution but i prefer official support.
Can you folks share the status or timeline regarding fastify support

@rsbh This isn't a huge priority for us at the moment, sorry! Official support means for more adapters is out of scope at the moment, but if we revisit this in the future we can revisit this discussion.


Small fix in wording

Category

[x] Content fix
[ ] New article

What's in this Pull Request?
Changed Remove an owner or member from a team, and to the unified group which backs the team. to Remove an owner or member from a team, and from the unified group which backs the team.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

teams/teams-ps/teams/Remove-TeamUser.md
:white_check_mark:Succeeded
View (teams-ps)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

@Szab - thanks for fix, much appreciated!


Add user defined time frame or frames

The user should be able to playblast a combination of small frames and frame ranges:
Example : 1-20, 25; 50; 75, 100-150
from-to, increment;
This translates to:

Frames 1 to 20:
Frames 25, 50 and 75
Frames 100 to 150

To elaborate a bit on this, this would be similar to printing specific pages in applications like Word as seen here:

Here's a quick function I wrote that parses it similar to how Word does it (and how V-Ray specific frame render settings supports it). Note that this supports both comma , and semicolon ; as separators.
import logging
import re

log = logging.getLogger(__name__)

def parse_frames(string):
    """Parse the resulting frames list from a frame list string.

    Examples
        >>> parse_frames("0-3;30")
        [0, 1, 2, 3, 30]
        >>> parse_frames("0,2,4,-10")
        [0, 2, 4, -10]
        >>> parse_frames("-10--5,-2")
        [-10, -9, -8, -7, -6, -5, -2]

    Args:
        string (str): The string to parse for frames.

    Returns:
        list: A list of frames

    """

    result = list()

    for raw in re.split(";|,", string):

        # Skip empty elements
        value = raw.strip().replace(" ", "")
        if not value:
            if raw:
                log.warning("Empty frame entry: '{0}'".format(raw))
            continue

        # Check for sequences (1-20) including negatives (-10--8)
        sequence = re.search("(-?[0-9]+)-(-?[0-9]+)", value)

        # Sequence
        if sequence:
            start, end = sequence.groups()
            frames = range(int(start), int(end) + 1)
            result.extend(frames)

        # Single frame
        else:
            try:
                frame = int(value)
            except ValueError:
                log.warning("Invalid frame description: '{0}'".format(value))
                continue
            result.append(frame)

    return result

Implemented with #25


Mousebites will generate emtpy board

I have two different boards (same shape)
But it generate empty board if I chose Mousebites

can you provide a pcb file with only the edge cuts?
it seems the edge is not in simple polygon, but it's covered by the red lines of debug view.

edge_only.zip
Please have a look
BTW, this project saved me !!
Thank you very much for your dedication

new problem found: Simulate Mill Fillets seems not working

Simulate Mill Fillets is only applied to preview, as the mill fillets should be part of actual milling.
But I can make it an option.

hmm it works here

really weird. Im using kikit-ui-v2.1-windows-x64.zip
and you can see the pictire I posted , the V-CUT lines(blue) still displaying when I chose Mousebites.

please try v2.2


does not do upgrade of snap

Affected Puppet, Ruby, OS and module versions/distributions

Puppet: 7
Ruby: 2.7
Distribution: bullseye 11
Module version: 1.1.1

How to reproduce (e.g Puppet code you use)
I declare the package in my module this way.  it works fine to install it.
package { 'microk8s':
ensure          => installed,
provider        => 'snap',
install_options => ['classic', 'channel=1.26'],
}
then i go back to the module, change the channel to 1.27
package { 'microk8s':
ensure          => installed,
provider        => 'snap',
install_options => ['classic', 'channel=1.27'],
}
running "puppet agent -t" does nothing.  I know my module gets executed as it does notice, but snap is not updated.  If I remove it manually with "snap remove microk8s" from the node, and then run "puppet agent -t" then it will install 1.27.
What are you seeing
What behaviour did you expect instead
it is supposed to do a snap refresh and update snap from 1.26 to 1.27
Output log
Any additional information you'd like to impart

Hey @max-chudnovsky ,
I believe the channels on microk8s snap are defined as 1.27/stable. Take a look here

sorry, I could not wait so I wrote my provider.  Snaps should use "install"
only when you add a new snap.  If you want to change the channel, you need
to use "refresh".  Your module would install the module, but if I changed
the version it was not doing anything.  In any case, good luck with your
module.
On Thu, May 30, 2024 at 10:38 AM Christos Papageorgiou <
@.***> wrote:

Hey @max-chudnovsky https://github.com/max-chudnovsky ,
I believe the channels on microk8s snap are defined as 1.27/stable. Take
a look here https://microk8s.io/docs/setting-snap-channel
—
Reply to this email directly, view it on GitHub
https://github.com/root-expert/puppet-snap/issues/23#issuecomment-2139740877,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A2MTLX4YKHWMWD7NDOHHPQTZE42XBAVCNFSM6AAAAABIH7WEOWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMZZG42DAOBXG4
.
You are receiving this because you were mentioned.Message ID:
@.***>


Rethink dev/prod/mode handling

Description
Rethink dev/prod/mode handling for Vite 4. See the discussion in https://github.com/vitejs/vite/discussions/9274
Suggested solution
There have been some proposals, but it probably needs discussion by the team
Alternative
No response
Additional context
No response
Validations

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

@benmccann Is this different from #9203? (Maybe we should add a link to https://github.com/vitejs/vite/discussions/9274 there though)

Thanks. I forgot about that one


修复"保存debugtalk代码不能立即生效"的问题

原因: 第二次导入的时候, 已经导入过同模块了, 不会从新的路径重新导入

Pull Request Test Coverage Report for Build 61

0 of 2 (0.0%)  changed or added relevant lines in 1 file are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.0001%) to 0.076%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

fastrunner/utils/loader.py
0
2
0.0%

Files with Coverage Reduction
New Missed Lines
%

fastrunner/utils/loader.py
1
0.0%

Totals

Change from base Build 60:
-0.0001%

Covered Lines:
1

Relevant Lines:
1323

💛  - Coveralls

发现已经有reload了, 没有pull新代码, 抱歉


k9s: 0.24.10 -> 0.24.11

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/derailed/k9s/releases.
meta.description for k9s is: "Kubernetes CLI To Manage Your Clusters In Style"
meta.homepage for k9s is: "https://github.com/derailed/k9s"
meta.changelog for k9s is: ""
Updates performed

Golang update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

/nix/store/dfidcw1kyfqp7rjzjs2ckakyvgr02qyd-k9s-0.24.11/bin/k9s passed the binary check.

1 of 1 passed binary check by having a zero exit code.

0 of 1 passed binary check by having the new version present in output.

found 0.24.11 with grep in /nix/store/dfidcw1kyfqp7rjzjs2ckakyvgr02qyd-k9s-0.24.11

found 0.24.11 in filename of file in /nix/store/dfidcw1kyfqp7rjzjs2ckakyvgr02qyd-k9s-0.24.11

directory tree listing: https://gist.github.com/77f2040d00e89cfcebfaf6cab0fe21f1

du listing: https://gist.github.com/7d0c67c2639a4cf4ae7b7e576b9de947

Rebuild report (if merged into master) (click to expand)

4 total rebuild path(s)

1 package rebuild(s)

1 x86_64-linux rebuild(s)
1 i686-linux rebuild(s)
1 x86_64-darwin rebuild(s)
1 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
k9s

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/dfidcw1kyfqp7rjzjs2ckakyvgr02qyd-k9s-0.24.11 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A k9s https://github.com/r-ryantm/nixpkgs/archive/40db7836fe09eadd3cfcf9127011b10ffd0e4f95.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/dfidcw1kyfqp7rjzjs2ckakyvgr02qyd-k9s-0.24.11
ls -la /nix/store/dfidcw1kyfqp7rjzjs2ckakyvgr02qyd-k9s-0.24.11/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

k9s

Maintainer pings
cc @Gonzih @markus1189 @kalbasit for testing.


Added mysqli extention documentation

Description 📜
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context.
Fixes # <issue_number_here>

Type of change 📝

[ ] Au#dio (Should be in mp3 format Includes speech clarity, Concise ,Low distortion)
[ ] Vi#deo (Animations, screen-recordings, presentations and regular explanatory films are all possibilties etc)
[x] Doc#umentation (Content Creation in the form of codes or tutorials)
[ ] Other (If you choose other, Please mention changes below)

Domain of Contribution 📊

[ ] Android Dev #(Flutter)
[ ] Android Dev #(Java)
[ ] Android Dev #(Kotlin)
[ ] Backend Dev #(Java)
[ ] Backend Dev #(.NET)
[x] Backend Dev #(PHP)
[ ] Backend Dev #(Python)
[ ] C/#CPP
[ ] Competitive #Programming
[ ] Cyber #Security
[ ] DS#A
[x] Data#base
[ ] Datascience with #Python
[ ] Datascience with #R
[ ] Frontend Dev #HTML/CSS/JS
[ ] Frontend Dev #React/Angular/Vue
[ ] Go#lang
[ ] Interview #Prep
[ ] Java_#Domain
[ ] Java#Script
[ ] ME#RN
[ ] Machine #Learning
[ ] Open #Source
[ ] Python_#Domain
[ ] Ru#st
[ ] Statis#tics
[ ] UI/#UX

Checklist ✅

[x] I follow Contributing Guidelines & Code of conduct of this project.
[x] I have performed a self-review of my own code or work.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[x] I'm GWOC'21 contributor

Screenshots / Gif (Optional) 📸

@meinhoonharsh I replaced it..

You can use PHP syntax highlighting in Markdown file
<?php
$var = "This is code with regular Code Highlighting";
?>

<?php
$var = "This is code written in PHP Syntax Highlighting";
?>

For this, you just need to add php after those three backticks, like
```php

@meinhoonharsh I did it. Sorry, I just learned markdown . I was not aware of this feature.

Here are few changes you must look for

Please test your documentation in any Grammar checking tool, because I saw spelling mistakes and some grammatical errors here
Also crosscheck your documentation from any trusted resource, because at one place you've stated, mysqli_connect() function. It takes three parameters
but mysqli_connect can take up to 6 parameters
Please try to avoid Extra spaces in the Code, you have given a lot of spaces in the code

@meinhoonharsh I made the document grammatically correct.
Remove all extra spaces given.
mysqli statment takes 6 parameter but i have explained 3 which are required to connect to the server.
socket , port and host parameter(remaining) are of no use in this documentation

@meinhoonharsh codes are tested in my pc by myself

At some places, you've changed Extention to Extension, but still some extention are there

@meinhoonharsh I update the file. Have a look.

I read my codes again. The only variable database was declared extra in the first 2 code section.
It is the 3rd parameter of the mysqli_connect() function. I explained it in the document later on. So I didn't use it in beginning.
I still removed it.

@meinhoonharsh Issue #1724

@praveenscience Sir please Review 🙃

@praveenscience @meinhoonharsh Sir, I made changes to the document. Please, Have a look.

Description
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. Fixes # <issue_number_here>
Type of change

[ ]  Au#dio (Should be in mp3 format Includes speech clarity, Concise ,Low distortion)
[ ]  Vi#deo (Animations, screen-recordings, presentations and regular explanatory films are all possibilties etc)
[x]  Doc#umentation (Content Creation in the form of codes or tutorials)
[ ]  Other (If you choose other, Please mention changes below)

Domain of Contribution

[ ]  Android Dev #(Flutter)
[ ]  Android Dev #(Java)
[ ]  Android Dev #(Kotlin)
[ ]  Backend Dev #(Java)
[ ]  Backend Dev #(.NET)
[x]  Backend Dev #(PHP)
[ ]  Backend Dev #(Python)
[ ]  C/#CPP
[ ]  Competitive #Programming
[ ]  Cyber #Security
[ ]  DS#A
[x]  Data#base
[ ]  Datascience with #Python
[ ]  Datascience with #R
[ ]  Frontend Dev #HTML/CSS/JS
[ ]  Frontend Dev #React/Angular/Vue
[ ]  Go#lang
[ ]  Interview #Prep
[ ]  Java_#Domain
[ ]  Java#Script
[ ]  ME#RN
[ ]  Machine #Learning
[ ]  Open #Source
[ ]  Python_#Domain
[ ]  Ru#st
[ ]  Statis#tics
[ ]  UI/#UX

Checklist

[x]  I follow Contributing Guidelines & Code of conduct of this project.
[x]  I have performed a self-review of my own code or work.
[x]  I have commented my code, particularly in hard-to-understand areas.
[x]  My changes generates no new warnings.
[x]  I'm GWOC'21 contributor

Screenshots / Gif (Optional)


Unable to reconfirm a users email

Hey I'm using devise_token_auth (0.1.38) along with the Devise confirmable module.
Everything is working fine but I am unable to reconfirm a users email address if they change it.
After looking through the source code, this is coming from some Rails method overides in app/models/devise_token_auth/concerns/user.rb:
def email_changed?
    false
end

This means that the before update method :postpone_email_change_until_confirmation_and_regenerate_confirmation_token in Devise's confirmable module is never run and the users email address is updated without reconfirmation.
What is this overide for? Any suggestions how to overcome this and reimplement Devise's initial implementation would be great.
Thanks

@Petecass please see this:
https://github.com/plataformatec/devise/issues/2688
I may have worked around the issue by putting the devise_token_auth include directive above the devise module directives:
Thus:
class User < ActiveRecord::Base
  include DeviseTokenAuth::Concerns::User

  devise :database_authenticatable, :registerable,
         :recoverable, :rememberable, :trackable, :validatable,
         :confirmable
end

Same here. Additionally even though account_update sanitizer is null, users are able to update their e-mail. How is it even possible by default, and how to disable e-mail updates?

I'm also having this issue and unsure how to get passed in.  Any resolution?

Hi there @Petecass @matthewnatali    ,
In an effort to cleanup this project and prioritize a bit, we're marking issues that haven't had any activity in a while with a "close-in-7-days" label. If we don't hear from you in about a week, we'll be closing this issue. Obviously feel free to re-open it at any time if it's the right time or this was done in error!
If you are still having the issue (especially if it's a bug report) please refer to our new Issue Template to provide some more details to help us solve it.
Hope all is well.


FAB in PDF tools ignores position defined in settings

that's intended behavior, because it's laying in bottom bar, position applied only on non containered FABs👀

Btw, how pdf tools are going? No bugs?

Tbh I don't use it, so I can't definitely say if there's any bugs


suggest future  and future.apply

Codecov Report

Merging #358 (f549294) into main (c16719d) will increase coverage by 0.32%.
Report is 5 commits behind head on main.
The diff coverage is 100.00%.

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

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
@@            Coverage Diff             @@
##             main     #358      +/-   ##
==========================================
+ Coverage   86.23%   86.55%   +0.32%
==========================================
  Files          24       24
  Lines        2230     2224       -6
==========================================
+ Hits         1923     1925       +2
+ Misses        307      299       -8

Files
Coverage Δ

R/A_nhdplusTools.R
100.00% <ø> (+13.33%)
:arrow_up:

R/downloading_tools.R
80.00% <ø> (ø)

R/geoserver_tools.R
91.30% <ø> (ø)

R/get_hydro.R
97.26% <ø> (ø)

R/get_nldi.R
88.88% <ø> (ø)

R/get_oaproc.R
39.06% <ø> (ø)

R/get_vaa.R
85.84% <ø> (ø)

R/plot_nhdplus.R
90.55% <ø> (ø)

R/prep_nhdplus.R
94.94% <ø> (ø)

R/rebuild_topology.R
100.00% <ø> (ø)

... and 3 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Introduce .first and .last to 64 implementation #583

SUMMARY

Introduce .first and .last to 64 implementation
If your pull request includes new functionality, then you must have corresponding new tests.

Automated Checks

[ ] I have run ./gradlew test and made sure that my PR does not break any unit test.
[ ] I have run ./gradlew checkstyleMain or the equivalent and corrected the formatting warnings reported.

The fetching of the first Node in Roaring64Bitmap is certainly not optimal. Any pointer how to get it without going through the iterator?

The fetching of the first Node in Roaring64Bitmap is certainly not optimal. Any pointer how to get it without going through the iterator?

I mean, you can duplicate some (simplified version) of the code that sets up the iterator to just do a left-most/right-most traversal of the ART. Not sure if it's worth it, though. Might not be that much faster...or simpler^^

I mean, you can duplicate some (simplified version) of the code that sets up the iterator to just do a left-most/right-most traversal of the ART.

Yep, but it would also need to duplicate from https://github.com/RoaringBitmap/RoaringBitmap/blob/bdd66cb6a2c8a73f481534f230360b7067b1536c/RoaringBitmap/src/main/java/org/roaringbitmap/art/AbstractShuttle.java#L46 (iterating until spotting the relevant leaf (even for first node)). As a whole, I anticipate it will lead to a quite large bunch of code, not necessarily much faster (as we still go through some loops).
I'll give it a first try, but I feel I'll end with some loops while better knowledge may lead to a unrolled implementation.

(e.g. can one confirm we could get the Art root, then iterate through .getChild(0) (or .getChild(getMaxPos()) for .last()) ?)

  private LeafNode getExtreme(Node root, boolean reverse) {
    Node parent = root;
    while (true) {
      if (parent instanceof LeafNode) {
        return (LeafNode) parent;
      } else {
        parent = parent.getChild(reverse ? parent.getMaxPos() : parent.getMinPos());
      }
    }
  }

  @Deprecated
  public long fastFirst() {
    assertNonEmpty();

    LeafNode firstNode = getExtreme(art.getRoot(), false);
    long containerIdx = firstNode.getContainerIdx();
    Container container = getContainer(containerIdx);
    byte[] high = firstNode.getKeyBytes();
    char low = (char) container.first();
    return LongUtils.toLong(high, low);
  }

The following looks to pass various tests. I'll push a clean commit for this.

I added a test by extracting a Set flagged as a way to generate all kinds of nodes. Debug confirmed various nodes where generated (I agree this is a weak argument given the source is generated by a plain 10k random longs, but it fits existing tests).

Merging.


Add BossRangeAttackComponent for developing new AttackTask

Description
This pull request only effect RangeAttackComponent with making some attibutes and method package-private for BossRangeAttackComponent. This does not impact the current Boss projectile attacks since the new component is not linked to any AttackTask but act as a template to work for an AttackTask.

This PR should not break anything as the feature does not work yet.

LGTM


no cardboard switch icon in fullscreen mode

in fullscreen mode there is no cardboard icon any more, this behavior changed from former version. i don't know if it's a bug or intentional, but i think it makes sense to go from embedded to fullscreen to cardboard, without having to go back from fullscreen to embedded so you can switch to cardboard.

Intentional.


Return the name of the currently loaded file in the load_dataset function.

Feature request
Add an optional parameter return_file_name in the load_dataset function. When it is set to True, the function will include the name of the file corresponding to the current line as a feature in the returned output.
Motivation
When training large language models, machine problems may interrupt the training process. In such cases, it is common to load a previously saved checkpoint to resume training. I would like to be able to obtain the names of the previously trained data shards, so that I can skip these parts of the data during continued training to avoid overfitting and redundant training time.
Your contribution
I currently use a dataset in jsonl format, so I am primarily interested in the json format. I suggest adding the file name to the returned table here https://github.com/huggingface/datasets/blob/main/src/datasets/packaged_modules/json/json.py#L92.

Implementing this makes sense (e.g., tensorflow_datasets' imagefolder returns image filenames). Also, in Datasets 3.0, we plan only to store the bytes of an image/audio, not its path, so this feature would be useful when the path info is still needed.

Hey @mariosasko, Can I work on this issue, this one seems interesting to implement. I have contributed to jupyterlab recently, and would love to contribute here as well.

@tsabbir96 if you are planning to start working on this, you can take on this issue by writing a comment with only the keyword "#self-assign"

#self-assign

@albertvillanova thank you for letting me contribute here.
@albertvillanova @mariosasko As I am totally new to this repo, could you tell me something more about this issue or perhaps give me some idea on how I can proceed with it? Thanks!

#self-assign

Hello there, is this issue resolved? @tsabbir96 are you still working on it? Otherwise I would love to give it a try

@EduardoPach This issue is still relevant, so feel free to work on it.

Hey @mariosasko, I've taken the time to take a look at how we load the datasets usually. My main question now is about the final solution.
So the idea is that whenever we load the datasets we also add a new column in the _generate_tables() method from the builders called filename (or file_name) that should be related files contained in each split, right?
Do you have any suggestions of where I could add that?

Is this issue still open? If yes, I'd like to work upon on it. Thanks

Is this issue still open? If yes, I'd like to work upon on it. Thanks

Definitely still open. I gave it a try, but then didn't get any feedback on my last question so I stop. Feel free to work on it.

It's still open, so feel free to work on it. This can be implemented by adding a param to the packaged builders' configs that inserts a column with file names (in _generate_tables) when True. Naming this column file_name sounds good to me.

Hi is the issues still open, is see no activity since September but it shows that it is still assigned to tsabbir96.  If
tsabbir96 is not planning to continue, can i get it assigned to me.

Looking forward to your implementation. I also really need this feature.
Thanks

Hi. I am new and would like to contribute to this issue @tsabbir96


Run apk update before apk upgrade.

I prefer running apk update before apk upgrade so that apk upgrade actually installs the latest stuff. Let me know what you think! :)

thanks! it make sense and lgtm.
but i do not know why the unit tests failed.  could u pls hav a look on it?


Crash about overflowing max_stackdepth

Summary
I tried to run scrapscript.py with RustPython. It crashed. I reduced the input code with CReduce and then manually from ~4k lines to 21 lines.
Expected
No crash. In the full example, I expect the test suite to run. In the reduced example I expect nothing to happen.
Actual
# scrapscript.py
class LeftParen:
    pass
class RightParen:
    pass
class Lexer:
    def read_char(self) :
        return None
    def read_one(self) :
        while self:
            c = self.read_char()
            break
        else:
            return x
        {
                "(": LeftParen,
                ")": RightParen}
def tokenize():
    lexer = Lexer()
    while (lexer.read_one()) :
        pass
tokenize()

cedar% rustpython scrapscript.py test
[vm/src/frame.rs:1942] self = ExecutingFrame {
    code: code: <code object read_one at ??? file "scrapscript.py", line 8>,
    state: FrameState {
        stack: [
            [PyObject PyStr { value: "(", kind: Ascii, hash: -1 }],
            [PyObject [PyType LeftParen]],
            [PyObject PyStr { value: ")", kind: Ascii, hash: -1 }],
        ],
        blocks: [],
        lasti: 16,
    },
}
thread 'main' panicked at vm/src/frame.rs:1943:9:
tried to push value onto stack but overflowed max_stackdepth
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
cedar%

#5273 fixed a max_stackdepth calculation bug.

Building from main (5d6d1a6da7b05daad3bedaa9546cbe07714ea04a) and running still crashes in the same way.

Reduced further:
# scrapscript.py
def read_one() :
    self = 1
    while self:
        break
    else:
        return self
    {")": 2}
read_one()

# scrapscript.py
def f() :
    while 1:
        break
    else:
        return 2
    {1: 2}
f()


Ctrl-C should interrupt the incomplete input/cancel current operation, Ctrl-D should quit

To reproduce:

Open kshell
(Optionally) run a blocking task like Thread.sleep(10000)
Press Ctrl-C

The shell terminates. However, most scripting shells like R, python, scala-ammonite use Ctrl-C to interrupt the current line input, and Ctrl-D to quit the shell (often asking for confirmation first). It would be nice if the kotlin repl could follow this pattern.
Also see
https://youtrack.jetbrains.com/issue/KT-21152
https://youtrack.jetbrains.com/issue/KT-11369

It will be in 0.2.1 (very soon). This is already implemented in the case of Kotlin interpreter for Zeppelin. But I need to check behaviour in Spark environment before make it generally available.


[cbuild schema] Backward compatibility introduced in version 2.3 of the schema

schema validation currently fails on some 2.* versions of generated *.cbuild.yml files due to the following issue:
MassStorage.cproject.yml solution: ../workspace/workspace/MassStorage/MassStorage.csolution.yml]]] against schema: jsonschema: '/build/generators/0' does not validate with https: [//raw.githubusercontent.com/Open-CMSIS-Pack/devtools/schemas/projmgr/2.3.0/tools/projmgr/schemas/cbuild.schema.json#/properties/build/$ref/properties/generators/$ref/items/$ref/required](https://arm-ce.slack.com//raw.githubusercontent.com/Open-CMSIS-Pack/devtools/schemas/projmgr/2.3.0/tools/projmgr/schemas/cbuild.schema.json#/properties/build/$ref/properties/generators/$ref/items/$ref/required): missing properties: 'from-pack'"

Generated cbuild file using cbuild 2.1 or 2.2 indeed does not have the following section from-pack
which was marked as required as part of release 2.3
Environment:
cbuild 2.1
cbuild 2.2
linux

And therefore, a breaking change was introduced in version 2.3 of the schema.
Potential Solution?
Make the schema backward compatible and remove from-pack from the list of required fields.
Indeed, any change to a required list represents a breaking change
Impact
cbuild.yml is now parsed by our tools to determine artefacts produced by a build and therefore, needs to abide by the schema so that there is no parsing issues. We expect to only have to define one parser of cbuild files for all 2.* versions of the schema. It is hence crucial that backward compatibility is ensured over all 2.* versions.
Additional context
Add any other context about the problem here.

@acabarbaye when I understand this request correctly, you ask for a schema that is compatible with CMSIS-Toolbox 2.0, 2.1, 2.2, 2.3, 2.4, and so one.
In an cloud environment we will use the latest versions of the tools. There should be no need for this and we should be pragmatic here. The schema should therefore match the latest toolbox, but should in case of from-pack also tolerate YML inputs from previous versions. I believe this is already the case.

@ReinhardKeil I probably didn't explain the problem we are facing properly. We are currently in the situation where "schema in case of from-pack" does not  "tolerate YML inputs from previous versions". Indeed previous versions of the files do not set the from-pack which was introduced in 2.3 and since it is also marked as required as part of that version, this version of the schema cannot be used to parse/validate YML files from previous versions. Hence the break of backwards compatility. My suggestion (and the patch we had to introduce) is to remove the from-pack from the required list.

Action:  remove the from-pack node from the required list and make it optional.


Including a Subdirectory in URI - ASP.NET Core 3.1 Identity Server

I am trying to add a subdirectory to my identity server, so I can use it with nginx.
To note, this is an identity server with the UI, see (quickstart ui)
After perusing the github issues for identity server, I managed to find the code to actually add the subdirectory.
Here is my configure:
public void Configure(IApplicationBuilder app)
{
	if (Environment.IsDevelopment())
	{
		app.UseDeveloperExceptionPage();
	}

	app.Map("/auth", app =>
	{
		app.UseRouting();

		app.UseStaticFiles();

		app.UseAuthentication();
		app.UseAuthorization();

		app.UseEndpoints(endpoints =>
		{
			endpoints.MapControllerRoute(
				name: "default",
				pattern: "{controller=Home}/{action=Index}/{id?}");
			endpoints.MapRazorPages();
		});

		app.UseIdentityServer();
	});
}

However, when I navigate to http://xxx:8888/auth/account/login and attempt to login and receive an identity cookie, the URL stays the same and I am presented with a blank screen and no cookies. What should happen is that I should be redirected back to the homepage with a particular user logged in.
This seems to happen only when I add a subdirectory.
To note, the well known endpoints work fine when getting an access token as password or resource owner with the /auth.
Here is my configuration services, is there something missing in here?:
public void ConfigureServices(IServiceCollection services)
{
	string connectionString = Configuration.GetConnectionString("AzureConnection");
	var migrationsAssembly = typeof(Startup).GetTypeInfo().Assembly.GetName().Name;

	services.AddCors(options =>
	{
		options.AddPolicy("CorsPolicy",
			builder => builder.AllowAnyOrigin()
			.AllowAnyMethod()
			.AllowAnyHeader());
	});

	services.AddControllersWithViews().AddRazorRuntimeCompilation();

	services.AddRazorPages()
		.AddRazorPagesOptions(options =>
			{
				options.Conventions.AuthorizeAreaFolder("Identity", "/Account/Manage");
			});

	services.AddDbContext<IdentityDbContext>(options => options.UseSqlServer(connectionString, sql => sql.MigrationsAssembly(migrationsAssembly)));
	services.AddDbContext<ConfigurationDbContext>(options => options.UseSqlServer(connectionString, sql => sql.MigrationsAssembly(migrationsAssembly)));

	services.AddIdentity<ApplicationUser, IdentityRole>(options =>
	{
		options.SignIn.RequireConfirmedEmail = true;
	})
		.AddEntityFrameworkStores<IdentityDbContext>()
		.AddDefaultTokenProviders();

	services.AddAuthentication()
		.AddOpenIdConnect("azuread", "Azure AD", options => Configuration.Bind("AzureAd", options));

	services.Configure<OpenIdConnectOptions>("azuread", options =>
	{
		options.GetClaimsFromUserInfoEndpoint = true;
		options.SaveTokens = true;
		options.Scope.Add("openid");
		options.Scope.Add("profile");
		options.Scope.Add("email");
		options.Events = new OpenIdConnectEvents()
		{
			OnRedirectToIdentityProviderForSignOut = context =>
			{
				context.HandleResponse();
				context.Response.Redirect("/Account/Logout");
				return Task.FromResult(0);
			}
		};
	});

	var builder = services.AddIdentityServer(options =>
	{
		options.IssuerUri = "http://xxx:8888";
		options.PublicOrigin = "http://xxx:8888";

		options.Events.RaiseErrorEvents = true;
		options.Events.RaiseInformationEvents = true;
		options.Events.RaiseFailureEvents = true;
		options.Events.RaiseSuccessEvents = true;
		options.UserInteraction.LoginUrl = "/Account/Login";
		options.UserInteraction.LogoutUrl = "/Account/Logout";

		options.Authentication = new IdentityServer4.Configuration.AuthenticationOptions()
		{
			CookieLifetime = TimeSpan.FromHours(10), // ID server cookie timeout set to 10 hours
			CookieSlidingExpiration = true
		};
	})
	.AddConfigurationStore(options =>
	{
		options.ConfigureDbContext = b => b.UseSqlServer(connectionString, sql => sql.MigrationsAssembly(migrationsAssembly));
	})
	.AddOperationalStore(options =>
	{
		options.ConfigureDbContext = b => b.UseSqlServer(connectionString, sql => sql.MigrationsAssembly(migrationsAssembly));
		options.EnableTokenCleanup = true;
	})
	.AddAspNetIdentity<ApplicationUser>();
}

This is actually availiable to test as its on a VM with a public URL upon request.

I am trying to add a subdirectory to my identity server, so I can use it with nginx.

Explain please.
I would start with removing the explicit issuer name and public origin and add the forwarded headers middleware. If the app is hosted on a sub dir on nginx you might need to set the BasePath in ASP.NET Core.
https://docs.microsoft.com/en-us/aspnet/core/host-and-deploy/linux-nginx?view=aspnetcore-3.1

Sorry that should be, I have added a subdirectory to my code so that /auth will be before anything else.
We have nginx sat there ready with https so I have all the forwarding headers ready and set in my code but commented out for now as I am just accessing my identity server from outside of nginx in order to see if everything behaves as it should with http://xxxxxx:8888/auth/xxxxx.
I think we have found the issue (when adding the subdirectory) and it seems to be with the [ValidateAntiForgeryToken] attribute in the Account Controller as when we comment this out for Login and Logout methods, it works in terms of logging people in and out with the browser.
To note, we get the same behaviour when going through nginx, so I thought I would deploy outside of it and start with the basics to eliminate other possible issues like https, etc.
Yes had already set the basepath to /auth and my other deployment after this post was commenting out the issuer name and public origin.


Discuss licensing

I am providing headers and a source under MIT license and I like it. GLib is LGPL. Can I leave things as-is? (Related discussion on StackExchange).
I also feel like I should contact RtMIDI developers when the project is a bit more mature and tell that I moved non-binding license clause to readme everyone sees.

Choose a license states that about GNU LGPL v3:
However, a larger work using the licensed work through interfaces provided
by the licensed work may be distributed under different terms
and without source code for the larger work.

(I wonder if GLib is using or will use LGPLv3 exactly.)

Can your MIT library use an LGPL library?
How does using a LGPL gem affect my MIT licensed application?

Apparently it's ok.


WIP: mobile betting in embeds

It seemed like a good idea, but the betting interface takes up too much room in the embeds, I think.

It seemed like a good idea, but the betting interface takes up too much room in the embeds, I think.

Yeah I also feel like maybe having anything expand out in embeds may be a bad idea?

Do you want to just wait until I push embed design changes before making these?

Yeah, whatever design you propose is probably better. I'll just close this PR.

Sorry! Hope you didn't spend too much time on it :(


Stop script in NPM Script Explorer

Feature to stop desired task in NPM Script Explorer

@deadcoder0904 you can terminate a running script from Tasks>Terminate Task, does this cover your needs?

A context menu on the script might be better than that. You don't need to go to the Menu bar to do it. I think that's why the NPM Explorer is made :)

Fair request, but you can also do it from the NPM Explorer today, just execute the script again and you will get a notification whether you want to terminate or restart the task ;-)

A square stop button would be nice for long-running tasks like start [server] since Terminate Task takes a couple more steps (search command and pick task as I often have 2-3 running).

Yeah I guess.
Since 1 script isn't going to be started twice or technically it shouldn't so a stop button  like 🔳 can replace the play button.

Agreed. @egamma I used the restart task notification today. Super helpful!

The idea is good, there is API to track task execution. I´ll investigate.

A elegant solution I think would be to switch the play icon out with a stop (and perhaps also a refresh icon) but stop or restart a running task in the "Scripts Explorer".


How do you bold a cell?

I've tried a bunch of different things (setting r, setting s:{fontId:1}, other stuff) but I can't seem to bold a cell when writing a workbook. How should this work?

@SheetJSDev first off just want to say thanks for this great repo.
In regards to what you wrote here: https://github.com/SheetJS/js-xlsx/issues/74#issuecomment-46981775 I don't understand a few things.
For example, in item 1 you write:

You can manually add a font with  there.

I don't understand that sentence at all. Can you clarify?
You then mention to take a look at a rich text stress, which generates a file with formatted cells, which is exactly what I am trying to accomplish. How do I do that? I didn't see a way to pull up the source code that generates this file for the purposes of trying to figure out how it was done.
Building off one of your example source codes, I added a function that takes an array of objects and builds the data needed for the excel file.  Below is what it looks like. For now, what I am trying to do is make the header row bold and underline. How can I change the code to accomplish this?
By the way, for anyone else wanting to use this function, it requires the use of either the lodash or underscore library.
function sheet_from_array_of_objects(data, opts) {
		var ws		= {},
			range	= {s: {c:10000000, r:10000000}, e: {c:0, r:0 }},
			cell,
			cell_ref,
			R_loc = 0;
			C_loc = 0;

		// Add Header Row
		_.forEach(data[0], function(value, key) {
			cell = { v: key };
			cell.t = 's';
			cell_ref = XLSX.utils.encode_cell({c: C_loc,r: R_loc});
			ws[cell_ref] = cell;
			range.e.c++;
			C_loc++;
		});
		R_loc++;

		// Add rows and their cells
		for(var R = 0; R != data.length; R++) {
			C_loc = 0;
			_.forEach(data[R], function(value, key) {
				if(range.s.r > R) range.s.r = R;
				if(range.s.c > C_loc) range.s.c = C_loc;
				if(range.e.r < R) range.e.r = R;
				if(range.e.c < C_loc) range.e.c = C_loc;

				cell = { v: data[R][key] };

				if(cell.v != null) {
					cell_ref = XLSX.utils.encode_cell({c: C_loc,r: R_loc});
					if(typeof cell.v === 'number') cell.t = 'n';
					else if(typeof cell.v === 'boolean') cell.t = 'b';
					else if(cell.v instanceof Date) {
						cell.t = 'n'; cell.z = XLSX.SSF._table[14];
						cell.v = datenum(cell.v);
					}
					else cell.t = 's';

					ws[cell_ref] = cell;
				}
				range.e.c++;
				C_loc++;
			});
			range.e.r++;
			R_loc++;
		}
		if(range.s.c < 10000000) ws['!ref'] = XLSX.utils.encode_range(range);
		return ws;
	}

@bchr02 Just thought I'd respond to you in case anyone else is confused, even though you've probably worked it out by now.
If you unzip the rich text stress that SheetJS linked and open the xl/style.xml you'll see what he meant by being able to add  to a new font.
He provides the following example:
o.push('<fonts count="2"><sz val="12"/><color theme="1"/><name val="Calibri"/><family val="2"/><scheme val="minor"/><sz val="12"/><color theme="1"/><name val="Calibri"/><family val="2"/><scheme val="minor"/></fonts>');

Presumably that was to replace line 158 here
That's pretty much all that's required to get his workaround working, he then referred to his newly created font in the get_cell_style function.
function get_cell_style(styles, cell, opts) {
    var z = opts.revssf[cell.z||"General"];
    for(var i = 0; i != styles.length; ++i) if(styles[i].numFmtId === z && (styles[i].fontId === 1 ? cell.bold : !cell.bold)) return i;
    styles[styles.length] = {
        numFmtId:z,
        fontId:cell.bold ? 1 : 0, // This line!
        fillId:0,
        borderId:0,
        xfId:0,
        applyNumberFormat:1
    };
    return styles.length-1;
}

@nathanKramer thanks very much for the clarification. Actually, I moved on to another library called exceljs which in my opinion is more in line with the node philosophy because it offers streaming support in addition to styles. This allows me to to build excel files in pieces (or streams of data) which is extremely beneficial with large data sets. And it is not blocking and does not consume as much memory.

Hi could any one help me in changing the deader of the table style, i want to change header cell color.. Please help me..

We offer this in the Pro compendium.  Since companies have paid for the features already, it would be unfair to them if we turned around and made it available as open source.  We have a longer comment in a gist.

Setting a cell to bold is a pro feature? Seriously?

Setting a cell to bold is a pro feature? Seriously?

lol, oneliner csv export is much easier

The open source libraries were originally created to solve a specific problem involving data import and export.  In fact, at the very beginning the CLI tools actually were called xlsx2csv and xls2csv (which, as you can imagine, converted xlsx and xls files respectively to csv).  Once you get past the easy 90% of the problem, the last 10% takes some serious testing and corner case discovery.
Prior to what is now the "Pro version", my personal savings and time, as well as the patience and monetary contributions from my younger sister and parents, subsidized that research and development.  Thanks to sales of the Pro version, we are able to continue providing support for the open source build without having to resort to questionable tactics like relicensing to a non-free license or abandoning entirely.
If you genuinely feel that removing the open source option or abandoning the open source project is a better outcome than the current situation, definitely let us know.  We were under the impression that keeping the main projects free and open source under an Apache 2.0 License was the best decision for us and the best outcome for the community.
PS: Microsoft uses the open source library in Office 365 Apps.  The fact that they continue to use this library despite its current shortcomings certainly suggests the current free/pro split is in a reasonable place (and no, they have not reached out to us in the interest of purchasing a Pro license, so Microsoft developers probably never once asked about the necessity of "setting a cell to bold")

Since companies have paid for the features already, it would be unfair to them if we turned around and made it available as open source.

Actually, not really. It is common practice for other teams, for example at GitLab. You might need to review your pricing model for the paid plans, as people may not want to continue paying money just for setting a cell to bold. (But maybe they would for advanced colorization, or support of certain format versions etc. This is somethings that will keep evolving.)


Access to exit button?

I couldn't find anything in the code for using the exit button (keycode 412).  Is there a way to do that?

Hi! Would you be able to test if https://github.com/supersaiyanmode/PyWebOSTV/pull/24 works for you? Thanks so much for your time!

I would love to.  Could you get the newest pushed out to pypi?

I was hoping you'd perhaps want to test the latest source directly from this repo. If if works for you, I'd push it to PyPI so everyone can get it. (I'm unable to test it -- I don't have access to an LG TV right now).
Not entirely sure, but if you're looking for steps, they are below:
$ source your_virtualenv/bin/activate
$ cd /tmp
$ git clone https://github.com/supersaiyanmode/PyWebOSTV
$ cd PyWebOSTV
$ python setup.py install
$ # You should now have the new exit button from #24 now.

Thanks again for your time and help!

Yea, I can do that.  Thanks

Confirmed exit button works!

That is awesome! I'll push it out to PyPi very soon. Thanks for testing.

Pushed it out to PyPI: https://pypi.org/manage/project/pywebostv/release/0.8.4/


SparkServer was updated to be runnable on http and https protocols at…

… same time. Api call was added to SparkBase to support that. Ability to specify special ssl port and justSsl flag were added.

When is supposed this to be merged? I need this.

+1

Pull request was synced with latest code base.

Where exactly is it synchronized? In the master code base I can not find the string 'SPARK_DEFAULT_PORT_SSL' neither the 'sslPort' variable?
And what about the wrong initialization of sslPort here: https://github.com/perwendel/spark/pull/329/commits/1e2cfa11aa25e2e4eb4c38ed4676b5abf60634fa

It was better to say that it is redone. @georgeyanev Please see commit e1b6144b63ef5346a35be3cffbf6f96134b4c784

@gimlet2 Is this feature is available in 2.5.3 release? I need this feature ASAP. thanks

Is this issue still relevant now when there's an instance API?
Service http = ignite()
Service https = ignite()

http.port(1234)
https.port(1235)
https.secure(...

As per mentioned, I think this should be taken care of by the instance-api, or in a layer in front of Spark.


protocol method names are parsed same as protocolname

🚨New Issue Checklist🚨

[ no] Updated swift-ast to the latest version
[yes ] I am aware that the swift-ast -github-issue /path/to/file.swift might help me generate this file

Issue Summary
the ast output of a protocol method name comes out as same as protocol name
Environment

OS Info: Mac OS 10.14.4
Yanagiba/swift-ast version: 0.18.10

Reproduction Steps

Sample code
protocol TableUpdatable {
    func reloadTable()
    func errorLoadingData()
}

Command to run swift-ast with the code above:
swift-ast sample.swift

Expected Result
proto_decl
name: TableUpdatable
0: kind: method
name: reloadTable
1: kind: method
name: errorLoadingData
Actual Behavior
proto_decl
name: TableUpdatable
0: kind: method
name: TableUpdatable
1: kind: method
name: TableUpdatable

There hasn't been any activity on this issue recently. Due to the high number of incoming GitHub notifications, we have to clean some of the old issues, as many of them have already been resolved with the latest updates.
Please make sure to update to the latest version and check if that solves the issue. Let us know if that works for you by adding a comment.


Deletion?

Hey, great package!
Is there any way to delete a attachment from the bin?
Thanks and have a good one!

Okay, I just implemented that. Checkout the new version v1.1.0

Haha amazing! 🔥 will check that out later


Allow @ queries for metadata

Metadata-based querying of Epochs is very powerful. For example, these work:
import mne
path_kword = mne.datasets.kiloword.data_path() + '/kword_metadata-epo.fif'
kword = mne.read_epochs(path_kword)
kword["NumberOfLetters == 4 & ConsonantVowelProportion < .5"], kword["WORD == 'area'"]

But what doesn't work is pulling a variable from the current scope with @:
word = 'area'
kword.metadata.query("WORD == @word")

This works, in Pandas. But this errors:
kword["WORD == @word"]

Would be cool to have!
@choldgraf

wouldn't this be redundant with PyBIDS ?

sorry my bad. Read too quickly

wouldn't this be redundant with PyBIDS ?

I'm kinda wondering how you misread this now ...

Probably living with too many open GitHub tabs or emails open, and clicking into one assuming it was the one you meant to respond to without actually checking. At least that's how it has happened to me in the past, too :)

I tried to understand how Pandas does it, and I don't. And a self-made workaround based on looking up stuff in locals probably wouldn't work either.
Why are we using pd.DataFrame.eval btw over .query?

It was a while ago but I think we did have some discussion about which one to use. Maybe one is/was deprecated?

I'm pretty sure that they accomplish mostly the same thing, but that eval returns a boolean mask according to whatever the query was, while query returns the DataFrame object itself. So probably we use eval because we want that mask somewhere else in the code, I'm guessing.

... 2 years later, I tried to get this to work by manipulating the level kwarg (how far up the call stack pandas looks for @variables), as well as passing local_dict=globals() and a few other hackish things; nothing worked.  Given that it's possible to generally replace use of @ with a fairly similar f-string nowadays (e.g., epochs['WORD in @words'] becomes epochs[f'WORD in {words}']) I think we can close this as YAGNI.


refactor(fonts): use fonts from unpkg cdn

Overview
This should make using fonts a lot easier now.
We can recommend our users pull fonts from from unpkg cdn and our dev env will pull from the same place.

👍 ✅


Macros file

Is there a way to specify a KaTeX macros file? Either JSON (to share with VSCode markdown plugins) or just plain TeX to include?

Thanks for the suggestions.
I have added this feature in the latest version. Please see README on how to use it.


Getting unwanted border with 0.23.0

After upgrading to 0.23.0 I get a unwanted 1.5px border around codeblocks.

Caused by this:

0.22.* or after unchecking the style above:

Also this style when showing frame title:

Interesting! The border style itself has always been there, I wonder why the new version makes it more visible for you.
The only thing I changed is that there is another possible source for the borderColor style now. Which theme are you using?

Note that you can always use the config option styleOverrides to override any theme's border color with a color of your choice, or even remove it completely by setting it to none:
// astro.config.mjs
import { defineConfig } from 'astro/config'
import astroExpressiveCode from 'astro-expressive-code'

export default defineConfig({
  integrations: [
    astroExpressiveCode({
      styleOverrides: {
        borderColor: 'none',
      },
    }),
  ],
})

Using theme nord.
In 0.22.2 the border is #454e60

Looks great - thanks for the quick fix! :)
The tabs look better also without the gap between tab and frame border


Request: Add Project Name as Variable to Export

It would be useful to be able to use the current project name in the export result set wizard. For instance, all of my exports are set to a general DBeaverExport folder which I then manually file away within an Export folder within the project. While the timestamp is useful to prevent overwriting or duplication, it would be nice to be able to specify the project name as a prefix to the filename, or as part of the path to the directory where the export is being saved.
E.g., something like this:

Ok. That's easy.


about mean_value and scale

Hi Robert, thanks for your paper and codes, great work!
I noticed that the in train_voc.py and train_coco.py, there are data transform_param:
scale：0.017
mean_value：103.94，116.78，123.68
and in MobileNet_SSD, the transform_param seems to be:
scale: 0.007843
mean_value: 127.5, 127.5, 127.5
both transform_params are ok for train and test as I have trained MobileNet_SSD and Pelee for my own data (to detect head and have only 1 class)
So I have two questions here:

where these transform_params come from?
how did they impact mAP and FPS?

looking forward for your reply, thank you!

@vincent8877, How to use your own data to train the accuracy of the model in pelee？I also have only one class


added analyze.sh to match existing build.sh (w/updated docs)

added posix analyze.sh script to simplify generating reports.  also updated docs to reflect

 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: pracplayopen sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

Thanks very much for sharing your pull request.
We're moving in a different direction at this point. We actually removed the build.sh file in favor of standardizing on PowerShell Core scripts which run cross platform.


add example notebook

What kind of example would we want here?

we have https://github.com/developmentseed/titiler-stacapi/blob/main/notebooks/demo_wmts_eoapi.ipynb
I think for now it's the minimum we can do 🙏


Add horizontal sliding as an effect.

Hi, I love the Nivo Slider. Is it possible to add a horizontal sliding effect?
Cheers,
Danny

I've been working on my own fork of nivoSlider and I've just added a "cycle" effect. I think this is the effect you're asking for here.
You can check it out here: https://github.com/Jaybuz/Nivo-Slider


Use a settings window to let user make customized settings

We could use a settings popup window to let user set preferences like:

Listen on what address:port
Dart/light theme
Path to custom certificate
...

@yuankunzhang after last update we introduced an initial setting popup where you can select the address and the port, I think that dark/light theme toggle is a standard in a window corner and so i prefer to keep it there, regarding the custom path I think that is a very good idea, if you want I can assign you this issue:

add a browse file button in the initial menu
check that the file is a real certificate
(probably is better to save that certificate in local machine certificate store and check if it is valid on each startup)


Fix client name

This is the .NET/C# client, and not Java client.

@jlesech thank you. Can you please submit this against the live branch so that we can deploy it immediately?


Mods to core-metadata addressable test

Test mods applicable to edgex-go PR https://github.com/edgexfoundry/edgex-go/pull/938
Issue #118

ALL - I am asking that this PR not be approved yet and be placed on hold!  This is sizable and will take some time to review and process.  DO NOT ACCEPT or MERGE.  Comment ONLY!!


Using Input variable inside task Command breaks path by prepending CWD

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

VS Code Version: 1.86.2
OS Version: Windows_NT X64 10.0.26063

Steps to Reproduce:

Add below snippet as tasks.json
Run Do Something with Path from PickString and select C:\Temp
See the current directory being appended to the path, resulting in {CWD}\C:\Temp\DoSomething.exe being called, instead of the expected C:\Tempt\DoSomething.exe

I first thought this was an issue with an extension I was using: https://github.com/rioj7/command-variable/issues/83
Tasks.json:
{
  "version": "2.0.0",
  "inputs": [
    {
      "id": "selectDirectory",
      "type": "command",
      "command": "extension.commandvariable.pickStringRemember",
      "args": {
        "description": "Deploy to where?",
        "options": [
          ["Use previous directory", "${remember:deployDirectory}"],
          ["Pick directory", "${openDialog:deployDirectory}"]
        ],
        "default": null,
        "openDialog": {
          "deployDirectory": {
            "description": "Which directory?",
            "canSelect": "folders",
            "defaultUri": "C:\\temp",
            "keyRemember": "deployDirectory"
          }
        }
      }
    },
    {
      "id": "pickADirectory",
      "type": "pickString",
      "description": "Pick a directory to use",
      "options": [
        "C:\\Temp"
      ]
    }
  ],
  "tasks": [
    {
        "label": "Task A",
        "command": "Write-Output",
        "type": "shell",
        "args": [
            "Selected '${input:selectDirectory}' for A"
        ]
    },
    {
      "label": "Do Something with Path",
      "command": "${input:selectDirectory}\\DoSomething.exe",
    },
    {
      "label": "Do Something with Path in workspace",
      "command": "${workspaceFolder}\\DoSomething.exe",
    },
    {
      "label": "Do Something with Path from PickString",
      "command": "${input:pickADirectory}\\DoSomething.exe",
    }
  ]
}

Based on your configuration, this is working as designed since you include the workspace folder variable

Why are you using input here if it's just one option?

It was a small reproducable setup - the one I ran into the issue in, is way longer so I created the smallest reproducable I could think off
But the issue is, that the CWD is being appended if the command contains a variable through input - if you use ${workspaceFolder} it does not - so Do Something with Path from PickString goes wrong

closing as I cannot reproduce the issue


VS Buildtools 2022 17.7 not detected

Hi,
i recently removed VS Buildtools Preview 17.8 and installed VS Buildtools 17.7 but now vswhere doesn't detect my installation anymore

Please see https://github.com/microsoft/vswhere/wiki/FAQ#why-doesnt-vswhere-find-visual-studio-build-tools.


remove unsupported extractors from meltano.com docs

Migrated from GitLab: https://gitlab.com/meltano/meltano/-/issues/1625
Originally created by @dmor on 2020-01-28 20:30:18

Currently, there are extractors listed at https://meltano.com/plugins/#extractors that are not available in the hosted Meltano UI. This is confusing.
We need to remove:

CSV
Carbon Intensity
Marketo
Mongo DB

If we are still going to allow users of the open source project to use them, we can note that there are additional unsupported extractors and link to documentation in the repo itself. The website should only be talking about what we have available for our target persona in the hosted version.

View 3 previous comments from the original issue on GitLab


General improvements

Including suggestions made by @blsqr:

Update development status to beta
Test now fail if there is any warning message
The tests that were still working with unittest have been moved to pytest
Test and add support for Python 3.10
Improve the test documentation
Move from SourceFileLoader.load_module() to importlib.util.spec_from_file_location.loader.exec_module() to avoid deprecation in Python 3.12
Correct some other warnings

Bug correction:

TabData objects' coordinates now are simplified in the model file, using the _subscript_dict as other objects.

Coverage increased (+0.0001%) to 99.851% when pulling 9c864cb8e6dfe0887c420468bbc2bd356b6e3e5d on dev into 4aedaf7a439e301d4e7788d0e098f378f0e3cee2 on master.


More lazy loading in HTTPServerRequest

Similarly to https://github.com/rejectedsoftware/vibe.d/pull/1677
Some values are actually a bit problematic thanks to D's type concept (mutable methods!), for example consider the method fullURL method:
@property URL fullURL()
const @safe {
     URL url;
...
	url.username = this.username; //  Error: mutable method vibe.http.server.HTTPServerRequest.username is not callable using a const object
	url.password = this.password; //  Error: mutable method vibe.http.server.HTTPServerRequest.password is not callable using a const object
        return url;
}

I initially had the idea to hack the type system with something like assumeConst:
import std.traits;
private auto assumeConst(T)(T t) @trusted
    if (isFunctionPointer!T || isDelegate!T)
{
    enum attrs = functionAttributes!T | FunctionAttribute.const_;
    return cast(SetFunctionAttributes!(T, functionLinkage!T, attrs)) t;
}

But. of course, without success

If I'm not overlooking something, I think this needs to set _form to .init first, or it would try to access the null field.

Excellent catch! Fixed.


TypeError: import_fs.default.cpSync is not a function[turborepo]

What version of Turborepo are you using?
create-turbo@latest
What package manager are you using / does the bug impact?
pnpm
What operating system are you using?
Linux
Describe the Bug
m@m-inspiron5567:~/code/allbetter/turbogame$ npx create-turbo@latest

npx: installed 182 in 20.132s

>>> TURBOREPO

>>> Welcome to Turborepo! Let's get you set up with a new codebase.

? Where would you like to create your turborepo? ./my-turborepo
? Which package manager do you want to use? pnpm

Aborting installation.
Unexpected error. Please report it as a bug:
TypeError: import_fs.default.cpSync is not a function
    at run (/home/m/.npm/_npx/223541/lib/node_modules/create-turbo/dist/index.js:328:21)
    at processTicksAndRejections (internal/process/task_queues.js:95:5)

1st time user of turborepo, not the best DX 🙋♂
Expected Behavior
no error should be printed.
To Reproduce
Follow provided steps.
Reproduction Repo
No response

m@m-inspiron5567:~/code/allbetter/turbogame$ npx --version
6.14.17
m@m-inspiron5567:~/code/allbetter/turbogame$ node --version
v14.21.1

Maybe try with a more recent version of node :)

Using node@18 works as expected
m@m-inspiron5567:~/code/allbetter/turbogame$ volta install node
success: installed and set<EMAIL_ADDRESS>(with<EMAIL_ADDRESS>as default
m@m-inspiron5567:~/code/allbetter/turbogame$ npx --version
8.19.2
m@m-inspiron5567:~/code/allbetter/turbogame$ npx create-turbo@latest
Need to install the following packages:
<EMAIL_ADDRESS>Ok to proceed? (y) y

>>> TURBOREPO

>>> Welcome to Turborepo! Let's get you set up with a new codebase.

? Where would you like to create your turborepo? ./my-turborepo

? Which package manager do you want to use? npm

>>> Created a new turborepo with the following:

 - apps/web: Next.js with TypeScript
 - apps/docs: Next.js with TypeScript
 - packages/ui: Shared React component library
 - packages/eslint-config-custom: Shared configuration (ESLint)
 - packages/tsconfig: Shared TypeScript `tsconfig.json`

>>> Success! Created a new Turborepo at "my-turborepo".
Inside that directory, you can run several commands:

  npm run build
     Build all apps and packages

  npm run dev
     Develop all apps and packages

Turborepo will cache locally by default. For an additional
speed boost, enable Remote Caching with Vercel by
entering the following command:

  npx turbo login

We suggest that you begin by typing:

  cd my-turborepo
  npx turbo login

npm notice
npm notice New major version of npm available! 8.19.2 -> 9.1.3
npm notice Changelog: https://github.com/npm/cli/releases/tag/v9.1.3
npm notice Run npm install -g<EMAIL_ADDRESS>to update!
npm notice

Maybe try with a more recent version of node :)

Yes, clearly.
However it's still under maintanace, and I'd expect the tooling to at least prompt about unsupported version of node, or best - Support all LTS under maintance node versions.
Best :+1:


npm run watch is not working

when we try on different laptop that is hosted in a local server there is a media issue

Are you trying to run this example with your own PeerJS server?
This project doesn't use NPM, so I'm assuming you're referring to running npm watch within the PeerJS repo.

No, i have cloned your project and used but it it's working.
Thank you for taking tome to respond to my problem
On Mon, Jun 24, 2019 at 12:58 PM Jack McKernan<EMAIL_ADDRESS>wrote:

Are you trying to run this example with your own PeerJS server?
This project doesn't use NPM, so I'm assuming you're referring to running npm
watch within the PeerJS repo.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/jmcker/Peer-to-Peer-Cue-System/issues/8?email_source=notifications&email_token=AFPN3ZIRVZFORKSZUGKJEADP4BWABA5CNFSM4H23K6PKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODYL6UVQ#issuecomment-504883798,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFPN3ZOAN7HZGJL2RHDXRBLP4BWABANCNFSM4H23K6PA
.

Can you post any errors that you're receiving or explain more closely what you're trying to do? I'm not sure I follow yet.

@jmcker ,
I am facing the same issue with Nginx server serving the peer JS demo project. The receiver is able to connect and the status stated "connected" but the sender status is blank and it is unable to send messages.
I am using Google Chrome : Version 77.0.3865.90 (Official Build) (64-bit)
and I do not have any error logs in console.

@harshaparjanya were you able to figure out your issue?


Document Difference between Woodpecker and Drone

For the biggest difference I would mention that,
Woodpecker is Community focused,
while last words on Drone Maintinance Harness does have.
on drone not all files are foss lisenced but are published under Polyform Small Business ...
-> https://www.drone.io/enterprise/opensource/#features
tldr: for drone to scale you have to pay *beside some cases (see FAQ)
tldr: to contribute you have to sign a CLA -> the code is not yours anymore (they can re-license anytime they want)

I'm still using Drone, but I would really like to switch to Woodpecker. What is blocking me is some missing features:

No starlark/jsonnet support, which make complex pipelines quite hard to manage
No org secrets
No autoscaler

The missing autoscaler approach is the biggest issue for me. Personally, I don't need the CI every day, and I also don't need it 24/7 most of the time. Using the autoscaler is pretty cost-effective, as it can automatically spin up as much runner VMs as needed and destroy them afterwards, instead of running them all the time.

tldr: for drone to scale you have to pay

That's not fully true, if you are a small company, non-profit org, open source developer, etc. (see FAQ. Still, it's not fully open source, but at least the source code is publically available and can be reviewed.
However, keep up the good work on Woodpecker :heart:

https://mastodon.technology/@WoodpeckerCI/107009887956204103


Fix release ZIP files

@freekmurze Please re-release 1.0.0
Fixes #17

Almost-empty distignore file mislead rsync: https://github.com/10up/action-wordpress-plugin-deploy/blob/b3f8b3c0d73bf0af43aea9a0c2cb398d12d46b25/entrypoint.sh#L61-L65

Almost-empty distignore file mislead rsync: https://github.com/10up/action-wordpress-plugin-deploy/blob/b3f8b3c0d73bf0af43aea9a0c2cb398d12d46b25/entrypoint.sh#L61-L65

Thanks!

Thanks!

Half-sized ZIP 🚀

Half-sized ZIP 🚀


Bundled dependancies - module initialization error - no read access

I'm having an issue with certain dependancies not having read access and that causes errors on deployed lambda.
Error:
module initialization error: Error
at Error (native)
at Object.fs.openSync (fs.js:641:18)
at Object.fs.readFileSync (fs.js:509:33)
at Object.Module._extensions..js (module.js:578:20)
at Module.load (module.js:487:32)
at tryModuleLoad (module.js:446:12)
at Function.Module._load (module.js:438:3)
at Module.require (module.js:497:17)
at require (internal/module.js:20:19)
at Object.<anonymous> (/var/task/node_modules/rest-facade/src/index.js:2:11)

locally when installing with npm the rest-facade module has the correct permissions for all files in /src when its packaged with the plugin it doesn't:
packaged /node_modules/rest-facade/src/:
13799175 -rw-------  1 amitay  admin  9993 Dec 31  1979 Client.js
13799176 -rw-------  1 amitay  admin   207 Dec 31  1979 defaultOptions.js
13799177 drwxr-xr-x  2 amitay  admin   170 Aug 16 11:45 exceptions
13799181 -rw-r--r--  1 amitay  admin   154 Dec 31  1979 index.js
13799182 -rw-------  1 amitay  admin    38 Dec 31  1979 mocha.opts
13799183 -rw-------  1 amitay  admin   389 Dec 31  1979 utils.js

project /node_modules/rest-facade/src/:
12991081 -rw-r--r--  1 amitay  admin  9993 Oct 19  2016 Client.js
12991079 -rw-r--r--  1 amitay  admin   207 Oct 19  2016 defaultOptions.js
12980542 drwxr-xr-x  2 amitay  admin   170 Aug 13 17:44 exceptions
12991080 -rw-r--r--  1 amitay  admin   154 Mar 28 18:30 index.js
12991084 -rw-r--r--  1 amitay  admin    38 Oct 19  2016 mocha.opts
12991083 -rw-r--r--  1 amitay  admin   389 Oct 19  2016 utils.js

using:
webpack: ^3.5.4
serverless-webpack: ^3.0.0-rc.1
serverless: 1.19.0
webpack.config.js:
const path = require('path');
const slsw = require('serverless-webpack');
const nodeExternals = require('webpack-node-externals');

module.exports = {
    entry: slsw.lib.entries,
    target: 'node',
    externals: [nodeExternals({
        modulesFromFile: true
    })],
    module: {
        rules: [
            {
                test: /\.jql$/,
                exclude: /node_modules/,
                use: ['jql-loader', 'babel-loader']
            },
            {
                test: /\.jsx?$/,
                exclude: /node_modules/,
                use: ['shebang-loader', 'babel-loader']
            },
            {
                test: /\.json$/,
                use: ['json-loader']
            }
        ]
    },
    output: {
        libraryTarget: 'commonjs',
        path: path.join(__dirname, '.webpack'),
        filename: '[name].js'
    }
};

serverless.yml:
plugins:
  - serverless-webpack
  - serverless-dynamodb-local
  - serverless-s3-local
  - serverless-offline-scheduler
  - serverless-offline
  - serverless-mocha-plugin
  - api-docs // custom plugin

package:
  include:
    - node_modules/**
    - yarn.lock
  exclude:
    - .git
    - .requirements
    - .ses-mailbox
    - apidoc.json
    - apidoc_shared.js
    - docs
    - cfn_lambda_helpers
    - seed
    - test
  individually: true

Seems to be an issue with the current archiver configuration. It might use the default UMASK to package files which is wrong. @amitaymolko Would it be sufficient to use 644 as general approach?

Yes, I think 644 would be perfect.

This seems to have been an issue with npm itself.
This was fixed in v3.0.4 https://github.com/npm/npm/commit/80c33cf5e6ef207450949764de41ea96538c636e

Thanks for finding the root cause. 👍

The reason for that is, that Serverless (the framework itself) optimizes uploads by checking their hash. So if a file is recreated by webpack, it would generate a ZIP with a different hash, even if the compiled file did not change. So the dates are set to "0" or unset, so that the ZIP hash is only dependent on the real content.


Connection leak issue while using Redis-Redisson

We have a Redis cluster(running in Sentinel mode) with 3 nodes and we are using Redisson (version 3.10.7) to interact with Redis. While checking for some other issue, we noticed a high number of client being connected to server.
On further investigation, we noticed the below scenarios.
+-------------+-----------------------+
|Client         | Connections         |
+-------------------------------------+
|Client A at T0 |   33                |
|               |(default by Reddison)|
|Client A at T1 |99                   |
|               |(post network glitch |
|               |or server crash)     |
|Client A at T2 | 33                  |
|               |  (post restart app) |
|               |                     |
+-------------------------------------+

In case app restart is not done as well, connection remains open even though we have a "timeout" set at server level. Is Redisson leaving the older connection open  and creates a new one in case of redis server crash or network glitch?
Also, I noticed that with each client connection to Redis server,Redis is opening 4 file descriptor. Is this normal?

Fixed in https://github.com/redisson/redisson/issues/2672

I tried taking the master branch code and I am still able to replicate the issue.
Just to let you know, we are using Redis with Sentinel.
Can we reopen this issue?

Is Redisson leaving the older connection open and creates a new one in case of redis server crash or network glitch?

Broken connections are reconnected by ConnectionWatchdog. What are TCP status of all these connections?

For all connection, TCP status is ESTABLISHED.

Does this happen after Redis failover?

we are using below command to see,
lsof -n | grep redis | grep 'my ip'
Connection are keep on increasing while reconnect to redis.
Connection are getting broken very frequency and we are seeing connection pool is getting created again and again.

Could you provide steps to reproduce the problem?
Did you try to set pingConnectionInterval setting https://github.com/redisson/redisson/wiki/2.-Configuration#pingconnectioninterval ?

I did try setting pingConnectionInterval to 1000 but didn't help.
This is happening in longevity run and connections are not getting closed and have in established.
This could due to sentinal down and in that case, connection pool gets recreated and few of the connection become as leak which get slowly increased.

Could you describe steps to reproduce the problem?

Test the application for a longer time around 24 hours continuously
Break in network intermittently between redis and application and reconnect within few milliseconds
The previous connections still have tcp established and redisson creates new pool of connection
In 24 hours it increased to 10 times without releasing connection.

Break in network intermittently between redis and application and reconnect within few milliseconds

You mean to break connection between all Redis nodes and application?

Do you see any errors in logs? like NullPointerException ?

Do you use ssl connection to Redis?

We're experiencing this with SSL connections to Redis.

@kazetsukaimiko
Which Redisson version do you use?
Could you attach heap dump?

3.12.2.
Cannot attach heap dump, too much red tape there as this is in our production environment.
How it manifests: We use Azure Redis, which is a Tiered service. There are limits to the number of connections based on tier. Redission eventually bombs out saying that it is unable to fill the connection pool with the minimum number of connections because the Azure service stops accepting additional connections (1000 is our limit).
When we go to check connections we see around 800 ESTABLISHED TCP connections on the offending node, and upwards of 5000 SYN_SENT typically. Between the "bad node" and all of its replicas, we have close to if not 1000 connections total.
I believe we are experiencing the same kind of "millisecond outages" described above.

Could you try to reproduce it with version attached?
redisson-3.13.2-SNAPSHOT.jar.zip

@mrniko I really appreciate the quick turnaround. This issue takes time (a few days) to manifest, let me see if I can put together a testing plan.
Do you have a diff for this build somewhere I can peek through as well?

Here is the fix https://github.com/redisson/redisson/commit/b707fe8e297bcc9dbf36e83684e667ecc387769b

!conectionPromise.isDone() check looks like a potential NPE? There are constructors where that field is not set. Seems like you should do the same null check as you do with command just above.
We are setting up our test case today.

We're going to have to wait for this fix to trickle down to a release and roll it out then.

conectionPromise can't be null

Fixed in https://github.com/redisson/redisson/issues/2845


Add ID to change that is disseminated over gossip

This will make traceability of how changes originate, how many hops they make, and when the last time they are applied, and who the last node was that applied it.

lgtm.


Use the atom-text-editor tag instead of the editor class.

In keymaps\highlight-line.cson: Use the atom-text-editor tag instead of the editor class.

Duplicate of #43


Update windows interceptor list

Summary of the Pull Request
What is this about?
Closes #3515
Info on Pull Request
What does this include?

Updates our default coverage allowlist list extension for windows to include a new llvm path

Validation Steps Performed
How does someone test & validate?

Manually tested this list against an example customer's binary and verified that the paths expected to be filtered are not present.

Codecov Report

Merging #3528 (07aa044) into main (e3c4a40) will decrease coverage by 8.44%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #3528      +/-   ##
==========================================
- Coverage   39.40%   30.96%   -8.44%
==========================================
  Files         300      135     -165
  Lines       36381    14603   -21778
==========================================
- Hits        14336     4522    -9814
+ Misses      22045    10081   -11964

see 165 files with indirect coverage changes


IML cannot detect filesystem when a client is mounted on a storage server

If a filesystem has a client mounted on a storage server, IML is unable to deploy an agent to that node.
IML is expecting a fs to be known when it knows about a client in this area (at least):
[2020-06-25 13:56:56,646: ERROR/job_scheduler] Job 16: exception in dependency check: Traceback (most recent call last):

  File "/usr/share/chroma-manager/chroma_core/services/job_scheduler/job_scheduler.py", line 549, in _check_jobs
    deps_satisfied = job._deps_satisfied(dep_cache)

  File "/usr/share/chroma-manager/chroma_core/models/jobs.py", line 439, in _deps_satisfied
    result = self.all_deps(dep_cache).satisfied()

  File "/usr/share/chroma-manager/chroma_core/models/jobs.py", line 420, in all_deps
    for dependent_dependency in dep_cache.get(dependent).all():

  File "/usr/share/chroma-manager/chroma_core/services/job_scheduler/dep_cache.py", line 39, in get
    self.cache[key] = self._get(obj, state)

  File "/usr/share/chroma-manager/chroma_core/services/job_scheduler/dep_cache.py", line 18, in _get
    return obj.get_deps(state)

  File "/usr/share/chroma-manager/chroma_core/models/client_mount.py", line 51, in get_deps
    fs = ObjectCache.get_one(ManagedFilesystem, lambda mf: mf.name == self.filesystem)

  File "/usr/share/chroma-manager/chroma_core/lib/cache.py", line 119, in get_one
    raise klass.DoesNotExist

DoesNotExist

Not only detecting a fs but also adding a new server to that node.


docs: add info that only older NRGkicks are supported for the moment

The list of supported charging cables has been updated to include only older NRGkick charging cables.
Currently, DiniTech GmbH refuses to publish the specification of the current communication protocol. According to discussions with NRGkick Support, it is also not guaranteed that a release is planned.
Further info::

https://github.com/evcc-io/evcc/pull/10558
https://github.com/evcc-io/evcc/discussions/4048

AUTO-GENERATED FROM TEMPLATE - PLEASE EDIT HERE https://github.com/evcc-io/evcc/tree/master/templates/definition/charger


feature/#55 - SQLite local database, and refactoring with StatefulWidget's

New files:

barcode_product_query.dart: API query / product by barcode (used to be in full_products_database.dart)
local_database.dart: local SQLite database with one table (product)

Deleted files:

choose_page_model.dart: moved the code to StatefulWidget ChoosePage
full_product_database.dart: database code was deprecated; API fields is now in ProductQuery

Impacted files:

alternative_continuous_scan_page.dart: now a StatefulWidget; refactored with ContinuousScanPage
build.gradle: unrelated - lowered the Android minSdkVersion from 24 to 19, in order to run the app on my old smartphone ;)
choose_page.dart: refactoring as changed to StatefulWidget
continuous_scan_model.dart: not a ChangeNotifier anymore - should be improved when we use product list
continuous_scan_page.dart: now a StatefulWidget
group_product_query.dart: minor refactoring due to the changes in product_query.dart
keywords_product_query.dart: minor refactoring due to the changes in product_query.dart
main.dart: added LocalDatabase to the providers; simplified the code; removed mentions to SharedPreferences
product_query.dart: refactored a bit; moved here some fields from full_product_database.dart
product_query_model.dart: now using LocalDatabase
product_query_page.dart: now using LocalDatabase; refactoring
pubspec.yaml: replaced sembast with sqflite`

@PrimaelQuemerais Working again on the subject, I think the things are clearer now.
Tell me what you think of ContinousScanModel: the idea was to remove everything related to Widgets and CarouselController, and to leave it managing product metadata only. The model is more simple. The StatefulWidgets are more simple too.
For the moment the notifyListeners method is a dummy method created in LocalDatabase. The solution would be either to create a dedicated ChangeNotifier, something like ScanNotifier, or to store the scan list data in the database with the appropriate notifications.

@PrimaelQuemerais Working again on the subject, I think the things are clearer now.
Tell me what you think of ContinousScanModel: the idea was to remove everything related to Widgets and CarouselController, and to leave it managing product metadata only. The model is more simple. The StatefulWidgets are more simple too.
For the moment the notifyListeners method is a dummy method created in LocalDatabase. The solution would be either to create a dedicated ChangeNotifier, something like ScanNotifier, or to store the scan list data in the database with the appropriate notifications.

Sorry for the delay, I will review this after my exams this week.

Sorry for the delay, I will review this after my exams this week.

@PrimaelQuemerais Actually I have things to fix, and yes, I'll probably be using providers. The fact is that the keywords query currently fails (as already mentioned - double/null), and this possibility was not taken into account in the page.
Give me one day or two to clean the page.

@PrimaelQuemerais Actually I have things to fix, and yes, I'll probably be using providers. The fact is that the keywords query currently fails (as already mentioned - double/null), and this possibility was not taken into account in the page.
Give me one day or two to clean the page.

@PrimaelQuemerais Now the part about ProductQueryModel looks good to me. I hope you'll find it good too ;)

@PrimaelQuemerais Now the part about ProductQueryModel looks good to me. I hope you'll find it good too ;)

I've started to look at it. I've made the necessary changes for the code to work with the new version of the openfoodfacts-dart plugin.
I need a little bit more time to look at everything, will let you know :slightly_smiling_face:

I've started to look at it. I've made the necessary changes for the code to work with the new version of the openfoodfacts-dart plugin.
I need a little bit more time to look at everything, will let you know :slightly_smiling_face:

@PrimaelQuemerais I wouldn't have tried to put the new openfoodfacts: ^0.3.14+1 in the same move. But I'm not a git expert.

@PrimaelQuemerais I wouldn't have tried to put the new openfoodfacts: ^0.3.14+1 in the same move. But I'm not a git expert.

@monsieurtanuki I somehow did it in the wrong order but the Master was also updated to use v0.3.14+1 (I added it to #65 which updates the dependencies).
But true it should've been done by merging Master into this branch, saw you did that thanks.
I'm facing quite a few display issues, I'll check all the database related code today so we can merge and then fix these issues.

@monsieurtanuki I somehow did it in the wrong order but the Master was also updated to use v0.3.14+1 (I added it to #65 which updates the dependencies).
But true it should've been done by merging Master into this branch, saw you did that thanks.
I'm facing quite a few display issues, I'll check all the database related code today so we can merge and then fix these issues.

@PrimaelQuemerais I've just added a minor merge fix. I'm done.

@PrimaelQuemerais I've just added a minor merge fix. I'm done.

I went through everything, I really like the implementation of the database it is really neat.
Regarding the state management context.watch<LocalDatabase> is working nicely, if I'm not mistaking we could set the ContinuousScanPage to a StatelessWidget or am I omitting something?

I went through everything, I really like the implementation of the database it is really neat.
Regarding the state management context.watch<LocalDatabase> is working nicely, if I'm not mistaking we could set the ContinuousScanPage to a StatelessWidget or am I omitting something?

@PrimaelQuemerais I'm glad you liked the code!
Regarding ContinuousScanPage being a StatelessWidget, maybe you're right, try that. That would involve putting _continuousScanModel = ContinuousScanModel(contributionMode: widget.initializeWithContributionMode) in the constructor, but I believe you're rather supposed to use const fields; with StatefulWidget there's more flexibility.
But I think there's a problem with StatelessWidget: the whole widget would be rebuilt (triggered by the provider notification) and you would then create a new model from scratch. With the initState solution, you create the model just once and reuse it in the next build(context) calls.
LocalDatabase.dummyNotifyListeners() was just an initial suggestion, a dedicated ScanNotifier would make more sense.
That being said, if you think this PR is an improvement, I would suggest you to approve it and to merge it quickly, as it impacts many files and I would like to avoid the same merge conflicts again and again.
And there is probably refactoring needed regarding the providers, at least for coherence reasons, but that could be a new issue / a new PR.

@PrimaelQuemerais I'm glad you liked the code!
Regarding ContinuousScanPage being a StatelessWidget, maybe you're right, try that. That would involve putting _continuousScanModel = ContinuousScanModel(contributionMode: widget.initializeWithContributionMode) in the constructor, but I believe you're rather supposed to use const fields; with StatefulWidget there's more flexibility.
But I think there's a problem with StatelessWidget: the whole widget would be rebuilt (triggered by the provider notification) and you would then create a new model from scratch. With the initState solution, you create the model just once and reuse it in the next build(context) calls.
LocalDatabase.dummyNotifyListeners() was just an initial suggestion, a dedicated ScanNotifier would make more sense.
That being said, if you think this PR is an improvement, I would suggest you to approve it and to merge it quickly, as it impacts many files and I would like to avoid the same merge conflicts again and again.
And there is probably refactoring needed regarding the providers, at least for coherence reasons, but that could be a new issue / a new PR.

@monsieurtanuki I think you're right we would not gain anything from making the widget stateless. It is now merged into master

@monsieurtanuki I think you're right we would not gain anything from making the widget stateless. It is now merged into master

Thank you very much for all the great progress @monsieurtanuki and @PrimaelQuemerais !

Thank you very much for all the great progress @monsieurtanuki and @PrimaelQuemerais !

@stephanegigandet You're welcome! Currently working on the match / ranking / color features.

@stephanegigandet You're welcome! Currently working on the match / ranking / color features.


Feature #000 userOrgService: Jenkins File with no test cases and build and deployemnt chnages for user org service

@indrajra  @manzarul  please review and merge it


[BigInt] Reorganize into extensions by conformance

I removed isSigned since it's implemented by the SignedInteger protocol.
-  public static var isSigned: Bool { true }

I've split this into smaller commits for review.


Make it clear that "n-m" stands for many-to-many relationship

It wasn't clear to me when I first read it.

@stansw : Thanks for your contribution to the Azure documentation! The author, @sharonlo101, has been notified to review your proposed change.

#sign-off