Swagger 1.2 Documentation missing info section

I am using springfox library version 2.3.0 along with spring-boot-1.2.5.RELEASE. I am able to generate Swagger 2 schema without any issues. However there is a requirement to generate the schema in Swagger 1.2 format as well. After reading the documentation and several posts, I managed to get the Swagger2 and Swagger12 schema co-exist using separate Docket definitions. I am able to see both swagger 1.2 and swagger 2 versions of the schema in the swagger-ui as well.
However, in the Swagger 1.2 version of the schema, i see that the info section is missing, which i would expect to be as below
 "info": {
  "title": "test title",
  "description": "sample description",
  "contact"<EMAIL_ADDRESS>  }

Also, the produces section is present, but is blank as below. I would expect it to have
"application/json".
 "produces": [

  ]

My Swagger configuration is as below
@Configuration
@EnableSwagger
@EnableSwagger2
public class SwaggerConfig {

	@Bean
	public Docket swagger2Api() {
		return new Docket(DocumentationType.SWAGGER_2)
				.groupName("group-1")
				.apiInfo(sampleApiInfo())
				.select()
                                .apis(RequestHandlerSelectors.basePackage("com.xyz.controller"))
				.build();
	}

	@Bean
	public Docket swagger12Api() {
		return new Docket(DocumentationType.SWAGGER_12)
				.apiInfo(sampleApiInfo())
				.produces(new HashSet<String>(Arrays.asList("application/json")))
				.forCodeGeneration(true)
				.select()
                                .apis(RequestHandlerSelectors.basePackage("com.xyz.controller"))
				.build();
	}

	private ApiInfo sampleApiInfo() {
		return new ApiInfoBuilder().title("sample title")
				.description("sample description")
				.termsOfServiceUrl("sample-url).contact("abcd").license("xyz)
				.licenseUrl("https://xyz.com/LICENSE").version("0.1").build();
	}

Am i missing something here?

@sankarbalu Not really sure. It should work just fine and there are functional tests that do exactly what you're doing

@dilipkrish The functional test do not seem to check the info section or the produces element in the result. May be if these tests are included, i guess they might fail.

It totally does. Not sure what you're talking about

My bad. I didn't read the Swagger 1.2 specification completely.
After RTFM (https://github.com/OAI/OpenAPI-Specification/blob/master/versions/1.2.md), I realized that the info section is supposed to be present only under the resource listing endpoint at /api-docs, and not at the /api-docs/default/ where i was looking for.
Also the other issue with the empty produces section was resolved when I added the produces and consumes builder methods while generating the ApiInfo.
Thanks a lot for your time @dilipkrish. Appreciate it.


Occasional 500s on discovery.flynn.io

Yesterday I upgraded my Flynn cluster to the most recent nightly build and had some issues getting the nodes to form a cluster. In the past this process was flawless but yesterday I saw some log messages about the discovery service returning a 500. When I run the same requests, i.e. GET https://discovery.flynn.io/clusters/<UUID>/instances, I occasionally see a server error.
Just wanted to let you know. Am I right in assuming this can cause issues when forming the cluster and causing nodes to drop out?

This issue should be resolved now. It could cause issues starting new clusters, but shouldn't affect existing ones.

From my side it looks fine now. Thanks!


Issue for Case Study #1 Deliverable Solved

Issue Link: https://github.com/RRK1000/OSMDashboard-Winter-2024-SOEN-6431/issues/1#issue-2131558280
Proposed Solution:
To resolve the ambiguity and prevent potential clashes, we should rename the trackId variable to something more distinctive, such as trackRecordId. This renaming will enhance readability and reduce the risk of misunderstanding or unintended side effects in the codebase.
Changes Made:

Renamed the instance variable trackId to trackRecordId throughout the TrackPoint class.

Impact:
By addressing this issue, we improve the clarity and maintainability of the codebase. Developers will have a clearer understanding of the purpose and usage of variables, reducing the likelihood of errors and misunderstandings during development and maintenance.

Issue for Case Study #1 Deliverable Solved and Merged into the main branch.


fix: policy hub  post get policy rules response structuring error value map

Description
Enhanced the filter GET/POST for policyContent end point
Why
The current implementation of the GET/POST endpoint in the Policy Hub does not respond as expected when fetching a policy based on specific criteria
Issue
N/a
Checklist
Please delete options that are not relevant.

[ ] I have created and linked IP issues or requested their creation by a committer
[ ] I have performed a self-review of my own code
[ ] I have successfully tested my changes locally
[ ] I have added tests that prove my changes work
[ ] I have checked that new and existing tests pass locally with my changes

@leandro-cavalcante out of curiosity, could you provide a request that was failing?

@Phil91 Yes, now when the value is not present in the DB an exception Is thrown. I.e:


ability to delete a value in JSON

there will be a method that takes the name and deletes the name and the value from JSON,
and there will be another method to dumb the value of that name inside the JSON.
json = Json()
json.delete(name='existing name in the json')
json.dumb(name='dumbing the name value in the json')

finished the features


Foreign error links lose type information

If I have a:
error_chain! {
    links {
        rustup_dist::Error, rustup_dist::ErrorKind, Dist;
        rustup_utils::Error, rustup_utils::ErrorKind, Utils;
    }
}

...and I do:
let e: Error = Error::from(rustup_dist::Error::Foo)

Then there is no way for me to get back a rustup_dist::Error instance from e that would let me determine whether it is a rustup_dist::Error::Foo.
I don't know if it's a good idea, but maybe ErrorKind::Dist could contain a type safe reference to the cause, and not have a boxed cause for that case?

Thanks for the report.
Boxing the cause is a fundamental tradeoff of the crate's design, but you should be able to recover the concrete value by downcasting. You can pull out the box with let (_, ref cause) = error; then use downcast_ref. Untested.

The pattern needed is actually let (_, (ref cause, _)).

Got this approach to work after rubber ducking on IRC. Thanks for the help


feat: add contract address in range validation

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#1799  👈
#1798
#1790
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @ArniStarkware and the rest of your teammates on  Graphite

Merge activity

Nov 5, 3:13 AM EST: A user started a stack merge that includes this pull request via Graphite.


NET-1608: Update host to nm before restart if endpoint changed

Describe your changes
Provide Issue ticket number if applicable/not in title
Provide link to Netmaker PR if required
Provide testing steps
Checklist before requesting a review

[ ] My changes affect only 10 files or less.
[ ] I have performed a self-review of my code and tested it.
[ ] If it is a new feature, I have added thorough tests, my code is <= 1450 lines.
[ ] If it is a bugfix, my code is <= 200 lines.
[ ] My functions are <= 80 lines.
[ ] I have had my code reviewed by a peer.
[ ] My unit tests pass locally.
[ ] Netclient & Netmaker are awesome.

Close it and I will raise a new one.

reopen with new code change.


Add testing

closes #5

Running filewriter tests locally works fine, however the CI workflow hangs on Test_writeFile_OneFile_Success which I imagine will be due to a write permission error, however it was able to successfully create and delete a directory under prior tests...

Adjusted Test_writeFile_OneFile_Success to implement a buffer so that an error can get put on the channel. This also allows errors to be read from that channel later on in the test.
Error is added to the channel, filewriter.go writeFIle line 74, which means it originates from writer.Flush().
Error message: invalid argument. which is strange as Flush() takes no arguments.

solved pipeline issue by changing directory creation to os.Mkdir(dirName, 0777). Files are still created as 0666 by default.

Reduced scope to filewriter.go only as more effort (including a refactor) is required to configure Dependency Injection


Updated computational chemistry

Hi, I went through the Computational chemistry tutorial and updated some alt texts to make them more explanatory hence improving the accessibility of visually impaired users of the materials.
What do you think?

The climate tutorial commit shouldn't be there. Any idea how to take it out? I already created a different PR for it here #3050


Create new release for swagger-ui  v2.1.4

https://github.com/swagger-api/swagger-ui/releases/tag/v2.1.4
Fixes many issues including better support for references. Which may fix
#10

They're up to https://github.com/swagger-api/swagger-ui/releases/tag/v2.2.5 now. I'll try and spend a little time to see how hard it'd be to get it upgraded.


Panic when signing userOp json

I copied this from my scripts
build/intents-sdk sign --u '{
"sender": "0x60ad1b86e41863376921233fff6956150439e576",
"nonce": "0x0",
"initCode": "0x",
"callData": "{\"chainId\":80001, \"sender\":\"0x18Dd70639de2ca9146C32f9c84B90A68bBDaAA96\",\"kind\":\"swap\",\"hash\":\"\",\"sellToken\":\"0xEeeeeEeeeEeEeeEeEeEeeEEEeeeeEeeeeeeeEEeE\",\"buyToken\":\"0xc2132D05D31c914a87C6611C10748AEb04B58e8F\",\"sellAmount\":10,\"buyAmount\":5,\"partiallyFillable\":false,\"status\":\"Received\",\"createdAt\":0,\"expirationAt\":0}",
"callGasLimit": "0x493E0",
"verificationGasLimit": "0x493E0",
"preVerificationGas": "0x493E0",
"maxFeePerGas": "0x0",
"maxPriorityFeePerGas": "0x0",
"paymasterAndData": "0x",
"signature": ""
}'
Signer private key: 0x860304ed3cb246bd492a84510742b5fd42358968e60e3ff9e9f8287b24a05f6a
Public key: 5b4258c6914408f003fccd952645fc9c64b5eb4dde18fa2d7252fcde8f0c73bc391d4c29c35631e5e5a0bc35721e6f17819e28bf25021d047c81997453a92656
Address: 0xa4BFe126D3aD137F972695dDdb1780a29065e556
Entrypoint Address: 0x5FF137D4b0FDCD49DcA30c7CF57E578a026d2789
panic: error parsing user operation JSON: empty hex string

goroutine 1 [running]:
github.com/blndgs/intents-sdk/utils.GetUserOps(0x49cdfdb0d437f15f?)
        /Users/mar/go/src/github.com/blndgs/intents-sdk/utils/common.go:64 +0x104
github.com/blndgs/intents-sdk/cmd.glob..func3(0x14000538800?, {0x102e19202?, 0x4?, 0x102e19206?})
        /Users/mar/go/src/github.com/blndgs/intents-sdk/cmd/sign_userop.go:30 +0x6c
github.com/spf13/cobra.(*Command).execute(0x1033d8ec0, {0x14000127160, 0x2, 0x2})
        /Users/mar/go/pkg/mod/github.com/spf13/cobra@v1.6.1/command.go:920 +0x654
github.com/spf13/cobra.(*Command).ExecuteC(0x14000142600)
        /Users/mar/go/pkg/mod/github.com/spf13/cobra@v1.6.1/command.go:1044 +0x320
github.com/spf13/cobra.(*Command).Execute(...)
        /Users/mar/go/pkg/mod/github.com/spf13/cobra@v1.6.1/command.go:968
main.main()
        /Users/mar/go/src/github.com/blndgs/intents-sdk/main.go:24 +0xd4

This panic is because signature is empty that is mandatory field by model and unmarshing giving error:
https://github.com/blndgs/model/blob/d1daed6a2d3a612f6967880a39ddb19943cc98d0/userops.go#L59
should we remove validate or maintain a seperate struct in SDK?

This panic is because signature is empty that is mandatory field by model and unmarshing giving error:
https://github.com/blndgs/model/blob/d1daed6a2d3a612f6967880a39ddb19943cc98d0/userops.go#L59
https://github.com/blndgs/intents-sdk/blob/aca334235729ee23f74b72f800a7e370e3f292b8/utils/common.go#L60
should we remove validate or maintain a seperate struct in SDK?

Gotcha, we can detect being empty and insert an acceptable 0 value, i.e., allocated empty or '0x' or '0x0'.
Considering the SDK's public profile usage, it's not good UX to require the user to fill it out with a placeholder for an unsigned userOp.
What do you think?


Updated the video backend to support the latest hxvlc release.

see https://github.com/FNF-CNE-Devs/CodenameEngine/pull/227
quoted from @NexIsDumb

the video library is receiving many updates though @NeeEoo is stuck with the older version cause he wants to use that for now
i already asked him and he told me to use the commit he was using on that branch instead of making changes about it in the source


Templated pages fail to build when generated from query with date input

Steps To Reproduce
Builds with templated pages fail when templated pages generated from query containing date input
https://github.com/davidgasquez/templated-pages-test/tree/repro-prerendered-issue
npm run sources
npm run build

Environment

Node version (node -v): 20
npm version (npm -v):
OS: macos
Browser:
Database: duckdb

Expected Behavior
Build completes
Actual Behaviour
Build fails at final stage
✓ built in 19.99s

node:internal/event_target:1083
  process.nextTick(() => { throw err; });
                           ^
Error: The following routes were marked as prerenderable, but were not prerendered because they were not found while crawling your app:
  - /orders/[order]

See https://kit.svelte.dev/docs/page-options#prerender-troubleshooting for info on how to solve this
    at prerender (file:///Users/archie/Projects/templated-pages-test/node_modules/@sveltejs/kit/src/core/postbuild/prerender.js:495:9)
    at async MessagePort.<anonymous> (file:///Users/archie/Projects/templated-pages-test/node_modules/@sveltejs/kit/src/utils/fork.js:22:16)
Emitted 'error' event on Worker instance at:
    at [kOnErrorMessage] (node:internal/worker:326:10)
    at [kOnMessage] (node:internal/worker:337:37)
    at MessagePort.<anonymous> (node:internal/worker:232:57)
    at [nodejs.internal.kHybridDispatch] (node:internal/event_target:807:20)
    at exports.emitMessage (node:internal/per_context/messageport:23:28)

Node.js v20.6.1
Build failed

file:///Users/archie/Projects/templated-pages-test/node_modules/@evidence-dev/evidence/cli.js:178
                        throw `Build process exited with code ${code}`;
                        ^
Build process exited with code 1
(Use `node --trace-uncaught ...` to show where the exception was thrown)

Node.js v20.6.1

Workarounds
Anything you can do in advance of this being fixed?

workaround:

use a non-filtered version of the query
populate a datatable with links based on that query, but put it inside a <div class="hidden">

svelte will then pick up the pages for rendering


Creation of Focus Groups for CHAOSS Asia

Fixes #6 and #9
cc: @xiaoya-yaya @afro-coder @sandeepkanabar - Please review the documents and advise if any changes are required.
FYA: @ElizabethN

I'll fix the DCO issue and incorporate it in my next commit. There are some issues with the network at my end and I'm unable to push any commits successfully to the branch.

Looks good to me, thanks Divya!

LGTM:) thanks Divya


fails to link with libtorch using torch-sys

with torch-sys in cargo deps by default if torch is not installed system-wide cargo will build and install libtorch by itself somewhere to target/ and application if run with cargo run will find and link with it.
with cargo-tarpaulin library is not searched on those paths and it fails with missing libtorch
my example: https://github.com/Miezhiko/Amadeus/blob/mawa/.github/workflows/codecov.yml

Let me try and figure out how cargo test makes them visible to the tests, because if you run cargo test it links to it, but then if you take the generated test binary and run it afterwards it will fail with the linker issue unless the correct path is part of the LD_LIBRARY_PATH. And that's the issue tarpaulin is having

yes, cargo somehow adds proper libs paths to -sys crates using bundled libs when you do cargo run or cargo test

I've got a PR branch for this, tested it on a small torch project and I'll check it on Amadeus later this evening  https://github.com/xd009642/tarpaulin/pull/904

It worked on Amadeus, so I've merged the PR. I'll aim to do a release next week most likely, but until then if you install via the develop branch or if you use docker the next develop image to be pushed out will have the fix in :+1:


Updated license. Please accept and merge.

Sorry.
I do not like ADs.
Remove ads and make new PR

You're free to modify it as you wish. This pull request is just an upstream
update pull with all the latest changes. And more importantly, an update to
the open source license. (and btw, the ads have always been there from
version 1 and was not added newly in this PR)... You can always modify it
and remove ads in your fork...
On Wed 16 Dec, 2015 5:38 pm rancidfrog<EMAIL_ADDRESS>wrote:

Sorry.
I do not like ADs.
Remove ads and make new PR
—
Reply to this email directly or view it on GitHub
https://github.com/rancidfrog/show-java/pull/1#issuecomment-165087416.

Since this PR has an important update to the open source license, I request to accept and merge it.


Upgrade Agones to 1.29.0 and Open Match to 1.7.0

This PR has a upgrade to Agones and Open Match and change of installing for Open Match

Upgrade GKE Kubernetes version from 1.22 to 1.24
Upgrade Agones from 1.27.0 to 1.29.0
Upgrade Open Match from 1.5.0 to 1.7.0
Change instaling way for Open Match ※1

※1 One of the Open Match installing way is YAML apply. The default YAML requires more than 4.5 cpu cores, because the StatefulSet's Redis has 3 containers(redis, sentinel, metrics) requre each 0.5 cpus(= total 1.5 cores) and StatefulSet has 3 reqlica(= 1.5cores * 3Pod = total 4.5 cores)

I'll assign review after 3/7.

Would you revise README.md in accordance with your PR ?

I added the helm section.
I think helm is the neccesary, not optional. So, I removed (Optional).

@kemurayama
What do you mean about need to remove the duplicate line.?
Could you please point out the line with link?

Hi @govargo I mentioned this line. Thanks.
https://github.com/govargo/space-agon/blob/b49c440a0f32ac405bd40609d19c695bc4bb1615/README.md?plain=1#L27

Thank you for your point out @kemurayama.
It's my fault. I fixed this duplicate line.


Passing variables to tmpdir and finaldir

Calling the plotter from a bash script with these variables:
TEMPDIR=/mnt/tmp/
DESTDIR=/mnt/dst/
./chia_plot --tmpdir ${TMPDIR} --finaldir ${DESTDIR} (rest of command here)
gives this error:
Invalid tmpdir: --finaldir (needs trailing '/' or '\')
Only a problem when using variables.
Notice also that error message mixes tmpdir and finaldir.

I don't think it is mixing tmpdir and finaldir. As far as it is concerned, you have specified '--finaldir' as the input to the tmpdir option.
Essentially your variables have disappeared / turned blank.

Upon closer examination, it's because you created a variable called TEMPDIR (with an E in its name), but you specified ${TMPDIR} without an E

You're absolutely right, how embarrassing. Sorry to waste your time.


Empty blocks when running mev networks without regular geth node

Hey, I'd like to run a network without a regular geth node. To do that I use the command:
kurtosis run github.com/kurtosis-tech/eth2-package '{"mev_type": "full", "participants": []}'
The network is created, however blocks generated by the network do not include any transactions.
Why do the network generate empty blocks only?

It appears fuzzers use eth_sendRawTransaction method while builder supports only eth_sendPrivateTransaction and eth_sendBundle.
https://docs.flashbots.net/flashbots-protect/additional-documentation/eth-sendPrivateTransaction
https://docs.flashbots.net/flashbots-auction/searchers/advanced/rpc-endpoint#eth_sendbundle

Sending transactions to builders work properly. I've just noticed that the problem with mev-flood is the fact that contract deploy transactions are included in blocks at 4th epoch, however the flood tries to spam transactions since 0 epoch. Thus the command crashes. There is no problem when a regular node is part of the network since it is including transactions in blocks since 0 epoch.
@h4ck3rk3y is there a way to delay the run of fuzzers? Starting fuzzers at 4th epoch would probably resolve the issue

@piwonskp I missed this message. Is this still an issue!?
https://github.com/kurtosis-tech/ethereum-package/blob/main/main.star#L217-L224
^ We wait for the capella fork epoch at the moment but we can move it to before the mev_flood spam in background to go with your logic in case capella fork epoch < 4


Can't start vite sh: vite: command not found

Describe the bug
I'm trying to start a project using vite, but I keep getting sh: vite: command not found.
After reading https://github.com/vitejs/vite/issues/5381, https://github.com/vitejs/vite/issues/1215, and the discussion https://github.com/vitejs/vite/discussions/5432, I tried installing vite globally via npm i -g vite. Now I get the following error:
Error [ERR_MODULE_NOT_FOUND]: Cannot find package '@vitejs/plugin-react' imported from /Users/.../vite.config.js.timestamp-1675787579111.mjs

Looking at node_modules, there's a folder named @vitejs, but the folder is empty. Was this expected?
I'm not sure how to get vite running.
PS: I won't add a reproduction URL because it seems to be a problem with my system being incompatible with vite, not from vite itself.
Reproduction

Steps to reproduce
npm create vite@latest

continue with default name "vite-project"
select React
select TypeScript + SWC

cd vite-project
npm i
npm run dev
result:
npm run dev

><EMAIL_ADDRESS>dev
> vite

sh: vite: command not found

System Info
System:
    OS: macOS 13.0
    CPU: (8) arm64 Apple M1 Pro
    Memory: 71.13 MB / 16.00 GB
    Shell: 5.8.1 - /bin/zsh
  Binaries:
    Node: 18.14.0 - ~/.nvm/versions/node/v18.14.0/bin/node
    Yarn: 1.22.19 - /usr/local/bin/yarn
    npm: 9.3.1 - ~/.nvm/versions/node/v18.14.0/bin/npm
  Browsers:
    Brave Browser: <IP_ADDRESS>
    Safari: 16.1

Used Package Manager
npm
Logs
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Make sure this is a Vite issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to vuejs/core instead.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

Just found the problem and solution. It seems I had NODE_ENV set to "production". Not sure how that happened, but setting it to "dev" made everything work.
Is this expected behavior or do we want to change it?
It's the first time I face a problem related to having NODE_ENV set locally. No other package I've used before has given me any problem. Is that a bit too fragile?
I'll close this issue and open a discussion to keep track of this.


Error in Configuring  .net web application project(.net framework) project

Problem encountered on https://dotnet.microsoft.com/en-us/learn/aspnet/blazor-tutorial/create
Operating System: windows
Provide details about the problem you're experiencing. Include your operating system version, exact error message, code sample, and anything else that is relevant.

This seems to fall outside of the Blazor tutorial since it's a .NET Framework project. Please report the issue through Visual Studio
https://learn.microsoft.com/en-us/visualstudio/ide/how-to-report-a-problem-with-visual-studio?view=vs-2022


Feature Request: support for WeChat's page

Type

[ ] General question or discussion
[ ] Propose a brand new feature
[x] Request modification of existing behavior or design

What is the problem that your feature request solves
I archived a page form Wechat Open Platform (such as https://mp.weixin.qq.com/s/ri4nDgPQo4OVWaIWG9EQZA)
but I found that all images on the page can't be archived.
(https://archive.sager.wang/archive/1580637904/mp.weixin.qq.com/s/ri4nDgPQo4OVWaIWG9EQZA.html)
and the title also show "Unable to detect page title" in index of the archive box.
WeChat is the biggest IM in China and it has the strictest censor there.
So I am hopping archive box can archive the page with images.
I am sorry for my bad English :-)
Describe the ideal specific solution you'd want, and whether it fits into any broader scope of changes
I hope archive box can archive the page (with images) from Wechat Open Platform.
What hacks or alternative solutions have you tried to solve the problem?
archive.is can archive the page from wechat open platform with images.
How badly do you want this new feature?

[ ] It's an urgent deal-breaker, I can't live without it
[x] It's important to add it in the near-mid term future
[ ] It would be nice to have eventually

[ ] I'm willing to contribute dev time / money to fix this issue
[x] I like ArchiveBox so far / would recommend it to a friend
[ ] I've had a lot of difficulty getting ArchiveBox set up

Unable to detect page title
It's caused by wechat media platform(article) page has an empty  field.

Maybe archivebox could try to parse og:title twitter:title fields to get title.

@pirate the title is not present, is it ok to look in the og:title tag as a fallback?  (the retrieval is stopping because of this)
About the images, they are being lazy-loaded. I am not sure if wget can handle that, but that is something we can check after the title issue is fixed.

Yeah you can look in og:title, but lets not handle the image lazy loading right now, that's a very complex problem.

Greetings. I wonder if it is possible at this point to revisit this problem?
Given the prominence of Wechat (and the most comprehensive censoring, as OP also mentioned), I'd wager that articles on mp.weixin.qq.com are probably the most common target for archiving for users in China. It certainly is the case for me, as most of the articles I have felt a need to archive are on there. It would be wonderful to have ArchiveBox available for this usage, especially as the archive.* sites now block all web host proxy users.
Apologies for digging up an old thread and thank you all for your hard work.

I recommend seeing if they can be archived with SingleFile, and if not, raising the issue on that repo.  ArchiveBox does not itself do any archiving, it's just a collection of other utilities that do the actual archiving. If there are issues with archive fidelity in general the issue is to raise those issues with the sub-utilities or add a new extractor.

ArchiveBox does not have granular control over how the singlefile capture is done, we only call the SingleFile CLI. Unless they provide a CLI option to scroll before capturing, we cannot do that.

Hello, I found that SingleFile CLI has an option --load-deferred-images-dispatch-scroll-event. When it is enable, lazy-loaded images can be saved perfectly (at least on WeChat's Page).
https://github.com/gildas-lormeau/single-file-cli/blob/master/args.js#L206

Thats a great find! 🥳 Thanks. Lets add that option to archivebox by default then.

On a lot of my pages, turning --load-deferred-images-dispatch-scroll-event to true causes some "subscribe to my newsletter" popup to come up while having it off prevents this and still loads deferred images. So it probably shouldn't be a default.
This is all moot though because the bundled singlefile is outdated and doesn't support deferred images at all right now


Added cregbundle parameter to mpl drawer

Summary
Fix #4290
Details and comments
Added the 'cregbundle' parameter to the matplotlib_circuit_drawer function and the MatplotlibDrawer class. The default value for all drawers is now True.
The style dictionary key, 'cregbundle' has been deprecated and a warning is displayed if used. For now if the user sets this key in the style dict, it will override the parameter setting. All references to 'bundle' have been removed from qcstyle.py.

I think @1ucian0 didn't set the default to True as there were some bugs in the text drawer still? I do prefer a default of True across the board if there aren't obvious bugs.

Yeah. @1ucian0 said to go ahead and set it to True in #4290, and there were no test errors.

I think @1ucian0 didn't set the default to True as there were some bugs in the text drawer still? I do prefer a default of True across the board if there aren't obvious bugs.

Just for the release. Now that we have some time in master, let's flip it to true and hunt those bugs.

MPL circuit drawer issues are piling up, since we dont have a good way to track changes on it. I'm putting this PR on hold (meaning "do not merge yet") until https://github.com/Qiskit/qiskit-terra/pull/4544 (or a similar solution) gets in.

#4544 is merged... Let's add a test with the new method!
I will try helping, but fill free to do it yourself and ask me if any question.

I added the test here 98e4a11 !
Check the result here: https://mybinder.org/v2/gh/enavarro51/qiskit-terra/fixissue4290branch?urlpath=apps/test/ipynb/mpl_tester.ipynb

@1ucian0 #4597 needs to be merged before this one will pass the auto tests.

Also I looked at your binder. I think there ought to be 2 cregbundle images, one for True and one for False. And each time I click on one of your binder links, it goes through a several minute long build. Is this normal?

@1ucian0 #4597 needs to be merged before this one will pass the auto tests.

Good point! I will start pushing that one. This PR on hold until #4597 gets merged.

Also I looked at your binder. I think there ought to be 2 cregbundle images, one for True and one for False.

Not sure what you mean by this... is the test failing?

And each time I click on one of your binder links, it goes through a several minute long build. Is this normal?

Yes. That's normal. If the branch has no updates, there is no need in click the link again. You would see the same results.

Oh, what I meant is that there should be a reference image that shows cregbundle True and a separate image that shows cregbundle False, each of which would be compared to the output of the changed code. One might be affected by a change while the other one isn't.

@1ucian0 #4597 needs to be merged before this one will pass the auto tests.

Good point! I will start pushing that one. This PR on hold until #4597 gets merged.

Hold on.. #4597 affects latex, not MPL. Why should that one block this one?

Because this PR sets cregbundle=True as default and the latex version crashes when cregbundle=True, for the test_teleport in test_visualization.py. That's what @rao107 fixed.

I see, this PR changes the default. That is not necessary, we have a different PR for that https://github.com/Qiskit/qiskit-terra/pull/4595
Smaller single-issue PR are prefered to avoid blocking.

I extended the test.

This one was covered by PR #4616.


Arrange food, drink, and venue access for OKC Ruby event | 2016-11-10

What: Adventures in testing a monolith
When: 2016-11-10 11:30
Where: 1141 W Sheridan Ave, Oklahoma City
Check meetup for RSVP count: http://www.meetup.com/OKC-Ruby/events/235463230/

2 large ultimate pepperoini, 1 large spinach and feta, 1 garden salad, 1 coke, 1 sprite. $81 total.
Turns out Jimmy John's doesn't deliver to ss46, it's too far away.


feat: customize Identity firstUser creation

Which problem does the PR fix?
Fixes: #645
Should be merged before: #728
What's in this PR?

Support disabling the creation of Identity firstUser.
Support using a custom secret for Identity firstUser.

Checklist
Please make sure to follow our Contributing Guide.

Before opening the PR:

[x] There is no other open pull request for the same update/change.
[x] The commits follow our Commit Guidelines.
[x] Tests for charts are added (if needed).
[x] The main Helm chart and sub-chart are updated (if needed).
[x] In-repo documentation are updated (if needed).

After opening the PR:

[x] Did you sign our CLA (Contributor License Agreement)? It will show once you open the PR.
[x] Did all checks/tests pass in the PR?

One small note: When you install the helm chart, there is this notice:
...
Now you can point your browser to one of the service's login pages. Example: http://localhost:8081 for Operate.

Default user and password: "demo/demo"

even if you set the first user to jesse/jesse.
I don't think we necessarily need to change this, but wanted to make you aware of it.

I manually tested that the first user creation works via secret as well as specifying password directly. I also manually disabled first user and that works as well. :+1:
LGTM

One small note: When you install the helm chart, there is this notice:
...
Now you can point your browser to one of the service's login pages. Example: http://localhost:8081 for Operate.

Default user and password: "demo/demo"

even if you set the first user to jesse/jesse. I don't think we necessarily need to change this, but wanted to make you aware of it.

That's actually a good point :star_struck:
I've fixed it :ok_hand:


General: add consistent handling if no more unassessed submissions are available

Checklist
General

[x] I tested all changes and their related features with all corresponding user types on a test server.
[x] This is a small issue that I tested locally and was confirmed by another developer on a test server.
[x] Language: I followed the guidelines for inclusive, diversity-sensitive, and appreciative language.
[x] I chose a title conforming to the naming conventions for pull requests.

Server

[x] I followed the coding and design guidelines.
[x] I documented the Java code using JavaDoc style.

Motivation and Context

Description
The endpoints for getting submission without assessments

/exercises/{exerciseId}/modeling-submission-without-assessment
/exercises/{exerciseId}/programming-submission-without-assessment
/exercises/{exerciseId}/text-submission-without-assessment
/exercises/{exerciseId}/file-upload-submission-without-assessment

used to throw an exception that returns an 404 error to the client.
Since not having anymore unassessed submissions is normal behavior, returning null is a better solution.
Steps for Testing

Prerequisites:

1 Instructor
1 Student

instructor: Setup exercises for each type i.e. Programming Exercise, Modeling Exercise, etc.
student: Create submissions for each exercise
instructor: Check if you can assess/access the submissions for the exercise dashboards.

(Additionally check if you get an error after reaching the last submission)
Review Progress

Code Review

[x] Review 1
[x] Review 2

Manual Tests

[x] Test 1
[x] Test 2

Screenshots

@kekskatze good catch. I noted that and will look into further inconsistencies in the follow up


State, Tribal, Local, and Territorial (STLT): enhancement to daily-data and report-detail pages

ON HOLD until pipeline team's refactoring of historyfunction.kt is done.
Epic used to bundle enhancements to current STLT feature on daily-data and report-details pages
Problem Statement
As a person working at a public health authority, I need pagination/sorting/filtering on detailed-report and daily-data pages. I also need a summary list of facilities and senders in production and in test modes. Later, I’ll need summary, descriptive stats of data coming from reportstream.

All issues complete.


Add a Calorie Calculator to the tools Page

CURRENT BEHAVIOUR: The Tools Page has a dummy card at the Last.

EXPECTED BEHAVIOUR: Make a Calorie Calculator Page and link the Dummy card to that page.
For Reference :  https://www.calculator.net/calorie-calculator.html

I would like to work under this issue for JWOC.


txadmin not working with zap vserver

i dont now why it is not working but we  want to release our Server today we worked hard for it ****
https://prnt.sc/tbz0js
https://prnt.sc/tbzjwz
please answer as fast as you can i Need to fix that

Hey, for support join the discord server, github issues are for... Issues...Anyways, what it's showing on the screenshots are a sigabrt, thst is not caused by txAdmin.

I Need the Dc link

you need to read the docs then
learn how to use your eyes


encoder updated twice for curl loss

I'm looking into the code and find that in def update_cpc()  both self.encoder_optimizer.step() and self.cpc_optimizer.step() are called. However the parameters of critic.encoder are carried by both optimizer. Isn't it true that, in def update_cpc(), critic.encoder is updated twice using the same gradient?

@xy9485 do you know why critic.encoder is updated twice in def update_cpc()?

Yeah that's true. I don't know the reason of doing this.


ePaper Library

Hi! I got the sketch working pretty quickly on my ESP & Waveshare 2.9" board.  Very cool! Did you write the library for the epd? It is substantially better than the one Waveshare provides on their website.  If so, could you publish it as a standalone library, I've tried merging your EPD_drive.cpp/.h with the ones from the Waveshare demo without much luck since I'm still new to cpp.
Thanks!

Waveshare library display strings with part refresh, stores font in header file.
My library write all strings and pictures into buffer and refresh the display once, stores font in spiff.
so they can't work together. even though you merge them, you can only use one method to refresh the display.

Thanks for the tip! I like the buffer approach much better than what was in the demo code.   What do you think is the best way to display text rotated 90°?  This epaper library for the ESP32 supports arbitrary rotation but I don't know how to adapt the code to work with Arduino

Add some simple codes to transform the coordinates when operating the buffer to  change the direction of text.  if you can't handle that, you can wait for some days, i will do that.

I can give it a try but pretty new to cpp so it may be ugly!
On June 30, 2017 5:24:20 AM EDT, Hui Lu<EMAIL_ADDRESS>wrote:

Add some simple codes to transform the coordinates when operating the
buffer to  change the direction of text.  if you can't handle that, you
can wait for some days, i will do that.
--
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub:
https://github.com/duck531a98/esp8266-weather-station-epaper/issues/1#issuecomment-312220228


Only one device (Mi smart bulb essential) not working

The plugin is great and works with all my devices except one: a Mi smart bulb essential.
Actually I have 2 Mi smart bulb essentials named as Hall 1 and Hall 2.
The Hall 2 lamp  works perfectly but the  Hall 1 lamp does not respond.
I have cross checked the IP address with the router and the Mi home app. LAN is enabled in both.
The one thing that is different from the other identical lamp is that the server detected is cn (china) while all other devices are (de) Germany.  I have also tried with the China server but it did not work.
Your config.json
"devices": [
{
"name": "Mi Hall 1",
"ip": "<IP_ADDRESS>",
"token": "ec992e8b2d66223bd88eff57b152c25c",
"deviceId": "248175669",
"model": "yeelink.light.color5",
"pollingInterval": 10,
"deepDebugLog": true,
"micloud": {
"username": "xxxxxxxxxxxxx",
"password": "xxxxxxxxxxxxx",
"country": "de",
"forceMiCloud": false
},
"buzzerControl": false,
"ledControl": true,
"childLockControl": false,
"modeControl": true
},
{
"name": "Mi Hall 2",
"ip": "<IP_ADDRESS>",
"token": "d1ad7fd9fd96b2ccf9b2bc56b48c6d47",
"pollingInterval": 10,
"deepDebugLog": false,
"micloud": {
"username": "xxxxxxxxxxxxx",
"password": "xxxxxxxxxxxxx",
"country": "de"
},
"buzzerControl": true,
"ledControl": true,
"childLockControl": true,
"modeControl": true
},
{
"name": "YL Office",
"ip": "<IP_ADDRESS>",
"token": "faf860772ef1e1464091479ed664d351",
"pollingInterval": 10,
"deepDebugLog": false,
"micloud": {
"username": "xxxxxxxxxxxxx",
"password": "xxxxxxxxxxxxx",
"country": "de"
},
"buzzerControl": true,
"ledControl": true,
"childLockControl": true,
"modeControl": true
},
{
"name": "YL Maria's",
"ip": "<IP_ADDRESS>",
"token": "d0e827d6b8a0234b4c86003ce616cdf5",
"pollingInterval": 10,
"deepDebugLog": false,
"micloud": {
"username": "xxxxxxxxxxxxx",
"password": "xxxxxxxxxxxxx",
"country": "de"
},
"buzzerControl": true,
"ledControl": true,
"childLockControl": true,
"modeControl": true
},
{
"name": "YL Living Room",
"ip": "<IP_ADDRESS>",
"token": "6f5cd8c64c574d4f03fb07a7b3d622ae",
"pollingInterval": 10,
"deepDebugLog": false,
"micloud": {
"username": "xxxxxxxxxxxxx",
"password": "xxxxxxxxxxxxx",
"country": "de"
},
"buzzerControl": true,
"ledControl": true,
"childLockControl": true,
"modeControl": true
}
],
"platform": "miot"
The Debug log is as follows ( I have removed irrellevant parts):
4/7/2022, 1:28:49 PM] [miot] [Mi Hall 1] (Protocol) <IP_ADDRESS> -> Handshake reply
[4/7/2022, 1:28:49 PM] [miot] [Mi Hall 1] (Protocol) <IP_ADDRESS> <- (2) {"method":"miIO.info","params":{"timeout":5000,"retries":3},"id":3619}
[4/7/2022, 1:28:49 PM] [miot] [Mi Hall 1] (Protocol) <IP_ADDRESS> <- !1p��5�a�9�+@L0FpW���
[4/7/2022, 1:28:53 PM] [miot] [Mi Hall 1] (Protocol) Start handshake <IP_ADDRESS>
[4/7/2022, 1:28:53 PM] [miot] [Mi Hall 1] (Protocol) <IP_ADDRESS> -> Handshake reply
[4/7/2022, 1:28:53 PM] [miot] [Mi Hall 1] (Protocol) <IP_ADDRESS> <- (1) {"method":"miIO.info","params":{"timeout":5000,"retries":3},"id":3719}
[4/7/2022, 1:28:53 PM] [miot] [Mi Hall 1] (Protocol) <IP_ADDRESS> <- !1p��5�e��|�[��#�"k
[4/7/2022, 1:28:54 PM] [miot] [YL Maria's] Splitting properties into chunks. Number of chunks: 1. Chunk size: 14
[4/7/2022, 1:28:54 PM] [miot] [Mi Hall 2] Splitting properties into chunks. Number of chunks: 1. Chunk size: 14
[4/7/2022, 1:28:54 PM] [miot] [YL Office] Splitting properties into chunks. Number of chunks: 1. Chunk size: 14
[4/7/2022, 1:28:54 PM] [miot] [YL Living Room] Splitting properties into chunks. Number of chunks: 1. Chunk size: 14
[4/7/2022, 1:28:57 PM] [miot] [Mi Hall 1] (Protocol) Start handshake <IP_ADDRESS>
[4/7/2022, 1:28:57 PM] [miot] [Mi Hall 1] (Protocol) <IP_ADDRESS> -> Handshake reply
[4/7/2022, 1:28:57 PM] [miot] [Mi Hall 1] (Protocol) <IP_ADDRESS> <- (0) {"method":"miIO.info","params":{"timeout":5000,"retries":3},"id":3819}
[4/7/2022, 1:28:57 PM] [miot] [Mi Hall 1] (Protocol) <IP_ADDRESS> <- !1p��5�i{�~����,�F!���
[4/7/2022, 1:29:01 PM] [miot] [Mi Hall 1] (Protocol) <IP_ADDRESS> <- Reached maximum number of retries, giving up miIO.info - {"timeout":5000,"retries":3}
[4/7/2022, 1:29:01 PM] [miot] [Mi Hall 1] Error: Call to device timed out
[4/7/2022, 1:29:01 PM] [miot] [Mi Hall 1] Could not connect to the device! Retrying in 40 seconds!
[4/7/2022, 1:29:04 PM] [miot] [YL Maria's] Splitting properties into chunks. Number of chunks: 1. Chunk size: 14
[4/7/2022, 1:29:04 PM] [miot] [Mi Hall 2] Splitting properties into chunks. Number of chunks: 1. Chunk size: 14
[4/7/2022, 1:29:04 PM] [miot] [YL Office] Splitting properties into chunks. Number of chunks: 1. Chunk size: 14
[4/7/2022, 1:29:04 PM] [miot] [YL Living Room] Splitting properties into chunks. Number of chunks: 1. Chunk size: 14
Screenshots

Thank you !

My suspicion would be that the information in your micloud account might be not up to date, like the ip or token might be incorrect. You should try to reset the device and re-add to your account.

Thank you for the suggestion. I reseted the lamp and added back to my account.
I then run the Discover all Devices via Micloud and nothing changed, same token, device name, ip address and server (cn).
Same unreachable behaviour as before, how could I refresh my Mi account data?

Re-add the device in your mi home app.

I re-added it to my account, re-enabled LAN control but without solving the problem.
Maybe this one lamp is different somehow and does not update with the Mi server or country.
My problem seems not to be related to the plugin itself, so you can close this issue if there are no further suggestions.

The lamp is the the same model as the other working lamp which you got, so this lamp should also work. Seeing that the handshake is failing indicates that it is probably the token what is wrong.

Thanks, I will try to retrieve the token in another way.

Problem solved. Extracted the tokens from an iOS backup with DB browser.
All tokens, except for the problematic lamp were correctly extracted.
I used the new token for the Mi Hall 1 lamp and it worked like  charm.

Perfect, good to hear :) For some reason the token for that device is outdated in your micloud account... not sure why.


Chinese traineddata does not use ISO 639-3 code

The chinese models chi_sim, chi_sim_vert, chi_tra and chi_tra_vert use chi instead of zho which is the ISO 639-3 standard.
Since all other files are named according to this standard I dont see a reason why this should not be the case for chinese.

AFAIK chi is (also) valid code for Chinese:
https://www.w3.org/WAI/ER/IG/ert/iso639.htm
https://www.loc.gov/standards/iso639-2/php/code_list.php
https://iso639-3.sil.org/code/chi
Never rely on one source (especially if anybody can change it).

The Tesseract is not clear about the question whether ISO 639-2 or ISO 639-3 is used:
doc/combine_lang_model.1.asc:   Tesseract uses 3-character ISO 639-2 language codes. (See LANGUAGES)
doc/tesseract.1.asc:  Tesseract uses 3-character ISO 639-2 language codes
include/tesseract/baseapi.h:   * The language is (usually) an ISO 639-3 string or nullptr will default to
src/api/baseapi.cpp: * The language is (usually) an ISO 639-3 string or nullptr will default to eng.

I don't mind fixing this and using ISO 639-3 everywhere, but would like to get more feedback here from people who are affected by a renaming of the models for Chinese.

Thats really unfortunate, that tesseract it self isnt even consistent with the used standard.
Note the (usually) 😆

There exist more ISO 639-3 codes for Chinese variants (at least cmn, yue, nan).

Yes those are for variants of the language although Id argue that the model is applicable for the macro language chinese hence zho fits best. Ofcause it would also be possible to have multiple models for individual chinese languages.

So the suggestion is that the OCR models starting with chi_ be renamed to zho_ instead.
Comments from Chinese users of Tesseract are welcome.


Add disclaimer to rendered book?

The Rust Reference has a big fat red disclaimer at  the top of every page:

For now, this reference is a best-effort document. We strive for validity and completeness, but are not yet there. In the future, the docs and lang teams will work together to figure out how best to do this. Until then, this is a best-effort attempt. If you find something wrong or missing, file an issue or send in a pull request.

In comparison, the rendered "RFC book" has nothing like that. That's a problem, because we generally don't update RFCs after they have been accepted -- so they are frequently outdated or at least not explained in the best possible way.
So I suggest we also add a disclaimer to the RFC book, saying something like
"An RFC is a historic document that reflects the consensus at the time it was accepted. They are, in general, not updated when things change later. For current information, check the reference and the nomicon."
Cc @Centril @gnzlbg

Yes this would be really nice.

cc @nikomatsakis


Fix typo (URDNA2014 => URDNA2015).

@gkellogg,
With this one change (SHA-1 to SHA-256) and updates to the tests (which I've updated in the json-ld.org normalization repo already), your Ruby implementation's output for URDNA2015 matches the one I just finished in jsonld.js.

Cool, thanks Dave!


errors when iterating a path

When iterating a path, but also activelty altering the database, an error is thrown at the end

07/08/2022 04:34:32 PM <6004> sqlalchemy.pool.impl.QueuePool: [ERROR] Error closing cursor
Traceback (most recent call last):
  File "/home/bonan/miniconda3/envs/aiida-2.0/lib/python3.9/site-packages/sqlalchemy/engine/cursor.py", line 1100, in fetchmany
    new = dbapi_cursor.fetchmany(size - lb)
psycopg2.ProgrammingError: named cursor isn't valid anymore

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/home/bonan/miniconda3/envs/aiida-2.0/lib/python3.9/site-packages/sqlalchemy/engine/base.py", line 1914, in _safe_close_cursor
    cursor.close()
psycopg2.ProgrammingError: named cursor isn't valid anymore

Example code template:
for path in base_path:
    .... do ... somthing
    submit(builder)

One way to get around this issue is to collect the iteration results first.
for path in list(base_path):
    .... do ... somthing
    submit(builder)

This did not raise any issue in AiiDA v1. Perhpas the collection of iteration results should be the default behaviour?

already solved


Cannot move to sub-forum

A user made two posts (cross-posts). One was in Italiano, and the other in Italiano -> Hardware.
After a "report to moderator" I tried to merge the two threads. I can't merge directly because I (still) get this error:

So until that is fixed, these days I first move the thread to the new forum section, and then merge them.
However when I tried to move one of the threads I see this:

You will note that there is no Italian section there at all.
However if I try to move something from another part of the forum, you can see all the Italian parts:

I understand suppressing the exact sub-forum from the move (after all you don't want to move onto itself) but the other sub-forums should be available. Otherwise that thread can't be merged or moved at all (in the Italian section).

When is this going to be fixed? Someone now wants me to move a Spanish post into Spanish -> Documentation. I cannot.

Hi nickgammon. Thanks so much for reporting this bug! Since the moderation system has been completely changed by the switch to Discourse this week, this issue is now obsolete. If anyone does spot this problem happening again, please comment here and I'll reopen the issue.


Management of Bitbucket repositories

I've been continued developing some small features I really need for using Deployinator as our main deployment-tool. The first one has been regarding to Bitbucket, our hosting for git projects.
Deployinator's helpers for git actions use GitHub as the default service. That means some variables and methods, and mainly some assumptions regarding to the url composition.
I've introduced my "case" within the git_url method, evaluating the github_host and composing specific url when this is set to bitbucket. Of course, there are ways for doing that cleaner and easier to add more services in the future. Do you think it would be interesting a pull-request for starting that?

Using Deployinator.git_info_for_stack you can specify a host:
Deployinator.git_info_for_stack = {
        :test_stack => {
            :host => "bitbucket.org",
            :user => "cyberrodent",
            :repository => "myg",
            :checkout_path => "/tmp/deployinator_dev"
        }

I guess its not too well documented (not in the readme) It can be seen in the code.

The problem is that Bitbucket compose its URLs in a manner that current helper method git_url does not cover.
That config value is useful, and in fact I am already using it for my patched method: evaluating the host, and then composing the URL the way each host waits for.

Ok, I see what's happening. The problem is that I am deploying private respositories, and for them, with the current method, I have to pass the param read_write as true, for getting the right URL, that both hosts compose the same way.
But for non read/write repositories, Bitbucket does not allow the URL that git_url method composes this way: git://bitbucket.org/user/repo.git.

I've only used this for r/w.   Sounds like there might be a pull request in here.

@inigomedina I have a pull request here that I think will resolve your issue: https://github.com/etsy/deployinator/tree/issue-46
I added a protocol pass through to the git_freshen_or_clone method so anything you put there will be passed through (http or https for example in your case). Can you please let me know if this works for you and I'll merge it into master?

This has been merged, please let me know if this resolves your issue, the last optional parameter to git_freshen_or_clone is the protocol you would like to use.


6.1.0 + Implement registry tweak for date format

Added code to change date format via registry command, and to create a batch file to restore it.
Added function to run restoration batch and then delete it.
Added function calls at all script exit points that I could identify.
Replaced all instances of %CUR_DATE% with %DATE%.
Removed CUR_DATE definition.
Left in DTS definition for use in LOGPATH/LOGFILE, but modified it to exclude decimal seconds and the trailing info. If you'd rather keep the decimal seconds, that's easy enough, but I really recommend ditching the trailing info.
Batch file allows for restoring format manually if Tron is interrupted. To make manual restoration easier for inexperienced users, we may want to add a shortcut on the desktop, or add explanatory text in the intro screens. At a minimum, something something should be added to the README file.
A note on function layout: I like to use a slightly modified version of the function layout proposed at http://www.dostips.com/DtTutoFunctions.php#FunctionTutorial.Summary, with functions defined as beginning at the label, and ending at the 'goto :eof' command. Comments about usage, intent, etc. go immediately after the label, and is not indented. The actual code that's run gets indented one step. This includes SETLOCAL/ENDLOCAL statements, if used. If you'd like to use a different layout, I can adapt to that, but we should define some kind of standard early on.

Comments . . are indented.

Closing as we went a different route with this.


armv7 version crashes with panic: unaligned 64-bit atomic operation

When running livekit on armv7 (32bit) livekit crashes as soon as a client connects to the server
Server

Version: [1.5.2]
Environment: local dev, debian experimental armv7

Client

SDK: element call livekit branch (uses livekit js 0.8.2 as far as I know)
Version: [0.8.2]

To Reproduce
join element call call
Expected behavior
simply working
Additional context
panic: unaligned 64-bit atomic operation

goroutine 39 [running]:
runtime/internal/atomic.panicUnaligned()
        /opt/hostedtoolcache/go/1.21.5/x64/src/runtime/internal/atomic/unaligned.go:8 +0x24
runtime/internal/atomic.Xadd64(0x26f51cc, 0x1)
        /opt/hostedtoolcache/go/1.21.5/x64/src/runtime/internal/atomic/atomic_arm.s:258 +0x14
github.com/ua-parser/uap-go/uaparser.(*Parser).ParseOs(0x26f5180, {0x2442050, 0x46})
        /home/runner/go/pkg/mod/github.com/ua-parser/uap-go@v0.0.0-20230823213814-f77b3e91e9dc/uaparser/parser.go:318 +0x17c
github.com/ua-parser/uap-go/uaparser.(*Parser).Parse.func2()
        /home/runner/go/pkg/mod/github.com/ua-parser/uap-go@v0.0.0-20230823213814-f77b3e91e9dc/uaparser/parser.go:249 +0xa0
created by github.com/ua-parser/uap-go/uaparser.(*Parser).Parse in goroutine 36
        /home/runner/go/pkg/mod/github.com/ua-parser/uap-go@v0.0.0-20230823213814-f77b3e91e9dc/uaparser/parser.go:246 +0x1ac

I tried a somewhat older version. that one crashed on a different place (probably fixed already), but since I don't see many issues with arm 32 bit I mentioning straight crashes I think nobody really uses arm32? If that is the case and you don't want to waste time with that kind of issues I have no problem with that, but in that case I would suggest removing the arm32 bit build to make everyone aware that only 64bit archs are expected to work/supported

I have some news: I got it working by reordering some structs. For atomics to work they need to be aligned. Only the beginning of a struct is guaranteed to be on a 64bit boundary, so every utils.TimedVersion must be at the front.
there is also a atomic in ua-parser. I will have a look at that project and maybe notify them about that issue (ua-parser will crash on 32bit arm and probably also i386)

@akarl10 Thank you for chasing this one down! We'd love to make it work for armv7/32.
Would you be interested in opening a PR to reorganize the structs around?

just for transparency: #2382 fixed that exact stacktrace, but the pr #2386 fixes the atomic alignment in this repository.
With both applied now it seems to work oob, at least it did not crash anymore in my case.


Ci

Cosmetic changes and fix missing argument in pull job

Just FYI, last commit is not signed

https://git.dotplex.com/nitrokey/pynitrokey/-/pipelines final test all passed ready to merge after i squash

woop woop


Quick fix select input

no longer needed (will be adjusted in https://github.com/UBC-MDS/Student_Living_Guide/pull/71


[BUG] ERR_PNPM_NO_MATCHING_VERSION  No matching version found for<EMAIL_ADDRESS>

What happened?
Hi guys, I'm trying out the amazing framework you provide but encountered a dependency issue when I run pnpm create plasmo:

Could you please help me with the issue? Thanks!
While I can install the dependencies by yarn but need to choose versions:

Version
Latest
What OS are you seeing the problem on?
MacOSX
What browsers are you seeing the problem on?
No response
Relevant log output
❯ pnpm create plasmo hello-plasmo2
../Library/pnpm/store/v3/tmp/dlx-32382   |   +2 +
Packages are hard linked from the content-addressable store to the virtual store.
  Content-addressable store is at: /Users/tangyenan/Library/pnpm/store/v3
  Virtual store is at:             ../Library/pnpm/store/v3/tmp/dlx-32382/node_modules/.pnpm
../Library/pnpm/store/v3/tmp/dlx-32382   | Progress: resolved 2, reused 2, downloaded 0, added 2, done
🟣 Plasmo v0.58.0
🟠 The browser extension development framework.
🔵 INFO   | Creating new blank project
🔵 INFO   | Copying template files...
🔵 INFO   | Copying BPP workflow...
🔵 INFO   | Installing dependencies...
 ERR_PNPM_NO_MATCHING_VERSION  No matching version found for<EMAIL_ADDRESS>
The latest release of @plasmohq/parcel-resolver is "0.5.4".

If you need the full list of all 22 published versions run "$ pnpm view @plasmohq/parcel-resolver versions".
Progress: resolved 104, reused 88, downloaded 0, added 0
🟠 WARN   | pnpm exited with non-zero code: 1
🔵 INFO   | Initializing git project...
🟢 DONE   | Your extension is ready in:  /Users/tangyenan/Playgrounds/hello-plasmo2

    To start hacking, run:

       cd hello-plasmo2
       pnpm dev

    Visit https://docs.plasmo.com for documentation and more examples.

(OPTIONAL) Contact Details
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct
[X] I checked the current issues for duplicate problems.

As @louisgv pointed out in Discord, this issue is due to a custom npm registry that not fully synced with the official one.
After I swithced back to the official registry, everything works fine! Thanks!


chore: upgrade stable structures to version 0.5.2

The API for stable structures changes quite significantly between the two versions, so the change is relatively involved.
Due to a (now known) performance regression in stable structures >= 0.4.0, address_utxos have been migrated to use a Blob in the BTreeMap rather than an AddressUtxo. Once the performance regression is fixed, we'll use AddressUtxo as the type again.
There's a minor performance regression, mostly due to the conversions between Blob and Vec. These will be addressed once the regression mentioned above is addressed, but given that it's minor, it's not a blocker for the upgrade.

There's a minor performance regression, mostly due to the conversions between Blob and Vec

Something that I thought about while reviewing and related to the small performance regression: it would be nice to see the diff in benchmarks in percentages to more easily confirm the regression is negligible (instruction numbers are quite large in nature which makes it harder to eyeball things).

Agreed. The stable-structures crate is now using criterion to display the output, and that's much easier to eyeball. I think we can do the same here.


arabic encoding support and image in header

Hello
I'm using this package to convert .docx file to .pdf file
my file contain variables filled by Django view.
all of this working correctly,
but the .docx file have Arabic statement, so the program show error during conversion, so I remove this statement and then the code works like a charm.
so how to support convert doc with Arabic words?
also the .docx file has header ( with logo ), the outputted pdf has no header at all !!
@author_required
def download_approval(request, project_id):
    project = get_object_or_404(Project, pk=project_id)
    doc = DocxTemplate('letter.docx')
    context = {
        'ref_num': project.ref_num,
        'author_name': project.author.get_full_name,
        'approval_date': project.approved_date.date(),
        'project_title': project.title_en
    }
    doc.render(context)
    doc.save('approval_letter.docx')
    pypandoc.convert_file('approval_letter.docx', 'latex', outputfile="research_permission_request.pdf",
                          extra_args=['--pdf-engine=C:\\Users\\HP\\Desktop\\mktex\\miktex\\bin\\x64\\pdflatex.exe'])
    pdf = open('research_permission_request.pdf', 'rb')
    response = FileResponse(pdf)
    return response

Hi @JessicaTegner
pypandoc    v 1.11
pypandoc-binary        v 1.11
python v 3.9.2
django v 4.2

also I have attach the docx that I wanna covert to pdf :
letter.docx
also this is the pdf that resulted from the conversion process:
research_permission_request.pdf
note1: I don't wanna support Arabic anymore, I have translate it to English.
the only thing I wanna solve is the the image..
note2: I'm running this code on windows machine.
also, does the previous code works on Nginx, Linux (ubuntu) server ?

So correct me if I'm wrong, but it seems like the images in the word documents are in the page header, which from my understanding of pandoc do not get converted as well.
I did some testing with pure pandoc and the files you provided, and they indeed od not get converted over, which is really strange.

so ? how to convert the docx with the images :/  is there any workaround ?

f


Kubernetes Resource Model - docker-compose extension

I'm looking to add support in the sandbox for running Tinkerbell components using the Kubernetes Resource Model (KRM) as outlined in tinkerbell/proposals#46. I was thinking of adding docker-compose support, but wanted to get some input on approach.
Currently there is a single docker-compose.yml in deploy/compose, however with KRM, there are some services/containers that won't be needed (database, tink-cli, etc). Given the overlap, could we have 3 compose files since compose supports combining multiple files? The result would be something like a docker-compose.base.yml, docker-compose.k8s.yml and docker-compose.db.yaml. In order to run the existing stack, you'd have to run:
docker-compose -f docker-compose.base.yml -f docker-compose.db.yml up -d

and for the KRM stack:
docker-compose -f docker-compose.base.yml -f docker-compose.k8s.yml up -d

This gets away from the simplicity of docker-compose up -d, but enables reuse of services like registry, and common config on services like boots, hegel, tink-server, and nginx.
The only alternative I see is to have a separate, largely duplicated compose file.
Any opinions here?

I think the user experience for the docker-compose quickstart would change most with this. I do like the very simple experience of docker-compose up -d.
What about doing something to define either the "existing stack" or the "KRM stack" as the default. So we still get the nice docker-compose up -d for the default and then with an additional -f we could get the non-default version? Would this add too much complexity to the non-default compose file?


Crash with FTB Chunks / FTB Teams

Describe the Bug
I counter a crash when deployer put blocks on claimed chunks with ftbchunks of fabric.
Playing on All of Fabric 5, but with the addition of the create
This bug happed when I make a simple gyratory tree farm, and when put seed of oak in the dirt my server dies XD
crash-reports_crash-2022-05-21_17.25.23-server.txt
Reproduction Steps

Claim the chunk
Gyratory mechanical belt with deployers with oak sapling
Use deployer to put oak sapling on dirt
Crash
...

Expected Result
Not crash when deployer put blocks on claimed chunks
Screenshots and Videos
No response
Crash Report or Log
---- Minecraft Crash Report ---- // This doesn't make any sense!  Time: 5/21/22, 5:25 PM Description: Ticking entity  java.lang.NullPointerException: Cannot invoke "dev.ftb.mods.ftbteams.data.PlayerTeam.getExtraData()" because the return value of "dev.ftb.mods.ftbteams.data.TeamManager.getInternalPlayerTeam(java.util.UUID)" is null 	at dev.ftb.mods.ftbchunks.data.ClaimedChunkManager.getBypassProtection(ClaimedChunkManager.java:141) 	at dev.ftb.mods.ftbchunks.data.ClaimedChunkManager.protect(ClaimedChunkManager.java:170) 	at dev.ftb.mods.ftbchunks.FTBChunks.blockPlace(FTBChunks.java:280) 	at java.base/java.lang.invoke.MethodHandle.invokeWithArguments(MethodHandle.java:732) 	at dev.architectury.event.EventFactory.invokeMethod(EventFactory.java:53) 	at dev.architectury.event.EventFactory$2.handleInvocation(EventFactory.java:81) 	at com.google.common.reflect.AbstractInvocationHandler.invoke(AbstractInvocationHandler.java:89) 	at jdk.proxy2/jdk.proxy2.$Proxy115.placeBlock(Unknown Source) 	at net.minecraft.class_1747.handler$zjj000$place(class_1747.java:1538) 	at net.minecraft.class_1747.method_7712(class_1747.java:80) 	at net.minecraft.class_1747.method_7884(class_1747.java:50) 	at net.minecraft.class_1799.method_7981(class_1799.java:250) 	at com.simibubi.create.content.contraptions.components.deployer.DeployerHandler.activateInner(DeployerHandler.java:310) 	at com.simibubi.create.content.contraptions.components.deployer.DeployerHandler.activate(DeployerHandler.java:131) 	at com.simibubi.create.content.contraptions.components.deployer.DeployerMovementBehaviour.activate(DeployerMovementBehaviour.java:90) 	at com.simibubi.create.content.contraptions.components.deployer.DeployerMovementBehaviour.visitNewPosition(DeployerMovementBehaviour.java:71) 	at com.simibubi.create.content.contraptions.components.structureMovement.AbstractContraptionEntity.tickActors(AbstractContraptionEntity.java:276) 	at com.simibubi.create.content.contraptions.components.structureMovement.ControlledContraptionEntity.tickContraption(ControlledContraptionEntity.java:136) 	at com.simibubi.create.content.contraptions.components.structureMovement.AbstractContraptionEntity.method_5773(AbstractContraptionEntity.java:242) 	at net.minecraft.class_3218.method_18762(class_3218.java:702) 	at net.minecraft.class_1937.method_18472(class_1937.java:485) 	at net.minecraft.class_3218.method_31420(class_3218.java:363) 	at net.minecraft.class_5574.method_31791(class_5574.java:54) 	at net.minecraft.class_3218.method_18765(class_3218.java:331) 	at net.minecraft.server.MinecraftServer.method_3813(MinecraftServer.java:907) 	at net.minecraft.class_3176.method_3813(class_3176.java:322) 	at net.minecraft.server.MinecraftServer.method_3748(MinecraftServer.java:851) 	at net.minecraft.server.MinecraftServer.method_29741(MinecraftServer.java:697) 	at net.minecraft.server.MinecraftServer.method_29739(MinecraftServer.java:273) 	at java.base/java.lang.Thread.run(Thread.java:833)   A detailed walkthrough of the error, its code path and all known details is as follows: ---------------------------------------------------------------------------------------  -- Head -- Thread: Server thread Stacktrace: 	at dev.ftb.mods.ftbchunks.data.ClaimedChunkManager.getBypassProtection(ClaimedChunkManager.java:141) 	at dev.ftb.mods.ftbchunks.data.ClaimedChunkManager.protect(ClaimedChunkManager.java:170) 	at dev.ftb.mods.ftbchunks.FTBChunks.blockPlace(FTBChunks.java:280) 	at java.base/java.lang.invoke.MethodHandle.invokeWithArguments(MethodHandle.java:732) 	at dev.architectury.event.EventFactory.invokeMethod(EventFactory.java:53) 	at dev.architectury.event.EventFactory$2.handleInvocation(EventFactory.java:81) 	at com.google.common.reflect.AbstractInvocationHandler.invoke(AbstractInvocationHandler.java:89) 	at jdk.proxy2/jdk.proxy2.$Proxy115.placeBlock(Unknown Source) 	at net.minecraft.class_1747.handler$zjj000$place(class_1747.java:1538) 	at net.minecraft.class_1747.method_7712(class_1747.java:80) 	at net.minecraft.class_1747.method_7884(class_1747.java:50) 	at net.minecraft.class_1799.method_7981(class_1799.java:250) 	at com.simibubi.create.content.contraptions.components.deployer.DeployerHandler.activateInner(DeployerHandler.java:310) 	at com.simibubi.create.content.contraptions.components.deployer.DeployerHandler.activate(DeployerHandler.java:131) 	at com.simibubi.create.content.contraptions.components.deployer.DeployerMovementBehaviour.activate(DeployerMovementBehaviour.java:90) 	at com.simibubi.create.content.contraptions.components.deployer.DeployerMovementBehaviour.visitNewPosition(DeployerMovementBehaviour.java:71) 	at com.simibubi.create.content.contraptions.components.structureMovement.AbstractContraptionEntity.tickActors(AbstractContraptionEntity.java:276) 	at com.simibubi.create.content.contraptions.components.structureMovement.ControlledContraptionEntity.tickContraption(ControlledContraptionEntity.java:136) 	at com.simibubi.create.content.contraptions.components.structureMovement.AbstractContraptionEntity.method_5773(AbstractContraptionEntity.java:242) 	at net.minecraft.class_3218.method_18762(class_3218.java:702) 	at net.minecraft.class_1937.method_18472(class_1937.java:485) 	at net.minecraft.class_3218.method_31420(class_3218.java:363) 	at net.minecraft.class_5574.method_31791(class_5574.java:54) 	at net.minecraft.class_3218.method_18765(class_3218.java:331)  -- Entity being ticked -- Details: 	Entity Type: create:stationary_contraption (com.simibubi.create.content.contraptions.components.structureMovement.ControlledContraptionEntity) 	Entity ID: 161766 	Entity Name: Stationary Contraption 	Entity's Exact location: -3108.00, 145.00, -431.00 	Entity's Block location: World: (-3108,145,-431), Section: (at 12,1,1 in -195,9,-27; chunk contains blocks -3120,-64,-432 to -3105,319,-417), Region: (-7,-1; contains chunks -224,-32 to -193,-1, blocks -3584,-64,-512 to -3073,319,-1) 	Entity's Momentum: 0.00, 0.00, 0.00 	Entity's Passengers: [] 	Entity's Vehicle: null Stacktrace: 	at net.minecraft.class_1937.method_18472(class_1937.java:485) 	at net.minecraft.class_3218.method_31420(class_3218.java:363) 	at net.minecraft.class_5574.method_31791(class_5574.java:54) 	at net.minecraft.class_3218.method_18765(class_3218.java:331) 	at net.minecraft.server.MinecraftServer.method_3813(MinecraftServer.java:907) 	at net.minecraft.class_3176.method_3813(class_3176.java:322) 	at net.minecraft.server.MinecraftServer.method_3748(MinecraftServer.java:851) 	at net.minecraft.server.MinecraftServer.method_29741(MinecraftServer.java:697) 	at net.minecraft.server.MinecraftServer.method_29739(MinecraftServer.java:273) 	at java.base/java.lang.Thread.run(Thread.java:833)  -- Affected level -- Details: 	All players: 3 total; [class_3222['Nakuras'/88531, l='ServerLevel[our_bonifacio_aof]', x=-3080.30, y=140.00, z=-403.19], class_3222['papudeiror'/157262, l='ServerLevel[our_bonifacio_aof]', x=-3080.80, y=140.00, z=-402.30], class_3222['Tlum__'/162314, l='ServerLevel[our_bonifacio_aof]', x=-3105.18, y=145.00, z=-442.14]] 	Chunk stats: 4226 	Level dimension: minecraft:overworld 	Level spawn location: World: (16,68,-48), Section: (at 0,4,0 in 1,4,-3; chunk contains blocks 16,-64,-48 to 31,319,-33), Region: (0,-1; contains chunks 0,-32 to 31,-1, blocks 0,-64,-512 to 511,319,-1) 	Level time: 2802188 game time, 3112275 day time 	Level name: our_bonifacio_aof 	Level game mode: Game mode: survival (ID 0). Hardcore: false.


New unit conversions

http://www.w3.org/TR/2012/WD-css3-values-20120308/#other-units
CSS now supports s and ms as time values. Sass should be able to convert between them at 1s == 1000m.
CSS also support Hz an kHz and Sass should convert between them at 1000Hz == 1kHz
CSS also now supports angle measurements deg, rad, grad and turn which can be converted using good old π and whatnot.
Lastly it seems like dpi and dpcm can be converted but it seems like any conversion would not be 100% accurate due to rounding issues implied by quantization.

Getting there...
https://gist.github.com/2004835

@scottkellum Sorry I really should have put some more details about this into the issue. We have conversions built into sass as it is a core syntax concern and part of our arithmetic system: https://github.com/nex3/sass/blob/master/lib/sass/script/number.rb#L424-430


Dummy throwing Unknown prop errors

Dummy is throwing a lot of Unknown props errors, as can be seen here
as well in my test app using redux auth.
They appear to be same problem of #43 that was closed before but the problem persists.
Any ideas of how it can be solved? I'd be glad to send a PR if I can.
Thanks

It's a problem with the react-bootstrap library. It may be fixed now,
here's the issue:
https://github.com/react-bootstrap/react-bootstrap/issues/1994
If it's fixed I'll push an update ASAP.
On Mon, Aug 22, 2016, 11:44 AM Alberto Harres Rocha <
<EMAIL_ADDRESS>wrote:

Dummy is throwing a lot of Unknown props errors, as can be seen here
http://pastebin.com/Y8Huwp1E
as well in my test app https://github.com/betoharres/redux-auth-test
using redux auth.
They appear to be same problem of #43
https://github.com/lynndylanhurley/redux-auth/issues/43 that was closed
before but the problem persists.
Any ideas of how it can be solved? I'd be glad to send a PR if I can.
Thanks
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/lynndylanhurley/redux-auth/issues/76, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AAckRT3MdMmvXrICCxs18qV5SsrHUmnFks5qidHlgaJpZM4JqEad
.

Nice, thanks!! 😃

@lynndylanhurley the RB issue has been closed and there's a PR related to it merged as well. Would love to see this fixed and updated if possible :) thanks for your great work


docs, rkt: Add missing --signature parameter

It is rather expected to be able to pass a custom signature to prepare
command too, as it may fetch the image from the network.

LGTM after one comment.

@krnowak ping

Updated.

Should tests be added here, or in a following PR from #2271?
Except this question, LGTM

I think we can do tests in a following PR.


Unused private constructor doesn't trigger unused_*

Minimal reproduction:
void example(PublicClassPrivateConstructor? e) {
  print(e == null);
}

class PublicClassPrivateConstructor {
  PublicClassPrivateConstructor._();
}

I'm guessing there is an argument this is often a "hint" to prevent extends or with, which leads me to believe we need an @interface annotation - but it's also probably reasonable to use // ignore: unused_constructor for this purpose as well.

Interestingly enough, adding a factory constructor starts triggering:
void example(PublicClassPrivateConstructor? e) {
  print(e == null);
}

class PublicClassPrivateConstructor {
  factory PublicClassPrivateConstructor() {
    throw UnimplementedError();
  }

  PublicClassPrivateConstructor._();
}

Yeah it's intentional for the reasons you give.

Want to change this to a request for @interface?

@srawlins I'm OK with you closing this as a dupe of https://github.com/dart-lang/sdk/issues/46331 in that case. Up to you.

I'll keep this open as we should change the behavior soon for Dart 3.0 interfaces.

It's not just a hint, it prevents the class from having an implicit public default constructor.
That means that having a single private constructor is a reasonable thing to do, even if nobody uses it.
(Today abstract interface can do the same thing - then it doesn't matter that the class has a default constructor, if you can't call it and can't use it as a super constructor. Used to be that the private constructor was the best way to achieve that.)

Alright let's call it meaningful then.

Alright let's call it meaningful then.

I have an inverse read of the situation, now that we have abstract interface, an unused private constructor is dead code :P.

While I see your perspective, the code serves a purpose. There's another (and maybe better) way to accomplish the same purpose, but that doesn't negate the fact that the code can't just be removed without making other changes.
The two styles are both valid Dart, and if would be irritating to a user that preferred the constructor approach to have diagnostics that they never intended to resolve.
I could argue that the lint ought to flag such a constructor if the class is already marked as abstract interface. I don't know whether we do.
In addition, we could have a lint to flag cases where a class could be marked with abstract interface. If we did add such a lint it should have a fix.
Or we could skip the lint and implement an assist that would convert the class and remove the constructor. It isn't clear to me that anyone would discover such an assist, however.

If you can guarantee that the default constructor of an abstract interface class is not shown in DartDoc, then I'll agree that there is a better way. (But also don't want to migrate too much code for that, and we still do allow pre-3.0 code to exist, so unless the lint can behave differently for pre-3.0 code than for post-3.0 code, it's a problem.)


Allow for destination address on report_stellar_transaction and redeem

Incentive
We support bridging assets from Stellar (which uses ed25119), and for ease of use, we want to also allow users to be able to connect an external wallet to the chain using spacewalk.
Most wallets (polkadot.js and Talisman), unfortunately don't let you chose the ed25519 scheme, and only import/create accounts using the sr25519 scheme.
Problem
So far, we derive the address from the Stellar public key from the incoming transaction's envelope XDR. We take this from the sender of the payment, and this public key will always be ed25519, and we cannot derive the sr25519 version from it.
What is required
Changing the signature of methods report_stellar_transaction and redeem to be able to specify a destination address in both ways.
So the funds can go from the ed25519 version of the address in Stellar, to the sr25519 version in Pendulum.

Closing this because it does not apply with the latest version of spacewalk anymore. (We don't derive the destination address).


Token signatures and issuers

based on and blocked by https://github.com/nspcc-dev/neofs-api-go/pull/446

integration test failures aint related to these changes. #572


BUG: save(query("test.jld"),a) is an endless loop

function save{F}(q::Formatted{F}, data...; options...)
    unknown(q) && throw(UnknownFormat(q))
    libraries   = applicable_savers(q)
    failures    = Any[]
    for library in libraries
        try
            Library = checked_import(library)
            return Library.save(q, data...; options...)
        catch e
            handle_current_error(e, library, library == last(libraries))
            push!(failures, (e, q))
        end
    end
    handle_error(failures)
end

When you execute the below code, an endless loop comes because Library is JLD.
using JLD
using FileIO
a = 1
save(query("test.jld"), a)

julia> versioninfo()
Julia Version 0.5.0-dev+1516
Commit a7b41f5 (2015-11-30 00:08 UTC)
Platform Info:
  System: Linux (x86_64-linux-gnu)
  CPU: Intel(R) Xeon(R) CPU E5-2699 v3 @ 2.30GHz
  WORD_SIZE: 64
  BLAS: libopenblas (USE64BITINT DYNAMIC_ARCH NO_AFFINITY Haswell)
  LAPACK: libopenblas64_
  LIBM: libopenlibm
  LLVM: libLLVM-3.3

Is there anyone else who encountered the same problem?

You need to provide an HDF5-name for a, for example
save(query("test.jld"), "a", a)

@timholy Thank you, very much.

No problem.
Out of curiosity, is there any reason you're calling query directly? You should just be able to say save("test.jld", "a", a).

I just remembered the wrong style  save("test.jld",a) .  When I try to debug it, called query.

Thanks for explaining.
It would be good to have this give an error rather than segfault/hang. I opened https://github.com/JuliaLang/julia/issues/14472, which appears necessary to fix the problem in a backward-compatible way.

That's great! Looking forward to.

Can this be reopened until it is fixed (ie. turned into an error)? I think this is a common typo and I was just about to open a duplicate issue.

Closed by #50


[BEAM-10897] Update the fastavro lower bound due to an issue on Windows

Please add a meaningful description for your change here

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

R: @AnandInguva

What was the issue on Windows?

See: https://issues.apache.org/jira/browse/BEAM-10897

LGTM!


[nrf fromtree] [app] Allow to define external clusters in chip_data_model.*

Original contribution: https://github.com/project-chip/connectedhomeip/pull/27886
zap_cluster_list.py raises an error when the ZAP file enables a cluster that is not implemented in the SDK. This forces users to patch the SDK in order to use custom, vendor-specific clusters.
Add EXTERNAL_CLUSTERS argument to chip_configure_data_model() CMake function that takes a list of external cluster names for which the default implementation should not included nor required.
Usage example:
chip_configure_data_model(app
    INCLUDE_SERVER
    ZAP_FILE lock-app.zap
    EXTERNAL_CLUSTERS BASIC_INFORMATION_CLUSTER MY_CUSTOM_CLUSTER
)

Replace the existing zap_clusters_with_custom_implementation argument of chip_data_model() GN template with new one: external_clusters. The previous argument took a list of cluster implementation directories to be skipped, so it was only useful for bypassing the default implementations, but could not be used for custom, vendor-specific clusters. The new argument addresses both scenarios.
Usage example:
chip_data_model("lock-common") {
  zap_file = "lock-app.zap"
  ...
  external_clusters = [
    "BASIC_INFORMATION_CLUSTER",
    "MY_CUSTOM_CLUSTER",
  ]
}

Please fix the commit message: [nrf fromtree] [app]

I approve, but please take a look at the CI. I think it was working fine on the last PR we pushed, so it's possible that this change broke something.

Yes, but they were pointing a master branch instead of v2.4. Here, the CI error logs indicate mainly that "Getting requirements to build wheel did not run successfully."

I approve, but please take a look at the CI. I think it was working fine on the last PR we pushed, so it's possible that this change broke something.

Yes, but they were pointing a master branch instead of v2.4. Here, the CI error logs indicate mainly that "Getting requirements to build wheel did not run successfully."

Ah ok, I didn't notice it's 2.4 branch.

In fact, the same wheel problem with mbedtls occurred on the last 2.4 PR:
https://github.com/nrfconnect/sdk-connectedhomeip/actions/runs/5608952159/job/15195590442

I approve, but please take a look at the CI. I think it was working fine on the last PR we pushed, so it's possible that this change broke something.

Yes, but they were pointing a master branch instead of v2.4. Here, the CI error logs indicate mainly that "Getting requirements to build wheel did not run successfully."

Ah ok, I didn't notice it's 2.4 branch.

In fact, the same wheel problem with mbedtls occurred on the last 2.4 PR: https://github.com/nrfconnect/sdk-connectedhomeip/actions/runs/5608952159/job/15195590442

This seems to be caused by:
https://github.com/markaj-nordic/sdk-connectedhomeip/blob/pull_custom_cluster_script_support/scripts/setup/requirements.esp32.txt#L12C1-L12C77
Which is already removed in the upstream.


DateTime.MinValue doesn't get parsed for dateRangeQuery

NEST/Elasticsearch.Net version:
7.4.0
Elasticsearch version:
7.3.0
Description of the problem including expected versus actual behavior:
In the search descriptor DateRange.GreaterThanOrEquals does not parse DateTime.MinValue, instead it sends a request with an empty string which ES 7.4 throws an error cannot parse empty date
Steps to reproduce:

Set DateTime.MinValue
Create any search query

isnt that because ES runs internally on EPOCH dates? offsets from January 1, 1970 so 01/01/0001 is bs
add 1950 years to datetime.mindate and its fine 🗡

Thanks for reporting @Anasbzr. It looks like the JSON serialization library  does not handle DateTime.MinValue
var response = client.Search<Question>(s => s
	.Query(q => q
		.DateRange(d => d
			.GreaterThanOrEquals(DateTime.MinValue)
			.Field(f => f.CreationDate)
		)
	)
);

emits
POST http://localhost:9200/posts/_search
{}

but
var response = client.Search<Question>(s => s
	.Query(q => q
		.DateRange(d => d
			.GreaterThanOrEquals(DateTime.MinValue.AddTicks(1))
			.Field(f => f.CreationDate)
		)
	)
);

emits
POST http://localhost:9200/posts/_search
{"query":{"range":{"creationDate":{"gte":"0001-01-01T00:00:00.0000001"}}}}

we'll look to address this in the next release.

It looks like the JSON serialization library does not handle DateTime.MinValue

JSON serialization is fine, it's the implementation for IsValid on DateMath:
https://github.com/elastic/elasticsearch-net/blob/e514e5b7f4a7fb555357d3e71bd455237ec8b542/src/Nest/CommonOptions/DateMath/DateMath.cs#L86
I think for a DateTime, any DateTime should be valid

opened #4278 to address


Secret rotation

Hi... I have a question. In an earlier commet you mentioned that you'd add a way to rotate the secret by introducing old_secret:
https://github.com/patrickpissurno/fastify-esso/issues/2#issuecomment-620825433
I was unable to find this in the code. Has the idea of rotating secrets been implemented. If not, how would you approach this?
Thanks!

I added a pull request as a suggestion on how to tackle this:
https://github.com/patrickpissurno/fastify-esso/pull/23


I keep seeing ImageView no longer exists. You should not use this PhotoViewAttacher any more. in the logs. Am I doing something wrong?

I have a view pager where I use PhotoView. I keep seeing this log: ImageView no longer exists. You should not use this PhotoViewAttacher any more. Do I need to be destroying the PhotoViewAttacher somehow when the view is destroyed?

Have you try  it and create a new object with new ImageView ?

What do you mean?

Sounds like the PhotoViewAttacher is staying around longer than the actual ImageView is. Are you holding onto references to the PhotoViewAttacher elsewhere? Also, are you changing the contents of your ViewPager by changing the items within the adapter dynamically?

@Jawnnypoo I am using PhotoView directly rather than using an attacher. And I am using it with Glide. I am setting the image in instantiateItem() so this would be the only place where the photoview is changed

There was a memory leak in PhotoView where local PhotoViewAttacher was not freed. this should be fixed with #373

Hi there!
I am still getting this issue and I am using version 1.3.1.
I am only using the PhotoViewAttacher in my pagerAdapter and yes I am changing the items (loading the image when the page turns) dynamically.  It seems to be working fine but I am getting a ton of these errors and cant figure out what the problem is or how to make them stop!
Thanks in advance!
Sounds like the PhotoViewAttacher is staying around longer than the actual ImageView is. Are you holding onto references to the PhotoViewAttacher elsewhere? Also, are you changing the contents of your ViewPager by changing the items within the adapter dynamically?


Refactor

This PR refactors code to:

Remove redundant tests from project. This stops build fails from tests for code that didn't exist anymore
Refactor project structure to follow original guidlines
Added doc comments to many methods for easier generation of documentation

I have had a look at the documentation, not at the code.

We should give a 'how to use' heading. Inside it will show an example of importing the library. We can give a list of the various color palettes as well.


Srcset problems

Hi. I'm adding a loading image, to index.html, in an Ember.JS project.
The problem is that, after fingerprinting this:
        <img
             src="images/myimage.png"
<EMAIL_ADDRESS>2x, images/myimage.png 1x"
             sizes="200px"
             alt="Loading" />

        <img
             src="//mycdn.cloudfront.net/images/myimage-515d3d97735aae0a950e74c8ad32ddbf.png"
<EMAIL_ADDRESS>             sizes="200px"
             alt="Loading" />

The problem is that, it removes the 1x and 2x directives, and this makes the desktop browsers load the 2x image by default. I don't really know why, but I guess is something to do with srcset compatibility.
Any way we can preserve both the 1x and 2x directives?

Thanks for reporting this. Just to be upfront, this might be a difficult problem to solve. I don't really have an ETA at this time of when I will be able to tackle this.

I'm hitting this too. Is there any solution in mind? The current approach of one-regex-to-rule-them-all doesn't seem like it can last much longer.
Out of curiosity, why does the regex explicitly try to match on string delimiters (e.g. " or ')? Matching on the bare filename would solve this problem, although I have to assume that breaks everything. :stuck_out_tongue_closed_eyes:
Maybe it would be best to switch to an Esprima-based parser for JS files and only pattern match inside string literals?

I'm not really sure what the best strategy to solve this use case is. It matches one of the delimiters so that it knows where to stick the prepend in (i.e. cloudfront domain).
I'm all ears on other solutions, but implementing an esprima-based parser is probably over my head.

Same problem here. srcset it's not handled properly.
Using ember-cli:
// No fingerprint

var el6 = dom.createElement("img");
dom.setAttribute(el6,"src","/images/product/calender_framed_@2x-60a51c9551fd5a60265de52e7212a271.jpg");
dom.setAttribute(el6,"srcset","/images/product/calender_framed-be441c32691eb35d8a0e0542462b81d1.jpg 1x<EMAIL_ADDRESS>2x");
dom.setAttribute(el6,"class","wow fadeIn img-responsive");

// Fingerprint
// fingerprint: {
//   prepend: '//s3-eu-west-1.amazonaws.com/smhw-frontend-prod/';
// },
var el6 = dom.createElement("img");
dom.setAttribute(el6,"src","//s3-eu-west-1.amazonaws.com/smhw-frontend-prod/images/product/calender_framed_@2x-60a51c9551fd5a60265de52e7212a271.jpg");
dom.setAttribute(el6,"srcset","//s3-eu-west-1.amazonaws.com/smhw-frontend-prod/images/product/calender_framed-be441c32691eb35d8a0e0542462b81d1.jpg");
dom.setAttribute(el6,"class","wow fadeIn img-responsive");

has there been any movement / thoughts on this?

I may take a stab at this. srcset is something we're making more and more use of as of recently. @rickharrison if you're OK wit it, I think there would need to be a special case for detecting and parsing srcset values. We can probably use: https://github.com/albell/parse-srcset as it claims to be spec compliant. Then just run the urls through the fingerprinter and spit them back out.

Yea the only thing I'm concerned with is breaking existing functionality. A lot of the PRs thus far have tended to break behavior to fix edge cases.

I myself conceded defeat a couple times. The regexp approach is unbeliable hard to modify without breaking some edge case. I tried for a while to have different parsers for different files (hbs, js...) but that is not a simple task to do.

I cannot reproduce this issue (here is my try https://serabe.github.io/check-asset-rev/ ). Can anyone send me one?
Thanks!

Ok, the problem was with the prepend option. rickharrison/broccoli-asset-rewrite#61 fixes it.


Create concept for mesh that fields can share

There should be a concept of a mesh that gets passed into the fields. This will enable sharing of data like the search structures. Constructing a search structure per field is a bit insane and creates a significant startup overhead.
Also, the initial communication round to determine partitioning can happen a single time per mesh (rather than per field).

This should actually not be a "mesh" as the fundamental unit, but a field layout. Since two fields with the same mesh could have different orders (i.e., different layouts of the local global DOF ids)


Pass thru WaitFor to parent resourcs.

Description
This PR removes health checks from child resources (e.g. SQL Server Database). Instead what we do here is that when WaitFor is invoked we determine whether we are adding a reference to a resource that has a parent, and then we add a WaitAnnotation for both the child resource and the parent resource. This combined with the check removal in the child resources has a few effects.
First, dependents will not block waiting for a database to be created, they'll only wait for the server to be created. After some experimentation with the way people use Aspire this seemed like a more natural experience.
Leaving the WaitAnnotation in place of the child resources is important though for future scenarios where we want to enable something like this:
var builder = DistributedApplication.CreateBuilder(args);
var db = builder.AddSqlServer("sql")
                .AddDatabase("db")
                .WithCreation(); // This extension does not exist yet.

What the WithCreation(...) extension would do in this case is add a health check to the child resource and register a event subscription for the resource healthy event for the parent.
This is just an explanantion as to why we are keeping the WaitAnnotation on the child resource ;)
Fixes #5868
Checklist

Is this feature complete?

[x] Yes. Ready to ship.
[ ] No. Follow-up changes expected.

Are you including unit tests for the changes and scenario tests if relevant?

[x] Yes
[ ] No

Did you add public API?

[ ] Yes

If yes, did you have an API Review for it?

[ ] Yes
[ ] No

Did you add <remarks /> and <code /> elements on your triple slash comments?

[ ] Yes
[ ] No

[x] No

Does the change make any security assumptions or guarantees?

[ ] Yes

If yes, have you done a threat model and had a security review?

[ ] Yes
[ ] No

[x] No

Does the change require an update in our Aspire docs?

[ ] Yes

Link to aspire-docs issue:

[x] No

Microsoft Reviewers: Open in CodeFlow

fyi @Alirexaa