New ebean leaves connections with active transactions

I have tested two ebean versions: 3.4.1 and 4.5.3 for transactions.
The code is:
	@Inject
	private TestTableDA tda;

	private static class TransactionInfo {
		public final String id;
		public final String status;
		public final String info;

		public TransactionInfo(String id, String status, String info) {
			this.id=id; this.status=status; this.info=info;
		}

		@Override
		public String toString() {
			return String.format("Transaction(%s, %s, %s)",id,status,info);
		}
	}

	private static class MySQLInfo {
		public List<SqlRow> connections;

		public List<TransactionInfo> transactions=new ArrayList<>();

		public String status;

		public MySQLInfo(List<SqlRow> connections, String status) {
			this.connections=connections;
			this.status=status;
			int p=0;
			for(;;) {
				p=status.indexOf("---TRANSACTION ",p);
				if(p==-1) break;
				p+=15;
				int eol1=status.indexOf("\n",p);
				int eol2=status.indexOf("\n",eol1+2);

				transactions.add(new TransactionInfo(status.substring(p,p+6).trim(), status.substring(p+8, eol1).trim() , status.substring(eol1+1, eol2).trim()));
			}
		}

		public void log() {
//			log.i("Connections: %s",connections);
			log.i("Transactions: %s",transactions);
//			log.i("Status: %s",status);
		}
	}

	private MySQLInfo getMySQLStatus() {
		MySQLInfo res=new MySQLInfo(db.createSqlQuery("show processlist").findList(), db.createSqlQuery("SHOW ENGINE INNODB STATUS;").findUnique().getString("status"));
		res.log();
		return res;
	}

	@Test
	public void mySQLTransactionTest() throws InterruptedException {
		Thread.sleep(10000);	// wait for all start (backgroud) operations to finish

		if(cfg.getGeneralDbType()!=DbType.MySQL) return;	// check only with mySQL run
		getMySQLStatus();
		tda.getAll();
		getMySQLStatus();
	}

TestTableDA.java:
	public List<T> getAll() {
		return ebean.find(TestTable.class).findList();
	}

In short it just gets information from mySQL server (Transactions) before select, then it makes test select and after select it gets once again information from mySQL server.
In ebean 3.4.1 I always have (I started this 6 times to be sure):
main|INFO |13:47:11|TransactionTests - Transactions: ArrayList#2[Transaction(19C835, not started, MySQL thread id 121, OS thread handle 0x7f53c0401700, query id 69191 localhost <IP_ADDRESS> test), Transaction(19C834, not started, MySQL thread id 122, OS thread handle 0x7f53c04f6700, query id 69190 localhost <IP_ADDRESS> test)]
main|INFO |13:47:11|TransactionTests - Transactions: ArrayList#2[Transaction(19C835, not started, MySQL thread id 121, OS thread handle 0x7f53c0401700, query id 69196 localhost <IP_ADDRESS> test), Transaction(19C836, not started, MySQL thread id 122, OS thread handle 0x7f53c04f6700, query id 69197 localhost <IP_ADDRESS> test)]

Two transactions that are not active before and after select - each time.
With ebean 4.5.3 I have (tested also 6 times):
main|INFO |13:51:46|TransactionTests - Transactions: ArrayList#2[Transaction(1A3A9C, not started, MySQL thread id 141, OS thread handle 0x7f53c0463700, query id 104650 localhost <IP_ADDRESS> test), Transaction(1A34EC, ACTIVE 10 sec, MySQL thread id 142, OS thread handle 0x7f53c0527700, query id 104651 localhost <IP_ADDRESS> test)]
main|INFO |13:51:46|TransactionTests - Transactions: ArrayList#2[Transaction(1A3A9D, ACTIVE 0 sec, MySQL thread id 141, OS thread handle 0x7f53c0463700, query id 104654 localhost <IP_ADDRESS> test), Transaction(1A34EC, ACTIVE 10 sec, MySQL thread id 142, OS thread handle 0x7f53c0527700, query id 104653 localhost <IP_ADDRESS> test)]

One transaction is ACTIVE before select (from test starting routine which is done at test start), and after select there are two ACTIVE transactions.
This difference between ebean versions makes for me problem with altering tables, because there cannot be any active transaction when altering:
mysql> show processlist;
+----+------+-----------------+------------+---------+------+---------------------------------+-----------------------------------------+
| Id | User | Host            | db         | Command | Time | State                           | Info                                    |
+----+------+-----------------+------------+---------+------+---------------------------------+-----------------------------------------+
| 18 | root | localhost:60603 | documaster | Query   |   33 | Waiting for table metadata lock | ALTER TABLE test_table DROP COLUMN name |
| 19 | root | localhost:60604 | documaster | Sleep   |   97 |                                 | NULL                                    |
| 20 | root | localhost:60605 | NULL       | Query   |    0 | NULL                            | show processlist                        |
+----+------+-----------------+------------+---------+------+---------------------------------+-----------------------------------------+

I'm not sure if leaving active transactions doesn't make anything other (slowing database?).

Try setting a system property:
ebean.transaction.onqueryonly=ROLLBACK
System.setProperty("ebean.transaction.onqueryonly", "ROLLBACK");
... This property defaults to CLOSE in 4.x

That worked for me (thanks a lot), but isn't it bad for database performace that transactions are active?
Like you can see in my logs. I have Thread.sleep(10000) command to wait for background processes to finish
main|INFO |13:51:46|TransactionTests - Transactions: ArrayList#2[Transaction(1A3A9C, not started, MySQL thread id 141, OS thread handle 0x7f53c0463700, query id 104650 localhost <IP_ADDRESS> test), Transaction(1A34EC, ACTIVE 10 sec, MySQL thread id 142, OS thread handle 0x7f53c0527700, query id 104651 localhost <IP_ADDRESS> test)]

and it makes transaction active for that whole time "ACTIVE 10 sec" and propably much more...
When I was doing alters to database it waited for about 90 seconds (mysql config timeout?).

If we were running at any isolation level higher than READ_COMMITTED then
we would have to rollback any read only transactions.  However, we
basically have to run at read committed isolation level (otherwise
optimistic concurrency checking does not work) and that is actually defined
in the JPA spec (to run at read committed).
The justification for not rolling back read only transactions when running
at read committed isolation level is that then the "transaction start" is
irrelevant to the execution of statements (well, in Oracle speak we get
'statement level read consistency', the view of the db is at statement
start time and not transaction start time).
Now that said, this obviously effects MySql where it should have no effect
on Oracle or Postgres say. So yes, it might be that for MySql we need to
change the default to ROLLBACK (and for Oracle/Postgres we can leave it as
is) so I'll have a look at that.
On 15 January 2015 at 23:14, Ryszard Trojnacki<EMAIL_ADDRESS>wrote:

That worked for me (thanks a lot), but isn't it bad for database
performace that transactions are active?
Like you can see in my logs. I have Thread.sleep(10000) command to wait
for background processes to finish
main|INFO |13:51:46|TransactionTests - Transactions: ArrayList#2[Transaction(1A3A9C, not started, MySQL thread id 141, OS thread handle 0x7f53c0463700, query id 104650 localhost <IP_ADDRESS> test), Transaction(1A34EC, ACTIVE 10 sec, MySQL thread id 142, OS thread handle 0x7f53c0527700, query id 104651 localhost <IP_ADDRESS> test)]
and it makes transaction active for that whole time "ACTIVE 10 sec" and
propably much more...
When I was doing alters to database it waited for about 90 seconds (mysql
config timeout?).
—
Reply to this email directly or view it on GitHub
https://github.com/ebean-orm/avaje-ebeanorm/issues/232#issuecomment-70064969
.

SUMMARY:
The OnQueryOnly.CLOSE optimisation is by default only on for Oracle and Postgres. Can be set on the DatabasePlatform or System property "ebean.transaction.onqueryonly"
DETAIL:
Ok, those 2 commits don't have good 'diffs' as I changed the line endings ... however, the guts of the change is:

OnQueryOnly moved to DatabasePlatform
DatabasePlatform defaults to use OnQueryOnly.ROLLBACK
Oracle and Postgres default to use the OnQueryOnly.CLOSE optimisation

The guts of the OnQueryOnly mode selection is (still) in TransactionManager and is:
/**

Return the behaviour to use when a query only transaction is committed.

There is a potential optimisation available when read committed is the default
isolation level. If it is, then Connections used only for queries do not require
commit or rollback but instead can just be put back into the pool via close().

If the Isolation level is higher (say SERIALIZABLE) then Connections used
just for queries do need to be committed or rollback after the query.

*/
private OnQueryOnly initOnQueryOnly(OnQueryOnly dbPlatformOnQueryOnly, DataSource ds) {
// first check for a system property 'override'
String systemPropertyValue = System.getProperty("ebean.transaction.onqueryonly");
if (systemPropertyValue != null) {
  return OnQueryOnly.valueOf(systemPropertyValue.trim().toUpperCase());
}

if (OnQueryOnly.CLOSE.equals(dbPlatformOnQueryOnly)) {
  // check for read committed isolation level
  if (!isReadCommitedIsolation(ds)) {
    logger.warn("Ignoring DatabasePlatform.OnQueryOnly.CLOSE as the transaction Isolation Level is not " +
							"READ_COMMITTED");
    // we will just use ROLLBACK and ignore the desired optimisation
    return OnQueryOnly.ROLLBACK;
  } else {
    // will use the OnQueryOnly.CLOSE optimisation
    return OnQueryOnly.CLOSE;
  }
}
// default to rollback if not defined on the platform
return dbPlatformOnQueryOnly == null ? OnQueryOnly.ROLLBACK : dbPlatformOnQueryOnly;

}

Closing this as 'fixed' in 4.5.4 (OnQueryOnly.CLOSE optimisation only using for Oracle and Postgres by default).


Use content resolver for content uri on android

We encountered exceptions trying to copy from content:// URI on Android. This happens as soon as the file type isn't one of the few types explictly supported by ReactNativeBlobUtilUtils.normalizePath. Trying to extend the supported file types looked like a dead-end: when picking a PDF document from the Files app using react-native-document-picker for example, we get a content:// URI which can't be accessed as a regular file as far as I can tell.
So, while ReactNativeBlobUtilUtils.normalizePath is definitely useful for destination files, trying to use it too much for source path is problematic.
In this pull-request, I am using Android ContentResolver.openInputStream() to deal with such URI in cp, hash and slice.
Interestingly, openInputStream was already used by readFile so I didn't need to fix that case.

Thank you.
Yeah having to run thorugh normalizePath feels in general a bit strange. Especially since we don't have to in most cases.
But tbh. very old parts of the lib, and not sure I want to tuch that^^


Add an option to generate puzzles with lowercase letters?

Firstly, thank you so much for developing this project. I've been searching for a word search generator for quite some time.
I understand that using uppercase letters is a tradition, but I plan to use this tool to assist my 7-year-old daughter in learning English as a second language. Lowercase letters would be more suitable for her.
Could you please consider adding an option to generate puzzles with lowercase letters?
Thank you again for your work on this fantastic project!

Hi @joshbduncan, thank you very much!

All letters in the puzzle are in lowercase.
Utilize this package directly within Python.

I plan to gather new words from her reading books (e.g., Frog and Toad), create word search puzzles, and engage in play together until she becomes familiar with these words. Afterward, I'll encourage her to start reading the book. This is why I'd like to have all lowercase letters in the puzzle and intend to manage new words by creating a Python file and executing this package within it.

Thanks for responding. I'll get this implemented in the next version (coming very soon) and let you know when it is live. Thanks!

Ok, I got implemented. It is not officially released yet but you can install it directly from GitHub using the following command...
pip install git+https://github.com/joshbduncan/word-search-generator

Once installed, you can set the lowercase=True argument in the show method.  Thank you so much!
I am not familiar with Python. I didn't realize it would take that much effort. I really appreciate your help!

My pleasure!


fetch releases count too

We need to fetch total count of releases for each repository

@rultor release, tag is 0.0.4

@rultor release, tag is 0.0.4

@h1alexbel OK, I will release it now. Please check the progress here

@rultor release, tag is 0.0.4

@h1alexbel Done! FYI, the full log is here (took me 7min)


Want to help to add new language translation

uhh hi guys , i was here to support this project , and i want to help translate this builds to the another language , especially indonesian , so ill need some equipment to do it .
thanks !!

What?
Developer called languages_full.mk config so this rom supports all languages which support AOSP.
I am sure that indonesian language is included

yes i know that , but that translation was not perfect , so i want to fix the wrong ones , to became the right ones .

You  could contribute to the official cyanogenmod    tree so that many people can benefit from


Ctlr locks after AltGr is pressed (windows 10)

Hello.
Windows strart support rouble sign by default (AltGr+8), and after that TiE works bad.
After I press AltGr combination, it print symbol as usual, but after that Ctrl locks, and all keypressings become Ctlr combinations. If press Ctrl again it unlocks.
On windows 7-8.1 I fixed it by uninstalling MS updates corresponding for rouble sign support. But in Win 10 this functionality integrated to core.

The root of the the problem is the russian keyboard layout with rouble sign support.
See https://tabun.everypony.ru/blog/computers/105151.html
You can install fixed keyboard layout using this installer: https://cloud.mail.ru/public/4Kv8AvkkxHZQ/norub.7z

@oulenspiegel Thank you! This is exactly same solution I use.

BTW, here is latest official binary + new keyboard layout: https://yadi.sk/d/5l77PHvX3VVG4J (because Nikolay's site is now down).


[CROSSDATA-426] [STREAMING] [WIP] Streaming test fixing

It was already resolved in https://github.com/Stratio/Crossdata/pull/510


Add event sourcing example

This is still a pretty rough idea but I'm curious if this is basically the kind of thing you were thinking. This example takes cues directly from this article: https://www.eventstore.com/event-sourcing.
@timostamm I'm curious if this is roughly what you had in mind.

@timostamm I'm curious if this is roughly what you had in mind.

This surpasses what I had in mind! I'm not sure if I understand the article correctly, but it looks like ProductAdded and ProductRemoved are mutations of an order. That would make more sense in my mind, because the events are basically persistable actions (add/remove a product, submit/pay) on a subject (the order). Because the actions are objects themselves instead of method names, it is trivial to persist them, and that's why this model lends itself to audit logs.


v.0.21.12

Version 0.21.12
Released March 13, 2023

Refactor NYSE holidays, #1018 (@KJhellico)
Fix the documentation build warning, #1026 (@arkid15r)
Optimize TestAllInSameYear::test_all_countries test, #1001 (@arkid15r)
Update Canada localization: fr, #1015 (@KJhellico)
Update Denmark holidays: remove "Store bededag" starting 2024, #1013 (@malthe)
Update PR template, #1019 (@arkid15r)
Update Portugal holidays, #1011 (@PPsyrius)
Update Ukraine holidays, #1014 (@KJhellico)
Update package build process, #1020 (@arkid15r)
Add multiple countries localization: en_US, es, uk, #1012 (@KJhellico)
Unify country imports order, #1003 (@arkid15r)
Use native apostrophe for uk locale holiday names, #1004 (@arkid15r)

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
7133

Relevant Lines:
7133

💛  - Coveralls


Some Questions about Your Implemention

Hi Jiaxuan:
Thank you for your source code. It helps me a lot. Recently I'm trying to replicate this paper by tensorflow. However, I encounter some questions when I'm trying to understand your source code.

As detailed in Page 4 "Concretely, the l-th P-GNN layer first samples k random anchor-sets S_i."  and algorithm， should I resample anchor-sets at every layer of P-GNN? In your source code, (mode.py line 90 - line 100), anchor-sets are resampled at every epoch.

In model.py, you say:"only pick closest node for message passing". Combining it with your code to undersatnd, I think that you only use feature of the closest node in a anchor set. It's  contradictory with this statement "We find that using simple MEAN aggregation function provides good result".

I look forward to your reply.

@fanninnypeom I got the same questions, as posted in #13. Another similar issue can be found in #11 .
Below is my guess:

Paper
Implementation

1
resample every epoch and every layer
resample every epoch, but fixed for each layer

2
$AGG_M$ is MEAN
$AGG_M$ is MAX

Please correct me if I'm wrong here.

@fanninnypeom @chao1224 Were you able to figure out Q2. "$AGG_M$ is MEAN"?
I have the same question and am wondering if changing the implementation to incorporate MEAN aggregator improves the performance.
Any help is appreciated.


Add location_url to event endpoint response

Short description
This adds the location_url to the event endpoint. I decided against adding the url in the transform_poi function, as it is also used in the poi endpoint and there this information would be duplicated.
Proposed changes

add attribute location_url to event endpoint

Resolved issues

Fixes: #2294

Pull Request Review Guidelines

Thanks, this looks mostly good to me. This update just needs to be reflected in our api documentation.

How do you update the api docs? I only found ./tools/make_docs.sh

How do you update the api docs? I only found ./tools/make_docs.sh

Unfortunately the api docs cannot be automatically generated from the source code, so it has to be manually changed in this document:
https://github.com/digitalfabrik/integreat-cms/blob/01c9631b145b0ca62dd2d4be37ef6e047df351f5/docs/src/api-docs.rst


parallel training

Why hasn't parallel training been initialized？

Hi @dreammyhome. We did not use parallel computing during the development of our model. Currently, there are codes in the main.py for parallel training, but they have not been tested. I will check if they are consistent throughout the repository.
Thank you for your interest.

Thank you very much.


Adding the createFailed and AnswerFailed events on the call automation SDK

Describe the problem that is addressed by this PR
Added the CreatCallFailed and AnswerFailed event.
This changed is tested against the Contoso app.

API change check
APIView has identified API level changes in this PR and created following API reviews.
@azure/communication-call-automation


Add detailed bash and powershell insructions.

For using visualizer using REST & for local hosting.

Oh crap! Yes.
On Mon, Aug 5, 2019 at 4:50 PM Joseph Snyder<EMAIL_ADDRESS>wrote:

@josephsnyder requested changes on this pull request.
One small thing to fix up in the Powershell script.
In README.rst
https://github.com/OSEHRA-Sandbox/SyntheaPatientViz/pull/7#discussion_r310785068
:

-Traditional Display
+Windows:
+
+.. code-block:: powershell
+

$folder="C:\Users\Hp\synthea\output\fhir" # Replace this with your correct path
del $folder/*.html
$files = Get-ChildItem $folder -Filter _.json
foreach ($f in $files) {
echo $f.FullName
Invoke_RestMethod -Uri https://code.osehra.org/synthea/synthea_upload.php -Method Post -InFile $f.FullName > ($f.FullName + ".html")

Seems like this should be "Invoke-RestMethod", not "Invoke_RestMethod" for
my Win10 machine.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/OSEHRA-Sandbox/SyntheaPatientViz/pull/7?email_source=notifications&email_token=AAIS3QUJIXGVILVCZ6KBFCLQDCHBPA5CNFSM4IJOO2T2YY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCATRKFQ#pullrequestreview-270996758,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAIS3QQOAR3DQFVGYOCGJJDQDCHBPANCNFSM4IJOO2TQ
.

Okay fixed and force pushed.q
On Mon, Aug 5, 2019 at 5:09 PM Sam Habiel<EMAIL_ADDRESS>wrote:

Oh crap! Yes.
On Mon, Aug 5, 2019 at 4:50 PM Joseph Snyder<EMAIL_ADDRESS>wrote:

@josephsnyder requested changes on this pull request.
One small thing to fix up in the Powershell script.
In README.rst
https://github.com/OSEHRA-Sandbox/SyntheaPatientViz/pull/7#discussion_r310785068
:

-Traditional Display
+Windows:
+
+.. code-block:: powershell
+

$folder="C:\Users\Hp\synthea\output\fhir" # Replace this with your correct path
del $folder/*.html
$files = Get-ChildItem $folder -Filter _.json
foreach ($f in $files) {
echo $f.FullName
Invoke_RestMethod -Uri https://code.osehra.org/synthea/synthea_upload.php -Method Post -InFile $f.FullName > ($f.FullName + ".html")

Seems like this should be "Invoke-RestMethod", not "Invoke_RestMethod"
for my Win10 machine.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/OSEHRA-Sandbox/SyntheaPatientViz/pull/7?email_source=notifications&email_token=AAIS3QUJIXGVILVCZ6KBFCLQDCHBPA5CNFSM4IJOO2T2YY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCATRKFQ#pullrequestreview-270996758,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAIS3QQOAR3DQFVGYOCGJJDQDCHBPANCNFSM4IJOO2TQ
.


Oleh syzonov patch sla 77

Update local_proxy_remote_solana.md with actual parameters for handling new version of the NeonProxy

changed reference to the docker compose file
changed command for starting Proxy program

added hint to put Solana private key to id.json

@OlehSyzonov I pulled the changes on my side and I see the following stuff:

Command wget https://github.com/neonlabsorg/neon-proxy.py/blob/develop/docker-compose/docker-compose-ro.yml is wrong and is downloading the page HTML instead of the docker-compose config file. This command has to be changed to wget https://raw.githubusercontent.com/neonlabsorg/neon-proxy.py/develop/docker-compose/docker-compose-ro.yml
Command docker-compose up -d is also wrong, because you when you run docker-compose command you need to specify which config file to run and also which services should be included. Replace this command with docker-compose -f docker-compose-ro.yml up postgres dbcreation proxy indexer
Replace:

EVM_LOADER=eeLSJgWzzxrqKv1UxtRVVH8FX3qCQWUs9QuAjJpETGU
VERSION=v1.13.20
SOLANA_URL=http://api.devnet.solana.com/
SOLANA_KEY_FOR_EVM_CONFIG=<your solana public key>

With:
export EVM_LOADER=eeLSJgWzzxrqKv1UxtRVVH8FX3qCQWUs9QuAjJpETGU
export VERSION=v1.13.20
export SOLANA_URL=http://api.devnet.solana.com/
export SOLANA_KEY_FOR_EVM_CONFIG=<your solana public key>

After I implemented 1. 2. and 3. on my side I was able to run the local proxy on my end, but there is another issue. Now I'm able to run in the console curl http://localhost:9090/solana -X POST -H "Content-Type: application/json" --data '{"method":"eth_blockNumber","params":[],"id":1,"jsonrpc":"2.0"}' and I see a response {"jsonrpc":"2.0","id":1,"result":"0x12ce9c7a"}, but this response is static. It's the same response if I run the requests 2 mins later and this method eth_blockNumber should return new blocks for every request, because Solana mints 2 blocks every second. I see the following error in the console after running the proxy which speaks of some limitation being crossed - indexer   | {“level”: “WARNING”, “date”: “2024-07-31T13:40:28.067182", “pid”: 42, “module”: “common/http/client.py:147”, “message”: “error on retry 21 on request to *****: 429, message=‘Too Many Requests’, url=URL(‘*****’)“, “reindex”: “continue”}

@OlehSyzonov I pulled the changes on my side and I see the following stuff:

Command wget https://github.com/neonlabsorg/neon-proxy.py/blob/develop/docker-compose/docker-compose-ro.yml is wrong and is downloading the page HTML instead of the docker-compose config file. This command has to be changed to wget https://raw.githubusercontent.com/neonlabsorg/neon-proxy.py/develop/docker-compose/docker-compose-ro.yml
Command docker-compose up -d is also wrong, because you when you run docker-compose command you need to specify which config file to run and also which services should be included. Replace this command with docker-compose -f docker-compose-ro.yml up postgres dbcreation proxy indexer
Replace:

EVM_LOADER=eeLSJgWzzxrqKv1UxtRVVH8FX3qCQWUs9QuAjJpETGU
VERSION=v1.13.20
SOLANA_URL=http://api.devnet.solana.com/
SOLANA_KEY_FOR_EVM_CONFIG=<your solana public key>

With:
export EVM_LOADER=eeLSJgWzzxrqKv1UxtRVVH8FX3qCQWUs9QuAjJpETGU
export VERSION=v1.13.20
export SOLANA_URL=http://api.devnet.solana.com/
export SOLANA_KEY_FOR_EVM_CONFIG=<your solana public key>

After I implemented 1. 2. and 3. on my side I was able to run the local proxy on my end, but there is another issue. Now I'm able to run in the console curl http://localhost:9090/solana -X POST -H "Content-Type: application/json" --data '{"method":"eth_blockNumber","params":[],"id":1,"jsonrpc":"2.0"}' and I see a response {"jsonrpc":"2.0","id":1,"result":"0x12ce9c7a"}, but this response is static. It's the same response if I run the requests 2 mins later and this method eth_blockNumber should return new blocks for every request, because Solana mints 2 blocks every second. I see the following error in the console after running the proxy which speaks of some limitation being crossed - indexer   | {“level”: “WARNING”, “date”: “2024-07-31T13:40:28.067182", “pid”: 42, “module”: “common/http/client.py:147”, “message”: “error on retry 21 on request to *****: 429, message=‘Too Many Requests’, url=URL(‘*****’)“, “reindex”: “continue”}

@romanova-natasha Can you confirm that?

@OlehSyzonov

replace the url, Miro is right
Initial option of the compose file was named docker-compose.yml, it is a default option for command docker-compose up -d, so it was absolutely valid. I can't see any reason why I have to list services by its names. docker-compose -f docker-compose-ro.yml up -d will work perfectly, we need all services listed in compose file, we can enumerate it separately in case we need to run some of it.
Why not, it is ok, we can place bash code instead of just the info.
I am not an author of this docker-compose file, the issue Miro described is about solana node requests limitations. To understand how you can avoid this you have to ask our devopses (I know that for tests on devnet we use another solana node).
For checking proxy is operable we can add a comment: check a couple of blocks and make sure blocks are updated.

About 2. I'm ok with what @romanova-natasha said, but the thing is in the tutorial the command is placed as docker-compose up -d which does not work. Let's change it to docker-compose -f docker-compose-ro.yml up -d
About 3. my idea is that we should follow some consistency in the docs and we've already showed on multiple pages that setting up environment variables in the terminal is done like export VARIABLE=123, for example - https://neonevm.org/docs/operating/basic#step-2-configure-environment-variables. If we remove the export keyword we should remove it everywhere.
About 4. I've tested the local proxy with a Solana node which is limitless and the proxy works fine, I even broadcasted a Devnet transaction through it. So we should not advise users to use any of the following public Solana nodes ( https://api.devnet.solana.com & https://api.mainnet-beta.solana.com ), because our proxy overflow the limits just few seconds after running it. Let's replace export SOLANA_URL=http://api.devnet.solana.com/ to export SOLANA_URL=<SOLANA_NODE> and let's place explanation that the public nodes have rate limits and they will not work with the local proxy. Providers like Quicknode, TritonOne and Infura ( and others ) offer nodes with higher limits, but I'm not sure if we should mention them in the docs.
Also let's remove this section:

@mnedelchev-vn please, confirm/correct my summary of the discussion:

The url was https://raw.githubusercontent.com/neonlabsorg/neon-proxy.py/develop/docker-compose/docker-compose-ro.yml before the comment of @afalaleev . I'm confused. Going to revert it.
I'll write docker-compose -f docker-compose-ro.yml up -d
I'll write

export EVM_LOADER=eeLSJgWzzxrqKv1UxtRVVH8FX3qCQWUs9QuAjJpETGU
export VERSION=v1.13.20
export SOLANA_URL=http://api.devnet.solana.com/
export SOLANA_KEY_FOR_EVM_CONFIG=<your solana public key>

I'll replace export SOLANA_URL=http://api.devnet.solana.com/ to export SOLANA_URL=<SOLANA_NODE> and add comment "Please note that public nodes have rate limits and they may not work with the local proxy".
I'll remove section about RPC endpoints.

@OlehSyzonov yes, in summary these are the stuff we've change. Please also replace export SOLANA_KEY_FOR_EVM_CONFIG=<your solana public key> with export SOLANA_KEY_FOR_EVM_CONFIG=<YOUR_SOLANA_PUBLIC_KEY>.

@romanova-natasha , @mnedelchev-vn the agreed changes are applied. Please, approve the PR.


Fixing valid dashboard refresh

We had a reactive statement dependant on $runtime. Since it changes every time jwt changes this lead to constant refresh of the dashboard.

All i am doing is not depending on the whole of $runtime here. This is essentially memoisation of host. The runtime.set is getting called since jwt refreshes. Which changes the contents of $runtime. Which triggers the reactive statement that refetches the queries.
Not sure I understand your concern here.

Ah its the race condition that is the problem. Lemme check

To go with your approach, but still avoid the race condition, maybe a derived store for the runtime host would work.

Alternatively, I think making the useProject hook that's used on the dashboard page into a useProjectDeploymentStatus custom hook (w/ a selector for the status) would work & is a more correct use of TanStack.

Changing useProject should be in a different PR. It is unrelated. I have just removed the reactive statement. This assumed instanceId always changes when the deployment for a project changes.


ng build hangs without error or warning.

Command
build
Is this a regression?

[ ] Yes, this behavior used to work in the previous version

The previous version in which this bug was not present was
No response
Description
Raising this issue on behalf of @TheBarenziah (question was originally asked on discord: https://discord.com/channels/748677963142135818/1234862759502549163)
Running ng build hangs without showing any errors or warnings. Running it with the --verbose flag results in the following output before it hangs: https://pastebin.com/4DTaaNXL (dist folder is not created even though verbose output shows (some of) the chunks).
No errors are shown in the codebase and ng serve  and ng serve --configuration production work as expected.
Steps taken to debug the issue:

ran command on different machines.
removed package-lock.json before install to ensure correct package versions were installed.
tried with different version of node (latest version of node 18)
tried with downgrading Angular packages (fixed on 17.1.0)
removed any code referencing firebase/database (firebase database provider, database.service.ts and code referencing the service in register.component.ts) as that was added last. ng build worked before that, but was couple weeks ago since last tried.

The absence of errors/warnings makes this hard to debug and I unfortunately do not have the ability to step through the different build steps to isolate the issue.
Minimal Reproduction
Repository: https://github.com/TheBarenziah/neuro-recovery-app (firebase api key will be replaced once issue is resolved, no data in database)

clone
npm install
ng build or ng build --verbose

Exception or Error
No response
Your Environment
Angular CLI: 17.3.6
Node: 20.11.1
Package Manager: npm 10.2.4
OS: linux x64

Angular: 17.3.6
... animations, cdk, cli, common, compiler, compiler-cli, core
... forms, material, platform-browser, platform-browser-dynamic
... platform-server, router, ssr

Package                         Version
---------------------------------------------------------
@angular-devkit/architect       0.1703.6
@angular-devkit/build-angular   17.3.6
@angular-devkit/core            17.3.6
@angular-devkit/schematics      17.3.6
@angular/fire                   17.0.1
@schematics/angular             17.3.6
rxjs                            7.8.1
typescript                      5.4.5
zone.js                         0.14.4

Anything else relevant?
No response

In this scenario, the application isn't becoming stable, preventing prerendering from finishing. This seems to be caused by the following code:
  ngOnInit(): void {
    this.authService.user$.subscribe(user => {
      if (user) {
        this.authService.currentUserSig.set({
          email: user.email!,
          username: user.displayName!,
        });
      } else {
        this.authService.currentUserSig.set(null);
      }
      // console.log(this.authService.currentUserSig());
    });
  }

@alan-agius4 Thank you for taking a look. ng build is working as expected now after wrapping this code in ngOnInit in a AfterNextRender.

Couldn't the CLI display a warning when pre-rendereing a page takes more than 10s for example ?
This would give some insights on the issue.

I agree that a warning would have been nice in this case, as we had no idea what/where the issue was.


Activate the ModifierComposable rule

Issue

close #511

Overview (Required)

Activates the ModifierComposable rule of twitter/compose-rules

Links

ComposeModifierComposableCheckTest.kt  ---  test code that describes how ModifierComposable should behave

Good!


fix(babel-plugin): fixed handling of default styles

Proposed changes
fixed handling of default styles for responsive objects at build time
Types of changes

[x] 🐛 Bugfix (non-breaking change which fixes an issue)

Checklist

[x] 👀 I have read the CONTRIBUTING doc
[x] ✅ Lint and unit tests pass locally with my changes
[x] 🧪 I have added tests that prove my fix is effective or that my feature works
[x] 📚 I have added the necessary documentation (if appropriate)
[x] 🔀 Any dependent changes have been merged and published in downstream modules

Codecov Report
Patch coverage: 100.00% and no project coverage change.

Comparison is base (b094895) 100.00% compared to head (88ba670) 100.00%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@            Coverage Diff            @@
##              main      #741   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files          103       106    +3
  Lines          914       944   +30
  Branches       160       167    +7
=========================================
+ Hits           914       944   +30

Impacted Files
Coverage Δ

packages/compiler/src/transformer.ts
100.00% <ø> (ø)

packages/jss/src/morfeoJSS.ts
100.00% <ø> (ø)

packages/babel-plugin/src/plugin.ts
100.00% <100.00%> (ø)

packages/babel-plugin/src/utils/css.ts
100.00% <100.00%> (ø)

packages/babel-plugin/src/utils/dynamicClasses.ts
100.00% <100.00%> (ø)

packages/babel-plugin/src/utils/orderStyles.ts
100.00% <100.00%> (ø)

packages/babel-plugin/src/utils/splitStyles.ts
100.00% <100.00%> (ø)

packages/babel-plugin/src/visitors/component.ts
100.00% <100.00%> (ø)

packages/babel-plugin/src/visitors/css.ts
100.00% <100.00%> (ø)

packages/compiler/src/plugin.ts
100.00% <100.00%> (ø)

... and 6 more

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


Leading label does not match separator margin

If you look at the default cells the leading label is aligned with the separator. I believe you can fix this by marking the Superview.Leading item in the respective constraints to be "Relative to Margin".

"Relative to Margin" constraints were introduced in iOS 8, and AutoLayoutCells was originally designed to support iOS 7.0+.
I may be okay with dropping support for iOS 7, especially as iOS 9 is on the horizon.
@AnthonyMDev, what are your thoughts on this? What do you think about targeting a 1.0 release that drops support for iOS 7?

I'd like to see an initial, stable release with support for iOS7, so that legacy projects can have a dependency on a major, non-beta version. I'm alright with dropping iOS7 in version 2.0, which could probably be released very soon after 1.0.


how to use own server

I want to use my own server resources. I used nodejs to write a program. www folder as a static resource. What can I do to update my program next. Or do you have the server code (nodejs) can share it?
I print out the address of the app request. Some parameters I don't know what it means. Does anyone know?
android:     /socket.io/?EIO=3&transport=polling
ios : /socket.io/?transport=polling&b64=1&sid=
Very grateful for your help!

I'm not shure what you want?
When using cordova-hot-code-push-local-dev-addon then:

socket.io is only used for creating a local server
ngrok which is used to create a tunnel to the local server.

But for serving updates none of the above stuff required instead you just serve your files on a server and pointing in the file cordova-hcp.json via the property content_url to the server.

@mnewmedia is right.

android: /socket.io/?EIO=3&transport=polling
ios : /socket.io/?transport=polling&b64=1&sid=

These are from cordova-hot-code-push-local-dev-addon. This add-on is used only for local development. When you run cordova-hcp server - CLI client launches local server with socket. Dev add-on connects to this socket to listen for rebuild events. Then it is captured - add-on send's command to the main plugin to perform the update.
You should not use this add-on in production release. For production you will have to setup your own file hosting server, that will contain:

chcp.json config.
chcp.manifest config.
all your application's web files. Basically, www folder's content.

Again, this server is just a file hosting. No need for any additional setup.
When mobile app launches - main plugin will ping server for update. If it's available - it will download and install it. Of course, you can control when update download and install will be performed through the configs and JS API.

@nikDemyankov ,Sorry, my English is too bad.I did it just as you do.I started a service written in nodejs. These are my code.
`var app=require('koa')();
var serve = require('koa-static');
// x-res
app.use(serve(__dirname + '/www'));
app.listen(3000);`
Then I put my local service using ngrok mapping to the network；
This is the address:http://609892bb.ngrok.natapp.cn;
This is the cordova-hcp.json:
{
"name": "hcp-test",
"ios_identifier": "",
"android_identifier": "",
"update": "resume",
"content_url": "http://609892bb.ngrok.natapp.cn"
}
This is the chcp.json:
{
"name": "hcp-test",
"ios_identifier": "",
"android_identifier": "",
"update": "resume",
"content_url": "http://609892bb.ngrok.natapp.cn",
"release": "2016.08.05-15.05.04"
}
This is the config.xml:

Then I changed my code and  Execute command ‘cordova-hcp ’.
At this time, my application is not it should be updated?  Regret is that he did not update;
@mnewmedia @nikDemyankov Thank you very much for your help.

This is the config.xml:
<chcp>     <local-development enabled="true"/>     <config-file url="http://609892bb.ngrok.natapp.cn/chcp.json"/>   </chcp>

If I understood correctly, you have 2 services: local server and the ngrok server. And when device connects to ngrok - it is actually redirects to your local one? Why not to use only one service: local, for example?
Implementation of the cordova-hcp server you can find here.
You can launch app from the native IDE (ios - Xcode; android - Android Studio) and check console output to get more info on what is happening. Also, make sure that app actually can reach configs on your server. If it can't - you should be able to see the error message in the native IDE.
To open native iOS project - open <CORDOVA_PROJECT>/platforms/ios/<PROJECT_NAME>.xcodeproj with Xcode.
To open native Android project - open <CORDOVA_PROJECT>/platforms/android in Android Studio.

@nikDemyankov Thank you so much!I will try to do it;


Live streaming

Hi,
How to mask the "STOP" live streaming to Guest ?
In fact anyone can stop the live streaming :(
I want the fonctionnality Start Live Streaming / Stop live streaming only for one specific user.
Is it possible to adjust this options per user ?
Cordialy

Thank you for your feedback. We currently do not have any plans to change this behavior in the provided default UI as we do not include any user roles there. However you can implement any kind of management in your application and simply forward requests as well as build your own custom interfaces. Best wishes!

Hi,
Ok there is no user role.
In the API i see the "broadcasts" point to integrate in my manage application.
But i dont understand how tu use it because VIMEO or LIBAST give two parameters for live streaming :
RTMP Server Url   and   Stream key
In the API i see platform, stream_url, player_url
So how to use it ?
Cordialy

Some platforms split up the base url and the streaming key, simply put them back together and use stream_url parameter.

Hi,
so i made a test but i get a 406 error
Exception: Response status code: 406
I make a POST at :
rooms/bqHE7ySU84gQFDKUgBto1ZeA/broadcasts
with parameters :
"platform": "generic"
"stream_url": "rtmp://rtmp-global.cloud.vimeo.com/live/eeb7a867-ddba-4f1c-aa81-4f8416e37835/"
Where is my mistake ?
Thank again

when my access_token is invalid i get 410 (normal).
But when it is valid i get 406.
My Vimeo RTMP is valid.

I've checked the broadcasting to vimeo for you but I cannot see any issue there. I'd guess you missed to set the platform which is either generic, youtube or facebook.
$ curl -X POST \
  -d "stream_url=rtmp://rtmp-global.cloud.vimeo.com/live/<stream-key>"  \
  -d "platform=generic" \
  https://api.eyeson.team/rooms/<access-key>/broadcasts

$ curl -X DELETE https://api.eyeson.team/rooms/<access-key>/broadcasts

Sorry, I missed that you set "platform" in one of your previous messages, however it looks your stream-url is broken there as you end it with a / which will be treated as part of the stream-key


test: remove old relay test tools part I

The type of this PR is: Enhancement

This PR enhances the morale
Description
Removes the old createTestEnv which works but is a high-level abstraction not needed thanks to the Relay testing tools we have. This is also one attempt to unblock the Relay 13 upgrade which has failing tests in special with createTestEnv and they're very hard to debug and find the issue.

@araujobarret - is this still WIP or ready to go?

Ready to go @damassi

Gonna open more PRs to remove one by one, to not create a massive one directly

@damassi friendly reminder 🎗

Ah! I'm sorry for dropping the ball on this @araujobarret. Mergin' 👍


more correct implementation of implicit regressor

closes #672
on the first 5 seeds, python src/main.py --approach nsrt_learning --env touch_point --option_learner implicit_bc now gets 50/50 in all cases (previously, one seed would get 49).
visualization with 50 demos (note: unlike before, the colors are now energies, not probabilities):

visualization with 500 demos:

a few observations:

the difference between 50 and 500 is no longer as obvious. but the results with 50 look good.
the NN apparently hasn't figured out that -np.inf == np.inf in this action space, as evidenced by the discontinuity in the color map on the left. I'm not sure what to think about that... maybe it's related to the fact that we're only training on expert data with no noise in it.

@ronuchit I added some comments and asserts to try to address your questions. can you take another look at let me know which parts I should try to further clarify? (sorry, this is hard... I implemented it this way mainly by looking at references)


Use MongoDB fuzzy search feature to filter the chats list

This feature requires MongoDB Atlas to use Lucene search.
https://github.com/GromNaN/symfony-chatgpt-ux/blob/1e51d3df33ba9b4ac7744cb9247df4fd1c738215/src/Twig/Components/ChatsListComponent.php#L26-L40

The index must be created in Atlas first. Doctrine ODM & MongoDB Driver doesn't seems to provide a way to create the index by code.
{
  index: 'default',
  text: {
    query: 'SymfonyGPT',
    path: 'title',
    fuzzy: {}
  }
}


When connecting my account, I still see Metamask listed as the primary option instead of AXS for MNS - Geneva

When connecting my account, I still see Metamask listed as the primary option instead of AXS for MNS - Geneva

Rainbowkit will record the most recent connection methods used, which is a normal phenomenon. Please ensure that the cache is cleared

Rainbowkit will record recent wallet connections, which is a normal phenomenon and cannot be changed. Please ensure that the cache is cleared

ok i will test more on this on the mobile app, closing this for now!


Package 'mlx_whisper.assets' is absent from the packages configuration

Installing, whisper by pip install . -v I get:
:
  writing manifest file 'mlx_whisper.egg-info/SOURCES.txt'
  /opt/homebrew/Caskroom/miniconda/base/envs/mlx/lib/python3.12/site-packages/setuptools/command/build_py.py:207: _Warning: Package 'mlx_whisper.assets' is absent from the `packages` configuration.
  !!

          ********************************************************************************
          ############################
          # Package would be ignored #
          ############################
          Python recognizes 'mlx_whisper.assets' as an importable package[^1],
          but it is absent from setuptools' `packages` configuration.

          This leads to an ambiguous overall configuration. If you want to distribute this
          package, please make sure that 'mlx_whisper.assets' is explicitly added
          to the `packages` configuration field.

          Alternatively, you can also rely on setuptools' discovery methods
          (for example by using `find_namespace_packages(...)`/`find_namespace:`
          instead of `find_packages(...)`/`find:`).

          You can read more about "package discovery" on setuptools documentation page:

          - https://setuptools.pypa.io/en/latest/userguide/package_discovery.html

          If you don't want 'mlx_whisper.assets' to be distributed and are
          already explicitly excluding 'mlx_whisper.assets' via
          `find_namespace_packages(...)/find_namespace` or `find_packages(...)/find`,
          you can try to use `exclude_package_data`, or `include-package-data=False` in
          combination with a more fine grained `package-data` configuration.

          You can read more about "package data files" on setuptools documentation page:

          - https://setuptools.pypa.io/en/latest/userguide/datafiles.html

          [^1]: For Python, any directory (with suitable naming) can be imported,
                even if it does not contain any `.py` files.
                On the other hand, currently there is no concept of package data
                directory, all directories are treated like packages.
          ********************************************************************************

  !!
    check.warn(importable)
  /opt/homebrew/Caskroom/miniconda/base/envs/mlx/lib/python3.12/site-packages/setuptools/_distutils/cmd.py:66: SetuptoolsDeprecationWarning: setup.py install is deprecated.
  !!

          ********************************************************************************
          Please avoid running ``setup.py`` directly.
          Instead, use pypa/build, pypa/installer or other
          standards-based tools.

          See https://blog.ganssle.io/articles/2021/10/setup-py-deprecated.html for details.
          ********************************************************************************

  !!
    self.initialize_options()
  installing to build/bdist.macosx-11.1-arm64/wheel
:

No idea whether this is intentional or not.

Not intentional, I sent a PR which should fix it.


group builtin classes into builtin package

a devise.

Can you provide some examples for this?

vm/builtin
array, http  and other class  except vm code goes builtin

or not in vm, a top level builtin is also ok.

Thanks for the suggestion. But moving them to another package means a lot of Go-level interfaces between vm and those classes need to be redefined. And atm we have other higher-priority tasks. So closing for now.


Can't load my smart home configuration

1. Development/Production
I use robonomics.app Polkadot Parachain
2. Page Address
/telemetry
3. Error Info/Story
When I open the dapp nothing happens

4. Expected behavior
I see the list of my devices
5. Console logs
In the dev console of my browser I see the following

 GET https://aira.mypinata.cloud/ipfs/QmWVBUJoC3YxsaxMELeFES26XbQbceqpBoE8Wq6eaBc9F5 403 (Forbidden)
6. Context
Browser: Brave Version 1.73.91 Chromium: 131.0.6778.85 (Official Build) (arm64)
Dapp release: check here

It is necessary to check where the file is pinned and whether it is accessible via public gateways. If not, you should add your own gateway.
For example, could you provide a hash of a file that cannot be downloaded?

How can I find this hash?


Units not shown after update to 0.0.2-alpha.2

I have updated to 0.0.2-alpha.2 today. After the update no units are displayed. Values are still shown as expected but without their units. Do I need to activate this somewhere? Units are still defined correctly in the "Source"-Configuration of the element.

Already done that. After I hit save a timestamp is displayed for all values until the next value is received. Looks funny for a minute but that's fine, however units don't return.
I have also tried to delete and reinstall the adapter (did a restore via the configuration object). Rolling back to the former version brings back the units but then the icons are gone.

Please check, if the option for that timestamp is enabled in element options. Set that to none.

Already done that. No impact as well. Data object also shows empty values for units:
"unit": {     "2": "",     "9": "",     "16": "",     "18": "",     "19": "",     "26": "",     "34": "",     "35": "",     "41": "",     "42": "",     "43": "",     "44": "",     "57": ""   },
I will try to start with some objects from scratch to see if it works.

Data object is feed from configuration object through Adapter interface.
Did you check, if option is set to "none"?
As you told, the timestamp is written - that is a new function and is not enabled by default.

Yes. I have checked that. Seems like I messed up my config somehow as the sample config after starting from scratch works as it should. I'll just build from scratch, probably faster than debugging my config - this is an Alpha Version anyway.

You can compare your config with the sample one and check.

I have no Idea what might have been the issue - didn't seen to be too different. I have rebuilt my dashboard exactly as it was before and everything works as expected now.

Strange. Do you perhaps have the configuration object, which caused the issue?
Might be possible, to have a look into it.

Sure.
config.json.txt

Thanks. Will have a look into it and give some feedback later tomorrow.

Hi,
i am sorry, but i was not able, to reproduce this issue on my test-system.

No worries. After rebuilding the dashboard everything seems to be working fine now.


CI/CD integration

If several unit tests are written, then introducing continuous integration for this project is appropriate.
Travis-CI and AppVeyor are two of the most popular CI/CD service providers. Should give them a try.
If we need some templates, give a look at trust.

Done at eeadfd1ad7e0d33978441b35f279842bc5e40b8f


Cannot connect to runtime process timeout after 10000 ms running Python on VS code

(reason : Cannot connect to the target connect ECONNREFUSD <IP_ADDRESS>:9229).
There is a Open launch.json button .  launch.json has an Add Configuration button, but that is of no help to me.  There is a debug heading 'Launch Program which is probably causing the problem; Tried to uninstall VS Code and install, but had no way of changing the settings so that a reinstall left me with the same problem.

@Alknoy Can you follow the bug template and provide additional information? Also please provide repro steps on what you were trying to do. There isn't enough detail here to understand the issue.

@Alknoy Closing this for now, please provide repro steps and additional info as requested.


Cursor Highlight does not appear on the start menu

Microsoft PowerToys version
0.55.2
Running as admin

[X] Yes

Area(s) with issue?
Mouse Utilities
Steps to reproduce
1- Open window start
2- double Ctrl
The highlighter appears underneath the menu
https://user-images.githubusercontent.com/4522358/154796560-f1d0bd9b-4b15-41ef-8860-570a293a94c7.mp4
✔ Expected Behavior
The highlight appear over the menu
❌ Actual Behavior
The highlight appear underneath the menu
Other Software
Window 10 build 19044.1566

Yes, it's a known issue. Haven't found a solution (afaik). See #14178 and #14718.

Duplicate of #14718


Version catalogs for compose-1.1

Removed the buildSrc module in favour of version catalogs. This PR is for the compose-1.1 branch.
Summary of changes:

Created the libs catalog: gradle/libs.versions.toml.
Migrated all build files.
Purged buildSrc.

@raamcosta I've cherry-picked the first couple commits and changed the version to the ones in this branch.

@raamcosta #215.


Timer intervals revised

Added function to replace text with "scifi" text
Added 'keepChars' array option to prevent replacing certain characters
Invervals are now responsible for clearing themselves
Expose interval continues until reaching end of content string
Scifi interval continues until display text matches original content
Config values renamed to 'scifiSpeed' and 'exposeDelay' for clarity
Support added for using a scifiSpeed higher than exposeDelay value

This request includes revised interval functions and should resolve the concern in #11 without undoing the fix for #4, which means if there are no concerns with these changes, we can merge this change and hopefully be able to close both of those open issues.

I've switched to using indexOf as requested, and also updated the documentation to reference the new set of options.


[1.16.2] Compatibility issue with JEI

Issue
Game client crashes when I try to open any of the CFB books.
Crash Report
Versions
Forge: 1.16.2 - 33.0.20
Cooking For Blockheads: 1.16.2 - 9.1.0
Just Enough Items: 1.16.2 - <IP_ADDRESS>

I have a similar issue with JEI, but also had issues with cooking table (it would  crash or wouldn't show the UI and just darken the screen but not show UI)


NC Refresh Optimization

Added API on CNS:
If received  "GET": Return all NCs in CNS's state file to DNC in order to check if NC refresh is needed
If received "POST": Store all the NCs (from the request body that client sent) into CNS's state file
Added unit test.

@bohuini is this a conceptually different change than https://github.com/Azure/azure-container-networking/pull/1544? If not, please don't open new PRs for the same feature, continue on the same PR - it provides historical context and documentation.

duplicate


Fix: increase stability of integration tests

Is there an existing proposal for this?

[x] I have searched the existing proposals

Is your feature request related to a problem?
Commonly failing tests:
BrokerSessionIT.closeQueueOnPendingConfigureRequest:2890 expected:<SUCCESS> but was:<TIMEOUT>
BrokerSessionIT.inboundWatermarksTest:5621->verifyQueueControlEvent:187 Expected 'QueueControlEvent', received 'com.bloomberg.bmq.impl.events.BrokerSessionEvent'
SessionIT.testQueueFlushDown:3566 Runtime com.bloomberg.bmq.BMQException: Failed to start session: TIMEOUT

Describe the solution you'd like
Check timeouts, check multithreaded problems
Alternatives you considered
No response

Temporarily disabled 2 ITs not working stable on slow hosts:
https://github.com/bloomberg/blazingmq-sdk-java/pull/27
This issue remains opened until we fix them.

See this comment, this might be the common cause of failing tests on slow hosts
https://github.com/bloomberg/blazingmq-sdk-java/pull/27#discussion_r1318177152


[aes/rtl] Enhance alert output in AES FSM

Follow the change in PR #12898, we think it is better to immediately
output error signal when the state goes to default unknown state.
The reason is because if the attacker continues to drive the FSM to
unknown states, the alert actually won't fire.
Signed-off-by: Cindy Chen<EMAIL_ADDRESS>

this is the relevant discussion #12990

Thanks for the PR, this looks reasonable. I think in the past I did the same but removed it at some point because of potential combo loops. Could you please run this through AscentLint before me merge it?

Ofc, thanks for letting me know! I will get back to you after running the ascentLint :)

Hi @tjaychen do you mind helping me run AscentLint for this PR?

@cindychip lint runs look good! no combo loops.

Thanks Tim!


'stdlib.h' file not found

Hello,
I was trying to compile a simple hello world c program with stdlib.h included. The toolchain was configured with the chromium sysroot, and I was compiling it in an ubuntu 20.04 docker container.  While it was compiling, it threw a fatal error: 'stdlib.h' file not found.
I tried running the omp_test.c test using the sysroot toolchain configured in MODULE.bazel, and even that threw the same error. The error goes away when I install libc6-dev package, which contains the required headers.
Is there any additional configuration required to use the headers present in the sysroot? and avoid installing packages in the container?
some additional information:
I'm using #235 instead of master, since I'm using bzlmod

You definitely need to have libc headers until #155 is resolved. Other dependencies can be ncurses, libtinfo, etc. Container tests in tests/scripts/ can be informative to some extent. Please reopen if you want more information.


Scrollbar overhaul & more

This PR contains the following breaking changes:

MacosScrollbar has been completely overhauled and now  [more closely] resembles the native macOS scrollbar in appearance and behavior. Previously, it wrapped the material scrollbar, and now creates a custom scrollbar that extends
RawScrollbar. This resulted in the removal of several material-based properties for the scrollbar, and
ContentArea.builder is once again a ScrollableWidgetBuilder.
Removes material-based scrollbar properties from MacosScrollbarThemeData

Other changes:

Updates dart_code_metrics and related rules
Moved scrollbar.dart from /indicators to /layout, as it is a more appropriate location.

Pre-launch Checklist

[x] I have run dartfmt on all changed files
[x] I have incremented the package version as appropriate and updated CHANGELOG.md with my changes
[x] I have added/updated relevant documentation
[x] I have run "optimize/organize imports" on all changed files
[x] I have addressed all analyzer warnings as best I could

@whiplashoo I'd like your review on this one, if you have the time.

The new scrollbar looks really nice, almost identical to macOS one.
My only note: shouldn't it also appear when hovering over it?
On macOS, after scrolling and while the scrollbar is visible, hovering over any point of the track, activates the scrollbar's animation and thickness effect:
https://user-images.githubusercontent.com/9117427/217931104-46eb59a3-c6ca-4c73-831e-7b1798aa8795.mov
There is a handleHover method we can override in ScrollBar.

Thanks! Yes, the scrollbar should absolutely show the track on hover. I meant to add that but I forgot. I also don't think I knew about the handleHover override. I'll try and implement that tonight.
One thing, though, and I can't check on this now as I'm AFK, but I believe that the behavior of the scrollbar differs slightly between  Apple's apps (classic).

Yes, sure. I can see that on Finder, the inner scrollbar doesn't have that hover effect, for example, but it's there on Safari, Notes, and Reminders 🤷♂

@whiplashoo Alright then, I'm inclined to say that the default behavior should follow how the majority of apps behave. I'll work on adding this.
As an aside, if you have any tips on testing the scrollbar, I'm all ears. I've been banging my head against the wall for like a week about that!

Alrighty @whiplashoo, hover effect has been implemented!

Yes, that looks and works great!
Can't think anything on the testing side, though. Since the scrollbar is a wrapper around RawScrollbar, I am not sure if there is a lot to test, apart from the visual stuff.

Amazing! And that's a fair point.
Thanks so much for the review!

The DCM check failing is driving me NUTS!!!


Added copy to clipboard function, responsive landing page and implemented routes

Closes #5 , #3 , #1

@KshitijDarekar Kindly review this PR.

@Aniket-508 issue[#1 ] was already assigned to someone else. It won't be a problem if the assignee hasn't started working on it already but do check out if any issue is taken up by someone already the next time, to avoid conflicts.

@KshitijDarekar I have added inline-flex on the line 40. And regarding the issue 1, the assignee has already given green signal to assign it to someone else. Maybe you can merge this PR then. One more request, can you make the Landing page as difficult since I also made it responsiveness and for all other issues that this PR solves, you can keep them as you want.

@Aniket-508 any update?

@KshitijDarekar I have already done that. Please read the comments above.

@KshitijDarekar I have already done that. Please read the comments above.

you have added the classbutton-group-editor in the last commit, but I can't see the CSS for the same .🤔

@KshitijDarekar Extremely sorry, I forgot to stage that file. Now it is done.


Css error #505

.nav-sm .menu_section h3, .nav-sm .profile, .nav-sm span.fa{display: none;} -> .nav-sm .menu_section h3,.nav-sm .profile,.nav-sm .menu_section span.fa{display:none}
This will fix the error as suggested by @WhiteRavenProject on #505
Tested and until now didn't saw any breaking errors on the pages

Here you go :)


Folding@home

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?

Microsoft Reviewers: Open in CodeFlow

@andmagdo No need to close the pr. Just commit fixes and push it. 👍

/azp run

/azp run


Suggestion: Patchnotes in releases

Hi,
maybe it's would be great to have a changelog in the releases.
Regards.

The commits section tells you what the changes were suppose to fix as well as you can see the actual code changes

Actually as a plugin developer this would be very helpful to have at least a brief changelog summary for each release. There have been breaking changes before that has broken my plugin.
@nfarina thoughts?

@plasticrake What were your thoughts on implementing it?  Some sort of Wiki page? The only issue I foresee is that so many changes get pushed that it would be time consuming to add a new entry every time a commit is pushed

I have a changelog like this that is updated with each new new version. You can also have a workflow that updates the changelog when the relevant code is committed. When someone is ready to release a new version you can run something like git log --oneline --decorate to see the recent commits and make sure the changelog isn't missing anything.
Considering this is used by other plugin developers changes should be communicated. Although the issue I was referring to before was in HAP-NodeJS so unless that repo starts a changelog I wouldn't have known about the breaking changes either.

@plasticrake Hmm not exactly sure I am understanding the benefit of this over the commit log. Are you wanting a more indepth explanation of what is occurring in the commit or just an easier way to read the changes than the commit log? The other pertinent person to get involved in making this happen would be @KhaosT. What are your thoughts?

I mean ideally yes, having changelog attached to each version would make sense... however, most of changes these day only contains one or two commits so I felt it's about the same of reading the commit log between last release and current one?

Really, how's that hard to maintain a Changelog or Release Notes?
Commit list works, but that's not the perfect choise:
https://github.com/nfarina/homebridge/commits/master
Check the example:
https://github.com/arendst/Sonoff-Tasmota/blob/development/sonoff/_releasenotes.ino


Error when we use Third party mscgenjs modules: "define cannot be used indirect" with webpack v3.6.0 working fine with v1.12.15

Bug:
Third party modules like mscgenjs are not compatible with webpack v3.6.0  .Getting error Uncaught Error: define cannot be used indirect .
{
"name": "mscgenjs",
"version": "1.9.0",
"description": "Implementation of MscGen in JavaScript",
"main": "index.js",
"dependencies": {
"amdefine": "1.0.1",
"requirejs": "2.3.2"
},
Working fine older"version": "1.12.15".


[iOS] Badges do not show if initially set to blank

It appears that on iOS that the badge number does not show unless you initialize the item with a non-empty string. In looking at the iOS code, it appears that it only adds the badgeLabel to the view when it's not an empty string.

Any news on when this might be fixed? As far as I can tell, the issue is actually in MiniTabBar rather than this plugin. Workaround is to initialise each item with non-empty string and then set them all back to empty strings in the bottombar loaded handler before they appear.

Please provide code to reproduce the issue, thanks 👍

#75 is related to this issue. if the badges is not initially at the first place(like the following example). when updating the notification will not display the badges on ios.

public bottomBarItems: Array = [
new BottomBarItem(0, "Chat", "ic_home_black_24dp", "whitesmoke", new Notification("white", "red", "")),
];

the badges will update if i declare the bottomBarItem like the following:

public bottomBarItems: Array = [
new BottomBarItem(0, "Chat", "ic_home_black_24dp", "whitesmoke", new Notification("white", "red", "0")),
];

Should be fixed with the pre-release 4.0.0-rc1.1. Be careful, there is a lot of breaking changes.

package.json on the NativeScript still showing that the latest version is 3.0.8. how can I apply the changes ?

Yes just follow the readme from the repo you should be good to go. It
doesn't show up because it's an RC release
Le mar. 12 mars 2019 à 7:42 AM, marcuschoong<EMAIL_ADDRESS>a
écrit :

package.json on the NativeScript still showing that the latest version is
3.0.8. how can I apply the changes ?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/rhanb/nativescript-bottombar/issues/68#issuecomment-471876753,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AJCcO3V5qhdnee_1XEY6-XdayW2Klwx7ks5vV0xQgaJpZM4TYQEr
.

the latest release doesn't fix this issue. its still need to set the initially badge to 0 in order to display the badge when increase to 1.

Could you provide a repo or something to reproduce the issue ? Wich version are you using ?
It should be fixed


Create admin-only menus for editing assets

Closes #52

Link edit (#23: 13.webp in exemple) is shown twice


ModuleNotFoundError: No module named 'pymongo'

Hello !
I'm seeing a regression on the latest version when not using pymongo.
In core.py, an import for pymongo was added for type hints: https://github.com/python-cachier/cachier/blob/50fb9dafaa7f8d127bca738d07165054d82e910d/cachier/core.py#L23
However the case where pymongo is not installed is not handled, which results in a
ModuleNotFoundError: No module named 'pymongo' when importing cachier without pymongo.

hi @corenting, I also experienced this issue and sent a PR in #126 with a fix.

A solution by the venerable @cthoyt was released in v2.2.1.

Thanks for the quick fix and the new release !


Rotating video clips video

The Transform widget can rotate, translate, and scale a widget without actually changing its underlying position or size. However, this can sometimes be a problem. I have turned on widget inspector to confirm Transform widget's behaviour.
https://github.com/LeGoffMael/video_editor/blob/f874eb19d76ab7d18f372445c225c471988c2860/lib/ui/transform.dart#L18
https://user-images.githubusercontent.com/8436039/235014615-d1e12adf-ec53-4917-a83f-d2b5ee28066a.mp4

#114 could resolve the problem

https://www.youtube.com/watch?v=qOl8bvus788


[idea] declarative JS

JSX is nice when we have tooling, but we may not want to run Babel in production if we don't have tooling.
Maybe it would be nice to have some sort of declarative form in JS. For example,
import {header, h1} from 'surplus/elements'

const seconds = S.value(0)
setInterval(() => seconds(seconds() + 1), 1000)

const TodoHeader = ({ addTodo }) => [
    [header, [
        ['class', 'header'],
        [h1, [
            'todos ', seconds,
            ...
        ]]
    ]]
]

or maybe something more explicit with objects, but more indentation:
import {header, h1} from 'surplus/elements'

const seconds = S.value(0)
setInterval(() => seconds(seconds() + 1), 1000)

const TodoHeader = ({ addTodo }) => header({
    class: 'header',
    children: [
        h1({
            children: [
                'todos ', seconds,
            ],
        })
    ]
})

In the end, they'd compile to basically the same thing as the following would:
const seconds = S.value(0)
setInterval(() => seconds(seconds() + 1), 1000)

const TodoHeader = ({ addTodo }) =>

todos {seconds}

    ...

Then, JSX would be a convenient way to use HTML-like syntax if that is wanted, and when tooling is available.

I played around with it more and took it past proof of concept. I've incorporated it into the core rendering library so that MobX-JSX and KO-JSX can use HyperScript as well.
I've update the example on codepen to show it's usage in Solid-StandAlone (I made the Babel dep optional). I also tried out HyperScript Helpers and coded out the JS Frameworks Benchmark for fun. Definitely noticeably slower. More comparable to Preact in performance. I'm sure I can make it faster but I doubt I'm going to spend time at this point optimizing this. I think the syntax is more inline with what you are thinking. It looks like:
import { useState } from 'solid-js';
import { h, selectWhen } from 'solid-js/dom';
import hh from 'hyperscript-helpers';

const { div, style, h1, span, a, button, table, tbody, tr, td } = hh(h);

const Button = (id, text, fn) =>
  div('.col-sm-6.smallpad', [
    button(`#${id}.btn.btn-primary.btn-block`, {
      type: 'button', onclick: fn
    }, text)
  ])

const RowInsert = () => {
  const [ state, setState ] = useState({ data: [], selected: null });

  /* Snip ....handler functions.... */

  return div('.container', [
      div('.jumbotron', div('.row', [
        div('.col-md-6', h1('SolidJS Keyed')),
        div('.col-md-6', div('.row', [
          Button('run', 'Create 1,000 rows', run),
          Button('runlots', 'Create 10,000 rows', runLots),
          Button('add', 'Append 1,000 rows', add),
          Button('update', 'Update every 10th row', update),
          Button('clear', 'Clear', clear),
          Button('swaprows', 'Swap Rows', swapRows)
        ]))
      ])),
      table('.table.table-hover.table-striped.test-data', tbody({onclick: clickRow}, [
        h.each(() => state.data, row =>
          tr({model: row.id}, [
            td('.col-md-1', row.id),
            td('.col-md-4', a(() => row.label)),
            td('.col-md-1', a({action: 'remove'}, span('.glyphicon.glyphicon-remove'))),
            td('.col-md-6')
          ])
        , selectWhen(() => state.selected, 'danger'))
      ])),
      span('.preloadicon.glyphicon.glyphicon-remove', {attrs: {'aria-hidden': true}})
    ]);
}

In any case I think I've resolved this issue. Thanks for suggesting it. It wasn't as hard as I anticipated and I think it brings some real value.

Updating on this I've figured out a way to use Tagged Template Literals that should have much less overhead than the HyperScript version. I will tuning it in the coming weeks but I imagine it should run close to JSX in performance. It is still early but you can check it out here.  https://github.com/ryansolid/lit-dom-expressions


Feature request: create an ice-breaking game in a group chat

Anyone who sends a text "/☕" in the group with the topic "The Big Community" will trigger the game as the initiator.
Two random members in the group will be selected to join the game as gamers
Create a new group with the initiator and gamers for them to chat with each other
The bot will send the first message, such as "Welcome to the coffee chat group! This group is requested by ${INITIATOR_NAME}, and your random partners are ${GAMERS_NAME}. I hope you guys have a good ice-breaking time!"
The new group will be dismissed if there are less than 3 people in it.

@ellipsis-dev Please implement this game; thank you very much.


Remove hyphen in backbone datatype name generation

Remove hyphen in backbone datatype name generation

Remove hyphen in backbone datatype name generation

Could you please add a sample of how these would look-like after this change?

@Thiyanwso2

This - is make the generated Ballerina project to malfunction and hence we need to avoid adding hyphens in the generated code.

@Thiyanwso2
This - is make the generated Ballerina project to malfunction and hence we need to avoid adding hyphens in the generated code.

Thanks


Update cognitive-search-working-with-skillsets.md

Working with Heidi on this project

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

#sign-off


Allow different components to be used in templates

Hi!
In my opinion it would be nice to be able not to only specify template but also the component for typeahead templates. Recently I've had to create a footer with custom action and this is what I've came to:
import Ember from 'ember';
import AupacTypeahead from 'ember-aupac-typeahead/components/aupac-typeahead';

const {run, Component} = Ember;

const Key = {
  BACKSPACE : 8,
  DELETE : 46
};

export default AupacTypeahead.extend({
  suggestionComponent: Component,
  notFoundComponent: Component,
  pendingComponent: Component,
  headerComponent: Component,
  footerComponent: Component,

  initializeTypeahead() {
    const self = this;

    //Setup the typeahead
    //noinspection ChainedFunctionCallJS
    const t = this.$().typeahead({
      highlight: this.get('highlight'),
      hint: this.get('hint'),
      minLength: this.get('minLength'),
      classNames: this.get('typeaheadClassNames')
    }, {
      component: this,
      name: this.get('datasetName') || 'default',
      display: this.get('display'),
      async: this.get('async'),
      limit: this.get('limit'),
      source: this.get('source'),
      templates: {
        suggestion: function (model) {
          const cmp = self.get('suggestionComponent');

          //noinspection ChainedFunctionCallJS
          const item = cmp.create({
            model: model,
            layout: self.get('suggestionTemplate'),
            parentView: self
          }).createElement();
          return item.element;
        },
        notFound: function (query) {
          const cmp = self.get('notFoundComponent');

          //noinspection ChainedFunctionCallJS
          const item = cmp.create({
            query: query,
            layout: self.get('notFoundTemplate'),
            parentView: self
          }).createElement();
          return item.element;
        },
        pending: function (query) {
          const cmp = self.get('pendingComponent');

          //noinspection ChainedFunctionCallJS
          const item = cmp.create({
            query: query,
            layout: self.get('pendingTemplate'),
            parentView: self
          }).createElement();
          return item.element;
        },
        header: function (query, suggestions) {
          const cmp = self.get('headerComponent');

          //noinspection ChainedFunctionCallJS
          const item = cmp.create({
            query: query,
            suggestions: suggestions,
            layout: self.get('headerTemplate'),
            parentView: self
          }).createElement();
          return item.element;
        },
        footer: function (query, suggestions) {
          const cmp = self.get('footerComponent');

          //noinspection ChainedFunctionCallJS
          const item = cmp.create({
            query: query,
            suggestions: suggestions,
            layout: self.get('footerTemplate'),
            parentView: self
          }).createElement();
          return item.element;
        }
      }
    });
    this.set('_typeahead', t);

    // Set selected object
    //noinspection NestedFunctionCallJS
    t.on('typeahead:autocompleted', run.bind(this, (jqEvent, suggestionObject /*, nameOfDatasetSuggestionBelongsTo*/) => {
      this.set('selection', suggestionObject);
      this.sendAction('action', suggestionObject);
    }));

    //noinspection NestedFunctionCallJS
    t.on('typeahead:selected', run.bind(this, (jqEvent, suggestionObject /*, nameOfDatasetSuggestionBelongsTo*/) => {
      this.set('selection', suggestionObject);
      this.sendAction('action', suggestionObject);
    }));

    //noinspection NestedFunctionCallJS
    t.on('keyup', run.bind(this, (jqEvent) => {
      //Handle the case whereby the user presses the delete or backspace key, in either case
      //the selection is no longer valid.
      if (jqEvent.which === Key.BACKSPACE || jqEvent.which === Key.DELETE) {
        //noinspection ChainedFunctionCallJS
        const value = this.get('_typeahead').typeahead('val'); //cache value
        this.set('selection', null);
        this.sendAction('action', null);
        this.setValue(value); //restore the text, thus allowing the user to make corrections
      }
    }));

    //noinspection NestedFunctionCallJS
    t.on('focusout', run.bind(this, (/*jqEvent*/) => {
      //the user has now left the control, update display with current binding or reset to blank
      const model = this.get('selection');
      if (model) {
        this.setValue(model);
      } else {
        this.setValue(null);
      }
    }));
  }
});

Notice that I've had to copy almost all the code just to change small part of it :)

Yep this seems reasonable.

Also I've "hacked" dynamically created components. Notice:
parentView: self

Is the reason for parentView: self because you wanted your actions to be handled by the controller and not swallowed by aupac-typeahead? We need a good and official way of handling this as there will be an unknown number of actions on the component.  Ideas are welcome.

Actually I was trying to trigger action in aupac-typeahead. Maybe something has changed now since I am using Ember 1.13.10. However action wasn't triggered and I've received something like

component::ember472 has no handler for action XXX

Yes, this was sort of hack - in order to write {{action 'someAction' target=parentView}} in component template. However this doesn't seem a correct way to me. Main problem here is that dynamically create component isn't linked to its parent in correct way. I've currently haven't studied more correct way since this change had to be made ASAP but I am sure that it exists.

Hi,
we are trying to trigger actions from the template, too. This has to fail as the template is rendered in its own component and therefore does not have access to any other scope. Here is a demonstration:
https://ember-twiddle.com/b90d4e28f787686e219d2fa133727f7c?openFiles=templates.suggestion-template.hbs%2C
I would be happy to help with implementing this (maybe in combination with allowing the use of different components), but I am not sure what might be the best way to approach this.


Update cumulusci.yml for release notes statement

Critical Changes
Changes

Add link to Known Issues in release notes statement.

Issues Closed
Community Ideas Delivered
Features Intended for Future Release
Features for Elevate Customers
New Metadata
Deleted Metadata

@melissabarber What is the status for this PR?

@melissabarber What is the status for this PR?

I'm ready for your and @jstvz's reviews to take this to check it in.


:arrow_up: selenium 2.47.1

@nikolas thank you. You should change https://github.com/cobrateam/splinter/blob/master/setup.py#L26 too.


Changes 'side affects' to 'side effects'

Coverage remained the same at 95.413% when pulling fdf4c7451482d34079fc59d89345b6b238e1cc61 on filipbarak:patch-1 into f25888ffc67d21beaa1572b53ab6704a67266b64 on observing:master.

Coverage remained the same at 95.413% when pulling fdf4c7451482d34079fc59d89345b6b238e1cc61 on filipbarak:patch-1 into f25888ffc67d21beaa1572b53ab6704a67266b64 on observing:master.

Thanks <3


Change Toggle window focus hotkey

Is it possible to change this hotkey?

Yes.

(The option is under the plugin's own settings instead of Obsidian's regular hotkey settings so it can be set as a global hotkey, and not just an in-app hotkey.)

Thank you! I'm really sorry about that.
On Sun, Oct 29, 2023 at 4:37 PM Tom @.***> wrote:

Closed #32 https://github.com/dragonwocky/obsidian-tray/issues/32 as
completed.
—
Reply to this email directly, view it on GitHub
https://github.com/dragonwocky/obsidian-tray/issues/32#event-10799606753,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A2RAIECVI4IDASUCOPX3543YBY2FHAVCNFSM6AAAAAA6UW2ZH2VHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJQG44TSNRQGY3TKMY
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
Regards,
Dharshan Jayalath


[stable/grafana] Set plugins as a list

The way we currently set plugins as a comma separated list makes it impracticable to extend.
Having them as a list is more convenient.

CLA signed

@lostick please sign the DCO - https://www.helm.sh/blog/helm-dco/index.html

@cpanato sure thing

/ok-to-test

/lgtm

@unguiculus this is fine to be merged

/lgtm

/approve


Shopify Customer Orders Page

Describe the feature you'd like to request
A page accessible only to logged in customers that displays all current and previous orders
Describe the solution you'd like
A new useOrders hook existing in packages/commerce and packages/shopify that queries all orders for the current logged in customer. Some of the graphQL data would include:

date
fulfillment status
line items
addresses
payment method
subtotal
taxes
order total
link to track the order
etc.

Describe alternatives you've considered
I've tried to add on to the current getCustomerQuery in packages/shopify/src/customer/use-customer.tsx but it only returns the order id and name. Anything else I add inside the node breaks the query and returns a field not found error. I think this might be because it's setup to query the Storefront API and not the Admin API.
I've also tried to duplicate the useCustomer hook to use as a starting point, but haven’t had any luck yet unraveling the code.
Any help I can get to solve this or with finding the right direction would be greatly appreciated. If I can't figure this out, I think I'll have to switch to liquid and abort this headless solution.

You could have a look at this: https://shopify.dev/api/admin-graphql because as far as I know, order information etc comes from the Admin API

Based on experience, stick with Liquid and do not even attempt headless, unless you want to spend a year debugging, custom coding etc. While this repo is an amazing achievement, it is at best a "skeleton" and requires extensive custom coding to actually get it to production. Never mind trying to unravel the code; it is a beast and overly complex.
That said, if you are glutton for punishment, like myself, there is no harm really, as you can learn a ton working with this repo. However, if you going to use Shopify, I'd recommend Shopify Hydrogen instead, because at least then you will get support from Shopify and their repo is much more active. But, Shopify Hydrogen probably will take another year or so until it is actually somewhat stable; they are already re-working their entire cart after just releasing it.

@osseonews oh yikes. leave it to me to undertake something entirely new for a client site. :| i was also slowing hooking it up to sanity for easy page & blog post building. hahaha wow.
do you have any recommendations for a starting liquid theme?

Sorry, I don't use liquid anymore, I'm so deep into headless with NextJs, I'm just sticking with it now.
Sanity is a good choice for a headless CMS. If you go down that route, I'd check out CommerceLayer.io - they have a good Sanity Starter and a bunch of good React repos that are far easier to implement than nextjs commerce. It really depends on what your client is willing to pay. Headless is just a ton of work and way more to maintain than liquid. But, over time headless will get easier as better frontends are released.

Sorry, I don't use liquid anymore, I'm so deep into headless with NextJs, I'm just sticking with it now.
Sanity is a good choice for a headless CMS. If you go down that route, I'd check out CommerceLayer.io - they have a good Sanity Starter and a bunch of good React repos that are far easier to implement than nextjs commerce. It really depends on what your client is willing to pay. Headless is just a ton of work and way more to maintain than liquid. But, over time headless will get easier as better frontends are released.

Well, its not as bad as you say it is haha. I started 2 years ago and made my first Headless Shopify store with Gatsby. Yes, there is custom code involved like getting the account and order pages to work, custom filtering, CMS and forms, but other then that it's fine and reasonable.
I made the mistake to get into Gatsby for these kind of things. It's working, but getting it multilingual is really a pain. For this reason I am now moving towards Nextjs and basically port my existing store. But I have no clue what is the actually reason behind this repo. Is it a proof of concept? Is it actually actively maintained? I do love the concept of those hooks, but it's really complex to get into and built your own hooks. Seems much easier to start from scratch and just query the data you need for each page.
Maybe look into Hydrogen, which is a more native solution.

If it weren't so difficult to create new data hooks and customize the commerce packages, this wouldn't be a bad repo. Hydrogen does seem exciting and I'll definitely try it out eventually. I found this neat marketing page all about their latest updates: https://www.shopify.com/editions/summer2022/dev.
For my client's site, I decided to switch to customizing the Dawn Shopify theme and it's been going really well so far. The updates to the theme customizer are great (it has been about five years since I last used it) and the shopify cli is easy to use.  Plus they have a ton of documentation, so I won't have to worry about getting stuck.
I did figure out how to add on to the customer query and get basic order data from the storefront api, so I had a simple order page up and running. But not knowing what other hurdles I might face in the future made it an easy decision to drop vercel commerce.
Thanks for your recommendations and comments, @DennisKraaijeveld and @osseonews.

Hey, I think you made a good choice with the Shopify theme. This repo still is a great reference for any headless project you might want to pursue. The code structure while complex is well though out .
However, in case you are interested in headless, I think the best way to go is in stages. First get all your products (ideally stored in a database but you can also use a JSON file) and display them in NextJs frontend. This is relatively easy and this repo greatly complicates that basic 1st step, but you can copy their product slug file to set it up correctly. I would then recommend just using Stripe and store a purchase link in the database, and on the product page just have the customer go to Stripe for checkout. Stripe's checkout pretty much has every feature any customer would want. Use stripe's webhooks to store orders back in the database. Over time you can add on more features for logged in users, like viewing orders etc. But, initially, just keep it simple by displaying products and relying on Stripe. Headless has been made extraordinarly complex for no particular reason, and the prices of the headless platforms are obscene relative to what you can get out of the box from Shopify and/or Stripe checkout.

@osseonews Did you by chance use this with Shopify Plus?
I loved the entire experience and everything I learned from making a fully functioning commerce site with Shopify using this template. It's truly on it's way to greatness.
The only thing that I feel Shopify fell short on is linking the checkout to the user.
How did you work around the requirement for the multipass authentication call to link the shopify checkout to the customer?

@Vinna152 No. I used another e-commerce backend, which I'm not happy with, so I won't recommend it.  So long-term Hydrogen seems like a better choice for Shopify users. Personally, I think Shopify made some terrible choices with Hydrogen, so I have no interest really in developing on it right now.
As I stated above, this repo is amazing, and you can learn a ton from it. The problem is that none of the providers (except maybe BigCommerce) actually support it, so you are left on your own for alot of functionality, which means that long-term maintaining any store build on this platform will prove very difficult (based on my experience). The mistake NextJs made here is precisely trying to be able to have a platform for every provider, as the basic structure of the platform would never be appealing to any provider due to competitive reasons. By trying to build a platform like this, they have made it needlessly complex. I mean useSwr is incredibly simple hook from Vercel, but this repo makes it into quite literally the most complicated data fetch library I have ever seen. That being said, I think this is an awesome repo to get started and I think you can borrow what is good from it to build a store and then just discard the unnecessary pieces. That's at least what I am going to do, going forward.

You can easilly implement the feature you want by adding the Admin API to your implementation. This is what I did in order to implement a wishlist feature to the shopify provider. You can check my PR here: https://github.com/vercel/commerce/pull/724 and you can ask me anything you need help with. It's totally doable though and it integrates to the current repo perfectly well without causing any problems. Everything in my implementation is strictly typed too and compatible with existing types.

As a follow up on this issue, Shopify acquired Remix today, and basically admitted that RSC (React Server Components) doesn't work. At the same time, Vercel is making a big bet on RSC with NextJs 13, which would undermine useSwr which is the basis of this entire repo. Obviously,  they will have to refactor this entire repo to use RSC. This would of course make the codebase much simpler, as the reason for what I see as the code bloat in this repo is due to useSwr (see the Remix version of this Commerce). However,  once they refactor it they will face the same problems that Shopify had with RSC, and I don't see how they will fare any better. So overall, I don't see how NextJsCommerce will evolve and I feel that it remains risky to build on, in my opinion.  It is unfortunate, as this is an amazing project which I learned a ton from, but Vercel seems to have not been focused on it (more like a side project for them) and is making some strange long-term decisions now with regards to RSC that I don't think will work with Commerce type websites.

Here's a link to the announcement article if anyone's curious.
I'll definitely be following along to see how Hydrogen shapes up.

Hey there! Thank you for your contribution. We have decided to take Next.js Commerce in a new direction and will be closing out current PRs and issues due to this change. Please see this PR for more details: https://github.com/vercel/commerce/pull/966


Very unusable example, No Podfile etc.

@samkohli
Now, example app of DifferenceKit is using Swift package manager for iOS, so you can simply run app with Xcode 11.


Custom Controller does not move Robot

Hello together,
I programmed a custom controller 'pseudo_bike' with ros2_control. With classic gazebo, it does run and I can move the robot with teleop_keyboard by changing the /cmd_vel value.
I tried to do the same thing with Fortress Gazebo. It loads the controller nicely with gz_ros2_control (Compiled from Source with Fortress and Ros2 Humble). But strangely by changing the /cmd_vel value, it does not move.
 [ruby $(which ign) gazebo-5] [INFO] [1669391816.505566917] [controller_manager]: Loading controller 'pseudo_bike'

[ruby $(which ign) gazebo-5] [INFO] [1669391816.527176421] [controller_manager]: Setting use_sim_time=True for pseudo_bike to match controller manager (see ros2_control#325 for details)

 [ruby $(which ign) gazebo-5] [INFO] [1669391816.537294163] [controller_manager]: Configuring controller 'pseudo_bike'

[ruby $(which ign) gazebo-5] [INFO] [1669391816.560380717] [pseudo_bike]: On activate: Initialize Joints

 [ruby $(which ign) gazebo-5] [INFO] [1669391816.560452567] [pseudo_bike]: Get Wheel Joint Instance

[ruby $(which ign) gazebo-5] [INFO] [1669391816.560736187] [pseudo_bike]: Get Wheel Joint Instance

 [ruby $(which ign) gazebo-5] [INFO] [1669391816.560771817] [pseudo_bike]: Get Steering Joint Instance

[ruby $(which ign) gazebo-5] [INFO] [1669391816.560890867] [pseudo_bike]: Get Steering Joint Instance

[ros2-7] Sucessfully loaded controller pseudo_bike into state active

[INFO] [ros2-7]: process has finished cleanly [pid 1367284]

Do I have to use something special in order to use gz_ros2_control package for fortress?
Edit:
In the meantime I've tried all the examples added in gz_ros2_control. I could manage to move the block through the effort, velocity and position controller.
The tricycle and diffdrive controller has the same problem as my custom controller.
And Thanks for the help

The tricycle and diffdrive controller has the same problem as my custom controller.

Strange, I am just running a robot using these controllers... So controllers are working, and (hopefully) we just have a configuration issue.
Please check if your controller is actually started using ros2 control list_controllers command.
I could be that we have a regression on humble, I am currently working on Rolling

Yes I tried on my controller.
Ros2 control list_controllers give following feedback:
`ros2 control list_controllers
joint_state_broadcaster[joint_state_broadcaster/JointStateBroadcaster] active
pseudo_bike         [pseudo_bike/PseudoBike] active `
Seems okay

Are you sure you need the gz plugin from source? Or...do you have the gz
plugin and/or ros2_control installed from binary as well? There could be
more than one ways for these things to break as overlays don't work well
with ROS2.
On Sun, 27 Nov 2022, 11:15 Pinoroby, @.***> wrote:

Yes I tried on my controller.
Ros2 control list_controllers give following feedback:
`ros2 control list_controllers
joint_state_broadcaster[joint_state_broadcaster/JointStateBroadcaster]
active
pseudo_bike [pseudo_bike/PseudoBike] active `
Seems okay
—
Reply to this email directly, view it on GitHub
https://github.com/ros-controls/gz_ros2_control/issues/101#issuecomment-1328223949,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA24PYNGWCLPVREKDDHQIKLWKM7DZANCNFSM6AAAAAASLOM52Q
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

No I don't need the gz_ros2_control from source. Until now, I just have the gz_ros2_control package from source, since I cannot find it in the ros-humble  repository.

Hi guys,
I am also facing a similar problem, the examples don't work with the exception of cart_example_velocity.launch.py.
I'm using :

ROS2 galactic
ign-sensors6 - (from sources)
ign-gazebo6 - (from sources)
gz_ros2_control - (branch galactic, from sources in my ros2 ws)
ros_gz - (branch galactic, from sources in my ros2 ws)

I need to build ign-sensors6 from sources to fix some bugs, I also need the segmentation camera sensor available only since fortress, so building needed I guess.
Behavior :
When starting ros2 launch ign_ros2_control_demos diff_drive_example.launch.py, everything is going well and I end up with :
[ign gazebo-1] [INFO] [1670195659.585000648] [controller_manager]: Loading controller 'diff_drive_base_controller'
[ign gazebo-1] [INFO] [1670195659.594966027] [controller_manager]: Setting use_sim_time=True for diff_drive_base_controller to match controller manager (see ros2_control#325 for details)
[ign gazebo-1] [INFO] [1670195659.603058745] [controller_manager]: Configuring controller 'diff_drive_base_controller'
[ros2-5] Sucessfully loaded controller diff_drive_base_controller into state active
[INFO] [ros2-5]: process has finished cleanly [pid 59725]

$ros2 control list_controllers
joint_state_broadcaster[joint_state_broadcaster/JointStateBroadcaster] active
diff_drive_base_controller[diff_drive_controller/DiffDriveController] active

$ros2 control list_hardware_interfaces
command interfaces
        left_wheel_joint/velocity [claimed]
        right_wheel_joint/velocity [claimed]
state interfaces
         left_wheel_joint/position
         left_wheel_joint/velocity
         right_wheel_joint/position
         right_wheel_joint/velocity

Seems to me everything is nicely started but unfortunately, the robot is not moving at all ... ;)
I tried to publishing on cmd_vel topic from rqt or by using ros2 run ign_ros2_control_demos example_diff_drive without success..
Do you have any ideas why ?

Ok this was an easy fix, the only question remaining is, do you test ? Is it working elsewhere ?
We need the bridge to publish the clock for the controller_manger node.
bridge = Node(
    package='ros_ign_bridge',
    executable='parameter_bridge',
<EMAIL_ADDRESS>    output='screen'
 )

Because of line in ign_ros2_control_plugin.cpp:
  this->dataPtr->controller_manager_->set_parameter(
    rclcpp::Parameter("use_sim_time", rclcpp::ParameterValue(true)));

Hey I tried it finally.
Publishing the time was the solution thanks!