Disk widget not working

Important notices
Before you add a new report, we ask you kindly to acknowledge the following:

[x] I have read the contributing guide lines at https://github.com/opnsense/core/blob/master/CONTRIBUTING.md
[x] I am convinced that my issue is new after having checked both open and closed issues at https://github.com/opnsense/core/issues?q=is%3Aissue

Describe the bug
Installed 25.1 on new nvme.
The disk widget on dashboard isn't working. It shows 'Free: 100%'.
To Reproduce
Steps to reproduce the behavior:

Go to 'dashboard'
Hover on 'disk'
See result

Expected behavior
Working disk widget
Screenshots

Additional context

Environment
Software version used and hardware type if relevant, e.g.:
OPNsense 25.1.b_20-amd64
FreeBSD 14.2-RELEASE
OpenSSL 3.0.15
Lenovo M720Q
Intel(R) Core(TM) i5-8400T CPU @ 1.70GHz

your df shows the same as the graphs, doesn't it? (100% free, roughly 0% used)

I get it now.
Total size: 453G
Used: 1.2G
To get 1% used, I need 4.5G used status.
My bad!


Japanese characters don't appear in msconvert output filenames

Steps to reproduce:
Run msconvert with the parameter --outfile "pete3繧.mzML"
Expected:
An output file is produced named "pete3繧.mzML"
Observed:
An output file is produced named "pete3___.mzML"
Console Output:
PS> .\msconvert.exe --outdir C:\Users\Waters\Desktop\ C:\Users\Waters\Desktop\pete.mzML --outfile "pete3繧.mzML"
format: mzML
    m/z: Compression-None, 64-bit
    intensity: Compression-None, 32-bit
    rt: Compression-None, 64-bit
ByteOrder_LittleEndian
 indexed="true"
outputPath: C:\Users\Waters\Desktop\
extension: .mzML
contactFilename:
runIndexSet:

spectrum list filters:

chromatogram list filters:

filenames:
  C:\Users\Waters\Desktop\pete.mzML

processing file: C:\Users\Waters\Desktop\pete.mzML
calculating source file checksums

writing output file: C:\Users\Waters\Desktop\pete3___.mzML

Thanks for the report. Easy to repro.


FLASK RUN [ERROR: Could not import "app".]

(env) C:\Users\PLAYER1\Documents\flask_app>set FLASK_APP=app.py
(env) C:\Users\PLAYER1\Documents\flask_app>flask run

Serving Flask app "app.py"
Environment: production
WARNING: This is a development server. Do not use it in a production deployment.
Use a production WSGI server instead.
Debug mode: off
Usage: flask run [OPTIONS]

Error: Could not import "app".
(env) C:\Users\PLAYER1\Documents\flask_app>

What is app.py? That is not part of this tutorial. Shouldn't FLASK_APP by set to microblog.py?

if this tutorial serves as a template for your own project, please check the filename in your project-folder. sometimes it is called "runserver.py" for example...so you have to correct the environment variable FLASK_APP to this filename

I have the same error. I'am currently going throught VS Code Flask  tutorial and reseaving this error massege after starting debug mode, but server is working properly.

@alextoritsin Your app.py should be outside your (venv) folder. "Could not import" error is related to the where your app.py resides.

I have the same error. I'am currently going throught VS Code Flask tutorial and reseaving this error message after starting debug mode, but server is working properly.
I think Problem is with the directory
Suppose your directory is like this /A/B/app.py
You need to cd into B then enter the directory where your app.py file is located. then export  then run
pip install flask
export FLASK_APP=app.py(for linux)
flask run

Yes @chanderbhanu096 In may case I was running flask outside the folder where the app.py was located. Once I cd to that folder and ran flask from there the WSGA worked

I had the same issue, but after I move app.py out of all the folder it seems to work. I encounter a new problem that the page shows 404 not found right now.

Remember to name file "init.py" and not "init.py"

I have the same error. I'am currently going throught VS Code Flask tutorial and reseaving this error message after starting debug mode, but server is working properly.

in Windows you can try this solution https://youtu.be/1gLTE6Zvb-Q

Could import app error while flask run in Windows you can try this Solution https://youtu.be/1gLTE6Zvb-Q


[TW#17222] Low Power Amplifier

Hello,
I will like to know when will be available the low noise amplifier: example code and schematics. Thanks in advance

we're working on this, but it seems to be long before release.
are you sure your signal is in the range of +/- 2mV?

It is lower than that level. it should be in uVolts.

okay, in that case LNA would help.


Plans for future code adoption?

Great work on this project.
Are there any plans for incorporating other international codes? US, Canada, Australia?

For the moment we do not have any concrete plans for incorporating other codes, but we are happy to have contributions related to this 😃 Is there a specific code you are looking for?

ACI 318 is probably the most useful in terms of the number of people using it.
I will consider this moving forward, if I can add something, I will submit forreview. 👍


Testing Story: Forever Scroll (Recursive Islands) [islands load islands]

I would like to load some islands dynamically. The Roadmap #563 mentions a couple things long term.

Introduce a first class type safe data fetching system for islands
Lazy loaded islands

These directly affect how something like a forever scroll case would be implemented. I think forever scroll is a great test case to study to see how this API is going to look. Keeping it simple, a post with a Date island is loaded a number of times (say 7) into a div. When the user scrolls I want to load a new set of posts. I could do this with a mega island, but is there a way to keep it uhh... "more fresh"?
#565 starts to allow lazy loading. But how would I go about having an island load more islands?

A Scenario:
I have an existing post type and I load 7 per page. Each Post (a server side component) has a Reaction and LocalDateTime Island. There are pagination links rendered if necessary at the bottom of each page. I want to turn the "Load More" link into an Island. It should load right away and use JS to detect when it scrolls into view.
Once the Island is in view, it should load the 7 more posts each with 2 islands. Then it can replace itself with the new posts in the DOM.
Obviously the Post component now has to be rendered client side. That's new, my brain says move it into /components but I don't see where to go from there. And I'm not really sure that's anything but a convention.
I also need to initialize the Islands in the server side component. That's where it really starts to get beyond my grasp.
I think there are a lot more use cases for this kind of API, but I think this one stands out. It's a pattern that worked pretty well with JQuery and the DOM. Is there a way we can keep Fresh fresh while enabling this recursive loading of islands?

TLDR; too long dear reddit

[ ] Introduce a first class type safe data fetching system for islands

Please let me load Components with Islands in them using this system? That would be 👍

With the back in the day example, I could grab the markup by loading the next page, but the bit of hydrating the interactions would be wired up manually. So maybe what I am grasping for here is an Island hydration API that could work from markup?
IDK if that is a good idea.
A better architecture might be the cure here, or not.
Just as an example. the pagination could be a big island in the first place. Loading the first 7 posts via whatever ajax methods the forever scroll paginator is going to use. That's probably closer to what the Island Hydration architecture demands, but I still want to instantiate some islands dynamically on the paged content.

I think you want to use an IntersectionObserver in an island. Once the observed node is visible you can load more data and add it to the list of the posts. So on the server side you would initally load constant number of posts, then on the client your island would load more untill the node is invisible or the number of returtned posts is less than limit.
I use an intersection observer to lazy-hydrate islands, this is where the lazy loaded islands would be useful to save some bytes.
Examples:  LazyHydrate Component |  Lazy Island Test
I don't know what they meant by "a first class type safe data fetching system for islands" though :)

Ok, so here is actual code that works like I said. It loads the elements into the DOM (using InstersectionObserver) but falls short at the hydration step. I don't use Fresh at all to get my elements into the DOM, just some nilla JS.
https://github.com/Hyprtxt/linceo.club/blob/foreverscroll/islands/ForeverScrollLoader.jsx
I still need to mix in a signal that tracks the current page and total pages. Then a bit of render logic for the pagination links. I can use the signal to make the render island do its thing the correct number of times. If only I could get the islands hydrated. In my head I think the code we need is all already there, I just have to call it somehow.

I added the signals to make it all work and realized it's good enough to post. The reactions are broken on the loaded posts, but most of my users (grandparents) only browse, and it works for that just great.
https://linceo.club/gram
If you scroll all the way, I have not hidden the pagination link, and you can see that my query string pagination is still intact (I mean it powers the forever scroll, but still) and it's spider/accessibility (I hope) friendly too.


1137 expose additional information on course radio buttons

Context
Candidates need to be able to:

See more information about the course they are selecting
Easily distinguish between similar course ie. one is part time and one is full time

Changes proposed in this pull request
Before

After

Guidance to review
None really.
Link to Trello card
https://trello.com/c/88LNlgb8/
Things to check

[x] This code doesn't rely on migrations in the same Pull Request
[x] If this code includes a migration adding or changing columns, it also backfills existing records for consistency
[x] API release notes have been updated if necessary
[x] New environment variables have been added to the Azure config

As discussed on Slack, the styling of the radio options will change so that the description is on a new line below the title and in a light grey colour.


Support hidden form fields

Thanks @jbeuckm! This will be a nice addition to the library.
In addition to updating the other field types, would you please update one of the integration tests to add a hidden field? That way this feature doesn't regress down the line.
Also, I'm curious what your use case for a hidden field is, if you don't mind sharing? I've never had occasion to use them myself. Just wondering what purpose they would serve.

Yes - I was just being lazy. I added hidden to the other fields and tested the flag in the TextField spec. See what you think.
I wanted a non-obvious way to mark a PDF so that I could check that it was (probably) the same original file later. I'm interested in hearing other ways to do the same. If there is a clear path to doing this with an encrypted certificate, I might help work on it.

Thanks @jbeuckm! This will be a nice addition to the library.
In addition to updating the other field types, would you please update one of the integration tests to add a hidden field? That way this feature doesn't regress down the line.
Also, I'm curious what your use case for a hidden field is, if you don't mind sharing? I've never had occasion to use them myself. Just wondering what purpose they would serve.

Yes - I was just being lazy. I added hidden to the other fields and tested the flag in the TextField spec. See what you think.
I wanted a non-obvious way to mark a PDF so that I could check that it was (probably) the same original file later. I'm interested in hearing other ways to do the same. If there is a clear path to doing this with an encrypted certificate, I might help work on it.


Marsh dweller fails to parse

Marsh dweller fails to parse due to Marsh-dweller not matching Marsh dweller

It now parses but does not catch the description


Different results for javascript.Promise and C-Sharp-Promise

Different results for javascript.Promise and C-Sharp-Promise.
Is there any problem?
javascript

C-Sharp-Promise

This is due to the way promises are always chained in the next "tick" (setTimeout) in JS, but RSG.Promise evaluates immediately without waiting a "tick" (as there usually is no such thing in C# as the event loop. But AFAIR you can implement the event loop yourself and tell RSG.Promise to utilize it).

@zsoi
Thank you for answering!


[Feature request] Implement skeleton, or a loading message, when opening the app/an album

Presently, opening the app or an album shows a message that there's nothing to show, before then loading the view:
https://github.com/IacobIonut01/Gallery/assets/57101103/143c3f66-43a7-4a32-a97f-016f6b0df637
It would be better if instead we showed a skeleton or some type of loading message, to make it look more professional.
App version: 1.1.0
Device: Galaxy A52s 5G
Android version: 13

Google Photos always have something to show! I guess they are pre-loading timeline and albums in advance
But the placeholder or loading message will totally kill the vibe of state-of-art gallery

Google Photos always have something to show!

That's not actually true. Maybe you have a fast device and so it appears instantaneous to you, but if you look at a screen recording slowed down, you can see the skeleton:
https://github.com/IacobIonut01/Gallery/assets/57101103/a0daa88e-86bc-4d35-94ca-c079ecd23035
There is probably performance still left on the table, and we should try to attain that definitely, but a skeleton can help with the perceived speed of the app, especially as you go lower down the device price spectrum. And even a (probably) optimised app like Google Photos uses it for that extra bit of perceived responsiveness, especially on lower-end devices.


New functions and tweaks

Added Get-Theme function to list available themes
Added Show-ThemeSymbols function to list defined symbols
Allow themes to be re-loaded by Set-Theme
Made Show-ThemeColors implementation more generic
ArgumentCompleter now re-uses Get-Theme function

Update: I'll have access to my Windows setup on Monday. If everything works as advertised, I'll deploy immediately. And I'll make sure to create a VM setup.

I've got a windows setup, does that count? :)

I've got a windows setup, does that count? :)

Well, as much as I'd love to put all my trust in strangers on the internet, I really want to check everything at least once myself for now :-)


Quick query

Just an idea I had... useful debugging/learning tool added to the Active Scans page.  Easy way to run a quick query on any table in Accumulo.  It uses the Monitor's credentials with EMPTY auths to do a restful query on Accumulo.. anything with Auths won't be counted.  Passes whatever the user puts in the text box as the Range(row) for a batch scan.  The result will be stored in a JSON object in the DOM.

I don't think it's a good idea to incorporate scanning into the web interface of the monitor.  The monitor is also supposed to be a limited status monitoring service, not a triage service. Adding scan features to the monitor would expand its scope significantly, and I think it's better to keep its scope limited.
Prior to 2.0, we did have the ability to scan via the monitor, because we had an embedded shell inside the monitor when it was running with SSL/TLS. We stripped it out, to simplify the service. It was fraught with security concerns anyway, so it's probably best that it was stripped out.

I don't think it's a good idea to incorporate scanning into the web interface of the monitor.

What concerns authentication?

The monitor is also supposed to be a limited status monitoring service, not a triage service.

I don't see a difference. I think Admins would like the ability to scan the metadata table.

I don't think it's a good idea to incorporate scanning into the web interface of the monitor.

What concerns authentication?

The monitor runs as the system user. It will be able to scan with system-level permissions. If we were to change this, we would have to create something like we had with the embedded shell where we track sessions and allow users to authenticate as themselves. User authentication and session management dramatically increases the scope and complexity of the monitor.
Beyond the authentication bits, one of our driving principles for the monitor is that it never exposes user data. We even hash tablet information, so we don't expose end rows and other potentially sensitive user data that leaks into the metadata table, and potentially out to the monitor, which is designed to run on the perimeter of the cluster. Having scan features baked in to the monitor that brings this data back to a web user completely undermines this security principle.

The monitor is also supposed to be a limited status monitoring service, not a triage service.

I don't see a difference. I think Admins would like the ability to scan the metadata table.

The difference is night and day. The shell provides a mechanism for simple query, troubleshooting, and management, in an authenticated, local process. The monitor runs as a service to support multiple simultaneous users (it is a web service, after all), with basic "what is Accumulo doing right now" kind of information. There's even the opinion (although I haven't heard it expressed recently) by some of us that the monitor is already doing too much, and it's possible it is teaching users, to rely on it for monitoring their clusters instead of deploying a proper monitoring infrastructure.
Admins want many things, but we can't always provide everything they want in the specific way they want it. We need to draw the line somewhere to keep Accumulo from dying of scope creep and complexity.

The shell provides a mechanism for simple query, troubleshooting, and management, in an authenticated, local process.

I was speaking with an Admin and they only use root on the shell, defeating the whole purpose of multiple users.  I guess its just easier for their setup.

I think we are in agreement that baking this into the Monitor isn't the best place... leaving open for now in case others have ideas.

I created a separate repo on my personal Github for this idea.  I have been experimenting some with Spring Boot.  If anyone is interested its here: https://github.com/milleruntime/rapido


Docstrings are not preserved in the output schema

Description
Docstrings in the tailcall configuration graphql file are not present in the generated schema, and so do not show up in tools like graphiql
Steps to reproduce
Add a docstring in jsonplaceholder.graphql, for example
"""
Get all posts
"""
posts: [Post] @http(path: "/posts")

Run tc start jsonplaceholder.graphql and note that there are no dosctrings in the output schema.
Expected behavior
Docstrings from the input graphql file are preserved in the final (output) schema.
Actual behavior
No docstrings are present in the output schema

/attempt #339

Which url do I see to check the output schema?

@rishi-raj-jain Updated the issue with a sample config. You should hit /graphiql to get the output schema.


mpw-5c are very noisy for output logs

Prompt
running make user_project_wrapper generates 100k lines of logs
Proposal
show only errors and top level information. Each tool can log into the log directories as before.

I believe that's not an openlane issue. It's rather openlane usage issue. Openlane has -disable_output in which it only prints the contents of openlane.log. Also for mpw-5c, there is a bug where make user_project_wrapper or make user_project_example or any make <block> will actually run all the blocks, fixed here https://github.com/efabless/caravel_user_project/pull/89/. That's one more reason why the logs you are seeing are huge

Still, the default should be the saner option. -disable_output should be inverted into -verbose.

Issue fixed in main- as this will continue to affect mpw-5c, the issue will remain open for documentation purposes.

... -disable_output (now the default behavior)

While it's nice to have less spammy output, it does make it harder to get a sense of how much progress the tooling doing, esp. for long running operation like [INFO]: Running Detailed Routing....
Should we open another issue to discuss further improvement or is that ok to keep this issue open?
(I also just happened to stumble upon https://clig.dev by @cli-guidelines w/ some good recommendations about output management https://clig.dev/#output and progress indicator https://clig.dev/#robustness-guideline)

You can just add -verbose, then.
I do agree about an exception for routing though. Maybe the OR team can add a more tty-friendly output format..? @maliberty Thoughts?

@donn do you have some hints on how the -verbose flag works (does it take a level? or is that just a all of nothing thing?); it doesn't seems to affect the output for me.

@donn my bad it's already documented in https://github.com/The-OpenROAD-Project/OpenLane/blob/master/docs/source/openlane_commands.md#general-commands, I just had to set it to something > 1 :)


[GHSA-5mcr-gq6c-3hq2] Local Information Disclosure Vulnerability in Netty on Unix-Like systems

Updates

Affected products

Comments
It seems all of the 3.x netty artifacts were also published to the io.netty namespace, so I need to go back and include those for all of my previous updates

Hi there @normanmaurer! A community member has suggested an improvement to your security advisory. If approved, this change will affect the global advisory listed at github.com/advisories. It will not affect the version listed in your project repository.
This change will be reviewed by our highly-trained Security Curation Team. If you have thoughts or feedback, please share them in a comment here! If this PR has already been closed, you can start a new community contribution for this advisory

Hi there @normanmaurer! A community member has suggested an improvement to your security advisory. If approved, this change will affect the global advisory listed at github.com/advisories. It will not affect the version listed in your project repository.
This change will be reviewed by our highly-trained Security Curation Team. If you have thoughts or feedback, please share them in a comment here! If this PR has already been closed, you can start a new community contribution for this advisory


add lowercase tokenizer docs

Description
add lowercase tokenizer docs
Issues Resolved
Part of https://github.com/opensearch-project/documentation-website/issues/1483 addressed in this PR.
Version
all
Checklist

[x] By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license and subject to the Developers Certificate of Origin.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@udabhas @varun-lodaya Please provide tech review approval to move this PR forward in the documentation process. Please review this week or provide a peer who can review it. Thank you.


[BEAM-6325] Cast cross compile output from []byte to string for printing

Fix an oversight in the cross compile mechanism for the universal runner.

Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

It will help us expedite review of your Pull Request if you tag someone (e.g. @username) to look at it.


[Runtime][String] - Split implementation does not exist

Although the libsrc indicates following, actual implementation doesn't exist
native function split(string s, string sep) (string[]);

Fixed with https://github.com/ballerinalang/ballerina/pull/2294


updated vendors path.

put the vendors in the frontend.

@Shivam-apna please update your conflinct.

Need to test more.


decorators for ConstituencyGroup, SeatIncumbency, HouseSeat, ContactPoint

Coverage remained the same at 100.0% when pulling 8738c614ddaf7527fc8a51e443b3fef37a909b33 on giusepped/decorators into 15c697bd3c70146fdc3a4c3da4e1ee989858c3a0 on master.

Coverage remained the same at 100.0% when pulling 3f72a99adabede757ab4416d7e63facf4ac3f432 on giusepped/decorators into 15c697bd3c70146fdc3a4c3da4e1ee989858c3a0 on master.

Coverage remained the same at 100.0% when pulling 3f72a99adabede757ab4416d7e63facf4ac3f432 on giusepped/decorators into 15c697bd3c70146fdc3a4c3da4e1ee989858c3a0 on master.


Update README.md (Fix broken link)

Description
Changed the first image displayed, the first one was broken.
Notes for Reviewers
Signed commits

[x] Yes, I signed my commits.

Thank you, @Abrams11IsBack 👍 Here's one-time instructions to sign-off on your commit - https://github.com/layer5io/meshery-operator/pull/149/checks?check_run_id=2188362358
More info in the Welcome message above.

@Abrams11IsBack will you sign your commit?


add answers to exercise 1-5 chapter 7

Hi @axel22,
Pls see this answers

Thanks, looking good overall!
I only left some minor comments for exercise 1.

@axel22
Thanks for your review, I'll try to fix

Hi @axel22,
Pls, see my changes
I use asInstanceOf[P] method to convert Q => P.
Pls suggest me, does a better way exist ?

Unfortunately, not that I know of. Ideally, =:= class should have a reverse method that swaps the type parameters. Alternatively, you could require implicit =:= in both directions, but I guess this is good as it is!
LGTM

Thanks for your contribution!

Thanks


GH-8993 added flow types to files js

Summary
added flow types to files.js
Ticket Link
https://github.com/mattermost/mattermost-server/issues/8993

Thank you! :):):):):)


Pasteing of space delimited list

I am able to copy and paste a comma delimited list. I have a requirement to allow pasting of a list on spaces and carriage returns. Is there currently a way to do that?

This is closed, finally figured out how to do it.


Prefix channels instead of hardcoding them?

First of all, thanks for this repo, it helped me wrap my head around preload.js scripts and such.
Now, my question. In preload-simple.js, it is written that you should always use hardcoded channels. I was wondering, are there any security issues to prefix channels instead of hardcoding (or filtering) them? For example:
function sendIPCMessage (channel, arg) {
  return ipcRenderer.sendSync(`secured-${channel}`, args);
}

With that, a malicious message cannot be sent (I suppose), since window.sendMessage('ELECTRON_BROWSER_GET_BUILTIN', 'app') will send the message on the "secured-ELECTRON_BROWSER_GET_BUILTIN channel instead.
Am I missing something? Because if that's not the case, then it would be way more flexible not to use hardcoded channels.
Thanks!

That would be equally secure as far as I know.
Isolated worlds was recently introduced so preload scripts have lost some of their interesting capabilities. I suggest looking at protocol handlers for a more interesting alternative.
I encountered a bug with them though,
https://github.com/kewde/electron-sandbox-boilerplate/tree/bug-stream-protocol/sandbox-stream-protocol


Release to packagist

:cry:

/cc @tmatsuo @bshaffer @JustinBeckwith @jgeewax
Would anyone be able to help get me get this set up?

Once we get this sorted out -- can we make sure that deploying new packages is only done by Travis ?

I'm pto today but I tried...
On Tue, Mar 29, 2016 at 9:52 AM JJ Geewax<EMAIL_ADDRESS>wrote:

Once we get this sorted out -- can we make sure that deploying new
packages is only done by Travis ?
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/GoogleCloudPlatform/gcloud-php/issues/26#issuecomment-202902539

Once we get this sorted out -- can we make sure that deploying new packages is only done by Travis ?

Yes, I can get a hook set up that should take care of that for us - https://packagist.org/about#how-to-update-packages

done

Awesome, thanks @bshaffer


Warning about permissible value lengths even though their length is less than 255 characters

From make all
For example:

WARNING:linkml.generators.excelgen:'FoodCookingMethod' has permissible values with total length > 255 characters. Dropdowns may not work properly in /home/mark/gitrepos/mifc/project/excel/mifc.xlsx

We should characterize this some more and possibly raise an issue in linkml/linkml

Come up sometimes for me as well when I run make all-all

Hah, the total character size of FoodCookingMethod's permissible value names is 330. So the warning is valid and the underlying problem is a constraint from Excel. I think we just have to live with this.

OK cool thanks @turbomam if it's just an excel issue then ya we'll have to live with it. Hopefully the enum values will work in excel. On a related note I'm thinking of for example having all iso3 country codes in a new enum, I hope something like that also works in the excel version. TBD. I guess we close for now if this isn't something we can fix. Thanks again Mark.


Add missing fragment specifier to a clap_app! rule.

Introduced in #238, this bug affects both clap 1.* and 2.* and was found by rust-lang/rust#39419.
I'd suggest releasing not only 2.20.5, but also 1.5.6, to cover downstream crates still on 1.5.5.

cc @james-darkfox @jseyfried

Oh and if anyone knows how to trigger this path to test it, that'd be helpful.

Coverage remained the same at 90.659% when pulling f28b2353b24229ac04708b6828a4bafb57d16182 on eddyb:patch-1 into cb81ce59e55f97b72fd09dcc93530f9bbded29d8 on kbknapp:master.

Thanks for catching this and letting me know! I'll push out the two new versions.
@homu r+

:pushpin: Commit f28b235 has been approved by kbknapp

:zap: Test exempted - status

v2.20.5 is out and v1.5.6 is in PR right now


Fix Virtual DHCP Server: Correct IP reassignment

Changes proposed in this pull request:

DHCP_NAK does not work correctlly

Changes make it work correctlly

Description of changes:
①If IP address in user's note is changed, then reply to DHCP REQUEST with DHCP NAK.
This makes DHCP sequence begin with DHCP DISCOVER.
②If there is any entry with the same MAC address, then remove it.
Unused lease time record is deleted from IP lease table.
③Reassigning static IP address is partially modified.
④Thease changes enable to switch between static and dynamic IP address assignment.
Users do not need to restart the VPN server.

Example below:
Already set to IPv4:<IP_ADDRESS> in user's note.
First, set to IPv4:<IP_ADDRESS> in user's note. Indicated with yellow line.
Second, delete IPv4:<IP_ADDRESS> from user's note. Indicated with yellow line.
Third, set to IPv4:<IP_ADDRESS> in user's note. Indicated with yellow line.
User can switch the way of IP address assignment at any time.

Attach A:

this looks like a followup of https://github.com/SoftEtherVPN/SoftEtherVPN/issues/1947
@PizzaProgram , can you please test it ?

@hiura2023 thank you very much for taking care of OpenSSL-3 issue.
I'm going to merge this PR (I would avoid nitpicking that it should be better to go into 2 separate PRs, anyway, there'll be separate commits, I am fine with that)


filtering by labels example + tests

in relation to:  https://github.com/apocas/docker-modem/pull/57
unfortunately I was not able to get the qs library to serialize the querystring to what the docker api expects. for now I think we need to stick w/ the manual string representation.
included is an example and unit test to help clarify this for existing users.

@rmg 9134d69 should address the false-passing issue when testing against agents that have 0 containers. thanks again for bringing that up and forcing me to clean this up :)

I've created a PR that supports auto-encoding maps as JSON strings (for easier filter management):
https://github.com/apocas/docker-modem/pull/58
and I've written a test for the patched docker-modem behaviour here :
https://github.com/briceburg/dockerode/commit/6ff5ff777d292accae84dea0fef5143aee649a8a
It doesn't make sense to include the new test unless the docker-modem PR is accepted, so I'll defer this submission until a decision has been made on that.
In the meantime this PR is still valid. Would you like to to make any changes?  Thanks!

I'm going to separatate this into two PRs, one for docs, one for tests.


merge from template-coveralls

package.json

chore(deps): add ^21.0.0 remove ^20.1.3 (devDependencies.semantic-release)

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

npm package (@latest dist-tag)
v6.4.0
GitHub release

Your semantic-release bot :package::rocket:


Cannot update or uninstall client

Hi,
I have had Keybase installed on my work computer for a while. Since some week ago the company migrated all the machines to the company domain, I can't really tell what all the changes were.
Anyways, after the migration I get an error message while updating or uninstalling. I want to uninstall so that I can install it fresh.

Is there any workaround to get rid of the current installation?
Thanks!

Download that specific MSI, and tell the installer where it is instead

Thank you, I will try to locate the that version of the installer, and it should do the trick.


Window not configured correctly to work with fancy zones

I often have to open three or more instances of service bus explorer to view various queues in different environments. I like to dock them. I use a tool called FancyZones from the Windows Power Toys offering to do this. However, Service Bus Explorer doesn't work correctly with this tool. I created an issue [https://github.com/microsoft/PowerToys/issues/17354] and they came back with the following:
Checked ServiceBusExplorer, and the window doesn't have WS_VISIBLE style, that's why it's ignored by FancyZones.
I am unsure what this means or how to correct it, or if it's truly an issue with Service Bus Explorer. Can a dev please take a look and correct if possible or report that it's a none issue if it's not?

@TheAtomAnt, this is a FOSS project. There are no "assigned" or "allocated" developers to look at the issues. A few of us volunteer here and there, and some contribute to the areas where they'd like to see some improvement. If you'd like to see this addressed sooner, I suggest rolling the sleeves and giving it a try or convincing someone from your organization to help.


[ios] Swipe gestures trigger prematurely

Swipe gestures on iOS trigger not on release, but after an initial, small movement. This makes it too easy to make an unintended move if there is an initial deviation in the wrong direction, even if the start and end points clearly indicate a different direction.
Please either make the gesture trigger on release, or increase the length of stroke at which the event is triggered.

I can't confirm that. I tested using Chrome and Safari on iOS 8.3, and movements trigger after release, not requiring another movement

Sorry, I guess I should have specified that I mean the native iOS app, not the web app on iOS.


[Question] Am I using this flag correctly? (nofollow-import-to)

I wrote a simple pandas script just to print a DataFrame to the terminal,
but because my pip installation included optional pandas dependencies like matplotlib and sqlalchemy, Nuitka compiled them as well.
As a result, I used --nofollow-import-to=matplotlib --nofollow-import-to=sqlalchemy to exclude them, and the final size of the compiled EXE file was reduced by about 30%.
However, I have a few questions.
(1) In the official documentation, the --nofollow-import-to flag seems to be followed by a glob expression.
(2) The lowercase 'sqlalchemy' I used here does not match the library name seen in pip freeze, but it seems to have worked anyway?

True, it accepts patterns. I added that to the help output. We do not have a user manual section yet, that explains things about module inclusion, we are going to add it eventually from the help output.

Module names and distribution names are different for some packages indeed. Nuitka Options accept one or the other.


Missing docker images on dockerhub

I could not find following images on dockerhub.
See also #175
for n in gcs syslog stackdriver graylog logzio kafka kinesis; do
   docker pull fluent/fluentd-kubernetes-daemonset:${n}
   docker pull fluent/fluentd-kubernetes-daemonset:stable-${n}
done
----
Error response from daemon: manifest for fluent/fluentd-kubernetes-daemonset:gcs not found
Error response from daemon: manifest for fluent/fluentd-kubernetes-daemonset:stable-gcs not found
Error response from daemon: manifest for fluent/fluentd-kubernetes-daemonset:syslog not found
Error response from daemon: manifest for fluent/fluentd-kubernetes-daemonset:stable-syslog not found
Error response from daemon: manifest for fluent/fluentd-kubernetes-daemonset:stackdriver not found
Error response from daemon: manifest for fluent/fluentd-kubernetes-daemonset:stable-stackdriver not found
Error response from daemon: manifest for fluent/fluentd-kubernetes-daemonset:graylog not found
Error response from daemon: manifest for fluent/fluentd-kubernetes-daemonset:stable-graylog not found
Error response from daemon: manifest for fluent/fluentd-kubernetes-daemonset:logzio not found
Error response from daemon: manifest for fluent/fluentd-kubernetes-daemonset:stable-logzio not found
Error response from daemon: manifest for fluent/fluentd-kubernetes-daemonset:kafka not found
Error response from daemon: manifest for fluent/fluentd-kubernetes-daemonset:stable-kafka not found
Error response from daemon: manifest for fluent/fluentd-kubernetes-daemonset:kinesis not found
Error response from daemon: manifest for fluent/fluentd-kubernetes-daemonset:stable-kinesis not found

Does anyone need these images?
From my experience, latest and non-version tags are only for testing.
For fluentd and k8s environment, such images are not used on production.

This issue should be outdated.
And we already provide gcs, syslog, stackdriver, graylog, logzio, kafka, and kinesis images.


It should only match tag word bondary

My tag or alias are todo and issue. I don’t expect every word beginning with this letter matches.
Here is a screenshot.

Should be fixed, a space or semicolon is required between the tag and anything else to signal that it is a tag not part of a word. Will work on allowing users to set their own required tag separators in a future update.

Tested and confirm the fix.

There is a drawback with this fix.
Before, I was able to style a whole line with this :
{
	"tag": "====",
	"aliases": [
		"-----"
	],
	"color": "white"
},

Maybe we need a property "isPrefix": true. It would treat the tag as a prefix implying the style should apply to the whole line. What do you think ?
The regex would be something like this in this case.

"evenbettercomments.highlightTagOnly": false can’t solve it.
Another tag property could be useful : styleCommentSymbol: boolean. Or even more fine-grain with styleMonoCommentSymbol: boolean plus styleMultiCommentSymbol: boolean.
Currently, with the style above, it is inconsistent. Monoline symbol is styled. Multiline symboles are not.

There is a drawback with this fix.
Before, I was able to style a whole line with this :
{
	"tag": "====",
	"aliases": [
		"-----"
	],
	"color": "white"
},

Maybe we need a property "isPrefix": true. It would treat the tag as a prefix implying the style should apply to the whole line. What do you think ? The regex would be something like this in this case.
"evenbettercomments.highlightTagOnly": false can’t solve it.
Another tag property could be useful : styleCommentSymbol: boolean. Or even more fine-grain with styleMonoCommentSymbol: boolean plus styleMultiCommentSymbol: boolean.
Currently, with the style above, it is inconsistent. Monoline symbol is styled. Multiline symbols are not.

Ah so technically I believe it actually is working in that example, what you are seeing as the problem is the start and end brackets are not coloured, only the text in between them is, this is because if you had a multi line comment and a tag just at the top on the same line as the block start, the colouring on the start and end bracket would be inconsistent and look off. I did add a setting which would fix that but it also may not be something you like, if you take a look at the read me, you’ll see an additional settings option for something like “allow full block highlighting” or something like that, basically with that setting, if you placed a tag on the same line as the start delimiter, the entire block will be given that style including the start and end delimiters which seems to be what you’d like, however I’m not sure how it will work in combination with highlighting tags only. There does not seem to be an easy way to get the symbols to also be highlighted in a consistent manner, I may need to add another parsing step to look for monoline block comments to control highlighting on them appropriately.

Ah so technically I believe it actually is working in that example.

No, the match is actually the same value as the tag. Same number of = in the tag and in the example. If I add a = in the comment, it doesn’t match anymore and it is predictable to avoid the problem of the title of this issue.
Personally, I prefer the delimiters not to be styled. I just noticed something that might be more predictable by user settings.

So I believe I understand what you are saying, the issue is that now the match is only made for exactly the value set as the tag and never more if someone wanted to highlight a line like:
// ==================

But only had the tag set as "===" then it would fail because it is requiring them to use it like:
// === ===============

To get the highlighting, did I understand you correctly?
I think that this problem actually was removed only partially because of me setting the separators to space or colon, I removed a little bit in the parser that was actually matching for any number of tags in a row, so something like
//TodoTodoTodo: Some comment stuff

Was also matching and so combined with a more strict setting for a separator between the tag and comment, the functionality was removed. This is also tricky to implement with how tags are currently found, I may be able to add in an option that lets you match for the last character repeated.

I hadn’t notice that the rendering for online comment is greedy. The comment symbol is styled too with 	"evenbettercomments.highlightTagOnly": true

Compared to multi lines comments

Moved further discussions to new issues: #9, #10 as to close this issue since the primary topic of it had been solved.


Rtsp client ffmpeg

use ffmpeg library to display rtsp stream (here bunny video)
There is also the branch using MobileVLCKit so we need to choose which one we want to merge

Ok, so let's keep MobileVLCKit for the sample app :-).
Thanks @ayameMBS!


always delete all apps when deleting clusters

Coverage remained the same at 42.772% when pulling 122500a40c478f13bd8a9658b6ef94e8b850a0a5 on josephkane:force-delete-cluster into f397d028dad55dcbd546e6d4752777b8b6752201 on supergiant:master.


There is a small bug in 'TBirdSocketClient.IsValidHeaders'

connection issue due to strict header evaluation:
Connecting to a websocket server implemented with Spring-Boot-websocket (3.0.1), causes an exception in the 'connect'-phase, with the message:
Unexpected return key on Sec-WebSocket-Accept in handshake!

Expected Connection header is just upgrade, compared with Equals. Recieved header is upgrade, keep-alive. As the server works perfectly fine with other WS-clients, I consider this comparison a bug. The code below shows a improved variant, which makes the library work.
function TBirdSocketClient.IsValidHeaders(const AHeaders: TStrings): Boolean;
begin
  Result := False;
  AHeaders.NameValueSeparator := ':';
  if (AHeaders.Count = 0) then
    Exit;
  if (not AHeaders[0].Contains('HTTP/1.1 101')) and (AHeaders[0].Contains('HTTP/1.1')) then
    raise Exception.Create(AHeaders[0].Substring(9));
  // Bug: currently there is: Equals('upgrade')
  // Improved/fixed:
  if AHeaders.Values['Connection'].Trim.ToLower.contains('upgrade') and AHeaders.Values['Upgrade'].Trim.ToLower.Equals('websocket') then
  begin
    if AHeaders.Values['Sec-WebSocket-Accept'].Trim.Equals(FSecWebSocketAcceptExpectedResponse) then
      Exit(True);
    if AHeaders.Values['Sec-WebSocket-Accept'].Trim.IsEmpty then
      Exit(True);
    raise Exception.Create('Unexpected return key on Sec-WebSocket-Accept in handshake!');
  end;
end;

Very very nice, I will merge your code.

Merged. Thanks

Thank you!


fix: fix mauvais lien plausible sur les stats

fix #2719

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

GitHub release
v3.14.0

Your semantic-release bot :package::rocket:


OpenAiAutoConfiguration does not recognize the API Key configured via property key: 'spring.ai.openai.image.api-key'

Bug description
When only an ImageModel bean is needed, while configuring the OpenAI API key via the below property:
spring:
  ai:
    openai:
      image:
        api-key: ${OPENAI_API_KEY}

I got the below error:
Caused by: java.lang.IllegalArgumentException: OpenAI API key must be set. Use the connection property: spring.ai.openai.api-key or spring.ai.openai.chat.api-key property.

The error persisted when I tried disabling the chat client autoconfiguration using:
spring:
  ai:
    chat:
      client:
        enabled: false

Ended up using the spring.ai.openai.api-key property to resolve the startup exception but the former key should've worked as well.
Environment
Using the Spring AI version : 1.0.0-M4 with Java 23
Expected behavior
The expectation of having the API key recognized when configured through the property spring.ai.openai.image.api-key was not met.
Steps to reproduce and Minimal Complete Reproducible example
Reproducible example can be found here. Execute the below command to get the exception:
OPENAI_API_KEY="api-key-goes-here" mvn test -Dtest=ImageGeneratorLiveTest

@markpollack can I check and contribute to this issue?


Update the cypher template to include full text search and Add neo4j browser style file.

In order to analyze various clusters I need the batch id and timestamp on when I ran the command.
Also added a full-text search to find nodes quickly.

Sorry for the really late reply. Your stylesheet looks really good. Thanks!


Confusing CodeQL Error: Go files were found but not processed

Error: Go files were found but not processed
We are seeing a weird error complaining not all Go files were processed and the error also appears on the default CodeQL scan configuration.

We have some Go files with build tags for testing purposes, for example: https://github.com/linode/terraform-provider-linode/blob/dev/linode/stackscripts/framework_models_unit_test.go#L1, which are the files not included in the scan, but I think it should not trigger an error because generally go build won't build test files (correct me if I'm wrong on this).
Is there any way to get more detailed error message or ignore the error from scanning status reporting?
And I think the correct behavior is to ignore test files not being built in the action.

That does indeed look a bit confusing. The error originates from extractor.go#L127-L136
Looking at the last lines of the Autobuild step I notice a No packages found. line:
 [2024-09-27 04:23:23] [build-stderr] 2024/09/27 04:23:23 Done extracting packages.
  [2024-09-27 04:23:23] [build-stderr] 2024/09/27 04:23:23 Installing dependencies using `go get -v ./...` in `tools`.
  [2024-09-27 04:23:23] [build-stderr] go: warning: "./..." matched no packages
  [2024-09-27 04:23:23] [build-stderr] go: warning: "./..." matched no packages
  [2024-09-27 04:23:23] [build-stderr] 2024/09/27 04:23:23 Running extractor command '/opt/hostedtoolcache/CodeQL/2.19.0/x64/codeql/go/tools/linux64/go-extractor [./...]' from directory 'tools'.
  [2024-09-27 04:23:23] [build-stderr] 2024/09/27 04:23:23 Build flags: ''; patterns: './...'
  [2024-09-27 04:23:23] [build-stderr] 2024/09/27 04:23:23 Running packages.Load.
  [2024-09-27 04:23:23] [build-stderr] 2024/09/27 04:23:23 Done running packages.Load.
  [2024-09-27 04:23:23] [build-stderr] 2024/09/27 04:23:23 No packages found.
  [2024-09-27 04:23:23] [build-stderr] 2024/09/27 04:23:23 Extracting universe scope.
  [2024-09-27 04:23:23] [build-stderr] 2024/09/27 04:23:23 Done extracting universe scope.
  [2024-09-27 04:23:23] [build-stderr] 2024/09/27 04:23:23 Running go list to resolve package and module directories.
  [2024-09-27 04:23:23] [build-stderr] 2024/09/27 04:23:23 Done running go list deps: resolved 0 packages.
  [2024-09-27 04:23:23] [build-stderr] 2024/09/27 04:23:23 Done processing dependencies.
  [2024-09-27 04:23:23] [build-stderr] 2024/09/27 04:23:23 Starting to extract packages.
  [2024-09-27 04:23:23] [build-stderr] 2024/09/27 04:23:23 Done extracting packages.
  [2024-09-27 04:23:23] [build-stderr] 2024/09/27 04:23:23 Success: extraction succeeded for all 2 discovered project(s).

Strangely enough, the summary lines in the log right before Go files were found but not processed suggest that CodeQL scanned 386 out 545 files. You might want to look at the "status" page . It should have a download link for a CSV file with information about which files were analyzed.
CodeQL scanned 386 out of 545 Go files in this invocation. Check the status page for overall coverage information: https://github.com/linode/terraform-provider-linode/security/code-scanning/tools/CodeQL/status/
Analysis produced the following diagnostic information:
Go files were found but not processed (1 result)
    * [Specify a custom build command](https://docs.github.com/en/code-security/code-scanning/automatically-scanning-your-code-for-vulnerabilities-and-errors/configuring-the-codeql-workflow-for-compiled-languages) that includes one or more `go build` commands to build the `.go` files to be analyzed.

Hi @zliang-akamai

Is there any way to get more detailed error message or ignore the error from scanning status reporting?

@aibaars' suggestion for getting more information makes sense. You can download the CSV file to see which files were extracted and which ones weren't to see whether any that you care about weren't extracted.

And I think the correct behavior is to ignore test files not being built in the action.

We do not extract test files by default, which is indeed deliberate.

The error you are seeing is ultimately just to warn you that there are .go source files in your repository, but that some of them weren't extracted. That may be deliberate (as with test files) or not, but you should check whether you are happy with the set of files that were extracted.

Hi @mbg and @aibaars,
Thanks for the response! I inspected the file and found all the not extracted files are test files. See attachment below.
code-scanning-files-extracted.csv
I think it is working as expected (test files aren't extracted by default), but I am not sure why it's still reporting the error, which is showing up in the status page... and I would like to get rid of the error from the status page if it indeed worked as expected.

Hi again @zliang-akamai 👋🏻
Great to hear that all the files you were expecting to have been extracted were extracted, and thank you for checking!
We agree that it would be better if this error message did not show up on the tool status page in a case  like this and we are working on some improvements in this area.


add domainname spec entity

add the domainname entity so that container runtimes can add special handling similar to hostname. The current workaround of adding a sysctl for kernel.domainname only works with rootful execution in most cases. This will allow for rootless execution.
Signed-off-by: Charlie Doern<EMAIL_ADDRESS>

@giuseppe PTAL

Nit:

This will allow for rootless execution

I think in commit message following must be extended to specify how a container-runtime should utilize this field, two approach i could  think are runtime invokes setdomainname( or the second one is write /proc/sys/kernel/domainname before setting up UTS namespace but is it possible in rootless session ? (Requesting maintainers to confirm)

as @flouthoc pointed out, please update the config schema as well

What should happen if the sysctl is also specified but with a different value? Should it fail?

@AkihiroSuda Afaik In crun i think sysctl is not allowed at all if spec already has a knob to configure it for instance i can see for kernel/hostname : https://github.com/containers/crun/blob/main/src/libcrun/linux.c#L3198

@opencontainers/runtime-spec-maintainers Can we merge this?


Cloudflare Web Analytics API?

Which Cloudflare product(s) does this pertain to?
Analytics
Subject Matter
Cloudflare Web Analytics API
Content Location
Within Cloudflare Web Analytics
Additional information
Does Cloudflare Web Analytics have an API? If so, how do we use it? I do not see any documentation of this.

Already documented.
https://developers.cloudflare.com/analytics/graphql-api/


Error when click on mudDatePicker

Bug type
Component
Component name
MudDatePicker
What happened?
Get error when click on MudDatePicker with the following Fixed Values Usage configuration:
<MudDatePicker Label="Month Picker (Fixed Year and Day)" HelperText="@_month?.ToShortDateString()" @bind-Date="_month" OpenTo="OpenTo.Month"<EMAIL_ADDRESS><EMAIL_ADDRESS>DateFormat="MMM" />
Error can be reproduced on https://mudblazor.com/components/datepicker#fixed-values-usage

Expected behavior
No error expected
Reproduction link
https://mudblazor.com/components/datepicker#fixed-values-usage
Reproduction steps
1.Click on MudDatePicker with "Month Picker (Fixed Year and Day)" configuration
Relevant log output
No response
Version (bug)
6.5.0
Version (working)
No response
What browsers are you seeing the problem on?
Chrome
On what operating system are you experiencing the issue?
Windows
Pull Request

[ ] I would like to do a Pull Request

Code of Conduct

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

Error caused by fixDay > max possible day in month.
Minimal repro https://try.mudblazor.com/snippet/cOwHkgQNipTfZtKj


Light Control for Smart Ceiling Fan CLF513S

Please add light control for the Smart Ceiling Fan CLF513S. Thanks!

Duplicate of #39


[Settings] Various UX tweaks

Summary of the Pull Request

Page header position corrected: now center aligned with the first navigation item, similar to W10 settings app. (#3837)
Changed FZ opacity NumberBox to Slider (#3686)
Set fixed widths for input elements, so they are consistent on every page and look better (#3692 and #3733 )
Opacity value is now shown next to the opacity sliders for Shortcut Guide and FZ (#3234)

Centered header page + consistent widths for input controls

Slider for FZ + value is now shown

[X] Applies to #3837 #3686 #3692 #3733 #3234
[X] CLA signed. If not, go over here and sign the CLA

@niels9001
Is it possible to have a var on each settings page to define the control width centralised?

In one of your comments on issue #3692 you pointed out that we should have a slider for jpeg quality level too. I think this is a great idea. Is this done in another PR?

On the other side with the number box the user can set the value more exactly.
And it looks better between the other input fields.

In one of your comments on issue #3692 you pointed out that we should have a slider for jpeg quality level too. I think this is a great idea. Is this done in another PR?

On the other side with the number box the user can set the value more exactly.
And it looks better between the other input fields.

Yeah, I left it a NumberBox because it fits in nicely with all other ComboBoxes on this page.

Excellent!
One control that still needs to be updated is this one for PowerRename:

@niels9001

Yeah, I left it a NumberBox because it fits in nicely with all other ComboBoxes on this page.

If #3835 will be applied, then I totally prefer the slider. Because of the logical min and max.

@niels9001
since you are there, can you also fix the spacing between options?

The one that should be changed is the smaller one.

@niels9001
since you are there, can you also fix the spacing between options?

The one that should be changed is the smaller one.
Good catch! They both have the SmallTopMargin margin applied. However, a ToggleSwitch doesn't end at the bottom of the blue toggle visual but runs a bit longer. Where a ComboBox / Numberbox does. This results in more whitespace.

This can be observed in the W10 Settings app as well - something for the WinUI folks to maybe look at it (or we could adopt the layout/form control they are planning to work on).
For now I'll hardcode the margin to remove the inconsistency in this particular layout where we use both a toggleswitch + 2 numberboxes.


Fixed issue with ```shell being displayed

Fixed the issue of ```shell being displayed due to missing code block

Thanks, for the PR @dinukadev! Looks great!
Preview: https://deploy-preview-13144--kubernetes-io-master-staging.netlify.com/docs/reference/kubectl/docker-cli-to-kubectl/
/lgtm
/approve

Thanks @cody-clark . Appreciate the quick review!

Any time, @dinukadev! And sorry about any confusion for that last PR.


Added index.js file for javascript code

Fixes: #1770
Description
Added separate index.js file for javascript code for better maintenance of codebase

Type of PR

[ ] Bug fix
[x ] Feature enhancement
[ ] Documentation update
[ ] Other (specify): _______________
[ ]

[x ] I have made this change from my own.
[ ] I have taken help from some online resources.
[x ] My code follows the style guidelines of this project.
[ x] I have performed a self-review of my own code.
[x ] I have commented my code, particularly in hard-to-understand areas.
[x ] I have made corresponding changes to the documentation.
[ x] My changes generate no new warnings.
[ x] I have tested the changes thoroughly before submitting this pull request.
[x ] I have provided relevant issue numbers and screenshots after making the changes.

Conflicting files
index.html
Please see the conflicts highlighted in the image above. Resolve the conflicts in the mentioned files. Ensure you pull all the latest changes before pushing your feature/bug fixes to the main branch. After resolving the conflicts, please open a new PR for review.
Thank you!


Fix copy plugins

the low level method CMSPlugin.copy_plugin messes up the reference to the old plugin changing it to the new one
This has no consequence if the old plugin instance is not "reused", but it may affect lower level use (like in blueprint)

code looks good to me. don't have a real project to properly test this for regressions though.

Was asked to look at this in #3637, I'm not clear on what bug this is solving but have no doubts there is a bug ;) Is there a ticket I'm missing that explains the details of this? As noted in #3637, the issue there is that plugin ordering is thoroughly screwed up on publish and that bug was introduced in the develop branch after the django-treebeard changes were merged. It doesn't appear that this PR addresses that issue.

yeah, after checking your PR the issue is slightly different, nevermind

OK, I've tested this on an existing project with some pretty custom and deeply-nested plugin setups. Seems to work! Great job!


General Question: Is possible to add custom buttons?

I see the buttons:  Add. Edit, Delete, Refresh and a comment  saying  // do not change name
is it possible to add custom buttons with any other name we want?
I know this is a Datatables feature but could this mess this plugin if I add buttons to each row?

If you mean to add other buttons more than "Add", Edit", "Delete" over the table: yes, there is no problem. Please just don't change the "name" if the 3 buttons above.
If you mean to add buttons "inline" on each row: this is more problematic. I don't know.

Mmm, but how to fire a function with a custom button?

what kind of function are you thinking of?
and, is this question related with altEditor, or is this a question on datatables? buttons are a standard component, see e.g. the docs

is related to AltEditor, we have the basic CRUD in AltEditor, but you could have a scenario like mine where you add a sub-table and you could take action on sub-table but need the Id from the parent, it's like put AltEditor in a subtable..
An image can speak more I put an image here

Please see https://luca-vercelli.github.io/DataTable-AltEditor/example/12_inline_buttons/example12.html


:seedling: Bump github.com/ossf/scorecard/v5 from v5.0.0-rc2 to v5.0.0

Note: we should wait until Monday to start cutting a release here.

Note: we should wait until Monday to start cutting a release here.

Roger that!


RDM6300 addtional output formats, XOR Checksum

I had recently some issue with the RDM6300 reader, during debugging I found I would be very nice to have matching Numbers between the entries in the Database and the ones printed on the ID-Cards itself.
On the ID-Cards I am having are two Numbers, one 10 Digit Number with leading zeros and a fractional number.
Both are based on the same card number, just with a different representation. The Format is called Wiegand26.
Also  I added a XOR checksum check  as it is described in the RDM6300 Manual.
Additionally I tried to Address the dependency Topic of #1105, but this is mostly untested, since I don't have this modules available. It would be very nice if someone could support here.

Hey @arne123,
the CI currently fails (e.g. https://github.com/MiczFlor/RPi-Jukebox-RFID/actions/runs/597448418).
./scripts/Reader.py.experimental:170:37: F821 undefined name 'MIFARE_WAIT_FOR_ENTRY'
        self.device.set_max_retries(MIFARE_WAIT_FOR_ENTRY)
                                    ^
1     F821 undefined name 'MIFARE_WAIT_FOR_ENTRY'
1
Error: Process completed with exit code 1.

I think the import doesn't work right. Could you have a look?
Thanks

I think I got it working with adding
from py532lib.mifare import MIFARE_WAIT_FOR_ENTRY
I can not test the Module without active i2c, I just checked for the availability of MIFARE_WAIT_FOR_ENTRY
which seems to have the right value (255/0xFF).
Can you do me a favor and test, if this will work in your environment?
Then I will update the PR
`class Pn532Reader:
def __init__(self):
    from py532lib.i2c import Pn532_i2c
    from py532lib.mifare import Mifare
    from py532lib.mifare import MIFARE_WAIT_FOR_ENTRY
    print(MIFARE_WAIT_FOR_ENTRY)  # just for test
    pn532 = Pn532_i2c()
    self.device = Mifare()
    self.device.SAMconfigure()
    self.device.set_max_retries(MIFARE_WAIT_FOR_ENTRY)

def readCard(self):
    return str(+int('0x' + self.device.scan_field().hex(), 0))

def cleanup(self):
    # Not sure if something needs to be done here.
    logger.debug("PN532Reader clean up.")`

Unfortunately I don’t have that reader to test it

Adding the import made the flake8 check run again, but the unit test fails due to changes in gpio control
==================================== ERRORS ====================================
21
__________________ ERROR collecting test/test_gpio_control.py __________________
22
ImportError while importing test module '/home/runner/work/RPi-Jukebox-RFID/RPi-Jukebox-RFID/components/gpio_control/test/test_gpio_control.py'.
23
Hint: make sure your test modules/packages have valid Python names.
24
Traceback:
25
/opt/hostedtoolcache/Python/3.7.10/x64/lib/python3.7/importlib/__init__.py:127: in import_module
26
    return _bootstrap._gcd_import(name[level:], package, level)
27
test/test_gpio_control.py:5: in <module>
28
    from components.gpio_control.gpio_control import get_all_devices
29
E   ImportError: cannot import name 'get_all_devices' from 'components.gpio_control.gpio_control' (/home/runner/work/RPi-Jukebox-RFID/RPi-Jukebox-RFID/components/gpio_control/gpio_control.py)
30
=========================== short test summary info ============================
31
ERROR test/test_gpio_control.py
32
!!!!!!!!!!!!!!!!!!!! Interrupted: 1 error during collection !!!!!!!!!!!!!!!!!!!!
33
=============================== 1 error in 0.21s ===============================
` ``

see https://github.com/MiczFlor/RPi-Jukebox-RFID/runs/1983037905?check_suite_focus=true

Fixed the import in #1312


Add Heart BioAmp Candy in "Hardware" section

Hey @lorforlinux
I have made the following updates in Heart BioAmp Candy documentation:

Added content including description, features, and how to use the sensor
Added Candy images, connection diagrams, and snapshot for ECG signals recording using the sensor.

Please review and merge.

@lorforlinux Thankyou for the inputs! I have resolved all the comments. You can go ahead and merge.

@bhawnasehgal Thank you for your contributions, I am merging this now.


Multiline label texts are incorrectly positioned in Safari

Problem
When using Safari browser on iPhone or Mac computer, if a text label is multiple lines (i.e., contains \n) then the entire label is shifted upwards when rendered on the graph (assuming label direction = south).
Likely Cause
https://github.com/dash14/v-network-graph/blob/4e91d30d6912a56697de8f693c4f3042c881d372/src/components/label-text.vue#L156-L161
When a <tspan> is rendered in Safari, it does not seem to inherit the dominant-baseline attribute from the parent <text> element. Chrome, Firefox, and Edge do not seem to have this issue.
Proposed Solution
Consider setting dominant-baseline directly on <tspan> in addition to <text>.

Reference: https://stackoverflow.com/a/42023579
"Apparently, Safari wants the inner tspan have the dominant baseline set..."

Hi @ctrueblood-epri,
Thank you very much for your report.
I have implemented the solution you suggested.
Please check it out v0.3.7.


Face tracking params sometimes default to 1 or -1

Sometimes when first joining a session... combined params default to 1 or -1 instead of 0... this can be fixed by quickly moving your face around a lot but is annoying to do.

I can also confirm this happens on the latest release. It appears to affect a random parameter on join, anyone else having this issue?

I am aware of the issue and the reasons behind it, just trying to finish up the UI before going back into the merger and fixing stuff up in there 😄

Should be fixed next update! Hopefully won't require any avatar modifications


Text

The text about me can be updated to be more selling

I've created a pull request #158 that changes the text to be slightly more professional and references your skills.


MC-1.12.1 Crash Serverside - Immidiate fix requested!

https://gist.github.com/P3rf3ctXZer0/67cfa05b72d0b0367c138edbfa1a8d19

Fixed.


FEATURE Job - when duplicating, also duplicated related objects

Such as Sections, Category
https://docs.silverstripe.org/en/4/developer_guides/model/relations/#cascading-duplications

Only implement if we can skip duplicating Submissions


Updates "val nameVersion = xxx" in sbt config

When the target's sbt config contains a line like:
val humbugVersion = "quoted version string"

update the version string to the latest version of the given artifact.
This generalises a previous feature that applies only to "val uniformVersion".
Also fixes bugs in the regex for updating depend.omnia() calls.

I know I haven't set a good precedence with commenting :smile: but are you able to add some comments for these new functions. They are getting a bit more complex.

:-1:


FAQs corrections

1. Does this PR affect any open issues?(Y/N) and add issue references (e.g. "fix #123", "re #123".):

[x] N
[ ] Y

2. What is the scope of this PR (e.g. component or file name):
FAQs

3. Provide a description of the PR(e.g. more details, effects, motivations or doc link):

[ ] Affects user behaviors
[ ] Contains syntax changes
[ ] Contains variable changes
[ ] Contains experimental features
[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[ ] Other

4. Are there any breaking changes?(Y/N) and describe the breaking changes(e.g. more details, motivations or doc link):

[x] N
[ ] Y

5. Are there test cases for these changes?(Y/N) select and add more details, references or doc links:

[ ] Unit test
[ ] Integration test
[ ] Benchmark (add benchmark stats below)
[ ] Manual test (add detailed scripts or steps below)
[ ] Other

Some codes were giving errors because of some logical errors and typos:

There is one commit incorrectly signed off. This means that the author of this commit failed to include a Signed-off-by line in the commit message.
To avoid having PRs blocked in the future, always include Signed-off-by: Author Name<EMAIL_ADDRESS>in every commit message. You can also do this automatically by using the -s flag (i.e., git commit -s).
Here is how to fix the problem so that this code can be merged.
Rebase the branch
If you have a local git environment and meet the criteria below, one option is to rebase the branch and add your Signed-off-by lines in the new commits. Please note that if others have already begun work based upon the commits in this branch, this solution will rewrite history and may cause serious issues for collaborators (described in the git documentation under "The Perils of Rebasing").
You should only do this if:
You are the only author of the commits in this branch
You are absolutely certain nobody else is doing any work based upon this branch
There are no empty commits in the branch (for example, a DCO Remediation Commit which was added using --allow-empty)
To add your Signed-off-by line to every commit in this branch:
Ensure you have a local copy of your branch by checking out the pull request locally via command line.
In your local branch, run: git rebase HEAD~2 --signoff
Force push your changes to overwrite the branch: git push --force-with-lease origin faq-corrections

Hi @Sumitwarrior7 😄
Good Job! DCO github action is failed, you can follow this https://www.secondstate.io/articles/dco/ to fix.


Changelog entry for v0.3.1

I can't seem to find any info in the CHANGELOG about the v0.3.1 release.

I have updated the CHANGELOGs: https://github.com/specta/expecta/commit/40c8ca3a91fd5b7766e6fe5a94e7f882fba86aef


example application, 'npm run test' - No tests found

Description
running npm run test in the example app as outlined in the testing notes does not perform any tests on Windows 10.
No tests found   In C:\Development\web\react-boilerplate   19410 files checked.   testMatch:  - 19410 matches   testPathIgnorePatterns: \\node_modules\\ - 19410 matches   testRegex: tests/.*\.test\.js$ - 0 matches   Pattern: "" - 0 matches
Steps to reproduce
Follow the quick start guidelines;
Clone this repo using git clone --depth=1 https://github.com/react-boilerplate/react-boilerplate.git Move to the appropriate directory: cd react-boilerplate. Run npm run setup in order to install dependencies and clean the git repo. Then run npm run test
Versions

React-Boilerplate (see package.json): v3.5.0
Node/NPM: v8.9.4 / v5.6.0
Browser: N/A

Can you re-clone this project and output the full stack trace of the error?

I'm sorry, perhaps that was not very clear. There is error occurring, npm run test runs successfully but it fails to find any tests to run - when there are clearly many tests available in the example application.
I'm assuming these are found and run by others environments as I don't see anyone with similar issues but fail to run on mine - I don't understand why.

Sorry, maybe I'm not understanding. I'm asking if what happens if you scrap the first original clone you did, and try again with a fresh start-- and then let me know what the output is. Sometimes a refresh can be more enlightening as you're watching each step closer

re-cloned and ran npm setup, then npm run test - here is the output. Notice, "no tests found"

There's a lot of known bugs with Jest and Windows. This is def a Jest issue, I suggest browsing their issue channel -- this one comes up first.


Algorithms solns

Solved issue #553
In Language- JAVA

[x] Updated README Link
[x] Added code with comments

Thanks for contributing!

Resolve conflicts

check


Feature Request: Make Thumbnails Follow Sym Links

The thumbnail path could follow system links. So thumbnails do not need to be regenerated.
I'm imagining of a workflow where we can use symlinks to make virtual folders, but do not regenerate thumbnails.
Examples are:
A file search presents its result in a "virtual folders"  with the found files linked in it.
"Virtual folders" based on file tags.
The follow system links feature could be enabled with a flag.

That's a nice feature, which also saves space.
Implemented on the last commit.
There's no need to save two files for the same thumbnail.


feat(cli): implement tests and dependency checks for init command

Description
This pull request changes the following:

Enable jest unit test framework
Enable npm test in CI/CD
Implement initial dependency checks as part of the init command (We would need to extend it as required)
Add tests with 84% code coverage
Restructure code (all commands classes are now in a single directory and inherits a base command class)

Related Issues

Closes #433

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

GitHub release
npm package (@latest dist-tag)

Your semantic-release bot :package::rocket:


Support for more initial life totals

To be selfish for a second, my primary desire for Simple Life is for playing a deckbuilding game called Star Realms. The initial life total is 50, an option I can't provide to Simple Life. It would be great to have a method for setting custom initial totals.
I'm happy to put together a PR if you decide this feature is desirable.

I've actually run into the same problem recently, where some friends and I started playing a new format that life totals start at 25 rather than 20, 30, or 40, and have been meaning to add more customization options.  This just gave me more motivation and I've just finished some mock-ups of how to implement the UI for it, and I'll be working on it now.

I don't know if you're still interested in working on this, but this is the closest app to having everything I want in a Magic Lifecounter so I've been playing around with it a bit in my own fork. I actually have a branch that just adds this feature and I can send a PR if you're interested.

I don't know if you're still interested in working on this, but this is the closest app to having everything I want in a Magic Lifecounter so I've been playing around with it a bit in my own fork. I actually have a branch that just adds this feature and I can send a PR if you're interested.


Add nebula-graph to the list of resources for database.

This is not a list of "apps" written in c++.

This is not a list of "apps" written in c++.

Thanks, got it.


Tidy Publication schema

Since we inherited publication schema from Encode without much modification, there are some fields that needs make up.

date submitted is a string
I think the supplementary data fields is too complicated, and all can be associated in the form of experiments, or documents.
Types of publication can be aggregated under 3, 4 categories, instead of >10.
I am not sure if we want things like "date revised"
Published by is an array, and does not contain anything we need.

I will implement the changes after discussing with @aschroed

I don't see this field.  I see date_pubilshed and date_revised.  I agree that these type of fields should be date-time.
I agree but we may want url as well if we are dealing with some 'external' publications.
Agreed
4 and 5. I think Nils may have thoughts on some of this as well as necessary status fields as he is involved in setting publication and data release policy and also is interested in using our system as a possible publication tracking tool.

I have a couple other possible modifications.  We can discuss tomorrow.

It is date published that is string.
I agree, we can keep the url.
I guess the date revised that Nils wants is different than the one it refers to now. Most of the fields we ask for can be filled in an automated way just by the pubmed id.

Agreed on the automated filling using PMID if the paper has already been published.  However, I believe Nils' use cases are for consortium members planning and tracking their publications through the complete process of conceiving the idea of publishing on a set of experiments through developing an abstract (though when I wrote papers this was the last thing I did - just saying) and then the full life cycle of submitting, revising and then finally publishing.
He has got feedback that some people will be leary about sharing details of publications that are in preparation if everyone can see them - hence the need for permission control via status on Publication.
I agree that published_by does not need to be an array and I think that you are correct that we can remove this and just use 'project' in the Award to indicate who to attribute the publication to i.e. 4DN or External.

Changes Summary
All changes are under bestseller branch "PublicationMadeSimple"
1) Required
Lab and Award not required (to be able to related external publications)
2) IdentifyingProperties
I took out title, for 2 reasons
. the title might change for publications in process
. it is a very long string to use as an id
3) Authors
gave a definition for naming convention
Name1 Surname1, Name2 X. Surname2,...
We can also add a regex, right now it is a string
4) Contact author
If the publication is progress, a user will be linked to the publication as the contact person as suggested by @ngehlenborg .(not the corresponding author)
5) Supplementary data
deleted
6) URL
added as string, I think we should have this as a calculated property from PMID
7) Details  (issue volume page)
I took out this fields, if we want to display such fields in the frond end, we can either create calculated properties or directly add them in the frond end by fetching them from pubmed.
8) Identifiers
I reduced them to PMID, I think this is the most capable one, if we need DOI we can also have a calculated property
9) Categories
The categories contained the following, is an array, I would say we should go through the list and  reduce the number of options. Maybe @burakalver can have a look.

"human disease",
"basic biology",
"software tool",

                "database",

                "integrative analysis",

                "technology development",

                "data standard",

                "key publication",

                "pilot phase publication",

                "model organism biology",

                "tutorial",

                "genomic annotations",

                "production"

10) Published by
The list contained the ENCODE related ones, right now we have

4DN
External

11)Abstract
Abstract has "rdfs:subPropertyOf": "dc:abstract", I left it there, but have no idea what it does.
12)Date revised
deleted

Date published
changed from free string to format=date and also added a description
there is a calculated property publication year, which uses date published. it is modified to work again with the new format

14)Internal tracking/creation date and last modification date
I think all these are calculated fields that will come with the tracking system
I did not implement them
15) Status
There is the mixin status, and also the publication status. They are both called status at the moment. Since this is related to permissions, I leave this part to @aschroed. I am tempted to call the publication status with some other name and have 2 status.


System.NullReferenceException if Assigned Coordinator for activity is removed from volunteering

If an assigned coordinator stops volunteering for a disaster and activity (removes their self from volunteers) and the Admin tries to access the Cluster Coordinators for that disaster, an error is thrown by the server.
Steps to reproduce:
Have a user volunteering for a disaster and activity/cluster
As Admin, assign that volunteer as the cluster coordinator
As the volunteer, remove that disaster and activity/cluster from the list of events you're volunteering for
As Admin, try to access the Cluster Coordinators for that disaster
Expected:
The volunteer is no longer listed as the cluster coordinator
Actual:
The System.NullReferenceException is thrown. If the volunteer re-volunteers for the same event at the same time, the Admin will be able to access that cluster coordinators
Looks like issue could be the same as #355

Looking at the code for this, the issue is in the ClusterCoordinatorController/GetDisasterClusterCoordinatorsViewModel method. As the code tries to build the clusterViewModel it tries to assign to the Coordinators property. That attempt involves the following invocation of the Select LinQ method:
.Select(x => new ClusterCoordinatorViewModel
{
Name = x.Person.FullName,
Id = x.Id,
})
The issue is that should there be a cluster which does not have a clusterCoordinators record matching it, then a null value is passed into the Select...
The specific issue could be resolved by generating the Coordinators list before trying to instantiate the ClusterViewModel, thereby giving us the control to be able to simply pass an empty list should there be no Coordinators for the selected cluster.
That just gets us past the immediate problem though - there could be issues further down the line with the UI code... I'll take a look at it and see what happens...
A word of caution - I am new to htbox...

@rwherrmann will work on this issue.

@rwherrmann - note my comment above as to cause...

@rwherrmann will work on this one.

Actually, a slightly different approach is needed...

You have it

My comments above were misleading - a ClusterCoordinator is found, but the Person property hasn't been populated - hence the call to Person.FullName on the Select lambda fails...
Sorry for confusion...

Thanks for the info.  I am thinking that the source of the issue is that
when a volunteer removes themselves from a commitment, we should also check
if they are a coordinator and remove that as well.  We can code around the
issue in the ClusterCoordinator controller but the underlying data issue
will still be there.
Does that make sense?
On Wed, Jan 6, 2016 at 10:36 AM, Martin Milan<EMAIL_ADDRESS>wrote:

My comments above were misleading - a ClusterCoordinator is found, but the
Person property hasn't been populated - hence the call to Person.FullName
on the Select lambda fails...
Sorry for confusion...
—
Reply to this email directly or view it on GitHub
https://github.com/HTBox/crisischeckin/issues/406#issuecomment-169360875
.

Yes - you're on the right track... I've kept looking at this, and it looks like the DataService's RemoveCommitmentById method needs to also remove any ClusterCoordinators for the Cluster we are looking at.
I'll leave that for you, unless you can't do it, in which case I will do it later tonight...

That's what I am thinking too - remove the cluster coordinator data at that
point if it exists.  I can finish that now.
On Wed, Jan 6, 2016 at 11:06 AM, Martin Milan<EMAIL_ADDRESS>wrote:

Yes - you're on the right track... I've kept looking at this, and it looks
like the DataService's RemoveCommitmentById method needs to also remove any
ClusterCoordinators for the Cluster we are looking at.
I'll leave that for you, unless you can't do it, in which case I will do
it later tonight...
—
Reply to this email directly or view it on GitHub
https://github.com/HTBox/crisischeckin/issues/406#issuecomment-169370735
.

It goes without saying (and indeed just did) that the only ClusterCoordinators we would remove would be where the cluster and person matched the cluster and person of the commitment we were removing though...
ClusterId is nullable on the Commitment by the looks of things as well...

It would be nice to do this at the schema level. I'm not seriously proposing a trigger, but that sort of idea - so if the commitment is removed, the clusterCoordinator dies with it as a consequence of the database alone. There could be an approach with ForeignKeys, but I'm too tired to think about it...
My advice then - go for the hack on the data service...

Fixed in PR #457


About using different version of moveit

Hello,
I find my moveit version is 0.9.18, but running aubo_i5_moveit_config gives the following error:

error while loading shared libraries: libmoveit_robot_trajectory.so.0.9.15: cannot open shared object file: No such file or directory

Is it possible to do minimal change to use the new version of moveit? Or the better way to install the old version of moveit? I am very appreciate for your advice.
Thanks.

i have the same question

You may have used melodic or a later version of ROS to compile the kinetic version of the code. You can select the code of the melodic version branch to compile.


[FEAT] Swap Journeys and Pages section on sidebar

Is your feature request related to a problem? Please describe.
Hi, my suggestion is to swap the Journeys and Pages sections on the sidebar. Because when a new user visits the website the links under the Pages section help the user to know more about the project. What are your thoughts folks?
Screenshot

-- I totaly agree with you vikas , swamp this 👍🏻
-- one thing I would like to add , kindly checkout Ebuka's journey .. there's an error

Yes. Will do this.
On Mon, 25 Jul 2022, 10:03 am Vikas Ganiga, @.***>
wrote:

Is your feature request related to a problem? Please describe.
Hi, my suggestion is to swap the Journeys and Pages sections on the
sidebar. Because when a new user visits the website the links under the
Pages section help the user to know more about the project. What are your
thoughts folks?
Screenshot
[image: image]
https://user-images.githubusercontent.com/74750414/180699336-92ff8504-d7a8-4c0f-95d6-4c3717729219.png
—
Reply to this email directly, view it on GitHub
https://github.com/collab-community/journey-book/issues/62, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AUGJUTOG7KBTX4CWFG3LB73VVYKKHANCNFSM54Q3M5IA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Thanks, Kendall.


invalid json

after enabling the auto export function with youtube as export location. After a restart of the docker i get the error and bootlop
01.08.2023 09:35:10
SyntaxError: Unexpected token '{', ..."closed":tr{"module":"... is not valid JSON
01.08.2023 09:35:10
    at JSON.parse (<anonymous>)
01.08.2023 09:35:10
    at <anonymous> (/usr/local/share/twitchautomator/server/src/Core/Log.ts:72:57)
01.08.2023 09:35:10
    at Array.map (<anonymous>)
01.08.2023 09:35:10
    at readTodaysLog (/usr/local/share/twitchautomator/server/src/Core/Log.ts:72:22)
01.08.2023 09:35:10
    at Function.init (/usr/local/share/twitchautomator/server/src/Core/LiveStreamDVR.ts:128:9)

Reverted back to an old config.json but still getting the error.
The I read a couple of lines above that the log is the file he looks at and crashes?

01.08.2023 09:42:08
Read today's log...
01.08.2023 09:42:09
undefined:1
01.08.2023 09:42:09
{"module":"YouTube","time":1690870443221,"level":"ERROR","text":"Could not upload video: The request cannot be completed because you have exceeded your quota.","pid":34,"metadata":{"response":{"config":{"url":"https://youtube.googleapis.com/upload/youtube/v3/videos?part=snippet&part=status&uploadType=multipart","method":"POST","userAgentDirectives":[{"product":"google-api-nodejs-client","version":"5.1.0","comment":"gzip"}],"data":{"_readableState":{"objectMode":false,"highWaterMark":16384,"buffer":{"head":null,"tail":null,"length":0},"length":0,"pipes":[],"flowing":false,"ended":true,"endEmitted":true,"reading":false,"constructed":true,"sync":false,"needReadable":false,"emittedReadable":false,"readableListening":false,"resumeScheduled":false,"errorEmitted":false,"emitClose":true,"autoDestroy":true,"destroyed":true,"errored":null,"closed":true,"closeEmitted":true,"defaultEncoding":"utf8","awaitDrainWriters":null,"multiAwaitDrain":false,"readingMore":false,"dataEmitted":true,"decoder":null,"encoding":null},"_events":{},"_eventsCount":2,"_writableState":{"objectMode":false,"highWaterMark":16384,"finalCalled":true,"needDrain":false,"ending":true,"ended":true,"finished":true,"destroyed":true,"decodeStrings":true,"defaultEncoding":"utf8","length":0,"writing":false,"corked":0,"sync":false,"bufferProcessing":false,"writecb":null,"writelen":0,"afterWriteTickInfo":null,"buffered":[],"bufferedIndex":0,"allBuffers":true,"allNoop":true,"pendingcb":0,"constructed":true,"prefinished":true,"errorEmitted":false,"emitClose":true,"autoDestroy":true,"errored":null,"closed":true,"closeEmitted":true},"allowHalfOpen":true},"headers":{"x-goog-api-client":"gdcl/5.1.0 gl-node/19.9.0 auth/7.14.1","content-type":"multipart/related; boundary=9016b555-b76c-46e7-9968-8a7afbe583ab","Accept-Encoding":"gzip","User-Agent":"google-api-nodejs-client/5.1.0 (gzip)","Authorization":"Bearer ya29.redacted","Accept":"application/json"},"params":{"part":["snippet","status"],"uploadType":"multipart"},"retry":true,"body":{"_readableState":{"objectMode":false,"highWaterMark":16384,"buffer":{"head":null,"tail":null,"length":0},"length":0,"pipes":[],"flowing":false,"ended":true,"endEmitted":true,"reading":false,"constructed":true,"sync":false,"needReadable":false,"emittedReadable":false,"readableListening":false,"resumeScheduled":false,"errorEmitted":false,"emitClose":true,"autoDestroy":true,"destroyed":true,"errored":null,"closed":true,"closeEmitted":true,"defaultEncoding":"utf8","awaitDrainWriters":null,"multiAwaitDrain":false,"readingMore":false,"dataEmitted":true,"decoder":null,"encoding":null},"_events":{},"_eventsCount":2,"_writableState":{"objectMode":false,"highWaterMark":16384,"finalCalled":true,"needDrain":false,"ending":true,"ended":true,"finished":true,"destroyed":true,"decodeStrings":true,"defaultEncoding":"utf8","length":0,"writing":false,"corked":0,"sync":false,"bufferProcessing":false,"writecb":null,"writelen":0,"afterWriteTickInfo":null,"buffered":[],"bufferedIndex":0,"allBuffers":true,"allNoop":true,"pendingcb":0,"constructed":true,"prefinished":true,"errorEmitted":false,"emitClose":true,"autoDestroy":true,"errored":null,"closed":true,"closeEmitted":true},"allowHalfOpen":true},"responseType":"json","retryConfig":{"currentRetryAttempt":0,"retry":3,"httpMethodsToRetry":["GET","HEAD","PUT","OPTIONS","DELETE"],"noResponseRetries":2,"statusCodesToRetry":[[100,199],[429,429],[500,599]]}},"data":{"error":{"code":403,"message":"The request cannot be completed because you have exceeded your quota.","errors":[{"message":"The request cannot be completed because you have exceeded your quota.","domain":"youtube.quota","reason":"quotaExceeded"}]}},"headers":{"alt-svc":"h3=\":443\"; ma=2592000,h3-29=\":443\"; ma=2592000","cache-control":"private","connection":"close","content-encoding":"gzip","content-type":"application/json; charset=UTF-8","date":"Tue, 01 Aug 2023 06:14:03 GMT","server":"UploadServer","transfer-encoding":"chunked","vary":"Origin, X-Origin, Referer","warning":"214 UploadServer gzipped","x-guploader-response-body-transformations":"gzipped","x-guploader-uploadid":"redacted"},"status":403,"statusText":"Forbidden","request":{"responseURL":"https://youtube.googleapis.com/upload/youtube/v3/videos?part=snippet&part=status&uploadType=multipart"}},"config":{"url":"https://youtube.googleapis.com/upload/youtube/v3/videos?part=snippet&part=status&uploadType=multipart","method":"POST","userAgentDirectives":[{"product":"google-api-nodejs-client","version":"5.1.0","comment":"gzip"}],"data":{"_readableState":{"objectMode":false,"highWaterMark":16384,"buffer":{"head":null,"tail":null,"length":0},"length":0,"pipes":[],"flowing":false,"ended":true,"endEmitted":true,"reading":false,"constructed":true,"sync":false,"needReadable":false,"emittedReadable":false,"readableListening":false,"resumeScheduled":false,"errorEmitted":false,"emitClose":true,"autoDestroy":true,"destroyed":true,"errored":null,"closed":true,"closeEmitted":true,"defaultEncoding":"utf8","awaitDrainWriters":null,"multiAwaitDrain":false,"readingMore":false,"dataEmitted":true,"decoder":null,"encoding":null},"_events":{},"_eventsCount":2,"_writableState":{"objectMode":false,"highWaterMark":16384,"finalCalled":true,"needDrain":false,"ending":true,"ended":true,"finished":true,"destroyed":true,"decodeStrings":true,"defaultEncoding":"utf8","length":0,"writing":false,"corked":0,"sync":false,"bufferProcessing":false,"writecb":null,"writelen":0,"afterWriteTickInfo":null,"buffered":[],"bufferedIndex":0,"allBuffers":true,"allNoop":true,"pendingcb":0,"constructed":true,"prefinished":true,"errorEmitted":false,"emitClose":true,"autoDestroy":true,"errored":null,"closed":tr{"module":"log","time":1690871423705,"level":"INFO","text":"Starting new log file for 2023-08-01, git hash undefined","pid":34}

fixed it by deleting the log file from today???

yeah it seems like there's a race condition corrupting the file


add eth_getCode method

So far I have only added what I believe to be the interface. I am not sure if there are other parts of the code that need to be altered for this to work. Didn't see any tests I could work from. Please let me know if there is more to be done here.
cc @nlordell

Ya, so as suspected the serializer doesn't like the Option:
the trait `AsRef<[u8]>` is not implemented for `std::option::Option<Vec<u8>>`

I can try to implement this.

Turns out the Option was not necessary since the endpoint returns 0x when no code is available at the given address


Date cannot be inserted nicely for 01-02-2024

Describe the bug
Nürnberg would like to enter a date by just using the keyboard, but when entering 01-02-2024 as a start date, 10-02-2024 is the result.
To Reproduce
Steps to reproduce the behavior:

Go to create card
Click on the start date field
Type (exactly 01022024)
See that date is actually 10.02.2024

Expected behavior
The date is 01.02.2024. There ar also other fauty dates. Get in touch with daniel regarding this
Additional context

On-hold until we hear from Daniel.

Unfortunately, the interaction with date input fields depends a bit on the browser, however I guess every browser allows to enter a date using the keyboard only.
I would not classify this as a bug, though. It's also non-trivial to fix. Most likely, we would have to use yet another dependency just for the datepicker.
I'd recommend to respond with something along the following lines:
There's a way to enter a date with your keyboard only but it depends on your browser. On Firefox, simply enter all numbers (single-digit days and months prefixed with "0")  in succession (e.g.  01022024) (Firefox recognizes the intentionally placed zero-prefixes) and on Chromium-based browsers, press <TAB> in the case of a single-digit day between 1 and 3 and in the case of January, e.g.  01<TAB>01<TAB>2024 -> 01.01.2024, 0401<TAB>2024 -> 04.01.2024, 04022024 -> 04.02.2024. On Safari (which does the best thing imo) press <TAB> after every group (no matter how large your numbers are): 01<TAB>02<TAB>2024

Then our solution is use firefox, as Nuernberg decided to "depriorize" this issue in favor of parametrized urls


Send other signals than SIGHUP

I'm working on a caddy based alternative to nginx-proxy, but caddy is listening for USR1 to reload the config instead of HUP. It would be useful if docker-gen could send other signals.

Afaik I know this is supported if using a configuration file since you write [container name|container id] = [signal]

Yeah, not sure what I was thinking when I created this task. It works fine :)


Can't research extension and update.

Type: Bug
Can't research extension and update.
XHR Faild

VS Code version: Code 1.87.2 (863d2581ecda6849923a2118d93a088b0745d9d6, 2024-03-08T15:20:17.278Z)
OS version: Windows_NT x64 10.0.19045
Modes:

Hi @yhgzs-111 ,
Thanks for reaching out. If you're using a VPN client or some local firewall, please turn these off and retry. If the error is still persisting, it will help us to investigate, if you can share the network log ( command: Developer: Open Logs Folder).
-VS Marketplace Team

Closing the issue as there have been no updates for a week. If the problem persists you can contact us with the requested network log.


hey I think this might be a solution to me as well but any chance u could try explaining ur code to me?

@aslamanver  Good to see that I'm not the only one in 2020 ;)
here's my workaround for that problem
my dependencies
import MUIRichTextEditor from 'mui-rte';
import { convertFromHTML, ContentState, convertToRaw } from 'draft-js';
import draftToHtml from 'draftjs-to-html';

and converting html to editors state (mui-rte requires stringfied object) part:
let editorDefaultValue = null;
if (initialData?.text) {
    const contentHTML = convertFromHTML(initialData.text);
    const state = ContentState.createFromBlockArray(contentHTML.contentBlocks, contentHTML.entityMap)
    const raw = convertToRaw(state);
    const { blocks, entityMap } = raw;

    const fixedBlocks = blocks.map((block) => {

      return {
        ...block,
        type: block.text === '📷' ? 'atomic' : block.type
      }
    })

   const fixedEntityMap = {}
    for (const [key, value] of Object.entries(entityMap)) {
      if (value.type === 'IMAGE') {
        value.data.url = value.data.src
      }

      fixedEntityMap[key] = value;
    }

    editorDefaultValue = JSON.stringify({
      blocks: fixedBlocks,
      entityMap:fixedEntityMap
    });
  }

I know that might be discussable to rely on emoji comparison, but at least it works for now ;)
And fixing entities is because somehow editor displays images src from 'url' property.
Originally posted by @KubaZachacz in https://github.com/facebook/draft-js/issues/818#issuecomment-695799397

Hey there! I'm guessing you meant to comment in the other thread, so I'm going to close out this issue for now :). If I made a mistake, please let me know!


Adds type checking for torque symbolizers

Fixes #3970, also related to https://github.com/CartoDB/Windshaft/issues/150
Prevents the editor from sending torque-specific rules with the wrong type. E.g.:

@javisantana

Frontend tests were OK :+1: (details)

Frontend tests were OK :+1: (details)

Frontend tests were OK :+1: (details)

what happens if I have a working style where the styles become wrong after this change? will they work?

(sorry, it seems I merged just as you sent the comment, dammit!) This check is only made when styles are submitted, so it won't affect existing visualisations. Also, when the style is detected wrong, no action is applied and the last working style is the one active, as far as I know. So this shouldn't break anything already working.

Frontend tests were OK :+1: (details)


corect a translation

I correct a translation.
Please, accept my PR.

please only commit js files and not the dist files.

Yes and make sure your branch is up to date with the current master!

Thank you for your advice.
I resend PR  #1595.