Slow conversion of book-size documents

git clone git@github.com:progit/progit2.git
cd progit2
touch book/contributors.txt
bundle install
time bundle exec asciidoctor-pdf progit.asc

Resulting PDF has 527 pages as of today.
On my Ryzen 1700, this book converts to HTML in 0.7s, to EPUB3 in 2.8s and to PDF in 1m46s.
$ bundle exec asciidoctor-pdf --version
Asciidoctor PDF 1.5.3 using Asciidoctor <IP_ADDRESS> [http://asciidoctor.org]
Runtime Environment (ruby 2.4.9p362 (2019-10-02 revision 67824) [x86_64-linux]) (lc:UTF-8 fs:UTF-8 in:- ex:UTF-8)

This is largely going to be addressed by the changes coming in Asciidoctor PDF 2, which will do away with the dry run in most cases and only use a single pass. Conversion is also slow because Ruby is slow at reading and processing PNG images. It's decidedly faster to use prawn-gmagick.
Beyond that, there isn't much we can do. That's just the limitation of this stack (and the PDF generator in particular). We'll focus on achieving better speed with the web-based PDF converter.

I believe it would not hurt to perform some profiling here to confirm that PNGs are the main issue.

I've done that and I'm quite confident of where the problems are. The first and main problem is the double rendering of almost every node, esp large blocks (which is being addressed). The second are images, particularly PNGs. But beyond that, Prawn is just not efficient and is not being improved, so there isn't much we're going to be able to do. If someone can identify hotspots that we can fix, I'm glad make what changes are necessary to improve them.
In my experience, PDF generation is slow no matter what tool is used. I've never seen a fast PDF converter. So there are inherent limitations in the format itself. We're never going to come anywhere close to the speed of converting to HTML (unless the toolchain itself uses HTML).

IME going through the asciidoctor -> latex -> pdflatex path was about twice as fast as asciidoctor-pdf on ginormous documents like the Vulkan specification, though I haven't done that for a couple of years. Maybe that's just because asciidoctor-pdf is doing twice as much work, but I will be grateful for any improvements as well. It takes 5.5 minutes on a Ryzen 2600 to generate our full specification document (ca. 2331 PDF pages), and there  are about 16 MB of SVG source incorporated in the output.

I believe eliminating the double rendering will make a big difference. But there are still inherent limits in the PDF generation library that we won't be able to do anything about (short of writing a new PDF generator library, which I'm definitely not going to do). The future will be the browser-based stack, which I'll talk more about once Asciidoctor PDF 2 is out.

Chromium takes about 1:45 to 'Print to PDF' of the same document from asciidoctor-generated HTML, so if you're planning to leverage a browser engine in some fashion as that name suggests, that could help. Not expecting miracles of course.

As I suspected, the lag largely has to do with decoding PNG images. Here are the results on my machine converting Pro Git 2 using Ruby 3.1 and Asciidoctor PDF 2. (11th Gen Intel® Core™ i7-1065G7 CPU @ 2.80GHz (4 cores / 8 threads), 15.4 GiB RAM)
real    0m53.429s (without prawn-gmagick)
real    0m33.780s (with prawn-gmagick)

Decoding PNG images requires a lot of mathematical computation, and that's not a task Ruby is particularly efficient at performing. That's why prawn-gmagick makes such a substantial difference.
Contrary to what I anticipated, Asciidoctor PDF 2 isn't any faster than Asciidoctor PDF < 2. But it's not slower either. I get almost identical results.
There's nothing fundamentally I know that we can change to make Asciidoctor PDF any faster. We're limited by the capabilities of the PDF library (Prawn) and Ruby itself. And PDFs are extremely complicated to create for a variety of reasons, so it's going to be slower than producing other formats. That's why we'll be looking to using web-based technologies to generate PDFs in the future. But that's still very much on the horizon.

Will the prawn-gmagick path be something supported OOTB with version 2?

It's supported out of the box with Asciidoctor PDF 1.5.x, 1.6.x, 2.0.x, and all future versions. This is explained in the README (and will be explained in the docs once they are available). See https://github.com/asciidoctor/asciidoctor-pdf#supporting-additional-image-file-formats


[Social Media] More, but shorter, videos about Ladybird (I'll close this issue request, to avoid having too many open issues here)

Hey there folks,
This is just a small comment in regards to videos about ladybird.
I, and probably others, are interested in the news about ladybird, e. g. "is it ready yet?".
One way to keep up to date is via videos, such as:
https://www.youtube.com/watch?v=0aXSBCtELZ4
The video is fine, but as I have the attention span of a squirrel on a sugar nut rush, a length of 17 to 19 minutes is too long, IMO. I'd like to see more a range of, say, 4-8 minutes or so - just the primary gist, and showcasing a little bit of what is new. Ideally without too much audio, e. g. more eye candy than spoken text, though spoken text is also fine. For me I just want slightly shorter videos.
Anyway, that is the gist of it. Kind of a request to have more smaller videos about ladybird. This can be about singular features too, by the way, e. g. "this website now works, let me show you its features".
Thanks for reading!

You've opened and immediately closed several issues like this. The issue tracker is for technical issues, please refrain from opening these


chore: add linter analysis options

An initial proposal for some linter rules. These are the ones I use personally, they are not very strict but might be a good start.
Check out https://dart.dev/tools/linter-rules

@BertVanMieghem Is this still being worked on ?

@BertVanMieghem Hi, what is the status of this PR, maybe some help needed?

@BertVanMieghem Hi, what is the status of this PR, maybe some help needed?

@mfederowicz sure you can help here

Ok @tenshiAMD but what is the plan, I have https://github.com/mfederowicz/exer_log/tree/lintr - cleanedup, 1 ahead to alpha, so should I create another PR?

Ok @tenshiAMD but what is the plan, I have https://github.com/mfederowicz/exer_log/tree/lintr - cleanedup, 1 ahead to alpha, so should I create another PR?

@mfederowicz yes then we close and reference this PR to the new PR. Thanks.

@tenshiAMD ok but can you lookup on content of pubspec.lock, because iam affraid of update some parts, especially material_color_utilities or flutter_riverpod those packages bump up version after:
flutter pub get

ok @tenshiAMD problem with pubspec.loc solved, i pushed changes related only with linter, we can close this PR and focus on: https://github.com/EXERLOG/exer_log/pull/191

ok @tenshiAMD problem with pubspec.loc solved, i pushed changes related only with linter, we can close this PR and focus on: #191

@mfederowicz ok thanks
@BertVanMieghem we are closing this one for now. thanks.


Update windows-update-issues-troubleshooting.md

Added a section for transient errors where users do not have to take remedial action.

#sign-off

I'll signoff, but want to inform @Deland-Han (probably should fix metadata on this article at some point 😄 )
#sign-off

We created a bug to triage this change request. We will response soon.

@kartv-ms The changes are live at https://learn.microsoft.com/en-us/troubleshoot/windows-client/deployment/windows-update-issues-troubleshooting. This PR will be closed.


chore: cache Crossref requests + improve error message

Fixes #370
Fixes #364

FYI sentry-io report is a false positive. Changes haven't been deployed to production yet.

Thanks @kesara . May I have access to Sentry anyway?

@stefanomunarini Sorry, can't provide access to Sentry. But happy to provide any logging information when required.


Fix Hydration Error in /sdk page by fixing ProductCard

Fixes https://github.com/thirdweb-dev/dashboard/issues/747
Code Change

Change <p> to <div> by Using <Box> instead of <Text> as a container of props.children for ProductCard component to fix hydration error in /sdk page and future hydration errors as well
Added color and lineHeight styles to maintain styles as it was earlier

Seems like it was broken from before but just checked on mobile and this is how it looks
Probably would be good to fix it on this same PR.

@nachoiacovino Yeah, It was broken already. I just fixed the Hydration error - and kept the UI as is.
Since this branch is already merged, Let me fix that in a new branch

The UI issue fixed in https://github.com/thirdweb-dev/dashboard/pull/749


Latest published Travis binary errors when run with Rust 1.27.0

The error is:
/home/travis/.cargo/bin/cargo-tarpaulin: error while loading shared libraries: libproc_macro-5a157e6e97a5a7d7.so: cannot open shared object file: No such file or directory

I assume libproc_macro is part of the Rust toolchain and that the hash part of its filename changes between releases.  I don't know if there's anything that can be done to avoid this error, but it would be good if the readme mentioned that published binaries were tied to Rust releases, and perhaps indicate which version a binary needs in its filename (e.g. cargo-tarpaulin-0.6.2-rust-1.26.1-travis.tar.gz) to make it more obvious. I've just switched to using cargo install and caching the result.

Good catch, I've updated the readme to warn about this issue in the installation section and am currently updating the release script to change the name. I'll close once it's updated and pushed

I ended up updating the travis-install.sh to print WARNING This method is no longer recommended. Use docker or crates.io to install tarpaulin to stdout after installing from github releases, and with the readme updated hopefully everyone will migrate away.
I'm going to close this issue now. Bujt if you see anyone in the wild using travis-install recommend they use crates.io or docker to install tarpaulin :+1:

Revisiting this because I rediscovered the travis binaries that are still being published, wondered why I wasn't already using them, and found that I'd filed this issue...
I checked what shared library the published binary for v0.8.6 is linked against, and it no longer refers to any Rust-specific .so files. I also ran a Travis build against Rust 1.35, 1.36 and 1.37 using the same binary, and all worked, so I don't think this is an issue any more. I don't know what changed, but I guess maybe it was stuff getting stabilised?


Port the script to mac

As a macOS user I want to ask if the script will be macOS friendly in future

Additionally, I get this error when I try to run the script:
Select a bot: 1   File "bots/server/discord_text_spam.py", line 33     async def on_ready():         ^ SyntaxError: invalid syntax

The script is fully universal on all OS.
windows, mac and linux
and what is your python version.

Python 2.7.15
It's also possible that I just don't know how to use the script properly, but it appears that I configured everything that is necessary for it to run.
I would suggest you to finish the Wiki thing with step by step explanation of how the whole thing works. Surely there are a lot of people who have a small experience with using suchlike scripts.
That's all for now. Thank you for your reply!

I tried running the script on fresh with everything configured again.
This error appears:

That would be the problem, this script is made for python 3.6.4

Oh alright, I'll try that out.


Fixes Problems that prevent it from running on linux

Fixed Line Ending Issue in Launcher
Renamed Launcher to match the naming scheme
Added Multiple Instances of switching the executable name depending on the OS
Added the use of ps a when killing previous instances on Linux
Added the RPC_BIND_PORT variable to allow to user to change it easily in case the default port is used
Changed to using full path in some instances to prevent errors
Generalized some print statements so they would make sense on all platforms

Thanks for this! I don't have a lot of Linux experience, so it's much appreciated!


Goto Definition & Autocompletion not working

Environment

Elixir & Erlang versions (elixir --version):

Erlang/OTP 22 [erts-10.6] [source] [64-bit] [smp:6:6] [ds:6:6:10] [async-threads:1] [hipe]

Elixir 1.10.0 (compiled with Erlang/OTP 21)

Operating system: macOS Catalina
Editor or IDE name (e.g. Emacs/VSCode): VSCode
LSP Client name:
If using VSCode, are you using "ElixirLS Fork: Elixir support and debugger" (note that it has "Fork" in the name)? Yes

Crash report template
These occur when pressing F12 (Goto definition) or when holding Command key + mouse click
[Info  - 5:55:42 PM] Returning location struct
[Info  - 5:55:42 PM] Returning location struct
[Info  - 5:55:45 PM] Returning location struct
[Info  - 5:55:45 PM] Returning location struct

These when calling/triggering auto-complete, but not right after: 'Loading...' appears and this is displayed after I click enter (even though nothing was displayed.)
[Error - 5:55:57 PM] Request textDocument/documentSymbol failed.
  Message: an exception was raised:
    ** (SyntaxError) nofile:20: syntax error before: ','
        (language_server 0.3.1) lib/language_server/providers/document_symbols.ex:35: ElixirLS.LanguageServer.Providers.DocumentSymbols.list_symbols/1
        (language_server 0.3.1) lib/language_server/providers/document_symbols.ex:20: ElixirLS.LanguageServer.Providers.DocumentSymbols.symbols/3
        (language_server 0.3.1) lib/language_server/server.ex:485: anonymous fn/3 in ElixirLS.LanguageServer.Server.handle_request_async/2
  Code: -32000

17:55:57.357 [error] Process #PID<0.350.0> raised an exception
** (SyntaxError) nofile:20: syntax error before: ','
    (language_server 0.3.1) lib/language_server/providers/document_symbols.ex:35: ElixirLS.LanguageServer.Providers.DocumentSymbols.list_symbols/1
    (language_server 0.3.1) lib/language_server/providers/document_symbols.ex:20: ElixirLS.LanguageServer.Providers.DocumentSymbols.symbols/3
    (language_server 0.3.1) lib/language_server/server.ex:485: anonymous fn/3 in ElixirLS.LanguageServer.Server.handle_request_async/2

'Loading...' sign also appears on mouse over but nothing happens.
Developer tools
[Extension Host] ok:localhost:/tmp/elixir-sense-1584724345293026000.sock
[Extension Host] [vscode-elixir] ElixirSense client connected on localhost:/tmp/elixir-sense-1584724345293026000.sock
[Extension Host] [vscode-elixir] ElixirSense client connection closed
[Extension Host] [ElixirSense] 18:12:25.306 [error] Task ElixirSense.Server.TCPServer started from #PID<0.200.0> terminating** (stop) exited in: GenServer.call(ElixirSense.Server.TCPServer.ConnectionHandlerSupervisor, {:start_task, [{:nonode@nohost, #PID<0.200.0>, #PID<0.200.0>}, [#PID<0.200.0>, #PID<0.199.0>], {:erlang, :apply, [#Function<1.131136907/0 in ElixirSense.Server.TCPServer.start_connection_handler/2>, []]}], nil, nil}, :infinity)
        ** (EXIT) no process: the process is not alive or there's no process currently associated with the given name, possibly because its application isn't started
        (elixir 1.10.0) lib/gen_server.ex:1013: GenServer.call/3    /Users/facundodiaz/.vscode/extensions/mjmcloug.vscode-elixir-1.1.0/elixir_sense/lib/elixir_sense/server/tcp_server.ex:58: ElixirSense.Server.TCPServer.accept/2
        (elixir 1.10.0) lib/task/supervised.ex:90: Task.Supervised.invoke_mfa/2
        (stdlib 3.11) proc_lib.erl:249: :proc_lib.init_p_do_apply/3
Function: &ElixirSense.Server.TCPServer.listen/3
        Args: ["unix", "localhost", "0"]
ok:localhost:/tmp/elixir-sense-1584724345307545000.sock

I can provide more info if needed. Thanks!!

It looks like you have vscode-elixir installed as well. I believe that the two extensions are not compatible with each other. Can you try removing vscode-elixir? Also are there any syntax errors in the file that you are operating on?

It works after removing vscode-elixir. Didn't imagine that.
Thanks for your time, I will close the issue :)

@axelson It seems that vscode-elixir is causing a lot of issues. Maybe we should print a warning if we detect it.

Yeah that's a good idea. Filed it at https://github.com/elixir-lsp/vscode-elixir-ls/issues/56


Sentence based selection strategy

Issue copied from https://github.com/lukasbach/textannotation/issues/2
Currently, the sentence based selection strategy discovers sentence boundaries based on the occurence of dots. In some instance, a dot may appear in the center of a sentence without it defining its end, such as "Today, Mr. Doe visited his aunt."
A more sophisticated strategy might be suitable. The current implementation is located at edu.kit.textannotation.annotationplugin.selectionstrategy.SentenceSelectionStrategy::evaluate. Altering this method also changes the strategy that is being leveraged.

Improved the strategy with commit b3ad8e7a4ffee06a7d4d4ebb40e33b996e6afac5. It is still heuritic-based and looks for punctuation marks like ".", "!", "?" and also respects multiple marks, e.g., "!?".
Also, the selection respects common abbreviations that are followed by a "." like "Mrs.", "Dr.", "Prof." etc.


[GKE v2] Imported Cluster Imported Flag is changed after sync

What kind of request is this (question/bug/enhancement/feature request):
Bug
Steps to reproduce (least amount of steps as possible):
Create a GKE cluster in GKE Console that will be imported.
In Rancher UI import the GKE cluster.
Allow the sync to occur between rancher and gke.
Result:
Once the sync is complete, view the cluster data in the api and see that in upstreamsepec imported is no longer true. This causes issues with syncing because the upstreamspec is the source of truth for the UI to sync the changes down to the empty GKE config when editing.
Other details that may be helpful:
Environment information

Rancher version (rancher/rancher/rancher/server image tag or shown bottom left in the UI):
Installation option (single install/HA):

Cluster information

Cluster type (Hosted/Infrastructure Provider/Custom/Imported):
Machine type (cloud/VM/metal) and specifications (CPU/memory):
Kubernetes version (use kubectl version):

(paste the output here)

Docker version (use docker version):

(paste the output here)

Duplicate of https://github.com/rancher/rancher/issues/32072


Make documentation readme fetching part of the runtime

It is currently done during the build using the  addons/launcher-addon/add-template-files.sh script.
The process should be close to the one used in the BCS.
As part of issue #185, it will improve mvn clean install perf

Take a look here: https://github.com/fabric8-launcher/launcher-booster-catalog-service/blob/master/src/main/java/io/fabric8/launcher/booster/catalog/AbstractBoosterCatalogService.java#L127-L208
It's the three places where we use CompletableFuture.

We could have a setupDocumentation() or initDocumentation() or whatever that pre-fetches the contents (without doing an actual get() to block) while the code that actually needs the information could do a ,get(). We could also have a resetDocumentation() or something similar to the reindex() method we have in the BCS. And we could call that from https://github.com/fabric8-launcher/launcher-backend/blob/master/web/src/main/java/io/fabric8/launcher/web/endpoints/BoosterCatalogEndpoint.java#L162 perhaps so that a reindex of the booster catalog also triggers a re-clone of the documentation.

Btw, for doing the clone itself you can look here: https://github.com/fabric8-launcher/launcher-booster-catalog-service/blob/master/src/main/java/io/fabric8/launcher/booster/catalog/spi/NativeGitBoosterCatalogPathProvider.java#L55-L71

I wonder if the BCS has some sort of hook that we could attach to its lifecycle. Considering that everytime the booster is re-indexed the README should also be read again, I think that would be an interesting feature to have

@gastaldi I don't think it makes much sense. The BCS doesn't have anything to do with reading the docs. And the one that triggers the re-indexing is the backend, so that one can simply also call the re-reading of the docs.

@quintesse regarding the clone part, IMO, using jgit may be a better idea (even in the BCS by the way).

@ia3andy no, we used JGit before and we removed it. JGit can be extremely slow. Native Git is just better for this.

@quintesse this is a good reason ;)

JGit doesn't support partial fetches. That's why we're using native Git

@gastaldi that's another good reason hehe

I wonder if the BCS has some sort of hook that we could attach to its lifecycle. Considering that everytime the booster is re-indexed the README should also be read again, I think that would be an interesting feature to have

I take that back, it's better to have the documentation in a separate endpoint, to be triggered as a webhook when changes to launcher-documentation are pushed

Sure, I don't have a problem with that. The webhook thing makes sense. Remember though that if we at some point merge the READMEs into the Booster Catalog we'd either have to trigger 2 endpoints with a single change or we'd have to merge them after all.


Android Picture improves image clarity and cannot be used

I got a picture of the current TF input and saw that the resolution is not high. I tried to change to an HD picture, but

After the modification, it cannot be identified; I would like to ask if it is impossible to use high-resolution graphics when designing.

after edited bitmap code

This question is better asked on StackOverflow since it is not a bug or feature request. There is also a larger community that reads questions there.
If you think we've misinterpreted a bug, please comment again with a clear explanation, as well as all of the information requested in the issue template. Thanks!


[AUDIT] consider utilizing a new callback readyForExecution

I speculate that we have scenarios in the plugin such as metrics where this callback is useful

@gerashegalov can you add the Spark commit/issue that this is related to?  That will help us assess priority and scope.

@mattahrens done!


outlookMsgToEmail fails with unsuccessful Exchange delivery receipts

If you work with Outlook/Exchange you can request delivery receipts for your emails. After sending an email, you'll get a receipt if the email has been successfully delivered to the recipient's server.
If the recipient's server supports these confirmations, the returning email can successfully be parsed and converted by Simple-Java-Mail's Outlook to email converter.
If the recipient's server doesn't support this feature, you'll get a different email from your Exchange server. This message contains an attached email without body (empty text, empty html, empty rtf). Therefore the OutlookEmailConverter.buildEmailFromOutlookMessage recursive call fails on line 111 because the mime-body of the attached mail is empty and the whole conversion is stopped with an IllegalStateException (InternalEmailConverterImpl.java:42).
https://github.com/bbottema/simple-java-mail/blob/f306e89d477ea64e48b0308339bd178af8593360/modules/outlook-module/src/main/java/org/simplejavamail/internal/outlooksupport/converter/OutlookEmailConverter.java#L111
I would suggest to make this conversion more fault-tolerant to support the mentioned Exchange messages.
Unfortunately, I can't attach a sample email because of privacy issues but I would be happy to help resolving this issue.
BTW: Because of some dependency issues, I still have to work with version 6.7.5.

It seems like I've ran into the same/a similar issue
I have an example mail that states that the recipient's server did not send a receipt for successfully delivering the mail so I thought this might be the same base use case. If not sorry for hijacking this issue, I'll open a new one then. I can't publish the mail here, but if you wand I can send it to you directly.
I've tested this with version 7.1.1, but the original problem was discovered on a system using 6.6.1.
Using the EmailConverter I get two warnings and finally the IllegalStateException.
Doing the same with the OutlookMessageParser works as expected.
public static void main(String[] args) throws IOException {
	String msgFileName = ".\\files\\Mittels Relay umgeleitet test.msg";

	try (FileInputStream fileInputStream = new FileInputStream(msgFileName)){
		EmailConverter.outlookMsgToEmail(fileInputStream);

		//OutlookMessageParser outlookMessageParser = new OutlookMessageParser();
		//OutlookMessage outlookMessage = outlookMessageParser.parseMsg(fileInputStream);
	}
}

14:01:43 [main] WARN  OutlookMessage - Skipping nested Outlook message as file attachment, writing Outlook messages back as data is not supported!
14:01:43 [main] WARN  OutlookMessage - To access the nested Outlook message as parsed Java object, refer to .getAttachments() instead.
Exception in thread "main" java.lang.IllegalStateException: This should never happen
	at org.simplejavamail.converter.EmailConverter.mimeMessageToEMLByteArray(EmailConverter.java:559)
	at org.simplejavamail.converter.internal.InternalEmailConverterImpl.mimeMessageToEMLByteArray(InternalEmailConverterImpl.java:41)
	at org.simplejavamail.internal.outlooksupport.converter.OutlookEmailConverter.buildEmailFromOutlookMessage(OutlookEmailConverter.java:126)
	at org.simplejavamail.internal.outlooksupport.converter.OutlookEmailConverter.outlookMsgToEmailBuilder(OutlookEmailConverter.java:85)
	at org.simplejavamail.converter.EmailConverter.outlookMsgToEmailBuilder(EmailConverter.java:239)
	at org.simplejavamail.converter.EmailConverter.outlookMsgToEmail(EmailConverter.java:221)
	at org.simplejavamail.converter.EmailConverter.outlookMsgToEmail(EmailConverter.java:213)
	at ParseP7s.main(ParseP7s.java:30)
Caused by: java.io.IOException: No MimeMessage content
	at jakarta.mail.internet.MimePartDataSource.getInputStream(MimePartDataSource.java:92)
	at jakarta.activation.DataHandler.writeTo(DataHandler.java:278)
	at jakarta.mail.internet.MimeBodyPart.writeTo(MimeBodyPart.java:1670)
	at jakarta.mail.internet.MimeMessage.writeTo(MimeMessage.java:1888)
	at jakarta.mail.internet.MimeMessage.writeTo(MimeMessage.java:1862)
	at org.simplejavamail.converter.EmailConverter.mimeMessageToEMLByteArray(EmailConverter.java:555)
	... 7 more
Caused by: jakarta.mail.MessagingException: No MimeMessage content
	at jakarta.mail.internet.MimeMessage.getContentStream(MimeMessage.java:1395)
	at jakarta.mail.internet.MimePartDataSource.getInputStream(MimePartDataSource.java:78)
	... 12 more

@Faelean I'm pretty sure you're mentioning the exact same issue. I could temporary fix it by doing a null check on nestedMsg's html, text and rtfText properties. If none of these properties are set, I skip the recursive call to ignore the nested (empty) email.
See https://github.com/bbottema/simple-java-mail/blob/f306e89d477ea64e48b0308339bd178af8593360/modules/outlook-module/src/main/java/org/simplejavamail/internal/outlooksupport/converter/OutlookEmailConverter.java#L107

@bbottema Just sent you an example by email. Thanks a lot!

Is there any purpose to the empty attached message or can we just ignore it, then.

Fixed in 7.2.1. I'm just ignoring this very specific exception cause. Might be good to ignore any and all exceptions here, but for now I rather take it case by case, so we slowly build up a catalogue of known Outlook issues.

About 6.x.x version range, I'm not sure I want to support it there. You can't move to 7.x.x? 6 is really old school.

Thanks a lot for solving this issue!
We are currently bound to 6.x.x because of the javax/jakarta dependency. But I do completely understand your point.

You can forget about V6 regarding this issue.
Thanks a lot for all your work!


Support count(*) function in QueryBuilder

Allow count() function in QueryBuilder.
Example:
new QueryBuilder()
.select()
.count("")
.as("count")
.from(commandContext.namespace(), commandContext.collection())
.where(conditions)
.build();

@maheshrajamani I know this exists:
https://github.com/stargate/stargate/blob/d6647b887e815c8578a8c150cec2e3e974745764/coordinator/persistence-api/src/main/java/io/stargate/db/query/builder/QueryBuilderImpl.java#L405-L408
is count not supporting it?

@ivansenic If I am not wrong this is to run query as select * from <table name>


Deployment failure

⚠ Please verify that this bug has NOT been raised before.

[X] I checked and didn't find similar issue

🛡 Security Policy

[X] I agree to have read this project Security Policy

📝 Describe your problem
I want to deploy to fly. io, but I don't know why the deployment failed.
uptime-kuma:base-debian

uptime-kuma:base-alpine

🐻 Uptime-Kuma Version
1.16.1
💻 Operating System and Arch
20.10.12 linux x86_64
🌐 Browser
Microsoft Edge 102.0.1245.33
🐋 Docker Version
Unknown, possibly new
🟩 NodeJS Version
No response

It even seemed to be working, but suddenly it wasn't
（Cry ）QAQ，I don't know why~~~~~

Have you checked out these links related to the issue:
https://fly.io/docs/getting-started/troubleshooting/
https://community.fly.io/t/failed-due-to-unhealthy-allocations/199/8

您是否查看了与该问题相关的以下链接：https://fly.io/docs/getting-started/troubleshooting/ https://community.fly.io/t/failed-due-to-unhealthy-allocations/199/8

I tried to modify the port, but the same problem still appeared

I tried for three hours, but it still didn't work

Unfortunately I think fly.io has modified init which has trouble with chown. I remember when I deployed to fly.io I had to modify entrypoint.sh. Removing lines 8-16 should do.

Unfortunately I think fly.io has modified init which has trouble with . I remember when I deployed to fly.io I had to modify . Removing lines should do.chown``entrypoint.sh``8-16

Thanks for your help. Looks like it's working
But it seems to be dead again

Looks like you are missing the compiled frontend files. You can get those by running npm run setup in the local directory.

deployed successfully!!!!!!
Thank you. Thank you both. Thank you very much！！！！！！！！

Hi. I have the same issue. How can I modify the entrypoint.sh on Windows ?


Update cloudflared.yaml

add network mode host to template, otherwise cloudflared has no access to localhost

Thank you for the PR!


Remove manifests and import ethpm manifests

What was wrong?
Unnecessary duplication of manifests in this library
Issue #
How was it fixed?
Removed unnecessary dupes and replaced with manifests in py-ethpm
Cute Animal Picture

merging since minor refactor


Entity Framework Repository

one more thing @asadsahi  please help
you have entity base repository like
public class EntityBaseRepository : IEntityBaseRepository where T : class, IEntityBase, new()
i am trying to as PhotoRepository : EntityBaseRepository
its throws erros can use IPhotoRepository as type parameter.
or please can you give at least one example how to use this class for DB interaction
as i am trying to implement UnitOfWork pattern

@sajidali2444 check this out as an example.

I cannot seem to get the included IUnitOfWork to persist changes to the database using the repository pattern. Could you provide an example, or give direction?
https://github.com/asadsahi/AspNetCoreSpa/blob/master/src/AspNetCoreSpa.Infrastructure/IUnitOfWork.cs


implements csv output mode. closes #2836

Hi,
in this version, I am implementing a simple way to output CSV using a third-party library 'csv-stringify'.
By default, the CSV output has a header and the default settings specified here. I had a discussion with @waldekmastykarz  about allowing users to configure the CSV output using our configuration file. I would suggest doing this as a separate issue once this feature is integrated.
On top of this, I added the new output mode in our documentation. I noticed that we still mentioned that the default output mode is text. I corrected also that part.
Cheers

I performed the following tests in case you are wondering how to test this new feature:

output CSV with a single object: expects a header + 1 record
output CSV with an array: expects a header + all records
output CSV with the reporting commands: expects at least a header

The last question to you guys before merging it. For the output mode JSON, we are writing the word in capital letters. Would it make sense to do the same with CSV?

The last question to you guys before merging it. For the output mode JSON, we are writing the word in capital letters. Would it make sense to do the same with CSV?

The output type is json, so I'd suggest using csv here.

Merged manually. Thank you! 👏


Merge coverage results

Coverage increased (+1.4%) to 97.473% when pulling e4325d2b7aaab18309c3f414e9eb6f01b286e6e0 on merge_coverage into d0a02975faa2ec3cbe727e7fdbda6d16992fce93 on master.


when pressing enter on a very large span attribute, expand it

Some people put large data in span attributes, so you won't be able to see it even if you change the width. It would be nice to be able to press enter and see the entire attribute value in this case. That, or something similar

We can simply show a modal or something when pressing enter. This could be a solution of https://github.com/ymtdzzz/otel-tui/issues/38#issuecomment-2204768531 .
I'll take a look at.

Current progress ( never mind about the brightness of my terminal 😅 )
This modal is shown when pressing Enter. To close it, we can simply press Enter again or change the selection.


MAINT: Check that scipy.linalg.lapack functions are documented

Would have caught an error I only saw in #10741 by manually checking the docs, and caught an existing doc error introduced in #10537.

I wonder why refguide_check does flag it. Maybe need to opt in scipy.linalg.lapack and .blas?

I wonder why refguide_check does not flag it.

Probably because scipy.linalg.lapack.__all__ == ['get_lapack_funcs']. Maybe this, too, is a bug.

In fact, linalg.blas and .lapack are specifically excluded: https://github.com/scipy/scipy/blob/master/tools/refguide_check.py#L120
Since this PR adds a test for them to be present, maybe avoid duplication and just add these names to __all__ and stop excluding them instead?

Yeah that sounds like a better approach

I briefly looked into this (https://github.com/scipy/scipy/compare/master...ev-br:refguide-lapack?expand=1) and I take my suggestion back.
Making refguide-check pass would require that all the linalg.lapack names are listed in the linalg docstring. And that is not desired, scipy.linalg docstring should keep listing the high-level stuff instead of being cluttered with low-level lapack names.
So, +1 for this PR from me.

Indeed I was hoping to keep track of these additions (I also noticed lots of missing entries and added in #10075) but failed in the last PRs. Also +1 from me. As a bonus I have some stuff to do from that PR which I will add now.

Sounds good to me, feel free to merge in that case

Merged, thanks @larsoner !


Error running francinette in Pipex project folder

It keeps running get_next_line tests. The NAME in my Makefile is "pipex". I have tried changing it to libft.a, it successfully runs libft tests. Any target it does not find, it goes to get_next_line, so I guess it is not recognizing pipex as a project it should test.

I checked the logs in francinette's dir and noticed it was running some cmds with get_next_line's source files which I had in the root of PIPEX. Moved the files to a subdir and it fixed the issue ...


Stabilized mode with semi-auto "flip" capability

Should be a good way to test high angular rates and unusual attitudes without requiring too much skill from the pilot. Setting the new parameter to e.g. 0.9 results in switching from stabilized to acro mode only when roll or pitch command exceeds 90%. Just hold the stick against the stop in the direction you want to flip and let go once you get past inverted to recover to level.
Log: http://dash.oznet.ch/view/zCRt9Qby6KtG8PMooMaLzb (FMUv1 on QAV250 running Q attitude estimator)
Onboard Video: https://youtu.be/xF5vHBSV5KU
Forward flip (first one) exhibited loss of heading lock (90 degrees), second 2 (backward) held heading. Need to repeat the forward case to see if it's repeatable. Note the much larger moment of inertia about the pitch axis.

Need to change the parameter default to 0.0 so that acro mode behavior is normal by default.

This seems pretty useful when you have the param set to a default of zero. Is that coming?

last commits are bench tested, will flight test shortly

flight test of manual, stabilize and acro modes OK

I've flown this today.
It worked, until I crashed my 250 quad with it ;). Couldn't do a complete flip because my rates were way to low, so I more or less rammed it into the ground.. Anyway, will try again, seems to be a cool addon :)

I increased my acro rates to 720 deg/sec. The default of 90 was unflyable
in my opinion.
Hope you didn't break anything.
On Tue, Jun 23, 2015 at 12:10 PM, Andreas Daniel Antener <
<EMAIL_ADDRESS>wrote:

I've flown this today.
It worked, until I crashed my 250 quad with it ;). Couldn't do a complete
flip because my rates were way to low, so I more or less rammed it into the
ground.. Anyway, will try again, seems to be a cool addon :)
—
Reply to this email directly or view it on GitHub
https://github.com/PX4/Firmware/pull/2390#issuecomment-114594173.

--
Mark Whitehorn
<EMAIL_ADDRESS>

No worries, had soft ground ;)

If you're flying an H quad like the QAV250, it's much easier to roll than to pitch (meaning it goes faster). Worse, when I do a backward flip with my 250, it also yaws 90 degrees, which is my excuse for hitting the tree trunk. Forward flips work OK, but take so long I lose a lot more altitude than when doing rolls. The rolls are lots of fun too, can't wait to try it with the goggles :)

Did another test with this today, very cool!
720dps and switch at 0.9 > triple flip followed by an almost nice landing ;)
I didn't notice any difference though between roll/pitch flips. Both went very good, and I had no unwanted yawing.

Triple flip  :+1:  Did you mean to do that?
How much altitude did you start with and did you do any throttle management?
Were you flying a QAV250? There must be something wrong with my setup, since the flips are much slower than the rolls.

Yes, QAV250 and the triple flip was intentional ;), from about 30m I'd say. I didn't manage throttle, but I'd like to try that. Not sure how much you can do at this roll rate though (or if I get the timing right).
When I look at my videos, my pitch flips are actually faster than my rolls.

For your amusement: https://youtu.be/n22Z4f8KvNo

Thanks, cool video. I'm guessing your pitchrate P gain is  higher than the rollrate P gain?
My pitchrate gain is lower than roll. will try increasing it when I get my QAV250 back together.
This is a perfect opportunity to add more intelligence; automatic thrust management for fixed pitch, then moving on to collective pitch.

Yes, I think I have about 1/3 higher pitch P than roll P.

Last commit removes manual mode parameters which had moved to the position controller. Test flown on QAV250.
@LorenzMeier  Also demonstrated that the yaw setpoint fix https://github.com/PX4/Firmware/pull/2481 works.

@LorenzMeier It looks like this is OK to merge; the default value for MC_ACRO_STAB is 0.0, so default Acro mode behavior will not change.


Can I use a Jameco 112366 with board 1.4 rev3?

As the title says, I have a board version 1.4 rev 3, but according to the firmware and documentation, I can only use a Jameco 157041: https://github.com/CircuitSetup/Split-Single-Phase-Energy-Meter/blob/master/Software/EmonESP/src/energy_meter.h#L63-L64
Is this actually the case or can I use a Jameco 112366 (essentially identical to the 15704 but is a 1.5A instead of a 1A) with a rev3 board?

Sorry I didn't see your question sooner.
Yes, that transformer can be used. I would recommend calibrating it to the meter so you get accurate readings.


Task System - Move responsibility for creation/disposal into TaskFlowGraph

Most of the data/job related objects are created in the TaskFlowGraph and then disposed the same way via TaskFlowGraph.DisposeFor
However some aspects like sub task drivers and cancel requests data stream are still manual and it's a bit weird.

Does this conflict with the effort in #68 where we want the graph to just be a set of relationships and not anything concrete? Or is this task just some bridge work to wrap up loose ends and #68 will move the logic again when migrating to a factory pattern?

I think this is just some cleanup work to make sure everything is happening in a consistent place. With #68 doing the split I think the ownership/disposal part will be separate from the relationships and it will move.

With this comment: https://github.com/decline-cookies/anvil-unity-dots/pull/77/files#r989150346
It's pretty apparent that the ownership should reside in the TaskSystem/TaskDriver and the disposal happens there too.
The graph should only handle the relationships.


#724 Fonts refactor

Notes: app.bundle.[hash].js is not cached. It's too large for our current max file size filter.
Console:

Application's Tab
Registration

Precache Assets:

Runtime: Google Fonts Stylesheets

Runtime: Google Fonts Webfonts

PWA is looking good, but yarn run build:package is not bundling the font files:

@galelis, it may not be worth bundling for UMD, but we'll need to run that by @swederik.
Most loaders simply split and spit out chunks in dist.
UMD may be able to import them from relative location, but would assume they're hosted alongside one-another.
This would work okay for unpkg, but not offline scenarios.

https://survivejs.com/webpack/loading/fonts/

We could base64 inline them, but it feels like a lot of bloat, and the loaders appear out of date:

https://stackoverflow.com/questions/41657087/webpack-inline-font-with-url-loader
https://webpack.js.org/loaders/url-loader/ (may not support)

file-loader does not appear to support what we're trying to do:

https://webpack.js.org/loaders/file-loader/

Recommended:
Either import using url-loader and publish alongside index.umd.js
Document that roboto needs to be loaded if using a source other than unpkg or your own bundler from the npm dependency
OR
Assume roboto is available globally.
Document that users should load roboto.
If they don't we fallback to the system font-stack.

CC: @swederik, review on Monday?


Use current models transaction client

I am running into problems where .assignRole is creating a new transaction and using up my transaction pool when used in a loop (think importing users from a CSV).
Patching the package so it uses the current models transaction fixes my issue.
public async assignRole(
  this: HasRolesModelContract,
  ...roles: Array<RoleContract | string>
): Promise<void> {
  const roleModels = await Promise.all(
    roles.map(async (r) =>
      typeof r === 'string' ? await Role.firstOrCreate({ name: r }, { name: r }, { client: this.modelTrx }) : r
    )
  )

  await this.related('roles').attach(roleModels.map((r) => r.id))
}

I guess this should be done for syncRoles and revokeRole as well.

We should patch it to do creation of strings in one go using fetchOrCreateMany then merging the array.
PR welcome

We should patch it to do creation of strings in one go using fetchOrCreateMany then merging the array.

FWIW, I view that as two separate issues.
Issue 1: It's currently not possible to assign roles inside of an ongoing transaction.
Issue 2: Assigning multiple roles results in querying the roles table multiple times.

Agreed


GROOVY-11192: Diamond inference should work on subclasses with less generic parameters

Bug Description
Something like
            interface A<K, V> {
            }
            class B<V> implements A<Long, V>{
            }
            A<Long, String> a = new B<>()

would cause groovyc to fail.
Analysis
https://issues.apache.org/jira/browse/GROOVY-11192
There is a bug in org.codehaus.groovy.transform.stc.StaticTypeCheckingVisitor#inferDiamondType.
The inferredType is java.util.Map<java.lang.Long, java.lang.String> -> java.util.Map<K, V> and cceType is MapLong<> -> MapLong<V>.
After adjustGenerics() on line 1124, cceType becomes MapLong<java.lang.Long, java.lang.String> -> MapLong<V>, which is incorrect. It should be MapLong<java.lang.String> -> MapLong<V>.
Fix
If we see (# of generic types for A) > (# of generic types for B's redirect), then only keep the generic types that correspond to B's redirect generic types.
A.genericTypes = [Long, String]
B.genericTypes = []
A.redirect().genericTypes = [K, V]
B.redirect().genericTypes = [V]
In this case, find the index of V in A.redirect().genericTypes, then find the corresponding type in A.genericTypes, which is String.

could you please add a test case:
        interface A<G1, G2> {
        }
        class B<G3> implements A<Long, G3>{
        }
        A<Long, String> a = new B<>()

oh yes.. using shouldCompile would be better than assertScript where possible. We have to test we repeating names for generic types because of mistakes, that have been don in the past where generic types leaked to other classes with the wrong name, and then solutions that worked only because of the just mentioned reason and not if the names are different.

Generics is a very tricky area of the compiler.  The diamond inference may assume that the target type's generics exactly correspond to the constructor type's generics.  I don't think the > check is sufficient.  You may have a type like "class C<V,K> implements  Map<K,V>".
inferDiamondType should be walking from 'Map' to MapLong calling parameterizeType each time to fill in the type parameters.  So, I would expect a fix in inferDiamondType not adjustGenerics if something is amiss.

Thank you so much @blackdrag @eric-milles for your comments . Now I understand the problem is more complicated than I thought. Closing the PR for now.

Generics is a very tricky area of the compiler. The diamond inference may assume that the target type's generics exactly correspond to the constructor type's generics. I don't think the > check is sufficient. You may have a type like "class C<V,K> implements Map<K,V>".

Can you share a bit more on You may have a type like "class C<V,K> implements Map<K,V>? This PR is about when subclasses have less type parameters.
Are you suggesting the compiler might wrongly inference class C as <K,V> instead of <V, K>?


Remove trailing whitespace from Python files

Thanks very much, but I think the current maintainers don't care about those aesthetics in some relatively short-lived tool-code. I am going to close, unless another maintainer wants this reopened.


[Compute] Fix #20133: az vm create: Fix --data-disk-delete-option not working when no --attach-data-disks are provided

Issue: #20133
Description

Fix #20133 data-disk-delete-option was not working when no attach-data-disks are provided with it.
Testing Guide

az vm create --data-disk-delete-option Delete --resource-group {rg} --name {vm-name} --image UbuntuLTS
After VM deletion, the data disk created should be deleted.
History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change.
[Component Name 2] az command b: Add some customer-facing feature.

This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] The PR title and description has followed the guideline in Submitting Pull Requests.

[ ] I adhere to the Command Guidelines.

[ ] I adhere to the Error Handling Guidelines.

Compute

Hello Team, Hello @calvinhzy
I tested again on my subscription and after I delete the VM from RG, the attached data disk is not deleted automatically.
As far as I understand, upon deployment when using the --data-disk-delete-option " Delete" the data disk should be deleted automatically, after I go for example in the RG , select VM and delete it.

Hi @gitoboc
This fix is only merged to dev branch right now. We have a monthly release cycle, of which the next one is on 12/07.


Improve import of Flow types

cc @PauuloG.
This gives us proper flow-type-vscode autocomplete too.

Codecov Report

Merging #115 into develop will decrease coverage by 3.12%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           develop     #115      +/-   ##
===========================================
- Coverage    73.33%   70.21%   -3.13%
===========================================
  Files            8        8
  Lines           45       47       +2
  Branches         6        6
===========================================
  Hits            33       33
- Misses          12       14       +2

Impacted Files
Coverage Δ

src/models/Project/index.js
0% <0%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


dev(obs): bump deps and removed unused

Summary
Removes some of the unnecessary values set for the meter and the tracer + bumps the deps.
Testing strategy
No testing set in place as this just modifies crate version. If needed can add some.

Thank you for this!


No longer works in Create 0.5.1

As the title suggests, lit forges under a boiler do not heat the boiler in Create 0.5.1.

Can confirm, no longer getting any heat registered on the boiler with a fully heated charcoal forge under it.


fix security connector segfault

Passing nullptr to absl::string_view (https://github.com/grpc/grpc/blob/c0e2fe9ea040b6e1e901b540eb29f318d9c1c541/src/core/lib/security/security_connector/security_connector.cc#L48) is not allowed and would lead to segfault.
@veblush

I don't think I have the permission to add a label :(

As far as I can tell, this change was merged via / included in https://github.com/grpc/grpc/pull/29346

The fix has been merged in another PR.


Enhancement - Customer

Is your feature request related to a problem? Please describe.
Enhance Features, fix issues, and improve performance
Todo's

[x] #464
[x] #465
[x] #466
[x] #467
[x] #468
[x] #469
[x] #470
[x] #589
[x] #633
[x] #634
[x] #635
[x] #890
[x] #891
[x] #1031
[x] #1032

Closing this issue since all tasks has been completed


Release v1.0.3 to CocoaPods

This is a useful little lib! I'd like to use it within an application I am building for an up-coming tutorial on raywenderlich.com - so hopefully it will give this project a bit more exposure.
I'm using iOS9, so could really do with having v1.0.3 on CocoaPods. Thanks :-)

bump
sorry to be a pain, but I am writing the article next week and would really love to use this code :-)

It's awesome that you're writing an article using this. I'm excited to see it :)
I got the earlier comment. I just moved and started a new job so things have been hectic the past week. I'll push the updates before Friday.

Thanks - much appreciated. And good luck with your move / new job :-)

Let me know if you have any issues with it


Remove Link

I accidentally clicked the link every time it popped up and pissed me off

It's still a bad idea


Update README.md

Update Example attest

Thanks @FrimIdan! Can you sign off on the commit? Syft requires the DCO check to pass and commits to be signed. Thanks!

@spiffcs i've signed off the commit.

@FrimIdan I'm so sorry for being unclear here. Thank you for signing off on the commit. Anchore also requires that contributors sign commits as noted here. If you have trouble configuring this let me know and I can help you get set up. At the end, you should have a verified tag on the commit.


where to place config.yaml?

Describe the bug
Unclear from the documentation where custom configurations shall be placed
Following this config file example I am trying to build my own in particular to specify the directory path where navi  should look for cheatsheets. Moreover, here you address to navi --help for further details about config.yaml, however no specifications are provided by the help page either.
Where shall I place my config.yaml? More in general, I would expect to have something along the lines of ~/.config/navi to be the default path where the application looks for anything: at the moment, on my Mac, the default path is ~/Library/Application Support/navi/cheats/denisidoro__cheats.
Navi is awesome, thank you for the great work!

Ignore: I have updated Navi to the latest version and the --help provided the info I was after!


add support for EXCHANGE PARTITION

What problem does this PR solve?
Add support for EXCHANGE PARTITION. #412
What is changed and how it works?
MySQL syntax:
standalone_alter_commands:
    ......
    | EXCHANGE_SYM PARTITION_SYM ident
          WITH TABLE_SYM table_ident opt_with_validation
          {
            $$= NEW_PTN PT_alter_table_exchange_partition($3, $6, $7);
          }

Check List
Tests

Unit test

with_validation is actually a part of alter_commands_modifier.
And for EXCHANGE PARTITION, there is a specific opt_with_validation rule.
The current solution is to treat "WITH/WITHOUT VALIDATION" as a part of AlterTableSpec, which   is maybe not appropriate.
I think we can add a type ValidationType to identify validation type of both AlterTableStmt and AlterTableSpec(for EXCHANGE PARTITION).

I'm just worried about the validation rules hard coded in AlterTableSpec may cause ambiguity. we  might need to modify them one day we need alter_commands_modifier.

@leiysky I think the parser does already support alter_commands_modifier.
alter_commands_modifier: alter_algorithm_option
| alter_lock_option
| with_validation

All of its branches are implemented.

@kennytm @leoppro PTAL again.

LGTM


Initial PoC for supporting #[diesel(serialize_as)]

Opening this as a draft PR to get feedback, specifically:

How do we want to handle insertion of references? I think inserting by reference doesn't make a lot lot of sense if we are doing a conversion internally. However, this cannot be a runtime error. Would it be acceptable to just omit the impl<'insert> Insertable block if any of the fields have a serialize_as attribute?
How is the feature going to interact with #[diesel(embed)]? I feel also this combination doesn't make much sense. The combination of #[diesel(embed)] and #[diesel(serialize_as)] could be a compile-error.

I left some initial notes.

Thank you @weiznich !
How do we want to deal with references in general?
Currently, I am emitting this:
impl<'insert> Insertable<users::table> for InsertableUser {
    type Values = <(std::option::Option<diesel::dsl::Eq<users::name, UppercaseString>>, ) as Insertable<users::table>>::Values;
    fn values(self) -> Self::Values {
        (std::option::Option::Some(
            users::name.eq(UppercaseString::from(self.name)),
        ), )
            .values()
    }
}

impl<'insert> Insertable<users::table> for &'insert InsertableUser {
    type Values = <(std::option::Option<diesel::dsl::Eq<users::name, UppercaseString>>, ) as Insertable<users::table>>::Values;
    fn values(self) -> Self::Values {
        (std::option::Option::Some(
            users::name.eq(UppercaseString::from(&self.name)),
        ), )
            .values()
    }
}

Passing InsertableUser as a reference is pretty pointless now because we always allocate in from anyway. It also forces users to have a 2nd From impl for a reference.
Should I just omit the impl<'insert> block in case the struct is using serialize_as?

Should I just omit the impl<'insert> block in case the struct is using serialize_as?

I think that's an reasonable thing to do as long as it is documented probably.

Should I just omit the impl<'insert> block in case the struct is using serialize_as?

I think that's an reasonable thing to do as long as it is documented probably.

Cool thank you! I will roll with that then!
Is is also reasonable to say this doesn't work with #[diesel(embed)]?

@weiznich this is ready for another review!
I think I've covered most of the comments. Tests are hopefully passing now! The failing compiletests also fail on master currently.

This PR looks good now modulo that two small things. Good work +1

Thank you!

The failing compiletests also fail on master currently.

That should be fixed now. I somehow manged to merge two PR's that changed similar things and lead combined to a failing compile test see_no_evil

That sounds like a job for bors :)
From the documentation:

Bors is a GitHub bot that prevents merge skew / semantic merge conflicts, so when a developer checks out the main branch, they can expect all of the tests to pass out-of-the-box.

We use it extensively at work and it works really well!

I've just noticed that this probably should have a Changelog entry here. Otherwise it's ready for merging now. (Sorry for finding just another thing that needs to be changed)

That sounds like a job for bors :)

I'm well aware of bors, but I hesitated to setup and maintain that for diesel yet.

I've just noticed that this probably should have a Changelog entry here. Otherwise it's ready for merging now. (Sorry for finding just another thing that needs to be changed)

Done!


Error 404 on Webhook Verification with Flask Integration

Description
While integrating the pywa library with a Flask server, the webhook verification step results in a 404 error. The following message appears in the log:
"GET /webhook/?hub.mode=subscribe&hub.challenge=<challange>&hub.verify_token=<verify_token> HTTP/1.1" 404
Steps to Reproduce
Configure the pywa library with the necessary credentials and a callback URL using pagekite.me.
Start the Flask server.
Observe the 404 error in the server logs when WhatsApp attempts to verify the webhook.

Expected Behavior
The server should correctly handle the GET request for webhook verification, respond with the challenge code provided by WhatsApp, and handle incoming POST requests for messages.
Actual Behavior
The server returns a 404 error during the GET request for webhook verification.
Example Code
Here is a simplified version of the code used to set up the Flask server and pywa integration:
from pywa import WhatsApp
from pywa.types import Message
from flask import Flask

flask_app = Flask(__name__)

wa = WhatsApp(
    phone_id='your_phone_id',
    token='your_token',
    server=flask_app,
    callback_url='https://your_pagekit.mel/webhook/',
    verify_token='your_verify_token',
    app_id='your_app_id',
    app_secret='your_app_secret',
)

@wa.on_message()
async def message_handler(_: WhatsApp, msg: Message):
    print(msg)

if __name__ == '__main__':
    flask_app.run(port=8000, debug=True)

Analysis
The issue arises because the pywa library does not automatically handle the webhook verification step, and there is no clear guidance in the documentation on how to configure Flask to handle this correctly. The workaround involves manually adding a route in Flask to handle the GET request for verification.
Proposed Solution

Documentation Update: Include clear instructions and example code in the pywa documentation for handling webhook verification with Flask.

Library Enhancement: Consider adding a method or functionality within pywa to automatically handle the webhook verification process, simplifying integration with web frameworks like Flask.

Additional Context
pywa Version: 1.20.2
Flask Version: 3.0.3
Python Version: 3.10.12
Operating System: Linux Mint 21.1

This issue is critical for developers attempting to integrate pywa with Flask, as it hinders the initial setup and functionality testing.

Hi @julianobergamo, thanks for the report.
If you register the callback url to a different endpoint other than "/", you need to tell pywa to register the route for this endpoint. so in your code move the endpoint to webhook_endpoint:
wa = WhatsApp(
    phone_id='your_phone_id',
    token='your_token',
    server=flask_app,
    callback_url='https://your_pagekit.mel',
    webhook_endpoint='/webhook',
    verify_token='your_verify_token',
    app_id='your_app_id',
    app_secret='your_app_secret',
)

I will soon try to add a separate guide for flask and even for django and others

Hey david-lev,
Thanks a lot for helping me out with this issue. Your response really helped me!
Thanks again!


ValidationEngineBuilder constructor needs updating to latest HL7 Org Validator

Latest HL7 Org. Validator ValidationEngineBuilder constructor added a new argument (useEcosystem), see:
https://github.com/hapifhir/org.hl7.fhir.core/commit/60bf358bfa978b95b39041d8412f92ea401cefa9#diff-f66d101dc8bdff1a1ed6716055c5d666b7c8af5219184252968ed4a9fc7ddf24
So the call to this method without this new argument now failes (NoSuchMethod).
Need to update the call in:
https://github.com/dmitry-zasypkin/fhir-profile-validation/blob/main/jgw/src/isc/ateam/validation/JavaValidatorFacade.java#L35

Fix by:
https://github.com/dmitry-zasypkin/fhir-profile-validation/commit/c244e7bcfcc36d292ddfed8ed48137449f75b044
https://github.com/dmitry-zasypkin/fhir-profile-validation/commit/543bd5b037f5d149f5e1ee6571a87ba018843349


ManuallyDrop droped

Utilization of ManuallyDrop replaced by mem::forget due the new rust 1.48 recommendation, avoiding warnings.
More info about this rust change in f3024073f92b15d38b42241e65067f0ba796896c, on std lib rust.

Pull Request Test Coverage Report for Build 378743004

0 of 3 (100.0%)  changed or added relevant lines in 2 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-13.5%) to 65.923%

Files with Coverage Reduction
New Missed Lines
%

rsfbclient-rust/src/client.rs
1
80.23%

Totals

Change from base Build 344605194:
-13.5%

Covered Lines:
1979

Relevant Lines:
3002

💛  - Coveralls


Use js-wp-editor in modal

Hi guys, I would like to use the js-wp-editor function inside a modal window but it doesn't work properly. The first time I open the modal window the function works fine. But if I close the window and then re-open it the Visual editor doesn't work. You can see what I mean here: https://cl.ly/1W190d2O0W0A
Do you know what is going wrong and how to solve it? Thanks already for your help!

Hi @NickHeurter ,
have you found a solution for this, I'm having the same issue.
Thanks

Hi @DonElyes, no not yet. I've stopped using the js-wp-editor in this modal and used the default Wordpress editor in the modal. But if you find the solution, please let me know!


Add bottom-tab navigator

According the design sheets, the app must have an internal navigator, known as bottom-tab navigator.
The first route must be for Home screen, you can add another fake page to test the movement.

on it


Could not connect to localhost on Windows.

Steps to reproduce

Puppeteer version: 0.13.0
Platform / OS version: Windows 7 x64
Node.js version: v8.9.4

What steps will reproduce the problem?

Fork https://github.com/eggheadio-projects/puppeteer-testing
npm i
Run tests

I see Error: net::ERR_CONNECTION_REFUSED using await page.goto('http://localhost:3000/').
But test works great with await page.goto('https://www.google.ru/');.
What is the expected result?
Puppeteer able to connect to localhost.
What happens instead?
Error: net::ERR_CONNECTION_REFUSED.

Argh. My bad. There was a mistake with start app.
Puppeteer deals perfect with localhost.

@YozhEzhi What was your mistake?

@bsurai I don't remember.
Maybe port was 8080 or something like that.

For others: I was unable to get puppeteer to connect to localhost:3000 (net::ERR_ABORTED at localhost:3000 I think?).
Adding http:// before localhost:3000 in page.goto fixed the issue for me.

I am getting the same error, i use pupeeter 11.0.0 on windows and get the "Error: net::ERR_CONNECTION_REFUSED"
when using "await page.goto(http://<IP_ADDRESS>:3030/app.html);"

@travisk-codes thxxx, i was get the same problem


build: Dependency Correction on Cloud Transfer Sample

What this PR changes/adds
This PR adds dependencies on the "transfer/transfer-05-file-transfer-cloud/cloud-transfer-consumer/build.gradle.kts" and "transfer/transfer-05-file-transfer-cloud/transfer-file-cloud/build.gradle.kts"
Why it does that
This dependencies are added because there was an error when trying to run the jar file.
Linked Discussion

@ndr-brt could you take a look at the other part of my correction on the linked discussion. Should I open another PR for the docs correction I suggested, although it's incomplete?


Change address book file path

Fixes #209

Not very sure if there are more places in the codebase to replace.


Allow Polars DataFrame as argument for tgb.table()

Description
The tgb.table() function does not accept yet polars DataFrame, this can be bypassed by using df.to_pandas() but since polars is growing and slowly becoming a replacement for pandas is a lot of area, it would make sense for me to have a native integration in taipy for it.
Here is the code that should work with the new feature :
from taipy.gui import builder as tgb
from taipy.gui import Gui
import polars as pl

df = pl.DataFrame({"name": ["Héricendre", "Kaiminus", "Germignon"]})
with tgb.Page() as page:
    tgb.table(data="{df}")
Gui(
    page,
).run(debug=True, use_reloader=True, port=5111)

For now it does not show a table and raise the following warning :
WARNING:root:
--- 1 warning(s) were found for page '/'  ---
 - Warning 1: Can't find Data Accessor for type <class 'polars.dataframe.frame.DataFrame'>.
----------------------------------------------

Acceptance Criteria

[ ] Ensure new code is unit tested, and check code coverage is at least 90%.
[ ] Create related issue in taipy-doc for documentation and Release Notes.
[ ] Check if a new demo could be provided based on this, or if legacy demos could be benefit from it.
[ ] Ensure any change is well documented.

Code of Conduct

[X] I have checked the existing issues.
[ ] I am willing to work on this issue (optional)

Taipy Enterprise will integrate Polars in 4.0 in tables; this feature has already been treated. It will also integrate Polars in its data management capabilities (data nodes). See Release Notes


In the parser, explicitly declare return types using child class instead of parent

Consider changing the return types of the methods contained within the Parser class to instead use the exact returned class, rather than the parent 'Expression' class.

Note: may help with issue #8


Add prefix and suffix text support

This change adds support for the new prefix and suffix text support in GOV.UK frontend. It allows text to be added before and after text inputs in a grey box to give users more context around the data, for example stating the currency of a expected monetary value.

[ ] Wait for the next version of the Design System to be released and make sure everything looks ok

Fixes #159

Also worth noting that the docs currently look like this and will do until the guide can be built with the unreleased prefix/suffix styles.

Looks good!


Added helper function to extract tags from an AWS TagSet

This PR adds a generic helper function to extract the tags from an AWS TagSet into a map.
A TagSet is of the following form:
"TagSet": [
    {
      "Key": "Name",
      "Value": "my-vpc"
    },
    {
      "Key": "CreatedBy",
      "Value": "Terraform"
    },
    {
      "Key": "Env",
      "Value": "test"
    },
]

An example of using this to extract the tags set on a VPC:
vpcID := "vpc-a5b9c5e7"

req := client.DescribeVpcsRequest(&ec2.DescribeVpcsInput{
	VpcIds: []string{
		vpcID,
	},
})

resp, err := req.Send(context.Background())
if err != nil {
	logger.Logf(t, "Failed to lookup the VPC %q. Error: %s", vpcID, err.Error())
}
assert.Nil(t, err)

tags := aws.ExtractResourceTags(resp.Vpcs[0].Tags)

Agreed with @yorinasub17. We are always grateful for contributions, but as Terratest grows both in size and popularity, we have to be more and more picky with what we add to the library to ensure it doesn't become too bloated. I think looping over the tags directly in your own code is simple and readable enough (and probably more type safe) than this helper, so I'm going to close this one out. Thanks!

Thank you for reviewing my PR.
I agree with you that this function is somewhat complex and that the alternative is easier and more readable, however if a user wanted to check the tags of all created resources, looping over the tagset leads to a lot of code duplication . Nonetheless, I appreciate the feedback.


“events.js:72  throw er; // Unhandled 'error' event” during ‘particle setup’ with Photon

Note: I updated particle-cli and the firmware before running particle setup.
➜  ~  particle setup
                  _   _      _        _
 _ __   __ _ _ __| |_(_) ___| | ___  (_) ___
| '_ \ / _` | '__| __| |/ __| |/ _ \ | |/ _ \
| |_) | (_| | |  | |_| | (__| |  __/_| | (_) |
| .__/ \__,_|_|   \__|_|\___|_|\___(_)_|\___/
|_|                     https://particle.io/

> Setup is easy! Let's get started...
> It appears as though you are already logged in as<EMAIL_ADDRESS>? Would you like to log in with a different account? No

! PROTIP: Hold the MODE/SETUP button on your device until it blinks blue!
! PROTIP: Please make sure you are connected to the internet.

> I have detected a Photon connected via USB.
? Would you like to continue with this one? Yes
! The Photon supports secure Wi-Fi setup. We'll try that first.

! PROTIP: Wireless setup of Photons works like a wizard!
! PROTIP: We will automagically change the Wi-Fi network to which your computer is connected.
! PROTIP: You will lose your connection to the internet periodically.

> No nearby Photons detected. Try the `particle help` command for more information.
? Would you like to wait and monitor for Photons entering setup mode? Yes
> Monitoring nearby Wi-Fi networks for Photons. This may take up to a minute.
? Found "Photon-P9D5". Would you like to perform setup on this one now? Yes

! PROTIP: You will need to know the password for your Wi-Fi network (if any) to proceed.
! PROTIP: You can press ctrl + C to quit setup at any time.

events.js:72
        throw er; // Unhandled 'error' event
              ^
Error: spawn ENOENT
    at errnoException (child_process.js:980:11)
    at Process.ChildProcess._handle.onexit (child_process.js:771:34)

Note: the same error is mentioned here (in a different context) but I don't know what to make of the answer.
http://stackoverflow.com/questions/17749055/nodejs-throw-er-unhandled-error-event-events-js72-when-using-child-proc

What OS are you using?

OS X 10.9.5


Multiple dvc merge can cause ordering issues

If you use DVCParams.merge() multiple times it can mess up the order in which they were put in there.
consider having a dictionary or list in the PyTrack class instead of merge
self.dvc={key:DVCParams}

DVC merge is now longer used


📝 Fix Chinese typos

I found a typo when I was studying

📝 Docs preview for commit 9be4d852a011e2e141cb40e8ae24d5b9e474d984 at: https://cd4da136.fastapitiangolo.pages.dev


Allow users to link to a specific reef or area on the map

Accept additional URL parameters:
Map:

a specific reef
or a lat/lon/zoom trio

Reef and ReefPoint Pages:

pre-selected custom date period

This PR should be built on top of https://github.com/aqualinkorg/aqualink-app/pull/509 if it has not been merged

Let's prioritize the map changes.
Ideally, we want to be able to specify a reef in the URL as follows: /map?reef_id=1630&zoom=7.

The map would load centered above this reef at a zoom level of 7
The reef 1630 becomes the selected reef in the top right corner of the page

From a code perspective, this probably means setting the “reefOnMap” with the id you get to the URL.


Fix assets with incorrect routes

Using /public/* instead of /assets/*. Fixes issue #117

🎉 This change has been released in version 1.2.0


Cannot find module<EMAIL_ADDRESS>

When run via gitlab CI, with this job description:
lint:nglint:
  stage: test
  image: trion/ng-cli:${CLI_VERSION}
  script:
  - ng lint

this error occurs:
Running with gitlab-runner 13.8.0 (775dd39d)
  on code-fbi-h-da-de-runner-6b97bf74f7-lkkgn uS4iLib3
Resolving secrets
00:00
Preparing the "kubernetes" executor
00:00
Using Kubernetes namespace: ci
WARNING: Pulling GitLab Runner helper image from Docker Hub. Helper image is migrating to registry.gitlab.com, for more information see https://docs.gitlab.com/runner/configuration/advanced-configuration.html#migrating-helper-image-to-registrygitlabcom
Using Kubernetes executor with image trion/ng-cli:${CLI_VERSION} ...
Preparing environment
00:06
Waiting for pod ci/runner-us4ilib3-project-15457-concurrent-0rlw7t to be running, status is Pending
Waiting for pod ci/runner-us4ilib3-project-15457-concurrent-0rlw7t to be running, status is Pending
	ContainersNotReady: "containers with unready status: [build helper]"
	ContainersNotReady: "containers with unready status: [build helper]"
Running on runner-us4ilib3-project-15457-concurrent-0rlw7t via code-fbi-h-da-de-runner-6b97bf74f7-lkkgn...
Getting source from Git repository
00:01
Fetching changes with git depth set to 50...
Initialized empty Git repository in /builds/istlujime/angular-ci-cd-test/.git/
Created fresh repository.
Checking out 71a77e4f as master...
Skipping Git submodules setup
Downloading artifacts
00:00
Downloading artifacts for build (270294)...
Downloading artifacts from coordinator... ok        id=270294 responseStatus=200 OK token=dsXdsdkz
WARNING: dist/: lchown dist/: operation not permitted (suppressing repeats)
Executing "step_script" stage of the job script
00:01
$ ng lint
An unhandled exception occurred: Cannot find module<EMAIL_ADDRESS>Require stack:
- /usr/local/lib/node_modules/<EMAIL_ADDRESS>- /usr/local/lib/node_modules/<EMAIL_ADDRESS>-<EMAIL_ADDRESS>-<EMAIL_ADDRESS>- /usr/local/lib/node_modules/<EMAIL_ADDRESS>- /usr/local/lib/node_modules/<EMAIL_ADDRESS>-<EMAIL_ADDRESS>-<EMAIL_ADDRESS>-<EMAIL_ADDRESS>-<EMAIL_ADDRESS>- /usr/local/lib/node_modules/@angular/cli/bin/ng
See "/tmp/ng-eoBgYw/angular-errors.log" for further details.
Cleaning up file based variables
00:00
ERROR: Job failed: command terminated with exit code 1

The expected behavior would be that the job runs trough fine. Cannot find module<EMAIL_ADDRESS>does not occur in my local development environment, as these modules are present, both in node_modules and package.json.
My package.json:
{
  "name": "angular-ci-test",
  "version": "0.0.0",
  "scripts": {
    "ng": "ng",
    "start": "ng serve",
    "build": "ng build",
    "watch": "ng build --watch --configuration development",
    "test": "ng test",
    "lint": "ng lint"
  },
  "private": true,
  "dependencies": {
    "@angular/animations": "~12.0.3",
    "@angular/common": "~12.0.3",
    "@angular/compiler": "~12.0.3",
    "@angular/core": "~12.0.3",
    "@angular/forms": "~12.0.3",
    "@angular/platform-browser": "~12.0.3",
    "@angular/platform-browser-dynamic": "~12.0.3",
    "@angular/router": "~12.0.3",
    "rxjs": "~6.6.0",
    "tslib": "^2.1.0",
    "zone.js": "~0.11.4"
  },
  "devDependencies": {
    "@angular-devkit/build-angular": "^12.0.3",
    "@angular-devkit/core": "^12.0.3",
    "@angular-eslint/builder": "12.1.0",
    "@angular-eslint/eslint-plugin": "12.1.0",
    "@angular-eslint/eslint-plugin-template": "12.1.0",
    "@angular-eslint/schematics": "12.1.0",
    "@angular-eslint/template-parser": "12.1.0",
    "@angular/cli": "~12.0.3",
    "@angular/compiler-cli": "~12.0.3",
    "@types/jasmine": "~3.6.0",
    "@types/node": "^12.11.1",
    "@typescript-eslint/eslint-plugin": "4.23.0",
    "@typescript-eslint/parser": "4.23.0",
    "ajv": "^6.12.6",
    "angular-eslint": "0.0.1-alpha.0",
    "eslint": "^7.26.0",
    "jasmine-core": "~3.7.0",
    "karma": "~6.3.0",
    "karma-chrome-launcher": "~3.1.0",
    "karma-coverage": "~2.0.3",
    "karma-jasmine": "~4.0.0",
    "karma-jasmine-html-reporter": "^1.5.0",
    "typescript": "~4.2.3"
  }
}

Please use ‘npm ci‘ to install your project specific dependencies.

Did using npm i  resolve your issue ?

Hi @everflux,
I have the same exact problem and using npm ci did not solve the problem. I think the reason is that @angular-eslint/builder is not installed globally (see this post). I use the newest (12.1.1) version of trion/ng-cli.

Could you show your angular.json and build configuration / execution?

It happens during the lint stage.
.gitlab.ci
lint:
  stage: lint
  image: trion/ng-cli:12.1.1
  script:
    - npm run lint

package.json
"scripts": {
    "lint": "ng lint"
},
"devDependencies": {
    "@angular-builders/jest": "^9.0.1",
    "@angular-devkit/build-angular": "^12.1.1",
    "@angular-eslint/builder": "12.1.0",
    "@angular-eslint/eslint-plugin": "12.1.0",
    "@angular-eslint/eslint-plugin-template": "12.1.0",
    "@angular-eslint/schematics": "12.1.0",
    "@angular-eslint/template-parser": "12.1.0",
}

angular.json
"lint": {
    "builder": "@angular-eslint/builder:lint"
    "option" {...}
}

Do you need more information?
Thank you very much in advance!

Please try to add 'npm ci' to your lint stage before you run 'npm run lint'

Adding npm ci to the lint stage helped. There seems to be something wrong with my cache configuration - the cache is not properly loaded.
Sorry for wasting your time!

I am having same error but it is when I run nx g @nwrl/angular:app some-app


Add newline after release title

Before changelist produced
# scientific-python-hugo-theme 0.3
We're happy to announce the release of scientific-python-hugo-theme 0.3!

This PR produces
# scientific-python-hugo-theme 0.3

We're happy to announce the release of scientific-python-hugo-theme 0.3!

See https://github.com/scientific-python/scientific-python-hugo-theme/pull/343.

Thanks Jarrod. :)


Set initial resolution from 480x640

Hi, how can I set the initial texture resolution?  It's setting to 480x640 and this is causing the layout to be a mess with a notification over the top of my password pattern, so I am unable to unlock the phone.  I have tried -m -c -b parameters.
Thanks.

Initially, it mirrors at the device definition.
You can limit the size with -m.
For example, if your device is 1920×1080, you can mirror at 1600×900 with:
 scrcpy -m 1600

Thanks for the quick reply!  There is nothing I can do to get it to use more than 480x640 and it is showing a tiny area of my screen.  I managed to get logged in by connecting over network because then the "select USB function" notification was not over the password pattern.  After unlocking I then connected back with USB.
How can I change the device definition?

Is it a real device (or an emulator)?
The width and height are retrieved here.
What is the result of:
adb shell wm size

It's a real phone, Samsung Galaxy Note 3. Running adb shell wm size returns "Physical size: 640x480".  Can I override this somehow?  The phone has a 1080x1920 display.

OK, so the device does not report its size.
You an edit the source code to force your values here (or here if you just want to change the client).

Thanks, I issued this to set the size:
adb shell wm size 1080x1920
However after doing that the screen is jumbled.

Resolved with:
adb shell size 1280x720
adb shell density 320
scrcpy
For some reason if I try to set size 1920x1080 then check what's set, it won't take 1920 and instead sets 1280x960 which isn't 16:9.
Well something seems screwed up with adb not getting the correct device profile.  Thanks for the help!

Well something seems screwed up with adb not getting the correct device profile.

(it's not adb, it's in your device rom, adb just gives a shell to get access to the value)

ypu can incresae the max size by typing scrcpy -m 1950

You can increase the max size by typing scrcpy -m 1950

Ask me a question, the screen resolution of my cel is 2400 x 1980 and my monitor is QHD.
And I would like to know if I can leave my resolution in QHD too and how to do it if there is a way ?


UR e-series monitoring

Hi everyone, I'm not sure if this is the place to ask this question but I think it is. Using ur_modern_driver can you read the current or the torque of all the joints?. I know that ther are UR Script functions in order to do that, but my question is if there is already a topic that publish it when you launch ur_modern_driver ur10e_bringup.launch robot_ip:=[ROBOT_IP] or do I have to do it by myself writing my own ROS node?

We currently publishes joint motor currents in the effort field of the JointState messages (here).
This data comes from the controller over the real-time interface, which only includes the I_actual field. There is no M_actual field corresponding to M_target.
There is no analogue to the get_joint_torques() urscript function in that datastream afaict, so we cannot publish it.

Closing as it's not an actionable issue at the moment.

Thank you for your explanation!


Generalize row-key-decoding

This PR replace 2 with a general variable.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


FairLock is freezed after run more than 9000 times

int j =0;
public void Redis() throws InterruptedException {
    int i=0;
    while (i<100) {
        i++;
        Thread ts1 = new Thread(() -> {
            while (true) {
                RLock lock = redisson.getFairLock("frozen");
                try {
                    lock.lock(-1, null);
                    log.info(String.valueOf(j++));
                }finally {
                    if (lock.isHeldByCurrentThread()) {
                        lock.unlock();
                    } else {
                        log.error("获取了锁，但是却不能解锁？？");
                    }
                }
            }
        });
        ts1.start();
    }
    Thread.sleep(1000000000);
}

this is my code.the log will not print after a few minute.

update to 3.13.3 version


opt: no need spill x28

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (14b90fc) 61.80% compared to head (440d317) 61.19%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@               Coverage Diff                @@
##           test/aarch64     #596      +/-   ##
================================================
- Coverage         61.80%   61.19%   -0.62%
================================================
  Files                84       83       -1
  Lines             10345    10172     -173
================================================
- Hits               6394     6225     -169
+ Misses             3568     3565       -3
+ Partials            383      382       -1

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


plugin gh-pages can't work well

SBCL show

Failed to find the TRUENAME of /home/xxxx/blog/.curr

I changed
(defmethod deploy :after (staging)
  (let ((blog (truename (rel-path (deploy-dir *config*) ".curr"))))
    (delete-file (rel-path blog "index.html"))
    (cl-fad:copy-file (rel-path blog "1.html") (rel-path blog "index.html"))
    (with-open-file (out (rel-path blog "CNAME")
                     :direction :output
                     :if-exists :supersede
                     :if-does-not-exist :create)
      (format out "~A~%" *cname*))))

to
(defmethod deploy :after (staging)
  (let ((blog (truename (rel-path (deploy-dir *config*) ""))))
    (delete-file (rel-path blog "index.html"))
    (cl-fad:copy-file (rel-path blog "1.html") (rel-path blog "index.html"))
    (with-open-file (out (rel-path blog "CNAME")
                     :direction :output
                     :if-exists :supersede
                     :if-does-not-exist :create)
      (format out "~A~%" *cname*))))

It can work well now, I don't learn common lisp much and don't understand why. Maybe I do something wrong.
Sorry for my poor English :-(

@eriFelaP I got gh-pages warnings too. Would you like to have a look ? @redline6561
Thanks


Remove retired contacts-frontend

It's now in the gds-attic :)

dupe https://github.com/alphagov/govuk-developer-docs/pull/209/files

Sorry, didn't see this one!


Template instantiations have a parent symbol table based on the location of their first instantiation

Templates should track the symbol table in which they were defined and use that as the resolver current symbol table when resolving the template instantiation. Currently, the template instantiation uses the symbol table in which the template was first instantiated.
$ cat test.sunder
struct foo[[T]] {
    var x: T;

    func init() foo[[T]] {
        return (:foo[[T]]){.x = yikes};
        #                       ^^^^^
        #                       Yikes is not in scope! This should be an error.
    }
}

func main() void {
    var yikes = 0xABCDu16;
    var bar = foo[[u16]]::init();
    dump bar;
}
$ sunder-compile test.sunder
$ echo $?
0

Fixed by 0218fd20df18f3b2b5a9c615859ac3c0f622d5b7.


APIScan failure on DefaultAzureCredentialOptions several properties

Library name and version
Azure.Identity v1.11.3
Describe the bug
APIScan is not able to find documentation for below properties on DefaultAzureCredentialOptions
SharedTokenCacheTenantId, VisualStudioCodeTenantId, VisualStudioTenantId.
We have received several bug reports similar to below
Tool: ApiScan: Rule: documentationnotfound (Documentation Not Found). https://aka.ms/apiscan/learnmore?page=protocol%2fdocumentationnotfound.htm
azure.identity!Azure.Identity.DefaultAzureCredentialOptions.set_SharedTokenCacheTenantId is either an undocumented HVP API or the documentation is not discoverable.

We were not able to find related source for these properties in published documentation.
Expected behavior
We would except to find documentation publicly for these properties.
APIScan should be able to find the documentation.
Actual behavior
Tool: ApiScan: Rule: documentationnotfound (Documentation Not Found). https://aka.ms/apiscan/learnmore?page=protocol%2fdocumentationnotfound.htm
azure.identity!Azure.Identity.DefaultAzureCredentialOptions.set_SharedTokenCacheTenantId is either an undocumented HVP API or the documentation is not discoverable.

Reproduction Steps
There is no specific repro steps. You can try finding public documentation for those properties which leads to nowhere.
Environment
No response

hi @JRahnama -
These properties are intentionally hidden

We were able to replace those properties with TenantId. Thank you.


MSSQL support limit&offset without order by

Hi kayak:
I'm using tortoise-orm to access mssql
but when i use Model.get(xxx="xxx") to fetch a record, it will be got by filter with limit, like this:

the sql will be: "SELECT "xxx" FROM "IVR_CALL" WHERE "xxx"='xxx' OFFSET 0 ROWS FETCH NEXT 2 ROWS ONLY"
then goes error;
So i want to add "ORDER BY (SELECT NULL)" under MSSQLQueryBuilder->the _offset_sql() like this:

how do you think of it

I think it would be best if tortoise orm declared the order by separately so no magic happens behind the scenes. Would you mind raising an issue there? Thanks!

fine


Utvide organization BFF til  å støtte manglende felter for metadatakvalitet

Se design:
https://eggedosis.github.io/fdk-design/redirects/portal/report/metadataquality.html
Gjelder blant annet feltene:

[ ] Implement data for dataset catalog page
  # 1. request paginated datasets from fdk-fulltext-search
  # 2. extract url (entityUris) from datasets
  # 3. request assessments using entityUris from /assessments/entities?entityUris=a,b,c
  # 4. interweave assessments into existing fdk-fulltext-search response
  # 5. return fdk-fulltext-search response with assessments

[ ] Implement data for dataset page
      # 1. request dataset from fdk-fulltext-search
      # 2. extract url (entityUris) from dataset
      # 3. request assessment using entityUri from /assessments/entity?entityUri=a
      # 4. interweave assessment into existing fdk-fulltext-search response
      # 5. return fdk-fulltext-search response with assessment

Dette fungerer nå som forventet. Lukker saken.


Duplicate "Enter" toast

Clicking on the input results in multiple toasts as Chakra generates a unique id for each toast by default. Fixing this by passing an id to the toast.

I would like to claim this issue myself, thanks.

I would like to claim this issue myself, thanks.

Perfect, you're good to go!


[Question] How to import CSV data?

Hi guys,
as the mailing-list link on your page does not work, I use the issue tracker for asking, how to import CSV data?
Background:
I am using the Emotic EPOC+ EEG headset with 14 channels (which is far  away from any medical EEG - I know). Nevertheless I wonder if there is an option to import the recorded channels which are written in simple CSV format.
F3	F4	P7	FC6	F7	F8	T7	P8	FC5	AF4	T8	O2	O1	AF3
-149	-125	-128	-259	312	-138	-113	-146	-132	-133	-101	-152	-170	-225
Subsequent question: does mne-python work with 14 channels at all?
Thanks for answering, even it is the wrong channel.
Greetz paul

Checkout the pandas library and it's io tools. pd.read_csv. From there
you can then create epochs or raw objects as described here:
http://martinos.org/mne/stable/auto_tutorials/plot_creating_data_structures.html
And to your second questions. Not sure why it shouldn't. It depends of
course on what is your goal :)
On Mon, Sep 19, 2016 at 11:44 AM ppasler<EMAIL_ADDRESS>wrote:

Hi guys,
as the mailing-list link on your page does not work, I use the issue
tracker for asking, how to import CSV data?
Background:
I am using the Emotic EPOC+ https://www.emotiv.com/epoc/ EEG headset
with 14 channels (which is far away from any medical EEG - I know).
Nevertheless I wonder if there is an option to import the recorded channels
which are written in simple CSV format.
F3 F4 P7 FC6 F7 F8 T7 P8 FC5 AF4 T8 O2 O1 AF3
-149 -125 -128 -259 312 -138 -113 -146 -132 -133 -101 -152 -170 -225
Subsequent question: does mne-python work with 14 channels at all?
Thanks for answering, even it is the wrong channel.
Greetz paul
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/mne-tools/mne-python/issues/3601, or mute the thread
https://github.com/notifications/unsubscribe-auth/AB0fimgyh2Bfl7TxgM9VI2m0fdMpiAvWks5qrlmEgaJpZM4KATZR
.

Hi Dennis,
thanks for the fast reply.
I will check out the link you posted (I kinda missed that one...) and see if this fits my needs.
If you are interested:
My research project at university is about detecting driver drowsines with an EEG. I recorded some data in our driving simulator and assumed that the drowsiness increases over time (which the probands comfirmed). I then wrote a litte application and did the processing by myself (eventhough I am not very experinced with EEG and signal processing). The result was promising, but not as good as I aspected. So I want to try mne-python to see if I made some mistakes in my calculations and eventually can improve the detection rate.

Works like a charm, thanks!

Great and glad to hear! Your project sounds interesting, I hope mne-python
will be of help for you!
On Mon, Sep 19, 2016 at 4:32 PM ppasler<EMAIL_ADDRESS>wrote:

Works like a charm, thanks!
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/mne-tools/mne-python/issues/3601#issuecomment-248010274,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AB0fihA_sqBYOytWlCQe8qaTlXjk4fqVks5qrpz5gaJpZM4KATZR
.


Add some basic trig functions

Just lazily adding some trig functions I needed, more might come later.

Would you mind holding off on this until you have a concrete use case for it? I think it technically counts as a breaking change, in which case I think it should be merged with a change in version number, something I might do if it's decided to rename the library to al-jabr
However, if you need it for something that you can't push yet I'm happy to merge it anyway

Would you mind holding off on this until you have a concrete use case for it? I think it technically counts as a breaking change, in which case I think it should be merged with a change in version number, something I might do if it's decided to rename the library to al-jabr
However, if you need it for something that you can't push yet I'm happy to merge it anyway

Just using it for some experiments, which are blocked by https://github.com/rust-lang/rust/issues/60551 regardless, so no rush at all. :)

If you have any use cases for this I would be happy to merge this (and the use cases) now. I'm planning on re-releasing aljabar under the more correct name al-jabr, and reset the version. Thus no worries on breaking changes.

If you have any use cases for this I would be happy to merge this (and the use cases) now. I'm planning on re-releasing aljabar under the more correct name al-jabr, and reset the version. Thus no worries on breaking changes.

Sorry for the late response, @maplant, I've been away for a short while. I can add the full set of trig functions and inverses later today, do you want the hyperbolic versions included? Have you given any thought to what trait they belong under? They seem perfect where they are, the only wart I can see is impl<T> Real for Complex<T>, but I guess that's non-issue. Should the Real-trait implement the common https://doc.rust-lang.org/std/primitive.f64.html#methods and https://doc.rust-lang.org/std/primitive.f32.html#methods interface?


Ephemeral disk exhaustion leads to eviction loop

When the ephemeral disks used by an EKS node group are exhausted, an eviction loop which includes the Sematic Servers starts, with pod statuses such as "message": "Pod The node had condition: [DiskPressure]. ",, or "message": "The node was low on resource: ephemeral-storage. ",
[...]
sematic-server-ee-584694978d-ntmlg                             0/1     Error                    0             56m
sematic-server-ee-584694978d-ntsv6                             0/1     Evicted                  0             7m9s
sematic-server-ee-584694978d-p9qtp                             0/1     Evicted                  0             7m12s
sematic-server-ee-584694978d-qww7t                             0/1     Evicted                  0             7m8s
sematic-server-ee-584694978d-r2llx                             0/1     ContainerCreating        0             57s
sematic-server-ee-584694978d-rtf22                             0/1     Evicted                  0             7m11s
sematic-server-ee-584694978d-rvq27                             0/1     Evicted                  0             7m12s
sematic-server-ee-584694978d-rxmmf                             0/1     Error                    0             64m
sematic-server-ee-584694978d-tsch7                             0/1     ContainerStatusUnknown   0             23m
sematic-server-ee-584694978d-tzkfq                             0/1     Error                    0             71m
sematic-server-ee-584694978d-v988v                             0/1     Evicted                  0             7m10s
sematic-server-ee-584694978d-vrbp8                             0/1     Evicted                  0             7m9s
sematic-server-ee-584694978d-w94nl                             0/1     Error                    1             27m
sematic-server-ee-584694978d-wc7f8                             0/1     Evicted                  0             7m11s
sematic-server-ee-584694978d-wv6tm                             0/1     Evicted                  0             7m10s
sematic-server-ee-584694978d-x4vqn                             0/1     Evicted                  0             7m8s
sematic-server-ee-584694978d-xc8wc                             0/1     Error                    0             90m
sematic-server-ee-584694978d-xf5np                             0/1     Evicted                  0             7m8s
sematic-server-ee-584694978d-xw2r2                             0/1     Evicted                  0             7m10s
sematic-server-ee-584694978d-z2svw                             0/1     ContainerStatusUnknown   0             28m
sematic-server-ee-584694978d-z6dgl                             0/1     Evicted                  0             7m10s
sematic-server-ee-cleaner-28084805-4pjmn                       0/1     ImagePullBackOff         0             4m30s
sematic-server-ee-cleaner-28084805-9r4hx                       0/1     ContainerStatusUnknown   0             6m49s
sematic-server-ee-cleaner-28084805-vzhtl                       0/1     ContainerStatusUnknown   0             7m55s
sematic-server-ee-socketio-64c49d4557-b7g24                    1/1     Running                  0             87m
sematic-server-ee-socketio-64c49d4557-xcjmr                    0/1     Error                    0             90m
sematic-worker-2f00879bea3e40a8b59ccbae731cec0d-65c88d-f94h9   1/1     Running                  0             8m38s

Mitigation ideas:

Increase node disk size (does not scale)
Isolate Servers on their own node group, so they are not impacted
Explore introducing protection against scheduling pods whose image sizes would exceed the ephemeral disk space available


Fix MarkupTest's missing array code convention

Summary
Found the missing code convention of the array in the MarkupTest code.
So, add it to this.
Checklist

[x] Added tests
[x] Ran the ./bin/test script and it succeeded

@franklinsch
I reviewed all the files to find and add two parts that did not fit the array convention.
I hope it will be reflected for the perfection of the code.

@swift-ci please test


Update definition-structure.md

@ljtill : Thanks for your contribution! The author(s) have been notified to review your proposed change.

LGTM! Thanks for catching and fixing this, @ljtill!
#sign-off


Auto generate link for emphasized headings (TOC of README broken)

How can the issue be reproduced
Try to click "Auto linking for clickable TOC" in the README.md of commit afe002a.
What was expected
To jump to #auto-linking-for-clickable-toc
What actually occurred
#auto-linking-for-_clickable_-toc was added to the url and nothing jumped.
What was the version of the involved component
afe002a

@mpvharmelen
Hi, I'm so sorry for late reply 😭
And thanks for your report, the link was dead certainly.
I will fix it ASAP.

@mpvharmelen
I found the 'markdown_preview' attribute was missing on README.


can not find ./config/config_datasets_preprocessing.yaml

Dear author,
I can not find "config_datasets_preprocessing.yaml" in ./config to test my own data, could you please answer this small question? Thank you very much.

Sorry for the wrong instructions.
Corrected it in commit bde890fb3045198da83feb061f62b96599b81c57
Best,
Julian

Sorry for the wrong instructions. Corrected it in commit bde890f
Best, Julian

Ok, thank you very much.
Best,
Shenhai


README, add reason to consider adopting re-frame

Thanks, I like it.

This is like one of the fastest merges I've gotten for a simple doc edit!
:)
On 15/12/2016 3:32 AM, "Mike Thompson"<EMAIL_ADDRESS>wrote:

Merged #287 https://github.com/Day8/re-frame/pull/287.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Day8/re-frame/pull/287#event-894933009, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AAg8qLeJpMXgv5RriNkQ_rmYZdqpPZTRks5rIQkkgaJpZM4LN03m
.

Nice expansion! Didn't know re-frame and elm have such a history


Rushikesh.sapate/prgb 10/function support for extract2

Function support for EXTRACT2 which is exactly similar to CAST(EXTRACT as FLOAT64)

L1 Done.