Getting hup events right after accept()

I wrote a multithreaded asynchronous http server in rust using mio. When I run a load test (using siege) the server works fine on the first load test but when the load test is done all requests to the server start failing. Using some logging I noticed that every new connection I get with accept() receives a hangup event immediately. The server connection itself doesn't get any error or hangup events.
I am running rust 1.12.0, mio 0.6 on OS X el capitan
Load test I am using: siege -b -c10 -d10 -t20S http://localhost:8080
Here's the main event loop of my server (full code with example https://github.com/syml/rust-webserver):
pub fn run(self) {
    let poll = Poll::new().unwrap();
    let server = TcpListener::bind(&SocketAddr::from_str(&self.host).unwrap()).unwrap();
    poll.register(&server, SERVER, Ready::readable(), PollOpt::edge()).unwrap();
    let mut events = Events::with_capacity(1024);
    let mut next_conn: usize = 1;
    let mut workers = Vec::new();
    // Create worker threads.
    for _ in 0..self.num_workers {
        let (tx, rx) = channel();
        let worker_handler = self.event_handler.duplicate();
        thread::spawn(move || {
            Self::process_events(rx, worker_handler);
        });
        workers.push(tx);
    }
    loop {
        println!("Polling...");
        match poll.poll(&mut events, None) {
            Err(e) => panic!("Error during poll(): {}", e),
            Ok(_) => {}
        }
        for event in events.iter() {
            match event.token() {
                SERVER => {
                    println!("Accepting..");
                    match server.accept() {
                        Ok((stream, _)) => {
                            println!("Registering new connection...");
                            match poll.register(&stream,
                                                Token(next_conn),
                                                Ready::readable(),
                                                PollOpt::edge()) {
                                Err(e) => panic!("Error during register(): {}", e),
                                Ok(_) => {
                                    println!("New connection on worker {} ",
                                             next_conn % self.num_workers);
                                    workers[next_conn % self.num_workers]
                                        .send(Msg::NewConn(next_conn, stream))
                                        .unwrap();
                                    next_conn += 1;
                                }
                            }
                        }
                        Err(e) => panic!("Error during accept() : {}", e),
                    }
                }
                Token(id) => {
                    println!("Sending event on conn {} to worker {}",
                             id,
                             id % self.num_workers);
                    workers[id % self.num_workers]
                        .send(Msg::ConnEvent(id, event.kind()))
                        .unwrap();
                }
            }
        }
    }
}

fn process_events(channel: Receiver<Msg>, mut event_handler: Box<EventHandler>) {
    loop {
        let msg = channel.recv().unwrap();
        match msg {
            Msg::NewConn(id, conn) => {
                event_handler.new_conn(id, conn);
            }
            Msg::ConnEvent(id, event) => {
                event_handler.conn_event(id, event);
            }
        }
    }
}

I run into the same issue with a similar test case:

do a stress test involving a lot of connection/disconnections, r/w
stop it (bulk of HUP events)
then new connections are getting a HUP event

Another piece of information is that the address then returned by:
match server.accept() {                         Ok((stream, addr)) =>
is wrong, it's an address associated with the previous stress load.

to fix this issue either:

set the server socket on level trigger
loop on server.accept
otherwise incoming connections are handled only once per socket state change.

Thanks @matwork for replying to this. Since it doesn't seem like a bug, I'm going to close the issue. Feel free to open a new issue if you think that there is actually a bug in Mio.


fix: rework package dependencies

Summary of Changes
Fixed issues with package dependencies that may cause problems.
Related Issues
n/a
Pull Request Checklist
Tick all boxes below to demonstrate that you have completed the respective task. If the item does not apply to your this PR check it anyway to make it apparent that there's nothing to do.

[x] All commits contain a DCO Signed-off-by line (we use the DCO GitHub app to enforce this);
[x] Updated LICENSE-3RD-PARTY.md for any added dependencies or vendored components;
[x] Updated documentation as needed for changed code and new or modified features;
[x] Added sufficient tests so that overall code coverage is not reduced.

If you have any questions to any of the points above, just submit and ask! This checklist is here to help you, not to deter you from contributing!
Pro Tip 🤓

Read our contribution guide at least once; it will save you a few review cycles!
Your PR will likely not be reviewed until all the above boxes are checked and all automated tests have passed.

PR template adapted from the Python attrs project.

@cvarjao Think this will be ok. I was thinking if I just put it in the dev and peer dependencies then it may get missed by a yarn publish? So I now have it in dependences and peer. Thoughts?


FEATURE: Generate identity from token

Generates the Identity from the token.
It uses the AppRoles defined at the token to attribute Reader, Writer and Owner privilege.
To be able to use the AppRoles claims a Hack was necessary to be added at the ServiceFactory.

Closing this down for now. Please reopen as updates happen.


UNMET PEER DEPENDANCY

My project is built on meanjs boilerplate. When I run, npm install --save yelp. I get the following results:
UNMET PEER DEPENDENCY<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>extraneous └── UNMET PEER DEPENDENCY<EMAIL_ADDRESS>What is the best way to resolve this?

I believe those warning messages will show up if you install any npm module. You should open an issue with meanjs.

Thanks, will do.


Update Set-CASMailbox.md

-MAPIBlockOutlookVersions.
Update the format of the value for this parameter from x.y.z.w to x.y.x, as only the x.y.z works when running the command in PowerShell.
@Linda-Editor @byszhoubin

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

File
Status
Preview URL
Details

exchange/exchange-ps/exchange/Set-CASMailbox.md
:white_check_mark:Succeeded
View (exchange-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

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

File
Status
Preview URL
Details

exchange/exchange-ps/exchange/Set-CASMailbox.md
:white_check_mark:Succeeded
View (exchange-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit ea13ff7:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/exchange-ps/exchange/Set-CASMailbox.md
:white_check_mark:Succeeded
View (exchange-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit ea13ff7:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/exchange-ps/exchange/Set-CASMailbox.md
:white_check_mark:Succeeded
View (exchange-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit 4d37dba:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/exchange-ps/exchange/Set-CASMailbox.md
:white_check_mark:Succeeded
View (exchange-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit 4d37dba:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/exchange-ps/exchange/Set-CASMailbox.md
:white_check_mark:Succeeded
View (exchange-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


How to mark a migration?

I have a migration that I have used. How me to mark "done" status? Or how me skip their? This migrations is structure the my project. I began use Phinx after wrote the structure.

If your phinx log is empty, then you could sort of cheat by commenting out the migrations methods (all of them) and then running it.
Or you could add an appropriate entry into the phinxlog table (or whatever you have called it, if you have overridden the name).

Have you in the plans to make the console command?

It would be problematic to have the ability to mark a migration as done without it actually being done. The idea is to be able to rollback and migrate as often as necessary. If a migration has been performed manually, then there really is no need to have it in Phinx.
As for the initial schema, it is often the case that a physical schema dump exists prior to using migrations.
So your first migration could be as simple as detecting if a table from the initial schema exists and doing nothing otherwise executing the initial schema query to create the schema.
As there is no down migration, this only needs to be an up. Maybe something like ...
<?php

use Phinx\Migration\AbstractMigration;

class InitialSchema extends AbstractMigration
{
    public function up()
    {
        if (!$this->hasTable('categories')) {
            $this->query(file_get_contents(__DIR__.'/initial-schema.sql'));
        }
    }
}

This allows new users to create the schema from the inital-schema.sql file. No rollback. Existing users will have no payload.
So, because this is such an edge case and easily solved within the existing Phinx mechanics, I don't see any reason to add an alternative mechanism.
Of course, if you can make a PR with functionality that is useful, then I'll certainly let the community decide.

If your initial schema is written in phinx, then just adding the detection of the first table around the whole up() method body should be enough. 2 lines of code provides you with everything.

@rquadling Ok. I shall be thought as to add mark command to Phinx. Example, in Yii2 Framework exists mark command in migration.


Add transfer_batch_to_device hook to DataModule

What does this PR do?

If transfer_batch_to_device is defined in datamodule, we'll use it instead of the default logic in trainer.
Fixes # (issue)
Before submitting

[ ] Was this discussed/approved via a Github issue? (no need for typos and docs improvements)
[x] Did you read the contributor guideline, Pull Request section?
[x] Did you make sure your PR does only one thing, instead of bundling different changes together? Otherwise, we ask you to create a separate PR for every change.
[ ] Did you make sure to update the documentation with your changes?
[x] Did you write any new necessary tests?
[x] Did you verify new and existing tests pass locally with your changes?
[ ] If you made a notable change (that affects users), did you update the CHANGELOG?

PR review
Anyone in the community is free to review the PR once the tests have passed.
If we didn't discuss your PR in Github issues there's a high chance it will not be merged.
Did you have fun?
Make sure you had fun coding 🙃

I still don't understand why we need to duplicate all data hooks from LightningModule into the DataModules. What prevents us from defining a data-interface and inherit in both classes? #2707

@awaelchli I felt the same thing as I submitted this. At this point (right before major release), I don't want to make major refactor to hooks. After 0.9.0 we can knock this out, because you're right...it seems pointless to copy over.
Does that sound reasonable?

I hope we can pick this up immediately after 0.9. I believe we otherwhise risk that these two defintions diverge and at some point we will have bugs because the Trainer does not call the hooks in one class the same way as in the other etc. This of course translates into maintenance effort that should not be.

Agreed!

@awaelchli - this test look good? Borrowed logic from you

well, given my previous comment I also have to answer no here 🤣 . Can we not parametrize the original test to use the LightningModule class and the DataModule class as inputs to the test?

well, given my previous comment I also have to answer no here 🤣 . Can we not parametrize the original test to use the LightningModule class and the DataModule class as inputs to the test?

opted to put it separately in datamodule file because I think it makes more sense to live there.

generally we are decoupling the data from the lightningmodule. but keeping the option to keep the data linked.
the last missing key here is the ability for a datamodule to define how it transfers data in case you have really weird formats

@awaelchli  we can do that after 0.9 but we need to think about it carefully as well to make sure we don't break functionality or bleed abstractions over.


<EMAIL_ADDRESS>NativeModule.RNCNetinfo is null

Try to use this library but i get<EMAIL_ADDRESS>NativeModule.RNCNetinfo is null error

There is an issue template, but you did not follow it. Please follow the issue template for your problems to receive attention https://github.com/react-native-community/react-native-netinfo/blob/master/.github/ISSUE_TEMPLATE/bug_report.md

Environment : Windows 8 64Bit
Versions
Android: 7.0
react-native-netinfo: 5.7.1
react-native: 0.62.1
react: 16.11.0
Description : Try to use this library but i get<EMAIL_ADDRESS>NativeModule.RNCNetinfo is null error


2019-06-12: Ogame Lobby updated : https://board.en.ogame.gameforge.co…

2019-06-12: Ogame Lobby updated : https://board.en.ogame.gameforge.com/index.php/Thread/808737-OGame-accounts-are-becoming-a-Gameforge-account/
This patch updates the URLs needed to login again.

Codecov Report

Merging #32 into master will not change coverage.
The diff coverage is 0%.

@@           Coverage Diff           @@
##           master      #32   +/-   ##
=======================================
  Coverage   43.97%   43.97%
=======================================
  Files          96       96
  Lines        6126     6126
=======================================
  Hits         2694     2694
  Misses       3377     3377
  Partials       55       55

Impacted Files
Coverage Δ

ogame.go
5.09% <0%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

Thanks for the PR. Highly appreciated.


Scroll to items as you navigate to them

Bug/issue #, if applicable: 90601877
Summary
Items should be scrolled into little by little, when focused, instead of jumping to the middle of the navigator scrollarea.
This is an alternative approach to #225
Dependencies
NA
Testing
Steps:

Using keyboard arrows or VO, navigate up/down across the navigator.
Assert items are scrolled as you reach the bottom

Checklist
Make sure you check off the following items. If they cannot be completed, provide a reason.

[x] Added tests
[x] Ran npm test, and it succeeded
[x] Updated documentation if necessary

I dont understand the issue.

oh... yeah... that is a whole other can of worms

This looks expected to me – feels natural. I'd leave it as is


[DO NOT MERGE] Add card details pattern

This pull request adds a new card details pattern including:

guidance
a video and screenshots

This is a contribution which needs to be reviewed by the Design System working group to determine whether it can be merged.

You can preview this change here:
Built with commit 97e92190dc7abc83712ce79501ec0bf2041cde1c
https://deploy-preview-476--govuk-design-system-preview.netlify.com

@timpaul Weird, that's not the size image I added...


Add logs to credx wallet migration

Codecov Report

Merging #1004 (ed53a5f) into main (78bdd33) will increase coverage by 0.00%.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #1004   +/-   ##
=======================================
  Coverage   29.88%   29.88%
=======================================
  Files         425      425
  Lines       27024    27024
  Branches     5256     5256
=======================================
+ Hits         8076     8077    +1
  Misses      16750    16750
+ Partials     2198     2197    -1

Flag
Coverage Δ

unittests-aries-vcx
29.88% <ø> (+<0.01%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Files
Coverage Δ

libvdrtools/indy-wallet/src/lib.rs
49.00% <ø> (ø)

... and 1 file with indirect coverage changes


Update download_dsyms.rb

Makes app identifier a required option, so we catch stuff like https://github.com/fastlane/fastlane/issues/9275

Don't forget to delete the branch 😉


Add Scatterer.netkan

Supercedes #949

KerbalStuff feeds "version" : "whatever" through the metadata. This is sure to create problems down the line.
Note to self to fix that to something less arbitrary,

#rebuild

That solved itself faster than I thought, merging.


Proposal Execute Callback Messages

This PR adds support for sending binary data through a proposal execute to improve message composability.
I was working on the arena-token-gateway and thought the data was currently being passed up, but it is only done during pre-propose module instantiation.
Use case:
DAO Executes Proposal -> Contract Generates Callback Messages -> DAO Receives Binary Data -> DAO Executes Callback Messages
In my case, I want the DAO to create a vesting contract on its cw-payroll-contract whenever the DAO accepts a token grant application.

This can actually be implemented better with authz.


Smart Git Sync

(docs): Adding a good draft for Amplication's Smart Git Sync.

Thanks @GreenMachine01 . I just implemented your feedback. I removed the section on build logs, created a better introduction, and rephrased the amplication branch section.
I'm trying to combine the Smart Git Sync and Basic Git Sync into one section here. My understanding is that they both work essentially the same. The only difference is the Basic Git Sync creates multiple branches. Is that correct?

@dericksozo @mulygottlieb I didn't deep dive but skimmed through it and it looked good to me.
I didn't deep dive since we plan to very soon terminate using the Basic Git sync and move all plans to use the Smart Git sync.

@dericksozo - an amplication branch is only created for the smart git sync. On the basic one, we create a branch per PR, which means a branch per each build in our platform. The current explanation about it, and starting by saying that Git Sync (in general) uses an amplication branch, is not accurate.

Hey @mulygottlieb, thank you for the feedback, but I went ahead and removed mention of Basic Git Sync as per #312. @GreenMachine01 please let me know if this resolves #312.
I apologize about all those extra commits. I was having some issues pushing my local branch for some reason, but I got it resolved.


Unable to use newer versions of megaparsec

Projects cannot use newer versions of megaparsec even when specifically pinned in the project's default.nix. This appears to be because obelisk's default.nix overrides it for some reason:
$ ag -C 1 megaparsec default.nix
91-            '';
92:          })).override { megaparsec = super.megaparsec_6_1_1; };
93-    algebraic-graphs =

Yeah, we should be applying this override before the user's overrides (unless of course we can get rid of it completely).

This is only a local override for modern-uri. The user's overrides already happen last! But, I thought I was able to reproduce your problem at first: If all you are doing is adding the override to your default.nix and then check with ghc-pkg list | grep megaparsec you will in fact only see megaparsec-6.1.1:
robert@xps13 ~/workbench/obelisk-test (master*) $ nix-shell ./default.nix -A shells.ghc
robert@xps13 ~/workbench/obelisk-test (master*) $ ghc-pkg list | grep megaparsec
    megaparsec-6.1.1

The reason is, nix is lazy! You have to add the dependency to your cabal file, for nix to install it. Simply put megaparsec to your build-depends section in frontend/frontend.cabal for example and you will see the following:
robert@xps13 ~/workbench/obelisk-test (master*) $ nix-shell ./default.nix -A shells.ghc
robert@xps13 ~/workbench/obelisk-test (master*) $ ghc-pkg list | grep megaparsec
    megaparsec-6.1.1
    megaparsec-5.3.1

if you have not overridden megaparsec yet, otherwise you will see your overridden version of course. You can verify that cabal is picking the right version by adding appropriate version bounds - cabal will pick yours!

But, btw. the newest megaparsec (6.5.0) won't build without updating some dependencies too:
Setup: Encountered missing dependencies:
parser-combinators >=0.4 && <1.0

It would appear that this issue has been long since resolved,  on or around https://github.com/obsidiansystems/obelisk/commit/169972db143da3dd3ae084939145d94327cc9ce5


ies.py - Multiple ies files

ies.py <path to files> <extract files to>
Can you add this feature? Thank you!

Sounds good, I will do that :)

done


⚠ Bofrost.de has degraded performance

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

HTTP code: 200
Response time: 1190 ms

Resolved: Bofrost.de performance has improved in 30995b8 after 20 minutes.


"Your obfuscator sucks"

Remember when u said that 😭😭😭

Yours is good, the thing is i wanted to make automated obf but, the math is wild and i  dont have energy to be solo dev doin this 😆 ,anyways abobus isnt that bad but that good..

Yours is good2 (somali), the thing is i wanted to make automated deobf but, the math is wild and i  dont have energy to be solo dev doin this 😆 ,anyways abobus isnt that bad but that good..

Nah u good lol this kid would join my support servers during early dev and try and clown on the project lol

kdot youre cool guy imo lol, before somali wasnt really good and i made deobfuscator for it before.. anyways current one is hard and im happy that it have been improved but kids abuse it. also one thing kdot ik how not to make it crash notepad so you gotta drag and drop the file in a replit and it doesnt crash there for me somehow..


Simple Fog canvas-layer-control button unresponsive and reveal tool remains active even when simple fog not active

This is 100% repeatable for me:

Open a scene.
Click on the Simple Fog icon in the left vertical menu (canvas layer controls)
Turn on fog, reveal a section of map, drop character icons
Click on a different canvas-layer-control icon, such as the Tokens Layer
Click on the Simple Fog layer icon.  NOTHING HAPPENS.

I am unable to get back to the Simple Fog control unless I refresh my screen.
Also, if I applied FOW with Simple fog and select a reveal tool, then switch to a different canvas layer control, such as the Tokens Layer, the reveal tool is still active. But there is no way to change to a different reveal tool (e.g. from brush to grid) because the Simple Fog icon is not responsive.
Also, after applying fog (clicking on the "eye" icon in the Simple Fog layer menu, I often have to refresh the browser before the fog is applied and before reveal controls can be used.
Do to the need to constantly refresh the browser, Simple Fog has become impractical to use.

Hey @MNBlockhead, sorry you are having trouble. This sounds like the issues we corrected a while back but just want to verify first. Can you verify that you are on version 0.1.16 of SimpleFog?
If not, we had to fork this repo from the original owner due to him not having time to keep updated. If you are still on his fork it will probably cap at 0.1.11 and say up to date. You will have to uninstall his and then reinstall the current fork, so that it gets you to 0.1.16. You can use this link when adding the module: https://raw.githubusercontent.com/League-of-Foundry-Developers/simplefog/master/module.json
If you are on 0.1.16, let me know, and we can start working on figuring out what is causing this.

Hey @MNBlockhead, sorry you are having trouble. This sounds like the issues we corrected a while back but just want to verify first. Can you verify that you are on version 0.1.16 of SimpleFog?
If not, we had to fork this repo from the original owner due to him not having time to keep updated. If you are still on his fork it will probably cap at 0.1.11 and say up to date. You will have to uninstall his and then reinstall the current fork, so that it gets you to 0.1.16. You can use this link when adding the module: https://raw.githubusercontent.com/League-of-Foundry-Developers/simplefog/master/module.json
If you are on 0.1.16, let me know, and we can start working on figuring out what is causing this.

It was 0.1.16.  I had uninstalled the prior version (Vance version) and installed the new LoED version.  But I just uninstalled and reinstalled again, and it seems to be working.  I also uninstalled the Less Fog module. Not sure if uninstalling Less Fog made a difference or if it was just uninstalling and reinstalling Simple Fog that fixed it. Anway, seems to be working now.

It was 0.1.16.  I had uninstalled the prior version (Vance version) and installed the new LoED version.  But I just uninstalled and reinstalled again, and it seems to be working.  I also uninstalled the Less Fog module. Not sure if uninstalling Less Fog made a difference or if it was just uninstalling and reinstalling Simple Fog that fixed it. Anway, seems to be working now.

Your can now simply uninstall and reinstall via the foundry module manager ui. Updating is not the same though.

Your can now simply uninstall and reinstall via the foundry module manager ui. Updating is not the same though.

Thanks! I'll do some research and see if Less Fog causes any weirdness and if there is anything we can do about it.
Thanks!!

Thanks! I'll do some research and see if Less Fog causes any weirdness and if there is anything we can do about it.
Thanks!!


hdapsgl: 0.5.0 -> 0.7.0

We now package a fork on github because:

gentoo does it
the fork is owned by the same organisation as the one we take hdapsd from.

Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 85005 1

1 package built:
- hdaps-gl


Impossible to reach max value with step more than 1

Hi guys I'm facing this issue in case I have range like: min 49 and max 100 with step 5 I can reach only 99. So I can not move slide to the end and display the maximal value.

Hi this is due to HTML base input type="range" element which do not handle that case, basically if the maxValue is not a multiple of the stepValue starting from minValue


multi-release jar for jdk11+ missing

I am using the lib for a javafx application which I have ported from javafx8 to javafx11 and openjdk 11.
In order to use the lib I had to modularize the lib myself and therefore the question if it is possible to get a multi-release jar which can be used in java8 and java 9+ ?
Here is my config to use the lib (I have used moditec to modularize the lib):
<dependency>
            <groupId>com.github.bbottema</groupId>
            <artifactId>emailaddress-rfc2822</artifactId>
            <version>2.1.3</version>
            <exclusions>
                <exclusion>
                    <groupId>*</groupId>
                    <artifactId>*</artifactId>
                </exclusion>
            </exclusions>
        </dependency>
        <dependency>
            <groupId>com.sun.activation</groupId>
            <artifactId>jakarta.activation</artifactId>
            <version>1.2.1</version>
        </dependency>
        <dependency>
            <groupId>com.sun.mail</groupId>
            <artifactId>jakarta.mail</artifactId>
            <version>1.6.4</version>
            <exclusions>
                <exclusion>
                    <groupId>*</groupId>
                    <artifactId>*</artifactId>
                </exclusion>
            </exclusions>
        </dependency>
        <dependency>
            <groupId>org.simplejavamail</groupId>
            <artifactId>simple-java-mail</artifactId>
            <version>5.5.1</version>
            <exclusions>
                <exclusion>
                    <groupId>*</groupId>
                    <artifactId>*</artifactId>
                </exclusion>
            </exclusions>
</dependency>

Moditec config file or input for module-info.java

<module>
                                    <artifact>
                                        <groupId>org.simplejavamail</groupId>
                                        <artifactId>simple-java-mail</artifactId>
                                    </artifact>
                                    <moduleInfoSource>
                                        open module simple.java.mail {

                                        requires java.desktop;
                                        requires jakarta.mail;
                                        requires jakarta.activation;
                                        requires emailaddress.rfc2822;
                                        requires org.slf4j;

                                        exports org.simplejavamail;
                                        exports org.simplejavamail.converter;
                                        exports org.simplejavamail.email;
                                        exports org.simplejavamail.internal.util;
                                        exports org.simplejavamail.mailer;
                                        exports org.simplejavamail.outlookmessageparser.simplejavamail;
                                        exports org.simplejavamail.outlookmessageparser.simplejavamail.model;
                                        exports org.simplejavamail.outlookmessageparser.simplejavamail.rtf;
                                        exports org.simplejavamail.outlookmessageparser.simplejavamail.rtf.util;
                                        exports org.simplejavamail.springsupport;
                                        exports org.simplejavamail.util;
                                        }
                                    </moduleInfoSource>
                                </module>

Uhm, modularity is a little beyond my knowledge currently, but I have done some module-related work on the upcoming 6.0.0 release in the develop branch. I'm close to releasing it, would you mind trying a snapshot of that to see if it meets your requirements?
Otherwise, I'm happy to work with you to make it Java9+ compatible.

I will try that out. It is not so complicated at all you have just create a module-info.java in the root of the source folder and compile it with Option -release 8 and afterwards again with -release 11 for java9+. So you have inside the source folder another source folder for java 9+.
But if you want to switch to java 9+ for the whole lib than create the module-info.java in the root folder and compile with java 9+ normally and you have the lib ready.
You can have a look here for more details:
https://www.baeldung.com/java-multi-release-jar
https://maven.apache.org/plugins/maven-compiler-plugin/multirelease.html

I was under the impression that, split packages notwithstanding, the module system is backwards compatible. Why would I need two releases?

Yes and no. The format of the module-info.java is only understand by a compiler greater than java 9. If you create the module-info.java and compile it with java 8 you will get errors. This is the only reason for the multirelease jar files. The multi-release jar prevents you from creating two jar files (one for java8 and one for java9) with two code bases.
Which JDK are you using to build the lib ?
For the reason above multi-release jar is the solution.
In this case in the directory for java 9+ only consists of the module-info.java. Therefore the compiler can create class files for java8 without the module-info.class and for the java9+ with the module-info.class. All other classes stay the same.

In the evening I will try it out and let write you what in the pom.xml must be changed and which file to place in which directory to archive all this above.

I have tried to commit but I think it was not successfully.
Please can you use the following maven plugin (readme explains that in more details):
https://github.com/metlos/multi-release-jar-maven-plugin
But you have to compile with java 9 at least. If you use java 11 you have to use additionally libs such as java.xml (was removed in java 11).
src/main/java-mr/9/module-info.java with the following content:
open module simple.java.mail {

                                        requires java.desktop;
                                        requires jakarta.mail;
                                        requires jakarta.activation;
                                        requires emailaddress.rfc2822;
                                        requires org.slf4j;

                                        exports org.simplejavamail;
                                        exports org.simplejavamail.converter;
                                        exports org.simplejavamail.email;
                                        exports org.simplejavamail.internal.util;
                                        exports org.simplejavamail.mailer;
                                        exports org.simplejavamail.outlookmessageparser.simplejavamail;
                                        exports org.simplejavamail.outlookmessageparser.simplejavamail.model;
                                        exports org.simplejavamail.outlookmessageparser.simplejavamail.rtf;
                                        exports org.simplejavamail.outlookmessageparser.simplejavamail.rtf.util;
                                        exports org.simplejavamail.springsupport;
                                        exports org.simplejavamail.util;

I'm still not clear on the issue here. What problem are we solving?
Why are you unable to use the library as-is? Newer JDK's should be backwards compatible with legacy jars, with the exception of split packages and J2EE modules.

I have now commit a working version which is usable with java9+ and has a minimal module-info.java included. For that I had to add some dependend jars and change to packaging goal from jar to multi-release-jar. At last I had to add two properties <maven.compiler.source>1.8</maven.compiler.source>
<maven.compiler.target>1.8</maven.compiler.target>
I commit to the master branch but I do not know if this was right because I am using github for the first time.
Have a look on it and see if that is ok for you.

@lanthale, you are focusing on the solution, while I'm still trying to understand the problem (or rather I fail to see a problem to begin with). So if you could please help me understand the problem if there really is one, then maybe we can move on to a solution.

Sorry I was too fast.
The problem is that actually your lib is treated as an automatic module in java 9 or newer which means that it is not possible to use jlink to create a distributable java app on any platform.
For a java-ee app on an application server this is not a problem, but for a desktop app it is.
So the files I added before are just adding the necessary information so that the lib is a valid named module with a defined interface for java beyond 9 and that the lib is in parallel useable under java 7 / 8.
Hopefully that clarified the issue a bit.
Do you have any questions ?

I just merged a pull request that fixes the module names. I hope to work on the multirelease item now.
One question, the requires/exports lines in module-info.java, is that manual work? Because that sounds aweful. Or did you generate that somehow?

@bbottema You can generate it with jdeps that comes with the JDK. But after that, you will most likely have to maintain it manually.

I'm using 6.5, and cannot compile using java 15:
java: module org.simplejavamail reads package org.simplejavamail.internal.modules from both org.simplejavamail and org.simplejavamail.core

Hi, same problem here with release 7.0.0 and java 17


create account with trust

This change allows to create an account and activate it at the same time.
the method 'create_account' now has a flag 'activate' that can be set to True to achieve that.
example:
import kin
from stellar_base.network import NETWORKS
from stellar_base.asset import Asset

NETWORKS['RESEARCH'] = 'scaling research'
kin_asset = Asset('KIN','GBSJ7KFU2NXACVHVN2VWQIXIV5FWH6A7OIDDTEUYTCJYGY3FJMYIDTU7')

sdk = kin.SDK('seed1',
              channel_secret_keys=['seed2'],
              network='RESEARCH',
              horizon_endpoint_uri='endpoint',
              kin_asset=kin_asset)

sdk.create_account('GDMBKYHJBPUOMULWQBBKQ7I62ZL5XBWXCEIAX5E7747TPFW3SEOG3UUI', starting_balance=30, activate=True)

Currently, kin-core-python does not know how to handle errors from transactions with multiple operations, so if this transaction will fail, the returned error will be 'internal error'

Codecov Report

Merging #32 into master will decrease coverage by 0.21%.
The diff coverage is 66.66%.

@@            Coverage Diff             @@
##           master      #32      +/-   ##
==========================================
- Coverage   94.11%   93.89%   -0.22%
==========================================
  Files          11       11
  Lines         798      803       +5
  Branches       96       97       +1
==========================================
+ Hits          751      754       +3
- Misses         36       37       +1
- Partials       11       12       +1

Impacted Files
Coverage Δ

kin/sdk.py
95.19% <100%> (+0.02%)
:arrow_up:

kin/stellar/builder.py
95.34% <50%> (-4.66%)
:arrow_down:

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 0dbba91...2ba16eb. Read the comment docs.


Add support for QML language

Ignore *.qmlc and *.js files (cached binary versions of their .qml counterpart, similar to .pyc and .py Python files)
### QML ###
# Cached binary representations of QML and JS files
*.qmlc
*.jsc

Can you submit a PR and add this template in data/custom?

Fixed by #278


Developer Resources

Links need to open in new tab
Make links text that says something like "see more" or "learn more" (editable) that are hyperlinked, instead of showing full URL

comment in google sheet asks for "Access Now -->" instead of URL. Is this the preferred verbiage?

Yes please

arrows look a little funky, want me to remove them:


Custom Auth Query Params (2) - DO NOT MERGE

Picking up where pull 650 failed and left off prior to the big changes and my bad commits, this add the ability for users to include their own custom query parameters when making an authorization URL. This is in response to issue 638 at request of @peleyal.
By providing a KeyValuePair<string, string>[] for the UserDefinedQueryParams member of the GoogleAuthorizationCodeFlow.Initializer, this collection will be used to update the resulting url via Google.Apis.Requests.Parameters.ParameterUtils.IterateParameters().
An attempt at a unit test has been provided with /GoogleApis.Tests/Apis/Requests/Parameters/ParameterUtilsTest.cs
The initial build will fail due to the issue with the logger, which I'll detail via a note on the code line.

@squid808 I'm sorry that it takes me too long. I'll try to get to it before the weekend.
Sorry!
Eyal

Awesome job, and thanks for moving from #650 to this one (huge merge.. sorry).
Looking forward for you to applying the comments and for us to push this change!
Enjoy the weekend, thanks!
Eyal

@peleyal I updated things based on your comments, thank you. I had indeed not noticed your note in the previous PR to add a new constructor, so I sorted that all out. Using the other logger method also worked, thank you. However, since I'm still on VS 2013 something wonky is happening with the Page Streamer tests for me, which seems to have killed all my other unit tests, and now VS isn't even recognizing the tests to run them. So, I didn't get the test fully fixed just yet, but I wanted to throw the rest of the code up in the meantime. I'll update once I sort out the tests.

Okey dokey, that should do it. I had the newer NUnit adapter installed for another project, and had disabled the older one in the VS extensions. I just had to enable both, didn't seem to be a conflict.
Also, fwiw I can only compile the solution if I comment out the PageStreamer tests. I haven't looked in to it too much, but I assume there is maybe some newer syntax that I can't compile in VS2013, so I've been commenting that whole file out and not committing that change as my workaround.

Awesome work!
Note my two small comments to this PR.
After addressing them, feel free to change to remove DO NOT MERGE from your PR.
Thanks for adding this functionality and being patient with my slow review.
Eyal

So, like I mentioned my bad about the change to Attribute. Like I explained you should actually revert it.
Regarding the broken tests, you can actually not test the Revoke method in this PR and just add a TODO to test it later on. We will focus only on your change here, and later on in another PR we will test more methods.
Thanks!
Eyal

I think that should about cover it. If you want me to add more info or a person's name in the TODO for the test, I certainly can. Once I confirm the build finishes (and that I didn't do anything stupid), I'll remove the DO NOT MERGE from the PR name.

We are really close!
I just have small nits (just comments), thanks for being so patient.
Eyal

Glad for the nits, sorry there were so many that were simply catchable. I'm running on fumes lately. I think that should square it all away.

Awesome! I'll wait for all the tests to pass, and then I'll merge it. Thanks so much for implementing it.
Eyal

@squid808, well done! :+1:


feat: upgrade bluesky_post

Checklist

[ ] The title of my PR starts with a Conventional Commit prefix (fix:, feat:, docs: etc).
[ ] I have read the Contributor Guide and followed the process outlined for submitting PRs.
[ ] I have updated/added tests for new/updated/fixed functionality.
[ ] I have updated/added dartdoc comments with ///.

Breaking Change

[ ] Yes, this is a breaking change.
[ ] No, this is not a breaking change.

Related Issues

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (dbc23d0) 78.06% compared to head (bc0eec0) 81.21%.
Report is 4 commits behind head on main.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1016      +/-   ##
==========================================
+ Coverage   78.06%   81.21%   +3.15%
==========================================
  Files         190       47     -143
  Lines        2671      724    -1947
==========================================
- Hits         2085      588    -1497
+ Misses        586      136     -450

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


set commit message to something by default

In the configuration, commit_message can't be empty but is set to null.  Could you set it to a valid default value?
 ->scalarNode('commit_message')
                            ->info('The message to use for the VCS commit.')
                            ->cannotBeEmpty()
                            ->defaultNull()
                        ->end()

Fixed by https://github.com/HypeMC/versioning-bundle/commit/b6cb1fa98390fb7c5fc22ce9d59d0727b49c4e6b


Adds optional labels to the controller

What this PR does / why we need it:
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Makes it possible to define labels for the deployment and the pods of the kserve-controller-manager.
Type of changes
Please delete options that are not relevant.

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

Feature/Issue validation/testing:
Please describe the tests that you ran to verify your changes and relevant result summary. Provide instructions so it can be reproduced.
Please also list any relevant details for your test configuration.

[ ] Test A

[ ] Test B

Logs

Special notes for your reviewer:

Please confirm that if this PR changes any image versions, then that's the sole change this PR makes.

Checklist:

[ ] Have you added unit/e2e tests that prove your fix is effective or that this feature works?
[ ] Has code been commented, particularly in hard-to-understand areas?
[ ] Have you made corresponding changes to the documentation?

Release note:

Optional labels can be defined for the deployment and the pods of the kserve-controller-manager.

/assign @yuzisun

My concern is there are endless options for users to customize, including labels, annotations, etc.
A better approach would be to use kustomize + helm: https://github.com/kubernetes-sigs/kustomize/blob/master/examples/chart.md

It's a pattern many projects follow. A few examples:

Superset
Istio
Grafana

I don't think this issue alone will justify introducing Kustomize in the stack of many projects.

My concern is there are endless options for users to customize, including labels, annotations, etc.
A better approach would be to use kustomize + helm: https://github.com/kubernetes-sigs/kustomize/blob/master/examples/chart.md

It's a pattern many projects as it's a very common need. A few examples:

Superset
Istio
Grafana

I don't think this issue alone will justify introducing Kustomize in the stack of many users. For example in our case, we basically cloned the project to bypass this limitation.

I agree that's common case that people need custom labels and annotations.


Update faq.md

Redacted spelling errors, improved translation, and included the missing links.
Closes #
✅ Checklist

[ ] I have followed every step in the contributing guide (updated 2022-10-06).
[ ] The PR title follows the convention we established conventional-commit
[ ] I performed a functional test on my final commit

Changelog
[Short description of what has changed]

Screenshots
[Screenshots]
💯

cc @Sboonny

Can we get this up to date and ready to ship?


Command error handling

Error handling when 1 or more command from list command that runs not worked or errors

make command class for each command


Added JSON Serialization Sample and File IO sample

Summary
Describe your changes here.
Fixes #Issue_Number, dotnet/docs#Issue_Number or dotnet/dotnet-api-docs#Issue_Number (if available)

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

@insiyakanchwala
Did you open this PR in the correct repository? This repo is for samples that we include in the .NET documentation on Microsoft Docs.  I don't see any article that would reference this sample. Is it incomplete? Was it meant for some other location?
Let us know, and we can provide guidance to route you to the correct additions so this is ready to merge.


Users did not understand the role of Verify in this context

Impact: Without a clear understanding of the role of Verify, users struggled to see the value of this part of the interaction.
Evidence: Some users repeatedly questioned both the inclusion of identity providers and the questions they asked.

Some users found identity questions discouraging as they did not understand why they were being asked.
These users tended to try to match the questions with their expectations of the concessionary travel application, this caused uncertainty and reduced confidence.
For example, some questioned why they must provide bank details for a free pass.

Referenced as part of #413


Question about the design of the model

Hi, thanks for the awesome work!
After reading the paper and codes I have one question about the model design. In section 3, you mention as

Note that compared to recent advances in neural volumetric representations [50, 45, 76], we do not use any neural network and directly optimize meshes, which are straightforward to use in computer graphic pipelines.

I was curious about the part about not using any neural networks because as the model is already light to train, I thought that attaching a small MLP layer after the primitive can highly increase the quality of the rendered images as from other NeRF type models. Although I do understand that attaching the MLP will make the entire model less practical in the sense that we cannot use it in computer graphics pipelines, I think this will make the model competitive with other existing MVS models. Have you ever conducted any experiments with neural networks?
Thanks!

Hi @crepejung00 thanks for reaching out!
That's a good point, we discussed it as some point to also increase the training speed but did not investigate it further. In general, I think it would be highly beneficial to model textures with MLPs because it will allow to automatically allocate memory to texture regions that require a high-level of details, which is an issue in our current formulation.


Example with slow getInitialProps

I have a page where the getInitialProps is a bit slow. I tried to make it even slower, but the loading indicator still didn't show up. The delayed-enter example does work, but it doesn't do its "loading" in getInitialProps.
Would it be possible for someone here to add a working example of a loader showing up while getInitialProps is "loading" something (on the client of course)?

Based on my understanding of Next, I don't think that's possible to make work. Next will wait for getInitialProps to complete before switching to the new page, so we can't actually detect the transition until the data finishes loading.

Wouldn't be possible to hook into router events either?  https://github.com/zeit/next.js/blob/canary/examples/with-loading/pages/_app.js
My understanding of Next is pretty newb, so you're probably right, but yeah. 🤔


issue on package.json file

Sails version: 1.0.1
Node version:  8.11.1
NPM version:  6.0.0
DB adapter name: N/A
DB adapter version: N/A
Operating system: ubuntu linux 16.04

plz change "coffee-script": "1.12.7" to "coffescript": "1.12.7" (no hyphen) on package.json

lol.? whats that mean?


Remove recursion error in accessors

Changes the implementation of accessors in dask-dataframe to avoid a
recursion error if the accessor was invalid for the given series. This
now relies on pandas to handle all validation, removing some code on our
end.
Also move all accessor tests (excluding categorical) into the
test_accessor.py file, keeping things more organized.
Fixes #5174

+1 for the idea and reorganisation; I haven't looked in detail.
The failure is actually a warning on test_cov.

Cool, merging.


Week 25 review Ticket

2.2

Notes and answers to questions
Red scale
smiley face
MCQ - data compresion

2.3

Notes
using tips model file to make a graph showing the tip ratio to the total bill
MCQ - 2 of them

MCQ TRI 2 final

2.8/3
Great job, Aditya! You did a good job showing understanding of the material this week. Your multiple choice questions have explanations for your errors and it appears you will learn a lot from your mistakes. You answered all the questions and used algorithms to encode your name in base64. Additionally, you showed a complex use of pandas and dataframes to create a linear regression model.


[Android] mapView onResume

In the Android doc i see that MapView has onPause and onResume activities.
So i thought i needed to use them in my app.
When do i see refreshing whenever i come back to my activity:

Looking at the sample app this does not seem to be called (which might be why we dont see the issue):
https://github.com/CartoDB/mobile-android-samples/blob/master/AdvancedMapAndroid/app/src/main/java/com/carto/advancedmap/baseclasses/activities/MapBaseActivity.java
Should i NOT call those methods? What the consequence of using/not using them?
Thanks

Yes, simply do not call these methods. Note that onPause releases map-related resources to make more space for other activities. This is so by design.

@mtehver thanks i wont!


LAB_04L02_Package_AVD_applications_ADDS

Module: 04
Lab/Demo: 02
Exercise: 02
Task: 02
Step: 07
Description of issue
Task 2, Step 7 instructs students to navigate to the F:\Apps folder to review the contents. They will be prompted with a dialog indicating they don't currently have access. Steps should include a note for them to click to gain access

updated


Wow no tests?

lbtm

Current coverage is 86.33%

Merging #1 into codecov will decrease coverage by -6.44% as of a2109b2

@@            codecov      #1   diff @@
=======================================
  Files             2       2
  Stmts           180     183     +3
  Branches         33      31     -2
  Methods           0
=======================================
- Hit             167     158     -9
- Partial           6       8     +2
- Missed            7      17    +10

Review entire Coverage Diff as of a2109b2

Powered by Codecov. Updated on successful CI builds.

Coverage decreased (-1.03%) to 95.082% when pulling 52784b50c55ddf878db23ec64dada616b4bf3691 on pr_test into 269a99f1a9be48ed98d3bf817d4d77685888e5b1 on codecov.

Coverage decreased (-1.03%) to 95.082% when pulling d4223d8c7a5b76eff2ca4c3a20805157366620b2 on pr_test into 269a99f1a9be48ed98d3bf817d4d77685888e5b1 on codecov.

Coverage decreased (-1.03%) to 95.082% when pulling e0dbfd8f4de61f6bb7a0d927b4c0026cad0993a9 on pr_test into 269a99f1a9be48ed98d3bf817d4d77685888e5b1 on codecov.

Coverage decreased (-1.03%) to 95.082% when pulling 7a8ecba3766ca24cd909dc7a1091801c35a28dbf on pr_test into 269a99f1a9be48ed98d3bf817d4d77685888e5b1 on codecov.

Coverage decreased (-73.3%) to 22.849% when pulling 1835c4db4d54405f216e2657c1b4fa8c33dee222 on pr_test into 269a99f1a9be48ed98d3bf817d4d77685888e5b1 on codecov.

Coverage decreased (-1.03%) to 95.082% when pulling cacd91e8b0f39f12a14c663ac321ef1ff96c5eb4 on pr_test into 269a99f1a9be48ed98d3bf817d4d77685888e5b1 on codecov.

Coverage remained the same at 96.111% when pulling 8b5d871f0ae6f7ff04bbec76cc9c24d1877ee290 on pr_test into 269a99f1a9be48ed98d3bf817d4d77685888e5b1 on codecov.

Coverage remained the same at 96.111% when pulling 8b5d871f0ae6f7ff04bbec76cc9c24d1877ee290 on pr_test into 269a99f1a9be48ed98d3bf817d4d77685888e5b1 on codecov.


Add ability to open Replica Sets over TLS

[ ] Requires dependency update

SUMMARY
This pull request adds the ability to connect to a Replica Set over TLS. Additionally, you can now connect to a Replica Set using a modified DNS Seedlist Connection format (specifically, you use just the host).
COMPLETED TASKS

Added openReplicaSet variants for connecting over TLS.
Added openReplicaSet variants for connecting via the DNS Seedlist Connection host, including both TLS & non-TLS variants with & without a custom connection timeout.

INCOMPLETE TASKS

Automatically reconnecting when the primary steps down & an election occurs.

NOTES
To connection a Replica Set over TLS by explicitly specifying the Replica Set name and seedist, use openReplicaSetTLS. To connect using a custom connection timeout, use openReplicaSetTLS'.
To connection a Replica Set using the DNS Seedlist Connection host, use one of the four openReplicaSetSRV variants. For an unsecure connection, use openReplicaSetSRV. For a secure connection over TLS, using openReplicaSetSRV'. For a custom connection timeout, use openReplicaSetSRV'' (unsecured) and openReplicaSetSRV''' (secured over TLS).
In order to authenticate, you will need a Pipeline reference which can be retrieved by calling the primary function with the previously opened replica set.
For Atlas, you will do something similar to this:
pipe' <- openReplicaSetSRV' clusterHostName >>= primary
result <- access pipe' master "admin" $ auth dbUser dbPass
.
.
.
close pipe'

Hopefully we can feed this back into the upstream repo and close this issue.

@albertov Thanks for the review. I’ve addressed your comments in 2f8894b


Update info.json for ton blockchain

@PolyProgrammist Did you create a pull request into original trust wallet repository?


Drives without a letter assigned displayed as a blank space (followed by a separator)

Etcher version: Beta 13
Operating system and architecture: Windows 10
Do you see any meaningful error information on DevTools? N/A

When I select a drive that doesn't have a letter assigned (either there is no partition on the drive or there isn't a partition with a letter) the drive is shown as  - 2.0 GB. The name is empty, so there's just a dash followed by the size.

I think it should say one of the following:

<no name> - 2.0 GB
<unnamed> - 2.0 GB
<unformatted> - 2.0 GB
2.0 GB
\\.\PHYSICALDRIVE3 - 2.0 GB

The last one would be preferred, because it is the most informative.

Hi @WasabiFan ,
Thanks a lot for reporting this. Great catch, I didn't consider a drive without a letter assigned. We should definitely fallback to \\.\PHYSICALDRIVEX. I'll send a PR soon.

Now that I think about it, the physical drive information would be useful even if it has a letter (command-lime tools, logs, etc.); how easy would it be to add a tooltip or a "More info" button which lets you see that info? Obviously you can always just look at disk manager to correlate letters with physical drives, but having that info present in the UI would be nice.

Hi @WasabiFan ,
A tooltip or something like that should be a simple addition. Let's loop @taahirisaacs , Etcher's designer, here for his UX input first.

I've fixed this issue by printing the device ID if a drive letter is not assigned to the drive. Lets keep this open for tooltip/more info discussion.

After some discussion, it is unlikely that we will show the physical drive internal path, since its irrelevant to the big majority of our users. We will of course do show it if there is no drive letter to supersede it.
Thanks a lot!


Make Add and Remove return boolean values

Hi,
When dealing with Roaring bitmaps in an application, I have found that it could be helpful if one is able to know, after inserting/deleting an integer to/from a Roaring bitmap using the add/remove method, if that integer was existing before in the bitmap or no. Same kind of thing done by Java collections add and remove methods.
If one has to maintain the cardinality of integers stored in a set of Roaring bitmaps, he needs to know if an inserted/removed integer was exsiting before in the bitmap, in order to increment/decrement the global cardinality. Such a job could be realized by calling the contains method before inserting or removing an integer. However, this will make two accesses to containers. However, if the add/remove method is able to return a true or false indicating if the inserted/removed integer was or not in the bitmap, one would be able to avoid a call to contains in such situations and preserve a supplementary access to containers.
Greetings!

Thanks for raising the issue. This seems sensible.
The next step would be to prototype an implementation and estimate the performance penalty we can expect (preferably using the JMH framework).
Possibly, we could start first with an intermediate API extension that has add and remove methods, maybe named checkedAdd and checkedRemove... and see if they are slower. This has the benefit of not breaking or slowing down any of the existing code, while meeting your needs.
The implementation could be identical to the current add and remove implementation, with an added check to see if the cardinality of the container has changed. Since existing container can return the cardinality in constant time this should be reasonably fast. However, getting the cardinality of a container involves polymorphism, and so, possibly, some branching. It is possible that this branching could introduce latency.
Still, this could be good enough. If the methods are distinct from add and remove (e.g., checkedAdd and checkedRemove), then nobody would care if they are slightly slower.
There are more intrusive changes that one could make, of course.
Samy? Do you want to have a go at writing a prototype, using the non-intrusive approach I suggest, with a JMH performance test, and submit a PR?

Ok. I also think that checking the size of the containers before and after insertion/deletion is the more effective way to do that. I'm going to try this solution.

Thanks!

Issue recently resolved.


any validation provided  ?

Is here any methods for fields validation on submit(). Probably i missed something but there is nothing about in docs.

I know this isn't ideal, but it is an option. Stripe has an excellent validation library which can be used in conjunction with card.js.
The library is located below:
https://github.com/stripe/jquery.payment

You can use Payment or jquery.payment to do validation - Card doesn't provide this.


Exclude certain controllers?

I set EnforceHeader=true, to require the client to add the correlationID header.  I need to exclude some of the controllers.  I have a health check endpoint and swagger documents endpoint that should not require a correlationID.  Is this possible?

Hi @chambersDon,
Thanks for raising this suggestion. With the current design, it's not available in the package. To implement this, I think the middleware would need to be made "Endpoint aware" so that when it runs, it knows which Endpoint will handle the request. This is technically possible if the middleware where to be registered after UseRouting() in the pipeline. An attribute could then be used to mark Endpoints with metadata to avoid the correlation ID from being enforced on those requests. The main downside is that for this to function, it may require consumers to call UseRouting earlier than they would normally. Often we want to grab the correlation ID right at the start of the request so the correlation ID is registered as the first middleware in the pipeline.
I'll put this into under consideration as I can see the requirement. As I type this, I can think of a few designs which may be a nice way to enable this where it is needed.
Right now, I think the best option is not to enforce this using the library with EnforceHeader = true. Instead, you could add you own piece of middleware to selectively check for the correlation ID when you deem that it is required for the Endpoint in question. Alternatively, since you likely only want the enforce to happen inside MVC, you could write a global ActionFilter and apply the check there, such that you reject any requests without the correlation ID which are routed to an MVC Action.


Ready never called with empty watcher

Describe the bug
When creating an empty watcher, the "ready"/"error" events are never called. I'd like to create the watcher and the add and remove paths dynamically.
Versions (please complete the following information):

"chokidar": "^3.5.2",
Node version v12.21.0
OS version:
ProductName:	macOS
ProductVersion:	11.5
BuildVersion:	20G5023d

To Reproduce:
watcher = require("chokidar").watch([]).on("change", console.log).on("ready", () => console.log("ready")).on("error", () => console.log("error"))

Expected behavior
I would expect this watcher to become ready basically immediately. And I'd also expect watcher.add to return a promise that resolves once that path has been traversed and is ready again.
Additional context
My app has a sidebar, similar to VSCode or Finder, and I'm just trying to watch for files added or removed. The main process of the Electron app creates a watcher and the renderer process will add and remove things based on the sidebar state.

This would be great!

And I'd also expect watcher.add to return a promise that resolves once that path has been traversed and is ready again.


Enhance MemorySchemaProvider to support register_listing_table

Which issue does this PR close?

Closes #1836
Rationale for this change

What changes are included in this PR?

Are there any user-facing changes?

Im looking into whether I should just have register_object_store do the same thing as the same named method on ExecutionContext

@alamb @seddonm1 @Igosuki @returnString i've extended the MemorySchemaProvider to support creating ListingTable from an ObjectStore.  This can further be used with object_store.list_file(...) to create many tables in a directory at once.
I havent yet tested with partitioned data, but I think this is a good start.
I'm interested in your views.

@houqp do you have any thoughts on this?

Its also quite interesting how this only fails CI tests on windows.
Has anyone seen that issue with the object store abstraction before?

Sorry for the late review. Looking now

@alamb do you think ObjectStoreSchemaProvider is the way to go here? Doing that makes sense to me to keep a cleaner separation of functionality without the bloat of ObjectStore when it isn't needed.

@alamb do you think ObjectStoreSchemaProvider is the way to go here? Doing that makes sense to me to keep a cleaner separation of functionality without the bloat of ObjectStore when it isn't needed.

Yes, I do think that would be a cleaner separation of functionality. Does anyone else have opinions (@rdettai  or @yjshen  for example?)

I don't understand why there should be various interfaces. If you want to achieve abstraction over tables, the best is to enable a single data abstraction/discovery interface that points to tables no matter where they are ?

I don't understand why there should be various interfaces. If you want to achieve abstraction over tables, the best is to enable a single data abstraction/discovery interface that points to tables no matter where they are ?

I think it comes down to if the InMemoryCatalog implementation would be used for interfaces on top of files -- aka do we want all InMemoryCatalog's to know about ObjectStores?
I can go either way -- if people prefer this way I am fine with that -- let's get the PR polished up and we can merge it in

I agree with @Igosuki that it would be nice to have a single abstraction but im not sure this is the PR for that.  I've seen talks of other table providers, for example on slack a BigTable table provider was mentioned / potentially in the works - so its not clear to me where we would want to draw the line yet.
My proposal would be to get this merged with the clear separation of functionalities (i.e. i migrate to ObjectStoreSchemaProvider and I can also make a separate issue to discuss having a single interface - for example maybe a TableRegistry.

My proposal would be to get this merged with the clear separation of functionalities (i.e. i migrate to ObjectStoreSchemaProvider and I can also make a separate issue to discuss having a single interface.

Sounds good to me -- it seems to me like the only blocker is whatever is causing the test failure on Windows

@alamb hopefully this is good now.


Adds the option to generate P2SH-P2WPKH address with Address()

The last request had a typo in the bytes which messed things up. This is fixed and tested.

Can you add unit tests?

Closed in favor of https://github.com/prusnak/bip32utils/pull/12


Upcoming API changes

These should be possible:

No need for container element, just render component if it is not provided

<InFlux namespace="sidebar" config={{
  [getDefault()]: 'close',
  open: SidebarOpened,
  close: SidebarClosed
}} />

Instead of passing Smart container to the config, pass it as child

Instead of
<InFlux namespace="sidebar" config={Container} />

Should be
<InFlux namespace="sidebar">
  <Container what={ever} />
</InFlux>

So we can have nesting and pass some props.

Having (2) in place, we would be able to specify config with default value.

<InFlux namespace="sidebar" config={{[getDefault()]: 'open'}}>
  <Container />
</InFlux>

Moved to #61

Remove pathname #64 (there is a react-router for that)


fix(popover-edit): incorrect spacing for form field inside popover

For some reason the padding on mat-form-field was reset inside the popover edit overlay which made it looked weird. From what I can tell, there's no reason to do this so these changes remove the override.
For reference, here's what it looks like at the moment:

IIRC I did that to fix weird layout when there's no heading in the popup. Can you add a screenshot of that use case without the style override?

Updated the description with an "after" screenshot.

Please also add a screenshot with one of the examples that does not have heading text. I think that’s the case that looked really bad with all the extra top padding in the form field.
Might be solvable with some negative margins?

Sure, here's a before and after. I don't think there's anything off about it:

That’s it. The space reserved for the label causes a mismatch in vertical spacing above and below. Unfortunately the visual spec I’m working off of is internal or I’d share it.
Is there another less objectionable way to fix the spacing issue without overriding the form field styling?

How about only clearing the padding-top? The main issue I was aiming to address with this PR was the text being flush against the bottom border. Here's what removing the top padding only looks like:

Looks good. Thanks.


Interacting with RichText doesn't work

...or doesn't work out of the box - anyway, this problem should be investigated and documented in the docs.
Reported by a nice guy @ Flutter Firebase Festival.
Code sample
import 'package:flutter/material.dart';
import 'package:flutter_test/flutter_test.dart';
import 'package:patrol/patrol.dart';
import 'package:patrol_challenge/main.dart';
import 'package:patrol_challenge/pages/quiz/form_page.dart';
import 'package:patrol_challenge/ui/style/colors.dart';

void main() {
  patrolTest(
    'test',
    nativeAutomation: true,
    ($) async {
      await $.pumpWidgetAndSettle(
        const MaterialApp(
          home: Scaffold(
            body: Column(
              children: [
                Text.rich(
                  TextSpan(
                    children: [
                      TextSpan(text: 'Hello', style: TextStyle()),
                      TextSpan(
                        text: 'World',
                        style: TextStyle(fontWeight: FontWeight.bold),
                      ),
                    ],
                  ),
                )
              ],
            ),
          ),
        ),
      );

      await $('World').tap(); // should work
      await $('Hello World').tap(); // should work as well
      print('tapped');
    },
  );
}

This works
import 'package:flutter/material.dart';
import 'package:patrol/patrol.dart';

void main() {
  patrolWidgetTest(
    'test',
    ($) async {
      await $.pumpWidgetAndSettle(
        const MaterialApp(
          home: Scaffold(
            body: Center(
              child: Column(
                mainAxisSize: MainAxisSize.min,
                children: [
                  Text.rich(
                    TextSpan(
                      children: [
                        TextSpan(text: 'Hello', style: TextStyle()),
                        TextSpan(
                          text: 'World',
                          style: TextStyle(fontWeight: FontWeight.bold),
                        ),
                      ],
                    ),
                  ),
                ],
              ),
            ),
          ),
        ),
      );

      await $('HelloWorld').tap(); // should work as well
      print('tapped');
    },
  );
}

I did some researching and looks like there's no easy way to tap on a particular TextSpan. For the time being, this workaround should work.

Awesome, thank you!


Screen reader does not announce search suggestion information after entering data in the 'Message Input Box' edit field present in the dialog.

Is it an issue related to Adaptive Cards?
No.
What is the PWD impact?
User who rely on screen readers are getting impacted if Screen reader does not announce searching result after entering data in the 'Message Input Box' present in the dialog, the screen reader user will not be able to get proper information on these controls and will be left unaware about the visual information. This will lead to distortion.
What browsers and screen readers do this issue affect?
Windows: Chrome with NVDA
Are there any code-based customization done to Web Chat?
No, I am using Web Chat without any customizations except "styleOptions".
What version of Web Chat are you using?
Latest production
Which area does this issue affect?
Others or unrelated
What is the public URL for the website?
Request Contributors - POC (sharepoint.com)
How to reproduce the issue?
Step 1: Open the above URL in New Edge/Chrome browsers with valid credentials.
Step 2: Invoke the 'Ask iris now' button which is present under the questions section.
Iris the bot dialog will appear. Type “How do I find” in type your message 'Message Input Box' edit field.
Step 3: Verify that Screen reader does not announce screen reader does not announce search suggestion information after entering data in the 'Message Input Box' edit field present in the dialog or not.
What do you expect?
Screen reader should announce search suggestion information after entering data in the 'Message Input Box' edit field present in the dialog.
What actually happened?
Screen reader does not announce search suggestion information after entering data in the 'Message Input Box' edit field present in the dialog.
Observation: When focus lands on message input box edit field, after entering data screen reader is not announcing any information.
Do you have any screenshots or recordings to repro the issue?

Did you find any DOM elements that might have caused the issue?
No response
MAS reference
https://aka.ms/MAS1.3.1
WCAG reference
https://www.w3.org/WAI/WCAG21/Understanding/info-and-relationships.html
WAI-ARIA reference
No response
Adaptive Card JSON
No response
Additional context
No response

#A11yMAS;#A11ySev2;#MAS1.3.1;#HCL;#Accessibility;#FeatureBotframeworkwebchat-Apr23;

This is outside of the scope of Web Chat. The auto-complete is a customization done by Iris Bot and this is not implemented in Web Chat.

As per the above comment, this issue is not under bot framework team. Hence closed the bug and partner team (WWL) also changed the ownership in bugs.


Skeletonization in libpysal.cg

@jGaboardi a question mostly for you. Given the relatively generic nature of the skeletonization, do you reckon it could be merged upstream to lib?

xref #25

now that the geometry.voronoi_skeleton() function is quasi-matured and tested, we may want to revisit this.

I'd say that this is not on top of the priority list. Eventually it would be nice but I'd try to finish the stuff that will make it into the publication before looking into this.


Create ASAP7 & Sky130 PDKs

Requires:

[x] layers.json
[ ] Spice models
[x] Specific circuit examples

[x] Must have functional schematic simulations
[x] Must have testbench

ASAP7 repo: https://github.com/ALIGN-analoglayout/ALIGN-pdk-asap7
Sky130 repo: https://github.com/ALIGN-analoglayout/ALIGN-pdk-sky130

Completed except the spice models.


(NVIDIA 525 Hyprland Arch) Waybar doesn't render properly due to GtkWindow being overriden.

Whenever I launch Waybar, I get the error shown below. The one that concerns me the most is the critical error, but having a bunch of warnings is still pretty worrying.
I tried installing all the dependencies again and tried a bunch of things, but nothing seems to work. I even reverted back to the default config and the default sytles css, but I still get the exact same problem. Not sure what to do, because I really want to use waybar, but I am struggling right now.
I added myself to the input group which solved other warnings that were happening, but the ones that are shown in the screenshot are still present even after doing that.
Any help would be greatly appreciated.

Same issue here, nvidia+hyprland trying to use waybar. Although for me, it's not critical and only a warning.
Any guidance would be soo apprciated!


composer shortcuts broken

it doesn't open composer, also some others doesn't work
will need update

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/28109137-composer-shortcuts-broken?utm_campaign=plugin&utm_content=tracker%2F1241747&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F1241747&utm_medium=issues&utm_source=github).

  "nbbpm": {
    "compatibility": "^0.6.0"
  }

says everything :D
Update is planned. No estimations thought.

Updated (completely reworked).
NodeBB comptibility: ~0.9.0 (at least not tested with <0.9.0, may work anyways)
Themes supported: lavender, persona
Composer supported: composer-default


Fix hierarchy formatting

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

Hi @gewarren This pull request was opened in the public repo. Would you make future content updates in the private repo? Thanks.


Small font sizes no longer rendering correctly?

It has been a few months since I last exported a new bitmap font.
While previously I had issues with font being blurry, which you fixed with the sharpening option, now I'm seeing issues where fonts aren't rendered properly due to being cut off and slightly transparent.
Here is what this font looked like a few months back when you did the sharpening fix:

Here is what it looks today with the same settings, no matter which browser I use:

For some reason, parts of the fonts are now transparent, and many characters are cut off. Is there a new setting I'm missing or something that broke in the interim? The font can be obtained from here for testing: https://datagoblin.itch.io/monogram

Hi,
Do you have this problem using Firefox?

Good call.  Firefox does render correctly.


Allow capabilities to be parsed with and without CAP_ prefix

What type of PR is this?
/kind feature
What this PR does / why we need it:
It is now possible to convert strings into capabilities which contain
and do not contain the CAP_ prefix. When serializing them back into a
string, we always get a CAP_ prefixed result.
Which issue(s) this PR fixes:

None
Special notes for your reviewer:
None
Does this PR introduce a user-facing change?

None

/lgtm


What about SharePoint 2019?

Please update to include SP 2019 as appropriate.

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

ID: 9579b467-9b55-1aae-589e-4864ab028984
Version Independent ID: f0c07ff9-52c9-d4d9-226a-e8b561de6581
Content: Configure Office Online Server for SharePoint Server 2016
Content Source: OfficeOnlineServer/configure-office-online-server-for-sharepoint-server-2016/configure-office-online-server-for-sharepoint-server-2016.md
Product: office-online-server
GitHub Login: @lorenahuangliu
Microsoft Alias: lohuan

Following up, does Office Online Server support SP 2019 ?

https://social.technet.microsoft.com/wiki/contents/articles/52608.office-online-server-for-sharepoint-server-2019.aspx
says yes. Will try it soon and see.

SP 2019 has been updated, as applicable


Cannot run your code

Hi,
I've been trying to run your code, but it does not run because of so many errors.
I fixed all the problems mentioned by previous issues, but yet another error came out..
I think you manually modified the gluoncv toolkit too much.
Can you please share all your modifications?
Thank you.

Sorry for the errors. Truly, gluoncv has been updated, which is slightly different from the version I used. I will try to update the code for the latest version. The main idea is to split the last normalization operation and perform this step after imposing noise.

The code has been revised and should work this time. Feel free to contact if there are any other questions.


Html WebResource in form, is reloades randomly when ejecute Xrm.Navigation.navigateTo

Hi.
I have this problem.
I have a Html WebResource in an entity form.
It has a button, that execute Xrm.Navigation.navigateTo to open another html webresource in a modal.
var datos = {
ClientId: 123,
ProductId: 123
};
        var jsonString = JSON.stringify(datos);
        var pageInput = {
            pageType: "webresource",
            webresourceName: "axx_/Html/SelectorCuentasBancarias/CuentasBancarias.html",
            data: jsonString
        };
        var navigationOptions = {
            target: 2,
            width: { value: 80, unit: "%" },
            height: { value: 80, unit: "%" },
            position: 1
        };

        Xrm.Navigation.navigateTo(pageInput, navigationOptions).then(
            function success() {
                console.log("ok");
            },
            function error() {
                console.log("err");
            }
        );

This work Ok, but, randomly, the WebResource in the Form is reloades, without the parameters thar de form pass to the webresouce, such as id, i don´t know why

@dga205 Thanks for contacting us. Unfortunately, we're unable to handle Support requests for the product here. To contact Support, visit https://powerapps.microsoft.com/support/. Closing the issue.


Feature/pdfocr

New features:

Adds pdfocr tool to the library.
Adds missing parameters to listSignatures.

Changes:

Now TaskFactory typechecks all tools are present.
Updated dependencies.

Fixes:

Fix several signature tests.

Do you want me to replace all inRange occurrences with the expectations pair? I only did those of the tests I had to fix.
Or do you want a helper that contains the expectations?
const expectInRange = (actual, expected, errorMargin) => {
    expect(data.length).toBeGreaterThan(expected - errorMargin)
    expect(data.length).toBeLessThan(expected + errorMargin)
}

Do you want me to replace all inRange occurrences with the expectations pair? I only did those of the tests I had to fix. Or do you want a helper that contains the expectations?
const expectInRange = (actual, expected, errorMargin) => {
    expect(data.length).toBeGreaterThan(expected - errorMargin)
    expect(data.length).toBeLessThan(expected + errorMargin)
}

Exactly 👍


CSR with subject containing commas not accepted by cert_write

Hello,
I noticed during testing that the cert_write program will not accept a CSR generated by cert_req, if the CSR contains a subject with one or more items therein containing commas, which is then followed by another item.  For what its worth, the CSR can be handled by OpenSSL.
Here are the commands I am testing with:
To generate the CSR:
cert_req.exe filename=test.key output_file=test.p10 subject_name="CN=example, O=Test\, Inc., C=US" key_usage=digital_signature,key_encipherment
To generate the certificate given the CSR:
cert_write.exe request_file=test.p10 issuer_key=issuer.key
cert_write fails with "  !  mbedtls_x509write_crt_set_subject_name returned -0x2100 - X509 - Requested
OID is unknown".
I suppose this is due to the fact that cert_write loads the CSR and encodes the subject into a string, but does not escape the resultant string before attempting to parse it.  When it tries to parse the string it fails since the comma inside "Test, Inc." is not escaped.
A reasonable way to handle this would perhaps be to copy the subject from the CSR directly into the destination certificate structure, rather than round-tripping the subject into string format and then parsing it again.  If that is not acceptable, perhaps escaping the string after loading it from the CSR would be another solution.
Also worth noting, is that if I leave off the ", C=US" part of the subject in the initial cert_req run, the CSR will be accepted by cert_write, but the resulting certificate will contain a subject with only "CN=subject, O=Test".  The, ", Inc." part just gets silently discarded.
NOTE: Edited to format commands as code, to fix escaping issues.
Thanks,
Chris

Thank you for your report. I can confirm that I am able to reproduce the issue reported.
As you say, the problem is that the string encoded into the CSR is not escaped. This causes a failure in the sample application at programs/x509/cert_write.c because the non-escaped string is used to craft the new certificate. We will further investigate the problem and issue an appropriate fix.

ARM Internal Ref: IOTSSL-1175


Uncaught (in promise) TypeError: getIP() is undefined thrown in risk.1.0.0.js

Describe the bug
In the dev console, following error is thrown inside risk.1.0.0.js:
  // ...

  function getIP() {
    try {
      // will throw cors error
      return window.parent.fetch(checkoutShopperUrl + "riskapp/v1/risk/ip", { signal: AbortSignal.timeout(750) })
                   .then(response => {
                     if (response.ok) {
                       return response.json();
                     }
                   })
                   .then(responseJson => {
                     return responseJson['ip'];
                   });
    } catch (e) {
    }
  }

  var doAllFingerPrints = async function(){
    // ...
    await getIP().then(result => { // Error is thrown here. window.parent.fetch throws cors error, catch is empty, therefor getIP() is undefined
      ip = result;
    });
    // ...

await window.parent.fetch('https://checkoutshopper-test.adyen.com/checkoutshopper/riskapp/v1/risk/ip')

// Uncaught (in promise) DOMException: Permission denied to access property "fetch" on cross-origin object

How to get rid of this error? Is window.parent possible?
Screenshots
If applicable, add screenshots to help explain your problem.

Using "@adyen/adyen-web": "^5.53.2",
Additional context
Add any other context about the problem here.

@MarkusJLechner - thanks for alerting us. This issue was caused by a feature added by another team. This feature has since been rolled back so this problem will no longer occur


Fix indexed column length and quotes in comments.

When using a MySQL (or MariaDB) server that does not have innodb_large_prefix enabled, indexed columns are limited to 191 characters when using utf8mb4.
Thus, when executing the query to create the user table, the following error occurs:
(HY000/1709): Index column size too large. The maximum column size is 767 bytes.

When importing the structure using mysql command or phpMyAdmin, all affected columns get set to COLLATE utf8mb4_unicode_520_ci on import, which prevents the error from popping up.
Since all columns at the moment are just set to CHAR(255), I think it's better to adjust the column sizes to something more realistic.
For now, the user.username column is the only one affected.
Read more about InnoDB limitations.

Guys?


Bad interaction between godot-vrm and vsk-many-bone-ik-4.3

Tested versions

failed 9ebed55048a5091a3b68e9c15d726b322f1c9646
failed https://github.com/godotengine/godot/compare/655e93d5846b2ef8ebb7d22c8878f51b8f22b312...V-Sekai:godot:vsk-many-bone-ik-4.3
worked 655e93d5846b2ef8ebb7d22c8878f51b8f22b312

System information
Windows 11, Nvidia 3090, Vulkan forward+
Issue description
Solving for spring bone fails with an ik based on ManyBoneIK3D
https://github.com/V-Sekai/godot/assets/32321/e374b369-a4d6-4120-bdab-e904ac92fb53
Steps to reproduce

Compiled branch
Install godot vrm
Go to any skeleton add ManyBoneIK3D
Configure many bone ik with res://many_bone_ik/configure_ik.gd editor script.
When godot-vrm is solving for springbones it fails

Minimal reproduction project (MRP)
fail-game-project.zip

This is caused from using set_bone_pose_override with only a subset of bones.
It seems that since SkeletonModifier3D, set_bone_pose_global_override appears to be no longer recursive when skeleton modifiers are in use.
Godot currently lacks a proper API, but I noticed that  @TokageItLab recommended in one of the comments using the modification_processed signal, and I can confirm this works as a workaround for now.
	if ClassDB.class_exists(&"SkeletonModifier3D"):
		if internal_modifier_node != null:
			if internal_modifier_node.get_parent() != null:
				internal_modifier_node.get_parent().remove_child(internal_modifier_node)
			internal_modifier_node.queue_free()
		internal_modifier_node = SkeletonIK3D.new()
		internal_modifier_node.name = "VRM_internal_skeleton_modifier"
		skel.add_child(internal_modifier_node, false, Node.INTERNAL_MODE_BACK)
		internal_modifier_node.connect(&"modification_processed", self._on_secondary_process_modification_processed)

When I combine the above with calling set_bone_pose_global instead of set_bone_psoe_global_override, the bone poses seem to be incorrect, so I need to debug further.
I think would rather only call set_bone_pose_rotation in 4.3.