Installing tectonic for AWS - Not completing

Issue Report Template
Tectonic Version
tectonic 1.4.5
Environment
Windows 7 Enterprise
Intel i5, 8 GB RAM, 64-bit
What hardware/cloud provider/hypervisor is being used with Tectonic?
AWS
Expected Behavior
Actual Behavior

Reproduction Steps

...
...

Other Information
Feature Request
Environment
What hardware/cloud provider/hypervisor is being used with Tectonic?
Desired Feature
Other Information

Hi @tnkumar, thanks for filing this issue! To help you troubleshoot, have you checked to make sure that you have the correct privileges for your AWS credentials?

Yes @mfburnett . I am the root administrator for the AWS account. Any other suggestions?

I had the same problem with the aws region eu-west-2. It seems that the regions are hard coded in kubernetes and london will be added with version 1.5.

I ran into this problem, see #19
You should be able to just scp the assets zip, explode it and run bootkube start. It appears to be waiting for the apiserver to start, but that's done by bootkube.

It is not actually waiting for the Kubernetes control plane, it waits for the Kubelet to be running, as resolved behind the ELB. The messaging was wrong. You can connect to your EC2 instances and verify the kubelet service is running. If that's ok, check the ELB or local network restrictions. Skipping over the validation can mask other issues.

Hi ... I am trying to install tectonic on bare metal, however I don't know how and where to run the matchboxes and get the informations for the "matchbox address" panel. Thanks

@Nourid5 this issue is about AWS. Please open a separate issue.

Hi @tnkumar, @naphthalene, and @webner - we recently released Tectonic v1.5.4-tectonic.1, which makes a bunch of improvements over Tectonic 1.4.5, including removing the scp step and adding HA by default for AWS (and bare metal). You can give it a try here to see if this resolves some of the issues you were having: https://coreos.com/tectonic/releases/

Going to close this issue due to inactivity. We recently released Tectonic v1.6.2-tectonic.1 with a ton of improvements - would recommend you all check it out! Feel free to open back up this issue with any comments or additional questions.


Polish ug

Codecov Report

Merging #229 (0b5db23) into master (9952ab9) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #229   +/-   ##
=========================================
  Coverage     69.08%   69.08%
  Complexity      759      759
=========================================
  Files           116      116
  Lines          2387     2387
  Branches        281      281
=========================================
  Hits           1649     1649
  Misses          621      621
  Partials        117      117

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 9952ab9...0b5db23. Read the comment docs.


possible to make a gitea version that's focused purely on security?

Feature Description

how secure is gitea for putting it out online?
possible to provide a security hardened military grade version of gitea to be put front facing online?

Screenshots
No response

Please elaborate exactly what you mean by that.
Also, why should there be security hardened military grade version?
Why shouldn't all instances have the same security features?

It really depends and impossible for anything to be pure secure.
Golang has bugs (then Gitea needs to use the fixed Golang to compile). Every software you are using including Linux kernel could have security bugs.
For Gitea, every problem in security report gets fixed in first time. And if you'd like to help, you could also to audit the source code and help for the security related problems.

@wxiaoguang @delvh i see plenty features of gitea and i'm just wondering if we can have a minimal version that has been thoroughly security audited and is extremely hardened because there are too many hackers and cybersecurity issues in the world today.

TBH I do not think it is possible for Gitea at the moment. There are a lot of dependencies between these features, and it's impossible to "partially" build Gitea.
For minimal "git" requirement, a SSH server is enough, then only the Linux & SSH & GIT code needs to be hardened.

I have a pretty hardened Gitea server. As stated in previous comments, defense-in-depth is what matters more than a single program/process being secure, since there are always bugs and even sandboxes can have bugs. It's all about what you're trying to protect and what you're willing to sacrifice in terms of usability and performance etc.
Some starting points:

Enforce firewall rules for fine-grained incoming and outgoing connections, blocking both directions by default unless rules are explicity added
Enforce server ciphers to take priority over client ciphers in your web server
Enforce TLS 1.3 is the only protocol used
Set the 3 TLS 1.3 ciphersuites to TLS_AES_256_GCM_SHA384:TLS_AES_CHACHA20_POLY1305:TLS_AES_128_GCM_SHA256
Ignore invalid headers
Enforce HSTS
Enforce specific key sizes for SSH keys, preferring modern ED25519 over becoming-deprecated RSA
Sign all commits and tags via SSH
Avoid PGP/GPG (it has many issues and security researchers such as myself wish it would just go away; see here, here, and here)

The most important rule of security is you are never unhackable. 100% security does not exist, but the more layers you have, the closer you can get to 99.99%.

We close issues that need feedback from the author if there were no new comments for a month. :tea:


Datepicker won't update when model changes after it ready

Datepicker initialize currDate when it ready, and won't update with model after that.

Add a watcher will fix it.
  watch: {
    value(value) {
      this.currDate = this.parse(value) || this.parse(new Date());
    }
  }


Create temporary inventory when execute a playbook

Create a temporary inventory and add hosts to it before job execution in the preprocess step.
Delete the temporary inventory after job execution in the postprocess step
Every job template is created with default inventory Demo Inventory which contains localhost. Demo Inventory comes with tower installation. We can change it if the embedded tower will configure a formal localhost inventory.
Temporary is created with default organization which has id = 1. Again we can change it if the embedded tower will configure a formal default organization.
Will skip the temporary inventory creation/deletion if the playbook runs at localhost.

@gmcculloug @bdunne @syncrou

👍 Looks good to me (once it can get to green.)


DOC Overlay Network Required TCP/UDP Ports + Cluster Options

Need to document the exact  required TCP/UDP ports to be open in order for overlay networks to work:
engine port (e.g TCP 2376),
vxlan: UDP 4789
serf: TCP + UDP 7946
key-value store ( e.g for consul TCP 8500)
Additionally, in Understand Docker container networks
need to edit --cluster-advertise=HOST_IP and make it --cluster-advertise=HOST_IP:ENGINE_PORT or --cluster-advertise=HOST_INTERFACE:ENGINE_PORT

Hi!
Please read this important information about creating issues.
If you are reporting a new issue, make sure that we do not have any duplicates already open. You can ensure this by searching the issue list for this repository. If there is a duplicate, please close your issue and add a comment to the existing issue instead.
If you suspect your issue is a bug, please edit your issue description to include the BUG REPORT INFORMATION shown below. If you fail to provide this information within 7 days, we cannot debug your issue and will close it. We will, however, reopen it if you later provide the information.
This is an automated, informational response.
Thank you.
For more information about reporting issues, see https://github.com/docker/docker/blob/master/CONTRIBUTING.md#reporting-other-issues

BUG REPORT INFORMATION
Use the commands below to provide key information from your environment:
docker version:
docker info:
uname -a:
Provide additional environment details (AWS, VirtualBox, physical, etc.):
List the steps to reproduce the issue:
1.
2.
3.
Describe the results you received:
Describe the results you expected:
Provide additional info you think is important:
----------END REPORT ---------
#ENEEDMOREINFO

@moxiegirl @mavenugo This was an issue when running on AWS, but could apply to other situations
Thanks @nicolaka

Got close on my first try but missed the VXLAN port.  +1 :)


Use URL and FileBlob/WebBlob to load binary data in Inference API instead of Blob

The Inference API use Blob or ArrayBuffer as argument for the request who result in loading huge file into RAM before sending them to the API.
We should rather use the new FileBlob or LazyBlob as for the commit method of the Hub API.
I already started to externalize the first version of the FileBlob and I could publish the WebBlob as well as the createBlob utility method
I would also understand if you want to:

keep the package in the huggingface namespace
create a shared package between hub and inference package in this monorepo.

I think publishing the package is neat and you did most of the work, so definitely go ahead! It'll be useful to the community / nice to show off :).
Still, for our usage we'll want to keep it in an internal local internal package.

This work was teamwork and I had good advices on a topic I rarely encounter ;-)
For our matter, I suggest to create an internal package then (if later I have more time then I will publish)
Shall I create a packages/shared for the code shared between both libraries?

I guess packages/utils? Taking inspiration from https://github.com/typescript-eslint/typescript-eslint/tree/main/packages/utils
Let's do this after merging the two PRs (web blobs & VCR), just in case it interferes

Sounds good :+1:
Do we really need to publish it as a separate package on NPM?
The alternative would be to import the classes directly from a relative path. Additional chunks will be added for the code coming from the packages/utils package

Yes I don't want to add it as a separate package, either importing the files directly works, or changing the tsup config (or associated rollup config)

:100: agree with KISS :+1:
I wil try something

Note that you can add internal dependencies in pnpm with workspace:*, just they will appear in the release's package.json by default

And if it's too much headache, a shared folder like you suggested is :+1:

Note that you can add internal dependencies in pnpm with workspace:*, just they will appear in the release's package.json by default

I'm not sure to understand what you mean.
For instance I tried to create a new package @huggingface/utils and then import it in hub but then the generated code still had my import, expecting it to be published on NPM as well.


javadoc fix javadocs

the javadocs are currently broken

fixed see https://opensource.adobe.com/aio-lib-java/


Support leaving a contractless agreement

What is included in this request?

[X] Bug Fix
[ ] New feature
[ ] Changes to an existing feature
[ ] New extension
[ ] Changes to an existing extension

:tada: This PR is included in version 1.28.0-dev.35 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.28.1-dev.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

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


change rake default and dkver

@windwiny Awesome, I haven't done this just because I don't have a good reason to but it's a good idea.


Add searchbox to herobox

Closes https://github.com/scylladb/sphinx-scylladb-theme/issues/466
Preview: https://www.loom.com/share/fa2c87ac70e54d038abfeaefb4f63191
How to test this PR

Make the docs with "make preview".

Open http://<IP_ADDRESS>:5500/examples/hero-box/#hero-box-with-search-box

Test the component.

To enable the searchbox on Scylla docs site, I'll submit another PR adding the :search_box: option to the directive hero_box once we release a new version of the theme.

The search is working as expected.


Download directory not checked for existence

The Issue
When changing the download directory string, lbry-app seems to assume it's a valid path.
Steps to reproduce

Go to the settings page
Append something to the download directory string (for instance "/LBRY")
Try to watch something

Expected behaviour
The full path should be checked for existence. If it fails, throw a GUI visible error in lbry-app and possibly ask if the directory should be created.
Actual behaviour
An error is printed on stdout if launced from a terminal, otherwise the stream silently fails with either the message "Requesting stream" or, on a single for me non-repeatable occasion, something about having acquired 0 blocks out of the total amount.
System Configuration

LBRY Daemon version: 0.9.1
LBRY App version: 0.9.1
LBRY Installation ID: A1tVXWpgt4vqV9eUZz7Q
Operating system: Linux (Linux-4.11.0-041100rc3-generic-x86_64-with-Ubuntu-16.04-xenial)

Anything Else

Optionally check for write permissions by creating a dummy file?

The app now uses a select folder dialog.


Kill locust slaves when test duration for test is completed

Description of issue / feature request
I am using locust in cluster mode with 1 master and N slaves with UI. Locust has capability to stop test execution when it reaches to time duration. But when test execution is completed, I would like to know if locust has feature to destroy whole cluster (including its master and slave processes). That will help test env to be clean.
Expected behavior
Kill Locust master and slave processes from test env.

Environment settings (for bug reports)

OS: Linux
Python version: 3
Locust version: Latest

Steps to reproduce (for bug reports)
Start Master and N slaves with some time duration. Once your time duration is completed, try to search locust master and slave processes. You will see them in your test env.
TBD - example code appreciated

Should be fixed now.


Fix typos in Soft-delete docs

Fix some typos in the documentation for Soft-delete will be enabled on all key vaults.

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

Looks good; thanks.


Enhancement: Promoting as an Admin on Speakers Page

This PR

Added a promoteAction method on AdminsController
Updated SpeakersController's index method for finding Admin grouped users.
Updated speaker/index template for showing Admin Grouped users or their promoting links.
Updated WebGateWayProvider for define promoting URL

Thanks @emir


AttributeError: 'module' object has no attribute 'pdtLocale_icu'

Get this error when running tests.
Was able to "fix" by reverting to
from .pdt_locales import load_locale

pdtLocales = {
    'en_US': load_locale('en_US'),
    'en_AU': load_locale('en_AU'),
    ...

Fix only worked for some tests.
Using python 2.7.10

Hi Phoebe, would you please provide some context as to where that code was added, what it was originally, and in which version of parsedatetime you noticed the problem?

I'll delete this in a minute.  I have a fork which comments out the force addition of . and - to the date separators and in syncing with your current master I have clearly made a mess!  Much quicker to start with a clean fork and change one line of code!

No problem, parsedatetime version 3.0 is underway and once the new test suite is complete it will be time to move on to fixes like #152 based on your original request.

I look foward to it.  Can I make a request?

Certainly, it's a good time to report any bugs or feature requests. For 3.0 it's unlikely that any major new features will be added unless pull requests are submitted since there are a lot of bugs to fix, but if you have an idea go ahead and make a new issue to share it.

Will do.

Workaround: pip install 'parsedatetime==2.5'


Multiple code spans with HTML comments does not working properly

It seems that using multiple code spans with HTML comments isn't working properly. Using a single code span with a HTML comment is working fine though. Consider the following examples.
A single code span with a comment at the end of a paragraph works fine.
foo `<!--comment-->`

✅ (document (paragraph (html_tag)))

A single comment at the beginning of a paragraph works fine as well.
`<!--comment-->` foo

✅ (document (paragraph (html_tag)))

However, having two code spans with HTML results in incorrect results.
`<!--comment-->` foo `<!--comment-->`

❌ (document (paragraph (html_tag) (code_span (code_span_delimiter) (code_span_delimiter)) (html_tag)))

I'm not experienced with writing Tree-sitter parsers, so I'm unsure how the tree should look when parsing the last example.
However, I think it's supposed to look like this:
(document (paragraph (html_tag) (html_tag)))

This is fixed with the most recent commit


Is it compatible with arch linux ?

To my knowledge, there is not specific reason why it would not be compatible with archlinux.

I am trying to install it from aur and its exiting with error code 1

this is an output of $ yaourt -S xeus

and at last i got this
100 13593  100 13593    0     0   9426      0  0:00:01  0:00:01 --:--:--  9419
  -> Found Findcppzmq.cmake
  -> Found Findcryptopp.cmake
  -> Found FindZeroMQ.cmake
  -> Found package.patch
==> Validating source files with sha256sums...
    xeus-0.18.1.tar.gz ... Passed
    zmq_addon.hpp ... FAILED
    Findcppzmq.cmake ... Passed
    Findcryptopp.cmake ... Passed
    FindZeroMQ.cmake ... Passed
    package.patch ... Passed
==> ERROR: One or more files did not pass the validity check!
==> ERROR: Makepkg was unable to build xeus.
==> Restart building xeus ? [y/N]
==> -----------------------------

We are not the authors of the arch linux package, which was done by a third party. This appears to be an issue with the archlinux package.

@SylvainCorlay  I got error while building from source
$ git clone https://github.com/QuantStack/xeus-cling.git
$ cd xeus-cling
$ mkdir build && cd build
$ cmake ..

You need to install the dependencies (xeus, cling, all the dependencies of xeus). If the package of xeus-cling form your distrib does not work, we highly recommend to install xeus-cling with conda rather than from source.

@JohanMabille  the native installation on arch linux fails so  i am using anaconda now


GetMeetingOptions fails with json unmarshal error

When using the package a commit 933a2eac3d1fc4de1f7d7de14159918042fb45d5. I attempted:
_, err := client.GetMeeting(zoom.GetMeetingOptions{MeetingID: "someID"})

The error returned was: json: cannot unmarshal string into Go struct field Occurrence.occurrences.occurrence_id of type int. I believe the occurrence ID should be a string instead of an int.

Thanks!


Gratipocalypse

Tracking consequences of the catastrophe, please add you own.
https://github.com/exercism/exercism.io/commit/ab9d3f2b1ca7ff83916566a82dd408ee0b8d937e

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/18346125-gratipocalypse?utm_campaign=plugin&utm_content=tracker%2F85909&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F85909&utm_medium=issues&utm_source=github).

No other consequences have been posted in a long time, closing this..


Completion is available inside bindings idents

When invoking code completion inside complete binding ident that is not pattern:

When binding is incomplete completion context is invalid as desired.

In some situations completion is not available inside binding idents, e.g. in parameter definitions:

Fixed via https://github.com/Microsoft/visualfsharp/pull/5083


Integration don’t remember colour temperature

Some devices (Xiaomi bedside lamp2, yeelight gu10 bulb color, maybe some other)
don’t remember color temperature, every time I open entity, color temperature settings reset. Device only remember rgb color
https://user-images.githubusercontent.com/27709949/196028027-ecdd6f4d-b2b1-4f48-8373-8336851324bd.mov

Show me the entity attributes.

Shure!

unavailable ?

No idea, man?
It seems that color_mode is always hs and dont't switches to color_temp

Still no changes. Device only remember color, but not color temperature

Fixed in https://github.com/al-one/hass-xiaomi-miot/commit/5cfe22f88a5059592d0f578b99af639a621fc02a


Fetch tokenizer from onnxruntime-extensions

With this PR, we depend on onnxruntime-extensions for the tokenizer instead of maintaining a duplicate copy inside ort-genai.

Thank you for the review. :)


added Mojo:US/Eastern

fix https://github.com/SickRage/sickrage-issues/issues/812

Thanks for the pull request! Before a real human comes by, please make sure your PR has all of the below criteria checked

[x] Give a description on what the PR is for.
[ ] Make sure your PR is based on the DEVELOP branch
[x] Don't send big changes all at once. Split up big PRs into multiple smaller PRs that are easier to manage and review

Please make sure you also read contribution guide and followed all of the steps.
Thanks!


Properly handle end of session

When a session ends, the WebGL app should return to a 2D usable state. Now, the canvas is black

unity scene freezes on end of session on android, i tried reloading scene after xr session ends, doesnt seem to work...
unity 2019.4

I know, all cameras are deleted, it should not

Corrected in last commit https://github.com/Rufus31415/Simple-WebXR-Unity/commit/e60b96d69689a0025deba7163d2ea2467b3848da


Vim9: wrong error message when passing unexpected type of argument to function

Describe the bug
In Vim9 script, passing an unexpected type of argument to a function called at the script level prints a wrong error message.
To Reproduce
Write this in /tmp/t.vim:
vim9script
def Func(n: number, s: string)
    eval 1
    eval 2
    eval 3
enddef
Func(123, [])

Start Vim like this:
vim -Nu NONE /tmp/t.vim

Run this Ex command:
:so%

E1012 is raised:
Error detected while processing /tmp/t.vim[7]..function <SNR>1_Func:
line    3:
E1012: type mismatch, expected string but got list<unknown>

The error message looks wrong for 3 reasons:

the error does not occur in the body of Func(); it happens at the script level (where the function is called)
it does not occur on line 3 in the body of Func(); it happens on line 7 in the script
the message does not tell us which argument in the function call has an unexpected type

Expected behavior
E1013 is raised (not E1012).
The error message does not tell us that the error was raised while processing Func(), but that it was raised while processing the script where Func() was called (i.e. ..function <SNR>1_Func should not be in the stack trace):
Error detected while processing /tmp/t.vim[7]..function <SNR>225_Func:
                                             ^----------------------^
                                                        ✘

The error message tells us that the error comes from the line 7 in the script, because that's the address of the line where Func() is called with an unexpected type of argument.
And it tells us that the argument with the unexpected type is the second one.
E1013: argument 2: type mismatch, expected string but got list<unknown>
       ^--------^

All in all, I would expect this error message:
Error detected while processing /tmp/t.vim:
line    7:
E1013: argument 2: type mismatch, expected string but got list<unknown>

Environment

Vim version: 8.2 Included patches: 1-1520
OS: Ubuntu 16.04.7 LTS
Terminal: XTerm(358)

The error messages do not include the column or argument number anywhere.  We still have errors that say nothing more than "invalid argument".  It's a bigger task to make this work better and it's not directly related to Vim9 script.  So let's make that a todo item.

The error messages do not include the column or argument number anywhere.

If the same error is raised from a :def function, then Vim gives the number of the argument.
vim9script
def Func(n: number, s: string)
    eval 1
    eval 2
    eval 3
enddef
def OtherFunc()
    Func(123, [])
enddef
OtherFunc()

E1013: argument 2: type mismatch, expected string but got list<unknown>
       ^--------^

It's a bigger task to make this work better and it's not directly related to Vim9 script. So let's make that a todo item.

Understood.  I just thought it was worth reporting any difference between an error message raised at the script level, and the same error raised from a :def function.

The error messages do not include the column or argument number anywhere.

If the same error is raised from a :def function, then Vim gives the number of the argument.
vim9script
def Func(n: number, s: string)
    eval 1
    eval 2
    eval 3
enddef
def OtherFunc()
    Func(123, [])
enddef
OtherFunc()

E1013: argument 2: type mismatch, expected string but got list<unknown>
       ^--------^

Ah, that's probably the only one.

It's a bigger task to make this work better and it's not directly related to Vim9 script. So let's make that a todo item.

Understood.  I just thought it was worth reporting any difference
between an error message raised at the script level, and the same
error raised from a :def function.

I'll just do this one, since it has enough code in common.
--
You were lucky to have a LAKE! There were a hundred and sixty of
us living in a small shoebox in the middle of the road.
/// Bram Moolenaar --<EMAIL_ADDRESS>-- http://www.Moolenaar.net   \
///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \
\\  an exciting new programming language -- http://www.Zimbu.org        ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///


[CELEBORN-1298][FOLLOWUP] Support Spark2.4 with Scala2.12

What changes were proposed in this pull request?
Why are the changes needed?
sbt is not tested in spark2.4 scala2.12 environment.
Does this PR introduce any user-facing change?
No
How was this patch tested?
GA

Thanks, merged to main(v0.5.0)/branch-0.4(v0.4.1).


ProtectedData: make unit test windows-specific

There is no (and seems won't be) managed implementation for ProtectedData in CoreFX. So related xunit tests are actual only for Windows. To avoid failures on non-windows targets I added [PlatformSpecific(TestPlatforms.Windows)] attribute. That's suggested in https://github.com/mono/mono/issues/9395.

I am fine to merge as-is it but could you try to upstream it as well

Okay, I'll send this change to upstream too, thanks.


Fix MiniEngine shaders for DXIL compat.

Rename 'ConstantBuffer' to 'MainConstantBuffer'
Fix variable-hiding warnings
Fix bitwise operator precedence warnings
Guard against negative operand to pow() with saturate()

The last point was committed separately by @stanard, but was in my change before rebasing, which is why it's still in the change description.

@tex3d,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

Sorry, it's a bit stale now, and I already renamed the cbuffers to something other than "ConstantBuffer".  W.r.t. operator precedence warnings, I'd prefer to disable the warning explicitly because operators have an order of precedence for a reason.


DCOS-37721: Latest PR 2

what happens, if you update a file which is cached for ages? -> added git sha to filename -> had to remove "force upload" flag in order to prevent overwriting of the file.
cleaned up the scripts after ee changes…

I spun up multiple clusters from this branch and it worked… lets see what breaks this time once its on master… 🤯

LGTM! Please squash 3880a92a018246390821e4b2425ac101cfd9de9c and before merging 5aea0a098f748861081a31973017b15cbc9f0756

@DanielMSchmidt @orlandohohmeier @Fabs addressed all comments and added a JIRA 👍 , please revisit 🙏


Introduce fused Dequantize+Matmul op with data-tiling and ukernel

https://github.com/openxla/iree/issues/14864 is about a matmul (in this case, a vec-mat) where the big matrix operand is produced by dequantizing narrow integer values. The intended use case is where memory traffic is the overriding concern, so the dequantization is meant to be fused into the matmul. That Issue is about this currently running slow, and the most obvious reason for the slowness is suboptimal codegen, leading one to consider writing a ukernel for this fusion while codegen catches up.
However, if one did overcome all codegen issues or had a good ukernel, the next performance bottleneck would be memory performance issues from not doing data-tiling.  So let's jump right away to doing this right with data-tiling.
In order for SetEncoding to be able to match this fusion and set the appropriate encodings to prepare for data-tiling, it will be good to have this fusion represented as a single op, perhaps something like LinalgExt::DequantMatmulOp (naming is tentative for now; to be debated on future PRs introducing that).
MaterializeEncoding will then rewrite that to usual PackOp, UnpackOp, and a new LinalgExt::DequantMmt4dOp.
DequantMmt4dOp is what we will have a ukernel for.
Steps:
graph TD;
a1["1. Create a new ukernel, `dequant_mmt4d`, with slow reference code for now.
That will solidify what exactly that op should be doing, and will provide an easy
lowering path for the ops created at the next steps - won't have to think about
lowering to linalg or vectorization etc."]
a2["2. Create the `LinalgExt::DequantMmt4dOp`
and implement its lowering to the `dequant_mmt4d` ukernel."]
a3["3. Create the `LinalgExt::DequantMatmulOp`
and implement data-tiling for it, with MaterializeEncoding picking for now any
dummy tile size and lowering it to `PackOp`, `UnpackOp`,
and a new `LinalgExt::DequantMmt4dOp`."]
a4["4. For some specific CPU ISA, pick appropriate data tile size
and implement a specific ukernel code path."]
a1-->a2-->a3
a1-->a4

Could we hold off on adding a new op and new encoding here. We already have a custom way of creating dispatches for dequant ops. We can use that as a place to introduce data tiling to experiment with. If we start building ops based on dequant + matmul that can cause an explosion in the number of operations


Select options are not centrally aligned

What version of starlight are you using?
0.19.1
What version of astro are you using?
4.4.4
What package manager are you using?
pnpm
What operating system are you using?
Windows
What browser are you using?
Brave
Describe the Bug
This is how they look

They should look like that

Same with the language selection box

Link to Minimal Reproducible Example
https://github.com/withastro/starlight
Participation

[ ] I am willing to submit a pull request for this issue.

this feels like a subjective opinion and less of a bug (both look fine to me). Could you share a concrete proof in form of a reference from MDN or a trusted article that your suggestions are applicable?

I would agree with a little bit more margin to the left side of the label icon.
That's used by Material UI for example:
https://m2.material.io/components/menus#exposed-dropdown-menu
Here's my take on the subject:

Let me know if this ok and I can submit it.

@rgilsimoes are you still up for submitting a PR? If your changes keep the current unfocused styles and only move the outline slightly, that seems great to me!

@rgilsimoes are you still up for submitting a PR? If your changes keep the current unfocused styles and only move the outline slightly, that seems great to me!

Will do, got it stashed!


ISO Homepage - User Tab Design

Table to view all users as well as remove and update users. View current active users. Create new users.


Provide location parameter to KS test, address issue #39

This pull request fixes the KS Test by providing the location (fixed) parameters and resolves issue #39.

Thank you. A few requests:

let's keep naming consistent and call the variable fixed_parameters
please update docstring for the new parameter
update tests

Added a new commit to address these points

Thank you James. I have reviewed the PR and I think I realized that I previously misunderstood the problem. A fix still needs to be pushed however, let me elaborate.
I reviewed ks test implementation and I think it's correct. The idea is that you pass sample (extremes), distribution, and fit parameters. Fit parameters are used to convert distribution's CDF function of quantiles + fit parameters to a function of only quantiles - this produces a function of only quantiles. Fit parameters has all parameters (free and fixed). Because of this, we shouldn't be modifying the KolmogorovSmirnov class itself.
Regarding the original issue. Pyextremes EVA class works by doing the following:

Extract extremes
Transform extremes to standard form expected by distribution (e.g. minima to pseudo-maxima) - this is done with the ExtremesTransformer class
Fit distribution to transformed extremes - this is where terminology goes a bit haywire (my bad). Distribution has fit_parameters attribute, which has estimated parameters, and fixed_parameters, which has frozen (e.g., floc) parameters.
Get extreme values by calculating ISF for given exceedance probability and then transforming the calculated value back (remember step 2) to original extreme value domain

Where the issue truly lies is how the EVA.test_ks method is implemented. To summarize:

KolmogorovSmirnov expects a sample, distribution, and all (free and fixed) parameters
EVA has raw extremes and transformed extremes (we must use transformed because that's what fit parameters are for)
EVA.model has fit parameters which are only free parameters (bad choice of terminology on my end)

What we need to do is to change the EVA.test_ks method to pass the following to to KolmogorovSmirnov:

self.extremes_transformer.transformed_extremes to extremes
self.model.fit_parameters (this is the same as self.model.distribution.mle_parameters) combined with self.model.distribution.fixed_parameters to fit_parameters - this way we will be passing all (free and fixed) parameters

My apologies for sending you on a wild goose chase earlier. Please let me know if you would be willing to revert the changes and update code.

George, thank you for the clarification. I see why changes to KolmogorovSmirnov class aren't needed -- I've changed the PR to only have the necessary changes to EVA.test_ks.
Using the term fit_parameters to mean (very slightly) different things in different places of the code is a bit confusing. However, you've already noted this and I think it is outside the scope of this PR!


Service Type Error

Describe the bug
I keep getting type errors when I create a custom service.  This happens when import { StoreService as MedusaStoreService } from "@medusajs/medusa/dist/services";. Seems like the types exist on medusa-interfaces because importing import { BaseService as MedusaStoreService } from "medusa-interfaces"; get rid of the errors
To Reproduce
Steps to reproduce the behavior:

Follow the Create Marketplace Tutorial part 1
The store.service and user.service throws the following errors

Expected behavior
Normally, we should not get any type Error, except I am doing something wrong
Screenshots

Package version:

Version 1.7.4

Additional context
# medusa versions
"@medusajs/medusa": "^1.3.5",
"@medusajs/medusa-cli": "^1.3.1",

In the newest version of medusa, the service extends the TransactionBaseService and not the baseService. So you don't need to implement the withTransaction anymore ☺

Thanks


How to set async content data?

I have data that loaded async after Tiptap is mounted. I can't set content
For now the only way I found is to watch data loaded, but this way I have the issue that it's setting after every charachter I type
watch: {
    value () {
      if (this.value) {
        this.editor.setContent(this.value)
        // this.html = this.editor.getHTML()
      }
    }
  }

Where and how to set data?

Where is facepalm icon?
in mounted value already setted. Watch is never called.
If you provide prop which already initialized you should use v-bind.
in data:
data() {
            console.log('editor created in data')
            return {
				/* html: '', */
				editor: new Editor({
					extensions: [
						new Blockquote(),new BulletList(),new CodeBlock(),new HardBreak(),new Heading({ levels: [1, 2, 3] }),
						new ListItem(),new OrderedList(),new Image(),
						new TodoItem(),new TodoList(),new Link(),
						new Bold(),new Code(),new Italic(),new Strike(),new HorizontalRule(),
						new Underline(),new History(),new Table({resizable: true,}),
						new TableHeader(),new TableCell(),new TableRow(),new TrailingNode({node: 'paragraph',notAfter: ['paragraph']}),
					],
					onUpdate: /* debounce( */({ getHTML }) => {//maybe we should make it debounce?
						/* this.html=getHTML(); */
						/* if (this.html === '

')this.html = ''; */
						this.$emit('input', getHTML());
					}/* , 500) */
				}),
			}
		},

in mounted:
mounted () {
            this.editor.setContent(this.value)
        }

@philippkuehn maybe we should provide initial_content prop into editor contstructor?

For those who are finding this search result now, I don't recommend checking empty content this way:
if(this.editor.getHTML() == '

' && this.value) {
    //...
}

It won't be correct if you've customized the paragraph classes. I'm currently using TipTap 2.0.3 which has the isEmpty property. So my solution to populating the editor with fetched content is:
const { error, data } = useFetch<JSONContent>('/some-url');

const editor = useEditor({...}); // Don't set content!

if(data && editor.isEmpty) {
    editor.commands.setContent(data);
}

Hope that helps the next person. :smile:

For those who are finding this search result now, I don't recommend checking empty content this way:
if(this.editor.getHTML() == '

' && this.value) {
    //...
}

It won't be correct if you've customized the paragraph classes. I'm currently using TipTap 2.0.3 which has the isEmpty property. So my solution to populating the editor with fetched content is:
const { error, data } = useFetch<JSONContent>('/some-url');

const editor = useEditor({...}); // Don't set content!

if(data && editor.isEmpty) {
    editor.commands.setContent(data);
}

Hope that helps the next person. 😄

This is the correct solution I believe


Image files paths with spaces are not parsed correctly

I have an example file named "circle bridge on lake.jpeg" (note the spaces) and am trying to load it in a markdown file with:
![circle bridge on lake](%config.gallery_art_url%/circle bridge on lake.jpeg)
*circle bridge on lake*

Instead of generating the following expected HTML:

circle bridge on lake

Pico generates the following broken half-parsed HTML:

![circle bridge on lake](http://localhost/assets/gallery-art/circle bridge on lake.jpeg)
circle bridge on lake

I understand that usually people don't have spaces in their file names on the web, but this is all being generated via bash scripts and the file names are the image descriptions, so I would prefer to keep the spaces.

Unfortunately Markdown requires URLs not to contain any spaces, and so does Parsedown, the Markdown parser Pico uses. The easiest solution is to rename all files using another bash script.
If your gallery is at a known place in your website, you can make things easier for you by adding a YAML structure to your page's YAML header and interpret it in your Twig templates accordingly:
---
title: My page
gallery:
  - img: %config.gallery_art_url%/circle_bridge_on_lake.jpeg
    description: circle bridge on lake
  - img: %config.gallery_art_url%/another_image.jpeg
    description: a different description
---

Some **Markdown** contents

{% for item in meta.gallery %}

    {{ item.description }}

{% endfor %}

Thank you for such a fast and thorough response!
I was afraid there wouldn't be a way to keep the spaces. I'll have to look into adjusting the bash scripts that are generating the markdown files and headers. :thinking: Hm... Maybe they could strip out underscores/dashes so that descriptions look right? I've never done that kind of string manipulation in bash before. I'm assuming it's possible.
For the curious:
My gallery artwork is in a known place, but there may or may not be multiple gallery pages in different locations. The bash scripts I have look through the gallery art directory and for each image generate a like-named single image display page markdown file complete with header, image link, and placeholders for user content. The gallery template then can display the excerpt from these pages.
If I can get the bash scripts adjusted properly to work around not having spaces in the file names they'll generate the following:
---
title: Circle Bridge On Lake
description: Artwork
excerpt: |
    ![Circle Bridge On Lake](%config.gallery_art_url%/circle-bridge-on-lake.jpeg)
    *Circle Bridge On Lake*
---

![Circle Bridge On Lake](%config.gallery_art_url%/circle-bridge-on-lake.jpeg)

Write a description here.

I have my bash scripts updated and all of the files no longer have any spaces in their names. Thank you again for the help @PhrozenByte .


feat: port Data.Nat.SqrtNormNum

I'm not sure about unsafe defs.. According to Data.Int.GCD they are all commented out. Does this also apply in this case?


Bayesian Tuning, Refactor

Major Changes:

Adds TuningBayes to tune with bayesian optimization
Refactors similar functions into a base class TuningBase
Refactors test suite
API changes for standardization

Closes #19

Codecov Report

Merging #21 into master will increase coverage by 1.57%.
The diff coverage is 92.85%.

@@            Coverage Diff             @@
##           master      #21      +/-   ##
==========================================
+ Coverage   92.08%   93.66%   +1.57%
==========================================
  Files           4        6       +2
  Lines         341      426      +85
==========================================
+ Hits          314      399      +85
  Misses         27       27

Impacted Files
Coverage Δ

tuningdeap/__init__.py
100% <100%> (ø)
:arrow_up:

examples/tune_basic_mlp.py
94.52% <100%> (ø)
:arrow_up:

tuningdeap/tuningbayes.py
100% <100%> (ø)

tuningdeap/base_tuner.py
89.62% <89.62%> (ø)

tuningdeap/tuningdeap.py
95.38% <90%> (ø)

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 55c2232...23a1e5b. Read the comment docs.


time-source duplicates?

It should be checked whether the following attributes are duplicates:

ptp-sync-pac/ptp-source-dataset/time-source
ptp-sync-pac/ptp-default-dataset/time-source (with an unsuitable data type int8)

IEEE 1588 defines the timeSource attribute as part of the timePropertiesDS. (This data set is not modeled as such in the Synchronization Model. Many of its attributes appear in the ptp-source-dataset container.)

read-only ptp-sync-pac/ptp-source-dataset/time-source: INT [1] = -1 (length undefined !! ) with comment "Values to be interpreted according to table 7 in section <IP_ADDRESS> of IEEE 1588-2008."
read-write ptp-sync-pac/ptp-default-dataset/time-source:INT8 [1] = -1 with comment "Values to be set according to table 7 in section <IP_ADDRESS> of IEEE 1588-2008."

Synchronization experts to discuss, whether it's two different attributes.

Feedback:
Regarding Sync #52 and Sync #53, we agree with Michael comments that attributes are redundant.
Sync #53: We also think it is duplicated, and we consider "ptp-sync-pac/ptp-default-dataset/time-source" should be deleted.

Proposal to the 5Gxhaul call on 19th of July 2023:
The ptp-sync-pac/ptp-default-dataset/time-source should be deleted from the SyncModel.


[IMPROVEMENT]  Enable Field Selectors for Longhorn CRDs

Is your improvement request related to a feature?
Currently, the CRDs don't yet support selection based on fields, but this was introduced with KEP-4358
Describe the solution you'd like
Implement selectableFields so that commands can filter based on states, e.g.,
kubectl get volumes -n longhorn-system --field-selector status.robustness!=healthy,status.state==attached

Ref: https://kubernetes.io/docs/tasks/extend-kubernetes/custom-resources/custom-resource-definitions/


Breakpoints on justMyCode: false not working properply in 1.90.0

Does this issue occur when all extensions are disabled?: Yes/No
Impossible to verify without extenstions

VS Code Version:

Version: 1.90.0
Commit: 89de5a8d4d6205e5b11647eb6a74844ca23d2573
Date: 2024-06-04T19:34:48.028Z
Electron: 29.4.0
ElectronBuildId: 9593362
Chromium: 122.0.6261.156
Node.js: 20.9.0
V8: 12.2.281.27-electron.0
OS: Darwin arm64 23.5.0

Python: v2024.8.0

OS Version:

Apple M1
Sonoma 14.5 (23F79)

Issue also submitted here here, but with no details: here
Steps to Reproduce:

VSCode version 1.90.0
"justMyCode": false,
set breakpoint anywhere in outside of your code.
hit run - the breakpoint will highlight somewhere in your code, the breakpoint outside of your code will be in green and steps will not move the selection forward (the app will proceed however). With each step, vscode will come back to some line or breakpoint in your code.

** Works in version 1.89.1 **

Please file on the python debug extension repo, thanks


feature request : audio (and video) filter profile

right now theres no way to set filter for different specific source afaik.
Just like crf had crf-profiles, would be great if we can have profile for audio / video filter too.
example:
if source audio = 5.1 and max-channels = 2 use this as a filter pan=stereo|FL=0.5*FC+0.707*FL+0.707*BL+0.5*LFE|FR=0.5*FC+0.707*FR+0.707*BR+0.5*LFE
if source audio = stereo and max-channels = 2 do not use filter at all

So the filter system would discriminate based on number of desired channels and number of source channels?
Assuming this setting:
pan=stereo|FL=0.5*FC+0.707*FL+0.707*BL+0.5*LFE|FR=0.5*FC+0.707*FR+0.707*BR+0.5*LFE

Source: 5
max-channels: 5
No filter applied

Source: 5
max-channels: 2
Filter applied

Source: 2
max-channels: 2
No filter applied

Is that essentially how the flow should work?
If source channels < output channels, apply, but you'd need to specific filters for each combination of input channels and output channels
That would also mean you would need to include something in the key value about source channels and target channels
6>2:pan=stereo|FL=0.5*FC+0.707*FL+0.707*BL+0.5*LFE|FR=0.5*FC+0.707*FR+0.707*BR+0.5*LFE
So something like inputchannels>outputchannels as the key?

Think I have a pretty solid concept working here
https://github.com/mdhiggins/sickbeard_mp4_automator/tree/fancyfilters
This adds a new section to the config that looks like this
[Audio.ChannelFilters]
6-2 = pan=stereo|FL=0.5*FC+0.707*FL+0.707*BL+0.5*LFE|FR=0.5*FC+0.707*FR+0.707*BR+0.5*LFE

The key 6-2 indicates that input channel count and the output channel count separated by a - and anything after the first = is the filter that will be applied
You could define plenty of other combinations as well, an example would be 8 channel input to stereo as 8-2
This should work in combination with other audio filters that are set in the [Audio] section of the config

Okay will test asap. Thanks!

Merged into master, please reopen if there is issues


Command binding on HamburgerButtonInfo not working

Hello
I'm trying to use MVVM all the way using the HamburgerMenu.
Having a ShellViewModel for the Shell.xaml page and then trying to bind a command  on the HamburgerButtonInfo control to an ICommand on the VM named ConnectCommand like this:
<Controls:HamburgerButtonInfo ClearHistory="True" ButtonType="Command" Command="{Binding ConnectCommand, Mode=TwoWay}">
The binding is not working and the BindingExpression message on output says "BindingExpression path error: 'ConnectCommand' property not found on 'Template10.Controls.HamburgerButtonInfo'."
The ButtonType name clearly promotes a "Command" but the documentation on https://github.com/Windows-XAML/Template10/blob/master/Samples/Search/Readme.md says that this gives the possibility to use the Tap event.
Is a real Command not possible to use here and in that case, why this naming?

Mode=OneWay ??

Has oc nothing to do with it! TwoWay since the button is NOT read only. This is a connect button and must be grayed out when logged on.

From the error, obviously ConnectCommand property is not found. Where does the property reside? If in view-model I am inclined to use the x:Bind as {x:Bind ViewModel.ConnectCommand, Mode=....} where ViewModel is set to the DataContext (usually in xaml file). An alternative is {Binding ElementName=PageName, Path=ConnectCommand }. Bottomline -- the binding should find where ConnectCommand is. It is also important to check how you defined your command property -- from experience it's easy to get it wrong as it's more than an ordinary property.
Sorry this is simply based on concept as you provided no code to be more specific.

Is the viewmodel defined in xaml as the datacontext or in codebehind as property of the  view and no name on the xaml definition. Somehow your context for the control is kicking back with the binding error

That's my bet too. You are using a binding statement that digs into a view-model that is not there. But there's more you should consider here. Are you planning on HamburgerInfo graying out your content for you when you set CanExecute to false? That is how a Command works with a button, but it is not how it works with HamburgerInfo. Instead, you will require a dedicated property in your view-model and bind to it inside your XAML. I hope this makes sense. In the end, HamburgerInfo is not a button.

Tanks for all the feedback. I will provide more info if I still have the problem after trying a bit more based on your tips.
If HamburgerButtonInfo is not a button, the naming is quite bad 😏

If HamburgerButtonInfo is not a button, the naming is quite bad
At the risk of being pedantic, the 'name' is actually Info to which the 'naming' applies in HamburgerButtonInfo. Consider HamburgerButton an adjuctive describing Info.

"At the risk of being pedantic, the name is actually Info to which the 'naming' applies in HamburgerButtonInfo. Consider HamburgerButton an adjective describing Info."
What I have today is a very well used app built using Caliburn Micro and the conventional SplitView control. Navigation is done there using Buttons! They work like buttons do and Command binding work as Command binding work targeting an ICommand even the CanExecute part of it.

More info:
DataContext is set on page using:
    <Page.DataContext>         <viewModels:ShellViewModel x:Name="ViewModel" />     </Page.DataContext>
A breakpoint in the page constructor confirms DataContext is ShellViewModel
I tested naming the page and then use:
Command="{Binding ElementName=TestPage, Path=DataContext.ConnectCommand}
Still no success...

Using Binding in a Button and TextBlock outside the Controls:HamburgerMenu works directly.
Sigh!

About half a dozen examples in samples folders, trade binding for x:bind, trade Datacontext for ViewModel in the Command="{x:Bind ViewModel.ConnectCommand}"

Not interested in x:bind! Using conventional Binding ok!

Ok I have been looking thru the source and stepping thru it and it appears to be a bug
Command="{Binding SomeCommand, ElementName=ViewModel}" never actually sets the Command it's always null and I can't seem to find where it ever would

Here is a OneWay binding with x:Bind that works as a starter:
XAML
<Controls:HamburgerButtonInfo ClearHistory="True" ButtonType="Command" Command="{x:Bind ConnectCommand, Mode=OneWay}">
	<StackPanel Orientation="Horizontal">
		<FontIcon Width="48"
				  Height="48"
				  Margin="0,-3,0,0"
				  FontFamily="Segoe MDL2 Assets"
				  FontSize="24"
				  Glyph="" />
		<TextBlock
				   Margin="12,0,0,0" Text="Connect"
				   VerticalAlignment="Center" />
	</StackPanel>
</Controls:HamburgerButtonInfo>

Code-behind
DelegateCommand connectCommand;
public DelegateCommand ConnectCommand => connectCommand ?? (connectCommand = new DelegateCommand(async() => await Connect()));
private async Task Connect()
{
    // Do your connection stuff here...
}

To do TwoWay binding, you obviuosly need to provide extra logic because DelegateCommand must return a value that ICommand understands otherwise you'll be greeted with Invalid binding path 'ConnectCommand' ...Cannot bind  ... DelegateCommand' to ... .ICommand' without a converter but depending on what you want there may be an easier workaround with OneWay binding (such as using another property for button text color/opacity/background). Hope this helps.

I can tell you it doesn't work with twoway delegatecommand  no binding errors, flat doesnt work, the command dependency property never passes its contents and he absolutely doesn't want to use x:bind.

I know @jonaswikstrom doesn't want to use x:Bind, but somehow helps to have a working reference. The code I provided works. Contrary to what you say, if you change the binding to TwoWay an error will be emitted (I have checked this).

Correct me then...

I get it. I suspect the difference is that you used ShellViewModel and I didn't. My x:Bind is just to the codebehind (Shell.xaml.cs). Does that explain?

its what I feared this doesn't work in this case with a viewmodel on the datacontext of the ShellView

Just so that you know:
I've also tested using ViewModel DataContext (command property in ShellViewModel.cs) and everything works as for the shell view. That is, I can hit the x:Bind bound Command property, and if Mode is changed to TwoWay the error is emitted.
Bottom line: works with x:Bind for view or view-model. Doesn't work with classical Binding as you indicate.


PP-288: Indexing for decision content and motion fields

To test

Checkout branch
SSH into the container and run drush cr;drush cim -y
If you have decision data already, try running: drush ap:dc -v

This should update the decision content and motion fields.
Edit a few decisions from:  https://helsinki-paatokset.docker.so/fi/admin/content/decisions
They should have HTML content in the decision and motion fields. The content should also be cleaned up a bit compared to the field_decision_content and field_decision_motion fields found in the decision data group (removed HTML and HEAD tags, and only including the main content for example).

Run drush migrate-rollback ahjo_decisions:latest;drush migrate-import ahjo_decisions:latest
OR if you didn't have decision data migrated, just run: drush ap:fs -v;drush mim ahjo_meetings:latest;drush mim ahjo_cases:latest;drush mim ahjo_decisions:latest;drush ap:ud --localdata
After the migration has been run, check the decisions list again. They should have the motion and content fields filled through the migration process
Check the decisions search index fields: https://helsinki-paatokset.docker.so/fi/admin/config/search/search-api/index/decisions/fields

The decision_content and decision_motion fields should be configured
Index the content and check that the fields are actually indexed

Noticed that decision_motion has a boost value of 0.5 - is this intentional? This does make the matches in decision_motion score lower than other matches (default boost is 1). If the motion content is not as relevant as other fields or contains a lot of duplicate text from the decision_content this boost value is OK.

Yeah, this is intentional, since the decision_content is probably more relevant.


Added Latest Tweet In the Readme

Added Latest Tweet In the Readme. Also added the path to the moksh's twitter account so that as soon as someone clicks the tweet it redirects to the moksh's twitter account

Hey @akshitagupta15june please review the pr


Improved safety of clipboard function when running outside a secure context or with very outdated browsers.

Linked Issue
Fixes #2030
https://discord.com/channels/1003691521280856084/1151104710850383972
Description
Added a try catch block to be able to handle insecure contexts or very, very outdated browsers. Or extensions breaking things.
Checklist
Please read and apply all contribution requirements.

[x] This PR targets the dev branch (NEVER master)
[x] Documentation reflects all relevant changes
[x] Branch is prefixed with: docs/, feat/, chore/, bugfix/
[x] Ensure Svelte and Typescript linting is current - run pnpm check
[x] Ensure Prettier linting is current - run pnpm format
[x] All test cases are passing - run pnpm test
[x] Includes a changeset (if relevant; see above)

Came up with a better solution in #contributors in Discord.


How to train a model tokenizer on my own datasets.

Hello, i want to know how to train a model tokenizer on my own datasets. Can you give me some meanings?

Thanks, i may know how to create a BPE model. Ii can use "--create_tokenizer" to create a BPE model!


fix: update unreleased splunk docker image (#294)

Update image of unreleased splunk with python 3.9 to build https://cd.splunkdev.com/splcore/main/-/tree/f8b1a92a256b (Splunk 9.3)
Tests:
https://github.com/splunk/splunk-add-on-for-servicenow/actions/runs/9692653841/job/26746592496?pr=741
https://github.com/splunk/splunk-add-on-for-google-cloud-platform/actions/runs/9692491380/job/26746170582?pr=735 To be released as a bugfix.

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


help

Excuse me, I have completed all the steps in the Installation section as per the README.md instructions. If I want to start the hanabi project to test the simulation game, what should I do next? Which command should I use to start it?

Hi! Sorry, I didn't receive an email notification for this issue. You can look inside the Hanabi-specific subfolder, which contains a README with examples for testing the simulation here.


The presence light application's about page doesn't show settings location

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Further details

OS: [e.g. iOS]
PresenceLight App Type (Desktop, Blazor)
PresenceLight Build (Nightly, Store, etc)
PresenceLight Version (The # in About)
Did you clear settings.json? (C:\Users\username\AppData\Local\Packages\37828IsaacLevin.197278F15330A.SomeValue\LocalState\settings.json
ASP.NET Core version

Please populate issue template


Unknown Error in scipy.interpolate.interpn

Dear Sir or Madame,
I am using scipy.interpolate.interpn to interpolate 3 dimensional data stored in an array with the shape [vertices, 1, 1] for a evenly grided space with the shape of [vertices, 3]. Since the scipy.interpolate.interpn function is able to interpolate thte data on an array with more vertices, but the same spacing I chose to use it. However after solving multiple reshaping issues I came across following ValueError.
Reproducing code example:
from scipy.interpolate import interpn
import numpy as np

subj_vol = np.array([[ 0.025, -0.055, -0.03 ],
       [-0.01 , -0.05 , -0.03 ],
       [-0.005, -0.05 , -0.03 ],
       [-0.01 , -0.03 ,  0.1  ],
       [-0.01 , -0.025,  0.1  ],
       [-0.015, -0.02 ,  0.1  ]])
temp_vol = np.array([[ 0.   , -0.055, -0.065],
       [ 0.005, -0.055, -0.065],
       [-0.005, -0.05 , -0.065],
       [ 0.   , -0.05 , -0.065],
       [ 0.005, -0.05 , -0.065],
       [-0.005, -0.045, -0.065],
       [-0.005, -0.02 ,  0.08 ],
       [ 0.   , -0.02 ,  0.08 ],
       [ 0.005, -0.02 ,  0.08 ]])
data = array([[[  1.54202606e-12]],
       [[  5.71852339e-14]],
       [[  3.70166527e-15]],
       [[  9.09036818e-16]],
       [[  1.83975258e-13]],
       [[  3.07202621e-14]]])
x, y, z = subj_vol.astype(bool).T
interp = interpn((x, y, z) , data, temp_vol, method='linear', bounds_error=False)

Error message:
Traceback (most recent call last):

  File "<ipython-input-1-8994199dc614>", line 1, in <module>
    runfile('/home/daniel/Dropbox/MasterThesis-dvdv/Volume Morphing/Interpolate_MFT_on_new_grid.py', wdir='/home/daniel/Dropbox/MasterThesis-dvdv/Volume Morphing')

  File "/home/daniel/anaconda2/lib/python2.7/site-packages/spyder/utils/site/sitecustomize.py", line 866, in runfile
    execfile(filename, namespace)

  File "/home/daniel/anaconda2/lib/python2.7/site-packages/spyder/utils/site/sitecustomize.py", line 94, in execfile
    builtins.execfile(filename, *where)

  File "/home/daniel/Dropbox/MasterThesis-dvdv/Volume Morphing/Interpolate_MFT_on_new_grid.py", line 73, in <module>
    bounds_error=False)

  File "/home/daniel/anaconda2/lib/python2.7/site-packages/scipy/interpolate/interpolate.py", line 2497, in interpn
    "ascending" % i)

ValueError: The points in dimension 0 must be strictly ascending

Scipy/Numpy/Python version information:
('0.18.1', '1.11.3', sys.version_info(major=2, minor=7, micro=13, releaselevel='final', serial=0))```

Have you noticed that your x, y, z are boolean arrays?
>>> x
array([ True,  True, True,  True,  True,  True], dtype=bool)

...which are of course not "strictly ascending" and thus give the ValueError.
You may have a look at this stack overflow answer for an example how to use interpn.

I am very sorry. There was a mistake in my Example.
from scipy.interpolate import interpn
import numpy as np

subj_vol = np.array([[ 0.025, -0.055, -0.03 ],
       [-0.01 , -0.05 , -0.03 ],
       [-0.005, -0.05 , -0.03 ],
       [-0.01 , -0.03 ,  0.1  ],
       [-0.01 , -0.025,  0.1  ],
       [-0.015, -0.02 ,  0.1  ]])
temp_vol = np.array([[ 0.   , -0.055, -0.065],
       [ 0.005, -0.055, -0.065],
       [-0.005, -0.05 , -0.065],
       [ 0.   , -0.05 , -0.065],
       [ 0.005, -0.05 , -0.065],
       [-0.005, -0.045, -0.065],
       [-0.005, -0.02 ,  0.08 ],
       [ 0.   , -0.02 ,  0.08 ],
       [ 0.005, -0.02 ,  0.08 ]])
data = np.array([[[  1.54202606e-12]],
       [[  5.71852339e-14]],
       [[  3.70166527e-15]],
       [[  9.09036818e-16]],
       [[  1.83975258e-13]],
       [[  3.07202621e-14]]])
x, y, z = subj_vol.T
interp = interpn((x, y, z) , data, temp_vol, method='linear', bounds_error=False)

Error Message is the same:
  File "/home/daniel/anaconda2/lib/python2.7/site-packages/scipy/interpolate/interpolate.py", line 2497, in interpn
    "ascending" % i)

ValueError: The points in dimension 0 must be strictly ascending

Is there a common way for sorting the 3D-grid arrays accordingly to the function restrictions ?

There's probably some misunderstanding in how to use interpn:
you have 6x6x6 grid, but only 6 data points, so you do not have gridded
data and cannot use interpn to interpolate it.

That is true. However the ValueError is raised by a false sanity input grid.
Since that is the case, I would like to solve this problem anyway.
I found the solution to sort my subj_vol to be strictly ascending.
subj_vol = subj_vol[subj_vol[:,2].argsort()] # First sort doesn't need to be stable.
subj_vol = subj_vol[subj_vol[:,1].argsort(kind='mergesort')]
subj_vol = subj_vol[subj_vol[:,0].argsort(kind='mergesort')]
In[8]: subj_vol
Out[8]:
array([[-0.015, -0.02 ,  0.1  ],
       [-0.01 , -0.05 , -0.03 ],
       [-0.01 , -0.03 ,  0.1  ],
       [-0.01 , -0.025,  0.1  ],
       [-0.005, -0.05 , -0.03 ],
       [ 0.025, -0.055, -0.03 ]])

So the ValueError shouldn't be raised by a false input grid.


styles for diff on markdown preview

Fixes #59288

Awesome! This change will be in the next vs code insiders build and will go to stable in 1.28


Fix plugin version script to include dist folder

When a plugin is published to npm, it doesn’t include the dist folder.
Here’s the version script: https://github.com/donejs/generator-donejs/blob/a50d2851bf788ec11473b49e9c8b46a1eff4bd82/plugin/index.js#L120
The issue is that the dist folder is added in a commit to the release branch, then it switches to master before npm publish is run, which means the dist folder is no longer there for it to be published to npm.
Here’s a commit that shows a potential fix: https://github.com/chasenlehara/chasen-number-input/commit/a08957f392eaeb46a25303b84151b3ccebaf52d8
That commit changes postversion to postpublish so it’s still on the release branch when publishing to npm. It also changes the commit message that includes the version bump.
Ideally, the version bump commit message would have the version number in it, but I haven’t figured out a way to do that without creating a script.

I think this change makes sense.
Another option is what we're doing on canjs/canjs, which runs the build during each of the release scripts: https://github.com/canjs/canjs/blob/master/package.json#L16-L19.
I think changing to postpublish is a better solution though since you only have to do it in one place.


Crashes withot logs when using oauth

Description
Crashes without logs when using authorization type "OAUTHBEARER".
client.OAuthBearerSetToken(token.TokenValue, token.Expiration, **null**);
OS -> Ubuntu Linux, MacOS, etc.
Nuget package: Confluent.Kafka, Version=<IP_ADDRESS>
How to reproduce
Reproduces on unix OS family: local or in docker container.
Works on windows.
Set principal to null in client.OAuthBearerSetToken(token.TokenValue, token.Expiration, **null**): https://github.com/confluentinc/confluent-kafka-dotnet/blob/master/test/Confluent.Kafka.IntegrationTests/Tests/OauthBearerToken_PublishConsume.cs
Works, when principal set to empty string.
Checklist
Please provide the following information:

[ ] A complete (i.e. we can run it), minimal program demonstrating the problem. No need to supply a project file.
[x] Confluent.Kafka nuget version.
[ ] Apache Kafka version.
[ ] Client configuration.
[x] Operating system.
[ ] Provide logs (with "debug" : "..." as necessary in configuration).
[ ] Provide broker log excerpts.
[x] Critical issue.

Can you provide debug logs?

Can you provide debug logs?
There are no logs becouse this issue
https://github.com/confluentinc/confluent-kafka-dotnet/issues/1834

Happens to me as well OSX with latest nuget
If principal is set to null it will exit with CODE 139 in with or without a debug session.
If principal is set to an empty string, all is ok.

This is also happening in docker env for me. Works fine outside of docker.
var adminClientBuilder = new AdminClientBuilder(_adminClientConfig)                 .SetOAuthBearerTokenRefreshHandler(async (c, p) =>                 {                     try                     {                         var token = await _tokenService.GetTokenAsync();                         if (token != null && !string.IsNullOrEmpty(token.AccessToken))                         {                             var tokenExpiration = DateTimeOffset.UtcNow.AddMinutes(15).ToUnixTimeMilliseconds();                             c.OAuthBearerSetToken(token.AccessToken, tokenExpiration, null);                         }                         else                         {                             c.OAuthBearerSetTokenFailure("Token is null or empty");                         }                     }                     catch (Exception ex)                     {                         _logger.LogError(ex.Message);                         c.OAuthBearerSetTokenFailure($"Token retrieval failed: {ex.Message}");                     }                 });


Values.yaml uses external containerd to k3s

Shoudl the values.yaml not be using contaird managed by k3s internally, otherwise hwo does the cluster leverage nvdia?
Maybe this is a more appropriate implementation
toolkit:
env:
- name: CONTAINERD_CONFIG
value: "/var/lib/rancher/k3s/agent/etc/containerd/config.toml.tmpl"
- name: CONTAINERD_SOCKET
value: "/run/k3s/containerd/containerd.sock"
- name: CONTAINERD_RUNTIME_CLASS
value: "nvidia"
- name: CONTAINERD_SET_AS_DEFAULT
value: "true"
devicePlugin:
config:
name: time-slicing-config-all
default: any

The pod will use the nvidia runtime if it's specified in the manifest, see the nvidia-smi.yaml example.


Mistake in documentation - viewers are not getting build permission

I think this documentation section is wrong or at least very confusing:
All users who already have Read permission for datasets via app permissions, sharing, or workspace access also get Build permission for those datasets. Those users get Build permission automatically because Read permission already grants them the right to build new content on the dataset by using Analyze in Excel or Export.
Viewers are not getting build permission for the dataset!
(https://learn.microsoft.com/en-us/power-bi/connect-data/service-datasets-build-permissions?source=recommendations#more-granular-permissions)

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

ID: 16dbd9ca-dcd3-a4f3-43a8-01ead54fc9f9
Version Independent ID: 66217fc3-a852-201c-a563-54ecbd9e5726
Content: Build permission for shared datasets - Power BI
Content Source: powerbi-docs/connect-data/service-datasets-build-permissions.md
Service: powerbi
Sub-service: pbi-data-sources
GitHub Login: @paulinbar
Microsoft Alias: painbar

Hi @AdamGlArla. Thanks for pointing this out. I am working with the product team to get clarification and will update the doc as necessary. So in the meantime I am closing this issue. Thanks again for your feedback!


CLA Bot exhibiting non deterministic behaviour

I have two PRs, and they have the same set of commit authors.  Yet one of them has clabot=yes, and one has clabot=no.
The two PRs are https://github.com/kubernetes-sigs/etcdadm/pull/170 and https://github.com/kubernetes-sigs/etcdadm/pull/162
I compared the authors using:
diff <(git log justinsb/add_etcd_manager_4 | grep Author | sort | uniq) <(git log justinsb/add_etcd_manager | grep Author | sort | uniq)
or equivalently: diff <(git log 8ec0490d748bcadb8900b6235c0f230a2c9b3109 | grep Author | sort | uniq) <(git log 2c7d16cbe9d50ebc90792d5d9e79f73e4aa21e6c | grep Author | sort | uniq)
No differences were shown.

Hello @justinsb  Those 2 PRs are using the CLA v1 bot that isn't managed by the EasyCLA system. All Kubernetes repos are still on the CLA v1 system right now.
In your case, the https://github.com/kubernetes-sigs/etcdadm/pull/162 has over 700 commits. Each commit must be authored by a user attached to their GitHub account so that we may verify their identity and that they have signed a CLA. Some of those commits are showing the grey octocat icon because they are incorrectly authored and not tied to a user. The link in the "details" of the CLA check shows more information about how to fix those commits. https://docs.github.com/en/github/committing-changes-to-your-project/why-are-my-commits-linked-to-the-wrong-user#commits-are-not-linked-to-any-user Quickly glancing through the commits, I see that several from "Daniel Lipovetsky" in 2019 aren't linked.
If you need further assistance with the CLA checks, please open a support ticket at support.linuxfoundation.org Thank you!


Running into issue with UDS example

https://github.com/hyperium/tonic/blob/master/examples/src/uds/server.rs
When I pull the whole tonic repo and run this example, it works. However, I'm trying to build from this example in my own directory and I keep getting the on line 81 of the example:
error[E0437]: type ConnectInfo is not a member of trait Connected
I copied the server.rs file along with some dependencies in Cargo.toml and a basic build.rs. Not really sure why this is happening.

Some changes have been made on master that have not yet been released on crates.io. Here is the example from version 0.4.3 https://github.com/hyperium/tonic/tree/v0.4.3/examples/src/uds.

Oh I see. Thank you!
Solved with using
tonic = { git = "https://github.com/hyperium/tonic.git", features = ["tls"] }
instead of
tonic = {version ="0.4.0", features=["tls"]}


Added Go installation for Ubuntu & Debian based Linux

Go installation on Ubuntu is really simple.

Excellently！I merged～！Hahahahaha

Great, will try your new version tmr.


Please provide cvdump.pdb as well.

You provide the executable, which is nice, but as long as things are in a state where we cannot build the thing ourselves, providing the PDB for the executable that you do include would at least let us step through and get a better feel for how it works.
Providing both debug and release builds of the tool might be useful as well.
Right now I am looking at an issue where cvdump says the TYPES stream is corrupt.  Why?  Hard to say - I am having to step through in assembly language to try and deduce why this might be the case, and as you might expect this is a tedious process.

Yes please, being able to step through cvdump source code in the debugger would be very useful :)


make pk argument required when querying single object

It's just a one line correction to make pk argument required when querying a single django object
edit: and test fot required pk argument

@stygmate looks good to me, can you update the tests? :)

@stygmate looks good to me, can you update the tests? :)

Hi the test are not failing because of my modifications. Looks like a error in creation of docker image. Tests are ok my side. But if you want I can add a test to verify that the field is actually required.

@stygmate looks good to me, can you update the tests? :)

Hi the test are not failing because of my modifications. Looks like a error in creation of docker image. Tests are ok my side. But if you want I can add a test to verify that the field is actually required.

yes, let's add that! so we don't get any regression 😊

@patrick91 the ugly pre-commit.ci was annoying me ;) but seems ok now !

LGTM 👍🏼


Error: RegexNotFoundError: Unable to extract content ID

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 this: [x])
Use the Preview tab to see what your issue will actually look like

Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2017.09.11. 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 2017.09.11

Before submitting an issue make sure you have:

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

What is the purpose of your issue?

[X] Bug report (encountered problems with youtube-dl)

Verbose output follows:
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: [u'--restrict-filenames', u'--simulate', u'--get-filename', u'--verbose', u'-o', u'%(series)s-%(title)s.%(ext)s', u'https://au.tv.yahoo.com/plus7/800-words/-/watch/29520134/800-words-season-1-episode-1/']
[debug] Encodings: locale US-ASCII, fs utf-8, out None, pref US-ASCII
[debug] youtube-dl version 2017.09.11
[debug] Python version 2.7.10 - Darwin-16.7.0-x86_64-i386-64bit
[debug] exe versions: ffmpeg 3.3.3-tessus, ffprobe 3.3.3-tessus
[debug] Proxy map: {}
ERROR: Unable to extract content ID; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
  File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 776, in extract_info
    ie_result = ie.extract(url)
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 434, in extract
    ie_result = self._real_extract(url)
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/yahoo.py", line 302, in _real_extract
    CONTENT_ID_REGEXES, webpage, 'content ID')
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 797, in _search_regex
    raise RegexNotFoundError('Unable to extract %s' % _name)
RegexNotFoundError: Unable to extract content ID; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

Testing youtube-dl to return series, title and filename from the Plus7 web site in Australia.  ytdl returned the error msg advising that I should do a bug report.  Can't find any other similar bugs and am definitely using youtube-dl 2017.09.11.  I am located in Australia and so am not affected by geo-restrictions.

I am receiving the same issue with version 2017.09.15
Plus7 recently changed the layout of their site so this maybe a contributing factor.
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: [u'--verbose', u'https://au.tv.yahoo.com/plus7/highway-patrol/-/watch/37132737/highway-patrol-mon-18-sep-season-9-episode-5/']
[debug] Encodings: locale UTF-8, fs utf-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2017.09.15
[debug] Python version 2.7.11 - Darwin-17.0.0-x86_64-i386-64bit
[debug] exe versions: ffmpeg 3.3, ffprobe 3.3
[debug] Proxy map: {}
[Yahoo] highway-patrol-mon-18-sep-season-9-episode: Downloading webpage
ERROR: Unable to extract content ID; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
  File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 776, in extract_info
    ie_result = ie.extract(url)
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 434, in extract
    ie_result = self._real_extract(url)
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/yahoo.py", line 302, in _real_extract
    CONTENT_ID_REGEXES, webpage, 'content ID')
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 797, in _search_regex
    raise RegexNotFoundError('Unable to extract %s' % _name)
RegexNotFoundError: Unable to extract content ID; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

Yes, there must have been a recent change @ Plus7.
I do not get the RegexNotFoundError if I force youtube-dl to use the generic extractor.  However, every video I've tried gets:
"ERROR: Unsupported URL: https://au.tv.yahoo.com/plus7/800-words/-/watch/29520134/800-words-season-1-episode-1/"

This should be fixable.  The commercial tool Allavsoft which uses youtube-dl is able to download videos from Plus7.  I have attached a log file from Allavsoft:
download.log
Cheers.


vcr tests

This checklist is used to make sure that common guidelines for a pull request are followed.
General Guidelines

[x] The PR has modified HISTORY.rst with an appropriate description of the change (see Modifying change log).

Command Guidelines

[x] Each command and parameter has a meaningful description.
[x] Each new command has a test.

(see Authoring Command Modules)

Codecov Report

Merging #5 into master will increase coverage by 0.26%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master       #5      +/-   ##
==========================================
+ Coverage   70.94%   71.21%   +0.26%
==========================================
  Files         426      426
  Lines       26910    26904       -6
  Branches     4095     4095
==========================================
+ Hits        19092    19160      +68
+ Misses       6578     6499      -79
- Partials     1240     1245       +5

Impacted Files
Coverage Δ

...azure-cli-interactive/azclishell/_dump_commands.py
84.21% <ø> (+66.38%)
:arrow_up:

...s/azure-cli-interactive/azclishell/az_completer.py
59.72% <100%> (ø)
:arrow_up:

.../azure-cli-interactive/azclishell/configuration.py
63.63% <0%> (+1.29%)
:arrow_up:

...azure-cli-core/azure/cli/core/commands/progress.py
89.89% <0%> (+5.05%)
:arrow_up:

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 f43cb89...5602ea6. Read the comment docs.


Multiple Examples updated to work on WebGPU

switched few examples to use EnvAtlas instead of cubemap
also switched them to use createGraphicsDevice to allow easy way to enable WebGPU
updated envMap setter to set up appropriate texture filtering
sky shader gets processed for Uniform Buffers / Bind Groups

I notice this PR switched to loading an envAtlas texture. I'm wondering whether it's 'nicer' to have the examples load a pano and reproject it at load time. It then easier for devs to experiment since panos are available all over the web.

The reprojection does not work on WebGPU, so I'm switching to this option at the moment. It's also the most optimal solution. But agreed that some examples should use other options, and perhaps even an example showcasing multiple options.


Changes about home href and feed

Both d563f10 and 58a8d85 tried to fix the link to root (i.e. home page), but that ternary is unnecessary.
And they missed out the link in the nav-menu, footer and path of all the resources files. With a prefix slash they all break when the blog lives in a sub-directory.
Replacing the root + path approach with url_for() helper as required by official doc should makes things clear.
To see what the different hrefs bring us to, see http://rmntn.net/href-test
I also make RSS feed header self-adaptive.

Thank you very much for so many improvements! But I am still wondering why we have to set the directory of home menu in _config.yml to "." instead of "/" or something else?

All changes in e0a8672 is made for the same reason: Using '/' will bring you to the very root of site. My demo proves this.

OK I see, thanks for your contribution.


Stop double defaults in match

Sadly we cannot stop repeated values, that will just be whichever one is the last one will be run.

Looks good

Sadly we cannot stop repeated values

Why not? We could keep track of a list of values and then if it's already in the list, it's repeated


Permalinks for bounce and scale modals

Appending ?bounce=true or ?scale=true on any request page will have the respective modal automatically open.

🚢


Add zcml and xml extension for --plone

Plone has some configuration files with zcml extension, and others with xml extensions that should be included into ag search.

Hey @ggreer could you still have a look at my pr?


Link device tracker entity to Rivian device

Previously, the device_tracker entity did not show up in the "Device info" view.  This PR links the device_tracker to the Rivian device so that it shows up in the "Device info" view:

Believe this is the entity that states if the vehicle thinks it is "at home" or "away"?

Believe this is the entity that states if the vehicle thinks it is "at home" or "away"?

Yep, and the device_tracker entity worked fine as-is.
This PR simply links that entity to the device config, so when you click on 1 device from the Integrations page, the device_tracker entity shows up on the device info page:


Fixed #25320 -- Reverted ManyToManyField.null to False for backwards compatibility.

https://code.djangoproject.com/ticket/25320

PR looks good to me in it's current state - very easy to comprehend now.
Do we think it'd make sense tod also revert (the newly introduced) has_null_arg as part of this PR? https://github.com/timgraham/django/blob/25320/django/db/models/fields/related.py#L2301
And revert the associated self.has_null_arg check here, back to it's original is_null check.
(One piece less of state / API that way)

I think it makes sense to show the warning whether null=True or null=False are specified.

@timgraham Yup - that's very reasonable.


[BanyanDB] Implement unify persistence data storage

Search before asking

[X] I had searched in the issues and found no similar feature requirement.

Description
At present, BanyanDB uses Badger for persistence storage, which has the following drawbacks.

Badger uses the mmap mode for storage. The mmap mode has some problems refer to https://db.cs.cmu.edu/mmap-cidr2022/.
The data organization mode of Badger is not suitable for BanyanDB.
Badger is designed for SSD optimization and has a relatively small scope of application.

Therefore, to solve these problem, we can implement  unify persistence data storage.And we can benefit from it:

Better data format for storage and query in APM scenarios.
Adapt to various hardware and s3 interfaces.
Better IO mode, such as io-uring.
Improve compaction performance and reduce data query latency.

Use case
No response
Related issues
No response
Are you willing to submit a PR?

[X] Yes I am willing to submit a PR!

Code of Conduct

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

I aligned the milestone between this and PR for 0.6

@HHoflittlefish777 how do you modularize the requirements of local storage for BanyanDB?
If Badger as an unmaintained solution with its mmap issue, a near drop-in solution can be trying to adopt pebble which shares a similar design. Making a local-fs from scratch is a bit confusing for me >_<

We have our design for local file structure, to improve performance. Nothing about badger  status.

@wu-sheng I noticed the design doc now - https://github.com/apache/skywalking-banyandb/pull/275

@HHoflittlefish777 how do you modularize the requirements of local storage for BanyanDB?
If Badger as an unmaintained solution with its mmap issue, a near drop-in solution can be trying to adopt pebble which shares a similar design. Making a local-fs from scratch is a bit confusing for me >_<

The goal is to replace the entire Badger, and BanyanDB will implement its own storage engine to adapt to the time serial data. The goals of this unified file system are as follows:

Mask differences between different operating systems
Mask the differences between different IOs, such as synchronous and asynchronous IOs
Mask the difference between local and remote storage

Therefore，not only local storage : )


error with main.go:124 +0x6ab

https://github.com/etsy/hound/blob/92e228e2be3d624b89e3eaf5d2a4098495937587/cmds/houndd/main.go#L124
/src/github.com/etsy/hound/cmds/houndd/main.go:124 +0x6ab

If no configuration file is available, perhaps the error message could be nicer. But hound cannot proceed in that state.

thank you very mach..

So, this can either be closed or retitled "Improve error message when configuration file not found"

因为我的config配置文件少了个大括号},所以就出错了。

Recent Go versions include more information in the unmarshal error message.  Given that JSON is tricky to generate by hand (extra comma, for example), maybe the error message should be made clearer.

thank you very mach..

It seems like from the comments that this issue may not need to stay open.  If that is not correct, please feel free to reopen or make a Pull Request.  Cheers!


[2.2] fix: fixed validation issue

Info
https://github.com/bagisto/bagisto/issues/10212

Checked and found that issue related to the Contact Us page validation has been fixed. Please check the video for reference.
Video
https://github.com/user-attachments/assets/d7004aca-890a-4d55-8709-7370bd4fbed2


Are u adding any filter section in ur upcoming update?🤔

from where did u get the REDIRECT_URI ??

I dont understand what u mean...😅


How to get statistics using this library

Hi @kohsuke ,
I am using this library for long in implementing many of my projects. I am facing an issue as I am not able to get statistics of a particular user in a repo? How to do that?

Can you point me to the relevant part of the GitHub API doc? I don't see anything statistics-ish in https://developer.github.com/v3/users/

No further input from the reporter


Putting white spaces in between if/while clauses and braces.

What's in this pull request?
Fix whitespaces.

Can you change the commit message so that it specifically says that you are putting white spaces in between if/while clauses and braces with an example.

Thx @gottesmm

Can you put a dummy example into the commit message so that people can understand without reading the actual diff what the patch does?
Something like.
[gardening] Put white spaces in between if/while clauses and braces where it is missing.
For instance:
'if (foo){' => 'if (foo) {'

Thank you, it's better.

@swift-ci Please test and merge

The test failure looks like some other issue. These changes should be NFC.

I had to revert this change since it introduced 80+ line violations. Please file a new diff with these changes, sans any of those 80+ line violations.

https://github.com/apple/swift/pull/2025 @gottesmm


Cross platform editing

Starting the editor for pages is currently linux only (and not even all distros) it is also hacky.

It should work on most *nix systems now, but not windows.

fixed


Preserving colour of drawable when setting avatar and icon

Hi @henrytao-me,
This seems to be a great lib. Something that Google should've done to encourage the adoption of material design. I tried to put drawables into the avatar and icon of MdListItem. But they are all turned dark when I use the MdTheme.Light and they are all turned light when I use the MdTheme.Dark. How do I keep the original colour of the drawable the way the avatars in the Gmail app works? It seems that there is a MdIcon tint and some other icon colours in the source. What should I override in my theme that extends MdTheme?
I think this library will attract a lot more attention once you have more documentation. Thank you.
Niko

Hi Niko,
If you want to keep original color of the drawable, you can custom MdListItem and use ImageView instead of MdIcon.

I will update list examples later. Thanks for your interest in this library. ;). I love it.

Hi @nikoladoherty
Please check out v1.9.10. I updated iconTintMode for list item. You raised a very good point where MdListItem shouldn't apply tint mode for avatar. In your case, if you want to keep icon color, you can declare view like this:
<me.henrytao.mdcore.widgets.MdListItem
      android:layout_width="match_parent"
      android:layout_height="wrap_content"
      android:background="?attr/selectableItemBackground"
      android:clickable="true"
      app:mdli_divider_layout="@layout/mdli_divider_left"
      app:mdli_icon="@drawable/ic_blank_circle"
      app:mdli_icon_tintMode="dst"
      app:mdli_title="@string/text_single_line_item"
      app:mdli_type="single_line_icon_with_text" />

Thanks for the update! The avatar now shows colour. Thanks for also implementing the icon tint mode attribute.


Ongoing: Fixing docstrings

Just pushed scripts/find_undoc_args.py. putting this here
for the occasional drive-by contributor, and newcomers who would appreciate
an opportunity to make a "low-risk" first PR.

[x] 1) pandas/compat/scipy.py:8 (scoreatpercentile()) -  Missing[1/4]: ['interpolation_method']    Extra(?): ['interpolation']
[ ] 2) pandas/core/algorithms.py:105 (factorize()) -  Missing[1/4]: ['na_sentinel']
[ ] 3) pandas/core/format.py:1287 (set_printoptions()) -  Missing[3/12]: ['max_colwidth', 'colheader_justify', 'encoding']
[x] 4) pandas/core/frame.py:5714 (boxplot()) -  Missing[3/6]: ['ax', 'grid', 'rot']    Extra(?): ['data']
[x] 5) pandas/core/generic.py:1005 (truncate()) -  Missing[1/3]: ['copy']
[ ] 6) pandas/core/groupby.py:529 (groupby()) -  Missing[1/2]: ['by']    Extra(?): ['groupings', 'axis', 'exclusions', 'name', 'level']
[x] 7) pandas/core/reshape.py:601 (lreshape()) -  Missing[1/4]: ['label']
[x] 8) pandas/core/reshape.py:725 (make_axis_dummies()) -  Missing[1/3]: ['frame']    Extra(?): ['call']
[x] 9) pandas/stats/math.py:93 (calc_F()) -  Missing[1/6]: ['R']
[x] 10) pandas/stats/moments.py:804 (expanding_apply()) -  Missing[1/6]: ['time_rule']
[x] 11) pandas/stats/moments.py:721 (expanding_count()) -  Missing[1/4]: ['time_rule']
[x] 12) pandas/stats/moments.py:741 (expanding_quantile()) -  Missing[1/6]: ['time_rule']
[x] 13) pandas/stats/moments.py:556 (rolling_apply()) -  Missing[1/7]: ['time_rule']
[x] 14) pandas/stats/moments.py:128 (rolling_count()) -  Missing[1/5]: ['time_rule']
[x] 15) pandas/stats/moments.py:528 (rolling_quantile()) -  Missing[1/7]: ['time_rule']
[x] 16) pandas/stats/moments.py:584 (rolling_window()) -  Missing[2/9]: ['time_rule', 'axis']
[x] 17) pandas/stats/moments.py:256 (_rolling_moment()) -  Missing[1/8]: ['time_rule']
[x] 18) pandas/stats/ols.py:1243 (_filter_data()) -  Missing[1/3]: ['weights']
[x] 19) pandas/tools/plotting.py:96 (scatter_matrix()) -  Missing[2/7]: ['marker', 'frame']
[x] 20) pandas/tools/plotting.py:1811 (boxplot_frame_groupby()) -  Missing[3/7]: ['grouped', 'grid', 'figsize']
[x] 21) pandas/tools/plotting.py:1981 (_subplots()) -  Missing[1/9]: ['data']    Extra(?): ['fig_kw']
[x] 22) pandas/tools/plotting.py:307 (andrews_curves()) -  Missing[1/4]: ['ax']
[x] 23) pandas/tools/plotting.py:1545 (boxplot()) -  Missing[3/8]: ['ax', 'grid', 'figsize']
[x] 24) pandas/tools/plotting.py:1671 (hist_frame()) -  Missing[3/11]: ['column', 'data', 'by']
[ ] 25) pandas/tools/plotting.py:1358 (plot_frame()) -  Missing[3/24]: ['frame', 'fontsize', 'figsize']    Extra(?): ['barh', 'bar', 'stacked']
[ ] 26) pandas/tools/plotting.py:1467 (plot_series()) -  Missing[1/16]: ['series']    Extra(?): ['barh', 'bar']
[ ] 27) pandas/tools/tile.py:14 (cut()) -  Missing[2/7]: ['include_lowest', 'precision']
[ ] 28) pandas/tools/tile.py:104 (qcut()) -  Missing[1/5]: ['precision']
[x] 29) pandas/tseries/frequencies.py:741 (infer_freq()) -  Missing[1/2]: ['warn']
[x] 30) pandas/tseries/offsets.py:1163 (generate_range()) -  Missing[2/5]: ['time_rule', 'offset']
[x] 31) pandas/tseries/tools.py:51 (to_datetime()) -  Missing[3/6]: ['box', 'dayfirst', 'format']

Hi,
I came across this and figured this was a good place for me to get familiar with and start contributing to pandas.
The list seems pretty old as when I click through the line of code of each item in the list, the referred line of code is not there anymore.
I went ahead and used scripts/find_undoc_args.py to identify more similar issues. There were several of them, and I chose one of them to start.
Pull request is here #9293.
Please let me know if this is a proper way to contribute.

@yyl Yes, improvements to documentation/docstrings are always appreciated :+1:.

Hi, I was hoping to contribute and this looked like somewhere good to start, but I'm having some issues with the script, for example:
./find_undoc_args.py -m pandas -G pandas-dev -p pandas | head -n1

- [ ] 1) [core/algorithms.py:527 (factorize())](https://github.com/pandas-dev/blob/master/core/algorithms.py#L527) -  __Missing__[1/5]: ['order']

any ideas? seem to be using the path flag wrong, but i can't figure it out.
thanks

I'd like to contribute, but I'm not quite sure what some of the ask is here...


Introducing list region feature

Description
This PR introduces a QoL feature where it allows Users to get an idea of which region is actually enabled (opted-in) in their account, and most importantly - To assist the users to put in the correct values for the region identifier.
Fixes # (issue)

N/A

Type of change
Please delete options that are not relevant, and add any that may be relevant.

[x] New feature (non-breaking change which adds functionality)

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

[x] Multiple Test Cases

Please refer to the video below.  Rest assured, it does not break any existing functionalities.

https://user-images.githubusercontent.com/20971020/227607532-d846f12a-f4ed-4f3a-8ff2-9707f083194f.mov

Test Configuration:

Firmware version: AWS CloudShell
Hardware:
Toolchain:
SDK: AWS SDK (PHP)

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] I have tested this change with all regions and services
[x] Any dependent changes have been merged and published in downstream modules

1/ Need to handle users type in ",<SPACE>" or "<SPACE>," for separator (maybe trim or str_replace all  to '')
2/ I think for consistency sake, we need to move function __getAllEnabledRegions into class AwsRegionSelector (remember to declare "use Aws\Account\AccountClient;"
3/ Move function __promptConfirmGetAllRegions as well into class AwsRegionSelector as static function
4/ in "__tools.php",
5/ screen.php to call AwsRegionSelector::__getAllEnabledRegions instead of __getAllEnabledRegions

Hi @kuettai ,
Thank you for the feedback. I've performed the changes as suggested;

1/ Need to handle users type in "," or "," for separator (maybe trim or str_replace all to '')
This line here should do the trick. In this, we're using array map to ensure trim(), strtolower() is used as a function.

2/ I think for consistency sake, we need to move function __getAllEnabledRegions into class AwsRegionSelector (remember to declare "use Aws\Account\AccountClient;"
Done. I've moved this to the Class itself.

3/ Move function __promptConfirmGetAllRegions in "__tools.php" as well into class AwsRegionSelector as static function
Done. Refer to the point above.

4/ screen.php to call AwsRegionSelector::__getAllEnabledRegions instead of __getAllEnabledRegions
Yup, ensured the existing calls / refers were changed, in conjunction to this change.

Some tests (Just works as normal):

Cheers

Last feedback, "trim" only remove the <space> at the LEFT & RIGHT. Need better handling, instead i am thinking of just doing a str_replace(" ", "", $region) straight away.

Okay, @kuettai . I've amended the code accordingly. Thanks!