Lossless video format (using jcodec) not supported by dl4j

I have generated labelled videos using jcodec 1.5 and 1.9, both using the standard SequenceEncoder.
SequenceEncoder encoder = new SequenceEncoder(new File());
BufferedImage bi = ImageIO.read();
encoder.encodeNativeFrame(AWTUtil.fromBufferedImage(bi));
encoder.finish.
This results in lossy videos, see screenshot below.
I also generate videos as was written here: http://stackoverflow.com/questions/11345343/png-image-files-to-video-lossless
This results in lossless videos, which are also much smaller (again, see screenshot). However, dl4j does not load these images. In jcodec 1.5 I receive codec not supported, and using jcodec 1.9 I receive the following error :
Exception in thread "main" java.lang.RuntimeException: Exception thrown in base iterator
at org.deeplearning4j.datasets.iterator.AsyncDataSetIterator.next(AsyncDataSetIterator.java:231)
at org.deeplearning4j.datasets.iterator.AsyncDataSetIterator.next(AsyncDataSetIterator.java:36)
at TrackingPredictions.VideoClassification.main(VideoClassification.java:154)
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
at java.lang.reflect.Method.invoke(Method.java:498)
at com.intellij.rt.execution.application.AppMain.main(AppMain.java:144)
Caused by: java.lang.NullPointerException
at org.jcodec.containers.mp4.demuxer.FramesMP4DemuxerTrack.getMeta(FramesMP4DemuxerTrack.java:203)
at org.jcodec.api.FrameGrab.detectKeyFrame(FrameGrab.java:211)
at org.jcodec.api.FrameGrab.decodeLeadingFrames(FrameGrab.java:188)
at org.jcodec.api.FrameGrab.(FrameGrab.java:85)
at org.datavec.codec.reader.CodecRecordReader.loadData(CodecRecordReader.java:107)
at org.datavec.codec.reader.CodecRecordReader.sequenceRecord(CodecRecordReader.java:85)
at org.deeplearning4j.datasets.datavec.SequenceRecordReaderDataSetIterator.nextMultipleSequenceReaders(SequenceRecordReaderDataSetIterator.java:200)
at org.deeplearning4j.datasets.datavec.SequenceRecordReaderDataSetIterator.next(SequenceRecordReaderDataSetIterator.java:133)
at org.deeplearning4j.datasets.datavec.SequenceRecordReaderDataSetIterator.next(SequenceRecordReaderDataSetIterator.java:115)
at org.deeplearning4j.datasets.datavec.SequenceRecordReaderDataSetIterator.next(SequenceRecordReaderDataSetIterator.java:29)
at org.deeplearning4j.datasets.iterator.AsyncDataSetIterator$IteratorRunnable.run(AsyncDataSetIterator.java:300)
at java.lang.Thread.run(Thread.java:745)
I assume that lossless videos yield in better learning performances, and that smaller videos most likely in speed performances..

Screenshot from lossy and lossless video:

Screenshot from both videos, lossy and lossless.

The plan is to switch to FFmpeg, which everyone -- including Chrome -- is using these days, but it's LPGL, so we need advice from our legal team, @agibsonccc ? :)

We can't have it as a first class module. We will have to maintain 2
versions of the software.
On Sat, Dec 3, 2016 at 9:00 AM, Samuel Audet<EMAIL_ADDRESS>wrote:

The plan is to switch to FFmpeg, which everyone -- including Chrome
https://www.chromium.org/audio-video -- is using these days, but it's
LPGL, so we need advice from our legal team, @agibsonccc
https://github.com/agibsonccc ? :)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/deeplearning4j/DataVec/issues/104#issuecomment-264595083,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABF18oPyANzILGyUP7AHjYu5m7bYE7fzks5rELEmgaJpZM4KsPJS
.

@someonedeep I've added NativeCodecRecordReader that uses FFmpeg and can decode pretty much any video file, but we need to add a dependency on JavaCV manually, for licensing reasons. For example, in the case of Maven:
  <dependency>
    <groupId>org.bytedeco</groupId>
    <artifactId>javacv-platform</artifactId>
    <version>1.3.1</version>
  </dependency>

@saudet could you add a description on this somewhere on the deeplearning4j.org website on the datavec page? Maybe create a section called video?

There's already something about that on the site: https://deeplearning4j.org/simple-image-load-transform#javacv--opencv-and-ffmpeg-filters  How do you think we should modify @tomthetrainer ?

Ahh even a link from the datavec page here woul dbe great I had forgotten about this page :D.

Hi, i tested the NativeCodecRecordReader (copy pasted BaseCodecRecordReader, NativeCodecRecordReader, CodecRecordReader to my own project, which uses also the 0.7.2 version of dl4j) and it now trains a model based on lossless videos. Thank you and keep up the excellent work.
If this is published in the new maven version, I hope to be informed.

Great! Thanks for the feedback. It will be in the next release, either 0.7.3 or more likely 0.8.0. To be informed, you can subscribe to the mailing list at the bottom on this page: https://deeplearning4j.org/

I will add a discussion about opencv and ffmpeg filters to the re-write of the data ingestion that includes scaler, transform, normalizer etc.


Upgrade to fs2-1.0.0-RC1

Needs fs2-jawn-1.0.0-RC1, which hasn't hit Central yet.  Any binary compatibility is coincidental.

Appveyor failure is spurious.


missing generated directory

In https://github.com/VasSkliris/aframev2/blob/main/projects/data/README.md there is an aframe/data/train/background directory that is not created when the commands are run

Did you create a .env file with these environment variables and run source .env?

No this is before I go to the sandbox when I build the data project and then run all instructions in readme. I created the directory and it runs fine, I just think it misses from the commands that create directories possibly?

Did you do this mkdir here? Which command does this break on?

Yes but there the command after this saves something in /aframe/data/train/background and that command does not create the background directory


WHY?!?!?!?! THIS IS PROBLEM.

https://snack.expo.io/@pixelkoduje/261028
WHY? This is official code.

What specifically makes you worry?

this package is deprecated. Use react-native-cn-quill instead.


feat:Cache DescribeClusterAttachScripts response for 6 hours

What would you like to be added:
Cache ack DescribeClusterAttachScripts response for 6 hours
Why is this needed:
Reduce unnecessary API calls.
Completion requirements:
This enhancement requires the following artifacts:

[ ] Design doc
[ ] API change
[ ] Docs update

The artifacts should be linked in subsequent comments.

I would like to take it.


Fix Seq2C for paired samples

When running on tumor/normal pairs, Seq2C's cohort calling function returns a cut sample list containing only tumors, leading to skipping normals in downstream steps, ending with no normals infinal. This PR fixes that.

Vlad -- thanks for catching this and for the quick fix.


Sync with upstream

The last time I updated this repo was on Nov 2018. This branch is 621 commits behind upstream. So, embrace yourself for this huge update.

LGTM


pachctl deploy --dashboard-only shouldn't require arguments

Is there any particular reason why pachctl deploy --dashboard-only requires arguments?  Seems strange that I had to hunt down the arguments I used to deploy the cluster like bucket names just to deploy the dashboard.

stale


[BACKLOG-6425] Tweak Run Dialogs across operating systems - Linux

@bmorrise Please Sir!

Build Commands:
Build:

ant -Dtestreports.dir=bin/reports/unit-test -Djunit.jvmargs=-XX:PermSize=256m -f ui/build.xml clean-all resolve jacoco && ant -Dpentaho.coding.standards.version=1.0.4 -Dtestreports.dir=bin/reports/integration-test -Djunit.jvmargs=-XX:PermSize=256m -f ui/build.xml jacoco-integration checkstyle publish-local

Cleanup:
rm -r ~/.ivy2/local || echo no publish local to remove
Changed files:
ui/src/org/pentaho/di/ui/core/dialog/ConfigurationDialog.java
ui/src/org/pentaho/di/ui/job/dialog/JobExecutionConfigurationDialog.java
ui/src/org/pentaho/di/ui/trans/dialog/TransExecutionConfigurationDialog.java
Checkstyle violations:
See: https://github.com/pentaho/pentaho-coding-standards
File /home/buildguy/project/ui/src/org/pentaho/di/ui/core/dialog/ConfigurationDialog.java

Message: 'if' is not followed by whitespace. at Line: 410 Column: 7
Message: Line has trailing spaces at Line: 413
Message: File contains tab characters (this is the first instance). at Line: 413 Column: 60
Message: 'if' is not followed by whitespace. at Line: 415 Column: 7

File /home/buildguy/project/ui/src/org/pentaho/di/ui/job/dialog/JobExecutionConfigurationDialog.java

Message: Line has trailing spaces at Line: 209
Message: Line has trailing spaces at Line: 212

File /home/buildguy/project/ui/src/org/pentaho/di/ui/trans/dialog/TransExecutionConfigurationDialog.java

Message: Line has trailing spaces at Line: 228
Message: Line has trailing spaces at Line: 231
Message: Line has trailing spaces at Line: 234

Build Commands:
Build:

ant -Dtestreports.dir=bin/reports/unit-test -Djunit.jvmargs=-XX:PermSize=256m -f ui/build.xml clean-all resolve jacoco && ant -Dpentaho.coding.standards.version=1.0.4 -Dtestreports.dir=bin/reports/integration-test -Djunit.jvmargs=-XX:PermSize=256m -f ui/build.xml jacoco-integration checkstyle publish-local

Cleanup:
rm -r ~/.ivy2/local || echo no publish local to remove
Changed files:
ui/src/org/pentaho/di/ui/core/dialog/ConfigurationDialog.java
ui/src/org/pentaho/di/ui/job/dialog/JobExecutionConfigurationDialog.java
ui/src/org/pentaho/di/ui/trans/dialog/TransExecutionConfigurationDialog.java

@bmorrise All clear... Please Sir!


smtp.connect() error

As while running the code ,
always the smtp connection error arises while making the tls connection with for eg gmail smtp server and port number 587
i  also tried to change the smtp port to 25 and 465 but nothing happened ...
Dont know why this error is arising :
smtplib.SMTPConnectError: (421, 'Cannot connect to SMTP server <IP_ADDRESS> (<IP_ADDRESS>:587), connect error 10060')

Have you tried making an app password within your Google account settings?
Cody Kaczynski
www.CodyKaczynski.com
On Dec 21, 2018, 10:54 AM -0500, Sarthak Gupta<EMAIL_ADDRESS>wrote:

As while running the code ,
always the smtp connection error arises while making the tls connection with for eg gmail smtp server and port number 587
i also tried to change the smtp port to 25 and 465 but nothing happened ...
Dont know why this error is arising :
smtplib.SMTPConnectError: (421, 'Cannot connect to SMTP server <IP_ADDRESS> (<IP_ADDRESS>:587), connect error 10060')
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

This is what is being talked about. You need to go into your google setting and toggle this setting. Then this application works.


ci: remove azureboards workflow

PR Checklist

[ ] Unit Tests have been added for new changes
[ ] API tests have been updated if applicable
[ ] All commented code has been removed
[ ] If you've added a dependency, you've ensured license is compatible with Apache 2.0 and clearly outlined the added dependency.

What are you changing?

Anything the reviewer should know when reviewing this PR?
If the there are associated PRs in other repositories, please link them here (i.e. open-amt-cloud-toolkit/repo#365 )

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

GitHub release
v2.3.7

Your semantic-release bot :package::rocket:


Strategy simulation in onPeriod do not take callback into account since latest refactoring

All talib strategies and indicators need a callback as calculations are async. therefore we moved all the calculation from calculate to onPeriod as this provides a callback. But onPeriod is not using the callback in simulations in the right way since the latests refactorings in engine code.
If we run a simulation 14 days it stops already before. you can test this behavior with the following code. if you higher sleep time simulator stops more earlier. it looks like some code snippet is not waiting for the callback
    function sleep (time) {
      return new Promise((resolve) => setTimeout(resolve, time));
    }

onPeriod: function (s, cb) {
    Promise.all([sleep(500)]).then(function() {
      console.log("aaa")
      cb()
    })
}

Looks like monodb data elements are now streamed and not collected. maybe @krystophv can help
https://github.com/DeviaVir/zenbot/commit/607a72c#diff-0b0ccd6820a6dc8b5de6b7c6868c126aR235
tradesCollection.find(opts.query).sort(opts.sort).limit(opts.limit).toArray(function (err, trades) {

        collectionCursor.on('data', function(trade){
          lastTrade = trade
          numTrades++
          if (so.symmetrical && reversing) {
            trade.orig_time = trade.time
            trade.time = reverse_point + (reverse_point - trade.time)
          }
          eventBus.emit('trade', trade)
        })

Ah, yeah - I can look into this. It's not directly related to the code chunks you've pointed out, but I think it's an effect of that refactor. It's likely that the sim is exiting before the internal processing queue has finished emptying. I'll see about adjusting the exit functions to wait for an empty queue.


Let Setting support additional description

As of 2.0.2 the Setting class holds a simple description and a value. It would be great if an additional description could be added to a particular Setting, see attached screenshot from IntelliJ

100% agree on this. We discussed this already before making PreferencesFX public and due to the limited time we had, we decided to not implement it in favor of other, more important features.
Feel free to implement it and submit a PR!

Very well, I'd suggest an API similar to what's provided at https://github.com/dlemmermann/formsfx/pull/21, that is, description should accept String or Node as arguments. If it's a String, then it'll be wrapped with a Label.

It makes sense for https://github.com/dlemmermann/formsfx/pull/21 to be reviewed and hopefully merged, then wait for the next release of FormsFX as Setting uses the Field API as is.

I think since description is an internal name anyways and refactoring that would not result in any signature changes, resulting in only non-breaking API changes I would even propose to refactor description to title or name and then you could use description, as it would make more sense. What do you think?

Thats somethink I ask me for myself. Why are the parameters in Setting, Group and Category spelled description when in the JavaDoc they promounced as name, title...
One possibility is that the text in the label describes the functionality from the component on the right side, but why isn't that terminology used in the JavaDoc?

@Naoghuman like I mentioned, the name description may not be ideal. As proposed in my previous comment, I would happily merge a PR that implements such a change if it benefits the readability.


Revert "mobile: Java using C++ builder under junit and testing equiva…

…lence (#25297)"
This reverts commit 72cbe0e8d6c398497d5d31e91114d05bda584e27.

Commit Message:
Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]

/retest

Or preferably fixing forward with https://github.com/envoyproxy/envoy/pull/25395


Mount an Individual View Component

Is it possible to mount an individual component?  If not, would it be a reasonable feature request?
My current approach is to unmount all components on the page and then mount them again - but, this resets the state of all components on the page - and I even get inconsistent results when I expect all components to reset.
Consider adding an instance of an Item component to a page through an ajax request:
document.getElementById('items')
  .insertAdjacentHTML('beforeend', '<%= j render partial: 'items/item', locals: { item: @item } %>');

WebpackerReact.unmountComponents()
WebpackerReact.mountComponents()

Here's a change to the webpacker-react-example that shows the full request/response in action.
https://github.com/renchap/webpacker-react-example/compare/master...sealocal:add-item-with-remote-request
Please share thoughts on mounting a single component on the page.

This should definitely be doable. I will work on implementing this.


emit Genomics API counters to the Dataflow UI

An Aggregator example can be found here: http://stackoverflow.com/questions/28119993/how-do-i-create-user-defined-counters-in-dataflow
Also look for opportunities to push the collection of these metrics further down the stack into utils-java so that the counters can be shared by both Spark and Dataflow.
Here's the current spark implementation:
https://github.com/googlegenomics/spark-examples/blob/master/src/main/scala/com/google/cloud/genomics/spark/examples/rdd/VariantsRDD.scala#L192

This was fixed with 1ad45fc982e55052df90f2a5f38430569d50a591

Cool upgrade to the previous counts that used to be in utils.


ld: framework not found webview_flutter

Im using the latest version of the webview_flutter android devices are working fine when i try to run ios simulator its shows error like
ld: framework not found webview_flutter
clang: error: linker command failed with exit code 1 (use -v to see invocation)
Please help me to solve this last two days checking the solution all over the internet...

Hi @sefin92, Thanks for filing the issue. please share the output of flutter doctor -v  and the complete flutter run -v logs in a text file as the output will be long.
Thanks


Revert breaking changes in casper-hashing

Fixes #3071
This PR reverts the breaking changes in casper-hashing since the v1.4.6 release.
Changes were found using the cargo-public-api tool.

bors r+


facial detection

[Enter feedback here]

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

ID: 735bd5b5-ae11-5bbc-6fcb-26aa8b3f022d
Version Independent ID: 3ab0fb5d-faba-2380-967a-13d4ae3afde0
Content: Face detection and attributes concepts - Azure Cognitive Services
Content Source: articles/cognitive-services/Face/concepts/face-detection.md
Service: cognitive-services
Sub-service: face-api
GitHub Login: @PatrickFarley
Microsoft Alias: pafarley

@blocboi512 Could you please add more details about the question. Thank you for your understanding.

@blocboi512 Could you please add more details about the question. Thank you for your understanding.

@blocboi512
We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.

@blocboi512
We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


Large number of transactions stay at "Not verified" even when network status is green

Last verified transaction is from 3 days ago. Appears that something isn't syncing the verification properly.

From the website, deleting %AppData%/Garlium/blockchain_headers and restarting apparently solves many problems.


Experimental: Split Operator::BrTable into Operator::{BrTableStart, BrTableTarget}

Implements https://github.com/bytecodealliance/wasm-tools/issues/182.
This PR does not serve as a proposal for the concrete implementation but rather as a place for discussion of #182 .
Tests are failing because I did not yet update the other crates in this repository that are depending on wasmparser, e.g.:
 error[E0107]: wrong number of lifetime arguments: expected 0, found 1
   --> crates/wasmprinter/src/lib.rs:701:48
    |
701 |     fn print_operator(&mut self, op: &Operator<'_>, nesting_start: u32) -> Result<()> {
    |                                                ^^ unexpected lifetime argument

CI dislikes formatting because I wanted to keep the diff of this PR at a bare minimum and formatting would have touched lots of places that are not interesting to this PR apparently.

Thanks for the PR! I've often though that it'd be great to avoiding having to skip ahead in various places inside of wasmparser, although this happens in a lot of places currently, not just in br_table parsing.
The addition of the state in BinaryReader feels a bit odd to me though. One possible fix for this, however, is to avoid updating  BinaryReader itself too much and instead document a contract that once BrTableStart is seen then read_var_u32 needs to be called the specified number of times, plus one, in succession. That could be automatically handled by higher-level operator iterators, but it would make BinaryReader avoid having any extra state in it.

Thanks for the PR! I've often though that it'd be great to avoiding having to skip ahead in various places inside of wasmparser, although this happens in a lot of places currently, not just in br_table parsing.
The addition of the state in BinaryReader feels a bit odd to me though. One possible fix for this, however, is to avoid updating  BinaryReader itself too much and instead document a contract that once BrTableStart is seen then read_var_u32 needs to be called the specified number of times, plus one, in succession. That could be automatically handled by higher-level operator iterators, but it would make BinaryReader avoid having any extra state in it.

Thanks for the PR! I've often though that it'd be great to avoiding having to skip ahead in various places inside of wasmparser, although this happens in a lot of places currently, not just in br_table parsing.
The addition of the state in BinaryReader feels a bit odd to me though. One possible fix for this, however, is to avoid updating BinaryReader itself too much and instead document a contract that once BrTableStart is seen then read_var_u32 needs to be called the specified number of times, plus one, in succession. That could be automatically handled by higher-level operator iterators, but it would make BinaryReader avoid having any extra state in it.

Thanks for the reply and for the new idea about how else we could go forward.
I am mainly interested in starting a discussion about this and whether it is needed at all.
The new state in BinaryReader indeed feels wrong but it is fine to get a feeling for the PoC - wouldn't merge it either in the current state tbh.
If you want we could experiment a bit with your proposal and see how it fleshes out.
However, is this generally something you anticipate or do you regard this not very important?

Thanks for the PR! I've often though that it'd be great to avoiding having to skip ahead in various places inside of wasmparser, although this happens in a lot of places currently, not just in br_table parsing.
The addition of the state in BinaryReader feels a bit odd to me though. One possible fix for this, however, is to avoid updating BinaryReader itself too much and instead document a contract that once BrTableStart is seen then read_var_u32 needs to be called the specified number of times, plus one, in succession. That could be automatically handled by higher-level operator iterators, but it would make BinaryReader avoid having any extra state in it.

Thanks for the reply and for the new idea about how else we could go forward.
I am mainly interested in starting a discussion about this and whether it is needed at all.
The new state in BinaryReader indeed feels wrong but it is fine to get a feeling for the PoC - wouldn't merge it either in the current state tbh.
If you want we could experiment a bit with your proposal and see how it fleshes out.
However, is this generally something you anticipate or do you regard this not very important?

I don't feel like I have a great compass for whether this is worthwhile or not. I don't know how slow the skipping of varuint32 actually is. There's a good number of places in wasmparser that "parse" things twice either by literally parsing or skipping bytes, and I'm not sure whether it's actually costly or how it factors "in the large". It might be reasonable to try to see how this affects microbenchmarks for reasonably-sized wasm modules.

I don't feel like I have a great compass for whether this is worthwhile or not. I don't know how slow the skipping of varuint32 actually is. There's a good number of places in wasmparser that "parse" things twice either by literally parsing or skipping bytes, and I'm not sure whether it's actually costly or how it factors "in the large". It might be reasonable to try to see how this affects microbenchmarks for reasonably-sized wasm modules.

I don't feel like I have a great compass for whether this is worthwhile or not. I don't know how slow the skipping of varuint32 actually is. There's a good number of places in wasmparser that "parse" things twice either by literally parsing or skipping bytes, and I'm not sure whether it's actually costly or how it factors "in the large". It might be reasonable to try to see how this affects microbenchmarks for reasonably-sized wasm modules.

Fair point and sorry if I falsely expected you to provide me with a compass here! :sweat_smile:
For me the more important part about this change is what I describe in this thread under the "What does this solve?" and "Streamlined Design" approach.
Can you point me to some other places where we currently have double parsing?

I don't feel like I have a great compass for whether this is worthwhile or not. I don't know how slow the skipping of varuint32 actually is. There's a good number of places in wasmparser that "parse" things twice either by literally parsing or skipping bytes, and I'm not sure whether it's actually costly or how it factors "in the large". It might be reasonable to try to see how this affects microbenchmarks for reasonably-sized wasm modules.

Fair point and sorry if I falsely expected you to provide me with a compass here! :sweat_smile:
For me the more important part about this change is what I describe in this thread under the "What does this solve?" and "Streamlined Design" approach.
Can you point me to some other places where we currently have double parsing?

That's all very interesting information, thanks!
Before you refactored the entire crate with the genius Payload construct I was trying to decouple the validation of the crate using traits that still exist in some way today.
The idea was to have a Parser<T> type that is generic over some T: Validator where Validator is a trait used by the parser to feed and query the validation process. If you only want to parse without validation you'd use something like Parser<NoopValidator>. The idea behind this is that oftentimes information required by backends is very similar to what a validator requires and therefore it oftentimes is possible to use the backend as the resource storage for the validator reducing memory footprint and eventually getting rid of double-parsing of the source.

That's all very interesting information, thanks!
Before you refactored the entire crate with the genius Payload construct I was trying to decouple the validation of the crate using traits that still exist in some way today.
The idea was to have a Parser<T> type that is generic over some T: Validator where Validator is a trait used by the parser to feed and query the validation process. If you only want to parse without validation you'd use something like Parser<NoopValidator>. The idea behind this is that oftentimes information required by backends is very similar to what a validator requires and therefore it oftentimes is possible to use the backend as the resource storage for the validator reducing memory footprint and eventually getting rid of double-parsing of the source.

Yeah I think that'd work, and another possible alternative I've thought of is to have more specialized methods on Validator for "here's the import I just parsed", so the application drives the iteration and then feeds items into the validator, sort of like how the opcode validation currently works.
I haven't worried too too much about all this though yet becaue the code section is almost always the vast majority of the time spent when validating, but it's always useful to plan to fix these sorts of things in the future!

Yeah I think that'd work, and another possible alternative I've thought of is to have more specialized methods on Validator for "here's the import I just parsed", so the application drives the iteration and then feeds items into the validator, sort of like how the opcode validation currently works.
I haven't worried too too much about all this though yet becaue the code section is almost always the vast majority of the time spent when validating, but it's always useful to plan to fix these sorts of things in the future!

Closing this since the PR has been stale for quite some time and would require a load of work to get working again. Besides that the most recent changes to the wasmparser crate have made the changes introduced in this PR less valuable. If the maintainers wish to proceed with this approach we can re-open or open another PR.


Add a wiki: Anime Last Stand Wiki

An anonymous user has submitted a new wiki via our submission form.
Link to origin wiki: https://alsroblox.fandom.com
Link to destination (independent) wiki: https://wiki.animelaststand.com/

Destination wiki only has 4 articles vs. > 100 on Fandom. The official website also links to the Fandom wiki (despite the destination wiki being on the same domain). Maybe a work in progress.


Can't render adaptive cards

I'm having this issue with the latest version (4.0.15 alpha). The adaptive cards work on final channels (Slack, Teams, etc) but the emulator can't render the cards itself.
Emulator | Teams
AdaptiveCard card = new AdaptiveCard();
card.Body = new List<AdaptiveElement>()
{
    new AdaptiveTextBlock()
    {
        Text = "Lorem ipsun"
    },
    new AdaptiveDateInput()
    {
        Value = DateTime.Now.ToShortDateString()
    },
    new AdaptiveTimeInput()
    {
        Value = DateTime.Now.ToShortTimeString()
    }
};

var msg = context.MakeMessage();
msg.Attachments.Add(
    new Attachment()
    {
        Content = card,
        ContentType = "application/vnd.microsoft.card.adaptive",
        Name = "Lorem Ipsun"
    }
);

await context.PostAsync(msg);
context.Wait(this.MessageReceived);

Issues read: #244 and #286

Thank you for the submission. There were a number of issues related to adaptive cards that were closed recently  in the Botframework-WebChat repo. We have incorporated most of these changes into the v4 branch which should go out with the next build.

Leaving this open until the fix is confirmed.

Thanks @dkonyayev! Now it works.
I also didn't knew about this "Developer Mode" (pretty much the chrome F12 overlay), so thanks again!
Now related to the issue: is this "All inputs must have a unique Id" restriction valid only for the Emulator? I mean, the cards were rendering on Teams and Slack without this change.
Should they display the card, even with the missing Ids? Or is the Emulator's "can't render card" message the right approach?


Fixes #253

@bledson please use symfony-5 branch for Symfony 5 support as mentioned in the docs.
$ composer require ankitpokhrel/tus-php:dev-symfony-5


Make basic scoring

[x] Make score update on screen
[x] Make pressing bowl button randomly choose a score
[x] Make so that each time you bowl the new score is added to previous score

Closed by PR #10


Partner Tier 1 Support and Partner Tier 2 Support

Hi,
I'm wondering what is the scope about these two accounts?
The page reports:
Do not use - not intended for general use
Many customers are wondering why the page reports them
The question are:

why have this information if we cannot see them in Azure AD subscription?
what is the scope behind this detail if they are only as "Do not use - not intended for general use."?
does it make sense to maintain them? I believe it could create more confusion to the reader unless there is a reason we are not able to understand. In this last scenario, maybe it could make sense to explain better the information reason.

Thanks
Alex

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

ID: 0767aac4-fcd9-eaac-2c30-8f17d8eadd92
Version Independent ID: a380a828-03f9-e037-3a62-e4165fda7cb6
Content: Azure AD built-in roles - Azure Active Directory
Content Source: articles/active-directory/roles/permissions-reference.md
Service: active-directory
Sub-service: roles
GitHub Login: @rolyon
Microsoft Alias: rolyon

@Gassela Thank you for your inputs. These two roles are intended to be used only by some small number of legacy resale partners which is the reason it is added to the document but not useful for everyone. Only useful for the legacy partners who were using this when it was introduced earlier. No new members can use it and hence it has been removed from the portal and it does not show there. These roles are going to be deprecated in future and hence the warning to not use them on the page.
However you are correct that it can confuse the reader so we will add some info there and request the content team to approve the same.
Thank you.

We are working on this internally with product group and will be updating the article as per internal discussion.


Display Empty State Message When No Facility Group is Associated with Store Pickup Group Type

Description
Currently, when no facility group is associated with the store pickup group type, the facilities are still visible in the facilities section, but the toggle button to enable/disable store pickup for those facilities is not present. This can cause confusion for users. To improve user experience, we should implement an enhancement where no facilities are shown in such cases, and an empty state message is displayed to notify the user.

Upon thorough review, I have determined that the message "No store pickup group found linked with current product store" is more appropriate than simply displaying facilities without a toggle button. This message clearly communicates to the user (retailer) that there is no association between the facility group pickup and the current product store, providing clarity and preventing potential confusion.
Upon further inspection, I identified two view entities responsible for retrieving the relevant data: ProductStoreFacilityDetail and ProductStoreAndFacilityGroup.
The ProductStoreFacilityDetail view entity ensures that facility details linked to the current product store are fetched. Meanwhile, ProductStoreAndFacilityGroup is responsible for linking the product store to the facility group.
In scenarios where a card of facilities is displayed but no toggle is available, it indicates that ProductStoreAndFacilityGroup has not retrieved any data. This implies that the current product store is not associated with any facility group pickup, which further justifies the display of the appropriate message.

@dongreparam have you done QA too for this ticket?

@dongreparam have you done QA too before closing this ticket?
Yes sir, I have thoroughly reviewed all the test cases and ensured that the issue is resolved across all scenarios.


AndEngine Truck Factor

As part of my PhD research on code authorship, we calculated the Truck Factor (TF) of some popular GitHub repositories.
As you probably know, the Truck (or Bus) Factor designates the minimal number of developers that have to be hit by a truck (or quit) before a project is incapacitated. In our work, we consider that a system is in trouble if more than 50% of its files become orphan (i.e., without a main author).
More details on our work in this preprint: https://peerj.com/preprints/1233
We calculated the TF for AndEngine and obtained a value of 1.
The developer responsible for this TF is:
Nicolas Gramlich - author of 100% of the files
To validate our results, we would like to ask AndEngine developers the following three brief questions:
(a) Do you agree that the listed developer is the main developer of AndEngine?
(b) Do you agree that AndEngine will be in trouble if the listed developer leave the project (e.g., if he wins in the lottery, to be less morbid)?
(c) Does AndEngine have some characteristics that would attenuate the loss of the listed developer (e.g., detailed documentation)?
Thanks in advance for your collaboration,
Guilherme Avelino
PhD Student
Applied Software Engineering Group (ASERG)
UFMG, Brazil
http://aserg.labsoft.dcc.ufmg.br/

(a) @nicolasgramlich is the owner of the repository.
(b) AndEngine is in trouble because @nicolasgramlich did not respond to pull request/issues for more than three years now.
(c) No, the project is simply missing a maintainer - without maintainer detailed documentation doesn't help.

Hi @winniehell, thank you for your answer.


TRG 5.09 Helm test and upgrade workflow steps missing

Our TRG 5.009 describes workflow steps that will test the default config of our helm charts and also verifies if old Chart releases can successfully be upgraded to the current Chart version.
This workflow should run automatically, but should also have a workflow_dispatch trigger to run it on demand.
The upgrade part and the workflow_dispatch trigger are completely missing

Final fixes done in #57


Usage Question: Create private/public key pair with Ed25519 using SHA3-512 instead of SHA512 (standard)

I'm struggling with finding a sample to do the following: I would like to create a private/public key pair with the Ed25519 public-key signature system. But unlike the standard Ed25519 implementation which uses SHA512 hash function, I would like to use the SHA3-512 hash function. Some help would be very much appreciated. Thanks a lot!

First of all, I must apologize for just replying now as I am just seeing this issue now.
GitHub unfortunately didn't notify me that an issue was created.
I have done something similar to what you want but for Blake2B.
Links can be found below
KeyPair Generator
Other relevant links
https://github.com/Xor-el/CryptoLib4Pascal/blob/master/CryptoLib/src/Math/EC/Rfc8032/ClpEd25519Blake2B.pas
https://github.com/Xor-el/CryptoLib4Pascal/blob/master/CryptoLib/src/Crypto/Signers/ClpEd25519Blake2BSigner.pas
using the above links as guidelines, you can create your own variant which uses SHA3-512.
do feel free to reach out if you encounter any other issues along the way.
Regards.

Hello
Thank you very much for your answer!
Hhm, unfortunately I did not succeed :-(
For some quick tests I'm save to "misuse" your Blake2B-512
implementation, and just replace the digest in ClpEd25519Blake2B.pas,
right?
Replace
result := TDigestUtilities.GetDigest('BLAKE2B-512');
with
result := TDigestUtilities.GetDigest('SHA3-512');
In my test code I'm assigning a private key, and want to generate the
public key:
LSk :=
DecodeHex('4ba47b6ace2abe73ef593c3cf32a999fb57349a6038bc334d3b50edff8bd33f6');
PrivateKey := TEd25519Blake2BPrivateKeyParameters.Create(LSk, 0);
PublicKey := PrivateKey.GeneratePublicKey;
Private Key: 4ba47b6ace2abe73ef593c3cf32a999fb57349a6038bc334d3b50edff8bd33f6
Expected Public Key:
576192500b07f5c78482d05cb17bfcde61fcd819e447447ada0f7ad149928bf2
Generated Public Key:
498a4762482840fa5cb1e6614a9ff3624625dac26df7b5250a2d3d9f9e10fc9b
So something went wrong.
When doing the same with SHA-512 it works:
Private Key: 9d61b19deffd5a60ba844af492ec2cc44449c5697b326919703bac031cae7f60
Expected Public Key:
d75a980182b10ab7d54bfed3c964073a0ee172f3daa62325af021a68f707511a
Generated Public Key:
d75a980182b10ab7d54bfed3c964073a0ee172f3daa62325af021a68f707511a
When doing the same with Blake2B-512 it works as well:
Private Key: 9d61b19deffd5a60ba844af492ec2cc44449c5697b326919703bac031cae7f60
Expected Public Key:
78e65bf30f893d32fc57ef051c341bdede242544fc2a2112f0fa2c7afdebc02f
Generated Public Key:
78e65bf30f893d32fc57ef051c341bdede242544fc2a2112f0fa2c7afdebc02f
The reason I want to do this: I would like to create a NEM address
from scratch. NEM is a cryptocurrency platform like Ethereum with its
native currency, XEM (like Ether is to Ethereum). I'm pretty sure that
the expected public key is correct. Becasue when importing the
corresponding private key into the NEM Nano Wallet I get the correct
address, and the public key is part of the address calculation.
Thank you very much for any help, hints etc.
Best regards
Guido

Quoting Ugochukwu Mmaduekwe<EMAIL_ADDRESS>
First of all, I must apologize for just replying now as I am just
seeing this issue now.
GitHub unfortunately didn't notify me that an issue was created.
I have done something similar to what you want but for Blake2B.
Links can be found below
KeyPair Generator[1]
Other relevant links
https://github.com/Xor-el/CryptoLib4Pascal/blob/master/CryptoLib/src/Math/EC/Rfc8032/ClpEd25519Blake2B.pas
https://github.com/Xor-el/CryptoLib4Pascal/blob/master/CryptoLib/src/Crypto/Signers/ClpEd25519Blake2BSigner.pas
using the above links as guidelines, you can create your own
variant which uses SHA3-512.
do feel free to reach out if you encounter any other issues along the way.
Regards.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub[2], or unsubscribe[3].

Links:
[1]
https://github.com/Xor-el/CryptoLib4Pascal/blob/master/CryptoLib/src/Crypto/Generators/ClpEd25519Blake2BKeyPairGenerator.pas
[2]
https://github.com/Xor-el/CryptoLib4Pascal/issues/9?email_source=notifications&email_token=ANXESRSFXEAWJKOUV43TAILQU7YVJA5CNFSM4JLFIRX2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEE54ZPA#issuecomment-557567164
[3]
https://github.com/notifications/unsubscribe-auth/ANXESRQBMYCKX2R4P2BMFY3QU7YVJANCNFSM4JLFIRXQ

Hello, strangely again, GitHub didn't notify me of your update.
Got to see it by chance.
To answer your question, Nem used SHA3-512 before it was standardized by NIST which essentially means that Nem uses KECCAK-512.
So the hash you need is
result := TDigestUtilities.GetDigest('KECCAK-512');

Hello again. No worries! And thanks a lot for checking from time to time.
I forgot to mention that I also tested with KECCAK-512. But unfortunately this didn't work out either :-(
Private Key: 4ba47b6ace2abe73ef593c3cf32a999fb57349a6038bc334d3b50edff8bd33f6
Expected Public Key: 576192500b07f5c78482d05cb17bfcde61fcd819e447447ada0f7ad149928bf2
Generated Public Key: 473c6df8dbbbde579b700323cd4f3d0df868573868bf1d9a53c30a2056d0ff5a
Hhmmm... I was pretty sure that with NEM the only difference to the standard Ed25519 implementation is the hash function. But maybe there is more...
Again, thank you very much for your support so far.
Best regards
Guido

Hello again. No worries! And thanks a lot for checking from time to time.
I forgot to mention that I also tested with KECCAK-512. But unfortunately this didn't work out either :-(
Private Key: 4ba47b6ace2abe73ef593c3cf32a999fb57349a6038bc334d3b50edff8bd33f6
Expected Public Key: 576192500b07f5c78482d05cb17bfcde61fcd819e447447ada0f7ad149928bf2
Generated Public Key: 473c6df8dbbbde579b700323cd4f3d0df868573868bf1d9a53c30a2056d0ff5a
Hhmmm... I was pretty sure that with NEM the only difference to the standard Ed25519 implementation is the hash function. But maybe there is more...
Again, thank you very much for your support so far.
Best regards
Guido

Hi, so I did some research about Nem and discovered that the private key is first reversed (as a byte array) before passed to the ED25519 method.
The code below gives me 576192500b07f5c78482d05cb17bfcde61fcd819e447447ada0f7ad149928bf2 as my public key.
var
  sk, skreversed: TBytes;
  privateKey: IEd25519Blake2BPrivateKeyParameters;
  publicKey: IEd25519Blake2BPublicKeyParameters;
  pk: string;
  i: Int32;
begin
  sk := DecodeHex
    (('4ba47b6ace2abe73ef593c3cf32a999fb57349a6038bc334d3b50edff8bd33f6'));
  System.SetLength(skreversed, length(sk));
  for i := Low(sk) to High(sk) do
  begin
    skreversed[High(sk) - i] := sk[i];
  end;

  privateKey := TEd25519Blake2BPrivateKeyParameters.Create(skreversed, 0);
  publicKey := privateKey.GeneratePublicKey;
  pk := EncodeHex(publicKey.GetEncoded);
end;

Hello again
Wow! Thanks a lot! Works like a charm!
Donation is on the way...
Best regards
Guido


ensime-vim: refactor and add more features

Start to split the user messages (feedback) and the config from the
logic

Use contextmanagers to better code abstraction

Document the most important bits of the logic

Split logic into internal functions to make their goal clear

Ensime extends object (pythonic good practice since 2.2)

Optimize and simplify conditionals

Add handler for ArrowTypeInfo to display types of functions

Add EnToggleFullType to enable to display either full types or
shallow types

Use decorator to remove boilerplate code in the neovim plugin

Add high-level library for threads threading instead of the
deprecated thread module

Make code more readable and mantainable

Fixed issue when starting client

Migrating to a more user-friendly structure for the classpath
configuration in the .config folder.

Allow both vim and neovim to share the sources of the plugin, hence
removing boilerplate and inconvenience when editing

Ensime-shared has been improved considerably and now contains most of
the logic of the ensime-vim plugin

My credits included in the ensime-shared python module

Improve vim documentation of the ensime-vim plugin (and README)

Reorder appearance of functions to better readability

Make EnDeclaration go to char correctly

Add refactoring rename in the client side although a lot of
improvements must be done in the ensime server before getting all the
goodies of this functionality

Add up-to-date gif demos

I have decided to PR on master because this has been extensively tested. Please, if you find any error let me know or fix it by yourself! :dancer:

Just curious what the status of this PR is?

I would suggest breaking this into two PRs to make it easier to see what changes you have added.

Sorry if changes are not clear (they can be found under ensime_shared) but I cannot split this again and support vim as well. As I said before, this was meant to be more of a general refactoring than several single changes. On the way (and because I needed it), I implemented EnRename and some more features.

Thanks @jvican, I tend to agree with @vito-c , splitting this would help.

The only remarkable thing that I did is to move the logic to ensime_shared (ensime.py file). I understand that it will help... but I don't have enough time for this in the near future and I would struggle a lot to make all the changes that I did.

Fair enough. I'll start by testing working with your branch, if anyone can do the same and give feedback, this would be nice.

Here is what I found using Neovim 0.1.1:
Got this error when using omni completion.
Error detected while processing function EnCompleteFunc:
line    1:
KeyError('filetype',)

Nice to have EnType working with functions, but it was pretty easy to make it hang in such a way that I had to restart Neovim.

Just realized I didn't clear my .ensime_cache directory prior to testing, so I'll need to retest after doing that.

Still seeing the same issues.

Yes, as I said before, I am fixing the issues with omnicompletion. Let's see what is the problem with EnType.

What's the status of this PR? ensime-vim could really use some better type information.

I am trying to close this today.

@jvican for future reference you should make your commits more incremental then they will be easier to break out if needed :)

I usually do @vito-c even though I rewrited the git history because some people like only one commit on PRs.

Ok, just tell me when you're good to go for mergin.

After some hours I didn't find any bug. If you do, please tell me.
There is a misbehaviour of the vim omni completion that causes to insert the entire suggested word and doesn't take into account the previous substring. For example, if we request a completion for Execu we'll get ExecuExecutionContext instead of ExecutionContext. I don't know how to solve this and I think it's not related to ensime-vim but the backend that uses it.
@yazgoo It's your turn to modify the docs. Feel free to remove any funny comment, I am quite prone to them... :laughing:

what's up with the build? This doen't look healthy...
$ python spec/ensime.py
Traceback (most recent call last):
  File "spec/ensime.py", line 21, in <module>
    import ensime_launcher
ImportError: No module named ensime_launcher

That was supposed to be set up to execute tests by @yazgoo and I think it's not finished and he got some problems. I already gave my opinion on the future testing of the ensime-plugin at https://github.com/ensime/ensime-vim/issues/136.

ok cool, so long as we have tests :smile:
thanks for doing all this work! It looks like you're really moving the vim support along.
Can I ask that you put the docs on http://ensime.github.io/editors/vim/ and make the README a short note like https://github.com/ensime/ensime-emacs/blob/master/README.md
We're consolidating all the documentation into one site.

Sure! I will do it in the next week as I am moving out in a few days and will be a little bit busy with the rapture stuff and my personal issues :smiley_cat:

BTW, last commit deletes AUTHORS file as well by request of @fommil, it's better to point out to the github graph here.

Any news on this?

@jvican are you ok for merge ?

Yep, everything's fine.

Now, you'll have to polish things and get rid of the parts of the docs you don't like. Feel free :+1:

Awesome,
Thanks for your contribution!


Copy to static site instead of making blob public

PR Summary

PR Context

PR Checklist

[x] PR has a meaningful title

Use the present tense and imperative mood when describing your changes

[x] Summarized changes
[x] Make sure all .h, .cpp, .cs, .ps1 and .psm1 files have the correct copyright header
[ ] This PR is ready to merge and is not Work in Progress.

If the PR is work in progress, please add the prefix WIP: or [ WIP ] to the beginning of the title (the WIP bot will keep its status check at Pending while the prefix is present) and remove the prefix when the PR is ready.

Breaking changes

[x] None
OR
[ ] Experimental feature(s) needed

[ ] Experimental feature name(s):

User-facing changes

[x] Not Applicable
OR
[ ] Documentation needed
- [ ] Issue filed:

Testing - New and feature

[x] N/A or can only be tested interactively
OR
[ ] Make sure you've added a new test if existing tests do not effectively test the code changed

Tooling

[x] I have considered the user experience from a tooling perspective and don't believe tooling will be impacted.
OR
[ ] I have considered the user experience from a tooling perspective and opened an issue in the relevant tool repository. This may include:

[ ] Impact on PowerShell Editor Services which is used in the PowerShell extension for VSCode
(which runs in a different PS Host).

[ ] Issue filed:

[ ] Impact on Completions (both in the console and in editors) - one of PowerShell's most powerful features.

[ ] Issue filed:

[ ] Impact on PSScriptAnalyzer (which provides linting & formatting in the editor extensions).

[ ] Issue filed:

[ ] Impact on EditorSyntax (which provides syntax highlighting with in VSCode, GitHub, and many other editors).

[ ] Issue filed:

Opened PR to fix test failure in PSResourceGet CI - https://github.com/PowerShell/PowerShell/pull/24339

I will rebase and re-run after the other PR is merged.


Is there a Python version?

I think Diffusers would be the best option for python
https://github.com/huggingface/diffusers
There would be no advantage to an OnnxStack python version as OnnxStack has significantly less features than the python libraries out there.

is it possible to use the models you published on hugging face in python?

is it possible to use the models you published on hugging face in python?

Yes, these are ONNX conversions so are language agnostic , can be use anywhere, python, rust, c++

is it possible to use the models you published on hugging face in python?

Yes, these are ONNX conversions so are language agnostic , can be use anywhere, python, rust, c++

I am new to those things.
I tried to use diffusers library to load your models but failed.
Do you have any library recommendations?

is it possible to use the models you published on hugging face in python?

Yes, these are ONNX conversions so are language agnostic , can be use anywhere, python, rust, c++

I am new to those things.
I tried to use diffusers library to load your models but failed.
Do you have any library recommendations?

is it possible to use the models you published on hugging face in python?

Yes, these are ONNX conversions so are language agnostic , can be use anywhere, python, rust, c++

I am new to those things.
I tried to use diffusers library to load your models but failed.
Do you have any library recommendations?

Unfortunately I don't know the python language so don't know the libraries that can be used


A pop-up appears but cannot be edited.

Even if a pop-up appears and I select it, the actual modification is not performed. I am using Redmine 5.0.3.stable.
and I downloaded the dynamic edit plugin today.
Every field edit is not working.

At the first install, it works, but now it doesn't.  Even after reinstalling, the error is the same.

From the logs, it seems that the problem is that the ID is changed from admin to anonymous (I logged in as admin).

Hello,
I have the same problem on redmine with the following environment:
Environment:
Redmine version 5.0.4.stable
Ruby version 2.7.4-p191 (2021-07-07) [x86_64-linux-gnu]
Rails version 6.1.7
Production environment
Database adapter Mysql2
Mailer queue ActiveJob::QueueAdapters::AsyncAdapter
Mailer delivery smtp
Redmine settings:
Redmine theme Redmine-theme-purplemine2 (includes JavaScript)
SCM:
Git 2.30.2
Filesystem
Redmine plugins:
iotbzh 1.0.0
iotbzh_menu_links 0.0.1
redmine_impersonate 2.0.0
redmine_indicator 0.2.16
redmine_issue_dynamic_edit 0.9.1
redmine_lock_out 0.0.1
redmine_openid_connect 0.9.4
The database was imported from a redmine 4.
And I have the impression that it is related to LDAP whose configuration was also imported, during my tests with a clean DB, no problem to modify dynamically, but with the imported DB it does not work, and indeed I see an anonymous user in the logs.
I also have a server identification popup, and when I log on, the plugin is working again.
I continue to investigate ...

Ilogeek and team,
Thanks so much for making this very important plugin. I've given you a star :). Let me know if anything else would help. We use this plugin in a healthcare application and it saves a lot of headache.
I have the same issue.  I'm on redmine 5.0.5 and dynamic edit 0.9.1
To reproduce, try:

create custom fields that have text/number types
make several custom fields, and delete a few and/or change their order in the custom fields list, e.g., have field IDs [1,2,5,4] where 3 was deleted, and 5 was moved above 4.
The PATCH call is sent with no field code, from production.log

I, [2023-03-31T22:55:47.425520 #306]  INFO -- : [XXXXX] Started PATCH "/issues/962" for XXXXX at 2023-03-31 22:55:47 -0400
I, [2023-03-31T22:55:47.426202 #306]  INFO -- : [XXXXX] Processing by IssuesController#update as */*
I, [2023-03-31T23:23:25.696092 #308]  INFO -- : [XXXXX]   Parameters: {"authenticity_token"=>"XXXXX", "id"=>"962"}
I, [2023-03-31T23:23:25.699237 #308]  INFO -- : [XXXXX]   Current user: admin (id=1)

Note Parameters have no field. For different custom field of a list type, a dynamic edit calls correctly
I, [2023-03-31T22:41:05.530059 #308]  INFO -- : [XXXXX]   Parameters: {"issue"=>{"custom_field_values"=>{"13"=>"Male"}}, "authenticity_token"=>"XXXXX", "id"=>"962"}

I also saw what @gousekid reported regarding a 403 error, but I logged out of all browsers and cleared passwords, and the 403 errors went away, but the Parameter problem remained

Hello


Add pool support

Coverage decreased (-0.6%) to 87.975% when pulling f0266ba9c0167ae6de18bd00149e01b930e3a93a on pool_v2 into b4e03289f47f4fd91830e4d855a3641f4db24e04 on master.


Network functionality of metering/rate limiting

The network functionality of metering/rate limiting is currently missing in dpservice. By considering the networking we have, what I understand by browsing the relevant pages from cloud providers is as following:

rate limiting mainly applies onto the egress direction from the VM's perspective. In our setup, performing rate limiting on the ingress direction of VMs by dpservice does not make too much sense, as it cannot actually reduce traffic for the underlay network;
other cloud providers such as GCP have different rate limiting parameters depending on the destinations of traffic flows, e.g, cross-region or per-project basis. As the starting point, in our setup, we could have two rate limiting parameters for each VM. One for public traffic (south-north) and one for private traffic (east-west), and they are both per-VM based. The absolute values of these two parameters can be very different, ranging from dozens of Mbits/s to several Gbits/s.

Possible solutions (non-offloading path):
(a) use DPDK's provided metering algorithm and perform egress rate limiting with CPU. Example. Cons: maximum flexibility in terms of implementing different rate limiting strategies. Feasible rate limiting strategies include: 1) rate limiting depending on traffic types (public vs. private); 2) rate limiting by bandwidth or pps, which is more crucial for small packets. Pros: it costs CPU cycles to determine which packets should be dropped.
(b) relying on VF tx rate limiting supported by Mellanox hardware (@guvenc ). Example. Cons: relatively easy to change on the system, as corresonding sys files of VFs needs to change. And hardware would take care of performing rate limiting on the egress direction. Pros: lack of the fexibility to implement various rate limiting strategies described above. For example, if a bandwidth cap of 1Gbits/s is written to the sys file, both directions, namely north-south and east-west, have to conform this value. Addtionally, pps based rate limiting is not supported.
What we could do here is to combine both approaches to achieve max benefits. For each VF, we could have a default bandwidth cap enforced by the approach (b), e.g., 10 Gbit/s for a normal VM. Or 100 Mbit/s for a small testing VM. For each created interface, via gRPC calls, we futher specify more concrete limiting strategies by (a), such as if using bandwidth-based approach or pps-based approach, and specify different values for public and private traffic.
inputs are welcomed. @MalteJ @guvenc

[x] feature update to dpservice
[x] Go and client adaptation
[x] Final test and PRs

closed with corresponding PRs in metalnet, dpservice, dpservice-go, dpservice-cli.


[V3][RoleManagement] Removing reaction does nothing

If i understand correctly, this code:
https://github.com/mikeshardmind/SinbadCogs/blob/258bb4b205f5a52405e84ab8690e8302572030b8/rolemanagement/events.py#L78-L101
Must remove role on removing of reaction. But this is not happens.

You need to set the role as self-removable for removing a reaction to remove the role:


needle's rawblock cache maybe hit by mistake occasionally in edge scenario

Chris:
Today I found one volume file become read only(id: 3002) after compaction, so I took time to find out the root cause
Below is my online log:
I0206 06:56:50 17728 volume_server_handlers_vacuum.go:16] checked compacting volume = 2957 garbageThreshold = 0.1 vacuum = false
I0206 06:56:50 17728 volume_server_handlers_vacuum.go:16] checked compacting volume = 3002 garbageThreshold = 0.1 vacuum = true
I0206 06:56:51 17728 volume_server_handlers_vacuum.go:25] compacted volume = 3002 , error = <nil>
I0206 06:56:51 17728 volume_loading.go:62] open to write file /data/seaweedfs/weed-volume/lawsuit-bucket_3002.idx
I0206 06:56:51 17728 volume_loading.go:69] volumeDataIntegrityChecking failed verifyNeedleIntegrity /data/seaweedfs/weed-volume/lawsuit-bucket_
3002.idx failed: File Entry Not Found. Needle 5040252 Memory 5040257
I0206 06:56:51 17728 volume_loading.go:73] loading index file /data/seaweedfs/weed-volume/lawsuit-bucket_3002.idx readonly true
I0206 06:56:51 17728 needle_map_memory.go:51] max file key: 258388066 for file: /data/seaweedfs/weed-volume/lawsuit-bucket_3002.idx
I0206 06:56:51 17728 volume_server_handlers_vacuum.go:34] commit compact volume = 3002 , error = <nil>

From above volume log,  we could conclude that the volumeDataIntegrityChecking failed after the 3002 volume is compacted.
But when I restart this volume server, everything seems to be OK
Log file created at: 2017/02/06 11:30:38
Running on machine: xxxxxxxxx
Binary: Built with gc go1.7.3 for linux/amd64
Log line format: [IWEF]mmdd hh:mm:ss threadid file:line] msg
I0206 11:30:38 51565 file_util.go:20] Folder /data/seaweedfs/weed-volume Permission: -rwxrwxr-x
I0206 11:30:38 51565 volume_loading.go:62] open to write file /data/seaweedfs/weed-volume/lawsuit-bucket_3002.idx
I0206 11:30:38 51565 volume_loading.go:73] loading index file /data/seaweedfs/weed-volume/lawsuit-bucket_3002.idx readonly false
I0206 11:30:38 51565 needle_map_memory.go:51] max file key: 258388066 for file: /data/seaweedfs/weed-volume/lawsuit-bucket_3002.idx
I0206 11:30:38 51565 disk_location.go:64] data file /data/seaweedfs/weed-volume/lawsuit-bucket_3002.dat, replicaPlacement=101 v=2 size=39271144 ttl=

And I suspect that the Needle's ReadData function get the cached needle data by mistake which lead to the following error:
3002.idx failed: File Entry Not Found. Needle 5040252 Memory 5040257
Because the current block cache key format is:
cacheKey := fmt.Sprintf("%d:%d:%d", r.Fd(), offset>>3, readSize)
If the fid is reused by OS and one needle's offset in old dat file and new compacted dat file is same(their file size diff is less than the needle alignment padding size), then the cache hitting is triggered. This is actually a edge case, but it did happen
Do you have any suggestion on this scenario?
Thanks!

your theory is possible.

Why the integrity check failed but compaction commit still happened?

Regarding the point 2, I review the volume's loading code

The commitCompact -> volume.load() will not throw error but log it.

Chris:
How about adding the compact version into block cache key?
Below is the code change
func ReadNeedleBlob(r *os.File, offset int64, size uint32,version Version) (dataSlice []byte, block *Block, err error) {
	padding := NeedlePaddingSize - ((NeedleHeaderSize + size + NeedleChecksumSize) % NeedlePaddingSize)
	readSize := NeedleHeaderSize + size + NeedleChecksumSize + padding
	return getBytesForFileBlock(r, offset, int(readSize),version)
}

func (n *Needle) ReadData(r *os.File, offset int64, size uint32, version Version) (err error) {
	bytes, block, err := ReadNeedleBlob(r, offset, size,version)
	if err != nil {
		return err
	}
...

func getBytesForFileBlock(r *os.File, offset int64, readSize int,version Version) (dataSlice []byte, block *Block, err error) {
	// check cache, return if found
	cacheKey := fmt.Sprintf("%d:%d:%d:%d", r.Fd(), offset>>3, readSize,version)
...

Adding compact version as part of the block cache key will reduce the incorrect hit probability(not sure if it could eliminate the issue thoroughly). And it will lead to the specific volume's entire cache miss after the compaction.
Does this code change make sense? Thanks!

This change sounds good. But it's good to let it run a while to see the results.

OK. I will ship this draft fix in dev environment firstly

@hxiaodon  how is it?

Chris:
I'm a little bit ashamed that I did not work on this issue since Feb 13.  I will sync the latest seaweedfs code and create the pull request by the end of this week.

I plan to remove the caching layer, since not much improvement are noticed.
https://github.com/chrislusf/seaweedfs/tree/skip_memory_pool

Chris:
Will this application caching be removed permanently? I feel that the OS page cache will not be always good enough since it will cache the whole 4KB page blocks(even this page just have partial valid data) instead of many possible far smaller files in a consolidated memory
The current lru cache used by seaweedfs is hashicorp one, how about bigcache or freecache, these 2 lib are efficient ones compared with others
Of course weedfs is a disk storage solution, adding memory cache layer out of it is also a good solution.

Depends on performance numbers.
On Fri, 9 Jun 2017 at 12:29 AM Xiaodong Huo<EMAIL_ADDRESS>wrote:

Chris:
Will this application caching be removed permanently? I feel that the OS
page cache will not be always good enough since it will cache the whole 4KB
page blocks(even this page just have partial valid data) instead of many
possible far smaller files in a consolidated memory
The current lru cache used by seaweedfs is hashicorp one, how about bigcache
or freecache https://github.com/allegro/bigcache#bigcache-vs-freecache,
these 2 lib are efficient ones compared with others
Of course weedfs is a disk storage solution, adding memory cache layer out
of it is also a good solution.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/chrislusf/seaweedfs/issues/450#issuecomment-307317122,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABeL73C096qsmHfG-OEgSCNl6B5gjdVHks5sCPRXgaJpZM4L3wzs
.

not applicable since the cache has been removed now.


[Feature request]: masked input component

Preflight Checklist

[X] I have read the Contributing Guidelines for this project.
[X] I agree to follow the Code of Conduct that this project adheres to.
[X] I have searched the issue tracker for an issue that matches the one I want to file, without success.

Request type
Request for a new component
It affects the following packages
Components
Functionality
It would be nice to have a text input field, which is masked with a predefined pattern to prevent wrong and assist user input.
Not to reinvent the wheel you can use a small simple existing library such: https://github.com/beholdr/maska
Link to design proposal file
No response

Hello @stefan-schubert-sbb
We discussed this internally. At the moment we do not want to provide support for this.
This would either force us to wrap an input element in shadow DOM, which is problematic for accessibility, or to just wrap the given library, which provides minimal value. Additionally, we do not yet see enough standardizes use cases for this.
If this changes in the future, we can revisit this.


mfb.css

Hello
in commit b3ce89c0d5d3d87dfd61f16081b53ccdd73c1c74 you removed mfb.css
but after install 2.0 version, mfb.css and jquery.form.min.js the required
Maybe you have some links to this files at other location. Check please

mfb.css is removed because it was integrated in lfm.css. Please upgrade to the latest alpha version and run php artisan vendor:publish --tag=lfm_public --force.


remove reference to write-stream and iterators

I'm also wondering if we should try to reformulate a bit. Since I'm not a native english speaker I might be off. In the beginning we mention bi-directional iterators as one of the features and we say this in plural, i.e. there are at least two types of iterators.
But since write-stream has been removed, does this mean we have one type of iterator, or are there still several types of iterators? Or is it a single bi-directional iterator that is exposed by createReadStream()? If there are different types of iterators I think we should be more specific in what they are to get rid of ambiguity.
Thoughts?

@juliangruber Any thoughts?

@ralphtheninja this seems fine. The bi-directional iterators in this case means that they can go either forward or reverse.


Adds the missing points to the chin feature (issue #48)

Addressed the issue with the face_landmarks wrapper not including all points from the output of _raw_face_landmarks.

Replaced by #55 that includes the test fixes.


Mupx Setup failed in AWS t2.micro

Hi,
I'm trying to deploy one application in AWS and when I run mupx setup is getting this error:
Started TaskList: Setup (linux)
[<IP_ADDRESS>] - Installing Docker
[<IP_ADDRESS>] x Installing Docker: FAILED
    -----------------------------------STDERR-----------------------------------
    bash: line 4: docker: command not found
    sudo: no tty present and no askpass program specified
    sudo: no tty present and no askpass program specified
    sudo: no tty present and no askpass program specified
    sudo: no tty present and no askpass program specified
    -----------------------------------STDOUT-----------------------------------
    ----------------------------------------------------------------------------

First thought is did you try the NOPASSWD in sudoers stuff described in here...
https://github.com/arunoda/meteor-up/blob/master/README.md
Second though does that t2.micro have docker installed or is that a dependency needed still. Double check with ssh if docker is installed in tye instance and in the users env path as well as permission to use/execute it.

@MasterJames I did the steps but still not working with same problem,

There is a scripts/linux shell called install-docker.sh
That checks on the instance if it has docker and attempts to install it if not found.
If that process is blocked etc you get an error there I suspect.
so if you SSH into the instance and type "docker version" you'll now to call this or not.
wget -qO- https://get.docker.com/ | sudo sh
and then you should get a clearer understanding of a problem in there or not.
#!/bin/bash

# Is docker already installed?
set +e
haveDocker=$(docker version | grep "version")
set -e

if [ ! "$haveDocker" ]; then

  # Remove the lock
  set +e
  sudo rm /var/lib/dpkg/lock > /dev/null
  sudo rm /var/cache/apt/archives/lock > /dev/null
  sudo dpkg --configure -a
  set -e

  # Required to update system
  sudo apt-get update

  # Install docker
  wget -qO- https://get.docker.com/ | sudo sh
fi

It works with this script! Thank you so much!


Replace Eventually() in controller tests by using channel or cond.

This change makes the waiting loop more efficient and robust.
addresses #3929

@xiaolanz: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-simpleTests.sh
8bb890e9bd2aaccfa93c8097d793451deca171e1
link
/test e2e-simple

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@xiaolanz: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-simpleTests.sh
8bb890e9bd2aaccfa93c8097d793451deca171e1
link
/test e2e-simple

prow/e2e-bookInfoTests.sh
8bb890e9bd2aaccfa93c8097d793451deca171e1
link
/test e2e-bookInfo

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@xiaolanz: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-simpleTests.sh
8bb890e9bd2aaccfa93c8097d793451deca171e1
link
/test e2e-simple

prow/e2e-bookInfoTests.sh
8bb890e9bd2aaccfa93c8097d793451deca171e1
link
/test e2e-bookInfo

prow/istio-pilot-e2e.sh
8bb890e9bd2aaccfa93c8097d793451deca171e1
link
/test istio-pilot-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest

@xiaolanz: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-bookInfoTests.sh
8bb890e9bd2aaccfa93c8097d793451deca171e1
link
/test e2e-bookInfo

prow/istio-pilot-e2e.sh
8bb890e9bd2aaccfa93c8097d793451deca171e1
link
/test istio-pilot-e2e

prow/e2e-simpleTests.sh
8bb890e9bd2aaccfa93c8097d793451deca171e1
link
/test e2e-simple

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@xiaolanz: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e.sh
8bb890e9bd2aaccfa93c8097d793451deca171e1
link
/test istio-pilot-e2e

prow/e2e-simpleTests.sh
8bb890e9bd2aaccfa93c8097d793451deca171e1
link
/test e2e-simple

prow/e2e-bookInfoTests.sh
8bb890e9bd2aaccfa93c8097d793451deca171e1
link
/test e2e-bookInfo

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@xiaolanz: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-simpleTests.sh
8bb890e9bd2aaccfa93c8097d793451deca171e1
link
/test e2e-simple

prow/e2e-bookInfoTests.sh
8bb890e9bd2aaccfa93c8097d793451deca171e1
link
/test e2e-bookInfo

prow/istio-pilot-e2e.sh
8bb890e9bd2aaccfa93c8097d793451deca171e1
link
/test istio-pilot-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@xiaolanz: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-bookInfoTests.sh
8bb890e9bd2aaccfa93c8097d793451deca171e1
link
/test e2e-bookInfo

prow/istio-pilot-e2e.sh
8bb890e9bd2aaccfa93c8097d793451deca171e1
link
/test istio-pilot-e2e

prow/e2e-simpleTests.sh
94cf9009a1fc4454640cb4591eb6bb63a44c442e
link
/test e2e-simple

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@xiaolanz: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e.sh
8bb890e9bd2aaccfa93c8097d793451deca171e1
link
/test istio-pilot-e2e

prow/e2e-simpleTests.sh
94cf9009a1fc4454640cb4591eb6bb63a44c442e
link
/test e2e-simple

prow/e2e-bookInfoTests.sh
94cf9009a1fc4454640cb4591eb6bb63a44c442e
link
/test e2e-bookInfo

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@xiaolanz: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-simpleTests.sh
94cf9009a1fc4454640cb4591eb6bb63a44c442e
link
/test e2e-simple

prow/e2e-bookInfoTests.sh
94cf9009a1fc4454640cb4591eb6bb63a44c442e
link
/test e2e-bookInfo

prow/istio-pilot-e2e.sh
94cf9009a1fc4454640cb4591eb6bb63a44c442e
link
/test istio-pilot-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@xiaolanz: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-simpleTests.sh
94cf9009a1fc4454640cb4591eb6bb63a44c442e
link
/test e2e-simple

prow/e2e-bookInfoTests.sh
94cf9009a1fc4454640cb4591eb6bb63a44c442e
link
/test e2e-bookInfo

prow/istio-pilot-e2e.sh
94cf9009a1fc4454640cb4591eb6bb63a44c442e
link
/test istio-pilot-e2e

prow/istio-unit-tests.sh
f1e4f9648694d70d6744bc716767ac6984e65f28
link
/test istio-unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@xiaolanz: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-simpleTests.sh
94cf9009a1fc4454640cb4591eb6bb63a44c442e
link
/test e2e-simple

prow/e2e-bookInfoTests.sh
94cf9009a1fc4454640cb4591eb6bb63a44c442e
link
/test e2e-bookInfo

prow/istio-pilot-e2e.sh
94cf9009a1fc4454640cb4591eb6bb63a44c442e
link
/test istio-pilot-e2e

prow/istio-unit-tests.sh
f1e4f9648694d70d6744bc716767ac6984e65f28
link
/test istio-unit-tests

prow/istio-presubmit.sh
f1e4f9648694d70d6744bc716767ac6984e65f28
link
/test istio-presubmit

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

Go in general discourage the use of common utils. These two better off adopting different patterns.
I added timeout.
utils.Eventually is still kept for some cases that fit the use better.

@xiaolanz: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-simpleTests.sh
94cf9009a1fc4454640cb4591eb6bb63a44c442e
link
/test e2e-simple

prow/e2e-bookInfoTests.sh
94cf9009a1fc4454640cb4591eb6bb63a44c442e
link
/test e2e-bookInfo

prow/istio-pilot-e2e.sh
94cf9009a1fc4454640cb4591eb6bb63a44c442e
link
/test istio-pilot-e2e

prow/istio-presubmit.sh
f1e4f9648694d70d6744bc716767ac6984e65f28
link
/test istio-presubmit

prow/istio-unit-tests.sh
dd63ed49d9ce2a385d4c0e61e4224245dbe4014a
link
/test istio-unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

PTAL

@xiaolanz: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-bookInfoTests.sh
94cf9009a1fc4454640cb4591eb6bb63a44c442e
link
/test e2e-bookInfo

prow/istio-pilot-e2e.sh
94cf9009a1fc4454640cb4591eb6bb63a44c442e
link
/test istio-pilot-e2e

prow/istio-unit-tests.sh
dd63ed49d9ce2a385d4c0e61e4224245dbe4014a
link
/test istio-unit-tests

prow/e2e-simpleTests.sh
dd63ed49d9ce2a385d4c0e61e4224245dbe4014a
link
/test e2e-simple

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@xiaolanz: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e.sh
94cf9009a1fc4454640cb4591eb6bb63a44c442e
link
/test istio-pilot-e2e

prow/istio-unit-tests.sh
dd63ed49d9ce2a385d4c0e61e4224245dbe4014a
link
/test istio-unit-tests

prow/e2e-simpleTests.sh
dd63ed49d9ce2a385d4c0e61e4224245dbe4014a
link
/test e2e-simple

prow/e2e-bookInfoTests.sh
dd63ed49d9ce2a385d4c0e61e4224245dbe4014a
link
/test e2e-bookInfo

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@xiaolanz: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e.sh
94cf9009a1fc4454640cb4591eb6bb63a44c442e
link
/test istio-pilot-e2e

prow/e2e-simpleTests.sh
dd63ed49d9ce2a385d4c0e61e4224245dbe4014a
link
/test e2e-simple

prow/e2e-bookInfoTests.sh
dd63ed49d9ce2a385d4c0e61e4224245dbe4014a
link
/test e2e-bookInfo

prow/istio-unit-tests.sh
07c8b95c74712b0c49478fcda31ed8173ac98d70
link
/test istio-unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@xiaolanz: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e.sh
94cf9009a1fc4454640cb4591eb6bb63a44c442e
link
/test istio-pilot-e2e

prow/e2e-bookInfoTests.sh
dd63ed49d9ce2a385d4c0e61e4224245dbe4014a
link
/test e2e-bookInfo

prow/istio-unit-tests.sh
07c8b95c74712b0c49478fcda31ed8173ac98d70
link
/test istio-unit-tests

prow/e2e-simpleTests.sh
07c8b95c74712b0c49478fcda31ed8173ac98d70
link
/test e2e-simple

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@xiaolanz: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e.sh
94cf9009a1fc4454640cb4591eb6bb63a44c442e
link
/test istio-pilot-e2e

prow/istio-unit-tests.sh
07c8b95c74712b0c49478fcda31ed8173ac98d70
link
/test istio-unit-tests

prow/e2e-simpleTests.sh
07c8b95c74712b0c49478fcda31ed8173ac98d70
link
/test e2e-simple

prow/e2e-bookInfoTests.sh
07c8b95c74712b0c49478fcda31ed8173ac98d70
link
/test e2e-bookInfo

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@xiaolanz: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
07c8b95c74712b0c49478fcda31ed8173ac98d70
link
/test istio-unit-tests

prow/e2e-simpleTests.sh
07c8b95c74712b0c49478fcda31ed8173ac98d70
link
/test e2e-simple

prow/e2e-bookInfoTests.sh
07c8b95c74712b0c49478fcda31ed8173ac98d70
link
/test e2e-bookInfo

prow/istio-pilot-e2e.sh
07c8b95c74712b0c49478fcda31ed8173ac98d70
link
/test istio-pilot-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@xiaolanz: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-simpleTests.sh
07c8b95c74712b0c49478fcda31ed8173ac98d70
link
/test e2e-simple

prow/e2e-bookInfoTests.sh
07c8b95c74712b0c49478fcda31ed8173ac98d70
link
/test e2e-bookInfo

prow/istio-pilot-e2e.sh
07c8b95c74712b0c49478fcda31ed8173ac98d70
link
/test istio-pilot-e2e

prow/istio-unit-tests.sh
6300b50cad393cf3a78a1854aec417e3d65272a4
link
/test istio-unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@xiaolanz: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-bookInfoTests.sh
07c8b95c74712b0c49478fcda31ed8173ac98d70
link
/test e2e-bookInfo

prow/istio-pilot-e2e.sh
07c8b95c74712b0c49478fcda31ed8173ac98d70
link
/test istio-pilot-e2e

prow/istio-unit-tests.sh
6300b50cad393cf3a78a1854aec417e3d65272a4
link
/test istio-unit-tests

prow/e2e-simpleTests.sh
6300b50cad393cf3a78a1854aec417e3d65272a4
link
/test e2e-simple

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@xiaolanz: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e.sh
07c8b95c74712b0c49478fcda31ed8173ac98d70
link
/test istio-pilot-e2e

prow/istio-unit-tests.sh
6300b50cad393cf3a78a1854aec417e3d65272a4
link
/test istio-unit-tests

prow/e2e-simpleTests.sh
6300b50cad393cf3a78a1854aec417e3d65272a4
link
/test e2e-simple

prow/e2e-bookInfoTests.sh
6300b50cad393cf3a78a1854aec417e3d65272a4
link
/test e2e-bookInfo

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@xiaolanz: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
6300b50cad393cf3a78a1854aec417e3d65272a4
link
/test istio-unit-tests

prow/e2e-simpleTests.sh
6300b50cad393cf3a78a1854aec417e3d65272a4
link
/test e2e-simple

prow/e2e-bookInfoTests.sh
6300b50cad393cf3a78a1854aec417e3d65272a4
link
/test e2e-bookInfo

prow/istio-pilot-e2e.sh
6300b50cad393cf3a78a1854aec417e3d65272a4
link
/test istio-pilot-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

PTAL.
Test failure due to unrelated reasons.

/retest

@xiaolanz: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-simpleTests.sh
6300b50cad393cf3a78a1854aec417e3d65272a4
link
/test e2e-simple

prow/e2e-bookInfoTests.sh
6300b50cad393cf3a78a1854aec417e3d65272a4
link
/test e2e-bookInfo

prow/istio-pilot-e2e.sh
6300b50cad393cf3a78a1854aec417e3d65272a4
link
/test istio-pilot-e2e

prow/istio-unit-tests.sh
6300b50cad393cf3a78a1854aec417e3d65272a4
link
/test istio-unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@xiaolanz: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-bookInfoTests.sh
6300b50cad393cf3a78a1854aec417e3d65272a4
link
/test e2e-bookInfo

prow/istio-pilot-e2e.sh
6300b50cad393cf3a78a1854aec417e3d65272a4
link
/test istio-pilot-e2e

prow/istio-unit-tests.sh
6300b50cad393cf3a78a1854aec417e3d65272a4
link
/test istio-unit-tests

prow/e2e-simpleTests.sh
6300b50cad393cf3a78a1854aec417e3d65272a4
link
/test e2e-simple

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@xiaolanz: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e.sh
6300b50cad393cf3a78a1854aec417e3d65272a4
link
/test istio-pilot-e2e

prow/istio-unit-tests.sh
6300b50cad393cf3a78a1854aec417e3d65272a4
link
/test istio-unit-tests

prow/e2e-simpleTests.sh
6300b50cad393cf3a78a1854aec417e3d65272a4
link
/test e2e-simple

prow/e2e-bookInfoTests.sh
6300b50cad393cf3a78a1854aec417e3d65272a4
link
/test e2e-bookInfo

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@xiaolanz: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-simpleTests.sh
6300b50cad393cf3a78a1854aec417e3d65272a4
link
/test e2e-simple

prow/e2e-bookInfoTests.sh
6300b50cad393cf3a78a1854aec417e3d65272a4
link
/test e2e-bookInfo

prow/istio-unit-tests.sh
f0ea6f584b90596c1b81725e26bc9a954a1de615
link
/test istio-unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@xiaolanz: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-bookInfoTests.sh
6300b50cad393cf3a78a1854aec417e3d65272a4
link
/test e2e-bookInfo

prow/istio-unit-tests.sh
f0ea6f584b90596c1b81725e26bc9a954a1de615
link
/test istio-unit-tests

prow/e2e-simpleTests.sh
f0ea6f584b90596c1b81725e26bc9a954a1de615
link
/test e2e-simple

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@xiaolanz: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
f0ea6f584b90596c1b81725e26bc9a954a1de615
link
/test istio-unit-tests

prow/e2e-simpleTests.sh
f0ea6f584b90596c1b81725e26bc9a954a1de615
link
/test e2e-simple

prow/e2e-bookInfoTests.sh
f0ea6f584b90596c1b81725e26bc9a954a1de615
link
/test e2e-bookInfo

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/lgtm

@xiaolanz: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-simpleTests.sh
f0ea6f584b90596c1b81725e26bc9a954a1de615
link
/test e2e-simple

prow/e2e-bookInfoTests.sh
f0ea6f584b90596c1b81725e26bc9a954a1de615
link
/test e2e-bookInfo

prow/istio-unit-tests.sh
86155bbe37c1ee72c87d396cc72042f8972970d3
link
/test istio-unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@xiaolanz: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-bookInfoTests.sh
f0ea6f584b90596c1b81725e26bc9a954a1de615
link
/test e2e-bookInfo

prow/istio-unit-tests.sh
86155bbe37c1ee72c87d396cc72042f8972970d3
link
/test istio-unit-tests

prow/e2e-simpleTests.sh
86155bbe37c1ee72c87d396cc72042f8972970d3
link
/test e2e-simple

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@xiaolanz: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
86155bbe37c1ee72c87d396cc72042f8972970d3
link
/test istio-unit-tests

prow/e2e-simpleTests.sh
86155bbe37c1ee72c87d396cc72042f8972970d3
link
/test e2e-simple

prow/e2e-bookInfoTests.sh
86155bbe37c1ee72c87d396cc72042f8972970d3
link
/test e2e-bookInfo

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@xiaolanz: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-simpleTests.sh
86155bbe37c1ee72c87d396cc72042f8972970d3
link
/test e2e-simple

prow/e2e-bookInfoTests.sh
86155bbe37c1ee72c87d396cc72042f8972970d3
link
/test e2e-bookInfo

prow/istio-unit-tests.sh
a2bfbb9dc7f1681fb826189b99091042fc50269f
link
/test istio-unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@xiaolanz: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-bookInfoTests.sh
86155bbe37c1ee72c87d396cc72042f8972970d3
link
/test e2e-bookInfo

prow/istio-unit-tests.sh
a2bfbb9dc7f1681fb826189b99091042fc50269f
link
/test istio-unit-tests

prow/e2e-simpleTests.sh
a2bfbb9dc7f1681fb826189b99091042fc50269f
link
/test e2e-simple

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@xiaolanz: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
a2bfbb9dc7f1681fb826189b99091042fc50269f
link
/test istio-unit-tests

prow/e2e-simpleTests.sh
a2bfbb9dc7f1681fb826189b99091042fc50269f
link
/test e2e-simple

prow/e2e-bookInfoTests.sh
a2bfbb9dc7f1681fb826189b99091042fc50269f
link
/test e2e-bookInfo

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


TypeAdapter[T] inferrence fails with mypy (quick fix)

Initial Checks

[X] I confirm that I'm using Pydantic V2

Description
TypeAdapter(T) fails to infer TypeAdapter[T] with mypy.
Adding __init__ replicating the existing __new__ type checking block in the TypeAdapter class fixes the issue.
Mypy 1.4.1
Will create PR if no objections.
Example Code
import typing as tp
from pydantic import TypeAdapter

# Before change:
ta = TypeAdapter(float)  # ERR: Need type annotation for "ta"  [var-annotated] (mypy)
tp.reveal_type(ta)
# Revealed type is "pydantic.type_adapter.TypeAdapter[Any]" (mypy)

# After change:
ta = TypeAdapter(float)  # Mypy happy
tp.reveal_type(ta)
# Revealed type is "pydantic.type_adapter.TypeAdapter[builtins.float]" (mypy)

Python, Pydantic & OS Version
pydantic version: 2.0.2
        pydantic-core version: 2.1.2 release build profile
                 install path: venv/lib/python3.11/site-packages/pydantic
               python version: 3.11.4 (main, Jun  7 2023, 12:45:48) [GCC 11.3.0]
                     platform: Linux-5.19.0-46-generic-x86_64-with-glibc2.35
     optional deps. installed: ['devtools', 'email-validator', 'typing-extensions']

Selected Assignee: @hramezani

Thanks @zakstucke for this 🙏
I can confirm the problem and also the fix that you provided.

What happens with your change if you do TypeAdapter(int | str)?

Sadly same as your comment in PR: pydantic.type_adapter.TypeAdapter[types.UnionType]. But I think that's mypy's problem rather than pydantic's.

@adriangb I've used this as a workaround for unions in the past with mypy, but it would require the user to enter such as TypeAdapter(Returns[str | float]), which is probably not something you'd want to support.
# https://github.com/python/mypy/issues/9003
class Returns(typing.Generic[T]):
    if not typing.TYPE_CHECKING:

        def __class_getitem__(cls, item: object) -> object:
            """Called when Returns is used as an Annotation at runtime.
            We just return the type we're given"""
            return item


⚠ Bofrost.de has degraded performance

In f2c51a3, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1402 ms

Resolved: Bofrost.de performance has improved in 9850205.


BUG [TEST]

Please fill out all the sections below for bug issues, otherwise it'll be closed as it won't be actionable for us to address.
Describe the bug
This should say something
To Reproduce
Steps to reproduce the behavior:
Expected behavior
A clear and concise description of what you expected to happen.
int main() {
  printf("hello world!");
}

Configuration:

Edition: [e.g. Windows 11, Windows Server]
Base Image being used: [e.g. Windows Server Core]
Container engine: [e.g. docker]
Container Engine version [e.g. 22]

Additional context
Add any other context about the problem here.

again


running a global installed deptrac.phar as non-root fails

deptrac should be callable as non-root user if it was globally installed. currently it results in an Exception.
vagrant@vagrant:~/test$ deptrac
PHP Fatal error:  Uncaught Humbug\SelfUpdate\Exception\FilesystemException: The current phar file is not writeable and cannot be replaced: /usr/local/bin/deptrac. in phar:///usr/local/bin/deptrac/vendor/padraic/phar-updater/src/Updater.php:430
Stack trace:
#0 phar:///usr/local/bin/deptrac/vendor/padraic/phar-updater/src/Updater.php(101): Humbug\SelfUpdate\Updater->setLocalPharFile('/usr/local/bin/...')
#1 [internal function]: Humbug\SelfUpdate\Updater->__construct(NULL, false)
#2 phar:///usr/local/bin/deptrac/vendor/symfony/dependency-injection/ContainerBuilder.php(927): ReflectionClass->newInstanceArgs(Array)
#3 phar:///usr/local/bin/deptrac/vendor/symfony/dependency-injection/ContainerBuilder.php(475): Symfony\Component\DependencyInjection\ContainerBuilder->createService(Object(Symfony\Component\DependencyInjection\Definition), 'updater')
#4 phar:///usr/local/bin/deptrac/vendor/symfony/dependency-injection/ContainerBuilder.php(984): Symfony\Component\DependencyInjection\ContainerBuilder->get('updater', 1)
#5 phar:///usr/loca in phar:///usr/local/bin/deptrac/vendor/padraic/phar-updater/src/Updater.php on line 430

the constructor of padraic/phar-updater calls $this->setLocalPharFile($localPharFile); which throws an exception on a not writeable phar file.
the constructor is called because of the services.xml.
am i right and lazyloading the service would help?
<service id="updater" class="Humbug\SelfUpdate\Updater" lazy="true">

thanks a lot for reporting this issue.
can you please self-update deptrac and retry?

now it works :)
root@vagrant:~# deptrac selfupdate
Updating deptrac to latest version...
Deprac was successfully updated.
root@vagrant:~# su - vagrant
vagrant@vagrant:~$ deptrac

deptrac is alpha, not production ready.
please help us and report feedback / bugs.

depfile "/home/vagrant/depfile.yml" not found, run "deptrac init" to create one.