Unable to release jar

Gitea version (or commit ref): Latest
Git version: Latest
Operating system: Arch Linux x64
Database (use [x]):

[x] PostgreSQL
[ ] MySQL
[ ] MSSQL
[ ] SQLite

Can you reproduce the bug at https://try.gitea.io:

[x] Yes (provide example URL)
[ ] No
[ ] Not relevant

Log gist:

Description
Trying to upload a jar fails, even when attachments are configured to allow application/java-archive
Screenshots

I had similar problem with exe files... you have to allow them in app.ini  https://docs.gitea.io/en-us/config-cheat-sheet/#attachment-attachment

I have that enabled, restarted the gitea service and made sure it's still enabled in the config; Yet I still can't upload them.

There are three places where an attachment can be denied:

The browser
The proxy-server - e.g. NGinx
Gitea

What does your browser report the attachment mimetype as? It's possible that dropzone.js is denying the upload because the mimetype the browser is reporting is different from what it is expecting.
Are you running behind a proxy server? Check that it is not denying the upload.
Check your logs to see if the mimetype you are expecting is being passed to Gitea.


[Proposal] Add a comment above the created step.

This would make it easy to find step in dart code.
A file with this
    Scenario Outline: Plus button increases the counter
        Given the app is running

Becomes that.
import 'package:flutter_test/flutter_test.dart';
import 'package:dummy_yaml/main.dart';

/// the app is running
Future<void> theAppIsRunning(WidgetTester tester) async {
  final widget = MyApp();
  await tester.pumpWidget(widget);
}

I'd expect some help from IDE here, i.e. when you type tair it should be smart enough to match it with the file named the_app_is_running.dart.
If you use search, things might become tricker when the step has parameters:
Given {2} plus {2} is equal {4}

Will give
/// plus is equal

or
/// {2} plus {2} is equal {4}

comment (depends on how you propose to handle parameters). Both of these comments might not help you when you are searching for something, as in the first case parameters are missing and if you ignore parameters — just search for plusisequal instead of plus is equal, and in the second case you'll need to know parameter values, which is unlikely is useful.
We may still add a raw line as a documentation comment to generated generic steps, not for search but for documentation purposes, if you see a value in it.

I agree with the idea of using the IDE to go directly to the step.
Yes, it may not be possible in some cases.
For me, I'd put the step in raw like /// {2} plus {2} is equal {4}.
I'm aware that it's not perfect, but it gives an alternative to the IDE solution.
PS: I'll have to see if it's not too complicated to make a vscode extension.

It should be straightforward to add such a documentation comment as the raw step line is already a parameter in GenericStep.

Yes, I don't think it's complicated.
I can do it.


purchase order

[x] mark as cancelled

[x] missing field - Tax , discount type field

[x] extra fields available in app they are - due date , purchase person ,  additional charges

[x] purchase order date should  kept non mandatory

[x] print not functioning

[x] purchase order downloading not functioning

[x] filter options missing in app - billed ,partially billed , cancelled

[x] while select organisation organisation address should be shown

[x] while selecting customer customer address should shown

[x] response message for delete with the status closed

[x] confirmation message for submitt for approval

[x] status change response message for approve

[x] status change response message for mark as issued

[x] status change response message for reject

[x] DRAFT : mark as issued ,convert to bill

[x] PENDING APPROVAL : mark as issued , convert to bill , convert to draft

[x] APPROVED : mark as issued , convert to bill , convert to draft

[x] ISSUED : mark as cancelled ,convert to bill

[x] msg for convert to bill
1 :
2:

3:

issues closed


Change Upload to Test S3 upload, not real AWS

Currently, even in testing in the CI, there is an actual S3 upload happening.
This should not be the case.
Let's implement some s3 test framework.
And activate it in the tests.
As well as write unit tests for the s3 functions implemented.

sr2silo_1  | Traceback (most recent call last):
sr2silo_1  |   File "/app/scripts/vp_transformer.py", line 19, in <module>
sr2silo_1  |     from sr2silo.s3 import compress_bz2, upload_file_to_s3
sr2silo_1  |   File "/app/src/sr2silo/s3.py", line 13, in <module>
sr2silo_1  |     from moto import mock_aws
sr2silo_1  | ModuleNotFoundError: No module named 'moto'
sr2silo_sr2silo_1 exited with code 1
Removing sr2silo_sr2silo_1 ...
Removing sr2silo_sr2silo_1 ... done
Removing network sr2silo_default

This is in docker test.
It should fail but it does not.
Once it properly fails add moto.
Probably moto is not found as it is installed as a dev dependency.


Cannot GET /sapbo.html

Hi all,

i've correctly installed resources with the command: "npm install ./build-sap-bo.bat"
i have correctly launched command "coffee coffee/sap_bo/proxy_server.coffee" using Nod.js.
i've port 3000 listening

Why Tableau returns me "Cannot GET /sapbo.html" ?
Thanks

Hey, the correct command is:
npm install
and after the
./build-sap-bo.bat
Can you please upload the console output after these commands?
Thanks

Hey, @QTarantella can you confirm that you still have the issue?

I close the ticket since there are not enough information to proceed. Feel free to reopen when you have your answers and the issue still persist


setup deploy app to azure with github and container app

setup deploy app to azure with github and container app

Changes approved


Unable to create .svg from .flowchart

Hi everyone,
here's my wf.flowchart :
`op2=>operation: D_FR = "Si demande d'acces par Francois"
op4=>operation: A_FR = 'Franþois a acces'
op6=>operation: D_OPE_PRAC = "demande d'acces par un consultant ope-PRAC"
op8=>operation: YS = 'Validation Yasmine'
op10=>operation: FR = 'Validation Franþois'
cond13=>condition: if D_FR
op17=>operation: YS
cond22=>condition: if D_OPE_PRAC
cond27=>condition: if A_FR
op31=>operation: FR
op35=>operation: YS
op2->op4
op4->op6
op6->op8
op8->op10
op10->cond13
cond13(yes)->op17
cond13(no)->cond22
cond22(yes)->cond27
cond27(yes)->op31
cond27(no)->op35`
I've just installed diagrams by running on powershell (Win10) the following command which rans successfully :
npm install -g diagrams
I then ran :

And nothing is happening.
Please, am I missing something obvious ?
Thanks in advance,
Kikala.

Your .flowchart name is not the same in your file and command.  Try to use wf.flowchart instead of wf_ope_prac.flowchart in your command line or try to rename the file from wf.flowchart to wf_ope_prac.flowchart.


Improve production perf

What about using some babel plugin for the production build, like https://github.com/callemall/material-ui/blob/master/docs/src/.babelrc?

Sorry, I'm already quite busy with an OSS projet.
I'm curiouse, why just adding one of the 4 plugins?

The rest of them don't make any sense for this boilerplate:

transform-runtime transforms ES6 to ES5 client side, we transpile it on build already and send the ES5 version to the client.

transform-react-constant-elements hoists all react elements to the outer most scope, but since we use modules we don't want that at all.

transform-react-inline-elements no idea what "exploded react objects" are and what the benefit of having JSX as them is.

From my point of view, we need them.

transform-runtime Externalise references to helpers and builtins. That meens it's reducing the output file by prevented helper function to be duplicated in each file.
transform-react-constant-elements reduces call to React.createElement and the resulting allocations. More importantly, it tells React that the subtree hasn't changed so React can completely skip it when reconciling.
transform-react-inline-elements it's preventing call to React.createElement by using a raw js object. This improve performances.

 I'll reopen the issue and fix them when I get the time. (or maybe somebody else has some time at their hand and wants to contribute?)
What are the cons of using these plugin, why isn't everybody using them?

What are the cons of using these plugin

Those plugins doesn't play well in a development environnement. That's their first drawbacks.
I have also noticed that transform-react-inline-elements increase the output build size.

why isn't everybody using them?

Because nobody is taking about them.

I'm still unsure what transform-runtime actually does. When I include it, it increases the output build size and throws a bunch of errors that the

module babel-runtime/helpers/jsx cannot be resolved

Why would I want to include babel-runtime/helpers/jsx in my build output?

Well, I'm joining your point, on some of my project transform-runtime increase the build size, on other it decrease it. Plus, I have noticed this issue with IE11 http://phabricator.babeljs.io/T6946.
I'm gonna close this issue. Thanks!


Port SyncAutoscaleSettings from Python to C#

Codecov Report

Merging #2407 (cbd724c) into main (f22dee1) will decrease coverage by 5.17%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main    #2407      +/-   ##
==========================================
- Coverage   29.90%   24.72%   -5.18%
==========================================
  Files         289      119     -170
  Lines       34905    11871   -23034
==========================================
- Hits        10437     2935    -7502
+ Misses      24468     8936   -15532

Impacted Files
Coverage Δ

src/ApiService/ApiService/Functions/TimerTasks.cs
0.00% <0.00%> (ø)

...rc/ApiService/ApiService/Functions/TimerWorkers.cs
0.00% <0.00%> (ø)

src/ApiService/ApiService/onefuzzlib/AutoScale.cs
0.00% <0.00%> (ø)

...ervice/ApiService/onefuzzlib/ScalesetOperations.cs
2.73% <0.00%> (-0.23%)
:arrow_down:

src/agent/onefuzz-agent/src/failure.rs

src/agent/coverage/src/block/windows.rs

src/agent/input-tester/src/lib.rs

src/agent/win-util/src/hr.rs

src/agent/onefuzz-task/src/local/radamsa.rs

src/agent/onefuzz/src/env.rs

... and 164 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Important config options not being set for ocean diagnostics

It appears that [input]/mappingDirectory and [regions]/regionMaskDirectory are not being set by a-prime.  As a result, all mapping files are being generated on the fly even when they already exist in a directory that should be being pointed to in the config file (see #38).  Presumably the MOC plots cannot be working properly without the region mask directory having been set.
The file that needs to be updated is:
https://github.com/E3SM-Project/a-prime/blob/master/python/setup_ocnice_config.py

Okay, great! Glad this is on your radar. This is the main issue in #38 so should be addressed sooner rather than later to not cause issues or require nasty workarounds in processflow. Hopefully, we can combine this with MPAS-Analysis v1...

Addressed by #40


Warning when compiling YGZ-slam

Please provide any relevant information about your setup.

Tutorial No.:Tutorial 3
Mavros: 0.18.4
ROS: Kinetic
Gazebo:7
Ubuntu: 16.04
System Platform: X86
*OpenCV :3.3.1

I'm trying to compile the YGZ-slam by "sh generate,sh" and i ran into the warning as shown below. Despite that, i'm still able to compile it successfully. May i know if this warning will affect the performance of the slam.
Warning log:
Could NOT find ensenso (missing:  ENSENSO_LIBRARY ENSENSO_INCLUDE_DIR)
** WARNING ** io features related to ensenso will be disabled
-- Could NOT find DAVIDSDK (missing:  DAVIDSDK_LIBRARY DAVIDSDK_INCLUDE_DIR)
** WARNING ** io features related to davidSDK will be disabled
-- Could NOT find DSSDK (missing:  _DSSDK_LIBRARIES)
** WARNING ** io features related to dssdk will be disabled
** WARNING ** io features related to pcap will be disabled
** WARNING ** io features related to png will be disabled
** WARNING ** io features related to libusb-1.0 will be disabled
-- Could NOT find ensenso (missing:  ENSENSO_LIBRARY ENSENSO_INCLUDE_DIR)
** WARNING ** visualization features related to ensenso will be disabled
-- Could NOT find DAVIDSDK (missing:  DAVIDSDK_LIBRARY DAVIDSDK_INCLUDE_DIR)
** WARNING ** visualization features related to davidSDK will be disabled
-- Could NOT find DSSDK (missing:  _DSSDK_LIBRARIES)
** WARNING ** visualization features related to dssdk will be disabled
-- Could NOT find RSSDK (missing:  _RSSDK_LIBRARIES)
** WARNING ** visualization features related to rssdk will be disabled

Thanks for the report, we will review it soon:)

No, the problem is PCL related and we are currently freezing PCL related functions in SLAM, so these warnings will not impact slam performance.
Are you using Visual Studio? PCL tend to have dependency issues with Visual Studio.


Ambiguous call to overloaded function when using per-feature includes

The following snippet produces error C2668: 'gl45core::glGetUniformSubroutineuiv' : ambiguous call to overloaded function
#include <glbinding/gl/functions.h>  // < imagine this was included by a 3rd party library (e.g. globjects.hpp)

#include <glbinding/gl/functions45core.h>
#include <glbinding/gl/enum45core.h>

using namespace gl45core;

int main(int argn, char** argv) {
    GLuint index = 0;
    glGetUniformSubroutineuiv(GL_VERTEX_SHADER, 0, &index);
}

Full log:
1>c:\users\jole\documents\visual studio 2013\projects\glbindingambiguity\glbindingambiguity\main.cpp(10): error C2668: 'gl45core::glGetUniformSubroutineuiv' : ambiguous call to overloaded function
1>          d:\projects\glbinding\source\glbinding\include\glbinding\gl\functions45core.h(311): could be 'void gl45core::glGetUniformSubroutineuiv(gl45core::GLenum,gl45core::GLint,gl45core::GLuint *)'
1>          d:\projects\glbinding\source\glbinding\include\glbinding\gl\functions.h(1017): or       'void gl::glGetUniformSubroutineuiv(gl::GLenum,gl::GLint,gl::GLuint *)' [found using argument-dependent lookup]
1>          while trying to match the argument list '(const gl::GLenum, int, gl45core::GLuint *)'

If the first line is removed, the program compiles as expected. The problem is caused by argument dependent lookup and the fact that enums constants in the gl45core namespace use a type from the gl namespace. In order to fix this, per-feature declarations should be fully independent from the gl namespace.

Yeah... http://en.wikipedia.org/wiki/Argument-dependent_name_lookup
We have to decide on the intended usage.


Does this support adding our own Conversation ID?

Does this support adding our own conversation ID to make bard remember it?

Hi, I can add this feature if you'd like! But for now, can I ask why you would? Conversations on Bard are ephemeral anyway, so I don't see the use compared to say an unofficial GPT plugin.


Latest Version Tag is not read from GIT

I am using the semver plugin for a library and have the issue that the last version is not correctly detected from the git tags.
In the GIT project there are the following tags (I also tried it without the v prefix)

v0.1.0
v0.2.0
(I also tried "0.1.0" with the same result). When

The versions generated by the plugin always are v0.1.0-XXX versions thus leading me to believe that the tag is not read and the plugin falls back to the default.
Plugin Configuration (version 0.2.2):
semver {
    version("alpha", "alpha.{COMMIT_TIMESTAMP}")
    version("beta", "beta.{COMMIT_TIMESTAMP}")
}

has been fixed in v0.4.1


[feature request] Option for instance-initiated termination

This is useful for preprocessing scripts with On-Demand instances.
The AWS syntax is at: https://stackoverflow.com/questions/10541363/self-terminating-aws-ec2-instance

Do you want a feature to terminate the instance in N hours? At the moment you can simply put the shutdown -h +120 command to the instance commands parameter  (not container commands) to shut it down in 2 hours, for example.

Will shutdown terminate the instance? Or just keep it allocated in in the turned-off mode? (that was my prior understanding)

Here they say that by default the instance is stopped (not terminated on shutdown), the good news is that it's still free, the bad news is that we need to check once in a while if the instance has stopped itself and only then terminate it. I think terminate-on-shutdown is a better fit for long-running preprocess jobs that don't need to keep the stopped instances hanging around.
https://superuser.com/questions/587727/does-shutdown-h-now-on-a-linux-aws-instance-stop-or-terminate-the-instance

@vadimkantorov A Spot Instance will be terminated (they cannot be stopped), but an On-demand Instance will be stopped. As you said you're not paying for a stopped instance, but it preserves a root volume (with OS), so the difference is you're paying for an extra volume (13GB by default).
I don't want to implement this behaviour by default, because in theory, for an on-demand instance, it would be possible to implement logic when you can stop the instance, run it again later and then Spotty would just run stopped container on it. This would be much faster than killing a whole stack and recreating it again, it also would preserve a container state.
If you're using Spot Instances for your jobs than there is nothing to be worried about, instances will be terminated and root volumes will be deleted. If you need to do it for on-demand instances, I would suggest trying to use AWS API to change shutdown behavior before the shutdown command (see modify-instance-attribute). You might also need to add some managed policy to the managedPolicyArns parameter to use this API call.

@vadimkantorov I'll close this issue. Feel free to reopen it if you have any other questions or suggestions.

Yes, this is finally for consistency mostly (so that on-demand instances terminate instead of stopping if shutdown is requested)


Add comments to process module and minor refactoring

@utam0k
I have added comments to process module and its contents, also adding some references for it in draft-doc.md
I have also changes the names of two functions that were defined and used in this module, as they seemed better :
changed ChildProcess::setup_uds -> setup_pipe
changed ChildProcess::ready -> notify_parent
let me know if you think that original were better, and I will revert them. Also let me know if any changes are required.
Thanks.

@utam0k I have changed the hard-coded numbers to the name of constants


Use existing roles in the debug stack

Zack mentioned:

My role policy is unable to create new roles. Is there a way to set default function props / stack synthesizer for the debug stack, so I can provide it the necessary roles and doesn’t try to create them?

Requests:

Zack McKenna https://serverless-stack.slack.com/archives/C01JG3B20RY/p1644958011631169

Debug stack will not be necessary to Live Debug, and is removed in SST v2.


Swap left & right mouse buttons

I have OSX in a VMWare guest and need to create an item to swap left and right mouse buttons. (it seems the system mouse settings are being ignored). Any ideas please?

Please use new setting by this private.xml:
https://github.com/tekezo/Files/blob/master/Karabiner/private.swap_mouse_buttons/private.xml
How to use private.xml:
https://pqrs.org/osx/karabiner/document.html#privatexml

Hi,
thanks but unfortunately this does not work.
The event viewer still shows left as left and right as right (maybe this is expected).
Right click still shows the context menu.
I’m wondering if this is because it’s a VMWare guest and the mouse driver is from VMWare.
Any ideas please?
Michael
From: Takayama Fumihiko<EMAIL_ADDRESS>Sent: 30 April 2016 12:26
To: tekezo/Karabiner<EMAIL_ADDRESS>Cc: Michael Tissington<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [tekezo/Karabiner] Swap left & right mouse buttons (#623)
Please use new setting by this private.xml:
https://github.com/tekezo/Files/blob/master/Karabiner/private.swap_mouse_buttons/private.xml
How to use private.xml:
https://pqrs.org/osx/karabiner/document.html#privatexml
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub https://github.com/tekezo/Karabiner/issues/623#issuecomment-215957656 https://github.com/notifications/beacon/AGdJ3CGDCM7JLYLsj-BnO3BRVXmmp6dkks5p8zxUgaJpZM4IS5TS.gif

I don't think the event viewer is supposed to change what it shows. I could be wrong, but...

Thank you for confirmation.

I’m wondering if this is because it’s a VMWare guest and the mouse driver is from VMWare.

I don't know about VMware well, but I think your assumption is right.
Please consider swapping buttons in Host OS.


Fix third-party dependency vulnerabilities for IS 6.1.0

Describe the issue:
When running the trivy scan, third party dependency vulnerabilities are reported. Need to fix the vulnerabilities of following libraries.

[x]  com.fasterxml.jackson.core:jackson-databind (kubernetes-membership-scheme-1.0.9.jar)

This issue is being closed due to extended inactivity. Please feel free to reopen it if further attention is needed. Thank you for helping us keep the issue list relevant and focused!


Remove call to NewRandomVMIWithEphemeralDisk

What this PR does
NewRandomVMIWithEphemeralDisk is deprecated. This PR removes calls to
this function from all test files (except for tests/vmipreset_test.go, that is done in #11826)
Release note

None

/sig code-quality

/retest

/retest


copy help message -- ADLS Gen2 support only Azure Active Directory Authentication

Help Messages shows the following.
./azcopy copy --help
Copies source data to a destination location. The supported directions are:

local <-> Azure Blob (SAS or OAuth authentication)
local <-> Azure File (Share/directory SAS authentication)
local <-> ADLS Gen 2 (OAuth or SharedKey authentication)
Azure Blob (SAS or public) <-> Azure Blob (SAS or OAuth authentication)
Azure File (SAS) -> Azure Block Blob (SAS or OAuth authentication)
AWS S3 (Access Key) -> Azure Block Blob (SAS or OAuth authentication)

But ADLS Gens does not support SharedKey authentication only AAD Auth.  Should this message be revised?
Source  : https://github.com/Azure/azure-storage-azcopy/blob/master/cmd/helpMessages.go
Version :AzCopy V10.1.0 GA Release

Hi, Shared Key authentication is different than SAS authentication. Furthermore, SAS authentication is in an open PR (#347).
Apologies for any confusion.

Just to provide clarity @gutzeit1106, Shared keys can be used using the environment variables ACCOUNT_NAME and ACCOUNT_KEY. These keys are intended to be full access to a single storage account. If you need them in a user-friendly way, you can obtain them using Azure Storage Explorer.

@adreed-msft  I understand that SharedKey for this help means ACCOUNT_NAME and ACCOUNT_KEY. Thank you for your response.
On the other hands,  README.md describe the following.
https://github.com/Azure/azure-storage-azcopy#authenticating-with-azure-storage

I think that this explanation should be revised.


Add arithmetic

Add arithmetic supported, like
goqu.C("a").Add(1) // "a" + 1
goqu.C("a").Sub(-1).As("b") // ("a" - 1) AS "b"

@mattn @Frug can you help to take a look at this PR?


Linux block select

The linux build doesn't seem to support block select like the Windows build; alt-shift-cursor keys.
I'd like (an option at least) for the cursor mechanics to behave identical in linux as windows. I shouldn't need to retrain my decade+ old muscle memory every time I need to fix/test the linux build.

Is there really block select in the Windows build?

I could have sworn there was? I don't have a Windows build with me now, but I distinctly recall remarking to myself when I tried it out that the block selection worked correctly...?

Nah.. never implemented. ;)


Pull Request for Bule Hora University domain name

@DSpike First stage of verification passed, official domain is bhu.edu.et as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision.

@DSpike Pull request merged. Thank you.

Thank you for your response.
On Wed, 14 Jun 2023, 2:33 pm Philip Torchinsky, @.***>
wrote:

@DSpike https://github.com/DSpike Pull request merged. Thank you.
—
Reply to this email directly, view it on GitHub
https://github.com/JetBrains/swot/pull/17770#issuecomment-1591018345,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEHZXBA2UUYEHP2KFEJCF5TXLGOQNANCNFSM6AAAAAAZFIFSFA
.
You are receiving this because you were mentioned.Message ID:
@.***>


UnicodeDecodeError: 'charmap' codec can't decode byte 0x9d in position 78552: character maps to

Description
pandas.CSVDataset default argument hit Unicode Error
UnicodeDecodeError: 'charmap' codec can't decode byte 0x9d in position 78552: character maps to <undefined>

Steps
Step to reproduce
catalog.yml
sample:
  type: pandas.CSVDataSet
  filepath: data2/sample.csv

python script to create and load dataset
df['test'] = ['123']
df['test'] = '\x9d'

df.to_csv('sample.csv') # to data/sample.csv
pd.read_csv('sample.csv') # success
catalog.load('sample') # UnicodeDecodeError: 'charmap'

Kedro: 0.16.4
Window 10

Potentially related: https://github.com/quantumblacklabs/kedro/issues/291

Not sure if it is identical, I use a plain pd.read_csv() will read the files normally

Hi @noklam,
I think this might indeed be related to https://github.com/quantumblacklabs/kedro/issues/291. Could you try and see if the solution provided there solves your issue?
"Try adding the following config to your catalog entry:
fs_args:
    open_args_load:
        mode: "rb"

We use fsspec underneath and it needs to open the file in binary mode. Try passing encoding to open_args_load or load_args as well, and that should work."

I pass Utf-8 to both load args and open_args_load and it work, I do not know why though.
I tried to debug with load args only and the encoding argument is passed to pandas successfully, not sure what is causing the issue.

Step to reproduce
catalog.yml
sample:
  type: pandas.CSVDataSet
  filepath: data2/sample.csv

python script to create and load dataset
df['test'] = ['123']
df['test'] = '\x9d'

df.to_csv('sample.csv') # to data/sample.csv
pd.read_csv('sample.csv') # success
catalog.load('sample') # UnicodeDecodeError: 'charmap'

I think this is a discrepancy in pandas' API, not Kedro.
When writing data, the pandas.CSVDataSet uses panda's to_csv method. This method uses utf-8 as the default encoding.
On the other hand, when reading data,  the pandas.CSVDataSet uses panda's read_csv method. The subtle difference here is that this method has an optional encoding, instead of an utf-8 default. If it's not provided, it will defer to the encoding of the open() method on the filesystem, which will use the system's default encoding. On some Windows system, it's cp1252, hence the charmap error message.
You can fix this by either changing the encoding on the reading side or the writing side. Both are equally valid.

On the reading side, you need to pass encoding to both load_args, which will then pass it to panda's read_csv, and open_args_load, which will then pass it to the open() method, so both use utf-8 correctly.
Or on the writing side, you can pass cp1252 to save_args so pandas writes your file with your system's default encoding and no need to adjust the load args.

I hope this helps.

What you have said suggested pd.read_csv do not use utf8. But my sample code actually just use plain read_csv and it works. Am I missing something?
thanks

Ah, sorry, I think I got a bit confused as well. The opening of the file happens in the CSVDataSet, not pandas, so I guess you are right. In the dataset, we open the file first before reading it. This opening doesn't use utf-8 but rather the system's default. So can I suggest you try just putting utf-8 in open_args_load, which when then be passed to the method, and see if it helps?

I test a few options

Success

fs_args:
    open_args_load:
        mode: "rb"

Success

fs_args:
    open_args_load:
        encoding: 'utf-8'
load_args:
  encoding: 'utf-8'

load_args:
  encoding: 'utf-8'

failed
4. To test if kedro did use both encoding,
fs_args:
    open_args_load:
        encoding: 'utf-8'
load_args:
  encoding: 'utf-1234'

Case 4 surprise me, as Case 2 suggested that load_args encoding is not passed to pandas properly, but it is actually used too.
load_args seems redundant here.

how to use fs_args for PartitionedDataset?

@noklam Does this change c3c378a871305bac30b0575aa8bbc49ee5b72777 that will go in 0.17 solve it for you? If not, could you please provide us with more information about your environment like Python version and conda environment?

@noklam I believe this is addressed in 0.17.0, as per commit referenced above. Could you please confirm if that's the case, and if so consider closing the issue?

@noklam I believe this is addressed in 0.17.0, as per commit referenced above. Could you please confirm if that's the case, and if so consider closing the issue?


Dynamic routes on Cloudflare fail trying to import node:fs/promises

The error in the workerd console is
Cannot process SSR Error: No such module "node:fs/promises".
    at null.<anonymous> (file:///waku-project/.wrangler/tmp/pages-tQnVbB/entries.js:485:11)

That is referencing these changed lines in #860. Before that PR, Cloudflare workerd must have been getting routing information from unstable_buildConfig[0].customData.data.
To reproduce
npm create<EMAIL_ADDRESS>cd waku-project
npm install
Then edit src/pages/about.tsx and change getConfig to dynamic:
export const getConfig = async () => {
  return {
    render: 'dynamic',
  };
};

Clone the waku repo.
cd packages/waku
pnpm i
pnpm run compile
Copy or link the built package to the waku project. I just replace the dist folder in node_modules/waku/dist.
In the waku project, run a cloudflare build and run a local workerd server via wrangler to serve the built files locally.
npm run build -- --with-cloudflare
npx wrangler pages dev dist
open http://localhost:8788/about in your browser.

For this we could definitely add logic into the fs-router and build process for --with-cloudflare to polyfill fs when dynamic routes are present.
I'll defer to @dai-shi on whether or not that's an ideal approach or not with this though. I haven't looked into the plugins a ton, but maybe this would be considered a responsibility of a cloudflare plugin.

Thanks for giving it a look. I'm not sure polyfilling fs is necessary... it seems the platformObject.buildData.fsRouterFiles should be populated. If the build process could pass the fsRouterFiles to the build output vite plugin, it could write to a source file during build that can be imported at runtime and added to the platformObject via the platform server.
For example, the platformObject here - https://github.com/dai-shi/waku/blob/main/packages/waku/src/lib/plugins/vite-plugin-deploy-cloudflare.ts#L86 - would need buildData.fsRouterFiles to be set. It could serialize that to a json file in the worker code bundle. That json file would be imported and passed to the waku server here https://github.com/dai-shi/waku/blob/main/packages/waku/src/lib/builder/serve-cloudflare.ts#L36-L38

Thanks for the catch. I'll have a look.


Huge clean-up: PEP8, sort imports, remove deprecated and unused imports

I think there is something more, but this is only the beginning. Just could not watch code without tears of blood.

@asnelzin you have merge conflicts.

@douglascamata fixed.

@TheSavior check this!

:+1


fix(core): should external terminal-link when build

Closes #19

Codecov Report
Base: 64.76% // Head: 64.59% // Decreases project coverage by -0.16% :warning:

Coverage data is based on head (e76ee42) compared to base (7fa3801).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #20      +/-   ##
==========================================
- Coverage   64.76%   64.59%   -0.17%
==========================================
  Files          44       44
  Lines        4129     4129
  Branches      195      195
==========================================
- Hits         2674     2667       -7
- Misses       1455     1462       +7

Impacted Files
Coverage Δ

packages/core/src/renderer/render.ts
69.09% <0.00%> (-3.64%)
:arrow_down:

packages/core/src/renderer/text.ts
51.82% <0.00%> (-1.46%)
:arrow_down:

packages/core/src/utils/index.ts
82.89% <0.00%> (-1.32%)
:arrow_down:

packages/core/src/focus/FocusManager.ts
78.03% <0.00%> (-0.94%)
:arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Confusing error message when an array of objects of types a is used instead of a plain object of type a

With following code :
interface IFoo {
  bar: string;
}

let arr = [{bar: "baz"}, {bar: "booz"}];

function doFoo(obj:IFoo){
  return obj;
}

doFoo(arr);

You are getting the error message on the doFoo(arr) line:
error TS2345: Argument of type '{ bar: string; }[]' is not assignable to parameter of type 'IFoo'.
  Property 'bar' is missing in type '{ bar: string; }[]'

I was just downvoted to oblivion on stackoverflow for asking such a stupid question, but at least for me, the error Property 'bar' is missing in type '{ bar: string; }[] is rather confusing (it doesn't make any sense) if you don't understand that the problem is that you have an object of type A on one side with an array of objects of type A on the other side.
IMHO, something like an array of IFoo is not assignable to a plain IFoo or whatever you prefer would be better than actual message.
Thank you.

I do not think there is much we can do here. the TS type system is a structural one. not all types have names, and even when they do, their names do not matter much. in your example, arr is not of type IFoo [] it is of type { bar : string}[], asking the compiler to relate this to a named type and give you the name instead is not by any means a simple task, and I do not think it will help resolve the issue either. if arr is defined as IFoo[] you should get the message you want.
the error message gives you a stack of the failed comparison, and drills down to reach the missing. i understand that this takes some training to get used to, but i believe this capture the most information he compiler can provide.


Try to run on eclispe but couldnot run

I Try to complie in my eclipse project , Couldnot complie it.
Any suggestion??

Can you share the compiler log?

when i put your exif.cpp it's give alot of error .

You need to activate C++11 support.

I am trying .
Thank you for your quick response.
Thank you again.

thank  you it's working .

Can you please close the issue.


Update ABTR model to latest doppler syntax

refs #95

Job VTB Documentation on c40aebb wanted to post the following:
View the site here
This comment will be updated on new commits.


dependencies in allErrors mode should create error for each missing dependency

Currently the error is created only for the first missing dependency.
It should be using code from "required" keyword - more refactoring to missing.def needed.

b92e6e86a78092e64c9938874395ec578c739d12


Make checkpoint header the canonical head

What was wrong?
We recently added support to persist checkpoint headers, in other words, headers, not connected to genesis, persisted with a score that we trust.
However, previously, when we set persist_checkpoint_header it did not make that checkpoint header the canonical head.
The feature is used for checkpointed beam sync (https://github.com/ethereum/trinity/pull/921)
and in a previous version of that PR it didn't mattered if that API call would make the checkpoint header the new canonical head or not.
The reason for that is because eventually a header on top would be persisted with persist_header_chain(headers, genesis_parent_hash) and then that would move the canonical head to the latest header.
But with a refactored https://github.com/ethereum/trinity/pull/921 this turned out to be a problem because we now assume that calling persist_checkpoint_header would also make that checkpoint header the new canonical head (Which strikes me as a logical thing).
How was it fixed?
This makes the persisted header the new canonical head without any further checking. I'm not entirely sure but I can't think of a case where we would not want to make that checkpoint header the new canonical head.
To-Do

[x] Clean up commit history

[x] Add entry to the release notes

Cute Animal Picture

@carver Good point! I have some additional questions here. I created an issue to not hold back the release + Trinity PR. https://github.com/ethereum/py-evm/issues/1833


Confusion with daily versions

Looking at how the Nextcloud version can be specified, something's a bit off.
The main issue is when choosing the daily channel, namely this part:
daily: "nextcloud-{{nextcloud_branch}}-daily-{{nextcloud_version}}.zip"
Looking into the daily folder on the download server, this format doesn't exist.
For daily stable releases it should be something like:
latest-stable{{ nextcloud_version_major }}.zip
And for bleeding edge daily is should be:
nextcloud-{{ nextcloud_version_major }}-daily-{{ ansible_date_time.date }}.zip
I've been working on a PR for this, but I'm not sure how best to tackle it. Main issue is the date for the bleeding edge daily archives, as it's possible that the date is 1 day behind (depending on timezone), resulting in an invalid URL. For that, I was thinking of a fallback system that would go back 1 day in case the archive doesn't exist.
Other option would be to put the date in a variable, allowing to choose an specific daily version, but not sure how necessary that would be. (I guess it could be an optional thing, default being today's date, falling back to yesterday's).
What do you think?

Indeed, this uses case is badly documented/explained, and maybe I didn't used the most appropriates words.
In fact I assumed that users can write whatever they like in{{ nextcloud_version}}because it is free form, including any date when they want a specific daily instance.
I also wanted to use a minimum number of variables.
For the bleeding edge daily there is also the lastest master file pointing to the most recent daily if i'm not mistaken.
from my perspective, {{ nextcloud_branch }} == {{ nextcloud_version_major }}.
What I did might be a little lazy and may need some refactoring and/or to be more documented, I don't known how much you worked on this one.

I missed this point:

The main issue is when choosing the daily channel, namely this part:
daily: "nextcloud-{{nextcloud_branch}}-daily-{{nextcloud_version}}.zip"
Looking into the daily folder on the download server, this format doesn't exist.
For daily stable releases it should be something like:
latest-stable{{ nextcloud_version_major }}.zip

Both formats exists,
The daily subfolder is veeeeery long and contains a LOT of daily versions from nextcloud 9 to 11, and master
daily: "nextcloud-{{nextcloud_branch}}-daily-{{nextcloud_version}}.zip" works well by specifying a date in nextcloud_version.
But, as I said before, this might be not well explained or counter-intuitive.

I understand your point of not using many variables, which I totally agree with!
Yes, the daily list is really long... It's just a matter of using right variables that do the job :+1:
For example: I don't think branch == major version. Branch should be what it is now (stable, only one at the moment I think, maybe also master?).
Major version is basically: "{{nextcloud_version.split('.')[0]}}", so this is a temporary variable to make things easier, not a separate one for the user to define.

...works well by specifying a date in nextcloud_version.

Yes, but it's just not correct, as specifying just the date could be any version, 9, 10, 11, master, etc. So the nextcloud_branch part there should really be the nextcloud_major_version (or master).
I think it makes sense to think this through a bit more before making a big move on this.

I've just noticed that since 15th jan -last patch release- there is a latest archive in releases for each major version.
Therefore your point make even more sense to rework the version selection tasks.

Nextcloud has added different archives, even in releases channel,
there are now cases :

Releases:

lastest
lastest + major
full_version

prereleases:

full_version + RC(n)

Daily:

latest
latest + major_branches
major + daily (date)

So far I get:

Update Channel : Releases|prereleases|daily
major version: 9|10|11|...
full version number
RC number
date (for daily) version: YYYY-MM-DD
major branch (for daily) : master|stable9|stable10|...
"latest" keyword

Ideally I whould like to keep url Statically generated (as it is done by now in vars/main.yml), without using a set_fact task and conditions, and keep a minimal number of variables...
Any suggestion ? 😃

Pushed a new branch Rework_version_selection_#18
can you make a Rewiew of the new download variables ?

@aalaesar Would be great if you could make a PR to continue the discussion, then I can comment directly on code lines.

here you go: #26


Update authentication docs to add Direct Usage using sockets

The docs are lacking how to authenticate via sockets without using feathers-authentication-client.  Please see this thread inside the feathers-authentication repo for more info: https://github.com/feathersjs/feathers-authentication/issues/523#issuecomment-310512114
This PR adds some documentation on how to achieve Local and JWT authentication using plain sockets.

Ah, yes.  This was definitely needed.  Thanks!


Adding OnICEGatheringFinish.

Description
It would also be possible to swap the order of event execution but, but that require breaking change to the execution flow maybe breaking some apps. (First setting state completed then executing onLocalCandidateHdlr and then onStateChangeHdlr but that still breaking change to the execution flow).
This is way safer.
Reference issue
Fixes #1051.

Fantastic, thank you so much @Jorropo :)

@Sean-Der I saw that and yes that works but here is 3 problems with that :

That not Intuitive (ok that not objective at all but still that maters for me).
That unsafe, what told me


⚠ RPC server - Astar has degraded performance

In aaa61e3, RPC server - Astar ($ASTAR_RPC_URL) experienced degraded performance:

HTTP code: 503
Response time: 163 ms

Resolved: RPC server - Astar performance has improved in d0ca519 after 6 minutes.


Make compile with MSVC 2015

This patchset contains changes needed to compile unshield at Windows with MSVC 2015.
It is successfully compiles and works.
I tested linkage with OpenSSL and zlib.

Very nice, thank you! But I'm afraid the WIN32 checks in CMakeLists.txt (e.g. if(WIN32)) and in the source code (e.g. #if defined(_WIN32) || defined(_WIN64) will break MinGW support.... ping @xantares !

yep, these flags you set in CMakeLists are msvc specific, so it should be if(MSVC), then the *printf fixes are not needed with mingw to you should check that "MINGW32" is not defined

Thanks a lot @xantares for your quick and concise response. @a-ilin would you update the PR so that it builds on both MinGW and MSVC?

Thanks for the notices! I made the changes, however I have no possibility to check the build against MinGW.

@xantares Are you able to test this PR with MinGW?

Yes it's building fine.
But if (WIN32 AND MSVC) is redundant, do just if (MSVC)
On a side note you also would make sure these warnings are fixed:
/home/xantares/projects/unshield/src/unshield.c: In function ‘make_sure_directory_exists’:
/home/xantares/projects/unshield/src/unshield.c:104:13: warning: implicit declaration of function ‘_mkdir’ [-Wimplicit-function-declaration]
         if (_mkdir(current) < 0)
             ^~~~~~
/home/xantares/projects/unshield/src/unshield.c: In function ‘extract_file’:
/home/xantares/projects/unshield/src/unshield.c:457:1: warning: label ‘exit’ defined but not used [-Wunused-label]
 exit:
 ^~~~
/home/xantares/projects/unshield/src/unshield.c: In function ‘list_files_helper’:
/home/xantares/projects/unshield/src/unshield.c:610:18: warning: unknown conversion type character ‘z’ in format [-Wformat=]
       printf(" %8" SIZE_FORMAT "  %s%s\n",
                  ^
/home/xantares/projects/unshield/src/unshield.c:610:14: warning: format ‘%s’ expects argument of type ‘char *’, but argument 2 has type ‘size_t {aka unsigned int}’ [-Wformat=]
       printf(" %8" SIZE_FORMAT "  %s%s\n",
              ^~~~~
/home/xantares/projects/unshield/src/unshield.c:610:14: warning: too many arguments for format [-Wformat-extra-args]
/home/xantares/projects/unshield/src/unshield.c: In function ‘extract_file’:
/home/xantares/projects/unshield/src/unshield.c:458:6: warning: ‘success’ may be used uninitialized in this function [-Wmaybe-uninitialized]
   if (!success)
      ^

Thanks @xantares for your help! @a-ilin do you see the same warnings?
By the way, this is incorrect even if it gives the expected output :-)
#ifndef __PRETTY_FUNCTION__
#define __PRETTY_FUNCTION__ __FUNCTION__
#endif

As __PRETTY_FUNCTION__ is undefined outside of a function, it will always be redefined to FUNCTION, but that's not a problem as PRETTY_FUNCTION seems to be the same as FUNCTION in C. Maybe I used it of habit from of my C++ days... :-)
So maybe I should replace all PRETTY_FUNCTION with FUNCTION in a separate patch?

Thanks for replies!
@twogood, macro PRETTY_FUNCTION is undefined at MSVC so using it produces a corresponding error. That define fixes it.

@a-ilin : You can remove the __PRETTY_FUNCTION__ stuff from lib/internal.h in your PR now! :-)

Ping @a-ilin :-)

Thanks, @twogood!
Unfortunately I'm quite busy right now, so I will fix the patchset in a couple of weeks.

Rebase and merge?

New PR: #81


Can you use Flow with Node.js scripts/apps?

I just heard about Flow and am trying to wrap my head around it. What considerations would I need to take differently using this for a Node app, rather than browser code?

Node should work for node apps as well as browser apps (we assume the node/npm name resolution algorithm for modules by default).
We ship interface definition files for both web APIs and Node modules with Flow. It's worth noting that  these interface definitions have been filled out for the most part, but may not be fully complete. If you do run into some part of the interface definitions that we haven't built yet hopefully it should be easy to send that in as a PR: https://github.com/facebook/flow/blob/master/lib/node.js

Feel free to re-open this if you have more questions!
(or hop in #flowtype on irc.freenode.net if you want chat more directly)


Model Status Updates

This change is

Reviewed 2 of 2 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved, some commit checks failed.

Comments from Reviewable


Update Common.sol

a more optimized solution to get the most significant bit

just clocked that this returns the index of the msb not the msb itself, should have gone through it


Modules circle on overview page too small

Version : 1.0.14
First, thank you for this tool, it is really useful.
The issue I am having is that the circle in the overview page is too small, around a quarter of its normal size.
It wouldn't be such a problem if is it wasn't for the text overlapping the inner most circles-dependencies.
For the record, I am using webpack through Karma, but I don't think it is relevent for this issue.

Hello. I am back to work and even though I cannot give you the full stats file because I have no idea if the infos in there can be considered sensitive, I can give you the screenshot.
The stats file, without the modules, starts like this:
[
  {
    "timeStamp":<PHONE_NUMBER>648,
    "time": 16871,
    "hash": "e4050dbc12d422eaa5af",
    "errors": [],
    "size": 4227134,
    "assets": [
      {
        "name": "spec.js",
        "chunks": [
          0
        ],
        "size": 4227134,
        "miniSize": 1402957,
        "minified": false
      }
    ],
    "chunks": [
      {
        "size": 1763271,
        "files": [
          "spec.js"
        ],
        "modules": [

Also I do not know if this is relevent, but I do have a module which name starts with multi  followed by multiple relatives paths of JS files separated by spaces.
Here are fullsize screenshots from Chrome. As you can see, I can only hover the outside-most circles, as the text will overlap and it will lose the cursor hovering.


fixed Markdown tables

A few Markdown tables were broken because they only have 3 table headings but 4 columns have been defined.
Screenshots
Before my fix

With my fix

Coverage remained the same at 96.019% when pulling de4744cfd7f786717579e93e4f0baa58a26bf833 on bennyn:master into 269a9dd15f5956bed65f5d353cd7277b2f6d8493 on Vincit:master.

@bennyn Thanks! The documentation site seems to show these correctly despite the error.


Gradle integration

Is your feature request related to a problem? Please describe.
Gradle is an official option for JHipster, it must be implemented just as Maven is.
Describe the solution you'd like
Being able to build a generated application with Gradle.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
See #23

Can you please add bug bounty labels and amounts to this project and give me permission to assign them? I want to spend Okta's donations! 😀

Is this one resolved with the merged commits? Or is it something that needs more work?

Fixed via #62


Raw json response

I am unsure if this is somehow possible or if this is a feature request.
It would be wonderful if there was a simple way to return a JsonResponse from a controller manually. I can work around this, but it would be a nice feature :)
Something like
public function show($id)
{
   return Inertia::json(User::find($id));
}

I am curious, why do you want this? The whole idea behind Inertia is that it detects the request type (regular HTML request vs Inertia request), and then returns the proper response accordingly. Why would you want to always for a JSON repsonse?

See that is a good question. I take a bit of liberties with inertia to make the user experience nicer.
Let me explain the setup:
The page is a list of statistics for different projects. It only shows those you want to see. In the corner there is a select, which lets you add new projects to the list by selecting them. Now I could reload all statistics on the page when a new element is added, but I would rather just send a request to an endpoint to get data for just the added project and manually add it to the list with react. For now i just let inertia reload the page
If I had such an endpoint it would never have a page affiliated with it. It is just a classic show route that returns a JSON response.
I am well aware that the point of inertia is to have a more classic build website that just functions as a SPA, but I am mixing it to get the best of both worlds :)

@Sinnbeck I apologize if I'm misinterpreting your use-case, but wouldn't this be solved pretty easily with Partial Reloads?

@Juhlinus The issue would be that my index method does not have a 'prop' for a single project's data, only for all subscribed projects. Currently I am indeed using a partial reload, to reload the stats for all projects (but there is a difference in having to reload 20 projects or 1)

@Juhlinus Another use case could maybe be polling? I had such a use case and had to return raw json.

Yeah, polling would be a solution, I suppose.
While we're now talking more in lines of solving your issue instead of addressing your question, I'd still like to give my two cents: You could structure your statistics response in such a way that each project is keyed by it's id, and then on selection change, or on an interval, make a call to a Sanctum/Passport-configured JSON API endpoint (such as /api/v1/projects/{id}/statistics/summary) and hot-swap the partial data in the component.
Alternatively, you could go the Laravel Echo-way (read: websockets-way) and do that instead, but I think polling might be easier :)
Good luck!

I have a question.vue component which shows the question and it's answers. Initially the answers are empty. I fetch the paginated answers in created hook because I don't want to load all answers related to the question. I don't know how to handle it with inertia.
question.vue
 <template>
     //....do some stuff

{{setCountAnswer}}

        <answer v-for="(answer) in answers" :key="answer.id" :answer="answer"></answer>

  </template>

`
In AnswersController.php
  public function index(Question $question) {
       $answers = $question->answers()->with('user')->simplePaginate(4);
       //how to handle it with inertia
      return response()->json('answers' => $answers);
  }

I think this would be nice to have. We use inertia because we are developing SPAs, sometimes we dont want a full component render, we are just making (for example) a multi-step form, in which, this would fit very well.

I had a requirement for this in my project. To get the page data as JSON I created the following function which I consumed via a hook.
The key parts are the headers which trigger a JSON response as opposed to a HTML response.
const { version } = usePage();

async function fetchInertiaPageJson(
    url: string
): Promise<AxiosResponse<Page<any>>> {
    return axios(url, {
        headers: {
            "X-Inertia": true,
            "X-Inertia-Version": version,
        },
    });
}

@reinink Use case could maybe be for example saving images of content by using tiptap headless text editor.
i need endpoint that response json final url of image to content to save in img src tag to show them

Thanks to @BenSampo I finally got it. idk what sort of voodoo magic he's talking about but for those curious:
PHP
namespace App\Http\Controllers;

use Inertia\Inertia;
use App\Models\User;
use Illuminate\Http\Request;

class UserController extends Controller
{
    /**
     * Display a listing of the resource.
     *
     * @return \Illuminate\Http\Response
     */
    public function index(Request $request)
    {
        return Inertia::render('User/index', [
            'users' => User::all()
        ]);
    }
}

Vue

Honestly, we just need $inertia.reload to be a promise, or in some way other than a bunch of callbacks. Not a fan.
Ben was just missing X-Inertia-Partial-Component and X-Inertia-Partial-Data.

I made this wrapper around form helper to work with JSON calls:
https://gist.github.com/mohitmamoria/91da6f30d9610b211248225c9e52bebe

I made this wrapper around form helper to work with JSON calls:
https://gist.github.com/mohitmamoria/91da6f30d9610b211248225c9e52bebe

Just in time.


nixos/qemu-vm: allow custom partition- and filesystem layouts

Description of changes
Related issue: #177963
This PR adds a new option to be used in test- and build-vm virtual machines to override the default filesystem layout (to some extent). It's useful for testing on "unusual" configurations such as an ephemeral rootfs (AKA "erase your darlings" setup).
Things done

Built on platform(s)

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

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

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

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

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

[ ] Fits CONTRIBUTING.md.

I've also manually checked that the two tests work in a VM built by nixos-rebuild build-vm.

@ofborg test installer.simple
@ofborg test simple
@ofborg test non-default-filesystem
@ofborg test swap-partition

Cool, thanks a lot for your help! One last question: Should this be backported? (I'm not familiar with when things are usually backported.)

Should this be backported?

Maybe. It's mostly an addition, but it'd wait a bit to make sure it doesn't cause any problems.


Add Spanish as a supported language

Adding spanish es-MX as a supported language in Arcade

@jwunderl I'll get to translating some of those to increase the % before merging this

#4381 outlines the percentages but it's been substantially improved today to a state which should be good enough.

"es-mx" not showing in up 1.7.21. Por que no?
RE: #3603

ah oh no! this change didn't get cherry-picked to stable1.7, i'll port it over and we can do a point fix either this week or next depending on how many people are around/how urgent this is haha


Fix doxygen and quickbook warnings

This PR fixes a lot of small doxygen and quickbook warnings

needs a rebase to develop

needs a rebase to develop

Done


missing token error  when we try to call login api

When we try to do passport js authentication to login to freecodecamp , we are not able to do any api calls because of error: accesstoken is required, how can we set token without login . Is this something related to the loopback or passport js. Can you help?

What are you trying to achieve? Explain detailed steps on how and where are you getting this? The more context you give us the better.
If this is just an general query about freeCodeCamp's codebase and the stack then, Contributors Chat room or the Forum is a better place to ask question than this issue tracker.


Not going to "my subscriptions"

Basically when I go to youtube through a new tab then it redirects me, but after i watch a video and click that "YouTube" button in the top left corner it doesnt redirect me.. never..

Hi, I would suspect that YouTube is changing the layout of some of their pages and as such my extension no longer points to the correct icon to change the link. This has not taken effect on my youtube so I cannot test locally. If you could provide some information so that I can proactively change my extension to help others in this situation I would be very appreciative.
Please get the site to a state where clicking the button does not take you to the expected page. In chrome:

Right Click the icon
Inspect Element
Developer tools window should appear.
Hover over the code in this window and you will notice things become highlighted on the main page.
Move up the code, hovering over the items, until the entire of the the top bar is highlighted (like this

)
Right click that item
Edit as html
Copy the entire code in the box that shows up.
Paste as a comment here.

I have received a copy of the new YouTube homepage from another use and the icon in the top left has not changed. My extension should still be able to alter the destination of this link so I am unsure why you are experiencing this issue. Unfortunately If you cannot send me a copy of the page I will be unable to fix the issue. Please take a look at capturing all of a page to easily send me a copy of your webpage. Thanks.

Sorry, still new to this git thing and haven't checked my email. Here is the page where clicking the home button does not redirect me.
http://www36.zippyshare.com/v/BYPY2snr/file.html

No worries, thanks for taking the time to do that. It looks like the button is the same as usual, I suspect that this will be due to YouTube loading a new page (video) and my extension not realising the page has changed. Not sure how I can fix this at the moment but i am looking into it! There should be a new release sometime soon and hopefully I will have this fixed for then.

Hi again, would you be able to install the attached testing build in chrome using these instructions and check if a popup box displays when you re-create the issue. Thanks.

Closing as no response for a while & likely fixed here 22fb0c7738217dd449d12dd5a7a749b560926a4b


Regarding code for training the model

Hi @ducdauge,
I was going through the Multi-task BabyAI repo and was wondering if you could please share the code which you used for training the model. That would really help me with my experiments. Please let me know if this possible.
Thanks

Hi Valay, thanks for your interest! I have sent you a copy of the code by email.


prohibited-package gives false warnings in org.eolang.string-test

In a unit test for org.eolang.string:
+package org.eolang
+tests

[] > foo
  true > @

I'm getting this warning:
org/eolang/string-tests.eo Package org.eolang is reserved for internal object only (prohibited-package)

I believe, we must avoid this message if +tests is present.

@yegor256 assign me please

@h1alexbel go ahead


Send \r\n instead of \n

Send \r\n after commands. We currently send \n but the spec says \r\n. And microsoft doesn't like \n...
We should also strip the \r\n from the commands we receive instead of only a \n.

Updating parser for it in dc5cdda. Please check if I'm not forgetting something :)


How do I send an email with attachment?

I'm trying to send an email with attachment but it's not working.
In the form, I used:
<%= simple_form_for @comprovante, html: { multipart: true }, defaults: { wrapper_html: { class: 'form-group' }, input_html: { class: 'form-control' } } do |f| %>
 <%= f.file_field :file %>

<% end %>
I receive the email but there is nothing attached.
Email`s Body
Name: John
Email<EMAIL_ADDRESS>Message: Testing email
File: #ActionDispatch::Http::UploadedFile:0x00007fc9094557c0

Please use StackOverflow for questions/help, where a wider community will be able to help you. We reserve the issues tracker for issues only.


[FR] - Allow usage of delegate keys for --required-signers

Internal
Area
Other   Any other topic (Delegation, Ranking, ...).
Describe the feature you'd like
Suggested by @CarlosLopezDeLara.
The new SPO polls are signed by delegate keys, and it would be useful to have a possibility to include delegate keys in required signers, for easier verification of poll validity by SPOs.
Currently the GenesisDelegateExtendedSigningKey and GenesisDelegateSigningKey are not expected types by --required-signers:

Command failed: transaction build  Error: /home/carlos/cluster/bft0/shelley.000.skey: TextEnvelope type error:  Expected one of: PaymentSigningKeyShelley_ed25519, PaymentExtendedSigningKeyShelley_ed25519_bip32, StakePoolSigningKey_ed25519 Actual: GenesisDelegateExtendedSigningKey_ed25519_bip32

Implemented on master by https://github.com/input-output-hk/cardano-node/pull/5189


Code checking and running

It would be nice if all the code mentioned in the docs is actually compiled, so that errors can be caught, and migration to new versions is easier. This would also help support multiple GHC versions: one could recompile with a different version, and quickly find out what breaks.
Tintin is a tool that does this, so it's clearly possible, however support for GHC-9 seemed to be lacking.
More ambitiously, it would also be nice to be able to run any of the code without leaving the site. This is maybe more possible with the advent of JS and WASM backends.

OK, select code now runs using literate haskell.

But only one file is checked. I need a set up where a whole bunch of the docs are compiled.


Add virtual audio mixer

Re: https://github.com/ppy/osu/pull/16771#discussion_r799795987
Context: The final goal is to expose audio mixers to mods so that they can apply audio filters in song select. After some discussion (linked above), the agreed-upon method is to include an AudioMixer parameter in IApplicableToTrack.ApplyToTrack method. Since this method is sometimes used in situations where the mixer is irreverent (e.g. in diff calc), this PR creates a dummy audio mixer that can be used in those cases.
Don't think tests are needed for this, since the only job of this dummy mixer is to carry the Effects list.

there's really not much to review here. i'm personally holding off to see if there's consensus on even adding this and the PR that depends on this in the first place.

really weird to leave a "request changes" review that doesn't request any actual changes.
also not seeing what the nested mixer thing has to do with this. if that gets merged (which is unclear if it will, since it's been kind of hanging there for a while), virtual mixers could just not support adding children, no?

also not seeing what the nested mixer thing has to do with this. if that gets merged (which is unclear if it will, since it's been kind of hanging there for a while), virtual mixers could just not support adding children, no?

Dunno, but fair point about the direction in #4915 not being set in stone yet I suppose.

I'm confused as to where this is actually used. I can't see any usage of it in any of the osu-side PRs, nor in this PR framework side.

This is related to https://github.com/ppy/osu/pull/16771. The virtual audio mixer would allow the whole IApplicableToTrackMixer interface to be removed. Instead, the audio mixer will be provided to mods via IApplicableToTrack.ApplyToTrack(Track, IAudioMixer). This is not implemented in that osu-side PR yet because I'm not sure if this virtual mixer approach is acceptable.

Do you happen to have a diff showing how you'd see that working instead? I think that's the minimum required to move forward with this (we usually don't add new classes in the framework without accompanying usage and tests, so I'd see that as being added here if this was a decided direction).

I believe it would pretty much amount to using the virtual mixer in this diffcalc code path (presuming that the mixer gets added to IApplicableToTrack as a second param).

Here's how https://github.com/ppy/osu/pull/16771 can be changed to make use of the virtual mixer: https://github.com/ppy/osu/commit/c500bccd2e9408359c2183b2fd0be2bd4b461dda
Full diff with master (along with basic usage of the provided audio mixer in muted mod): https://github.com/ppy/osu/compare/master...hlysine:use-virtual-audio-mixer

Are you sure that diff even works? The fact you are making a local mixer for the track seems to imply that all existing filters which are being applied to AudioManager.TrackMixer would no longer affect the game track (which I believe would require #4915 to resolve if that's the correct direction even).


fix: Build Node.js bindings from source on CI

Description of change
Forces CI to build the Node.js bindings from source. Currently, there are cases where it may use the prebuilt binaries instead.
Links to any relevant issues
Fixes #1614
Type of change

Bug fix (a non-breaking change which fixes an issue)

How the change has been tested
Describe the tests that you ran to verify your changes.
Make sure to provide instructions for the maintainer as well as any relevant configurations.
Change checklist

[x] I have followed the contribution guidelines for this project
[x] I have performed a self-review of my own code
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have checked that new and existing unit tests pass locally with my changes

@Tuditi yep! Just note that this is specific to prebuild-install so it may not work for packages that install prebuilt binaries another way


Bump platform curl from 7.59.0 to 7.65.1.

High-level description
This bumps platform curl from 7.59.0 to 7.65.1.
Various components use curl/libcurl for some of the heavy lifting HTTP machinery within DC/OS, such as the Mesos fetcher.
Corresponding DC/OS tickets (required)
https://jira.mesosphere.com/browse/DCOS_OSS-5319

@mesosphere-mergebot label Ready For Review


Improving test coverage

Working on improving test coverage.

Pull Request Test Coverage Report for Build 6

0 of 0 (NaN%) changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.9%) to 77.778%

Totals

Change from base Build 4:
0.9%

Covered Lines:
1

Relevant Lines:
1

💛  - Coveralls


conf file flooding, e.g. /etc/doas.conf

Describe the bug
look attached file doas.conf, I've executed ansible script many times
doas.conf.txt
To Reproduce
Steps to reproduce the behavior:

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

Expected behavior
A clear and concise description of what you expected to happen.
Actual behaviour
List what actually happens when you run the aforementioned steps.
Error log
Add any error logs that might help with debugging the problem.
Additional context
Add any other context about the problem here.

The issue should be fixed now. Can you check if it works for you?

No longer needed in the newer setup.


AssertionError case sensitivity issue

The exception looks like this:
AssertionError: Last split: ab not in ~AB~~
The low-case word cant match the upper-case word in the sentence. Exceptions may be caused by the abbreviation(e.g. BGM) or single letter in the sentences.

To install directly from the git repository, use the following command:
pip install git+https://github.com/zh-plus/Open-Lyrics

I have made recent commits to remove the assertion and fix a case-related issue. But these commits are not currently available on PYPI. I plan to create a new release this week that includes additional improvements.

I also had this error and your latest update fixed it for me!

thank you for fixing the issues. One more thing, the latest version seems did not generate the lrc file but only json files, any idea?

thank you for fixing the issues, it is working now. One more thing, the latest version seems did not generate the lrc file but only json files, any idea?

Fixed by https://github.com/zh-plus/Open-Lyrics/commit/7aee5fedcf4ad9c6cd55807d120c23db2a4b0b24


new: Image mark

Usage
import bqplot
import ipywidgets as widgets

with open("sample.png", 'rb') as f:
    data = f.read()
image = widgets.Image(value=data)

scale_x = bqplot.LinearScale(min=-1, max=2)
scale_y = bqplot.LinearScale(min=-0.5, max=2)
image_mark = bqplot.Image(image=image, scales={'x': scale_x, 'y': scale_y})
fig = bqplot.Figure(padding_x=0, padding_y=0)
fig.marks = [image_mark]
fig.axes = [bqplot.Axis(scale=scale_x), bqplot.Axis(scale=scale_y, orientation='vertical')]
fig

It uses the widget.Image widget. Maybe it doesn't do all the things, like colorscales and all, but I think this makes a lot possible already.

@maartenbreddels Awesome

@maartenbreddels Somehow I cannot get the Image mark to work with your example above. I can get ipywidgets.Image to display fine in the output area but not on a Figure.

Are you using ipywidgets7?

Yeah I have ipywidgets 7.0.0b1, widgetsnbextension 3.0.0b1

Can you show me some error msg'es from the javascript console?

I'm curious - why use the ipywidgets image here instead of just having the image source directly? Does setting image layout attributes mess this up?

Well, i'm not sure actually what would be the best strategy, since it would be useful to have everything that ipywidgets' Image can have, especially after https://github.com/jupyter-widgets/ipywidgets/pull/1676 gets in. So either we copy all of that, or none of that and reuse Image. Other ideas are welcome.

@ssunkara1 Maybe we can get this in by PyData NYC?

@maartenbreddels looks good to me. Any idea why travis is failing?

We have not released a version with that feature yet. This feature has just been merged recently.

Yes I built from source, followed the instructions for devs on the readme

I see this wants a bytes string, whats the easiest way to get this to work with an image in the form of  a numpy array?


No images in docker images after snap install dive

I was trying to get a better understanding of what was taking space in a local image, so I ran docker inspect ... just before installing dive through snap, and after doing snap install dive my images in docker images were gone.
The snap install dive downloaded something and then entered some "configuration stage" (don't recall what it said, but the terminal replaced the line with dive 0.12.0 from Guillaume Belanger (gruyaume) installed when it completed) that took quite a long time. I had 30-40 images, so the time in the "configuration stage" is on par with how long it would take to delete the images with docker system prune.
I do not have Docker installed through snap, I'm using the Docker deb packages from Docker.com on Ubuntu 24.04, currently with Docker 27.1.1. I also only have the default Docker context.
$ snap info dive
name:      dive
summary:   A tool for exploring each layer in a docker image
publisher: Guillaume Belanger (gruyaume)
store-url: https://snapcraft.io/dive
license:   MIT
description: |
  Dive is a tool for exploring a docker image, layer contents, and
  discovering ways to shrink the size of your Docker/OCI image. To learn more
  about dive, see https://github.com/wagoodman/dive.

  **Usage**

  Install the Docker and Dive snaps and connect them:

     sudo snap install docker
     sudo snap install dive
     sudo snap connect dive:docker-executables docker:docker-executables
     sudo snap connect dive:docker-daemon docker:docker-daemon

  Run `dive`:

     dive ubuntu:latest
snap-id: MudyYZqdJlVtqfObbi6wN5bOzZM7JKft
channels:
  latest/stable:    0.12.0 2024-03-03 (20) 7MB -
  latest/candidate: 0.12.0 2024-03-03 (20) 7MB -
  latest/beta:      0.12.0 2024-03-03 (20) 7MB -
  latest/edge:      0.12.0 2024-03-10 (26) 7MB -

It seems @RafaelWO had the same problem a couple of weeks ago:
Originally posted by @RafaelWO in https://github.com/wagoodman/dive/issues/542#issuecomment-2257812333

Hi Team, Same Issue here. My Images got deleted after install drive through snap

I started getting permission denied errors when doing docker compose down. Turns out that is caused by AppArmor. After some more digging, it turns out I had somehow gotten docker through snap, which mixed with the Debian docker packages from Docker.com that I want to use:
# docker info
Client: Docker Engine - Community
 Version:    27.1.2
 Context:    default
 ...
Server:
 Containers: 6
  Running: 5
  Paused: 0
  Stopped: 1
 Images: 16
 Server Version: 24.0.5                <<--------------------- THIS
 ...
# snap list | grep docker
docker                     24.0.5           2915   latest/stable    canonical**     -

Removing the snap-installed docker killed my running containers, and then ran tar+gzip for half an hour:
# snap remove docker
2024-08-19T14:22:10+02:00 INFO Waiting for "snap.docker.dockerd.service" to stop.
Save data of snap "docker" in automatic snapshot set 3

until it finally said docker removed, so I guess I should have run docker system prune -af --volumes to remove my docker images+cache+volumes before asking snap to remove docker. While snap was running tar+gzip, docker info looked like this:
Client: Docker Engine - Community
 Version:    27.1.2
 Context:    default
 Debug Mode: false
 Plugins:
  buildx: Docker Buildx (Docker Inc.)
    Version:  v0.16.2
    Path:     /usr/libexec/docker/cli-plugins/docker-buildx
  compose: Docker Compose (Docker Inc.)
    Version:  v2.29.1
    Path:     /usr/libexec/docker/cli-plugins/docker-compose

Server:
ERROR: Cannot connect to the Docker daemon at unix:///var/run/docker.sock. Is the docker daemon running?
errors pretty printing info

So suddenly getting a new containerd when doing snap install dive explains why the images disappeared. It would be nice with a big warning both in the README and in the dive-snap-package that it's not "compatible" with the upstream Docker packages.

it's a disaster without any warning. I got traped too. The snap approach is not tested and should get removed from the README.md.

+1 how to push this issue more? It's not self explanatory that it requires docker installed via snap (who does this anyway?) and it just install it without asking (Or did I miss this? I did not install it with 'snap install docker').

Same issue, using snap install and it ruined my life 👎👎👎👎👎

I installed Dive (likely with sudo snap install), and afterward, my containers disappeared. I rebuilt them the next day, but then ran into numerous Docker issues. Even with help from O1 and claude3.5sonnet for an entire day, I couldn’t resolve the problems (didn't connect the dots to dive and find this thread) and ended up reinstalling Linux.
Here’s the error I was getting:
Error response from daemon: failed to create task for container: failed to create shim task: OCI runtime create failed: runc create failed: unable to start container process: error during container init: error running hook #0: error running hook: exit status 1, stdout: , stderr: Auto-detected mode as 'legacy'
nvidia-container-cli: initialization error: load library failed: libnvidia-ml.so.1: cannot open shared object file: no such file or directory: unknown

I don’t know if Dive caused it, but it’s the only thing I know of that changed before things stopped working.


Possible mismatch of virials sign and magitude between ML-PACE LAMMPS and JuLIP.jl implementation?

Hello folks,
Just wanted to confirm about the magnitude virials for ACEpotentials calculated using
(i) ML-PACE implementation in LAMMPS  and (ii) JuLIP.jl:
(i) I have relaxed a structure using a fitted ACE model in LAMMPS (a big shoutout to @wcwitt 🎉 for their help ).
(ii) Read the potential into Julia and the final relaxed structure also, and evaluate the virials by: ACE1.virial(struct1). From my understanding of definition of virial, virial = -stress * volume.
stress from lammps:
Pyz =   1698.4627  MPa
virial from JuLIP:
virial(2,3) = 3.4385 eV
volume from JuLIP: 3243.5700963735135 ang3
stress(2,3) = - 0.00106009733 eV/ang3
1 eV/ang3 --><PHONE_NUMBER> MPa
stress(2,3) = -169.846317 MPa.
so the sign and magnitude seems off. I don't know if this will be a major factor in the results. Nonetheless, I want to bring this up.
For the energies, I get good matching:
LAMMPS: -1068.6558 eV
JuLIP.jl: -1068.6557962647173 eV
For the forces,
both codes give zero forces for all atoms. This is great.
CC: @cortner
Attached are the LAMMPS log file for step(1) and julia script, geometry, ACE model for step(2).
Archive.zip

You are using metal units in LAMMPS, so the unit for pressure (and therefore stress) is Bar, not MPa. I think that answers the magnitude. https://docs.lammps.org/units.html
I'm guessing the sign is just a difference in convention (internal vs applied stress).
Please double check all this and let me know if you agree.

 For some reason, VASP uses MPa, I clearly got mixed up with LAMMPS and VASP units for pressure. You are right that LAMMPS uses bar and 1 bar = 0.1 MPa. The magnitude checks out.
For the sign convention, LAMMPS does not get into the details on the sign in the docpage:
https://docs.lammps.org/compute_pressure.html
Only thing stated is that r.F is used for virial for molecular statics.
From ACE1 source code, I could only find this:
https://github.com/ACEsuit/ACE1.jl/blob/8ac52d2128241a01d8b9a036f41b1d5106cbeb07/src/pairpots/pair_basis.jl#L146
There is no apparent reason for the mismatch on the sign.
I will run some tests with simple bulk geometry with larger than equilibrium lattice constant to see what signs each code gives me. This will already give me the answer. 😃
Thank you again for your help on this. 🙏
We can close this issue too.


Update Links “tiktok-34”

Automatically generated by Netlify CMS

👷 Deploy Preview for distracted-hoover-7b89cc processing.
🔨 Explore the source changes: 12063b5ce135becfcbf2a0108f7233b3b7f7c53a
🔍 Inspect the deploy log: https://app.netlify.com/sites/distracted-hoover-7b89cc/deploys/6219ebbcc67df5000803aebe


Incorrect "uniq" result, or I did it wrong?

This three number should be equal in my head:
> clickhouse --client -q "select user from funnel" > users
> cat users | sort | uniq | wc -l
2000654

> clickhouse --client -q "select uniq(user) from funnel"
1991337

> clickhouse --client -q "select uniq(user) from (select user from funnel order by user)"
1991337

Also not right on primary key column:
> clickhouse --client -q "select ts from funnel" > ts
> cat ts | sort | uniq | wc -l
86200

> clickhouse --client -q "select uniq(ts) from funnel"
86234

Rows counts are matched:
> clickhouse --client -q "select count(ts) from funnel"
10003270
> wc -l ts
10003270 ts

Table schema:
CREATE TABLE IF NOT EXISTS funnel  (
	partk Date,
	ts    UInt32,
	user  UInt32,
	event UInt16,
	props String
) ENGINE = MergeTree(partk, (ts), 8192);

Some rows:
> clickhouse --client -q "select * from funnel limit 3"
2017-01-01<PHONE_NUMBER>	1015021	10007	{"price_all": 6000}
2017-01-01<PHONE_NUMBER>	1020295	10003	{"content": "watch"; "page_num": 4}
2017-01-01<PHONE_NUMBER>	1076833	10009	{"how": 0}

Already final optimized table to one partittion 201701 (data dir: 20170101_20170101_1_10_2)
Version: https://github.com/yandex/ClickHouse/tree/c99a9b1c551066fa1cda0a34768ea2520e19c9a2

Can somebody help me out?

uniqExact, my fault...


Incorrect path concatenation

I'm using these settings (version 0.11.0)
app.use((req, res, next) => {
        const dir = 'one'; // dynamic value
        return sassMiddleware({
                src: path.join(__dirname, '../src/assets/projects', dir)),
                dest: path.join(__dirname, '../public/projects', dir)),
                debug: true,
                outputStyle: 'compressed',
        })(req, res, next);
});

File structure:
├── src
│   └── assets
│       └── projects
│            └── one
│                └── css
│                    └── main.scss
│            └── two
│                └── ...
│            └── three
│                └── ....
├── public
│   └── projects
│       └── one
│            └── css
│                 └── main.scc
│       └── two
│            └── ...
│       └── three
│            └── ...

and in the logs I see that the path along which the script looks for scss files is created on correctly:
__dirname/src/assets/projects/one/projects/one/css/main.scss
in the source code i see this https://github.com/sass/node-sass-middleware/blob/master/middleware.js#L146
where src is the path from options.src = __dirname/src/assets/projects/one
and path is the path to the current scss file, for example, it equals /projects/one/css/main.scss
which, when concatenated, creates the wrong path

Given the structure you showed, the configuration is missing css directory. Please try:
- src: path.join(__dirname, '../src/assets/projects', dir)),
+ src: join(__dirname, '../src/assets/projects', dir, 'css'),
- dest: path.join(__dirname, '../public/projects', dir)),
+ dest: join(__dirname, '../public/projects', dir, 'css'),

This did not help, now in the logs I see this path: __dirname/src/assets/projects/one/css/projects/one/css/main.scss

This code (in /tmp/server/my-server.js) worked on my Mac with nsm v0.11:
var sassMiddleware = require('node-sass-middleware');
var app = require('express')();
var join = require('path').join;
var http = require('http');
var port = process.env.PORT || 8000;

const dir = 'one';

app.use(sassMiddleware({
  src: join(__dirname, '../src/assets/projects', dir, 'css'),
  dest: join(__dirname, '../public/projects', dir, 'css'),
  debug: true,
  outputStyle: 'compressed',
}));

http.createServer(app).listen(port);

when I navigate to http://localhost:8000/main.css in the browser, CSS is rendered and logs show:
  source: /tmp/src/assets/projects/one/css/main.scss
  dest: /tmp/public/projects/one/css/main.css
  read: /tmp/example/public/projects/one/css/main.css
  render: /tmp/example/src/assets/projects/one/css/main.scss


Add all torch-vision models to run via shark-inference.

The models are taken from here: https://pytorch.org/vision/stable/models.html

@powderluv PTAL.


AiPlayer Poses

Allows AiPlayer to use player poses and also fixes an issue with them not updating on the client.

Closing it as clone of reviewed PR version: #1530.


Datapack Management not possible with Current Symlink Implementation

Problem: The folder for datapacks resides in /world/datapacks.  Since the links created by with servers.<name>.symlinks results is defined to be the name of a variation and it is impossible to include special characters in the name of a derivation. My first immediate workaround trying create a symlink  in the entire /worlds folder being a read only symbolic link to the nix store, thus making it impossible for the server to generate the world.
Fix: It would be nice to have a system for directly managing datapacks, or a modification to the current method of defining symlinks to get this to work. :)

This might be solvable using this recent pull request: https://github.com/Infinidoge/nix-minecraft/pull/29 but based on my understanding of the code it only works with text files.

Hey, can you show an example? Does this not work for you?
{
  services.minecraft-servers.servers.foo = {
    symlinks = {
      "world/datapacks/fóóbãr.zip" = a-derivation;
    };
  };
}

My apologies, my previous config had mistakenly tried to apply
{
  services.minecraft-servers.servers.foo = {
    symlinks = {
      "world/datapacks/fóóbãr.zip".value = a-derivation;
    };
  };
}

With that change everything is working perfectly :) Thanks for the help!


Try to improve build times

See if using gradle-build-action makes a meaningful difference.

None of this seems to make a difference.


test refactor: use address logic from utils

Coverage increased (+3.8%) to 80.89% when pulling 2534dd64141641b7fe4f58dd8e2c2d90836b9a6c on debug-node-17-ssrf into 3a738ee4d270f8fcf092a5e50b7634020298aa94 on main.


Bump up default log level, key import/export logs

makes warn the default log level on notary CLI
increases the -v log level from error to info
bumps up key import/export failure logs from info to warn

cc @cyc115 @ecordell
Closes #1162, #1178
Ex: here's what an offline notary list looks like now if there's a cache with a valid timestamp:
🐳 $ ./notary -s https://notary.docker.io list docker.io/library/alpine
ERRO[0000] could not reach https://notary.docker.io: Get https://notary.docker.io/v2/: dial tcp: lookup notary.docker.io on [::1]:53: read udp [::1]:56941->[::1]:53: read: connection refused
WARN[0000] Error while downloading remote metadata, using cached timestamp - this might not be the latest version available remotely
NAME               DIGEST                                                              SIZE (BYTES)    ROLE
----               ------                                                              ------------    ----
2.6                9ace551613070689a12857d62c30ef0daa9a376107ec0fff0e34786cedb3399b    528             targets
2.7                9f08005dff552038f0ad2f46b8e65ff3d25641747d3912e3ea8da6785046561a    1374            targets
<..snip..>

LGTM


coreutils binary contains build path / build is not reproducible

At the moment, the coreutils binary contains several occurrences of the absolute path under which coreutils was built, which makes the build not reproducible (the binaries will be different depending on under which path they are built). Reproducibility is important in some industries ( see also https://github.com/rust-lang/rust/labels/A-reproducibility and https://github.com/rust-lang/rust/issues/111540 ) and also triggers a "buildpath" build error which I disabled in the yocto integration of coreutils as workaround.
For instance, in the Ubuntu package  rust-coreutils 0.0.24-2, the binary /usr/bin/coreutils contains occurences of absolute paths starting with "/usr/share/cargo/registry/coreutils-0.0.24-2/src/uucore/..."
I can also reproduce in current master branch:
~/dev/coreutils$ cargo build --release --features unix
~/dev/coreutils$ cat target/release/coreutils | grep -a "$(pwd)"
/home/ecordonnier/dev/coreutils/src/uu/truncate/src/truncate.rsSIZE is an integer with an optional prefix and optional unit.
    '%'  => round up to multiple ofError writing clap::Error/home/ecordonnier/dev/coreutils/src/uu/cp/src/cp.rs/home/ecordonnier/dev/coreutils/src/uu/pr/src/pr.rs/home/ecordonnier/dev/coreutils/src/uu/fmt/src/fmt.rs--adjustment/home/ecordonnier/dev/coreutils/src/uu/nice/src/nice.rsA command must be given with an adjustment.: warning: setpriority: " is not a valid number: getpriority: cannot stat attributes of  (multi-call binary)
�
 �
���(�)�*�+�,�-�.�/�(�)�*�+�,�-�.�/�h�i�j�k�l�m�n�o�h�i�j�k�l�m�n�o��������B�B����������B�B����BB��BB�������B�B���FFFIFLFFIFFLSTSTDFD5D;NFD=&&&&�R�R�RS>S/home/ecordonnier/.cargo/registry/src/index.crates.io-6f17d22bba15001f/num-prime-0.4.4/src/buffer.rs/home/ecordonnier/dev/coreutils/src/uu/unexpand/src/unexpand.rsSpecifierNotAtStartOfNumberSpecifierOnlyAllowedWithLastValueTabSizeCannotBeZeroTabSizeTooLargeTabSizesMustBeAscending/home/ecordonnier/dev/coreutils/src/uu/mv/src/mv.rsThe argument '<>...' requires at least 2 values, but only 1 was providedoptions --backup and --no-clobber are mutually exclusive/home/ecordonnier/dev/coreutils/src/uu/ptx/src/ptx.rsLineOutOfRangeLineOutOfRangeOnRepetitionMatchNotFoundMatchNotFoundOnRepetitionLineNumberSmallerThanPreviousInvalidPatternSuffixFormatIncorrectSuffixFormatTooManyPercentsNotRegularFile/home/ecordonnier/dev/coreutils/src/uu/fold/src/fold.rsillegal width value (elvishpowershellzsh/home/ecordonnier/dev/coreutils/src/uu/chgrp/src/chgrp.rs/home/ecordonnier/dev/coreutils/src/uu/touch/src/touch.rsmissing file operand
...

The string "SIZE is an integer with an optional prefix and optional unit." right after "/home/ecordonnier/dev/coreutils/src/uu/truncate/src/truncate.rs" comes from https://github.com/uutils/coreutils/blob/d1f253481172786b819044f3717de9517506e910/src/uu/truncate/truncate.md

There's this option that might fix your problem: https://doc.rust-lang.org/rustc/command-line-arguments.html#--remap-path-prefix-remap-source-names-in-output
I'm not sure whether this is something that we should fix necessarily if it can be configured. Let me know if this option works for you!

I tested this in .cargo/config.toml and it works for the X86 release build:
[build]
rustflags = ["--remap-path-prefix=/home/ecordonnier=foobar"]

However there are some instances left in the debug build., and it does not work completely for the yocto cross-build (there is one occurence left), so I'll try to debug this:
__gmon_start__^@_ITM_deregisterTMCloneTable^@_ITM_registerTMCloneTable^@__cxa_finalize^@__stdbuf^@_Unwind_Resume^@memcmp^@setvbuf^@fileno^@stdin^@stdout^@stderr^@memcpy^@_Unwind_GetTextRelBase^@_Unwind_GetDataRelBase^@close^@syscall^@munmap^@memmove^@bcmp^@__errno_location^@open64^@__tls_get_addr^@abort^@_Unwind_GetIP^@_Unwind_Backtrace^@getcwd^@strlen^@fstat64^@write^@lseek64^@writev^@read^@__xpg_strerror_r^@memset^@malloc^@posix_memalign^@free^@realpath^@pthread_setspecific^@pthread_getspecific^@realloc^@calloc^@readlink^@getenv^@exit^@getauxval^@sigaltstack^@mmap64^@__cxa_thread_atexit_impl^@statx^@_Unwind_GetLanguageSpecificData^@_Unwind_GetIPInfo^@_Unwind_GetRegionStart^@_Unwind_SetGR^@_Unwind_SetIP^@pthread_key_create^@pthread_key_delete^@dl_iterate_phdr^@_Unwind_RaiseException^@libgcc_s.so.1^@libc.so.6^@ld-linux-x86-64.so.2^@GLIBC_2.3^@GCC_4.2.0^@GCC_3.0^@GCC_3.3^@GLIBC_2.28^@GLIBC_2.16^@GLIBC_2.14^@GLIBC_2.33^@GLIBC_2.34^@GLIBC_2.3.4^@GLIBC_2.18^@GLIBC_2.2.5^@/home/ecordonnier/dev/poky/build/tmp/work/cortexa57-poky-linux/uutils-coreutils/0.0.27/recipe-sysroot-native/usr/lib:/home/ecordonnier/dev/poky/build/tmp/work/cortexa57-poky-linux/uutils-coreutils/0.0.27/recipe-sysroot-native/lib

I also tested "trim-paths" in .cargo/config.toml which is available only with cargo nightly at the moment, with the same result:
[unstable]
trim-paths = true

[profile.release]
trim-paths = ["diagnostics", "object"]

[profile.debug]
trim-paths = ["diagnostics", "object"]

Thanks for the hint. It looks like this is a generic rust issue not specific to uutils-coreutils.

I'm glad it works! We could enable this by default for release builds. You mention debug builds, why would you need the reproducibility there? And I wonder where the trouble with the cross-build is!

I mentioned debug builds because I was wondering if that's the reason there is one instance left in yocto (yocto generates debug symbols for all binaries and then split them to some extra debug symbol file). You are correct that in principle for the normal cargo debug binary,  reproducibility is not important.

The remaining occurence of the buildpath in the yocto build is caused by libstdbuf.rlib (from src/uu/stdbuf/src/libstdbuf). However I have not yet found a way to reproduce outside of yocto. When compiling uu_stdbuf directly in the uutils-coreutils repository this buildpath is not there. I'll debug further when I find some time. I also want to double-check "remap-debuginfo = true" as simpler alternative to "remap-path-prefix=XXX".


Can't use custom light shader if vertex shading is enabled

Godot version:
3.2.2rc1, GLES2 and GLES3
OS/device including version:
Windows 10
Issue description:
If you enable Force vertex shading in the editor, the light() function in custom shaders has no effect. I think this is because light() is called in the fragment shader for each light source and if you disable per pixel lighting this won't work of course.
Since it's nowhere mentioned that custom shading only works if per pixel shading is active this is either a bug or a documentation issue. I hope it's a bug tough and not by design 😄.
Steps to reproduce:

Enable Force vertex shading in the editor
Make a material with a custom shader and do e.g. DIFFUSE_LIGHT = vec3(1.0, 0.0, 0.0); in the light() function
Make a simple scene with a mesh and assign the material
Lighting did not change

Minimal reproduction project:
PerVertexShadingBug.zip

I think this feature needs to be implemented from scratch.

This was documented a long time ago by https://github.com/godotengine/godot-docs/pull/3685.
Since then, vertex shading was removed in 4.0 and readded in 4.4. The warning is still present in the spatial shaders page, so this does not need to be re-documented.


Configuration Consistency

Is your feature request related to a problem? Please describe.
Data Prepper configuration names inconsistently use snake_case or lowerCamelCase. Configuring Data Prepper is clearer when the naming convention is consistent.
Describe the solution you'd like
Most importantly Data Prepper should have a consistent naming convention for configuration names. The standard we are proposing is snake_case. This convention is used by most of the existing properties. Thus, users will have fewer properties to reconfigure.
Data Prepper will need to:

Have documentation clearly stating the standard.
Provide a migration for names which are not in compliance. The first part of this migration is adding snake_case properties for each camelCase property. The existing camelCase properties will remain as deprecated properties. The next major release of Data Prepper will remove the cameCase versions.

Tasks

[ ] #2184
[ ] #2202
[ ] #2203

As there are several different types of configurations, I created new issues to track them independently.


Allow sharing an advanced search via link

It would be great if it was possible to share an advanced search via link so others don't need to rebuild advanced search from scratch. Either copying the link from url bar manually or with a button that copies the link to clipboard would be nice.
On cacombos.com site for example, it is possible to search for a key and share the link because it contains the query param, like
https://cacombos.com/search?key=n78A4A2

I am not sure how this could like like in practice but perhaps it is possible to put the json body that's sent to the server in the url as a parameter. For LTE Combos 20-3C-1 as an example, something like:
https://uecapability.smartphonecombo.it/library/advancedsearch/?query={"criteriaList":[{"type":"combos","field":"LTE_COMBOS","comparator":"HAS_ANY","value":[{"type":"simple","dl":[{"type":"lteDl","band":20},{"type":"lteDl","band":3,"minBwClass":"C"},{"type":"lteDl","band":1}],"ul":[]}]}]}

@HandyMenny Sound like a good idea.
However, my question is: What would be a standard workflow to query using json in the browser? Because if I share the json query to a non-techsavvy user who wants to view available logs in the browser, how would they be able to check the results of a query in the browser?

What would be a standard workflow to query using json in the browser?

Open advanced search -> select json -> paste the query -> click submit
In any case, the purpose of this demo is more to show what the parser is capable of, than a "user friendly" tool.
Suggestions are welcome since it's something that I also use every day, however being user friendly isn't really the top priority


Adding Trapping Rainwater problem solution (python, cpp)

Please assign this issue to me under Hacktoberfest2022 @anomekumar08

@anomekumar08


added hass_is_global config parameter

This PR adds a config parameter hass_is_global (default False), which if set provides hass as a global variable in all pyscript contexts.
@raman325 and @dlashua - I invited you to the forked repository.  Please review and feel free to push any changes to craigbarratt/pyscript-hass-is-global, or just comment here.
This will resolve #46.

I just pushed the change I had suggested here: https://github.com/custom-components/pyscript/pull/42#issuecomment-713189490

Thanks!  Should that also be done in __init__.py (it's aways used cv.boolean)?

no cv.boolean can be used there because the backend is aware of the custom config validators that HA provides. The issue is that the serializer that delivers the schema to the frontend (for config and option flows) was not aware of the custom validator before I submitted that PR, so it doesn't know what to do and throws an error instead of showing a form.

Ok, thanks for the explanation.

I've loaded this branch and it seems to work fine with no issues. I haven't found anything interesting to do with hass just yet, but the things I expect to be there are there and are usable.

Thanks for the feedback and fixes.  I just released 0.32 (which has the HASS restart fix, config improvements from @raman325 and @state_trigger support for multiple arguments, lists and plain state variables to mean any).  I'm going to merge this and we have time to test and tweak if necessary.

excellent, I'll pull master now so it gets a good test run.