Strange behaviour with indentation

When running the following flux:
https://gitlab.com/oersi/oersi-marc/-/blob/2-createBasicTransformation/localTestWorkflow.flux?ref_type=heads
There are more indentations added to every new record in a file.
The first (0) has a normal indentation :
<marc:collection xmlns:marc="http://www.loc.gov/MARC21/slim" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:schemaLocation="http://www.loc.gov/MARC21/slim http://www.loc.gov/standards/marcxml/schema/MARC21slim.xsd">
	<marc:record>
		<marc:leader>00000cam a2200000  4500</marc:leader>
		<marc:controlfield tag="005">20240412102249.3</marc:controlfield>
		<marc:controlfield tag="007">cr</marc:controlfield>
		<marc:datafield tag="264" ind1="#" ind2="0">
		...
	</marc:record>

</marc:collection>

the second has:
<marc:collection xmlns:marc="http://www.loc.gov/MARC21/slim" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:schemaLocation="http://www.loc.gov/MARC21/slim http://www.loc.gov/standards/marcxml/schema/MARC21slim.xsd">
		<marc:record>
			<marc:leader>00000cam a2200000  4500</marc:leader>
			<marc:controlfield tag="005">20240412102249.4</marc:controlfield>
			<marc:controlfield tag="007">cr</marc:controlfield>
			<marc:controlfield tag="008">240228t||||||||xx#|||||o|||||||||||fin||</marc:controlfield>
			...
		</marc:record>

</marc:collection>

record 14(15) has a lot:
<?xml version="1.0" encoding="UTF-8"?>
<marc:collection xmlns:marc="http://www.loc.gov/MARC21/slim" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:schemaLocation="http://www.loc.gov/MARC21/slim http://www.loc.gov/standards/marcxml/schema/MARC21slim.xsd">
															<marc:record>
																<marc:leader>00000cam a2200000  4500</marc:leader>
																<marc:controlfield tag="005">20240412102249.6</marc:controlfield>
																<marc:controlfield tag="007">cr</marc:controlfield>
																<marc:controlfield tag="008">240228t||||||||xx#|||||o|||||||||||fin||</marc:controlfield>
																<marc:datafield tag="040" ind1="#" ind2="#">
																	<marc:subfield code="b">fin</marc:subfield>
....
															</marc:record>

</marc:collection>

Every record additional indentations are added.
I am not sure why.

This example cannot be run in the playground, but I have added an example for the runner here, that also shows the wrong indentation that it produces. This error seems to be specific for the combinantion batch-reset and  encode-marcxml, when I use encode-yaml or encode-xml the error is not happening.

@dr0i you wanted a playground example, see comment above this is not possible, but I added a small example in a repo and it shows the strange behaviour.


CryptoTokenExtension - is it necessary or should it be removed?

Is your feature request related to a problem? Please describe.
According to this info CryptoTokenExtension "IS a tracking tool" although actual details are not given and personally I can't find additional info about it.
Describe the solution you'd like
Consider if CryptoTokenExtension is actually needed and if it has anti-privacy implications, or if it is "bloatware" which can be removed.

I had a quick look at the source code. Not a JS expert but from my understanding it is used for hardware 2FA. It's main purpose is for identity verification with a USB hardware authentication device, e.g. Yubikey. In a sense it DOES track you because by definition for 2FA to work you need to use the same device to verify it is you every time.
Now I don't know whether remove it will be a good idea or not. It's basically a tradeoff: either you want to use a hardware authentication device which can track you, or you want the whole functionality gone at all.

I had a quick look at the source code. Not a JS expert but from my understanding it is used for hardware 2FA. It's main purpose is for identity verification with a USB hardware authentication device, e.g. Yubikey. In a sense it DOES track you because by definition for 2FA to work you need to use the same device to verify it is you every time.

Thanks for clarifying.
So if things are as you explained, we should probably consider that:

Not everyone uses such devices

We should not assume that ungoogled-chromium users (who are supposedly more privacy and security minded) would trust by default a proprietary non-RYF device, even less - being aware of things like BadUSB etc. So I am inclined to think that UC users would rather not need this extension.

Even if we suppose someone uses such device for some unfortunate reason they can simply be given the option to enable this extension instead of enforcing it for everyone without an option to disable it

Now I don't know whether remove it will be a good idea or not. It's basically a tradeoff: either you want to use a hardware authentication device which can track you, or you want the whole functionality gone at all.

Perhaps there is no need for a trade off. Perhaps it is best to be up to the user to turn it ON or OFF. By default any form of tracking should be OFF, just like any unnecessary functionality.

i second that

The last thing I can recall about CryptoTokenExtension is this: https://github.com/Eloston/ungoogled-chromium/issues/874#issuecomment-559604434. If what I said then still holds, then it isn't useful (especially with Webauthn available), but it doesn't need to be removed either.
If you guys want to patch it out, that's fine by me.

If you guys want to patch it out, that's fine by me.

Can't you add a flag for it?

@emanruse Correct me if I'm wrong, but I think CryptoTokenExtension is already broken because of domain substitution (based on what I said in the issue). We would need to fix that first to make a flag useful.

@Eloston unfortunately I can neither confirm or deny that because I have never used that such 2FA which it is supposed to assist.
But what I am thinking:
If it is "broken" (ungoogled) through domain substitution then it is already dysfunctional. Hence - it can be safely removed and a flag is not necessary at all.
IIUC "fixing" it would mean "googling" it again which would contradict the essence of the current project. Relying on a flag to keep the whole browser ungoogled (as it is expected to be) seems somewhat risky, even if it is "off by default".
Perhaps the best approach (in case the minority who uses 2FA insists on using it with ungoogled-chromium) would be to have the browser itself without the extension (clean and ungoogled) and the extension separate. Then those who want to use it can simply install it, accepting the risk of re-googling part of the browser for themselves only without that influencing anyone else who uses ungoogled-chromium.
What do you say?

If it is "broken" (ungoogled) through domain substitution then it is already dysfunctional. Hence - it can be safely removed and a flag is not necessary at all.

Yes, but leaving it broken is also less work for us too (doing nothing). However, if there are still privacy/security issues with this approach, then we should deal with them.

Perhaps the best approach (in case the minority who uses 2FA insists on using it with ungoogled-chromium) would be to have the browser itself without the extension (clean and ungoogled) and the extension separate. Then those who want to use it can simply install it, accepting the risk of re-googling part of the browser for themselves only without that influencing anyone else who uses ungoogled-chromium.

Unfortunately CryptoTokenExtension is embedded in the browser and can't be easily installed like a regular extension because it uses private extension APIs that aren't available to regular extensions. I don't know what security risks we may run into if we convert CryptoTokenExtension to be an external extension, but it would be a nice solution to this problem.

Of course it is easiest to do nothing but long term: keeping and rebuilding a piece of software which does nothing is a waste of time and power, especially considering the tremendous increase of build time in latest versions. I don't know how much exactly this particular extension adds to the overall build time but multiplied by the hundreds of users - perhaps another thing to consider.

Well either way, I think my time is better spent addressing other longer-term technological issues like automated testing. Not only is this issue not very interesting, I'm not convined many people are affected by this issue.
However, I still welcome others to fix a solution if they're interested.


Windows builds are failing

Not sure if this is a low hanging fruit to fix:
cl : Command line error D8021 : invalid numeric argument '/Wno-unused-function'
See: https://ci.appveyor.com/project/conda-forge/pyresample-feedstock/build/1.0.8/job/pp68nena62gt0whf

Fixed in #6


CLI execution - no documents exist

Hi
Its me again... :(
I am having problems when reading from the db. Let me explain what i am doing:
1 - I have a job that downloads daily the NVD database updates.
2 - I am copying this db file in my workspace and running this
3- I have another job that runs depcheck against this database.

dependency-check.sh --scan /jenkins/workspace/DCTest --project test --noupdate
[ERROR] No documents exist

4- Ok... I need to specify the database as an argument. Lets go for it

dependency-check.sh --scan /jenkins/workspace/DCTest --project test --noupdate -d /path/to/dc.h2.db
[ERROR] Unable to connect to the database

5 - According to https://github.com/jeremylong/DependencyCheck/issues/410, i need to send the connection string jdbc:h2:file:%s;FILE_LOCK=FS;AUTOCOMMIT=ON;

dependency-check.sh --scan /jenkins/workspace/DCTest --project test --noupdate -d /path/to/dc.h2.db --connectionString jdbc:h2:file:%s;FILE_LOCK=FS;AUTOCOMMIT=ON;
[ERROR] Unable to connect to the database

Any idea on how can i solve this?
Thanks

The issue appears to be your -d argument.  -d is "The location of the data directory used to store persistent data."  It looks like you are specifying the full file path - and in this will cause ODC to fail.

It solves the issue now. Thanks!


Logging

Add logging...

Populate handlers with informative log messages.
Allow for more customizable logging functionality (different levels of verbosity/specific options)


Cxf-soap: tests are not working in FIPS environment

fixes https://github.com/apache/camel-quarkus/issues/5979
@ppalaga I thought, that the regenaration of keystores + some additional logic handling FIPS in tests would be needed (similar to https://github.com/quarkiverse/quarkus-cxf/pull/1297 and https://github.com/quarkiverse/quarkus-cxf/pull/1174). With the current simple fix the test suite works in FIPS environment.

There are no tests verifying that non-FIPS configuration fails -> which is the reason of not requiring special handling in the  test, so it is OK
I'm not sure about the keystores though:

On one side - keystores can not be used "fully" to encrypt a message, because the proces should fail in FIPS
On the other side, it is not necessary to test such usecase, because it is covered by Quarkus-cxf, which is leveraged here.

WDYT?
a. Should we keep the fix this small? (I think so)
b. Should we consider making the tests more covering (to use keystores properly)? (I'm not sure of it)

Failure of CI (MongoDB Grouped and Debezium) is not caused by this change

@JiriOndrusek I assume this PR is still needed? If so, please rebase.


fixed issue with sessionId in fsa-pre-built

closes #75
Infinite get loop no longer happens on login.
Not sure wether sessionId needs to be something else but setting it to null seemed to fix all of my problems.

Alex,
This conflicts with the recent fix to sessions. I think you're working off of an older copy of FSG.


Half-Life: armhf update

I have only tested this on the Anbernic RG35XX H running MuOS since I don't own any other supported handhelds.

Hi @SiliconExarch i am opening a testing thread on the PortMaster discord so we can test a few other devices. On the surface it looks good though. 👍

Tested on:

[x] ArkOS RG353V
[x] AmberELEC 351V
[x] muOS RG35XX PLUS
[x] TrimUI Smart Pro


Tukan + Cicada (sidebar-hover-ipsec) and other fixes

@fichtner
@fabianfrz
@marjohn56
sidebar hover, ipsec and other important fixes

have to fix this too


Azure Iot Hub Sample Issue

Section link

Reason:
DataResidency is not available in 'centraluseuap' region.

Note:
1.I have used locations such as eastus, westus, japaneast, southafricanorth, but it also returns the error above.
2.If disable the DataResidency, it will be created successfully.
Suggestion:
Set property enableDataResidency to false.
@xirzec, @drwill-ms, @dw511214992 for notification.

Hi @zzhxiaofeng
This issue has been fixed.
Here is the package<EMAIL_ADDRESS>

@kazrael2119 Thank you, I will close the issue now


replace, test: Add unit test to replace with cap

One of the feature is replacing with the captured state from a capture
entry path. This change add support for it at the parser and create a
unit test for the resolver since resolver has the feature already.

/kind enhancement


19-07-2024 15:30:05 [ErrorReported] ERROR (ThrottledFolderBind): Exception calling "Bind" with "3" argument(s): "The request failed. The remote server returned an error: (403) Forbidden."

Despite adding the EWS/Exchange impersonation rights, I am getting below error continuously. Can anyone help find the missing piece in the puzzle?
19-07-2024 15:30:05 [ErrorReported] ERROR (ThrottledFolderBind): Exception calling "Bind" with "3" argument(s): "The request failed. The remote server returned an error: (403) Forbidden."
Failed to open source message store<EMAIL_ADDRESS>Please check that you have impersonation permissions

Hi Team, Thankfully The impersonation issue was resolved automatically. To my best guess, I just had to wait a bit longer. However now I am stuck with another issue. see below and please help resolve this if you will.
I see all success logs in Azure sign-in, however it keeps failing.


Remove the use of unique_ptr and simplify destructors

This PR removes the std::unique_ptr and res_type::destruct bits, and uses simple destructors for each Adbc structs.

:green_heart: :blue_heart: :purple_heart: :yellow_heart: :heart:


Roles can also be mixed in using 'is'.

The problem
The docs say a Role can be mixed into a class using "does" but fails to mention that "is" also works.
Solution provided
Update the docs to clarify how "is" also works to mix a Role into a class.

The semantics of is with a role are quite different from those offered by does. With is, a class is punned from the role, and then inherited from. Thus, there is no flattening composition, and none of the safeties it provides.

Thank you for the comment.  Relying heavily on your explanation I updated my PR to explain the difference a bit more.

s/none of the safeties C provides./none of the safeties which C provides./
??


Use a secret has a nixos option value

Hello,
I’d like to know if there is a way to use a nix sops secret as a value of a nixos option ? Some module don’t have things like « passwordFile » options but only « password ». In these kind of cases I never know what to do.

Possibly a duplicate of #159?

Agreed. This is a documented limitation of the design.


Update TROUBLESHOOT.md

add troubleshoot

Please use gerrit for the changes , see https://github.com/hyperledger/blockchain-explorer/blob/master/CONTRIBUTING.md thank you, if you need more assistance please let me know Nik
…
On Thu, Dec 13, 2018 at 6:24 AM sunyoubo @.***> wrote: add troubleshoot ------------------------------ You can view, comment on, or merge this pull request online at: #24 Commit Summary - Update TROUBLESHOOT.md File Changes - M TROUBLESHOOT.md https://github.com/hyperledger/blockchain-explorer/pull/24/files#diff-0 (52) Patch Links: - https://github.com/hyperledger/blockchain-explorer/pull/24.patch - https://github.com/hyperledger/blockchain-explorer/pull/24.diff — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#24>, or mute the thread https://github.com/notifications/unsubscribe-auth/AKjK1ZgXzKNdvyV8b1lfe92KiVYw-4SJks5u4jjogaJpZM4ZRb9V .
-- Nik Frunza

Please use gerrit for the changes , see https://github.com/hyperledger/blockchain-explorer/blob/master/CONTRIBUTING.md thank you, if you need more assistance please let me know Nik
…
On Thu, Dec 13, 2018 at 6:24 AM sunyoubo @.***> wrote: add troubleshoot ------------------------------ You can view, comment on, or merge this pull request online at: #24 Commit Summary - Update TROUBLESHOOT.md File Changes - M TROUBLESHOOT.md https://github.com/hyperledger/blockchain-explorer/pull/24/files#diff-0 (52) Patch Links: - https://github.com/hyperledger/blockchain-explorer/pull/24.patch - https://github.com/hyperledger/blockchain-explorer/pull/24.diff — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#24>, or mute the thread https://github.com/notifications/unsubscribe-auth/AKjK1ZgXzKNdvyV8b1lfe92KiVYw-4SJks5u4jjogaJpZM4ZRb9V .
-- Nik Frunza

thank you very much, I get it


un-html-escaping dblp author

Seems that the DB table for DBLP has xml entities in it.
ARXIVNG-1214

Pull Request Test Coverage Report for Build 277

26 of 26 (100.0%)  changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.05%) to 84.273%

Totals

Change from base Build 274:
0.05%

Covered Lines:
2706

Relevant Lines:
3211

💛  - Coveralls


Bluetooth: Controller: Missing LL_ENC_RSP after HCI LTK Negative Reply

Describe the bug
In the Encryption Start Procedure, for the case when the host does not provide an LTK, the controller is supposed to send an LL_ENC_RSP before sending LL_REJECT_IND or LL_REJECT_EXT_IND.
To Reproduce

Modify hci_core.c, le_ltk_request() to always call le_ltk_neg_reply():

//	if (bt_smp_request_ltk(conn, evt->rand, evt->ediv, ltk)) {
//		le_ltk_reply(handle, ltk);
//	} else {
		le_ltk_neg_reply(handle);
//	}

Build the peripheral_hr sample
Connect and try bonding, expect it to fail.
Observe the sniffer log.

Expected behavior
Send LL_ENC_RSP before LL_REJECT_IND. See the MSC in Bluetooth Core Specification v5.1, Vol 6, Part D, Section 6.7, or Part B Section <IP_ADDRESS>.

If encryption is not supported by the Link Layer of the slave, the Link Layer of
the slave shall send an LL_REJECT_IND or LL_REJECT_EXT_IND PDU with
the ErrorCode set to Unsupported Remote Feature / Unsupported LMP
Feature (0x1A).
…
The Link Layer of the slave shall then send an LL_ENC_RSP PDU. The Link
Layer of the slave shall then notify the Host with the Rand and EDIV fields.
Impact
Possible showstopper. Peer devices may cache that the controller does not support encryption, thereby leaving the link unencrypted.
Screenshots or console output

Environment (please complete the following information):
f79fbac101f8febb459acee577e9f6b76e00acc0

@cvinayak, FYI

@rugeGerritsen Please specify which controller you are using, legacy or split

Updated the bug description to state that this bug is present in 2.0.0, both the legacy and split controller

@rugeGerritsen Please try with CONFIG_BT_CTLR_FAST_ENC=y

Thanks @rugeGerritsen, this is indeed a bug. It will only be fixed in the split controller.

@rugeGerritsen Please try with CONFIG_BT_CTLR_FAST_ENC=y

The bug is not present when CONFIG_BT_CTLR_FAST_ENC=y. This is only enabled in the hci_uart example


admin/store/create GET POST handlers

The store/create GET and POST handlers need to be created for the admin user.
A skeleton for the GET handler is available here.  It should be relatively simple.  We may not really want to to do anything else with it until we figure out how we are doing the templating (#7).
The POST handler needs to do the following things:

authenticate the admin user

you can look at how the GET handler is doing authentication

receive the email address for the store user

parse this from one of the query params

create a new CompanyEmail row in the database

add a new class method to MonadKucipongDb

send the new store user a registration email

This is similar to the one we send the admin after adding a new admin.

redirect the admin to a success (or error) page

The login handler for admins is doing a redirect, so you can look at how they are doing it.

@arowM Feel free to work on either the GET or POST handler.

I'm on this.

@cdepillabout
What's the reason we should receive email address as query params rather than request body on POST handler?

@arowM I'm sorry, I should have clarified that.
Either query params or request body is OK.  In the API document I believe it is specified as coming inthe request body.  Which ever is easier for you.

@cdepillabout
Thank you!

@arowM
Would you be able to create a simple PR (or even just commit right to master) with a commit that creates the CompanyLoginToken table?  I imagine it would be almost identical to the AdminLoginToken table.
I am going to start #10, and I need the CompanyLoginToken model to actually be able to write the Kucipong.Handler.Store.login method.

@cdepillabout
OK, I'll be on this today.
What's the reason you separated company email table from company table?
Which is better to merge these tables or leave them as they are?

I found that all codes exclude src/Kucipong/Db/Models/EntityDefs.hs uses word "Store" for store user, but EntityDefs.hs calls it "Company".
I'll rewrite them with "Store".

@arowM

What's the reason you separated company email table from company table?

This was so that the Company table didn't have to use Maybes.
The Company and CompanyEmail table currently looks like this:
CompanyEmail
    email                   EmailAddress
    created                 CreatedTime
    updated                 UpdatedTime
    deleted                 DeletedTime Maybe

    Primary email

    deriving Eq
    deriving Show
    deriving Typeable

Company
    created                 CreatedTime
    updated                 UpdatedTime
    deleted                 DeletedTime Maybe
    emailAddr               CompanyEmailId
    name                    Text
    businessCategory        Text
    businessCategoryDetail  Text
    image                   Image
    salesPoint              Text
    address                 Text
    phoneNumber             Text
    businessHours           Text
    regularHoliday          Text
    url                     Text

    deriving Eq
    deriving Show
    deriving Typeable

However, if we combined the tables, it would look like this:
Company
    email                   EmailAddress
    created                 CreatedTime
    updated                 UpdatedTime
    deleted                 DeletedTime Maybe
    name                    Text Maybe
    businessCategory        Text Maybe
    businessCategoryDetail  Text Maybe
    image                   Image Maybe
    salesPoint              Text Maybe
    address                 Text Maybe
    phoneNumber             Text Maybe
    businessHours           Text Maybe
    regularHoliday          Text Maybe
    url                     Text Maybe

    Primary email

    deriving Eq
    deriving Show
    deriving Typeable

It is possible it would be annoying to have to handle all these Maybes when writing the handlers and html templates.  However, if you can think of a good way to deal with then when writing the handlers and templates, then it might be easier to join the tables.

I found that all codes exclude src/Kucipong/Db/Models/EntityDefs.hs uses word "Store" for store user, but EntityDefs.hs calls it "Company". I'll rewrite them with "Store".

Okay, thanks.  I agree completely with this.

I got it. I agree with the idea to remove annoying Maybes.
However, in the view of business side, the Kucipong should not require all columns on first log in of store user. It means we have to use Maybe even if we separate the tables.
Of course, we can do without Maybes if we separate Store table to StoreRequired and StoreUnrequired, but it is hard to migrate on the day we'll move some columns from one to another.
I'll merge them.


refactor plugins naming.

hi, @qicz , This branch has conflicts that must be resolved.

@midnight2104  I can not do it.


updated MP health intro with link to Guide

Replaced "Page under construction" with a small intro a link to the new mp health check guide.

@m0mus Hi Dmitry, I just need to have this one reviewed so I can squash and merge. Thanks.


Installer should make use of Widget::wrapFormElementWithError()

This way, we could reuse the existing styles for error messages.

Working on now, should be ready by Hackathon :)


Issue #2386 Fix

This pull request is in relation to #2386 so we comply with RFC 2616, Section 9 and ensure that the Response body is empty on HEAD requests which potentially fall back on same path GET route handler due to FastRoute's routing logic.

Coverage increased (+0.02%) to 97.405% when pulling 50c2ec998bfd9ae1bb4f6907427819da0f2d94a6 on l0gicgate:Issue-2386 into 6a7a2c79ca0d1ea876c6d50a9e1c8669b851a75e on slimphp:4.x.


Coverity 1615999

Need to address https://scan5.scan.coverity.com/#/project-view/60762/10222?selectedIssue=1616000

False positive


Testing and cleanups

bumps our coverage for the redux side of the code.
minor cleanups
closes #135
cc @ajithranka


cannot find function supports_feature in crate rustc

Hey,
In the test mode, I found the following error using version 1.7, 1.6...
But, out of the test mode, the error was not reported.
#[cfg(test)]
mod tests {
#[test]
fn t() {}
}
error[E0425]: cannot find function supports_feature in crate rustc
--> C:\Users\admin.cargo\registry\src\rsproxy.cn-8f6827c7555bfaf8\smartstring-1.0.1./build.rs:9:44
|
9 |     let has_feature = Some(true) == rustc::supports_feature("allocator_api");
|                                            ^^^^^^^^^^^^^^^^ not found in rustc
if I remove the following from toml, there will be no error.
rhai =  { version = "1.5.0", features = ["sync","unicode-xid-ident"] }
Thank you!
Regards,
Yang

What is your compiler version?
Rhai requires 1.57 mainly because of SmartString.

Dear schungx,
Thanks for your supper fast reply, as you always did!
rustup --version
rustup 1.24.3 (ce5817a94 2021-05-31)
info: This is the version for the rustup toolchain manager, not the rustc compiler.
info: The currently active rustc version is rustc 1.61.0 (fe5b13d68 2022-05-18)
Thank you!
Yang

@youngzhaozju   Do you still have this problem?  Because I don't seem to be able to reproduce it...
Maybe you can do a cargo update to refresh your Cargo.lock?
This can be a dependency version problem.

@youngzhaozju Do you still have this problem? Because I don't seem to be able to reproduce it...
Maybe you can do a cargo update to refresh your Cargo.lock?
This can be a dependency version problem. The version_check crate needs to be version 0.9.4.

@schungx I still haven't solve this problem. I tried cargo update but it was not helpful. I think it might be caused by conflicting crates. I will report it if I found any solution. Thank you!

Did you end up resolving  your problem?

Closing this for now.  Feel free to reopen if you still have prob.


Compile backend for faster install times

compile meaning take it from all the source structure, and tree-shake -> bundle -> and end up with a lot less files overall
related to npm run build make static assets

Duplicate of #468


python3Packages.maya: 0.3.3 -> 0.6.1

Description of changes

Things done

Built on platform(s)

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

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

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

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.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

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review run on x86_64-linux 1

16 packages built:

python310Packages.flask-common
python310Packages.flask-common.dist
python310Packages.maya
python310Packages.maya.dist
python310Packages.secure
python310Packages.secure.dist
python310Packages.snaptime
python310Packages.snaptime.dist
python311Packages.flask-common
python311Packages.flask-common.dist
python311Packages.maya
python311Packages.maya.dist
python311Packages.secure
python311Packages.secure.dist
python311Packages.snaptime
python311Packages.snaptime.dist

Result of nixpkgs-review pr 256787 run on aarch64-darwin 1

16 packages built:

python310Packages.flask-common
python310Packages.flask-common.dist
python310Packages.maya
python310Packages.maya.dist
python310Packages.secure
python310Packages.secure.dist
python310Packages.snaptime
python310Packages.snaptime.dist
python311Packages.flask-common
python311Packages.flask-common.dist
python311Packages.maya
python311Packages.maya.dist
python311Packages.secure
python311Packages.secure.dist
python311Packages.snaptime
python311Packages.snaptime.dist


*: reject empty endpoints in EtcdBackup CR

fix https://github.com/coreos/etcd-operator/issues/1893
Tested with empty endpoints:
  status:
    Reason: spec.etcdEndpoints could not be empty
    succeeded: false

Should be better handled by initializer/admission plugin: https://github.com/coreos/etcd-operator/issues/1392

lgtm

LGTM


[AFURLConnectionOperation cancel] crashes ...

... when trying to execute [self.connection cancel]. It seems that the problem is with connection property being marked as "assign" instead of "retain". If the connection terminates before we get there, the "connection" object would be a dangling pointer. The solution would be revert back to "retain" and do self.connection = nil in:

connectionDidFinishLoading:
connection:didFailWithError:
end after each [self.connection cancel]

connection is correctly assigned rather than retained because of a quirk of NSURLConnection retaining its delegate. By retaining the connection, we would essentially be creating a retain cycle.
I suspect that the crash your describing reflects an underlying memory management issue somewhere else in the application (see Issue #129)

I actually just ran into this issue this morning. Line 354 in AFURLConnectionOperation with the latest branch.
EXC_BAD_ACCESS on [self.connection cancel].
I believe I have seen another project suffer the same issue. Not much more info yet, but wanted to get it reported.

@kcharwood Have you merged in Pull Request #216? That appears to have solved a whole class of concurrency issues having to do with connection cancellation. Let me know if that fixes it!

I actually am. I integrated the latest as of yesterday, and saw the problem this morning.

Huh, that's not great. With these kinds of things, it's hard to really tell if the patch is causing a problem in AF, or exposing a problem in the code using AF. Any context / insight you come by would be much appreciated, of course :)

Yeah completely understand. I've only seen it once thus far. I was scrolling up and down a tableview that has images using the UIImageView+AF category to load them when it crashed...
No other info to report at the moment. I'll try and put some effort on stress testing that part of the app again this afternoon. We'll post back details here if I find any.

And I see it again when stressing it a bit...
Here is a thread dump:
* thread #13: tid = 0x2f03, 0x02aaf09b libobjc.A.dylib`objc_msgSend + 15, stop reason = EXC_BAD_ACCESS (code=1, address=0x508c90d5)
    frame #0: 0x02aaf09b libobjc.A.dylib`objc_msgSend + 15
    frame #1: 0x001c96d4 REDACTED`-[AFURLConnectionOperation cancelConnection] + 180 at AFURLConnectionOperation.m:354
    frame #2: 0x00dffe72 CoreFoundation`-[NSObject performSelector:withObject:] + 66
    frame #3: 0x012839ef Foundation`__NSThreadPerformPerform + 254
    frame #4: 0x00dd297f CoreFoundation`__CFRUNLOOP_IS_CALLING_OUT_TO_A_SOURCE0_PERFORM_FUNCTION__ + 15
    frame #5: 0x00d35b73 CoreFoundation`__CFRunLoopDoSources0 + 243
    frame #6: 0x00d35454 CoreFoundation`__CFRunLoopRun + 1012
    frame #7: 0x00d34db4 CoreFoundation`CFRunLoopRunSpecific + 212
    frame #8: 0x00d34ccb CoreFoundation`CFRunLoopRunInMode + 123
    frame #9: 0x012a341f Foundation`-[NSRunLoop(NSRunLoop) runMode:beforeDate:] + 300
    frame #10: 0x012a32e3 Foundation`-[NSRunLoop(NSRunLoop) run] + 76
    frame #11: 0x001c78a2 lyndadotcom`+[AFURLConnectionOperation networkRequestThreadEntryPoint:] + 226 at AFURLConnectionOperation.m:130
    frame #12: 0x0126f4e6 Foundation`-[NSThread main] + 76
    frame #13: 0x0126f457 Foundation`__NSThread__main__ + 1258
    frame #14: 0x96d20ed9 libsystem_c.dylib`_pthread_start + 335

Retain cycle can be easily avoided if my suggestion is implemented. Otherwise NSURLConnection will get released automatically and calling "cancel" will occasionally cause a crash.

@kcharwood Can you try @zdzisiekpu's proposed fix to see if that works? I'd love to be able to get some empiricism to backup this change if you're able to pretty consistently reproduce your crash.

Ya I'll try it out tomorrow and report back.

@kcharwood @mattt I actually saw this yesterday as well, but still haven't been able to reproduce this if I include the locks mentioned in Pull Request #216, but thay may be pure coincidence.

See Pull Request #223 for a proposed change. @kcharwood @zdzisiekpu please let me know if this eliminates the crash.

Pulled in @erikolsson's change this morning, and still see the crash.
Just to give some insight into how I am reproducing this - I have a pretty big tableview that displays an image in each cell, and I am using the UIImageView+AF category to load that image. Using the TableViewIndex, I can quickly pan around the table causing cells to quickly be dequeue'd, which cancels a lot of in flight image requests, and after a few seconds of doing that I run into the crash.
I'm going to implement @zdzisiekpu's change above and see if that solves the issue.

@kcharwood that was my exact use case, scrolling table view with images, something similar to the friends list in Facebook for iPhone.
It kept crashing until I submitted pull request #216 , which I was fairly certain it fixed it, but unfortunately you are still seeing this crash.
The crash you are seeing now, is that still on line 353 in AFURLConnectionOperation.m?

Ya that's right crash was always EXC_BAD_ACCESS on [self.connection cancel].  However, with the fix suggested by @zdzisiekpu I haven't been able to get it to crash.

So looks like the changes from @zdzisiekpu have solved my issue with all the changes I put in #224. Looks like @steipete went down a very similar path in #226.

Great to hear! Now to figure out which one to merge in...
Good call, @zdzisiekpu on your fix!

We just submitted an app to the app store using the latest version. Still in review, I'll let you know what happens to the crash rates in a few days hopefully.
@zdzisiekpu @kcharwood did you still see the crash after introducing this fix?

I haven't seen a crash since the fix on this end.

great! let's hope it works for us too!

Closing this thread (fingers crossed). If anyone comes across this problem again, please open a new issue. Thanks!


FR: Swift codable with arrays

Feature proposal

Firebase Component: FirestoreSwift

I'm implementing swift codable as guided in doc using data.as(DecodableType.self).
There's a case where I have an array of doc references (referenced to another collection)
final class TodoModel: DocumentReference {
    var name: String?
    var createdAt: Date?
    var deadLine: Date?
    var startTime: Date?
    var endTime: Date?
    var spendTime: Date?

    // MARK: - Realtionship
    // @ObjectID var tasks: [TodoTaskModel]? // @ObjectID Not working on this.
    // @ObjectID var tasks: [DocumentReference]? // This is not working also.
}

final class TodoTaskModel: DocumentReference {
   var name: String?
   @ObjectID var todo: TodoModel?
}

This yields decodingIsNotSupported : "DocumentReference values can only be decoded with Firestore.Decoder" even after comment all the tasks.
After digging the fireStoreSwift for a bit, it seems the decoding for array reference is not yet implemented. Any helps to implement this is appreciated. Thanks.

Digging for more, it seems this part is somehow incorrect in the logic:
FireStoreDecoder.h
Line 1039:
func unbox<T: Decodable>(_ value: Any, as _: T.Type) throws -> T? {
....
    if let v = value as? T { // T is TodoModel, but value is [String: Any]
      if isFirestorePassthroughType(v) {
        // All the native Firestore types that should not be encoded
        return (value as! T)
      }
    }
...

The debugger passed the check and go to this:
storage.push(container: value)
let decoded = try T(from: self)

Here it jumps again to CodableDocumentReference and throw FirestoreDecodingError.decodingIsNotSupported(       "DocumentReference values can only be decoded with Firestore.Decoder"     )

@EddieLukeAtmey Thanks for filing this. I will look at this today and report back with an estimate on how long it might take to fix.

@EddieLukeAtmey Sorry for the slow response time here. I had to check back with the team.
It turns out that we explicitly decided to not provide a general purpose encoding of DocumentReferences. While they work within Firestore, we don't provide a JSON mapping that allows them to be used outside of Firestore.Encoder/Firestore.Decoder. The reason for this is that there is no appropriate general purpose way to serialize the inherent properties of a DocumentReference (e.g. the path, project and database and the properties of the Firestore instance from which it was created).
While we could theoretically provide a JSON mapping that conveys this data, doing so would lock us into a format and prevent us from making internal SDK changes that could affect the representation (such as adding new private fields).
I am able to serialize arrays of DocumentReferences using the Firestore.Encoder and Firestore.Decoder. If that doesn't work for you either, do you mind providing a reproduction so we can continue our investigation? I will close this feature request for now, but we can re-open if there are issues in this code path.


Iskeyword match

Hi,
As I was using ddc-buffer on programming source files, I found that the candidates were not respecting the programming language iskeyword option.
I would like to submit this PR to make use of iskeyword. Most of this PR is inspired by Shougo/ddc-around.
Don't hesitate to modify/discuss/merge at your will.
Cheers
Kawas

Hi matsui54,
After playing with the code, analysing when the source is initialized and which events trigger the source (and initialization if not already done...), I have made some changes to use ddc options.keywordPattern.
Because of the way events trigger a current buffer indexation, in most case we only need to use ddc options.keywordPattern (as you told me at the very beginning of our conversation).
As always, waiting for your remarks and ready to squash stuff before a merge :)

Tested the PR and it seems perfect! Thank you for your contribution!


[book typo] Quaternion dot product not scalar?

Hi
thanks for the great read at https://raw.org/book/algebra/quaternions/. The issue is related to that page, not to the JS library, so not sure if it's ok to raise the issue here. In case, please do share if there's a more appropriate place where to notify about the book.
In Quaternion Dot Product chapter, the operator "signature" so to say is shown as:
⋅:H×H↦H
But, being a scalar product, shouldn't the output be in R? And thus:
⋅:H×H↦R
Thanks

Oh! That is a very good finding, thanks a lot for that! And yes it's okay to file that issue here. Fixed it.


fix: updated docs TOC

Description
Updated Composer documentation TOC.
#minor

Coverage remained the same at 54.64% when pulling f74562d872b6d08a866ff49cd0899e051d3c5050 on zxyanliu/update/doc/toc into 2253f114a12c6332efcf2f1cba2c6faedd3d75dc on main.


New Item Filter

Make a new item filter that connects to the end of a pipe that only allows specific items through. We have an extractor, now why not a separate item that filters out items in the same pipe line? Like, let's say I have an extractor, whitelisted to only Diamonds, Emeralds, and Gold. Since they're all in the same pipe, I need a filter. Diamonds only in this chest, Emeralds in this one, and Gold in this one, rather than them all being put in one chest due to being unfiltered.

check other issues first before creating a new one


Create an NPC Macro

Given Name p. 333
Family Name p. 333
Callsign p. 333
First look - 2 p. 330
Initial Disposition p. 330
Character Role p. 331
Character Goal p. 331
Revealed Character Aspect 2 p. 332
Requested by enmelm (Discord)

complete


New version: JetBrains.FleetLauncher.Preview version 1.40.86

Created by 🥟 Dumplings in workflow run #8029.

Log
Updated: 1.40.83 → 1.40.86
Submitting WinGet manifests
Creating manifests
Uploading manifests and making commits
Creating a pull request

Microsoft Reviewers: Open in CodeFlow

Possible duplicate package entry.
Similar installer SHA256 hash found in manifest:

InstallerSha256: 1f6a650e776bb719954b0cfc4b70b374410df3f00d0a684f0c432268a5b220a1

manifests\j\JetBrains\FleetLauncher\Preview\1.40.83

It appears that the developer might have reverted to this version.


Support other encryption standards

We currently only support RSA keys for signing and verifying of the response.

Right now this is a nice to have. For the forseeable future we'll be using RSA.


List view expansion expands only on toggle click but entire row shows hand cursor

When setting the list view to include expanding rows the entire row gets the hand cursor but the click doesn't do anything unless it is on the expand toggle. In Patternfly examples clicking anywhere in the row expands the item (unless there is  a link, button, or kebab).

I cannot reproduce this in angular-patternfly v3.x.  With expanding rows, clicking anywhere in the row triggers the expansion open/close.  I believe this was fixed via PR #400


Add additional labels to metrics

Checklist

[x] Tests pass (if any)
[x] Code has been linted/formated
[x] Documentation updated (where applicable)

Description
Add additional fields to metrics to allow fine graded separation in grafana

request_protocol
request_path
request_host
request_port (if exists)
request_query (if exists)

@john-dev heads up you've got a problem the linter's complaining about

To be fair, that error is in you main branch ever since and is raised in every single pull_request you got.

i would advice to fix this in main and rebase the pull_requests to avoid merge conflicts

Could we remove the overlapping changes in this PR so it just adds the labels, it seems to overlap with the merged PR #15


Add docs for new features

We need to write docs for dynamic subscriptions, and possibly other new features that have been added that haven't been documented.

As far as I'm aware, dynamic subscriptions is the only thing that needs further docs.

Possibly I'm missing something, but it seems that documentation is missing for very basic features.   Maybe there is a lot of assumed knowledge that I don't have (I'm not a front end developer).  For example, I can't find an example of using  a drop down anywhere.  If there are examples available, it would be great to link to.
this doesn't work properly:
(defn dropdown.
[]
(fn []
[:select {:on-change #(re-frame/dispatch [:change-letter (-> % .-target .-selected)])}
'([:option "a"] [:option "b"] [:option "c"])]))

@BenMacKenzie re-frame is an application framework, not so much a UI library. You could take a look at re-com which is a UI library designed to work with reagent.

Yes, thanks, i think this is what i was looking for.

I'm going to close this.  The dynamic subscriptions now has first cut docs.  More could be done, but that's always the case.

@BenMacKenzie  Support requests and usage questions should go to the Clojure Slack channel (http://clojurians.net/), the ClojureScript mailing list (https://groups.google.com/forum/#!forum/clojurescript), or the Reagent mailing list.


Fix initialization of DigitalOcean ingestion

I'm not sure if this module ever worked before, but it did raise an error on initialization:
digitalocean.TokenError: No token provided. Please use a valid token

This PR addresses this issue.

@kunaals, with the next scheduled release planned for the 10th of November,
do you think this fix will deserve a hotpatch release? (0.47.2)


show background color for duration widget

related https://github.com/demisto/web-client/pull/8266

Closing this PR as relevant parts extracted to - https://github.com/demisto/sane-reports/pull/159


Allow Pipfile to consume environment variables

It would be awesome if Pipfile could be parsed to allow for environment variables.
While it's great that pipenv supports private indexes, it's annoying if those indexes require credentials—it's bad practice to store them in source control:
[[source]]

url = "https://{{ MY_USERNAME }}:{{ MY_PASSWORD<EMAIL_ADDRESS>verify_ssl = true
name = "pypi"

Happy to contribute a PR for this if this gets some thumbs.

:+1:

@kennethreitz what do you think about making this section able to inherits values from a global setting file, such as /etc/pipenv.conf (a-la /etc/pip.conf), so all Pipfile within a private organisation can use the same internal repository (nexus, artifactory, other,...)

no

@gsemet I believe pipfiles will use sources specified in your pip.conf now though

When we create a pipfile for you, it will be populated with sources from pip.conf.

not sure, we use our internal nexus cache in pip.conf:
$ cat /etc/pip.conf
[global]
trusted-host = nexus.mycompagny.fr
index = http://nexus.mycompagny.fr/repository/pypi/pypi
index-url = http://nexus.mycompagny.fr/repository/pypi/simple

when executing pip install on its own, it uses this server.
when creating a pipfile:
$ pipenv install request
...
$ cat Pipfile
[[source]]
name = "pypi"
verify_ssl = true
url = "https://pypi.python.org/simple"
...```

we pick up user pip conf i think, not global pip conf.

here's the code: https://github.com/pypa/pipenv/blob/master/pipenv/project.py#L419

That sounds potentially dangerous if you’re storing credentials in pip.conf.
@kennethreitz, I was reading another issue where you referred to the Pipfile as the source of truth. I think this is a great idea and should stay that way.

On Mar 14, 2018, at 9:14 AM, Kenneth Reitz<EMAIL_ADDRESS>wrote:
here's the code: https://github.com/pypa/pipenv/blob/master/pipenv/project.py#L419
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

we could detect high-entropy strings automatically (or just urlparse).

@dvf pipfile is def the source of truth.

i think environment variable injection is a good idea though

url =<EMAIL_ADDRESS>

or just<EMAIL_ADDRESS>

We raise an error if PYPI_CREDS isn't in os.environ.

If we only check PYPI_CREDS, we're assuming that a user would only have a single source defined in their Pipfile. I think we should allow for any environment variable—similar to how Docker allows you to use Environment Variables in Compose files.

Pipfile is the ultimate source of truth
Any value can contain an environment variable, but nothing else

Of course we'd support any.

i'm just using that as an example

Oh gotcha 👍
Do you have any ideas about how you see this being implemented?

@kennethreitz - env variable expansion is something I'd be happy to implement unless somebody else wants to (prob have time next weekend), but also happy to leave to other ambitious and/or helpful folks

@jtratner I was going to take a stab at it. Feel free to leave some suggestions in advance though.

oh go for it @dvf :)

Resolved by #1769 and #1809

It would be nice if we can have environment variables loaded throughout the entire Pipfile. E.g.
[dev-packages]
my-package = {path="${PATH_TO_REPO}", editable = true}

@njgrisafi The reason this does not work appears to be due to https://github.com/pypa/pipenv/blob/8aa9283fda2e8cc532f517969f5f3e1969e8d4a2/pipenv/project.py#L519 using tomlkit instead of pipfile.api.PipfileParser
However, this gets further complicated by the fact that the path is copied into the Pipfile.lock. The way inject currently works, it that it injects the values at parse time, which means by the time you get to writing the lock file, it will write the value of the the environment variable, not the ${FOO} syntax. :(
I agree, this feature working on path would be very much desired for me too. I want to run a Pipfile in and out of a docker container, but currently implementation limits this.

So where are the docs?


Added 1.12.0 and 1.12.1 version of gTest

New versions of googletests

@waruqi The environment setup failed on archlinux. Seems it's not a matter with the package


Fairy ring helper says invalid location

No matter which ring I try to use it says invalid location

I'm only getting invalid location when the code isn't used for a location. Can you try reinstall the client and see if you get the error again?

Is there any more information on this @drnfc ?

Looks this is non-issue, reopen if it still persists for you + include more information


Add navbar-logo

add navbar-logo.png to static/pickATrip_django_apps
use django static to inculde the image file

maybe just open a small issue and link this PR to it.

Maybe you can add the logo photo to the 'media' folder that we have?

If I want to use the Django static option I think I have to use a pickATrip_django_apps\static\pickATrip_django_apps path.


fix: skip content type for non-body methods

Summary by CodeRabbit

New Features

Enhanced request body handling by allowing only specific HTTP methods (POST, PUT, PATCH) to include request bodies.
Added comprehensive tests for router configuration across multiple adapter modules to ensure consistent response for GET requests.

Bug Fixes

Improved robustness of the request body processing by bypassing parsing for unsupported methods.

TIL body on GET is not actually spec-forbidden, but it just has no semantic meaning in the spec, and is generally "not recommended"

TIL body on GET is not actually spec-forbidden, but it just has no semantic meaning in the spec, and is generally "not recommended"

then Node aint spec compliant, nor is Chrome

Interesting... then Node aint spec compliant, nor is Chrome

I think since the spec doesn't specify behaviour, they can implement it how they want, and everyone has just agreed that GET doesn't have a body, even if the spec is wishy-washy on it hahah


Add helpers to compute type hashcode from native metadata

We will be converting the type reflection mapping table (the one that
converts between EETypes and type metadata handles) to a hashtable.
In order to do that, we need to be able to compute the type hash code
from metadata. In this commit:

I'm making it possible to compute the type hash code without
allocating and formatting the complete type name (to cut down on
allocations)
I'm adding a helper to compute hash codes from TypeReferenceHandle
and TypeDefinitionHandle
A bunch of unit tests for both.

(I'm submitting this against the nmirror branch because I'll be using this from TFS side very soon and don't want to wait/block on full integration.)

👍

@davidwrighton If you have any comments, I can make adjustments in subsequent commits.


[SampleAndHoldSim] Version 0.5.0, when the game is restarted and the save is loaded, there will be intermittent stuttering

https://youtu.be/5dS6l6sdI6o
As the title says, when entering the game for the first time, the game will continue to stutter (as shown in the picture above), and it will not be relieved until the first automatic save or manual save.
The milliseconds of lag depends on whether Icarus is moving. For example, the milliseconds of lag will be higher when he keeps moving (200~500ms)

Hmm.. This seems weird.
Are there other mods? Does this happen only when SampleAndHoldSim is installed?
If UPS can handle, consider using AlterTickrate instead. May it is more stable.

There is also DSPOptimizations, and there are no other mods

Since the stuttering happened in different parts, it may due to system garbage collection.
I try to reduce memory allocation of the mod in version 0.5.1.
The stuttering solves after game save is weird because the mod doesn't modify stuff with saving process.
If the stutter still occurs, try to seek mods that help game save, for example BulletTime, CompressSave, LossyCompression

It has been updated to version 0.5.1. After entering the game for the first time, there will only be about 3 stuttering times and then it will stop stuttering. Thank you.


JsonConverter.bas won't run as it is HTML

Hi all,
I have downloaded and installed JsonConverter.bas in accordance with the install instructions. I am using Excel 2016 (Office 365).
As soon as I run my VBA script I get an error "Compile Error: Expected: line number or label or statement or end of statement". The cursor displays the first < of the line  line in the JsonConverter.bas Module.
What am I doing wrong?
Many thanks!
Cheers
...Steve

Found the problem. If you download the file (right click>save link as) rather than use the download ZIP button it downloads HTML.


Update from_json to use null as line delimiter

I am leaving this as draft for two reasons.

it does not fix having \r in the JSON. See https://github.com/rapidsai/cudf/issues/16915
There is a very large performance hit going to a regexp for stripping the characters from the input. I am a bit conflicted here because we do need a fix for this at some point even without trying to support \r and \n in the data because \t is not being treated as an empty line and will fail.

We are going to do this a different way.


As an administrator of mrbelvedere, I want to be able to create an installation step that can add a picklist value to a field if it doesn't already exist.

Retrieve the current values, add the new value, redeploy to org

@jlantz another CCI


Make auth manager indepentent

This is a tiny refactor. It just moves encryption module into a separate independent folder. This will help future default constructors refactor + building new independent modules

Codecov Report

Merging #52 (565b093) into refactor-did-auth (1c24451) will decrease coverage by 0.61%.
The diff coverage is 95.91%.

@@                  Coverage Diff                  @@
##           refactor-did-auth      #52      +/-   ##
=====================================================
- Coverage              84.96%   84.35%   -0.62%
=====================================================
  Files                     17       18       +1
  Lines                    379      377       -2
  Branches                  53       49       -4
=====================================================
- Hits                     322      318       -4
- Misses                    57       59       +2

Impacted Files
Coverage Δ

.../ipfs-cpinner-client/src/auth-manager/constants.ts
100.00% <ø> (ø)

modules/ipfs-cpinner-client/src/constants.ts
100.00% <ø> (ø)

...ules/ipfs-cpinner-client/src/auth-manager/store.ts
33.33% <33.33%> (ø)

...ules/ipfs-cpinner-client/src/auth-manager/index.ts
100.00% <100.00%> (ø)

...pfs-cpinner-client/src/encryption-manager/index.ts
100.00% <100.00%> (ø)

modules/ipfs-cpinner-client/src/index.ts
78.43% <100.00%> (-0.82%)
:arrow_down:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1c24451...565b093. Read the comment docs.


Issue with newest version of homebrew-core

I ran into an issue where the newer version of the homebrew-core tap, caused an issue since it used a new function, not present in the current version installed by nix-homebrew.
https://github.com/Homebrew/homebrew-core/blob/master/Formula/x/xz.rb#L26
The error I receive is:
Error: homebrew/core/xz: undefined method `deny_network_access!' for Formulary::FormulaNamespaced6af8689f9d67fe78906ee1d8991056d52b8b4735aad55203c61e4b64950bbbd::Xz:Class

I assume this issue is quickly resolved once the brew command version is updated, but it might be worth thinking about a way of locking taps to the version that is known to be supported.

I'm experiencing this too, and can't find a way to fix it. Have you?
I've tried manually using brew update, but get: /nix/store/na5j7bphjjfqn39hvn439x8qg83jpqbr-source/.git: Permission denied
I've tried disabling homebrew and nix-homebrew in my nix-darwin flake, deleting /opt/homebrew and then re-enabling homebrew and nix-homebrew.
Sadly, I still get the same error as you @Dekker that prevents multiple formulae from installing,
eg
Installing libxml2 ==> Downloading https://ghcr.io/v2/homebrew/core/libxml2/manifests/2.12.6 Error: homebrew/core/xz: undefined method deny_network_access!' for Formulary::FormulaNamespaced6af8689f9d67fe78906ee1d8991056d52b8b4735aad55203c61e4b64950bbbd::Xz:Class`
This includes if I try to re-install xz by adding it as a brew formula in brews.
I have also tried to disabling homebrew and nix-homebrew in my nix-darwin flake and re-installing homebrew without nix, which hasn't fixed this.
Any help is appreciated as sadly, I need to use some homebrew forumlae that aren't yet in nixpkgs.
Thanks
PS Is homebrew-bundle deprecated from nix-homebrew, as I don't see it in the readme, but it's in my flake?

I actually managed to work around the problem by adding
    brew-src = {
      url = "github:Homebrew/brew/4.3.0";
      flake = false;
    };

to my flake inputs. By then adding a inputs.XXX.follows to the nix-homebrew input:
    nix-homebrew = {
      url = "github:zhaofengli/nix-homebrew";
      inputs.brew-src.follows = "brew-src";
    };

I was able to upgrade the version of homebrew that was being used.

Many thanks @Dekker1, that worked!
I thought I'd try what you no doubt did first, using master instead of 4.3.0 to avoid having to remember to change versions again in future if forgeting to change the url back, but was hit with a ruby version issue with a psych gem issue, so reverted to your approach. Mentioned should someone else see this and try to skip your workaround for master.


[Question]: NULL IN PHYSICAL DEVICES IOS

What is your question?
Hello,
Guys I am using the latest version and I have iPhone 14 plus on the latest version IOS.
it's working fine on the ios emulator. but returns null on the physical devices.
here is the log.
[] NEHotspotNetwork nehelper sent invalid result code [1] for Wi-Fi information request
I have added wifi access capability and tried everything but nothings work on latest version of IOS.
please help. i am new to ios,
Thanks
Checklist before submitting a question

[X] I Google'd a solution and I couldn't find it
[X] I searched on StackOverflow for a solution and I couldn't find it
[X] I read the README.md file of the plugin
[X] I am using the latest version of the plugin
[X] All dependencies are up to date with flutter pub upgrade
[X] I did a flutter clean
[X] I tried running the example project

Please provide a better description of the issue, starting from the plugin you use and what is null as currently it is hard to understand what the problem is.
However, I think that you are talking about network_info_plus. There is an issue on latest iOS versions as discussed in #1652

sorry for the less info.
yes, I was talking about the same issue. regarding the network_info_plus plugin.
I am using only getWifiIP() method it's returning null, and here are the logs.
[] NEHotspotNetwork nehelper sent invalid result code [1] for Wi-Fi information request
code : final wifiIP = await info.getWifiIP();
Thanks,

guys any updates, please?

Hello, I have the same problem and I still couldn't solve it. Now I am looking to implement UDP to find the devices in the client's network but there are parts of the application closely linked to knowing the IP that are difficult to resolve in a clear way for the client.
I hope they can find a solution soon.
Thank you!

I've same problem ! :(

Hello everyone.
I also continue with the error and unfortunately I couldn't wait any longer because we already had many cases with problems.
Finally we implement a broadcast message via UDP so that the ESP32 responds to us and then we can capture the IP and connect.
In our case, we were able to solve it and for clients with old products we gave them the option of loading the IP by hand or updating the ESP32 firmware to incorporate the UDP tracking option.
I hope this helps others move forward with a possible resolution.
Greetings

Any solution？


[DOCS] Use metadata to assign annotations

Which page or section is this issue related to?

Now that it's possible to filter records in Feedback Datasets using metadata, we can modify this page to explain how to make assignments using this method.

@nataliaElv, did you already asign users by metadata?

Yes for old datasets, but not for the new Feedback, so maybe we can modify the page to use that in favor of duplicating datasets 😄


Fixed #3804

Fixes #3804

Hi, and thanks for contributing.
This is an exact duplicate of #3802. Please close this and go +1 that one.

Thanks!


fixed appirio-tech/topcoder-app/issues/1167 clean up constants

fixed appirio-tech/topcoder-app/issues/1167 clean up constants

@shubhendusaurabh please try to use props instead of process.env

Updated. process.env will still be required in setting default props as constants object is not present in react-components as they are present in topcoder-app.

@tladendo Did you review changes after your comment?

@shubhendusaurabh you still have a direct ref to process.env in UserDropdownMenuExamples.jsx

@shubhendusaurabh could you also try to make sure the right props are getting passed into the components, where possible? thanks

@tladendo Updated.

@shubhendusaurabh added one last question in code comments

@tladendo What is the question? Can't find any.

@shubhendusaurabh in components/Navbar/NavbarExample.jsx, why did you take INTERNAL_API_URL out of the props and drop the call to process.env right into the code?

@tladendo The function suggest in which INTERNAL_API_URL is being used is not part of the component. It is called by the child component with only search term passed in.

@shubhendusaurabh is this supposed to work when running react-components by itself? because that's not working for me. updated webpack as well

@tladendo yes, it should. Did you restart webpack after updating. In my case I made the changes in /react-components/node_modules/appirio-tech-webpack-config/webpack.config.coffee file directly then worked on it.

@shubhendusaurabh do challenge details page links work for you? that's where i'm seeing the problem

@tladendo Details link issue was present before my changes but I have fixed this now.

@shubhendusaurabh thanks!


feat: unescape escaped characters when value is not quoted

Thank you for this handy gem. We use it regularly.
When using Ruby's shellwords library to escape values, the following env file is created.
export FOO=one\ two

Quotes cannot be used around those values, otherwise the backslash is considered a literal.
# incorrect
$ export FOO="one\ two"
$ echo $FOO
one\ two

# correct
$ export FOO=one\ two
$ echo $FOO
one two

This PR updates the code to correctly unescape characters when quotes are not used
The parse_value method had to be refactored to keep Rubocop happy.

Much appreciated!

Is there an expected date for the next release?

My plan is to review outstanding PRs/issues tomorrow and cut a release before the end of the day in the pacific timezone.

Hi. I know it's been a while, but as far as I can tell, this was never released. We're currently pointing our gem source at a private repo for this feature, but would like to change that to use the public gem. Could this be released please?

2.8.0 is out with this change. Sorry for the delayed release. Enjoy!

Better late than never! 😆 Thanks!


Fix PullRequestMergeCommandTest failing for some tests

Q
A

Fixed tickets

License
MIT

@sstok could you please reference the error that is giving while running the tests here? Thanks

Aborted. Because of pat it expects a selection from the list.


Opt-in region State Machine enhancements

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.
This PR introduces a new step in the State Machine that executes a new State Machine that enables opt-in regions, and waits until the enabling is complete. The new SM is configured to run 15 accounts concurrently.
Main State Machine:

New OptIn State Machine:

This improves the experience when creating new AWS accounts by having the SM enable the region instead of using an external script.

-Waiting eagerly for this feature. This update is crucial for our clients and will significantly enhance our service offerings.


internal/osutil/gce: staticcheck

Closing this, see my comment here:

https://github.com/perkeep/perkeep/pull/1405#issuecomment-894727096


feat: 🎸 new component inputSearch, now you can use the x to clean input value

As you can see, I use a lot this page

But was annoying me, the fact that the input isn't a type search, so I made this 🥲
now you can use esc to clean the input or if you in a mobile device you can click on the X  button ❤

I really wanna this feature on the site PLEASE MERGE ME 😩 (will help me alot)

Hey how's it going ? @MadeBaruna
I now you need to take your time, but when you have a free time, could you see this significant improvement ? 🥺
Will help me a lot if some day I could use it


Storybook should prompt user when it runs on a different port

Is your feature request related to a problem? Please describe.
In SB4, storybook was improved to run on a different port if the desired port is busy, inspired by CRA. However it's confusing when you're expecting it to be running on port X and it's running on port Y.
Describe the solution you'd like
Storybook should prompt the user when it switches ports, like CRA does:

Are you able to assist bring the feature to reality?
Yes, but I'd love it if somebody else could take this.

I'm interested in working on this. What's the desired functionality if the user doesn't want to run the app on a different port? Should it exit?

Hi @mpaarating! If you didn't change your mind and its feature request its actually: yes if user choice "no" app should be an exit.
Let me know if you want to start working about this.
Thank you!

@Armanio great! I'll start working on this!

@shilman I think this feature already exists. If not, can you provide a bit more detail about the change you'd like to see?
Current behavior:

Hmm, looks like it got added, or maybe I goofed!? Anyway thanks for investigating @mpaarating 👍


TrayPublisher: Fill missing data for instances with review

Changelog Description
Fill required data to instance in traypublisher if instance has review family. The data are required by ExtractReview and it would be complicated to do proper fix at this moment! The collector does for review instances what did https://github.com/ynput/OpenPype/pull/4383
Additional info
Ideally we should fill frame start/end from ranges of passed files, and use frame start from asset -> calculate frame end based on range from files. There is a lot of issues connected to this approach that may break other suff. We should also collect frame ranges for image frame sequences which are not right too (e.g. in burnins).
Testing notes:
Simple creators

Open TrayPublisher
Select a creator (e.g. render)
Pass files to source file input and to review input
Publish
ExtractReview should not fail on missing data on instance

Editorial

Publish via TrayPublisher editorial which should also create reviews.

I think the only way is doing it per representation. The biggest issue with that approach is that it would require to rewrite ExtractReview, probably ExtractBurnin and some more which don't expect that approach. That's a lot of work and a massive amount of work to review the PR.


[Question] why design a resource stage server

spark-on-yarn uses HDFS to store app dependencys, why spark-on-k8s use a  resource stage server not HDFS?
there may be some problem about resource stage server:
1, single point, if restarted or migrated the resource may lost
2, if many jobs concurrently uploading/downloading resources, the performence is likely poor(not tested yet)

Not all spark-on-k8s users are running HDFS in k8s.  In YARN there's a dependency on HDFS so YARN applications can assume that an HDFS exists.  But in k8s there's no HDFS dependency, and no other feature of k8s for stateful storage exist.  So for Spark had to make something ourselves.

Kubernetes and YARN differ in the sense that YARN is tightly coupled with the Hadoop ecosystem where Kubernetes is not. Allowing Kubernetes applications to be launched without HDFS allows for more flexibility for users who wish to avoid having to stand up an entire distributed system just to run one-off applications. The user may use a different distributed storage medium for their industry’s data that isn’t suitable for mounting dependencies, such as a storage layer that is massive but only eventually consistent. In this case, it is much easier for a user to deploy a single HTTP service than it is for them to consider the provisioning of an entire distributed storage backend.
This matter is further reinforced when we consider that Mesos does not allow users to submit using local files at all. In mesos-cluster mode, all dependencies have to be pre-staged over some remote storage layer like HDFS or HTTP.
The resource staging server is also only a strictly optional component. Application writers can choose to pre-stage their dependencies in a different storage layer like HDFS or HTTP. We expect a very common use case to be having the application writers mount their dependencies into the Docker images to allow these dependencies to be cached.

thx, so HDFS maybe an optional backend for stage server ?

Yeah, I could see that.  Preferable to both HDFS and staging server though is to bake the necessary jars right into the docker image you're running -- then they're cached on each node by dockerd


Allow verbs in the specification

I have built a similar (toy) language for recipes in the past, and noted that such a language could be interpreted by a robot.
For this to take place, verbs should be available to the language, similar to the @, #, and ~ nouns. $ may be a sensible character to use.
The "standard library" should not provide any verbs (just as no @ or # nouns are provided).

@LSWdigital thanks for sharing the example. It's still not clear for me what is the use case and how this will be used in the end. Also, I assume we need to express a link between the verb and ingredient to make any use of it, right?

The fact that i'm mentioned in a repo i never heard about is hilarious, i kinda like the markup ;)
Great project btw!

@potato haha, great username!


Game Crash

crash-2017-05-02_16.21.45-client.txt
it crashes when look to homecraft mineware's bed...
forge 2294

Problem in their mod. They are returning null for a block.getItem() and that's invalid in 1.11


Improve and test archive upload of large compendia via browser

If workspaces get really large, we should make sure the browser upload still works well (though for large data we'd surely prefer cloud-based uploads or submissions via repositories). The API might need to be updated for this, see
https://github.com/zenodo/zenodo/issues/833#issuecomment-249785341

The instructions provided at https://github.com/zenodo/zenodo/issues/833#issuecomment-262442303 work well. Deposition of files will be implemented in o2r-shipper.


Wrong TestCases in C-Plus-Plus/stack/Check_for_balanced_parenthesis.cpp

🐛 Bug Report
(A clear and concise description of what the bug is.)
Have you read the Contributing Guidelines on Pull Requests?
The given test cases has not been handled in code. The test cases talk about an expression but in the code it is no where handled thus results in wrong output. Also the given sample I/O is not correct.

/assign

/assign


cotp: init at 1.1.0

This is a follow-up on #200427 which was closed due to me messing up while rebasing (sorry!)
In case I need additional changes, I'm opening it as a draft so I can take more time to review.
Description of changes

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[x] 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
[x] 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

[x] Fits CONTRIBUTING.md.

Sorry about the format issues @azahi , I read somewhere that alejandra was being used for formatting and applied it, but perhaps I read it somewhere else. For future contributions, should I use nixfmt instead?

should I use nixfmt instead

No, don't use that. There are no specific guidelines on how to format Nix code, just try to mimic what's already there. If you need a formatter you acn use nixpkgs-fmt, but it's not "official" per se.
There's also an RFC about that if you are interested.

Evaluation is currently failing, you need to fix that and then please squash your commits. You should have two commits ready to merge: one which adds yourself as a maintainer, and one which creates the package itself. You can find out how to correctly format commit messages in CONTRIBUTING.md or just look at the git logs and see what other people are doing.

Looks great! The only think left is that you should remove the v prefix for version in your commit message, it should be like this: cotp: init at 1.1.0.

Looks great! The only think left is that you should remove the v prefix for version in your commit message, it should be like this: cotp: init at 1.1.0.

Done!

Result of nixpkgs-review pr 200434 run on x86_64-linux 1

1 package built:

cotp


Anonymous specialization constants support

Fixes Vulkano Shaders bug when compute shader local group layout values bound to specialization constants.
Prior to this fix it was not possible to redefine local group size of compute shader in run time. Attempts to do so lead to macro compilation errors.
Now it is possible to define the layout in form of
layout(local_size_x_id = 12, local_size_y_id = 13, local_size_z = 1) in;

and then set the values as
SpecializationConstants {constant_12: 8, constant_13: 4, ...}

during Compute Pipeline creation.
This patch fixes #1320 and #914
Tested with AMD RADV VEGA M, Ubuntu 18.04.4, Vulkan 1.2.131.

Is this a standard way for the names. Seems weird to me that you'd set the variable names to the values instead of a key.

@AustinJ235 Well, glsl requires integer literals in local size layout definition. Even for the local_size_<x>_id fields. Unfortunately there is no way to specify constant name as far as I know. As such shaderc doesn't provide constant name as well and vulkano simply skipping such constants when building SpecializationConstants struct because a field name cannot start with a number in Rust. My suggestion is to consider constant_<x> as Vulkano interface convention. It doesn't violate Vulkan/GLSL standard anyway.

Thanks I just wanted some clarification. Seems reasonable enough. I think you may need to push a dummy commit to re-trigger travis ci. It was offline for a bit and acting funny, so It isn't aware of this pull request.


fix(gha): Fix conditional homebrew release script

This fixes the conditional statement for releasing to homebrew. Finishes up the work from this PR:
https://github.com/awslabs/cdk8s/pull/82#issuecomment-603308942
Tested in the closest way I could replicate main repo:

This commit (same as the one in this PR): https://github.com/campionfellin/cdk8s/commit/5ecb1434a8e4b2622cf3e001929a72c511ab89fa

Commit to skip things that my fork can't do (other releases, etc). Also changes release_homebrew to just print the $GITHUB_CONTEXT:
https://github.com/campionfellin/cdk8s/commit/32f5af04d88d87fbde9cdb02c39be7ffdcea1771

Run tools/bump.sh and release 0.19.0:
https://github.com/campionfellin/cdk8s/commit/a5a31b88a21a381fe0f8282ece0393d03d0d96c8

Push to my master branch. Triggering this workflow:

https://github.com/campionfellin/cdk8s/runs/578391052?check_suite_focus=true

I can see that the workflow ran, printing out the $GITHUB_CONTEXT

I think this solution will work for us. If you see anything in the printed out $GITHUB_CONTEXT that you might think is better, let me know!
Signed-off-by: campionfellin<EMAIL_ADDRESS>By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license

Mergify is blocking it with error Resource not accessible by integration, I believe it's because I'm editing one of the workflow files.
Similar error happened in this PR: https://github.com/awslabs/cdk8s/pull/90
Going to override and merge.


filter option by years

Is there option to filter movies by periods of time?

Unclear what you're asking. This repo concerns the open source workflow UI "Conductor".


New Feature: Omnibar

 Issue by chaorace
Saturday Jan 07, 2017 at 07:17 GMT
Originally opened as https://github.com/CQUI-Org/cqui/issues/225

So, if you've used an editor like Sublime or an IDE like IntelliJ, you've probably encountered a bar like this:

It's a bar which contains all of the available commands for the application and allows you to dynamically search for them. I'd like to add something like that to CQUI, because I think there's lots of potential for dramatically improving UI speed for powerusers this way.
Example: open omnibar, start typing city name, tab to autocomplete, type "pro", tab to autocomplete "produce", type "hor", tab to autocomplete "horseman", enter to send a new horseman entry to the start of the selected city queue.
The possibilities are basically endless here, so I definitely think it's worth the effort investment. Still, it's a considerable undertaking that would come at the cost of other features if I started working on it right now, which is why I'm hoping to gauge excitement for something like this. So what do you guys think? What kind of actions would you like to be completeable using an omnibar, and what kind of syntax do you think would work best?

 Comment by Remolten
Saturday Jan 07, 2017 at 16:04 GMT

I think this would probably go beyond the scope of the project. The implementation of this feature would be incredibly expensive, and I don't believe it is useful for enough users to warrant development.

 Comment by chaorace
Saturday Jan 07, 2017 at 21:51 GMT

@Remolten Ultimately, the stated goal of CQUI is to reduce clicks and make useful information more readily available. An omnibar feature falls squarely within the reducing clicks category, it's effectively just a very flexible way to implement keyboard shortcuts when you get right down to it.
As for implementation cost, it wouldn't be an all-or-nothing endeavor. This is definitely the kind of feature that can be implemented piecemeal over time


[SPARK-38210][DOCS] Improve documentation generation README

What changes were proposed in this pull request?
Current instructions in README file is not complete and not sufficient to complete site build for testing and validation.
After number of trial and errors I have managed to build it. In the process I had to install number of additional packages.
This PR purposes improvements to the documentation to avoid spending similar efforts for contributors.
Why are the changes needed?
Improve Spark documentation generation procedure
Does this PR introduce any user-facing change?
No
How was this patch tested?
I have started a docker container:
docker run --name spark_doc_build_new -p 4000:4000 -it spark_doc_build_image
and installed everything as per below
apt-get update
apt-get -y install  git nano
apt-get -y install  curl
apt-get -y install  ruby-full
apt-get -y install  python3 pip
apt-get -y install  gnupg

echo "deb https://cloud.r-project.org/bin/linux/ubuntu focal-cran40/" >> /etc/apt/sources.list
apt-key adv --keyserver keyserver.ubuntu.com --recv-key '95C0FAF38DB3CCAD0C080A7BDC78B2DDEABC47B7'
apt-get update

apt-get -y install  r-base
apt-get -y install  pandoc libxml2-dev
apt-get -y install  libcurl4-openssl-dev
apt-get -y install  libssl-dev
apt-get -y install  libfontconfig1-dev libharfbuzz-dev   libfribidi-dev libfreetype6-dev libpng-dev libtiff5-dev libjpeg-dev

Rscript -e 'install.packages(c("curl", "knitr", "devtools", "testthat", "rmarkdown", "markdown", "e1071"), repos="https://cloud.r-project.org/")'
Rscript -e 'devtools::install_version("roxygen2", version = "7.1.2", repos="https://cloud.r-project.org/")'
Rscript -e "devtools::install_version('pkgdown', version='2.0.1', repos='https://cloud.r-project.org')"
Rscript -e "devtools::install_version('preferably', version='0.4', repos='https://cloud.r-project.org')"

echo 'deb http://security.debian.org/debian-security stretch/updates main' >> /etc/apt/sources.list
apt-get update
apt-get -y install  openjdk-8-jdk
apt-get -y install  scala

git clone https://github.com/apache/spark.git
cd spark/doc

gem install bundler
bundle install
bundle exec jekyll build

and checked via jekyll serve from host
bundle exec jekyll serve --host <IP_ADDRESS>

I have also made this Dockerfile to make the process even easier. Would that be valuable to add to the repo?
https://github.com/khalidmammadov/spark/pull/1/files

Can one of the admins verify this patch?


i cant craft custom items

Hello, i faced such problem. I added a few items with ContentTweaker to my modpack. I easy added these items, applied textures, even added recipes to special machines from other mods. But when i tried to add a one crafting table recipe there was a problem. There is recipe:
recipes.addShaped("salthide", <contenttweaker:salted_hide>, [ 	[<contenttweaker:trimmed_hide>, null], 	[<minecraft:dye:15>, null] ]);
JEI shows me that recipe is avalible, /ct syntax also told me there are no problems, but when i try to fold this recipe it just do not crafts output item.
with items not from the ContentTweaker everything works fine

Without a log I can't tell you much.
However, be sure that the script where you add the recipes runs during the crafttweaker loader, not the CoT one

your recipe form like that
[T ,]
[d ,]
(T : trim , d:dye, _ : blank)
so it need one blank slot at right side
than its working
[T,]
[d,]
[,T,]
[,d,]
[,,_]
but its not working
[,T]
[,d]
[,,]
[,,T]
[,_,d]
also your recipe need 'what craft' and don't need 'where i place' than use "addShapeless"is better work.


Various minor corrections to the text

This PR fixes a number of minor issues in the text.  Some further comments follow (line numbers for each file are provided in the first column - apologies if there is a better way to do context comments on GitHub).
basic.md
457 Any authoritative server which does not implement 'zones' in this way will
458 eventually run into trouble. It is not enough to consult a list of known
459 names and answer records attached to those names.

The phrase "answer records" is not used earlier in the text.  Later on, it is used when negative answers are discussed, in which context it makes a lot of sense.  Here, though, I think the reader might wonder whether "answer records" are records which were somehow specially designated for being used in answers.  Perhaps just say "records" here?  Dunno, maybe it's just me.
608 ## Query types that are not RRSET types
609 In addition to the resource record types covered above, like A, AAAA, NS and
610 SOA, two additional types exist that can only be used in queries: ANY, AXFR
611 and IXFR.

You say "two additional types" and then list three of them.  I understand AXFR and IXFR are related but IMHO it might be tricky to get for people reading about this stuff for the first time.
auth.md:
 61 1. No applicable zone is loaded. Send REFUSED answer.
 62 2. From best zone, there was an exact match for the qname and qtype, send RRSET, set NO ERROR
 63 3. From best zone, the name queried exists, but no matching qtype and no NS type present (send NO DATA)
 64 4. From best zone, the name may exist, but there is a node or a parent has an NS record. Send delegation

Item 4 sounds unclear to me.  Is there a word missing between the words "a" and "node"?
Perhaps use "NOERROR" instead of "NO ERROR"?  IMHO the reader is more likely to come across the former e.g. when using various diagnostic tools.
"From best zone" sounds reasonably okay in point 2 but sounds off in points 3 and 4.  (Maybe use "In the best zone" there?)
Trailing punctuation is inconsistent between the items of this list.  (Other punctuation nit: "(...), send RRSET" vs. "(...). Send delegation".

 89     2. If a match would take us out of the authoritative data,
 90        we have a referral.  This happens when we encounter a
 91        node with NS RRs marking cuts along the bottom of a
 92        zone.

I realize this bit is copied verbatim from RFC 1034 section 4.3.2 but this is the first time you use the word "referral" in the text.  While obviously correct from a technical standpoint, perhaps use "delegation" here instead since you use that word elsewhere and it could prevent the reader from wondering whether there is a difference between a "referral" and a "delegation"?
optional.md:
 38 The EDNS content is attached to a pseudo-record called OPT in the additional
 39 section of a message and an answer.

AIUI, an "answer" is a type of a "message".  Perhaps use "a query" instead of "a message" here?
 58  * NSID (3): Return the 'nameserver identifier' [RFC
 59    5001](https://tools.ietf.org/html/rfc5001)
 60  * EDNS Client Subnet (8): Convey part of client subnet [RFC 6891](https://tools.ietf.org/html/rfc2671).
 61  * Cookie (10): Lightweight transaction security [RFC  7873](https://tools.ietf.org/html/rfc7873)
 62
 63 [RFC 7871](https://tools.ietf.org/html/rfc7871) is a concise specification
 64 that is completely applicable to 2018 DNS. Its implementation is highly
 65 recommended.

RFC 7871 is the ECS RFC.  Is this paragraph tongue-in-cheek? ;)
The "RFC 6891" anchor text is linked to the wrong RFC (2671).
I think you meant to put "RFC 7871" in the "EDNS Client Subnet" bullet point and "RFC 6891" on line 63.

Final remark: perhaps there is not a lot you can do about it quickly but the JavaScript Markdeep renderer apparently tries to outsmart itself when rendering the articles on a mobile device.  It attempts to attach hyperlinks to the numbers in SOA records, treating them as phone numbers.  However, since these records are inside triple backticks, the link tags appear on the screen:

Hope this helps, thank you again for doing this, it is much appreciated!

thanks so much @kempniu! I have copied this into a separate issue so it does not get lost with this PR.

Happy to help!  Sorry for not putting comments in a separate issue myself, it makes sense in retrospect.  I guess I initially thought I will have just a one or two extra comments that could be addressed in the same PR after discussing them, then I got slightly carried away and never reconsidered the original plan ;)


Password sent in query params to callback

I've set up HTTP form auth with SQL provider. See my SecurityModule
When I log in, there is an HTTP GET made to the callback url with username and password fields in the query string.
http://localhost:9000/callback?username=admin&password=admin
I'm new, so most likely doing something wrong. But even if I have misconfigured my app, I don't think this should ever happen.

I guess the method of your form is GET (or nothing) instead of POST. Change that to hide your credentials.


Copyright and resposiveness

We can add a copyright at footer with year changes dynamically and page collapse for small screen so we can fix that issue also.

refer to my fix #70 for the dynamic copyright year change


[Bug] Build error with API version 28 or 30

Is there an existing issue for this?

[X] I have searched the existing issues.

Would you like to work on the issue?

[x] Assign this issue to me.

Describe the bug
When I tried to build the project with API version 28 or 30,  I ran into some errors.
I failed to build and faced message ninja: build stopped: subcommand failed. even though test cases were all passed.

What I've tried as far is as follows, but they didn't work.

Change API version 30 to 28.
Reduce test cases.
Delete /.cxx directory and rebuild.
Delete euphony repository in my local storage and clone again

Expected Behavior
This is what I expected to happen.

Environment
- OS: Windows 10
- API level: 28, 30

Steps to reproduce

Run Android Studio.
Create device using API version 28 or 30.
Run 'java-sample'.
See all test cases are passed.
See build errors.

Anything else?
I refered to these issues and discussions.

https://github.com/euphony-io/euphony/issues/116#issue-1022335831
https://github.com/euphony-io/euphony/discussions/63
https://github.com/euphony-io/euphony/discussions/45

When I used Pixel XL API 32, the project completely succeed to build.

Proposal
I think it is good to notice users who try to build with API version 28 or 30 that the API versions can make build errors. And also, how about do not run gtest with the API versions?

@haan823 Thank you for reporting issues 👍 And your suggestion is also perfect! It would be good to block gtest with API 28 and 30.


Remove trivial TransportModule

In gitlab by @simonspa on Apr 26, 2017, 11:36
propagater outputs charge collected in implant, digitizer takes them directly (same type)
for DC coupling, just add module in between and specify inputs/outputs

In gitlab by @Koensw on Apr 26, 2017, 12:49
assigned to @Koensw

In gitlab by @Koensw on Apr 27, 2017, 14:28
changed milestone to %2

In gitlab by @Koensw on Apr 26, 2017, 13:15
changed milestone to %1


No data passed to templates when alerts are present

https://sonarcloud.io/project/issues?id=muety_wakapi&issues=AXUOqv3qu1Oc5-RXBONp&open=AXUOqv3qu1Oc5-RXBONp

Fixed with latest commit.


BarButton Item Badge Cut  Off

How do you keep the badge from getting cut off of a bar button Item?

Oops my bad.. didn't see hub.moveCircleBy(x: CGFloat, y: CGFloat)


refactor: rm async_trait and add trait_variant

Close #139
Use trait_variant::make to support async fn in pub traits. It creates 2 traits ---- LocalCatalog for single thread and Catalog with Send for multithreaded runtime.

cc  @odysa Should we close this?

@liurenjie1024 Yes, we can close this and reopen it after they release the rewrite feature.

@liurenjie1024 Yes, we can close this and reopen it after they release the rewrite feature.


Potential issue with query-building in Redis OM

Hi there, I'm new to Redis. I'm using Redis OM and I'm experiencing an interesting issue. I created a model based off of the JSON response from a web service. It loads data correctly. However, when I call find and pass some criteria, it seems to swap the order of parts of the query randomly. For example, I can call the .find() code and 3 times out of 5 it won't return any results because it forms the query incorrectly. Here is an example from the profiler: Incorrect:
"FT.SEARCH" ":Models.vulnerability.Vulnerabilities:index" "@cve_cvssMetricV31_metrics_cvssData_baseSeverity:{critical}" "LIMIT" "0" "1000"
Correct:
"FT.SEARCH" ":Models.vulnerability.Vulnerabilities:index" "@cve_metrics_cvssMetricV31_cvssData_baseSeverity:{critical}" "LIMIT" "0" "1000"
notice how cvssMetricV31 and metrics are swapped. It's almost as if there's some instantiation issue happening when the models classes are being instantiated.
Update:
I stepped into the resolve_redisearch_query function and it seems like the Parents property is out of order. Is there a way to control the order in which the classes are instantiated? I'm currently doing Vulnerabilities.cve.metrics.cvssMetricV31.cvssData.baseSeverity == 'critical' as my query. I would have thought that would retain the order of the objects, but it seems to randomly move around the order of the Parents

Hi @vladimirberns - can you please share the model this was occuring with?


fix: [Validation] exact_length does not pass int values

Description
From Slack

exact_length in traditional rules passes int values
but exact_length in strict rules passes only string values
it is not intuitive that the rule ‘required|numeric|exact_length[10] is invalid, and if you pass<PHONE_NUMBER>, you will get the error "The XXX field must be exactly 10 characters in length."

Checklist:

[x] Securely signed commits
[ ] Component(s) with PHPDoc blocks, only if necessary or adds value
[ ] Unit testing, with >80% coverage
[ ] User guide updated
[x] Conforms to style guide

Thank you for reviews! Guys.


Convert aten.sum.dim_IntList to ttnn.sum

Ticket
None
Problem description
To support aten.sum.dim_IntList, we need ttnn.sum to support a tuple of dims.  I'm investigating how to patch the kernel.
What's changed

I have no idea about the failing test case.


fix: keyboard focus looping issue

Resolved an issue where changing focus with keyboard tabbing would focus on un expected element (date-picker__dialog).
Issue

When user expects to switch keyboard focus from close button to previous month button (forward tabbing) they first had to go through the unexpected focus on date-picker__dialog before focusing on previous month button.

When user expects to switch keyboard focus from previous month button to close button (backwards tabbing) they would tab first to date-picker__dialog unexpectedly but then would be unable to tab backwards from there to the close button as focus was essentially stuck on the date-picker__dialog until they switched focus with forward tabbing.

Resolution

On switching focus with forward tabbing focus now goes from close button to previous month button
On switching focus with backward tabbing focus now goes from previous month button to close button

Are there any tests that you can add that demonstrate the issue and that your solution resolves it?


follow up flutter symbolize and split-debug-info

https://github.com/flutter/flutter/issues/59321#issuecomment-886343823
right now it's not possible to use flutter symbolize since the address doesn't match, flutter symbolize does the symbolication differently than Sentry probably.

@fzyzcjy sorry for the late reply.
I'd not say its a bug since symbolication works on Sentry, if we change that, it'll probably break it, I
'll see if we can find a way around it.

This could be a feature where opt'ing in to options.attachSymbolizeStacktrace during init would add a Context called symbolize with the raw stacktrace to the event.
Is this something you could contribute a PR for?

Working on #444


[opencolorio] doesn't build python bindings - windows

Host Environment

OS: Windows 10
Compiler: revision

To Reproduce
 Edit the portfile.cmake with -DOCIO_BUILD_PYGLUE:BOOL=ON
./vcpkg install opencolorio:x64-windows

Build completes successfully, but no python bindings were created.
config-x64-windows-out.log

Is this still an active issue?


CFFL

CFFL

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_2122redv3l/clone/recipe/meta.yaml:17: pip_install_args: pip install should be run with --no-deps.
ERROR: /tmp/abs_2122redv3l/clone/recipe/meta.yaml:23: missing_wheel: For pypi packages, wheel should be present in the host section
ERROR: /tmp/abs_2122redv3l/clone/recipe/meta.yaml:69: missing_pip_check: For pypi packages, pip check should be present in the test commands
WARNING: /tmp/abs_2122redv3l/clone/recipe/meta.yaml:81: missing_description: The recipe is missing a description
ERROR: /tmp/abs_2122redv3l/clone/recipe/meta.yaml:81: missing_license_family: The recipe is missing the about/license_family key.
ERROR: /tmp/abs_2122redv3l/clone/recipe/meta.yaml:81: missing_documentation: The recipe is missing a doc_url or doc_source_url
ERROR: /tmp/abs_2122redv3l/clone/recipe/meta.yaml:81: missing_dev_url: The recipe is missing a dev_url
Errors were found


caddy: use mainProgram for binary

This allows exchanging caddy with a different caddy built binary but different program name. Example frankenphp in nixpkgs
I had to update the devenv.lock as in that commit that mainProgram wasn't added in nixpkgs.

https://www.shellcheck.net/wiki/SC2064 -- Use single quotes, otherwise this...
https://www.shellcheck.net/wiki/SC3010 -- In POSIX sh, [[ ]] is undefined.
In examples/mailpit/.test.sh line 6:
trap "pkill -P $DEVENV_PID" EXIT
^---------^ SC2064 (warning): Use single quotes, otherwise this expands now rather than when signalled.
For more information:
https://www.shellcheck.net/wiki/SC2064 -- Use single quotes, otherwise this...

I fixed the failure in mailpit. I think it was because shellcheck got updated with the lock file change

There's more failures

I unified all tests scripts now:

fixed all shell checks errors
use always trap to stop devenv up
use same shebang for all scripts

last 10 log lines:
>   https://www.shellcheck.net/wiki/SC1091 -- Not following: /nix/var/nix/profi...
>   https://www.shellcheck.net/wiki/SC2181 -- Check exit code directly with e.g...
>
> In build_cloudflare.sh line 1:
> set -xe
> ^-- SC2148 (error): Tips depend on target shell and yours is unknown. Add a shebang or a 'shell' directive.
>
> For more information:
>   https://www.shellcheck.net/wiki/SC2148 -- Tips depend on target shell and y...
>
For full logs, run 'nix log /nix/store/96b5m76d34iz50bar5hzqn6k467l32a2-pre-commit-run.drv'.

So devenv ci now runs without errors

Looks the gitignore change of examples broke almost all things

That flag fixed it. Can you trigger again the pipeline?

I had to revert the ignoring of devenv.lock. I think we would need to adjust nix itself? to not run git add on it. 🙈

So I fixed clickhouse, vault, modern-c, supported-languages. There is now something failing with ruby and rails. I have there no experience.
But more jobs are green than current main branch 😅

Thanks for fixing those :heart: