Connection pool exhausted

I met similar problem these days online  used  of sub/pub.
org.redisson.client.RedisConnectionException: Connection pool exhausted!  Disconnected hosts: [<IP_ADDRESS>/<IP_ADDRESS>:7091]
at org.redisson.connection.pool.ConnectionPool.get(ConnectionPool.java:169) ~[redisson-2.2.17.jar:na]
at org.redisson.connection.pool.MasterConnectionPool.get(MasterConnectionPool.java:25) ~[redisson-2.2.17.jar:na]
at org.redisson.connection.MasterSlaveEntry.connectionWriteOp(MasterSlaveEntry.java:371) ~[redisson-2.2.17.jar:na]
at org.redisson.connection.MasterSlaveConnectionManager.connectionWriteOp(MasterSlaveConnectionManager.java:600) ~[redisson-2.2.17.jar:na]
at org.redisson.command.CommandAsyncService.async(CommandAsyncService.java:401) ~[redisson-2.2.17.jar:na]
at org.redisson.command.CommandAsyncService$7.run(CommandAsyncService.java:450) ~[redisson-2.2.17.jar:na]
at io.netty.util.HashedWheelTimer$HashedWheelTimeout.expire(HashedWheelTimer.java:581) ~[netty-common-4.1.0.Final.jar:4.1.0.Final]
at io.netty.util.HashedWheelTimer$HashedWheelBucket.expireTimeouts(HashedWheelTimer.java:655) ~[netty-common-4.1.0.Final.jar:4.1.0.Final]
at io.netty.util.HashedWheelTimer$Worker.run(HashedWheelTimer.java:367) ~[netty-common-4.1.0.Final.jar:4.1.0.Final]
at java.lang.Thread.run(Thread.java:745) [na:1.7.0_65]
The version I use is 2.2.17, I have 2 redis server, one master and one slave, in sentinel mode. It is in test environment so net work and cpu usage is low.
This error appears about every 1 day.

Have you tried to increase connection pool size?

i tried it but no effect .  i distrust connection pool  release  Invalid connection

Invalid connections are always return back to pool. You can switch on log level to trace and see it by yourself.

As I see from your trace problem occurs during write operation (not pub/sub command) on master node. Have you checked connection with <IP_ADDRESS>:7091?

Do you see disconnected due to failedAttempts substring in logs?

I use useSentinelServers.  when  Someone  switching Master slave  , will cause it .
2016-07-14 15:37:55.926 WARN  o.r.c.SentinelConnectionManager.slaveDown:285 slave: <IP_ADDRESS>:7091 has down
2016-07-14 15:37:55.926 WARN  o.r.c.SentinelConnectionManager.slaveDown:285 slave: <IP_ADDRESS>:7091 has down
2016-07-14 15:37:56.084 WARN  o.r.c.SentinelConnectionManager.onNodeDown:272 master: <IP_ADDRESS>:7091 has down
2016-07-14 15:37:56.084 WARN  o.r.c.SentinelConnectionManager.onNodeDown:272 master: <IP_ADDRESS>:7091 has down
2016-07-14 15:37:56.174 WARN  o.r.c.SentinelConnectionManager.slaveDown:285 slave: <IP_ADDRESS>:7091 has down
2016-07-14 15:37:56.174 WARN  o.r.c.SentinelConnectionManager.slaveDown:285 slave: <IP_ADDRESS>:7091 has down

Does <IP_ADDRESS>:7091 slave became a master?

NO  it is  <IP_ADDRESS> -> <IP_ADDRESS>

NO it is <IP_ADDRESS> -> <IP_ADDRESS>

Sorry I didn't understand you. Do you shutdown <IP_ADDRESS> right? and then <IP_ADDRESS> node became a master? How many slaves do you have?

yes  shutdown <IP_ADDRESS> right   and then <IP_ADDRESS> node became a master   I have  one slave

If you don't want to get command execution exceptions during master node migration you can increase retryAttempts setting

I mean it do not auto switch  <IP_ADDRESS> node became a master . always  retryAttempts master <IP_ADDRESS> r

Can't reproduce this bug with 2.2.20 version.
Here is my test:
        Config config = new Config();
        config.useSentinelServers()
                .setMasterName("mymaster")
                .addSentinelAddress("<IP_ADDRESS>:26389", "<IP_ADDRESS>:26379", "<IP_ADDRESS>:26399")
                .setRetryAttempts(100);

        RedissonClient r = Redisson.create(config);
        RTopic<Object> topic = r.getTopic("1234");
        topic.addListener(new MessageListener<Object>() {

            @Override
            public void onMessage(String channel, Object msg) {
                System.out.println("received message: " + msg);
            }
        });

        Thread t = new Thread() {
            public void run() {
                for (int i = 0; i < 100; i++) {
                    r.getTopic("1234").publish("" + i);
                    try {
                        Thread.sleep(1000);
                    } catch (InterruptedException e) {
                        // TODO Auto-generated catch block
                        e.printStackTrace();
                    }
                }
            };
        };
        t.start();

        t.join();
```java

you mean when switch Master slave , I increase retryAttempts setting to switch end . how much retryAttempts it is right

you mean when switch Master slave
Yes.
You can increase retryAttempts either retryInterval . Try retryAttempts=25


Nim golfed implementation

Is it possible to make a pure Nim implementation of Keccak like the ones in http://www.cypherspace.org/rsa/ ?
Source of short hash in Python for reference:
https://www.nayuki.io/res/cryptographic-primitives-in-plain-python/sha3hash.py
https://github.com/XKCP/XKCP/blob/master/Standalone/CompactFIPS202/Python/CompactFIPS202.py

External reference: https://github.com/coruus/keccak-tiny/issues/6

cc @cheatfate

@DonaldTsang, take a look at the digest proc from the hash module in nimcrypto. It can be used like this:
keccak256.digest(bytes)

@zah Maybe a secondary template for lines 147~171 and 173~196 and 198~222, they are all loops of range(24) in Python speak if you understand what I mean


Update code to support heterogenous graphs

Update __preprocess_transition_probs_xn2v() to use indexes instead of strings for keys, per convo with Vida
@vidarmehr I think this is what needs to be done to get heterogenous graph support working (feel free to edit PR or suggest changes)

Yes, Justin! __preprocess_transition_probs_xn2v() needs to be updated. Also, some tests in test_hetnode2vec fail. I have commented them out for now. But, I think there is an issue in get_alias_edge_xn2v(self, src, dst)
The tests that fail are:
test_raw_probs_2 and test_raw_probs_simple_graph_2.
I would appreciate it if you can help me with these issues. Thank you.

Update - I spent some time tonight and:

refactored the tests a lot, to DRY them out and make them a little faster.
renamed things for clarity
uncommented and @skip'd the 2 failing tests

I'll hopefully fix the 2 failing tests tomorrow

@justaddcoffee @pnrobinson
Dear Justin and Peter, I have modified the hetnode2vec.py and tested the probabilities of going from one node to another node. I still use the node names and not their indices. But, I will work on it. I think the changes would be similar to what Justin had done it before. If you agree, I think it is better if we close this pull request. I create another pull request and merge this branch to develop. Please let me know if you have a comment or suggestion. Thank you.

Why is this line failing in travis?
The command "nosetests --with-coverage --cover-package=xn2v --cover-xml" exited with 1.

It seems all of the unit tests are being passed?

Why is this line failing in travis?
The command "nosetests --with-coverage --cover-package=xn2v --cover-xml" exited with 1.

It seems all of the unit tests are being passed?

I found the issue. I fix it quickly.

@pnrobinson All checks have passed. Can I merge it?


Source bashrc after updating it

This would only work after a reboot as we don't source the bashrc file once updating it to include the maven path.

Thanks,
I added it in the latest update.
But I'm not sure that call in source in the bash file works that well though.

You're right,  it won't do anything. My mistake. Maybe just updating the readme file would be better advising linux user's to source the .bashrc after running the installer. It may just be me that struggled!


Removing a Localization in IOS

New Issue Checklist

[x ] Updated fastlane to the latest version
[x ] I read the Contribution Guidelines
[x] I read docs.fastlane.tools
[x ] I searched for existing GitHub issues

Issue Description
The addition of a new localization is rather simply, you just simply create a folder and named it with the corresponding language code. However, is there a way to remove/delete a localization using Fastlane?
Complete output when running fastlane, including the stack trace and command used

You can use: --capture_output as the last commandline argument to get that collected for you

N/A
Environment
Please run fastlane env and copy the output below. This will help us help you :+1:
If you used --capture_output option please remove this block - as it is already included there.
N/A

@pparmaei Just to make sure I understand, are you asking about localized app metadata (titles, description, screenshots, etc) here, or something else? Is there a particular use case you have in mind?
Thanks for writing in!  🚀

@mpirri yes, I am talking about localized app metadata. I just want to know if I am able to remove a localization from fastlane or spaceship. For example, if I have the 'Korean' localization enabled, and want to remove it in the next version update, is it possible to do this with fastlane or spaceship commands? Or is the only way of doing this through iTunes Connect?
Thanks

That is correct, currently there isn't a way to do this in spaceship.  I don't suppose that you might be interested in trying to build this functionality and submitting it as a pull request to fastlane though?  We are always looking for really cool fastlane enhancements from contributors.  We have some pretty helpful docs to help you get started.  Let us know if you might be interested - thanks!


pkg.browser => pkg.main, and update the build path.

R: @philipwalton @addyosmani @gauntface
Fixes #911
While moving from browser to main, I've also taken the opportunity to remove the browser/ subdirectory from build/. I don't think there are any packages that have both browser and non-browser builds, so it doesn't seem necessary.

Changes Unknown when pulling bfebb84757817adeafd2d904179829d65bd29739 on drop-browser-package-field into ** on v3**.


docs(FormApi): Include link to TanStack Store docs beneath FormApi's Store instance

The TanStack Store instance has some useful methods of its own. It may be helpful to provide a link to its docs from here.

Result of this discussion.


Specify license

Hey, I was just considering copying your tests over to pydux but I don't know what license your code is and whether you would allow it.

Hey, thanks for reaching out. Feel free to copy and modify all tests and code. I will upload a license when I get a chance (MIT License).

Added License


query opiration bug $dlka

If there is an error in writing the operation keys such as: $eq $ne etc
It does not return an error. Rather, it responds and considers the standards to be consistent with the documents. It returns all documents.
For example:  filter:{ $ldsa :  23 }  will be considered valid
Therefore, all documents will be considered to comply with this condition

fixed with change api.


⚠ Bofrost.be has degraded performance

In 1565f01, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 3620 ms

Resolved: Bofrost.be performance has improved in 5c88909 after 8 minutes.


Remove generator dependence on globally installed gulp instance (#172)

If you try to generate a theme without a globally installed copy of gulp you'll get Error: spawn gulp ENOENT at the end. We do tell people (in the README) to install gulp globally, but that shouldn't be necessary: it is already a dependency of the generator and we can just use the generator's copy.
Test plan: yo ./packages/generator-liferay-theme without a globally installed copy of gulp and see it work without blowing up.
Closes: https://github.com/liferay/liferay-themes-sdk/issues/172

Just started reviewing :)

Checked with @julien about this one over Slack and got the green light to merge it.


PowerToys not running at startup

Microsoft PowerToys version
0.81.0
Installation method
PowerToys auto-update
Running as admin
No
Area(s) with issue?
General
Steps to reproduce
I have "Run at startup" turned on in Settings.
✔ Expected Behavior
PowerToys should run at startup.
❌ Actual Behavior
PowerToys does not run at startup, with this version, for the first time in several years of use. Windows 11.
Other Software
No response

I would note that PowerToys is not listed in my Startup apps list, neither in Task Manager or in Settings>Apps

I found a fix. I have several utilities that can be used to control startup programs. One of them, Revo Uninstaller Pro, had disabled auto start for Power Toys.  So the fault was on me.


feat: start using semantic release

Description:
This PR introduces Semantic Release, a tool for automated version management and package publishing. It follows the Semantic Versioning specification and automates the versioning process to ensure it's consistent and reliable.
Changes:

Semantic Release Configuration: Added a .releaserc configuration file for Semantic Release. This file specifies the branches to track and the plugins to use for the semantic release process.

GitHub Actions Workflow: Updated the docker-publish.yml workflow file to include a step for running Semantic Release. This step will automatically determine the next version number, generate release notes, and create a GitHub release.

Package.json: Updated the scripts section of the package.json file to include a semantic-release script.

Dependencies: Added semantic-release and related plugins as devDependencies in the package.json file.

By implementing Semantic Release, we aim to streamline our release process, reduce the possibility of human error, and provide meaningful and consistent release notes for our users. This change should not affect the functionality of the application.

:tada: This PR is included in version 1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Fixed some bugs in README.md ;-;

Because in some links there was a header id with capital letters after the #, this don't work and you were therefore not redirected to the method or event.

Fixed a bug where it was not shown how 'boosterCard' was defined in the boostImage() method.

Added www. to links that didn't already have it ._.

On line 3 the bin methodhad the link of the welcomeImage() method, this has now also been solved.

I would approve for rest except the www on domains. Discord never have these exist and it'll redirect you to the original domain. www.discord.com -> discord.com which is already correct at first. You're just making things unclear. The opposite of the Discord domain, NPM uses www if you take a look closely when directing to the NPM site.

I don't use discord.com anywhere here, just checked and all links work except the discord.js.org with the www in front, so I'll remove that and then everything is fine :)

If you fix them just to satisfy your basis, I prefer your own personal fork instead of PR since you're not inputting the original resources (modified) and I assume this PR needs few changes before merging but eh it's merged either way...

If you fix them just to satisfy your basis, I prefer your own personal fork instead of PR since you're not inputting the original resources (modified) and I assume this PR needs few changes before merging but eh it's merged either way...

I just tweaked the readme a bit, it's not like much can go wrong, go cry 😂

Ok, this seems a bit off-topic but decisions are on the owner of this repo. If this repo was mine, I would've declined most PRs for being low effort.

Top development quality.

Top development quality.

SEE I SAID U @Iliannnn  he is in the server using a alt

Top development quality.

Sureeeeee

Top development quality.

SEE I SAID U @Iliannnn  he is in the server using a alt

Iknow 😂

Karim is dead as always

Karim is dead as always

Are you sure about that?

Karim is dead as always

Are you sure about that?

@BIntelligent has more commits than you ._.

ye im intelligent

ohok


[Ubuntu] Use new Hashicorp Releases API

Description
Changing the legacy API endpoint URL to the new endpoint that was announced recently:
ref. https://www.hashicorp.com/blog/announcing-the-hashicorp-releases-api
legacy
curl -s https://checkpoint-api.hashicorp.com/v1/check/terraform | jq -r .current_version

new
curl -s https://api.releases.hashicorp.com/v1/releases/terraform/latest | jq -r .version

Related issue:
Check list

[ ] Related issue / work item is attached
[ ] Tests are written (if applicable)
[ ] Documentation is updated (if applicable)
[ ] Changes are tested and related VM images are successfully generated

Just realized that there's also one for Packer, so updating that script as well. Thanks for rpproving that @al-cheb

Thanks for your review @al-cheb
Made some more changes as per your suggestions.

/azp run ubuntu1804, ubuntu2004, ubuntu2204


Hidden row values

Is there an elegant way to assign hidden values to rows or to cells?
For example, a record - roughly speaking: a row - may have a UUID assigned. This  UUID is useful for DB operations, but not something that end users should see.
In Java tables (Swing and JavaFX) the underlying data and view are separate, so it can be directly controlled what to show and how. As for PlutoGrid, I can think of:

custom column type with custom renderer -> not possible, cf. #243
having a user-defined data table in parallel -> this is like re-inventing the wheel
hide column -> this may work

Is there an elegant way to handle this task?

Okay, it seems PlutoRow has a key member that I can use.
PlutoRow(
    key: ValueKey<String>(myRecord.uuid),
    cells: { ... }
)

Using the key as above may be a solution for single, unique properties, such as UUIDs.


[hotfix][ci] Sync CI setup & Reuse workflow

Migrates this repo to the common CI setup and newly reusable workflow. AFAICT the main change is not having 2 separate steps for normal/e2e tests.
Please let me know if that's a significant problem for you.

The test failure is likely caused by an actual packaging problem, that was previously masked because you were using Maven 3.8.5 which has known issues w.r.t. dependency propagation.
Actually, you probably shouldn't be using Maven 3.3+ due to the usual shading issues yet unless you cherry-pick the relevant changes from https://github.com/apache/flink/pull/21349.


Delta's return inversed value

val tile = Tile(100, 100)
val other = Tile(105, 106)
val delta = tile.delta(other)
assertEquals(Delta(5, 6), delta)

The delta from tile1 -> tile2 is returning the negative delta instead of the positive. That is to say x1-x2 not x2-x1.
Will need to flip all existing usages

There's no reason it has to be one way or another, might it make more sense the other way though? Hard to say


Issue 5428: Allow JRE testing for other testsuite aside from JCK

JAVA_TO_TEST definition moved to allow for JRE testing of other than just JCK testsuite
https://github.com/adoptium/aqa-tests/issues/5428

abandoned approach


Make fact set key stringification opt-in for now

This will allow us to push out a bugfix/non-breaking release and release this change in a way that will allow users to migrate over without immediately breaking their tests.
/cc #91

Coverage increased (+0.01%) to 99.254% when pulling 25634f4481f20f2fc7444e867928ca607234e33e on rodjek:opt_in_string_keys into 1a02b7d24609f8a4a69ee05cabf88d4348a2ff90 on mcanevet:master.

Coverage increased (+0.01%) to 99.254% when pulling 25634f4481f20f2fc7444e867928ca607234e33e on rodjek:opt_in_string_keys into 1a02b7d24609f8a4a69ee05cabf88d4348a2ff90 on mcanevet:master.


PRAGMA table_info returns Data type error on node.js environment

Hello,
I tried executing PRAGMA table_info using the node.js environment and it returned a Data type error. Other commands seem to work fine.
Code used (file main.js):
var duckdb = require('duckdb');

var db = new duckdb.Database('database.duckdb');
var con = db.connect();

var n_rows = 20;

// Create a new table
console.log('Creating table.')
con.run('CREATE TABLE a (i INTEGER, j INTEGER)');
var stmt = con.prepare('INSERT INTO a VALUES (?, ?)');
for (var i = 0; i < n_rows; i++) {
  stmt.run(i, i*10);
}
stmt.finalize();

console.log('Table created, now getting its info.')
// Iterate over the tables and get their info
con.all('PRAGMA show_tables;', function(err, res){
  if (err){
    throw err;
  }
  res.forEach(e => {
    console.log(e['name'])
    con.all(`PRAGMA table_info(${e['name']})`, function(err, res){
      if (err){
        throw err;
      }
      console.log(res);
    });
  });
})

Command executed:
node main.js

Output:
Creating table.
Table created, now getting its info.
a
undefined:0
Error: Data type is not supported BOOLEAN

PRAGMA table_info seems to work fine when the table is not present. But, it returns an error with a table that is present.
I searched the issues page but couldn't find anything related. Is this a bug (or known issue), or is it a problem somewhere on my side?

Indeed we are probably missing boolean support in the nodejs connector. Happy to review a PR!

Indeed we are probably missing boolean support in the nodejs connector. Happy to review a PR!

Good to know, I am going to fix it in the near future.

Good to know, I am going to fix it in the near future.

Thanks, see here: https://github.com/cwida/duckdb/blob/master/tools/nodejs/src/statement.cpp#L139

Thanks, see here: https://github.com/cwida/duckdb/blob/master/tools/nodejs/src/statement.cpp#L139

Thanks for the pointer.
How do I compile the changes I make on the statement.cpp file?
Here is what I tried:

Running make from the root of DuckDB, but it didn't create a node package (at least not that I could find).
I also tried looking for NodeJS flags on the root's Makefile.
Running make from within tools/nodejs, but the command failed (example of error ERROR:root:code for hash sha512 was not found).

Thanks for the pointer.
How do I compile the changes I make on the statement.cpp file?
Here is what I tried:

Running make from the root of DuckDB, but it didn't create a node package (at least not that I could find).
I also tried looking for NodeJS flags on the root's Makefile.
Running make from within tools/nodejs, but the command failed (example of error ERROR:root:code for hash sha512 was not found).

Check out our CI setup here, that is "sort of" documentation on how we set up these things ourselves :)
If that still doesn't work feel free to ask again.

Check out our CI setup here, that is "sort of" documentation on how we set up these things ourselves :)
If that still doesn't work feel free to ask again.

From file you mentioned, I tried the following command:
./scripts/node_build.sh 15

(I am not sure about that 15).
Initially it would fail compilation because some unique_ptrs didn't have std:: in front of them. So I just added it.
Now, it fails during compilation because it cannot find the module utf8proc_wrapper.hpp
../src/duckdb.cpp:62605:10: fatal error: 'utf8proc_wrapper.hpp' file not found

I am not sure where I went wrong.

From file you mentioned, I tried the following command:
./scripts/node_build.sh 15

(I am not sure about that 15).
Initially it would fail compilation because some unique_ptrs didn't have std:: in front of them. So I just added it.
Now, it fails during compilation because it cannot find the module utf8proc_wrapper.hpp
../src/duckdb.cpp:62605:10: fatal error: 'utf8proc_wrapper.hpp' file not found

I am not sure where I went wrong.


Mouse event support

Hello,
Is it possible to set mouse events to the nodes and links ? In normal d3, I use .on
https://github.com/mbostock/d3/wiki/Selections#on
I could think of selecting the elements through d3, and then add this attribute. But was thinking more like a function callback which sets the correct event handler (or even other attributes like style,..). Or is a better way.
thanks for this project.
bsr.

this code gives me a syntax error, has it changed in the meantime?


tests/vbank: add a flag to set conn params

Signed-off-by: zyguan<EMAIL_ADDRESS>What problem does this PR solve?
Cannot specify connection parameters currently.
What is changed and how does it work?
Check List
Add a flag to allow us set connection parameters so that we can enable some new features (like async-commit, clustered index) by system variables, (eg. -conn_params='tidb_enable_async_commit=1&tidb_enable_1pc=1')
Tests

Manual test (add detailed scripts or steps below)

Code changes

Has Go code change

Side effects

None

Related changes

None

Does this PR introduce a user-facing change?:

NONE

LGTM. I just know this way. Then we don't need prepareSql to enable async commit/1PC.

I like this way too.


release code

Hi, Thanks for your very nice work! When do you plan to release the code?

Hello~ Thanks for sharing this great work with the community. When will you finish the instructions for this project?

Code and instructions are updated :)


how to generate more text

hi sir, the number of text is random, how can i generate more text like 10 lines or 20 lines in each image?
thx

@larrydemons  Have u solved it.

Check #86 and #47.

thx


Identity scaffolder: Navigating to localhost:5001/Identity/Manage/Index for the first time does not redirect to Login

Repro:

Run dotnet new razor --auth Individual
Run dotnet aspnet-codegenerator identity
Remove AddIdentity bit from Startup.cs as mentioned in ScaffoldingReadme.txt
Run all the necessary ef migrations
Go to https://localhost:5001/Identity/Manage/Index

Expected:

Redirected to Login page

Actual:
Response saying Unable to load user with ID ''.

@prafullbhosale I think the ScaffoldingReadme.txt has a typo. You shouldn't remove the call to AddIdentity. You just need to remove the call to AddDefaultUI.

We need to update the code that sets up authorization to look like this:
                services.AddMvc()
                    .AddRazorPagesOptions(options =>
                    {
                        options.AllowAreas = true;
                        options.Conventions.AuthorizeAreaFolder("Identity", "/Account/Manage");
                        options.Conventions.AuthorizeAreaPage("Identity", "/Account/Logout");
                    });


fix: problem with endless loading of balances and names

Try out Leather build 3dd7d70 — Extension build, Test report, Storybook, Chromatic

This pr fixes "endless" loading of balances and names in only stacks/btc mode, since part of requests are not enabled and useQuery always returns isLoading: true, I changed it to isFetching

isFetching fires more often than isLoading right, as fetching is always new regardless of if it has data.
So doesn't fetching show the loader more often?

@kyranjamie  isn't it what we wanted actually? to show user that we fetch something?

Do we show the shimmer or the skeleton?

isFetching fires more often than isLoading right, as fetching is always new regardless of if it has data.
So doesn't fetching show the loader more often?

hm, wonder if it's critical to show shimmer animation every time we fetching? otherwise we need to add something like
isInitialFetching: query.isLoading && query.fetchStatus !== "idle"

Do we show the shimmer or the skeleton?

for this change we show shimmer

since part of requests are not enabled

can you explain this a little more? Why is part of the request not enabled?
Questioning this just because the changes don't map to my understanding of these props. isLoading = initial load only. isFetching = all refetches.
So, if we have a problem with it showing loading state too often, how does changing it to a property that is true more often, fix this problem?

can you explain this a little more? Why is part of the request not enabled?

e.g. in ledger mode, when we don't have stx address => we set enabled property of stx queries (balance, bns name etc.) to false. in useQuery when enabled is false => it always returns isLoading: true

e.g. in ledger mode, when we don't have stx address => we set enabled property of stx queries (balance, bns name etc.) to false (enabled: !!stxAddress). in useQuery when enabled is false => it always returns isLoading: true

This is actually surprising to me ...shouldn't that disable the query entirely, so isLoading would be false?


Splits track.js errors into multiple apps

Refs #2658.

I like the approach, @viddo, what do you think?

:ok_hand:

Frontend tests were OK :+1: (details)

@javisantana PTAL at this. It's ready to be merged but first we need to setup those keys in production. @viddo can help me with that. Thx

:es:

Frontend tests were OK :+1: (details)


Normalmap has no effect on water shader

Relates to #17

This has been fixed. Forgot to set the size uniform


power launcher uses discrete gpu

Environment
Windows build number: Microsoft Windows [Version 10.0.18363.836]
PowerToys version: 0.18.1
PowerToy module for which you are reporting the bug (if applicable): launcher

Steps to reproduce
have powertoys running
Expected behavior
not to use the discrete gpu (or at least have an option to not use it)
Actual behavior
nvidia gpu activity monitor shows powerlauncher.exe is using discrete gpu
Screenshots

also, this should only be touching the GPU if the window is visible.

What is the user scenario to justify adding such a setting?

@JaspalX how do you disable this for other apps that don't have this option

TBH is this a big drama? probably not, that said it would be a nice to have...I obsess over battery consumption when not plugged in!
I certainly see it reported in the nvidia gpu monitor regardless of whether I can see the window or not.
There's an option in (some) other apps to disable hardware rendering, which seems to do the trick - chrome has it, mailbird.

@JaspalX
do you have actual numbers to show that the GPU usage is effecting the battery?
Hardware rendering is supposed to use less power than software rendering.

I'm afraid not and I'm not sure how I'd be able to spot the difference as there's always something else going to happen on the pc. TBF it is my belief (TM) that it would use more power...I'm happy to be persuaded otherwise!

Actually I don't have to persuade you, if you find evidence that it consumes more power we'll be happy to consider adding the option, otherwise it would be pointless to do the work without knowing its effectiveness.
Closing the issue for now.

I have a laptop that has the option of turning off the dGPU and when I do that, I have to restart FancyZones and it's a bit annoying to do every time I unplug my laptop. An option to only use PowerToys with the iGPU would solve this and I think this is a valid use case for this feature.

An option to disable unused features would be awesome, for example I'm using it just for remapping keys, and there is this color picker UI thing that is running on GPU. (Using 96W USB-C charger for a laptop with 130W TGP due to some special circumstances , I found this app while trying to ensure nothing is using the GPU, not an expert of hardware and power delivery, just wanted to share the findings)

@tilwinjoy  Apps use GPU depending on their framework to render, if you don't want the features to be used, you can go into the PowerToys settings app turn both those applications off and they won't start.

I was able to solve this issue by going into NVIDIA control panel and setting specific apps (FancyZones, AlwaysOnTop) to use the integrated graphics.


Documentation is misleading

On this page
https://learn.microsoft.com/en-us/azure/aks/operator-best-practices-identity#use-pod-managed-identities
You read to read almost 30 lines before having the notation specifying that the pod identity is deprecated.
Secondly, workload identity does not have a chapter, it's hidden among notes of pod identity
Why not :

removing the pod identity documentation since it's deprecated for a long time or at least, adding the warning at the beginning
adding a clear note that the recommandation is workload identity and that we have a migration path

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

ID: 177c5ca2-6358-a4d9-63b4-71d9b21472e9
Version Independent ID: 115868ae-93cb-3bea-eea8-4de3fe5ce39a
Content: Best practices for managing identity - Azure Kubernetes Service
Content Source: articles/aks/operator-best-practices-identity.md
Service: azure-kubernetes-service
GitHub Login: @MGoedtel
Microsoft Alias: magoedte

#reassign:schaffererin

@lgmorand
Thanks for bringing this to our attention.
I'm going to assign this to the document author so they can take a look at it accordingly.

Hi, @lgmorand! Thank you for your feedback.
I just signed off on an update to the article, adding a clearer warning of the pod-managed identity deprecation and workload identity recommendation. You can see my update in this PR. The changes to the article will be live in the AKS documentation soon. I will now close this GitHub issue.
Thanks again :)
#please-close


input_player_connected not working as expected

When connecting a new gamepad, my game uses input_source_set to assign it to the lowest unconnected player, and it seems to work, except that when attempting to refreshing my game's objects, the information they receive about who's connected doesn't reflect this.
As you can see, the console tells me 'Input: Gamepad 0 connected', and even tells me in player 1's sources that gamepad 0 is in fact connected to player 1. Yet when input_player_connected(0) is called, it returns false. I can't use input_player_get_gamepad for this because it doesn't tell if the player is actually connected, only that their last assigned source was a gamepad. input_players_get_status also fails to reflect changes, as even after a gamepad has been added or removed, its variable __any_changed is always 0.
The gamepad works, and functions such as input_check(<verb>, 0) work, but in order for my game to work, it needs a consistent way of gathering gamepad information when it changes, not just when a verb is triggered.

The solution I've settled on for this specific issue is to treat the array length of input_source_get_array as a boolean for whether or not the player is connected.

Can you please share a (minimal) code example to demonstrate the issue ?


[DRAFT] Sample Statistics sqlite version

What changes were proposed in this pull request?
a sqlite version of sample statistics
for design review and verify
How was this patch tested?
no test

@linhong-intel need some comments or a design doc to help us understand the sample algorithm

paste some result here, seems this PR is a better statistics. Full result can be found in https://travis-ci.org/Intel-bigdata/OAP/builds/398138589#L4711
interval	real	sqlite	sample	partbyvalue
(-12,185) 	1000	 925	  50	1000
(-96,154] 	1000	 925	  50	1000
(48,78)   	 315	 318	  19	 428
[-37,117] 	1000	 925	  50	1000
[-8,62)   	 615	 598	  28	 715
(-68,76)  	 764	 747	  37	 715
[-41,-26] 	   0	   2	   0	   0
[-98,137] 	1000	 925	  50	1000
(-100,6]  	  72	  40	   2	 110
(20,89]   	 705	 709	  42	1004
[15,160)  	 855	 812	  45	1092
[77,92)   	 163	 168	   8	 417
(0,157)   	 991	 925	  49	1000
[76,155]  	 236	 178	  15	 505
(58,129)  	 413	 370	  16	 713
(-100,51) 	 488	 453	  22	 499

Seems great! @linhong-intel Probably we can disable the last 2 statistics in near future

@gczsjdy updated the result. I made a mistake on SampleBasedStat

Got it @linhong-intel Seems in the random(non-skewed) condition, in a few cases we don't do as good as the last 2, is this expected?
I will deep dive into this code and see

Close this. please refer #818 for the formal PR.


[Server] Late model Kubernetes components are missing namespace property

Current Behavior
Neither the Kubernetes model v1.31 Deployment nor the Pod (as examples) are recognized as namespaced components. This is a bug in the code for generating later versions of Kubernetes components.
Environment

Meshery Server Version: stable-v0.7.117

Contributor Guides and Handbook

🛠 Meshery Build & Release Strategy
📚 Instructions for contributing to documentation

Meshery documentation site and source

🎨 Wireframes and designs for Meshery UI in Figma (open invite)
🙋🏾🙋🏼 Questions: Discussion Forum and Community Slack

can i work on this issue ?

@Jougan-0 fixed this, I believe.

@Jougan-0 fixed this, I believe.

Good eye, @winkletinkle. Yes, @Jougan-0 is getting it done.


Revert polygon in Inspector in NavigationPolygonInstance doesn't delete this polygon

Godot version:
3.1 3e128a6
OS/device including version:
Linux Mint 19
Issue description:
When I try revert polygon from navigationpolygonInstance(2d), thenin inspector I see that this polygon was removed but in 2d viewport I still see it but I can't delete it and also show this error:

ERROR: _navpoly_unlink: Condition ' !nm.linked ' is true.
At: scene/2d/navigation2d.cpp:144.

Steps to reproduce:
I also try to revert collision shape and that works so removing polygon should do the same
https://streamable.com/2a1kw

Works in 3.2, but throws
 servers/visual/visual_server_canvas.cpp:760 - Condition ' pointcount < 3 ' is true.

Fixed in 3.2.4 beta3
Although a similar issue still exists: #44191


Install failed at the<EMAIL_ADDRESS>install script

Posting this for anyone having the same issue. Since it's a local mac install, the dependency issues may not apply to other setups.
My Setup

Mac OS X 10.13.5
Xcode 9.4
Git 2.15.1
Node 10.3.0

Steps
Followed Mac OS X installation instructions in INSTALL.md (using Homebrew):
brew install pkg-config cairo pango libpng jpeg giflib ffmpeg
git clone https://github.com/nypublicradio/audiogram.git
cd audiogram
npm install

Received 15 warnings and 7 errors. Errors were for 'TryCatch' and 'HasOwnProperty' in /src/Canvas.cc:
error: no matching constructor for initialization of 'v8::TryCatch'
error: no matching member function for call to 'HasOwnProperty'

Upgraded to Canvas 2.0.0-alpha.12
npm install canvas@next
That fixed the install errors, but when running npm start and trying to generate mp4, rec'd this error:
TypeError: Canvas is not a constructor
    at drawFrames (/Applications/audiogram/audiogram/draw-frames.js:12:19)
    at /Applications/audiogram/audiogram/index.js:112:5
    at initializeCanvas (/Applications/audiogram/audiogram/initialize-canvas.js:12:12)
    at Audiogram.drawFrames (/Applications/audiogram/audiogram/index.js:104:3)
    at start (/Applications/audiogram/node_modules/d3-queue/build/d3-queue.js:74:11)
    at poke (/Applications/audiogram/node_modules/d3-queue/build/d3-queue.js:58:11)
    at /Applications/audiogram/node_modules/d3-queue/build/d3-queue.js:90:23
    at /Applications/audiogram/audiogram/index.js:61:14
    at Stream.<anonymous> (/Applications/audiogram/audiogram/waveform.js:27:12)
    at Stream.emit (events.js:182:13)

A Fix
Found this issue which identifies new function: Canvas.createCanvas()
In /audiogram/audiogram/draw-frames.js, changed line 12 from
canvases.push(new Canvas(options.width, options.height));
to
canvases.push(new Canvas.createCanvas(options.width, options.height));
Not sure if the code change above should be added as a pull request since it involves Canvas 2.x - maybe a version check could switch between the two?

I created a github account just to say: thank you!

Hi - could someone from WNYC update the instructions to include this:
npm install canvas@next
The default instructions failed on FreeBSD, so I went chasing around for that, then the OS-X instructions failed on OS-X so I installed Ubuntu in a VM and again, similar errors during the Canvas build.
After a few hours, I landed on this issue and I now have this running on all 3 OSes. If this info were in the README, it would have been super helpful!


Move the documentation over to Read The Docs

Describe the Task
Move the documentation over to Read The Docs
Time Estimate
1 to 3 days of work
Sub-Issues
Consider breaking the task down into sub-issues.

[ ] Add a checkbox for each sub-issue here.

Relevant Deadlines
ASAP
Funding Source
Define the Metadata
Assignee

[x] Select engineer(s) or no engineer required
[ ] Select scientist(s) or no scientist required

Labels

[x] Select component(s)
[x] Select priority
[x] Select requestor(s)

Projects and Milestone

[ ] Review projects and select relevant Repository and Organization ones or add "alert:NEED PROJECT ASSIGNMENT" label (NO REPOSITORY LISTED TO CHECK)
[x] Select milestone to next major version milestone or "Future Versions"

Define Related Issue(s)
Consider the impact to the other METplus components.

[x] METplus, MET, METdatadb, METviewer, METexpress, METcalcpy, METplotpy

Task Checklist
See the METplus Workflow for details.

[ ] Complete the issue definition above, including the Time Estimate and Funding Source.
[ ] Fork this repository or create a branch of development.
Branch name: feature_<Issue Number>_<Description>
[ ] Complete the development and test your changes.
[ ] Add/update log messages for easier debugging.
[ ] Add/update unit tests.
[ ] Add/update documentation.
[ ] Push local changes to GitHub.
[ ] Submit a pull request to merge into development.
Pull request: feature <Issue Number> <Description>
[ ] Define the pull request metadata, as permissions allow.
Select: Reviewer(s), Project(s), Milestone, and Linked issues
[ ] Iterate until the reviewer(s) accept and merge your changes.
[ ] Delete your fork or branch.
[ ] Close this issue.

The work has been done on this issue, a pull request was submitted and approved, and the code was merged.  This issue can be closed.


Inline Menus

Table cells provide inline menus for selecting from a list of things.

resolved in version 0.8.2


Fix recursive link resulted flutter doctor stucking.

Fix issues like : #25885 and #25870

Thanks! This fix looks reasonable; @gspencergoog may be a better reviewer here.

LG but please add a test. See android_studio_test.dart for examples of a mock file system being created in memory. You can use fs.link to make a recursive symlink to test.

@xster
I don't think it's proper to use the LocalFileSystem for testing.
However, the MemoryFileSystem way also don't work.
@tvolkert Please have a look at this issue and https://github.com/google/file.dart/issues/118


bundle update causes app to begin to fail

Thanks for creating an issue! Please fill out this form so we can be sure to have all the information we need, and to minimize back and forth.

What are we trying to do?

Keep everything running smoothly!

What feature or behavior is this required for?

being able to safely run bundle update

How could we solve this issue? (Not knowing is okay!)

I'm very suspicious of the platform-api line in our Gemfile @colinxfleming says its a legacy gem from before Heroku gave us a cert, but somewhat hard to extract from our system.

Anything else?

Reproduce by checking out master, running rake test (tests should start, cancel out), run bundle update (tests will fail to run).

After I commented out platform-api, faker began throwing an error. Seems to be a problem for others as well

faker released an update rolling back their error.
platform-api is still causing trouble. Commenting it out of the Gemfile & running bundle update allows the server to start again, but we should probably extract the gem more carefully than that.

platform-api is being shot into space via #1137 and #958 , which I hope to have into production tomorrow. (I think Heroku has a much smoother process around this now.) tough break for faker, yikes.
There are two open 'update a bunch of dependencies' PRs ( #1122 and #1123 ), so that hopefully fixes a good part of this. @lomky what say we call this good when we get our gemnasium badge to change from update! to green?

Platform-api is gone. One down one to go.


Improve GetEntryText

Right now it can be fooled if the code contains "\n> Output:" or "\n> Solution:"

resolved here: 5ab6b73bb99b7572b6e8ffd6b3d0f86c7d3612ef


Flexible data slicing

Enable flexible data slicing mechanism, based on all of the following fields in state:

isManualSegmentation: whether to apply manual (filter-based) or automatic (kmeans) data slicing
baseCols: use which columns to slice (either through creating filters for these columns, or through inputting them to kmeans clustering)
nClusters: number of clusters to use in automatic slicing (only applicable to automatic slicing)
segmentFilters: filter logic corresponding to data segment (only applicable to manual slicing)
segmentGroups: which segments to group together for comparing against each other

Enable automatic updating all fields mentioned above once one of the fields gets changed (usually this happens when user triggers an action to update one of the fields)

States form a dependency chain: {data, isManualSegmentation} -> {baseCols} -> {nClusters, segmentFilters} -> {segmentGroups}.
Any change in upstream states could cause changes in downstream states (e.g. change in data will result in change in baseCols, if the current value of baseCols becomes invalid given the current value of data
Downstream states change won't affect upstream states (e.g changing segmentGroups won't cause nClusters to become invalid
States update logic: If a state A changes, for one its downstream states B, check if it has become invalid, if so, set it to default based on all other current upstream states; if still valid, do not change the value; change in B could result in further changes in C, so we need to recurse on all downstream states.
Created a helper function validateAndSetDefaultStatesConfigurator to execute the logic above.

Please clarify a thing. Where should the validateAndSetDefaultStatesConfigurator function to be used. Since this function modifies the state, I suppose it shouldn't be used inside a reducer, but rather inside the components which modify the related states (isManualSegmentation, etc), e.g. via triggering an action insidecomponentDidUpdate. If this is the case, you must prepare for the complexity that this type of pattern introduces.

validateAndSetDefaultStatesConfigurator doesn't mutate state, and should be used in reducer. In particular, it should be used in action handlers such as handleUpdateFieldA, where updating fieldA would cause other fields in current state state.fieldB state.fieldC to be invalidated:
const handleUpdateFieldA = (state, action) => {
    return validateAndSetDefaultStates({
        ...state,
        fieldA: action.payload,
    });
}

re-open


workers not showing up on dashboard

I'm using dask-jobqueue with Slurm on the USGS "Yeti" HPC system, and while my cluster is working:
https://gist.github.com/rsignell-usgs/a40a47f87a48534d3f678f5ddfc9018c
with workers connecting (cell [14] ),  I'm not getting workers showing up in the dashboard:

I'm connecting with:
ssh -N -L 8877:igskahcmgslih04.cr.usgs.gov:8877 -L 8878:igskahcmgslih04.cr.usgs.gov:8787<EMAIL_ADDRESS>
How can I figure out what I'm doing wrong?

@mrocklin oh man.   Yep.   Thank you.


when i run my root config in the browser after buildinig the rooot config it showing cannot use import statement outside a module .

I started by typing create-single-spa --moduleType root-config . and i created the root config boiler plate code that is autmotically generated by the single spa . When i started the file it showing the error . when i click that error it showing the js file of the root config in the dist folder. the dist folder has the import statement int the first file it showing the error there . I will attach my git hub link of that file . Can you please check it and give me a suggestio

n . Its almost 4 days im workling on it to fix the issues.https://github.com/donexfience/MF_Testing_debug_purpose

I am also facing the same issue from today mrng,  have you fixed this issue bro ?

+1
facing same issue

Facing same issue

Facing same issue

you can downgrade the single spa version and try to install the root config
Uninstall the v5 of the cli 'create-single-spa' and install v4.1.6 and regenerate your root config
I think they already fixed the issue because he posted on issue they fixed it have a new realease of root config generator

Local root config worked for me today. This was likely fixed in https://github.com/single-spa/create-single-spa/releases/tag/generator-single-spa%405.0.1


How can I run the project without using Apex?

The Git homepage mentions that the project can be run without using Apex.
#set enable_layernorm_kernel=False in config to avoid using Apex
What is the specific command for this?
I added --enable_layernorm_kernel=False after the inference command, but it reported an error that the parameter was unrecognized.
I also modified the 'enable_layernorm_kernel': False under the script opensora/configs/opensora/inference/16x256x256.py, but it still doesn't work.

Hi, you need to navigate to the file "configs/opensora-v1-1/train/stage2.py" and make the following modifications:
model = dict(
type="STDiT2-XL/2",
from_pretrained=None,
input_sq_size=512,  # Note: the pretrained model is trained on 512x512
qk_norm=True,
enable_flashattn=True,
enable_layernorm_kernel=False,
)

That's correct. You need to manually modify the config file.


fix: missing NFTs in NFT tab

Description

Pull Request Type

[ ] :bug: Bug fix (Non-breaking Change: Fixes an issue)
[ ] :hammer_and_wrench: Chore (Non-breaking Change: Doc updates, pkg upgrades, typos, etc..)
[ ] :nail_care: New Feature (Breaking/Non-breaking Change)

Issue (if applicable)

closes #
Risk

High Risk PRs Require 2 approvals

What protocols, transaction types or contract interactions might be affected by this PR?

Testing

Engineering

Operations

Screenshots (if applicable)

Current dependencies on/for this PR:

develop

PR #6122   👈

This stack of pull requests is managed by Graphite.


bug fix of room initialization and invitation popup from crashing app

#in ButtonClicker
#commit 1: roomId need to initialize in onConnectedToRoom(Room room) as this callback will surely call in all players app rather than in onRoomCreated(int statusCode, Room room) which guarantees to call only in the player who initiate play invitation or room creation for other players.
#commit 2: To avoid crash of application from ignored invitation request like closing application just when invitation pop up (edge case appeared in some use case)

I signed it!

Thanks! a couple minor comments.

hey done


log determinant of HMatrix

A question please: how do I go about calculating the log-determinant of an HMatrix? Thanks.

It seems you can use the Cholesky decomposition:
\log(det(H)) = \log(det(L L^\top)) = \log(det(L)^2) = 2 \log(det(L))

Do you have a MWE example? You can call cholesky to factor an HMatrix, but you may need to tweak a bit with the internals to extract the diagonal of L.
Disclaimer: I know nothing about log determinants 😅


QR code not fully visible on widescreen [ android TV/nexus player ]

ABCore is fully working and syncing on google nexus player.
There's a glitch showing the QR code in Dashboard: the qr code doesn't fully fit the screen on my widescreen TV.

I'm running official Android 8 firmware.

the bug is also replicable using an android tv emulated device (AVD)
Type: Android TV (1080p)
Resolution is: 1920 x 1080

The screen should be scrollable, do you have any navigation command on Android TV that does that?

confirmed, the scrollview/scrollbar is not visible but working with a remote


Provide a counterpart to .run() usable for unit tests

It would be nice to be able to write unit tests for the argument parser - e.g. that certain combinations of arguments are accepted or rejected, that the custom code in .map() or .parse() is correct, or when reimplementing/mimicking an existing interface with bpaf and keeping the tests from the previous ones to avoid API breaks.
Right now parsing can only be performed with .run(), which gets the data from the command line, and prints to stdout. There doesn't seem to be a way to pass a hardcoded list of arguments or capture the output, which is necessary for unit tests.
This could be solved by providing a function that accepts T: IntoIterator<OsStr> and writes to an arbitrary T: Write, or at least non-generic version of that such as &[OsStr], so that the parser could be unit-tested.

There's some private API to do exactly that, I just made it public: 3af6481e45dbf1e51738aaf42e975ff3bcd591cf. For now you can use git version, I'll make a release in a day or two once I figure out proper documentation for that.
You can find some examples here:
https://github.com/pacak/bpaf/blob/master/src/tests.rs

Pushed some more changes with documentation. Please let me know if it solves your problem. If it is - I'll publish it to crates.io.

Thank you for such a quick implementation!
I have added unit testing to cargo supply-chain in https://github.com/rust-secure-code/cargo-supply-chain/commit/b05efdeeb00f32639a7bfc8c401754aade0b1600, and the API for that looks great to me!
 Perhaps they would be more clear if named .capture_stderr() or some such?
I'm also not fond of panicking if nothing is printed, because it makes it difficult to verify that nothing is printed. Perhaps return an empty string if there is no output?
Both of the capture_* methods could use code examples as well. I think --help and passing an invalid flag would make good examples for stdout and stderr respectively.
Thanks again!

Also, should the unwrap_* methods return String or OsString? I think testing handling of non-UTF-8 arguments could be valuable, and they would not be expressible in a String.

It is not really capturing anything. If parser detects --help or --version
it will always produce only something that can be consumed with
unwrap_stdout. Otherwise it will produce "stderr" generated either by the
parser if someone is missing or by user's guard or parse functions. API for
those is constructed to only produce a String. If you try to print
something inside map or parse - it will not be captured. Depending on a
test case you'll know what to use. Unwrap_stdout if you want to test
generated help or unwrap_stderr if you are testing parse/guard/ missing
parameters. Exact string reperentations are probably not stable between
versions.
In theory it should not be possible to print nothing at all - bpaf should
add at least a field name that failed to parse. Will check on that
tomorrow. I guess I'll add most of this email to docs as well.
On Sun, Mar 13, 2022, 22:37 Shnatsel @.***> wrote:

Thank you for such a quick implementation!
I have added unit testing to cargo supply-chain in
@.***
https://github.com/rust-secure-code/cargo-supply-chain/commit/b05efdeeb00f32639a7bfc8c401754aade0b1600,
and the API for that looks great to me!
 Perhaps they would be more clear if
named .capture_stderr() or some such?
I'm also not fond of panicking if nothing is printed, because it makes it
difficult to verify that nothing is printed. Perhaps return an empty string
if there is no output?
Both of the capture_* methods could use code examples as well. I think
--help and passing an invalid flag would make good examples for stdout
and stderr respectively.
Thanks again!
—
Reply to this email directly, view it on GitHub
https://github.com/pacak/bpaf/issues/2#issuecomment-1066115568, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAQFI6JDO33AWOAKVQ4PBTU7X4T3ANCNFSM5QSQMYIA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you commented.Message ID:
@.***>

Released 0.2.1


Timezone: America/Chicago does not account for DST

Version tested against: microsoft/mssql-server-linux:2017-latest
Steps to reproduce:
1. Run a container, passing in desired Timezone
docker run -e "TZ=America/Chicago" microsoft/mssql-server-linux:2017-latest

2. Connect to running SQL instance
3. Get Date using SELECT GETDATE()

Expected outcome:
Given the current offset for America/Chicago is -5, I would expect SELECT GETDATE() to return UTC time - 5 hours
Example:
UTC time: 12:00:00
expected: 07:00:00

Actual Result:
SELECT GETDATE() is returning the time related to the standard (non DST) offset -6 i.e UTC time - 6 hours
Example:
UTC time: 12:00:00
actual: 06:00:00

Hi,
That's a known issue. One option is install tzdata mannually in your container. Or you can add the following lines to your Dockerfile:
apt-get update
apt-get install tzdata

Please refer to Wrong offset for Europe/Moscow timezone.

SQL Server DateTime data type is not DST aware. Please file a feature ask for the same here: https://feedback.azure.com/forums/908035-sql-server
You can use the timezone offset for DST calculations. Reference: https://docs.microsoft.com/en-us/sql/t-sql/data-types/datetimeoffset-transact-sql?view=sql-server-2017
If you have questions regarding SQL Server database engine like the one here, then please leverage MSDN and StackOverflow.
I will close this thread as this is not a container specific issue.


[Style] Hovering Effect

What's the style idea?
We need to improve the hovering effect  in the navbar on main page.
Checklist

[X] I have checked the existing issues.
[X] I follow Contributing Guidelines of this project.
[X] I am a GSSoC'24 contributor.

Code of Conduct

[X] I agree to follow this project's "Code of Conduct".

I would like to work upon it. Kindly assign it.

I cannot assign it to you, as I'm a contributor.And moreover I raised the issue to work up in it...

#7 already raised the issue on navbar work on it


Hallucination err

Hallucination magic causes the client to crash
client version 6061
server build 3364 / GIT hash 0fc15b5ba49e4a579c9973ce7da57d13220b0d3d

Is the hallucination flag or the hallucination sphere spell?

spell 1005

Ehm try use a most recent build

Compiled at Aug 21 2021 (16:15:29) [build 3468 / GIT hash d69c5220788bd3a09cadfbc7d8990507ef01b74b]
The error still exists
Client crash

Can't reproduce, check if your hallucination spell has the spellflag_tick

have
[SPELL 1005]
DEFNAME=s_Hallucination
NAME=Hallucination
SOUND=snd_Z_END_MYSTERY1
RUNES=WN
CAST_TIME=60
RESOURCES=i_reag_garlic,i_reag_nightshade
RUNE_ITEM=i_rune_MIND_BLAST
SCROLL_ITEM=i_scroll_MIND_BLAST
FLAGS=SPELLFLAG_TARG_CHAR|SPELLFLAG_DIR_ANIM| SPELLFLAG_HARM | SPELLFLAG_FX_TARG| spellflag_curse|spellflag_tick
EFFECT_ID=i_fx_curse
EFFECT=2,5
LAYER=layer_flag_hallucination
DURATION=260.0,460.0
MANAUSE=15
SKILLREQ=MAGERY 30.0, NECROMANCY 40.0
INTERRUPT=100.0,100.0

The crash happens when the spell is cast or when the spell expires? Can you reproduce it in another client?

Successful casting
It is possible to crash when NPC changes
I have three computers that crash

Can't reproduce, try a different client like 7.0.20

I'm not going to upgrade the client version for the time being
Because I changed some functions in the 6061 client

You modified the client?

yes
But it will not affect its operation
Because I use the client without modifying the memory will still crash

I'll try with client 6061

No crash, what's the error in console about the client crashing?

No hint
Client flash back

You modified client?  So its the open source classicuo?

Memory modification
Even if I don't modify it
It will also collapse

But what client you use? Classic, cross, osi, orion?

Classic 6061

https://ia800906.us.archive.org/view_archive.php?archive=/27/items/UltimaOnlineTM9thAnniversary<IP_ADDRESS>Patch72/Ultima Online™ - 9th Anniversary [<IP_ADDRESS> [Patch 72]].iso&file=Ultima Online™ - 9th Anniversary <IP_ADDRESS> [Patch 72].exe
Client crash

yes
<IP_ADDRESS> Normal operation
The client will not crash

<IP_ADDRESS> Will collapse, just for a long time

id = (CREID_TYPE)(Calc_GetRandVal(CREID_SUCCUBUS));
Change here, client crash fixed


Devilbox for android

What is your idea or feature suggestion?
There are 2 good web server apps for Android. Bitwebserver and Ksweb server. Devil box has more features than the above-mentioned apps. It would be great is we ported DevilBox for android.
Benefits

Can be useful for developers to develop on the mobile device itself

Will be more popular and gain users around the globe as Devilbox has more compatibie databases than its competitors

Where can we find information about this?
No response
Are you willing to provide a PR to address this?
No

I would question the benefit to "develop" on a phone where you can barely type. Besides, Devilbox has a pretty high storage and memory footprint relative to what is nowadays available on mobiles.

It would be great is we ported DevilBox for android.

Devilbox uses Docker and Docker Compose and supports amd64 and arm64 architectures only (see requirements). If these work on android, then you might be able to run it, otherwise you will be out of luck.
You can start here: https://gist.github.com/arno01/ebf570af208e28c1a0cf78da4f63bc9c and see if you can get something together, but there is not going to be any official support for this atm.


Update manage-resource-accounts.md

@JohanFreelancer9 Copy editing is needed for this PR. Thanks.

@MikePlumleyMSFT PR has been copyedited and is ready for final review, could you please check and merge? Thanks!


Intrinsics and overflow

fixes #151

Hmm, but I think we are supposed to always mask the shift width...
Compare the Debug and Release mode MIR and LLVM IR here: https://is.gd/LglRp7
Debug mode uses CheckedShl and an assert MIR statement on the overflow result, while Release uses a plain Shl. Both do a shift width mask in the LLVM IR, but only Debug mode can panic (from the assert).
So I think we need to leave the code for Shl alone and change something for CheckedShl?

Oh, I guess this code handles both Shl and CheckedShl. Maybe we just need to adjust the cases where the overflow result is true to make sure the assert is triggered.

In debug mode, the mir shows a concrete assert. In the release mode, we still forward and the masking is done by the forwarded to function. The masking was entirely redundant

Ohhh. In release mode we still use the checked_shl intrinsic and that does the masking?

Huh, TIL the masking is done in std code, not the LLVM backend: https://doc.rust-lang.org/stable/src/core/num/mod.rs.html#1092-1094


Code blocks width fix

Fixed a couple of issues with code sizes
closes #274

It wasn't as easy as I thought 😆 CSS is hard, man!  Now it should be good.

I can confirm this! Man, it took me quite some time to understand how to set the right background theme using highlight.js to match the correct color scheme without having two different background colors overlapping each other. All of this, while for some reason, I wasn't able to see how the highlighted block would look. It was HELL! 😂

Thanks! Appreciate that I didn't have to look into it 😅


fix: revert PR #2

PR #2 is wrong.  egg-jwt/app/extend/application.js is reading this.config.jwt, please read https://github.com/eggjs/egg-jwt/blob/master/app/extend/application.js#L9

看来还是单元测试没有覆盖的问题~


Feat connect issue deploy

Summary

Create DataEnricher and BatchUpdate
Connect issue to deploy

Does this close any open issues?
Closes #2879
Screenshots
Include any relevant screenshots here.
Other Information
Any other information that is important to this PR.

For github/gitlab issues, we can use repoId to connect issues to deployment.
But for jira issues, we have to use repoId and board_repos to connect.
Can we unify these tow ways by also adding board_repo for github/gitlab issues?


Check daily combo profit and delta profit from upgrades

Check daily profit
If worth from collecting cards for a daily combo is less than profit from the combo
Delta profit from upgrades:
Uses variable MIN_CARD_PROFIT_DELTA (default 0). Cards won't be upgrade if delta profit is less than defined in MIN_CARD_PROFIT_DELTA

My fix:

https://github.com/shamhi/HamsterKombatBot/issues/251#issuecomment-2175933781

@ButCheR-PRO good fix. I didn't pay much attention toif common_price < bonus and balance > common_price and is_combo_accessible (it's my first experience with python). But what about delta?


Support for for handling duplicates?

I'm using MyBatis Dynamic SQL with MySQL, and would like to know whether there are any methods to handle duplicates, such that I can do something similar to:
INSERT INTO ...... ON DUPLICATE KEY UPDATE...
or
INSERT IGNORE INTO...
Thanks! :)

The library doesn't support those types of inserts. You could alter the insert statement after it is generated, but it might be simpler to just code the insert statement manually in this case.

Oh that's unfortunate... any chance your team will add that functionality? I do currently code it manually on my xml mapper, so it's a little less elegant of a solution. But hey, it works, so no biggie if that's not in line with the library's plans. I appreciate the response though :)

I'm trying very hard to keep the library vendor neutral - we don't really have the time to support all the different SQL dialects. If something is missing that is relatively standard, then I will consider adding it. I did this recently for limit/offset support because it is widely, but not universally, supported.
I'm not sure about this - does any database other than MySQL support this syntax?
I will also say that the insert statements are the hardest to extend in the current version of the library. I will look into adding some extension points where you might be able to plug in to the code generation cycle and make some changes.

@jeffgbutler Apologies for the resurrection but I was just investigating "upsert" support and found this.
PostgreSQL also supports a very similar syntax using INSERT INTO ... ON CONFLICT (<keys or constraint>) DO UPDATE.
To your point this means the library would have to wade into the vendor specific territory. Even so these statements are great and finding a way to integrate them would be awesome.

@kdubb No worries. I am thinking about some different options to make it easier to do things like this.

@jeffgbutler I hava found the solution for several days. Now I know that it has not been supported yet.
The issue has been opening for such long time, if it possible to know whether there is a solution now?

There is no solution in the library now. As far as I know, there isn't a vendor neutral way to handle this issue. My current thinking to provide some kind of hook to make it easier to modify the generated SQL, but that thought isn't fully formed yet.

There is no solution in the library now. As far as I know, there isn't a vendor neutral way to handle this issue. My current thinking to provide some kind of hook to make it easier to modify the generated SQL, but that thought isn't fully formed yet.

Supporting it would fit well with the concept of saving aggregate roots in the DDD domain, and would require additional SQL save child entities, which I use for now in xml <select>  tag completes saving the aggregation root and storing the child entities in batches, which is not very elegant.

hope i can help u
<insert id="testDebugFunction1" parameterType="UrParameterType">
  INSERT INTO your_table (c1, c2, c3)
  VALUES (#{p1}, #{p2}, #{p3})
  ON DUPLICATE KEY UPDATE column1 = #{p1}, column2 = #{p2}, column3 = #{p3}
</insert>

<insert id="testDebugFunction2" parameterType="UrParameterType">
  INSERT IGNORE INTO your_table (c1, c2, c3)
  VALUES (#{p1}, #{p2}, #{p3})
</insert>

Yes, you can do it in xml, but hopefully using mybatis-dynamic-sql

Saving aggregation roots in an RDBMS is a hassle, and to avoid using JPA annotations, you have to do a lot of things with mybatis, such as implementing the save method:
AggregateRoot save(AggregateRoot aggregateRoot);

<select id="save" parameterType="AggregateRoot" resultMap="aggregateRootResults" flushCache="true">
        INSERT INTO <include refid="aggregateRootTable"/>( <include refid="aggregateRootColumns"/> )
        VALUES ( #{id}, #{prop1}, #{prop2}, #{prop3} )
        ON CONFLICT (id) DO UPDATE SET column1 = #{prop1},
                                       column2 = #{prop2},
                                       column3 = #{prop3}
        RETURNING <include refid="aggregateRootColumns"/>;

        <if test="elementCollection != null and elementCollection.size > 0">
            INSERT INTO <include refid="elementCollectionTable"/> ( <include refid="elementCollectionColumns"/> )
            VALUES
            <foreach collection="elementCollection" item="e" separator=",">
                (#{e.id}, #{id}, #{e.prop1}, #{e.prop2}, #{e.prop3})
            </foreach>
            ON CONFLICT (id) DO UPDATE SET column1 = EXCLUDED.prop1,
                                           column2 = EXCLUDED.prop2,
                                           column3 = EXCLUDED.prop3
            RETURNING <include refid="elementCollectionColumns"/>;
        </if>

        DELETE FROM <include refid="elementCollectionTable"/> WHERE aggregateRootId = #{id}
        <foreach collection="elementCollection" item="e" open="AND" separator="AND">
            id != #{e.id}
        </foreach>;
    </select>

Saving aggregation roots in an RDBMS is a hassle, and to avoid using JPA annotations, you have to do a lot of things with mybatis, such as implementing the save method:
AggregateRoot save(AggregateRoot aggregateRoot);

<select id="save" parameterType="AggregateRoot" resultMap="aggregateRootResults" flushCache="true">
        INSERT INTO <include refid="aggregateRootTable"/>( <include refid="aggregateRootColumns"/> )
        VALUES ( #{id}, #{prop1}, #{prop2}, #{prop3} )
        ON CONFLICT (id) DO UPDATE SET column1 = #{prop1},
                                       column2 = #{prop2},
                                       column3 = #{prop3}
        RETURNING <include refid="aggregateRootColumns"/>;

        <if test="elementCollection != null and elementCollection.size > 0">
            INSERT INTO <include refid="elementCollectionTable"/> ( <include refid="elementCollectionColumns"/> )
            VALUES
            <foreach collection="elementCollection" item="e" separator=",">
                (#{e.id}, #{id}, #{e.prop1}, #{e.prop2}, #{e.prop3})
            </foreach>
            ON CONFLICT (id) DO UPDATE SET column1 = EXCLUDED.prop1,
                                           column2 = EXCLUDED.prop2,
                                           column3 = EXCLUDED.prop3
            RETURNING <include refid="elementCollectionColumns"/>;
        </if>

        DELETE FROM <include refid="elementCollectionTable"/> WHERE aggregateRootId = #{id}
        <foreach collection="elementCollection" item="e" open="AND" separator="AND">
            id != #{e.id}
        </foreach>;
    </select>

Another elegant way is to encapsulate query support for JSON/BSON columns with mybatis-dynamic-sql, aha！


Call for re-structure : 10 cascade locks till db write which is pretty bad.

Lock acquisition improved in #308, #300 and #296.


workspace property in devcontainer.json

Hi! first of all thanks for the extension. I've just found it and it's really cool. However, I think that adding a workspace property directly in devcontainer.json could be a bit conflicting cause it's effectively interferring with the spec.
I would expect the configuration for the extension to be added in the settings of the respective extension separately. What do you think about this?
thanks.

Yep I agree. It should be simple enough to add support for configuring this via settings.json as well. Let me take a look at that this afternoon 👍

I may need to update the readme, but the workspace property can already be added to the .vscode/settings.json file in a workspace. Let me know if you try that and it doesn't work.

I've just tried that but it's not opening the files that I'm specifying there (not even with the top-level workspace property), that's why I created the other issue separately.
Thanks


MAX_STATEMENT_TIME

Please remember to follow the Lasair Pull Request checklist when issuing & reviewing this pull request:
https://lsst-uk.atlassian.net/wiki/spaces/LUSC/pages/2612264961/How+to+do+Pull+Request#STEP-3%3A-Issue-a-Pull-Request-on-Github
CHANGES

This is what the error in Jenkins looks like:
Running in /var/lib/jenkins/workspace/asai_hotfix_max_statement_time/tests/integration/pipeline/filter
[Pipeline] {
[Pipeline] sh
+ python3 test_run_query.py

Running tests...
----------------------------------------------------------------------
EEE
======================================================================
ERROR [0.000s]: test_1_run_query (__main__.RunQueryTest)
Read from the database
----------------------------------------------------------------------
Traceback (most recent call last):
  File "test_run_query.py", line 60, in test_1_run_query
    query_results = run_query(querydict, RunQueryTest.msl)
  File "/var/lib/jenkins/workspace/asai_hotfix_max_statement_time/tests/integration/pipeline/filter/../../../../pipeline/filter/run_active_queries.py", line 180, in run_query
    sqlquery_real = ('SET STATEMENT max_statement_time=%d FOR %s LIMIT %d' % (settings.MAX_STATEMENT_TIME, sqlquery_real, limit))
AttributeError: module 'settings' has no attribute 'MAX_STATEMENT_TIME'
...

Maybe add a MAX_STATEMENT_TIME to tests/unit/pipeline/filter/settings.py?


trying to fix import at docstring

Maybe it fixes #26 Travis failure
At least, here locally it passes!

And now that travis is done we can see that the issue persists :disappointed:
https://travis-ci.org/pyoceans/python-oceans/jobs/54083726#L553
I suspect some connectivity limitation, travis might be trying to save bandwidth and netCDF cannot download the variables.  I will try to make a slimmer example/doctest later.

maybe including such .nc file inside a data folder, until you include the migration to iris dataset.
btw, where I can find the install instructions to travis? .travis.yml I got the list of packages to be installed inside setup.py [only pytest and pytest-cov]
It's a good idea to include the file requirements.txt and dev-requirements.txt? or some instructions for deving proccess inside any document?
Cheers.

maybe including such .nc file inside a data folder

I am reluctant to bloat the repo with binaries. Anyways, oceans is such a small project that we can just live with that :grimacing:

where I can find the install instructions to travis?

.travis.yml this line install everything.

It's a good idea to include the file requirements.txt and dev-requirements.txt?

Yes.

or some instructions for deving proccess inside any document?

Also, yes :grinning:


Deprecated options on Mongodb Drivers

When i try no connect to Atlas Mongodb, the connection was succesful. Although, the following message appeared on terminal:
**

error in authentication man! [Function: error]
(node:9712) [MONGODB DRIVER] Warning: useNewUrlParser is a deprecated option: useNewUrlParser has no effect since Node.js Driver version 4.0.0 and will be removed in the next major version
(Use node --trace-warnings ... to show where the warning was created)
(node:9712) [MONGODB DRIVER] Warning: useUnifiedTopology is a deprecated option: useUnifiedTopology has no effect since Node.js Driver version 4.0.0 and will be removed in the next major version**

The connection coding:
`const { error } = require('console');
const { MongoClient } = require('mongodb');
const uri = "mongodb+srv://kelwinesechiel8:@cluster0.vxfjhgw.mongodb.net/?retryWrites=true&w=majority";
async function run() {
const client = new MongoClient(uri, { useNewUrlParser: true, useUnifiedTopology: true });
try {
await client.connect();
const dbo = client.db('keldb');
const obj = { name:'Kaleb' };
const colecao = 'names';

await dbo.collection(colecao).insertOne(obj);
console.log('succesfully inserted');

} finally {
await client.close();
}
}
run().catch(console.log('error in authentication man!', error));
`
Config:
Node 20.8.1
Mongodb ^6.3.0

Hello!
I am sending you a test example of correct connection to MongoDB via code:
How to use it:

execute the following code once at the application start:
FastReport.Utils.RegisteredObjects.AddConnection(typeof(MongoDBDataConnection));
now you should be able to create a new MongoDB data connection from code:
Report report = new Report();
report.Load(@"YourReport.frx");
//...
MongoDBDataConnection conn = new MongoDBDataConnection();
conn.ConnectionString = "your connection string";
conn.CreateAllTables();
report.Dictionary.Connections.Add(conn);

Best regards,
Bogdan


crail-site.conf configure mistake leads to java arrayindexoutofboundsexception 1

In current version, I find that crail.storage.types replace the crail.datanode.types.
In the conf/crail-site.conf.template, it is like below:
crail.storage.types		com.ibm.crail.datanode.rdma.RdmaDataNode
but in the README.md, it is this
crail.datanode.types                  com.ibm.crail.storage.rdma.RdmaStorageTier
In fact crail.storage.types  com.ibm.crail.storage.rdma.RdmaStorageTier is the right one in current version.
Here com.ibm.crail.storage.rdma.RdmaStorageTier is in the StorageServer.java.
If using com.ibm.crail.datanode.rdma.RdmaDataNode, then storageTierIndex will be 1 which leading to java arrayindexoutofboundsexception at the namenode.
So this STORAGE_TYPES should be consistent in README.me, core-site.conf.template, CrailConstants.java and StorageServer.java.

Thanks for pointing this out. Indeed all docs should be consistent as com.ibm.crail.storage.rdma.RdmaStorageTier is the right property here. We'll fix this with the next stable check in. There was a DiSNI<->Crail version incompatibility from April 5 to April 6 which is fixed now, in case you had problems to compile Crail today.

fixed in e20461d87a345b531118bde3149f85264cc0087e


Specify payload for triggers

Please consider re-open https://github.com/MicrosoftDocs/azure-docs/issues/20971 issue as it was not resolved, yet still actual. Initial author provided all the information on what's wrong.

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

ID: f538a557-7e88-e077-6a6f-6fb14d89dbf6
Version Independent ID: 84a1693b-2795-c865-35df-3ad3b9b61e2c
Content: Manually run a non HTTP-triggered Azure Functions
Content Source: articles/azure-functions/functions-manually-run-non-http.md
Service: azure-functions
GitHub Login: @craigshoemaker
Microsoft Alias: cshoe

Hello @dmytro-hlushchenko-rft Thanks for reaching out to us!.
As the referenced issue is very old, I kindly request you to provide the error details from your side on this issue, so that our SME and team would investigate the issue and get back to you soon.
Please add more details to this new issue!

Unfortunately, this article can't be a reference for the payloads of all triggers. I will make it a point to indicate to where you should look for this information.

#please-close and see the referenced commits for the updates.


Uncaught TypeError: Cannot read property 'currentProvider' of undefined

Uncaught TypeError: Cannot read property 'currentProvider' of undefined
http://artonomous-ethberlin.s3-website-us-east-1.amazonaws.com/

We solved it via Twitter. Metamask wasn't unlocked.


Missing Ingress pathType with k8s >=1.22

Describe the bug

When installing the helm chart on K8s version >= 1.22 with ingress enabled, the path type is missing and gives the following message:
Error: INSTALLATION FAILED: unable to build kubernetes objects from release manifest: error validating "": error validating data: ValidationError(Ingress.spec.rules[0].http.paths[0]): missing required field "pathType" in io.k8s.api.networking.v1.HTTPIngressPath
Set up

** Needs to be done on a k8s cluster version 1.22 or greater.
Enable the ingress and specify a path in values.yaml:
kafka-ui:
  ingress:
    enabled: true
    path: /

Steps to Reproduce
Steps to reproduce the behavior:

helm install chart-name .

Expected behavior

Chart is successfully installed is correctly formatted ingress.

@PhilDulac hey, thanks for reaching out. This has been fixed in master-labeled image, we'll also get a minor release with this change to update the charts on artifacthub.

Still missing pathType if  only kafka-ui.ingress.path  is configured.


fix: umd externals assets error

Summary
releated to: https://github.com/web-infra-dev/rspack/pull/3725/
This error occurred in umd library :

Test Plan

!canary


If the uploaded file for an event is an image, display that image on the Event View page

 If the uploaded file for an event is an image, display that image on the Event View page #902


feat: add the initial tests.md

First approach to the topic of testing the reusable Terraform modules. Go beyond the simplistic "does it pass plan" and "does it pass apply" scenarios.

@damianfedeczko Appreciate your approval, but I am no longer permitted to see the green "Merge" button. :smile:  (I've been removed from the org - as expected.) Care to hit that button for me? :pleading_face:

@damianfedeczko Appreciate your approval, but I am no longer permitted to see the green "Merge" button. smile (I've been removed from the org - as expected.) Care to hit that button for me? pleading_face

How's it going boys!


Server pins

About the PR=
I added server pins
Why / Balance
Pride for the servers you came from.
Technical details
Created new art for the pins and added the entities. Also added them to the misc loadouts.

Isn't this kind of meta?

yeah, i think based on the lore (what should be in the future), these pins is bot quite right

Touching git too late. This was unintentionally sent here.


Map UI not working?

Hi,
it seems that the Routing Map UI is not working. It just points to assets which contains a path where the UI is stored (graphhopper) but this is not going to work like this.
Is there a way how to make it work?
Regards
Stefan

Sorry, I have been serving the assets on my dev machine with Apache for a few weeks. It also acts as reverse proxy to Jetty/Dropwizard. That's why I did not notice that the assets are not working any more. I will have a look into it.
My production systems use the orignal GraphHopper web interface (logo and tile servers in main.js changed). Apache works as reverse proxy with a configuration similar to this:
<VirtualHost *:443>
        DocumentRoot /var/www/routing-webinterface/
        ServerName mycool.hostname.tld
        ProxyPass /route http://<IP_ADDRESS>:8989/route
        ProxyPass /i18n http://<IP_ADDRESS>:8989/i18n
        ProxyPass /info http://<IP_ADDRESS>:8989/info
        ProxyPass /nearest http://<IP_ADDRESS>:8989/nearest
        ProxyPass /match http://<IP_ADDRESS>:8989/match
</VirtualHost>

https://github.com/Nakaner/graphhopper/tree/master/web/src/main/resources/assets is checked out at /var/www/routing-webinterface/

I just tested it and it works even without Apache by simply following the steps in the readme.


Make access request redirect to dashboard if user already has access

With the access request feature, users who have access often revisit the access request page and are still shown the access request page.
This PR checks to make sure the user still doesn't have access before showing the access request page and redirects to the dashboard if they have access.
@john-bodley @mistercrunch

Looks like it's a quirk with Python. I intitally didn't have it but the flake8 rules required me to put the trailing comma and somehow python reads it correctly.
From my interpreter
>>> all((True, True, True),)
True
>>> all((True, True, False))
False
>>> v = 1,
>>> v
(1,)

PING

Yeah I ran similar tests in an interpreter and still I'm confused as to how this works properly. I agree we cannot rely on this odd behavior as it may flap in the future or across python versions. LGTM


Minor improvements

I'm sorry that reviewing is late. :bowing_man:
What will you do timing to merge? It is failing to tests but I will fix this problem later.

Sorry, what do you need from me?

No. I will merge this when it's convenient.


Added board files for ebaz4206 with Zynq7020

Hello,
Recently I purchased EBAZ4205 board from Aliexpress which was sold as EBAZ4205. The label on the PCB is still the EBAZ4205, but the Zynq was upgraded to Zynq702. Thus the seller probably named it 4206.
I modified your board files and tested it with Vivado. They work.
Could you, please add those files for everybody who bought this revision of the EBAZ4205?
Best Regards,
Krasimir Kostadinov


Issue inside the serverside example

The serverside rendering itself is working fine. But there is a problem on the client when the app is started (https://github.com/Raynos/mercury/blob/master/examples/server-rendering/browser.js#L30):
hg.app(null, app, App.render, {
    initialTree: prevTree,
    target: targetElem
});

This code does not seem to work with the current version of mercury as hg.app looks like the following (https://github.com/Raynos/mercury/blob/master/index.js#L109):
function app(elem, observ, render, opts) {
    if (!elem) {
        throw new Error(
            'Element does not exist. ' +
            'Mercury cannot be initialized.');
    }

    mercury.Delegator(opts);
    var loop = mercury.main(observ(), render, extend({
        diff: mercury.diff,
        create: mercury.create,
        patch: mercury.patch
    }, opts));

    elem.appendChild(loop.target);

    return observ(loop.update);
}

The problem in this example is that elem is null and an error is thrown.

To answer my own question: simply removing any usage of elem from inside the app function solves the issue. It would be nice if this could be stated inside the example.

@martintietz you can pass document.body into app() if you don't have a dom node.


migrate to new documentation format

Related: https://github.com/bit-bots/bitbots_tools/issues/61

Ready to review?

Ready to review?


Boilerplate Init and Readme

Description
Included Boilerplate , Readme, MockUps, and Designs to the Initial project.
Fixes # (issue)
No Issues Fixed Yet
Type of change
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[x] 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

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

[ ] Yarn Started, and Installed Properly

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] My code has been reviewed by at least one peer
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes

:+1:


test_models: check relay malfunction

just checked the RAV4 2017 failure, looks legit (not dashcamOnly, but we're not sending anything nor breaking the relay :thinking:)

If we suspect the routes are bad, let's just add new ones.

Yep, will do!

So these routes were all either dashcammed when they were made, or the openpilot toggle was off (we don't check that). Checking this, no failures.


docs: add a sample app using OAuth2 authorization flow

PLEASE READ THIS ENTIRE MESSAGE
Hello, and thank you for your contribution! Please note that this repository is
a read-only split of googleapis/google-cloud-php. As such, we are
unable to accept pull requests to this repository.
We welcome your pull request and would be happy to consider it for inclusion in
our library if you follow these steps:

Clone the parent client library repository:

$ git clone git@github.com:googleapis/google-cloud-php.git

Move your changes into the correct location in that library. Library code
belongs in AnalyticsData/src, and tests in AnalyticsData/tests.

Push the changes in a new branch to a fork, and open a new pull request
here.

Thanks again, and we look forward to seeing your proposed change!
The Google Cloud PHP team

This sample was very useful to me @ikuleshov and should definitely be added somewhere :)


LoadBalancer constructor description isn't clear

See:
https://github.com/dart-lang/isolate/blob/6d3656f0f0b0a67a31fcf1ada7744df0f1d5d87c/lib/load_balancer.dart#L29-L30
Nor service, neither size are symbols in class or constructor parameter list.

Fixed.


tick.time do not include microseconds

Getting tick.time value from client.tick_by_tick_bid_ask we can receive the following data:
2023-08-04 19:20:58.0 +00:00:00
However, do not include miliseconds/microseconds, any workaround to get this data from IB server?

TWS gateway is returning seconds since Jan 1970.


Doesn't work on MacOS

On MacOS there is a limitation: You can't run a gui outside the MAIN Thread. @NeuralNine please find a way to run the the gui in the Main Thread to have compatability with MacOS.

Thanks for the information. I don't have any MacOS systems I can test this package on at the moment. I will work on it once I have the possibility. Of course I am also happy to accept pull requests from Mac users that are able to fix this issue.

Thanks for the information. I don't have any MacOS systems I can test this package on at the moment. I will work on it once I have the possibility. Of course I am also happy to accept pull requests from Mac users that are able to fix this issue.

Thanks for replying to my issue. I also have a Windows machine but my mac is my main programming computer! I will also try it on the Windows machine.

I studied the source code of vidstream and if i find a way to not put the _start_stream function on the StreamingClient class inside a Thread it may work on MacOS

I studied the source code of vidstream and if i find a way to not put the __client_connection function on the StreamingServer class inside a Thread it may work on MacOS

Hello, I'm also a mac user. Do you can share your solution ?

Hi,
Had this bug also :)
This helped me - https://github.com/opencv/opencv/issues/22602
The thing is macOs doesn't allow UI actions from threads other then main thread.
Meaning in StreamingServer.start_server program initiate new thread of __server_listening.
So when we try to do UI action in line 179
cv2.imshow(str(address), frame)
We'll get an error.
I tried to init __server_listening in another process, i.e:
    def start_server(self):         """         Starts the server if it is not running already.         """         if self.__running:             print("Server is already running")         else:             self.__running = True             processId = os.fork()             if processId > 0:                 self.__server_listening()
and it worked correctly.
@NeuralNine wonder should I make a PR checking server's OS and for macOS answers we'll run __server_listening in another process


Different subscription to be used for ARS/ERGW

Customer requires ARS/ERGW (and VNET for them) to be deployed in different Azure Subscription than Transit GWs
Current version they use is 6.9.223

Requirement was removed


Local Environment Setup 💻

Hey @kiranwoth  assigning you your first "before we begin" type task!
Let's make sure your local setup is ready for development 😄
Couple things to setup (check these off as you go):

[x] Download an IDE (if you already have one then woo!, I like vscode personally as it feature rich but minimal)
[x] Get Python installed on your computer (we will use Python as the backend of our app unless you prefer a different language, feel free to comment on here if you don't want to use Python 😄) I would recommend using pyenv to manage your Python installations as it allows you to easily switch Python versions on the fly.
[x] Get familiar with Github a little. Run through the quickstart to get Git downloaded on your computer and maybe try cloning this repo and pushing a "test" commit by adding a test.txt file with "Hello World!" on the first line 🚀
[ ] Optional: If you are not super comfortable with Python it is not a problem, we will go at whatever pace you want and learn as we go. A good hour tutorial to get the basics of Python that I like is this video on youtube.

When your done just close out this issue! Then we can start discussing more tasks and break them down into really small pieces. Feel free to comment here and at me @jackwotherspoon if you have any questions along the way or get stuck :smile:

@jackwotherspoon should commits not be on the main branch?

Great question! Normally no, I see you created a branch which is perfect 😄 Normally you want to create a branch for feature or dev work and then open a pull request to the main branch so someone can review your code prior to merging it.
You can make this easier by disabling commits directly on the main branch. To do so, go to the repository "Settings" tab and then under "Branches" you can add a branch protection rule to main branch to require pull requests and disable merging directly to main.


storage: add ci for storage

Signed-off-by: Boris Glimcher<EMAIL_ADDRESS>

@mestery do we want to make CI for network & storage completely parallels  ?


[Migration]  MenuItem component crashes with  "Cannot read properties of undefined (reading 'root')" on dividerClasses

Duplicates

[X] I have searched the existing issues

Latest version

[X] I have tested the latest version

Steps to reproduce 🕹
Link to live example:
Steps:

Any component that renders a <MenuItem> in the tree be it static or dynamic like a <Select>
Render the application in a browser
React app crashes

Current behavior 😯
This is a super weird behavior and unfortunately I can't seem to reproduce except on these internal systems. I have exhausted what could be the difference from the working versions I have of it and the one that doesn't work.  This feels like something obvious that I just can't see even with all the debugging and testing I have done to try and figure it out.
Even using the example of https://mui.com/material-ui/react-table/#data-table where the drop-down for rows per page is clicked once the MenuItem is attempted to be rendered the error rears it's ugly head.
I would also add that whether the code is in a ThemeProvider or not the bug persists so I feel like it might be something witt WebPack (5) or something else with that process i am not seeing.
Here is a stack-trace of the error when rendering any menu item which points to this line in source  which implies that dividerClasses is undefined and I cannot for the life of me figure out why.  I have tried an exhaustive number of things to try and get this error fixed but nothing seems to work.
  [`& + .${_Divider.dividerClasses.root}`]: {
    marginTop: theme.spacing(1),
    marginBottom: theme.spacing(1)
  },

Actual stack-trace from chrome.

Context 🔦
Been in the process of upgrading from Material UI v4 to v5 and this is the only component that doesn't work. It doesn't feel like a bug in Material UI but I guess it could be? I just feel like i am missing something from the upgrade process and just cannot see it
Your environment 🌎

npx @mui/envinfo
  Chrome and Edge yield the same result; can't test on firefox cause it's not installed
  Output from `npx @mui/envinfo` goes here.
  System:
    OS: Windows 10 10.0.19044
  Binaries:
    Node: 21.2.0 - C:\my-project\node_modules\.bin\node.CMD
    Yarn: Not Found
    npm: 10.2.4 - C:\Program Files\nodejs\npm.CMD
  Browsers:
    Chrome: Not Found
    Edge: Spartan (44.19041.3636.0)
  npmPackages:
    @emotion/react: ^11.11.1 => 11.11.1
    @emotion/styled: ^11.11.0 => 11.11.0
    @mui/base:  5.0.0-beta.26
    @mui/core-downloads-tracker:  5.14.20
    @mui/envinfo: ^2.0.14 => 2.0.14
    @mui/icons-material: ^5.14.18 => 5.14.19
    @mui/material: ^5.14.18 => 5.14.20
    @mui/private-theming:  5.14.20
    @mui/styled-engine:  5.14.20
    @mui/styles: ^5.14.18 => 5.14.20
    @mui/system:  5.14.20
    @mui/types:  7.2.10
    @mui/utils:  5.14.20
    @mui/x-date-pickers: ^6.18.3 => 6.18.3
    @types/react: ^18.2.42 => 18.2.42
    react: ^18.2.0 => 18.2.0
    react-dom: ^18.2.0 => 18.2.0
    styled-components: ^6.1.1 => 6.1.1

It doesn't look like this bug report has enough info for one of us to reproduce it.
Please provide a CodeSandbox (https://material-ui.com/r/issue-template-latest), a link to a repository on GitHub, or provide a minimal code example that reproduces the problem.
Here are some tips for providing a minimal example: https://stackoverflow.com/help/mcve
I know you mentioned that it's hard to reproduce, but we really can't do anything without reproduction.


Feature select video

All feature specific components are under src/components we have explicitly called out that this shouldn't be the case in the contribution guidelines, please go through the same.

@chirgjn
I did not get what is this for?

Can you share some options that you can thing of or were able to find?

Ideally I'd want to show the user an error message on the page like: "Something went wrong, please try again later."

@chirgjn
I wanted to check this with you so that it doesn't come up later and we don’t have multiple iterations over the same task.
I would prefer to return an error and handle it at the calling place, but that way conflicts with the return type stated.
An empty result will be no different from the initial state.
Instead of searching for something, then implementing it, and then finding that it is not as per your expectations in the review, is what I wanted to avoid by consulting you for this. Additionally, you have mentioned clarifying such things early on.

I would prefer to return an error and handle it at the calling place, but that way conflicts with the return type stated.
An empty result will be no different from the initial state.

This is useful. It would have saved an iteration if you'd have shared this problem earlier.

Instead of searching for something, then implementing it, and then finding that it is not as per your expectations in the review

You don't have to implement it and send it for review. Share the approach and the pros and cons.

You have mentioned clarifying such things early on.

This is still the case

Can you search about error handling and suggest some other options?

We can use a promise chain and handle it in a single place. Applies to sequential calls like in init() Gapi.ts.
We can change the return type to Promise<type | any> so that it can handle both the returned promise and returned err.
We can also use return Promise.reject(err), and handle this in a try{} catch{} block from the calling point.

I will need to implement these to know for sure which one fits our needs.
There are many combinations in which Promise and try{} catch{} can be used.


Pre-work to monitoring kubemark via internal IP.

Ref. https://github.com/kubernetes/perf-tests/issues/503

/lgtm
/approve