How to test a query which returns no result

I have just migrated to angularfire2 5.0.0-rc.2.
Before I had this :
const searchHashtag = this.searchHashtag(hashtag);
const subscribe = searchHashtag.subscribe(snapshots=>{         if(snapshots.length > 0){            ....        }else{           ....        }
and now i replace it by this:
const searchHashtag = this.searchHashtag(hashtag);
const subscribe = searchHashtag.snapshotChanges().subscribe(snapshots=>{         if(snapshots.length > 0){            ....        }else{           ....        }
But when the searchHashtag return no results, the subscribe are no trigger.
How i can fix it ?
thx

@Anthony2539
Try This,
const searchHashtag = this.db.list('hastag/').valueChanges()
const subscribe =  searchHashtag.subscribe( (snapshots:any)=>{ if(snapshots.length > 0){ .... }else{ .... }

Working on it now #1220


Color change during transcoding?

Hi Don,
I've been using other-transcode since you released it and I like it a lot, thank you for your efforts in making lickety-split hardware encoding look as good as software.
Something odd happened today though and I wanted to flag it for you. When I transcoded a 4K HEVC BD rip of "Casino Royale" to 1080p H.264, I was surprised to find that the transcoded 1080p had very different colors than the original. I've used other-transcode to transcode VC-1 BluRay rips to 1080p H.264 at least a dozen times and never noticed this issue until now.
Here's a screenshot of the original 4K HEVC BluRay playing in IINA:

And here's a screenshot of the 1080p file after other-transcode, also playing in IINA:

It seems that transcoding HEVC to H.264 changes the gamma somehow, or at least with my setup and this particular BluRay. Did I miss something in terms of options when transcoding 4K HEVC to 1080p H.264? I tried transcoding again but with the --no-filters option and it did not fix the colors.
It's worth mentioning that playing the BluRay rip with IINA shows the true color balance, but if I use IINA to generate a screenshot, the PNG shows the same color anomalies as the transcoded 1080p file. If I use MacOS's native Screenshot utility instead, the PNG shows the proper colors.
Anyway, not life or death, but definitely weird enough to report back to you in case anyone else is seeing this issue.
FWIW my system is a Skylake i3-6100 running MacOS Mojave, with an AMD 280X GPU (although other-transcode seems to ignore the AMD and just use the Intel HD 530 internal GPU).
Any ideas? Thank you again for all your amazing work!
Corey

@coreygreenberg
If you compare software-based transcoding (other-transcode --x264-avbr) with VideoToolbox-based hardware encoding (other-transcode), then your mileage will vary. Yes h/w transcoding will be faster as you've seen and the quality will be similar in most cases but there will be scenarios when the VT-based h/w transcode may look worse due to more excessive colour-banding or blockiness. It's mainly because the h/w cannot be tuned due to the closed nature of what's available. So give both a shot, use other-transcode for the majority and then if something doesn't look quite right via h/w, drop back to other-transcode --x264-avbr
Also, I suspect we'll find that IINA is another player which decodes HEVC incorrectly ... which is why you're seeing the blockiness. Native MPV and Plex's Enhanced Player correctly decode 8- and 10-bit HEVC.
And one point to note ... we're not constantly upgrading at every point. It doesn't work that way at all - perhaps if you're into Fortnite, yeah sure but transcoding is a different beast.

Intel gen 9 -> gen 10 = no transcoding benefit.
The new Ryzen's = no transcoding benefit (even though they can be seriously multi-core but we're not focusing on s/w transcoding anymore), $250 Nvidia 1650 Super -> $1000 Nvidia 2080 Super = a small percentage faster transcoding due to memory but the NVEnv chip is the same so effectively no benefit
I could also talk about motherboards, storage and I/O (M.2, USB 3, USB-C, SATA, SSD etc.), networking and NASs ... marginal benefits.

So it's important to realise that hardware transcoding is not about the latest new thing but does the latest new thing actually bring any transcoding benefits. Most don't until there's a major leap in things like NVEnc, Intel Xe or AMD's new stuff later this year.
Anyhow, enjoy "One Upon a Time ... in Hollywood" ... it's a good movie :-)

@rhapsodians
So I compared the x264 encode with what I got from other-transcode default H.264 and despite it taking approx 6X as long to encode, the image quality was indistinguishable to me. So as you said, I’ll stick with VT for now and if I see something jicky I’ll try it with the x264 option.
I’m pretty sure IINA is just MPV with a GUI, at least that’s what the dev claims and what everything I Googled seems to confirm. And I first noticed the weird 10-bit HEVC artifacts watching “Once Upon” on my TV via Plex on Apple TV, not on a desktop media player. I merely confirmed that the artifacts were there by taking a second look on the desktop with IINA. I’ll try native command line MPV though and report back if I see any difference playing that problematic 10-bit HEVC encode.
Maybe my comments about needing to upgrade hardware every year to keep up with the Alpha Don were a bit overblown but I am afraid if getting the very best that other-transcode can deliver means building a Windows 10 PC with a newish Nnvidia GPU, I’ll settle for the lower tier I’m getting on a Mac with VT. I’ve got enough spare parts to throw together a Windows box and a 1050 off eBay will cost me $100 or so, but I swore off Windows decades ago and every time I’m asked to sort something on a friend or colleague’s Windows PC, it’s an awful experience. I don’t wish to keep a Windows box running on my network just for transcoding. My hope is that FFmpeg catches up the Mac and Intel/AMD GPUs to where Windows and NVidia are with Don’s scripts now, but if that never happens, I’m happy with what I’m seeing from my current setup. I look at this like I do Catalina — if this is really where Apple is going with macOS, I never thought I’d ever be that guy but I guess I’m standing pat with Mojave and if future apps won’t run on my OS, I’ll use whatever still runs under Mojave until Apple gets macOS back to where it was before this deal-breaking debacle.
Agreed on “Once Upon”. Easily my favorite of his films since the “Kill Bills”. I could watch Brad Pitt’s stunt double throw fake Bruce Lee into the side of that beautiful car on endless loop all day. And I’d watch a whole movie of the fake Steve McQueen. I never realized how much Damien Lewis looks like him. He needs to star in the remake of “Sand Pebbles”.
So I have a question for you and the other Meltonians. Don at one point listed some specific movies and scenes that made life particularly hard for transcoding and thus were good benchmarks for tasting. Do you have any specific discs and/or scenes you lean on when comparing different transcoding schemes? Other than that weird glitching when my Skylake puked on 10-bit HEVC, I have yet to notice anything awry when transcoding any of my discs since even the first rev of other-transcode. Any torture tests you recommend would be welcomed and definitely explored here.

@coreygreenberg The Blu-ray version of "Blade Runner 2049" is an excellent test case rife with many scenes prone to color banding. And they're hard to miss. I would recommend trying that first.
My thanks to everyone adding content, but I think we've strayed from the original topic here. While the color change problem you noticed is definitely undesirable, it is, as I mentioned before, a limitation of both FFmpeg and the various encoders. It's not something I can fix or even work around in other-transcode.
I'll close this now but feel free to continue commenting. Thanks.


Update scalafmt-core to 3.0.6

Updates org.scalameta:scalafmt-core from 3.0.5 to 3.0.6.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scalameta", artifactId = "scalafmt-core" } ]

labels: library-update, semver-patch

Codecov Report

Merging #198 (c546613) into main (7beaf11) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #198   +/-   ##
=======================================
  Coverage   88.85%   88.85%
=======================================
  Files          91       91
  Lines        2772     2772
  Branches       70       70
=======================================
  Hits         2463     2463
  Misses        309      309

Continue to review full report at Codecov.

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


[T2A2][W10-A4]Li Zihan

Refactor & MethodAbstraction

Ready for review

Ready for review

@lizihan97 You seemed to have included all exercises in this one PR.  Please only do the exercises for a specific assignment on the associated and make a PR for each assignment.  It's difficult to review as everything is mashed together.
Next time please?  Accepted without review for this one...

@hwkchia the commit HashMap+Enum+Varargs is the original file before I did the activities for T2, each commit after HashMap+Enum+Varargs in this PR shows one step of T2A2. You can review my commits for T2A2 starting from Improve boolean variable and function names.  Sorry for the mess.


cmd + B doesn't toggle sidebar visibility

Type: Bug

open any directory
open explorere
Try performing cmd + B

VS Code version: Code 1.76.1 (5e805b79fcb6ba4c2d23712967df89a089da575b, 2023-03-08T16:48:08.231Z)
OS version: Darwin arm64 22.3.0
Modes:
Sandboxed: No

System Info

Item
Value

CPUs
Apple M2 (8 x 24)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmetal: disabled_offmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_renderer: enabled_onvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: disabled_off

Load (avg)
2, 2, 2

Memory (System)
16.00GB (0.28GB free)

Process Argv
--disable-extensions

Screen Reader
no

VM
0%

Extensions disabled

I see that the key binding is there

Closing due to lack of info, please feel free to re-open if you disagree.

The issue still persists in March release of VS code.

Please follow the steps here to file useful keybindings issues https://github.com/microsoft/vscode/wiki/Keybinding-Issues


manual cursor observers may attempt to update when orwell wrapped component is unmounted

use this hack: https://github.com/facebook/react/issues/3417#issuecomment-121649937
handleCursorChanged() should become noop when orwell component is unmounted. and should output a dev message warning
pass function to test if component is mounted to manual:

manual((update, isMounted) => { ... });

fixed by https://github.com/Dashed/orwell/commit/d60812fcb62d96f2835a62c81def114e339bfb26 and https://github.com/Dashed/orwell/commit/099e4ef45adadd13e674fbb944e1e47a06364abd


Modified entry widget to interact properly with keyboard

I have modified the Entry widget to be set to focus throughout the program right from the start. This is because the user won't have to click on the widget to write in it from the physical keyboard.
The user will only be able to input numbers and operators to the Entry widget from the physical keyboard. The operators include: "+", "-", "*" and "/". You can add more in the checkIfNumberOrOperator function.

Thanks a lot. I will review it as soon as the holiday ended!

Unfortunately, I didn't add the function to use the keyboard to interact with the program yet. I will add this in the future.

Thank you.

Should I help in adding this feature?

Always welcome!


🐛  Remove spurious escaping of brackets in NOTES.txt template

Summary
This PR removes spurious escaping of the brackets in the NOTES.txt template in the core Helm chart.
Related issue(s)
Fixes #

/cc @francostellari

I tested on my Mac with test/e2e/run-test.sh. Successful function.
This reminded me that the chart's Values.KUBESTELLAR_VERSION in this case is set to the git commit hash (short version) --- which, nonetheless, makes for a working URL (https://github.com/kubestellar/kubestellar/blob/825c79a01/docs/content/direct/core-chart.md#kubeconfig-files-and-contexts-for-control-planes in this case).

\lgtm


User confirmation flow

I am not sure what should be the exact flow for user confirmation, so creating an issue and ready to discuss before any hack/implementation.
If a user(say userA) is already logged in into the system and another user(say userB) tries to activate her/his account via confirmation link, then which user should be logged-in after confirmation?

[ ] userA ??
[ ] userB ??

\cc @simonv3 @roryaronson

I think User A should be logged out and the confirmation should go to User B. I think User B should definitely not get access to User A's account, and I think logging User A out is the safest option.

I agree with Simon. Upon clicking a confirmation link User A should be logged out and User B should be logged in and have their account confirmed.

@simonv3 @roryaronson This is exactly happening into the system, so I think we can close this issue.


Class refactor

Refactor into classes to add decoupling and scalability, change db system and with it, other parts of code also for scalability, add sqlite implementation to run tests locally and change CI pipeline to accommodate it.

Wrong syntax on CI pipeline...


Board Review: Event Hubs - Load balancing options for processor client

Thank you for starting the process for approval of the client library for your Azure service.  Thorough review of your client library ensures that your APIs are consistent with the guidelines and the consumers of your client library have a consistently good experience when using Azure.
** Before submitting, ensure you adjust the title of the issue appropriately **
To ensure consistency, all Tier-1 languages (C#, TypeScript, Java, Python) will generally be reviewed together.  In expansive libraries, we will pair dynamic languages (Python, TypeScript) together, and strongly typed languages (C#, Java) together in separate meetings.
The Basics

Service team responsible for the client library: Event Hubs
Link to documentation describing the service: https://azure.microsoft.com/en-us/services/event-hubs/
Contact email (if service team, provide PM and Dev Lead): SeongJoon Kwak, Javier Fernandez & Shubha Vijayasarathy

About this client library

Name of the client library: azure-messaging-eventhubs
Languages for this review: Python, Java, .NET, JS
Link to the service REST APIs: N/A

Background and alternatives considered
Event Hubs provides message streaming through a partitioned consumer pattern in which each consumer reads a specific subset, or partition, of the message stream. This pattern enables a horizontal scale for event processing and provides other stream-focused features that are unavailable in queues and topics.
To scale your event processing application, you can run multiple instances of Event Processor Client and have it balance the load among themselves. The processor client periodically checks the current state of partition ownership and determines if partitions have to be redistributed among active instances.
In track 2 SDK for Event Processor client, the load balancing task runs at a fixed interval of 10 seconds. When the load is not balanced among active processors, partitions are claimed/stolen one at a time per load balancing loop. This can take a long time for the processors to reach a balanced state. The proposal in this issue is to provide options to configure the load balancer to speed up the process of reaching a stable state. Specifically, we will allow users to customize:

The time interval for running the load balancer
The time duration to wait before assuming a partition is orphaned
The load balancing strategy to use: balanced (default) or greedy.
-- BALANCED: In this mode, one execution of load balancer task will claim at most 1 partition.
-- GREEDY: In this mode, one execution of load balancer task will claim as many partitions required to balance the load.

For details and other options considered, please refer to this gist.
Artifacts required (per language)
We use an API review tool (apiview) to support .NET and Java API reviews.  For Python and TypeScript, use the API extractor tool, then submit the output as a Draft PR to the relevant repository (azure-sdk-for-python or azure-sdk-for-js).
.NET

API view
Scenario Samples

Java

API view
Link to samples for champion scenarios

Python

API view
Link to samples for champion scenarios

TypeScript

Upload output of api-extractor as a Draft PR.  Link to PR
Link to samples for champion scenarios

Champion Scenarios
A champion scenario is a use case that the consumer of the client library is commonly expected to perform.  Champion scenarios are used to ensure the developer experience is exemplary for the common cases.  You need to show the entire code sample (including error handling, as an example) for the champion scenarios.

Champion Scenario 1:

Describe the champion scenario: User has a single instance of Event Processor Client subscribed to an Event Hub with 30 partitions and want to start processing from all partitions with minimum delay. To achieve that, the user can configure the processor client to use GREEDY strategy that claims all partitions with no delay when the processor starts.

Estimate the percentage of developers using the service who would use the champion scenario - Users who are tolerant to partitions changing ownership frequently at startup before reaching stable state and prefer to start processing from all partitions immediately will use it.

Link to the code sample - https://gist.github.com/srnagar/366cb66279406db738f8eff82fbfb851#sample-user-code

Champion Scenario 2:

Describe the champion scenario: User has an Event Hub instance configured with 5 partitions and there are usually 5 instances of Event Processor Clients running. Also, the application requires extensive initialization and cleanup before and after processing a partition. So, the user prefers to not have partitions jumping around when the application starts up. The Event Processor Client can be configured to claim a single partition for each load balancing update interval by using the default BALANCED strategy. Moreover, the user can also tune the update interval to suit the application needs.

Estimate the percentage of developers using the service who would use the champion scenario - Applications that have expensive initialization and cleanup for processing a partition will prefer to use the default strategy.

Link to the code sample - https://gist.github.com/srnagar/366cb66279406db738f8eff82fbfb851#sample-user-code

Link to the TypeScript PR for API review: https://github.com/Azure/azure-sdk-for-js/pull/9140

@kyle-patterson Now that we are done with the meeting for this issue, can we post the conclusions and close it?

This feature is released. Closing the issue.


Text composables: Android Basics with Compose

URL of codelab https://developer.android.com/codelabs/basic-android-kotlin-compose-classes-and-objects?continue=https%3A%2F%2Fdeveloper.android.com%2Fcourses%2Fpathways%2Fandroid-basics-compose-unit-2-pathway-1%23codelab-https%3A%2F%2Fdeveloper.android.com%2Fcodelabs%2Fbasic-android-kotlin-compose-classes-and-objects#6
In which task and step of the codelab can this issue be found? https://developer.android.com/courses/pathways/android-basics-compose-unit-2-pathway-1#codelab-https://developer.android.com/codelabs/basic-android-kotlin-compose-classes-and-objects Pathway 4
Describe the problem
The course material requires re-writing to make clear what the learner has to do, for example, in the below screenshot, the training team have not explained where to insert the subclass and also, the screenshot shows that we need to write the subclass above the superclass.

This is the solution code, although what is taught in the above course does not align with the solution code.
https://developer.android.com/codelabs/basic-android-kotlin-compose-classes-and-objects?continue=https%3A%2F%2Fdeveloper.android.com%2Fcourses%2Fpathways%2Fandroid-basics-compose-unit-2-pathway-1%23codelab-https%3A%2F%2Fdeveloper.android.com%2Fcodelabs%2Fbasic-android-kotlin-compose-classes-and-objects#9

Hi @AR102023
Thank you for writing in and providing images of the issue.
I will share with the team so they can address it as needed.
Regards.

Thank you

From: John Shea @.>
Sent: Wednesday, March 13, 2024 8:16:21 PM
To: google-developer-training/basic-android-kotlin-compose-birthday-card-app @.>
Cc: Andrew @.>; Mention @.>
Subject: Re: [google-developer-training/basic-android-kotlin-compose-birthday-card-app] Text composables: Android Basics with Compose (Issue #519)
Hi @AR102023https://github.com/AR102023
Thank you for writing in and providing images of the issue.
I will share with the team so they can address it as needed.
Regards.
—
Reply to this email directly, view it on GitHubhttps://github.com/google-developer-training/basic-android-kotlin-compose-birthday-card-app/issues/519#issuecomment-1995650556, or unsubscribehttps://github.com/notifications/unsubscribe-auth/BDNTKDLXTIHFGBZZJ7JZCTDYYCXZLAVCNFSM6AAAAAA7WB7G6KVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSOJVGY2TANJVGY.
You are receiving this because you were mentioned.Message ID: @.***>


Updated plugin SDK

Updated to the upcoming plugin SDK
Simplified CI workflow, removed useless steps
Now CI also builds release plugin
Adapted plugin tickers (following https://github.com/LedgerHQ/app-ethereum/pull/392)

No version bump needed ? Maybe not as it is just boilerplate?


oopsy: add summary view when opened in a browser

This was asked for in #361.

Looks something like:
I'm not sure if it makes sense to use "am I opened via websocket in a browser vs in an overlay" as the determining factor for which view to use, but I think this is a way better view if you have the room for it.  I could also make a separate html file here, even if it uses the same js underneath.
I think this creates a lot of room for followups:

summary view over a zone / entire night
loading old logs into it (like raid emulator, but for oopsy)
shortcut for creating links to timestamps in twitch video? (provide a video link and a start time?)

I'm not sure if it makes sense to use "am I opened via websocket in a browser vs in an overlay" as the determining factor ... I could also make a separate html file here, even if it uses the same js underneath.

I think the latter makes more sense here; something like oopsy_report.html/oopsy_summary.html, but I wonder which option drives feature adoption more -- I can see an argument for both, and I can see making either option fairly painless for the user. I wonder if we should write something out to the console when oopsy loads saying, "Summary report available at " or something similar; that way we could realistically do whatever made the most sense for us from a technical standpoint.

Unfortunately, it'd be hard to say "summary at url".  You'd need to know what port the OverlayPlugin WSServer was running on.  I think documentation is probably the best bet here?
Ok, so assuming I make a oopsy_summary.html page.  The other wrinkle to making this more accessible is that the OverlayPlugin WSServer tab will give you such URLs, but it currently only does so for overlays you have enabled.  (ngld has mentioned maybe making this use presets instead of the set of current overlays, but it's the same problem.) In either case, there won't be an oopsy_summary preset or overlay, because it's not meant to be added as an overlay.  So it will still be somewhat difficult to find this url.
I think for backwards compatibility, it'll be tough to rename oopsyraidsy.html proper.  So the questions are, should oopsyraidsy.html behave magically and check WSServer status? (This would help with the use via the OverlayPlugin-provided url?) Should we have separate html for summary? and live list too?
Options:

html proliferation: have oopsyraidsy (can be both) / oopsy_summary (only summary) / oopsy_live (only live).
somewhat magic: have oopsyraidsy (can be both) / oopsy_summary (only summary)
entirely magic: only oopsyraidsy (can be both)
no magic: oopsy_live (only live) and oopsy_summary (only summary)

Ok, I ended up going with the "no magic" option.  oopsyraidsy.html is the live list and oopsy_summary.html is the summary view and there's no "do something different if you're loading via websocket" shenanigans (which I think would cause issues with people loading oopsy as an OBS overlay anyway).


Selected item from suggestions widget is truncated when displayed as gray text

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version: 1.60.0-insider (user setup)
OS Version: Windows_NT x64 10.0.19043
List of extensions: None, all are disabled

Steps to Reproduce:

Create a new typescript file.
Type fun, the suggestions widget opens and the first suggestion is function.
The gray text however, only shows functio, it's missing the last character.
I've attached a gif. Notice how when I type fu, the gray text displays function but when I type fun, it shows functio.

@hediet

@juihanamshet1 can you please verify todays insider build fixes this?

@hediet Yes, it has been fixed!


fix(mtls) update to use newer mTLS api

The code path requires unmerged commits to both lua-nginx-module and
lua-resty-core to work. Previously those commits introduced an
alternative handshake method, tlshandshake. They have since been
revised, in the hope that the feature will reach an OpenResty mainline
release soon.

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

This can wait until the mTLS API arrives in mainline OpenResty.

Hi @pintsized !
Thank you for this contribution! We've merged these changes in another PR: https://github.com/Kong/lua-resty-healthcheck/pull/133. They were merged to 1.6.x branch and the master branch still was missing this change back then - but with the latest cleanup of https://github.com/Kong/lua-resty-healthcheck/pull/144 the master branch caught up to speed and now it has this code.
Thanks again for bringing this up 🏆


add kramdown, github-pages to README install

these are needed in addition to jekyll

I believe all these are dependencies of github-pages gem. Should we recommend to install that ?


Different configuration key prefixes are confusing

For axon-server, the prefix is axoniq.axonserver.maxMessageSize: 20_000_000;
For client application, the valid prefix is axon.axonserver.maxMessageSize: 20_000_000;
"axoniq" vs "axon".
Not sure if it's designed to be like this, or just a not finished renaming.

Understandable that it's confusing, but simply changing would mean breaking changes with minor releases; something which we aim to avoid always. We'll discuss whether there's an opportunity to change this around too.

Acceptable, hope this page may help someone out, if entered this situation.


Backup snapshot volume data to an external store

It would be great if this tool could also back up the data in the snapshot to s3, minio, or nfs. Restic is a great tool for this and would love to see the integration here. This would cover most disaster/recovery if your cluster was ever nuked or completely fubar.
Thanks.

Unfortunately something like this is out of scope for Gemini at this time. We want to avoid creating one-off integrations with storage providers, and instead rely on the CSI to handle the backups themselves.
That said, if the cluster is nuked, your EBS snapshots (assuming you're on AWS) should still be in tact, though it might take some work to migrate them back into a new cluster. The underlying snapshots on your cloud provider will only get deleted if you kubectl delete the corresponding VolumeSnapshot.
You could also try velero as a more heavy-weight Disaster Recovery solution


Asic miner Setup

So I'm trying to get away from using a pool to mine. Currently my Asic miner is on poolflare but would rather use my own personal mining client. My problem is your directions are almost impossible to understand.

Same here, we can use some help!

کمک کردن

The README contains more details now about how to use an ASIC miner. An example for a complete compose based setup is available here: https://github.com/kadena-io/docker-compose-chainweb-node/blob/main/mining-node/docker-compose.yaml


Fatal: number of used blobs is larger than number of available blobs!

1 snapshots have been removed, running prune
counting files in repo
building new index for repo
[2:30] 100.00%  1150 / 1150 packs
repository contains 1150 packs (41230 blobs) with 5.557 GiB
processed 41230 blobs: 0 duplicate blobs, 0 B duplicate
load all snapshots
find data that is still in use for 1 snapshots
[0:04] 100.00%  1 / 1 snapshots
Fatal: number of used blobs is larger than number of available blobs!
Please report this error (along with the output of the 'prune' run) at
https://github.com/restic/restic/issues/new
but i have no idea what where that output might be found

Closing this issue in favor of #2663, please add the error message there if you still need help resolving this problem.


[PROJECT PROPOSAL]:Water Quality Analysis

Project Request

---Perform EDA on water quality dataset to determine whether the water is drinkable or not
| About | performing data analysis to determine water quality |
| Github | Venkatapranay |
| Email  |<EMAIL_ADDRESS>|
| Label  | GSSOC'23 |
https://github.com/Venkatapranay

Define You

[x] GSSOC Participant
[x] Contributor

Project Name
Water Quality Analysis
Description
performing data analysis to determine water quality
Scope
can be used in water supply
Timeline
August 6 2023
Video Links or Support Links
dataset : https://www.kaggle.com/datasets/adityakadiwal/water-potability

Hi @miraj0507 @adithya-s-k ! I have raised this issue under GSSOC'23. Can you assign this issue to me.

Hello @adithya-s-k, I would like to work on this issue. Can you please assign this issue to me?

Full name : Sai Bhaskar Kandula
GitHub Profile Link: https://github.com/SaiBhaskar0987
Email ID<EMAIL_ADDRESS>Approach for this Project :
Conducting exploratory data analysis (EDA) on a water quality dataset to evaluate drinkability entails several essential steps. Initially, comprehending the dataset's structure and rectifying it by addressing missing values and ensuring uniformity is imperative. Descriptive statistics and visual exploration aid in comprehending data distributions and detecting any anomalies or trends. Correlation analysis unveils interrelationships among water quality parameters, while hypothesis testing validates assumptions. Optionally, machine learning models can prognosticate drinkability based on quality parameters. Ultimately, interpreting findings and meticulously documenting the process ensures transparency and facilitates informed decision-making concerning the water's potability.
Dataset: https://www.kaggle.com/datasets/adityakadiwal/water-potability
What is your participant role: GSSOC 2024 Contributor
If you are ok with all sir or any changes you want me to include, please tell me and please @Venkatapranay  sir assign me this issue

Sir as I reqested you earlier, Can you please assign me the issue @Venkatapranay.


Bitrise step activate tuist remote cache 1.0.2

https://github.com/bitrise-steplib/bitrise-step-activate-tuist-cache/releases/1.0.2
What to do if the build fails?
At the moment contributors do not have access to the CI workflow triggered by StepLib PRs. In case of a failed build, we ask for your patience. Maintainers of Bitrise Steplib will sort it out for you or inform you if any further action is needed.
New Pull Request Checklist
Please mark the points which you did / accept.

[x] I will not move an already shared step version's tag to another commit
[x] I read the Step Development Guideline
[x] I have a test for my Step, which can be run with bitrise run test (in the step's repository)
[x] I did run bitrise run audit-this-step (in the step's repository - note, if you don't have this workflow in your bitrise.yml, you can copy it from the step template.)
[x] I read and accept the Abandoned Step policy

New Step
Thank you for the new Step share! The CI check might will fail due to our extended validation engine. Nothing to worry about yet, we will get back to you shortly.

Fixed in https://github.com/bitrise-io/bitrise-steplib/pull/3884


Missing Icon Link

The json data that is pulled back has an ICON attribute with a filename;  Where can I get this filename?

/src/drivers/webextension/images/icons
Found it.  You have to build the whole source


:D

@lovell and others, thank you for this library!!!
That is all. 😺

Thanks, it's always interesting to learn more about what people are using sharp for in the survey at #35 if you're able to share details publicly.

I went ahead and dropped a comment.
https://github.com/lovell/sharp/issues/35#issuecomment-350875930


1: updated readme with a template and edited template

Resolves #1
updated readme with template
edited most of the text and markup to reflect needs for this project

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


exmaple to connecting obsidian and svelte together

this is just my solution, how do you think

@caasion I connect Svelte and CodeMirror, and implement some basic components, if you feel ok please merge it and work on top of it

btw I'm not sure if it is something you have done in these commits, but I wasn't able to start the plugin after I complied it.

Good work on getting started though. If this is your first time with svelte then you learn way to quick lol :dizzy_face:.

A bug happens if there is no data.json in the plugin directory. It is fixed already in the main and svelte branches.

yeah, it is my first time with svelt, but I tried React before and that teaches me a lot 😉


Failure during a migration command: ... The value provided for column Some("text") is too long. - Report id 8248

This is an old issue and the error is already clear enough, we're just going to close it.

But has the issue been addressed?

The issue is that a string in the migration (probably a default) is too long for the column — we could provide our own error for that, but it wouldn't be any clearer than the database error. Also note that this type of errors is much more visible with recent versions of migrate, since you actually commit and hopefully review the SQL migration directly.


Add new REST APIs

These APIs are added:

create cachegroup
create server
edit server
assign deliveryservice servers
snapshot CRConfig
create profile
copy profile
create division
create region
create physical location.

@mitchell852 I make changes as your comments, except parent_cachegroup and secondary_parent_cachegroup, since I think parent_cachegroup and secondary_parent_cachegroup  are requried in the API. Thanks,

@dangogh @mitchell852 , I modified with your comments, please check. Thanks.

@dangogh @mitchell852 ,  I resolved conflicts because there are other pull requests merged to master. How the status of this pull request? thanks

unfortunately,   it looks like your merge didn’t go so well.   Your PR now appears to include all the changes that have been submitted,  and so it’s hard to tell what changes you actually made and whether the changes others made are included in the  result.  That makes it really difficult to evaluate this change.     I suggest you go back to the version before this merge (git reset --soft ) and use "git rebase -i upstream master" to make sure only your changes are included in the PR.   I'm happy to try to help walk you through that if you have any trouble..   -Dan

@dangogh @mitchell852 , my fault in git rebase, thanks for pointing out. I cherry-pick to another branch and create a new PR https://github.com/Comcast/traffic_control/pull/1142 . Let's use new PR, thanks.


add port to starter json

I missed the discusion with Alex so I'm curious, how is the port value used?


Error when using !youtube command

"Error: Project<PHONE_NUMBER>92 has been scheduled for deletion and cannot be used for API calls. Visit https://console.developers.google.com/iam-admin/projects?pendingDeletion=true to undelete the project." is returned when using the !youtube command. Untested with other commands. Going to the link it provides gives nothing

Try generating a new api key

Should be fixed, thanks for noticing and sorry for the inconvenience.

still doesnt work. gives "Please setup YOUTUBE_KEY in config.js to use the !youtube command."


How to add iOS Lead Essentials Program mention?

Hi guys and thanks for taking the time to create and maintain this directory, it's really useful!
I wanted to contribute by adding a mention to the iOS Lead Essentials Program by Essential Developer (you probably know about it), but I'm not sure how to fit it to the current structure:

it's not a blog, so it shouldn't go under Blogs (unless you think it should)
there's no Courses / Programs section, should we create one?
other ideas ...

Not sure if this is relevant, but I did finish this program and I'm recommending it.
Thank you!

Thank you @daveverwer for explaining in detail, it makes total sense!


update rctf to v1.0.99

Trying out a small change to trigger the CI/CD pipeline

Closing this- will open a new request to see if pipeline can be triggered


Use double quotes instead of single when running piptools with --pip-args flag

Linked issue is #1621 .
This PR replaces single quotes by double quotes in commands using the --pip-args flag to be compatible with more shells.
Contributor checklist

[ ] Provided the tests for the changes.
[ ] Assure PR title is short, clear, and good to be included in the user-oriented changelog

Maintainer checklist

[ ] Assure one of these labels is present: backwards incompatible, feature, enhancement, deprecation, bug, dependency, docs or skip-changelog as they determine changelog listing.
[ ] Assign the PR to an existing or new milestone for the target version (following Semantic Versioning).

Thanks, looks good!
I don't know whether this will make it to the changelog, but can you re-title this to reflect that it's a documentation change, rather than a behavioral change?


feat(docs): Documenting noir codegen

WIP, waiting on https://github.com/noir-lang/noir/pull/4448
Description
Problem*
Resolves
Summary*
Additional Context
Documentation*
Check one:

[ ] No documentation needed.
[ ] Documentation included in this PR.
[ ] [Exceptional Case] Documentation to be submitted in a separate PR.

PR Checklist*

[ ] I have tested the changes locally.
[ ] I have formatted the changes with Prettier and/or cargo fmt on default settings.

Please tag me when this is RFR @catmcgee 😊

@signorecello it looks like the page didnt get added to the docs preview. It should be there since the sidebar is auto-generated, correct?


Implemented MITRE D3fend

I implemented the MITRE D3fend tags and modified the code so that they can be added automatically with the extension commands.

Thank you for the contribution. Sorry for the long wait time :)


error with el.up()

Uncaught TypeError: Object # has no method 'up'
any ideas?

It was my mistake copying the code but i got now
POST http://www.vajillahogar.es/index.php/ajax/product/quickview/id/65/ 404 (Not Found)
Ajax.Request.Class.create.requestprototype.js:1234
Ajax.Request.Class.create.initializeprototype.js:1196
(anonymous function)prototype.js:248
klassprototype.js:51
ProductInfo.loadInfoproductInfo.js:100
(anonymous function)prototype.js:214
wrapper

This is not a issue just find 'p' tag product image on exist or not so just change on jQuery file with your tag.

the same think, do you have some example? because I could not run anywhere!

Sorry as now I have a full time job, I'm not planning to maintain this repository unless one of my clients needs a new feature / bugfix.
Pull requests are welcome, however.

I was having the same error. But, at least in my case, the problem was in my list.phtml I put the quick view button outside of .product-image block, making the following lines in productInfo.js expendable:
        $$(x_image).each(function(el, index){
            el.observe('mouseover', that.showButton);
            el.observe('mouseout', that.hideButton);
        })

I just commented this block out and the error went away.

 I'll rewrite this thing later. In case you want to implement it yourself, please make a pull request. ;)


CreateMemo documentation possibly out of date

"Memos only notify dependents when returned value changes" However, https://codesandbox.io/s/async-resource-vznh2?file=/index.js logs to console every time the button is clicked despite the value not changing
https://codesandbox.io/s/async-resource-pp4mq?file=/index.js does not exhibit this behavior (has custom comparator).
I think the issue is https://github.com/ryansolid/solid/blob/33436589f3abeecef2de892bc4856ce384c2039a/packages/solid/src/reactive/signal.ts#L113 where areEqual just doesn't have a default value. Of course, the documentation may have just grown stale.

Yep. It worked the way as documented for a very brief time period. I must have changed it and missed updating it back. I decided it was better to make it consistent with Signals in favor of updating to handle more scenarios.


Error encoding 437 .net core 3.1

Raise this error
No data is available for encoding 437. For information on defining a custom encoding, see the documentation for the Encoding.RegisterProvider method.
as workaround i comment the lines
private static Encoding DefaultEncoding = Encoding.UTF8/* Encoding.GetEncoding(437)*/;

I have added it to the static constructor


Improve test setup step docs, and make Ansible run them automatically

This updates the docs to make it clearer how to do DB setup to run the tests, and updates the ansible setup to ensure this gets done immediately so you can successfully run tests in new Vagrant instances (which requires adding SUPERUSER to the Vagrant postgres user, so they can drop/recreate the test DB)

Very handy 👍


Can we ignore non-external, non-annotated JS Interop members?

We have some cases where there are JS interop class members that are not external and not annotated with @JS(<name>) themselves. For example, default constructors of @JS() classes which by definition the user does not specify. We consider the name of these null and store them alongside the members which are annotated or external. When the annotation is present on a member but the name is not specified the name gets stored as an empty string. When the member is external but no annotation is provided then the name is defaulted to the Kernel IR member name.
We should consider whether or not it's worth ignoring these non-external, non-annotated members when processing JS Interop classes. There are already some members we ignore: https://github.com/dart-lang/sdk/blob/main/pkg/compiler/lib/src/js_backend/native_data.dart#L197
It's unclear whether or not there's a valid use case for compiling this specific subset of members but if there's not it might make sense to ignore it in that above line and then make the map where we persist those names have a non-null value type:
https://github.com/dart-lang/sdk/blob/main/pkg/compiler/lib/src/js_backend/native_data.dart#L40

I don't believe we have a specification of behavior for synthetic constructors on an interop class. In fact, I'd argue the use of such a constructor should be an error, but I don't know what the current behavior is for these members. If they error on both compilers, I'm fine with ignoring them/doing whatever with them.
tests/lib/js/external_member_static_test.dart contains tests for the disallowed members in a @JS class. Worth noting are:

non-external factories: These should be allowed (ignoring the fact that they don't work in DDC). They're useful for creating variants of constructors.
non-external non-annotated (and annotated) instance fields: This is an unfortunate relic that we've been keeping until we fix all the current usages. These members are equivalent to external instance fields. There's a longstanding bug to handle these usages we have not gotten around to. Since they are explicitly disallowed in static interop, there's a good chance these will just become obsolete.
non-external non-annotated static members: These should be allowed (and like factories, I believe they don't work in DDC).

We also don't do a great job making sure @JS annotations are used only on external members (as they're useless otherwise). Like non-external instance fields, there's no point to these annotations, but we haven't gone through the breaking change process of disallowing them.
I hope this helps. Let me know if you have more questions.

That all makes sense, thanks! Beyond what we allow and disallow, when we're actually processing these interop members it seems like we can ignore a large subset of those fields. We don't need to track their names to include in the compiled JS.
Specifically the answer to this TODO seems to be that we ignore most of the entities where the name would be null. Can we generalize that membersToIgnore to capture this and add any exclusions if we need them?

Specifically the answer to this TODO seems to be that we ignore most of the entities where the name would be null.

Yes, that's reasonable. The only exception I can think of is an unnamed external constructor that is non-synthetic. I don't know what the CFE guarantees there but the analyzer has a null name for those constructors, so we might want to double-check that.

Can we generalize that membersToIgnore to capture this and add any exclusions if we need them?

If the goal is to just make the above map non-nullable, then I believe the only possible way a name could be null for an interop member is the above constructor.


Add no-stack-check manifest option

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @huonw (or someone else) soon.
If any changes to this PR are deemed necessary, please add them as extra commits. This ensures that the reviewer can see what has changed since they last reviewed the code. The way Github handles out-of-date commits, this should also make it reasonably obvious what issues have or haven't been addressed. Large or tricky changes may require several passes of review and changes.
Please see the contribution instructions for more information.

I would very much like to be able to pass the -C no-stack-check to the compiler. I'm writing an emulator in Rust and I would like to:

squeeze as much performance as I can - it's probably not much, but always something, and I don't care too much about safety in this project,
not crash; I implemented an extremely lightweight userspace/green threads library for my emulator in assembly, and while it works fine when called from C it crashes when called from Rust due to stack checks; I don't really feel like unnecessarily complicating my code supporting a feature I don't even need, and it's not even possible to support it without copy-pasting the code from rust's stdlib, since sys is private; I don't want that kind of maintenance burden in case you'll decide in the future to store the stack bounds in a different TLS slot or something,
still use cargo for building my project (otherwise I'll just be forced to make a Makefile wrapping cargo and panic in my build.rs when the user uses cargo directly; ugh...)

Thanks for the PR! Right now, however, Cargo is not currently trying to expose all the options of the compiler at the top-level configuration. There's definitely a real use case here of configuring the codegen options of the entire compile, but we'd want to think about this in a more principled fashion than adding all the options at the top-level.
For now I'd recommend using the RUSTC environment variable or the build.rustc config option to specify your own custom compiler, and you can then customize various flags through that.

I expected as much; it was worth a shot though. (:
Sorry for asking a vaguely related thing, but - if I'd make a pull request exposing record_os_managed_stack_bounds from libstd/sys/common/stack.rs under std::rt would it have a chance to be accepted?
At first I thought that no-stack-check would be enough, but upon further investigation I've discovered that any userspace threading implementation (mine and coroutines-rs included) needs access to record_os_managed_stack_bounds to not crash - while you can use the no-stack-check codegen option for code you compile the standard library crates still have __morestack calls embedded in them. The only thing you can do right now is to copy-paste the stack.rs from Rust into your own crate (e.g. see here) which is not ideal for obvious reasons.

Yeah feel free to send a PR! We're unlikely to stabilize the interface any time soon (we'd prefer to move to stack probes on all platforms), but intermediate changes are fine :)


Feature/n UI 1566 restore wizard state

add restore wizard state functional
update wizard-v2 docs page

@Blakesalot  @lukasholcik please check whether this solution is better. It now stores the state only if the wizard is finished, if not user has to go step by step, just like JR requested. I adjusted the example as well to show that the dynamically added steps are also preserved on wizard restore since it all is being stored in forms.


update lavaplayer to 1.3.64 with has the soundcloud fix 👀

Closed due to conflict. Did it myself. Please use dev next time.

uh I thought I'd use master because you don't want to push the changes from dev to master yet online 👀


Add SquarePrism shape into mesh

This commit add RectangularPrism, a cube-like shape where you can specify the size of each dimensions (x,y,z) independently. It also includes refactoring of the shape Cube to use it in order to reduce duplication of code. This commit also normalize the normal and UV values of the mesh created.

@cart could you rerun the CI jobs for this PR? (Artifact of #380)

How is this different to using a shape::Cube and a NonUniformScale
(Or at least, in theory - it appears that using any NonUniformScale only serves to stop the Translation working presently)

Does #883 make this PR obselete?

Yup! I'll close this one out.
#883 actually uses @TheJasonLessard's code as a base. @e00E did "the right thing" here by working directly off of the commit from this pr (which leaves @TheJasonLessard's git account as an author), but GitHub did "the wrong thing" by not maintaining the correlation between the "commit author" and the github account. I'm annoyed with that ... attribution is important.
@TheJasonLessard if you want "github" attribution and not just "git commit" attribution, feel free to create a pr that "moves" the relevant code to a different spot in the file. That way you get the relevant lines associated with your account (and the new PR creates a paper trail back to this conversation).

@TheJasonLessard do you consent to relicensing this per https://github.com/bevyengine/bevy/issues/2373?

Sure
On Thu, Oct 20, 2022, 5:06 PM Alice Cecile @.***> wrote:

@TheJasonLessard https://github.com/TheJasonLessard do you consent to
relicensing this per #2373
https://github.com/bevyengine/bevy/issues/2373?
—
Reply to this email directly, view it on GitHub
https://github.com/bevyengine/bevy/pull/377#issuecomment-1286151655, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AQQ4JGFRZSRIRP6N7BGJFC3WEGX65ANCNFSM4QNMJBRQ
.
You are receiving this because you were mentioned.Message ID:
@.***>


install tools: ReadItAndKeep, Freyja, Cojac

This PR installs 5 tools: ReadItAndKeep, Freyja:demix, Freyja:aggregate and plot, Cojac:mutbamscan, Cojac:tabmut
https://toolshed.g2.bx.psu.edu/repos/iuc/read_it_and_keep
https://toolshed.g2.bx.psu.edu/repos/iuc/freyja_demix
https://toolshed.g2.bx.psu.edu/repos/iuc/freyja_aggregate_plot
https://toolshed.g2.bx.psu.edu/repos/iuc/cooc_mutbamscan
https://toolshed.g2.bx.psu.edu/repos/iuc/cooc_tabmut
Installation sequence for tool-installers

[x] Test using @galaxybot test this
[x] Inspect CI output for expected changes
[x] Deploy using @galaxybot deploy this if test install was successful
[ ] Merge this PR

@galaxybot test this

@galaxybot deploy this

@galaxybot deploy this


Mouse Cursor still present when Stream from Xbox One

Hello :)
I hava discover your app and thanks for it. I install all the prerequite and when I launch the Xbox App the mouse block on center and I can move with keyboard to the app but when I launch the stream from my Xbox One, the mouse still block on center of screen but nothing with keyboard and mouse.
Furthermore when I'm in stream, when I move the mouse the cursor appear :)
Thanks in advance for answer ;)

I recall this error had occurred before, please give me a sec while I test the released build.

Hello,
Now it work. I don't know why the first time all it's block. I create a new profil and it's OK 👍
The mouse cursor still present on screen but block at center even when i setup the parameter is this normal ?
Other question ^^ . I would like to remap the mouse click button to LB and RB instead of LT and RT. I see in the .ini file the MOUSE parameter but all is on -1 and I don't know how to change it.
Thanks for help :)

@Kiroha This will be addressed in a update tomorrow, i'm sorry i have not addressed this issue sooner.
The update will address: The Cursor Issue and Remapping the Mouse Triggers.

I pulled and built source from master and still see the mouse cursor over the window (being bound to center).  Is this code still pending somewhere?

I've noticed this rarely.  I don't think it's fixed.


Moving AST docs to build step, adding path, file and ignores config options

AST docs now included as a post-compile step
Optional config options

path: path to output folder (uses cache by default)
file: name of ast file
ignores: simple string ignore.

Notes:

I tinkered with making ignores a RegExp in the config but it seems the buidler is parsing it as an object but it's losing (cc @alcuadrado)
I had issues testing no configuration - something to do with how you useEnvironment in the tests is preventing the buidler lazy loading of config to work properly I suspect.

Nice, really like having it automatically generate on post-compile


Executor returned by llm_chain_openai::chatgpt::Executor::for_client() does not implement llm_chain::traits::Executor ([E0277])

So, in my use case, I need an openai executor that uses an organization-id read from an environment variable, looking at the source code for the llm_chain_openai crate (as the documentation is kinda lacking), it would seem that the llm_chain_openai::chatgpt::Executor::for_client() would let me do that by letting me specify the async_openai client to use.
However, while this works
use async_openai::Client;
use lazy_static::lazy_static;
use llm_chain_openai::chatgpt::{Executor, Model, PerInvocation};

lazy_static! {
    pub static ref OPENAI_EXECUTOR: Executor = Executor::for_client(
        {
            let org_id = std::env::var("OPENAI_ORG_ID").unwrap_or_else(|_| "".to_string());
            if org_id.is_empty() {
                Client::new()
            } else {
                Client::new().with_org_id(org_id)
            }
        },
        Some(PerInvocation::new().for_model(Model::ChatGPT3_5Turbo)),
    );
}

it doesn't give me an Executor that implements the llm_chain::traits::Executor trait, meaning I can't actually use it for anything useful
is this intended behavior? if so, why? and if not, what can I/we do about it?
here is the specific error: rustc: the trait bound `OPENAI_EXECUTOR: llm_chain::traits::Executor` is not satisfied the trait `llm_chain::traits::Executor` is implemented for `llm_chain_openai::chatgpt::Executor` [E0277]

in terms of potential solutions, I think the easiest is to just add organization IDs as a supported PerExecutor option, adding async_openai clients as a supported option, adding a function that modifies the client of an existing executor (that way someone could use the new_with_options function to create a functional executor, then modify the client to fit their needs), or modify the for_client function to return an executor that implements the Executor trait
I could probably make a PR for any one of those solutions, the real question is which one, if any, is most in line with the goals of this project

more in depth review of the options:

the most ergonomic approach is changing llm_chain_openai::chatgpt::options::PerExecutor to be something like this:

pub struct PerExecutor {
    pub api_key: Option<String>,
    pub organization_id: Option<String>,
}

but that would be a breaking change,

we could also change llm_chain_openai::chatgpt::Executor::for_client
from

pub fn for_client(
    client: async_openai::Client,
    per_invocation_options: Option<PerInvocation>,
) -> Self {
    let client = Arc::new(client);
    Self {
        client,
        per_invocation_options,
    }
}

to
pub fn for_client(
    mut self,
    client: async_openai::Client,
) -> Self {
    self.client = Arc::new(client);
    self
}

but this is also technically a breaking change because it changes the functionality of a (admittedly broken) public function

last option I'm considering is to add a new function, with_client to llm_chain_openai::chatgpt::Executor

pub fn with_client(
    mut self,
    client: async_openai::Client,
) -> Self {
    self.client = Arc::new(client);
    self
}

Summary:
realistically, unless you're planning a major release soon, option 1 is too destructive to existing functionality, so that leaves 2 and 3,
IF the for_client function is currently operating how it is intended, then I should do option 3, otherwise option 2 because this is a bug.
So, final question: is the for_client function currently operating as is intended?

option 4, probably the best:
change the for_client function to something like this:
    pub fn for_client(
        client: async_openai::Client,
        per_invocation_options: Option<PerInvocation>,
    ) -> Result<Self, ExecutorCreationError> {
        use llm_chain::traits::Executor as _;
        let mut exec = Self::new_with_options(None, per_invocation_options)?;
        exec.client = Arc::new(client);
        Ok(exec)
    }

changes the resume type, but that's about it

Option 5 is to make the client member variable of llm_chain_openai::chatgpt::Executor public, but that isn't a great solution imo


Create Sprite for Raw Meat #168

If you find this PR acceptable, then I will submit another with a recolored version for cooked meat as well.

Hi @beatyt! Very nice sprite, thank you. It would be great if you could do the cooked meat, yes. :smiley:


How to get NFT from TON masters in mainnet?

I followed the 01-wallet tutorial but did everything in mainnet.
I send 0.05 TON to EQChHpu8-rFBQyVCXJtT1aTwODTBc1dFUAEatbYy11ZLcBST address.
Will I have an NFT minted to my address? (EQC5Bjo-w76nmo-yc_QhV1JkWOAO9et65s5Oh6zUMIgK41Zy)

hello @zjor,
you sent it to the wrong address. You should send it to: EQA4V9tF4lY2S_J-sEQR7aUj9IwW-Ou2vJQlCn--2DLOLR5e
source: (Step 9) https://ton-community.github.io/tutorials/01-wallet/


Excluding the property drawer from template expansion

My org-capture setting automatically adds an ID to each captured entry. When I create a new snippet using yankpad-capture-snippet command, an ID is assigned to the snippet as well, and whenever I expand the snippet using yankpad-insert, a property drawer containing the ID of the snippet is prepended to the result. I don't want this to happen.
Could you add support for preventing the property drawer of a template from being inserted as part of a result?

The issue has been fixed, and here is how it went wrong:
After following your instruction, I found that function included in org-capture-finalize-hook, but this condition test seems to have failed. org-capture-last-stored-marker contains an absolute path, while my yankpad-file is abbreviated, i.e. beginning with ~, so the test failed. After setting yankpad-file in an absolute path using expand-file-name, a created snippet no longer has an ID. Thank you for your response.

I've commited a small change which hopefully fix the cases of relative vs absolute paths. Thanks for notifying me about this!


site.github  certificate verify failed.

[x] This is a question about using the theme.
[ ] I have updated all gems with bundle update.
[x] I have tested locally with bundle exec jekyll build.

Environment informations

Minimal Mistakes version:3.3.6
github-pages : 84
Operating system:Win 10

$ bundle exec jekyll serve
Configuration file: C:/Users/Bruce/Documents/Github/blog/_config.yml
            Source: C:/Users/Bruce/Documents/Github/blog
       Destination: C:/Users/Bruce/Documents/Github/blog/_site
 Incremental build: disabled. Enable with --incremental
      Generating...
  Liquid Exception: SSL_connect returned=1 errno=0 state=SSLv3 read server certificate B: certificate verify failed in _includes/head.html, included in _layouts/default.html
jekyll 3.1.6 | Error:  SSL_connect returned=1 errno=0 state=SSLv3 read server certificate B: certificate verify failed

Sorry bothering you, I am a noob in ruby or jekyll. I want to use the latest version ,but I encountered the new problem. I've tried the ways I known and even searched the stackoverflow. I appreciate it if you can give me some instructions. Thanks in advance!

Perhaps this thread will be helpful. I don't think the error is theme related.
https://github.com/jekyll/jekyll/issues/3985

Thanks a lot! :blush:

Also got this too on Windows 10. The other thread doesn't help.

@ceefour Looks to be related to the jekyll-gist gem. Please file an issue upstream as I don't control that gem.

https://github.com/jekyll/jekyll-gist/issues/30

@mmistakes as I mentioned before Tried commenting jekyll-gist in _config.yml but didn't work.
So it's probably a similar root cause , that still happens without jekyll-gist.

@ceefour That said the issue isn't theme related. It's something with either your Ruby environment, the Jekyll gem, one of its dependencies, or a combination of these. You'll have to look for help there.
Only other thing I can think of is you're using github-pages gem in Gemfile, which autoloads the Jekyll Gist gem regardless if you comment it out or not. Try replacing with gem "Jekyll", then bundle update, then bundle exec jekyll serve.
Other than that I can't really offer any other advice or help. You're better off hoping someone from the core Jekyll team or one of its plugin maintainers comes up with a fix. Windows and Jekyll are known to have issues. Well rather Ruby and Windows,

@ceefour  Maybe you can try delete the https://github.com/mmistakes/minimal-mistakes/blob/master/_includes/seo.html#L7 .  It worked fine on my site https://github.com/BZRLC/bzrlc.github.io/blob/master/_includes/seo.html#L7 . Just a advice, don't know if it work on your site.

 That line is the culprit:
{% assign seo_url = seo_url | default: site.github.url %}

Curiously, I don't see any mention of github.url in _config.yml. Probably a bug?

Not a bug @ceefour, that's related to the github-metadata plugin. I was going to suggest it but seemed more tied to jekyll-gist so I didn't. When you push your site to GitHub it uses that plugin to automatically set site.github.url for you.
You'll get all kinds of other Liquid errors if you don't properly configure it to. That's why I suggested you switch from using the GitHub Pages gem to vanilla gem "jekyll" in the Gemfile as it won't use the GitHub Metadata plugin.
Usually all you need to do is add repository: username/repo-name to your _config.yml and the error will go away.
Give that plugin docs a read over. It tells you how to set things up so it properly authenticates. If that doesn't work you might have encountered a bug and if so file an issue with them.

Wait, so the best practice is to use gem "jekyll" instead of github-pages gem ? This seems contradictive with the minimal-mistakes defaults and what GitHub itself suggests.
BTW I did have repository: username/repo-name in my _config.yml from the start.

Commenting github-pages gem and using just jekyll gave me:
C:\Users\ceefour\git\matagaruda.github.io>bundle exec jekyll serve
Configuration file: C:/Users/ceefour/git/matagaruda.github.io/_config.yml
Configuration file: C:/Users/ceefour/git/matagaruda.github.io/_config.yml
  Dependency Error: Yikes! It looks like you don't have jekyll-paginate or one of its dependencies installed. In order to use Jekyll as currently configured, you'll need to install this gem. The full error message from Ruby is: 'cannot load such file -- jekyll-paginate' If you run into trouble, you can find helpful resources at http://jekyllrb.com/help/!
jekyll 3.3.0 | Error:  jekyll-paginate

It depends on where you're hosting. If you hosting with GitHub Pages then you'll want to use gem "github-pages" as it uses versions of Jekyll and its dependencies that mirror how GH Pages..
If you're self hosting then use gem "jekyll" as it gives you the more recent version.
I don't know all the intricacies of how github-metadata works, but I know i've come across all kinds of similar authentication errors if things are configured correctly on Windows. Like I said usually setting repository is enough, but there are some other options you may need to do.
Reach out to the maintainers of that repo for further help. Entirely possible you hit a legit bug. I can't say either way as I have no involvement with that gem/plugin.

@ceefour If you switch to gem "jekyll" you'll need to add all the gems/plugins to your Gemfile. github-pages has jekyll-paginate, jekyll-sitemap, jemoji, etc. as dependencies so they come along for the ride. Otherwise you'll get those error (which tells you exactly how to solve the problem).
This is all documented.

The GitHub Pages gem installs additional dependencies that may need to be added to your Gemfile if you decide to remove the gem "github-pages".

Your Gemfile should look like this when using vanilla Jekyll:
source "https://rubygems.org"

gem "jekyll"
gem "wdm", "~> 0.1.0" if Gem.win_platform?

# If you have any plugins, put them here!
group :jekyll_plugins do
  gem "jekyll-paginate"
  gem "jekyll-sitemap"
  gem "jekyll-gist"
  gem "jekyll-feed"
  gem " jemoji"
end

Thanks @mmistakes , that works. And I see no more SSL errors even with the original seo.html .....


use uint16 for indices

A single mesh may not have more than max GLushort vertices so this
can already be restricted in mesh building.
(otherwise the vertex index will overflow VboMesh::compile)

:+1:


[SYSTEMML]Fix DMLAppMasterStatusReporter.java thread cancel bug.

Summary
This PR created for fixing multiple thread bug in DMLAppMasterStatusReporter.java.
1.Use a volatile variable for flag.
2.Invoke interrupt method to avoid block method such as Thread.sleep invocation.

thanks @GrapeBaBa - since you explicitly interrupt the thread, could you please also add the related  handling of InterruptedException (e.g., something like LOG.warn("Status reporter interrupted with stop="+_stop)

@mboehm7 Done.

LGTM

Thank you @GrapeBaBa . I will merge.


docs: Documentation for creating UDFs using Decorators.

@IshSiva The documentations looks very good. Left a few comments.

@IshSiva Is this ready to merge?

Yes, this can be merged.


Docs: Fix broken link to Aikar's Flags

Replace broken https://mcflags.emc.gs with
https://aikar.co/2018/07/02/tuning-the-jvm-g1gc-garbage-collector-flags-for-minecraft/
which presumably was the original where that came from.

The new link isn't working for me,

FYI it eventually worked for me just now, but it's really slow to load; it's likely some slow dynamic site generator, not static content.


Unsolicitedssorelaystate

During an unsolicited SAML SSO the supplied target parameter got lost, since it was only saved in the request not in the message context. With this patch it's preserved and the relay state is supplied to the service provider.
Testing just requires a targer paramter in an unsolicited SAML request, the debug output will show an empty relay state without the patch and the supplied target parameter with the patch in the SAML response enconding.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Große Lengerich seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

This is great, thank you. Please submit another PR targetted at the master branch and we should be able to merge both quickly.


Fix minor typo in readme

Fix minor typo in var definition in readme.
This is only present in the python readme, all is fine on the rust side.

@FullMetalMeowchemist thanks for catching this!


Duplicate storage for the same component

I've been running into the problem that some components went "missing". After investigating I noticed at some point a second storage for the same type is being created.
I used following code for outputting info (trying with different namespaces and without typedef)
      TRACE << " E        " << (Uint32)e.entity();
      TRACE << " R        " << e.registry();
      TRACE << " S        " << &e.registry()->storage<::Children>();
      TRACE << " S (size) " << e.registry()->storage<::Children>().size();
      TRACE << " T1       " << entt::type_id<::Children>().hash();
      TRACE << " T2       " << entt::type_id<std::vector<entt::entity>>().hash();

I was able to get two different outputs right after creating the entity
 E        418
 R        0x55a490312658
 S        0x55a490341680
 S (size) 3
 T1<PHONE_NUMBER>
 T2<PHONE_NUMBER>

 E        418
 R        0x55a490312658
 S        0x55a4903d5e90 // different storage
 S (size) 0
 T1<PHONE_NUMBER>
 T2<PHONE_NUMBER>

There now seems be be a new storage for the type Children.
When printing out all storages with
for (auto [n, s] : e.registry()->storage())
{
    TRACE << s.type().name() << " : " << &s << (entt::type_id<::Children>().hash() == n ? " <<<<" : "");
}

I get following output (:exclamation:  this is from a different run so the pointers are different from above):
std::vector<entt::entity> : 0x55e27bf4cbe0 <<<<
[...]
std::vector<entt::entity> : 0x55e27bd9eb30

Things to note

I started to notice it only in release mode
This might have started suddenly without any code change regarding this entity (but I'm not sure)
Happens in code that is part of the same executable (not between shared libs or anything like that)
I've noticed this even in the same compilation unit!
Hard to reproduce. Slight changes in code affect this behaviour.
compiler is gcc 13.2.1 20230801
entt version 3.12.2

I suspect it has something to do with hashes being created differently, since in the output above only one of the strorages hash matches. But I'm a bit at a loss here on how to proceed :sweat_smile:

It would be the first case of conflict in many years but anything is possible. How is Children defined?
You can try to give it a user defined hash that is different from the auto-generated one.
If you're right on the root cause, this would solve the problem at once.

How is Children defined?

using Children = std::vector<entt::entity>;

You can try to give it a user defined hash that is different from the auto-generated one.

You mean like this?
namespace entt
{
template<>
struct type_hash<::Children, void> final
{

  [[nodiscard]] static constexpr id_type value() noexcept
  {
    return ~0;
  }

  [[nodiscard]] constexpr operator id_type() const noexcept
  {
    return value();
  }
};
} // namespace entt

This solve the issue for me. Hash is forced to<PHONE_NUMBER> and I only get one storage
 E        418
 R        0x55c8934b98e8
 S        0x55c893445970
 S (size) 3
 T1<PHONE_NUMBER>
 T2<PHONE_NUMBER>
[...]
std::vector<entt::entity> : 0x55c893445970 <<<<
[...]

Wait a moment.
std::vector<entt::entity> and Children ARE the same type. It's not a conflict.
using declarations don't introduce new types. They define aliases and nothing more.

A repro would help but 🤷♂ no idea what you're doing that can cause this, I'm sorry.
I'm glad the custom hash fixed it though. I'm afraid I can't do much more for this issue.

This looks like a nasty bug somewhere
When I print the storage at my first usage I get:
TRACE << "First usage : " << i.registry()->storage<Children>().type().hash() << " "
          << &i.registry()->storage<Children>();
Output:
First usage :<PHONE_NUMBER> 0x560da62d7fc0

When I then later print all storages I get:
(name, address, hash)
std::vector<entt::entity> : 0x560da6526a10<PHONE_NUMBER>
std::vector<entt::entity> : 0x560da62d7fc0<PHONE_NUMBER>

It looks like the hash of the initial storage at 0x560da62d7fc0 changed.
The hash seems to be computed as<PHONE_NUMBER> consistently throughout the program. So I'm really unsure as to why the custom hash helped at all.
I'll try to get minimal repro but I don't have a lot of hope :laughing:

Two new things.
Defining ENTT_STANDARD_CPP also helps.
I added debug printing to the entt code (basic_registry::assure).
  template <typename Type>
  [[nodiscard]] auto &
  assure([[maybe_unused]] const id_type id = type_hash<Type>::value()) {
    if constexpr (std::is_same_v<Type, entity_type>) {
      return entities;
    } else {
      static_assert(std::is_same_v<Type, std::decay_t<Type>>,
                    "Non-decayed types not allowed");
      auto &cpool = pools[id];

      if (!cpool) {
        using storage_type = storage_for_type<Type>;
        using alloc_type = typename storage_type::allocator_type;

        auto op = type_id<Type>();
        printf("Type=%s \ntype_id<Type>().hash()=%u \nid=%u "
               "\ntype_hash<Type>::value()=%u\n",
               op.name().data(), op.hash(), id, type_hash<Type>::value());

        if constexpr (std::is_same_v<Type, void> &&
                      !std::is_constructible_v<alloc_type, allocator_type>) {

And for std::vector<entt::entity> it prints
Type=std::vector<entt::entity>]
type_id<Type>().hash()=2301207559
id=2361607097
type_hash<Type>::value()=2301207559

The id that is passed as default value is inconsistent with those computed in the method.
While for other types it"s consistent
Type=Light]
type_id<Type>().hash()=2329082063
id=2329082063
type_hash<Type>::value()=2329082063

Any ideas? Is this a compiler issue?

To be honest, it really really really smells of UB. As if you were a moved-from registry or a dangling reference. Sort of.
Have you already considered this and investigated to exclude it? Without a repro, I'm just trying to think aloud but it's very difficult to help. I'm sorry.
Is this a public project btw? Maybe I can try to reproduce it locally.

To be honest, it really really really smells of UB. As if you were a moved-from registry or a dangling reference. Sort of. Have you already considered this and investigated to exclude it?

Yes I'm definitely using the same registry (you can see the same pointers for the registry in the first post). I'm also adding thousands of components in the same function, but only std::vector<entt::entity>  causes issues. Running the program with address sanitizer or undefined behavior sanitizer shows no issues as well.
Also I don't see how UB would cause id = type_hash<Type>::value() (function arg) to be different from type_hash<Type>::value() (function body) in the same method call. This should be constexpr right? Imo this has to be an issue with the compiler...

Without a repro, I'm just trying to think aloud but it's very difficult to help. I'm sorry.

Sorry I wish I had one :sob: But all I'm doing is
reg.emplace<std::vector<entt::entity>>(e)

with different random result.

Is this a public project btw? Maybe I can try to reproduce it locally.

No sorry,
I realize this is difficult to remote diagnose. Still I really appreciate your help :+1:

Btw if I change Children to
using Children = llvm::SmallVector<entt::entity, 4>;

I also have no issues ... I guess the hasher does not like std::vector<entt::entity> :laughing:

Maybe the using declaration gives it a headache? I guess you tried to reproduce it like this, right?
I can't really explain why it should be a problem but I don't have access to the real code either, so 🤷♂ hard to say.

I personally would not use using to specify a component type. You could use
class Children : public std::vector<entt::entity> {} or similar to create a new type.

I personally would not use using to specify a component type. You could use class Children : public std::vector<entt::entity> {} or similar to create a new type.

I don't want to create a new type. I was just using a typedef for convenience. The issue persists regardless of using the typedef or std::vector<entt::entity> directly.

I have a repro, but I almost certain you won't get the same result :disappointed:
https://github.com/hfn92/entt-repro
This is my output. It shows two storages with the same type but different hash. (The hash of the type in storage is different from the has used as map key)
2361607097 : std::vector<entt::entity, std::allocator<entt::entity> >
2301207559 : std::vector<entt::entity, std::allocator<entt::entity> >
Error detected!!!<PHONE_NUMBER> vs<PHONE_NUMBER>

Minor  unrelated changes such as this already fix this issue ...
--- a/Prototypes.h
+++ b/Prototypes.h
@@ -11,6 +11,6 @@ class Prototypes
 public:
   static void Create(entt::registry& reg, std::string_view i)
   {
-    entt::handle h{ reg, reg.create() };
+    // entt::handle h{ reg, reg.create() };
   }
 };

Ehy, sorry for the late reply, I've been busy last week. Gonna try it soon. 👍

I can't reproduce your result with this code. To be honest, I don't even see where Prototypes is used and therefore I don't see how changing it would change anything. As far as I can tell, any decent compiler should ignore and discard this class since it's irrelevant. 🤔
Are you sure you pushed the right code?

I can't reproduce your result with this code.

Yeah I didn't think it would work for you :sweat_smile:

I don't even see where Prototypes is used and therefore I don't see how changing it would change anything. As far as I can tell, any decent compiler should ignore and discard this class since it's irrelevant. 🤔
Are you sure you pushed the right code?

The class is not used. The code is utter nonsense since I just removed code from my engine just until before the error would stop occurring.
I still think this I a compiler issues unless entt invokes UB during the constexpr calculation of the hash, which I don't believe. So unless you use gcc 13.2.1 20230801 in RelWithDebInfo you wont get the same result.

I have confirmed that the compiler uses two different strings to compute the hash during compile time.
std::vector<entt::entity, std::allocator<entt::entity> >  // "correct"
std::vector<entt::entity>

With hashes
2361607097   // "correct"
2301207559

I have no idea what to do with that information :rofl:

I mean, have you tried with a different compiler too? Like, dunno, clang or even gcc 14?
I'm not 100% convinced that your error is due to a compiler issue but I can't help much if I can't reproduce it.

Hi @hfn92 , @skypjack
I want to chime in here, because I'm getting some very similar behaviour as well. I have a project that was running perfectly using entt 3.6, but recently updated it to 3.12 and I noticed one of my unit tests was failing. But it would only fail when running in the Continuious Integration system we are running.
I too tracked problem down to having two storage locations for the same type. When creating the object it would create one storage, then when looking it up, it would search in another storage location.
My component is also a std::vector<some_type> so maybe it has to do with that.
I got the same error using gcc8 and gcc11. But ONLY when they are running inside a docker container. When I compile the same unit tests on my main system (which use gcc11), all the tests pass and I have no problems.
I tried the suggested solution of implementing a custom type_hash function. This worked! Thank you!
I also tried not using std::vector as a component, but instead defined a wrapper struct around the vector like so:
template<typename T>
struct VectorComponent
{
    std::vector<T> _vector;
}

This also seemed to work.  So the problem may be due to how std::vectors are being type_hashed. It may be a compiler bug. I will try to compile using clang and see if I get the same issue

To be clear, the type hash relies on a string, which is implementation defined behavoir (/ compiler specific extension?), and as such is just best effort. But it rarely breaks. eg: I had to define the hashes myself, because I went across the compiler boundary.
@GavinNL if you just want it to work and are ok with extra types (like the VectorComponent<T>) you can just inherit from the original type and you don't need to adjust code (like with a wrapper).
eg class Children : public std::vector<entt::entity> {} or similar.

Thanks for the info @Green-Sky .  I was going to inherit from the vector, but I heard it's bad practice to inherit from std containers. I was only using that component in one place, so it was pretty easy to fix.
I just did some testing with the hashs here's what happened on my tests.

This problem only occurred on GCC8 and GCC11 in Ubuntu Docker containers. It worked fine on my main Linux Mint host
It works fine on MSVC and Clang 14
implementing the type_hash<std::vector<MyType> > fixed the problem on GCC but MSVC started failing unit tests. Didn't look into this too much, I just ended up wrapping the type_hash implementation in #if blocks

The type_hash is used to basically get a number so that it can be used as a key for a map, correct?  Is there a reason std::type_index wasn't used instead? constexpr reasons, maybe?

This problem only occurred on GCC8 and GCC11 in Ubuntu Docker containers

This is interesting though. We have these compilers on the CI, so in theory we can reproduce the issue with a minimal repro?

Hi @skypjack , I will try to get you a minimal code that reproduces this error

Not sure if this helps to fix it but it would be great if we had something that fails on the CI because I cannot reproduce the error locally and I don't have gcc11 up and running locally, I'm sorry. 🙏

I'll do my best.  The main problem was using std::vector<SomeStruct> as a component on an entity.  When creating the component, the has that was being calculated (and therefore the storage) for the component was different from when looking up the component using registry.get< std::vector<SomeComponent> >(e)
It was a really odd issue because it wasn't very inconsistent in when it would fail. Only my unit tests were failing, my actual application was running fine. I had the registry.emplace< std::vector<SomeComponent> >(e) in a header file, while registry.get< std::vector<SomeComponent> >(e) was in a source file
Its likely the "string" that's being generated by gcc that is used for computing the hash for std::vector was not being generated the same way in different parts of the code.

I just tested @hfn92 example code on GCC11 and I'm getting the same errors he is.
I also tested in Clang 14 debug/release, they both seem to be okay
I can confirm that the error is happening on GCC Release builds, it does not happen on Debug. That would explain why my tests were failing in the CI. Our CI was building in release mode, and I was building in debug mode locally.
@skypjack ,Is your CI building in debug mode or release mode?

Here's @hfn92 test, with some cmake and conan files
entt_test.zip
My system is: Linux Mint 21/Ubuntu 22.04, GCC11.4 + release build

Thank you a lot. That's really funny btw. I wonder what changes in release mode on the gcc side.

I just did some additional testing. In the code I gave you, there are two calls to emplace<Children> one in main and one in Create().  If you add the following right after each call to emplace
    std::cout << "Calling  emplace<Children>()" << std::endl;
    constexpr auto stripped = entt::internal::stripped_type_name<Children>();
    constexpr auto value = entt::hashed_string::value(stripped.data(), stripped.size());
    std::cout << "   Stripped name :" << stripped << std::endl;
    std::cout << "   hashed   name :" << value << std::endl;

This is the output I got when running with GCC11.4+Release+Ubuntu22.04
In Create
Calling  emplace<Children>()
   Stripped name :std::vector<entt::entity, std::allocator<entt::entity> >
   hashed   name :2361607097
2361607097 : std::vector<entt::entity>
In Main
Calling  emplace<Children>()
   Stripped name :std::vector<entt::entity>
   hashed   name :2301207559
2361607097 : std::vector<entt::entity>
2301207559 : std::vector<entt::entity>
Error detected!!!<PHONE_NUMBER> vs<PHONE_NUMBER>
Ok!<PHONE_NUMBER> ==<PHONE_NUMBER>

It looks like the stripped_name  is different on each of the different calls resulting in a different hash being calculated

This seems to be happening when the two emplace< > are in different compilation units.  Moving the Create() function to main.cpp causes it to work again and results in the following:

In Create
Calling  emplace<Children>()
   Stripped name :std::vector<entt::entity>
   hashed   name :2301207559
2301207559 : std::vector<entt::entity>
In Main
Calling  emplace<Children>()
   Stripped name :std::vector<entt::entity>
   hashed   name :2301207559
2301207559 : std::vector<entt::entity>
Ok!<PHONE_NUMBER> ==<PHONE_NUMBER>

Could it be that at some point in the code the compiler is unable to see the whole type declaration, making it skip the allocator... part?

Some more poking around...
In Prototype.h there is a class with a static member function. This header file is included in both cpp files, but its not being used.
If I comment out the static member function, I get the following, which seems like it's working. Note that the name does not contain the allocator part.
In Create
Calling  emplace<Children>()
   Stripped name :std::vector<entt::entity>
   hashed   name :2301207559
2301207559 : std::vector<entt::entity>
In Main
Calling  emplace<Children>()
   Stripped name :std::vector<entt::entity>
   hashed   name :2301207559
2301207559 : std::vector<entt::entity>
Ok!<PHONE_NUMBER> ==<PHONE_NUMBER>

I don't know which one should be considered the correct name, std::vector<entt::entity> or std::vector<entt::entity, std::allocator<entt::entity>> .  Clang's name resolves to std::vector<entt::entity>
In release mode, I think if it can detect a function isn't being used, it doesn't include it in the compilation, but why that would change the name of the compile-time string, I don't know.
It might be worth asking a GCC developer. I don't know if this is a bug or not

I find this one fascinating but I don't see what EnTT can do here. It really looks like an issue on the compiler side.
Fortunately, the library provides a tool to get around the problem and you confirmed that it works like a charm.
I'm open to suggestions though. If you think we can do more, please drop a few lines to describe your idea.
Otherwise, I think I'll close the issue as won't fix sooner or later because I don't see what can I do for that.

I'm open to suggestions though. If you think we can do more, please drop a few lines to describe your idea.

When defining ENTT_STANDARD_CPP entt switches to another method of computing the hash, which I think solves the issues. But we seem to loose the ability the use .name() since it just returns an empty string. This makes debugging harder and breaks my ImGui ui for displaying compoments :sweat_smile:
Would it be reasonable to provide a way to switch to the counting method for hashes while still retaining the name? In this case I don't mind that the string might not be "correct", since I just use it for display purposes

Well, you can specialize type_name and make it return the pretty function name. EnTT is fully configurable in this sense.
Also, keep in mind that the 'ENTT_STANDARD_CPP` way doesn't work across boundaries because it uses local static variables. The standard doesn't rule on them and different compilers treat them differently.
If you don't care about this, then yes, this is another viable solution. 👍

We have a solution, but I think we should at least have this issue searchable in case other people have have come across the same issue.

Fair enough but how do we make the issue searchable? I mean, it's so already and it contains lot of words that match with the right keywords. How can we improve this eventually?

Instead of setting this issue as closed/wont fix, is there a way to mark it as "Known Issue" ?
If not, maybe we can put it in the documentation under a section called "Known Issues"? We can explain the error along with the solution.

I don't think of a way to do that in GH but EnTT has a FAQ doc that fits the purpose maybe?

Sure, that will work :)


Doc enhance for gpt

Added several snippets of example code for how to use PropertySet, not just for fun but because this was necessary for GPT to make working code for this class.

Added full single-node SIR example to doc -- can be improved still, but it really helps LASER-GPT make working code faster.

I'm still testing how well the pdf generated from these changes informs LASER-GPT in terms of building working code with vital dynamics.

Just realized my reference example doesn't use the new LaserFrame constructor properly...


iframe style not apply in windows

Hi, I tried to apply styles to iframe using the apc.iframe.style setting but the styles where not applied.
I use windows pc.
Thank you.

I can't seem to get this to work either. I've tried both inline and with a css file.


Kontena Lens 1.3.0

What would you like to be added:
Kontena Lens 1.3.0
Why is this needed:
Better performance, new features.

Fixed in #842


License?

What is the OSS license?  Will the project include a LICENSE.txt file?

I was asking me the same question. Any updates regarding the licence ?


Participation in Hacktoberfest

@0xdabbad00 is this repository participating in Hacktoberfest?

No, unfortunately I'm not doing a very good job of spending the time to properly maintain this project, and thus I would not be able to spend the time to help new contributors.


tabBarView's title can not be displayed  properly in Swift Example

hi, MerrickSapsford,
I downloaded and tried your the Example. Objective-C version worked great, but Swift version can not work finely.
Following your ReadME, I had run the Example, result as follow: (Xcode 7.3.1)
Can you help me ? Thanks in advance !

Can not upload the pic

Fixed as of 0.3.3

It works, thanks


Nested DropZone

Feature request type
Performance improvement
Component name
MudDropZone
Is your feature request related to a problem?
Nested drop zones are not working due to event propagation that can be easily fixed.
Describe the solution you'd like
if you add few html properties to drop zone it can be used as nested drop zone :
@ondragover:preventDefault       @ondragover="()=> { }"       @ondragenter:preventDefault       @ondragenter="()=> { }"       @ondrop="()=>{ }"       @ondrop:preventDefault       ondragstart="event.dataTransfer.setData('text', event.target.id);"       @ondrop:stopPropagation       @ondragenter:stopPropagation      @ondragend:stopPropagation      @ondragover:stopPropagation      @ondragleave:stopPropagation      @ondragstart:stopPropagation
Have you seen this feature anywhere else?
there is few drop zone components for blazor which  support nested drag and drop :

Plk.Blazor.DragDrop
WAELKASSEM / DragDropBlazor

Describe alternatives you've considered
No response
Pull Request

[X] I would like to do a Pull Request

Code of Conduct

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

Any ways to patch it until your PR goes through ?

Any ways to patch it until your PR goes through ?

I created my own component using MudBlazor original one. That is the quickest way.

I raised this issue ages ago. Glad to see a PR has been submitted. @ZeidLab is your new component available on Github?

@jimitndiaye Unfortunately I have customized and personalized my own component and it is nothing like MudBlazor DragAndDrop know. But you can always make your own component using  PR

@jimitndiaye you can easily copy and past DragAndDrop component folder to your project and rename it. it will work for you.

Is there any news on this ?


allreduce_robust.cc: Allow num_global_replica to be 0

In some cases, users may not want to have any global replica of
the data being broadcasted/all-reduced. In such cases, set the
result_buffer_round to -1 as a flag that this is not necessary
and check for it.

please do a rebase so that travis can restarts checks, there a few lint errors that you might want to fix

@tqchen Do you mean the guard for the header files ?
I can fix them if you like (I thought you might have prefered it in the way it is there right now. Hence left it as is)

@tqchen Wanted to confirm here - setting the num_global_replica to 0 shouldn't break anything else in the rest of the library right ? i.e. That is a valid config if I want no replicas of my data to be stored ?

I think it is valid, except that there is no fault tolerance now

But there is still fault tolerance inside a single broadcast call and a single all reduce call. (As there is a loop inside Which checks and reinitiates the broadcast/all reduce) Which is what I want.

I see, that is correct


Add version to api end point

The api end point should be:
/v1/api-endpoint
instead of just: /api-endpoint
It's better if later have to support multiple versions.

#50 implement this change


scylla_repository: expect -SNAPSHOT in scylla-manager reloc URL

it turns out the latest scylla-manager's reloc tarball's URL contains contains "-SNAPSHOT" instead of "-dev-" in it. so let's update the test to reflect the fact.
the name should match with the name specified by
scylla-manager/dist/.goreleaser.yaml 's name_template.

Lgtm
Maybe we should check for both ?

i'd prefer not. IMHO, the uncertainty in test is a source of the confusion. that's why i also checked scylla-manager's CI.


Error on every HA start: 'DeviceTools' object has no attribute '_device_modifications'

Hey,
I get this error every time HomeAssistant starts:
Traceback (most recent call last):
  File "/config/custom_components/device_tools/device_tools.py", line 84, in async_run
    self.async_get_entries()
  File "/config/custom_components/device_tools/device_tools.py", line 72, in async_get_entries
    if self._device_modifications == device_modifications:
       ^^^^^^^^^^^^^^^^^^^^^^^^^^
AttributeError: 'DeviceTools' object has no attribute '_device_modifications'

I can't replicate this on the newest version. Did you check HACS if there are any available updates for this integration?

I updated from 1.8.2 to 1.8.3 and the message is gone.


Doesn't work in the browser when not using require js

@gr2m thanks for this great tool ! however, it does not work in the browser... 😢
i did `bower install moment-parse-format --save'
and the console would show require is not defined.
Simple way to fix it is to put the whole thing in an iife

can you use npm to install instead of bower?
After installing with npm, you can add the add it from node_modules/moment-parseformat/dist/moment-parseformat.js.
Or if you want to install it via bower, try bower install https://npmcdn.com/moment-parseformat/dist/moment-parseformat.js

Unfortunately, i can't use npm, we use gulp-wiredep and it can not see that the bower version is installed, and yes i have run bower with --save
Error: Error: moment-parseformat is not installed. Try running `bower install`.

I’m sorry I don’t know how to work around that, I don’t know much about bower, do you have an idea? Can we somehow point bower install moment-parseformat to install from bower install https://npmcdn.com/moment-parseformat/dist/moment-parseformat.js?

you have to run npm buildwhen you update the package, and then add a bower.json' that points to dist/moment-parseformat.js`

and there are bower postinstall hooks that can automate that

Is it enough to have the bower.json and dist/moment-parseformat.js on npm only though? I do not want to commit the dist folder on github

i don't know, but if you look at other packages (momentjs included) they all commit the dist to github. you can easily automate it with gulp

yes, I know, they do it only because of bower, but it comes with overhead that I cannot add in order to be able to maintain this package, I’m sincerely sorry

alright, thanks anyways...

in your bower.json, can you set a url instead of a version? Something like this
{
  "name": "test",
  "dependencies": {
    "moment-parseformat": "https://npmcdn.com/moment-parseformat/dist/moment-parseformat.js"
  }
}

You can also set a specific version like this<EMAIL_ADDRESS>

no i get :
moment-parseformat is not installed. Try running `bower install`.

so this works for me with bower 1.5.2

create a bower.json with the content from by comment above
run bower install
copy the content of bower_components/moment-parseformat/index.js and paste it directly into the console on momentjs.com. moment.parseFormat is working

yes, but i have a gulp script running that injects automatically in the index.html, and that doesn't work, i think i found  a workaround though

@gr2m my workaround didn't work, how about i submit a pull request that allows bower install?
only thing you have to do more is npm run buildand bump the versions

the versions get bumped automatically with semantic-release. But as I said before, I will not commit built files to GitHub. But you can do it on a fork?

ah i see. well i would have tried, i'll work off a fork then


Improve overflowing sidebar labels

Overview
Update sidebar to add ellipsis and a tooltip for overflowing items.
Also fix an alignment issue with the category text.
Fixes: https://github.com/unisonweb/codebase-ui/issues/147
Loose ends
Note that the ideal behavior would be for the category to disappear when
the label is overflowing - this commit doesn't attempt to do that.

merging since, this is pretty uncontroversial