[Feature Request] Add class to app element when modal open

We would like to lock the app from scrolling when the modal is open. Currently we have a wrapper around Modal to set the class on our app element based on the isOpen prop.

Ran into the same problem. Will work on a patch for this Monday.

@astephenb created PR if you'd like to test.

@KyleAMathews Did this get merged?

Nope. This is the PR https://github.com/rackt/react-modal/pull/41

@KyleAMathews I haven't seen any movement on this project in about 2 months. They must be really busy with the launch of the React training course.

@bartimaeus oh hey Eric! :)
Yeah, something like that. Though @mzabriskie isn't working with Michael and Ryan.
@mzabriskie any chance you can add more maintainers here?

Sorry for the delay. Spent some time giving this project some much anticipated attention. Will be cutting a new release soon.

Thanks!
On Tue, Jul 14, 2015 at 11:07 PM Matt Zabriskie<EMAIL_ADDRESS>wrote:

Sorry for the delay. Spent some time giving this project some much
anticipated attention. Will be cutting a new release soon.
—
Reply to this email directly or view it on GitHub
https://github.com/rackt/react-modal/issues/29#issuecomment-121501452.


Allow users to use a specific version of npm

The current set of buildpacks installs a version of npm based on the chosen version of node. Sometimes users will want to choose a different version of npm than the version that came with the node version they've chosen. We should create a mechanism for users to specify a version of npm to install.

@ryanmoran or @trevorblades, is this still a relevant feature request for you?

Yes. I would like to see this happen.

@paketo-buildpacks/nodejs-maintainers Any blockers or things to keep in mind when implementing this feature?

@ryanmoran What would be the reason for choosing a different version of npm? A more current one?

@c0d1ngm0nk3y yes. Or to be able to roll back to a different version if a bug is discovered in the version that ships with Node.js.

Is this feature still under consideration? There are reasons to use a different npm version than the default one, such as bugs or testing new features

What about if the npm-install buildpack receives a BP_NPM_VERSION which results in npm install npm@<version> being installing the actual node-modules?


Dependabot, images, README

Fixes dependabot hint
Images were missing, I fixed it
Updates README with a few notes and hint

I had to resolve conflicts after the #62 merge - another review, sorry @siegfriedweber


atlas networking containers list throws error

Describe the bug
Attempted to execute atlas networking containers list after auth login.
I get the following error back
Error: template: output:2:31: executing "output" at <.RegionName>: can't evaluate field RegionName in type interface {}
Desktop (please complete the following information):
atlas --version output
atlascli version: 1.9.1
git version: 730ed8bebeaa69592c31ef47539795ac4a18291f
Go version: go1.19.10
   os: windows
   arch: amd64
   compiler: gc

Additional context
First time trying the atlas cli, I just wanted to find out what network containers existed.
In my case, there were no containers when I queried the http api myself without specifying a provider.

Thank you for reporting issue. We are working for the fix.
As workaround we recomend to use -o JSON to get output of the command in JSON format

We have made a fix for the issue. It will be included in the next release of the atlas CLI.

The issue has been fixed in the latest release.


Proposal: Increase scope of committership

Recently, the ESLint Reviewers concluded that it would be good to give ESLint Committers more authority in the project. This pull request proposes:

Committers should be allowed to merge some pull requests (bug fixes, docs, build-related, upgrades) on their own so long as they are non-breaking.
Committers should be allowed to label an issue as "accepted" if it is a bug fix, a new rule, or a non-breaking change to an existing rule. I think this change is good because it didn't make sense to allow Committers to accept new rules but not changes to existing rules.

Please let me know your thoughts. @eslint/eslint-team

:+1:

While I have absolutely no problem with this change, my question is what's the difference between committer and reviewer at this point? Is it that reviewers can merge any pull request, and committers can only merge non-breaking?

Is it that reviewers can merge any pull request, and committers can only merge non-breaking?

Not quite, but the primary difference is that committers can merge some pull requests now.

Kind of agree with @ilyavolodin. :+1: for accepting things, :-1: for merging.

@BYK why :-1: for merging?

:+1:

Is the intention to limit merges to trivial PRs?

@nzakas that blurs the line between committers and reviewers a bit too much if you ask me. As a committer, if I can do merges too what separates me from the people who has spent much more time on the project? How about "they can merge non-breaking changes at least one reviewer says 'LGTM'`?

@alberto yes.
@byk I explicitly named what type of PRs committers can merge both in the original description of this PR and in the changes files. There is just a small subset that committers can merge on their own. Please read those over.
The overall goal here is to remove bottlenecks by increasing the number of things committers can do.

@nzakas - I already did :) I think it would confuse me since now I have to keep in mind which patches I can merge and probably err on the safe side and not merge anything at all.

@BYK The way I understand it, it would be only for trivial PRs consisting of a few lines of code, in order to get PRs merged faster and alleviate some of the work of the reviewers to allow them to focus on the non trivial ones. I'm :+1: for that

@byk you wouldn't have to merge anything, it's optional. If you don't want to, you don't have to.

Alright, if I'm the only one concerned I'll yield :)

@btmills @IanVS @lo1tuma @xjamundx thoughts?

I'm a little nervous with so many people being able to accept new rules.  It seems to me that this should be a responsibility of the project leader(s) (after getting input from the team if desired).

@IanVS that hasn't changed. Committers have always been able to accept new rules, the only change here is that you'll now also be able to accept modifications to the rules.

Ah okay, I didn't look closely enough. Sounds good to me.
On Mon, Mar 7, 2016, 16:08 Nicholas C. Zakas<EMAIL_ADDRESS>wrote:

@IanVS https://github.com/IanVS that hasn't changed. Committers have
always been able to accept new rules, the only change here is that you'll
now also be able to accept modifications to the rules.
—
Reply to this email directly or view it on GitHub
https://github.com/eslint/eslint.github.io/pull/210#issuecomment-193449573
.

Great I didn't realize we could even label issues. 0 concerns from me.

No objections here!

By the way, just in case it's not clear, this is about adding capabilities for committers. You are not required to do anything you're not comfortable with, so of you are unsure of accepting something, or you want someone else to review a PR because you don't feel comfortable doing it, that's OK! Just leave a comment asking for someone else to help (which I think we all basically do anyway).

OK, I think we've reached consensus here, so merging.


Customer View Details does not correctly show the detail information

Hi;
When I run the application and click detail on customer, the program always show "Kelly She" information doesn't matter the customer I clicked. Please fixed it.
Thanks

Hi,
I have a workaround for the bug. I do not know the exact cause of the problem but the following was my workaround and it worked:
In the file: CustomerDetailPage.xaml.cs
In the function: OnNavigatedTo
After the if else statement:
Add:
ViewModel.AddNewCustomerCanceled += AddNewCustomerCanceled;
EmailText.Text = string.IsNullOrEmpty(ViewModel.Email) ? "Null" : ViewModel.Email;

Do this for each textbox and parameter and it works.
Another way of doing the same thing would be to add in the above mentioned place:
ViewModel.AddNewCustomerCanceled += AddNewCustomerCanceled;
EmailText.Text = ViewModel.Email;

And also add some code in the CustomerViewModel.cs:
In the public string definitions of the class atributes(for example FirstName or Email):
Change their get functions to:
get => string.IsNullOrEmpty(Model.Email) ? "Null" : Model.Email;
Once again you have to change the line in every attribute.

Sorry for the delayed response to this issue. We've just released a bunch of updates for this sample, including a fix for this bug. The solution is just that the NavigationCacheMode for the CustomerDetail page should be set to Disabled so the customer data isn't cached. See this commit for the fix: https://github.com/microsoft/Windows-appsample-customers-orders-database/commit/423b80bbaadfcda1a0bbd6784b11899ae020f1b7


Fix order of name parts in plant name.

Basics

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

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

Checklist

[ ] I have installed and I am using pre-commit hooks
[ ] I fully described what my PR does in the documentation
[ ] I fixed all affected documentation
[ ] I fixed the introduction tour
[ ] I wrote migrations in a way that they are compatible with already present data
[ ] I fixed all affected decisions
[ ] I added automated tests or a manual test protocol
[ ] I added code comments, logging, and assertions as appropriate
[ ] I translated all strings visible to the user
[ ] I mentioned every code or binary not directly written or done by me in reuse syntax
[ ] I created left-over issues for things that are still to be done
[ ] Code is conforming to our Architecture
[ ] Code is conforming to our Guidelines
[ ] Code is consistent to our Design Decisions
[ ] Exceptions to any guidelines are documented

Review

[ ] I've tested the code
[ ] I've read through the whole code
[ ] I've read through the whole documentation
[ ] I've checked conformity to guidelines

Please add tests (also for seed search etc.), maybe together with @4ydan and clarify which parts of the application are fixed now. Is ExtendedPlantsSummaryDisplayName already used everywhere?

Is ExtendedPlantsSummaryDisplayName already used everywhere?

It is used on the map and in the seed overview.
Are you aware of any other uses of plant names?

In the map there are several places (e.g. plant selection, plant labels, plant hover text).

In the map there are several places (e.g. plant selection, plant labels, plant hover text).

As long as I have not missed anything, it should be used in all these places.


SDK cannot be used with ethers.js v6

context
A user reported this bug in Discord here https://discord.com/channels/592843512312102924/1077277654132215960
initial report
The initial report is from a user running the basic sdk example from the docs:
import { Database, helpers } from "@tableland/sdk"
import { Wallet } from "ethers"
import * as dotenv from "dotenv"
dotenv.config()

async function main() {
    const privateKey: any = process.env.PRIVATE_KEY
    const rpc: any = process.env.RPC_URL

    const wallet = new Wallet(privateKey);
    const provider: any = helpers.getDefaultProvider(rpc)
    const signer: any = wallet.connect(provider);
    interface Schema {
        id: number;
        name: string;
    }

    const db = new Database<Schema>({ signer: signer })

    const prefix: string = "my_sdk_table";

    const { meta: create } = await db
        .prepare(`CREATE TABLE ${prefix} (id integer primary key, name text);`)
        .run();

    console.log(create.txn?.name)
}

main();

This results in: Error: RUN_ERROR: signer.getChainId is not a function
details
Ethers.js v6 was recently released https://docs.ethers.org/v6
With this release the signer interface no longer has the getChainId method.  The chainId is potentially available elsewhere within the ethers v6 signer, but we might want to decide if there's a more robust way to get the chainId.
The v3 SDK had the requirement that a specific network had been connected to, which made this problem a little easier.
ideas for a solution

We could potentially try to sniff out what kind of interface the passed in signer supports.  The  downside I see here is that this could be cumbersome.
We could determine the chainId from the query.  I think this makes sense, but I'd have to dive into implementation to be sure.

Looks like we will have to wait until https://github.com/ethers-io/ethers.js/issues/3734 is fixed to properly fix this and upgrade this package to Ethers v6


chore(main): release 0.58.10

:robot: I have created a release beep boop
0.58.10 (2023-08-14)
Miscellaneous

remove print (0ea5890)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/BLSQ/openhexa-app/releases/tag/0.58.10 :sunflower:


React 15.0.0-rc1 key warning

React 15.0.0-rc1 add new warning

Elements will now warn when attempting to read ref and key from the props.

recompose seems touch key from props, so this warning show up

key is not a prop. Trying to access it will result in undefined being returned. If you need to access the same value within the child component, you should pass it as a different prop.

Thanks for the bug report. This can be fixed by using hasOwnProperty here: https://github.com/acdlite/recompose/blob/ca7c6d33de8afa4e48cee299ba9a16feba4b1d76/src/packages/recompose/createElement.js#L7
I'll fix soon!

Closed by #113. It'll go out in the next release.


Can Android applications initialize without calling sensors and reading Android ID

Yes. They only call those when you use them.


⚠ Bofrost.de has degraded performance

In c7ec2b7, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 5966 ms

Resolved: Bofrost.de performance has improved in 47930ad.


Publish OIS package

Blocked by https://github.com/api3dao/airnode/issues/1209

Just published the npmjs packages. Build the docker images now.

Can you assign milestone, priority, size, and iteration?

Yeah, except for milestone because that is purely Airnode repository thing.

I'm just noting here that I've seen this.

This has now been published:
https://www.npmjs.com/package/@api3/ois
and https://github.com/api3dao/ois/releases/tag/v1.0.0

@Siegrift let me know when this is ready for testing :)

I think there is not much to test here apart from that it was published right, which I've done as part of the review. I think this can be moved to DONE.
I'll need your help to test https://github.com/api3dao/airnode/issues/1274 though :)

I've confirmed the package is published correctly and can be used in a test project.
Closing.


Window resize event handler should be debounced then removed in removeListeners

It looks like this may be an oversight, but the resize hanlders are never removed. It might also be nice to debounce this.

The image above is simplebar in a modal that I was just opening and closing.

You're right. Forgot to remove that listener! Thanks!


Add Message class

The Message constructor should take the message string as a parameter,
and check the message string to be valid as per the relevant RFCs.
The Message objects will contain the irc message,
and parsed prefix, command, and suffix.

Nice work!


fix(package): add typings, add main

Codecov Report

Merging #51 (f48b907) into master (e90b157) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##            master       #51   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           19        19
  Lines          157       157
  Branches        35        35
=========================================
  Hits           157       157

Continue to review full report at Codecov.

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


Update events

Adding EMBL-EBI and ELMI workshops.
cc @jburel

Do you also want to add the upcoming one in Panama?

Do you also want to add the upcoming one in Panama?

Added in https://github.com/ome/www.openmicroscopy.org/pull/712/commits/81ce784c0928a91a71f4a2cefc2c122679fa83bd

built at https://snoopycrimecop.github.io/www.openmicroscopy.org/events/

Happy for me to merge ?


[5.9] Refactor: Move References to Stores

Explanation: Refactor: Move References to Stores
Scope: All pages/components using references
Issue: rdar://107635892
Risk: Medium
Testing:

All unit tests should still pass.
Make sure this change doesn’t introduce regressions for all pages

Reviewer: @mportiz08 @dobromir-hristov @marinaaisa
Original PR: https://github.com/apple/swift-docc-render/pull/594

@swift-ci test


⚠ TiendaEnLinea has degraded performance

In 8e39c25, TiendaEnLinea (https://www.fahorro.com/) experienced degraded performance:

HTTP code: 200
Response time: 10130 ms

Resolved: TiendaEnLinea performance has improved in e8692b1.


Add a private message system

Is your feature request related to a problem? Please describe.
While it's great to have communicaation over Telegram or Discord, a user suggested it would be cool to be able to send private messages on the platform. YouTube had it for some time, but then they disabled it.

Jasmin suggested on the regional meeting that this could be a great feature for memberships:
The channel owner could either allow DMs from anyone, just followers, just tier 1 (tier 2,3 and so on) members and also embedding some kind of tipping system into the DMs, so we could have special personal services like onlyfans going on, just without the mature content.


Convert playground to use Monaco Editor

Currently we use https://ace.c9.io/ however it is proving to be very difficult to get some features working, notably context menus and error highlighting. An alternative is https://microsoft.github.io/monaco-editor/index.html which is the editor used by VSCode. This has the added advantage that it might be easier to transfer any Marlowe logic across to a VSCode extension.

This is done :)

Lol, this was done 2 years ago I think 😄

Good times

💯


Server connection failure message in grpc node should be explicit

If  the grpc node is given a wrong server address, should it fail with an explicit and more descriptive error message (for example: Could not connect to server  xx.xx.xx.xx:yy) instead of complaining about not being able to fetch the submap list?
 Seems gRPC has a way to explicitly let the channel connect and then wait for a state change. This should work to ensure the channel is connected before proceeding.
https://github.com/grpc/grpc/blob/master/doc/connectivity-semantics-and-api.md


storage.rs:59-62: Implement #xml function in Storage....

The puzzle 17-1a69e44f from #17 has to be resolved:
https://github.com/h1alexbel/fakehub/blob/31636e6ff542ff1317c28ae7bbac232572a06a30/server/src/xml/storage.rs#L59-L62
The puzzle was created by @h1alexbel on 11-Jul-24.
Estimate: 35 minutes,  role: DEV.
If you have any technical questions, don't ask me, submit new tickets instead. The task will be "done" when the problem is fixed and the text of the puzzle is removed from the source code. Here is more about PDD and about me.

@0pdd thanks for the report, here is a feedback: * The todo message mentions implementing the #xml...

I see you're talking to me, but I can't reply since I'm not a chat bot.

@0pdd 2 puzzles #98, #99 are still not solved.

@0pdd all 2 puzzles are solved here: #98, #99.


Updated the readme file

Added the program section.
Please merge it @visheshrwl

Hey @Aman-G-upta ! 👋
First off, I want to extend a huge thank you to all of you for your amazing contributions to this project! Your support and hard work mean a lot, and I truly appreciate everything you do. 🙏
I have a small favor to ask: if you think I’ve been helpful and supportive throughout your contributions, I would be honored if you could nominate me by filling out this form: Nominate me.
Your nomination would mean the world to me, and it’s a great way to help me continue supporting our awesome community. Thanks again for all the amazing work, and let's keep making great things happen! 🚀
Best,
Vishesh


Push IQ# install page updates to live

Merging a few commits from myself and @geduardo to live. Mainly contains updates to the IQ# installation page.

Docs Build status updates of commit 8a0600c:
:clock10: Pending: waiting for processors (6 builds ahead of you)
:warning: Docs Build is busy, currently there are 6 builds ahead of this one, for more information you can view the Build queue graph on the Docs Portal.

Docs Build status updates of commit 8a0600c:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

articles/install-guide/pyinstall.md
:white_check_mark:Succeeded
View

articles/install-guide/qjupyter.md
:white_check_mark:Succeeded
View

articles/intro-to-katas.md
:white_check_mark:Succeeded
View

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.


[BUGFIX] Throw exception for requests in broken test environment

1. Why is this change necessary?
In case the testing environment is broken (e.g. missing JWT secrets, invalid .env.test file), the request done in Shopware\Core\Framework\Test\TestCaseBase\SalesChannelApiTestBehaviour::assignSalesChannelContext() returns an error, but the code expects a token, leading to following issue:
ErrorException: Notice: Undefined index: token

vendor/shopware/core/Framework/Test/TestCaseBase/SalesChannelApiTestBehaviour.php:289

This hides the real issue and makes debugging hard.
2. What does this change do, exactly?
The response is now checked for an error item. If such an item exists, an exception is thrown containing the message of the very first error.
3. Describe each step to reproduce the issue or behaviour.
After upgrading Shopware from 6.3 to 6.4, I missed MAILER_DSN in my .env.test file. The following shortened code revealed the issue:
class RegisterRouteDecoratorTest extends TestCase
{
    use IntegrationTestBehaviour;
    use SalesChannelApiTestBehaviour;
    use CountryAddToSalesChannelTestBehaviour;

    protected function setUp(): void
    {
        $this->ids = new TestDataCollection(Context::createDefaultContext());
        $this->browser = $this->createCustomSalesChannelBrowser([
            'id' => $this->ids->create('sales-channel'),
        ]);
    }

    public function testRegistrationOfUser(): void
    {
        $this->browser->request(
            'POST',
            '/store-api/account/register',
            [],
            [],
            ['CONTENT_TYPE' => 'application/json'],
            json_encode($this->getRegistrationData(), JSON_THROW_ON_ERROR)
        );
        $response = json_decode($this->browser->getResponse()->getContent(), true, 512, JSON_THROW_ON_ERROR);
    }
}

4. Please link to the relevant issues (if any).
/
5. Checklist

[ ] I have written tests and verified that they fail without my change
[X] I have squashed any insignificant commits
[X] I have created a changelog file with all necessary information about my changes
[ ] I have written or adjusted the documentation according to my changes
[ ] This change has comments for package types, values, functions, and non-obvious lines of code
[X] I have read the contribution requirements and fulfil them.

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

If there is any way to cover this test internal with tests, please let me know.

Cypress fails due to
Stderr:
In Kernel.php line 415:

  The environment "grep=@base" contains invalid characters, it can only contain characte...

This looks unrelated to my change.

Hello,
thank you for creating this pull request.
I have opened an issue on our Issue Tracker for you. See the issue link: https://issues.shopware.com/issues/NEXT-18917
Please use this issue to track the state of your pull request.

Hey @andreasfernandez  thanks for your contribution! :blue_heart: :tada:


etlegacy: fix .desktop files

Fix an issue introduced in #296713
Description of changes

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

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

Yeah, thanks for letting me know the issue. I updated my laptop just now and noticed that etlegacy doesn't compile any more on it, so, mission succeeded !

Nice work!


Added YAML files for quickstart #4: Configuring granular permissions by service

This is a new quickstart for User Access (https://issues.redhat.com/browse/HCCDOC-334), which originally was going to go on the Learning resources page for Settings, but now needs to go into the IAM Learning resources (once created) as users will configure User Access settings from there.
RHCLOUD JIra: https://issues.redhat.com/browse/RHCLOUD-24721
@Hyperkid123 or @ryelo - can you please review and merge/let me know if anything needs fixing? Thank you!

/ok-to-test


⚠ RbxVeri has degraded performance

In 59fe523, RbxVeri (https://verify.triza.dev) experienced degraded performance:

HTTP code: 200
Response time: 1659 ms

Resolved: RbxVeri performance has improved in f50d282.


tab complete the init script

could you change the aptly-api to be symlinked to init.d so that it can be tab completed?

This is something we'd rather not have in the core module. The service command will correctly tab complete files in /etc/init.d/ and /etc/init and with the forthcoming Ubuntu move to systemd we'd rather keep this as simple as possible.
I'd suggest doing it in your own module.


Hide copy button if flash isn't installed

Since it is broken in that case.

Going to close this since we are planning on moving to a solution that doesn't involve flash at all.


Validate metadata file

If metadata.json is present in data/ directory, it must be a valid JSON or JSON-LD file. See https://github.com/RockefellerArchiveCenter/project_electron/blob/master/transfer/bagit-specification.md#metadatajson

Still needs to be done.


Fix update route in SQL Authenticator

Codecov Report
Base: 82.52% // Head: 81.62% // Decreases project coverage by -0.89% :warning:

Coverage data is based on head (95691aa) compared to base (9399645).
Patch coverage: 40.00% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #568      +/-   ##
==========================================
- Coverage   82.52%   81.62%   -0.90%
==========================================
  Files          78       78
  Lines        6059     6068       +9
==========================================
- Hits         5000     4953      -47
- Misses       1059     1115      +56

Impacted Files
Coverage Δ

quetz/testing/fixtures.py
92.35% <40.00%> (-3.30%)
:arrow_down:

quetz/pkgstores.py
42.45% <0.00%> (-11.60%)
:arrow_down:

quetz/tasks/mirror.py
90.49% <0.00%> (+1.05%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


More env var cleanup

closes #4172

todo: add a flag to disable Path.exists validation

overall i like the descriptor implementation

Thanks, i wanted to try binding the descriptors to rx.Config as well in a followup.

i'm not too sure about .get being a property though, to me that kind of looks weird. did you consider having it as a normal method like .set(...)

Yeah i was thinking about making it a normal metehod.. This would also allow us to add f.e. a cache arg to get as well.
If @adhami3310 agrees i will refactor it.

If @adhami3310 agrees i will refactor it.

yea that would be great

Refactor done :heavy_check_mark:


INFO: successfully loaded

When I call the svd function, I get lots of diagnostic messages printed.
Is this normal?
Apr 04, 2021 12:41:17 PM com.github.fommil.jni.JniLoader liberalLoad
INFO: successfully loaded /var/folders/c1/sqxjhjm15gdcyr49z7r4k1mr0000gn/T/jniloader14881482479103767951netlib-native_system-osx-x86_64.jnilib
Apr 04, 2021 12:41:17 PM com.github.fommil.jni.JniLoader load
INFO: already loaded netlib-native_system-osx-x86_64.jnilib

I'm using the version recommended on the breeze GitHub page.
libraryDependencies  ++= Seq(
  // Last stable release
  "org.scalanlp" %% "breeze" % "1.1",

  // Native libraries are not included by default. add this if you want them
  // Native libraries greatly improve performance, but increase jar sizes.
  // It also packages various blas implementations, which have licenses that may or may not
  // be compatible with the Apache License. No GPL code, as best I know.
  "org.scalanlp" %% "breeze-natives" % "1.1",

  // The visualization library is distributed separately as well.
  // It depends on LGPL code
  "org.scalanlp" %% "breeze-viz" % "1.1"
)

Yes, the first time you call any linear algebra function in breeze (or
something that calls one) breeze will attempt to load native blas and
lapack libraries via netlib-java. The logging comes from netlib-java. It's
possible to disable but I don't recommend it.
On Sun, Apr 4, 2021, 3:42 AM Jim Newton @.***> wrote:

When I call the svd function, I get lots of diagnostic messages printed.
Is this normal?
Apr 04, 2021 12:41:17 PM com.github.fommil.jni.JniLoader liberalLoad
INFO: successfully loaded /var/folders/c1/sqxjhjm15gdcyr49z7r4k1mr0000gn/T/jniloader14881482479103767951netlib-native_system-osx-x86_64.jnilib
Apr 04, 2021 12:41:17 PM com.github.fommil.jni.JniLoader load
INFO: already loaded netlib-native_system-osx-x86_64.jnilib
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/scalanlp/breeze/issues/801, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAACLIL3HYZ5FPXU4G3GVJ3THA7CNANCNFSM42LIQUPA
.

Is it normal that I get a lot of such messages?

are you seeing this more than once per session/program?

It seems the full message just appears once.
Perhaps it is a bit misleading as it appears every time I test a function, but I suppose
that is because IntelliJ starts a new JVM session each time I press the little green triangle to run a test.
Apr 08, 2021 11:52:50 AM com.github.fommil.jni.JniLoader liberalLoad
INFO: successfully loaded /var/folders/c1/sqxjhjm15gdcyr49z7r4k1mr0000gn/T/jniloader8766867026396728044netlib-native_system-osx-x86_64.jnilib
Apr 08, 2021 11:52:50 AM com.github.fommil.jni.JniLoader load
INFO: already loaded netlib-native_system-osx-x86_64.jnilib


Improve the documentation of  POD links

The problem
I read nothing about that issue in https://docs.perl6.org/language/pod#Links or in in Rakudo or in the converter module code. So this ticket my be one for Rakudo -I don't know yet.
Suggestions
Naturally i would look to http://perldoc.perl.org/perlpod.html and suggest L < Label | SubDir / Module.pod  # anchor > (defaulting to .pod when no file ending) and as cherry on top L< Label | https://perl6.org # anchor> as well.

Sorry, but can you please clarify? Are you asking for a change in the Pod spec? Pods are parsed by perl6 itself...

I asked how to do a link from one POD file into another. Perl 5 can do it, so there is no reason Perl 6 can not. Either we forgot to document it, or (and myself looking into relevant sources think it is more probable) it is not implemented yet. This will be just an addition, not  a change.

yes, thank you,
than should we probably document this more clearly, because the example there is about an IP address not local file name and I assumed it just works for internet links and not local ones

OK, I'll change the issue to say that if you don't mind.

sure

implemented with 178d2e367586d6623e4c67f6be59767c53f12d3a

implmented with
5ddb34165f6ff786068cb0fc1b58ecc6acb8bea8 fixed:
178d2e367586d6623e4c67f6be59767c53f12d3a


How to Set PostgreSQL Session Variables with @vercel/postgres

I'm currently trying to set a PostgreSQL session variable in next 13.4.3 using the @vercel/postgres module. I'm trying to use the SET command to set a custom session variable before executing other queries, but I'm encountering an issue.
Here is an example of what I'm trying to do:
const {userId} = auth()
console.log("USERID: ",userId)
await sqlSET request.jwt.claim.uid = ${userId};
I'm getting the following error:
error: syntax error at or near "$1"

so got my answer from the neon group, set_config should be used instead as follows:

import {db} from '@vercel/postgres'
const client = await db.connect()
await client.sqlselect set_config('auth.uid',${userId},false)
await client.sql ...another query


Properly handle go mod init failures.

Also, do no attempt to run it if go.mod already exists.
What this PR does / why we need it:
See subject.
Fixes #1051

Is the failed check a flake?

Is the failed check a flake?

yep, that is already fixed on master

@porridge could you bring in the latest master to this branch?


How to connect to docker endpoints via the hosts VPN tap0 interface?

Portainer version: 1.14.0
Docker version 17.05.0-ce, build 89658be
Ubuntu 16.04 LTS
Cannot determine where to ask this, so please redirect me if needed.
I have Portainer setup and working well with the hosts unix socket. Portainer is running in a docker container on the master swarm node.
Between my swarm-nodes I have a mesh VPN with IPs 10.0.0[1-3]  for the three nodes. Docker API is listening on TCP port 2375 on the two non-master nodes. I am trying to connect Portainer to them, however it gets a connection refused. From the master host however I can curl the 10.0.0.[2-3]:2375 url successfully.
Right now I am fairly positive that this is due to the portainer instance running as a container and docker is blocking the communication across the VPN Interface. I suppose an option would be to run the container in net=host mode, but I'm not confident that's the best approach. Is there a way to make portainer just have access to the one interface? Should I just install portainer directly on the masters so they can access the VPN interface? Is there some sort of NAT trickery I should use instead? What is the best approach?

Hi @Routhinator
This is probably a Docker networking issue as the Portainer container do not have access to the VPN network. I'm not sure how did you configure that mesh VPN ?
You can join us on https://portainer.io/slack/ for some live support if you prefer.

Right-o, slack seems a better place for this. I would also agree, however was presuming I missed something in this due to my docker experience being only about 6 months.


Use @Configuration(proxyBeanMethods=false) wherever possible?

What do you think about using ´@Configuration(proxyBeanMethods=false)´ instead of @Configuration to avoid the (potentially expensive) generation of cglib proxies?
See: https://docs.spring.io/spring-framework/docs/current/javadoc-api/org/springframework/context/annotation/Configuration.html#proxyBeanMethods-- and: https://github.com/spring-projects/spring-boot/issues/9068 for details.
By the way: Thank you very much for providing mq-jms-spring. 👍

I'll look to make that change next time I do some updates.

Thank you very much! Looking forward to it.

Done in 2.2,5

Thank you very much! 👍


Monocular unable to fetch charts from repo

I've created an S3 bucket for https://github.com/bitnami/charts which is accessible at https://s3.amazonaws.com/charts.bitnami.com/incubator.
I am able to download charts from this repo using the helm cli
helm repo add bitnami https://s3.amazonaws.com/charts.bitnami.com/incubator
helm fetch bitnami/memcached

But monocular is unable to fetch charts from this repo with the following repo config
  - name: bitnami
    url: https://s3.amazonaws.com/charts.bitnami.com/incubator
    source: https://github.com/bitnami/charts/tree/master/incubator

Log:
api_1      | INFO[0004] Local cache missing                           name=nginx version=0.4.0
api_1      | INFO[0004] Local cache missing                           name=tensorflow-inception version=0.0.3
api_1      | INFO[0004] Local cache missing                           name=apache version=0.3.7
api_1      | INFO[0004] Local cache missing                           name=tomcat version=0.4.15
api_1      | INFO[0004] Local cache missing                           name=mariadb-cluster version=0.5.0
api_1      | INFO[0004] Local cache missing                           name=jenkins version=0.4.41
api_1      | INFO[0004] Local cache missing                           name=wildfly version=0.4.9
api_1      | INFO[0004] Downloading metadata                          dest="/root/monocular/repo-data/bitnami/tomcat/0.4.15/chart.tgz" source=tomcat-0.4.15.tgz
api_1      | INFO[0004] Downloading metadata                          dest="/root/monocular/repo-data/bitnami/jenkins/0.4.41/chart.tgz" source=jenkins-0.4.41.tgz
api_1      | INFO[0004] Downloading metadata                          dest="/root/monocular/repo-data/bitnami/nginx/0.4.0/chart.tgz" source=nginx-0.4.0.tgz
api_1      | ERRO[0004] Error on DownloadAndExtractChartTarball       error="Get tomcat-0.4.15.tgz: unsupported protocol scheme """
api_1      | ERRO[0004] Error on DownloadAndExtractChartTarball       error="Get jenkins-0.4.41.tgz: unsupported protocol scheme """
api_1      | INFO[0004] Local cache missing                           name=postgresql version=0.4.14
api_1      | ERRO[0004] Error on DownloadAndExtractChartTarball       error="Get nginx-0.4.0.tgz: unsupported protocol scheme """

Any ideas?

It's a similar issue to https://github.com/kubernetes-helm/monocular/issues/367, my understanding was the --url was required when creating the index, but it looks like the Helm supports relative URLs. I can confirm the helm fetch works for me too.
So we would need to modify Monocular to understand relative links.


Extend EDA for attributes to CONUS reaches

Depends on #142.
Create target(s) that create maps and violin plots of the reach attributes across CONUS.

Would help to have a violin plot with one side for the CONUS distribution and the other side for the training site distribution. That way we can see if any attribute distributions are really different than our training data coverage.
Not sure how to indicate on a map where we have training sites. Might be too crowded to do.

@jds485   This isn't ready for a pr yet but I have a couple questions.  So far, I have just done comparisons of the gagesii features and the NHD conus features. I haven't looked into the area of applicability yet, I thought it would be good to just look at the raw CONUS data for a sense of how different the distributions are.  I can look into the aoa and try to add those as a 3rd violin plot on these.  These are for the high aggregated quantiles. I'll add another target to do the mid quantiles.
Right now I just sort of eyeballed the plots and made a list of ones that should be log transformed - things like drainage area. A lot of the plots need to be transformed but have zeros in them.  So - I could either just add a constant and transform just for visualization purposes.  Or the scales packages has a 'pseudo log transform' which I have never used but might be ok for our purposes?
Here are some examples:
Definitely a difference in drainage area distributions

Should be transformed but has zeros

Also, the function makes these in a loop and its pretty slow.  It might go faster if it were parallelized but I'm not sure I know how to do that.  I'll try and look at some of the other functions that are parallelized and see if I can figure it out.

I can look into the aoa and try to add those as a 3rd violin plot on these

Just so I'm understanding what would be in that 3rd violin plot: are you thinking that you would apply the aoa method, select the regions within the aoa, and then the 3rd violin plot would be for only those retained reaches? Also, the aoa method would be applied to only the attributes that are used in the models, so some of these attributes would not have a 3rd violin.

I'll add another target to do the mid quantiles.

Sounds good. Those targets can be added after the high quantile results are working.

So - I could either just add a constant and transform just for visualization purposes.

That's fine by me. The idea of pseudo log transform makes sense, though. I've not applied, but for visualization purposes it might be better than adding an arbitrary constant.

It might go faster if it were parallelized

We can discuss in the meeting later if you can't figure it out.


Why enable DDOS Standard for Key Vault?

This sentence: "Enable Azure DDoS Protection Standard on the Azure Virtual Networks associated with your Key Vault instances for protection against distributed denial-of-service attacks" implies that DDOS Standard should be enabled on a VNet because Azure Key Vault is deployed. Azure Key Vault only accepts HTTP traffic. It is already protected by Azure DDOS Basic. It is also a rate limited with thresholds that are well below a typical DDOS attack. So why enable DDOS Standard? This messaging may be confusing for security assessment teams and may lead customers to deploy an expensive resource that is not adding value (to the protection of Key Vault).

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: cd6bd6f1-4ba8-0104-1a49-0819e6c43fdf
Version Independent ID: 2ee92535-e22a-8348-1af9-ffc272a39dd4
Content: Azure Security Baseline for Key Vault
Content Source: articles/key-vault/general/security-baseline.md
Service: security
GitHub Login: @msmbaldwin
Microsoft Alias: mbaldwin

@DanielLarsenNZ Thank you for your feedback . We will have this reviewed and update this thread.

@DanielLarsenNZ We have requested the content team to review this ask and see how best this information can be restructured in the document.

This feedback references a version of the Azure Key Vault baseline that no longer exists (v1).  The v3 AKV baseline will be published shortly. Thank you! #please-close


Add AbstractVssApi, get/put/list api handlers

Add AbstractVssApi, get/put/list api handlers

Added in #12

Added in #12

No need to make separate PR for tests, IMO.

Closing as it was reviewed as part of #12


Improve the animation autokey editor icon

The icon was redrawn to snap to a 16x16 grid, with a lower number of SVG nodes in the process.
Preview
100% editor scale

200% editor scale

Thanks!


Number schema

It's needed to define a number schema.
const schema = new Config({
  port: Config.number({ coerce: true, min: 3, max: 6 }),
});

expect(() => schema.parse({ port: '3' })).toThrow(new ValidationError('Invalid configuration: The "port" expected to be greater than or equal "3" it is "1"'));

expect(() => schema.parse({ port: '123456789' })).toThrow(new ValidationError('Invalid configuration: The "port" expected to be less than or equal "5" but it has "9"'));

expect(() => schema.parse({ port: '3' })).toEqual({ port: 3 });

Hi :)
assign it to me :)

I've already done it.
Could you please take another one?

Yup


:robot: Triggering CI on branch 'release-next' after syncing to upstream/main

/test 415-test-e2e-aws-415


olPath at times not removed after clearing paths[]

Hi tombatossals,
I'm using path drawing from https://github.com/tombatossals/angular-openlayers-directive/blob/master/examples/110-path-example.html, which is really useful, thanks!
However, when I tries to clear paths ($scope.paths = []) dynamically, it doesn't clear. I have to transit to another view and come back to the view (with angular-openlayers) to clear paths.
Not sure if olPath faces similar resolved marker issue: https://github.com/tombatossals/angular-openlayers-directive/issues/110.
Will you (or someone knowledgeable) be able to look into it? I tried to edit the angular-openlayers-directive.js but not getting any good result.

Hi there,
Managed to find a possible solution ~
Under angular-openlayers-directive.js, find olPath directive. Just before the line "if (isDefined(attrs.coords))", add the following:
scope.$on('$destroy', function() {
layer.getSource().removeFeature(feature);
});
After which, you should be able to clear existing olPath.
Thanks to @zzats and #110. Hi @tombatossals, maybe you can test it out and commit into the codebase :)

Hi there,
I don't know if anyone can help me but I'm trying to create a 'drawing function' that would allow users to create paths dynamically and I can get it to work using Openlayers 3 standard functionality but I don't seem to be able to get anything drawn using this directive at all. Any help would be greatly appreciated, Thanks.

Hi @dvmac ,
I'll try to help out (I'm still a newbie). You can do something like:
<openlayers ... >
  <ol-path ng-repeat="path in paths" proj="{{path.projection}}" coords="{{path.coord}}"></ol-path>
</openlayers>

Then in your controller where you extend the $scope:
angular.extend($scope, {
... ,
paths: []
});
When you want to create path dynamically:
$scope.paths.push({
  projection: '<INSERT PROJECTION TYPE>',
  coord: [[<INSERT START COORD> ,<INSERT END COORD>]]
});

Thanks style-x7, I hadn't quite figured the coords="{{path.coord}}" part but that ties in with what I've been trying so far.
For the actual function itself, I've tried to adapt an existing markers control (which works) but I'm getting an SyntaxError: Unexpected end of input at Object.parse (native).
This is my function:
$scope.drawPath = function () {
$scope.$on('openlayers.map.singleclick', function (event, data) {
var prj = ol.proj.transform([data.coord[0], data.coord[1]], data.projection, 'EPSG:4326').map(function (c) {
return c.toFixed(4);
})
$scope.$apply(function () {
$scope.paths.push({
lat: data.coord[1],
lon: data.coord[0],
projection: data.projection,
label: {
message: "Lat: " + prj[1] + ", Lon: " + prj[0],
show: false,
showOnMouseOver: true
}
});
});
});
};
If you could shed any light I'd really appreciate it but thanks for your input.

I'm doing some housekeeping and closing the issue since it's quite old. Please feel free to re-open it if it still persists. Also maybe try to upgrade to the latest version as it might fix it for you.


Override default styling of cookie banner to hide

The cookie/survey banners have a display: none styling on which overrides the styling in the gem.
Need to add !important flag to force this styling.

That old chestnut 👍


fixbug fix memory leak of new_json_reader

Proposed changes
Issue Number: close #xxx
Problem summary
Describe your changes.
Checklist(Required)

[ ] Does it affect the original behavior
[ ] Has unit tests been added
[ ] Has document been added or modified
[ ] Does it need to update dependencies
[ ] Is this PR support rollback (If NO, please explain WHY)

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

run buildall

TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 33.37 seconds
stream load tsv:          424 seconds loaded<PHONE_NUMBER>9 Bytes, about 168 MB/s
stream load json:         24 seconds loaded<PHONE_NUMBER> Bytes, about 93 MB/s
stream load orc:          58 seconds loaded<PHONE_NUMBER> Bytes, about 18 MB/s
stream load parquet:          30 seconds loaded 861443392 Bytes, about 27 MB/s
https://doris-community-test-1308700295.cos.ap-hongkong.myqcloud.com/tmp/20230426030617_clickbench_pr_135289.html

There is no this problem on  1.2-LTS


Chat permissions on TDLib

I used my Telegram Premium account to disable the ability for people to send voice note and video note messages to me, via the official app's Settings > Privacy and Security > Voice Messages > Set to "Nobody".
On the client that I've made, which is logged in to another number, I was expecting to receive either updateChatPermissions or chatEventPermissionsChanged when the above change happened, but seemingly none was received.
Using the same number on the official client shows the mic icon being grayed out, which suggests that the permission change did take effect.
Am I looking at the wrong update events?
Please advice, thank you!

For users you need to additionally check the flag userFullInfo.has_restricted_voice_and_video_note_messages.

Thanks! I will check this out later.


Silently deleting directory contents

I noticed that if I have a folder (e.g. test):
~
❯ mkdir test

~
❯ touch test/hi.txt

~
❯ ls test
hi.txt

And I want to persist test folder across reboots, then I add it to:
{
  environment.persistence."/nix/persist" = {
    # ...
    users.will = {
      directories = [
        "test"
      ];
      # ...
    };
  };
}

~
❯ sudo nixos-rebuild switch --flake '/home/will/code/system/#blazar'
[sudo: authenticate] Password:
warning: Git tree '/home/will/code/system' is dirty
building the system configuration...
warning: Git tree '/home/will/code/system' is dirty
warning: `--preserve-env` has not yet been implemented and will be ignored
warning: `--preserve-env` has not yet been implemented and will be ignored
warning: `--preserve-env` has not yet been implemented and will be ignored
activating the configuration...
Warning: Source directory '/nix/persist/home/will/test' does not exist; it will be created for you with the following permissions: owner: 'will:users', mode: '0755'.
setting up /etc...
reloading user units for will...
restarting sysinit-reactivation.target
the following new units were started: home-will-test.mount, libvirtd.service, sysinit-reactivation.target, systemd-tmpfiles-resetup.service

~
❯ ls test

~
❯ mount | rg test
/dev/mapper/crypted on /home/will/test type ext4 (rw,relatime,x-gvfs-hide)

It silently deletes the contents of the test folder. Shouldn't this be an error similar to when persisting files?

Looked into this a bit more and realized it's not deleting anything.
~/tester
❯ tree -a
.
├── a
│   └── test.txt
└── b

3 directories, 1 file

~/tester
❯ sudo mount -v -o bind ./b ./a
mount: /home/will/tester/b bound on /home/will/tester/a.

~/tester
❯ tree -a
.
├── a
└── b

3 directories, 0 files

~/tester
❯ sudo umount ./a

~/tester
❯ tree -a
.
├── a
│   └── test.txt
└── b

3 directories, 1 file

When using bind mount (mount -o bind) a directory (./b) to another directory
(./a), the contents of ./a are "hidden", and instead, the contents of ./b
become visible within ./a.
In my case, tmpfs root, the data is actually only lost when rebooting.
I still think there should be an error message though. Perhaps a check can be added to create-directories.bash?
Perhaps a more robust solution would be to move the data into /tmp and then back after mounting?

Yes please. The current behavior of silently bind mounting is not convenient or obvious.
Apparently, there's a project that automates persisting: https://github.com/Geometer1729/persist-retro . I think this is not the best approach, and we should just throw an error if the directory is detected. The reason for that is there might be running processes that are working on the folder already. Unless we can figure out a heuristic that helps us not break running processes while moving the contents, we shouldn't do that.


Multi-level navigation property, entities above level 2 could not be saved in database

Suppose there is entity A defines navigation property List bs, and entity B defines property List cs, then I use
 var bulkConfig = new BulkConfig                     {                         PreserveInsertOrder = true,                         SetOutputIdentity = true,                         UseTempDB = true,                         IncludeGraph = true                     };
 await DbContext.BulkInsertOrUpdateAsync(new List<A> { entity }, bulkConfig);,the result is A and B save sucsess,but C save failed.But it works well by ef core db.savechanges(); I think there is something wrong with mutity level entities, when create datatable to sqlserver, the FK relation by son and grandson cann't translate beacuse PK of son didn't create?

Dupe: https://github.com/borisdj/EFCore.BulkExtensions/issues/520


Release v. Next

Follow https://github.com/dblock/danger-toc/blob/master/RELEASING.md.

@mr-dxdy this one is all yours

Done.  I fixed some typos in RELEASING.md. Link.