Be more compliant with RFC7280 regarding Content-Length

Addresses #1163 by removing the Content-Length from the request when there is no body. There are still some concerns with this fix. The body_size method can be expensive. Also when considering the line of the RFC this addresses:

A user agent SHOULD NOT send a Content-Length header field when the request message does not contain a payload body and the method semantics do not anticipate such a body.

This PR only addresses the first part and does not consider the portion that states "and the method semantics do not anticipate such a body".

and the method semantics do not anticipate such a body.

I still think you should sent the header in POST requests. That's why I checked for method eq GET in my PR.

I think what's good enough for curl is good enough for us, curl -v -X POST http://mojolicious.org does not send a Content-Length: 0 header.

@kraih directly above the cited text we can read:
A user agent SHOULD send a Content-Length in a request message when
no Transfer-Encoding is sent and the request method defines a meaning
for an enclosed payload body. For example, a Content-Length header
field is normally sent in a POST request even when the value is 0
(indicating an empty payload body).

I think this makes it very clear that they propose to send it.

@forki And that's merely a SHOULD, that other very popular HTTP implementations don't follow.

I know. but the other one is also a SHOULD NOT. This is a weird discussion tbh. The RFC proposes how a client should behave. Of course you can do whatever you want. It's your project. But these suggestions actually make sense IMHO. In the GET case you can't have a body, so sending a Content-Length makes no sense. In POST, the method semantics usually use the body, so sending the Content-Length is useful. Even sending Content-Length 0, because then server may or may not shortcut things because it does not need to look at the body stream.
I mean our problem is solved in both cases. If this PR gets through then we will be very happy.

@kraih: What about...
$ curl -v --data-raw "" http://localhost:3000
...
> Content-Length: 0

It doesn't look like we will be able to reach consensus on this, perhaps we should leave everything the way it was for now. At least until a strong enough argument for one of the proposed changes emerges.

I'm pro getting rid of Content-Length: 0 for GET.

there are 2 strong arguments:

spec is very clear on what it favors
it increases compat with a server technology

@jhthorsen And about...
$ curl -v -X GET --data-raw "" http://mojolicious.org
...
> Content-Length: 0

Wouldn't removing Content-Length for GET requests only make this change less intrusive in addition to following the spec recommendations more closely? I don't see why this is even a discussion 😄

@kraih
$ curl -v -X GET --data-raw "" http://mojolicious.org

curl is actually smart here and sends a POST in that case. So it's consistent with what we discussed.

@forki Not with curl 7.54.0 or 7.57.0.

@khellang Then we also have to ask the question what the real goal here is. Why would we pick a single HTTP method and add a workaround specifically for it, instead of compiling a list of all affected HTTP methods to get actual spec compliance?

@kraih yes wrong code in clipboard. I confused myself and removed the comment.
@kraih and I think I now leave the discussion. I tried to help with making your lib support yet another server and keep it minimal invasive. You closed WIP PR over syntax issues. Fine. It's your project.
This was my first excursion into perl ecosystem. Can't say it was a nice trip. I'm really disappointed right now.
Well, on the other hand kestrel is clearly violating spec and I hope they have more interest in getting things fixed. So bye.

Why would we pick a single HTTP method and add a workaround specifically for it, instead of compiling a list of all affected HTTP methods to get actual spec compliance?

Depends how pragmatic you want to be. Either way, you'd be better off than the status quo :smile:
Checking for GET would probably get you 90% there. As RFC 7231 states:

GET is the primary mechanism of information retrieval and the focus of almost all performance optimizations.

If you want to compile a list of every single HTTP method where "the method semantics do not anticipate [...] a body" that would be even better, but will probably require more work for little value.

@forki Spec compliance is hard, that has nothing to do with the Perl ecosystem. I closed your WIP PR because you were just throwing stuff at the wall to see what sticks, instead of participating in a discussion first to figure out how to solve the problem correctly. It just wasn't very productive, and neither are your snide remarks about those trying hard here to find a solution.

@khellang You really don't think it would be very odd to generate different requests for GET and HEAD by default?

You really don't think it would be very odd to generate different requests for GET and HEAD by default?

Yes, I think it would be weird. But I think it's equally weird to send Content-Length: 0 for GET, HEAD and OPTIONS requests, or omit it for POST, PUT and PATCH.

I think we can conclude that the proposed solution here is definitely incorrect and not what we want to do.

But, we also do not want to treat GET and HEAD requests differently. So we have no alternative to move ahead with either.

I kept being troubled by the quote

and the method semantics do not anticipate such a body

I wanted to understand what it meant and see if I could find any such definitions. After much reading of specs, I'm going to lay out some findings.
The closest thing I can find to support for removing the Content-Length header comes from RFC 7230 3.3

The presence of a message body in a request is signaled by a
Content-Length or Transfer-Encoding header field.  Request message
framing is independent of method semantics, even if the method does
not define any use for a message body.

However this would therefore also include any method that does not include a body, including say POST which this thread has already shown people prefer to have. Note that this section replaced the obsoleted RFC 2616 4.3 states that

A server SHOULD
read and forward a message-body on any request; if the request method
does not include defined semantics for an entity-body, then the
message-body SHOULD be ignored when handling the request.

which would have given stronger credence to removing the Content-Length for any such request, however as it was removed in the new version, I think that is an indication that any method that may have a body is allowed to.
Later when considering how new methods might be added in the future RFC 7231 8.1.2 states (referring back to the previous section)

Since message parsing (Section 3.3 of [RFC7230]) needs to be
independent of method semantics (aside from responses to HEAD),
definitions of new methods cannot change the parsing algorithm or
prohibit the presence of a message body on either the request or the
response message.  Definitions of new methods can specify that only a
zero-length message body is allowed by requiring a Content-Length
header field with a value of "0".

This states outright that method parsing ought to be independent of the method semantics. It further states that new methods that require empty bodies must require the Content-Length header and that the value be 0. This indicates for me that this behavior of setting such a header is encouraged.
I still wanted to understand what "do not anticipate such a body" meant though so I continued looking. The methods themselves are defined in RFC 7231 4.3. The definitions of GET and HEAD include the statement

A payload within a GET request message has no defined semantics;
sending a payload body on a GET request might cause some existing
implementations to reject the request.

This is a far cry from saying that the body is not anticipated, just that it has no meaning. Indeed servers like Elasticsearch allow such requests. When thinking about it, this usage makes sense. If you'd like to submit a safe request (as defined by RFC 7231 4.2.1), as one wants when simply searching, GET is the clear choice. The fact that any such body has no defined meaning does not imply that it is without use, clearly it is for being safe, and additionally it is not required that the server ignore it. No this cannot be the meaning of "not anticipated".
So the question is then, what is? There is only one request method with prohibited from having a body, TRACE. As seen in RFC 7231 4.3.8.

A client MUST NOT send a message body in a TRACE request.

This is the kind of language I would expect to see when stating that a body is not anticipated. I can see why a useragent SHOULD NOT send a Content-Length in such an event, because it certainly would not be anticipated. The contrast of that definition with the definition of GET is clear.
With all that stated I can say that there are two remaining implications for Mojo.
First, if we feel it is worthwhile, Mojolicious should remove Content-Length from TRACE requests and forbid them to have message bodies.
Second, we may consider removing Content-Length headers for messages that do not have bodies, however, I see no reason to treat GET and HEAD (and OPTIONS incidentally) and differently than POST, PUT, and DELETE in that regard. After all, as quoted from RFC 7230 3.3 before

Request message
framing is independent of method semantics, even if the method does
not define any use for a message body.

In regards to interpersonal dynamics at play so far in this PR and the previous one, let me say, @forki while we appreciate your bringing this to our attention and we do hope that your problem is resolved, whether here or on the server side, Pull Requests are not intended as workspaces for running code. If you must work that way, feel free to enable Travis on your clone. There are many people that subscribe to our repository and giving them all notifications so that you may experiment is not fair to them.
Additionally, it is not encouraging for us who work on Mojolicious to hear that you will not give us the courtesy of installing the interpreter and cloning the repo to test (we have worked hard on making the dependencies effectively nil) while you expect us to give you our time and effort to consider your changes. Certainly you can see an asymmetry there?
Now I have put in hours of reading RFCs and writing code to evaluate this problem and attempt a sane mitigation. I see that Kestral is on the matter too. I hope that any future correspondence from your shows an equal amount of effort on your part or at least appreciation for those of us who have. Specifically any future references to what the RFC "clearly state" should come with clear quotations that refute my above.
This response is not intended in any way as a personal attack. Indeed quite the opposite. This is stating that this situation is taken very seriously and is being dealt with as professionally as possible. I would hope you would treat us as such too.
I'm going to leave this PR open for a little while to encourage discussion of my above points, however, without clear RFC evidence or very strong indication that the "living standard" of HTTP is markedly and decidedly different, and widely implemented as such, I don't see this PR being merged.

Just Googling "content-length zero" makes my head spin 😖
There's a myriad of issues, StackOverflow posts and forum threads for browsers, client libraries and servers.
Some servers expect Content-Length: 0, some reject it. Some clients omit it (for some or all HTTP methods), some include it.
There's tons of references to different RFCs, both old and new. Some use strong language, some are very vague.
I don't think we'll be able to reach consensus here. Let's hope MS will merge the fix on the server-side 🤞

sorry @jberger that's just not true. I cloned the repo and tried to run it locally. It did not work for me, but you are right I could probably just spent more time to get it work eventually. But I still tried to find a way to participate and was actively working on a solution to the problem. Regarding assymetry: I spent a lot of time to look through the code base to pick up your style. I was learning. /shrug
I don't even use your project myself.
I actually sent WIP PPs to both sides. Server and Client in order to bring you guys together and find a way to resolve the incompat.
This is the first project that I'm trying to work on that did not appreciate WIP PRs. If you look an my github profile then you will see that I'm working on a couple of projects. I guess that policy is OK but it's new to me.

@forki I'm sorry to hear that it didn't work for you. If you were having trouble, you could have asked us, we want to make things easy.
As to the "not appreciate WIP PRs": actually, yes our project is more stringent than most, a PR is usually assumed to be in complete state unless explicitly said so. That being said, we closed the PR on the 17th commit, and one that would (a) not have compiled and (b) would have broken GET requests with bodies; we didn't know how to take such a request and so we closed it.
Your reply to the effect of asking us to reopen the PR so you could continue to use the CI was our first indication that you weren't running the tests locally (or at least the first time I realized it). I have similarly been involved in open source for a long time and have many project involvements and it was the first time I had seen that.
Look I'm not trying to be unreasonable, we would have been happy to help get your perl and mojo installed. We still would though now that I've done this research I don't see it helping your chances of getting a fix included. Code is no longer the issue, RFC compliance is.

@khellang yeah, I've been doing those searches too. Amazing isn't it? The problem here of course is that if we change something, soon the next person will have some other, likely incompatible edge case. Say a server that requires Content-Length on all posts. We would have to justify this change for why we broken their application and without RFCs I couldn't do so. Seeing as Kestral appears to be making moves to correct the (actually) clear non-compliance on their end, I think everyone can walk away satisfied.

Since I'm trying to find a positive outcome here. Just a suggestion: maybe
add a contribution guideline. I actually looked for one but didn't find one
immediately.
So instead of just complaining about an issue I tried to get my hands dirty
and investigate myself. And as you said: code was never the issue.
Am 15.12.2017 17:56 schrieb "Joel Berger"<EMAIL_ADDRESS>
@forki https://github.com/forki I'm sorry to hear that it didn't work
for you. If you were having trouble, you could have asked us, we want to
make things easy.
As to the "not appreciate WIP PRs": actually, yes our project is more
stringent than most, a PR is usually assumed to be in complete state unless
explicitly said so. That being said, we closed the PR on the 17th commit,
and one that would (a) not have compiled and (b) would have broken GET
requests with bodies; we didn't know how to take such a request and so we
closed it.
Your reply to the effect of asking us to reopen the PR so you could
continue to use the CI was our first indication that you weren't running
the tests locally (or at least the first time I realized it). I have
similarly been involved in open source for a long time and have many
project involvements and it was the first time I had seen that.
Look I'm not trying to be unreasonable, we would have been happy to help
get your perl and mojo installed. We still would though now that I've done
this research I don't see it helping your chances of getting a fix
included. Code is no longer the issue, RFC compliance is.
@khellang https://github.com/khellang yeah, I've been doing those
searches too. Amazing isn't it? The problem here of course is that if we
change something, soon the next person will have some other, likely
incompatible edge case. Say a server that requires Content-Length on all
posts. We would have to justify this change for why we broken their
application and without RFCs I couldn't do so. Seeing as Kestral appears to
be making moves to correct the (actually) clear non-compliance on their
end, I think everyone can walk away satisfied.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/kraih/mojo/pull/1166#issuecomment-352056355, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AADgNFsVu2v6qnH3byxBXckrC_xGwQGBks5tAqTZgaJpZM4RCJHv
.

Like perhaps in the Contributing Guide which is linked to by Github in a yellow box when opening a pull request? I'm sorry if you didn't see it, I'm not sure how much clearer it could be. If this conversation doesn't come back to technical merits quickly I think it might be about time to wrap it up.

Yes. Did not see it. Looked into readme. But thanks.
Am 15.12.2017 18:41 schrieb "Joel Berger"<EMAIL_ADDRESS>
Like perhaps in the Contributing Guide
http://mojolicious.org/perldoc/Mojolicious/Guides/Contributing#CONTRIBUTING-CODE
which is linked to by Github in a yellow box when opening a pull request?
I'm sorry if you didn't see it, I'm not sure how much clearer it could be.
If this conversation doesn't come back to technical merits quickly I think
it might be about time to wrap it up.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/kraih/mojo/pull/1166#issuecomment-352067061, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AADgNOnNmvHzALT1JKgq5AW6qk43l-l9ks5tAq86gaJpZM4RCJHv
.

With no technical rebuttal forthcoming, I'm going to close this at least for now


Add country codes for Puerto Rico

What has been done

Added country codes for Puerto Rico #111

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Add comments on RoPE initialization

Does this explain the difference between HF and vLLM? I.e. if you enable the same CPU initialization with greedy sampling, so we get same outputs?

@casper-hansen Not always. Because floating-point arithmetics is not associative, different kernel implementations might lead to different outputs. The difference is more significant when reduction operation is involved. Therefore, our custom CUDA kernels for attention and RMS normalization do not produce exactly the same outputs as the original HF implementation, and thus outputs of vLLM models can be different from the HF models. We've checked that the outputs usually match when using FP32 and greedy sampling, but there are some cases where the outputs do not match. However, please note that this does not hurt task accuracy, as vLLM's implementation is mathematically equivalent to HF's.


Error occured. Invalid payload provided. No JSON object could be decoded.

Please help me to fix this issue in the webhook.php

Hi @adam-gweleg can you provide some more context on your issue? Are you following the instructions in examples/Webhook/README.md? I've updated it to mention the shared secret you must copy/paste into the example code.

I’m closing this issue because it has been inactive for a while now. However, please feel free to reopen it if you still encounter this issue with the latest stable version, or please submit a PR if you've managed to fix it. Thanks!

help please I have the same problem in node js and I can't solve it

`app.post('/pay/crypto/charge', rate, async (req, res) => {
const webhookSecret = 'my-secret';
var event;
console.log(req.headers);

try {
	event = Webhook.verifyEventBody(
		req.rawBody,
		req.headers['x-cc-webhook-signature'],
		webhookSecret
	);
console.log(event);
} catch (error) {
	console.log('Error occured', error.message);

	return res.status(400).send('Webhook Error:' + error.message);
}

})`


When copy-pasted payment proof details assets have name Beam

There are transactions with assets
(sent, sent offline, sent max privacy)
When we copy-pasted payment proof detail to notepad - assets have name "Beam"
Steps:

Find the completed transaction and expand it to see the transaction's details
Click "Payment proof" and copy transaction code
Open the Payment proof verification window and paste transaction code
Click buttons "copy details"
Paste the copied information into notepad

Actually:
The assets have name "Beam"
Expected:
The name of the assets matches in "Payment proof" and Notepad

Checked


Library not found

Please integrate .xcworkspace so dynamic librarys also loaded ..
e.g. pods .. since there are nice pods out there

infer -- xcodebuild -workspace AppName.xcworkspace -scheme AppName  -configuration Debug -sdk iphonesimulator

I use xcodebuild -workspace AppName.xcworkspace -scheme AppName -configuration Debug -sdk iphonesimulator returns ** BUILD SUCCEEDED **
But infer -- xcodebuild -workspace AppName.xcworkspace -scheme AppName -configuration Debug -sdk iphonesimulator returns
/Users/louise/workspace/ios/Pods/pop/pop/POPVector.h:13:10: fatal error: 'iostream' file not found
#include <iostream>
         ^
** BUILD FAILED **

The following build commands failed:
	CompileC /Users/louise/Library/Developer/Xcode/DerivedData/App-gqzswdekjpuqdcdldgboeqbvbprd/Build/Intermediates/Pods.build/Debug-iphoneos/Pods-pop.build/Objects-normal/armv7/POPAnimatableProperty.o pop/pop/POPAnimatableProperty.mm normal armv7 objective-c++ com.apple.compilers.llvm.clang.1_0.compiler
(1 failure)

I tried out simple example of Infer;
infer -- clang -c Hello.m
and got this error.
Hello.m:1:9: fatal error: 'Foundation/Foundation.h' file not found
#import <Foundation/Foundation.h>
        ^
1 error generated.

@Elmundo That example is meant to be executed on a Mac with Xcode installed (with the command line tools) and the iOS SDK.

i got a Mac which has Xcode installed with its command line tools and iOS SDK also.
Bariss-MacBook-Air:examples barisyilmaz$ xcode-select -p
/Applications/Xcode.app/Contents/Developer
Bariss-MacBook-Air:examples barisyilmaz$ gcc
clang: error: no input files
Bariss-MacBook-Air:examples barisyilmaz$

Still got this error.

@Elmundo what happens if you just run clang -c Hello.m does it work?

@martinoluca Just warnings, no error.
$ clang -c Hello.m
Hello.m:4:1: warning: no 'assign', 'retain', or 'copy' attribute is specified - 'assign' is assumed
      [-Wobjc-property-no-attribute]
@property NSString* s;
^
Hello.m:4:1: warning: default property attribute 'assign' not appropriate for non-GC object
      [-Wobjc-property-no-attribute]
2 warnings generated.

Hey @Elmundo, @martinoluca, I had the same issue...
Reinstalling command line tools has fixed it for me.
I'm running OSX 10.11 (15A178w), and command line tools for Xcode 6.3.2.
Xcode 7 beta command line tools break this in a completely different way for me.


Using perfetto from android app

Hi,
I'm trying to integrate perfetto in an app so that I can capture system trace from the app only, just like system tracing in developer options.
Looked into the source code of Traceur and found things like persist.traced.enable. However, no success yet.
Could you please tell if it's feasible or not? If yes, please suggest the best way to do it.

This request seems extremely generic.
If your question is "can I do this" in general the answer is "yes" but YMMV depending on the details.
Note that you will never be able to start/stop/capture the trace from a standard app.
Traces contains extremely sensitive contents and apps are not allowed to access the trace controlling endpoint, to prevent they can tell which other apps are running in the system.
In this sense Traceur is special. Traceur is a system app (i.e. is built as part of the system package). As such it gets assigned a dedicated selinux domain traceur_app.te.
That domain is allowed to control tracing because of this (still from taceur_app.te)
# Allow to access traced's privileged consumer socket.
unix_socket_connect(traceur_app, traced_consumer, traced)

Any other apps is not and will never be able to collect tracing data.
What's your use-case here? What's the problem you are trying to solve?

Being from an OEM development team, I can make system app. That won't be a problem. In fact, I already tried it, but it cannot find system property and gives me this:
Access denied finding property "persist.traced.enable"
I'm trying to use the same code as Traceur but in a different system app. Looks like normal system apps don't have rights to control tracing.

As I explained above, that's governed by selinux. This is what makes the difference between a "normal" system app and Traceur.

Thanks a lot. It was really helpful. :)

Cool!
Beyond the technical aspect, I'd love to understand more about your use case. Always curious to hear how OEMs use perfetto, what works and what not.
Mind dropping me a mail<EMAIL_ADDRESS>? We can take it from there.
Thanks!

Seems like a solution was found offline so closing now.


feat: Added alpha parameter to RidgeRegression

Closes #164 .
Summary of Changes
Added alpha parameter to RidgeRegression for regularization.

🦙 MegaLinter status: ❌ ERROR

Descriptor
Linter
Files
Fixed
Errors
Elapsed time

✅ PYTHON
black
2
2
0
0.58s

✅ PYTHON
mypy
2

0
1.6s

❌ PYTHON
ruff
2
2
8
0.02s

✅ REPOSITORY
git_diff
yes

no
0.01s

See detailed report in MegaLinter reports
Set VALIDATE_ALL_CODEBASE: true in mega-linter.yml to validate all sources, not only the diff
MegaLinter is graciously provided by

https://github.com/Safe-DS/Stdlib/pull/232 is now merged, please use this as the baseline and adjust your PR accordingly.

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

v0.12.0
GitHub release

Your semantic-release bot :package::rocket:


State reset upon input change

Fixes #21
There are a couple formatting changes in here (axis_arr_in --> msg_in; from typing import X, Y --> import typing) that aren't really related to the main topic.
The main topic is that this PR enables nodes to detect changes to inputs that invalidate state variables then recalculate state variables. The nodes can then live on in the graph despite their inputs changing. This has the downside that there are now more checks happening on EVERY message, when input changes should ideally be infrequent.
I kind of need this for a particular workflow -- ezmsg is running as a service and the pipeline source is an LSLInlet node that might change the stream it is pulling from. Resetting the whole pipeline is proving to be very slow.
I'm considering adding another argument to each of these nodes like detect_change: bool = False and simply not doing the reset check if that is False. (I'm open to other name suggestions. input_changeable, adapt_input, etc.). But before I do that, I want to profile this PR so far, compared to upstream, and see how much of a performance penalty these comparisons really are.

I created some small perf tests. Here are the best times of 5 runs for each, under dev and this state_reset branch:

generator
dev
this branch
slowdown (pcnt)

scaler_np
1.895
1.914
0.94

downsample
0.802
0.814
1.49

slicer
0.972
1.021
5.04

sampler
1.591
1.603
0.75

ranged_aggregate
1.412
1.399
-0.93

spectrum
1.258
1.263
0.40

windowing
1.012
1.061
4.84

filterbank
??
??
< 0.93

cwt
2.683
2.708
0.93 - fbank

I arbitrarily modified the size of my messages so each run through the data would take about 1-2 seconds.
windowing was highly variable run to run
cwt includes filterbank so I didn't bother testing filterbank
ranged_aggregate was actually sped up a bit despite the new input checks. I must have made some unrelated small changes that were faster than the predecessor.

I'm not super concerned about windowing because it's quite fast generally which was reflected in the fact that the numbers were highly variable. Even if the ~5% slowdown is true, it's going from "extremely fast" to "almost just as extremely fast".
I'll take another look at slicer...

IMO, the penalty of doing this checks is acceptable so I don't think it's necessary to make them optional.
This is ready for review.


Fixed the MarkDown

Hey @GreenJoey I am so sorry. I don't know, how I missed this commit.
Thanks for contributing to AlgoWiki.
Apologies for the delayed review.
Thank you!


Chatdown examples

The hyperlink for chatdown example files is dead. Does there still exists examples?

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: fa3a3039-9b21-fcb9-3fd7-4ae8c69bce93
Version Independent ID: 36ceb86e-828a-03d7-3245-44ab6ea4c09a
Content: Test and debug bots using the Bot Framework Emulator - Bot Service
Content Source: articles/bot-service-debug-emulator.md
Product: bot-framework
GitHub Login: @DeniseMak
Microsoft Alias: v-demak

It is now here. We'll update the link.


DPL-964 Research - Ability to track sample volumes and enable top-up management

User story
As a long-read user (James W.) I want to be able to track sample volumes so that the customers can be effectively view and manage sample re-reruns (top-ups) without manual intervention.
An unknown number of ToL samples require additional sequencing - estimated 25-50%. Currently, the process requires an email from the customer to SeqOps to confirm if any sample remains. Then the lab team manually checks the volume remaining. By tracking the volume in Traction and having the data accessible through MLWH, the customers will be able to see without manual intervention if a sample can be re-run. It also opens the way to automating the process on their end as well. Around 500 samples are estimated to need a top-up of data. This is a very significant amount of manual work that we would like to prevent in the future
Who are the primary contacts for this story
James W.
Who is the nominated tester for UAT
James W.
Acceptance criteria
To be considered successful the solution must allow:

[ ] System allows/detects to track estimated Library volumes remaining
[ ] System enables customers to view volume tracking data

Dependencies
This story is blocked by the following dependencies:

#<issue_no.>
sanger/#<issue_no.>

References
This story has a non-blocking relationship with:

#<issue_no.>
sanger/#<issue_no.>

Additional context
This will reduce the need for multiple emails and manual sample checks per sample. Up to 50% of ToL submitted samples (200-400 per month) could require a top-up. The extend is largely unknown as this processing is essentially not happening at present, which means ToL is unable to complete and assemble genomes.

Volume Tracking:
Volume is recorded for a library (pool) when a pool is created on Traction.
As a lab user, we would like to be able to manage the library volume automatically
The volume would need to be updated on two occasions
When a run is created i.e. volume is taken from a pool create a complex - this would need to be user specified
When a pool is made i.e. when volume is taken from 2 or more samples to create a new pool
Also required is a way to flag a library as exhausted i.e. 0uL volume. (It would be great if this could link with Labwhere. Destroyed location = 0uL or vice versa)
Note: Library volume may decrease outside of these scenarios e.g. due to evaporation, spillage, loss of sample
As a lab manager I would like to be able to track these volumes using the MLWH in conjunction with other data to determine sequencing schedules e.g. top up of samples that failed to meet initial coverage
ToL as a user of long read services would also like to be able to access this volume information via the MLWH to facilitate their top up management system in STS

Library Volume.pptx

https://lucid.app/lucidchart/3c12fbb3-5265-49b5-9f6a-97c9a69a59eb/edit?invitationId=inv_4a6b1f8d-e886-4ffc-b784-40f2135c0679&page=0_0#

From James (02/01)
Question:
Hi Steve it looks good - just worth flagging early that it will need a bit of flexibility due to evaporation of sample, minor pipetting errors etc. so it wont always be a clean sum up of used volumes. e.g. 15uL and 15uL might use up a 35uL sample.
On the occasions where we:
a. have more sample than expected in traction:
It would be good if we could mark a sample as 0uL remaining - library exhausted.
b. have less sample than expected in traction:
Would it be best to have the used volume be accurate i.e. we used 15uL so that is recorded accurately, but then round up any negative values to 0?
Or perhaps the simplest option would be to have a manual adjustment volume to bring things in to line if they fall out - like a used volume that can be manually specified?
Will be great to have volumes in traction though
Answer:
The design I have created is flexible enough to cover all of those scenarios, how we decide to implement them is up to you e.g. if 15uL and 15uL might use up a 35uL sample it might be that you want say that the original volume was 30uL in which case we could exhaust the original sample and modify the volume of the primary aliquot to 30uL. We could then push this data to the warehouse as a new record.
The design itself will enforce correctness so you will not be able to have volumes in the system that do not make sense.

My comments:
For short read library pools there is already in Limber the functionality to manually record volume (as QC information) against the tube after it has been created.
Possibly 'volume' could be set on creation of the library tube by having it as a field in the pooling screens. You could then give it a default value from the purpose config that varies by pipeline, and allow the lab user to modify that default if needed.
For DNA in plate wells earlier in the pipelines uploading information in QC files is probably the only way to do it, it is impractical to manually set volumes on all the individual wells in 384 and even on 96-well plates.
Automatically decrementing volume remaining as transfers are made from plates and tubes in pipelines is a bit risky, as there is no direct connection between the transfer volumes set in liquid handler robot methods and the LIMS tracking. We could only hardcode a transfer volume in the configs for a step and assume it was not been changed. API interactions with liquid handler robots to pull information about what was actually transferred is not implemented. And in any case would only return what the robot was told to transfer, not what was actually physically transferred (e.g. programmed to take 30ul, but there was only 17ul in the well).
As I understand it the initial volumes in stock wells is largely an assumption based on what the supplier tells them, and varies greatly. Whether volume checking can be performed at all also varies by plate type, and is not always possible (opaque wells, deep wells, frozen and don't want to thaw, etc.).
Evaporation, loss during transfers with tips, etc. also play a part during high throughput processing as labwares sit open on robots.
I think volume checking is more practical at the library step for short read, once samples are pooled. Tracking the high numbers of DNA/RNA samples in plate wells earlier in the pipelines is more difficult.

NB. Flagging wells/tubes as 'exhausted' has been requested for 2 pipelines in short read.
To make it flexible maybe it is more of a 'do not use this any further' flag with a reason (exhausted. contaminated, etc.).
We are thinking for short read that flag would be at receptacle (well/tube) level and prevent any further Submissions for work being created for that receptacle. It could be automatically set when we know the liquid handler robot is being set to take all the sample, and also manually set for specific situations (like in Bioscan where the insect has gone from a well).

Closed as done.


Access to model constant

If I try to get constant from model
<option value="{{ constant('USER::ROLE_MASTER') }}">Master</option>

I get error:\
constant(): Couldn't find constant USER::ROLE_CUSTOMER

What class do you have? USER? Or User or App\User?


Ubuntu network not showing in UTM

So I created a fresh VM with Ubuntu on my M1 Macbook pro and I realized that I had no network. I don’t know why I tried emulated Vlan , bridged and shared It doesn’t work. I tried to change the network card but didn’t work. Please help I appreciate.
Configuration

UTM Version: I think the latest
OS Version: MacOS Monetary/ VM: Ubuntu 20.04.4 LTS
Apple Silicon

Have you tried checking your network settings on your MacBook?

Yes it works perfectly on my Mac but not on the VM

I have the same problem.


WRKLDS-1491: Revert back to single arch temporarily

There is a bug in pipeline that fails when FBC image is multi arch image list and this will be fixed by https://github.com/konflux-ci/release-service-catalog/pull/559 and releasing to prod will take time.
In order to unblock cli manager FBC release and continue testing, this PR reverts back to FBC configuration to single arch to get a release. After the release, there will be new PR to add multi arch again.

/lgtm


[ENG-4838] Update search-result card to show preprint word

Ticket: ENG-4838
Feature flag: n/a

Purpose

Don't show "PREPRINT" on search result card for branded preprint provider discover page

Summary of Changes

Pass provider to search-result card

Check result type and provider's preprintWord

Update Preprint Provider word to just Provider

Screenshot(s)

For preprint provider's discover page:

Side Effects

QA Notes

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

2 of 2 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.004%) to 70.668%

Totals

Change from base Build<PHONE_NUMBER>:
-0.004%

Covered Lines:
5950

Relevant Lines:
8187

💛  - Coveralls


New version: SuiteSparseGraphBLAS v0.10.0

Registering package: SuiteSparseGraphBLAS
Repository: https://github.com/JuliaSparse/SuiteSparseGraphBLAS.jl
Created by: @Wimmerer
Version: v0.10.0
Commit: 51075dfa73ccfd23dc6cb0dc458b1b99cee0b7a1
Reviewed by: @Wimmerer
Reference: https://github.com/JuliaSparse/SuiteSparseGraphBLAS.jl/commit/51075dfa73ccfd23dc6cb0dc458b1b99cee0b7a1#commitcomment-127539671
Description: Sparse, General Linear Algebra for Graphs!

block


ReferenceError when a submodule can not be found

Hi there is bug  in lib/snooze.js
You throw an error when a submodule could not be found:
throw new Error(
                'Unable to find submodule in node_modules (recursively): ' +
                submodule);

This ends in a ReferenceError because your variable is called submod and not submodule here.

Good catch. This would have passed the unit tests because referencing a variable that doesn't exist would throw an error and the unit test checked if an error was thrown.
Would you like to make this PR to dev?


Added loadticketcloud.py to parse the tgt_cloud returned by roadtx prt

The file tgt_cloud.ccache can be used to request TGS for accessing Azure file shares using Kerberos authentication.

Thanks for all your work on roadtools, it helped me a lot to understand how Entra and Kerberos work!


cluster and nodejs 0.6.x

Cannot get the cluster function to work with nodejs 0.6.x. Apparantly cluster is now integrated within nodejs 0.6.x and the api's have changed. Is there any workaround to get it run?

Any news?

the nodejs 0.6.x has its native cluster module . see doc:  http://nodejs.org/docs/latest/api/cluster.html
I test  the  native cluster  on node 0.6.3.
maybe haraka require the thirdparty's  cluster(which  doesn't support node > 0.4.x) .

I think Harak should using the nodejs native cluster module.


APPS-3018: Create integrator manifest strategy for navigation and add navigation manifest

Ticket: https://spryker.atlassian.net/browse/APPS-3018

Please change this according to current master file structure
Best to rebase and also check if this is now in line with current integator specs.


[Bug]: What's between DefaultEmbeddingFunction() and SentenceTransformerEmbeddingFunction(model_name="all-MiniLM-L6-v2")?

What happened?
I save some embeddings by default like this:
collection = client.get_or_create_collection(name=db_name)
Then, i can fetch data by DefaultEmbeddingFunction() like:
emb_fn = embedding_functions.DefaultEmbeddingFunction()
collection = client.get_or_create_collection(name=db_name, embedding_function=emb_fn) # It's work
But i can't fetch data by all-MiniLM-L6-v2 like:
emb_fn = embedding_functions.SentenceTransformerEmbeddingFunction(model_name="all-MiniLM-L6-v2")
collection = client.get_or_create_collection(name=db_name, embedding_function=emb_fn) # It's not work
What's the different between DefaultEmbeddingFunction and SentenceTransformerEmbeddingFunction(model_name="all-MiniLM-L6-v2")?
Versions
ChromaDB V0.5.3, python v3.10.11, MacOS 15.0 Beta (24A5327a)
Relevant log output
No response

@h3clikejava, thanks for raising this. Let's start with some background:

Default EF uses onnx runtime + mean pooling, resulting in 384-dimensional embeddings
ST does everyting out of the box and again produces 384-dimensional embeddings

Under normal circumstances, you should not have trouble swapping between the two, as Chroma will accept queries using 384-dimensional embeddings, despite the fact that there are slight differences in the output embeddings (in the order of 1-e4/1-e5 range).
That said, when you say it won't work, do you mean you get an error? Can you share the error?

@h3clikejava happy to re-open if you can help out! closing for now as @tazarov did a good job addressing it


Use takeEvery or takeLatest based on action payload?

Based on the payload of the action that is being watched I want to either trigger a takeLatest or takeEvery.
Right currently I have a (half)working solution. The first time the GET_ORDERS_REQUEST with the Open type gets dispatched, the effect is not happening. If that action would be dispatched again, fetchOpenOrders gets called :)
Anything I'm missing?
function* ordersRequestSaga() {
 const action = yield take(OrderActions.TypeKey.GET_ORDERS_REQUEST);
 switch (action.payload.requestType) {
 case OrderData.RequestType.Open:
  yield takeLatest(action.type, fetchOpenOrders);
  break;
 default:
  break;
 }
}

The way to implement what you're trying to do is much simpler than you think. There's no need for the switch statement because Saga helpers like takeLatest and takeEvery only spawn a new saga when the first argument matches with the action your dispatched to the store.
function* ordersRequestSaga() {
  yield takeLatest(OrderData.RequestType.Open, fetchOpenOrders);
  yield takeEvery(OrderData.RequestType.OtherRequestType, fetchOtherRequests)
}

You can also pass predicate to takeLatest/takeEvery instead of action type. Smth like
function* ordersRequestSaga() {
  yield takeLatest(ac => ac.type === OrderActions.TypeKey.GET_ORDERS_REQUEST && ac.payload.requestType === OrderData.RequestType.Open, fetchOpenOrders);
  yield takeEvery(ac => ac.type === OrderActions.TypeKey.GET_ORDERS_REQUEST && ac.payload.requestType !== OrderData.RequestType.Open, fetchOtherRequests)
}


How to generate schema + gpu_data using latest GpuArrowReader

Hi,
We used to have a script that works for pygdf and h2o4gpu, but it no longer works due to a change in pygdf that I'm confused about.  Can I get help?
See:  https://github.com/h2oai/gpuopenai/blob/master/pygdf/notebooks/mapd_to_pygdf_to_matrix_3-smallipums_4matrices.ipynb
I don't see how in this notebook to create some schema needed now by GpuArrowReader.
I might be able to re-write the whole thing, but I'm trying to show something tomorrow.
I can see the changes made here:
https://github.com/gpuopenanalytics/pygdf/commit/50347914a1b230e5d8a420e8f6e285dd1315331e
But it's not obvious how to do this within the mapd-numba-pygdf framework.

@sklam

I'm basically looking for the minimal change to get this schema.  I think once that's done, the rest should work fine.

Somewhat related: https://github.com/mapd/mapd-core/issues/43

With the new changes, users can use pymapd to get a pygdf.DataFrame directly from a MapD query.  See docs about pymapd.Connection.select_ipc_gpu.

I can still get the actual directly copy of a simple data pointer to a contiguous data block like before?


error with mseed_predictor

I've got the following error when running EQT using mseed_predictor:

11-06 21:25 [INFO] [EQTransformer] Started working on JNAK, 7 out of 36 ...
11-06 21:25 [INFO] [EQTransformer] 20200101T000000Z__20200102T000000Z.mseed
11-06 21:25 [INFO] [EQTransformer] 20200102T000000Z__20200103T000000Z.mseed
11-06 21:25 [INFO] [EQTransformer] 20200103T000000Z__20200104T000000Z.mseed
11-06 21:25 [INFO] [EQTransformer] 20200104T000000Z__20200105T000000Z.mseed
11-06 21:26 [INFO] [EQTransformer] 20200105T000000Z__20200106T000000Z.mseed
11-06 21:26 [INFO] [EQTransformer] 20200106T000000Z__20200107T000000Z.mseed
11-06 21:26 [INFO] [EQTransformer] 20200107T000000Z__20200108T000000Z.mseed
11-06 21:26 [INFO] [EQTransformer] 20200108T000000Z__20200109T000000Z.mseed
11-06 21:26 [INFO] [EQTransformer] 20200109T000000Z__20200110T000000Z.mseed
11-06 21:26 [INFO] [EQTransformer] 20200110T000000Z__20200111T000000Z.mseed
11-06 21:26 [INFO] [EQTransformer] 20200111T000000Z__20200112T000000Z.mseed
11-06 21:26 [INFO] [EQTransformer] 20200112T000000Z__20200113T000000Z.mseed
11-06 21:27 [INFO] [EQTransformer] 20200113T000000Z__20200114T000000Z.mseed
11-06 21:27 [INFO] [EQTransformer] 20200114T000000Z__20200115T000000Z.mseed
11-06 21:27 [INFO] [EQTransformer] 20200115T000000Z__20200116T000000Z.mseed
Traceback (most recent call last):
File "./runEQT.py", line 83, in
batch_size=500)
File "/home/tatar/Programs/miniconda3/envs/eqt/lib/python3.7/site-packages/EQTransformer/core/mseed_predictor.py", line 311, in mseed_predictor
predD, predP, predS = model.predict_generator(pred_generator)
ValueError: not enough values to unpack (expected 3, got 0)

this is occurred for one station and it worked for rest of them.

@saeedsltm that station might have very short duration of data. Trying smaller batch size might help.

@smousavi05 thanks, it worked!


Refactor ~ enable rc search when needed

clang may require the use of MSVC rc.
This modification enable the needed check_vstudio() call when rc is requested.

On windows, for the default "windows" platform, this patch does require using --mrc= or --mrc=<ALTERNATE_RC> to avoid the check_vstudio() call. I'm not sure if there's another way, but this markedly speeds up clang and gcc configurations when either no or alternate resource compilers are used.


RelayOptions - UseOpaqueCursors

Is your feature request related to a problem? Please describe.
Just like global ids, cursors would ideally be opaque, as per docs here.
Describe the solution you'd like

bool UseOpaqueCursors setting on RelayOptions.
if true, would set up:

serialization of the cursor field on a PageInfo type, and
deserialization of the before and after

It'd also be nice if the ICursorSerializer could be swapped out by the developer so they could specify their own (to aid adoption allowing them to do a non-breaking change by keeping all ids the same).
Describe alternatives you've considered
It's possible to do this yourself with middleware, but it'd be nicer to be in the framework since it's a common need.

how would the interface look like?

how would the interface look like?

Taking inspiration from IIdSerializer, it could be this. The "Serialize/Deserialize" feels a bit weird though. Particularly given it's string <--> string and I imagine it still will be, unless you wanted to have the Edge have object Cursor and then there's always a registered ICursorSerializer, but there's an option to toggle .ToString() or .ToString().ToBase64().
Almost could be IIdEncoder or IIdMasker, but probably doesn't matter too much either way.
I'm mostly interested in this as I know our FE have a couple of global ids in their JS for icon maps or something, so with this abstraction I could swap in my exact base64 implementation (or fallback onto HC's built-in if it turns out to be the same) without having a breaking release. But it also lets people use whatever serialization/encoding they want, if they wanna make it opaque some other way, use some future faster encoding method.
Also OK with not even bothering as I can easily update the FE code AOT to handle both before/after ids and maybe we don't care about making this flexible - it's just on/off base64 encoding, end of story. Up to you :)
#nullable enable

namespace HotChocolate.Types.Relay
{
    public interface ICursorSerializer
    {
        /// <summary>
        /// Creates an serialized cursor returned to clients.
        /// </summary>
        /// <param name="cursor">The cursor.</param>
        /// <returns>
        /// Returns a serialized cursor.
        /// </returns>
        /// <exception cref="CursorSerializationException">
        /// Unable to create a serialized cursor string.
        /// </exception>
        string Serialize(string cursor);

        /// <summary>
        /// Deserializes a serialized cursor.
        /// </summary>
        /// <param name="serializedCursor">
        /// The serialized cursor.
        /// </param>
        /// <returns>
        /// Returns the deserialized cursor string.
        /// </returns>
        /// <exception cref="CursorSerializationException">
        /// Unable to deserialize the serialized cursor string.
        /// </exception>
        string Deserialize(string serializedCursor);
    }
}

Taking inspiration from IIdSerializer, it could be this. The "Serialize/Deserialize" feels a bit weird though. Particularly given it's string <--> string and I imagine it still will be, unless you wanted to have the Edge have object Cursor and then there's always a registered ICursorSerializer, but there's an option to toggle .ToString() or .ToString().ToBase64().
Almost could be IIdEncoder or IIdMasker, but probably doesn't matter too much either way.
I'm mostly interested in this as I know our FE have a couple of global ids in their JS for icon maps or something, so with this abstraction I could swap in my exact base64 implementation (or fallback onto HC's built-in if it turns out to be the same) without having a breaking release. But it also lets people use whatever serialization/encoding they want, if they wanna make it opaque some other way, use some future faster encoding method.
Also OK with not even bothering as I can easily update the FE code AOT to handle both before/after ids and maybe we don't care about making this flexible - it's just on/off base64 encoding, end of story. Up to you :)
#nullable enable

namespace HotChocolate.Types.Relay
{
    public interface ICursorSerializer
    {
        /// <summary>
        /// Creates an serialized cursor returned to clients.
        /// </summary>
        /// <param name="cursor">The cursor.</param>
        /// <returns>
        /// Returns a serialized cursor.
        /// </returns>
        /// <exception cref="CursorSerializationException">
        /// Unable to create a serialized cursor string.
        /// </exception>
        string Serialize(string cursor);

        /// <summary>
        /// Deserializes a serialized cursor.
        /// </summary>
        /// <param name="serializedCursor">
        /// The serialized cursor.
        /// </param>
        /// <returns>
        /// Returns the deserialized cursor string.
        /// </returns>
        /// <exception cref="CursorSerializationException">
        /// Unable to deserialize the serialized cursor string.
        /// </exception>
        string Deserialize(string serializedCursor);
    }
}


Create game, author classes and add class properties

Create a game, author classes, and add class properties:
In this pull request I did the following:

Created a Game class in a separate .rb file.
Created an Author class with an association to the Item class (in a separate .rb file).
All Game class properties visible in the diagram have been defined and set up in the constructor method.
All Author class properties visible in the diagram have been defined and set up in the constructor method.

Approved :+1:
Congratulations your project is done, There's nothing else to say other than merge it.


Create aliases for the imports to omit long import statements (dev issue not end user's)

Problem:
In the Screen Shot below we have a very long import statement which disrupts the code appearance and also make importing components harder which can be done relatively easy manner.

Can be seen in these files as well :

Solution
We can use path aliasing like @components/AppbarClient there are some alias already present inside next.config.js and we can describe some more and use it as a standard to bring uniformity in the code base.
correct imports examples :

Hi, I'd like to work on this issue. Could you please assign it to me?

Ok I will work on that then raise a pr

To create aliases in a React codebase, particularly in a Next.js project, you can use path aliasing for import statements. This can be achieved by making modifications to the next.config.js file and then using the defined aliases in the codebase.
Here are the steps to create aliases in a React codebase, specifically in a Next.js project:
Modify next.config.js: In the root of your Next.js project, create or open the next.config.js file. If it doesn't exist, create this file at the root of your project.
Inside next.config.js, you can use the resolve.alias property to define your path aliases. Here's an example of how to define aliases for commonly used paths:
const path = require('path');

module.exports = {
  webpack: (config, options) => {
    config.resolve.alias['@components'] = path.join(__dirname, 'components');
    config.resolve.alias['@styles'] = path.join(__dirname, 'styles');
    // Add more aliases as needed
    return config;
  },
};```

Using the Aliases in Code: After defining the aliases in next.config.js, you can now use these aliases in your import statements throughout your codebase. For example, if you defined an alias for '@components', you can import components like this:
import AppbarClient from '@components/AppbarClient';

i hope it has answered your issue

i have also raised the PR #500 , on merging it resolved this issue.
Happy coding 👍

Yes but can you also use these aliases and update the PR?

yes! thanks for updating me


specific.html info alignment issue

At any resolution, the specific.html page information section has an alignment issue on several elements, see attached screenshot. Some items are centered, others are justified space-between/space-evenly. Perhaps a more uniform approach would be ideal?

Converted to table so bidder amount and placed all line up with each other, buttons to be adjusted with skeleton loading


Custom parameter injection and multiple tests run

This PR has a double purpose:

allowing to inject custom parameters for the tool (currently it is enabled for VPR only, but could be expanded in the future to be more flexible).
make it possible to easily running multiple times the tests to gather more samples.

Note: Given that the exhaust script is used mainly to perform a batch of tests, I have limited the freedom degree to be able to choose only toolchains and projects (from a user perspective).
The user can still run single tests using fpgaperf.py.
TODO:

[x] test multiple runs with all of them gathering information on CI
[x] test multiple runs with different options for VPR

@kgugala I have updated the colab page with the new results

@kgugala I have updated the colab page with the new results
Those results are identical (up to 7th digit after comma). Are you sure this is correct?

What does this None parameters mean? I don't think VPR was ran there w/o any parameters

What does this None parameters mean? I don't think VPR was ran there w/o any parameters

Actually this is a bug I discovered just now, the parameters run had an additional build with no parameters override set, hence the None value. I'll push a fix in a moment

Those results are identical (up to 7th digit after comma). Are you sure this is correct?

Looking at the meta.json that is produced by each run it seems that those are indeed identical. I think this is good news actually as this seems to indicate that, with identical input conditions we get identical outputs.

In addition, there are some fixes still to be made:

[ ] when using the parameters build type, the dataframe should contain columns with the parameters values, so that they can be correctly displayed in a more readable format.
[ ] fixes to results parsing, as, by looking at the table, there seem to be some issues related to that (e.g. not all clocks are correctly gathered, missing placing time for vivado, resource usage in VPR is way to less than vivado, etc.)

I believe that, if everything is ok with this PR we should get to merge it as it is quite big, and move with the fixes in a second step.


how to test data ??

how to test data ??

I can load the model from this Checkpoint (https://github.com/CharlesShang/FastMaskRCNN/issues/86#issuecomment-309022377), and get predictions from it.
However, I can only get predictions for an output_tensor before the ROIAlign layer,
but not for any tensor in the graph after that layer. Example:
# this works
output_tensor = graph.get_tensor_by_name("pyramid_1/ROIAlign_2/Crop:0")
# this doesn't work
output_tensor = graph.get_tensor_by_name("pyramid_1/ROIAlign_2/Reshape_2:0")

There's a bug in the ROIAlign layer with the py_func used to generate anchors.

My understanding is that the py_fun is not stored in the graph and cannot be reconstructed.
Or is this bug related to casting anchor coordinates to ints?
Can anyone here (@amirbar, @sairin1202 , @hari-sikchi , @Cangrc , @mariolew , @cuihuitao) try to load the graph this way, and reproduce the error / find a workaround ?

tf.reset_default_graph()
checkpoint_dir = "/my/directory/for/segmentation/FastMaskRCNN/output/mask_rcnn/500k/"
meta_file_path = checkpoint_dir + "coco_resnet50_model.ckpt-499999.meta"
ckpt_file_path = checkpoint_dir + "coco_resnet50_model.ckpt-499999"

# load graph
saver = tf.train.import_meta_graph(meta_file_path, clear_devices=True)
graph = tf.get_default_graph()

# initialize and load weights
sess =  tf.Session()
sess.run(tf.global_variables_initializer())
saver.restore(sess, ckpt_file_path)

# Input images
input_tensor = graph.get_tensor_by_name("Reshape_2:0")
output_tensor = graph.get_tensor_by_name("pyramid_2/MaskEncoder/Reshape_2:0")

# Mask output
results = sess.run(
    [output_tensor],
    #[tf.reduce_mean(input_tensor)],
    feed_dict={input_tensor:images}
)

@simaoh Hi，simaoh ， have you fixed it? are you able to test your own image now?


[Internal] - sap/m/demokit/helloworld using AMD style

Adapting the demokit helloworld to AMD style using sap.ui.define

Yes, i hadn't synced with the latest changes made to the File.

Ummm... I thought I had commented before, but I haven't.
I'd think that in a Hello World example simplicity wins over being clean and best-practice compliant. Shouldn't a "Hello World" show about the minimum code required to display this text within a typical environment of the framework? Without distracting constructs that do not serve this goal?
I'm totally for AMD syntax and showing it in our very next example, but maybe not in the very first?
Regards
Andreas

@akudev didn't really think about that. But it makes sense.


UDP with multiple type

Hello,
I'm wondering if it is possible to create a UDP that accept multiple type? I searched the docs but can't find an example. I want to do something like:
property assign {type=ref|boolean|string; component=signal;};
Thanks

Doesn't look like it.
Section 15 doesn't mention the ability to do so, and the BNF grammar definition in Annex B is pretty clear that it is not supported:
property_attribute ::=
    property_type
    | property_usage
    | property_default
    | property_constraint
property_type ::= type = property_data_type [ array_type ] ;
property_data_type ::=
    component_primary_type
    | ref
    | number
    | basic_data_type
property_usage ::= component = property_comp_types ;
property_comp_types ::= property_comp_type { | property_comp_type }

Would you be able to point me to Section 15? Also, I guess to support it first we need to modify the BNF grammar in the parser huh?

I was referring to Section 15 of the SystemRDL 2.0 language spec: https://accellera.org/downloads/standards/systemrdl
See page 85.
The spec also defines the BNF grammar in Annex B, on page 107.
As this is part of the official language specification, I'm not interested in changing the compiler to violate that.

Thanks for pointing me to the specs. Closing this issue 👍


ca-bc-chilliwack

{
    "coverage": {
        "country": "ca",
        "state": "bc",
        "city": "Chilliwack"
    },
    "data": "http://opendata.chilliwack.com/SHP/Parcels_SHP.zip",
    "website": "http://www.chilliwack.com/main/page.cfm?id=2331",
    "license": "http://www.chilliwack.com/main/page.cfm?id=2391",
    "type": "http",
    "compression": "zip",
    "fingerprint": "b1c098d36137ea82dbd1f7044b44b459",
    "version": "20140502",
    "cache": "http://s3.amazonaws.com/openaddresses/ca-bc-chilliwack.zip",
    "note": "No Address Data",
    "skip": true
}

No address data

We have a statewide layer - I support source duplication but going to close for now unless someone wants to go after this.


Remove thinid (diana.js)

Remove thinid, it uses Math.random() for getting random numbers so it's cryptographically insecure and uses non url safe characters which makes it useless, since in node environment you can easily use Crypto for random number generation. Also generates many random numbers every function call which hurts performance

And again. Thanks :)


Show the rule that is causing the issue

In the message from SCI, the rule is not mentioned that was failing.
FUGR /JCS/BC > PROG LBTCHDEF [ @518 ]  Variable "rc_tgt_host_chk_has_failed" not used, #JCS#BC.FUGR.LBTCHDEF.ABAP
like in the pipeline:
/tmp/abaplint-aigijg/src/cx_salv_bs_tools_conv_error.clas.abap[1, 1] - Expected CLASSDEFINITION (structure)

where is the first text from?
also see https://github.com/abaplint/abaplint-cloud-foundry/issues/21

We might want to use an other field for the information. We are using the inform method:

It contains 4 attribute parameters (only one is used). Or embed it in the message (at the end in round brackets).
Reference

yea, so embed the abaplint key as last thing in the message text
and then we can hash the message code to SCI_ERRC in order to have the correct grouping

try again, latest version of the abap code

using the latest change (also on docker) shows the rule.
Would propose though to change to p_text = '&1 (&2)'. which makes it the same as with abaplint and is also more visible.
When running SCI using ADT the message seems to be messed up

Had a case where the message is getting to long.

Would it not be better to remove the file path? or put it at the end or a new line?

ATC? please move that to a separate issue
description fixed in https://github.com/abaplint/abaplint-abap-backend/pull/54/commits/765693d99dd6ed7d518802d56be237c9e9be01e9
idea is to only output the filename in some cases, it now handles namespaces, https://github.com/abaplint/abaplint-abap-backend/pull/54/commits/b57c0fb226afa42934baf31a524c0c98b34db94f

Much better. Thanks


[BUG] get_movielens throws on exception on the ml-100k dataset

Bug description
get_movielens throws on exception on the ml-100k dataset

Steps/Code to reproduce bug
from merlin.models.data.movielens import get_movielens
train, valid = get_movielens(variant="ml-100k")

Throws an exception like
   train, valid = get_movielens(variant="ml-100k")
  File "/home/ben/code/models/merlin/models/data/movielens.py", line 53, in get_movielens
    movielens_download_etl(path, variant)
  File "/home/ben/code/models/merlin/models/data/movielens.py", line 210, in movielens_download_etl
    movies = pd.read_csv(os.path.join(local_filename, "ml-100k/u.item"), names=cols, sep="|")
  File "/home/ben/miniconda3/envs/nvtabular/lib/python3.8/site-packages/pandas/util/_decorators.py", line 311, in wrapper
    return func(*args, **kwargs)
  File "/home/ben/miniconda3/envs/nvtabular/lib/python3.8/site-packages/pandas/io/parsers/readers.py", line 586, in read_csv
    return _read(filepath_or_buffer, kwds)
  File "/home/ben/miniconda3/envs/nvtabular/lib/python3.8/site-packages/pandas/io/parsers/readers.py", line 482, in _read
    parser = TextFileReader(filepath_or_buffer, **kwds)
  File "/home/ben/miniconda3/envs/nvtabular/lib/python3.8/site-packages/pandas/io/parsers/readers.py", line 811, in __init__
    self._engine = self._make_engine(self.engine)
  File "/home/ben/miniconda3/envs/nvtabular/lib/python3.8/site-packages/pandas/io/parsers/readers.py", line 1040, in _make_engine
    return mapping[engine](self.f, **self.options)  # type: ignore[call-arg]
  File "/home/ben/miniconda3/envs/nvtabular/lib/python3.8/site-packages/pandas/io/parsers/c_parser_wrapper.py", line 69, in __init__
    self._reader = parsers.TextReader(self.handles.handle, **kwds)
  File "pandas/_libs/parsers.pyx", line 542, in pandas._libs.parsers.TextReader.__cinit__
  File "pandas/_libs/parsers.pyx", line 734, in pandas._libs.parsers.TextReader._get_header
  File "pandas/_libs/parsers.pyx", line 843, in pandas._libs.parsers.TextReader._tokenize_rows
  File "pandas/_libs/parsers.pyx", line 1917, in pandas._libs.parsers.raise_parser_error
UnicodeDecodeError: 'utf-8' codec can't decode byte 0xe9 in position 76620: invalid continuation byte

I could not repro the error. It works fine for me. I tested it in merlin-tensorflow-training:22.03 container with models, core and nvtabular latest main branches pulled.

This looks like a difference between pandas 1.2 and pandas 1.3. In pandas 1.3 - the default encoding seems to be utf-8 in pandas 1.3 which causes this issue. You can replicate by changing the encoding to utf-8 on pandas 1.2.
I have a fix in https://github.com/NVIDIA-Merlin/models/pull/249 - that should work for both pandas 1.2.x and pandas 1.3.x+


Document pull_from_upstream

Fixes #495

@jpopelka @lbarcziova rereview please, added a bit more clarification and rewording.


[Chained] ruler,receive,sidecar: StoreAPI Series encodes chunks to 120 samples instead of single, big one.

Chained PR to https://github.com/thanos-io/thanos/pull/2305
This is to have a unified chunk size and should reduce the load on queriers.
This also will be much more comparable when a chunk iterator will be done.
See following benchmark results for Receive (multiTSDB):
 benchstat -delta-test none _dev/bench_outs/0-receiveseries/benchBenchmarkMultiTSDBSeries.out _dev/bench_outs/1-receiveseries/benchBenchmarkMultiTSDBSeries.out
name                                                                                old time/op    new time/op    delta
MultiTSDBSeries/1000000SeriesWith1Samples/headOnly/4_TSDBs_with_1_samples,_250000_series_each-12         6.41s ± 0%     6.16s ± 0%   -3.85%
MultiTSDBSeries/1000000SeriesWith1Samples/blocksOnly/4_TSDBs_with_1_samples,_250000_series_each-12       5.77s ± 0%     6.16s ± 0%   +6.61%
MultiTSDBSeries/100000SeriesWith100Samples/headOnly/4_TSDBs_with_25_samples,_25000_series_each-12        3.68s ± 0%     3.96s ± 0%   +7.43%
MultiTSDBSeries/100000SeriesWith100Samples/blocksOnly/4_TSDBs_with_25_samples,_25000_series_each-12      4.04s ± 0%     4.02s ± 0%   -0.46%
MultiTSDBSeries/1SeriesWith10000000Samples/headOnly/4_TSDBs_with_2500000_samples,_1_series_each-12       1.53s ± 0%     1.57s ± 0%   +2.08%
MultiTSDBSeries/1SeriesWith10000000Samples/blocksOnly/4_TSDBs_with_2500000_samples,_1_series_each-12     1.67s ± 0%     1.66s ± 0%   -1.12%

name                                                                                old alloc/op   new alloc/op   delta
MultiTSDBSeries/1000000SeriesWith1Samples/headOnly/4_TSDBs_with_1_samples,_250000_series_each-12        4.08GB ± 0%    4.08GB ± 0%   +0.03%
MultiTSDBSeries/1000000SeriesWith1Samples/blocksOnly/4_TSDBs_with_1_samples,_250000_series_each-12      4.08GB ± 0%    4.08GB ± 0%   -0.01%
MultiTSDBSeries/100000SeriesWith100Samples/headOnly/4_TSDBs_with_25_samples,_25000_series_each-12       1.73GB ± 0%    1.72GB ± 0%   -0.37%
MultiTSDBSeries/100000SeriesWith100Samples/blocksOnly/4_TSDBs_with_25_samples,_25000_series_each-12     1.66GB ± 0%    1.67GB ± 0%   +0.57%
MultiTSDBSeries/1SeriesWith10000000Samples/headOnly/4_TSDBs_with_2500000_samples,_1_series_each-12      2.71GB ± 0%    2.47GB ± 0%   -8.68%
MultiTSDBSeries/1SeriesWith10000000Samples/blocksOnly/4_TSDBs_with_2500000_samples,_1_series_each-12    2.68GB ± 0%    2.46GB ± 0%   -8.14%

name                                                                                old allocs/op  new allocs/op  delta
MultiTSDBSeries/1000000SeriesWith1Samples/headOnly/4_TSDBs_with_1_samples,_250000_series_each-12         44.9M ± 0%     44.9M ± 0%   +0.00%
MultiTSDBSeries/1000000SeriesWith1Samples/blocksOnly/4_TSDBs_with_1_samples,_250000_series_each-12       44.9M ± 0%     44.9M ± 0%   -0.00%
MultiTSDBSeries/100000SeriesWith100Samples/headOnly/4_TSDBs_with_25_samples,_25000_series_each-12        27.1M ± 0%     27.1M ± 0%   -0.01%
MultiTSDBSeries/100000SeriesWith100Samples/blocksOnly/4_TSDBs_with_25_samples,_25000_series_each-12      27.1M ± 0%     27.1M ± 0%   +0.01%
MultiTSDBSeries/1SeriesWith10000000Samples/headOnly/4_TSDBs_with_2500000_samples,_1_series_each-12       1.02M ± 0%     1.69M ± 0%  +65.15%
MultiTSDBSeries/1SeriesWith10000000Samples/blocksOnly/4_TSDBs_with_2500000_samples,_1_series_each-12     1.02M ± 0%     1.69M ± 0%  +65.34%

Signed-off-by: Bartlomiej Plotka<EMAIL_ADDRESS>

[ ] I added CHANGELOG entry for this change.
[x] Change is not relevant to the end user.

Changes

Verification

Linting seems to have failed.

all is fine now, would be useful for future benchmarks.
CC @brancz


Is rapidblock abandoned?

No updates to the block list in months.
rapidblock.org is down.

Yeah.  Other people are doing a better job than I could; I have too much personal stuff going on right now to keep up.  See https://codeberg.org/oliphant/blocklists .


req.parseBody should allow multiple keys (and converted to array)

Currently
Only the last key is recorded into the resulting object after req.parseBody().
Expected
When parsing request parameters that consists of multiple keys, the keys should be converted to array automatically.
tests=1&tests=2&tests=3

{ tests: [ 1,2,3 ] }

Also, another use case where using tests[] which has been around for some time also need to be supported.
tests[]=1

{ tests: [ 1 ] }

Related: https://github.com/honojs/hono/issues/257

please see https://hono.dev/api/request
// Captured params
app.get('/entry/:id', (c) => {
  const id = c.req.param('id')
  ...
})

// Get all params at once
app.get('/entry/:id/comment/:commentId', (c) => {
  const { id, commentId } = c.req.param()
})

@bogordesaincom The one that's more relevant is this https://hono.dev/api/request#queries. Thanks for the hint.

ahh yes. that query section

this doesn't answer the question, as the question is clearly about req.parseBody().
I have run into this issue as well today.  They are right that only the last value is parsed from the body.

Hi. We may fix it. But it will be a breaking change. So, will be included in the next major release.

I saw a similar issue which suggested manually parsing form data, so it’s not a priority for me. Thanks!

this doesn't answer the question, as the question is clearly about req.parseBody().

Because parseBody and queries has the same format, I hope both enable single key multiple value parsing to array.

I think this implementation is good, if passing the all: true as an option, it return multiple values as an array:
c.req.parseBody({ all: true })

With this way, it will not be a breaking change.

I ran into this as well.
I think it would be better to parse multiple keys into an array by default for consistency & predictability. { all: true } is inconsistent with the way Hono handles queries. Parsing multiple keys into an array by default also makes it clear for users what to expect: post a key once if you want a single value and post it multiple times if you want an array.
The { all: true } option also would mean that the validator & zValidator middleware would need this argument as well.

The PR #1488 will resolve this issue.

My concern is that we expect an array to come in, but a single value comes in and it is not an array. This is not the behavior the user expects.
So I suggested { all: true }. When using a validator, the user specifies whether it is an array or not, so the all option is not necessary.

@yusukebe these small things would be nice tips on the docs 👍

how to use this with zod validator?

Hi @mrevanzak
Currently, the validator does not support array values in form. We may make the validator support it, but it's not soon.

did it mention anywhere in the docs?


Custom reference

Hello,
Sorry to bother, but rather than templates I prefer to use the --reference-doc=custom-reference.odt option. However, wherever I put custom-reference.odt, the plugin doesn't find it, saying openBinaryFile: file does not exist, even if I insert a full path in quotes. I'm using Linux Mint, and the file exists in my Vault as well as various other places that I try to put it.
Any help would be greatly appreciated, as this has been a block in my workflow for a very long time now.
Thank you
Lucas

Hello,
I have been able to partially solve the problem, by putting the custom reference document in the same folder as the file to be converted. Putting a full path to the file doesn't work.
Lucas


Enable testing mode for the Datalad-registry Flask app in tests

This PR is based on https://github.com/datalad/datalad-registry/pull/138. It should be rebased after https://github.com/datalad/datalad-registry/pull/138 is merged to get the relevant commits of this PR.

what would be the effect of this?


Landing Page Improvements

Landing page is a little out of date.
Have a few improvements in mind.

[ ] Add a link to the Docs page
[ ] Add feedback to the Terminal Command button

https://github.com/dojoengine/dojo/assets/3332646/c8909cf2-e381-4c83-8938-a73f9209488d

Desktop

Mobile

Resources
Figma
^ You may need to request access ^

i can make changes


crystal: make Crystal default target match Nix target platform

Description of changes
When building the Crystal compiler, it picks a default LLVM target triple by either looking at the env var CRYSTAL_CONFIG_TARGET if set, or by calling LLVMGetDefaultTargetTriple on the linked LLVM. https://github.com/crystal-lang/crystal/blob/master/src/compiler/crystal/config.cr#L47
On nix linux builds of Crystal using the LLVM function seems to be fine. However on MacOS, it can cause a mismatch with nix's MacOS stdenv.
stdenv.targetPlatform.darwinMinVersion defaults to "11.0" and this makes it's way into all the clang and ld wrapper scripts. However, the default LLVM target can be something such as "aarch64-apple-darwin22.3.0" which would translate to kernel version 13. Then Crystal would build .o files targeting 13, and the wrapped clang/ld would try and link them to be compatible with 11 and generate hundreds of warning messages such as " was built for newer macOS version (13.0) than being linked (11.0)".
By setting the CRYSTAL_CONFIG_TARGET to match the nix targetPlatform, the default target for the compiler becomes just "aarch64-apple-darwin", and everything works as expected.
On linux setting this env var has no effect, since the LLVM detected triple matches that of the nix targetPlatform.
Things done

Built on platform(s)

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

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

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

fixes: https://github.com/NixOS/nixpkgs/issues/219512
ref: https://github.com/NixOS/nixpkgs/issues/164648#issuecomment-1320673682

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/showing-first-time-contributors-some-love/29105/14

This looks like a good quick solution.
I believe this could be fixed in the Crystal compiler in a generic way, see https://github.com/crystal-lang/crystal/issues/13594


JedisFactory does not provide an implementation of the passivateObject method, will Jedis also remove idle objects?

We use jedis 2.9.0 in project to provide a redis connection pool, and enbabled test and evict idle object by setTestWhileIdle(true) and setTimeBetweenEvictionRunMills(30000L).
However, we found that in the production environment jedis will send a lot of ping command, which may cause a sudden increase of redis server cluster ops.
I read the jedis code and found that jedis uses  GenericObjectPool from apache, which will start an evictor to evict idle object regularly if TestWhileIdle and TimeBetweenEvictionRunMills configured, in jedis, validateObject() provided by JedisFactory will invoke ping(). Besides,  I found that JedisFactory does not provide an implementation for method passivateObject, I am curious if there is no implementation of method passivateObject(PooledObject pooledJedis), will jedis remove idle objects? Thanks.
The key code is as follows
org.apache.commons.pool2.impl.GenericObjectPool#evict()
if(evict) {
        destroy(underTest);
        destroyedByEvictorCount.incrementAndGet();
    } else {
        if (testWhileIdle) {
            boolean active = false;
            try {
                factory.activateObject(underTest);
                active = true;
            } catch (final Exception e) {
                destroy(underTest);
                destroyedByEvictorCount.incrementAndGet();
            }
            if (active) {
                if (!factory.validateObject(underTest)) {
                    destroy(underTest);
                    destroyedByEvictorCount.incrementAndGet();
                } else {
                    try {
                        factory.passivateObject(underTest);
                    } catch (final Exception e) {
                        destroy(underTest);
                        destroyedByEvictorCount.incrementAndGet();
                    }
                }
            }
        }
    }

I am curious if there is no implementation of method passivateObject(PooledObject pooledJedis), will jedis remove idle objects? Thanks.

Yes，seeredis.clients.jedis.JedisFactory#destroyObject.

I found that JedisFactory does not provide an implementation for method passivateObject

passivateObject(org.apache.commons.pool2.PooledObject) is invoked on every instance when it is returned to the pool, means uninitialize an instance to be returned to the idle object pool, see doc, Jedis doesn't have to implement it, so the code is:
@Override
  public void passivateObject(PooledObject<Jedis> pooledJedis) throws Exception {
    // TODO maybe should select db 0? Not sure right now.
  }

@yangbodong22011 Thanks for your attention, I found that I misunderstood how the redis connection pool works.


Admin SDK User/Group/Member support and misc bug-fixes

Okay, lots of changes being submitted in this pull request.
The ngGAPI has now been updated and tested against the current google-provided javascript client, using OAuth2, and now supports the users, groups and members API's from the Admin SDK portion of the Google API.
Change Summary:

Added API configs to support Admin Directory users, groups & members.
Refactored OAuth2 wrapper and proved that it works in immediate=true mode.
Updated existing unit tests to work with PhantomJS so it runs headless instead of in Chrome. (Feel free to change this back if you want)
Added main: gapi.js to the bower file for better auto-injection support in other projects.
Updated README.md file to include the fact that support for Admin SDK users, groups & members have been added.
Changed references to console.log to $log.log so it doesn't break older versions of MSIE javascript.
Fixed bug in cases where HTTP DELETE is called, where the transaction status could not be read by the caller to verify that the transaction succeeded. Google's API doesn't return any object when DELETE is successful, only a status code as part of the HTTP response. So the caller needs to have access to the transaction object, not just the expected response body object. (lines 133-136 in gapi.js)
Added deferred.notify calls during OAuth2 process to help with communicating to the caller about what is happening during the OAuth2 process.
The OAuth2 process will attempt login 4 times before sending deferred.reject.
Cleaned up trailing white space throughout gapi.js and test spec files.

NOTE: When accessing any of the Admin SDK API's, (currently only users, groups and group members are supported), you need to do the following:

Access the Google Developer console and create an API Client ID for a Web Application. The interface keeps changing every few months, so hopefully the key words "Client ID, and Web Application" are enough to help you find what you're looking for. You want a key that allows you to restrict access by referrer domain name.

Log into your Google Apps domain account as an administrator. By the time you read this, the admin screen might have changed, but here's the path to what you need as of this writing:

Security -> Show More (at bottom) -> Advanced Settings -> Manage API client access

Once you're on that page, you need to enter your client ID and a comma-delimited list of scopes, as defined by the Admin SDK Documentation here: https://developers.google.com/admin-sdk/directory/v1/guides/authorizing

Once you have done that, you should be able to use your Client ID in your javascript call to the Google API as noted in the ngGAPI documentation that Christian wrote.

@jcwatson11
Great work, thanks!
This is quite a lot in one PR. Would you mind doing a quick code review with me in a hangout before I merge?

Yeah. It sure is. And sure. Let's hang... or whatever you call it on a Google hang-out. (I usually Skype).
Is there some way to send a private message through github so I can send you my contact details?

Email me.
<EMAIL_ADDRESS>

Hey Christian,
It was really good to meet, and talk with you during our code review. The changes you requested are committed in this pull request. Thanks for all your support. I look forward to working with you again in the future!
Jon

Thanks Jon! Enjoyed meeting you as well. Please stay in touch.


Automatic shutdown

Often, GPIO 26 triggers shutdown automatically without pressing power button. How to solve this? Please help me.

Please alway provide information on your configuration with any bug reports

what version of Pi? What version of raspbian, etc?

I have had problems with random shutdown events in the past - make sure
that you minimize the lengths of any breadboard wires as these can pick up
interference which is enough to trigger shutdowns.
On Thu, Mar 15, 2018 at 9:47 PM, manikandan-enability <
<EMAIL_ADDRESS>wrote:

Often, GPIO 26 triggers shutdown automatically without pressing power
button. How to solve this? Please help me.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/craic/pi_power/issues/3, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAAqOTnJ0QDxt-VPWDVogZkz5tRxublVks5te0PKgaJpZM4StPhP
.


feat(history): Reset history.current when all apps are destroyed

This resolves #3294

@posva @joeldenning loot at the PR, please

Thank you!

Thank you for your effort of supporting and developing vue :)


Server side RUI docs

yeah

I just found out uniboi also made a serverside rui PR

bro

bro

I didnt notice you also made one

yours are cool but mine got your images so I win

@F1F7Y yo when you did said "Send a small popup to player which will appear in the lower half of their screen under their cursor." By cursor, did you mean crosshair?

ye

ah okay well i forgor to change
anyways #95 should be the most updated pr. Suggest you check it out. I basically just merged both pr's into unibois

uniboi supremacy

Says the dude who asked me how to compile a singe c file from an entire library

I will not elaborate further


Deprecate RAC2 support

As the title states, ReactiveCocoa 2 support has been deprecated.
Closes #298

Current coverage is 80.13%

Merging #301 into master will not affect coverage as of afc012f

@@            master    #301   diff @@
======================================
  Files           13      13
  Stmts          579     579
  Branches         0       0
  Methods
======================================
  Hit            464     464
  Partial          0       0
  Missed         115     115

Review entire Coverage Diff as of afc012f

Uncovered Suggestions

+8.47% via ...rkLoggerPlugin.swift#28...76
+2.77% via Source/Moya.swift#18...33
+1.56% via ...rkLoggerPlugin.swift#12...20
See 7 more...

Powered by Codecov. Updated on successful CI builds.

Probably worth mentioning in the README too

Form maybe clarifying that its 3 and above, rather than saying what isn't supported

Agreed with Orta – some clarification in the readme would be good, but that's not a blocker. I'll open an issue.


:bug: Fixed last wave flickering and minor docs update

Fixes #44

Any update about this bug that the last wave flickering

@PassantAbdelatif This PR still need to be tested for AudioFileWaveforms. That's why it hasn't been merged. We will merge this as soon as possible. Thank you for your patience.

@ujas-m-simformsolutions @dhvanit-v-simformsolutions @vatsaltanna-simformsolutions when will this be merged?
I have been using the latest commit on this pull request as my version for the package for a while now.
In case anyone is interested, this is how I do it:
  audio_waveforms: #WAITING FOR UPDATE https://github.com/SimformSolutionsPvtLtd/audio_waveforms/pull/197
    git:
      url: https://github.com/SimformSolutionsPvtLtd/audio_waveforms.git
      ref: d1f8972d8f4d2c957e9077b67cdd9c420774b2a9

@vatsaltanna-simformsolutions Thank you very much for the merge!
Are you planning on creating a new release soon?


⚠ Jenkins Plugins Artifactory has degraded performance

In dd10a52, Jenkins Plugins Artifactory (https://get.jenkins.io/plugins/artifactory/) experienced degraded performance:

HTTP code: 200
Response time: 1386 ms

Resolved: Jenkins Plugins Artifactory performance has improved in 0519f2d after 12 minutes.


nixos/engelsystem: pin php at 8.1

Upstream supports php 8.0/8.1 for the 3.3.0 release. The upgrade to 8.2 caused a type mismatch in carbon.

PHP message: Exception: Code: 0, Message: Carbon\Carbon::setLastErrors(): Argument #1 ($lastErrors) must be of type array, bool given, called in /nix/store/2prnw9qya9kaks2rwvd6fkrz0c7l5ygd-engelsystem-3.3.0/share/engelsystem/vendor/nesbot/carbon/src/Carbon/Traits/Creator.php on line 98, File: vendor/nesbot/carbon/src/Carbon/Traits/Creator.php:928

Description of changes

Things done

Built on platform(s)

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

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

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

@ofborg test engelsystem


Update conformance terms and forms for support providers

Signed-off-by: Steve Winslow<EMAIL_ADDRESS>

cc @jmertic

@swinslow - this works, thank you! I broke up the Support Conformance types into discrete rows - let me know what you think.


Background repeat outlook

Hi guys,
I'm having an issue with the "background-url" within  "mj-section".
The problem that the background repeats in Outlook, while Gmail etc is fine.
I've tried background-repeat: no-repeat, but nothing seems to work.

Email Client: Outlook
OS: Windows
Browser: N/A

Thanks
Adam :)

Background image are handled by VML, so sadly there's no way to support no-repeat on background for section. There's a small hack to play with vml background size/height but the results aren't that great (it distort the background image).

Agh!!
I'll try and find a way around it myself too!
Thanks for the quick reply man!
Adam :)

I appear to have fixed the issue, well on my email anyway.
I've been sizing the image to 600px, to fit the width of the email.
But after trying a 1920x1080 the image sizes itself to fit the container!!
Adam :)


Version issue when using with githubflow

I'm using githubflow to manage branches and tag versions.  The tag is created with I do a
git hf release finish v1.2.3
I've got a yml in my workflows folder that triggers a GitHub action when this tag is created:
name: Release
on:
  create:  # Publish release on tag creation

jobs:
  publish:
    runs-on: ubuntu-20.04
    steps:
      - uses<EMAIL_ADDRESS>        if: github.event.ref_type == 'tag'
        with:
          fetch-depth: 0
      - uses: olafurpg/setup-scala@v10
        if: github.event.ref_type == 'tag'
      - uses: olafurpg/setup-gpg@v3
        if: github.event.ref_type == 'tag'
      - run: sbt ci-release
        if: github.event.ref_type == 'tag'
        env:
          PGP_PASSPHRASE: ${{ secrets.PGP_PASSPHRASE }}
          PGP_SECRET: ${{ secrets.PGP_SECRET }}
          SONATYPE_PASSWORD: ${{ secrets.SONATYPE_PASSWORD }}
          SONATYPE_USERNAME: ${{ secrets.SONATYPE_USERNAME }}

This actually runs when its supposed to (when I finish the release) and actually publishes to Sonatype, so I must have correctly set all my tokens (yay!).  But... what I see isn't version 1.2.3.  The version I see is b4e7bd_unknown, which I presume is some mash-up of the git commit id.
How can I modify my publish script, preferably without changing when it runs, to correctly version from the created tag?

Never mind.  This is an issue with my packager for gitflow.  GitHub changed master to main, which screwed up some logic.


improvementadd the queryid to the temporary file path

Proposed changes
_temp_<table_name> to _temp_<queryid>_<table_name>.
Prevent users from having a table with the name _temp_<table_name>.
Issue #31442

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall


Multiple Image Maps

Is there some way to make multiple image maps on one page work?

was there a resolution for this??

Also interesting in solution...

anyone get an answer to this issue? Thanks!

What "doesn't work" with them?

I cannot use multiple image maps on a single page.. the maps on the 2nd instance don't appear over image and therefore are not clickable.

Anyone has figured this out? How to make it work for multiple images in a single page?

make it so that you add class to your  that has different usemap name
For ex.
my first one
my second one
and for the first tag, I use                 jQuery('.contact-background-image').rwdImageMaps();
and for the second tag, I use                jQuery('.d3-image-default').rwdImageMaps();

make it so that you add class to your img tag that has different usemap name
For ex.
my first one <img class="contact-background-image" src="" usemap="#image-map"
my second one <img class="d3-image-default" src="" usemap="#d3-image-map"
and for the first tag, I use jQuery('.contact-background-image').rwdImageMaps();
and for the second tag, I use jQuery('.d3-image-default').rwdImageMaps();

this didn't work for me


Resolve-Path -Relative does not prefix dotfiles in current directory with .\ but does so for other files

Prerequisites

[X] Write a descriptive title.
[X] Make sure you are able to repro it on the latest released version
[X] Search the existing issues.
[X] Refer to the FAQ.
[X] Refer to Differences between Windows PowerShell 5.1 and PowerShell.

Steps to reproduce
Resolve-Path -Relative on items in current directory generally prefixes them with .\, but not if they're dotfiles (i.e. begin with a period). Any reason they're treated differently in this situation?
Out-File testdot
Out-File .testdot
Resolve-Path -Relative *testdot

Expected behavior
.\.testdot
.\testdot

Actual behavior
.testdot
.\testdot

Error details
No response
Environment data
PSVersion                      7.4.1
PSEdition                      Core
GitCommitId                    7.4.1
OS                             Microsoft Windows 10.0.22631
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

Visuals
No response

This just seems like a bug.  Probably something incorrectly checking if it starts with a .


Disable "beep" sound on start and stop

Hi
is there any way to disable the beep sound on starting and stopping ?
I'm using Ionic 2 and building for Android.
I tried Media Plugin but it does not seem to be able to affect system sounds.
Thanks in advance.

You should mute the Multimedia (music) stream.
Edit "SpeechRecognition.java" and add this to the startListening method to line 157 (before initiating the recognition):
AudioManager audioManager = (AudioManager) context.getSystemService(Context.AUDIO_SERVICE);
audioManager.adjustStreamVolume(AudioManager.STREAM_MUSIC, AudioManager.ADJUST_MUTE, 0);

@ragcsalo  Hi I did this but still getting beep sound.. :(

this one works for me
AudioManager audioManager=(AudioManager)context.getSystemService(Context.AUDIO_SERVICE);
audioManager.setStreamVolume(AudioManager.STREAM_MUSIC, 0, AudioManager.FLAG_REMOVE_SOUND_AND_VIBRATE);
I advise first to get the current volume using:
audioManager.getStreamVolume(AudioManager.STREAM_MUSIC);
then set volume to 0 and after you are done with recording you can reset volume to the initial one

Hi @JackJohnG , can you please provide a full working example? Where i have to put the reset volume to the initial one?
Thanks

Hi @JackJohnG , can you please provide a full working example? Where i have to put the reset volume to the initial one?
Thanks

I mute the media volume in the onCreate function of my MainActivity, and restore the volume in the onDestroy function. I had to create the variable first (which stores the original volume level). My code is:
private Integer original_volume_level;
private AudioManager audioManager = (AudioManager) getSystemService(Context.AUDIO_SERVICE);

onCreate:
=========
original_volume_level = audioManager.getStreamVolume(AudioManager.STREAM_MUSIC);
audioManager.adjustStreamVolume(AudioManager.STREAM_MUSIC, AudioManager.ADJUST_MUTE, 0);
Toast.makeText(this, "Media volume muted.", Toast.LENGTH_SHORT).show();

onDestroy:
==========
audioManager.setStreamVolume(AudioManager.STREAM_MUSIC, original_volume_level, 0);
Toast.makeText(this, "Media volume restored.", Toast.LENGTH_SHORT).show();

I hope this helps. :-)

@texano00 I created a very small cordova plugin to control volume in android
you can just download it as zip, unzip it into the folder of your project and then add it using command
ionic cordova plugin add ./cordova-plugin-volumecontrol
Then you can use it in any place of the application like this:
window.VolumeControl.setVolume(0);
So, you just need to call the above method just before you start speech recognition and after you are done you can call:
window.VolumeControl.resetVolume();
In case you need to change some behaviour inside plugin, you can simply edit VolumeControl.java file inside plugin but then do not forget to remove old version of plugin and add the new one.
If you have any questions you can write me an email

@JackJohnG  Nice one! I tried it and setVolume is working. resetVolume doesn't seem to be working though.


THRIFT-5634: thrift docs picture was broken

[x] Did you create an Apache Jira ticket?  (not required for trivial changes)

https://issues.apache.org/jira/browse/THRIFT-5634

[x] If a ticket exists: Does your pull request title follow the pattern "THRIFT-NNNN: describe my issue"?
[x] Did you squash your changes to a single commit?  (not required, but preferred)
[x] Did you do your best to avoid breaking changes?  If one was needed, did you label the Jira ticket with "Breaking-Change"?
[ ] If your change does not involve any code, include [skip ci] anywhere in the commit message to free up build resources.

change :
https://thrift.apache.org/docs/doc/images/cgrn.png -> https://thrift.apache.org/doc/images/cgrn.png
https://thrift.apache.org/docs/doc/images/cred.png -> https://thrift.apache.org/doc/images/cred.png

i wonder what is a better fix, to change all relative paths to absolute one or to just update somewhere once to fix the docs prefix?

i wonder what is a better fix, to change all relative paths to absolute one or to just update somewhere once to fix the docs prefix?

This is determined by the location of the static resource (image).
For example:
https://thrift.apache.org/docs/Languages.html was under /docs/, so that relative assets will load with prefix  /docs/
doc/images/cgrn.png-> /docs/doc/images/cgrn.png ❌
However if we change correct relative path -> ../doc/images/cgrn.png, this will break github preview.
Move doc/images to docs/images, then add the /docs prefix to every static resource(image), maybe this is a better fix.
Change all doc will break search result of search engine.