nixos/jellyfin: better defaults for hardware acceleration

Description of changes

I have not tested any of that. I just changed this based on recent additions to the wiki: https://nixos.wiki/wiki/index.php?oldid=7605&rc_id=7898
cc @makefu for testing
Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

@peperunas

The "render" group came from the jellyfin documentation, just added it ( https://jellyfin.org/docs/general/administration/hardware-acceleration.html )
I thought about adding a flag for enabling access to /dev/dri, however i am unsure if it would really provide some security benefits in contrast to the complexity of users trying to find out why hardware transcode does not work.

I haven't tested this yet, but in principle I'm all for it. One thing to note is from the systemd.resource-control(5) man page, under the DeviceAllow= section:

The device group is matched according to filename globbing rules, you may hence use the "*" and "?"  wildcards. (Note that such globbing wildcards are not available for device node path specifications!)

From what I understand, this would mean that DeviceAllow = [ "/dev/dri/*" ]; isn't understood by systemd? We might need to put DeviceAllow = [ "char-drm rw" ]; instead.

@minijackson  i will test your proposal

I checked the different parameters, however i have not managed to get jellyfin to transcode without setting privateDevices=false
Upstream also essentially gave up: https://github.com/jellyfin/jellyfin/commit/dd8b9e9d2367893f2865d07dbe3a11c1adf6c631#diff-3d806a09406ac27e205da315a5f9ad987eec663f657b3dda296b4712c8a73d73R27
according to https://github.com/NixOS/nixpkgs/pull/163491#issuecomment-1064385734 adding nvidia-specific whitelisting works, however i cannot get it running for vaapi with my intel card.
Error:
[AVHWDeviceContext @ 0x1d119c0] No VA display found for device /dev/dri/renderD128.
Device creation failed: -22.
Failed to set value '/dev/dri/renderD128' for option 'vaapi_device': Invalid argument
Error parsing global options: Invalid argument

adding "render" supplementaryGroup is mandatory for transcoding to work

we should use upstream settings unless we need to do something nixos specific
https://github.com/jellyfin/jellyfin/blob/master/debian/jellyfin.service

we should use upstream settings unless we need to do something nixos specific https://github.com/jellyfin/jellyfin/blob/master/debian/jellyfin.service

I synched this up now.

@ofborg test jellyfin


Finding Matrix follows Teoplitz Matrix Algorithm or Not ?

Hi,
I am GSSOC'21 participant
My code will be in JAVA with proper comments and test cases.
@rudrakshi99 @nakul-19 @plazzy99 @tarun26091999  please assign me this issue

Hi,
I am a GSSOC'21 participant.
I would like to solve this issue in C++. Please assign it to me.


How can get comments of specific post ?

General Question
Read the Notes and fill out the form.
Switch to Preview for reading.
Notes
Your issue will be closed if you violate any rule below.

You need to include a meaningful descriptions and e.g. a screenshot.
You do not include screenshots of your code.
Read the Docs.
Capturing endpoints and similar can be found here
(read it if you are going to ask about it).
You can delete this section after reading it.

Form
Put an [x] if you meet the condition, else leave [ ].

[x] I've searched the Issues
[x] I've read the basic concepts
[ ] I'm using the latest version

Question
How can get comments of specific post ?
here is my code
first i get ids of posts
Code
// get post ids
 async getAllPostsByHandle(targetHandle) {
        // get instagram id (pk)
        const pk = await this.ig.user.getIdByUsername(targetHandle);

        // created feed
        const feed = this.ig.feed.user(pk);
        // get all iitems from feed
        let items = [];
        let page = 1;
        do {
            try { // in case you get blocked by instagram
                console.log(`fetching page... ${page++}`);
                items = items.concat(await feed.items());
                await this.delay(Math.floor(Math.random() * (10000 - 4000 + 1)) + 4000)
            } catch (err) {
                console.log("error");
                console.log(err);
            }
        } while (feed.isMoreAvailable() && page < 4);

        return items;
    }

then i  try to get comments
//get comments
    async getCommentator(id) {
        let comments = await this.ig.feed.mediaComments(items[3].id)
        console.log(comments)

        return comments
    }

Error and Output
but when i try to get comments i have this output
{"attemptOptions":{"delay":60000,"factor":1.5,"maxAttempts":10,"minDelay":60000,"maxDelay":300000,"jitter":true},"rankToken":"1536d705-86f5-5b81-96a1-204ee397d691","id":"2588689759012251944"}

Take a look at how you used the User-Feed. It's basically the same with the comment-feed. ig.feed.mediaComments returns a feed.

thanks i solved this problem . i wrote
let comments = await this.ig.feed.mediaComments(items[3].id)
but i should make let comments = await this.ig.feed.mediaComments(items[3].id).items()

but i should make let comments = await this.ig.feed.mediaComments(items[3].id).items()

This really depends on how many comments there are. If there are too many, you'll need to call .items() multiple times like you did with the user-feed.


Suggestion, Count the number of pokemon which were caught/failed to catch while the app is running

Hello,
Would it be possible to add a counter inside the app which increments everytime the go plus catches/fails to catch a pokemon?
https://play.google.com/store/apps/details?id=com.sebas.gocompanion&hl=en_IE&gl=US
Go Companion used to have a similar feature but it hasn't been updated in a few years I would love to see this option added to this app, if it's outside of the scope of the project, that's fine too.

No plan for it for now. Right now I am trying to find people to test (I assume it works for you) and get it on Play store.

The counter doesn't seem to add anything. I'm seeing 0 0 0 0

@Smultie Make sure you enable "Monitor game notifications".

Yes, it's working now! Thanks!

Just wanted to say thank you for implementing the feature! I really do appreciate it ❤


Bio texts in mentor applications are not properly formatted

Describe the bug
The bio texts in mentor applications are not properly formatted.

To Reproduce
Steps to reproduce the behavior:

Apply as a mentor
Add following to the bio:

Lorem Ipsum
Dolalr sit amet:<PHONE_NUMBER>35

• Community Development and Capacity Building
• Social Mobilization
• Project Management
• Site-Based Emergency Responses
• Training Programs
• Project Evaluation
• Report Writing
• Excellent Computer, Multimedia and Social Media Skills
• Strong Communication and Analytical Skills
• Design Campaigns
• Artist – drama, paintings • Author

Login to the admin dashboard
View the application

Expected behaviour
The text should be properly formatted. Please follow this StackOverflow question:
https://stackoverflow.com/questions/11599859/forcing-a-long-line-of-text-without-spaces-to-line-break-according-to-parent-c

@jayasanka-sack I would like to work on this.

Assigned @kumuditha-udayanga


websocket disconnect

if websocket is disconnect with server, is the method "websocketDidDisconnect" called all the time or just call once?

That is a delegate method and is called when the WebSocket is disconnected. It is called whenever the client gets disconnected. It is only called on disconnect, so unless your connection to the WebSocket keeps dropping off, it would not generally called that often.

To co sign @acmacalister note, websocketDidDisconnect  wouldn't get called expect in the case of disconnections, but it can be called more than once for the same disconnect depending how the stream was disconnected (if it was wasn't a gracefully disconnection, etc). #81 has more info on this.

@acmacalister  @daltoniam  :+1:


[DOCS] userStatus/speakingTopic missing from events guide

Description
There are a few optional fields missing from the events docs here https://linkfree.eddiehub.io/docs/how-to-guides/events
"userStatus": "speaking",
"speakingTopic": "Open Source",
This issue is to add these items to the example as optional parameters with an explanation of what data should be stored there.

Screenshots
No response
Additional information
No response

Hey I would like to contribute to this issue. @amandamartin-dev

Hey I would like to contribute to this issue. @amandamartin-dev

This issue is "awaiting triage" please do not ask to be assigned unless you see the status "ready for dev". We appreciate your interest and enthusiasm, but please follow the guide to. help keep confusion down and decrease the work for maintainers.
See the contributing guide for more information: https://github.com/EddieHubCommunity/LinkFree/blob/main/CONTRIBUTING.md

Assign to me, If no one working on this project

Oh good spot Amanda on the missing docs 👍

I'd love to be assigned this!

Sure @wrmacdonald I have assigned it to you


How to use timesteps?

I have noticed the output of ctcdecode includes timesteps, which the description says it can be used as alignment.
But I just get shape (Batchsize，N_beams，N_timesteps). I don't know how to use it.
timesteps - Shape: BATCHSIZE x N_BEAMS
The timestep at which the nth output character has peak probability. Can be used as alignment between the audio and the transcript.
Thanks in advance.

I have the same question, thanks


Create EditRequestCommand class

This PR adds the EditRequestCommand class and it's relevant tests. The tests and build for the edit request command pass, and the checkstyle also pass.
The build fails due to the a bug in PatientBook(), which triggers a null pointer exception when called. Refer to issue here [https://github.com/CS2103-AY1819S2-W09-2/main/issues/78]


Return a 404 for badges if there is no published certificate

Coverage increased (+0.03%) when pulling bcfb7d3476b586e461f0c62399cf8f8867b9d258 on feature-404-draft-badges into 86ab71ae74622575e0e7151fd91a4545b17d3de5 on master.

Coverage increased (+0.03%) when pulling<PHONE_NUMBER>4a0ab12eec506cd2e3b63ef41c0dcd on feature-404-draft-badges into 86ab71ae74622575e0e7151fd91a4545b17d3de5 on master.


Pls docker version

Please create a docker version, I can’t set up the environment in any way, it swears on dependencies! and youtube video removed

Yes I know about the YouTube video, I was thinking about re-uploading it somewhere, I'm currently thinking about this.
I let the issue open, I'll respond again here when the video will be online again.


Fix MSYS hanging of PowerShell

Fixes #6688.
I moved gpg-agent.exe termination code from #6407 into vcpkg_acquire_msys so that none of the ports using MSYS would block build automation tools.

Hi @emptyVoid, thanks for your PR. Could you please also bump the version in CONTROL version?

@NancyLi1013, do you mean ports/icu/CONTROL file?

Yes, you're right. if there are some changes about the port, you should also update the version in ports/port/CONTROL file. Thanks.

Same errors as with #6723:
error: failed to update mingw32 (unable to lock database)
error: failed to update mingw64 (unable to lock database)
error: failed to update msys (unable to lock database)
error: failed to synchronize all databases

It looks like when the task is killed it does not free its lock on the database.  Because we are reusing VMs in the CI system the lock is causing failures in unrelated PRs.

We'll want to trigger a rebuild of all ports using vcpkg_acquire_msys before putting this change through. Thanks @emptyVoid!

Waiting on CI results from #6760 were I added white-space changes to trigger CI builds of ports using MSYS.

#6760 has finished and this PR is now passing Eager. Thanks again for this fix, @emptyVoid!


Issue 316 rename with inner types

side types are also working
only changeset 24da348e is needed for this fix, but it's late and I forgot to reset before fixing, and I'm too tired (and too lazy) to rebase it to master

Closing, will open a new pull request


Abstract BrowserRunner and decouple from wd.js

At the moment plugins are limited in how much they can customize the actual BrowserRunner. One particular use case is Browserstack's REST API, which does not use WebDriver.
It would be useful for WCT to decouple the runner from WebDriver and allow plugins more control over the configuration and execution of the BrowserRunner.
I've created a sample web-component-tester-custom-runner package and wct-browserstack plugin that demonstrate how this relationship could work. Both are published and can be installed from npm.
Abstract class
There could be many ways to achieve this, such as through hooks or classes.
My sample repos go the abstract class route. WCT ships with an abstract BrowserRunner class with shared functionality and its own WdBrowserRunner implementation.
If a definition does not provide a runnerCtor to use, WCT will default to WdBrowserRunner.

that sure looks like a nice solutions :)
would love an easy way to test with browserstack. :)
currently I'm forced to use some sort of workaround...

it works really NICE :)
I just used it to test ~40 Webcomponents... it's a huge help... way more stable then my previous workaround.


build error (make PythonAPI)

Windows 11
Visual Studio 2019
When I build a PythonAPI(make PythonAPI) in x64 Native Tools Command Prompt for VS 2019,
error is occured.
Please somebody help me.

C:\test\carla>make PythonAPI
-[Setup]: [Batch params]: --boost-toolset msvc-14.2 --all
-[Setup]: Asynchronous jobs:  16
-[Setup]: Boost toolset:      msvc-14.2
-[Setup]: Install directory:  "C:\test\carla\Build"
-[Setup]: Installing zlib...
-[install_zlib]: [Batch params]: --build-dir "C:\test\carla\Build"
-[install_zlib]: A zlib installation already exists.
-[install_zlib]: Delete "C:\test\carla\Build\zlib-install" if you want to force a rebuild.
-[install_zlib]: Exiting...
-[Setup]: Installing libpng...
-[install_libpng]: [Batch params]: --build-dir "C:\test\carla\Build" --zlib-install-dir "C:\test\carla\Build\zlib-install"
-[install_libpng]: A libpng installation already exists.
-[install_libpng]: Delete "C:\test\carla\Build\libpng-1.2.37-install" if you want to force a rebuild.
-[install_libpng]: Exiting...
-[Setup]: Installing rpclib...
-[install_rpclib]: [Batch params]: --build-dir "C:\test\carla\Build"
-[install_rpclib]: Not cloning rpclib because already exists a folder called "rpclib-src".
-[install_rpclib]: Generating build...
-- Configuring done
-- Generating done
-- **Build files have been written to: C:/test/carla/Build/rpclib-src
-[install_rpclib]: Building...
Error: could not load cache
-[install_rpclib]: [Visual Studio 16 2019 Win64 ERROR] An error ocurred while installing using Visual Studio 16 2019 Win64.
-[install_rpclib]: [Visual Studio 16 2019 Win64 ERROR] Possible causes:
-[install_rpclib]:                - Make sure you have Visual Studio installed.
-[install_rpclib]:                - Make sure you have the "x64 Visual C++ Toolset" in your path.
-[install_rpclib]:                  For example using the "Visual Studio x64 Native Tools Command Prompt",
-[install_rpclib]:                  or the "vcvarsall.bat".
-[install_rpclib]: Exiting with error...**

-[Setup]:
Ok, and error ocurred, don't panic!
We have different platforms where you can find some help :)

Make sure you have read the documentation:
http://carla.readthedocs.io/en/latest/how_to_build_on_windows/

If the problem persists, you can ask on our Github's "Building on Windows" issue:
https://github.com/carla-simulator/carla/issues/21

Or just use our Discord channel!
We'll be glad to help you there :)
https://discord.gg/42KJdRj
make: *** [setup]

Did you manage to resolve this? I am having the same problem. I am working on Windows Server 2022.

Hello, I am having the same error message pop up when I use the cmd make PythonAPI.
I have:
Windows 10, python 3.8, admin x64 Native Tool Cmd Promt for VS2019 (VS 2019 plus all packages listed in instruction doc for windows build), version 3.81 of make etc.
I feel like I have tried everything... I was even on discord trying to find a solution and got nothing.... any help would be greatly appreciated :) I have started from scratch many times at this point.


BTS Admin - Modify resources after BTDF deploy - Dependency not found

Hi,
I'm posting this to verify that the behavior is by design;
Let's say Orchestration 'OrderBroker' is using component 'HelperLibrary'. The BTDF project deploys these artefacts based on 2 defined itemgroups:
Orchestrations
Components
After deployment, the Components will only be installed in the GAC. When you click on 'Modify Resource' in the BizTalk Administration Console, on the Orchestration 'OrderBroker', you will see a dependency missing error.
I know that at runtime, the GAC due to the .Net framework / probing and stuff, and that the Orchestration works fine, however i would like to ensure that i'm not missing anything.
Can this be confirmed?
Attachments
dependency.png
This work item was migrated from CodePlex
CodePlex work item ID: '10103'
Vote count: '1'

[tfabraham@5/12/2013]
By design.  The Deployment Framework does not add .NET assemblies as resources in the application.  One problem with adding assemblies as resources is that a particular assembly must be unique in the BizTalk group, so take our SSOSettingsFileReader.dll as an example.  It may be used by nearly every BizTalk app, but it can be added as a resource to only one app... so what to do?  I don't have a good answer and there is no functional requirement to add as a resource except to satisfy the dependency analysis tool in 2013.  Open to suggestions...

[fkuiper@5/13/2013]
Hi Thomas,
Maybe this could be as simple as adding an extra property to the items in itemgroup "Components". If this property is set (let's call is AddAsResource) the assembly is added to the BizTalk application. If this property is not set (or present) this step is skipped and the assembly will only be added to the GAC.
Regards,
Ferdinand.


Playground feature

Looks great :) I have one thought.. is it possible/good to load values from example.json automatically?

Also is it good to have a playground link here also? (to open in new tab) :

And when we have any error is it possible to show it within UI? or at least a message saying "There is an error please check console"

Looks great :) I have one thought.. is it possible/good to load values from example.json automatically?

Yes that would be great, any ideas how we could easily do that in the frontend?


Possible fix for raw event change in Proton

Creating PR to make sure this doesn't break anything before makeing the corresponding change in Proton

@kgiusti I've tried this change against the new version of proton on my own machine and I'm still getting the qd_adaptor_buffer_t leaks so I guess it's not as simple as just this change,

ok I'm getting the same leak run against an unmodified proton too. so I'm confused


08.08.01

As an approver, I want to approve a submitted expense claim that was approved, denoting the claim status as approved and setting my name as the approver for the expense claim.

Previously done


AIY VISION - The class list for the video_capture.py file is unavailable

AIY VISION - The class list for the video_capture.py file is unavailable in the code examples.
It's only written .

This file is available in the latest code revision: https://github.com/google/aiyprojects-raspbian/blob/aiyprojects/src/examples/vision/video_capture.py

All I see is this. How do I get it? "
A full list of possible categories can be obtained here:
 "
D A V I D  C Ô T É
Whatsapp: +1<PHONE_NUMBER>
"The best way to predict the future is to create it" - P. Drucker
On 10 May 2018 at 17:25, Dmitriy Kovalev<EMAIL_ADDRESS>wrote:

This file is available in the latest code revision:
https://github.com/google/aiyprojects-raspbian/blob/
aiyprojects/src/examples/vision/video_capture.py
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/google/aiyprojects-raspbian/issues/364#issuecomment-388191024,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AHQTU2lWF_0aZB3YMcjqbKLRGZgGC_6kks5txLBlgaJpZM4T6f86
.

All classes supported by image classification can be found here: https://github.com/google/aiyprojects-raspbian/blob/aiyprojects/src/aiy/vision/models/image_classification_classes.py
That file (specified by --classfile option) contains a subset of these classes, we'll put an example and update the doc in the next release.

Unfortunately, the link is broken. I'd love to have the list of all the classes used in the 'image_classification_camera.py' example

This  comment line in "video_capture.py" refers to "image_classification_classes.py", which does not exist in current release "v20181116".
In previous release " v20180803" the file was still there. So you can go to the older branch ("Branch"-->"Tag"-->" v20180803") and look for "image_classification_classes.py" or just use this direct link


Move insertSorted from server to core, use for diagnostic collections

This causes us to spend considerably less time sorting diagnostics in projects with many files where we alternate between getting the file's diagnostics and adding diagnostics (~10s, or 8% on a very large project). Additionally, I've upped the time rwc tests have to verify js output, because our newest RWC test takes 5s (!!!) just to textually compare its js emit, and mocha's default 2s timeout was causing it to timeout when I ran it via mocha directly (to profile it).

@sheetalkamat quicksort (as is usually done by the JS engine) has an average time complexity of n log n, but a worst case complexity of n^2 (though it may use another algo if it heuristically determines it's in a worst-case scenario). Inserting sorted is a log n operation, done n times, so is just always n log n time complexity. We always ask for sorted diagnostics so this is minimally as good as the previous delayed-sorting method from a computational standpoint, since we will always query for diagnostics at least once, and this significantly reduces the constant factor if we query for diagnostics (between mutations) more than once. Plus, it makes the implementation way cleaner to just keep a sorted list at all times, since they we don't need to keep dirty bits in the state.

@weswigham i am not sure i understand the original issue, why were we alternating between getting errors and adding them? is that for global? or for file diagnostics?

getSemanticDiagnostics, as used by emitWorker or getPreEmitDiagnostics, takes a source file (or not). When it is called with an individual source file (as is always true in emitWorker and  sometimes true for getPreEmitDiagnostics), it called into checker's getDiagnostics function, which in turn calls getDiagnosticsWorker, which, when called with a sourceFile, gets the global diagnostics, then checks the file, then gets the file diagnostics and the global diagnostics again (so it can return only global diagnostics added during the check of that file). All three of those calls were capable of triggering diagnostic sorting (though only the second is likely to). Previously when any of the methods on diagnostic collection were called to get any subset of the diagnostics, it would sort all diagnostics (unless it had already been sorted and hadn't been modified in any way) - including those from files that weren't currently being looked at. Additionally, whenever you queried for all diagnostics, it would sort and dedupe everything... then combine them all into an all diagnostics array... then sort and dedupe them again. (The first sort and dedupe being extraneous.)
While a sort-once strategy is likely fine for per-file diagnostics (provided it's actually only sorted once ever), both the list of all diagnostics and the global diagnostics list need to be constantly resorted, since additions (can) occur in every file. Rather than holding a dirty flag for every file and sorting file diagnostics once, but also inserting sorted for the global and all diagnostics lists, it's much simpler, implementation-wise, to use insertSorted for all of the lists, and is just as effective to use for the per-file lists provided we always actually query for them (and we always query for diagnostics if the checker is configured to generate them), and is way less likely to become a unintended performance pitfall in the future (for example, if there's some future feature where you only check part of a file at a time).

I like your suggestion of creating allDiagnostics list on demand by combining all file diagnostics in the right order instead of having to keep a second sorted list. let's do that as well.

@mhegazy done.


lt does not support torch.complex Tensors

To reproduce
import torch
import lovely_tensors as lt
lt.monkey_patch()

x = torch.randn(4, dtype=torch.complex32)
x

Gives
RuntimeError: isposinf does not support complex inputs.

Hi. Thank you for the report.
Do you have a concrete use case for complex tensors?
I will make sure lt can handle them, but do you have ideas on what a concise and informative representation of a complex tensor should look like?
Besides a text summary, is there a graphical representation that would make sense?

I've pushed a quick fix that just displays the plain version for complex tensors.
# We don't really supposed complex numbers yet
c = torch.randn(10, dtype=torch.complex64)
c

tensor([ 0.5135+0.0645j, -0.2751+0.3733j, -0.2552-0.0428j,  0.0518+0.5789j,  1.0469+0.2439j, -1.0070-0.0823j,
     0.1538-0.0330j, -1.0137-0.4006j, -0.3007+0.1856j, -1.0176+0.3687j])

Let's keep the issue open because there is definitely room for improvement.

A simple solution would be to treat real and imaginary parts of the tensor as two different tensors, perhaps using torch.view_as_real

A simple solution would be to treat real and imaginary parts of the tensor as two different tensors, perhaps using torch.view_as_real

Could you suggest a codebase, ideally a notebook, that is heavily using complex tensors? I feel that I still need to better understand how people use them in real code.


Add support for Truncate

Add TRUNCATE TABLE <TABLENAME> to product.
result of truncate:
Table definition exists for table but all data on disk and in memory is removed.
Will currently remove data from dictionary but there is an open question of whether we may want an option to keep dictionary.

Not a great idea to purge the dictionary if it's shared, but maybe the truncate is going to be followed by reloading with updated comment and substantially same dictionary encodings.  Maybe this and shared dictionary means there should be something like a clean or truncate dictionary?

closed with 45f8069ac34ba1b4b07c7759ed822f44397f6f44


Use wide class if no ToC

Is there a simple way to make the single layout always use classes: wide, unless toc: true is also set?
In other words, I want posts to be wide by default so they fill the right side, unless I have specifically added a Table of Contents, in which case I want that to still appear on the right side.
Unless I'm mistaken, this is not easily achievable since you can't turn off a class in front matter?

This is what Jekyll’s front matter defaults are for. Set the default to classes: wide and on those posts/pages where you set toc: true you can also set classes: false to disable it.
https://jekyllrb.com/docs/configuration/front-matter-defaults/

Might be easier to implement if you override the layout file to add the class automatically when page.toc is not enabled.

@mmistakes Thanks, I knew about the front matter defaults but wasn't aware I could do classes: false
@iBug That sounds like a very good solution, thank you!

@iBug Thanks for getting me on the right track. There's probably a much better and shorter way to do it than this, but I'm not too familiar with Liquid. It works, though!
_layouts/default.html
  {% unless page.toc %}
    {% if page.classes or layout.classes %}
      {% assign classes = page.classes | default: layout.classes | push: 'wide' %}
    {% else %}
      {% assign classes = 'wide,' | split: ',' %}
    {% endif %}
  {% else %}
    {% if page.classes or layout.classes %}
      {% assign classes = page.classes | default: layout.classes %}
    {% else %}
      {% assign classes = '' | split: '' %}
    {% endif %}
  {% endunless %}

  <body class="layout--{{ page.layout | default: layout.layout }}{{ classes | join: ' ' | prepend: ' ' }}">


.php_cs - add few rules

Add few rules to PHP CS Fixer configuration that are already followed.

@Ocramius with other CR team developers have you thinking a set of common roles for cs-fix? (thoughts after split :P )

@gianarb that would be great. But currently we do not know which rules are followed in actual codebase - they are followed in most cases, but to turn then on they usually need little fixing to made Travis happy about them.

Np for this PR.. It's a general question :+1:
Sorry for this intrusion :smile:

@keradus thanks, merged after some further thoughts about it


Add innerRef prop to components

It would be nice to be able to get the underlying ref node for some of the components. For example, for this line chart I made, I needed to modify @vx/shape's LinePath to take an innerRef prop that would then assign it to the path element's ref. This way I could gain access to the svg path element, and glean some data from it (path.getTotalLength() and path.getPointAtLength()) Other libraries that I know of that do something similar are glamorous and styled-components.
Thoughts?

yup will make refs opt-in

since forwardRef is part of react nowadays, wouldn't it make sense to change innerRef to ref?


Build error on RCTWebSocket

I was trying to build the application on my computer but the build failed, I thought it was node version I had 5.0, but even with 4.0 I have problems, apparently different problem, but still I can't succeed.
Which version of node do you have there, and which version of IOS do you target as well?
Here a screenshot. Thanks! :-D

Hi Daniele,
which version of XCode are you using?
I have the 7.1 installed-
bye
A

I don't know how but it's working on my new mac :+1: I'll close the issue, and I'll send you pull requests instead :-D


Use crispEdges rendering for octicon-internal-repo

Came up in https://github.com/go-gitea/gitea/pull/11771#issuecomment-640438994
Applied only to this specific octicon so that it doesn't interfere with others - that octicon is kinda special in a way that it's 13px, unlike others that are 10px, 12px or 14px.

CI failure unrelated

So your theory is that this brower bug only triggers on odd widths only?

Most likely with combination of font size/line height or specific positioning and size; for example I can not reproduce the issue on 32px for example. It's less visible on 24px but still can be noticed if two were stacked near each other.

Interestingly can reproduce it on 30px but it's crisp on 32px (which is exactly double of 16px which is also broken)

Might have to do something with this particular SVG, I don't know. This icon was added late in octicon lifecycle and looking at some issues on GitHub they might not even have original files for it.
It's very possible that like in original Firefox issue, it's not about odd width but sub-pixel anti-aliasing and it just so happens that this icon is made in some unique way that causes it to manifest on certain resolutions.

Just noticed that it's blurry on 32px but only when zoomed; except for multipliers of 100% such as 200% or 300% - on those it's crisp.

Any drawback to using crispEdges on all SVG icons?

In the end adding this CSS rule for this icon only seems like a good compromise to me; it works in all cases I tested and doesn't affect anything else. We'll be able to remove it whenever we switch to new octicons (once they're out officially) or when we get to parsing octicon JSON (whichever is first I suppose).

Oh, and does geometricPrecision have any effect on your issue? I remember using that once.

Any drawback to using crispEdges on all SVG icons?

Jagged edges on icons such as notification bell; generally I don't think it's recommended to use it on all icons; it's a workaround.

Oh, and does geometricPrecision have any effect on your issue?

No, but I believe disabling anti-aliasing is what actually fixes it for this icon and it doesn't has any large rounded shapes so it's fine either way.


Don't open the Edit window by default

Is your feature request related to a problem? Please describe.
Sorry if I misunderstood something.
After displaying a plugin, the Edit window appears by default.
That window can only be closed after clicking the Edit button.
Describe the solution you'd like
I feel that the Edit window should be opened after the Edit button is clicked.
Additional context
Screencast:
https://github.com/user-attachments/assets/9e075bc9-457a-46d4-876a-681b6de4571a
Environment:

OS: Arch Linux x86_64
ysfx-s: WIP version
renoise 3.4.4-1
sway-git 1.10.r7397.7288f77-1
wlroots-git 0.19.0.r7218.54321088-1
mesa 1:24.2.1-1
linux 6.10.8.arch1-1

That's indeed a bug and I was already wondering about this behavior when I saw your previous screen grabs.
This might take a bit of figuring out. I hope you're up for testing a few builds with me as I cannot reproduce this locally.
Right now, my suspicion is that setContentNonOwned (that's where ysfx binds the IDE to that (then invisible) window) behaves differently on different platforms.
Alright, attempt 1: Linux 64-bit VST3.zip.

Alright. If setting it to invisible doesn't work, then let's try just deferring the window entirely until you hit edit. Here is attempt number two: Linux 64-bit VST3.zip

It works fine.
https://github.com/user-attachments/assets/15899720-21f5-4ff0-b2f8-66c778101bb8
You can close it as you wish, Thanks!

Wonderful. Glad to hear that.
Thanks for testing and please don't hesitate to report future bugs!
I'll do a few more tests and then make another quick pre-release, so that hopefully not too many people end up using the plugin with the old naming convention.


install dlib error

Hi, your image helped me a lot in saving time.👍
By the way, I want to install the dlib library, but an error occurs. Do you know a solution?
I'm really sorry. I've tried all the methods on the internet, but I can't solve it, so I'm asking

I'm sorry, I decided to go the other way, so I'll close it. If you can, please delete it.


Implement PG identifier parsing algorithm

As proposed by @alco in https://github.com/electric-sql/electric/issues/1764#issuecomment-2391485961:

I would argue against going to Postgres for name lookup. Replicating Postgres' identifier parsing is much simpler because it can be implemented as a pure function. Postgres is open source and it's not like its definition of what is an identifier is going to change any time soon: no DB queries or caching required.
I have to admit that Postgres' definition of the identifier token (as specified in its lexer code) is a bit cryptic: it explicitly mentions the ranges A-Z, a-z and byte values in the range \200-\377 (those are octal numbers, in decimal they are 128 to 255), but I think the intent there is to allow any valid UTF-8 code point beyond the 7-bit ASCII Plane, so not only letters. For example, this works in psql:
=# create table 🪷(id text);
CREATE TABLE
=# \d
        List of relations
 Schema │ Name │ Type  │  Owner
────────┼──────┼───────┼──────────
 public │ foo¡ │ table │ postgres
 public │ Δ    │ table │ postgres
 public │ 🪷   │ table │ postgres

There is, apparently, a separate validation to determine that the given byte sequence is valid UTF-8 encoding (or any other encoding that the database server is configured to use) because not all sequences of bytes are valid identifiers.
Regardless, the algorithm for lower-casing identifiers is very small and straightforward. Here's its full implementation -
https://github.com/postgres/postgres/blob/259a0a99fe3d45dcf624788c1724d9989f3382dc/src/backend/parser/scansup.c#L55-L73
So it is conceivable that we can implement our own validation of user-provided table identifiers by checking they are valid UTF-8 strings and don't contain certain ASCII characters. Then we can normalize the identifier by implement Postgres' downcasing algorithm to obtain a string that we can use to fetch table info from Postgres and as a cache key for storing table info in memory.

@alco @kevin-dp the postgres algorithm depends on the server encoding as well - should we grab the encoding in the metadata acquisition stage, or just default to UTF8 for start?

I've implemented a basic Electric.Postgres.Identifier.parse in https://github.com/electric-sql/electric/pull/1829 to serve the immediate needs of that feature.
Postgres makes use of it's server encoding and max identifier character length configuration, so we'd need to retrieve those as well as part of our "prep" and pass them along to the parsing routine. I will address that in a separate PR, probably overlaps with @alco 's work on putting all prep queries in the replication connection.

@alco @kevin-dp the postgres algorithm depends on the server encoding as well - should we grab the encoding in the metadata acquisition stage, or just default to UTF8 for start?

In the spirit of "any complex system that works has evolved from a simpler system that worked" let's start with assuming UTF8 first.

Addressing this in https://github.com/electric-sql/electric/pull/1873 - if we don't want to merge it then I suggest we close this issue


Feat/sidebar meta info

This PR includes the changes from "Feat/label with meta info on hover" So review that first, merge it, and then review this. THX

Nice works:)
I played around a bit, I noticed that for aois over the river I did not find a timestamp where something was detected, even though I clicked around a bit and theoretically half of the timestamps should have something. Is it because our detection did not work properly or is it just coincidence that I selected only the onces where nothing was detected?


MINOR: Kafka And ZooKeeper Stop Scripts doesn't return PID

In Centos 6.5, 'ps -ax | grep java' command doesn't returns the full process name. So, the
command doesn't returns the PID even Kafka and ZooKeeper is running.
Updated the script to use Java Process command (jps) which is elegant to retrieve the running
java process PID

I'm not sure it is safe to assume jps will be available. I think its packaged as a jdk tool and some installations that only have JRE won't have the tool (at least that's how things are organized in the Ubuntu OpenJDK packages and I assume that falls out directly from upstream JRE/JDK distinctions).

Agreed. Thanks for the feedback.
In Linux, the ps command seems to truncate the result to 4096 characters (it expands the full path of all the libraries in the classpath.). So, grep -i 'kafka.Kafka' returns nothing.

Yes, we've run into this with system tests. I think generally the solution was to reduce CLASSPATH length by making sure * is used properly without expansion to minimize the number of CLASSPATH entries (or setup the full CLASSPATH and export it), although I think writing a pid file is a much more reliable solution.


New Library Request: react-native-bundle-splitter

In order to properly evaluate if a new library can be added to package.json, please fill out this request form. It will be automatically assigned someone from our review team that will go through and vet the library.
In order to add any new production dependency, it must be approved by the App Deployer team. They will evaluate the library and decide if it's something we want to move forward with or if other alternatives should be explored.
Note: This is only for production dependencies. While we don't want people to add packages to dev-dependencies willy-nilly, we recognize that there isn't as great of a need there to secure them.
Name of library:
Details

Link to package: https://www.npmjs.com/package/react-native-bundle-splitter?activeTab=readme
Problem solved by using this package: https://github.com/Expensify/App/pull/48899
Number of stars in GH: 415
Number of monthly downloads: 10557
Number of releases in the last year: 2
Level of activity in the repo: Low
Alternatives: We can write a preloading mechanism ourself. It's just a matter of an additional code that will be present in Expensify codebase which we'll need to support.
Are security concerns brought up and addressed in the library's repo? No
How many dependencies does this lib use that will be brought into our code? 0
What will the effect be on the bundle size of our code? Size of the package is 20kb, but bundle will be increased by ~11kb

I haven't had a chance to look into this- @dangrous let me know if you have additional context. I'll take a look shortly

Summary is, it allows for preloading of components:

1⃣ preload the component in advance to be able to mount it synchronously on a demand
If we mount the component and the component is lazy and haven't been loaded yet, then when we switch navigator we remove old navigator, but a new one navigator doesn't have routes yet, so react-navigation clears a state, and when new routes are attached then a default route will be selected, because previous state was lost.

So we need to mount the screen/navigator synchronously.

I think we can 👍 this.

Alternatives: We can write a preloading mechanism ourself. It's just a matter of an additional code that will be present in Expensify codebase which we'll need to support.

@kirillzyusko would this be time-consuming or hard to build on our own?

@grgia I don't think it'll be time consuming, but from the other side I don't see a lot of sense in re-implementation that code directly inside the Expensify app, because:

it will be an additional code that we need to support;
new code will be identical to what's present in react-native-bundle-splitter

So I would vote for adding the library and don't add additional code to the Expensify codebase if possible 🙃

@kirillzyusko makes sense to me, I'll bring this to the appdeployers team for a vote

Posted in slack https://expensify.slack.com/archives/C03TQ48KC/p1730456781086199 for vote

the library looks good, I'm just curious:
are there are alternative libraries?
Instead of just fixing the underlying issue, would it be worth asking
@Kiryl Ziusko
to do a pass of the entire app for other rendering time improvements?
When we document the usage in Readme, are there any gotchas or complexities we'll need to be aware of?
as
@Kiryl Ziusko
is the library owner maybe he can answer these

@kirillzyusko could you take a look at these points from a comment in slack?

Looks like we are not going to move forward with this library as there are discussions about closing the original issue. I'm going to close this out.

@grgia just to confirm you are going to close the original issue https://github.com/Expensify/App/issues/44600 right?

are there are alternative libraries?

The other library I was thinking of is react-loadable. But this is web-only, so I thought it would make sense to use cross-platform library equivalent (react-native-bundle-splitter works on all platforms supported by RN).

Instead of just fixing the underlying issue, would it be worth asking
@Kiryl Ziusko
to do a pass of the entire app for other rendering time improvements?

Yeah, I can do it, but if the issue got closed, then I don't think it's worth to do that anymore?..

When we document the usage in Readme, are there any gotchas or complexities we'll need to be aware of?

I don't have we have any complixities you need to be aware of. At least when you add a library for web only (as it was done in PR) you don't need to change any metro configurations etc.
If you are adding it to the native (mobile) side, then most likely you will need to change metro config.


Modify Add and Edit + Testing to include GitHub Field

Test cases for AddressBookParser keep failing despite output being the same.
Update: Found the error 😄 (Added GitHub field to PersonUtil class)
Fixes #47.

Codecov Report

Merging #64 (7093555) into master (b7eb070) will increase coverage by 0.29%.
The diff coverage is 79.48%.

@@             Coverage Diff              @@
##             master      #64      +/-   ##
============================================
+ Coverage     72.33%   72.63%   +0.29%
- Complexity      399      406       +7
============================================
  Files            70       71       +1
  Lines          1229     1257      +28
  Branches        125      127       +2
============================================
+ Hits            889      913      +24
- Misses          308      310       +2
- Partials         32       34       +2

Impacted Files
Coverage Δ

.../java/seedu/address/logic/commands/AddCommand.java
100.00% <ø> (ø)

.../java/seedu/address/model/util/SampleDataUtil.java
20.00% <ø> (ø)

src/main/java/seedu/address/ui/HelpWindow.java
0.00% <ø> (ø)

src/main/java/seedu/address/ui/PersonCard.java
0.00% <0.00%> (ø)

...c/main/java/seedu/address/model/person/GitHub.java
70.00% <70.00%> (ø)

...c/main/java/seedu/address/model/person/Person.java
97.82% <85.71%> (+0.20%)
:arrow_up:

...java/seedu/address/logic/commands/EditCommand.java
97.22% <100.00%> (+0.20%)
:arrow_up:

...a/seedu/address/logic/parser/AddCommandParser.java
100.00% <100.00%> (ø)

...ain/java/seedu/address/logic/parser/CliSyntax.java
85.71% <100.00%> (+2.38%)
:arrow_up:

...in/java/seedu/address/logic/parser/ParserUtil.java
97.56% <100.00%> (+0.33%)
:arrow_up:

... and 1 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Adding Semantic Versioning to roadmap for version 1.0

Coverage increased (+0.03%) to 82.27% when pulling eee6636933a0668a9b95a682127be761ac59ac81 on fishcharlie:roadmap into 8f5c192cd50c96728494cb5fa615be8a3d9d7e44 on automategreen:master.


Dialog

is this imageview zooming works in full screen dialogs?

I've never tried that. Have you tried it?

Yes it is working perfect. Thanx @davemorrissey for such a beautiful lib.  One thing I am missing is, I want image to be full screen irrespective of aspect ratio is it possible?

Have a look at the configuration page in the wiki, you'll find what you need under the minimum scale type option.


[PF-278] First pass at Kubernetes common library

@jaycarlton I do not know a useful way to unit test this. Its entire purpose is interacting with Kubernetes.
I have been testing it by building data repo with it and running the repo integration tests that run under Kubernetes.


accessibility patterns and aria

Hello,
It appears several components dont expose ARIA roles and behave not very well with keyboard and assistive technologies.
I propose to follow these simple patterns :

give correct aria role on non-native components (like dialog, alert, etc...)
allowing keyboard navigation and actions (user should navigate with arrows inside a component, with tab in a form, activate a link/button with space ou enter, cancel with ESC)
trap keyboard focus in a modal dialog and reposition focus to the button that trigged the dialog when closing it.
animations-less mode

Thanks @grisha78
Yes, this is something that's really important to us. Please see #61


Add browserslist config to esbuild

Autoprefixer currently uses browserslist with the default browser set.
To view this:
In styles.js, change plugins: [ autoprefixer ] to plugins: [ autoprefixer.info() ] and run yarn styles and see browserslist output.
This PR adds back the browserslist string to package.json so that both esbuild and autoprefixer are using the same browser set.
We could do "browserslist": "defaults", in package.json if we want to, but I think as written it's a little clearer.
Additions

Adds esbuild-plugin-browserslist for feeding browserslist string in esbuild.
Adds browserslist string to package.json, without IE11.

Removals

Unreferenced postcss-unmq and require-dir

How to test this PR

In the root directory, run npx browserslist to see the browser targets.
Change browserslist string to "browserslist": "ie >= 11", in package.json
Re-run npx browserslist and see that it only outputs ie 11.
Run export DEBUG=esbuild-plugin-browserslist
5 Run yarn scripts and see it fails and esbuild info shows IE11 was targeted (which isn't enough for esbuild).
In styles.js, change plugins: [ autoprefixer ] to plugins: [ autoprefixer.info() ]
Run yarn styles and scroll up to see browserslist output targets is only IE11.
Revert browserslist string in package.json to > 0.5%, last 2 versions, Firefox ESR, not dead
Re-run yarn scripts and see the targets are updated and it passes.
Re-run yarn styles and see the targets are updated.

I actually think autoprefixer and esbuild should not use the same browserslist since our CSS support and JS support targets differ!

I fully agree that esbuild and autoprefixer should have the ability to have different targets. I also think the apps should have this ability as well, lest we can't update the whole site targets because of some edge-case app stakeholder hold-up.
If we want everything to have the same targets, I think that looks like removing the target in esbuild and having the browserslist string in package.json, where esbuild will scoop it up (with the plugin in this PR).
If we want to have differing targets, then any that differ from what is in base package.json defines its own targets (e.g. esbuild uses the base package.json values and autoprefixer sets its own target). I haven't checked how this would work with apps (can we add a browserslist file to their directory and will esbuild scoop that up?).
But back to the question of esbuild/autoprefixer targets differing. We do kinda have four domains: un-transpiled JS (no-js state), transpiled JS (js state), un-prefixed CSS, and prefixed CSS.

The major thing on the CSS-side of the house is dropping gulp-bless, which will majorly bust the layout on IE 10<, which at this point we're saying we don't care due to such low usage. In this PR the targets are > 0.5%, last 2 versions, Firefox ESR, not dead, which actually picks up IE11 (you can see with npx browserslist). For whatever reason esbuild doesn't care about this, but autoprefixer gets the IE11 prefixing and all is as it is now from a CSS-perspective in IE11. If we explicitly exclude IE with > 0.5%, last 2 versions, Firefox ESR, not dead, not ie 11, the layout begins to break in certain spots, but I don't think it will be completely destroyed like it will be in earlier IE without gulp-bless.

So similar to the argument of our supported browser set being evergreen, maybe at this point it makes sense to have the targets the same for esbuild and autoprefixer? (Again though, with the possibility they could differ, if required).

In https://github.com/cfpb/consumerfinance.gov/commit/ce6ea22f2109178c7315e0f73a2fa844c0f5c7d6 I set the esbuild > target to include the lowest version of the listed browsers with >0.2% usage (arbitrary cutoff). I know you wanted to use
real analytics data & to automate the generation of this list, but I think that work might be in conflict with this PR.
Thoughts?

So my thoughts on this are something like this: wouldn't it be nice if we could quickly and clearly answer what our browser support set is, what our visitor browser set is, and how to quickly and cleanly move our support to meet our visitors over time?
As far as live analytics data, what I have in mind is not necessarily a direct feed into cfgov, but rather a dependency that can become outdated so we know it needs updating. What that might look like is https://github.com/cfpb/cfpb-analytics/blob/main/packages/browserslist-config/browserslist-stats.json gets updated on a regular interval and https://www.npmjs.com/package/@cfpb/browserslist-config gets bumped and released at the same cadence.
Then in cfgov, the browserslist set in package.json could be the direct metrics (with syntax similar to here).
The site wouldn't be constantly shifting around in its build targets automatically, potentially mysteriously breaking stuff, but instead on a regular cadence we could bump @cfpb/browserslist-config in a PR for review and over time move our targets to a more modern set.
We could also incorporate eslint-plugin-compat and stylelint-no-unsupported-browser-features to be able to lint the codebase against the PR that bumps the browserslist and flag any issues that way.
Side note: I appreciate this discussion! I feel like we're narrowing in on a lean-mean-great-config-machine!


Latest master: f5 fails due to a build failure, ctrl+f5 works fine

This has been a constant annoyance for at least 2 years, where f5 regularly fails due to being unable to find referenced .dlls in project output directories. But ctrl+f5 works just fine.

This prevents me from seeing if anything will fail to build. For example, if I'm in FSharp.Editor and it cannot find a referenced .dll, I cannot see if I fail to implement an interface or not.


feat: init provider console with next, tailwind and shadcn

added Next js
added tailwind CSS (merged with shared CSS)
added shadcn

Minor changes

Fixed all the minor changes.


More ways to have commandlines?

Doing a project I realized many options and ways to configure commandline are missing and being told by many that this is one of the best libraries out there, makes me think that there is a reason that these options were not implemented.
So my question is, is there a guideline or something that developers follow? or just nobody needed these features or ...?
For example I couldnt make commandline to work to understand followings
 myapp.exe -ab
 myapp.exe --metadata title=title --medata comment=comment
 myapp.exe =otherseperator

So problem with first one is two letter verb.
Second line, it seems you can not have multiple verbs
Third one, I just didnt see a way to change default '-' to something else.

So problem with first one is two letter verb.

The option -ab is considered as two flags (boolean options)
          -a -b

Flags  can be merged together as  -ab

Second line, it seems you can not have multiple verbs

This is considered multiple options, but it's implemented as (with space as a separator, or you can assign your own):
                   myapp.exe --metadata title=title comment=comment

You can define multi verbs, but one verb at a time and verb has no preceding - or --
            merge -a file1 file2 -o file

Third one, I just didnt see a way to change default '-' to something else.

The symbol - and -- are GNU and POSIX specifications standard and commandlineParser follow this standard
See this example that is based in your usecase:
		  -ab --metadata  title=title   comment=comment

	 void Main(string[] args)
		{
				  var  result = Parser.Default.ParseArguments<Options>(args);
					result.WithParsed(opts => Run(opts))
					 .WithNotParsed(err=>Console.WriteLine(err));

		}

	 void Run(Options options)
		{
			Console.WriteLine(options.FlagA);
			Console.WriteLine(options.FlagB);
			options.MetaData.ToList().ForEach(x=>Console.WriteLine(x));
		}

	public class Options
	{
		[Option('a', "flag-a", HelpText = "flag a")]
		public bool FlagA { get; set; }
	         [Option('b', "flag-b", HelpText = "flag b")]
		public bool FlagB { get; set; }
		[Option('m', "metadata",Separator=' ', HelpText = "MetaData")]
		public IEnumerable<string> MetaData { get; set; }

	}


Api versioning for web api core 2 does not work with TestServer

Hi,
I'm trying to test web api controller(.net core 2 ) that use api-versioning.
The configuration of api-versioning:
ReportApiVersions = true;
AssumeDefaultVersionWhenUnspecified = false;
ApiVersionReader = new MediaTypeApiVersionReader();
The controller works well using swagger and postman, the problem when I use TestServer client ( for integeration tests), I get all the time bad request with error message "api version is required", the accept header is set correctly.
If I change the configuration key AssumeDefaultVersionWhenUnspecified to true I did not get the exception anymore even if I don't use the default version.
Any advice ?
Thanks.

I presume you found a resolution. If you haven't resolved the issue, feel free to come back. Thanks.

I set AssumeDefaultVersionWhenUnspecified to true, and it fixed the issue.

You can use AssumeDefaultVersionWhenUnspecified = true, but that isn't necessary to make it work. I should elaborate that you have configured things to use media type negotiation. If you're getting 400 with this error message, then that means you are not specifying the API version in the media type. By default, API versioning requires an API version to always be explicitly specified.
I'll assume that you're using JSON payloads. This means that the client now needs to explicitly specify a media type that includes the API version in requests. For requests that do not specify a body such as GET or DELETE, the client needs to specify Accept: application/json; v=1.0. For requests that have a body, the client should specify Content-Type: application/json; v=1.0.
The behavior of using AssumeDefaultVersionWhenUnspecified = true will allow the client omit the v media type parameter or even omit the media type altogether. Remember that only one API version can be implicitly matched this way (by default). You should never change the default API version if you enable this feature because you might break existing clients.
I hope that helps.


Doc update

This documentation introduces a 3rd section: "the pySTEPS developer guide" with many guide and guidelines useful for pySTEPS developers and contributors.
The subsections are:

Contributing to Pysteps
Installing pysteps
Testing pySTEPS
Packaging the PySteps project (guide to create a distribution and uploaded to pypi)
Creating a pySTEPS conda package (here for reference mostly. It won't be very useful if the package is in conda forge)

Also, the already existing sections in the documentation were renamed:

User reference -> User guide
Developer reference -> pySTEPS reference

I upload an online version of this documentation here.

Great step forward for our documentation!


Fix DCCRN and DCUNet-Large

This is the real DCCRN architecture. I got some of the details wrong in the initial version.

Are we simply going to release the fixed version without any backwards compat? I think that's reasonable since we have no pretrained models so far and it's only been a few days since the release.
The code the concatenative skip connection with the intermediate layer is a draft. I'm not sure how to do it in a clean way just yet. The way the intermediate layer works in this new version requires that we are aware of its existence in the u-net params calculation, so we can't really keep them entirely separate anymore. Maybe we have to add the concept of this concatenative intermediate layer to the u-net param calculation code. If you have any ideas that would be helpful as well!

@kssk16 Note that our LSTM is of size 640 rather than 512 because we're not stripping the Nyquist frequency so have 257 rather than 256 frequency bins.
Maybe we should change this as well so the implementation matches the paper.

@kssk16 Note that our LSTM is of size 640 rather than 512 because we're not stripping the Nyquist frequency so have 257 rather than 256 frequency bins.
Maybe we should change this as well so the implementation matches the paper.

@jonashaag Thanks for letting me know.

More differences:

They use PReLu, not Leaky ReLu
They use (2, 1) padding for all encoders and (2, 0) padding for all decoders
In the output layer they use output_padding=(1, 0)
They use a window of size 400, pad it (centered) to 512 and then use a 512 FFT. We don't do the 400 thing.

Note: New WIP version here https://github.com/jonashaag/asteroid/blob/b053ed67806da48270742ff01202d158a3faaef8/asteroid/masknn/recurrent.py#L549 https://github.com/jonashaag/asteroid/blob/b053ed67806da48270742ff01202d158a3faaef8/asteroid/models/dccrnet.py

OK, this should be ready to review.
The DCCRN implementation isn't 100% identical to the paper one, but very similar. Differences I know of: using 512 FFT instead of 400 + 2 * 56 zero padding; slightly different padding in T dimension.
While fixing the DCCRN implementation I realized that you don't need the intermediate_layer concept because you can simply replace it by an "asymmetric" u-net structure where the deepest layer's "partner" is the identity. In fact, the intermediate_layer concept doesn't actually fit the DCCRN architecture, so I removed it.
I also included a fix to the DCUNet architectures that I realized was wrong.
In general, I'd appreciate if someone could compare my implementations to the papers because I'm quite unsure if there are more differences that I didn't spot. Code can be found here https://paperswithcode.com/paper/dccrn-deep-complex-convolution-recurrent-1 https://paperswithcode.com/paper/phase-aware-speech-enhancement-with-deep-1
Commit fc0232cdf9892de50ecdaff23183da293776ccf4 is sort of unrelated, we can exclude it.

Maybe @kssk16 @huyanxin @sweetcocoa and @chanil1218 would like to have a look? That would be nice!

If I print in the BaseUNet's forward in this way
    def forward(self, x):
        enc_outs = []
        for idx, enc in enumerate(self.encoders):
            x = enc(x)
            enc_outs.append(x)
            print(f"Encoding {idx}", x.shape)
        for idx, (enc_out, dec) in enumerate(zip(reversed(enc_outs[:-1]), self.decoders)):
            x = dec(x)
            print(f"Decoding {idx}", x.shape)
            x = torch.cat([x, enc_out], dim=1)
        return self.output_layer(x)

I get
print(tf_rep.shape) torch.Size([1, 256, 36])
Encoding 0 torch.Size([1, 16, 128, 35])
Encoding 1 torch.Size([1, 32, 64, 34])
Encoding 2 torch.Size([1, 64, 32, 33])
Encoding 3 torch.Size([1, 128, 16, 32])
Encoding 4 torch.Size([1, 128, 8, 31])
Encoding 5 torch.Size([1, 128, 4, 30])
Encoding 6 torch.Size([1, 128, 4, 30])
Decoding 0 torch.Size([1, 128, 4, 30])
Decoding 1 torch.Size([1, 128, 8, 31])
Decoding 2 torch.Size([1, 128, 16, 32])
Decoding 3 torch.Size([1, 64, 32, 33])
Decoding 4 torch.Size([1, 32, 64, 34])
Decoding 5 torch.Size([1, 16, 128, 35])

where the time dimension is not divided by two at every iteration, but just looses one. Is this intended? At least this doesn't seem consistent with the docstring and also your comments that suggest that the time dim has to be divisible by the prod of strides. Are the strides on both freq and time dimension?
By the way, for DCCRNet, the jit test passes even with different size, which is great news !

Could it be that you confused DCUNet and DCCRNet here?
$ python -c 'import asteroid.models as m, torch; m.DCUNet("DCUNet-10")(torch.randn(1,50000))'
Encoding 0 torch.Size([1, 32, 129, 97])
Encoding 1 torch.Size([1, 64, 65, 49])
Encoding 2 torch.Size([1, 64, 33, 25])
Encoding 3 torch.Size([1, 64, 17, 13])
Encoding 4 torch.Size([1, 64, 9, 13])
Decoding 0 torch.Size([1, 64, 17, 13])
Decoding 1 torch.Size([1, 64, 33, 25])
Decoding 2 torch.Size([1, 64, 65, 49])
Decoding 3 torch.Size([1, 32, 129, 97])
...

The paddings are different in DCUNet. In DCCRNet we use the "official" padding (or very similar), that is, the encoders each lose 1 frame in the T dim and the decoders add it back using output padding.
In DCUNet, there is not official implementation available, so I used what's most common with other implementations, which seems to be padding so that for each encoder, T -> (T+1)/2 roughly, for example 17 -> 9 -> 5 -> 3 -> 2 -> 2. We can also use some other padding, I'm not an expert in u-nets (not expert in anything in deep learning really 😅)

It is completely possible, yes !
I'm not an expert in either of these architectures and didn't read the papers in detail recently.
Given that DCCRNet is much more flexible, I guess it makes sense to isolate the tests of DCUNet, right?
I'll do that

IMO, this should be good.
I found the correct durations for DCUNet by hand as well, so we test 3 shapes and the traced model is consistent, which is great !
Few things:

Having to guess input shape is very bad, and the error message is half helpful because it is in the frequency domain, so it doesn't directly help to get the right shape in the time domain. How can we improve that? Do we want to improve that?
Those tests are pretty slow, so it would be nice to have "mini" architectures.

Extending DCCRNet to separation shouldn't be too hard, we just need to overwrite the second entry of the last line in the architecture file that is past to the complex conv, right. So we could easily add a n_src arg to it and enable separation. Am I missing something?

I made "mini" architecture.
Still wondering about the guessing of the input shape.
Maybe padding trimming the time axis and raising a warning would be better? WDYT?

In the line of:
Index: asteroid/masknn/convolutional.py
<+>UTF-8
===================================================================
--- asteroid/masknn/convolutional.py	(revision dd47fce87c6d72878c6fffc38f14b1ef74ff1c27)
+++ asteroid/masknn/convolutional.py	(date<PHONE_NUMBER>145)
@@ -516,15 +516,29 @@
             **kwargs,
         )

-    def _check_input_dims(self, x):
-        # TODO: We can probably lift the shape requirements once Keras-style "same"
-        # padding for convolutions has landed: https://github.com/pytorch/pytorch/pull/42190
+
+    def fix_input_dims(self, x):
         freq_prod, time_prod = self.encoders_stride_product
-        if (x.shape[1] - 1) % freq_prod or (x.shape[2] - 1) % time_prod:
+        if (x.shape[1] - 1) % freq_prod:
             raise TypeError(
                 f"Input shape must be [batch, freq + 1, time + 1] with freq divisible by "
-                f"{freq_prod} and time divisible by {time_prod}, got {x.shape} instead"
+                f"{freq_prod}, got {x.shape} instead"
             )
+        if (x.shape[2] - 1) % time_prod:
+            div, mod = divmod((x.shape[2] - 1), time_prod)
+            pad_shape = [0, time_prod - mod]
+            x = nn.functional.pad(x, pad_shape, mode="constant")
+        return x
+
+    # def _check_input_dims(self, x):
+    #     # TODO: We can probably lift the shape requirements once Keras-style "same"
+    #     # padding for convolutions has landed: https://github.com/pytorch/pytorch/pull/42190
+    #     freq_prod, time_prod = self.encoders_stride_product
+    #     if (x.shape[1] - 1) % freq_prod or (x.shape[2] - 1) % time_prod:
+    #         raise TypeError(
+    #             f"Input shape must be [batch, freq + 1, time + 1] with freq divisible by "
+    #             f"{freq_prod} and time divisible by {time_prod}, got {x.shape} instead"
+    #         )

 class SuDORMRF(nn.Module):
Index: asteroid/masknn/base.py
<+>UTF-8
===================================================================
--- asteroid/masknn/base.py	(revision dd47fce87c6d72878c6fffc38f14b1ef74ff1c27)
+++ asteroid/masknn/base.py	(date<PHONE_NUMBER>225)
@@ -92,8 +92,9 @@
         )

     def forward(self, x):
-        self._check_input_dims(x)
+        x = self.fix_input_dims(x)
         return super().forward(x.unsqueeze(1))

-    def _check_input_dims(self, x):
+    def fix_input_dims(self, x):
         """Overwrite this in subclasses to implement dimension checks."""
+        return x

which doesn't work because the padding doesn't match the tf_rep when applying the mask but we could also correct for that without problems.

So we could easily add a n_src arg to it and enable separation. Am I missing something?

I think so, yes! Probably also applies to DCUNet.

Maybe padding trimming the time axis and raising a warning would be better?

I like the padding/trimming approach but I dislike warnings when you can raise an error instead. I suggest to make things explicit: By default it raises an error that tells you to enable padding/trimming or fix your input size. PoC, maybe the choice between padding and trimming is over engineered?
# time_prod
strideprod = 16
# x.shape[1]
l = 35
# New self.fix_length parameter to __init__
fix_length = None  # or "pad" or "trim"

l_remainder = (l - 1) % strideprod
if l_remainder:
    if fix_length == "trim":
        l -= l_remainder
        print(f"Trimmed length to {l}")
    elif fix_length == "pad":
        l += strideprod - l_remainder
        print(f"Padded length to {l}")
    else:
        print('Invalid shape, use fix_length="trim" or fix_length="pad"')

I understand.
In the current code, I don't raise a warning, I just pad. Feel free to adapt if how you'd like.

In the current code, I don't raise a warning, I just pad. Feel free to adapt if how you'd like.

OK, working on it now!

In the current code, I don't raise a warning, I just pad. Feel free to adapt if how you'd like.

OK, working on it now!

Made some changes, what do you think? It didn't have time to look at the tests yet, but the tests work for both fix modes, and fail for None.
Btw there is a test failure related to Torch complex number support (already fixed in Torch nightly)

Do you have a link to the issue or the PR?
Maybe we should run tests on torch-nightly as well?
I'll have a look at the changes.

Do you have a link to the issue or the PR?

I think it is https://github.com/pytorch/pytorch/issues/47157

Maybe we should run tests on torch-nightly as well?

That would be a good idea.

Comment the test that's failing, or skip if torch is under 1.8, sounds ok?

Better skip if Torch < 1.8

I let you take care of those changes (skip the test and masknet_kwargs). We'll add tests on nigthly afterwards.

Ok, done! I also changed the args specifications to be all tuples because I accidentally modified some of the lists because they are mutable, which is a terrible thing to debug.

I created a PR against PyTorch for better version checks https://github.com/pytorch/pytorch/pull/48414

I'll review the whole PR again today but I think it's ready to be merged !

I'll review the whole PR again today

Thank you! Much appreciated. Somehow this PR got MUCH bigger than expected :-D

Found missing shape check for DCCRN, added tests for shape checks, and also fixed the PyTorch 1.8 issue (it only breaks if you do foo[:x] where x = len(foo), ie. a noop slice)

So the trimming padding can be done with a single function, trim_x_to_y  and pad_or_trim_x_to_y are not needed it seems.

It's quite dangerous to push to the same branch in the same time (my bad :sweat_smile: ) but we did well ^^

So the trimming padding can be done with a single function

Indeed it seems so! https://github.com/pytorch/pytorch/blob/367426494759ddde0896665ed55c6f9af2870cf0/aten/src/ATen/native/ConstantPadNd.cpp#L25

So we could remove the trim functions and change the pad docstring to say "right-pad or right-trim".
Btw I won't be pushing any code for the next few hours, so feel free to push :P

I'll do that.

I re-reviewed everything and it LGTM
Waiting for your approval to merge it.

Also did a quick review, LGTM, go ahead :)


Companion 3.1.1 Error

Since upgrading from Companion 2.4.2 to Companion 3.1.1, PVP has not responded to commands despite Companion saying that it is connected. Whenever pressing an action, the error message that comes up says "Error Sending Command Error: certificate has expired". When returning to 2.4.2, the issue goes away. Any help would be appreciated!
Update: This only occurs when HTTPS is active. When HTTPS is off, companion buttons function as normal.

Adding further confirmation of this problem. Upgrading from 2.4.0 to 3.1.1 the PVP3 connection shows status as  "Connected", but no response from software when commands are given. Confirmed issue resolves when turning off HTTPS.


TypeError: Invalid non-string/buffer chunk

As soon as I try to run good/good-file I get this:
There was a problem (TypeError: Invalid non-string/buffer chunk) in fileReporter and it has been destroyed.
Nothing gets output to a file at all, ever. I'm not exactly doing anything complex with the config:
options: {
			ops: {
				interval: 1000
			},
			reporters: {
				consoleReporter: [{
					module: "good-console",
					args: [{
						format: ""
					}]
				}],
				fileReporter: [{
					module: "good-file",
					args: [`./logs/${process.env.RELEASE_ENV}.log`]
				}]
			}
		}

Any ideas?

First your consoleReporter isn't writing to anything, so you're going to run out of memory eventually.
Second, your fileReporter is going to be passed an object which is what the error you're seeing is about. You need to use a transform stream before good-file to stringify objects.

Ahhh I'm updating from the 5.x version of Good and 6.x works quite differently.
Hence the confusion. Thanks for clearing that up.

Am seeing the same error. and the log file is created.
`{
    ops : {
      interval : 1000
    },

reporters : {
      fileReporter : [{
        module : 'good-file',
        args: ['./log/serverlog.log']
      }]
    }
  },`

@arb Actually Glue gave me this error, when its initializing plugins for the server.
But as for turning object to string. i will, to that in my code, to implememented during runtime.
So how am i going to do it?


Quoting error cited as indentation error

Trying to parse the document:
blah:
  - "He said "Hi!" to his friends"

gives the following error:
bad indentation of a sequence entry at line 2, column 15:
      - "He said "Hi!" to his friends"
                  ^

This is really a quoting issue (inner quotes need to be escaped).  Is it possible to have the Error that gets thrown reflect that, instead of calling it an indentation error?

No, that's not possible to guess right error without significant parser rewrite.


Renaming proposition

Proposition for renaming classes, attributes and actions, plus users categories:
Naming conventions
Interview manipulation by use case:

Validation: Live validation of interviews data, shown to respondent and interviewers during the interview
Correction: Manual correction of interviews data, done by admin/correctors
Audit: Automated validation of interviews data, done by the server and shown to admin/correctors

Users:

Admin: Users with the admin role can administer the surveys, including monitoring and correcting interviews
Corrector: Users with the correctors role can correct interviews data
Participant: Users with the participant role can fill the surveys (also called Respondent)
Interviewer: Users with the interviewer role can fill out interviews for a participant by phone, in person or remotely, or can assist participants with their interviews

Please discuss if you agree, and if not, propose better (If you don't like a name, propose a better one :-) )

For the admin user, do we want to put some role, because right now admin is too broad of a type ?

For the admin user, do we want to put some role, because right now admin is too broad of a type ?

Yes we do, eventually, none of us should be admin, but we should all have permissions of some kind. We have all the framework for this and we have interviewer/validator roles, but we stopped there as it was enough at the time and didn't get time to work on it...

Sounds good for the renaming. But is that exhaustive enough? What about the case of somebody visualizing the audits (in the summary page) and just setting the interview as valid/invalid, complete/incomplete, questionable? This process does not require correction and we may not want all users who can do that to actually have the permission to correct the interview?
You think of OD surveys, but there are many others where we may just want to manually [in]validate them without giving right to edit.

I think we can still do better renaming and separating users/permissions

Roles/permissions and renaming are 2 separate things anyway.

@tahini Waiting for our review :-)


I keep getting this CORS issue

I just released my website and I keep getting this CORS issue from prismic. I don't know where to turn for this.
Any idea?
Failed to load https://automate-your-brand.prismic.io/app/authenticated/v2: No 'Access-Control-Allow-Origin' header is present on the requested resource. Origin 'https://automateyourbrand.com' is therefore not allowed access. The response had HTTP status code 401. If an opaque response serves your needs, set the request's mode to 'no-cors' to fetch the resource with CORS disabled.

Same for me.
I see no options in the Prismic UI to allow a specific set of domains for the preview control.
One of the previews has the domain for me, I thought that would be enough

Hello @ericraio and @birkir by adding the domain of your website to the settings / Previews. This should be fixed.

Happening to me as well. I'm using netlify and when I set the preview URL to my CNAME domain, it does not work.

I'm also having this issue. I have the url configured in the previews settings.
I'm using the gatsby plugin and my site is hosted on netlify

I encountered this error becuase I added an extra slash at the end of the client url, removing that slash fixed the issue

i have the same
Access to fetch at 'https://xxxxx.prismic.io/graphql/' from origin 'http://localhost:3000' has been blocked by CORS policy: Response to preflight request doesn't pass access control check: No 'Access-Control-Allow-Origin' header is present on the requested resource. If an opaque response serves your needs, set the request's mode to 'no-cors' to fetch the resource with CORS disabled.
i use apollo for nuxt
apollo: {
clientConfigs: {
default: {
httpEndpoint: process.env.PRISMIC
//   browserHttpEndpoint: '/graphql'
}
}
}
PRISMIC=https://xxxx.prismic.io/graphql

i fixed it i just followed the way he makes https://github.com/markperez1904/personal-website


Fixed dropdown menu’s border having incorrect color under the error state

Before, it was showing red border only on hover, and I thought it wasn’t correct:

After, it shows red border at all times:

~Also, I updated my fork to 2.2.10, and I think I might have messed up something, but although it’s showing 3 commits to the branch, it seems that only one file was changed for next—please double check it for me in case I’m missing something.~

git fetch upstream
git checkout next
git reset --hard upstream/next
git checkout dropdown-error-border
git reset --hard upstream/next
git cherry-pick 5436dae820b3e07ec16834cd455347931ecba195

This should do the trick 👍  always prefer to rebase when puling instead of merging as creates ugly merge bubbles.

Hi @sstok, thank you, that did it. 👍 Will be doing it properly from now on.


Fixed #32624 -- Avoided using JavaScript for "Local vars" show/hide in technical 500 debug page.

This helps get rid of some unsafe-inline javascript which helps allow a more strict Content-Security-Policy.
https://code.djangoproject.com/ticket/32624
It uses newish <details> / <summary> elements. In Internet Explorer it should fall back to always being visible.

@collinanderson Thanks :+1:

Thanks!


Add envconfig to project

Trying solve issue #6 :smile:

This looks good @rafaelhenrique 😄 🎉
Just a few suggestions to make this ready for merge:

As @hypnoglow mentioned above, we can provide a required flag to enforce that both the variables are set. 👏
Use a default value for PORT(say 8080) which can be simply set using the default tag.
I would appreciate it very much if you can add a description using the desc tag for each of the variables.
Define the Config struct in common package and import it in server.go.
Process the input as:

if err := envconfig.Process("hercules", &config); err != nil {
		fmt.Fprintln(os.Stderr, err)
		envconfig.Usage("hercules", &config)
		os.Exit(1)
	}

Applied changes @kshitij10496 @hypnoglow ... thanks for tips :wink:

Great work @rafaelhenrique! 🎉
I will wait for @hypnoglow's review before merging it in. 😄

Thanks, @rafaelhenrique, and @hypnoglow for your invaluable contributions! 🎉 🤗

:tada:

UPDATE:
I deployed the changes to Heroku only to later realise that the platform sets the PORT environment variable for each application. What this means is that it won't update the new HERCULES_PORT variable. Thus, I had to force the envconfig to recognize PORT and read it's value into config.Port variable. 😅


I cannot change the model to Curie for some reason

when i change that to text-curie-001, it doesn't work. I am very new to this. Also, where do you store the api key? Thanks!

When using the text-curie-001 model, its request parameters are different from those of the gpt-3.5-turbo model and need to be adapted according to the development documentation. Currently, this project only supports dialogue with the gpt-3.5-turbo model. If you need to select a different model for dialogue, I will add this feature as soon as possible. Additionally, the API key is stored on the client-side browser and is stored using the browser-supported LocalStorage. I hope this answers your questions.


Change pipeline storage account ID to Build ID

What this PR does / why we need it:
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
Special notes for your reviewer:
Release note:

/azp run

/azp run

/azp run

/azp run


List_models_of_datasets

Listing  all models of a specific datasets
function main() {
  // [START automl_list_models]
  /**
   * TODO(developer): Uncomment these variables before running the sample.
   */
  const projectId = 'projectId';
  const location = 'us-central1';
  const datasetId = 'datasetId';
...

Hi, can you provide more details on this issue please?

We have not heard any updates. If you are able to provide more details on your question, please feel free to reopen the issue. Thanks!


increase test coverage on tags methods

The tags code has been around since the beginning and is some of the worst - i spent some time refactoring it and deprecating things and then wrote tests for uncovered lines.

Closing this and merging tags and tools together because of test dependencies


[BUG] "Wrap Around" property messes up the order of items

Describe the bug
I'm witnessing this bug, that when declaring the property wrap-around=true, when looking at the elements it renders, they change the order in which they were declared.
To Reproduce
To reproduce we do the following:

First, we create the component and pass this prop to true.

<template>
  <Carousel :items-to-show="3" :wrap-around="true" :itemsToScroll="1">
    <slide v-for="slide in 4" :key="slide">

{{ slide }}

    </slide>
  </Carousel>
</template>

Then we go to observe the site
Here we can observe the following, the items appear in the following order:  3 - 4 - 1

However when we pass the slide once, the latter is in the following order: 1 - 2 - 3

When the true order that should happen is: 4 - 1 - 2

Expected behavior
The expected behavior would be that this first of all would start with the item 1 - 2 - 3
and with each change only move 1 or the amount indicated in the property
Additional context
As additional information, this was tested on Chrome, using Vue 3 and Nuxt 3.
For the rest, this behavior can also be observed on the demo page of the library:
https://ismail9k.github.io/vue3-carousel/examples.html#wrap-around
If you reload the site you will be able to observe the same thing, it appears with an order and then when you slide, it changes completely.

Hi @chescobarf, Acutely this is the normal behavior for the carousel component.
By enabling wrapping around the carousel reorder the slides so that there will be enough slides when you move to back and forward. Moreover, on this, there's a property called snapAlign which accepts the following values start, end, center, center-odd, and center-even. You can try one those values to find the best alignment that would fit your use case.


LLM-friendliness

Large Language Models (LLMs) are all the rage in the programming world. Some models like Claude 3 Sonnet are reaching capabilities making them very useful for scaffolding code, especially Elm code, which is simple, verbose, and strongly typed.
I remember reading that some of these models can take specially formatted files with concentrated knowledge of a language syntax, and a library API to help it make extra useful. I’m especially hopeful that With the amount of documentation with have and the simplicity of Elm, we can make elm-cardano an extremely LLM-friendly framework.
It would be super fun and useful to explore this.

Just some resource worthy of read: https://llmstxt.org/

A proposal to standardise on using an /llms.txt file to provide information to help LLMs use a website at inference time.


Updated forward only migration resharper file template.

Included file name in the migration class.

Yes, that was an improvement. I added it to the wiki too. Thanks again!

Thank you.


Doesn't work with systems that have deprecated ssh-rsa

This doesn't work on my system with a newer version of OpenSSH that has deprecated the ssh-rsa host key algorithm.
durrw@hahub ~ [1]> ssh <IP_ADDRESS>
Unable to negotiate with <IP_ADDRESS> port 22: no matching host key type found. Their offer: ssh-rsa

This causes the code to raise a ConnectionError
  File "/srv/hass/lib/python3.9/site-packages/pyruckus/__init__.py", line 18, in __init__
    self.connect()
  File "/srv/hass/lib/python3.9/site-packages/pyruckus/__init__.py", line 27, in connect
    ssh.login(self.host, username=self.username, password=self.password, login_timeout=self.login_timeout)
  File "/srv/hass/lib/python3.9/site-packages/pyruckus/RuckusSSH.py", line 34, in login
    raise ConnectionError(CONNECT_ERROR_EOF)
ConnectionError: Could not establish connection to host

The option -oHostKeyAlgorithms=+ssh-rsa needs to be passed to the ssh command

Thank you! I didn't realize these devices only supported ssh-rsa. I just submitted a PR to fix this. Will open a PR on home-assistant/core soon.


vite-plugins-vue-i18n will listens for changes to all .json files

Reporting a bug?
When I use the vite-plugins-vue-i18n plugin, it listens for changes to all the .json files in the project and causes vite to keep reloading
Expected behavior
should only listen to the directory specified by the include parameter
Reproduction
// vite.config.js
import path from 'path'
import { defineConfig } from 'vite'
import vue from '@vitejs/plugin-vue'
import vueI18n from '@intlify/vite-plugin-vue-i18n'

export default defineConfig({
  plugins: [
    vue(), // you need to install `@vitejs/plugin-vue`
    vueI18n({
      // if you want to use Vue I18n Legacy API, you need to set `compositionOnly: false`
      // compositionOnly: false,

      // you need to set i18n resource including paths !
      include: path.resolve(__dirname, './path/to/src/locales/**')
    })
  ]
})

// main.js

// When I import this line of code, the problem arises
import messages from '@intlify/vite-plugin-vue-i18n/messages'

Issue Package
vite-plugin-vue-i18n
System Info
windows 10

Screenshot

Additional context
No response
Validations

[x] Read the Contributing Guidelines.
[X] Read the README
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion.

it also doesn't listen for languages in node_modules.
    VueI18n({
      include: [
        resolve(__dirname, '../../node_modules/@oku/i18n/src/locales/**'),
      ],
    }),

how to listen ?

unplugin-vue-i18n  the same problem occurs.

@kazupon Any updates on this one?

If you would use to possible, I recommend to use unplugin-vue-i18n.
vite-plugin-vue-i18n only supports up to Vite 3, support security fix only.
unplugin-vue-i18n is going to official v1 release, when nuxt i18n module will have been released near the future.
(unplugin-vue-i18n is currently under development with me along with use cases for the nuxt i18n module)

@kazupon I do use unplugin-vue-i18n.
And the problem is that it also listens to changes of all .json files in the project's folder.
Even when include option contains only locales folder.
How can it be fixed?

@kazupon I do use unplugin-vue-i18n. And the problem is that it also listens to changes of all .json files in the project's folder. Even when include option contains only locales folder. How can it be fixed?

@kazupon I have the same problem，until now there is no solution, the same is used unplugin-vue-i18n .


Question: Google Analytics Plugin doesn't track subsequent page views?

I have installed the Google Analytics plugin as described in the docs.
When I open a page, a request is sent to GA. But when I follow an internal link on that page, no request is being sent to GA anymore (checked with the network activity monitor in Google Chrome dev tools). What am I doing wrong?
Thx!
Chris

Could this be part of it? https://github.com/MatteoGabriele/vue-analytics/blob/master/docs/untracked-hits.md

I've used the 'vue-analytics' package now instead of the '@gridsome/plugin-google-analytics' package. From the layout component I then called the send pageview function on mounted and it now works.


Collect the dashboard hosts external FQDN

This should be used by Dex to redirect back to Velum, allowing for scenarios
where the admin node's private IP is not accessible to the outside world - e.g.
as is the case on OpenStack and possibly other cloud environments.
Part of bsc#1062291
Depends-On: https://github.com/kubic-project/salt/pull/251
Depends-On: https://github.com/kubic-project/automation/pull/164

Also, the oidc_controller.rb will need some tweaking. Basically on the client method adapting the redirect_uri using this pillar value on the url_for helper, adding the host key.


adds fov_watcher template

Purpose
Adds watcher as described in issue #28
Implementation
Adds fov_watcher.py.  Watcher is built on top of the watchdog package.
Remaining issues
Notebook integration + tests.  Also, the callback function signatures might need to be changed to bin file paths, instead of the current signature requiring the xarray format.  The former is preferable, as it allows callback functions to use individual/internal panels.

looks like tests are failing because of a breaking change from an xarray minor version update... submitted a fix to mibi-bin-tools. should be ready for review regardless; only thing missing is testing for the file watching functionality itself (as opposed to the callbacks).

Do you think it would be valuable to put a some more comments / or a paragraph at the top to discuss the purpose and usage of the watcher_example notebook. Are the individuals intended to use the watcher script already going to know why they would need to use it?

@srivarra Good call.  I'm planning on adding some more verbose usage instructions once everything's functional, i.e probably will add that for the next review cycle.

The qc_metrics function expects the run folder to already exist. However, the watcher function doesn't create it before starting fov-level callbacks, which leads to the watcher crashing with a bad filepath error

The print statements get printed for each callback, rather than for each fov

this is ready to merge in, just needs approval @ngreenwald . callback 'chaining' design doc should be up in a bit, and that implementation will go in a new branch off of main


feature/440-setup-shared-creds-and-load-dev-environment-values-from-keybase

 Setup shared creds and load dev environment values from Keybase


Colab: extract mesh | installing torchsearchsorted fails

This is the error:
ERROR: Command errored out with exit status 1: /usr/bin/python3 -u -c 'import sys, setuptools, tokenize; sys.argv[0] = '"'"'/tmp/pip-req-build-688c9eye/setup.py'"'"'; file='"'"'/tmp/pip-req-build-688c9eye/setup.py'"'"';f=getattr(tokenize, '"'"'open'"'"', open)(file);code=f.read().replace('"'"'\r\n'"'"', '"'"'\n'"'"');f.close();exec(compile(code, file, '"'"'exec'"'"'))' install --record /tmp/pip-record-vcq4akmd/install-record.txt --single-version-externally-managed --compile Check the logs for full command output.

This was due to wrong pytorch version in requirements.txt. It should be fixed now.

Hi, I just wanna ask so if I use the lastest torch version (1.5) then torchsearchsorted doesnt work ?

if you use 1.5 I think you need cuda10.2 installed. I tried torch1.5 with cuda10.1 and it can't compile successfully, but torch1.5(.0) with cuda10.2 works for me.

Yeah seems like it is time for me to upgrade to cuda 10.2 =)))

I upgrade to torch1.6, and the 3rd party torchsearchsorted package is not required anymore since it's already included in pytorch natively. Please checkout the branch torch1.6 with all other packages also upgraded up-to-date.


bug: uint64 values are truncated on insert

Hi 👋 Thank you for this lovely module, it works great 🚀
I think I discovered a bug when trying to insert unit64 values. It seems those are casted to int64 before encoded, so data is loss.
See:
https://github.com/sijms/go-ora/blob/41ca433ee475e64869448c27e6e20ad038613ae4/v2/parameter_encode.go#L308
https://github.com/sijms/go-ora/blob/41ca433ee475e64869448c27e6e20ad038613ae4/v2/parameter_encode.go#L337
Can you confirm my findings?

sorry for late replay
I am fighting to add UDT array as an output parameter and finally I do
I fix the issue and add encodeUint64 for uint64 type

Thanks @sijms! I tried testing the new release and getting:
ORA-01461: can bind a LONG value only for insert into a LONG column

For a query that was working before (didn't manage to test the unit64 due to it). Do you know what might be the problem? I'm not sure any LONG types in my query.
I can try and provide a reproducible query if you need one.

would you send tested code to regenerate the error
because I re-code all functions related to parameter encoding and decoding

would you send tested code to regenerate the error
because I re-code all functions related to parameter encoding and decoding

Sure, let me prepare the smallest possible query to reproduce this

Hi @sijms, the follow test should reproduce the error. You'll need to update the connection string
func getTestDB(ctx context.Context) (*sql.DB, error) {
	db, err := sql.Open("oracle", "oracle://user:pass@localhost:/db")
	if err != nil {
		return nil, err
	}
	conn, err := db.Conn(ctx)
	if err != nil {
		return nil, err
	}
	defer conn.Close()
	return db, err
}

func TestLongError(t *testing.T) {
	ctx := context.Background()
	db, err := getTestDB(ctx)
	if err != nil {
		t.Fatal(err)
	}
	defer db.Close()
	_, _ = db.ExecContext(ctx, `DROP TABLE "long_error"`)
	createTableQuery := `CREATE TABLE "long_error" (
		"_cq_source_name" clob,
		"_cq_sync_time" timestamp,
		"_cq_id" raw(16),
		"_cq_parent_id" raw(16),
		"binary" blob,
		CONSTRAINT "long_error" PRIMARY KEY("_cq_id")
	  )`
	if _, err := db.ExecContext(ctx, createTableQuery); err != nil {
		t.Fatal(err)
	}

	insertQuery := `INSERT INTO "test_long_error" ("_cq_source_name", "_cq_sync_time", "_cq_id", "_cq_parent_id", "binary") VALUES (:1, :2, :3, :4, :5)`
	name := ""
	now := time.Now()
	id := []byte{251, 235, 184, 140, 247, 88, 75, 71, 142, 177, 147, 116, 124, 130, 216, 154}
	id2 := []byte{100, 71, 16, 9, 252, 62, 67, 67, 145, 79, 194, 78, 177, 138, 58, 135}
	blob := []byte{239, 78, 61, 82}
	if _, err := db.ExecContext(ctx, insertQuery, name, now, id, id2, blob); err != nil {
		t.Fatal(err)
	}
}

Please let me know if that helps

sorry for late again
I fix the issue it is related to parameter.MaxLen this value should not be 0 so when it is 0 (means null or empty value) I convert it to 1

Hi @sijms, sorry for the delay the latest version (v2.7.9) works great, so I'll go ahead and close this issue


POC: histogram widget

PSA: I haven't yet cleaned up the code & typings, but that can be done after this demo's features have even approved
Try the story: https://deploy-preview-4358--instantsearchjs.netlify.com/stories/?path=/story/refinements-rangeslider--with-histogram

Will revisit this if the engine ever has bandwidth to ship it :)


(BKR-349) AwsSdk Hypervisor Specs (review only - not ready for merge)

Hey puppet people, this is still blocked on BKR-348 and (probably) BKR-318 that kevpl submitted yesterday.
Putting this up prematurely so you guys can review when it's convenient.  It's grown to a large enough project I want to make sure I halt early if you guys hate it.
I'll continue pushing as pieces fall in place but it is currently only missing initial tests for #create_instance, #launch_nodes_on_some_subnet, #launch_all_nodes, and #configure_hosts

Can one of the admins verify this patch?

Forgot I had another question.  I've removed #kill_zombie_volumes and #kill_zombies since they don't appear to be called anywhere in the code base for beaker.  Are these really orphaned methods or is it possible to actually call methods directly out of the hypervisors in your beaker code and I should revert?

Yes, please revert.  We use those methods for monitoring our ec2 account for orphaned instances.

@anodelman Reverted.  #log_instances, #kill_zombie_volumes, and #kill_zombies are all back in place.
Would it be ok with you to merge this PR once  BKR-318 & BKR-367 are merge?
If that's cool I can rebase those two PRs in and then make the changes for the final few refactors&specs in a new PR.  I imagine I'm looking at a messy rebase with those two so I figure it will be easier to do the remaining refactors in a more isolated PR.

Needs a rebase.

@anodelman Heyo.  This has been rebased and polished up.  If you're comfortable merging the tests we have here I'll do the refactors on the launch nodes and create_instance methods separately.

ok to test

Refer to this link for build results (access rights to CI server needed):
http://jenkins-beaker.delivery.puppetlabs.net//job/qe_beaker_btc-intn/1132/
Test FAILed.

retest this please

Refer to this link for build results (access rights to CI server needed):
http://jenkins-beaker.delivery.puppetlabs.net//job/qe_beaker_btc-intn/1135/
Test PASSed.

Just doing some local sanity checking.

:+1:

Great work on this!