Add Elasticsearch component

Add the Elasticsearch resource.
Add the Elasticsearch client (Elastic.Clients.Elasticsearch)

Is there any plan to add these components to aspire?

We haven't got it on the backlog right now but we would definately be open to having a PR for it if you are up for contributing. If not we can pop this on the backlog and see when someone can pick it up. Seems like it would be a useful addition.

I am ready to contribute to this. if there is nothing to say I can start it.

Elasticsearch resource added in this commit but for the client, Elastic.Clients.Elasticsearch package is not in the package sources. only Elasticsearch.Net is available (already deprecated)

Hey, @Alirexaa. Thanks for starting on this. It's something we'd been planning to do at Elastic, but we hadn't managed to prioritise it yet. I might have some time in a week to send on this. I'm happy to review and build on what you're doing. We had some thoughts on the different options for resources as it would be useful to support the local docker container but also Elastic cloud for ease, too. I'm OOO this week, but I'm happy to discuss more next week.

@stevejgordon  ... @Alirexaa is getting pretty close to have this done and ready to merge. If you wanted to take a look now would be a good time before it merges!


Rename MISO LMP Weekly to Real Time 5 Min Final

Summary

Renames MISO().get_lmp_weekly() to MISO().get_lmp_real_time_5_min_final() because these 5 minute LMPs are final

Details

see one failing test in MISO that appears related: test_get_lmp_real_time_5_min_final_historical_date_range


Allow upload to GH with new LC UI

the success result in new version is
<span data-e2e-locator="submission-result">Accepted</span>


Trade bug - not enough ETH to cover gas

Overview
When connected to MetaMask on mobile any trade from ETH to ERC-20 comes up with an error "not enough ETH to cover gas"
References and additional details

Attempt to trade ETH for any ERC-20 asset and notice the error of “Not enough ETH to cover gas”

Acceptance Criteria
Users should be able to complete ETH to ERC-20 trades without error
Need By Date
n/a
Screenshots/Mockups
n/a
Ownership

[X] If my bounty needs engineering or needs product I have added the respective labels on the right
[X] As the sponsor of this bounty I will review the changes in a preview environment (ops/product) or review the PR (engineering)

Estimated effort
4 hours
Sponsor / Stakeholder
@MBMaria @0xdef1cafe
Bounty Hunters

Join our discord
Include an expected timeline for you to complete work in the work plan when you apply for this bounty!
Please refer to this link for some basic info
Please do not start work on this issue until you are approved in Gitcoin.

This is probably not related to ios or even Metamask, because I'm going through the same thing with Portis wallet connected, both on chrome and safari desktop on the macOS.

re-reviewing this. It sounds like this is just a gas estimate bug that needs addressing.

This has been fixed


Not allowed in Visual Basic for Applications 7.1

Hey there!
I'm getting syntax compile-time errors when using this operator (and the "+=" operator) in Visual Basic for Applications 7.1 (2012). Just a heads up because you might want to warn people about this.

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

ID: 9428c377-2798-2955-b41b-353bcf64c150
Version Independent ID: 0e08839d-6f7d-ae82-293d-504b3802a93b
Content: &= Operator (Visual Basic)
Content Source: docs/visual-basic/language-reference/operators/and-assignment-operator.md
Product: dotnet-visualbasic
GitHub Login: @dotnet-bot
Microsoft Alias: dotnetcontent

Thanks for submitting your feedback, @justinTM. Unfortunately, https://docs.microsoft.com/dotnet/visual-basic documents Visual Basic .NET only, and not Visual Basic for Applications. Since differences between VBA and Visual Basic .NET are beyond the scope of the documentation, I'll close this issue.

You can find the list of VBA operators at https://msdn.microsoft.com/en-us/vba/language-reference-vba/articles/operators.


Azure Cache for Redis: DNS name is not available

I'm trying to set up the Azure Cache for Redis. For this you have to type a DNS name before you can crate it. The problem is that no matter what I type it says "the name ... is not available".
Dont tell me rho-test-redis-fgghfgkljcfcfgqweglkjhfdghfghfgh345543.redis.cache.windows.net, dfhfghfgjghjdgsgjtryertytughfgcfvbdf and dfgfsaddhjdfghfgy-54643425dfgdffdh are already taken ... because I do not think so.

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

ID: 5e83695b-8934-80ef-2f82-9fb42b5dd2f2
Version Independent ID: f14f110b-b01b-10d2-e8d6-075a591c8d24
Content: Quickstart to learn how to Use Azure Cache for Redis with .NET apps
Content Source: articles/azure-cache-for-redis/cache-dotnet-how-to-use-azure-redis-cache.md
Service: cache
GitHub Login: @yegu-ms
Microsoft Alias: yegu

@SpikePy Thank you for your query. We will investigate and update this thread .

@SpikePy Thank you for reporting this issue! We're sorry that you weren't able to create a cache. To help us figure out what could be causing this, would you mind sending a screenshot to<EMAIL_ADDRESS>so that we can look into it?

Sent you the example to the given mail address.

@SpikePy I hope you were able to get this resolved.
We will proceed to close this issue now.
If there are further questions regarding this matter, please comment and we will gladly continue the discussion.

No, nothing got solved, I didn't even got an answer to my mail. The webinterface still seems broken. Setting it up via azure-cli is possible but that does not mean that the webinterface is fixed!
Is this quality ok for you azure guys? Is this the azure standard---broken webinterfaces, but you could use the cli? I'm sure what platform not to recommend to our customers.

did you solved the problem? im having the same issue

for anyone that gets here, i have found creating the resouce group before hand and then selecting that resource group then creating a dns name works

I had the same UI field validation about the availability of the name, but just clicked 'Review + Create' anyway and it seemed to work...


GH-36610: [CI][C++] Don't enable ARROW_ACERO by default

Rationale for this change
Because it's not a required component.
What changes are included in this PR?

Are these changes tested?
Yes.
Are there any user-facing changes?
No.

Closes: #36610

Can you rebase and then merge?

Ah, OK. I'll do it.


HDFS-16230 Use trim() method returned value.

Strings being immutable, you need to use the trim() method return value.
Description of PR
Minor bug in TestStorageRestore.
How was this patch tested?
Issue is trivial.
For code changes:

[x] Does the title or this PR starts with the corresponding JIRA issue id (e.g. 'HADOOP-17799. Your PR title ...')?
[ ] Object storage: have the integration tests been executed and the endpoint declared according to the connector-specific documentation?
[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[ ] If applicable, have you updated the LICENSE, LICENSE-binary, NOTICE-binary files?

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 46s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  1s

codespell was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

+1 :green_heart:
test4tests
0m  0s

The patch appears to include 1 new or modified test files.

_ trunk Compile Tests _

+1 :green_heart:
mvninstall
31m 52s

trunk passed

+1 :green_heart:
compile
1m 24s

trunk passed with JDK Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04

+1 :green_heart:
compile
1m 15s

trunk passed with JDK Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10

+1 :green_heart:
checkstyle
1m  1s

trunk passed

+1 :green_heart:
mvnsite
1m 22s

trunk passed

+1 :green_heart:
javadoc
0m 56s

trunk passed with JDK Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04

+1 :green_heart:
javadoc
1m 24s

trunk passed with JDK Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10

+1 :green_heart:
spotbugs
3m  8s

trunk passed

+1 :green_heart:
shadedclient
21m 28s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
1m 14s

the patch passed

+1 :green_heart:
compile
1m 15s

the patch passed with JDK Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04

+1 :green_heart:
javac
1m 15s

the patch passed

+1 :green_heart:
compile
1m  8s

the patch passed with JDK Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10

+1 :green_heart:
javac
1m  8s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

+1 :green_heart:
checkstyle
0m 50s

the patch passed

+1 :green_heart:
mvnsite
1m 13s

the patch passed

+1 :green_heart:
javadoc
0m 46s

the patch passed with JDK Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04

+1 :green_heart:
javadoc
1m 19s

the patch passed with JDK Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10

+1 :green_heart:
spotbugs
3m  5s

the patch passed

+1 :green_heart:
shadedclient
20m 49s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

+1 :green_heart:
unit
226m 53s

hadoop-hdfs in the patch passed.

+1 :green_heart:
asflicense
0m 45s

The patch does not generate ASF License warnings.

321m 55s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3455/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/3455

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell

uname
Linux 4d4abcd5db1c 4.15.0-156-generic #163-Ubuntu SMP Thu Aug 19 23:31:58 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / 6189a8e38d2f0b6ab340b0e4a76a27da362f013e

Default Java
Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3455/1/testReport/

Max. process+thread count
2942 (vs. ulimit of 5500)

modules
C: hadoop-hdfs-project/hadoop-hdfs U: hadoop-hdfs-project/hadoop-hdfs

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3455/1/console

versions
git=2.25.1 maven=3.6.3 spotbugs=4.2.2

Powered by
Apache Yetus 0.14.0-SNAPSHOT https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 52s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  1s

codespell was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

+1 :green_heart:
test4tests
0m  0s

The patch appears to include 1 new or modified test files.

_ trunk Compile Tests _

+1 :green_heart:
mvninstall
31m 43s

trunk passed

+1 :green_heart:
compile
1m 22s

trunk passed with JDK Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04

+1 :green_heart:
compile
1m 14s

trunk passed with JDK Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10

+1 :green_heart:
checkstyle
1m  0s

trunk passed

+1 :green_heart:
mvnsite
1m 23s

trunk passed

+1 :green_heart:
javadoc
0m 57s

trunk passed with JDK Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04

+1 :green_heart:
javadoc
1m 30s

trunk passed with JDK Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10

+1 :green_heart:
spotbugs
3m 10s

trunk passed

+1 :green_heart:
shadedclient
21m 25s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
1m 12s

the patch passed

+1 :green_heart:
compile
1m 14s

the patch passed with JDK Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04

+1 :green_heart:
javac
1m 14s

the patch passed

+1 :green_heart:
compile
1m  7s

the patch passed with JDK Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10

+1 :green_heart:
javac
1m  7s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

+1 :green_heart:
checkstyle
0m 51s

the patch passed

+1 :green_heart:
mvnsite
1m 15s

the patch passed

+1 :green_heart:
javadoc
0m 46s

the patch passed with JDK Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04

+1 :green_heart:
javadoc
1m 19s

the patch passed with JDK Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10

+1 :green_heart:
spotbugs
3m 10s

the patch passed

+1 :green_heart:
shadedclient
21m  6s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

+1 :green_heart:
unit
227m 10s

hadoop-hdfs in the patch passed.

+1 :green_heart:
asflicense
0m 45s

The patch does not generate ASF License warnings.

322m 34s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3455/2/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/3455

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell

uname
Linux cee30baef7a2 4.15.0-156-generic #163-Ubuntu SMP Thu Aug 19 23:31:58 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / ae46a1a5c9afa6306f688f3dd8ac143278d5e0d5

Default Java
Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3455/2/testReport/

Max. process+thread count
2848 (vs. ulimit of 5500)

modules
C: hadoop-hdfs-project/hadoop-hdfs U: hadoop-hdfs-project/hadoop-hdfs

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3455/2/console

versions
git=2.25.1 maven=3.6.3 spotbugs=4.2.2

Powered by
Apache Yetus 0.14.0-SNAPSHOT https://yetus.apache.org

This message was automatically generated.


Passing and failing tests for #624

@ericgreenmix I'm looking at this right now.

Ah, okay, I get it. I think the way I'd want to approach this is to make the daemon on a rebuild to first look for the latest sequence id, then keep the fetching going until then.  It's getting defeated right now because the gap in sequence is bigger than the page, and the daemon is stopping when it hits an empty set.

Ah that makes sense. Sounds like a good solution to me

@ericgreenmix Okay, the next question is "how did you get the event store into this state in the first place?"

Eh, what exactly do you mean by that? We have tons of events going into our event store. When an event is stored and then over 100 different event types get stored, then that same event type gets stored again, that is when we start seeing the issue.

@jeremydmiller looking at the Fetcher code, it looks like the issue might be the fetchNextPage.  We don't actually have gaps in our event store (there are events for every sequence number), but the query
select seq_id from {_selector.Events.DatabaseSchemaName}.mt_events where seq_id > :last and seq_id <= :limit and age(transaction_timestamp(), {_selector.Events.DatabaseSchemaName}.mt_events.timestamp) >= :buffer order by seq_id;
{_selector.ToSelectClause(null)} where seq_id > :last and seq_id <= :limit and type = ANY(:types) and age(transaction_timestamp(), {_selector.Events.DatabaseSchemaName}.mt_events.timestamp) >= :buffer order by seq_id;

seems to be filtering on type and grabbing the seq_id window at the same time.  So if there aren't any events of the particular type in that window, you'll get 0 results.  This I think is what is causing the issue to manifest itself in our system.

@jarroda Yeah, I bumped into that w/ Eric's test. I'll add another test that runs continuously with big gaps somehow. This is turning into a much bigger issue than I was anticipating.


use user ids for everything instead of user emails

Summary
Test Plan

Please consider the impact of your changes on the other developers.

#29  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @Silver-Sword and the rest of your teammates on  Graphite


[AMBARI-23868] Log Search web alert gets 401 http status code if KNOX SSO is enabled.

What changes were proposed in this pull request?
Root is protected when KNOX SSO is enabled, therefore the basic web alert.
So i can use logsearch address for http and https property instead of the port. (i can use <IP_ADDRESS> address as that is replaced with the hostname by web_alert.py)
I did not include any upgrade changes, the issue is not that critical and its easy to fix (as logsearch is tech preview there wont be that use case anyway)
How was this patch tested?
manually. if logsearch was stopped, it showed connection was refused during access the /api/v1/info api
please review @zeroflag @g-boros @swagle @kasakrisz

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/2318/
Test PASSed.


RELEASE: v.2.1.0-alpha.1

Coverage remained the same at 100.0% when pulling c2d9590a5ede46202ff0ffb816ca681690b78463 on dev into cf51ffd1a93363a84903ae35692a704df2991a3f on next.


relative path for starting node

Currently the node should start with an absolute path. We need to accept relative path like: ~/gallactic/chains/chain1

I hope the command works for relative path as well


COH-23644?

I am working in a project where we due to limitations of using JDK 8 and other components used are "stuck" with Coherence CE 14 and is having a challenge where evicting entries in the local cache (part of a near cache) takes a LONG time.
I realize this is not a supported release so we are looking into solving the problem ourselves but have a specific question that could help us in this...
We are profiling and seeing that the line that seem to take a lot of time is " iterEvict.remove()" in OldCache and is marked with an issue "COH-23644" that we assume introduced the line.
Not sure if this is a reference to an issue in Coherence CE or an internal Oracle issue to the enterprise edition but either way we have not been able to find any info about it and are very curious to what the issue was and why the line was introduced...
Naively when looking at the code it seems to remove keys from an ArrayList of elements to be evicted as it is iterated - it is not clear to us WHY there is a need to do this and in general removing elements from ArrayLists is expensive (at least in the past required copying the array) so we are suspecting this could be at least one reason we see slow eviction (our caches are big and so could the list of keys to evict be)....
Our first work-around that we are testing now is to increase the low-units to be about 99% of the high-units to ensure the size of the array list will be small (then the ~O(n ** 2) effect of the repeated array.remove operation is minimized) - lets see if it works...
We are also considering, as a further work around, try using Caffeine (2.X version ) as a local cache with this Coherence CE release (using the old lower level mechanism that has been available to provide your owen cache implementation) but not sure how well it would work - are experiences of this would be welcome!
I have noticed that the above mentioned line is present also in the latest Coherence CE release so if this is the cause of the problem we are seeing it may be present also with the latest...

Can report that the work around to increase the low-units to be very close to high-units (this way ensuring that a small number of entries are evicted each time a "prune" is performed seem to have improved the situation but now we instead have very frequent prunes which is not ideal either...
Woulds appreciate info about IF that line that remove the entries from the list as they are processed is really needed (I would naively assume the whole prune operation is executed inside a "synchronized" block preventing any other operations on the local cache and in that case I do not immediately see the need to remove the elements instead of just letting them be and afterwards empty the whole array at once if it is retained ror just discard it if it is temporary) and if so if you have some other suggestions of improvements to the code we can add in a "patch" for our own use... maybe use a double linked list instead of an array list that have a O(1) complexity for deleting an entry?!

Hi @javafanboy
So far as I can tell, this is a memory use improvement to address instances of OOM that we saw. It's just to release references early, and I believe should be safe for you to just undo that line. Said references will be released eventually.
If memory usage is not a concern for you, I would say just go ahead and "patch".
We'll investigate this on our side. Thanks for finding it.

I have never tried building Coherence before and when following the instructions it eventually reported build complete but nothing shows up under the /dist directory?! Any ideas what is up?
I have for instance tried "mvn -am -pl coherence clean install -DskipTests -Dtde.compile.not.required" that reports the build as successful but leaves dist empty (only original readme file is there)?
A quick search for "coherence*.jar" results in:
/prj/coherence-core-components/target/coherence-core-components-14.1.1-0-13-SNAPSHOT-sources.jar
./prj/coherence-core-components/target/coherence-core-components-14.1.1-0-13-SNAPSHOT.jar
./prj/coherence-core/target/coherence-core-14.1.1-0-13-SNAPSHOT.jar
./prj/coherence-core/target/coherence-core-14.1.1-0-13-SNAPSHOT-tests.jar
./prj/coherence-core/target/coherence-core-14.1.1-0-13-SNAPSHOT-sources.jar
./prj/coherence-discovery/target/coherence-discovery-14.1.1-0-13-SNAPSHOT.jar
./prj/coherence-docker/target/coherence-docker-14.1.1-0-13-SNAPSHOT.jar
./prj/coherence-http-netty/target/coherence-http-netty-14.1.1-0-13-SNAPSHOT.jar
./prj/coherence-javadoc/target/coherence-javadoc-14.1.1-0-13-SNAPSHOT.jar
./prj/coherence-jcache/target/coherence-jcache-14.1.1-0-13-SNAPSHOT.jar
./prj/coherence-loadbalancer/target/coherence-loadbalancer-14.1.1-0-13-SNAPSHOT.jar
./prj/coherence-login/target/coherence-login-14.1.1-0-13-SNAPSHOT.jar
./prj/coherence-management/target/coherence-management-14.1.1-0-13-SNAPSHOT.jar
./prj/coherence-metrics/target/coherence-metrics-14.1.1-0-13-SNAPSHOT.jar
./prj/coherence-mock/target/coherence-mock-14.1.1-0-13-SNAPSHOT.jar
./prj/coherence-rest/target/coherence-rest-14.1.1-0-13-SNAPSHOT-tests.jar
./prj/coherence-rest/target/coherence-rest-14.1.1-0-13-SNAPSHOT.jar
./prj/coherence-testing-support/target/coherence-testing-support-14.1.1-0-13-SNAPSHOT.jar
./prj/coherence/target/coherence-14.1.1-0-13-SNAPSHOT.jar
./prj/test/performance/framework/target/lib/14.1.1-0/coherence.jar
./prj/test/performance/framework/target/coherence-performance-framework-14.1.1-0-13-SNAPSHOT.jar
./prj/test/performance/psr/target/lib/14.1.1-0/coherence.jar
./prj/test/performance/psr/target/coherence-performance-psr-14.1.1-0-13-SNAPSHOT.jar
./prj/test/performance/target/lib/14.1.1-0/coherence.jar
./tde/core-net/3.0/target/coherence.jar
./tde/core/1.3/ext/coherence-core.jar
./tde/core/1.3/ext/coherence-discovery.jar
./tools/tde/lib/coherence.jar
Also in the v14.1.1 I downloaded there is a readme.txt (in addition to the README.md) that seem to contain some old / misleading instructions (Oracle internal for the Enterprise edition?) that had me confused at first...

When you build like that (mvn ... install), the jar ends up in the maven repository, usually in ~/.m2/repository/com/oracle/...
Otherwise the main jar can also be found in ./prj/coherence/target/coherence-14.1.1-0-13-SNAPSHOT.jar it's the same file. Useful for debugging or prototyping purposes.
I wouldn't worry about making any mistakes running these config files, Coherence doesn't depend on any "hidden" magic ala Windows registry :)
I'll look into those, at any rate.

Thanks for the info and happy to hear there are no similarities between
Coherence and with Windows :-)
On Fri, Feb 24, 2023 at 4:49 PM Maurice Gamanho @.***>
wrote:

When you build like that (mvn ... install), the jar ends up in the maven
repository, usually in ~/.m2/repository/com/oracle/...
Otherwise the main jar can also be found in
./prj/coherence/target/coherence-14.1.1-0-13-SNAPSHOT.jar it's the same
file. Useful for debugging or prototyping purposes.
I wouldn't worry about making any mistakes running these config files,
Coherence doesn't depend on any "hidden" magic ala Windows registry :)
I'll look into those, at any rate.
—
Reply to this email directly, view it on GitHub
https://github.com/oracle/coherence/issues/95#issuecomment-1443877731,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AADXQFZGADBL7766F74AKQTWZDJ7DANCNFSM6AAAAAAVFJRRQ4
.
You are receiving this because you were mentioned.Message ID:
@.***>

Off the top of my head, I wonder if the DuratiionTest.java failure is a region thing. I notice in your error message it says, expected:<1[.]50ms> but was:<1[,]50ms>, i.e the test expects a decimal point (as this is hard coded in the test code), but your region settings use a comma in place of a decimal point (like we do here in Turkey, although my laptop uses English for it region settings). We run all our CI builds on a number of different OS'es including versions of Windows and they all pass but they are all on US based Jenkins slaves. I'm sure we do not run builds with non-English regions, maybe we should try one and see what happens.

The "patch" of removing the line worked - we no longer see minute long delays in eviction where one core run  at 100%. No negative effects observed in our testing.


Controller.get_all_nodes_conn_status fails when zero nodes are connected.

Problem area
elif (resp.status_code == 200):
            p1 = 'nodes'
            p2 = 'node'
            if(p1 in resp.content and p2 in resp.content):
                status.set_status(STATUS.OK)
                itemlist = json.loads(resp.content).get(p1).get(p2)
                for item in itemlist:

As far as I can tell resp.content is just the body of the html response (a string) which means any response that contains "nodes" will also contain "node" ("node" is a substring of "nodes"); This makes the second half of the if statement on line 396 meaningless. The function json.loads(resp.content).get(p1).get(p2) then silently fails and returns None when the key "node" is absent (zero nodes are connected). The iteration over itemlist on line 399 then fails.
Traceback (most recent call last):
  File "poll.py", line 37, in <module>
    res = ctrl.get_all_nodes_conn_status()
  File "/usr/local/lib/python2.7/dist-packages/pybvc-1.1.1-py2.7.egg/pybvc/controller/controller.py", line 399, in get_all_nodes_conn_status
    for item in itemlist:
TypeError: 'NoneType' object is not iterable

Agreed. Will fix it. Thanks for the pointing on it.

Fixed.


schema diff returns schema error

After this issue: https://github.com/TBD54566975/ftl/issues/2095
calling schema diff <endpoint> resulted in this error
ftl: error: failed to get other schema: url https://tbd-ftl.tbd.sqprod.co/: failed to parse schema: /home/runner/_work/pfi/pfi/backend/modules/tbdex/usd_btc_exchanges.go:56:40-40: unknown reference "customers.getCustomerId", is the type annotated and exported?

First thought is that this is expected because the cluster's schema was in a half-way state. But is that fair?

Seems to be fixed by https://github.com/TBD54566975/ftl/pull/2110


removed unused bioimageio-pytorch submodule

as we're not using it anymore, it shouldn't clutter tiktorch

done in #186


How can I use htdemucs.yaml (Demucs V. 4) with MDX-Net (error message)?

I want to use the MDX-Net models with the newest Demucs model (version 4, htdemucs). I added htdemucs.yaml it to the Demucs models folder and I receive this error when I want to use this model with MDX-Net:

Last Error Received:
Error Received while processing "hello demo test.flac":
Process Method: MDX-Net
If this error persists, please contact the developers with the error details.
Traceback Error: "  File "inference_MDX.py", line 1692, in main
File "inference_MDX.py", line 321, in prediction_setup
File "demucs\pretrained.py", line 68, in get_model
File "demucs\repo.py", line 154, in get_model
File "demucs\repo.py", line 136, in get_model
File "demucs\repo.py", line 136, in
File "demucs\repo.py", line 104, in get_model
"
ModelLoadingError: "Could not find pre-trained model with signature 955717e8."
Error Time Stamp [2022-12-09 15:07:19]

What can I do to solve this?

I searched for 955717e8.th but the search result is a txt-file where 955717e8-8726e21a.th is mentioned.
How can I get 955717e8.th?

Short Answer: Wait until version 5.5 comes out. It should be out before the end of the year.
Long Answer: Get whichever audio you're trying to separate and run it through your preferred MDX models on UVR. Take the same audio and run it through the new demucs 4 model in your terminal. Check this for instructions on how to get that set up. Also, I'd recommend using the fine-tuned model by putting "n htdemucs_ft" into your command.
Then, take both of these separations and go back to UVR. Use Manual Ensemble mode and select both separations (MDX separation and the Demucs 4 separation). This is a workaround way to start using htdemucs with MDX-Net. But, this feature will be added in a couple of weeks maximum.
Feel free to reply if you have any questions!

Short Answer: Wait until version 5.5 comes out. It should be out before the end of the year.
Long Answer: Get whichever audio you're trying to separate and run it through your preferred MDX models on UVR. Take the same audio and run it through the new demucs 4 model in your terminal. Check this for instructions on how to get that set up. Also, I'd recommend using the fine-tuned model by putting "n htdemucs_ft" into your command.
Then, take both of these separations and go back to UVR. Use Manual Ensemble mode and select both separations (MDX separation and the Demucs 4 separation). This is a workaround way to start using htdemucs with MDX-Net. But, this feature will be added in a couple of weeks maximum.
Feel free to reply if you have any questions!

I followed your long answer and the results sound better than using any of the already existing demucs models.
Thank you for helping me. I will do it this way until the update is available.


Threads can obtain priveleged mode

Threads within a process can obtain priveleged mode by modifying the
return stack of another thread. Effectivly changing the CPSR and gaining provileged level after the PL1 context swtich.

Just do a logical orr with a privileged indicator in the thread control block, not using the stack value for other things the the flags

Added thread->privilede variable in the thread contorl block. The CPSR is manually modified during context swtich and during sched start


UI 769 one click deploy to digital ocean

Description

Add script to take the docker image and deploy to a Digital Ocean Droplet with it.
Update README to include deploy button.

This currently works when deploying via Digital Oceans doctl cli tool, but doesn't work with their "Deploy with DO" button because it appears to be broken currently (I tried a few other repositories and all are broken ATM). I've lodged a support ticket to figure out what's going on.
Type of change

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

How should this be tested?

Switch to this branch on the homepage of this repository
Click the Deploy to Digital Ocean button
Sign into your Digital ocean account and deploy
After deploy is complete, visit the URL Digital Ocean gives and check deploy is working

Checklist:

[x] I have performed a self-review of my own code
[x] I have commented my code where relevant, particularly in hard-to-understand areas
[x] My changes generate no new console warnings
[x] The base of this PR is master if hotfix, develop if not

Digital ocean got back to me and confirmed this is a bug and they're working on a fix.


How to add context menu to DGL plugin UI?

Hi FalkTX!
I want to add context menu to my plugin UI, so that I can provide some advanced features (e.g. view presets, plugin configurations, etc.). But it seems to be a hard job.
Maybe these ideas worth a try:

Use native menu: This should be the best solution. It can be achieved via Win32 API on Windows, and GTK on Linux. But seems to be hard for non-GTK applications to use GTK's context menu.
Integrate Dear ImGui: Is it possible to use Dear ImGui together with picture-based DGL UI?
Create our own context menu like JUCE: Seems that JUCE has their own context menu implementation.
Use NanoVG: Wolf Shaper uses NanoVG to implement its own context menu.

Do you have any ideas? Seems that I have to do it from scratch.

That is something to be done on the developer side, DPF only provides a simple window + view.
So rolling your own custom code that mimics the style of a context menu is typically the best choice.
Using gtk or win32 is not a good idea as they can block the main loop and prevent other things from running, resulting in deadlocks.

an alternative that never materialized so far is borderless windows from dpf side, but that seems like abusing things for presenting a non-native menu.
really best to rendering a menu as part of the view, making it so that it also nicely closes together with the view without leaving some weird top-level dangling windows around (a problem that can happen with juce if one is not careful)

integrating imgui on existing dpf UIs is quite straightforward, assuming they use opengl already.
2 examples of this: https://github.com/DISTRHO/DPF-Widgets/blob/main/tests/opengl.cpp and https://github.com/trummerschlunk/master_me/blob/master/plugin/widgets/DoubleClickHelper.hpp

Using gtk or win32 is not a good idea as they can block the main loop and prevent other things from running, resulting in deadlocks.

Is it possible to use a second thread? Sure that FLTK-based or NTK-based UIs has this issue: when menu is active, even the host is blocked.

integrating imgui on existing dpf UIs is quite straightforward, assuming they use opengl already.
2 examples of this: https://github.com/DISTRHO/DPF-Widgets/blob/main/tests/opengl.cpp and https://github.com/trummerschlunk/master_me/blob/master/plugin/widgets/DoubleClickHelper.hpp

They are based on pure ImGui.
What I'm going to do is to show ImGui window on DGL UI (picture-based). So I can reuse windows and widgets from Dear ImGui (especially sub-windows). Just like this ideal image (simply make it with GIMP):

Solved! A good solution is to apply Wolf Shaper's menu widget to my project, just like this:

Source:

https://github.com/AnClark/Minaton-XT/tree/develop/ext/WolfWidgets
https://github.com/AnClark/Minaton-XT/blob/develop/plugin/MinatonUIMenu.cpp


Request: Per Project style.css/header.js/footer.js

I would like to use different styles, headers and footers for every project. My suggestion is, to look for these files in the project root, if they exist - use these - otherwise use global defaults.
Thank you by the way for this awesome project. Its easy to use and very fast.

Hmm, good Idea... I will think about it :)

Can you describe to me briefly your advantage?
What's better to create in each project the footer, header and style, than changing in the plugin settings the path to this files? The latter is already possible. :) Generally I find the idea interesting.

Okay, here are my 2 cents. Firstly I think having to open the settings page searching for the module and write out the path to the file is a bit tedious (since there is no file picker at the moment). Secondly the way if you switch branch or project the alternate headings are already in place, that speeds up the workflow. For the third point: I use atom-sync-settings and different OS's. The path specified for the one OS wouldn't work with the other.
And by the way - couldn't you put header and footer into one file? You could save one file-read like this.
module.exports = function () {
    return {
          header {
            height: '2cm',
            contents: 'This is the header'
          }
          footer {
            height: '1cm',
            contents: 'This is the footer'
          }
    };

To improve the workflow, I refer to your idea. Query whether css, header and footer in the current project root are found. For simplicity I also use a directory ./markdown-themeable-pdf/* to contain the custom pdf settings.
For my purposes, it is actually easier if headers and footers are in separate files. That should make no problems. :)

Thank you. I think using the same seperate folder as for global config is a good idea. :-)

@tairun Can you try that once?
Must also rewrite the documentation ...

@cakebake Do you want me to pull 0b8f28c and testdrive?
(Sidenote, I don't know how to install packages outside of atom)

Okay nevermind, the update just showed in atom.

Changes in the header and footer file don't seem to carry over to the pdf (global and local files). Need to do more testing on this one. But I got an error while updating (or reinstalling the package). There seems to be a missing backslash in joined paths: See picture below (Bremse is my username)

I will test it again with a clean installation... Thank´s for fast response

Oh, I think you must only restart atom after updating markdown-themeable-pdf...
The wrong path is only a display issue by atom notifications...

Sorry that embarassing. Did not think of that. :+1: Now its working like a charm!

Great, as it should be :)


MWPW-138024: Smoke test for CaaS block

Resolves: MWPW-138024

Codecov Report

Merging #441 (009726a) into stage (ef93071) will increase coverage by 0.03%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##            stage     #441      +/-   ##
==========================================
+ Coverage   96.58%   96.61%   +0.03%
==========================================
  Files          26       26
  Lines        2867     2867
==========================================
+ Hits         2769     2770       +1
+ Misses         98       97       -1

see 1 file with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Add Element#contains() to README

nvm, realized this is not DOM level 4...

You are right, it should have been mentioned.
Now it is, under Other fixes/standardized behaviors include list
Thanks


add an option to not console.warn/console.error things

when using gl-shader just to check if a GLSL is valid, I expect the library not to side-effect any logs.

also I would like to access the original error message, not the one wrapped in formatCompilerError.
Maybe we should have a custom Error class, that contains different fields (originalError,..)

also I would like to access the original error message, not the one wrapped in formatCompilerError.
Maybe we should have a custom Error class, that contains different fields (originalError,..)


Combined Log Format response times have wrong number of seconds/microseconds

Context
Requests are logged with an INFO level log message that extends the standard Combined Log Format message with the time it took for the handler to process the request, nominally in seconds and microseconds:
[1681511864174] [INFO] 2023-04-14T22:37:44.174Z 755cd95f-b898-4a99-a247-cb1e97e2777e <IP_ADDRESS> - - [14/Apr/2023:22:37:44 +0000] "GET /main/ready/ HTTP/1.1" 200 5104 "https://some.app.net/auth/" "Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/605.1.15 (KHTML, like Gecko) Version/16.4 Safari/605.1.15" 0/3.615

In this example the execution time is represented by the 0/3.615 at the very end of the line.
The code that implements the format expects to be passed a number of microseconds and this is divided by 1000000 to get the number of seconds. Unfortunately the code that calls this passes the response time in milliseconds, not microseconds. This leads to the number of seconds more or less always being 0.
Expected Behavior
In the last part of the format, either the last number is in milliseconds and the second-to-last number is the last number divided by 1000, or the last number is in microseconds and the seconds-to-last number is that last number divided by 1000000.
Actual Behavior
The last number is in milliseconds but the second-to-last number is that last number divided by 1000000, essentially making the second-to-last number be in a unit of thousands of seconds.
Possible Fix
Either change the code that calculates the time to compute the time in microseconds, or change the code that computes the number of seconds to divide the input by 1000.
It likely makes more sense to change the semantics of the logging function to expect milliseconds and change the code that computes the number of seconds, since that leaves the output more consistent with the values that are seen in existing logs. It is also generally more useful to have execution times in milliseconds since that it the smallest unit of billing in Lambda.
While this is being fixed it would also make sense to change the formatting string to limit the number of significant digits in the printout of the floating point time, since it for larger values it often shows the precision errors in the float representation.
Steps to Reproduce

Fire up anything in Zappa
Submit a request
Look at the log files.

Your Environment

Zappa version used: 0.56.1

merged and released in 0.58.0.
closing


Unexpected tokenwhen I try combinate load() funtion

I tryed combinate with load and I get a unexpected token erorr in console.
my code looks like this:
$('.more').infinitescroll({
binder: $('.more'),
$( "video-body" ).load( "next.php" );
});

Sorry to hear you're having trouble with Infinite Scroll.
Infinite Scroll v3 is out!. It's an all new re-write, so take a look at how to upgrade from v2.
I recommend upgrading to v3. Please open a new issue if you run into trouble with Infinite Scroll v3.


Add tag-filter-negate flag

Fixes #65
To keep tag myprecious and delete everything else do this:
gcr-cleaner-cli -grace 720h -repo gcr.io/foo/bar -tag-filter-any 'myprecious' -tag-filter-negate

I won't be signing the CLA. Feel free to close the PR

@porjo Can I submit this PR for you since I've previously signed the CLA?

@StevenACoffman yes that's fine


Compilation error with Swift 3.2 and Xcode 9

When trying to build the latest version of this library (0.3.7) as a CocoaPod, using Swift 3.2 in Xcode 9 beta 1, the following compilation errors occur:
.../CryptoKit/Sources/CryptoKit/Hash Algorithms/Implementations/SHA-2/SHA2Variant.swift:66:34: Binary operator '&+' cannot be applied to two 'Self.BaseUnit' operands
.../CryptoKit/Sources/CryptoKit/Hash Algorithms/Implementations/SHA-2/SHA2Variant.swift:75:31: Binary operator '&+' cannot be applied to two 'Self.BaseUnit' operands
.../CryptoKit/Sources/CryptoKit/Hash Algorithms/Implementations/SHA-2/SHA2Variant.swift:78:32: Binary operator '&+' cannot be applied to two 'Self.BaseUnit' operands
.../CryptoKit/Sources/CryptoKit/Hash Algorithms/Implementations/SHA-2/SHA2Variant.swift:95:29: Binary operator '&+' cannot be applied to two 'Self.BaseUnit' operands

Thanks for testing and reporting. Updated code and placed in swift-3.2 branch. Please verify if this works.


Automatically send steps from agents to predictive intellisense buffer after getting result

Summary of the new feature / enhancement
Instead of waiting for user to run /code post to get the responses in the predictive intelisense buffer we should send the code results to predictive intellisense by default.
Proposed technical implementation details (optional)
I ask a question to the side car, get code response back, I then move to my working terminal and start typing the code given to me, the code from the side car should be suggested by predictive intellisense. This helps users discover the integration piece without knowing how to use /code post

I don't think it's desired behavior to always send generated command to the shell. There is a good chance the generated command is not ready for execution or is not what the user actually want.
Also, it seems not typical for a user to switch the shell and typing the same command. We have key bindings for copying/posting the command.


GSD Request

--- GSD JSON ---
{
  "vendor_name": "Linux",
  "product_name": "Kernel",
  "product_version": "versions from v4.15 to before v5.15.13",
  "vulnerability_type": "unspecified",
  "affected_component": "unspecified",
  "attack_vector": "unspecified",
  "impact": "unspecified",
  "credit": "",
  "references": [
    "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=7d5cb45655f2e9e37ef75d18f50c0072ef14a38b",
    "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=f68599581067e8a5a8901ba9eb270b4519690e26"
  ],
  "extended_references": [
    {
      "type": "commit",
      "value": "7d5cb45655f2e9e37ef75d18f50c0072ef14a38b",
      "note": "introduced"
    },
    {
      "type": "commit",
      "value": "f68599581067e8a5a8901ba9eb270b4519690e26",
      "note": "fixed"
    }
  ],
  "reporter": "joshbressers",
  "reporter_id": 1692786,
  "notes": "",
  "description": "i2c: validate user data in compat ioctl\n\nThis is an automated ID intended to aid in discovery of potential security vulnerabilities. The actual impact and attack plausibility have not yet been proven.\nThis ID is fixed in Linux Kernel version v5.15.13 by commit f68599581067e8a5a8901ba9eb270b4519690e26, it was introduced in version v4.15 by commit 7d5cb45655f2e9e37ef75d18f50c0072ef14a38b. For more details please see the references link."
}
--- GSD JSON ---

This issue has been assigned GSD-2022-1000051


implement with_timeout()/with_deadline() method style call on Future

This allows writing
rx.read(&mut read_buf).with_timeout(Duration::from_millis(200)).await

rather than
with_timeout(Duration::from_millis(200), rx.read(&mut read_buf)).await

This looks nicer to me.

I agree that it looks nicer, I was left wondering why the rest of the rust async ecosystem like tokio did not go for such an approach? Is there some disadvantage I am not seeing. Then I found out tokio does support such an api: https://docs.rs/tokio-util/latest/tokio_util/time/trait.FutureExt.html#method.timeout its just in tokio-util (still part of the tokio project) not tokio itself.


git: Add option for no_config in git shell

This causes git commands to run without considering any
system or user configuration. In some cases it's easier to
ban all overrides rather than have to override things on
a flag by flag basis.
Topic: gitconfig

Reviews in this chain:
└https://github.com/Skydio/revup/pull/188 git: Add option for no_config in git shell

#
head
base
diff
date
summary

0
1d315c52
740c5cdb
diff
Jul 3 13:56 PM
1 file changed, 16 insertions(+), 2 deletions(-)


Version 0.3.8 doesn't dispatchEvent 'ready'

Hi @kreopt @victorgrenke
We use your latest 0.3.8 where we don't see any VIDEO played in  tag.
It is difficult to figure out the problems, because we are not an expert on video and on your lib.
But we figured out Remuxer is never init because in flush() method there is never fired the dispatchEvent('ready').
On you last version we used which should be 0.3.5 anything is fine. So our question is what did you changed or what is causing this problem?
please help
thx

in flush method there is always this true
if (!this.tracks[track_type].readyToDecode || !this.tracks[track_type].samples.length) return;
and so dispatchEvent('ready') is never fired

Ok I figured our the problem!!!
We have the problem in the H264Remuxer with the new remux() method
If we uncomment the nalu.toString() log we got this output
NDR: NRI: 1, PTS: -1506046017672, DTS: -1506046017672
NDR: NRI: 1, PTS: -1506046014072, DTS: -1506046014072
NDR: NRI: 1, PTS: -1506046010472, DTS: -1506046010472
NDR: NRI: 1, PTS: -1506046003272, DTS: -1506046003272
NDR: NRI: 1, PTS: -1506045999672, DTS: -1506045999672
NDR: NRI: 1, PTS: -1506045996072, DTS: -1506045996072
NDR: NRI: 1, PTS: -1506045992472, DTS: -1506045992472
NDR: NRI: 1, PTS: -1506045985272, DTS: -1506045985272
so as you can see DTS is less then 0, but the default lastGopDTS is set to 0 so this
if (this.lastGopDTS < nalu.dts) {
is never true.
if we set lastGopDTS to -99999999999999 anything is fine.
So honestly we have no idea what is NALU and what is DTS, but this workaround does work for us.
If you like I can do a PR?

Hi! All pull requests and bug reports are welcome! Thanks for investigation
the root of this problem. We'll look deeply and fix things as soon as it
will be possible.
22 сент. 2017 г. 21:22 пользователь "mburger81"<EMAIL_ADDRESS>написал:

Ok I figured our the problem!!!
We have the problem in the H264Remuxer with the new remux() method
If we uncomment the nalu.toString() log we got this output
NDR: NRI: 1, PTS: -1506046017672, DTS: -1506046017672
NDR: NRI: 1, PTS: -1506046014072, DTS: -1506046014072
NDR: NRI: 1, PTS: -1506046010472, DTS: -1506046010472
NDR: NRI: 1, PTS: -1506046003272, DTS: -1506046003272
NDR: NRI: 1, PTS: -1506045999672, DTS: -1506045999672
NDR: NRI: 1, PTS: -1506045996072, DTS: -1506045996072
NDR: NRI: 1, PTS: -1506045992472, DTS: -1506045992472
NDR: NRI: 1, PTS: -1506045985272, DTS: -1506045985272
so as you can see DTS is less then 0, but the default lastGopDTS is set
to 0 so this
if (this.lastGopDTS < nalu.dts) {
is never true.
if we set lastGopDTS to -99999999999999 anything is fine.
So honestly we have no idea what is NALU and what is DTS, but this
workaround does work for us.
If you like I can do a PR?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Streamedian/html5_rtsp_player/issues/55#issuecomment-331461282,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADb0QBF8bSoIRaAc8Z6Q5ApABHODv5jhks5sk8K1gaJpZM4Pgto4
.

we created a PR for this issue #56


[ci] Add checkout step to verify-fpga job

Without this, Azure automatically checks out the repo at the time this job runs, not necessarily at the same commit as the previous FPGA jobs.
This can lead to a test not existing when the FPGA tests ran, but now existing when this verify job runs. The verify job then complains that you didn't run that test.

FPGA failure (//sw/device/tests:i2c_target_test_fpga_cw310_sival) looks unrelated, merging


MODE-2674. Modeshape 5 open database connections with HikariCP.

Configure the minimum idle connection property of the HikariDataSource.
https://issues.jboss.org/browse/MODE-2674

@mohideen it's good enhancement, thanks for submitting the patch
I will do a bit of investigation to see how easy/hard it would be to actually support all the HikariConfig params transparently from ModeShape.  I think that would provide the greatest degree of flexibility.

Closing this, as https://github.com/ModeShape/modeshape/pull/1641 adds generic support for custom Hikari configuration options.
The default minimumIdle will be set to 1.


Lost the theme format for my webpage

I am facing difficulties as I seem to have lost the theme format for my webpage https://sagihaider.com/
It was loading perfectly two months ago, but after adding publications, it is not displaying properly on a desktop/PC. Interestingly, the webpage works perfectly on mobile devices.
Could you please provide a solution or assistance?
Thank you!

I am attaching a screenshot for the reference.

I meet the same problem...

@sagihaider @penguinaugustus, you should try to uncheck "Enforce HTTPS", it worked for me.
Another option is replacing all the http occurrences by https ones.

Just tried enforcing HTTPS and no effect.

Just tried enforcing HTTPS and no effect.

Well I did the opposite and it worked. But I think your problem is about SSL : if I consult your website using http, everything works well.

Are you using desktop/pc? Because on mobile browser it is working well. However, when loading from PC issue remains the same.

I thinks the problem is even i uncheck enforcing HTTPS, sometimes my browser would automatically add https. I can fix it by changing the prefix to http but is there anyway to automatically enforce HTTP?

Are you using desktop/pc? Because on mobile browser it is working well. However, when loading from PC issue remains the same.

@sagihaider I wonder if you can share the method you used to solve this issue? Thanks.

Hi, I am using MAC so I did a hard refresh and it worked (⌘ Cmd and ⇧ Shift key and then press R).

It sounds like this issue is resolved, correct?

Yes, I think so. We can close the issue.
Thank you!

I haven't been able to resolve the issue on my end. It's interesting to know that a hard refresh solves it, given that the problem stems from the browser automatically adding https. I would greatly appreciate any additional suggestions. Thank you.

@jwangjie When I've been doing development I need to do a hard refresh as well, plus sometimes restarting Jekyll depending upon what I'm working on. However, you may have some libraries out of date so you may want to try running bundle update and see if that helps. You may also want to open up a new topic in the discussion since closed issues don't get much attention.


[Bug] Codegen fails for SyncReturn<Result<T, E>>

Describe the bug
Codegen fails for this function:
pub fn bug() -> SyncReturn<Result<i32>> {
	SyncReturn(Ok(0))
}

This is quite suprising because this works fine:
pub fn bug() -> Result<i32> {
	Ok(0)
}

and it looks like that the codegen should be handled by the inner type:
https://github.com/fzyzcjy/flutter_rust_bridge/blob/7dea25d68cc0433f653cc673da4440759b94e09e/frb_codegen/src/ir/ty_sync_return.rs#L19-L30
Codegen logs with RUST_LOG=debug environment variable
RUST_LOG=debug just gen
flutter pub get
Resolving dependencies...
  collection 1.17.1 (1.17.2 available)
  ffigen 7.2.11 (8.0.2 available)
  file 6.1.4 (7.0.0 available)
  http 0.13.6 (1.0.0 available)
  matcher 0.12.15 (0.12.16 available)
  material_color_utilities 0.2.0 (0.5.0 available)
  puppeteer 2.24.0 (3.0.0 available)
  source_span 1.9.1 (1.10.0 available)
  test_api 0.5.1 (0.6.0 available)
Got dependencies!
flutter_rust_bridge_codegen --rust-input native/src/api.rs --dart-output lib/bridge_generated.dart --c-output ios/Runner/bridge_generated.h --extra-c-output-path macos/Runner/ --dart-decl-output lib/bridge_definitions.dart --wasm
2023/06/06 11:39:18 [DEBUG] configs=[Opts { rust_input_path: "/home/alexandre/Downloads/flutter_rust_bridge_template/native/src/api.rs", dart_output_path: "/home/alexandre/Downloads/flutter_rust_bridge_template/lib/bridge_generated.dart", dart_decl_output_path: Some("/home/alexandre/Downloads/flutter_rust_bridge_template/lib/bridge_definitions.dart"), c_output_path: ["/home/alexandre/Downloads/flutter_rust_bridge_template/ios/Runner/bridge_generated.h", "macos/Runner/bridge_generated.h"], rust_crate_dir: "/home/alexandre/Downloads/flutter_rust_bridge_template/native", rust_output_path: "/home/alexandre/Downloads/flutter_rust_bridge_template/native/src/bridge_generated.rs", class_name: "Native", dart_format_line_length: 80, dart_enums_style: false, skip_add_mod_to_lib: false, llvm_path: ["/opt/homebrew/opt/llvm", "/usr/local/opt/llvm", "/usr/lib/llvm-9", "/usr/lib/llvm-10", "/usr/lib/llvm-11", "/usr/lib/llvm-12", "/usr/lib/llvm-13", "/usr/lib/llvm-14", "/usr/lib/", "/usr/lib64/", "C:/Program Files/llvm", "C:/msys64/mingw64"], llvm_compiler_opts: "", manifest_path: "/home/alexandre/Downloads/flutter_rust_bridge_template/native/Cargo.toml", dart_root: Some("/home/alexandre/Downloads/flutter_rust_bridge_template"), build_runner: true, block_index: BlockIndex(0), skip_deps_check: false, wasm_enabled: true, inline_rust: false, bridge_in_method: true, extra_headers: "" }]
2023/06/06 11:39:18 [DEBUG] Trying to parse "/home/alexandre/Downloads/flutter_rust_bridge_template/native/src/api.rs"
2023/06/06 11:39:18 [DEBUG] Trying to parse "/home/alexandre/Downloads/flutter_rust_bridge_template/native/src/bridge_generated.rs"
2023/06/06 11:39:18 [WARN] Skipping unresolvable module web (tried /home/alexandre/Downloads/flutter_rust_bridge_template/native/src/web.rs, /home/alexandre/Downloads/flutter_rust_bridge_template/native/src/web/mod.rs, /home/alexandre/Downloads/flutter_rust_bridge_template/native/src/bridge_generated/web.rs, /home/alexandre/Downloads/flutter_rust_bridge_template/native/src/bridge_generated/web/mod.rs, )
2023/06/06 11:39:18 [WARN] Skipping unresolvable module io (tried /home/alexandre/Downloads/flutter_rust_bridge_template/native/src/io.rs, /home/alexandre/Downloads/flutter_rust_bridge_template/native/src/io/mod.rs, /home/alexandre/Downloads/flutter_rust_bridge_template/native/src/bridge_generated/io.rs, /home/alexandre/Downloads/flutter_rust_bridge_template/native/src/bridge_generated/io/mod.rs, )
2023/06/06 11:39:18 [DEBUG] parse_function function name: Ident { sym: platform, span: bytes(679..687) }
2023/06/06 11:39:18 [DEBUG] parse_function function name: Ident { sym: rust_release_mode, span: bytes(2411..2428) }
2023/06/06 11:39:18 [DEBUG] parse_function function name: Ident { sym: bug, span: bytes(2483..2486) }
2023/06/06 11:39:18 [ERROR] panicked at 'not yet implemented: generate code for type "Result < i32 , Error >"', frb_codegen/src/ir/ty_unencodable.rs:59:9
thread 'main' panicked at 'not yet implemented: generate code for type "Result < i32 , Error >"', frb_codegen/src/ir/ty_unencodable.rs:59:9
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
error: Recipe `gen` failed on line 11 with exit code 101

To Reproduce
Clone this repo and run the codegen: https://github.com/DontBreakAlex/flutter_rust_bridge_template
Expected behavior
The codegen should work
Generated binding code
No response
OS
Linux
Version of flutter_rust_bridge_codegen
1.77.1
Flutter info
No response
Version of clang++
No response
Version of ffigen
No response
Additional context
No response

Looks like a bug. Feel free to PR!

Do you have any idea of why this does not work ? Where should I start looking ?

It says "not implemented". Thus, I suggest using the stack trace (RUST_BACKTRACE=1) to see which line is not implemented and work on it.

But is it really "not implemented"? I don't think so, the docs say this should work, plus SyncReturn is supposed to delegate to the underlying type, which is Result, which is definitely implemented. Seeing that the panic is in frb_codegen/src/ir/ty_unencodable.rs:59:9, I would guess that the issue is more likely to be an issue in dispatching somewhere...

Looks reasonable. Then maybe firstly have a look at https://cjycode.com/flutter_rust_bridge/contributing/design.html. Then print/debug on each phase. For example, is the parser correct? Is the code generator correct? etc


Bugs if grid size differ, ex. if image is not square

This code fails if you try setting grid_w and grid_h with different values, for example, you decided to use Yolo for images which are not square:
cell_x = tf.to_float(tf.reshape(tf.tile(tf.range(self.grid_w), [self.grid_h]), (1, self.grid_h, self.grid_w, 1, 1)))
cell_y = tf.transpose(cell_x, (0,2,1,3,4))
cell_grid = tf.tile(tf.concat([cell_x,cell_y], -1), [self.batch_size, 1, 1, self.nb_box, 1])

https://github.com/experiencor/keras-yolo2/blob/master/frontend.py#L92
ValueError: Dimension 1 in both shapes must be equal, but are 13 and 23 for 'loss/reshape_1_loss/concat' (op: 'ConcatV2') with input shapes: [1,13,23,1,1], [1,23,13,1,1], [] and with computed input tensors: input[2] = <-1>.

You should change that part of code in this way:
cell_x = tf.to_float(tf.reshape(tf.tile(tf.range(GRID_W), [GRID_H]), (1, GRID_H, GRID_W, 1, 1)))
cell_y = tf.to_float(tf.reshape(tf.tile(tf.range(GRID_H), [GRID_W]), (1, GRID_W, GRID_H, 1, 1)))
cell_y = tf.transpose(cell_y, (0,2,1,3,4))

cell_grid = tf.tile(tf.concat([cell_x,cell_y], -1), [BATCH_SIZE, 1, 1, BOX, 1])

So we get a grid where every cell keeps cell's coordinates. It will work with squaresd grid too

it is working in my repo, you can check it here. I did something very similiar from what @evgevd said

Nope @rodrigo2019 still the same error with your fork


[tts] tts does not work when app is in the background on ios

See https://github.com/dlutton/flutter_tts/issues/262

Hmm strange, we already have that though: https://github.com/muxable/rtchat/blob/main/ios/Runner/Info.plist#L47-L50

Also, the messages get added to the tts queue, and they will all be played once you get back to the app.


Marker Cluster Plugin

This PR ports 2 open-source clustering algorithms from googlemaps/android-maps-utils.

Non-hierarchical distance based algorithm
Grid based algorithm

cc @mapbox/android

@zugaldia I was able to make the code work with a configurable flag for MarkerView though the performance wasn't great. I'm thinking on punting on Markers as we aim to deprecate MarkerView in upcoming releases in https://github.com/mapbox/mapbox-gl-native/issues/9365.

Small issue with this plugin is there is no option to disable clustering at certain zoom level, say we need to plot 5 markers in 5 mtrs radius even at last zoom level we are not able to see all markers.
Is there any workaround for this?

I'm thinking on punting on Markers as we aim to deprecate MarkerView in upcoming releases in mapbox/mapbox-gl-native#9365.

👍 on reducing the scope to markers for now.

Is there a timeframe on getting this PR merged? I'm currently in the process of migrating from Google Maps and this is one of the main features missing on MapBox for my usage.

@simon-the-canadian I'm working on rebasing this PR on latest changes of master.
Will merge once CI has approved.
cc @lilykaiser

@tobrun Are you guys planning on releasing an artifact to maven soonish?

When I run this code. Not working for me.
org.maplibre.android.exceptions.CalledFromWorkerThreadException: Map interactions should happen on the UI thread. Method invoked from wrong thread is latLngForPixel. at org.maplibre.android.maps.NativeMapView.checkState(NativeMapView.java:119) at org.maplibre.android.maps.NativeMapView.latLngForPixel(NativeMapView.java:713) at org.maplibre.android.maps.Projection.fromScreenLocation(Projection.java:96) at org.maplibre.android.maps.Projection.getVisibleRegion(Projection.java:153) at org.maplibre.android.maps.Projection.getVisibleRegion(Projection.java:122) at commichelin.clustering.view.DefaultClusterRenderer$RenderTask.run(DefaultCl


Added NullTime, NullBytes & Null* helpers

Null helpers&types

Wrappers for sql.Null* types: NullBool, NullString, NullFloat64, NullInt64, NullTime, NullBytes
Helpers for initializing Null* values:  NullBoolFrom, NullStringFrom, NullFloat64From, NullInt64From, NullTimeFrom, NullBytesFrom

Hi Samuel,
Could you please review this pull request?
Thanks

Hi Derkan,
I'm sorry, but I'm really busy at the moment.
Just an advice : you should use separate branches per features (not master one).
The NullXXXX helpers are useful. But I don't understand why using new types, as their carry no information (except sql.NullXXX), and they're not used as method receivers.
Why not simply create helpers like this one ?
func NullStringFrom(v string) sql.NullString  {
	return sql.NullString{String: v, Valid: true}
}

Another big problem is the change in dbreflect : https://github.com/derkan/godb/blob/master/dbreflect/scannable.go#L17
You didn't add new types, you changed existing code removing the use of sql.NullXXXX ones. Users should use sql.NullXXXX if it's their choice.
But if types.NullXXXX types are removed to just have helpers, the problems will be solved.
Adding new Null types wrapping standard types like time.Time or []byte could be useful.
What I thing is :

NullXXX helpers are useful, but no the associated type.
Wrapping standard types (simple or from stdlib) in nullable new types is useful.
keeping them together in a types sub package is ok.

In any case, thank you for your contribution.
Sam.

Thanks for review, it was useful. From your suggestions I'll make these changes:

Remove, Null* type wrappers for sql.* types, but leave  NullFrom* helpers. ( I put sql.* wrappers for convenience, import single package and look types in a single place for IDE autocomplete. But it is not a must)
Revert changes for sql.* types in dbreflect.
Erkan.

Hi Sam,
Please review changes when you have a time.
I found this document very useful, and will try to use branch'es for my next pull request as you suggested. :-)
Erkan.

I've merged the PR. I just made some correction in comments and doc, I think you made a generous replace of nullablewith types ;)
Thanks for the contribution.


Move gcloud.storage.iterator into gcloud's common package

We'll need this for Cloud DNS in the ManagedZones.list method (https://developers.google.com/cloud-dns/api/v1beta1/managedZones/list)

Closing. We'll address this if / when it comes up, but YAGNI for now.

Relevant again due to #933.


Frost demon??

I just wanted to ask if there is any ice/frost based demon. I can't find any.    Thanks

Ancient Apparition ?

Oooooooooo

If you're asking about mythology , then you could say the Leviathan  (since it controls water) can control ice too. But true - there aren't that many "ice-based" demons. On the other hand there are tons of ice angels you can search up.
Also , if your question was answered , please close this issue , since it's not related to any kind of suggestion. If you want to ask questions , please use the aaa_discussion discord channel.
Once you have a ready boss mechanic(or item or boss_item , etc.), feel free to post the suggestion here ^_^

And , please , use this https://github.com/AngelArenaAllstars/suggestions/commit/ea0184157763dd320303937cb71472e815999ca9
as a base for your boss suggestion

Thanks


Best Before date not working

Whatever best before date I put in when I purchase something it comes up on Grocy with today's date, unless the product is set not to expire in which case it has today + 100 years.
I am using the android app (updated a few days ago) with Grocy version 2.7.1.
I am in the UK so I wonder if it's a locale issue?
Otherwise great app! Thank you!

I am in the US, and experience same issue on Android with Grocy 2.7.1

Same here (Germany). So it's probably not locale related? But weird anyhow.

Sorry all I'll take a look this afternoon.
On Fri, May 22, 2020, 11:40 AM Andreas Schneider<EMAIL_ADDRESS>wrote:

Same here (Germany). So it's probably not locale related? But weird anyhow.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/PantryParty/pantry_party/issues/81#issuecomment-632759489,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABQ6MQTJBEZU2ZUVWYYDELRS2MF7ANCNFSM4NHW6URA
.

Sorry all I'll take a look this afternoon.

Don't be sorry! Thanks for all the effort building, maintaining and improving this app! This makes grocy so much better to work with.

Fix for this will be released later today

Sorry all I'll take a look this afternoon.
…
On Fri, May 22, 2020, 11:40 AM Andreas Schneider @.***> wrote: Same here (Germany). So it's probably not locale related? But weird anyhow. — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#81 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AABQ6MQTJBEZU2ZUVWYYDELRS2MF7ANCNFSM4NHW6URA .

Working for me now, thank you!


Dependency issue with test requirement

This component ships a trait used by other components in tests.
Now with 07d53d35fdeb89ade8d6f72fe5ca2497f89b57f1 the tests have been upgraded to PHPUnit5 and tests of other components fails because they still using PHPUnit4.
E.g.: https://travis-ci.org/marc-mabe/zend-cache/jobs/203261654
The main problem is not to have a reusable trait for tests or the upgraded PHPUnit dependency but:

The dependency of PHPUnit5 is defined in require-dev - so only valid running zend-servicemanager in development environment but not verified by other components
Can't be moved to require as then every production environment would load PHPUnit and it's dependencies
The test trait will be available for everyone even in production environment but they shouldn't use it :confused:
The test trait is in namespace Zend\ServiceManager\Test but it should be in ZendTest\*

To solve this I would do the following (like what I started to do with splitting zend-cache):

create own repo for reusable test classes (like zend-servicemanager-test)

set autoload (not autoload-dev) to load the common test classes in namespace ZendTest\*
set require (not require-dev) to depend on zend-servicemanager and everything else required to use these test classes

add require-dev: zend-servicemanager-test in this component and all components using this class
-> Now the dependencies of such test classes are defined by composer and they are not part of any production environment

How this would look like as I started to do that with zend-cache:

https://github.com/marc-mabe/zend-cache-test
https://github.com/marc-mabe/zend-cache
https://github.com/marc-mabe/zend-cache-apc

Ping @weierophinney @Ocramius

This is causing problems with Zend Validator too: https://travis-ci.org/zendframework/zend-validator

i also start working to update all modules, which i found, to work with the latest phpunit version.

Honestly, I think the approach by @kokspflanze is the best one for us to take. PHPUnit 5.7 is considered "legacy" currently (technically "old stable"), with version 6.0 the current stable release; the PHPUnit 4 series is no longer supported. As such, we need to start updating all components to pin them to newer versions. PHPUnit 5 is needed when testing against PHP 5.6, and PHPUnit 6 for versions 7 and up. We've done this across a number of components recently, most notably the Expressive components, and have approaches that work across each in terms of testing on Travis.
The main "gotchas" we've observed when updating are:

setExpectedException() goes away, and needs to be replaced with expectException(), plus optionally expectExceptionMessage().
PHPUnit\Framework\TestCase should be used instead of PHPUnit_Framework_TestCase.
A number of commonly used classes from the phpunit package do not have namespaced, forwards-compatibility shims, which require doing things like class_alias hacks, to allow using namespaced versions of classes that will work under both major versions of PHPUnit.

I've found that most adaptations to components take less than an hour, and I see that @kokspflanze has already started submitting a number of these in the past week; if we can get these merged soon, and get others to help in the effort, we can get these cases sorted fairly quickly.
While we could certainly split the test trait(s) into their own package, I hate to do this, as it adds yet another dependency during normal development, particularly when this is a transient problem.

@weierophinney from my site i see the biggest problem to support PHPUnit 4, 5 and 6, some modules like zend-validator start with min php 5.5 so they want PHPUnit 4 but there is no expectException method, these method added with phpunit 5.2 code, so i stuck atm there to find a good solution without writing hacks.
maybe someone have a good solution for that.
from my site i try 1 or 2 modules each week, some modules take bit longer some are less than 1hour done.

@kokspflanze

…to support PHPUnit 4, 5 and 6…

Why? PHP 5.5 and PHPUnit 4.8 is no longer supported!

can i realy remove php 5.5 in modules like zend-validator?

@kokspflanze — We can issue new minor releases that bump the PHP requirement to ^5.6 || ^7.0 for any component at any time; I'm a bit surprised we haven't done so with zend-validator already, as we've been steadily doing so with new component releases for the past year!

okay thats cool=) didnt know it that we can easy set it to the latest version=) that save many time

This repository has been closed and moved to laminas/laminas-servicemanager; a new issue has been opened at https://github.com/laminas/laminas-servicemanager/issues/19.


Always visible layers

While looking into problem #606, I found another problem. The demo project creates no new Map object and than replace the MapControl.Map property, but uses the predefined MapControl.Map and than adds another layer for the raster map. Why is this a problem? The Pin layer (and the other two layers) that the Forms MapView uses, are added to the Map, when the MapControl.Map property change. But in this case, this never happens.
I think, we have three possible ways to handle this:

Document, that the user has to create a new Map, add layers s/he wants and than set this Map to MapView.Map, so I could add the Forms layers on top.
MapView attaches to Map.Layer.OnLayerAdded and OnLayerRemoved. Each time one of this events is fired, the 3 layers of MapView will be removed and added again. If not, they are not the top most layers and so aren't visible.
LayerCollection has the possibility to define special layers, which are always on top (in this case the last layers in array) of all other layers.

Any suggestions, which is the right/best way?

Seems to be the same as #598.

I see the problem. We will need some  better way to order layers. Let's discuss this further in #598. I will close this one.


drm_info: 2.3.0 -> 2.7.0

Description of changes
Was looking for a utility to dump drm nodes, found out this package was moved to gitlab. So hereby adopted and updated to the most recent version.
Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

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

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/4654

Please rebase on master

Please rebase on master

Done


Feature/allow client to request specific related counts

Purpose
Issue: https://openscience.atlassian.net/browse/OSF-5725
?related_counts parameter should be modified to allow the client to request specific relationship accounts, passed as a comma-separated list.
To retain backwards-compat, i.e. passing a boolean should also still work.
Changes
Modified relationship serializer.
The following now works:

?related_counts=children
?related_counts=children,comments

The following still works:

?related_counts=true

Minor changes above made during merge.

@pattisdr @sloria Does this work for linked_nodes in collections?
/v2/collections/?related_counts="linked_nodes"

It doesn't work for me locally so I wanted to check if there is a spelling issue?

No quotes I think?
On Mon, Feb 15, 2016 at 10:52 AM, Caner Uguz<EMAIL_ADDRESS>wrote:

@pattisdr https://github.com/pattisdr @sloria
https://github.com/sloria Does this work for linked_nodes in
collections?
/v2/collections/?related_counts="linked_nodes"
It doesn't work for me locally so I wanted to check if there is a spelling
issue?
—
Reply to this email directly or view it on GitHub
https://github.com/CenterForOpenScience/osf.io/pull/5028#issuecomment-184267428
.

--
Dawn Pattison
Developer
Center for Open Science
<EMAIL_ADDRESS>

Yep - http://localhost:8000/v2/collections/?related_counts=linked_nodes. No quotes.
Any relationship field on a serializer should work.

No quotes is not working for me locally. I'm getting this:

The self.parent change to self.root before the merge seems to be causing @caneruguz's issue above. @sloria - is it okay to change back to self.parent?

Made that change. Thanks for looking into it, Dawn. @caneruguz develop has the fix.


Attempt to open cnn_infer failed  (libcudnn_cnn_infer.so.8 linking)

Which image/versions are related to this issue/feature request?
pytorch-cuda12:latest
Issue/Feature description
/usr/share/torchvision/.venv/lib/python3.11/site-packages/torch/nn/modules/conv.py:456: UserWarning: Attempt to open cnn_infer failed: handle=0 error: libcudnn_cnn_infer.so.8: cannot open shared object file: No such file or directory (Triggered internally at /tmp/build-via-sdist-qyw2z4w4/torch-2.2.1+cu121/aten/src/ATen/native/cudnn/Conv_v8.cpp:78.)

This may be a linking issue, see https://github.com/microsoft/WSL/issues/5663#issuecomment-1068499676
To replicate, try running the script for the pytorch-cuda getting started  guide with the provided data.
It seems it would be best if this were resolved before the CUDA beta launch.

I duplicated this as https://github.com/chainguard-dev/image-requests/issues/1755 we can close this one now.


Make transmute_copy<T,U> with sizeof T != sizeof U a compile time error.

Make transmute_copy<T,U> with sizeof T != sizeof U a compile time error.
rendered

Unfortuntately, the suggested implementation doesn't compile:
use std::{mem, ptr};
pub unsafe fn transmute_copy<T, U>(src: &T) -> U {
    mem::transmute(ptr::read(src as *const T))
}

fn main() {}

<anon>:3:5: 3:19 error: cannot transmute to or from a type that contains type parameters in its interior [E0139]
<anon>:3     mem::transmute(ptr::read(src as *const T))
             ^~~~~~~~~~~~~~

I guess I should have tried it first. In that case it should just be another intrinsic maybe.

Here are some miscellaneous thoughts in no particular order:

The sanity checks on transmute are not there to guarantee safety (after all, they do not). They are there because otherwise we had the problem that a library which transmutes from T to U can cause its clients to fail to compile for invalid combinations of T and U, and that is something we try to avoid. (To make matters worse, the failure occurred in inlined code, which had at the time (and maybe still does) invalid spans, making it very had to track down what transmute was actually causing the problem.)

When we adopted the rules for transmute, we were aware that the fix we adopted imposed a limitation on clients that they could no longer use transmute from generic code. This was deemed acceptable precisely because transmute_copy (and other related functions) offered a workaround.

My preferred fix for this would be to remove the special transmute rules entirely and instead introduce a special trait (Transmutable). This would allow generic code to indicate to its caller that T and U must have the same size and have everything be properly checked. Such a trait could be introduced later post 1.0 and we could simply deprecate transmute and transmute_copy, though I say that if we do so, we must give the replacement for transmute a similarly awesome name.

Given the facts above, at the triage meeting we decided to close this RFC. If we're going to change anything here, I'd want to change it to use a special trait as described above.

It seems like we could probably add a Transmutable bound to transmute backward compatibly, since the current rules are designed to emulate that.


Support for passing a function for click events on the map

Describe the feature you'd like to request
As far as I see there is no support for passing a function that will be executed for click events on the map. It would be useful, e.g. to un-select markers (there is support for onClick events for markers already...)
Describe the solution you'd like
Pass functions for on-click events to the underlying google maps API
Describe alternatives you've considered
I think it can be achieved in a direct way

Since version 1.2.2, you can now pass any supported event to the main div.
<GoogleMap
  ...
  events={[
    {
      name: 'onClick',
      handler: (e) => console.log(e),
    }
  ]]
/>


You take too little oxygen to breathe

Description
When using an internals oxygen tank, you only need 3.1 kPa to breathe without the o2 level popping up. Normally it's be somewhere around 16 kPa. You can also survive indefinitely on 2.5 kPa of pure oxygen, but if you were asphyxiating prior 2.6 is recommended.
Screenshots

Additional context

fixed by #6022


Frontend integration of measure filters

Reference design doc: https://www.notion.so/rilldata/Measure-Filters-1c408bb456e64e2284392847d700b130

Good catch! Have fixed the issues. We really should move to shadcn's select components. The ones we have are lacklustre.


array sort returns an array

Changelog category (leave one):

Bug Fix (user-visible misbehavior in official stable or prestable release)

Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):
...
Array sort return an array so that the result can be used for further processing.

Information about CI checks: https://clickhouse.com/docs/en/development/continuous-integration/

Merging as per confirmation from @kashwy in slack.


[Tests] TabBar snapshot test is causing ObjC test suite to fail

Appears to have started at: https://github.com/facebook/react-native/commit/3b6d029a55efe1ffa5afcafe60d2d2445aadd502. Looks like it may just require new snapshots.

@javache fixed this. Thanks!


[BUG]: HRMS UI does not show a department in the dropdown

Describe the bug
An employee cannot be created via the HRMS UI without a department assignment. Currently, department dropdown shows "No options" and therefore, the "SUBMIT" button is greyed out.
To Reproduce
Steps to reproduce the behavior:

Go to https://dristi-kerala-uat.pucar.org/workbench-ui/employee/
Enter a HRMS Admin user or super user credentials and click on "LOGIN"
Click on the HRMS card in the UI and click on "Create Employee"
In the create screen, click on the Department dropdown. It shows "No options".

Expected behavior
The department master should be loaded from common-masters or egov-hrms masters. These are not configured in the DB currently. Hence this problem.
Screensho

ts
Fix
Check the MDMS schema to ensure HRMS masters are configured in MDMS v2. If not, add the department and designation masters to MDMS v2 and load appropriate data. @Taherabharmal can you please provide the values for department and designations for HRMS.

@shashikesh12 - Please refer to this document here for reference. You can also ping @aaradhya-egov for some clarification.

You can reach out to me in case you need clarifications. Please check if the appropriate masters are loaded in MDMS v2. @shashikesh12

@subhashini-egov i checked the API call that we are making from that page and we are getting data there from backend, but that's not coming up as part of dropdown. So added UI Label to it.So, someone from UI team can investigate on it.

@shashikesh12 can you point out to what are the values, because I could not find the right department vales in MDMS data being inserted via script as shared by @anirudh-0  (https://github.com/pucardotorg/pucar-DevOps/blob/Solutions-Pipeline/deploy-as-code/charts/backbone-services/postgresql/init-scripts/uat.sql)
it rather has eGov's PGR data which should be removed.
@subhashini-egov @manimaarans @suresh12 @Ramu-kandimalla

@atulgupta2024 this is the data we are getting

Hi @Taherabharmal  Can you please share the MDMS data

hi @rajeshcherukumalli this issue is fixed. can you please test it. cc: @Ramu-kandimalla

Its working as expected.

url: https://dristi-kerala-uat.pucar.org/workbench-ui/employee/hrms/create
reference link: https://jam.dev/c/9ef82ffc-a3fc-4125-ba1c-172f6ceb3e9a
Test scenario:
1.Verified that 'Department' dropdown have option.

Verified that options is selectable.
-Test cases:
https://docs.google.com/document/d/1BX4MSbvZqal3yAhH3sitVlHe58xrzdR9RlGQt4HDYRo/edit?tab=t.0

@kashish384 @Ramu-kandimalla - https://docs.google.com/spreadsheets/d/1FnqPw9iBYr12K-eMY-urPC2t7AHZvw1ceORkSPekQWw/edit?usp=sharing -- Masters for HRMS should be loaded from this sheet. Only use what is marked in green. EmployeeType, Department, Designations need to be updated. Can you please let me know if you need another ticket for this?? This is not yet done.

@Taherabharmal please update this ticket here if there are changes. Thanks.

@Ramu-kandimalla @subhashini-egov master data is updated according to all green entries in master data sheet that's shared above

Its working fine in QA[sol env.

url:https://dristi-kerala-qa.pucar.org/workbench-ui/employee/hrms/create
Reference link: https://jam.dev/c/3cbd3a42-7e28-48de-b37c-c6497d2405e8
Test Scenario:
1.Verified that user able to see options for 'EmployeeType' dropdown.
2.Verified that user able to see options for 'Department' dropdown.
3.Verified that user able to see options for 'Designations' dropdown.
Test cases:
https://docs.google.com/document/d/1isjcatPZyRIFfGVJ3Et-4ziWFZlKAD4FFJvYZM4y-WY/edit?tab=t.0


Fix false positives for Style/RedundantDoubleSplatHashBraces

Fixes #12263.
This PR fixes false positives for Style/RedundantDoubleSplatHashBraces when method call for no hash braced double splat receiver.
The code shown in #12263 is incompatible as shown below and is a false positive:
def x(opts) = puts(opts)

h = {foo: :bar}

x(**h.merge(k: :h)) #=> {:foo=>:bar, :k=>:h}
x(k: :v, **h)       #=> {:k=>:v, :foo=>:bar}

Before submitting the PR make sure the following are checked:

[x] The PR relates to only one subject with a clear title and description in grammatically correct, complete sentences.
[x] Wrote good commit messages.
[x] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[x] Added tests.
[x] Ran bundle exec rake default. It executes all tests and runs RuboCop on its own code.
[x] Added an entry (file) to the changelog folder named {change_type}_{change_description}.md if the new code introduces user-observable changes. See changelog entry format for details.

Thanks!


My Popular Food

Ask any Question

Hi @96mypopularfood!
Unfortunately, this isn't the right place for your contribution.

Your code seems to be Ruby, but this is the repository for the PHP SDK. You probably want to use the Ruby SDK
Pull Requests are used to change the SDK code, you seem to want to create a specific implementation using one of our SDKs. No PR is needed for that.

In case you're struggling with implementing our clients, please contact us at<EMAIL_ADDRESS>and we will be happy to help you get the most out of our SDKs!


Feature/optimize get names values

BTW, there is already nuget package for it, https://github.com/andrewlock/NetEscapades.EnumGenerators/

@YuriiBabenko Thank you for your participation
Please run the benchmark and share the improvement result here


Use noexcept for move constructors

We should not throw exceptions during move constructors.

@derekmauro @MBkkt Done!


color: inherit causes links to look like plain text

inherit causes the element to take the computed value of the property from its parent. As that's a <p> it makes it the same colour as the text.
 Pretty sure he was fast asleep!

Oh I was, but I'm all for making an experience consistently bad -- regardless of your ability.

:rofl:


transmit branch not compiling

Hi,
I just tried to npm install you transmit branch, but it complains about a missing cec.h file. Any clue ?
Thanks.

The cec.h file is part of the libcec library here: https://github.com/Pulse-Eight/libcec
I have the libcec library but I still can't get this to compile.