Not parsing the track titles from .cue

Hi,
Your tool is not using the title from the cue file to rename the splitted tracks
ex :

FILE "mycd.bin" BINARY
TRACK 01 AUDIO
TITLE "My Title 01"
PERFORMER "ABCD"
INDEX 01 00:00:00
TRACK 02 AUDIO
TITLE "My Title 02"
PERFORMER "DEFG"
INDEX 00 02:36:64
INDEX 01 02:38:64

I'd expect to get
My Title 01.flac
My Title 02.flac

Yes, it is lacking this feature, I made the cd parser explicitly for game cds. It is not implementing the whole cue file specification which is mostly for audio cds, so many metadatas are ignored.
Currently the only metadata CBAE can extract is the root level TITLE for the whole CD.
I could look into it, sure.
Did you encounter a game release that has metadata in the .cue file? or is it an audio CD?

Not exactly, it's actually from an audio CD that includes background music from video games! :P

Oh interesting! Turns out I did code the parser to read and store the individual track titles, but in the generateCue function they are ignored. (doh!)
Seems like a nice feature to have, I will add it soon! 😸

Pushed a new version, check it out.
Can now read both TITLE and PERFORMER tags (from tracks + root CD), and will autoname the output files accordingly,
also added an option to customize the track filenames based on a template.
I tested it on a few scenarios and it worked fine. Let me know if you encounter any weird behavior or something.

I'd love to but it seems that the windows binary is not updated ?

yeah.. when I updated the codebase to use the newer ECM modules for including files, it entirely broke the thing that I was using to convert it to .exe 😕
I'll take a look for other tools that can produce an .exe, but no promises. Try setting up node.js and installing the script with npm.


Changed install source for dask, distributed and tornado to git.

Codecov Report

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

@@           Coverage Diff           @@
##           master     #565   +/-   ##
=======================================
  Coverage   75.97%   75.97%
=======================================
  Files          75       75
  Lines        7039     7039
  Branches     1431     1431
=======================================
  Hits         5348     5348
  Misses       1518     1518
  Partials      173      173

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 71e2b50...7b78338. Read the comment docs.

is running off their master safe? I remember lots of trouble when we were following this approach with PETSc in Firedrake. Especially if these projects are particularly active... what do you think? I guess you wanna do this to get all of the latest bug fixes, correct? The risk is that we get sudden breakages of our own master.
Also, don't forget that these three packages should belong to a layer on top of that of Devito, so having them as compulsory dependencies feels weird. I think we've been there already, but since we're now also adding tornado...
If you feel that this "is just gonna be OK", then fine.
At the end of the day, the way I see it, there are three solutions:

We fork these three projects, just like we do with YASK: https://github.com/opesci/yask
This way, we would update our forked masters only when we now that the upstream ones are actually making all of our tests passing. The downside is that this require some effort on our side.
We make these three packages optionals (don't know what's the best way of doing that)
We just say "fine", we run this "risk", as this is only going to affect our Dask tests, so it shouldn't be a big deal for users

Also, can you squash your commits into a single one?
git checkout master
git pull
git checkout deadlock
git rebase -i master
<now you mark with ``s`` all commits except the root one, save and go on>
git push -f origin deadlock

There's also this: https://stackoverflow.com/questions/13685920/install-specific-git-commit-with-pip
We've touched on this before, I think.  There's a reasonably strong case for having predictable versions of dependencies used by travis.  Otherwise it breaks suddenly and we assume it's something we did, when in fact it's because some other package got upgraded.
The user-facing install should probably not do that, however.  Nominating exact version requirements is far more likely to cause unpleasant version interactions if their environments include packages not in devito's list.


Create embeddings locally

Is this possible to create embeddings locally? I mean I have few files ~200 files , I want to create embeddings locally and perform search locally instead of external embeddings providers ?
I have folder A contains 100 files .. I want to create embeddings based on those files and store into another folder.
Is this possible?

We can see if we can add local models like all-in-mini-lm for local generation.
Maybe for reference: https://github.com/langchain4j/langchain4j-embeddings


How retain login session in the in app browser?

Once login if I do not logout when I launch the android app again I shouldn't be ask login info aagain right?

I used a workaround, by adding "&auth_type=reauthenticate" to the login url, this worked as I wanted. I hope it helps.

Thanks
Sent from my iPhone

On 16 Sep 2015, at 10:18 am, brtwister2<EMAIL_ADDRESS>wrote:
I used a workaround, by adding "&auth_type=reauthenticate" to the login url, this worked as I wanted. I hope it helps.
—
Reply to this email directly or view it on GitHub.

hi
well I am too late to answer this but
Setting "cache=yes" in "function login ()>> loginWindow" in the openfb.js file works for me.


Snapshot/restore/bdd

test(snapshot/bdd): add initial restore delete

A bug was found where after snapshot delete the parent volume needs to have
ts allocation cache deleted.
( The WA for now is to restart the io-engine container and force update )

todo: add multiple layers of snapshots and restores

Signed-off-by: Tiago Castro<EMAIL_ADDRESS>

fix(snapshot/volume): split allocation counters

Splits allocation counters into replicas, snapshots and both.

Signed-off-by: Tiago Castro<EMAIL_ADDRESS>

fix(snapshot/pool): add specific error when pool cannot be used

todo: enhance filtering of resources to include reasons why a certain
resource could not be used.

Signed-off-by: Tiago Castro<EMAIL_ADDRESS>

bors merge


isort crashed five times...

Whenever I start VSC on my Windows 10 Pro notebook and connect to a remote linux server via the remote-ssh extension I get the message shown in the screenshot below:

Even if it says that it crashed five times in the last three minutes, this message is shown as soon as VSC connects to the remote server. isort is version v2022.6.0.
I uninstalled all the Microsoft python extensions, deleted the folders ~/.vscode-server/extensions/ms-python.*  and resintalled everything but the problem is still here.
Version: 1.73.1 (user setup)
Commit:<PHONE_NUMBER>f055b99068d3688932416f2346dd3b
Date: 2022-11-09T04:27:29.066Z
Electron: 19.0.17
Chromium: 102.0.5005.167
Node.js: 16.14.2
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.19043
Sandboxed: No

I got the same error
There is the log from isort.
[Info  - 6:47:31 AM] Connection to server got closed. Server will restart.
  File "/home/vscode/.vscode-server/extensions/ms-python.isort-2022.6.0/bundled/tool/server.py", line 4
    from __future__ import annotations
    ^
SyntaxError: future feature annotations is not defined

For some reason the isort server is trying to start with a Python 2 interpreter. If set the isort.interpreter setting to a Python 3 interpreter (e.g. python3) then isort works again.
I get a slightly different error in the console than the one above:
[Info  - 12:39:53 PM] Connection to server got closed. Server will restart.
[Error - 12:39:53 PM] Connection to server is erroring. Shutting down server.
  File "/home/vscode/.vscode/extensions/ms-python.isort-2022.6.0/bundled/tool/server.py", line 19
    def update_sys_path(path_to_add: str, strategy: str) -> None:
                                   ^
SyntaxError: invalid syntax

Error on the line from __future__ import annotations means that you are using python version 3.6 or older. We dropped support for this in python extension with version 2022.9.* onwards. There are two options:
[Option1] If you have to work with python 3.6 or older, downgrade the python extension to 2022.8.*. Then uninstall isort extension.
[Option 2] Upgrade your python to 3.7 or greater.

@karthiknadig this is quite disappointing.  I don't have a choice as to what version of Python that I have available to me on all of the servers I use and legacy code with virtualenvs might have older pythons.

@alanhoyle with the Downgrade option you will get the isort support the old way. Just that it won't give you the organize imports on save feature. you will still be able to use isort.

That's true today, but one presumes that you all aren't just pushing out updates with no other features...
Yes, I get that supporting legacy versions is probably a pain....

You can specify either the path to the Python interpreter to use for executing the copy of isort we ship in the extension or the path to isort itself (that's new enough, e.g. we use the --filename flag which hasn't always been there). See the settings on how to specify either.

Today VSCode auto update itself to version 1.74.1 and it starts to show me this error, right after runnig the editor. No remote session involved, although I have some extension installed. I haven't changed anything else in the system.
My default Python is 3.10.4.
I need help.
Version: 1.74.1 (system setup)
Commit: 1ad8d514439d5077d2b0b7ee64d2ce82a9308e5a
Date: 2022-12-14T10:30:51.966Z
Electron: 19.1.8
Chromium: 102.0.5005.167
Node.js: 16.14.2
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 6.3.9600
Sandboxed: No

This is how I fixed it for me:
$ cd projects

$ mkdir .isorts-special-little-house

$ cd .isorts-special-little-house

$ pyenv shell 3.11.1

$ python -m venv --upgrade-deps .venv

$ source .venv/bin/activate

$ pip install --progress-bar off -q isort

$ which python
/home/lexi/projects/.isorts-special-little-house/.venv/bin/python

and then I set
    "isort.interpreter": [
        "/home/lexi/projects/.isorts-special-little-house/.venv/bin/python"
    ],

in my user settings and it's not crashed since.
It would have been nice if the extension had done things like

Warn me that this was about to happen
Given me a sensible error message about using the wrong version of Python
Managed its own python env perhaps
Fallen back to calling isort ${filename} in the project's environment so the menu option didn't just disappear unexpectedly
but hey, it's Microsoft.

Where can I see error log?

You can see it in the Output panel and selecting the "isort" channel.

... a snarky comment made at our team's expense.

I am going to ask that people please be patient and civil about this issue. I'm sorry that your situations don't allow you to necessarily use a newer version of Python, but we have to draw a line somewhere for what Python versions we support, and so we have chosen to support Python versions that are supported upstream by the Python team.
We do hear your suggestions and will consider how we might be able to make this easier to deal with when it happens, but also know that our team has been down by 50% since this issue was reported due to vacations, so our time has been unfortunately spent elsewhere across all of the Python extensions we maintain.

I fully respect your right not to support out of date versions of python but perhaps when you drop 3.7 in 6 months time bear in mind that your extension will be autoupdated by the 10 million people who have it installed and crashing with an import error hidden in the output panel is a really confusing way to find out that your virtual environment is no longer supported.

I am having trouble with the extension auto-updating no matter what I do.
Perhaps there's something that I'm overlooking?
-a
--

Alan Hoyle  -  @.***  -  http://www.alanhoyle.com/  -

On Thu, Dec 15, 2022 at 4:02 PM Lexi Robinson @.***>
wrote:

I fully respect your right not to support out of date versions of python
but perhaps when you drop 3.7 in 6 months time bear in mind that your
extension will be autoupdated by the 10 million people who have it
installed and crashing with an import error hidden in the output panel is a
really confusing way to find out that your virtual environment is no longer
supported.
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/vscode-isort/issues/152#issuecomment-1353702640,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AACGX45ROMGVSBDPAWVIH5LWNOBMRANCNFSM6AAAAAAR4JEPYE
.
You are receiving this because you were mentioned.Message ID:
@.***>

I am having trouble with the extension auto-updating no matter what I do.

How are you selecting an older version of the extension?

bear in mind that your extension will be autoupdated by the 10 million people who have it installed and crashing with an import error hidden in the output panel is a really confusing way to find out that your virtual environment is no longer supported.

As I said, we will be taking all of this into consideration. I expect we will be adding some UX for when people try to run the extension w/o isort installed where their version isn't supported by isort (as happened here), or if our infrastructure code has to move on beyond the Python version isort supports.

This is how I fixed it for me:

I wonder if using pipx will be simpler here. So just
pipx install isort
and we done.

I also had this error message in WIN11, where i def. dont have any python2 installed.
After setting the "isort.interpreter": ["python"] it works again.
But how will this be different to the default behaviour?
Because its still just called "python".

After setting the "isort.interpreter": ["python"] it works again.

This seems like a bug in Python extension when providing the path to selected python. Please file a bug on the python extension with logs from Output > Python where you can reproduce this issue.

This should now be fixed. It should give you details on where it is getting python form, and if it is not able to get a python it will provide details on what to do next.


Error: /articles/2/0/6/7/4/elife-20674-v1.xml could not be found

https://sentry.io/codeforscience/sciencefair/issues/304052017/
Error: /articles/2/0/6/7/4/elife-20674-v1.xml could not be found
  at notFound (/opt/ScienceFair/resources/app.asar/node_modules/append-tree/index.js:669:13)
  at apply (/opt/ScienceFair/resources/app.asar/node_modules/thunky/index.js:44:12)

@mafintosh the above error is being reported by a small number of clients. They are using the hyperdrive:07e82e6ea1ab7e7c3d4872f035b2320e8aefe26ba35033b983f4b2e033d07d44, which definitely has en entry for /articles/2/0/6/7/4/elife-20674-v1.xml.
Any suggestion as to why it would throw this error?

Note, this is only happening on linux clients

@blahah my guess is that you are accessing the hyperdrive archive at version 0 on linux sometimes. lets look at it tmw :)

I'm having a similar issue whenever I attempt to download any article. I get the following:
/opt/ScienceFair/resources/app.asar/node_modules/raven-js/src/console.js:26 Content server serving /home/billbrod/Documents/sciencefair/datasources at 8000
/opt/ScienceFair/resources/app.asar/node_modules/fs-readstream-progress/index.js:19 Uncaught Error: /articles/0/5/2/4/1/elife-05241-fig1-v1-600w.jpg could not be found
    at notFound (/opt/ScienceFair/resources/app.asar/node_modules/append-tree/index.js:669)
    at /opt/ScienceFair/resources/app.asar/node_modules/append-tree/index.js:110
    at /opt/ScienceFair/resources/app.asar/node_modules/append-tree/index.js:362
    at /opt/ScienceFair/resources/app.asar/node_modules/append-tree/index.js:372
    at /opt/ScienceFair/resources/app.asar/node_modules/hypercore/index.js:120
    at apply (/opt/ScienceFair/resources/app.asar/node_modules/thunky/index.js:44)
    at _combinedTickCallback (internal/process/next_tick.js:80)
    at process._tickDomainCallback (internal/process/next_tick.js:128)
notFound @ /opt/ScienceFair/resources/app.asar/node_modules/append-tree/index.js:669
(anonymous) @ /opt/ScienceFair/resources/app.asar/node_modules/append-tree/index.js:110
(anonymous) @ /opt/ScienceFair/resources/app.asar/node_modules/append-tree/index.js:362
(anonymous) @ /opt/ScienceFair/resources/app.asar/node_modules/append-tree/index.js:372
(anonymous) @ /opt/ScienceFair/resources/app.asar/node_modules/hypercore/index.js:120
apply @ /opt/ScienceFair/resources/app.asar/node_modules/thunky/index.js:44
_combinedTickCallback @ internal/process/next_tick.js:80
_tickDomainCallback @ internal/process/next_tick.js:128

I get this for the jpg of every file in the image and for the xml file, as mentioned in the first post. I am running the Linux client (on Ubuntu 16.04 LTS). Has any progress been made on this?

Curiously, this paper and the associated data are missing from the original dataset. I am not quite sure how they came to be indexed in the dat, but subsequently removed and unavailable in the dat, but I will investigate.

I've had this happen for every paper I've attempted to download, am I just incredibly unlucky?

@billbrod OK no that sounds like a different bug. Sorry for the poor experience. Investigating this now too!

No problem, thanks for looking into it!


Call to private method CloudflareBypass\RequestMethod\Stream::getHeader()

Description

When try to use with latest CloudflareBypass from Karyna's latest version.
Version you are using (you may find this information in your composer.lock file)
"kyranrana/cloudflare-bypass": "^v2.0",
"jaymoulin/guzzlehttp-cloudflare": "^0.2.0"
 Uncaught Error: Call to private method CloudflareBypass\RequestMethod\Stream::getHeader() from context 'GuzzleCloudflare\Middleware' in /var/www/fetch/vendor/jaymoulin/guzzlehttp-cloudflare/src/Middleware.php:116
Stack trace:
#0 /var/www/fetch/vendor/jaymoulin/guzzlehttp-cloudflare/src/Middleware.php(69): GuzzleCloudflare\Middleware->hackRequest(Object(GuzzleHttp\Psr7\Request))
#1 /var/www/fetch/vendor/jaymoulin/guzzlehttp-cloudflare/src/Middleware.php(54): GuzzleCloudflare\Middleware->checkResponse(Object(GuzzleHttp\Psr7\Request), Object(GuzzleHttp\Psr7\Response), Array)
#2 /var/www/fetch/vendor/guzzlehttp/promises/src/FulfilledPromise.php(39): GuzzleCloudflare\Middleware->GuzzleCloudflare\{closure}(Object(GuzzleHttp\Psr7\Response))
#3 /var/www/fetch/vendor/guzzlehttp/promises/src/TaskQueue.php(47): GuzzleHttp\Promise\FulfilledPromise::GuzzleHttp\Promise\{closure}()
#4 /var/www/fetch/vendor/guzzlehttp/promises/src/Promise.php(246): GuzzleHttp\Promise\TaskQueue->run(true)
#5 /var/www/fetch/vendor/guzzlehttp/p in /var/www/fetch/vendor/jaymoulin/guzzlehttp-cloudflare/src/Middleware.php on line 116

Note to the owner. Just replace getHeader with getRequestHeader
A good place to look for public methods you can use is Base/RequestMethod/RequestMethod.php
Love what you done here by the way. great work @jaymoulin

@KyranRana Thank you for your message and fixing the bug :smile: (even though you introduced it :rofl: )


Quarantine branch

Editing Homer's The Iliad for INFO6540 GitHub assignment. I believe that I have completed all of the steps.

Thank you, and goodbye


OpenWrt Package installation fails

When I try to install my packages. It works first time. Executing the playbook again throws an error. I assume it tries to install ""

fatal: [openwrt_router]: FAILED! => {
"changed": false,
"invocation": {
"module_args": {
"executable": null,
"force": null,
"name": [
""
],
"state": "present",
"update_cache": true
}
},

---
- name: Set up and configure routers
  hosts: routers
  become: true
  gather_facts: false

  tasks:
    - name: Ensure Python is installed
      ansible.builtin.include_role:
        name: python
    - name: Gather facts
      vars:
        ansible_user: root
      ansible.builtin.setup:
    - name: Install packages
      ansible.builtin.include_role:
        name: imp1sh.ansible_managemynetwork.ansible_openwrtpackages
      vars:
        openwrt_packages_installrole:
          - "nano"
          - "htop"
          - "stubby"

    - name: Configure router
      ansible.builtin.include_role:
        name: dhcp

Ineed more information in order to debug. Please expand verbosity. Also show playbook file, maybe inventory / variables.
A quite common problem though is that the ansible_openwrtpackages role fails due to the OpenWrt host not being update to update their package caches. Does it work, when you try to install some package manually via opkg?

Hey sorry, my bad. It turned out to be an variable name issue. After i switched to a FQDN. :+1:
Thanks!

It doesn't seem to like the FQDN ansible lint suggestes?

TASK [imp1sh.ansible_managemynetwork.ansible_openwrtpackages : List packages to install] ***
ok: [openwrt_router] => {
"openwrt_packages_install": [
"nano",
"htop",
"stubby"
]
}
That works openwrt_packages_installrole that breaks ansible_openwrtpackages_openwrt_packages_installrole


Default reconnect strategy uses exponential backoff and jitter

Description
The current default reconnect strategy increases delay linearly with no jitter. This can lead to a thundering herd problem if many clients lose their connection at once, which is common during events like a Redis upgrade.
A reconnect strategy that uses exponential backoff plus a random jitter helps mitigate this risk and is recommended in many best practices documents for Redis clients, so I propose making this the default.
The default strategy I implemented will do the following:

Delay will be equal to (n^2) * 50 ms where n is the current retry count, with a maximum value of 2000 ms.
Jitter is a random value between 0 and 200 ms.

The millisecond values I choose are somewhat arbitrary, and I'm happy to change them if we think they're not right.
Checklist

[x] Does npm test pass with this change (including linting)?
[x] Is the new or changed code fully tested?

Existing unit tests for reconnectStrategy still pass. We have no tests for the default strategy, nor am I sure if and how we'd want to test it. Any thoughts on this?

[x] Is a documentation update included (if this change modifies existing APIs, or introduces new ones)?

Hi @qsymmachus, thanks for contributing!
We are working on V5 which will add support for sentinel, RESP3, and client-side caching. If you can redo this change but for v5 it would be awesome! :)

@leibale no problem. I changed the base branch to v5 and reworked my code slightly in 4a36c6f

@leibale I responded to your latest feedback; sorry for the delay!

@leibale bumping this one! Thanks for taking another look.

@guyroyse @leibale can we merge this guy, then?
Thank you for your help!

@qsymmachus done! thanks for taking the time to contribute, and sorry for the delays..


Remove dependency on dist (which is not available for most externals)

'externals' meaning SUSE-Employees here? because test-updates should not be available to anyone outside.
And then dist.nue.suse.com is as available as clouddata - but it probably makes it easier to mirror your own.
Maybe we could also s/dist/download/ to use mirrorbrain download redirection?

@bmwiedemann problem is that not all ip ranges are allowed to do nfs mounts from dist.

@bmwiedemann adjusted commit mesage


resolve escape char error when loading ayarc

On windows, the ayaDir will contain \ for example: D:\Aya\bin\.
These will be treated as an escape character by the String Parser and create nonsense.
The fix is to escape these characters, resulting in e.g. D:\\Aya\\bin\\

Thanks for your help with windows compatibility!


chore(release): v0.3.1

Version bump in package.json for release v0.3.1

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Fix ssl option

This PR is to help fix the inadvertent mistake of setting the Ssl protocol to Tls11.  It is now set back to Tls12.  I have also added an example to Wait-RabbitMqMessage Cmdlet.

Hi!
I might just need sleep, but the diff seems odd, includes additions from your previous PR?
GitHub diff:

Existing code:

Might be missing the merge commit from @gpduck that included a tls12 fix:

Not sure what the easiest route is, just accept this?  Re-clone, add doc update, resubmit PR (or, git-fu, if you prefer)?
Cheers!


refactor(booking_collection): rewrite serializable methods

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

17 of 23 (73.91%)  changed or added relevant lines in 4 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage increased (+0.2%) to 42.935%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

lib/src/model/cabin_collection.dart
0
2
0.0%

lib/src/model/booking_collection.dart
14
18
77.78%

Files with Coverage Reduction
New Missed Lines
%

lib/src/model/cabin_collection.dart
1
0%

Totals

Change from base Build<PHONE_NUMBER>:
0.2%

Covered Lines:
474

Relevant Lines:
1104

💛  - Coveralls


limactl list --json now returns an array instead of newline delimited JSON

Description
Originally reported in https://github.com/abiosoft/colima/issues/572
I don't think this was intentional, and should be fixed before the next release, as it breaks backwards compatibility.
And I think limactl inspect should be changed accordingly for consistency.

Yes, this is a bug, regression in https://github.com/lima-vm/lima/commit/75594fa33bd73a2cc80a389993c7573b4bf3ec58

PR https://github.com/lima-vm/lima/pull/1281


Support for asymmetric embedding retrieval techniques

Many of the current SOTA retrieval methods like instructor use asymmetric embeddings where the query Q and the document D use different embedding methods/prefixes (albeit in the same number of dimensions). How should we handle that? At the application level or possibly at the API level?

Thanks @creatorrr. With asymmetric embeddings the current work-around would be to supply your own vectors (i.e. using sentence transformers directly https://huggingface.co/flax-sentence-embeddings/multi-QA_v1-mpnet-asymmetric-A), write the vectors to weaviate, and then use near vector to search in Weaviate.
ccing @antas-marcin for built-in asymmetric support for the vectorizer modules.


Crash when trying to smelt mod items

Tried to smelt Chaos Ore from Silent's gems, and as soon as I try to put it in the furnace this happens:
http://pastebin.com/Yse9d1yp
Also happens with Thaumcraft's native clusters.
Running
Alt-F4-1.8.9-<IP_ADDRESS>
forge-1.8.9-<IP_ADDRESS>7-universal
Edit: Also trying to use alumentium from Thaumcraft broke things as well.
http://pastebin.com/Nz4Yuwp4
Double Edit: Trying to smelt with Coal and make smooth stone crashes as well.
http://pastebin.com/s21vpdNU

Can I get your full log?, things don't break for me with any of these items, but there is a chance that the recipe registry copying failed, because it's trying to reference a blank arraylist.

You mean my full startup log or something else?

Yes please, it'll contain information that might tell me why the recipe arraylist is empty

https://drive.google.com/file/d/0B0mCmvEcGbUUeDRJM20tQ3BRM00/view?usp=sharing
this should be it I think.

Sorry, can you test with the latest version http://minecraft.curseforge.com/projects/alt-f4/files/2275418 and then tell me what happens when you try to do the same thing as you did in the initial crash?
As well, tell my if you can see the registered recipes in JEI?
Really I need a log that has information from when you start up a world, as that's when the recipe registry list is filled.

will do,it will be a little while though. work unfortunately

Looks like it's all fixed. Possibly had the wrong version on the server as well. So I see everything/nothing crashes me now. Sorry for the hassle!


nucleoside-triphosphatase activity (GO:0017111) = EC:<IP_ADDRESS>?

Currently there is:
id: GO:0017111
name: nucleoside-triphosphatase activity
namespace: molecular_function
def: "Catalysis of the reaction: a nucleoside triphosphate + H2O = nucleoside diphosphate + phosphate." [EC:<IP_ADDRESS>]
xref: EC:<IP_ADDRESS>
xref: RHEA:23680
is_a: GO:0016462 ! pyrophosphatase activity
Problem is that EC:<IP_ADDRESS> seems to be for a specific enzyme (https://enzyme.expasy.org/EC/<IP_ADDRESS>), while the GO term is used as a grouping term for 6 is_a children with different EC/RHEA xrefs:
GO:0050333 thiamin-triphosphatase activity (EC <IP_ADDRESS> / RHEA 11744)
GO:0003774 motor activity
GO:0140603 ATP hydrolysis activity (RHEA 13065)
GO:0043273 CTPase activity
GO:0003924 GTPase activity (RHEA 19669, which maps back to EC:3.6.5.-)
GO:0050339 thymidine-triphosphatase activity (EC <IP_ADDRESS> / RHEA 19013)
So, is "GO:0017111 nucleoside-triphosphatase activity" meant to be broader than EC:<IP_ADDRESS>?  If so, then we should remove its EC:<IP_ADDRESS> xref and create a new, separate term for EC:<IP_ADDRESS> under GO:0017111.
Related point -  if GO:0003924 GTPase activity accurately maps back to EC:3.6.5.-, then maybe GO:0003924 needs moving to be a direct child of child of "hydrolase activity, acting on acid anhydrides (GO:0016817)" (= EC:3.6)??

@hdrabkin Appreciate your thoughts on this one too.

So the EC appears to just be mechanistic; the other GO terms are substrate specific.
The EC parent of 3.6.1
GO:0003924 GTPase activity (RHEA 19669, which maps back to)  ARGH!
19669 is the GTP specific substrate rexn;
The EC 3.6.1 is only found in an obsoleted GO term which was a grouping term specific to GTP.

Thanks @kaxelsen - makes sense!  It did look odd to see this GO term with a 4-digit EC xref having several children with other 4-digit EC xrefs.  As you both say, removing EC:<IP_ADDRESS> from GO:0017111 will resolve and clarify things.

@kaxelsen - one other thing, looking the EC parent here (and how it relates to GO structure):
EC:3.6.1.- is "Hydrolases -> Acting on acid anhydrides -> In phosphorus-containing anhydrides"
but there are also these 'sister' groupings:
EC:3.6.4.- Hydrolases -> Acting on acid anhydrides -> Acting on ATP; involved in cellular and subcellular movement
EC:3.6.5.-Hydrolases -> Acting on acid anhydrides -> Acting on GTP; involved in cellular and subcellular movement
Aren't ATP and GTP specific examples of "phosphorus-containing anhydrides"? Naively, that makes me think those ECs would be subclasses of 3.6.1.-?  But maybe it's the "involved in cellular and subcellular movement" aspect that distinguishes these classes and makes them a separate 3-digit grouping?  Do you mind clarifying that for me, thanks!

GO:0017111 fixed; will close ticket; Other issue maybe goes in separate ticket?

Thanks Harold.  Yes, I'll pursue my wider query about EC:3.6.-.- with Kristian elsewhere (if he doesn't answer here!) and make a new ticket if required.

@sjm41 It is exactly as you say. The enzymes in EC 3.6.4- and EC 3.6.5.- are all involved in movement. Note that most of the EC numbers earlier classified here have been transferred to the new EC 5.6.-.- (Isomerases altering macromolecular conformation) since the movement involves conformational changes.


Latest Atom broke rich presence for me

After updating to atom 1.26.0 discord wont show any details. Only Playing Atom Editor and the rich presence file icon.. bot nothing else anymore.
I am using Arch linux with the atom-editor-bin version.

"atom-discord":
behaviour:
alternativeIcon: "atom-3"
preferType: true
showFilenameOnLargeImage: true
updateTick: 150
useRestIcon: false
privacy: {}

ive added a screenshot.. i mean the filetype icon at the username after Playing Atom Editor which you only see from rich presence enabled apps

Try restarting discord and atom.

If you come to official discord server, I'll try checking your rich presence is showing to other people.

i restarted both many times.. reinstalled atom-discord, atom and discord.
what official discord server?

You could find it at the discord badge in readme file.
Please check if atom editor is added to game list.

it is.. but does not show any info.

you mean thte dev server?
https://discordapp.com/invite/discord-developers

Just send me the link to the server plz. i dont know wich readme you mean
:)

https://discord.gg/zfEs3K6
Here you are!


Get current sink track playback time

Is there any way to get current sink track playback time? I have searched in the docs but haven't found anything yet.

I have the same issue.

Hi! When trying to solve this for myself I found this test in the buffer.rs file.
    #[test]
    fn duration_basic() {
        let buf = SamplesBuffer::new(2, 2, vec![0i16, 0, 0, 0, 0, 0]);
        let dur = buf.total_duration().unwrap();
        assert_eq!(dur.as_secs(), 1);
        assert_eq!(dur.subsec_nanos(), 500_000_000);
    }

I can imagine some function in a sink that calls buf.total_duration() of the currently playing song.
Reading through the sink.rs file I can see that there is a datatype queue_tx that looks promising.
If the repository authors are comfortable with it I'd be happy to try implementing it.
    #[inline]
    pub fn get_duration(&self) -> u64 {
        // Somehow get first element of queue_tx as buffer
        // Return duration of that buffer
    }

There a bit of a problem, because the currently played track is not in the queue_tx, as it only has the "to-be-played" sources. The currently played is hold by queue_rx, which is immediately after construction moved into the OutputStreamHandle, so you don't have very much leverage on it.
Also Source::total_duration() is total track length, not remaining, not sure if relevant here.
The easiest way way to get the total_duration is on track load, create your own trackname->duration map and save the information from the Decoder you create by calling its total_duration() before appending it to the sink.

If you're still interested you could look over #510 and see if that solves part of the problem


qt55.qtbase: use more system libraries

Things done:

[x] Tested using sandboxing (nix-build --option build-use-chroot true or nix.useChroot on NixOS)
Built on platform(s)

[x] NixOS
[ ] OS X
[ ] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

More
cc @qknight @ttuegel

Hm, I'm unsure that this is a "mass" enough rebuild, but this would take whole KDE at least.

I think this is big enough to deserve the mass-rebuild tag. I have another Qt patch that I was sitting on, so I'll merge them together.

What's the effect on the Qt closure size?

Hm, good question, I haven't evaluated this. How can I (i.e., what's the command for measuring total closure size of an output)?

du -schl $(nix-store -qR $(nix-build -A pkg))

Without the patch: 654M
With: 659M
Most size is from harfbuzz (pcre16 is only 500K), but harfbuzz is also used in pango which is almost ubiquitous.

I don't understand why the closure size goes up. Without -system-harfbuzz, Qt includes its own copy of harfbuzz, which should surely increase the closure size, right?

It links it statically, dropping everything that it doesn't need -- I think this is the reason.

Well, FWIW I think that in practice harfbuzz is popular enough and this  to justify less static libraries (i.e. one place where bugs gather) -- and if harfbuzz is not in store already, increase is ~0.8%.

The odds may well change after #7701. Until then, when you link dynamically, you also add headers, docs, etc.


[BUG] mcdev init fails if installed package has insufficient privileges

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior

Expected Behavior
it should guide the user to update the permissions and afterwards wait for 10 minutes (because it takes time to cascade the updated permissions into every server of sfmc and it might fail if you try right after adding the crednetials (does not apply to new installed packages)
Steps To Reproduce

Go to '...'
Click on '....'
Run '...'
See error...

Version
4.3.4
Environment

OS:
Node:
npm:

Participation

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

Additional comments
No response

hi @FiB3 good thinking, but it's not that mcdev doesn't allow it: it will try to do whatever it can and simply issue a generic warning if access is missing. it does not test for anything in particular.
in this bug, us not testing actually led to a malfunction and I was hoping to make things more intuitive eventually.
with that said, my current client is also picky about access and hence I created a serious of new tickets to cover a more common scenario:

#1186

Ah ok, but still, mcdev did not initialised for me yesterday. With following:
...
? MID of Parent Business Unit 00000000
13:34:30 warn: Installed package has insufficient access. You might encounter malfunctions!
13:34:30 warn: Missing scope: accounts_read, accounts_write, approvals_read, approvals_write, calendar_read, calendar_write, campaign_read, campaign_write, contact_bu_mapping_create, contact_bu_mapping_delete, contact_bu_mapping_update, contact_bu_mapping_view, deep_linking_asset_delete, deep_linking_settings_read, deep_linking_settings_write, dfu_configure, event_notification_callback_create, event_notification_callback_delete, event_notification_callback_read, event_notification_callback_update, event_notification_subscription_create, event_notification_subscription_delete, event_notification_subscription_read, event_notification_subscription_update, file_locations_read, file_locations_write, journeys_aspr, journeys_delete, journeys_execute, key_manage_revoke, key_manage_rotate, key_manage_view, marketing_cloud_connect_read, marketing_cloud_connect_send, marketing_cloud_connect_write, ott_channels_read, ott_channels_write, ott_chat_messaging_read, ott_chat_messaging_send, package_manager_deploy, package_manager_package, push_read, push_send, push_write, sms_read, sms_send, sms_write, social_post, social_publish, social_read, social_write, tags_read, tags_write, tracking_events_write, users_read, users_write, webhooks_read, webhooks_write, workflows_read, workflows_write
13:34:30 info: Loading BUs
13:34:30 error: Credentials failure. SOAPError: API Permission Failed.
 $ mcdev retrieve
13:43:38 info: mcdev:: Retrieve
13:43:38 error: Could not find .mcdevrc.json in /my/path/.../.
13:43:38 error: Run 'mcdev init' to initialize your project.

Seems to me, that there's some API call, that failed due to a missing permission. :(
As per the new ticket - my Installed Package is in Parent BU, however it does not have all the permissions. So the issue seems to be a bit different (with the same outcome).

it failed when trying to load BUs. Since you entered "00000000" for the parent BU's MID I'm assuming you don't know it & didn't have access to it. The current solution requires that because it's used in multiple areas (for anything shared basically)
That's exactly the case that these other tickets are supposed to tackle.
This ticket on the other end, is supposed to make it easier for you to understand the typical problems (including yours) more easily.

U just anonymised the output. :D I put the correct Main BU MID. :)

ah, in that case, it's likely the missing accounts_read permission - just a guess. Working out what permission allowed what was never high on my list for mcdev TBH.


perf: replace empty functions with noopQrl

This is a fairly trivial improvement, but it does avoid loading empty functions on the client.
To make it work, I needed to run the simplification twice (before and after segmentation) but it's still fast.
It's on top of #6850 so don't merge this until that is merged

hmm the e2e aren't happy :(


Escape 'complex characters' characters in SurrealThing Key

This fixes keys such as Guids failing
As per documentation
https://surrealdb.com/docs/surrealql#:~:text=Alternatively complex characters within record IDs can be surrounded by the ⟨ and ⟩ characters

yeah, no problem. sounds like a plan.


Collect all Plutus dependencies in one module

This also shows how much we depend on plutus-apps.

Is this change still relevant now that we no longer intend on relying on cardano-ledger directly ? It might be, let's discuss about it here.


Update RM to Jersey 2.35

Done on top of code-generated adaptor. NOTE that moving rest-assured and junit to the bottom of the POM is NOT OPTIONAL. We need to add another user code block in the POM file template. I tried to work around with <useTestScope>false</useTestScope> but it does not give a desired effect.
com.sun.xml.bind:jaxb-impl is preferred to the jaxb-runtime from Glassfish because it's a bit leaner.
Now that we fixed many dependency mismatches with Jersey 2.35, we needed to add more overlay excludes for oauth-webapp. All of them are needed, but I think we can get rid of the WEB-INF/lib/oauth-*.jar one if we stop duplicating the oauth dependencies in the adaptor POM that are already in the overlay WAR.
Obsoletes #80

Potentially, we can simplify the dependency on Jena to only exclude commons-logging instead of excluding the httpclient: https://github.com/eclipse/lyo/commit/db238eb0292db60fec2cfc28fea110f2ddaedb84#diff-73298416a2ce32f13a821c3b7c721487bd0d3ca8b475cf55b389ed099235a633R66


Added berarma/cakephp-redsys

https://github.com/berarma/cakephp-redsys

plugin name is invalid as per guidelines
Berarma/Redsys

Please open a new PR with all the issues checked/fixed.


Could a feature (autoToRef) be implemented to automatically return (toRefs) when destructuring the state?

What problem is this solving
I actually came up with this because it is a bit annoying to add storeToRefs to each store if you want to destructuring
Proposed solution
I'm very new to this, it's the first time I do it, and while I was thinking how it could be implemented I found a repository where this function is better explained, I don't know if I can put the link?

One or multiple example would help understand what you want by auto refs and also if it's technically possible or not

Maybe this is possible with a vite plugin and I'm open to adding it to the main repo but I don't have the bandwidth to do it.
Another solution is to create your own defineStore() wrapper that calls storeToRefs() on the resulting store when passing an argument (to still enable extracting actions). Or even better a custom version that allows destructuring anything.
I think that last version is the most interesting one

this is my solution to this:
@helpers/pinia-auto-refs.js
import { storeToRefs } from "pinia";

const storeFiles = import.meta.globEager("./../store/*.js");

let storeExports = {};

for (const fileName in storeFiles) {
  const { default: storeObject } = storeFiles[fileName];

  storeExports[storeObject.$id] = storeObject;
}

export function useStore(storeName = "global") {
  if (!Object.keys(storeExports).includes(storeName)) {
    throw "Unknown pinia store name: " + storeName;
  }

  const store = storeExports[storeName]();
  const storeRefs = storeToRefs(store);

  return { ...store, ...storeRefs };
}

and unplugin autoimports config:
{
  dts: "auto-imports.d.ts",
  include: [
    /\.vue$/,
    /\.vue\?vue/, // .vue
  ],
  imports: [
    // presets
    "vue",
    "vue-router",
    {
<EMAIL_ADDRESS>["useStore"],
    },
  ],
}

add unplugin to vite config plugins:
import unpluginAutoImport from "unplugin-auto-import/vite";

//..

plugins: [
      // ...
      unpluginAutoImport(unpluginAutoImportConfig),
     // ...
]

example usage:
const { statePropExample, getterExample, actionExample } = useStore("storeName");
// leave useStore() to access to store named global -> useStore("globall")

pros:

easy to use

cons:

no intelisense

I have an alternative for my useStore function that has intelisense support
// defineRefStore.js
import { defineStore, storeToRefs } from "pinia";

export function defineRefStore(id, storeSetup, options) {
  const piniaStore = defineStore(id, storeSetup, options);

  return () => {
    const usedPiniaStore = piniaStore();

    const storeRefs = storeToRefs(usedPiniaStore);

    return { ...usedPiniaStore, ...storeRefs };
  };
}

also a d.ts file for intelisense
// defineRefStore.d.ts
import {
  DefineSetupStoreOptions,
  _ExtractStateFromSetupStore,
  _ExtractGettersFromSetupStore,
  _ExtractActionsFromSetupStore,
  StoreDefinition,
} from "pinia";

export declare function defineRefStore<Id extends string, SS>(
  id: Id,
  storeSetup: () => SS,
  options?: DefineSetupStoreOptions<
    Id,
    _ExtractStateFromSetupStore<SS>,
    _ExtractGettersFromSetupStore<SS>,
    _ExtractActionsFromSetupStore<SS>
  >
): StoreDefinition<
  Id,
  _ExtractStateFromSetupStore<SS>,
  _ExtractGettersFromSetupStore<SS>,
  _ExtractActionsFromSetupStore<SS>
>;

now when defining stores just use defineRefStore instead of defineStore from pinia
example:
import { defineRefStore } from "./defineRefStore.js";

import { ref, computed } from "vue";

export const useCartStore = defineRefStore("cart", () => {
  const products = ref([]);

  return { products };
});

NOTE I WROTE THIS FUNCTION ONLY FOR SETUP STORES

Hi，I implemented ts version based on your idea.And publish it as a Vite plugin.I used it in one of my projects and it worked fine and looked fine.It's easy to use and has intelisense, but I haven't had time to add tests to it yet.

pinia-auto-refs

Couldn't this be done as a helper of sorts?
const warehouseStore = useWarehouseState();
const { setWarehouseState } = warehouseStore;
const { warehouse, resourceVolume } = storeToRefs(warehouseStore);

As an example, this is quite annoying.
Couldn't this be wrapped up into a helper function that returns a combined object to the effect of:
function someHelper() {
    const warehouseStore = useWarehouseState();
    const { setWarehouseState } = warehouseStore;
    const { warehouse, resourceVolume } = storeToRefs(warehouseStore);

    return {
        setWarehouseState,
        warehouse,
        resourceVolume
    };
}

Or does this just end up breaking things?

Found this annoying as well. There's no point in unwrapping all refs in a setup store, since everything returned from it will already have been a ref/reactive/computed or marked raw.
I ended up writing this version of defineStore which includes a modified version of storeToRefs in it.

Keeps refs/computed/markedRaw as is
Additionally, keeps reactive as is (unlike storeToRefs which wraps them)
Removes extra methods from the store (allows combining stores)
Has working Go to definition (tested in webstorm)
Doesn't break dev tools (as far as I can tell)

https://gist.github.com/unshame/1deb26be955bc5d1b661bb87cf9efd69

const useInnerStore = defineStore('innerStore', () => {
    return {
        innerRef: ref('baz'),
    };
});

const useStore = defineStore('store', () => {
    return {
        reactiveProp: reactive({ field: 'foo' }),
        refProp: ref('bar'),
        rawProp: markRaw({ key: "I'm raw" }),

        ...useInnerStore(), // combining stores
    };
});

const { reactiveProp, refProp, rawProp, innerRef } = useStore();

console.log(reactiveProp.field); // foo
console.log(refProp.value); // bar
console.log(isRef(rawProp) || isReactive(rawProp)); // false
console.log(innerRef.value); // baz

The only downside is that some internal types had to be imported and rewritten to remove Unwrap from them.

@vieruuuu Love your work!
I think your thin wrapper over defineStore achieves the goal of removing storeToRefs() boilerplate code that this topic is about. Nonetheless I wanted to mention that if not using destructuring, there are some differences that people who are used to pinia's pattern need to be cautious, for example,
without defineRefStore:
const store = useStore();
store.prop1 = v1;
store.prop2++;

with defineRefStore:
const store = useStore();
store.prop1.value = v1;
store.prop2.value++;

Proposal for storeToRefs alternative!
--> store
const show = getter(false);
const countObj = reactiveGetter({ count: 0 });
const user = ref(false);
const getUser = () => console.log('user');

--> component
const { show, countObj, getUser } = useUserStore();

Essentially this would mean that ref and reactive would not be auto exported, getter and reactiveGetter (or whatever the term we are using here) on the other hand will be. Functions will be always exported. This way we would reduce the amount of work wherever a store is required.
To get a ref or reactive, this would mean storeToRefs would be still required.

It would be better if it didn't get converted to reactive in the first place, instead of turning it into reactive and then back into refs.

I don't know if this solves things for you guys, but I ran into this while coding an example application for an assignment:
here's my store:
import {ref} from 'vue'
import {defineStore} from 'pinia'

export const useTodoStore = defineStore('counter', () => {
    const todos = ref<{text:string, completed:boolean}[]>([])
    function addTodo(todo: {text:string, completed:boolean}) {
        todos.value.push(todo)
    }
    return {todos, addTodo}
})

here's how I destructure it
const {todos, addTodo} = useTodoStore()

this destructures the store with the todos ref still functioning as intended (it can be updated and my template adjusts accordingly) which would not be possible if I had done:
const store = useTodoStore()
const {todos, addTodo} = store

might be intended, might not be, either way, seems to be a good fit for this issue and I didn't see anyone bring it up.

Here's my version of defineStore, I've been using this for some time now, this version of defineStore won't have any noticeable runtime impact.
As a role of thumb, I don't use reactive, and this function converts every reactive  variable in store to ref.
store.ts
import { type StoreGeneric, defineStore, storeToRefs } from "pinia"
import type { ToRefs } from "vue"

// Extrancted from "utility-types" package
type NonUndefined<A> = A extends undefined ? never : A

type FunctionKeys<T extends object> = {
	[K in keyof T]-?: NonUndefined<T[K]> extends Function ? K : never;
}[keyof T]

type NonFunctionKeys<T extends object> = {
	[K in keyof T]-?: NonUndefined<T[K]> extends Function ? never : K;
}[keyof T]

/**
@description Creates a useStore function that retrieves the store instance
@param id — id of the store (must be unique)
@param storeSetup — function that defines the store
 */
export function defineComposable<Id extends string, SS>(id: Id, storeSetup: () => SS) {
	const piniaStore = defineStore(id, storeSetup)

	return () => {
		const store = piniaStore()
		const storeRefs = storeToRefs(store as StoreGeneric)

    // Pick only function properties from source store.
    // And append refs to final output.
    type StoreFunctions = Pick<typeof store, FunctionKeys<typeof store>> & NonNullable<unknown>
		type NonFunctions = ToRefs<Pick<typeof store, NonFunctionKeys<typeof store>>>

		type OmittedValues = Omit<NonFunctions, "_customProperties" | "$state" | "$id">
		type OmittedFunctions = Omit<StoreFunctions, "$onAction" | "$patch" | "$reset" | "$subscribe">

		return { ...store as OmittedFunctions, ...storeRefs as OmittedValues }
	}
}

And here's the usage example.
counter.ts
import { defineComposable } from "./store.ts"

export const useCounter = defineComposable("counter", () => {
	const count = ref(0)
	const name = ref("Eduardo")
	const doubleCount = computed(() => count.value * 2)
	function increment() {
		count.value++
	}

	return { count, name, doubleCount, increment }
})

app.vue
<script setup lang="ts">
import { useCounter } from "./counter.ts"

const counter = useCounter()
counter.count.value = 10
</script>

<template>
{{ counter.count.value }}
</template>

If you want to keep the reactive as is, use the gist provided by @unshame

Using storeToRefs to create a new defineStore
Here's my version of defineStore, I've been using this for some time now, this version of defineStore won't have any noticeable runtime impact. As a role of thumb, I don't use reactive, and this function converts every reactive variable in store to ref. File store.ts

As of 2.2.3 this no longer works, the types are lost when consuming the store. If i understand correctly it's related to #2771

Hello There !
Here's my updated version after 2.2.3 update:
store.ts
import { defineStore, storeToRefs } from "pinia"

export function defineComposable<Id extends string, SS extends () => any>(
	id: Id,
	storeSetup: SS,
): () => ReturnType<SS> {
	const piniaStore = defineStore(id, storeSetup)

	return () => {
		const store = piniaStore()
		const storeRefs = storeToRefs(store) as ReturnType<SS>
		const composable = {} as ReturnType<SS>

		for (const key in store) {
			if (key in storeRefs) {
				composable[key] = storeRefs[key]
			}
			else {
				// @ts-expect-error Incorrect store typings
				composable[key] = store[key]
			}
		}

		return composable
	}
}


for debugging purposes, we should offer an option to switch on plain text .pcap recording

for debugging purposes, we should offer an option to switch on plain text .pcap recording.
It should be very clear that it's for debugging because NIOWritePCAPHandler.SynchronisedFileSink blocks the event loop (we could write an async file sink ofc)...
A pretty terrible patch that can serve as example (but not the final impl ofc) is here:
diff --git a/Package.swift b/Package.swift
index bce6b69..0575c2d 100644
--- a/Package.swift
+++ b/Package.swift
@@ -31,7 +31,7 @@ let package = Package(
         .target(
             name: "AsyncHTTPClient",
             dependencies: ["NIO", "NIOHTTP1", "NIOSSL", "NIOConcurrencyHelpers", "NIOHTTPCompression",
-                           "NIOFoundationCompat", "NIOTransportServices", "Logging"]
+                           "NIOFoundationCompat", "NIOTransportServices", "Logging", "NIOExtras"]
         ),
         .testTarget(
             name: "AsyncHTTPClientTests",
diff --git a/Sources/AsyncHTTPClient/ConnectionPool.swift b/Sources/AsyncHTTPClient/ConnectionPool.swift
index 46e0b61..03abfc3 100644
--- a/Sources/AsyncHTTPClient/ConnectionPool.swift
+++ b/Sources/AsyncHTTPClient/ConnectionPool.swift
@@ -20,6 +20,7 @@ import NIOHTTP1
 import NIOHTTPCompression
 import NIOTLS
 import NIOTransportServices
+import NIOExtras

 /// A connection pool that manages and creates new connections to hosts respecting the specified preferences
 ///
@@ -257,6 +258,10 @@ struct ConnectionKey: Hashable {
     }
 }

+let fileSink = try? NIOWritePCAPHandler.SynchronizedFileSink.fileSinkWritingToFile(path: "/tmp/ahc-\(getpid()).pcap") { error in
+    print("AHC ERROR: something went wrong creating the file sink: \(error)")
+}
+
 /// A connection provider of `HTTP/1.1` connections with a given `Key` (host, scheme, port)
 ///
 /// On top of enabling connection reuse this provider it also facilitates the creation
@@ -529,9 +534,22 @@ class HTTP1ConnectionProvider {
             let requiresSSLHandler = self.configuration.proxy != nil && self.key.scheme.requiresTLS
             let handshakePromise = channel.eventLoop.makePromise(of: Void.self)

+
             channel.pipeline.addSSLHandlerIfNeeded(for: self.key, tlsConfiguration: self.configuration.tlsConfiguration, addSSLClient: requiresSSLHandler, handshakePromise: handshakePromise)

+            func addWritePCAP() -> EventLoopFuture<Void> {
+                if let fileSink = fileSink {
+                    return channel.pipeline.addHandler(NIOWritePCAPHandler(mode: .client, fileSink: fileSink.write(buffer:)),
+                                                position: .last)
+                } else {
+                    return channel.eventLoop.makeSucceededFuture(())
+                }
+
+            }
+
             return handshakePromise.futureResult.flatMap {
+                addWritePCAP()
+            }.flatMap {
                 channel.pipeline.addHTTPClientHandlers(leftOverBytesStrategy: .forwardBytes)
             }.flatMap {
                 #if canImport(Network)

more hacky but newer patch for this
diff --git a/Package.swift b/Package.swift
index f72de84..d572ea0 100644
--- a/Package.swift
+++ b/Package.swift
@@ -39,6 +39,7 @@ let package = Package(
                 .product(name: "NIO", package: "swift-nio"),
                 .product(name: "NIOCore", package: "swift-nio"),
                 .product(name: "NIOPosix", package: "swift-nio"),
+                .product(name: "NIOExtras", package: "swift-nio-extras"),
                 .product(name: "NIOHTTP1", package: "swift-nio"),
                 .product(name: "NIOConcurrencyHelpers", package: "swift-nio"),
                 .product(name: "NIOHTTP2", package: "swift-nio-http2"),
diff --git a/Sources/AsyncHTTPClient/ConnectionPool/HTTPConnectionPool+Factory.swift b/Sources/AsyncHTTPClient/ConnectionPool/HTTPConnectionPool+Factory.swift
index 1444df9..bd57a18 100644
--- a/Sources/AsyncHTTPClient/ConnectionPool/HTTPConnectionPool+Factory.swift
+++ b/Sources/AsyncHTTPClient/ConnectionPool/HTTPConnectionPool+Factory.swift
@@ -20,6 +20,8 @@ import NIOPosix
 import NIOSOCKS
 import NIOSSL
 import NIOTLS
+import NIOExtras
+import Foundation
 #if canImport(Network)
 import NIOTransportServices
 #endif
@@ -448,6 +450,17 @@ extension HTTPConnectionPool.ConnectionFactory {
                     .tlsOptions(options)
                     .channelInitializer { channel in
                         do {
+                            let fileSink = try! NIOWritePCAPHandler.SynchronizedFileSink.fileSinkWritingToFile(path: "/tmp/packets-nw-\(sslServerHostname ?? "unknown")-\(UUID())-\(getpid())-\(Date()).pcap",
+                                                                                errorHandler: { error in
+                                logger.error("ouch: \(error)")
+                            })
+
+                            try sync.addHandler(NIOWritePCAPHandler(mode: .client,
+                                                                    fileSink: fileSink.write(buffer:)))
+                            channel.closeFuture.whenComplete { _ in
+                                try! fileSink.syncClose()
+                            }
+
                             try channel.pipeline.syncOperations.addHandler(HTTPClient.NWErrorHandler())
                             try channel.pipeline.syncOperations.addHandler(NWWaitingHandler(requester: requester, connectionID: connectionID))
                             // we don't need to set a TLS deadline for NIOTS connections, since the
@@ -485,6 +498,16 @@ extension HTTPConnectionPool.ConnectionFactory {

                         try sync.addHandler(sslHandler)
                         try sync.addHandler(tlsEventHandler)
+                        let fileSink = try! NIOWritePCAPHandler.SynchronizedFileSink.fileSinkWritingToFile(path: "/tmp/packets-\(sslServerHostname ?? "unknown")-\(UUID())-\(getpid())-\(Date()).pcap",
+                                                                                errorHandler: { error in
+                            logger.error("ouch: \(error)")
+                        })
+
+                        try sync.addHandler(NIOWritePCAPHandler(mode: .client,
+                                                                fileSink: fileSink.write(buffer:)))
+                        channel.closeFuture.whenComplete { _ in
+                            try! fileSink.syncClose()
+                        }
                         return channel.eventLoop.makeSucceededVoidFuture()
                     } catch {
                         return channel.eventLoop.makeFailedFuture(error)


Setup as nuxt 3 plugin problem

After upgraded to new version 3.0,0, there is a problem with Nuxt. It worked perfectly with old version 2.0.1
import OpenlayersMap from 'vue3-openlayers'

export default defineNuxtPlugin((nuxtApp) => {
  nuxtApp.vueApp.use(OpenlayersMap)
})

Problem Message:
  Type '{ install: ((app: App<any>, ...options: any[]) => any) & { install?: ((app: App<any>, ...options: any[]) => any) | undefined; }; }' is not assignable to type '{ install: (app: App<any>) => any; }'.
    Types of property 'install' are incompatible.
      Type '((app: App<any>, ...options: any[]) => any) & { install?: ((app: App<any>, ...options: any[]) => any) | undefined; }' is not assignable to type '(app: App<any>) => any'.

Maybe from the plugin install declaration not matching with nuxt plugin declaration

What nuxt version are you using? I don't have this problem and my install declaration is accepting both the plugin and the options.

It wont work with nuxt 3.7.4 or below, it works with nuxt 3.8.0


PathPrefix / not working

Do you want to request a feature or report a bug?
Bug
Did you try using a 1.7.x configuration for the version 2.0?

[ ] Yes
[x] No

What did you do?
read the documentation, tried to catch all requests and pass them to a defined frontend
      rule = "PathPrefix(`/`)"

# i also tried
      rule = "Host(`<IP_ADDRESS>`)"
      rule = "Host(`localhost`)"
      rule = "Host(`*`)"
      rule = "Host(`.*`)"
      rule = "HostRegexp(`.*`)"

What did you expect to see?
traefik to forward the request internally to my nginx on port 8080
What did you see instead?
404 page not found
or
Internal Server Error
Output of traefik version: (What version of Traefik are you using?)
/ $ traefik version
Version:      2.0.0-alpha4
Codename:     faisselle
Go version:   go1.12.4
Built:        2019-04-17T11:38:21Z
OS/Arch:      linux/amd64

What is your environment & configuration (arguments, toml, provider, platform, ...)?
[global]
  checkNewVersion = false
  sendAnonymousUsage = false

[entrypoints]
    [entrypoints.web]
    address = ":40700"
    [entrypoints.admin]
    address = ":40701"

[log]
logLevel = "INFO"
filePath = "/var/log/traefik/traefik.log"
format = "json"

[accessLog]
filePath = "/var/log/traefik/access.log"
format = "json"

[api]
  entryPoint = "admin"
  dashboard = true

[ping]
  entryPoint = "admin"

[providers]
  [providers.file]
    filename = "/etc/traefik/rules.toml"
    watch = true

[http] # http routing section
  [http.routers]
    [http.routers.to-web] # Define a connection between requests and services
      entrypoints = ["web"]
      rule = "HostRegexp(`.*`)"
      service = "web" # If

[http.services]
       [http.services.web.LoadBalancer]
          [[http.services.web.LoadBalancer.servers]]
             url = "http://<IP_ADDRESS>:8080"

finally got it to work.
don't skip the weight on the route!
[http.services]
  [http.services.web.LoadBalancer]
    [[http.services.web.LoadBalancer.servers]]
      url = "http://<IP_ADDRESS>:8080"
      weight = 1 # without this it will not work


feat: Add classes to image emojis

Successor to #113
Addresses : https://github.com/UziTech/marked-emoji/pull/113#issuecomment-1646451888
Credit : @HowToCodeDE

Tests are green @UziTech

I'm still uncertain that this will be a positive change. It seems like adding classes and styling should be separate from marked.
Not to mention that there is an easy work around in https://github.com/UziTech/marked-emoji/issues/112#issuecomment-1646454563


fix(app): just add another console log

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


Adjust colliding aggregator cache IDs.

Renumbered ApproximateHistogramAggregatorFactory from 8 to 12,
8 was taken by CardinalityAggregatorFactory
Renumbered ApproximateHistogramFoldingAggregatorFactory from 9 to 13,
9 was taken by FilteredAggregatorFactory

👍


Two negating pins for dependencies - libnetcdf and numpy

🚀 Pull Request
libnetcdf !=4.9.1
Closes #5245, closes #5187. Thanks very much @zklaus for sorting this out in conda-forge/libnetcdf-feedstock#175 🌟. I've tested locally and stdout is no longer polluted with numerous HDF5 errors.
numpy !=1.24.3
Updating the lock files introduced errors in iris.tests.test_analysis.TestLatitudeWeightGeneration:

test_cosine_latitude_weights_2d
test_cosine_latitude_weights_2d_latitude_first
test_cosine_latitude_weights_2d_latitude_last

Breadcrumb trail starts with numpy/numpy#23651. This relates to ordering in masked arrays, and having looked at the way the tests are constructed, I don't believe they are asserting for the wrong answer.

Consult Iris pull request check list

numpy !=1.24.3

I can't be 100% sure if the below was originally written accounting for broken NumPy behaviour that has now been fixed. Anyone care to weigh in?
https://github.com/SciTools/iris/blob/1afae0d2e24b503c92a9c1b66ef311dca6cc2e28/lib/iris/analysis/cartography.py#L559-L569

@trexfeathers We can watch numpy and see whether the 1.24.3 avoidance resurfaces in their next minor/patch release 👍


Performance regression Date.toISOString()

These are the last results of nodejs-bench-operations

v18 - https://github.com/RafaelGSS/nodejs-bench-operations/blob/main/RESULTS-v18.md#date-toisostring
v19 - https://github.com/RafaelGSS/nodejs-bench-operations/blob/main/RESULTS-v19.md#date-toisostring

and as you can see, toISOString might have been affected by a regression.

And it seems to be there since v19.3.0 (at least) https://github.com/RafaelGSS/nodejs-bench-operations/tree/main/v19.

We updated to V8 10.8 in v19.3.0. Could this be related?
[c8421204b0] - deps: patch V8 to <IP_ADDRESS> (Michaël Zasso) https://github.com/nodejs/node/pull/45749

No. This was caused by a semver-major change. I've adjusted the repository to run the benchmark against 18.0.0 and 19.0.0 and the regression is still there.

v19.0.0 https://github.com/RafaelGSS/nodejs-bench-operations/blob/main/v19/RESULTS-v19_0_0.md#date-toisostring
v18.0.0 https://github.com/RafaelGSS/nodejs-bench-operations/blob/main/v18/RESULTS-v18_0_0.md#date-toisostring

It's probably the update to V8 10.7

Implementation of toISOString was changed in https://github.com/v8/v8/commit/126d4779256509a492671bba8bdc440916a9f89a

Is there an open issue in the V8 bug tracker for this?

probably not

Is the regression still there with V8 11.3? @RafaelGSS

I don't think this is actionable?

Yeah, it's more a tracking issue in case someone wants to fix it in the upstream.


Import a full org setup

(Excuse me for skipping the template, this is a simple case of doc suggestion, feel free to just close if not relevant)
Migrating an existing org, with many repos and existing teams setup is, I am sure, a common use case that is really nicely solved with:
https://github.com/chrisanthropic/terraform-import-github-organization
At the very least, I suggest to add to readme a note about this project (happy to PR if this makes sense).
If there is a nice way to import that using the provider, I suspect it's not an easy to find in the docs, happy to contribute there too, with a relevant pointer.

🤔 maybe we add a new md file with recommended tooling / examples. I've had other projects reach out in the past as well. Happy to review a PR related to this effort if you'd like to get one started 😀

Closing this for now, referenced project is a bit stale, so not sure I can whole heartily recommend this in the official TF repo.
Maybe this issue in and of itself will help someone at some future.


Question: is there any way to control window transparency with button control in winui3+win32

I need to create mainwindow in that child window and I need to add 2 buttons in child window first btnclick need to control transparency of Main window and 2 nd btnclik2 need to remove transparency of Mainwindow is this possible to do using winui3 or win32?

You can just use SetLayeredWindowAttributes :

Can u please send be code for this please
On Fri, 19 Nov, 2021, 5:09 pm castorix, @.***> wrote:

You can just use SetLayeredWindowAttributes :
[image: WinUI_Transp]
https://user-images.githubusercontent.com/22345506/142616842-7e985076-316e-47a5-a089-8b638cc8d1fb.gif
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/microsoft/microsoft-ui-xaml/issues/6351#issuecomment-973998882,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AWRMZ3NEMVUDJAAGIXDGPY3UMYZQ3ANCNFSM5ILN3XRQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

@castorix can u please send a bit of source code of ur application im making as exactly urs it would be helpful if u send

I had posted a test sample in this thread Need my button click also should work with transparency and dont want the app to gain the focus.

Can we make multiple window ? In winui3 like window on window so that I can
put my controls on 2nd window and I will try to control transparancy of
first window through button click so that my controls won't get transparent
On Tue, 23 Nov, 2021, 2:27 pm castorix, @.***> wrote:

I had posted a test sample in this thread Need my button click also
should work with transparency and dont want the app to gain the focus.
https://docs.microsoft.com/en-us/answers/questions/630720/need-my-button-click-also-should-work-with-transpa.html
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/microsoft/microsoft-ui-xaml/issues/6351#issuecomment-976296593,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AWRMZ3MXFCKRRUZMYTE2UYTUNNJOPANCNFSM5ILN3XRQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

@codendone I think transparent windows are not supported in 1.0, correct?


Misleading success message

When running gem release a success message gets printed even if the release didn't succeed (I disconnected from internet for testing purposes).
Check this line:
https://github.com/svenfuchs/gem-release/blob/06106d11d6d31f013e55e16894c7944f6440f856/lib/rubygems/commands/release_command.rb#L43

This should be fixed in 2.0.0.pre.1, see https://github.com/svenfuchs/gem-release/pull/64
$ gem release
Releasing foo with version 0.0.5

Building foo.gemspec
$ gem build foo.gemspec
  Successfully built RubyGem
  Name: foo
  Version: 0.0.5
  File: foo-0.0.5.gem

Pushing foo-0.0.5.gem
$ gem push foo-0.0.5.gem --key localhost --host http://localhost:9292
Pushing gem to http://localhost:9292...

Deleting left over gem file foo-0.0.5.gem
$ rm -f foo-0.0.5.gem
ERROR:  While executing gem ... (Errno::ECONNREFUSED)
    Failed to open TCP connection to localhost:9292 (Connection refused - connect(2) for "localhost" port 9292)

Thanks for reporting, @jekuno! I'm going to close this here.


bgfxplatform.h -> platform.h in documentation

I'm not sure where the github is for the documentation to put the issue there (or pr fix it), so posting here instead:
In https://bkaradzic.github.io/bgfx/overview.html#sdl-glfw-etc
#include <bgfx/bgfxplatform.h>
should be
#include <bgfx/platform.h>

Fixed.


[Ticker] Adds all the ticker endpoints

I added a structure and did some abstractions that I believe to be
common place in a REST API. If they prove to be wrong, we can roll them
back.
I added the concept of a multi find, because the finding a ticker by id
returns an array, which is not the norm, and thus is not the regular
find.
Closes #4.

:+1: after renaming Base


Need globally unique buckets

Add a prefix parameter passed when calling cdk (synth, deploy) to prefix the bucket names.

I'm currently using the bucket names, so any other deployments will fail.
I'm not using the auto-generated names for the buckets since you need to upload to them, and the auto generated names are gook.

Done in 69d4e929968d27038251f0f53259cc32ad8b6e1d


Add led path to 6UL-VAR-DART, update provisioning instructions for iMX7-VAR-SOM

Fixes: https://github.com/balena-os/balena-variscite/issues/129
Fixes: https://github.com/balena-os/balena-variscite/issues/37

@balena-ci I self-certify!


workflow-controller pod doesn't expose any ports but replies to /metrics on 8080

Summary

workflow-controller pod doesn't expose any ports but replies to /metrics on 8080
workflow-controller-metrics service doesn't respond to /metrics

As a result, this is breaking prometheus' service discovery based scraping.
Diagnostics
What Kubernetes provider are you using?
Azure Kubernetes Service
What version of Argo Workflows are you running?
2.11.0
Impacted by this bug? Give it a 👍. We prioritise the issues with the most 👍.

Not sure what the issue is here. Can you clarify what your ideal fix would be and why?

I'd expect svc/workflow-controller-metrics to serve the metrics on /metrics endpoint. Right now, the metrics are not served from either a discoverable port, or a discoverable service.

@agniveshadhikari do you have an idea of the correct YAML for the service? Can you share anything about the Prometheus auto-discovery?


Primary controller w/ WiFi Bridge mode

Need a primary controller that is driving the Tank LEDs but also pushing out a signal that all other Tubes will obey
This must coexist with WiFi UI for the Tank

WiFi coexistence seems to be causing issues right now - tubes can't connect to wifi?

Going a different direction - by default, will not connect to WiFi at all


Lossless webp?

First of all, I'd like to thank you for this awesome project, it solved a problem I've had for years with node-canvas: https://github.com/mozilla/pdf.js/issues/17188
I know the current version is at pre-release stage, although I'd like to report something, which I think is already "checked" on the roadmap, although I don't think it's actually finished completely.
canvas.toBuffer("image/webp", 100) is not working perfectly. First of all, it's not very obvious what should happen when I provide 100 for quality. Does the image become a lossless webp format in this case? Or it's still lossy with a large filesize?
Based on my tests, I've found that when I use image/webp, it's significantly slower than if I use image/png (about 18% slower), and the filesize produced by it it significantly larger - it produces around 73% larger files compared to when I use imagemin-webp
Can we somehow improve these things in the future?

canvas.toBuffer("image/webp", 100)

Yes, this is lossless

Based on my tests, I've found that when I use image/webp, it's significantly slower than if I use image/png (about 18% slower)

With same quality?

it produces around 73% larger files compared to when I use imagemin-webp

The same quality or less quality with imagemin-webp?

With same quality?
png is always lossless. I don't think "quality" makes sense when it comes to "lossless" formats.
The same quality or less quality with imagemin-webp?
I'm using lossless option for imagemin-webp, and it that case, the quality "number" is ignored, I believe. This is how I use it:

await imagemin([`${source}/*.png`], {
	destination: destination,
	plugins: [
		imageminWebp({
			lossless: true,
		}),
	],
});

By the way, I just realized the quality shouldn't be 100, because the valid range is between 0 and 1 (at least in mozilla's documentation for HTMLCanvas). However, changing it from 100 to 1 in @napi-rs/canvas significantly reduces the filesize, but the quality is also compromised.
So I guess you're using the range between 0 and 100? Shouldn't that be consistent with the browser's canvas and set it between 0 and 1?

Thanks!

What system and CPU are you using for test? On my MacBook Pro, the webp is faster than png

What system and CPU are you using for test? On my MacBook Pro, the webp is faster than png

Wow, that's interesting!
I'm using an i5 7600-K CPU on an Ubuntu 22 system. I can try on windows as well, give me a few minutes.

Probably I was wrong... Not exactly sure how, but I swear, the first time I tried, I measured 15% slower times with webp than with png. Now it seems to be the other way around, webp is indeed faster.
So I guess the only remaining issue is the filesize, which is - for some reason - a lot smaller when I use imagemin
Thank you for your quick feedbacks @Brooooooklyn

The underlying webp encoder in this library is libwebp, the same with underlying library in imagemin-webp: https://github.com/imagemin/imagemin-webp/blob/main/index.js
The size differences come from the imagemin-webp call the cwebp under the hood and it would apply 75 quality factor to the original webp: https://github.com/webmproject/libwebp/blob/main/examples/cwebp.c#L548
Which means whatever your input webp images are  optimized, it would be smaller after processed by imagemin-webp

@Brooooooklyn I'm a bit confused. 75 quality would mean it's NOT lossless, right? But then, how is it still lossless?


Small hisilicon defconfig fix to BR2_EXTERNAL_ variable

Thank you so much for the corrections.
I will include your patch in the system soon.


Add marlowe-run to the deployment

From @jonringer :

allocations should be mounted at /var/lib/nomad/$NOMAD_ALLOC_ID

Will update this.

the volume stanza should probably be used to mount a directory inside of the allocation.


Backdrops no longer working with latest update

I'm not sure what changed, but it broke backdrops integration.

Sounds like you should contact Backdrops then, given that every other Muzei source continues to work and Muzei 3.4 is backward compatible with sources using previous versions of Muzei's API.
Installing Backdrops myself, the app doesn't seem to be loading any wallpapers at all at the moment, much less via Muzei.

If no images appear when you tap "Browse" under Backdrops, then the issue is on Backdrops side - it isn't sending any wallpapers to Muzei to display. That would be an issue on their side and you should contact them.

Yup, they're looking into it. I emailed them on the weekend.


Activate the health module

You need this on the compose?

I used it on skywalking-client-test.

From compose files in main repo, you need both of them:
SW_HEALTH_CHECKER: default
SW_TELEMETRY: prometheus

@hanahmily  Done, please review  :)


People listed as NomCom candidates should be linked to their personal profiles

Description
Similar to #7031, when reviewing people for NomCom, it would be useful to be able to see their personal profiles easily.
Code of Conduct

[X] I agree to follow the IETF's Code of Conduct

Be aware of the discussion at #6068

Added contrib easy as turning the list of names into links to profile pages will be simple.

If you're talking about the "Enter feedback" tab, which then provides a list of names on the right-hand side, I am confused.  The names are links as shown by this screenshot.

If you click on someone's name, you get the "enter feedback" page.  And if you hover over their name is says "view their datatracker profile"  With a possibly-confusing envelope icon next to it:

So what should be done here?

the envelope should be a link to send them email.

Yeah, but it's a separate link from the "point to datatracker profile" which is why I called it possibly-confusing.
I think it should be with_email=False since you can get their email address from the public profile.

@paulehoffman reopen if you were looking at a different spot for links


Support for JSR-303 Custom ConstraintValidator

I would like to be able to specify something in the Swagger definition of an object that allows code-generation to annotate the model with @MyConstraintValidator where MyConstraintValidator.java is an existing constraint validator defined something like the simple constraint definition defined here: http://beanvalidation.org/1.0/spec/#example-definition-notnull

swagger-tools doesn't do any codegen.  You should look into swagger-codegen.


Fix deprecation warning

DeprecationWarning: The loop argument is deprecated since Python 3.8, and scheduled for removal in Python 3.10.

LGTM
@cybergrind How do you think of this PR?

I'll be happy to remove remaining loop arguments if it's needed :)

@decaz yep, I think that we can completely ditch the loop parameter, and if you have time and desire it would be great to have this change merged.


added hint to .travis.yml about deactivated state

added hint that the main CI is http://ci.hibernate.org/ and that Travis is supposed to be used for github forks (see https://hibernate.atlassian.net/browse/HSEARCH-2712)

Can one of the admins add this person to the trusted builders? (reply with: "add to whitelist" or "ok to test")

thanks, good idea!


fix: webchat markdown bug

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Update limits-specifications-teams.md

Update limit for Assignment and Classnotebook from 200 to 300

Updated new limits

@JohanFreelancer9 Copy editing is needed for this PR. Thanks.

Updated

@SerdarSoysal PR has been copyedited and is ready for final review, could you please check and merge? Thanks!

Can we get this PR published out on Docs
Leif
From: JohanFreelancer9 @.>
Sent: Friday, May 14, 2021 10:15 PM
To: MicrosoftDocs/OfficeDocs-SkypeForBusiness @.>
Cc: Leif Brenne (HE/HIM) @.>; Mention @.>
Subject: Re: [MicrosoftDocs/OfficeDocs-SkypeForBusiness] Update limits-specifications-teams.md (#7213)
@JohanFreelancer9 approved this pull request.

You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2FOfficeDocs-SkypeForBusiness%2Fpull%2F7213%23pullrequestreview-660307852&data=04|01|Leif.Brenne%40microsoft.com|7f72ff7c599e4eb7983408d917606eae|72f988bf86f141af91ab2d7cd011db47|0|0|637566525217297874|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=88OM0%2F1tikXMVoDq6yve3xoxYs5KK13TdBqSP2NzRsU%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAHTYUDCONHKTYEGQTKGRC43TNX7ONANCNFSM443PV3BQ&data=04|01|Leif.Brenne%40microsoft.com|7f72ff7c599e4eb7983408d917606eae|72f988bf86f141af91ab2d7cd011db47|0|0|637566525217307823|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=jnuBSQe4Itbgqj517LhecP9Ay%2FO4ZI6qxKWk0nqq8Dk%3D&reserved=0.

Sorry if this information somewhere, but when does the doc go from live to actually Live on the docs site
From: Serdar Soysal [MSFT] @.>
Sent: Tuesday, June 1, 2021 9:07 AM
To: MicrosoftDocs/OfficeDocs-SkypeForBusiness @.>
Cc: Leif Brenne (HE/HIM) @.>; State change @.>
Subject: Re: [MicrosoftDocs/OfficeDocs-SkypeForBusiness] Update limits-specifications-teams.md (#7213)
Merged #7213https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2FOfficeDocs-SkypeForBusiness%2Fpull%2F7213&data=04|01|Leif.Brenne%40microsoft.com|59b125401f304de03e1308d92517429f|72f988bf86f141af91ab2d7cd011db47|0|0|637581604106280817|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=7NzvibkQmhLzMwRDKMUl61Vssu6gfFdoo5TxG%2FylWms%3D&reserved=0 into live.

You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2FOfficeDocs-SkypeForBusiness%2Fpull%2F7213%23event-4826398262&data=04|01|Leif.Brenne%40microsoft.com|59b125401f304de03e1308d92517429f|72f988bf86f141af91ab2d7cd011db47|0|0|637581604106290775|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=VFXFxoEru2XiSiUnLUaK1xl2swS6uAKUVxgNSpAHb4E%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAHTYUDFBBZ7WPDACVU6RK6LTQUARPANCNFSM443PV3BQ&data=04|01|Leif.Brenne%40microsoft.com|59b125401f304de03e1308d92517429f|72f988bf86f141af91ab2d7cd011db47|0|0|637581604106300731|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=WSdp%2Fg1hM8Mhssik5Nmo5ala23NecgOK0fHouAh16LA%3D&reserved=0.

@lbrenne - Should be live on the docs site today.


integration

The problem
Hello, I installed this integration via HACS (on HA OS and Raspberry pi3).
The installation is OK when I search for "AsusRouteur" via the additional modules of HA, I only find the native integration AsusWRT ?
Your device model
RT-AC52U B1
Firmware type
Stock
Firmware version
<IP_ADDRESS>.380
Integration version with the issue
0.8.0
Method of the integration installation
HACS
What version and type of Home Assistant installation do you use
Home Assistant 2022.8.7
Is there anything useful in the logs?
No response
Diagnostics information
No response
Additional information
No response

Hello,
After installing the integration, did you restart your Home Assistant? This is required for the HA to discover new integration

Yes...

After uninstall a unless integration and another reboot, it's OK I have AsusRouteur.
Thanks for your time.


Access to path denied

From time to time I get an error while running paket install
Paket failed with:
    Access to the path 'C:\SomeProject\packages\Microsoft.Data.Services.Client\lib\portable-net45%2Bwp8%2Bwin8%2Bwpa' is denied.

this happens when VS is locking files inside this folder. stuff like XML docs. Very annoying and also happens with nuget. There is a private issue in some microsoft issue tracker that apparantly nobody cares about (see https://github.com/Microsoft/visualfsharp/issues/57)
// @latkin

Bummer.


Add 8 basis test setup from ncatestify

Hi,
thx a lot. Works perfect.
Add plugin as npm dependency
https://github.com/ncatestify/cypress-base-plugin
Add ncatt-base-tests
https://github.com/ncatestify/cypressio-base/tree/main/template/cypress/e2e/ncatt-base-tests
Tests and project will have new features and improvements over the plugin commands.
Feel free to get in touch with me. You also can explain everything with me in a live screenshare session on YouTube.
Keep green
Roland

Closing.
Not sure what "issue" you are presenting. Or how the comments relate to this repo.🤷


Use .to(device) in place of .cuda() to support CPU and other devices

Currently only CUDA-enabled devices can run Dreamtalk. However, CUDA-specific logic is only used in a handful of places.
This PR:

Adds an argument --device to specify a device to run on (supports valid arguments to torch.device)
Changes instances of .cuda() to use .to(device)

I verified this successfully runs the README example snippet on CPU using --device=cpu

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

Thank you very much for your help! 😊


Can't pass "&" in dmtx or qr

Hi, I cannot be able to pass url with & in barcode dmtx or qr.
If I use %26 (& in urlencode), barcode seams do not draw correcly.
I need to pass a url like "https://www.domain.ext/wp-admin/post.php?action=edit&post=123"
Any idea?
Thank you, Luca

This library allows to build and display a barcode by calling it directly from your browser. If it seems like a fancy and easy way to produce a barcode. Actually it is a very bad idea. It allows other people than yourself to call build their own barcodes with your code. This is absurd.
You should instead build your barcode from your own script like below. It will also solve your problem.
<?php

require_once('barcode.php');

$generator = new barcode_generator();
$data = 'https://www.domain.ext/wp-admin/post.php?action=edit&post=123';
$format = 'png';
$type = 'qr';
$options = [
	'w' => 600,
	'h' => 600,
];
$generator->output_image($format , $type, $data, $options);


Feature Request: Vertical Grid Rotation

Please could you make the grids rotatable on the up and down axis
Mainly so we can put the grid face up in the floor of our bases
thank you

This would require storing the horizontal placement position (axis) as well, which RS doesn't do. I like it the way it is.


Use monitor info cpus to get a qemu thread id

Fixes #22
I.e. parsing:
(gdb) monitor info cpus
* CPU #0: thread_id=1234
  CPU #1: thread_id=1235

Note that this is a thread id. We could parse proc to find the process id but it's not necessary since threads share memory. So /proc/[tid]/mem is the same as /proc/[pid]/mem.

Note this fixes pwndbg/pwndbg#1587

Hi @martinradev
Please could you review/merge this so that we can get the pwndbg bug fixed?
Thanks.

I hadn't even considered namespaces, thanks for pointing that out!
I've come up with a more robust method now that is namespace agnostic.
We can add a chardev file backend via the monitor commands. If we create this with a known file path (e.g /tmp/foo123), we can then check for that path as a link target in /proc/[pid]/fd.
This even works for mount namespaces. It won't be the same file, but the path relative to the root of the mount namespace will be.
We only do this when pgrep returns more than one pid.

Looks reasonable. I should really add a proper integration test suite for this project.
Thanks a lot Will!


Wrong answer passes: Use Destructuring Assignment with the Rest Operator to Reassign Array Elements

Challenge can be passed using global variable instead of array passed as parameter
There is no test to dissuade using the global variable source instead of the argument list.
const source = [1,2,3,4,5,6,7,8,9,10];

function removeFirstTwo(list) {
  // change code below this line
  const [,, ...arr] = source; // change this - SHOULD BE list NOT source
  // change code above this line
  return arr;
}
const arr = removeFirstTwo(source);
console.log(arr); // should be [3,4,5,6,7,8,9,10]
console.log(source); // should be [1,2,3,4,5,6,7,8,9,10];

I think that a test confirming removeFirstTwo([10,20,30]) returned [30] would be enough to prevent that.

https://learn.freecodecamp.org/javascript-algorithms-and-data-structures/es6/use-destructuring-assignment-with-the-rest-operator-to-reassign-array-elements/
Tell us about your browser and operating system

Browser Name: Chrome
Browser Version: Version 69.0.3497.100 (Official Build) (64-bit)
Operating System: Ubuntu

If possible, add a screenshot here

I would like to work on this issue.

@fraindz Are you still working on this issue? If not, I'd like to tackle it.

@ezioda004 I am not working on it. You are free to take it.


Add missing instruction to click button

A tiny addition to the instructions. Step 2 was missing the last but to actually click the button "add secret"

Many thanks.


Implement a modal for converting lines to other types

Add associated methods to Interval and Scale
ref #163

The modal is now called Convert interval values and the select element is labeled as Interval type.


On mouseover the label disappears

Expected Behavior

Current Behavior
The shape label disappears.

You can reproduce the issue on this example: https://d3plus.org/examples/d3plus-hierarchy/selecting-container/

Steps to Reproduce (for bugs)

Load a treemap chart : https://d3plus.org/examples/d3plus-hierarchy/selecting-container/
Place the mouse over of any shape.
You will see.

Your Environment

Browser Name: chrome
Operating System and Version: OSx El capitán

Working on this now. Kudos to @palamago for actually filling out the full bug report 😍

Reopen, because I think this bug is related:

Happens when you have a chart with timeseries.
After you hover the label always stay on top.


Add lua minification script

Add a lua minification script that accomplishes the following tasks:

Merge all files into a single file by replacing each unique require call once by the file content (if it points to a file and not to a library) in main.lua

This must also remove trailing "return <Class>" lines

Minify classes

Replace class names by single letters ascending
Replace class attribute names by single letters ascending
Replace class methods by single letters ascending
Remove class constants whose value is nil
Apply the minified attributes and methods to the class instances (search all files for "<Class>(.*)")

Minify global variables
Keep global method names
Remove unnecessary symbols (such as ";", "for a = 0, 5, >1<" or unnecessary brackets and white space)

This would probably cause problems with the new EventCallback's and serverEventListeners, so closing it for now.


Check changestatus in Service Contract Header

Can you explain what you actually want us to do with this?

Replace Testfield with CheckStatus, so we can use the function SuspendStatusCheck. There is already a function CheckChangeStatus, but its only used in a couple of places.
And add HideValidationDialog so we can override the confirm.
Do you not agree?

@AlexanderYakunin

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.


Add SlashCommand to show a list of commands

The "Slash Command" has become more popular, with the ease of typing a simple "/" character to show a list of command suggestion.  I wonder if you can add that as an extension. Tiptap is also looking to add that as an extension as well so it would be nice if this editor can support Slash Command
Describe the solution you'd like
I want a drop down list of commands shown to me when I type "/" in the editor. I also can search for command by typing commands alias. For example, if I type "/big heading", the command to set heading to level 1 shown up. I can choose this command by either click on it or hit the "Enter" key.

U can use titptap extension command for this purpose. I use it so, but maybe it will be rewrote.


Add Gotify recipe

Pre-flight Checklist

[x]  I have read the Contributing Guidelines for this project.
[x]  I agree to follow the Code of Conduct that this project adheres to.

Description of Change
Adding support for Gotify

I don't use this service but it seems appropriate. I am adding also the code to support a custom service.css as it is better to give that possibility to the user in any case.
Thanks for this PR.

@all-contributors please add @kromonos for code