ACM-15430 Fix control plane type calculation for details page

https://issues.redhat.com/browse/ACM-15430
Remove code duplication between cluster list and details, and use most up-to-date version (cluster list).

/test unit-tests-sonarcloud

/cc @vishsanghishetty


Pipe downloading video to opencv

Please follow the guide below

You will be asked some questions and requested to provide some information, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your issue (like that [x])
Use Preview tab to see how your issue will actually look like

Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2016.11.22. If it's not read this FAQ entry and update. Issues with outdated version will be rejected.

[X ] I've verified and I assure that I'm running youtube-dl 2016.11.22

Before submitting an issue make sure you have:

[X ] At least skimmed through README and most notably FAQ and BUGS sections
[ X] Searched the bugtracker for similar issues including closed ones

What is the purpose of your issue?

[ ] Bug report (encountered problems with youtube-dl)
[ ] Site support request (request for adding support for a new site)
[ ] Feature request (request for a new functionality)
[ X] Question
[ ] Other

The following sections concretize particular purposed issues, you can erase any section (the contents between triple ---) not applicable to your issue
Description of your issue, suggested solution and other information
I am writing a program to analyze the contents of a Youtube video using opencv. Ideally, I would like to be able to analyze the video while it is downloading (i.e. as if it were streaming on Youtube), and then throw away frames that have already been analyzed without ever writing to disk. Is it possible for me to pipe youtube_dl's download directly to another process rather than to disk, or some other workaround that would achieve my aims?

Like this:
youtube-dl -o - <the URL> | your_program

Thanks for your reply yan12125, but you didnt answer my question. Perhaps I need to be more explicit...
Let's say I have a Youtube URL and I'm interested in analyzing the video without saving it to disk. Further, I am interested in doing this analysis using opencv in Python. In my Python script, I would write:
import cv2, youtube_dl options = {'outtmpl': #SOMETHING#} Y = youtube_dl.YoutubeDL(options) Y.extract_info(r'http://some.url')
Does this make more sense?

The youtube_dl Python library doesn't support streaming to Python objects yet, so the current solution is using subprocess.Popen. For YouTube, things are simpler as they serve plain HTTP files, so you can do something like this:
info_dict = Y.extract_info(url, download=False)
video_stream = urllib.request.urlopen(info_dict['url'])
# video_stream is a file-like object with read() method

Thanks for your response. I am able to get a stream as you have indicated, but I dont understand what to do with it. How do I decode the stream into a useable format? Others suggest that for live streams you can use opencv's cv2.imdecode and numpy's numpy.fromstring, but I am only getting nonsense from this. Is there further I can do using youtube_dl or do I need to start looking elsewhere after I get the url?

YouTube videos are served as mp4 or webm files. Those files are usually compressed, so you need to decompress them first to get frame-by-frame images.

Thanks for your help. I'm at a loss for now and can live with downloading the video.


How to merge two GoogleSpreadsheetWorksheet?

I have two different GoogleSpreadsheetWorksheet with the same content format. I would like to create a third one that will be as a result GoogleSpreadsheetWorksheet1 + GoogleSpreadsheetWorksheet2.
Is it possible to do it with the current API?

I dont think there is a way to do this using google's api. YOud have to just fetch the data and then write it into another.
You could start by using the copy worksheet method for the first sheet and then reading/writing the data from the second sheet.


patterns codelab: invalid constant value in DocumentScree.build at step 5

Is there an existing issue for this?

[X] I have searched the existing issues
[X] I have read the guide to filing a bug

Steps to reproduce
Work through the patterns code lab to step 5 ( )
Make the code modifications as shown on that page.
The second modification creates a field Text('Last modified ${metadataRecord.modified}'
In this Text field, VS Code highlights metadataRecord with the error message "Invalid constant value"
Expected results
No Dart error message
Actual results
Dart error message "Invalid constant value"
Code sample
Code sample
  @override
  Widget build(BuildContext context) {
    final metadataRecord = document.metadata;
    ;
    return Scaffold(
      appBar: AppBar(
        title: Text(metadataRecord.$1),
      ),
      body: const Column(
        children: [
          Center(
            child: Text('Last modified ${metadataRecord.modified}'),
          ),
        ],
      ),
    );
  }
}

Screenshots or Video

Screenshots / Video demonstration
[Upload media here]

Logs
Logs
No Flutter commands performed. The linter reveals the error when the file is saved.

Flutter Doctor output
Doctor output
[✓] Flutter (Channel stable, 3.16.0, on Ubuntu 23.10 6.5.0-10-generic, locale en_AU.UTF-8)
    • Flutter version 3.16.0 on channel stable at /home/bruce/snap/flutter/common/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision db7ef5bf9f (5 days ago), 2023-11-15 11:25:44 -0800
    • Engine revision 74d16627b9
    • Dart version 3.2.0
    • DevTools version 2.28.2

[✗] Android toolchain - develop for Android devices
    ✗ Unable to locate Android SDK.
      Install Android Studio from: https://developer.android.com/studio/index.html
      On first launch it will assist you in installing the Android SDK components.
      (or visit https://flutter.dev/docs/get-started/install/linux#android-setup for detailed instructions).
      If the Android SDK has been installed to a custom location, please use
      `flutter config --android-sdk` to update to that location.

[✓] Chrome - develop for the web
    • Chrome at google-chrome

[✓] Linux toolchain - develop for Linux desktop
    • clang version 10.0.0-4ubuntu1
    • cmake version 3.16.3
    • ninja version 1.10.0
    • pkg-config version 0.29.1

[!] Android Studio (not installed)
    • Android Studio not found; download from https://developer.android.com/studio/index.html
      (or visit https://flutter.dev/docs/get-started/install/linux#android-setup for detailed instructions).

[✓] VS Code (version unknown)
    • VS Code at /snap/code/current
    • Flutter extension version 3.76.0
    ✗ Unable to determine VS Code version.

[✓] Connected device (2 available)
    • Linux (desktop) • linux  • linux-x64      • Ubuntu 23.10 6.5.0-10-generic
    • Chrome (web)    • chrome • web-javascript • Google Chrome 119.0.6045.159

[✓] Network resources
    • All expected network resources are available.

! Doctor found issues in 2 categories.

Hi @bruceesmith, you need to remove  const from your Column. Notice that it is not in the code for the codelab either.
Closing from here.
Thank you


WARNING: Could not crop because no foreground detected

The output is blank. Nothing segmented for task = "heartchambers_highres". Input file is a .nii.gz file. I used this file as an input to nnU-Net code and it worked. But it's working for totalSegmentor.
I get this message:
WARNING: Could not crop because no foreground detected
Did anyone face this problem?

Can you send me the image then I can try to reproduce the issue?

Hi Jakob Wasserthal,
The following link contains a cardiac 3D MRI sample that I want to segment
through TotalSegmentor. Output is all blank.
Could you please help me with this?
Link: HVSMR_003_0000.nii.gz
https://o365coloradoedu-my.sharepoint.com/:u:/g/personal/moju3572_colorado_edu/EZet2Q3VhhhErOwdMYGABQcB7xtC3Bzp1hqyf5X7f-PrsQ?e=H49Mbx
On Wed, Apr 3, 2024 at 4:33 AM Jakob Wasserthal @.***>
wrote:

Can you send me the image then I can try to reproduce the issue?
—
Reply to this email directly, view it on GitHub
https://github.com/wasserth/TotalSegmentator/issues/290#issuecomment-2034197628,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AC7Y6ZLX57XRI2HUXPYITRTY3PLF3AVCNFSM6AAAAABFUXBNBSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAMZUGE4TONRSHA
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
Jubair


Try really running PHPStan

🧪

Now it is working.

Thanks, I'll ignore a few errors for now that I don't see necessary to be fixed right now. (argument.templateType, missingType.iterableValue)
I'll also ignore trait.unused for now, because, to fix this error, I'd need to include tests/Enums in the scan (where the trait is used in the tests), which would raise quite a few "errors" that I'd have to fix, none of them which will actually cause any issues runtime, e.g.: the last() method returns \Callable|null, which actually is "not true", so I'd have to typehint that.
If you feel like you'd want to contribute to this, I'd appreciate it, otherwise I'll fix these issues later and roll out an update.


Update microformat-shiv dependency reference

I am getting build errors with Travis-CI due to an intermittent 403 response from this api.github URL during npm install, even though it seems to install fine locally:
"microformat-shiv": "https://api.github.com/repos/glennjones/microformat-shiv/tarball/"

Would you be opposed to using the owner/repo convention outlined here?
"microformat-shiv": "glennjones/microformat-shiv"

Hi Richard, Thanks for the PR. I had to make some larger changes to get microformat-node to play nice with microformat-shiv. This change was roll into the latest update and is now in v2.x of microformat-node
Everything should now pass the tests, if you have any more problems let me known.


cannot import name 'save_open' from 'safetensors'

System Info

python 3.9.20
safetensors 0.4.5

Information

[X] The official example scripts
[ ] My own modified scripts

Reproduction
I tried pip install safetensors and conda install -c huggingface safetensors.
Other API such as safetensors.torch.save_file/load_file are correct as the docs described.
But can not use save_open from from safetensors import save_open
Expected behavior
from safetensors import safe_open

tensors = {}
with safe_open("model.safetensors", framework="pt", device=0) as f:
    for k in f.keys():
        tensors[k] = f.get_tensor(k)

Please check your spelling of safe_open, if you still get an error, please post your error message.

Sorry, I think it may be due to a caching issue. I was unable to import safe_open earlier, but after trying again, it returned to normal.


MultiTax version 1.1.0

Extended name search

Codecov Report

Merging #2 (5ee4dfd) into main (f0e962d) will increase coverage by 0.12%.
The diff coverage is 98.78%.

@@            Coverage Diff             @@
##             main       #2      +/-   ##
==========================================
+ Coverage   97.07%   97.19%   +0.12%
==========================================
  Files          15       15
  Lines        1059     1176     +117
==========================================
+ Hits         1028     1143     +115
- Misses         31       33       +2

Impacted Files
Coverage Δ

multitax/ncbitx.py
94.44% <96.15%> (+0.07%)
:arrow_up:

multitax/otttx.py
96.00% <97.61%> (+0.54%)
:arrow_up:

multitax/customtx.py
89.47% <100.00%> (ø)

multitax/greengenestx.py
94.28% <100.00%> (ø)

multitax/gtdbtx.py
91.42% <100.00%> (ø)

multitax/multitax.py
97.81% <100.00%> (+0.04%)
:arrow_up:

multitax/silvatx.py
94.44% <100.00%> (ø)

multitax/utils.py
92.40% <100.00%> (+0.19%)
:arrow_up:

tests/multitax/integration/test_common.py
100.00% <100.00%> (ø)

tests/multitax/unit/test_functions.py
100.00% <100.00%> (ø)

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 f0e962d...5ee4dfd. Read the comment docs.


[20.09] gnome3.mutter334: 3.34.5 -> 3.34.6, update patches

Now managing patches using my git-series.

drop xwayland-path
It now uses find_program.path()

add 0008-build-bump-ABI-to-sysprof-capture-4.patch
This allows us to build with the sysprof in nixpkgs.

This is a version of the update using a series just for the gnome
stack on 20.09 (21.03 has 3.38 and sysprof-4).

Motivation for this change
This is blocked on https://github.com/NixOS/nixpkgs/pull/98316 being merged
It's living in https://github.com/NixOS/nixpkgs/pull/100224 for someone to test but it shouldn't be merge from there.
Things done

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

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

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

That update is in staging now


Level difficulty and player speed

I need to implement different speed settings for the player movement (but not game) depending on difficulty. This will likely mean keeping the FPS constant (120) but changing movement (ex: factor of 1/2). I can do this by multiplying the player physics by 3/2n, then have n represent the difficulty (something like Easy = 3, Normal = 2, Hard = 1) as an integer counter.

An easier way would use a dictionary with the difficulty settings (easy, normal, hard) to translate into their respective physic modifiers (0.6, 0.8, 1.0)

Sounds like everything is good. I'm assuming right now based on the speed, the difficulty is now hard.

Yea, I'll implement it in the next patch

This feature has been added in dsn_class v.0.2.4 and dsn_levels v.0.3.1


Linter not appearing in plugin list after install/enable

Obsidian: v0.12.15
Linter: v1.2.4
I've attempted to install and enable the plugin and, while it appears in the Community Plugins page and shows it's enabled, it does not appear in the Plugin Options list. Also, linting commands do not appear in the Command Palette. I've tried disabling/re-enabling, uninstalling and reinstalling the plugin, and restarting Obsidian.
I'm wondering if I'm missing a step or if it's a bug. Thank you for what looks to be a great plugin and I'm really looking forward to using it. 😄

fixed in latest release


Fault Tolerance

Fault Tolerance
A system is an entity with a well-defined behavior in terms of output it produces and
which is a function of the input it receives, the passage of time and its internal logic.
By “well-defined behavior” we mean that the output produced by the system is previ-
ously agreed upon and unambiguously distinguishable from output that does not
qualify as well-defined behavior. The well-defined behavior of a system is called the
system specification. A system interacts with its environment by receiving input from
it and delivering output to it. It may be possible to decompose a system into constitu-
ent (sub)systems. In Component-based software engineering (CBSE) terms, a system
is a component that may consists of the assembly of a number of smaller components.
In OO terms a system is a composition of objects, each of which may be itself a composition
of smaller objects.
A failure is said to occur in a system when the system’s environment observes an
output from the system that does not conform to its specification. An error is the part
of the system, e.g. one of its constituent (sub)systems, which is liable to lead to a
failure. A fault is the adjudged cause of an error and may itself be the result of a fail-
ure. Hence, a fault causes an error that produces a failure, which subsequently may
result to a fault, and so on. Let us consider the following example:
A software bug in an application is a fault that leads to an error when the
application execution reaches the point affected by the bug, which in turn
makes the application crash which is a failure. By crashing, the applica-
tion leaves blocked the socket ports it used which is a fault and the com-
puter on which the application crashed has socket ports which are not
used by any process nevertheless not accessible to running applications
which is an error, and which in turn leads to a failure when another appli-
cation requests these ports.
Based on the above, a fault in a system may propagate to the system's environment.
A system is called fault tolerant when it can deal with faults and their consequent er-
rors in such a way that it does not violate its specification, i.e. the environment of a
fault tolerant system does not perceive a failure of the system. Hence, a fault tolerant
system does not propagate faults to its environment. Fault tolerance techniques are
practical methods that describe how to detect an error and confine it within a system.
The confinement can be based on the restoration of the subsystem on which the er-
ror was detected before that error infects other parts of the system, or it can be based
on the masking of the error occurrence (e.g. by isolating the subsystem on which the
error was detected and using some form of redundancy to deliver the expected out-
put).
In general terms, fault tolerance provides techniques to confront faults and their con-
sequences in a system. These techniques describe the detection of errors in a sys-
tem, and the means that ensure the recovery of a system from errors or the masking
of errors in a system.
Three constituents of fault tolerance are error detection, recovery and masking.
Resource:

https://hillside.net/europlop/HillsideEurope/Papers/EuroPLoP2002/2002_Saridakis_ASystemOfPatternsForFaultTolerance.pdf
[aviz:95] A. Avizienis, “Building dependable systems: How to keep up with complexity”, Twenty-Fifth Fault-Tolerant Computing Symposium Special Issue, pp. 4–14, June 1995. (Historical Interest).
[siew:95] D. P. Siewiorek, “Niche successes to ubiquitous invisibility: Fault-tolerant computing past, present and future”, Twenty-Fifth Fault-Tolerant Computing Symposium Special Issue, pp. 26–33, June 1995. (Historical Interest).
[cris:91] F. Cristian, “Understanding fault–tolerant distributed systems”, Communications of the ACM, vol. 34, no. 2, , February 1991. (Historical Interest).
[aviz:04] A. Avizienis, J. Laprie, B. Randell, and C. Landwehr, “Basic concepts and taxonomy of secure computing”, IEEE Transactions of Dependable and Secure Computing, vol. 1, no. 1, , January 2004. (Taxonomy and Definitions).
[abra:86] J. A. Abraham and W. K. Fuchs, “Fault and error models for VLSI”, Proceedings of the IEEE, vol. 74, no. 5, pp. 639–654, May 1986. (Fault Models and Historical Interest).
[mull:93] V. Hadzilacos and S. Toueg, “Fault tolerant broadcasts and related problems”, In Distributed Systems, S. Mullender, editor, pp. 100–102, Addison Wesley, 2nd edition, 1993. (Definition).
[kala:13] R. Kalayappan and S. R. Sarangi, “A survey of checker architectures”, ACM Computing Survey, vol. 45, no. 4, pp. 48:1–48:34, August 2013. (Fault Definitions and Architecture Level Techniques).
[goel:81] P. Goel, “An implicit enumeration algorithm to generate tests for combinational logic circuits”, IEEE Transactions on Computers, vol. C-30, no. 3, pp.215–222, March 1981. (Test Generation).
[duarte:11] E. P. Duarte, R. P. Ziwich, and L. Albini, “A survey of comparison-based system-level diagnosis”, ACM Computing Survey, vol. 43, no. 3, pp. 22:1–22:56, April 2011. (System Level Diagnosis).
[siew:92] C. L. Chen and M. Y. Hsiao, “Error-correcting codes for semiconductor memory applications: A state of the art review”, In Reliable Computer Systems - Design and Evaluation, D. P. Seiwiorek and R. S. Swarz, editors, pp. 771–786, Digital Press, 2nd edition, 1992. (ECC).KKS753–11–spring 2
[lu:13] S. Lu, H. Jheng, M. Hashizume, J. Huang, and P. Ning, “Fault scrambling techniques for yield enhancement of embedded memories”, Proceedings of IEEE Asian Test Symposium, pp. 215–220, November 2013. (Memory Reconfiguration).
[mahm:88] A. Mahmood and E. J. McCluskey, “Concurrent error detection using watchdog processor- A survey”, IEEE Transactions on Computers, vol. C-37, no. 2, pp. 160–174, February 1988. (Watchdog and Historical).
-[rotenberg:99] E. Rotenberg, “AR-SMT: A microarchitecture approach to fault tolerance in microprocessors”, Proceedings of IEEE Fault-Tolerant Computing Symposium, pp. 84–91, June 1999. (Architecture Level).
[rashid:00] F. Rashid, K. K. Saluja, and P. Ramanathan, “Fault tolerance through re-execution in multiscalar architectures”, Proceedings of IEEE International Conference on Dependable Systems and Networks, also known as FTCS-30, pp.
482–491, June 2000. (Architecture Level).
[subra:10] P. Subramanyan, V. Singh, K. K. Saluja, and E. Larsson, “Energy efficient fault tolerance in chip multiprocessors using critical value forwarding”, Proceedings of IEEE International Conference on Dependable Systems and Networks, pp. 121–130, June 2010. (Architecture Level).
[aggarwal:07] N. Aggarwal, P. Ranganathan, N. Jouppi, and J. Smith, “Isolation in commodity multicore processors”, IEEE Computer, vol. 40, no. 6, pp. 49–59, June 2007. (Architecture Level).
[brooks:96] R. R. Brooks and S. S. Iyengar, “Robust distributed computing and sensing algorithm”, IEEE Computer, vol. 29, no. 6, pp. 53–60, June 1996. (Sensor Networks).
[Clouqueur:04] T. Clouqueur, K. K. Saluja, and P. Ramanathan, “Fault tolerance in collaborative sensor networks for target detection”, IEEE Transactions on Computers, vol. 52, no. 3, pp. 320–333, March 2004. (Sensor Networks).
[lapr:90] J.-C. Laprie, J. Arlat, C. Beounes, and K. Kanoun, “Definition and analysis of hardware and software fault-tolerant architectures”, IEEE Computer, vol. 23, no. 7, pp. 39–51, July 1990. (Historical).
[gray:90] J. Gray, “A census of Tandem system availability, 1985-1990”, IEEE Transactions on Reliability, vol. 39, no. 4, pp. 409–418, October 1990. (Historical).

Fault Tolerance Strategy via isolation from external dependencies so that one may not affect the other
Thread Based Isolation
In this isolation method; the external call gets executed on a different thread (non-application thread), so that the application thread is not affected by anything that goes wrong with the external call.
Semaphore Based Isolation
In this method, the external call runs on the application thread and the number of concurrent calls are limited by the semaphore count defined in the configuration.
Resource

[ ] https://medium.com/bliblidotcom-techblog/latency-and-fault-tolerance-for-distributed-system-2c6d3213e439

Byzantine Fault

[ ] https://en.wikipedia.org/wiki/Byzantine_fault
[ ] https://blog.cloudflare.com/a-byzantine-failure-in-the-real-world
[ ] https://www.youtube.com/watch?v=fqmhbMyA1qk
[ ] https://www.coursera.org/lecture/blockchain-scalability/consensus-with-byzantine-failures-Mcy0a
[ ] https://www.youtube.com/watch?v=_e4wNoTV3Gw
[ ] https://www.youtube.com/watch?v=LKpIirL8f-I SLI/SLO/SLA


AndroidX Support

Xamarin.Facebook.Android has a dependency on Android support Libraries.  Is there any timeline on an AndroidX version?

Xamarin.Facebook.Android

Xamarin.Facebook.Share.Android

Xamarin.Android.Support.v7.AppCompat

The updated version of Facebook for Android point to AndroidX.
https://github.com/xamarin/FacebookComponents/blob/master/Facebook.Android/source/facebook-share/facebook-share.csproj#L62

what does this exactly mean? Because when i try to update to the latest version, I get a warning that I must install Xamarin.Android.Support.v7.AppCompat. Was that AndroidX update not pushed yet as nuget ?


Document limitations related to centering text with RichTextLabel

This is a relatively common question on various community channels.

Thanks!

Cherry-picked for 3.2.3.


changed 'URL into a PDF file' to 'Web Page into PDF file'

Hello! This is to close issue #37, which requests to replace all references of 'URL into a PDF file' to 'Web Page into PDF file'. Please let me know if there's anything else you would like me to improve!

Hi, Thank you for your PR. Someone else, already working on this! you can add "Night Mood" in the todo section as unchecked (below Translation)

Thank you for the quick response!
Just wanted to confirm - would you like me to create a separate issue for this and separate branch? Or would you like me to continue committing to this branch?

Create a separate issue


chore: remove utility classes from components

Jira: https://dialpad.atlassian.net/browse/DLT-1125
Dialtone vue: https://github.com/dialpad/dialtone-vue/pull/1314
Description
Adds styles for the following components in dialtone library:
avatar
chip
codeblock
checkbox
collapsible
combobox
input
input group
image viewer
keyboard shortcut
list item group
(more to come)
Pull Request Checklist

[ ] Ask the contributors if a similar effort is already in process or has been solved.
[ ] Review the contribution guidelines.
[ ] Use staging as your pull request's base branch. (All PRs using production as its base branch will be declined).
[ ] Ensure all gulp scripts successfully compile.
[ ] Update, remove, or extend all affected documentation.
[ ] Ensure no private Dialpad links or info are in the code or pull request description (Dialtone is a public repo!).

Obligatory GIF (super important!)

avatar
chip
codeblock
checkbox
LGTM 🚀

collapsible
combobox
input
radio-checkbox
LGTM 🚀

image-viewer
keyboard-shortcut
list-item-group
LGTM 🚀


Run additional QA on contract stats for popular projects [JS]

Related to #501
Also need to investigate other contracts that behave like factories / have lots of internal transactions.
Rabbithole is a good example -- should have more than 0.098 ETH in fees given there are ~150K transactions.

Nothing new ... there remaining issues around completeness of onchain transactions (see #501) and historic fork/star events (see #500).
Here is the Google Sheet I used for QA
And here's the query that generated the data:
SELECT
    p.slug AS project,
    a.name AS artifact,
    SUM(CASE WHEN e."typeId" = 25 THEN e.amount ELSE 0 END) AS txns,
    SUM(CASE WHEN e."typeId" = 26 THEN e.amount ELSE 0 END) AS fees,
    MAX(CASE WHEN e."typeId" = 14 THEN e.amount ELSE 0 END) AS num_stars,
    MAX(CASE WHEN e."typeId" = 22 THEN e.amount ELSE 0 END) AS num_forks,
    MIN(CASE WHEN e."typeId" = 4 THEN e.time ELSE NULL END) AS first_commit,
    SUM(CASE WHEN e."typeId" = 4 THEN e.amount ELSE 0 END) AS sum_commits,
    SUM(CASE WHEN e."typeId" = 3 THEN e.amount ELSE 0 END) AS sum_prs_merged,
    SUM(CASE WHEN e."typeId" = 9 THEN e.amount ELSE 0 END) AS sum_downloads
FROM
    event e
LEFT JOIN
    project_artifacts_artifact paa ON e."toId" = paa."artifactId"
LEFT JOIN
    project p ON paa."projectId" = p."id"
LEFT JOIN
    artifact a ON paa."artifactId" = a."id"
WHERE
    p.slug IN ('{slugs_list}')
    AND e."typeId" IN (3, 4, 9, 14, 22, 25, 26)
GROUP BY
    p.slug,
    a.name;


Help on Usage

Thank you so much such a wonderful blog and tool. I followed everything except the first part of usage because I don't know what to do. I already have fish_title function.  What else to do? The code below doesn't look like config for WezTerm.
  {
    "window_id": 0,
    "tab_id": 167,
    "pane_id": 350,
    "workspace": "default",
    "size": {
      "rows": 48,
      "cols": 175,
      "pixel_width": 2975,
      "pixel_height": 1776,
      "dpi": 144
    },
    "title": "hx . ~/C/p/helix-wezterm",

~/.config/fish/config.fish
function fish_title
    # If we're connected via ssh, we print the hostname.
    set -l ssh
    set -q SSH_TTY
    and set ssh "["(prompt_hostname | string sub -l 10 | string collect)"]"
    # An override for the current command is passed as the first parameter.
    # This is used by `fg` to show the true process name, among others.
    if set -q argv[1]
        echo -- $ssh (string sub -l 20 -- $argv[1]) (prompt_pwd -d 1 -D 1)
    else
        # Don't print "fish" because it's redundant
        set -l command (status current-command)
        if test "$command" = fish
            set command
        end
        echo -- $ssh (string sub -l 20 -- $command) (prompt_pwd -d 1 -D 1)
    end
end

Sorry for not making it clear. I've just pushed a fix to clarify.

Sorry for not making it clear. I've just pushed a fix to clarify.

Thank you so much! Works great!


[FEATURE REQUEST] Reduce size of navigation bar

Is your feature request related to a problem? Please describe.
Navigation bar takes up too much space which could be reduced.
Describe the solution you'd like
With labels removed there is lot of space around icons which could be reduced further to free up more space for the feed.

Describe alternatives you've considered
If possible hiding the navigation bar when scrolling would be nice feature also.
Additional context
In true black navigation bar still have some subtle material theme that doesn't blend with true black.
Many thanks

I was looking into removing the extra space before the UI refresh though I thought most users wouldn't care

I'll try and add this feature before the next release

This issue has been fixed in v3.4.1


Attribution information is hidden under images

https://priviblur.zangetsu.kaizoku.cyou/vonkarn/742356566754426881:

The attribution info is still there in the DOM:

Removing the image shows the attribution information:

Should be a pretty easy fix. The attribution element just needs to be moved from the image-container to the parent image-block.
I'll push a fix for this in the morning


PHP 7.1 feature

After checking the Travis build, it seems that the function arguments are serious for checking the function argument counter.
Here is the exception ,message snippet during executing the PHPUnit test:
EmailTemplatesTest\InputFilter\TemplateInputFilterTest::testValidationOfEachProperty with data set #0 ('updateParameters', true, null, true)
ArgumentCountError: Too few arguments to function Roave\EmailTemplates\Validator\CanRenderValidator::__construct(), 0 passed in /home/lee/EmailTemplates/vendor/zendframework/zend-servicemanager/src/AbstractPluginManager.php on line 291 and exactly 1 expected

Please fix this.
Thanks.

Could you elaborate with an example?
On 7 Feb 2018 20:21, "peter279k"<EMAIL_ADDRESS>wrote:

After checking the Travis build, it seems that the function arguments are
serious for checking the function arguments. Here is the code snippet
EmailTemplatesTest\InputFilter\TemplateInputFilterTest::testValidationOfEachProperty with data set #0 ('updateParameters', true, null, true)
ArgumentCountError: Too few arguments to function Roave\EmailTemplates\Validator\CanRenderValidator::__construct(), 0 passed in /home/lee/EmailTemplates/vendor/zendframework/zend-servicemanager/src/AbstractPluginManager.php on line 291 and exactly 1 expected
Please fix this.
Thanks.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/Roave/EmailTemplates/issues/11, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAJakDgN5SHG43hIwut9tny-pRsshaPkks5tSffKgaJpZM4R9QFl
.

Reading the Travis-ci logs, it looks like it's installing Zend framework components that are no longer supporting the old style of factories. I'll update the factories and tag a new master based on the latest components tomorrow.


Fixed #31659 -- Made ExpressionWrapper preserve output_field for combined expressions.

Regression in df32fd42b84cc6dbba173201f244491b0d154a63.
I think it makes sense to set output_field on a combined expression, used in ExpressionWrapper, when it cannot be resolved.
ticket-31659

buildbot, test on oracle.

Thanks! Curious about whether or not making BaseExpression.select_format return sql, params if self._output_field_or_none is None worked as well?

Yes it works, but I assumed it's too invasive for a backport to Django 3.1.

Yes it works, but I assumed it's too invasive for a backport to Django 3.1.

Makes sense!

Merged in aeb8996a6706cad3e96d8221760c1cb408ee7ed9.


Feature Question: Frame analysis with cv.VideoCapture().read() instead of VideoReader

Hi,
i love your code and project and wanted to test out the implementation of deepposekit generated models in my own pipeline (DeepLabStream) . Unfortunately I am already working with the "base" class VideoCapture from OpenCV and cannot simply implement your VideoReader into my existing framework (as far as I can tell).
While your code suggests that any model exported and loaded needs the VideoReader for prediction, I assume it is a function that will pass the frames directly, with the "pure" predict() function somewhere deep within. Even though batch-size = 1 is possible to set and the VideoReader is only passing frames without altering them (if gray = False), I was not able to pass frames directly to model.predict(). I did not find the code for your model.predict() yet. When passing a single frame from VideoCapture.read() to model.predict(frame), I get a dimension error (I can post it in more detail if you'd like) which I assume means that simply passing it a frame does not work.
So my question is: Is there a way to directly access the model.predict function within each loaded model and pass it a single frame.
BTW I am using your (fly) data repository for testing now, thank you for providing it!

found a solution myself.


Fixed the broken link with modification of lines

Problem:
This is regarding this section on this page: https://kubernetes.io/docs/concepts/scheduling-eviction/assign-pod-node/#an-example-of-a-pod-that-uses-pod-affinity
See the design doc for many more examples of pod affinity and anti-affinity.
The link to https://git.k8s.io/community/contributors/design-proposals/scheduling/podaffinity.md is dead.
Solution:
This issue has been resolved by adding a simple link and enhancing the document line so that it can look beginner-friendly.

Hey @davidopp , @kevin-wangzefeng  Please go through this PR and feel free to suggest some changes if necessary.

Hi @NitishKumar06
you  have to update the link to the below one:
https://github.com/kubernetes/design-proposals-archive/blob/main/scheduling/podaffinity.md

Hey @Shubham82 , Could you please tell me to which link this should be updated? This is already working fine.

Okay I'll do it .
On Sat, 28 May, 2022, 4:14 pm Shubham, @.***> wrote:

@.**** requested changes on this pull request.
@niteshseram https://github.com/niteshseram Update this link to
https://github.com/kubernetes/design-proposals-archive/blob/main/scheduling/podaffinity.md
Actually, this
https://git.k8s.io/community/contributors/design-proposals/scheduling/podaffinity.md
redirects to Design proposals that have been archived, and All design
proposals are moved under the kubernetes/design-proposals-archive
https://github.com/kubernetes/design-proposals-archive/ repo.
—
Reply to this email directly, view it on GitHub
https://github.com/kubernetes/website/pull/33926#pullrequestreview-988480868,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AUQQSLU7SH3KFF5JT56MXMLVMH2KFANCNFSM5W24LQEA
.
You are receiving this because you were mentioned.Message ID:
@.***>

Is this a dup?

Duplicate of https://github.com/kubernetes/website/pull/34019
/close

Update to which link? This link is working fine.
On Wed, 25 May, 2022, 9:50 pm Shubham, @.***> wrote:

Hi @NitishKumar06 https://github.com/NitishKumar06
you have to update the link to the below one:
https://github.com/kubernetes/design-proposals-archive/blob/main/scheduling/podaffinity.md
—
Reply to this email directly, view it on GitHub
https://github.com/kubernetes/website/pull/33926#issuecomment-1137494927,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AUQQSLTDA7HRGGQMZDJZHPLVLZHLFANCNFSM5W24LQEA
.
You are receiving this because you were mentioned.Message ID:
@.***>


Additional Data Security Rules

This https://matthewdf10.medium.com/aws-accounts-as-security-boundaries-97-ways-data-can-be-shared-across-accounts-b933ce9c837e excellent article was published about all the way AWS services can share data. I suspect there are a lot of valuable rules that can be created from this. This task is to analyze the paper and the services and develop rules that would fit in the data security policy.

https://github.com/openraven/magpie/issues/133


TypeError: Cannot read properties of null (reading 'bot')

/root/AlfredBot/index.js:586
if (message.author.bot || message.channel.type == "dm") return; //if message is from a bot or is in a DM with the bot
^
TypeError: Cannot read properties of null (reading 'bot')
at ProcessMessage (/root/AlfredBot/index.js:586:24)
at processTicksAndRejections (node:internal/process/task_queues:96:5)
Node.js v17.9.1

https://github.com/rhit-corridg/AlfredBot/issues/51


changing the HLIP address used in apipa endpoint

Using the different HLIP prefix.

/azp run

@vinayakawanti looks like this PR not merged yet. If you don't want this PR to be merged, please close the PR


Update tool to 2021-11-19.

maclain27

asking for this repository to run Arduino in vs code readme anytime I need change!

maclain27

asking for this repository to run Arduino in vs code readme anytime I need change!

I don't understand what you mean @Maclain27

Hi it me  i said  i need access to vcpkg to run Arduino in vs code us it was recommeder and feel  free to tell if i need something wrong!!!
Get Outlook for Androidhttps://aka.ms/AAb9ysg

From: BillyONeal @.>
Sent: Monday, November 22, 2021 6:09:45 PM
To: microsoft/vcpkg @.>
Cc: Maclain Makanda @.>; Mention @.>
Subject: Re: [microsoft/vcpkg] Update tool to 2021-11-19. (PR #21550)
I don't understand what you mean @Maclain27https://github.com/Maclain27
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/microsoft/vcpkg/pull/21550#issuecomment-975995018, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ASH7BWMRJQOUILVVHOAFTXTUNLETTANCNFSM5IND6H3Q.

I mean I was runing Arduino in vs code and than I got error it was recommder to install vcpkg to found the library!


A comment on hooks

I've been following your new commits on dbgui branch and I think it's a great idea that you are planning to add hooks for debugging purposes. I've been planning to do it myself but now I'm gonna wait for this update.
So i just had a little suggestion if you are interested.  We could have a macro function to filter those if checks for hooks all together in release builds. much like assert macro.
thank you!

Yes, a similar thought occured to me when writing the stuff. I would most likely put it behind a new define (something like SOKOL_NO_TRACE_HOOKS).


Ipc 2581

Failures here are from #476.  Branch protections for our CPython unit testing are now enabled so this doesn't happen again.


[1.15] Marine Fisher missing texture

Industrial Foregoing Version: 2.3.3
Titanium Version: 2.4.2
Crashlog If Applicable (Upload to pastebin/gist): N/A
When viewing the Marine Fisher in JEI or holding it in your hand, it has a missing texture (it uses the black and purple squares). Once placed in the world it looks correctly.
Forge 31.2.43

Fixed in the 1.16 version, the 1.15 version is unsupported


Extra information about Image-based Coordinate Systems

This issue relates to the image based coordinate systems as described in the appendix.
https://bids-specification.readthedocs.io/en/stable/99-appendices/08-coordinate-systems.html
I was asking @gllmflndn  about the proper space label to use for images that are normalized to MNI with SPM and that discussion ended up looping in @cmaumet to cross check what decisions had been made for NIDM-results.
@cmaumet then send a link that excellent summary table
While going through it, I realized:

some of the information I was looking for already in the BIDS spec appendix but hard to "parse" from the table
some extra information contained in that NIDM issue could be added to BIDS spec.

I suggest:

adding a column "used by" in our appendix table to help readability and add "missing information" (softwares not mentioned so far)
possibly adding a "reference" column to get the any eventual link out of the description

Also is there somewhere a correspondence table" between the names of the BIDS coordinate systems and those of NIDM? The names are pretty transparent (to a human), so not sure how needed this is.
Maybe some of that info belongs better in the starter kit: what do y'all think?

Assuming that you are talking about the Standard template identifiers table, I think your two suggestion to make it more readable are good (adding two new columns)

Also is there somewhere a correspondence table" between the names of the BIDS coordinate systems and those of NIDM?

I don't know


Cannot attach multiple files to a message.

when attempting to pass multiple filenames as a list to the files= parameter in a message creation, I receive the following Error:
ValueError: The length of the files list is greater than one (1). The files parameter is a list, which accepts multiple values to allow for future expansion, but currently only one file may be included with the message.
Are there any plans to support multi file messaging?

@mechaHarry, good question.  This is a limitation of the Messages Create API.  I recommend reaching out to the Developer Support team in the #webex4devs Teams space or via the<EMAIL_ADDRESS>e-mail address for roadmap questions.

Thanks for pointing me in the right direction.
Just out of curiosity, from an implementation standpoint,
if and when the API decides to support multiple files via REST,
will your sourcecode need to be adjusted for this, or will it support it off the bat?
Thanks,

mH

Right now we have a check in the SDK that verifies that only one item is in the list of files.  We do this so that we can provide an articulate error message about why there can only be one file attached to the message.  We can remove the check and allow the Webex Teams API to throw an error; doing so would allow the SDK to support multiple files as soon as the APIs do.


fix(readdirSync): 🐛  Error no such file or directory

readdirSync throws ENOENT: no such file or directory, scandir when
the directory does not exists one of the file systems.
Problem solved checking also if there are some results before throw.

Fixes #240

:tada: This PR is included in version 4.2.1 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


feat: adding vendoring

Can you please amend the commit message to give more rationale about this?  I personally get why you want this: vendoring means the repository is now stand-alone, and any time we upgrade a dependency, it now shows in the repo without CI needing to access the web - but the commit message should state it for the sake of someone less familiar with the why.
Also, if you can include a Signed-off-by tag (with DCO semantics used by the Linux kernel), it will make it easier to merge this without waiting for Red Hat legal to state whether it will be mandatory for patches going forward in this project.

One more thing - this looks like a very automated patch - can you include the command line(s) that you ran to produce this patch, so that it is easy to reproduce?  When a patch this large needs to be backported across downstream branches (not that KubeSAN has any yet, but it may in the future), it is much easier to resolve merge conflicts by re-running the generating commands on the downstream branch.

Closing in favor of upstream MR https://gitlab.com/kubesan/kubesan/-/merge_requests/109


docs(operators): add documentation for last

add documentation and an example for the last operator
close #94
This is my first PR for an open source project, so I'm really looking forward for any kind of feedback. Thank you!

Codecov Report

Merging #208 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #208   +/-   ##
=======================================
  Coverage   74.49%   74.49%
=======================================
  Files          10       10
  Lines         149      149
  Branches        7        7
=======================================
  Hits          111      111
  Misses         38       38

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 4ee958d...49d2baa. Read the comment docs.

@JWO719 this is very exciting! i'm so happy to have you here! we'll be reviewing shortly! :)


ignore consoleError when run promise a+ test to avoid annoying error …

when run promise a+ test, don't output console.error for uncaught promise errors.

Close this one because I open a new PR #627 to add more features.


[BUG] Please set up chatMemory in order to use tools

In the single chat scenario, in order to reduce the use of tokens, I did not set chatmemory, and I wanted to use the tool, but I got an error：

This doesn't fit my scene 😣

Hi @chenhaoqiang, ChatMemory is required to use tools. To explain, here is the simple example:

User sends a message to the LLM. List of available tools is provided as well.
LLM chooses to execute one of the tools and responds with a request to execute a particular tool.
Tool is executed and the result is sent back to the LLM.
LLM processes tool execution result responds in natural language to the user.

As you can see, on the step 3 LLM should receive not only the result of tool execution, but also the initial message from user (from step 1) and it's request to execute a tool (from step 2). Otherwise, it will not understand what is going on and it's response will be random.
Thus, when using tools, you need to have a ChatMemory, at least MessageWindowChatMemory with capacity 3.

Hi @chenhaoqiang, ChatMemory is required to use tools. To explain, here is the simple example:

User sends a message to the LLM. List of available tools is provided as well.
LLM chooses to execute one of the tools and responds with a request to execute a particular tool.
Tool is executed and the result is sent back to the LLM.
LLM processes tool execution result responds in natural language to the user.

As you can see, on the step 3 LLM should receive not only the result of tool execution, but also the initial message from user (from step 1) and it's request to execute a tool (from step 2). Otherwise, it will not understand what is going on and it's response will be random.
Thus, when using tools, you need to have a ChatMemory, at least MessageWindowChatMemory with capacity 3.

Thank you for answering so carefully and patiently! I realized that I had misunderstood 😱


Added coverage report

This might work 🤞

Looks great, thanks again for the contribution :)

Great! Thanks for accepting it 👍


Add a step in the tutorial to select map enrichments

Tutorial URL: https://developers.sap.com/tutorials/cai-bot-shipping-3-price-parcel.html
Step 1: Create entity for parcel size
Here after clicking on Enrichment, please add a step in the Tutorial to select map enrichments since I got the below screen after clicking on Enrichment, had to select the map enrichments to proceed forward.

Best Regards,
Priyanka

Labels to be attached:
Landscape label: trail1, trail2
Sprint Label: SCPTest-2008A
Scenario Labels: SCPTest-cai
Best Regards,
Priyanka

Added that they must select Map Enrichments


Should be able to have if statement inside each

This does not work:
{{#each user.permissions}}
{{#if this.has_access}}
    {{/if}}

{{/each}}
because it detects that i'm closing the if and thinks it conflicts with the closing of {{/each}}.  The exact error is:
  throw new Handlebars.Exception(open.original + " doesn't match " + close.original);

at line 609

i'm preety sure if works inside each. can you provide a more complete code sample.


v3: helm template is missing the flag to render only selected templates

In helm v2 we had helm template mychart-1.2.3.tgz -x templates/your.yaml that can be used to render only selected template files in the chart.
If you run helm template --help, this is featured in the beginning of the output like:
To render just one template in a chart, use '-x':

	$ helm template mychart -x templates/deployment.yaml

Usage:
  helm template [flags] CHART

Flags:
  -x, --execute stringArray      only execute the given templates

This feature isn't announced for removal so I'm assuming this is a regression in v3. Can we just reimplement it for v3?
If we are to reimplement this for helm v3, a concern is that helm template now supports templating the remote chart directly, without firstly helm fetching the chart. The v3 version of the feature should just work regardless of the chart is local or remote.

Output of helm version: helm-3.0.0-beta.1~4
Cloud Provider/Platform (AKS, GKE, Minikube etc.): Any

yes, it is the regression, so it can be reimplement in v3

It is NOT a regression in the sense that it was forgotten.
The -x flag was intentionally removed from Helm 3 as it would introduce a problem with the Lua hook system or library charts: if a template relied on a Lua hook being executed, the -x flag would prevent that hook from running first. That is not a good design.
What we wanted to do post-3.0 is to re-implement -x as a selective post-render step, where it would select certain templates based on a label selector. That way, the chart goes through its full execution and you still receive a single template post-render.
Does that make sense?

closed via #6634

Should the helm2-to-3 migration document be adapted to be more true to the change:
https://helm.sh/docs/topics/v2_v3_migration/
template: -x/--execute argument renamed to -s/--show-only

This reads like the logic for both the flags is the same, but it's not.
-x in helm2 was doing single-file template, but -s is doing template on all chart and only showing the file given.


Hide Bar for the User

Will the serialization still work? 😃 This would be fantastic!

Let's see.
I have started creating the Beam(), Truss(), Column() objects (which can be serialised) but Bar is still not fully hide.
I need to explore the keyword internal, private ect ect


Allow serialization of broadcasted ndarrays

The solution in this PR is simply to create a contiguous copy of the broadcasted array and save it to disk. While this uses more memory on disk than strictly required by numpy, we need to account for the fact that broadcasting is a concept that is specific to numpy and may not translate to other implementations of ASDF. For this reason, it seems to make the most sense to create a copy of the broadcasted array so that it is available for other implementations.
This fixes #506.
cc @Cadair

Coverage increased (+0.003%) to 93.976% when pulling 492d877dc1e3008b0891b06e23b91cddde2b5d0a on drdavella:ndarray-broadcast into cbdaaa318c55838a08212c71cc23036a0a1db50b on spacetelescope:master.

Are you checking for c contiguous because that's how you are saving it to disk? I mean vs f contiguous?

@Cadair I believe the answer is yes, but in truth I'm checking for c_contiguous here because that's what was being checked for prior to this change as well. This should actually probably be defined by the standard somewhere, or encoded in the schema, though.


added .gitignore to project

Added the .gitignore file to maven project to avoid showing target files and temporary files as changes.

thanks @edegaudenzi !

It's a pleasure! ;)


Ensure each animation only affect its animated nodes.

I checked all animated scenes we use.

I rebased on master.


Issue with Scaling down the nodes

Using Gardener, I have created a cluster on AWS with 5 worker groups with auto scaling limits set minimum as 5 and maximum as 10.
Observation1:  I expect gardener to create a 25 Node Cluster but it created 20 node cluster(no errors from my AWS console regarding the quota limits).
Observation2: Using kubectl, I have deleted 19 nodes out of 20 and the nodes got deleted successfully. So when I execute kubectl get nodes -- shows only one node.
But surprisingly when I look in to AWS EC2 console I still see that 20 Instances are still running. Shouldn't it delete all the un-needed VM's?
Observation3: I need to cut down my AWS costs, so I have deleted all the un-needed 19 VM's from the EC2 console manually. But due to the auto scaling groups AWS again spawned up 20 more VM's. But surprisingly, when I now execute kubectl get nodes-- gives me 20 nodes back again.
Could someone explain this behavior which is pretty odd.

Hi @K7king,
regarding observ. 1: I have checked that all five auto scaling groups were created successfully on AWS and have been configured with the correct desired/minimum/maximum counts. Therefore, I'm very unsure what we/Gardener can do further about the situation? It has fulfilled the "contract" with AWS by successfully creating the resources there. I suggest to contact AWS and ask what's wrong with the one auto scaling group that does not produce any VMs.
Regarding observ. 2: You cannot delete VMs by executing kubectl delete node, that won't work. The current Gardener is not (at least not gracefully) able to scale down, however, the latest master version will support that as well as hibernating clusters completely or even updating them. Once deployed, the proper way to scale down would be to modify the autoScalerMin/autoScalerMax configuration in the Shoot manifest.
Regarding observ. 3: This behaviour is caused by the AWS auto scaling group - if you only delete the created VMs without changing the desired/minimum/maximum settings in the auto scaling group configuration itself, AWS will always replace terminated VMs with new ones (to have always as many VMs as the desired flag tells). To scale down manually, simply change this configuration on the auto scaling group. However, as said above, with the latest Gardener you should only do that by adapting the Shoot resource. You won't be able to manually modify infrastructure resources at it will reconcile its desired state every 10 minutes, discarding your manual changes.
It would be great and very interesting if you could get back to us regarding observ. 1. Let's keep the ticket open therefore.
Best regards, Rafael


Update README.md

more details for XSite example
fixes #106

do you mind adding the config1.yaml and config2.yaml? any new user seeing those commands may not understand where the configuration comes from.

@oraNod when you have time, can you take a look?

do you mind adding the config1.yaml and config2.yaml? any new user seeing those commands may not understand where the configuration comes from.

good Idea, I'll add it to the readme, too

@pruivo and @ryanemerson I've been thinking about this a little more and wonder if this readme is the right place for these xsite-details? This feels a little bit like the simple-tutorial that already uses the image. I don't wish to discourage any contributions but wonder if there is potent to bloat the readme, duplicate content, add maintenance overhead? Basically is this content being added to the right place?

First of all, thanks for the contribution @shivan much appreciated.
@oraNod I agree that this feels more like a tutorial of a specific use-case. The original idea of the readme was to just detail the images capabilities and config options.
I do think this content is of value though, so maybe we should create a docs dir that can be used as a home for tutorials/guides like this. We can then link to this from the main readme page. WDYT? I'm open to any suggestions here if you have something in mind.

Well, the cache creating can be removed because it isn't specific to the image (create a reference to the Infinispan docs?)
The map information is missing "Map the local port 7900 to the port defined in the configuration (7200). On Docker, for example, use -p 7200:7900" and should be in the README.
The rest, I'm OK to be someplace else (or create a link to the existing tutorial).


fix: correctly parse message format arg

Codecov Report

Merging #42 (86c63d8) into main (a6f6d8f) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main      #42   +/-   ##
=======================================
  Coverage   68.25%   68.25%
=======================================
  Files          10       10
  Lines        1131     1131
=======================================
  Hits          772      772
  Misses        359      359

Impacted Files
Coverage Δ

sarif-fmt/src/bin.rs
50.18% <100.00%> (ø)

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 a6f6d8f...86c63d8. Read the comment docs.


Error: Binary(NoSentinel)

review code, do some configure for electron fuses ?
here i lack of electron detail.
how to config electron.
please git comment

Hey there. Are you running the tool on a packaged Electron application? electron-hardener won't work on an unpackaged one.
A somewhat-easy way you could double-check is by opening what you believe to be the Electron app in your hex editor of choice and searching for the following string: dL7pKGdnNz796PbbjQWNKmHXBZaB9tsX. If that isn't anywhere in the binary, you have the wrong build kind.

thanks for answer. but turn to tauri, trouble is gone


Adding Seek buttons for Android

Adding Forward and Backward seek buttons for Android.
Fixes #615

If you can accomplish this for CupertinoControls, then I would accept this PR.
@brianegan any thoughts on this proposed change, since that would be nice if users were able to seek.

Please merge this PR, because Andriod need this feature

Please merge this PR, because Android need this feature

We can't merge changes without investigating and the changes are done which @diegotori requested. Please update your PR accordingly and we will test this :)

@Ahmadre if you have capacity, please review his changes by running them locally.
I wanna say that there were exceptions thrown due to the current library calling setState while in the middle of one.
Thanks.

LGTM @diegotori
Your change requests seems to be implemented. We merge this if you want.

@Ahmadre can you please submit a follow-up PR that fixes lint issues related to this one? Thanks.


WIP: bump Seldon ROCKs version 1.15.x -> 1.17.1 for release 1.8

This PR updates Seldon ROCKs from Seldon version 1.15.x to 1.17.1. The process has been thoroughly documented in the issue below and reviewer may consult it about details.
Closes #37

Closing in favor of PRs linked in issue #37


After moving config oof s3-box-s3 to separate repository display does not work correctly

Hi, I'm trying to add some sensors to my s3 box. However, I experienced some issues with changing the configuration yaml.
When I use
esphome.voice-assistant: github://esphome/firmware/wake-word-voice-assistant/esp32-s3-box-3.yaml@main

it works ok but I cannot add sensors
when I change this line to
esphome.voice-assistant: github://esphome/wake-word-voice-assistants/esp32-s3-box/esp32-s3-box.yaml@main

I can add sensors but the display is always turned on an max brightness and no images

It was my mistake I flashed for esp32-s3-box.yaml but I had to use esp32-s3-box-3.yaml


Can I use a clean Ubuntu image to make a control computer without downloading the img file?

Can I use this library and make a record on the computer myself

Yes, but it's not so easy to install everything required, that's why the ready image is made.

You can take some info in the simulator installation manual (https://clover.coex.tech/en/simulation_native.html), which is kind of a superset of the required software.

Your response speed is too fast, awesome! ! ! I just want to study the implementation of clover Python package in from clover import srv

There is no separate Clover Python library, the Python modules are generated by ROS while package building (from .srv and .msg files). Programs for Clover use rospy library to access Clover services and topics.

Let me study the .srv and .msg files of ROS. thank you for your reply


Consolidate publish_tf and enable_tf in launch/includes/realsense_r200_nodelet.launch.xml

This should just be a single variable "publish_tf", right now the behaviour is backwards from what is expected and the inclusion of "enable_tf" is confusing.
I have been using the launch file realsense_r200_rgbd.launch with publish_tf set to true from existing OpenNI compatible launch files and wondering why no tfs are being published from the nodelet.

@liz-murphy , I understand why you could get confused. But let me try to explain the reason why we have two flags "publish_tf" and "enable_tf":
This was done to follow the standards in freenect.launch, where "publish_tf" is set to true when the transforms are generated statically using "static_transform_publisher".
By default, the realsense package generates transforms from within the nodelet.
So when the "publish_tf" is set to true, we wanted to turn-off the default behavior of the realsense package and not generate transforms from within the nodelet, as the expectation/assumption was that the transforms will be generated statically. We have a comment in the realsense_r200_nodelet.launch.xml that states that.
"enable_tf" will be set to false only if "publish_tf" is true. Else it is set to true.
Ensures that the nodelet does not publish tf if the transforms are published statically.
Let me know if this helps.

@liz-murphy, Also, since you are using realsense_r200_nodelet.launch.xml, I would like to mention that you could change the resolution of the depth and color streams in "manual" mode. Please refer to the sample launch file manual launch.
We will update the realsense_r200_nodelet.launch.xml in the next release to show this option, in case users overlook the manual.launch example.

@reaganlo OK, my bad, I'm sorry. You are not providing a default static transform file though, so it is still odd to set publish_tf=true and have nothing published.
https://github.com/ros-drivers/freenect_stack/blob/edaf577f6db318ca12f2ac767619b2c6a971cff2/freenect_launch/launch/freenect.launch#L129
With respect to resolutions etc - you should be using a combination of dynamic reconfigure and/or ros params to load the configuration. It's just a more standard way of doing things and allows more transparency in remapping. It also means you can get rid of all those constants here https://github.com/intel-ros/realsense/blob/indigo-devel/camera/src/realsense_camera_nodelet.h#L78
I've made a start on that in my fork, needed it for compatability with our existing launch files on the robot.
https://github.com/liz-murphy/realsense/blob/dev/camera/src/realsense_camera_nodelet.h
https://github.com/liz-murphy/realsense/blob/dev/camera/src/realsense_camera_nodelet.cpp
Note that it is also a lot more convenient to set the resolution by enumerating the device modes in the dynamic reconfigure configuration and fetching the width/height etc in the code via
dev->get_stream_mode(strm, k, width, height, format, framerate);
which stops the user from setting an invalid combination.

@reaganlo if it helps with development I can send you the launch file we currently use to launch 2 cameras. Let me know if that'd be useful.

@liz-murphy, Yes, your suggestions are all valid. We are working on converting args to params and will have it part of the next release (next week). I was about to do a quick test on your pull request #8 .
I will post my comments on your pull request in that thread.
Yes, it would be great if you could send the launch files that you are using, so that we can understand the requirement better.
I will review this issue #9 with my team in the morning and decide how to resolve it. Meanwhile, if you feel it could be "closed" based on our discussion (i.e let publish_tf be set to false which would let the nodelet publish the transforms) , please feel free to close it :)

Hi, Does the publish_tf flag exist in the current version ?

No. The static transformations are published.

Is there any way to turn the static publishing off? In our case the camera is already in the robot URDF model and transforms are present there

We could add a flag in future versions, if it is a common demand. In the meanwhile, could you rename and ignore them?

Are you referring to using tf_prefix? If I use that it also renames the frame_ids of images and point clouds and then the transform between image frame and other frames is lost as it is not present in the tf tree

I see now what you mean.
It could take me a while to set a flag to disable publishing tf as there are more issues I'd like to address in the process. In the meanwhile, I think that if you disabled this line you'll get what you need.

Thanks @doronhi. I will go ahead and comment this line for now.

fixed in #843

@liz-murphy  Any other questions for this ticket? Looking forward to your update. Thanks!

@liz-murphy  Will close this if no other questions. Thanks!


feat: allow to explicitly disable helm rendering certgen resources

What type of PR is this?
feat: allow to explicitly disable helm rendering certgen resources
Which issue(s) this PR fixes:
Fixes #3501

@ShyunnY we need to move hmacSecretKey out of the certgen if we want to disable it.

@ShyunnY we need to move hmacSecretKey out of the certgen if we want to disable it in helm template.

@zhaohuabing
Sorry, I don't quite understand what you mean by hmacSecretKey. Could you please mark it out? Thank you very much  :)

@ShyunnY we need to move hmacSecretKey out of the certgen if we want to disable it in helm template.

@zhaohuabing Sorry, I don't quite understand what you mean by hmacSecretKey. Could you please mark it out? Thank you very much :)

@ShyunnY We can't disable certgen job since it also generates the hmac secret for OIDC.
https://github.com/envoyproxy/gateway/blob/371265969fe8b4246de3b78290a44d08ba8cb20a/internal/crypto/certgen.go#L157

@ShyunnY we need to move hmacSecretKey out of the certgen if we want to disable it in helm template.

@zhaohuabing Sorry, I don't quite understand what you mean by hmacSecretKey. Could you please mark it out? Thank you very much :)

@ShyunnY We can't disable certgen job since it also generates the hmac secret for OIDC.
https://github.com/envoyproxy/gateway/blob/371265969fe8b4246de3b78290a44d08ba8cb20a/internal/crypto/certgen.go#L157

I think I understand.
Is there a better way to solve this problem? Do you have any good ideas?

I think it's fine to keep this job here, but we should consider giving it a better name to clearly convey that it does more than generate certificates.

changing name will be a breaking change, and even if we do change name, whats an apt name ?
we could add helm doc strings highlighting that this job does 2 things - creates control plane cert and and the hmac secret

It seems that we need to highlight this in the Install with Helm and Control Plane Authentication using custom certs docs.
When users choose to set certgen: disabled, we must inform the corresponding warning message

changing name will be a breaking change, and even if we do change name, whats an apt name ?
we could add helm doc strings highlighting that this job does 2 things - creates control plane cert and and the hmac secret

It seems that we need to highlight this in the Install with Helm and Control Plane Authentication using custom certs docs. When users choose to set certgen: disabled, we must inform the corresponding warning message

We should leave the certgen job and just add helm docs to explain:

By default, certgen won't override existing certs generated by cert manager.
certgen also generates the HMAC for OIDC

changing name will be a breaking change, and even if we do change name, whats an apt name ?
we could add helm doc strings highlighting that this job does 2 things - creates control plane cert and and the hmac secret

It seems that we need to highlight this in the Install with Helm and Control Plane Authentication using custom certs docs. When users choose to set certgen: disabled, we must inform the corresponding warning message

We should keep the certgen job(won't add a certgen: disable option in the helm) and just add docs to explain:

By default, certgen won't override existing certs generated by cert manager or other third-party tools.
certgen also generates the HMAC for OIDC.

If certgen is disabled, OIDC will fail due to lacking of HMAC secret.

I may not be very clear about the design of OIDC. Can we let OIDC generate a default random value when HMAC cannot be obtained to solve this problem?
Similar to func generateHMACSecret() ([]byte, error)?

The HMAC generation is a one-time thing. The certgen job is the best place for it, or we need a separate job for it.

This is a difficult decision.
We would like to provide a certgen: disable Job to allow users to use custom certs. Using a new Job to generate HMAC to solve the problem of missing HMAC is a good idea, but it may increase complexity.
I would like to hear your opinions and see if we can find a balance to solve the problem :)

Hi  @ShyunnY I may miss something here, but AFAIK, the certgen job won't prevent users from using their own certs because certgen won't override these certs if they already exist.

I may not understand it very well, so what should I do next?
@zhaohuabing

hey @ShyunnY thanks for working on this one, after bringing this up in the community meeting today, we decided to not support disabling this job, to avoid any issues with the OIDC feature.

嘿@ShyunnY感谢您对此所做的努力，在今天的社区会议上提出这个问题后，我们决定不支持禁用这项工作，以避免 OIDC 功能出现任何问题。

@arkodg  That's okay! It's great to hear that the community meeting discussed this and came to a conclusion.
To sum up, I think we should add a little bit of documentation in doc: Highlight what certgen does, inform users to avoid manually disabling the certgen Job when using custom cert.

should that certgen explanation be moved to helm README instead https://gateway.envoyproxy.io/v1.0.1/install/api/ ?

Thanks everyone for the comments, I will close this PR


docs: CONTRIBUTING.md draft

Description of change
CONTRIBUTING.md draft
*reference

nestjs
vercel

Pull-Request Checklist

[x] Code is up-to-date with the main branch
[ ] There are new or updated unit tests validating the change
[x] Documentation has been updated to reflect this change
[x] The new commits follow conventions explained in CONTRIBUTING.md

please wirte description of change

LGTM


CustomLearningAdmin.aspx first use not working with Firefox

When running the manual installation procedure, going to CustomLearningAdmin.aspx leads to an error using Firefox. Works fine using Chrome. Not a big deal since there is a workaround then.
Error with Firefox:

No error with Chrome:

Category

[x] Bug

Version used for the portal: <IP_ADDRESS>
Firefox version: 68.0.2
Chrome version: 76.0.3809.100
Applied the following procedure. The errors occurs when browsing the CustomLearningAdmin.aspx page for the first time.

@FranckyC -- Based on the console it looks like for some reason firefox is blocking you from downloading the manifest files from the CDN. Based on Chrome working for you that doesn't seem environmental. I tested the same version of firefox with that same version of the web part and it's loading ok for me so I do not think there is a bug with the code/browser combination. That I guess leaves browser configurations? Please take a look at the network tab as based on the error messages that's the only path I can see right now to diagnosing the issue you're having.

Thanks @juliemturner I will dig into the network tab. Like I said, this is not a major issue. Maybe it is a wrong browser configuration on my side. Thank you for confirming me it works with the same combination on your side!

Ok understood. It was a Firefox fox plugin blocking some content ;) Thanks!


unknown directive 'application/octet-stream'

Hello,
When I add this into the nginx configuration in server block, the parser gives me unknown directive error:
    types {
        application/octet-stream mp4;
        application/octet-stream srt;
        application/octet-stream ass;
    }

exact error message: unknown directive 'application/octet-stream' on line 10, column 9
Is there anyway to skip this or even better make it parse this?
Thank you in advance.

As a workaround I did put them in a file and included them in the config.

You can use parser.WithSkipValidDirectivesErr to skip syntax checks, as in the following code snippet: parser.NewParser(filePath, parser.WithSkipValidDirectivesErr())

You can use parser.WithSkipValidDirectivesErr to skip syntax checks, as in the following code snippet: parser.NewParser(filePath, parser.WithSkipValidDirectivesErr())

Shouldn't these be parsed? I mean I got issues with this and using variables in server blocks, I eventually switched to crossplane and parsed the whole nginx file then got my needed directive arguments but this would be very better as crossplane parsed whole nginx file instead of individual files.


Always use workers under the hood to execute commands.

By default, the number of workers is set to the number of commands to be executed.
Using -p, one can set the number of workers to use.

Ok, now that issue #59 is closed, is something missing for this PR?

I guess I'll create a new PR. Anyway, I should rebase master first.


other languages possible?

The plugin is only in english right now, right?
Is there a way to implement other languages, like german?
or how difficult/much work is it?
I would be happy to provide the translation file, if that would help.

The plugin itself can be translated via: https://github.com/nystudio107/craft-recipe/tree/develop/src/translations
The content stored by the plugin can be in any language, just like any other Craft CMS field.


Unexpected token export

Hey! Fantastic library, first of all, this is so useful - thank you for it!
I'm getting an unexpected token export error when I use the library in Nuxt and wanted to see if there was any quick fix I was missing. To reproduce:

Clone and install this repo
npm run dev

Expected: See a blank page with an h2 reading "export bug demo"
Actual: See Nuxt's "Unexpected token export" error page
If this is something Nuxt-specific, I can do more digging, but I wanted to mention it and see if you've come across it before. In any case, thanks again for the excellent work!

Also, to clarify, the only relevant Vue template file is here - everything else is Nuxt defaults.

Found a fix! Instead of these lines:
import Phenomenon from 'phenomenon-px'

export default {
    mounted() {
        let fragment = ''

        const uniforms = {}

        const render = () => {}

        // kick loop
        Phenomenon(fragment, uniforms, render, this.$refs.canvas)
    }
}

I moved Phenomenon from an import to a require statement:

export default {
    mounted() {
        let fragment = ''

        const uniforms = {}

        const render = () => {}

        // here!
        const Phenomenon = require('phenomenon-px').default

        // kick loop
        Phenomenon(fragment, uniforms, render, this.$refs.canvas)
    }
}

Works as expected now!

Hey Sander, that's an odd issue I have not seen before, glad you found a fix! 🙌


Beaglebone AI-64 support

For the moment this depends upon https://github.com/balena-os/balena-beaglebone/pull/574


How to construct a runes-based transaction script for direct transmission without going through a Bitcoin full node?

How to construct a runes-based transaction script for direct transmission without going through a Bitcoin full node?

You would need to write a custom transaction contructing program using our ordinals crate.


OAK-10114 : created new query API with projections to limit memory us…

…age when fetching huge no. of records

@mreutegg @stefan-egli could you please review and share the feedback?

It would be good if the JIRA issue was more clear about the fact that a new DocumentStore is being introduced; and that API change should actually be described over there.

Other than moving the tests, this PR now looks good to me.


Update azure-maps-authentication.md

as these features are now in not in preview please change

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

@stevemunk
Can you review this PR?
IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

as these features are now in not in preview please change

Hi @rajparmar1004, I'm from Azure Maps team. SAS Token is in preview right now as an offering from Azure Maps.

thanks for letting me know @stack111
preview feature is not indicating in the portal in Azure maps service that's why created pull request.
CORS feature also in preview?

Yes CORS is preview too.


fail to pip install zipline with compile issue

Dear Zipline Maintainers,
Before I tell you about my issue, let me describe my environment:
Environment
mic mini with m2 pro micro chips, 64 bits
MacOS Sonoma 14.4.1
python 3.10.14 & 3.11.9
fail by "pip install"
package: zipline-1.4.1.tar.gz (5.4 MB)
error: subprocess-exited-with-error
× pip subprocess to install build dependencies did not run successfully.
│ exit code: 1
╰─> [177 lines of output]
Ignoring numpy: markers 'python_version == "3.5"' don't match your environment
Ignoring numpy: markers 'python_version == "3.6"' don't match your environment
thus, which package would support python. 3.10.x or 3.11?

Now that you know a little about me, let me tell you about the issue I am
having:
Description of Issue

What did you expect to happen?
What happened instead?

Here is how you can reproduce this issue on your machine:
Reproduction Steps

...
What steps have you taken to resolve this already?
...
Anything else?
...
Sincerely,
$ whoami

The really bad news:  Your Zipline issue is unlikely to be looked at by anyone.
The bad news: Unfortunately since Quantopian closed, the original Zipline stopped development.  This repository is no longer active.
The good news: Zipline was released as open source.
The even better news: A fork of Zipline called Zipline Reloaded is under active development by Stefan Jansen and several other amazing contributors:  https://github.com/stefan-jansen/zipline-reloaded


Fix aborts when an exception occurs in the execution engine

The peloton process dies, when there is an exception underneath the execution engine.  ExecutePlan would be able to handle this. This is reproducible with any test case with an constraint exception.
[Peloton output]
Exception Type :: Constraint
Message :: NOTNULL constraint violated on column 'b' : FIXME
terminate called after throwing an instance of 'peloton::ConstraintException'
  what():  NOTNULL constraint violated on column 'b' : FIXME
Aborted (core dumped)

@apavlo I think this should be assigned to the networking team, since this also happens with other exceptions. So, I change this to @ChTimTsubasa.


Export Private Key Functionality

Give users the option to export their private key as an encrypted file to backup elsewhere. An example entry point would be in More > Reveal 12 Words > Export Private Key Instead
Then the user would be prompted to save the file elsewhere.

Instead of exporting a file could we just allow the user to copy the private key? Similar to how it works on the wallet app today

i like it


about dataset

Thank you for your answer ~
As a novice, I would like to ask you another question. What are the functions of "blur", "sharp" and "blur_gamma" in training?
I wish you a happy life ~

I would like to ask when to input the blur_gamma  during the test or training?thank you!

I would like to ask when to input the blur_gamma  during the test or training?thank you!

You can use whatever for training but to compare with the results, I set the blur_gamma as the standard.

You can use whatever for training but to compare with the results, I set the blur_gamma as the standard.


Update media-modal-content.blade.php

fix view to work with youtu.be links.

Thanks for your improvement !


txn: Fix missing perf context tag for Flush and BufferBatchGet

What is changed and how it works?

Issue Number: ref #16291

What's Changed:
Add FLUSH command to storage metrics and check API version for Flush command

Related changes

[ ] PR to update pingcap/docs/pingcap/docs-cn:
[ ] Need to cherry-pick to the release branch

Check List
Tests

[ ] Unit test
[ ] Integration test
[ ] Manual test (add detailed scripts or steps below)
[x] No code

Side effects

[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[ ] Breaking backward compatibility

Release note

None

/test

/merge

/test


Interface compatibility with Distributions.jl ?

Hey,
Thanks for this great addition to the ecosystem !
From Distribution.jl, it seems like the first argument to the fit_mle function should be the distributions type and not an instance of the type :
julia> fit_mle(Gamma,rand(1000))
Gamma{Float64}(α=1.604973623956157, θ=0.3097630701718876)

julia> fit_mle(Gamma,rand(100))
Gamma{Float64}(α=1.6985042802071995, θ=0.31065614192888746)

julia> fit_mle(Gamma(1,1),rand(100))
ERROR: MethodError: no method matching fit_mle(::Gamma{Float64}, ::Vector{Float64})
Closest candidates are:
  fit_mle(::Type{<:LogNormal}, ::AbstractArray{T}) where T<:Real at C:\Users\lrnv\.julia\packages\Distributions\bQ6Gj\src\univariate\continuous\lognormal.jl:163
  fit_mle(::Type{<:Weibull}, ::AbstractArray{<:Real}; alpha0, maxiter, tol) at C:\Users\lrnv\.julia\packages\Distributions\bQ6Gj\src\univariate\continuous\weibull.jl:145
  fit_mle(::Type{<:Beta}, ::AbstractArray{T}; maxiter, tol) where T<:Real at C:\Users\lrnv\.julia\packages\Distributions\bQ6Gj\src\univariate\continuous\beta.jl:217
  ...
Stacktrace:
 [1] top-level scope
   @ REPL[14]:1

julia>

This is not really a problem for yo as you are free to overload this function as you want, and your interface actually makes a lot of sense since you exploit the guesses in your algorithm. But would it be possible to add methods following this convention, maybe with automatic guesses ? I have fit_mle bindings in Copulas.jl that assume this convention, and thus do not work directly with your package :(
Edit: I was trying to make a code example of what i would like, but I saw that mixures types do not include components types... More specifically, I would like to be able to type :
fit_mle(MixtureModel{Gamma,Gamma,Normal},data)
instead of
fit_mle(MixtureModel([Gamma(),Gamma(),Normal()],[1/3 1/3 1/3]),data)
Would that be possible ?
It would allow composability, as I am currently using :
using Copulas, Distributions, ExpectationMaximization, Random
X₁ = MixtureModel([Gamma(2,3), LogNormal(1,1)],[1/2,1/2])
X₂ = Pareto()
X₃ = LogNormal(0,1)
C = ClaytonCopula(3,0.7) # A 3-variate Frank Copula with θ = 0.7
D = SklarDist(C,(X₁,X₂,X₃)) # The final distribution

# This generates a (3,1000)-sized dataset from the multivariate distribution D
simu = rand(D,1000)

D̂ = fit(SklarDist{FrankCopula,Tuple{Gamma,Normal,LogNormal}}, simu) # works
# But how can i specify that i want a mixture for one of the variables ?

which, under the hood, calls fit_mle(Marginal_Type,marginal_data) on each marignals.

Thanks for the great issue! I was actually starting to wonder about mixtures of copula (or copulas of mixtures, as you wrote).
I believe we could make packages work togheter.
BTW, I just released v0.2 of the package with brand-new docs.
I address the type vs instance problem in Instance vs Type version section of the doc.
There is also a discussion in the PR#1670 in Distributions.jl I made. Main criticism from @devmotion was that it could add confusion. Maybe denoting the instance version with fit_mle! would be clearer (however one cannot mutate in place distribution, so it would not be true in place, this would also be confusing).
My point was that it would allow more flexibility and simpler writing (at the cost of a slight warning on what the function fit_mle does depending on the case).
As you noted: mixture models do not include the type of every component, so fit_mle(Type{mix}, y) cannot work.
Similarly, I noticed that ProductDistribution do not have types in it, so again fit_mle(Type{Product}, y) cannot work.
Base on these observations, using the instance version fit_mle(mix, y) (or product) made more sense.

For mixture, the input mix is used as initial state of EM algo, so it is 2 in 1.

I added this conversion line for
Interface compatibility with Distributions.jl
function fit_mle(g::D, args...) where {D<:Distribution}
    fit_mle(typeof(g), args...)
end

So first when encountering a g::Distribution, it will search for an instance version (for MixtureModels or Product), if it does not find it, it will fall back with the instance version thanks to previous definition.
Note that I did not define the same for the fit function fit(g::D, args...) (only fit). I could very well add it.
In your (great) package, you so far used the type convention which IMO for Copulas (and other multivariate) case is a bit heavy to write.
You could very well use the instance convention.
The following works in your (modified) exemple!
function Distributions.fit(D::SklarDist, x)
    # The first thing to do is to fit the marginals :
    @assert length(D) == size(x, 1)
    # One could put fit but EM works for with fit_mle only
    m = Tuple(Distributions.fit_mle(D.m[i], x[i, :]) for i in axes(x, 1))
    u = pseudos(x)

    # τ⁻¹(FrankCopula, τ) works but not τ⁻¹(FrankCopula{3, Float64}, τ)
    # I found https://discourse.julialang.org/t/extract-type-name-only-from-parametric-type/14188/20
    # to just extract the typename.
    # Again defining fit(C::ArchimedeanCopula, u) = fit(::Type{CT},u) where {CT <: ArchimedeanCopula} would simplify the writting
    C = Distributions.fit(Base.typename(typeof(D.C)).wrapper, u)
    return SklarDist(C, m)
end

D̂mix = fit(D, simu)

Note that I add to define the weighted fit_mle(LogNormal, x, w) (only fit_mle(LogNormal, x) is in Distributions.jl).
I believe this does the job
using StatsBase
function Distributions.fit_mle(::Type{<:LogNormal}, x::AbstractArray{T}, w::AbstractArray) where T<:Real
    lx = log.(x)
    μ, σ = mean_and_std(lx, weights(w))
    LogNormal(μ, σ)
end

(Note that StochasticEM() version only use unweighted version of fit_mle.)
Hence, IMO, it seems that for compatibility between EM.jl and Copula.jl, it is Copula.jl that needs to simply add Distributions.fit(D::SklarDist, x) as shown here. I believe it should not have any consequences on your existing functions.

I apologize for not reading your documentation and posting without restraint. Thanks for the clear explanation and the link to the PR, I can appreciate that you already put a lot of efforts into this;
I agree with you that, for mixtures, the fit_mle(D::Dist,...) makes more sense than the fit_mle(D::Type,...) version, since, as we noted, (1) mixtures types do not include component types and (2) EM fitting uses guesses for parameters.
I also agree with the PR reviewers that this adds different behaviors to the function. I see two outputs:

Your PR gets merged, I adapt the Copulas.jl code to accept this new version as you proposed. Low hanging fruit (now that you already did all the work and wrote the PR).
We parametrize MixtureModel. Harder but I think better. Let me make the case for it.

Plan:
MixtureModel become MixtureModel{T1,T2,T3} for small mixture, potentially NMix{T,N} like we have NTuple in base to represent a mixture with N components all of type T.
Issues
What if you want 100 Gaussians + a dirac ? Maybe via nesting as MixtureModel{Dirac,NMix{Gaussian,100}} ? This is unclear to me yet.
Also, this might be breaking...
Implications
Implementations of fit_mle that require guesses could pass these guesses as later parameters as fit_mle(MixtureModel{T1,T2,T3},guesses::MixtureModel{T1,T2,T3},...). Yes, this looks redundant, but you could also bind once and for all fit_mle(X::MixtureModel,..) = fit_mle(typeof(X),X,...) to keep your infrastructure in place.
A good argument
Another argument for this change of parametrization on the MixtureModel type is that the ProductDistribution already does it :
julia> typeof(product_distribution(Gamma(),Normal()))
Distributions.ProductDistribution{1, 0, Tuple{Gamma{Float64}, Normal{Float64}}, Continuous, Float64}

julia>

Although, the current type structure is kinda clunky (what are these 1 and 0 ?). Expanding fit_mle to product distributions should therefore be possible for both interfaces.
Conclusion
Whatever gets decided, I will adapt Copulas.jl accordingly.
Post-scriptum
I agree with you on the fact that abusing the meaning of ! using fit_mle!(D::Dist,...) is probably not the best solution as, as you noted, distributions are immutable.

Thanks for your comments.
Since I just noticed (thanks to you) that Product have been updated, I now agree more with what you propose.
In particular, the part fit_mle(X::MixtureModel,..) = fit_mle(typeof(X),X,...)!
I think now my main point against is that if one wants to use an algorithm that implies using fit_mle for generic distributions (like EM) the arguments would have to change depending on the distribution:

fit_mle(Exponential, y)
fit_mle(Categorical, k, y) This one can be transformed as fit_mle(Categorical, y) using that, but in some unlucky scenarios where the last category is not drawn maximum(y)<k. To fix that Type wise, one might define some subtype like Categorical{k}?
Now we also have fit_mle(MixtureModel{T1,T2,...}, guesses, y)

These are 3 ways to use the fit_mle depending on the distribution, so for generic algo it does not seem convenient.
Whereas with the instance formulation all write fit_mle(dist, y).
Coming back to your two scenarios:

It does not sound like it will be merged. Maybe I need to make a stronger case.
Should we open a PR in Distributions for that? Why would it be breaking?

About Categorical and type parameter conventions.
Some naming conventions so that we understand each others :

By type parameters I mean exactly what julia means , the T in MyType{T}
By distribution parameters, I mean what statiticiens mean : the (\mu,\sigma) of a gaussian.
By parameter field I mean the set of all possible parametrisation of a distribution. For the gaussian, this is something like $\mathbb R \times \mathbb R_+$.

I think in a dream world, I would like the parameter field to be clearly defined from the type parameters. This is in fact what is implicitly supposed by Distributions.jl's docs at this page:

[ The statement d = fit(D,x) ] fits a distribution of type D to a given dataset x, where x should be an array comprised of all samples. The fit function will choose a reasonable way to fit the distribution, which, in most cases, is maximum likelihood estimation.

Let's deal with the Categorical case as an exemple. The definition of categorical in Distributions.jl's code  is :
const Categorical{P<:Real,Ps<:AbstractVector{P}} = DiscreteNonParametric{Int,P,Base.OneTo{Int},Ps}

function Categorical{P,Ps}(p::Ps; check_args::Bool=true) where {P<:Real, Ps<:AbstractVector{P}}
    @check_args Categorical (p, isprobvec(p), "vector p is not a probability vector")
    return Categorical{P,Ps}(Base.OneTo(length(p)), p; check_args=check_args)
end

Categorical(p::AbstractVector{P}; check_args::Bool=true) where {P<:Real} =
    Categorical{P,typeof(p)}(p; check_args=check_args)

function Categorical(k::Integer; check_args::Bool=true)
    @check_args Categorical (k, k >= 1, "at least one category is required")
    return Categorical{Float64,Vector{Float64}}(Base.OneTo(k), fill(1/k, k); check_args=false)
end

Hence, the number of categories k is NOT part of the type parametrization, and thus can be considered a distribution parameter of the distribution to be derived from the data, which is what fit(Categorical,y) is currently doing if I read you correctly. The parameter field of Categorical is thus $\mathbb N$ for the parameter k times the probability simplex of size k.
If we introduce Categorical{k}, the number of categories of this distribution would not anymore be a distribution parameter but rather a type parameter and thus fit(Categorical{k},data) should respect it even if data does not contain the maximum possible value.
The version fit(Categorical, k, data) feels like it should not exist. I thus strongly agree with you that having one common interface fit(Distribution{EnventuallyWithTypeParams},data) is the way to go. If you want k not to be a distribution parameter, make it a type parameter. It just looks like the distribution we want, here Categorical{k}, does not exists yet.
About MixtureModel
Same thing as for Categorical : The fit(MixtureModel,data) is clearly dumb, as there is no way to infer the parameter field from the type and (absent) type parameters MixtureModel. On the other hand, in fit(MixtureModel{Gamma,Normal},data), we can infer the parameter field from the type, and thus it should work.
About Your interface and going forward.
Yes, defining in your package fit_mle(X::MixtureModel,..) = fit_mle(typeof(X),X,...) would probably be enough to keep your interface if and only if we make a strong case on Distributions.jl to change these stuff (namely, change mixtures so that their parameters fields are inferable from their type parameters, and maybe a few other distributions as well ?).
If you agree with that, I could try making the PR myself. Or maybe an issue first to ask their opinion before ?

If you agree with that, I could try making the PR myself. Or maybe an issue first to ask their opinion before, linking to this discussion ?

Yes, thank you!
About the initialization, other distributions could benefit from the notation fit_mle(typeof(X),X,...) to warm start.
Basically, all distributions where the MLE is not known explicitly and where numerical optimization is needed, i.e. most distributions.
For example, the Gamma distribution needs a starting point for the $a$ parameter. Currently, the $a_0$ is given as a kwargs.


[feat] : Add Login Signup with email

What would you like to share?
@punyakrit
we can add login form somethign like this
and store user info in localstorage or databasse

Additional information
it will help user to provider other login types
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

we are following google auth signup its better for now dont require custom signup option


Include math.Floor and math.Ceil in builtin library.

In #727, I suggested pouring all of Go's math library into a builtin library.  It was suggested, instead, that these math functions be implemented directly in Elvish.  This is a plausible idea, and I have started exploring/attempting that.  However, the floating-point floor and ceiling functions (and more generally, floating point modulo) I think cannot be trivially/efficiently implemented in Elvish, because they rely on the underlying bitwise encoding of double-precision floating points.
Thus I think it would be nice to do one (or both) of two things:

Include floor and ceil (and maybe round, if we're feeling excited) in the builtin library.
Include floating-point modulo, i.e. math.Modf, as a builtin arithmetic operator.  I might propose that this operator be name %f, and be used like %f 8.3 3 (outputs 2.3).
Include "truncated" (floored) division, a la Python.  I propose this operator be named //, just as in Python.  Then it would be used as // 8.3 3 (outputs 2).

The last suggestion is especially appealing to me, because it's, I think, the most minimal solution, and and everything else can be implemented using it.

Actually, better yet, I propose that the builtin modulo be replaced by a floating-point modulo.  But I also still kinda like the idea of a // operator.

I agree. Let's put floor, ceil and round in a new math module, teach % how to deal with floats and add //.

A math: namespace was introduced by commit 2d8a045. Also, my PR #937 introduces the floor and ceil commands as part of resolving issue #727.  So it would appear this is no longer an unresolved issue.

@xiaq, This can be closed now that PR #937 has been merged.


z-index of Bars

Is there an specific reason that scrollbars don't have any z-index value? I think they should have high z-index values to ensure they are visible.

You can customize every element to your needs. See: https://github.com/malte-wessel/react-custom-scrollbars/blob/master/docs/customization.md

Hi Malte, first of all I really love this library. Thanks for the amazing work.
Regarding to ghkno's question, I wonder if there is a way to customize the thumb style on top of the out-of-the-box element. I understand each element is fully customizable because we have full control over the rendered element. On the other hand, you've already done an amazing job giving us a great place to start. In most cases, we just need to tweak a little bit from the existing code and rewriting the entire render function and handling the scroll position manually seems to be an overkill.
Would you be open to allowing customizers to inject styles or classNames onto the existing thumb element?