Samples not working

I must be missing something very simple but I pulled the project fresh from git and opened in vs 2019.  I set up startup to run Identityserver project followed by api fullowed by mvc project without changing any other settings.
It seems to get stuck here:
http://localhost:5000/connect/authorize?client_id=mvc&redirect_uri=http%3A%2F%2Flocalhost%3A5002%2Fsignin-oidc&response_type=code&scope=openid profile api1 offline_access&code_challenge=K0iGfhQJOd_wWeUz79Brf4AYW9_r8cpiF6dqcc7q3eQ&code_challenge_method=S256&response_mode=form_post&nonce=637096827373579828.ZTBhMzNjOWUtNTFhMC00NTE2LTk2OTItODA1NWFlOGI3YTA3YWJhZWM1YWUtYjMzNi00ZGYyLWJmN2EtODAyYjVhNzhjYzA3&state=CfDJ8F8F_CrHwnFOkz4ygPVptdBV5EB7t1l27pbUsMjYfGSsZ6C7PY4cTKvxs13JHe2dV2-IkjLlwUCvRpuBJteCcdhX-3YU0rRoYDNAlj7JKqkIOeb7RSSdhXX5p7RxqnwGgLJV4bzwHCdj-9-jpPV7v3eI5Eefrkc7n3Lk0iUvqJIYYTRCia6BClUsQ2wkGWtNj1FFBIOjExOWS9GQ5rvVoKy2pj73QRE3c79Fwsy-RSSXeU5_CBCvV1x6s6ISXpJYyivn6DrCx3p81C26Mch-AbuQhaZcjPBtTL31I2B4JYDCPHPZf_j7MnaBt6vBEF7mo2cM7lwzB8orivzWb_9zDG58fQpHv-D8PzChR5BdPD4RXhVn4pg6EwzONcCOXPp3Iw&x-client-SKU=ID_NETSTANDARD2_0&x-client-ver=<IP_ADDRESS>
Without any errors in the logging.
FYI the console client project works fine.

Make sure everything is on the right port.

Definitely on the right ports like I said client app is working.  Just MVC client is failing.  Using the sample project no changes.


Support to be able to set Files as the default file explorer

Is your feature request related to a problem? Please describe.
It's not a problem, just a convenience request and a question
Describe the solution you'd like
Support to be able to set Files as the default file explorer
Describe alternatives you've considered
The default files explorer, but if that was good enough, I wouldn't be here :D
Additional context

Also, is there a Discord for Files?

You can already do this but it's not recommended because of the side effects, see here for more information.


RSpec/DescribeClass applies to view specs when using config.infer_spec_type_from_file_location!

#90 only fixed it for when the spec is explictly tagged as being a view type.

@jcoyne unless you have an idea of how to statically determine this configuration I'm going to close this as WONTFIX since the cop cannot know your RSpec configuration

I agree, since inference is disabled by default after RSpec 3.


feat: improving performance 2

Description
#65

Additional context
Use recursion to replace flat.

What is the purpose of this pull request?
Just code golfing performance, inspired by @TIMMLOPK
Original
CVA x 931,787 ops/sec ±0.62% (89 runs sampled)
CVA with tailwind-merge x 503,360 ops/sec ±1.16% (88 runs sampled)
TV with slots x 97,211 ops/sec ±0.44% (91 runs sampled)
TV without slots x 113,086 ops/sec ±0.57% (90 runs sampled)
Fastest is CVA

@TIMMLOPK's branch
CVA x 957,529 ops/sec ±0.54% (90 runs sampled)
CVA with tailwind-merge x 475,375 ops/sec ±1.19% (86 runs sampled)
TV with slots x 151,382 ops/sec ±3.36% (86 runs sampled)
TV without slots x 180,776 ops/sec ±0.53% (88 runs sampled)
Fastest is CVA

This branch
CVA x 940,169 ops/sec ±0.57% (91 runs sampled)
CVA with tailwind-merge x 489,335 ops/sec ±0.79% (90 runs sampled)
TV with slots x 181,157 ops/sec ±0.72% (89 runs sampled)
TV without slots x 210,579 ops/sec ±0.60% (91 runs sampled)
Fastest is CVA

[ ] Bug fix
[ ] New Feature
[ ] Documentation update
[X] Other

Before submitting the PR, please make sure you do the following

[X] Read the Contributing Guidelines.
[X] Follow the Style Guide.
[X] Check that there isn't already a PR that solves the problem the same way to avoid creating a duplicate.
[X] Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. fixes #123).

Hey @Handfish could you please update this PR pulling the main changes?

@Handfish awesome job


shrug, simpleCommands, and time tests

Created snapshots + tests for the shrug.js, simpleCommands.js, and time.js files.
The snapshots for shrug are a bit wonky, but I think it's due to escaping characters. I checked the output being returned with a console.log, and it doesn't look the same as the output being returned in the snapshots, even if it does match up properly when run through the bot and manually checked. I'll likely need some review here.
Additionally, I had to make a few lines dedicated to checking that all of the permutations of the word "shrug" would trigger the regex. I wasn't sure if I needed to do the same in other areas, and am looking to discuss how to improve this piece. checking each one would result in thousands of tests for this one file, so I am not sure if that was necessary if we already checked if the permutations all worked and we only need to know if ONE permutation triggered the regex for sections such as "command should be its own word/group - no leading or trailing characters", or "command can be anywhere in the string".

@I3uckwheat I unintentionally added some other spacing changes I had made to the commit.

@twalton83 We'll get it merged then worry about adding eslint and prettier after the fact.

@I3uckwheat Thanks!


Update most dependencies

Also fix the callback examples

Can't merge until https://github.com/lodash/lodash/issues/1880 is released. According to https://github.com/lodash/lodash/issues/1886 it will be sometime tonight

Just a note that ff5714712bfe8064ebe242013734aae5c747c101 technically could be considered breaking since the promises returned from our public API have changed in accordance with bluebird 3.0


settings.xml file format changed

Brief description of your issue
I am using systeminformer-3.1.24258-canary-bin with the SystemInformer.exe.settings.xml file from systeminformer-3.1.24244-canary-bin which worked on ysteminformer-3.1.24244-canary-bin without issues but with systeminformer-3.1.24258-canary-bin, it appears to be ignoring everything in the file.  Did the format or options in the file change?
Steps to reproduce (optional)
Use a existing  SystemInformer.exe.settings.xml file
Expected behavior (optional)
It is supposed to use the settings in the file.
Actual behavior (optional)
All settings are ignored and System Informer seems to use the default settings.  In fact, it seems even if one uses the default  SystemInformer.exe.settings.xml file and then configure things, it seems to be reading and saving the settings elsewhere and not to  SystemInformer.exe.settings.xml
Environment (optional)
systeminformer-3.1.24258-canary-bin
Windows 11 x64 Professional Edition 23H2 Build 22635.4145 Beta Insiders

Did the format or options in the file change?

Nothing changed.

Use a existing SystemInformer.exe.settings.xml file

I just tested this and no issues?

Something changed because let me show you something I didn't mention, as I have closed System Informer and then opened it again and this is what my settings looks like on screen which means I obviously have different things enabled on screen as you can tell form the columns itself and what's on the bottom status bar for example.

Now, please take a look at the contents of the SystemInformer.exe.settings.xml file itself.  Do you even see anything other than the defaults in it?  Where is it storing the other settings since it seems to be reading the configuration from somewhere else.

It is reading and savings somewhere else other than C:\Users\vince\Downloads\systeminformer-3.1.24258-canary-bin\amd64\SystemInformer.exe.settings.xml

Where is it storing the other settings since it seems to be reading the configuration from somewhere else.

The same default location we've been using for years?
%APPDATA%\SystemInformer\settings.xml

Also, where is the GeoIP ID and license key actually stored specifically?

The settings file?

Where is it storing the other settings since it seems to be reading the configuration from somewhere else.

The same default location we've been using for years?
%APPDATA%\SystemInformer\settings.xml

Also, where is the GeoIP ID and license key actually stored specifically?

The settings file?

I thought if one ran it as a portable which is extracting the zip file to a directory of my choice, it would be using the systeminformer-version-bin\amd64\SystemInformer.exe.settings.xml file and then running SystemInformer.exe as I have always replaced only that file and I would have the exact settings without manually configuring System Informer and if I didn't, it would just be the default because there has to be a reason that the amd64 directory contains a SystemInformer.exe.settings.xml file by default.
and this confirms what I just wrote:
https://github.com/winsiderss/systeminformer

So basically the issue is atleast in my case, it is not reading or writing to the SystemInformer.exe.settings.xml in the same directory where the SystemInformer.exe is.  Remember I didn't write the screenshot above....

So I have confirmed, it is not reading the SystemInformer.exe.settings.xml file from the same directory as SystemInformer.exe like it did in previous builds which is actually in the instructions as in lines 51-62 in README.md as previously mentioned.

For me the xml is still used on all systems.
run ProcMon and trace SI startup and look at IO events when settings are read if you see any errors in Result column.

For me the xml is still used on all systems.
run ProcMon and trace SI startup and look at IO events when settings are read if you see any errors in Result column.

Thanks for the pointers, it only happens with this build as I reverted to the previous build with the same SystemInformer.exe.settings.xml file in program directory and it works fine and that's with  %APPDATA%\SystemInformer deleted.
To be honest, I still don't know how to use functions in System Informer and obviously not ProcMon or Process Explorer for that matter but how exactly do I do what you mentioned as I am running ProcMon from wscc.

As I do not know if https://github.com/winsiderss/systeminformer/issues/1 works or not as I never tried it. We know https://github.com/winsiderss/systeminformer/pull/3 works. https://github.com/winsiderss/systeminformer/issues/2 is what I am referring to and one of these commits dated September 8, 2024 and later to main.c excluding July 3, 2024 probably is responsible for the issue I mentioned,

None of these commits changed the settings.

So can you give step by step instructions as always good to learn something new...

Search for the settings.xml filename in procmon and right-click the filename (must be the filename) then select Include - this will exlude everything but the file and there should be 15 or so results on screen. Make sure the full string for the filename is visible and the status column.

In the Filter select SystemInformer.exe as process to Include and only show disk and thread/process events (registry, Network, Profiling can be disabled from toolbar)

Click to ok to apply filter, clear view and run the new systeminformer.exe build. In ProcMon you can configure symbols, add here the Si symbols and now look in result column for non success events.

@MagicAndre1981 - A few questions...
Basically what I did was select Filter and then SystemInformer.exe as you mentioned.  Then on the toolbar at the top, I have only the 2nd and 4th icons selected, am I supposed to let it do the Applying Event Filter process when I click on the 2nd and 4th icon.
Am I supposed to do the toolbar at the top after the Filter part or before.  And am I correct the symbols file for SI is in systeminformer-3.1.24258-pdb.zip and then do I need to click the Capture button on the top before running systeminformer.exe.  I also renamed the %APPDATA%\SystemInformer\settings.xml to settings.xml.1 first just to make sure it doesn't use it.

in the symbols entry and ;C:\sym\systeminformer-3.1.24258-pdb\ to add the localy expanded PDBs

Not sure what I am doing wrong but basically the Configure symbols screen is as above, I have the filter as this is what it shows when I open the filter again after clicking OK.

I basically clicked on the Capture button and then ran SystemInformer.exe but the procmon page is blank.

Never mind, I have it on Architecture instead of Process Name.

This is what it shows, is it supposed to be 33,641 events?

Apparently, now it works fine

and what's interesting is unlike before, the %APPDATA%\SystemInformer directory no longer creates a settings.xml on it's own like it did before when I ran SystemInformer.exe after settings.xml was already renamed and only the settings.xml1 file which is what I renamed the settings.xml to before running SystemInformer.exe

Ignore the symbols they're not relevant. You need to filer by the settings.xml filename because that's the only file you're having issues with. If the window is blank then you're not using the correct filename.
You're trying to filter by process architecture with a filername?

Change to Path and Ends With file settings.xml

@dmex, actually as I mentioned in two posts above.  As soon as I got procmon working, something changed as I said:
"%APPDATA%\SystemInformer directory no longer creates a settings.xml on it's own like it did before when I ran SystemInformer.exe after settings.xml was already renamed as settings.xml1 and only the settings.xml1 file exists in that directory as there is no cache or the GeoLite2-City.mmdb yet which is what I renamed the settings.xml to before running SystemInformer.exe
I did not reboot other than opened and closed SystemInformer.exe a few times that was started with Run as Adminstrator because I had the filter on Architecture instead of Process Name for procmon.
And after doing the GeoIP database update, the cache folder is created in the same directory as SystemInformer.exe and no longer in %APPDATA%\SystemInformer while the GeoLite2-City.mmdb is in %APPDATA%\SystemInformer so it's a mystery what actually happened."
so something happened as the cache folder is now created in the SystemInformer.exe folder instead of %APPDATA%\SystemInformer and it reads SystemInformer.exe.settings.xml file from the SystemInformer.exe directory correctly and it no longer creates the settings.xml file in %APPDATA%\SystemInformer.

So it's now resolved?


ImportError: cannot import name 'DataLoaderIter'

hi @thstkdgus35
thank you for your great work in this project
i try to test with in windows
python main.py --data_test Demo --scale 4 --pre_train ../experiment/model/EDSR_baseline_x4.pt --test_only --save_results
but i get this erro

python main.py --data_test Demo --scale 4 --pre_train ../experiment/model/EDSR_baseline_x4.pt --test_only --save_resu
lts
Traceback (most recent call last):
File "main.py", line 4, in
import data
File "D:\EDSR PyTorch\code\data_init_.py", line 3, in
from dataloader import MSDataLoader
File "D:\EDSR PyTorch\code\dataloader.py", line 13, in
from torch.utils.data.dataloader import DataLoaderIter
ImportError: cannot import name 'DataLoaderIter'
Traceback (most recent call last):
File "main.py", line 4, in
import data
File "D:\EDSR PyTorch\code\data_init_.py", line 3, in
from dataloader import MSDataLoader
File "D:\EDSR PyTorch\code\dataloader.py", line 13, in from torch.utils.data.dataloader import DataL

how can i solve this please.

This is because of the pytorch update. DataLoaderIter was underscore suffixed to prevent subclassing. I think a quick non-code fix would be to move to pytorch 0.3.1, until it is fixed in this code, IMHO.

As hak23 said, that error occurs due to PyTorch update.
Fortunately, I pushed a commit for PyTorch 0.4.0 yesterday.
Although I did not test my code on Windows, I think it will work fine.
Thank you!

thank you @hak23 and @thstkdgus35 for your help
i use the last EDSR Project update, Unfortunately seem like we need some time to things works well in new PyTorch windows, thank you sir.


Table doesnt work in production mode?

I have been using CRA to create the production build. Everything works fine in development mode, but then in production mode the table isnt even displayed. I dont know where to start to check for the bugs.
You can see the  element in the inspector. But it doesnt load, and the thead is missing, and no data is displaying. Making the whole table go blank

fixed it downgrading to hte previous version

@romanpastu Thank you so much, I just switched all my tables over to reactable and it was working just fine on local, but on deploy, nada. You just saved me a whole lot of stress!


Powershell Syntax Errors

Hi!
I would really love to have syntax errors displayed on code that is written in powershell. Next to syntax highlighting this would be a great addition I think!

We closed this issue because we don't plan to address it in the foreseeable future. If you disagree and feel that this issue is crucial: we are happy to listen and to reconsider.
If you wonder what we are up to, please see our roadmap and issue reporting guidelines.
Thanks for your understanding, and happy coding!


[Suggestion]: Allow invokers to take spell damage from the RPG Series mods

Mod Loader (Optional)
Fabric
Minecraft Version(s) (Optional)
1.20.1
Mod Version(s) (Optional)
8.0.5
Suggestion (Required)
Its just a simple request and the title says it all.
playing with the rpg series mods you can be a wizard/mage, priest, Paladin, rogue etc.
but the wizards are forced to go melee when fighting the invoker, their spells do nothing, I had to melee the invoker the entire fight only able to use the spells on the addons that the invoker summons.
it seem that the classes that mainly use staves cant do any damage except melee and that would be arcane, fire, frost wizards, and preists, I tried magic on all of them and they dont work.
so idk if I have to open a suggestion for them as well but it would be nice to allow invokers to take spell projectile damage.

also add a config that allows projectiles against this boss, archers also are force to go melee because the projectile barrier goes up too fast for the archer to do any damage except a few shots, so far only melee classes can fight this boss without issues. I'm tryna make a modpack, and I'm currently testing bosses out with the classes


Clean up the homepage

The homepage is full of markup that was generated using https://github.com/hniksic/emacs-htmlize. It's extremely hard to work with it, so we should clean up all the excessive markup and inline styles. I think most of the font-locking is non-essential anyways (and the cases where it's useful can be replaced with screenshots).

@AndreaCrotti
FWIW, I put together the site originally. I had reasons for why I did what I did at the time, but I agree with everything Baz said. It makes sense to overhaul to make it significantly easier to work with.
To answer your question, I doubt that Baz is attached to much, or anything, about the current style. I, obviously, have not made time for sayid in quite a while, so I too would personally appreciate just about anything that anyone contributes.

Ah thanks @bpiel, well I did a simple export with pandoc
https://github.com/AndreaCrotti/sayid/blob/export-to-markdown/docs/index.md
(which needs a bit of manual cleanup but not so much).
If that plus a style in the github pages styles is enough there isn't really much else to do to get a much nicer format to work with for the website.

@AndreaCrotti
Nice! I think the only possibly important things you lose here is the styling (especially colors) of the sayid output. This is what I believe Baz was referring to when he mentioned "screenshots". That said, if the conversion artifacts were cleaned up, this seems like a step in the right direction to me. Screenshots can always be added later. Not that my involvement/approval is at all necessary here, just sharing my thoughts.

Yes sure, in theory I can even take some screenshots of the actual page.
With a theme we'd still get syntax highlighting though, so for the code at least it might not be necessary.

I'm fine with just having a README. Generally going with AsciiDoc is slightly better as it's easier to generate a nice landing page from it. There's not a lot of documentation, so it's probably fine to keep it only in a README and use screenshots instead of the exported markup.

Ah yes that's probably better, there isn't enough to split the docs between two places 👍
but the README should still be markdown right or not?

I'm fine with both Markdown or AsciiDoc. I use them every day, so it's all the same to me.

I haven't done the screenshots, but something like this would be fine?
https://github.com/clojure-emacs/sayid/pull/51
For the code blocks we don't need screenshots right?

Yeah, we don't - just put those examples in fenced code blocks.

Cool done

I guess we can close this one now

Yep. Thanks for moving this one forward!


WFLY-18994 Remove illegal uses of Operation.hasOptionalCapability(String, null, null) from JGroups subsystem

https://issues.redhat.com/browse/WFLY-18994

Thanks @pferraro


re-export typing.Literal: F821 Undefined name bar

Hello,
If I import typing.Literal in lib.py then do from lib import Literal in main.py, ruff fails to treat the Literal type annotations properly.
# lib.py
from typing import Literal

__all__ = ["Literal"]

# main.py
from lib import Literal

foo: Literal["bar"] = "bar"

$ ruff check *.py
main.py:4:15: F821 Undefined name `bar`
Found 1 error.
$ ruff --version
ruff 0.0.280

Expected behavior:
Ruff should recognize that Literal["bar"] is an annotation for a string literal and not flag the name bar as undefined.

For now, you'll need to mark lib in typing-modules, like:
[tool.ruff]
typing-modules = ["lib"]

This tells Ruff to treat exports from lib as if they are re-exports from typing. Let me know if it doesn't work for whatever reason :)

Thanks for the tip, @charliermarsh.
This still does not work for my use-case as, in lib.py, I have from typing import Literal as L. Here's an updated repro:
# lib.py
from typing import Literal as L
from typing import Literal

__all__ = ["L", "Literal"]

# main.py
from lib import L, Literal

foo1: Literal["bar"] = "bar"
foo2: L["bar"] = "bar"

# pyproject.toml
[tool.ruff]
typing-modules = ["lib"]

$ ruff check *.py
main.py:5:10: F821 Undefined name `bar`
Found 1 error.

I'll update this issue's name accordingly and re-open the issue.

Makes sense, thanks. To set expectations, this won't be fixed in an imminent release since it requires us to resolve symbols across files, but it will be fixed eventually.

I also stumbled upon this while evaluating if I want to use nptyping in my work project
from nptyping import Float, NDArray, Shape

my_array: NDArray[Shape["x, y, z, 3"], Float]

results in
$ ruff check nptyping_test.py
nptyping_test.py:3:26: F821 Undefined name `x`
nptyping_test.py:3:29: F821 Undefined name `y`
nptyping_test.py:3:32: F821 Undefined name `z`
Found 3 errors.

even with
typing-modules = ["nptyping"]

in my pyproject.toml.
As it seems this is a real-world example of the issue described by @Jasha10 - nptyping uses Literal for their shape Keyword.

Thanks for adding this context @rlipperts. My usecase is also related to ntyping, as I'm using a personal fork of ntyping in my own project

We could special-case nptying.Shape as an alias of Literal. (But it wouldn't solve cases in which that's re-exported under some other alias.)

As outline in the referenced link it is possible to replace Shape with Literal. For me that is Hotfix enough so that there is no need to clutter your codebase, @charliermarsh,  if you intend to fix it properly in the long run.

We also encounter something very similar in pylint when upgrading to 0.0.280, but on the actual typing.Litteral and not an alias:
pylint/utils/utils.py:56:31: F821 Undefined name `ignored`
pylint/utils/utils.py:56:39: F821 Undefined name `modules`
pylint/utils/utils.py:64:36: F821 Undefined name `py`
pylint/utils/utils.py:64:39: F821 Undefined name `version`
Found 4 errors.

When there are multiple values in the Literal it seems to works:
Code:
https://github.com/pylint-dev/pylint/blob/a57dd01c465a1357fc96833bec278fbf47147a4c/pylint/utils/utils.py#L27C1-L64

@Pierre-Sassoulas - I believe that issue is separate and related to the use of future annotations. It’s fixed on main and will be out soon. (Similar to https://github.com/astral-sh/ruff/issues/6055.)


Remove $ in all code blocks in docs.minio.io site

Any codeblocks that have $ & > in it has to be removed. We are enabling copy paste.
The commands when pasted on terminal need to work without editing.

@deekoder can we close this 😄


Add description of footprint benchmark test

Add description of footprint benchmark test, including the test structure and "how to use".
The section is part of "7.7. RackHD Debugging Guide". I put it before the last section "Logged warnings FAQ".
@heckj @RackHD/corecommitters @cgx027

:+1:

:+1:


Define item alignment for grid

This is a (multiple allowed):

[ ] bug
[x] enhancement
[ ] feature-discussion (RFC)

It would be great if we could use helper classes choose alignment for grid items. With grid being flexbox it should be easily doable and would come handy for stuff like image thumbnail galleries where you would expect items to align to the left instead of the default center alignment.
Exposing other flexbox properties through helper classes may be appreciated by others as well, but as flexbox contains a lot of properties, it might bloat the API.

Issue is closed because of outdated/irrelevant/not actual
If this issue is still actual and reproducible for latest version of Framework7, please create new issue and fill the issue template correctly:

Clearly describe the issue including steps to reproduce when it is a bug.
Make sure you fill in the earliest version that you know has the issue.
Provide live link or JSFiddle/Codepen or website with issue


Fastify asset server

Overview

Updating the testing server to use fastify for the asset and cors server. This simplifies the server code and improves maintainability by relying on an server framework that has a well established community following. Performance is not affected as fastify is near on-par with raw node server performance.
Related Issue(s)

https://issues.newrelic.com/browse/NR-77365
Testing

Nothing should have changed around how we run jil tests or the local test server.

Superseded by https://github.com/newrelic/newrelic-browser-agent/pull/382


add envFrom for env from secrets

Adds envFrom support for values.  Personal use case is for secrets created from vault for things like OAUTH_CLIENT_SECRET, while still keeping most of the config in the helm chart.
Bumps version to 3.2.0

Hope I'm not missing some simple way to merge the config secret and creds secret.
Thanks!

I kinda feel like #33 is a bit more sustainable


Using an extra call.request() in streaming readRows.

This should improve throughput.

When the blocking queue was added, there was no flow control from gRPC. There is now (in the form of call.request). Remove the blocking queue and replace it with a concurrent queue and every time a partial response is read in the scan side, invoke call.request(N) (or something similar).
Wouldn't you want the initial buffer size to be the maximum number of messages you want buffered before the flow-control window is closed and then remove the automatic call.request(1) from the observer?

Closing this in favor of a different solution, as per Angus.


feat: Add GPG_SECRET_KEY and GPG_PASSPHRASE secrets for craft

This PR is a prerequisite for https://github.com/getsentry/craft/pull/346
To generate a valid secret key, run gpg --armor --export-secret-keys YOUR_ID.

No, @bruno-garcia will generate it and add once he is awake.


[4.x] Use PHP CS Fixer

Fixes #767

Thanks for the PR. See https://github.com/archtechx/template/blob/master/.php-cs-fixer.php, #767 mentioned that we should be using config from there.
And the workflow can also be the same, you can just add this section to our existing workflow file: https://github.com/archtechx/template/blob/master/.github/workflows/ci.yml#L42-L56

Could you change the workflow as well?


Changing "case" match causes join failure

Repro is with Zed commit 6ff0586.
In a recent Slack thread, a community user was getting help debugging their join use case. We were building off this working example of a "self-join".
$ cat all.json
{"attr": {"addrrss": "<IP_ADDRESS>"}, "sessionid": 1, "thing": "evilthing", "s": "start"}
{"attr": {"addrrss": "<IP_ADDRESS>"}, "sessionid": 1, "thing": "evilthing", "s": "end", "message": "A match!"}

$ cat self-join.zed
switch (
  case thing=="evilthing" and s=="start" => Fieldx:=ip(attr.addrrss), sessionstartid:=sessionid
  case thing=="evilthing" and s=="end" => Fieldx:=ip(attr.addrrss), sessionendid:=sessionid
) | left join on sessionstartid=sessionendid did_it_match:=message

$ zq -version
Version: v1.11.1-21-g6ff05865

$ zq -I self-join.zed all.json
{attr:{addrrss:"<IP_ADDRESS>"},sessionid:1,thing:"evilthing",s:"start",Fieldx:<IP_ADDRESS>,sessionstartid:1,did_it_match:"A match!"}

The user had some difficulty applying this to their specific data, then circled back to report where they struggled. In their own words:

Omg I found the problem - I was doing a regular search as part of the switch conditions - so something like case "searchterm" and attr=="val"

We can repro their experience by dropping the leading thing== portions in each case, which results in the join no longer producing output.
$ cat case-oops.zed
switch (
  case "evilthing" and s=="start" => Fieldx:=ip(attr.addrrss), sessionstartid:=sessionid
  case "evilthing" and s=="end" => Fieldx:=ip(attr.addrrss), sessionendid:=sessionid
) | left join on sessionstartid=sessionendid did_it_match:=message

$ zq -I case-oops.zed all.json
[no output]

What I find interesting is that if I run the case matches from each variant a la carte they produce the same values.
$ zq 'thing=="evilthing" and s=="start"' all.json
{attr:{addrrss:"<IP_ADDRESS>"},sessionid:1,thing:"evilthing",s:"start"}

$ zq 'thing=="evilthing" and s=="end"' all.json
{attr:{addrrss:"<IP_ADDRESS>"},sessionid:1,thing:"evilthing",s:"end",message:"A match!"}

$ zq '"evilthing" and s=="start"' all.json
{attr:{addrrss:"<IP_ADDRESS>"},sessionid:1,thing:"evilthing",s:"start"}

$ zq '"evilthing" and s=="end"' all.json
{attr:{addrrss:"<IP_ADDRESS>"},sessionid:1,thing:"evilthing",s:"end",message:"A match!"}

The canonical Zed tells us a subtle difference: One becomes a where and one becomes a search.
$ zq -C 'thing=="evilthing" and s=="start"'
where thing=="evilthing" and s=="start"

$ zq -C '"evilthing" and s=="start"'
search "evilthing" and s=="start"

My main concern is that the silence from the case-oops.zed example doesn't give the user much to go off. If we need to maintain the behaviors shown here, I'm hoping there's room for a syntax error or runtime error/warning to help the user confirm if they made a mistake.

Yeah feels like you case shouldn't allow and expression "evilthing" and s=="start"

Ok actually the runtime error is surfaced in the switch statement, it just gets suppressed in the join because it doesn't pass the equi join test with its counter part. Perhaps if join encounters an error value it should emit it?

We took a shot at this in #4931 but then realized it was not the right approach. Will revisit at a later time.


feat: standardize nftApi types

Description
This PR refactors the nftApi types for both the Covalent and Zapper resolvers to be standardized into our own NftItem and NftCollectionItem types, so we don't have to hack around using Zapper's V2NftUserItem for all APIs we fetch NFTs from.
The only user facing change here would be the NFT metadata - which was wrong in prod - not being displayed anymore for mainnet NFTs.
Though this accidentally brings support for:

token standard detection for some NFTs (see screenshots)
description/website when available, which was previously missing for Polygon NFTs e.g the "FOXy Fury" NFT in willywonka frame-injected wallet

While at it, cleans up / comments the abused Zapper v2/nft/balances/collections endpoint to return a single NftCollectionItem, lift up its consumption to the nftApi so we can eventually leverage multiple sources for the metadata as well, and disables the resolution from Zapper for the time being until they sort their collectionAddresses[] queryparam not being honored.
Pull Request Type

[ ] :bug: Bug fix (Non-breaking Change: Fixes an issue)
[ ] :hammer_and_wrench: Chore (Non-breaking Change: Doc updates, pkg upgrades, typos, etc..)
[x] :nail_care: New Feature (Breaking/Non-breaking Change)

Issue (if applicable)

tackles https://github.com/shapeshift/web/issues/4566 - disable NFT collection meta resolution for Zapper (meaning, altogether since it's our only source currently) for the time being. Note this is for the collection meta resolution, we still get NFT item meta

Risk

Low - ensure all NFT data is the same as in prod, minus NFT meta for mainnet NFTs, which was wrong.
Testing

See risks

Engineering

☝🏽

Operations

☝🏽

Screenshots (if applicable)
Prod

This diff

Only other comment would be if it would be worth defaulting floorPrice and price to '0' to show something related to price, unless we are omitting any price data on empty string, more of product/ux question maybe

Only other comment would be if it would be worth defaulting floorPrice and price to '0' to show something related to price, unless we are omitting any price data on empty string, more of product/ux question maybe

Yeah omitting was actually a product decision, we don't display it unless unavailable (thus the difference between 0 and actual null/undefined/now empty string). Ty for the review ! As we plumb more of this through, consolidate queries and add a normalized slices, we'll be able to do even better with missing values 🙏🏽


Allow use of default key for ssm parameters

The last change to fleece made it possible to encrypt the rendered
SSM parameters using a custom KMS key, but in the process broke the
ability to use the default key.
That's because boto3's ssm put_parameter call is sort of confusing;
the way to use the default is to avoid passing in the KeyId arg
entirely, but the code was setting it to None which led to a runtime
error.

Circle tests have passed, but the notifications aren't showing up. Circle says there's a problem currently with notifications getting sent to github.


chore(deps): upgrade dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-2.x" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 434

Questions ?
Please refer to the Backport tool documentation


fix plutus V2 conversion in error report

We were attempting to convert all script contexts to the V1 context when reporting a Plutus script failure.
I've also refactored the function that produces the error report.

Just a curiosity: how did this problem manifest itself? Was it throwing errors for transactions with failing V2 scripts? Was it causing problems beyond inadequate reporting of what went wrong?

yep, exactly! the translation from Data originating from a V2 script context to a V2 script context failed fromData, and returned the "does not decode to a context" message.


Feature/di friendly server

Server side refactoring to be more DI friendly:

Adding some (rudimentary) interfaces as needed
Adding DI annotations as needed
Code refactoring and clean up

Initial eureka-server-karyon3 sub project for karyon3 integration
[from @aroger-r7] add route53 binding alternative to eip

NetflixOSS » eureka » eureka-pull-requests #465 SUCCESS
This pull request looks good

NetflixOSS » eureka » eureka-pull-requests #472 FAILURE
Looks like there's a problem with this pull request

NetflixOSS » eureka » eureka-pull-requests #475 SUCCESS
This pull request looks good


GStreamer error: Forbidden when playing youtube video

I have noticed when trying to play youtube videos that I had previously played the previous day I would start getting a "GStreamer error: Forbidden". I suspect this is because the youtube plugin is caching the streaming URL when the video is first looked up, but that URL becomes invalid after some time.

Even with your fix, what happens if you start playing, pause the song, wait a long time, then resume playback? And also when the song is really long (hours)? Mopidy-Gmusic suffers from both these issues due to the URI expiring.

It's possible we might be able to get around the issues you pointed out using something like GStreamer's download buffering? I'm pretty new to this but the description here makes it sound like it could work https://gstreamer.freedesktop.org/documentation/application-development/advanced/buffering.html?gi-language=c#download-buffering. But whether that works or not the cache in the youtube plugin would still be an issue when playing a previously played track. I still think it makes sense to include these changes and then open a separate issue for the other issue you brought up.

Thanks @jaminh and @kingosticks - I have wondered about how useful the cache in the YouTube extension is at all, in typical use. Maybe it is just an unnecessary complication?
Regarding the other thing, is there any way in the mopidy environment to give a URI a lifetime? And then, whenever an action is taken using the URI (play, resume playing, keep playing), to check whether the lifetime has been exceeded (and if so, grab a new URI)?

You could always re-lookup the uri just before you start playing a track in translate_uri. Problem is that once playing, gstreamer doesn't let you modify the uri without effectively changing tracks. So if the uri expires once you start playing you are still stuffed. The best place to handle that would be within the gstreamer element itself.
Or, as I experimented with for Mopidy-gmusic once upon a time , a proxy layer that hides the real uri from gstreamer. But that didn't work in environments where people were already using a network proxy.
Having gstreamer buffer the entire song is one solution which there was (is?) a Mopidy PR for. But the memory requirements for very long tracks (hours) get a bit out of hand and there didn't seem to be a way to control that.

Argh.  I've mucked something up, by running mopidy as sudo, instead of as the service.
I now get
2020-04-18 14:57:22,467 ERROR [13693:MainThread] mopidy.audio.gst: GStreamer error: Forbidden

for some files when running as a service.  Everything works fine when running as sudo.  LRU v TTL doesn't make any difference.
I assume that either mopidy or tornado or youtube-dl or gstreamer or something else has made a file, or modified the permissions of an existing file, when running as sudo such that the file can't be accessed (or overwritten) my mopidy when it is running as a service (as user mopidy).  Very frustrating!

youtube-dl caches some things (in /var/lib/mopidy/.cache) that seem to muck playback up - passing "cachedir": False to YoutubeDL seems to have fixed it.

I'm also having the exact same issue, running mopidy as a service. The URL of the youtube video is valid. Mopidy can play the video after restarting the service. I have to restart mopidy every few hours to keep it working at the moment.

Could you try this branch: https://github.com/natumbri/mopidy-youtube/tree/feature/download-buffering
Probably will not help, but worth a try...?

Now, this is a good example of what download buffering can help with!

Could you try this branch: https://github.com/natumbri/mopidy-youtube/tree/feature/download-buffering
Probably will not help, but worth a try...?
NB. download-buffering relies on a feature of mopidy that has been merged into the github develop branch, but has not yet been released. It looks like it will be included in mopidy 3.1, when that is released. Until then, if you want to test this fix, you'll need to be running a version of mopidy fromdevelop on github.

I installed the new package from that branch but It didn't help. Still got the same error: GStreamer error: Forbidden.

Can you provide a debug log?

I'll reproduce the issue with with debug log on. Will report back when I have the detailed log related to the error.
What I have at the moment is not very informative:
Oct 25 16:20:51 mopidy[3142758]: INFO     2020-10-25 16:20:51,960 [3142758:YouTubeBackend-6] mopidy_youtube
Oct 25 16:20:51 mopidy[3142758]:   youtube PlaybackProvider.translate_uri "youtube:video/video-name"
Oct 25 16:20:52 mopidy[3142758]: ERROR    2020-10-25 16:20:52,021 [3142758:MainThread] mopidy.audio.gst
Oct 25 16:20:52 mopidy[3142758]:   GStreamer error: Forbidden

Here's what I got today. Looks like the URL it tried to play expired. But how come the same track can be played after restarting mopidy? Does it have something to do with how mopidy-youtube calls youtube-dl?
Oct 27 13:23:25 mopidy[4017904]:   Iris websocket message received: {"jsonrpc":"2.0","id":160381940562287,"method":"broadcast","params":{"method":"notification","params":{"notification":{"content":"asdf123101 changed track"}}}}
Oct 27 13:23:25 mopidy[4017904]: DEBUG    2020-10-27 13:23:25,626 [4017904:HttpServer] mopidy_iris.core
Oct 27 13:23:25 mopidy[4017904]:   {'method': 'notification', 'params': {'notification': {'content': 'asdf123101 changed track'}}, 'connection_id': '36D6NP33MWPX'}
Oct 27 13:23:25 mopidy[4017904]: DEBUG    2020-10-27 13:23:25,626 [4017904:HttpServer] mopidy_iris.core
Oct 27 13:23:25 mopidy[4017904]:   {'id':<PHONE_NUMBER>62287, 'jsonrpc': '2.0', 'method': 'broadcast', 'result': {'message': 'Broadcast to 1 connections'}, 'recipient': '36D6NP33MWPX'}
Oct 27 13:23:25 mopidy[4017904]: ERROR    2020-10-27 13:23:25,840 [4017904:MainThread] mopidy.audio.gst
Oct 27 13:23:25 mopidy[4017904]:   GStreamer error: Forbidden
Oct 27 13:23:25 mopidy[4017904]: DEBUG    2020-10-27 13:23:25,841 [4017904:MainThread] mopidy.audio.gst
Oct 27 13:23:25 mopidy[4017904]:   Got ERROR bus message: error=GLib.Error('Forbidden', 'gst-resource-error-quark', 15) debug='../gst-plugins-good/ext/soup/gstsouphttpsrc.c(1436): gst_soup_http_src_parse_status (): /GstPlayBin:playbin0/GstURIDecodeBin:uridecodebin1/GstSoupHTTPSrc:source:\nForbidden (403), URL: https://manifest.googlevideo.com/api/manifest/dash/expire/1603779897/ei/2GiXX8byOKqJ2_gP0MKqgAM/ip/<IP_ADDRESS>/id/948da0b3c8941b28/source/youtube/requiressl/yes/playback_host/r8---sn-cxaaj5o5q5-tt1e6.googlevideo.com/mh/Mk/mm/31%2C26/mn/sn-cxaaj5o5q5-tt1e6%2Csn-vgqsknel/ms/au%2Conr/mv/m/mvi/8/pl/22/tx/23951660/txs/23951655%2C23951656%2C23951657%2C23951658%2C23951659%2C23951660%2C23951661/hfr/all/as/fmp4_audio_clear%2Cwebm_audio_clear%2Cwebm2_audio_clear%2Cfmp4_sd_hd_clear%2Cwebm2_sd_hd_clear/gcr/ca/initcwndbps/1671250/vprv/1/mt/1603758161/fvip/5/keepalive/yes/itag/0/sparams/expire%2Cei%2Cip%2Cid%2Csource%2Crequiressl%2Ctx%2Ctxs%2Chfr%2Cas%2Cgcr%2Cvprv%2Citag/sig/AOq0QJ8wRQIgCNnPF0kmoeDeRvmFNXXjTbQ8DrtiZ_odUVI0LBUPyKoCIQCFhpYP5Qdn5Ubi99gLrQFWwXjZUy4OlKemdRhaayN8zA%3D%3D/lsparams/playback_host%2Cmh%2Cmm%2Cmn%2Cms%2Cmv%2Cmvi%2Cpl%2Cinitcwndbps/lsig/AG3C_xAwRgIhAMnB6p3Ox-cG6anxBCGm__gfuaR8xmXY6l4Pl8ZYvAJ7AiEAzU1b6raDvKJ6N0H_l5w98X8ICsDxDSlWqENp_i2bQJw%3D, Redirect to: (NULL)'
Oct 27 13:23:25 mopidy[4017904]: DEBUG    2020-10-27 13:23:25,845 [4017904:MainThread] mopidy.audio.gst
Oct 27 13:23:25 mopidy[4017904]:   Changing state to GST_STATE_NULL: result=GST_STATE_CHANGE_SUCCESS

What if we don't load the video url until it is played, instead of loading it when it is searched (see https://github.com/natumbri/mopidy-youtube/compare/fix/issue-134, and try that branch)?

Looks like the error is gone but the track is still unplayable:
DEBUG    2020-10-29 14:57:38,934 [882437:Audio-2] mopidy.audio.actor
Track is not playable: youtube:video/- I will... THE FIRST TAKE.1R5jREc_joY
Position query failed

The link to the track is valid: https://www.youtube.com/watch?v=1R5jREc_joY.
I guess we are making some progress here...

For me it just happened again with the newest development version (freshly installed yesterday with python3 -m pip install https://github.com/natumbri/mopidy-youtube/archive/develop.zip):
Nov 17 12:04:44 raspi4 mopidy[21062]: INFO     2020-11-17 12:04:44,835 [21062:YouTubeBackend-3] mopidy_youtube
Nov 17 12:04:44 raspi4 mopidy[21062]:   youtube PlaybackProvider.translate_uri "youtube:video/Stephan Bodzin live Piz Gloria in Murren Switzerland for Cercle.xF_QkfZI1mM"
Nov 17 12:04:45 raspi4 mopidy[21062]: ERROR    2020-11-17 12:04:45,090 [21062:MainThread] mopidy.audio.gst
Nov 17 12:04:45 raspi4 mopidy[21062]:   GStreamer error: Forbidden

I loaded the tracks yesterday evening, paused the playback and resumed this morning. It worked for a few minutes of that video and then suddenly stopped.

I installed @fix/issue-134 and still have the issue

I installed @fix/issue-134 and still have the issue

I installed @fix/issue-134 and don't have the issue anymore. thanks

fix/issue-134 was merged. Assume this can be closed.

Hi, I've got the same issue in the latest release (3.2). I guess this has not been merged into mainline and released right?
Thanks for maintaining this plugin. It is simply the best!

Hi, I've got the same issue in the latest release (3.2). I guess this has not been merged into mainline and released right?
Thanks for maintaining this plugin. It is simply the best!

+1

v3.3 now has ttlcache - any improvement?

Is 3.3 on Pypi yet, or build/pull from github?

Nope, still happening on 3.3 ... Is it worth adding ttlcache as an environment variable perhaps?

@ned-kelly are you able to manually edit cache_ttl =  in youtube.py and set it to something small, say 600?

I'll change and see as I do want to try and fix this long term ... I ended up just making a program that checks if the playback state is "playing" but the time is not elapsed and just removing the playlist then re-adding it...

@natumbri confirmed works. Nice. Thanks !


Docker Integration with Bind Mounts for Development and Production

I have addressed the issue #139 by integrating Docker with Docker Compose files for both development and production environments. This integration incorporates bind mounts to enhance flexibility and efficiency in managing codebases and configurations within Docker containers.
Changes Made:
Created Docker Compose files specifically designed for development and production environments.
Implemented bind mounts within the Docker Compose configuration to enable live code reloading and seamless updates.
Ensured clear segregation between development and production configurations within the Docker Compose files.
This pull request aims to improve the development workflow and deployment process by leveraging Docker's capabilities and enhancing the maintainability of the application across different environments.
Please review and provide feedback @hkirat

Closing in favour of https://github.com/code100x/daily-code/pull/111#pullrequestreview-1985266790


Warning: Discriminator source is absent or null, use base class ResourceDetails." when iterating SecurityAssessment object

Package Name: security
Package Version: 2020-01-01
Operating System:Windows10
Python Version: 3.8.10

Describe the bug
getting this warning for every iteration over SecurityAssessmentList object.
Discriminator source is absent or null, use base class ResourceDetails.
After investigation, it was found that the cause of this warning is in types.json:

However, the content responded to in the SDK is:

This will cause the SDK to generate this Warning log after deserialization.
To Reproduce
run the following code:
from azure.mgmt.security import SecurityCenter
from azure.identity import DefaultAzureCredential
security_client = SecurityCenter(DefaultAzureCredential(), os.environ['AZURE_SUBSCRIPTION_ID'], 'westcentralus')
assessments = security_client.assessments.list(scope='/subscriptions/' + os.environ['AZURE_SUBSCRIPTION_ID'])
for assessment in assessments:
    print(assessment)

Expected behavior
no warnings

https://github.com/Azure/azure-sdk-for-python/issues/24815

https://github.com/Azure/azure-sdk-for-python/issues/24815

Still an issue:
{
  "azure-cli": "2.55.0",
  "azure-cli-core": "2.55.0",
  "azure-cli-telemetry": "1.1.0",
  "extensions": {
    "account": "0.2.5",
    "aks-preview": "0.5.173",
    "azure-devops": "0.26.0",
    "ip-group": "0.1.2"
  }
}


Fix calculation of skinned AABB for unused bones.

Fixes bug where bounding box of 1 unit was used in some models and had wrong LODs. (this could become very large if the mesh is scaled, such as FBX conversions)
Use LOD distance of 0 if camera is inside mesh bounds, rather than distance to surface.
Also fixes a mistake in calculating bone index.
This PR is part of #76184 by clayjohn
Fixes #67890
It might address part of #76436 but I have not tested MultiMesh

Hmmm. I may have made a mistake by combining two changes here.
The first fix for the AABB -1,-1,-1 works.
The is_inside() check is not working. I'm still hitting cases where the incorrect LOD is used when the bounds are too large or I'm inside them. And it's very dependent on camera angle.
so there's something wrong with the math here that I'm not understanding.

malcomnixon reported some weird LOD issues with addons/godot-xr-tools/hands/scenes/lowpoly/left_hand_low.tscn from GodotVR/godot-xr-tools, so I'm testing this right now.
For now I'll split it off so this just fixes the -1,-1,-1 bounding box size which we already tested. And if I figure out the other issue, I'll make a separate PR for it.

Thanks!

Cherry-picked for 4.0.4. While this is not a complete fix, code improvement seems good to me to include.


Maven/Kotlin build does not incl. HTML files in target/classes

Hey man. Interesting project. 💪
I have tried this with Maven and Kotlin (which seems to be the only case, which is not included in your examples). It seems something is not working. I cannot find the HTML files in the target/classes folder after mvn clean compile.
I have tried adding the mvn resources plugin to move the HTML files and that kind of works until you want to use HTMX and ViewActions.
I have created a sample project here which reproduces the issue.
It seems that maybe the annotation processor is not running.

Hey there,
yes I have not tried Kotlin with Maven yet.
You could try out to use the kapt maven plugin instead of the maven compiler plugin: https://kotlinlang.org/docs/kapt.html#use-in-maven
That could maybe fix the issue already, as in gradle-kotlin configuration you need to use the kapt plugin aswell.
Should add that to the readme 😄
https://github.com/tschuehly/spring-view-component/blob/ca6d4ccda16299c3b9e4418529188247e288296d/examples/kte-example/build.gradle.kts#L22

Hello. That works. This commit fixes it.
If I understand correctly, then render() will always return the "full" page or am I mistaken? My understanding is that the idea of HTMX is that you would return a smaller piece of HTML and that would be replaced in the page. Have you thought about sending only the diff?

Great that it works now!
Will add it to the docs.
The Idea of ViewComponent is to seperate the page into individual components where you can use composition to build your UI.
You would always rerender a whole component.
How small a component is up to you. The size of the html sent over the wire is negligible imo.
You can always use OutOfBounds Swap to swap multiple components on the page in a single request.
https://htmx.org/attributes/hx-swap-oob/
To send only the diff you would need some kind of information of the current representation on the client which goes against HATEOAS principle. I would advise against that and just let HTMX handle the diffing.

Hey, thanks for following up. But I think there might be some misunderstandings.

To send only the diff you would need some kind of information of the current representation on the client which goes against HATEOAS principle.

The HATEOAS principal talks about the knowledge the client has of the server, not the other way around. It says: "I should send enough information so the client knows what can be done with the state (ie. which links (actions) are available - by including the links in the HTML for ex.).
But I think the word "diff" is misleading. What I meant that the server is just sending the "relevant" bit (the one which changed). In the counter example, this would be the
X
 (not also sending the button, etc.). Your view-actions need to be very specific.

I would advise against that and just let HTMX handle the diffing.

HTMX does not do any diffing, but just swapping HTML in a target. What view-components seems to do, is to give the HTML body an id and use that as target.
if (htmlLine.contains("<body")) {
    return@map htmlLine.replace("<body", "<body id=\"$viewComponentName\"")
}

By default HTMX targets the element, which triggered the AJAX call, right?

The size of the html sent over the wire is negligible imo.

Maybe. But to stay with the counter example, why not just submit a form then re-load the whole page then? Except "dependencies from the header" maybe.
Maybe this issue is not the place for this kind of discussion. 😄
Anyhow, I will keep playing with view-components.
Regards


Hotfix avoid entering a bogus equation loop through recursion

Yesterday's change in #430  to include a top-level notBool (so that simplifications can be applied to it) ends up recursively calling evaluation of the same term (i.e., a bogus evaluation loop of size 1) in some KEVM test proofs.
This fix adds back the recursion breaking flag to protect against this (the test behaves normally again with this fix).


GitHub composite action for adding KinD user

Issue link

RHOAIENG-2632
What changes have been made

Adding GitHub composite action for adding KinD user.
Verification steps

Verified using own operator repository fork.
Checks

[x] I've made sure the tests are passing.
Testing Strategy

[ ] Unit tests
[x] Manual tests
[ ] Testing is not required for this change

/lgtm

/approve


How to change spectrum extracted from World into mgc?

Dear all,
I have tried the following steps:

./analysis arctic_a0001.wav 1.f0 1.sp 1.ap
x2x +df 1.sp | sopr -R -m 32768.0 | mgcep -a 0.58 -m 59 -l 1024 -e 1.0E-8 -f 0.0 -q 3 >tmp.mgc

however, the final tmp.mgc is quite different from arctic_a0001.mgc which is provided in data/
So how can I extract the correct mgc from sp based on WORLD?
Many thanks

Have you tried with mgcep -a 0.42? It looks like the arctic_a0001.wav is 16000Hz so then you would use 0.42 instead of 0.58.

The scripts for extracting parameters are here:
https://github.com/CSTR-Edinburgh/merlin/tree/master/misc/scripts/vocoder/world

Thank ronanki,  I have found the problems by compare my script with
https://github.com/CSTR-Edinburgh/merlin/tree/master/misc/scripts/vocoder/world

There are two "analysis" , one is located in /WORLD/ and the other is located in /WORLD/build, you should use /WORLD/build/analysis , It seems the results are quite different from each other.
you should use mcep instead of mgcep.
Here is the fixed version:
cd tools/WORLD
2./build/analysis arctic_a0001.wav 1.f0 1.sp 1.ap
x2x +df a.sp | sopr -R -m 32768.0 | mcep -a 0.58 -m 59 -l 1024 -e 1.0E-8 -j 0 -f 0.0 -q 3 >tmp.mgc


Update log-effect-core, log-effect-fs2 to 0.11.1

Updates

io.laserdisc:log-effect-core
io.laserdisc:log-effect-fs2

from 0.10.2 to 0.11.1.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "io.laserdisc" } ]

labels: semver-minor

Superseded by #87


spring-cloud-stream with RabbitMq and STOMP

From @lex-em on July 1, 2016 13:36
Hi!
Does it posible to use STOMP over RabbitMq with spring-cloud-stream?
If it doesn't possible, are there any ways to declare different listeners for different types of objects within one Sink? Maybe via Jackson's inheritance? I'd like something like this:
@EnableBinding(Sink.class)
public class EventHandler {

    @StreamListener(Sink.INPUT)
    public void handle1(Event1 event){
        System.out.println(String.format("Id: %s, model: %s", event.getId(), event.getModel()));
    }

    @StreamListener(Sink.INPUT)
    public void handle2(Event2 event){
        System.out.println(String.format("AuthorId: %s, userId: %s", event.getAuthorId(), event.getUserId()));
    }
}

Copied from original issue: spring-cloud/spring-cloud-stream#585

Closing this as we don't plan to support STOMP.


Add GEMM transposed tests to unit test and remove invalid logging code

This PR implements the nine possible transposed/other variants for gemm, as well as removing code in logging macros which is invalid when compiled in debug mode.
The gemm tests use a function _gemm_test_impl, which implementes the generic functionality common to each test. The individual variants are expressed using the _GEMM_TEST macro, which instantiates a test based on the _gemm_test_impl function.

Okay, so it seems that having all the tests under one umbrella may be too slow for travis. I'll look into breaking them up a bit.

Note that Travis is testing using the host device, which is known to be particularly slow on jenkins.


Can't eval defprotocol in cljs source file

Reproduce

Create project using Lein new mies
Add some code in core.cljs
Eval code by hitting ctrl + enter
Error occurs in defprotocol

Environment

Windows 10 64bit
LT 0.8.1
Default user.keymap and user.behaviors
Default plugins (Didn't download any external plugins)

To clarify, with Lein new mies do you mean to use the mies template and not a new default project named mies?

Yeah, I use the mies template

Relevant info for refer: defprotocol fails oddly in repl only

I am able to reproduce this, so at least there is that.
Curiously, if you adjust the project to be Clojure rather than ClojureScript it evals correctly.
Unfortunately, my guess as to the actual problem would be the internal version of ClojureScript that Light Table uses for a repl does not have the fix reported in your link... or otherwise some other incompatibility. We have issue #1973 to address the ClojureScript update and intend to start the update this month as it has become too significant an issue to ignore.
I could very well be incorrect, but I can't think of another reason for the problem you are experiencing... though perhaps someone else will.


ES-6130 Fix bug causing drafts to not inherit the initial created_at

Please follow our PR Guidelines for submitting or reviewing a PR
Description
When a draft was created, it would have a new created_at rather than using the created_at from the original record. This fixes that bug.
Related issue(s)
https://technologyadvice.atlassian.net/browse/ES-6130
Related PR(s)
How to test / repro
Supposedly you can follow these steps (in products repo) to link this repo locally, but it didn't work for me. Alternatively, just copy the HasDraft's line into vendor/technologyadvice/laravel-drafts/src/Concerns/HasDrafts.php.

Run unit tests
Create a new product and verify it is successfully created
Edit new product and verify the created_at has not changed in the DB (since UI doesn't show timestamps)

Screenshots
Changes include

[x] Bugfix (non-breaking change that solves an issue)
[ ] New feature (non-breaking change that adds functionality)
[ ] Breaking change (change that is not backwards-compatible and/or changes current functionality)

Checklist

[X] I have tested this code
[ ] If this is a bugfix, I've added testing to check this in the future
[ ] I have updated the Readme
[ ] I have added appropriate logging at the correct levels

I don't see any actual code changes. I see the README and a test is updated, but no code.

Somehow it didn't get committed. Fixed


Request: Add to database instead of overwriting

I would like to add events by importing a .jsonl file without deleting the current database.

Added in 0.3.1 release


use jQuery functions instead of custom ones

As I see it, this library is heavily dependent on jQuery already
For example this line here: https://github.com/h5bp/Effeckt.css/blob/master/js/modules/modal.js#L41
So why are we using functions like these:
https://github.com/h5bp/Effeckt.css/blob/master/js/core.js#L43
I just tested this function in my browser after opening the demo page and the values returned by Effeckt.prototype.getViewportHeight and $(window).height(); are exactly the same.
Do we even need functions like Effeckt.prototype.getViewportHeight? Can I remove functions like these since these add nothing new to already battle hardened jQuery functions?

Sure. The actual syntax was something we agreed on when we first started the project. Since you seem to have motivation with this I would highly suggest you go through some old tracker issues to become more acquainted with the project and it's intentions :)

Since you seem to have motivation with this I would highly suggest you go through some old tracker issues to become more acquainted with the project and it's intentions :)

Sure! Though be warned that many decisions from back 2 years ago might be worth it to reconsider.

Sure will do grayghostvisuals.
So can I raise a PR with this change?

sure

@grayghostvisuals @ShashankaNataraj If we depend on jQuery we should really use it, I agree. Though, wouldn't it be appropriate to revision the decision to depend on jQuery one day? I like jQuery but effeckt's JavaScript looks rather easy so it's quite a big dependency.

We should eliminate jQuery. No need for it.

Agreed - jQuery is a massive overhead for something like this that just needs simple values and simple DOM interaction
I vote remove the jQuery references

ok, whats the final word on this @grayghostvisuals @arthurvr ?

@ShashankaNataraj  remove

@grayghostvisuals @AaronLayton Have updated the title of this issue to make sense in this context instead of raising a new one!

Just to quickly clarify: I like jQuery (am even a team member over there).

I think we are all in agreeance - lets remove and get a PR up

@grayghostvisuals @arthurvr @AaronLayton I see references to Modernizr too! Are you guys in agreement that we need to remove references to Modernizr too?

@ShashankaNataraj You can discuss that in another ticket.


[MM-996]: removed jwt instance from setup autocomplete

Summary
Remove "cloud" JWT instance type from install, uninstall paths & help text.
Exisiting

Updated

Issue
Fixes https://github.com/mattermost/mattermost-plugin-jira/issues/996

@Kshitij-Katiyar Are there any remaining references to cloud or JWT?

@Kshitij-Katiyar I think we need to remove the command from help command response as well

@Kshitij-Katiyar Are there any remaining references to cloud or JWT?

@mickmister Removed jwt/cloud from helpText and instance autocomplete help text


removing RGBa -> RGB conversion

Pillow starting from next release probably will support this conversion itself(https://github.com/python-pillow/Pillow/issues/6706) so it is not needed to have it's implementation here.

Since Pillow added such a transform yesterday, we remove such a transform from Pillow-Heif.


Build-error from fresh solid-scripts app-ts generated with typescript.

npm init solid app-ts my-app generates an app properly and npm run start starts a functioning app and dev-server, but the console shows an error:
Attempted import error: '!!../node_modules/babel-loader/lib/index.js??ref--5-oneOf-2-1!./index.tsx' does not contain a default export (imported as 'Comp').
Error shows without any code changes, or with some, after generating a new project, with Typescript

Thanks for reporting. Probably related to the new HMR module I added. I will address thus as soon as I can.

Should be fixed in scripts v0.0.28. I haven't done a full from scratch install yet but it worked fine updating my existing project with the structure. I'm going to do a full install and double check but you should be good.

Yeah confirmed with new install.

That was quick, thanks!


docs: When to use nixpacks over buildpacks

Ask
As an external user I want to understand when I should be using nixpacks or "what type of issues are addressed by this project"
context
I read
alternative to [Buildpacks](https://buildpacks.io/) and attempts to address a lot of the shortcomings and issues that occurred when deploying thousands of user-apps to the Railway platform

and I would like to get context over what I am gaining testing this.

Nixpacks is overall a much better choice, since it's officially maintained by Railway, rather than a 3rd party. Plus it can build many different languages, and handles edge cases much better than Buildpacks.


First attempt to verify connections and pull process

Not sure what went wrong yesterday.  I'm trying more green buttons on Github after learning (20 min ago) that I hadn't completed last week's lessons.  long story.

Not sure what went wrong yesterday.  I'm trying more green buttons on Github after learning (20 min ago) that I hadn't completed last week's lessons.  long story.

what do I do now?

what do I do now?

This is feeling more like a circle.

This is feeling more like a circle.

This time I noticed the "only those with write access can merge pull requests" announcement.  I'm wondering how that compares to what I saw last week in the branching/forking labs.  Did I not invite @chrisdaaz as an editor?  Did I miss a different step?  I'll keep looking tomorrow.

This time I noticed the "only those with write access can merge pull requests" announcement.  I'm wondering how that compares to what I saw last week in the branching/forking labs.  Did I not invite @chrisdaaz as an editor?  Did I miss a different step?  I'll keep looking tomorrow.

Hi @jas58 ,
This assignment doesn't require a pull request. Here are the instructions (from Moodle):

Follow the steps in this tutorial to test GitHub Actions and GitHub Pages on your own GitHub repository. When you're done, submit the following to complete the lab assignment:
Send me a link to demonstration OER textbook you made, or the GitHub repository you forked as part of the tutorial.
Write a paragraph or more with potential ideas about how you might use GitHub Actions or GitHub Pages in your final project. You are not committing to use either in any specific way, but think about the possibilities and the instructor will help you pursue those ideas if possible.

I can see that you've done the tutorial successfully (looks great!): https://jas58.github.io/minibookdown/
Now I just need the "paragraph or more" submitted via Moodle.

Hi @jas58 ,
This assignment doesn't require a pull request. Here are the instructions (from Moodle):

Follow the steps in this tutorial to test GitHub Actions and GitHub Pages on your own GitHub repository. When you're done, submit the following to complete the lab assignment:
Send me a link to demonstration OER textbook you made, or the GitHub repository you forked as part of the tutorial.
Write a paragraph or more with potential ideas about how you might use GitHub Actions or GitHub Pages in your final project. You are not committing to use either in any specific way, but think about the possibilities and the instructor will help you pursue those ideas if possible.

I can see that you've done the tutorial successfully (looks great!): https://jas58.github.io/minibookdown/
Now I just need the "paragraph or more" submitted via Moodle.


Extend GitHub Actions Workflow to include Markdown linting

https://github.com/igorshubovych/markdownlint-cli
https://github.com/markdownlint/markdownlint

the markdownlint VSCode extension (https://github.com/DavidAnson/markdownlint) is based off of this

https://github.com/igorshubovych/markdownlint-cli (CLI-tool for using the above via command-line)

Very light research indicates that both of those options are popular. Since I believe I read that virtual environments for GHAWs already contain NodeJS and Ruby support, using either shouldn't be too difficult.

https://matthewsetter.com/tools-that-make-technical-writing-easier-markdown-linter/
<EMAIL_ADDRESS>

https://twitter.com/golangweekly/status/1190319844102692866
Not sure whether that is a parser only or if there is any kind of validation/linting support included (haven't checked their repo yet).

https://twitter.com/golangweekly/status/1190319844102692866
Not sure whether that is a parser only or if there is any kind of validation/linting support included (haven't checked their repo yet).

Doesn't look relevant for this task.

https://github.com/igorshubovych/markdownlint-cli
Note that some rules can be ignored (e.g., the CHANGELOG currently fails several of these due to the format that the keepachangelog project has created/suggests).

The current idea is to only enforce that Pull Requests pass the checks specific to the Go source code with Markdown linting being entirely optional for now until the linting rules are worked out.
Reading over the docs, it appears that we can build out a config file to disable/alter specific rules which will be read by default if found.

https://github.com/igorshubovych/markdownlint-cli#configuration

For example, here are the current linting failures:
CHANGELOG.md:65 MD024/no-duplicate-heading/no-duplicate-header Multiple headings with the same content [Context: "### Fixed"]
CHANGELOG.md:72 MD024/no-duplicate-heading/no-duplicate-header Multiple headings with the same content [Context: "### Fixed"]
CHANGELOG.md:78 MD024/no-duplicate-heading/no-duplicate-header Multiple headings with the same content [Context: "### Changed"]
CHANGELOG.md:96 MD024/no-duplicate-heading/no-duplicate-header Multiple headings with the same content [Context: "### Fixed"]
CHANGELOG.md:110 MD024/no-duplicate-heading/no-duplicate-header Multiple headings with the same content [Context: "### Added"]
CHANGELOG.md:122 MD024/no-duplicate-heading/no-duplicate-header Multiple headings with the same content [Context: "### Changed"]
CHANGELOG.md:138 MD024/no-duplicate-heading/no-duplicate-header Multiple headings with the same content [Context: "### Removed"]
CHANGELOG.md:147 MD024/no-duplicate-heading/no-duplicate-header Multiple headings with the same content [Context: "### Fixed"]
CHANGELOG.md:158 MD024/no-duplicate-heading/no-duplicate-header Multiple headings with the same content [Context: "### Added"]
CHANGELOG.md:166 MD024/no-duplicate-heading/no-duplicate-header Multiple headings with the same content [Context: "### Changed"]
CHANGELOG.md:168 MD013/line-length Line length [Expected: 80; Actual: 94]
CHANGELOG.md:183 MD024/no-duplicate-heading/no-duplicate-header Multiple headings with the same content [Context: "### Removed"]
CHANGELOG.md:187 MD024/no-duplicate-heading/no-duplicate-header Multiple headings with the same content [Context: "### Fixed"]
CHANGELOG.md:194 MD024/no-duplicate-heading/no-duplicate-header Multiple headings with the same content [Context: "### Fixed"]
CHANGELOG.md:211 MD024/no-duplicate-heading/no-duplicate-header Multiple headings with the same content [Context: "### Changed"]
CHANGELOG.md:216 MD024/no-duplicate-heading/no-duplicate-header Multiple headings with the same content [Context: "### Fixed"]
CHANGELOG.md:223 MD024/no-duplicate-heading/no-duplicate-header Multiple headings with the same content [Context: "### Added"]
CHANGELOG.md:233 MD024/no-duplicate-heading/no-duplicate-header Multiple headings with the same content [Context: "### Changed"]
CHANGELOG.md:241 MD024/no-duplicate-heading/no-duplicate-header Multiple headings with the same content [Context: "### Fixed"]
CHANGELOG.md:247 MD024/no-duplicate-heading/no-duplicate-header Multiple headings with the same content [Context: "### Added"]
CHANGELOG.md:271 MD024/no-duplicate-heading/no-duplicate-header Multiple headings with the same content [Context: "### Changed"]
CHANGELOG.md:286 MD024/no-duplicate-heading/no-duplicate-header Multiple headings with the same content [Context: "### Removed"]
CHANGELOG.md:294 MD024/no-duplicate-heading/no-duplicate-header Multiple headings with the same content [Context: "### Added"]
README.md:164 MD013/line-length Line length [Expected: 80; Actual: 270]
README.md:165 MD013/line-length Line length [Expected: 80; Actual: 270]
README.md:166 MD013/line-length Line length [Expected: 80; Actual: 270]
README.md:167 MD013/line-length Line length [Expected: 80; Actual: 270]
README.md:168 MD013/line-length Line length [Expected: 80; Actual: 270]
README.md:169 MD013/line-length Line length [Expected: 80; Actual: 270]
README.md:170 MD013/line-length Line length [Expected: 80; Actual: 270]
README.md:171 MD013/line-length Line length [Expected: 80; Actual: 270]
README.md:172 MD013/line-length Line length [Expected: 80; Actual: 270]
README.md:173 MD013/line-length Line length [Expected: 80; Actual: 270]
README.md:174 MD013/line-length Line length [Expected: 80; Actual: 270]
README.md:175 MD013/line-length Line length [Expected: 80; Actual: 270]
README.md:176 MD013/line-length Line length [Expected: 80; Actual: 270]
README.md:177 MD013/line-length Line length [Expected: 80; Actual: 270]
README.md:178 MD013/line-length Line length [Expected: 80; Actual: 270]
README.md:179 MD013/line-length Line length [Expected: 80; Actual: 270]
README.md:187 MD013/line-length Line length [Expected: 80; Actual: 165]
README.md:188 MD013/line-length Line length [Expected: 80; Actual: 165]
README.md:189 MD013/line-length Line length [Expected: 80; Actual: 165]
README.md:190 MD013/line-length Line length [Expected: 80; Actual: 165]
README.md:191 MD013/line-length Line length [Expected: 80; Actual: 165]
README.md:192 MD013/line-length Line length [Expected: 80; Actual: 165]
README.md:193 MD013/line-length Line length [Expected: 80; Actual: 165]
README.md:194 MD013/line-length Line length [Expected: 80; Actual: 165]
README.md:195 MD013/line-length Line length [Expected: 80; Actual: 165]
README.md:196 MD013/line-length Line length [Expected: 80; Actual: 165]
README.md:197 MD013/line-length Line length [Expected: 80; Actual: 165]
README.md:198 MD013/line-length Line length [Expected: 80; Actual: 165]
README.md:199 MD013/line-length Line length [Expected: 80; Actual: 165]
README.md:200 MD013/line-length Line length [Expected: 80; Actual: 165]
README.md:201 MD013/line-length Line length [Expected: 80; Actual: 165]
README.md:202 MD013/line-length Line length [Expected: 80; Actual: 165]
README.md:240 MD013/line-length Line length [Expected: 80; Actual: 172]
README.md:244 MD013/line-length Line length [Expected: 80; Actual: 106]
README.md:246 MD013/line-length Line length [Expected: 80; Actual: 153]
README.md:247 MD013/line-length Line length [Expected: 80; Actual: 94]
README.md:248 MD013/line-length Line length [Expected: 80; Actual: 131]
README.md:255 MD013/line-length Line length [Expected: 80; Actual: 94]
README.md:256 MD013/line-length Line length [Expected: 80; Actual: 131]
README.md:287 MD013/line-length Line length [Expected: 80; Actual: 172]
README.md:291 MD013/line-length Line length [Expected: 80; Actual: 156]
README.md:293 MD013/line-length Line length [Expected: 80; Actual: 202]
README.md:294 MD013/line-length Line length [Expected: 80; Actual: 148]
README.md:295 MD013/line-length Line length [Expected: 80; Actual: 177]
README.md:301 MD013/line-length Line length [Expected: 80; Actual: 140]
README.md:302 MD013/line-length Line length [Expected: 80; Actual: 148]
README.md:303 MD013/line-length Line length [Expected: 80; Actual: 177]
README.md:309 MD013/line-length Line length [Expected: 80; Actual: 140]
README.md:317 MD013/line-length Line length [Expected: 80; Actual: 118]
README.md:322 MD013/line-length Line length [Expected: 80; Actual: 271]
README.md:325 MD013/line-length Line length [Expected: 80; Actual: 101]
README.md:327 MD013/line-length Line length [Expected: 80; Actual: 138]
README.md:328 MD013/line-length Line length [Expected: 80; Actual: 111]
README.md:330 MD013/line-length Line length [Expected: 80; Actual: 95]
README.md:332 MD013/line-length Line length [Expected: 80; Actual: 88]
README.md:334 MD013/line-length Line length [Expected: 80; Actual: 145]
README.md:337 MD013/line-length Line length [Expected: 80; Actual: 127]
README.md:339 MD013/line-length Line length [Expected: 80; Actual: 94]
README.md:340 MD013/line-length Line length [Expected: 80; Actual: 120]
README.md:342 MD013/line-length Line length [Expected: 80; Actual: 105]
README.md:352 MD013/line-length Line length [Expected: 80; Actual: 127]
README.md:353 MD013/line-length Line length [Expected: 80; Actual: 129]
README.md:354 MD013/line-length Line length [Expected: 80; Actual: 130]
README.md:362 MD013/line-length Line length [Expected: 80; Actual: 156]
README.md:363 MD013/line-length Line length [Expected: 80; Actual: 158]
README.md:364 MD013/line-length Line length [Expected: 80; Actual: 159]
README.md:375 MD013/line-length Line length [Expected: 80; Actual: 219]
README.md:376 MD013/line-length Line length [Expected: 80; Actual: 223]
README.md:377 MD013/line-length Line length [Expected: 80; Actual: 225]

The MD013 rule is a great candidate for disabling since it is very likely complaining about the command-line examples using various flags and the associated output.

The MD013 rule is a great candidate for disabling since it is very likely complaining about the command-line examples using various flags and the associated output.

After adding the following config file with a disable setting for MD013:
# We know that line lengths will be long in the main README file
"MD013": false

these are the remaining linting failures:
CHANGELOG.md:65 MD024/no-duplicate-heading/no-duplicate-header Multiple headings with the same content [Context: "### Fixed"]
CHANGELOG.md:72 MD024/no-duplicate-heading/no-duplicate-header Multiple headings with the same content [Context: "### Fixed"]
CHANGELOG.md:78 MD024/no-duplicate-heading/no-duplicate-header Multiple headings with the same content [Context: "### Changed"]
CHANGELOG.md:96 MD024/no-duplicate-heading/no-duplicate-header Multiple headings with the same content [Context: "### Fixed"]
CHANGELOG.md:110 MD024/no-duplicate-heading/no-duplicate-header Multiple headings with the same content [Context: "### Added"]
CHANGELOG.md:122 MD024/no-duplicate-heading/no-duplicate-header Multiple headings with the same content [Context: "### Changed"]
CHANGELOG.md:138 MD024/no-duplicate-heading/no-duplicate-header Multiple headings with the same content [Context: "### Removed"]
CHANGELOG.md:147 MD024/no-duplicate-heading/no-duplicate-header Multiple headings with the same content [Context: "### Fixed"]
CHANGELOG.md:158 MD024/no-duplicate-heading/no-duplicate-header Multiple headings with the same content [Context: "### Added"]
CHANGELOG.md:166 MD024/no-duplicate-heading/no-duplicate-header Multiple headings with the same content [Context: "### Changed"]
CHANGELOG.md:183 MD024/no-duplicate-heading/no-duplicate-header Multiple headings with the same content [Context: "### Removed"]
CHANGELOG.md:187 MD024/no-duplicate-heading/no-duplicate-header Multiple headings with the same content [Context: "### Fixed"]
CHANGELOG.md:194 MD024/no-duplicate-heading/no-duplicate-header Multiple headings with the same content [Context: "### Fixed"]
CHANGELOG.md:211 MD024/no-duplicate-heading/no-duplicate-header Multiple headings with the same content [Context: "### Changed"]
CHANGELOG.md:216 MD024/no-duplicate-heading/no-duplicate-header Multiple headings with the same content [Context: "### Fixed"]
CHANGELOG.md:223 MD024/no-duplicate-heading/no-duplicate-header Multiple headings with the same content [Context: "### Added"]
CHANGELOG.md:233 MD024/no-duplicate-heading/no-duplicate-header Multiple headings with the same content [Context: "### Changed"]
CHANGELOG.md:241 MD024/no-duplicate-heading/no-duplicate-header Multiple headings with the same content [Context: "### Fixed"]
CHANGELOG.md:247 MD024/no-duplicate-heading/no-duplicate-header Multiple headings with the same content [Context: "### Added"]
CHANGELOG.md:271 MD024/no-duplicate-heading/no-duplicate-header Multiple headings with the same content [Context: "### Changed"]
CHANGELOG.md:286 MD024/no-duplicate-heading/no-duplicate-header Multiple headings with the same content [Context: "### Removed"]
CHANGELOG.md:294 MD024/no-duplicate-heading/no-duplicate-header Multiple headings with the same content [Context: "### Added"]

Can we configure linting rules to match specific files? In other words, ignore rules X, Y, Z for README.md only, but be strict about the others (or vice versa)?

Might be worth creating an entirely new workflow if we don't want the entire Action to be marked as failed because the markdown linting portion was the problem.

https://github.com/olivierlacan/keep-a-changelog/issues/274
We can ignore the "duplicate header" warnings for sub-headings like so:
"MD024":
  "siblings_only": true

This will still alert on top-level headings (which we likely want), but will ignore repeat (normal) occurrences of Fixed, Added and so on.


Don't show "connected to" on connection error

If there is a connection error, the balance correctly shows a warning sign.
However, it still shows "connected to" and shows the "disconnect" button. This is confusing since it contradicts that there was a connection error.
https://github.com/getAlby/bitcoin-connect/assets/27162016/f20a5fce-b17b-4575-8851-23a801ccc71a

I think this can be fixed by adding validation in the init function of the connector as I mentioned here

Ideally with LNbits though we would add a connection check to see if the connection is valid (by doing an API call). This can be done in the LNbits connector init function, and if the call fails an error should be thrown which will stop Bitcoin Connect from connecting. https://github.com/getAlby/bitcoin-connect/blob/master/src/connectors/LnbitsConnector.ts#L22


SNOW-656580 support geometry data type in snowpark

Please answer these questions before submitting your pull requests. Thanks!

What GitHub issue is this PR addressing? Make sure that there is an accompanying issue to your PR.
Fixes #SNOW-656580

Fill out the following pre-review checklist:

[x] I am adding a new automated test(s) to verify correctness of my new code
[ ] I am adding new logging messages
[ ] I am adding a new telemetry message
[ ] I am adding new credentials
[ ] I am adding a new dependency

Please describe how your code solves the related issue.
Geometry support was added to python connector in release 3.0.4. This PR update snowpark to use leverage the connector changes and add support for Geometry type.

Looks like test_write_pandas_with_timestamps is flaky.


merge package from arlac77/npm-package-template

package.json

chore(package<EMAIL_ADDRESS>chore(package<EMAIL_ADDRESS>chore(scripts): cover@#overwrite c8 --temp-directory build/coverage ava && c8 report -r lcov --temp-directory build/coverage
chore(engines<EMAIL_ADDRESS>chore(package): add nyc from template
chore(package): set $.ava.require='esm' as in template
chore(package): set $.ava.files='tests/-test.js,tests/-test.mjs' as in template
chore(package): set $.ava.extensions='js,mjs' as in template

.travis.yml

chore(travis): remove node versions 10.14.2
chore(travis): add node versions 10.15.0

Codecov Report

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

@@          Coverage Diff          @@
##           master   #439   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           1      1
  Lines          41     41
=====================================
  Hits           41     41

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 d5de758...6ca180b. Read the comment docs.

Coverage remained the same at 94.545% when pulling 6ca180b28e57aa668a9eca3bdd418ad11d1eba89 on template-sync-1 into d5de758d8bf1eb77640254a71d4af94169ce42ab on master.


Parser adds new line symbol to the source being parsed

@auduchinok Could you give more context on why this is a problem
Context I know is

New line is only added if the file doesn't already end in a new line
This applies only to ParseAndCheckFileInProject and ParseFileInProject

But I don't know what specific problems it causes to add the new line character.  Equally, I don't know why our tests are so sensitive to the character being there (beyond line number range reports - it's more dramatic than that I believe?)
Thanks!
don

@dsyme Adding new line changes AST ranges which we rely on. We currently map these ranges to R# document offsets using approach very similar to the one used in RoslynHelpers.fs except getting min of lines count and a range line. It works fine when an empty line isn't added to the source.

@auduchinok OK thanks, that makes sense

@auduchinok I'm labelling this a "Urgency Later" since we workaround it at the moment, but please feel free to submit a fix for it.

Fixed in https://github.com/dotnet/fsharp/pull/6001.


[ain-py] github actions fails for docs.yml

With error:
    ...
    Building wheel for aiohttp (pyproject.toml): finished with status 'error'
    error: subprocess-exited-with-error

    × Building wheel for aiohttp (pyproject.toml) did not run successfully.
  Successfully built coincurve
  Failed to build aiohttp
    │ exit code: 1
    ╰─> [173 lines of output]
        *********************
        * Accelerated build *
        *********************
        running bdist_wheel
        ...

Issue moved to:
https://github.com/ainblockchain/ain-py/issues/35


Document

Version
4.1.6
Reproduction link
router.vuejs.org
Steps to reproduce
Version 3 of the Vue Router had a page dedicated to the  component and the slots API. It also included examples.
https://v3.router.vuejs.org/api/#router-link
What is expected?
It would be lovely if a similar page existed for the current library.
What is actually happening?
No such page exists.

It'd be lovely if the documentation were a bit more focused on the modern use-case where one uses it with Vite.
e.g.

installation
imports (ES Module imports)
composition API example
options API example
...
"Template Navigation" page which is dedicated to the router-link component, notes that the slots API exists, and maybe recommends using programmatic navigation instead of the slots API. After all, the slots API for the Vue router doesn't seem to always be advantageous, and breaks the Vue Testing Utils' Router Link Stub https://test-utils.vuejs.org/api/#routerlinkstub

There is already a few pages about router link. the search shows them.
Other issues exist about adapting the getting started guide.
I if you have other proposals, I think the best is to directly work on a or proposal 🙂

The first result that the search shows is https://router.vuejs.org/guide/advanced/extending-router-link.html#extending-routerlink , which is not a very useful result, since it's an advanced topic.
The second result is https://router.vuejs.org/guide/#router-link , which doesn't include an example.


Support norm by BitLinear

Summary
By this PR, BitLinear and BitMultiheadAttention start to support layer normalizations.

Codecov Report
Attention: Patch coverage is 86.53846% with 28 lines in your changes missing coverage. Please review.

Project coverage is 82.71%. Comparing base (310d431) to head (822b556).

Files
Patch %
Lines

audyn/modules/bitnet.py
85.43%
22 Missing :warning:

audyn/modules/normalization.py
84.61%
6 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff            @@
##             main     #151    +/-   ##
========================================
  Coverage   82.70%   82.71%
========================================
  Files         155      156     +1
  Lines       14000    14162   +162
========================================
+ Hits        11579    11714   +135
- Misses       2421     2448    +27

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Howto add label to QR image URL?

I tried several formats for name parameter for adding "label" for account (the one that appears on top of changing 6-digit code). According https://github.com/google/google-authenticator/wiki/Key-Uri-Format it should go like this: Company<EMAIL_ADDRESS>

See https://github.com/Daplie/node-authenticator/blob/master/authenticator.js#L60
Here's a live demo in the browser:

https://daplie.github.io/browser-authenticator/ and code on github

Scroll down and you can put in the values you want and then click "Regenerate" to update the URI string and the QRCode.
You can even use the Authy App to test it on your own phone.
Here's the corresponding node example:  https://github.com/Daplie/node-authenticator

Also with live demo you posted, there is no Label when using google Authenticator.
See examples here:
With label:

Without:

I know this is old but for anyone interested adding &issuer=Google to the otpauth url before creating the QRCode would set the label to Google.

issuer is now an option to otpauthURL()


feat(sequencer): implement minting module

Summary
implement minting module which allows the sudo PoA key to mint funds at will
Background
needed for devnet for a faucet
Changes

implement MintAction which mints funds to some account, can only be signed by the sudo address
feature gate this under the mint feature for both astria-proto and astria-sequencer
add warning log which prints on sequencer startup if mint feature is enabled

Testing
unit tests
Related Issues
closes #358

@SuperFluffy I think having 2 env vars along with the feature is a bit overkill, maybe one env var is good for now? should I add in this PR?

@noot

@SuperFluffy I think having 2 env vars along with the feature is a bit overkill, maybe one env var is good for now? should I add in this PR?

Yeah, let's add one of them to really make sure. 😊


Extension freezes inkscape

Hi, I'm trying to export an inkscape design as .eps using your extension, but no message appears, inkscape freezes and nothing is saved nor exported.. am I doing something wrong??.. thank you!!

Sorry, I thought I'll get mail when somebody opens an issue, so I did not check this page in a while.
Anyway, if the problem still exists, could you upload the SVG that you tried to save?


Use the ecosystem convention for naming directories

https://github.com/pypa/distutils/pull/133 was recently merged, and it's great to see better support for alternative Python implementations, but unfortunately it was implemented in a way that is distinct from what I've seen in the rest of the community.
In particular the issue is the usage of sys.cache_tag for naming implementation directories. This works but is a deviation from the community norm of using sysconfig.get_config_var("VERSION"): for example see build_scripts.py from this repo and the uwsgi build system. This is workable for an alternative implementation as long as we set sys.cache_tag = sysconfig.get_config_var("VERSION"), which I guess isn't the end of the world, but I believe the correct resolution is to use the appropriate configuration flag.
I'd like to advocate for #107 being merged, which simply means replacing sys.cache_tag with sysconfig.get_config_var("VERSION") in #133.

Unfortunately, I don't have a strong opinion on the right thing to do here.
@mgorny Do you have an opinion on kmod's proposal?

I don't think that this is going to work.
$ python3.9 -c 'import sysconfig; print(sysconfig.get_config_var("VERSION"))'
3.9
$ pypy3.9 -c 'import sysconfig; print(sysconfig.get_config_var("VERSION"))'
3.9

Unlike #133, it doesn't give unique directories for alternate interpreters.


Make variable for hz

Signed-off-by: Jean Wasilewski<EMAIL_ADDRESS>

@DavidWittman, any chance to see this PR merged? :)

Thanks @henyxia


Update readme.md

updated the spell in readme

Coverage remained the same at 98.218% when pulling 9e583a4e7e3c6327760bb8255283d07e901bf60c on nikhilgupta2001:master into 9cc92f3589175e5329904d1a5743424ee3e278d7 on ansble:master.


Remove unnecessary fields in stack devfile

This PR implements the discussion https://github.com/devfile/api/pull/282#discussion_r552151109, which removes the unnecessary fields that are copies from odo registry repo in stack devfile.
Signed-off-by: jingfu wang<EMAIL_ADDRESS>

/approve

/approve


zip behavior

Is the following behavior intended in zip?  I googled for a tiny bit and there didn't seem to be any official implementation.
Lazy([1,2,3]).zip([1,2]) // [[1, 1], [2, 2], [3, undefined]
Lazy([1,2]).zip([1,2,3]) // [[1, 1], [2, 2]

FWIW this differs from the implementation of zip in both Haskell and .NET (Linq), both of which truncate the result to the length of the shortest input.

If any maintainers want this changed, I'll create a PR for a fix.

The no-truncation behavior matches that of Underscore's zip, which is probably intended giving Lazy's heredity

@bchociej - I just tested it on jsfiddle and I assume you skimmed my example.  Underscore doesn't truncate in either scenarios whereas lazy.js truncates if the first array is shorter than the second array.  I didn't test more than two arrays.

@olsonpm Sorry, I guess I said that really oddly. I agree with you -- I think that your first example, which matches Underscore, is probably the intent. In which case, I agree that the second example should give an array of length 3! I meant my comment to be in contrast to @djcsdy 's examples where the output is of the minimum length, truncating longer inputs.

Ah gotcha.  Not a problem - and I think the no-truncation behavior would be most valuable since it allows you to easily compare the equality of two sequences without first knowing the length.

Hm, thanks for bringing this up. I think I don't have a strong opinion as to which behavior is correct, although to me this actually feels like a scenario where it's likely a mistake on the developer's part. So maybe I do have an opinion, which would be not to truncate, so as to expose as much information as possible (and also match Underscore's behavior).
Yeah... I think I will update this to follow Underscore. Will close this issue when I've pushed the change.

Haskell’s behaviour is motivated by type theory (there is no null or undefined in Haskell), and also by use cases that involve zipping an infinite list with a finite list.
For example, consider the following code to filter out every other item in a list:
Lazy(["Sam", "Sarah", "Susan", "Steve"])
    .zip(Lazy.range(1, Number.POSITIVE_INFINITY))
    .filter(function (pair) { return pair[1] % 2 == 0; })
    .map(function (pair) { return pair[0]; });
// -> Lazy(["Sarah", "Steve"])

This construction is quite clumsy with lazy.js, which is a shame, because it can be quite useful.
In Haskell:
map snd $
    filter (even . fst) $
    zip [1..] ["Sam", "Sarah", "Susan", "Steve"]
-- -> ["Sarah", "Steve"]

Neither consideration applies to Underscore, but the above use case does apply to lazy.js and can be quite useful, so it’s worth considering carefully.

That's interesting.  Thanks much for taking the time to share as I never would have thought of that.

Folks following this thread who are interested in Haskell style zip semantics may want to check out my change to zip at jacksonfish/lazy.js@f556830, based off of the work in PR #178.

This was fixed in version 0.4.3.

To clarify: by "fixed" I mean Lazy now behaves the same as Underscore and Lo-dash re: truncation. Generally speaking Lazy isn't super flexible with regards to passing sequences as arguments to methods (sometimes it works, sometimes it doesn't). I intend to revisit this and make the library more consistent on this front in the future; at that time I may take another look at the behavior here and ensure it makes sense when handling things like infinite sequences.