Add logAeson (#578)

Starts work on  #578  .
Pre-review checklist

[x] All code has been formatted using our config (make format for Purescript, nixpkgs-fmt for Nix)
[x] All Purescript imports are explicit, including constructors
[ ] All existing examples have been run locally against a fully-synced testnet node
[ ] The integration and unit tests have been run (npm run test) locally
[x] The changelog has been updated under the ## Unreleased header, using the appropriate sub-headings (### Added, ### Removed, ### Fixed)

I'm going to mark this as a draft again since it needs some more work


Fix for Issue #42

Added timezone support for recurring events and thus fixing issue #42 .
Additionally fixed a typo in a variable name.

Thanks for the PR and spotting the spelling error. However, I do have a few concerns with your PR.
Your commit messages don't appear to match the file changes and I'm not familiar with the language you've used in places (e.g. 'aktuelle'). As this library is written in English for consistency we need to keep with that.
I know you were having a problem with week start but forcing Monday rather than Sunday as the first day of the week as it fits your use case is not the best course of action. Instead you should parse WKST if it is set and then adjust to use Monday accordingly.
Lastly, your fix using $gmt_offset will only grab the timezone from your installation of PHP and won't necessarily match that of the iCal file you're attempting to parse. Instead the timezone should be taken from the iCal file itself if it is present.

Hey, thanks for your review. The points you listed make sense and here are my words:
Regarding the commit messages, I am sorry I messed them up a bit. It was my local git repository and the fixes happened in the first commit which I gave a dummy name (aktuelle version means "initial version" in this case). I will find a way to change the commit messages and send them anew.
Using Sunday as week start is conflicting with PHP and the .ics because both go with ISO-8601(https://en.wikipedia.org/wiki/ISO_8601#Week_dates) from what I learned in my research looking for the error. Anyways, taking the value from WKST is a good idea, since I also learnt that some arabic countries' first day of week is saturday. I will make Monday the default then.
About the $gmt_offset you are right but I am not sure if every .ics has their timezone included. Will also make a check for that.

If you want I can create a new pull request with fixed messages

I made major changes to the comment above, if you haven't reread it please take a look.


Banner in front on Player

Hello!
Thanks for great job, my issue is :
How i put a banner in front of the player?
Im directly use the player in fullscreen, and my code of banner of Mopub is
MoPubView banner = (MoPubView) findViewById(R.id.bannerAd);
banner.setAdUnitId("MyIdBanner");
banner.loadAd();

and im declare the same banner in the layout like that:
<com.mopub.mobileads.MoPubView
            android:id="@+id/bannerAd"
            android:layout_width="wrap_content"
            android:layout_height="wrap_content"
            android:layout_gravity="center_horizontal"/>

How can I make the banner load, if pressing the button to play the video automatically opens the player in full screen without Layout.
Thanks!

Fixed.


[FLINK-12804] Introduce mailbox-based ExecutorService

What is the purpose of the change
This PR introduced an ExecutorService frontend to the mailbox that can be used by client code. For this purpose, the mailbox was also extended with open->quiesce -> close lifecycle methods.
Brief change log

Intoduce TaskMailboxExecutorService, an ExecutorService with additional  non-blocking alternative to execute and a yield() functionality.

Enhanced Mailbox with lifecycle methods.

Improvements to the mailbox usage in StreamTask

Verifying this change
Introduced TaskMailboxExecutorServiceImplTest and extended MailboxImplTest.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (no)
The serializers: (no)
The runtime per-record code paths (performance sensitive): (yes)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: (no)
The S3 file system connector: (no)

Documentation

Does this pull request introduce a new feature? (no)
If yes, how is the feature documented? (not applicable)

CC @1u0 @pnowojski

Updated the PR in a couple of ways, for example enforcing that some methods of the executor must be called from inside/outside of the mailbox thread, a check if we are alreay inside the mailbox thread and I removed potentially dangerous and not really required waiting (for mail or capacity) methods.

Hi everybody! I updated and extended the PR quiet a bit, including a refactoring that makes testing mailbox execution possible and an improved signaling for (un)availability of default action. PR now also sits on top of #8858.

Thanks a lot for the reviews, will squash and merge when Travis is green.

Have added the requested changes and rebased. Failing test throwingExceptionOnCallbackWithNoRestartsShouldFailTheSuspend also fails on master for me so I guess it is unrelated.

Thanks for the reivew @pnowojski . Will merge.


Flutter 1.12 app crashes on iPhone 4S (iOS 9.3.6) device

Hi there,
I have an app which recently I have updated to Flutter 1.12. Once I have pushed the release to Apple Store, I have reported crash from one of my users - they have iPhone 4S (iOS 9.3.6) device.
Prior that when my app was built with Flutter v1.9.1+hotfix.6 I have no such issues.
Here is my flutter doctor -v output:

[√] Flutter (Channel stable, v1.12.13+hotfix.8, on Microsoft Windows [Version 10.0.18362.720], locale en-US)
• Flutter version 1.12.13+hotfix.8 at d:\DevTools\flutter
• Framework revision 0b8abb4724 (5 weeks ago), 2020-02-11 11:44:36 -0800
• Engine revision e1e6ced81d
• Dart version 2.7.0
[√] Android toolchain - develop for Android devices (Android SDK version 29.0.3)
• Android SDK at D:\DevTools\Android\Sdk
• Android NDK location not configured (optional; useful for native profiling support)
• Platform android-29, build-tools 29.0.3
• ANDROID_SDK_ROOT = D:\DevTools\Android\Sdk
• Java binary at: D:\DevTools\android-studio\jre\bin\java
• Java version OpenJDK Runtime Environment (build 1.8.0_212-release-1586-b04)
• All Android licenses accepted.
[√] Android Studio (version 3.6)
• Android Studio at D:\DevTools\android-studio
• Flutter plugin version 44.0.2
• Dart plugin version 192.7761
• Java version OpenJDK Runtime Environment (build 1.8.0_212-release-1586-b04)

I realize the feedback I am giving you guyes is limited, however, since I have no such hardware in possession and in general I have no much of experience with iOS devices, I would appreciate any advice how to proceed on this topic.
Thanks!

Hi @angel1st
can you please provide the crash logs you are receiving;
ideally I'd for flutter run --release --verbose,
but if I understand correctly you don not have the possibility
of testing you app on a physical iPhone 4s, therefore
even if not a comprehensive representation of the situation
could you provide your flutter run --verbose (on a iOS 9.3.6)
your flutter build ios --verbose and your pubspec.yaml
Thank you

Hey @iapicca and thanks for answering. I can provide 2 out of 3 from the required deliverables, since I have no iOS 9.3.6 device on my end.
Here they are:
flutter build ios --verbose output
pubspec.yaml zip
One note though - the very same app has been successfully built with Flutter v1.9.1+hotfix.6. Once, I moved to Flutter 1.12, I have had the complain.

thank you for the logs @angel1st

Hey @iapicca, any chances for an issue update sooner rather than latter?

@iapicca - any news on this one. At least a feedback the team is working on it?

Hi @angel1st
are you still experiencing the issue with the latest stable (or above)?
If so could you please provide your flutter doctor -v
and your crash logs
Thank you

@iapicca - let me check and see if the end customer is still available. I release an updated app today and hopefully will get the customer on board in the next few days.

@iapicca - I have compiled and released my app with Flutter 1.17.1. Unfortunately, the user reported, she still has the very same issue on her iOS 4s phone.
Below is my flutter doctor -v output:

[✓] Flutter (Channel stable, v1.17.1, on Mac OS X 10.14.6 18G1012, locale en-GB)
• Flutter version 1.17.1 at /Users/angel/development/flutter
• Framework revision f7a6a7906b (8 weeks ago), 2020-05-12 18:39:00 -0700
• Engine revision 6bc433c6b6
• Dart version 2.8.2

[✗] Android toolchain - develop for Android devices
✗ Unable to locate Android SDK.
Install Android Studio from: https://developer.android.com/studio/index.html
On first launch it will assist you in installing the Android SDK components.
(or visit https://flutter.dev/docs/get-started/install/macos#android-setup for detailed
instructions).
If the Android SDK has been installed to a custom location, set ANDROID_SDK_ROOT to that location.
You may also want to add it to your PATH environment variable.

[✓] Xcode - develop for iOS and macOS (Xcode 11.3.1)
• Xcode at /Applications/Xcode.app/Contents/Developer
• Xcode 11.3.1, Build version 11C504
• CocoaPods version 1.8.4

[!] Android Studio (not installed)
• Android Studio not found; download from https://developer.android.com/studio/index.html
(or visit https://flutter.dev/docs/get-started/install/macos#android-setup for detailed
instructions).

[✓] VS Code (version 1.45.1)
• VS Code at /Applications/Visual Studio Code.app/Contents
• Flutter extension version 3.12.1

[!] Connected device
! No devices available

Without a stack trace or a reduced test case, there isn't much progress the team can make on this.

@chinmaygarde - if you have a look at the issue's initial discussion you would understand I cannot send you a stack trace due to specific limitations.
I can however refer to another case where the Flutter team was resourceful enough to find a specific device and test on its own end. I can provide iOS binary upon request.
I understand this is probably an edge case, however, if you state Flutter supports certain devices and OSes, you as a team should stand behind your statement.

In the linked issue, there was a reduced test case. Something that only used Flutter APIs with all the extraneous dependencies removed. If there is an issue in such a reduced test case, that is a good starting point for an investigation. Unfortunately, having just the binary is not sufficient for an investigation.
An observation: The pubspec.yaml you linked contains numerous dependencies as well. Without a backtrace or any logs, it is hard to tell where the issue originated.
Closing this as in-actionable. Please re-open if you are able to provide more information. Thanks.


fix: run integration tests on python 3.10

why did we needed to make this change in the first place? is grpc/protobuf broken on 3.9? is fal broken on that version? this PR creates more questions than it answers (how does it fix the CI breakage)

why did we needed to make this change in the first place? is grpc/protobuf broken on 3.9? is fal broken on that version? this PR creates more questions than it answers (how does it fix the CI breakage)

The integration tests were failing and updating to 3.10 fixed them. But then it looks like we realized tests were failing because of infra issues in our staging deployment. This PR will be reverted.


is local recording working?

hello!
I found your blog about your project for local recording on jitsi meeet. At the bottom there was a link to your github so I search "jitsi" and found this fork.
How was the development of that idea? any work in progress that may be resumed?

I just found
https://github.com/jitsi/jitsi-meet/pull/3223
and its merged!
great thank you!


bugfix: removed the useless param

relevant issue: https://github.com/tokers/lua-resty-requests/issues/8


Branch compatibility check not performed when switching on disjoint union distinguishing property.

Here is the bug: flow.org/try
The bug occurs when indexing into an Array<T> where T is some disjoint union type.  I've found that when attempting type refinement upon the distinguishing property, branch compatibility checks are not performed.
When explicitly annotating the type of the array element, however, branch compatibility checks are performed as one might expect.
This bug seems particularly dangerous because type refinement using strings may contain slight misspellings which Flow won't catch (unless of course, explicit type annotation is performed, but this in and of itself is not enforced by Flow).

This bug seems particularly dangerous because type refinement using
strings may contain slight misspellings which Flow won't catch

There’s a much better fix for this in general. Use an empty assertion
in your default case:
// @flow
type A = {type: "a", a: "a"}
type B = {type: "b", b: "b"}
type AB = A | B

function foo3(x: Array<AB>): string {
  const first = x[0];
  switch (first.type) {
    case "a":
      return first.a;
    case "bad_branch":  // no error
      return first.b;
    default:
      (first.type: empty);  // error! (good!)
      return "default";
  }
}

The empty assertion verifies that every case has been covered in the
switch. In the above example, it fails because the case "b": has
been omitted. The error message explains that:

Cannot cast first.type to empty because string literal b is
incompatible with empty.

This check makes your code robust to changes in the type AB. Suppose
that you redefine the type to add a third case: type AB = A | B | C,
where C has type: "c". With the empty assertion, this code will
fail at compile time, and you will be able to add the appropriate case.
Without the empty assertion, you would have no way of remembering that
this function needs to be updated.
Therefore, a switch against a discriminated union tag should always have
an empty assertion. I speak from experience when saying that this
feature—exhaustiveness checking—is extremely powerful. It enables
large-scale worry-free refactors of the form “change the definition, fix
all the type errors, and the code will work on the first try”.
By the way, I usually write
  switch (first.type) {
    // …
    default:
      throw new Error((first.type: empty));
  }

to get both a static failure and a runtime failure in the case where
something unsound slips through.
(As to whether the observed behavior is a bug… I suppose that it should
be, but honestly the semantics of errors on string-enum equality checks
are so fundamentally broken that I don’t think it’s worth fixing.)

Now both correctly error


Auto-hiding of labels is too aggressive

Describe the bug
A clear and concise description of what the bug is.
There are medium zoom levels where you want to see the labels and you can't, especially if you have a Windows computer that doesn't do smooth zooming/scrolling and doesn't have a high-DPI screen.
Secondly, the users expect the image export to look exactly like what's on screen.  If we hide labels conditionally based on zoom, then the image export doesn't look the same.  Users are hesitant to export, because the "figure labels are broken".
To Reproduce
Steps to reproduce the behavior:

Go to enrichmentmap.org on a Windows computer with an HD screen -- not high-DPI like a Mac, not 4K
Click on the demo
Zoom in a bit
You still can't see the labels

Expected behavior
A clear and concise description of what you expected to happen.
You should be able to see the labels all the time.  Hiding the bubble children labels is fine, because you don't want to see them and they're not shown in the image export.

Auto hide appears to be disabled.


Returning to CVCalendar with a selected date renders day label incorrectly

I am having an issue where after selecting a day (shown in blue), transition to a detail controller, and then returning back to the calendar, the selected day has the number label drawn in black on top of it.
Running through the drawing code it appears there is a check for "current day", but not "selected day", and it is draws the regular label on top of the blue circle.

Hello @jeffcardillo,
The issue comes with the wrong way of CVCalendar updating. It's already been reported in issue #14, and the simplest solution is to ensure that method commitCalendarViewUpdate() is called once (but not on every view's appearing).
I'll fix the documentation as soon as I have enough time for completing the new release version of CVCalendar.
Hope it helps.
Regards,
Eugene

Perfect, thank you.


Observability requirements for FOM

Documenting on FOM's confluence for now, partly because FOM GitHub doesn't have a wiki enabled
https://apps.nrs.gov.bc.ca/int/confluence/display/FOM/Observability+Requirements
Please advise if this should go elsewhere.

Documenting on FOM's confluence for now, partly because FOM GitHub doesn't have a wiki enabled https://apps.nrs.gov.bc.ca/int/confluence/display/FOM/Observability+Requirements
Please advise if this should go elsewhere.

@basilv Wikis now enabled!

Awesome, thanks Derek - requirements moved to https://github.com/bcgov/nr-fom/wiki/Observability-Requirements

@basilv sounds good to keep the scope to performance and availability. I think priority should be given to public forms performance related to latency, CPU utilization, memory utilization, peak traffic, and error rate. We have a distribution email that we can use for alerts, I will add you and Ian to the distribution group.
I wonder if it is important to get metrics on Mean time between failure to understand the average time between repairs. This might help with tracking patterns.
At this point, I think it might be time to start exploring Opensearch. We can look into this next sprint. Thanks for documenting the requirements.


Rename the existing gem to asciidoctor-rfc

This commit rename the gem to asciidoctor-rfc, and it also updates the directory structure and others files as necessary, For simplicity it is using the Asciidoctor::Rfc namespace for the gem files.

@ronaldtse : This PR includes the changes to rename the gem, I am using the Asciidoctor::Rfc for simplicity but please let me know if you would prefer Asciidoctor::RFC or anything else.

This is totally fine! Sorry I missed this PR... merging now


Add an overview or description.

Please add a high-level overview or description for this project. Thanks!

added: https://github.com/CloudWebManage/cwm-worker-operator/blob/main/README.md#cwm-worker-operator


[eas-ci] revert UpdateBranchWithCurrentGroupFragment

Why
In my query, I forgot I had to parameterize one of the edges so I can't make it a fragment after all. This PR switchies it over to an explicit type.
            query ViewUpdateChannelOnApp(
              $appId: String!
              $channelName: String!
              $filter: UpdatesFilter
            ) {
              app {
                byId(appId: $appId) {
                       ...
                      updateGroups(offset: 0, limit: 1, filter: $filter) { <------------ Had to parameterize this one
                      ...
            }

Test Plan

[ ] tests still pass

/changelog-entry chore revert UpdateBranchWithCurrentGroupFragment


Errors reading the session-data-default.js file get hidden

Several times I've been tripped up by having an mistake in my session-data-default.js that means it doesn't read, but not realising.
When the kit reads session-data-default.js it does throw an error if it can't read it - but this error is immediately hidden by some other lines. I tend to look to the end of my console to see if I have problems - since the error isn't there, I don't see it.
Example with an error:

Since the kit keeps working - if you browse a few pages the original error message quickly scrolls out of view.
I wonder if we can do anything about this? stop the kit if there's an error rather than continuing?

Agree this might better to crash on. process.exit(1) might do it here:
https://github.com/alphagov/govuk-prototype-kit/blob/2542c176bd072c99e43f15d02b44eee205fe68b8/lib/utils.js#L267
Though I recall process.exit doesn't always work for us for some reason (for example if you say no to 'do you want a different port?')

A followup to this - it would be really handy if the errors could be shown in the console output like normal js errors do. Even if you spot the above message, debugging what's gone wrong is really hard because no details are given - and you can't use console.log() to print anything to the console from this file.

you can try adding e to the log on this line
https://github.com/alphagov/govuk-prototype-kit/blob/master/lib/utils.js#L229


Show error condition on Insights

If there is a network error or something wrong happened, show an error somewhere.

Related: #224

@isaackeyet do you have any idea of how you'd convey an error on Insights? If you scroll down the sections showing comments have the error display there. Not sure what's the best way to show an error on the top sections. Maybe a header or something? For now leaving the dashes and Unknown can work as it's better than showing "(null) of views" LOL.

We haven't really solved this on the web side either. We have a basic message for when data didn't load for whatever reason and it works well for the normal modules (see screenshots below) but we don't currently have anything for the tabs based modules.

Since there's a lot of repetition currently if your connection drops (train entered a tunnel) or a serious error occurs (API unavailable) I'd much rather prefer a small greyed out error icon in each of the modules and a floating error message, toaster style.
Something with the same effect as this, conveying a loss of connection/data:

I mean, on web Calypso if your connection drops we now have an app-wide notice display that the connection has been lost and that "some information may be out of sync". It's really neat and perfectly appropriate since the entire app is dependent on the same things any one view in the app are: the API and an internet connection. So if data doesn't load in the iOS/Android app because flight mode was turned on or whatever, the whole app really has the same problem. Each module with incomplete information within could highlight and not highlight this, as appropriate, but the app-wide notice would make sure that you wouldn't have to be excessive about your inline messaging.
This is what the message looks like on web/mobile:

So we don't need any of this then?

@isaackeyet The issue got automatically closed with the PR - I'm going to reopen this and do another PR for the UX improvements. :smile:

Gotcha :)

Closing issue and migrating to new repo.


net: custom DNS-over-TLS Resolver doesn't work anymore

Please answer these questions before submitting your issue. Thanks!
What version of Go are you using (go version)?
go version go1.11beta2 darwin/amd64

Does this issue reproduce with the latest release?
Yes, tested on
go version devel +4f9ae7739b Tue Jul 24 14:29:38 2018 +0000 darwin/amd64

What operating system and processor architecture are you using (go env)?
GOARCH="amd64"
GOBIN=""
GOCACHE="/Users/artyom/Library/Caches/go-build"
GOEXE=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOOS="darwin"
GOPATH="/Users/artyom/go"
GOPROXY=""
GORACE=""
GOROOT="/Users/artyom/Repositories/go"
GOTMPDIR=""
GOTOOLDIR="/Users/artyom/Repositories/go/pkg/tool/darwin_amd64"
GCCGO="gccgo"
CC="clang"
CXX="clang++"
CGO_ENABLED="1"
GOMOD=""
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/lb/3rk8rqs53czgb4v35w_342xc0000gn/T/go-build210171313=/tmp/go-build -gno-record-gcc-switches -fno-common"

What did you do?
https://play.golang.org/p/g1be5wdYdgy
This is an example of using Cloudflare's DNS-over-TLS. This code works in go1.10.3
What did you expect to see?
repro ¶ time ./repro-go1.10.3
resolver dial returned
resolver dial returned
[{<IP_ADDRESS> } {2606:2800:220:1:248:1893:25c8:1946 }]

real	0m0.417s
user	0m0.199s
sys	0m0.054s

What did you see instead?
repro ¶ time ./repro-go1.11beta2
resolver dial returned
resolver dial returned
resolver dial returned
resolver dial returned
i/o timeout

real	0m10.057s
user	0m0.250s
sys	0m0.053s
1 repro ¶ time ./repro-go+4f9ae7739b
resolver dial returned
resolver dial returned
resolver dial returned
resolver dial returned
i/o timeout

real	0m10.067s
user	0m0.242s
sys	0m0.054s

Caught this issue first on linux/arm binary built with go version go1.11beta2 darwin/amd64.

I modified example to log (unencrypted) data passed over TLS connections: https://play.golang.org/p/Lhb8bOP7pw6
Here's what results look like:
repro ¶ go version
go version go1.10.3 darwin/amd64
repro ¶ go run repro.go
resolver dial returned
resolver dial returned
conn 1 Write:
00000000  00 1d 50 45 01 00 00 01  00 00 00 00 00 00 07 65  |..PE...........e|
00000010  78 61 6d 70 6c 65 03 63  6f 6d 00 00 1c 00 01     |xample.com.....|
conn 2 Write:
00000000  00 1d 31 8d 01 00 00 01  00 00 00 00 00 00 07 65  |..1............e|
00000010  78 61 6d 70 6c 65 03 63  6f 6d 00 00 01 00 01     |xample.com.....|
conn 2 Read:
00000000  00 2d                                             |.-|
conn 2 Read:
00000000  31 8d 81 80 00 01 00 01  00 00 00 00 07 65 78 61  |1............exa|
00000010  6d 70 6c 65 03 63 6f 6d  00 00 01 00 01 c0 0c 00  |mple.com........|
00000020  01 00 01 00 00 0d e3 00  04 5d b8 d8 22           |.........].."|
conn 1 Read:
00000000  00 39                                             |.9|
conn 1 Read:
00000000  50 45 81 80 00 01 00 01  00 00 00 00 07 65 78 61  |PE...........exa|
00000010  6d 70 6c 65 03 63 6f 6d  00 00 1c 00 01 c0 0c 00  |mple.com........|
00000020  1c 00 01 00 00 09 2a 00  10 26 06 28 00 02 20 00  |......*..&.(.. .|
00000030  01 02 48 18 93 25 c8 19  46                       |..H..%..F|
[{<IP_ADDRESS> } {2606:2800:220:1:248:1893:25c8:1946 }]

This is for tip:
repro ¶ go version
go version devel +4f9ae7739b Tue Jul 24 14:29:38 2018 +0000 darwin/amd64
repro ¶ go run repro.go
resolver dial returned
resolver dial returned
conn 2 Write:
00000000  17 ca 01 00 00 01 00 00  00 00 00 00 07 65 78 61  |.............exa|
00000010  6d 70 6c 65 03 63 6f 6d  00 00 1c 00 01           |mple.com.....|
conn 1 Write:
00000000  ec 77 01 00 00 01 00 00  00 00 00 00 07 65 78 61  |.w...........exa|
00000010  6d 70 6c 65 03 63 6f 6d  00 00 01 00 01           |mple.com.....|
resolver dial returned
resolver dial returned
conn 4 Write:
00000000  ac 10 01 00 00 01 00 00  00 00 00 00 07 65 78 61  |.............exa|
00000010  6d 70 6c 65 03 63 6f 6d  00 00 01 00 01           |mple.com.....|
conn 3 Write:
00000000  96 09 01 00 00 01 00 00  00 00 00 00 07 65 78 61  |.............exa|
00000010  6d 70 6c 65 03 63 6f 6d  00 00 1c 00 01           |mple.com.....|
i/o timeout
exit status 1

Bisected this to 672729ebbd15e0b0dfac1ba22e35e92557215e1a

CC @mdempsky @iangudger

That code is broken. It always returns a TCP connection even if a UDP one is requested. The TCP and UDP DNS protocols are different.

@iangudger from the second paragraph of the doc for net.Resolver.Dial I assumed that UDP wire protocol is only used if Dial returned net.PacketConn, which tls.Conn does not implement, so RFC 7766 path expected here.
    // If the Conn returned is also a PacketConn, sent and received DNS
    // messages must adhere to RFC 1035 section 4.2.1, "UDP usage".
    // Otherwise, DNS messages transmitted over Conn must adhere
    // to RFC 7766 section 5, "Transport Protocol Selection".
    // If nil, the default dialer is used.

What am I missing here?

Wow, that is dumb. I will throw together a patch to fix compatibility.

Assigning as release-blocking due to it being a regression.

Wow, that is dumb.

Or by design. 🤷

Wow, that is dumb.
Or by design.

I think this is one of the design flaws of the net package API surface, unfortunately, from the beginning. Looks like representing various characteristics in a single interface, like net.Conn, is not good for people who don't care about the hidden hierarchy on the interface. Surely, we need a good solution, at least for application-layer transport API still increasing complexity by adding fancy features; security (TLS 1.3, 1.4 or above), multipath (MPTCP, QUIC), newly flow and congestion controls and platform-dependent control knobs, in Go 2.


TD 3111 Add Status Count Bar Component Bug

Contributes TD-3111
Changes
Changed Props of the StatusCountBar component, for the count object which currently is Record<Status, number>. This change is necessary, because there is an error currently in Virto Test. This error is caused by differences in types for status in ipguk/types and Status types in RUI. In RUI there are many types supported and listed below:
type Status = "aborted" | "completed" | "disrupted" | "errored" | "failed" | "no-metrics" | "not-ready" | "not-run" | "operational" | "passed" | "pending" | "queued" | "ready" | "running"
In ipguk/types there are the following status types:
type Status = "aborted" | "completed" | "errored" | "not-ready" | "queued" | "ready" | "running";
This is a mismatch and that's why I have updated the StatusCountBar component props instead of Status type in the record, to Partial.
A brief description of what this pull request solves / introduces.

Fixed type mismatch between the Test app types and RUI types.

Maybe a short description of what is not included and will come in future work where appropriate.
Dependencies
N/A
UI/UX
N/A
Testing notes
N/A
Author checklist
Before I request a review:

[x] I have reviewed my own code-diff.
~[ ] I have tested the changes in Docker / a deploy-preview.~
[x] I have assigned the PR to myself or an appropriate delegate.
[x] I have added the relevant labels to the PR.
~[ ] I have included appropriate tests.~
[x] I have checked that the Lint and Test workflows pass on Github.
~[ ] I have populated the deploy-preview with relevant test data.~

Looks to me like StatusCountBarProps and StatusCountTableProps are identical. Don't duplicate code. Just make one of them the master and the other one can simply reuse that type and rexport it as it's own.

I think you missed the point about the duplication. The whole type is duplicated, both title and count, not just count. Easier if I just fix it I think.

I think you missed the point about the duplication. The whole type is duplicated, both title and count, not just count. Easier if I just fix it I think.

I thought you are talking about the StatusCount, but now I see you meant the whole props type

I was thinking we can reuse this StatusCount type if needed somewhere else.

If you did need to do that (which we don't yet) you can just extract it. Don't need to export it per se.
import type {StatusCountTableProps} from "./StatusCountTable.types"

type StatusCount = StatusCountTableProps["count"];


listener: add note for the reuse port old default value compatible code

Signed-off-by: He Jie Xu<EMAIL_ADDRESS>Commit Message: listener: listener: add note for the reuse port old default value compatible code
Additional Description:
according the discussion https://github.com/envoyproxy/envoy/pull/19977#discussion_r812126652
Risk Level: low
Testing: n/a
Docs Changes: n/a
Release Notes: n/a
Platform Specific Features: n/a

cc @mattklein123


feat(batch): support extended declare query cursor

I hereby agree to the terms of the RisingWave Labs, Inc. Contributor License Agreement.
What's changed and what's your intention?
As titled, resolve https://github.com/risingwavelabs/risingwave/issues/17661
An example:
DECLARE c1 CURSOR FOR  SELECT id, trading_date, volume FROM public.tb_cjg_1 WHERE ((id = CAST($1 AS INT)));

Note: DECLARE SUBSCRIPTION CURSOR with parameter is not supported in this PR.
Checklist

[ ] I have written necessary rustdoc comments
[ ] I have added necessary unit tests and integration tests
[ ] I have added test labels as necessary. See details.
[ ] I have added fuzzing tests or opened an issue to track them. (Optional, recommended for new SQL features #7934).
[ ] My PR contains breaking changes. (If it deprecates some features, please create a tracking issue to remove them in the future).
[x] All checks passed in ./risedev check (or alias, ./risedev c)
[ ] My PR changes performance-critical code. (Please run macro/micro-benchmarks and show the results.)

[ ] My PR contains critical fixes that are necessary to be merged into the latest release. (Please check out the details)

Documentation

[ ] My PR needs documentation updates. (Please use the Release note section below to summarize the impact on users)

Release note
If this PR includes changes that directly affect users or other significant modifications relevant to the community, kindly draft a release note to provide a concise summary of these changes. Please prioritize highlighting the impact these changes will have on users.

Shall we add a test case in e2e_test/extended_mode?

Shall we add a test case in e2e_test/extended_mode?

Currently, we cannot send a extended query via psql or python (psycopg2), so I added some tests in integration test instead.

The integration test passed. https://buildkite.com/risingwavelabs/integration-tests/builds/937#0192bc47-eb61-411e-b512-42df73c50f86

Ping. Any update?

Ping. Any update?
Forget to merge.


fix: added cross-env

fixes 'NODE_ENV' is not recognized as an internal or external command on windows

Could you please:

Rebase (we merged some changes)
Remove Yarn lockfile (we want this repo to use npm)
Update the npm lockfile (just running npm install should do it)

Thanks!

Looks like https://github.com/reactjs/server-components-demo/pull/18 has this fix and another fix so I think we'll take that one, but thanks for the idea!

Looks like #18 has this fix and another fix so I think we'll take that one, but thanks for the idea!


Having a 2.10.0 and 2.10.0b installed on RZ causes conflicts

I installed a 2.10.0b for Cody to test out on the RZ. However, the RZ 2.10 version of visit now defaults to 2.10.0b, and so if you try to run visit 2.10 locally, it crashes the moment it tries to connect to a remote compute engine on rz
-----------------------REDMINE MIGRATION-----------------------
This ticket was migrated from Redmine. As such, not all
information was able to be captured in the transition. Below is
a complete record of the original redmine ticket.
Ticket number: 2482
Status: Pending
Project: VisIt
Tracker: Bug
Priority: Normal
Subject: Having a 2.10.0 and 2.10.0b installed on RZ causes conflicts
Assigned to: -
Category: -
Target version: -
Author: Kevin Griffin
Start: 12/15/2015
Due date:
% Done: 0%
Estimated time:
Created: 12/15/2015 06:30 pm
Updated: 12/15/2015 07:01 pm
Likelihood: 3 - Occasional
Severity: 2 - Minor Irritation
Found in version: 2.10.0
Impact:
Expected Use:
OS: All
Support Group: Any
Description:
I installed a 2.10.0b for Cody to test out on the RZ. However, the RZ 2.10 version of visit now defaults to 2.10.0b, and so if you try to run visit 2.10 locally, it crashes the moment it tries to connect to a remote compute engine on rz
Comments:

@brugger1 Could this have been avoided by installing with the '-private' flag?

‘-private’ adds a ‘+’ to the name, so it fixes the conflict, but then the internal launcher can’t start it, so I don’t consider it a solution.


Add an option to add also event sources, small translation cleanups

Description
Changes proposed in this pull request:

...
...
...

Related issue(s)

Resolves #800

/test pre-busola-smoke-test-docker

/lgtm

Luigi failed in replicaset test
/test pre-busola-integration-test-k3s

/retest

/test pre-busola-integration-test-k3s


Block entries in common::math::SolveLinearMPC kernel matrix_k

The block entries in kernel matrix matrix_k in common::math::SolveLinearMPC seem to be erroneously pre-multiplied by an additional matrix_a term . The lower-triangular kernel matrix matrix_k is defined according to the following code:
  std::vector<Matrix> matrix_a_power(horizon);
  matrix_a_power[0] = matrix_a;
  for (unsigned int i = 1; i < matrix_a_power.size(); ++i) {
    matrix_a_power[i] = matrix_a * matrix_a_power[i - 1];
  }

  Matrix matrix_k =
      Matrix::Zero(matrix_b.rows() * horizon, matrix_b.cols() * horizon);
  for (unsigned int r = 0; r < horizon; ++r) {
    for (unsigned int c = 0; c <= r; ++c) {
      matrix_k.block(r * matrix_b.rows(), c * matrix_b.cols(), matrix_b.rows(),
                     matrix_b.cols()) = matrix_a_power[r - c] * matrix_b;
    }
  }

This implies

The diagonal matrix_k is thus filled with matrix_a * matrix_b. However, this diagonal should really consist of just matrix_b, and all the other entries offset accordingly. In otherwords, initially we should write
matrix_a_power[0] = Matrix::Identity(matrix_a.rows(), matrix_a.rows());

Please let me know if this is a correct assessment. I argue this is the case, as block operations in matrix_m1 and matrix_m on stacked transition state/control vectors should be contemporaneous:
matrix_m.block(0, 0, matrix_a.rows(), 1) =
      matrix_a * matrix_initial_state + matrix_c;

@berthu Sorry that we could not get back to you on your request. Is this a question you still need an answer to?

Closing this issue as we have not heard back in a while. Feel free to reopen if you have additional questions. Thanks!


Current situation of the EPG repository

Hello,
As everyone know, this repository is in a dormant state since GitHub removed Actions from our repositories.
While their sanction is accurately taken (and we respect it), lots of persons have been asking for the return of the main functioning of the EPG system (being the file creation).
This is why #2017 has been created.
As of today, I partially take care of creating updated EPG files (as I only upload a few specific EPG files, cannot take care of all of them).
For now, we'll ask each and every person to be patient while we take a solution that will correspond to everyone's wantings, and make sure not to create any issue related to this.

Integration with WebGrab+Plus could be a solution?
http://<IP_ADDRESS>
http://<IP_ADDRESS>/epg-channels

Thanks for update.
I think alot of users have had issues with getting consistent updates as one day we'll get a user-to-user chat but won't know what the project updates are.
So thank you for at least doing the bare minimum.

Integration with WebGrab+Plus could be a solution?
http://<IP_ADDRESS> http://<IP_ADDRESS>/epg-channels

This is a freeware solution and so far we would prefer not relying on semi-paid resources to access any content.

Integration with WebGrab+Plus could be a solution?
http://<IP_ADDRESS> http://<IP_ADDRESS>/epg-channels

WGP+ is theft. You pay a fee for premium access then pay an additional fee for individual guides AND you're limited to certain number of EPGs

Is there any way to run updates locally? In my own server? Or only direct access to xml files here? Thanks.

Is there any way to run updates locally? In my own server? Or only direct access to xml files here? Thanks.

See https://github.com/iptv-org/epg#how-to-use

Thanks for update.
I think alot of users have had issues with getting consistent updates as one day we'll get a user-to-user chat but won't know what the project updates are.
So thank you for at least doing the bare minimum.

Thank you for your comprehension.
As of now i'm still doing a specific number of those, but with @freearhey 's recent changes we might make it faster. Stay tuned :)

Is there any way to run updates locally? In my own server? Or only direct access to xml files here? Thanks.

See https://github.com/iptv-org/epg#how-to-use

As I understood, this method is only for download and share files locally. Wrong?
I was asking a method to update epg data (each channel) locally.

@albertovm This manual is exactly how to download the manual yourself from one of the supported websites.
I have already added a small clarification to the text to make it clearer: https://github.com/iptv-org/epg/commit/963de4fb5ef2b110ca4767b29296ba287c5a71fc


Update Travis CI config

Codecov Report

Merging #35 (c216c3d) into master (47bb7f1) will decrease coverage by 3.14%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master      #35      +/-   ##
==========================================
- Coverage   54.97%   51.82%   -3.15%
==========================================
  Files           5        4       -1
  Lines         191      164      -27
==========================================
- Hits          105       85      -20
+ Misses         54       49       -5
+ Partials       32       30       -2

Impacted Files
Coverage Δ

reuseport.go
80.00% <0.00%> (-5.72%)
:arrow_down:

reuseport_linux.go
41.17% <0.00%> (-3.27%)
:arrow_down:

tcp.go
48.52% <0.00%> (-2.18%)
:arrow_down:

udp.go
53.62% <0.00%> (-1.94%)
:arrow_down:

reuseport_bsd.go

Continue to review full report at Codecov.

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

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@47bb7f1). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #35   +/-   ##
=========================================
  Coverage          ?   51.95%
=========================================
  Files             ?        5
  Lines             ?      179
  Branches          ?        0
=========================================
  Hits              ?       93
  Misses            ?       54
  Partials          ?       32

Continue to review full report at Codecov.

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


Update docs and add unittests

As requested. Update docs to reflect addition of new DINO models.
Added unittests in test_transformations.py for two of the models.
I then amended the naming conventions a bit in extractors.py because there was a discrepancy between model size and image patch resolution between the ViT and XCiT models. It makes it easier to follow a pattern, i.e. by specifying patch resolution with p prior to the number in the ViT numbers (which didn't do it already, but XCiT models did). This steps away from the naming conventions used in the source material, so I'm not sure it's the best idea but from a user perspective, I think it adds more clarity. However, might slip up someone who knows the original conventions. Basically, ViT-S/16 -> (dino)-vit-small-p16. I'm totally open to amending this to be more in line with naming conventions used elsewhere, if others disagree it is a helpful change. No problem to change it back again :)
Also, prior update had named the ResNet version of DINO (dino-resnet50) but I changed it to dino-rn50 to be in line with the naming convention of the other ResNet models in thingsvision.

I can see some unit tests are failing. I verified everything in local python but had trouble running the unit test properly on my laptop so if it's a really obvious mistake, would appreciate a pointer as someone else will likely see this soon as I won't be able to check back in for next 24 hours.

@Alxmrphi, could you rebase and try again? I think I've found the issue with the unittests.

As requested. Update docs to reflect addition of new DINO models.
Added unittests in test_transformations.py for two of the models.
I then amended the naming conventions a bit in extractors.py because there was a discrepancy between model size and image patch resolution between the ViT and XCiT models. It makes it easier to follow a pattern, i.e. by specifying patch resolution with p prior to the number in the ViT numbers (which didn't do it already, but XCiT models did). This steps away from the naming conventions used in the source material, so I'm not sure it's the best idea but from a user perspective, I think it adds more clarity. However, might slip up someone who knows the original conventions. Basically, ViT-S/16 -> (dino)-vit-small-p16. I'm totally open to amending this to be more in line with naming conventions used elsewhere, if others disagree it is a helpful change. No problem to change it back again :)
Also, prior update had named the ResNet version of DINO (dino-resnet50) but I changed it to dino-rn50 to be in line with the naming convention of the other ResNet models in thingsvision.

The naming is fine with me. I agree that this is indeed a helpful change.

I can see some unit tests are failing. I verified everything in local python but had trouble running the unit test properly on my laptop so if it's a really obvious mistake, would appreciate a pointer as someone else will likely see this soon as I won't be able to check back in for next 24 hours.

@Alxmrphi, could you rebase and try again? I think I've found the issue with the unittests and resolved it.

Done! I couldn't locate any in-line comments though. I had presumed you had solved an issue and then removed them, hence needing the rebase. However, if I've blindly overlooked them please let me know and I will amend the PR to incorporate them.

Codecov Report
Base: 74.63% // Head: 74.71% // Increases project coverage by +0.08% :tada:

Coverage data is based on head (b8033ab) compared to base (06f2992).
Patch has no changes to coverable lines.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #131      +/-   ##
==========================================
+ Coverage   74.63%   74.71%   +0.08%
==========================================
  Files          25       25
  Lines        1226     1226
  Branches      177      177
==========================================
+ Hits          915      916       +1
  Misses        259      259
+ Partials       52       51       -1

Flag
Coverage Δ

unittests
74.71% <ø> (+0.08%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

thingsvision/core/extraction/extractors.py
77.58% <ø> (+0.86%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

Done! I couldn't locate any in-line comments though. I had presumed you had solved an issue and then removed them, hence needing the rebase. However, if I've blindly overlooked them please let me know and I will amend the PR to incorporate them.

Now, you should be able to see my in-line comments. For some reason, they were only visible to me.

Yes, I saw the comments now. It looks like that extra line came along for the ride.

Argh, sorry about this. It's all so messy. It seems there was another deleted line that didn't show up, defining transform immediately preceding. This should fix it now.


Add navbar to view layout

Could use Bootstrap.

This issue needs more precision in its declaration. What should be available on the nav bar? Should any of it work? What pages do the buttons go to if they are there?

Nav bar will have a home button that navigates to the home page, an add ride button that will navigate to the add rides page, and a rides button that will navigate to a page where a list of available rides will be displayed. All buttons will work properly.


add doc for local dev setup for legend studio/query with depot/sdlc

This PR is stale because it has been open for 30 days with no activity. Please remove stale label or add any comment to keep this open. Otherwise this will be closed in 5 days.

This PR is stale because it has been open for 30 days with no activity. Please remove stale label or add any comment to keep this open. Otherwise this will be closed in 5 days.

This PR was closed because it has been inactive for 35 days. Please re-open if this PR is still relevant.


Support list of structs definition for the launch file generator

Thanks to the changes introduced here we are able to generate the following rosparam structure:
https://github.com/ipa-nhg/ros-model-examples/blob/master/RosSystems/agriculture_demo_sprint2/src-gen/demo/launch/demo.launch#L76-L94
From the system model:
https://github.com/ipa-nhg/ros-model-examples/blob/master/RosSystems/agriculture_demo_sprint2/demo.rossystem#L53-L57

@ipa-nhg  LGTM!


[Bug] Using sharp only works the first time, no error msg

I'm trying to generate multiple sizes of an array of image files. However this just works the first image, the next ones will fail. I'm using a static function, so there should be few error potential.
The error appears just after the input file changes. All iterations for the first file work properly.
Do I need to manually clear some sharp cache after all sizes were generated?
public static async createSizesOfImage(path: string, sizes?: number[]): Promise<number[]> {
    if (!sizes || sizes.length === 0) sizes = [250, 640, 1280, 1600, 2200];
    let error: Error, info: any;

    const generatedSizes: number[] = [];

    for (const size of sizes) {
        const newFilePath = path.replace('.', '_s' + size + '.');
        LoggerService.log('ImageService - createSizesOfImage(): Generating image s' + size);
        console.warn('hi i am here');
        [error, info] = await to(
            sharp(path)
                .resize(size, size, { fit: 'inside', withoutEnlargement: true })
                .toFile(newFilePath)
        );
        if (error) {
            console.error('ImageService - createSizesOfImage(): Failed to generate s' + size + ' of ' + path, error);
            throw new Error();
        }

        generatedSizes.push(size);
    }

    return generatedSizes;
}

Workaround: Pass a buffer to sharp() instead file path.

Yes, buffers are usually better to work with to avoid hitting the disk. I'm new to sharp, but I think that clone() might be a good fit for what you are doing.
http://sharp.pixelplumbing.com/en/stable/api-input/#clone

Also, out of curiosity. What's to()? Is it a wrapper for try-catch?

Does "The error appears just after the input file changes" mean that the contents of the input file is changing but the path of the input file remains the same? If so, try sharp.cache(false). http://sharp.pixelplumbing.com/en/stable/api-utility/#cache

sharp.cache(false) didn't help in this case. The file content itself remains the same all the time, just the path variable changes and provides the next file path from the array.
It seemed for me the readStream from sharp or something like that was not closed afterwards, so that a new file would be rejected by the instance, because sharp already holds an open file.
@msageryd yes, it's a try catch wrapper, adopted from Go language, which makes error handling really gorgeous. The function itself is pretty easy, it's worth it to take a look inside the code. await-to-js github

This should all work and I can't see anything obviously wrong with the code presented here so far. Are you able to provide a standalone code sample, removing some of the other dependencies such as to? Which version/platform is this?

Closing due to inactivity but please feel free to re-open with a standalone code sample if more help is required.


Cannot subscribe. Processor is already terminated

Describe the bug
When stopping/closing the first ServiceBusSenderClient/ServiceBusProcessorClient in our spring boot app while Application shutdown in k8s, this two errors are logged:
logger: com.azure.core.amqp.implementation.AmqpChannelProcessor
message: {"az.sdk.message":"Cannot subscribe. Processor is already terminated.","exception":"Cannot subscribe. Processor is already terminated.","connectionId":"MF_bdec02_1713997242025","entityPath":"$cbs"}
logger: reactor.core.publisher.Operators
message: Operator called default onErrorDropped
The error is logged everytime, when a deployment is shutting down and the first ServiceBusProcessorClient is closed. When closing the second client, no error is logged/happening. When the client was not running, no error is logged.
The error for the ServiceBusSenderClient.close() is happening inconsistly for our cronjobs.
Exception or Stack Trace
for ServiceBusSenderClient.close():
reactor.core.Exceptions$ErrorCallbackNotImplemented: java.lang.IllegalStateException: Cannot subscribe. Processor is already terminated.
Caused by: java.lang.IllegalStateException: Cannot subscribe. Processor is already terminated.
at com.azure.core.amqp.implementation.AmqpChannelProcessor.subscribe(AmqpChannelProcessor.java:270)
at reactor.core.publisher.Mono.subscribe(Mono.java:4568)
at reactor.core.publisher.MonoWhen$WhenCoordinator.request(MonoWhen.java:229)
at reactor.core.publisher.MonoIgnoreThen$ThenIgnoreMain.onSubscribe(MonoIgnoreThen.java:135)
at io.opentelemetry.javaagent.shaded.instrumentation.reactor.v3_1.TracingSubscriber.onSubscribe(TracingSubscriber.java:61)
at reactor.core.publisher.MonoWhen.subscribe(MonoWhen.java:101)
at reactor.core.publisher.Mono.subscribe(Mono.java:4568)
at reactor.core.publisher.MonoIgnoreThen$ThenIgnoreMain.subscribeNext(MonoIgnoreThen.java:265)
at reactor.core.publisher.MonoIgnoreThen.subscribe(MonoIgnoreThen.java:51)
at reactor.core.publisher.Mono.subscribe(Mono.java:4568)
at reactor.core.publisher.MonoIgnoreThen$ThenIgnoreMain.subscribeNext(MonoIgnoreThen.java:265)
at reactor.core.publisher.MonoIgnoreThen.subscribe(MonoIgnoreThen.java:51)
at reactor.core.publisher.Mono.subscribe(Mono.java:4568)
at reactor.core.publisher.Mono.subscribeWith(Mono.java:4634)
at reactor.core.publisher.Mono.subscribe(Mono.java:4395)
at com.azure.core.amqp.implementation.ReactorConnectionCache.dispose(ReactorConnectionCache.java:191)
at com.azure.messaging.servicebus.ServiceBusClientBuilder$V2StackSupport.onClientClose(ServiceBusClientBuilder.java:1280)
at com.azure.messaging.servicebus.ServiceBusSenderAsyncClient.close(ServiceBusSenderAsyncClient.java:762)
at com.azure.messaging.servicebus.ServiceBusSenderClient.close(ServiceBusSenderClient.java:474)
at  our code....
for ServiceBusProcessorClient.close():
reactor.core.Exceptions$ErrorCallbackNotImplemented: java.lang.IllegalStateException: Cannot subscribe. Processor is already terminated.
Caused by: java.lang.IllegalStateException: Cannot subscribe. Processor is already terminated.
at com.azure.core.amqp.implementation.AmqpChannelProcessor.subscribe(AmqpChannelProcessor.java:270)
at reactor.core.publisher.Mono.subscribe(Mono.java:4568)
at reactor.core.publisher.MonoWhen$WhenCoordinator.request(MonoWhen.java:229)
at reactor.core.publisher.MonoIgnoreThen$ThenIgnoreMain.onSubscribe(MonoIgnoreThen.java:135)
at io.opentelemetry.javaagent.shaded.instrumentation.reactor.v3_1.TracingSubscriber.onSubscribe(TracingSubscriber.java:61)
at reactor.core.publisher.MonoWhen.subscribe(MonoWhen.java:101)
at reactor.core.publisher.Mono.subscribe(Mono.java:4568)
at reactor.core.publisher.MonoIgnoreThen$ThenIgnoreMain.subscribeNext(MonoIgnoreThen.java:265)
at reactor.core.publisher.MonoIgnoreThen.subscribe(MonoIgnoreThen.java:51)
at reactor.core.publisher.Mono.subscribe(Mono.java:4568)
at reactor.core.publisher.MonoIgnoreThen$ThenIgnoreMain.subscribeNext(MonoIgnoreThen.java:265)
at reactor.core.publisher.MonoIgnoreThen.subscribe(MonoIgnoreThen.java:51)
at reactor.core.publisher.Mono.subscribe(Mono.java:4568)
at reactor.core.publisher.Mono.subscribeWith(Mono.java:4634)
at reactor.core.publisher.Mono.subscribe(Mono.java:4395)
at com.azure.core.amqp.implementation.ReactorConnectionCache.dispose(ReactorConnectionCache.java:191)
at com.azure.messaging.servicebus.ServiceBusClientBuilder$V2StackSupport.onClientClose(ServiceBusClientBuilder.java:1280)
at com.azure.messaging.servicebus.ServiceBusReceiverAsyncClient.close(ServiceBusReceiverAsyncClient.java:1509)
at com.azure.messaging.servicebus.ServiceBusProcessor$RollingMessagePump.lambda$beginIntern$2(ServiceBusProcessor.java:237)
at reactor.core.publisher.MonoUsing$MonoUsingSubscriber.cleanup(MonoUsing.java:191)
at reactor.core.publisher.MonoUsing$MonoUsingSubscriber.cancel(MonoUsing.java:185)
at reactor.core.publisher.Operators$MultiSubscriptionSubscriber.drainLoop(Operators.java:2425)
at reactor.core.publisher.Operators$MultiSubscriptionSubscriber.drain(Operators.java:2393)
at reactor.core.publisher.Operators$MultiSubscriptionSubscriber.cancel(Operators.java:2205)
at reactor.core.publisher.Operators$MultiSubscriptionSubscriber.drainLoop(Operators.java:2425)
at reactor.core.publisher.Operators$MultiSubscriptionSubscriber.drain(Operators.java:2393)
at reactor.core.publisher.Operators$MultiSubscriptionSubscriber.cancel(Operators.java:2205)
at reactor.core.publisher.FluxRetryWhen$RetryWhenMainSubscriber.cancel(FluxRetryWhen.java:167)
at reactor.core.publisher.SerializedSubscriber.cancel(SerializedSubscriber.java:157)
at reactor.core.publisher.LambdaMonoSubscriber.dispose(LambdaMonoSubscriber.java:217)
at reactor.core.Disposables$ListCompositeDisposable.dispose(Disposables.java:268)
at reactor.core.Disposables$ListCompositeDisposable.dispose(Disposables.java:161)
at com.azure.messaging.servicebus.ServiceBusProcessor$RollingMessagePump.dispose(ServiceBusProcessor.java:268)
at com.azure.messaging.servicebus.ServiceBusProcessor.close(ServiceBusProcessor.java:119)
at com.azure.messaging.servicebus.ServiceBusProcessorClient.close(ServiceBusProcessorClient.java:353)
at our code...
To Reproduce
Have a running spring boot app with a running ServiceBusProcessorClient. Let the container be replaced with a new deployment. In the logs you should find the error.
Code Snippet
// Creating the client:
public ServiceBusProcessorClient createServiceBusProcessorClient(String connectionString, String queueName) {
return new ServiceBusClientBuilder()
.transportType(AmqpTransportType.AMQP_WEB_SOCKETS)
.connectionString(connectionString)
.processor()
.queueName(queueName)
.prefetchCount(0)
.maxAutoLockRenewDuration(Duration.of(60L, ChronoUnit.MINUTES))
.maxConcurrentCalls(1)
.processMessage(this::processMessage)
.processError(this::processError)
.disableAutoComplete()
.buildProcessorClient();
}
// Start/Stop/Close:
@Override
public void start() {
try {
this.client = createServiceBusProcessorClient(connectionString, queueName);
((ServiceBusProcessorClient) client).start();
} catch (Exception e) {
LOGGER.error("Error while starting QueueProcessor {}", loggingContext, e);
}
}
@Override
public void stop() {
if (client != null) {
((ServiceBusProcessorClient) client).stop();
}
}
@Override
public void close() {
if (client != null) {
((ServiceBusProcessorClient) client).close();
}
}
// Component, which manages the clients:
@Component
public class DataReceiver implements DisposableBean {
private final List clients = new ArrayList<>();
// Clients is filled with multiple clients in the constructor and not shown here.

@Override
public void destroy() {
    for (ServiceBusReceiveClient client : clients) {
        if (client != null) {
            try {
                client.stop();
                client.close();
            } catch (Exception e) {
                LOGGER.error("Error while closing client", e);
            }
        }
    }
}

// Triggered from business logic.
public synchronized void startReceiving() {
    for (ServiceBusReceiveClient client : clients) {
        client.start();
    }
}

Expected behavior
No error message is logged
Screenshots

no screenshots

Setup:

OS: Win11
IDE: IntelliJ
Library/Libraries: "azure-messaging-servicebus" Version 7.16.0
Java version: 17
Frameworks: Spring Boot 3.2.5

Additional context

@frederikHartung-extern  thank you for reporting this!
Could you please confirm that the issue here is the noise introduced by logs or is there something else that does not work?

I confirm, that just the ERROR Logs a problem for me. I didnt notice any other errors or that something is not working.

Level for this logging has been changed to warn in this pr, so no longer reported as error

Thanks!


Cannot chain multiple ilike or filter with ilike

Unable to chain multiple ilike like this
supabase.table("products").select("*", count='exact').filter("name","ilike",f"%KitKat%").filter("category","ilike",f"%Candy Bar%").order(sort).limit(limit).execute()
Getting error like

Still getting the same error on single ilike as well

Please check if your project is paused in the Supabase dashboard.

I am also getting those requests when using any kind of filtering, like:

ilike
like
filter

One operation that works is eq. Using version 2.4.5

Also receiving these errors as soon as i use % in ilike. If i drop them the query executes fine.

The % in the built query is causing your error, as the % is used for url encoding. Use * instead.

Yeah adding * instead of % worked.

@silentworks can you try chaining multiple ilikes as single ilike is working. I said multiple ilike chaining
for example select().ilike("title","%Home%").ilike("description","%father%").
It is now working when I replace % with *

@harshdeep-techstax Its the same results for me with either single of multiple chaining ilikes.
https://github.com/supabase-community/supabase-py/assets/79497/2f1c360e-15ba-4f90-abe6-ee41021a82f4

Closing as this is no longer an issue as demonstrated by the video in the previous comment.

Having the same problem, looks like the fix works


bugfix: support more built-in parameters when set chash balancer

bugfix: support more built-in parameters when set chash balancer
Summary
support more uri|server_name|server_addr|request_uri|remote_port|remote_addr|query_string|host|hostname)|arg_[0-9a-zA-z_-]+
Full changelog

lua\apisix\schema_def.lua
t\admin\upstream.t
t\node\chash-balance.t
doc\architecture-design.md
doc\architecture-design-cn.md

Issues resolved
Fix https://github.com/apache/incubator-apisix/issues/717

@lilien1010 There are some minor issues with the documentation, and the rest is fine. We will merge after you modify it.


SIP Video needed

Dear Team,
I am successfully installed openfire, ofmeet, sip-pluging and ofswitch. WebRTC is good enough and sip call through phone also good. I'm very much happy to using openfire-ofmeet. But my major needs is sip video call which may not available right now or may be needed to further configuration.
If I get any suggestion about enabling sip video call that will be very helpful to me
Thank you,

It should be easy to do this now. Openfire Meetings client now includes sip.js which is used for SIP audio conferencing with FreeSwitch using ofswitch plugin.

Dear Dele,
Yes it was successfully implemented sip audio. But I need sip video. Have
there any option to get sip video in ofmeet?
Thank you,
Tarikul
On Nov 7, 2016 4:26 PM, "Dele Olajide"<EMAIL_ADDRESS>wrote:

It should be easy to do this now. Openfire Meetings client now includes
sip.js which is used for SIP audio conferencing with FreeSwitch using
ofswitch plugin.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/igniterealtime/community-plugins/issues/23#issuecomment-258798944,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AUUrmhAtmOn1ptb6Wt5fXuVEtnusr1Qkks5q7vy-gaJpZM4KF2Hz
.

Ok. I will have a look at it. It will require FreeSwitch and the ofswitch plugin

I have both.
On Nov 7, 2016 5:38 PM, "Dele Olajide"<EMAIL_ADDRESS>wrote:

Ok. I will have a look at it. It will require FreeSwitch and the ofswitch
plugin
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/igniterealtime/community-plugins/issues/23#issuecomment-258813790,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AUUrmtQxLhEbK1LUsm10FiG0voq3LWtMks5q7w2fgaJpZM4KF2Hz
.

Dear Dele,
Have there any update about this issue.
Thank you

Is there any update about this issue

Still now there are no update.
Thank you,
Tarikul
On Sun, Jun 3, 2018 at 1:17 PM, cnscns<EMAIL_ADDRESS>wrote:

Is there any update about this issue
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/igniterealtime/community-plugins/issues/23#issuecomment-394142530,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AUUrmj_vpXSvl415qRL0g3U5YX4IH8bSks5t441sgaJpZM4KF2Hz
.

--
MD. TARIKUL ISLAM
3B, 3rd Floor, H#340, R#6, Kajipara, Mirpur, Dhaka-1216, Bangladesh.
Mob:<PHONE_NUMBER>181
*E-mail: *<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>.
*Skype/G-Talk/Yahoo/Nimbuzz: *sunshuvo.

Still now there are no update.

This version of openfire meetings in this repository has been deprecated. See the discussion here for more info about SIP integration with Openfire Meetings. Only audio is being considered.
I don't see SIP Video happening with Openfire Meetings. It is easier to use FreeSWITCH Verto Communicator for MCU type video conferencing.  That is what I added to Pade, the Openfire Meetings chrome extension.


Include .mdx in the title of a blog file

The .mdx file extension was missing from the 2023-mini-hackathon blog

Great catch! However, that file is actually supposed to be deleted~ Updated your PR and merging now! Will count toward your score tho :)


SSL error on OSX

$ cargo update --verbose
    Updating registry `https://github.com/rust-lang/crates.io-index`
Unable to update registry https://github.com/rust-lang/crates.io-index

Caused by:
  failed to fetch `https://github.com/rust-lang/crates.io-index`

Caused by:
  [12] SSL error: error:140E0114:SSL routines:SSL_shutdown:uninitialized
$ cargo --version
cargo 0.3.0-nightly (1777ab7 2015-06-07) (built 2015-06-07)

Was this just an intermittent error? This is just a normal network error so there's not really much Cargo can do with this (other than perhaps retry, which is covered by another issue). Can you curl that URL successfully?

This was an intermittent error, worked fine on a retry. Thanks @alexcrichton !


pydot error.

@xamyzhao Hi, thanks for your excellent work. But when I run python main.py trans --gpu 1 --data mri-100unlabeled --model flow-bidir by default, there are following errors.

stdout, stderr:
b''
b'Format: "jpg" not recognized. Use one of: canon cmap cmapx cmapx_np dot dot_json eps fig gv imap imap_np ismap json json0 mp pdf pic plain plain-ext png pov ps ps2 svg svgz tk vml vmlz xdot xdot1.2 xdot1.4 xdot_json\n'
Traceback (most recent call last):
File "main.py", line 337, in
test_every_n_epochs=test_every_n_epochs)
File "/root/userfolder/workspace_python/brainstorm/src/experiment_engine.py", line 109, in run_experiment
exp.create_models()
File "/root/userfolder/workspace_python/brainstorm/src/transform_models.py", line 442, in create_models
super(TransformModelTrainer, self).create_models()
File "/root/userfolder/workspace_python/brainstorm/src/experiment_base.py", line 77, in create_models
self._print_models()
File "/root/userfolder/workspace_python/brainstorm/src/experiment_base.py", line 161, in _print_models
show_shapes=True)
File "/root/userfolder/anaconda3/lib/python3.6/site-packages/keras/utils/vis_utils.py", line 138, in plot_model
dot.write(to_file, format=extension)
File "/root/userfolder/anaconda3/lib/python3.6/site-packages/pydot.py", line 1817, in write
s = self.create(prog, format, encoding=encoding)
File "/root/userfolder/anaconda3/lib/python3.6/site-packages/pydot.py", line 1945, in create
assert process.returncode == 0, process.returncode
AssertionError: 1

It seems that this might be an issue with your pydot or graphviz installation: https://stackoverflow.com/questions/33678636/pydot-error-file-format-png-not-recognized. Alternatively, you could remove the calls to plot_model -- they are useful for debugging your network architecture, but not necessary for training.

@xamyzhao Hi, Amy! Thanks for your great job! When I run python main.py trans --gpu 1 --data mri-100unlabeled --model flow-bidir , I got the error as follows, but I think the input shape is supposed to be (None, 160,192,224,1) right? Or do I miss something?
Traceback (most recent call last):
File "main.py", line 355, in
test_every_n_epochs=test_every_n_epochs)
File "/Users/apple/brainstorm-master/src/experiment_engine.py", line 133, in run_experiment
run_metadata=None,
File "/Users/apple/brainstorm-master/src/experiment_engine.py", line 184, in train_batch_by_batch
joint_loss, joint_loss_names = exp.train_on_batch()
File "/Users/apple/brainstorm-master/src/transform_models.py", line 589, in train_on_batch
loss_vals = self.transform_model.train_on_batch(X, Y)
File "/Users/apple/anaconda3/lib/python3.6/site-packages/keras/engine/training.py", line 1826, in train_on_batch
check_batch_axis=True)
File "/Users/apple/anaconda3/lib/python3.6/site-packages/keras/engine/training.py", line 1411, in _standardize_user_data
exception_prefix='target')
File "/Users/apple/anaconda3/lib/python3.6/site-packages/keras/engine/training.py", line 153, in _standardize_input_data
str(array.shape))
ValueError: Error when checking target: expected vm2_cc_fwd to have shape (None, 160, 192, 224, 3) but got array with shape (1, 160, 192, 224, 1)
Thank you very much!

@xamyzhao Dear Amy, thanks for your reply. But there is still a problem. It seems to be  the OOM error. The free memory of my GPU is about 11.7GB, unfortunately It happens this. Can you tell me what is your GPU configuration or how to avoid it ?

2019-05-02 22:42:07.649458: I tensorflow/core/common_runtime/bfc_allocator.cc:680] Stats:
Limit:<PHONE_NUMBER>8
InUse:<PHONE_NUMBER>
MaxInUse:<PHONE_NUMBER>
NumAllocs:                    1702
MaxAllocSize:<PHONE_NUMBER>
2019-05-02 22:42:07.649560: W tensorflow/core/common_runtime/bfc_allocator.cc:279] _______**********************************************____*******************
2019-05-02 22:42:07.649622: W tensorflow/core/framework/op_kernel.cc:1318] OP_REQUIRES failed at conv_ops_3d.cc:398 : Resource exhausted: OOM when allocating tensor with shape[1,32,160,192,224] and type float on /job:localhost/replica:0/task:0/device:GPU:0 by allocator GPU_0_bfc
Traceback (most recent call last):
File "main.py", line 337, in
test_every_n_epochs=test_every_n_epochs)
File "/root/userfolder/workspace_python/brainstorm/src/experiment_engine.py", line 133, in run_experiment
run_metadata=None,
File "/root/userfolder/workspace_python/brainstorm/src/experiment_engine.py", line 184, in train_batch_by_batch
joint_loss, joint_loss_names = exp.train_on_batch()
File "/root/userfolder/workspace_python/brainstorm/src/transform_models.py", line 569, in train_on_batch
loss_vals = self.transform_model.train_on_batch(X, Y)
File "/root/userfolder/anaconda3/lib/python3.6/site-packages/keras/engine/training.py", line 1217, in train_on_batch
outputs = self.train_function(ins)
File "/root/userfolder/anaconda3/lib/python3.6/site-packages/keras/backend/tensorflow_backend.py", line 2715, in call
return self._call(inputs)
File "/root/userfolder/anaconda3/lib/python3.6/site-packages/keras/backend/tensorflow_backend.py", line 2675, in _call
fetched = self._callable_fn(*array_vals)
File "/root/userfolder/anaconda3/lib/python3.6/site-packages/tensorflow/python/client/session.py", line 1454, in call
self._session._session, self._handle, args, status, None)
File "/root/userfolder/anaconda3/lib/python3.6/site-packages/tensorflow/python/framework/errors_impl.py", line 519, in exit
c_api.TF_GetCode(self.status.status))
tensorflow.python.framework.errors_impl.ResourceExhaustedError: OOM when allocating tensor with shape[1,160,192,224,32] and type float on /job:localhost/replica:0/task:0/device:GPU:0 by allocator GPU_0_bfc
[[Node: vm2_cc_bck/unet_dec_conv3D_3_0/convolution = Conv3D[T=DT_FLOAT<EMAIL_ADDRESS>data_format="NDHWC", dilations=[1, 1, 1, 1, 1], padding="SAME", strides=[1, 1, 1, 1, 1], _device="/job:localhost/replica:0/task:0/device:GPU:0"](vm2_cc_bck/leaky_re_lu_7/LeakyRelu, unet_dec_conv3D_3_0/kernel/read)]]
Hint: If you want to see a list of allocated tensors when OOM happens, add report_tensor_allocations_upon_oom to RunOptions for current allocation info.
[[Node: loss/vm2_cc_fwd_loss/Mean_3/_621 = _Recvclient_terminated=false,recv_device="/job:localhost/replica:0/task:0/device:CPU:0",send_device="/job:localhost/replica:0/task:0/device:GPU:0",send_device_incarnation=1,tensor_name="edge_16995_loss/vm2_cc_fwd_loss/Mean_3", tensor_type=DT_FLOAT,_device="/job:localhost/replica:0/task:0/device:CPU:0"]]
Hint: If you want to see a list of allocated tensors when OOM happens, add report_tensor_allocations_upon_oom to RunOptions for current allocation info.

@chengjianhong I just added a flow-fwd and flow-bck option to train the two spatial models separately if they don't both fit on the GPU (simply replace --model flow-bidir with --model flow-fwd). Let me know if you run into any issues.

@Liiiii2101 can you also try using --model flow-fwd instead of --model flow-bidir? Thanks.

@xamyzhao Dear Amy. I run it according to your suggestion. The error is as follow.

2019-05-03 12:45:16.718132: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1084] Created TensorFlow device (/job:localhost/replica:0/task:0/device:GPU:0 with 11366 MB memory) -> physical GPU (device: 0, name: TITAN Xp, pci bus id: 0000:83:00.0, compute capability: 6.1)
Traceback (most recent call last):
File "main.py", line 199, in
arch_params = named_arch_params[args.model]
KeyError: 'flow-fwd'

@chengjianhong can you check if you pulled the latest version?

Hi, Amy! After using --model flow-bidir, I still got the same error:
Traceback (most recent call last):
File "main.py", line 355, in
test_every_n_epochs=test_every_n_epochs)
File "/Users/apple/brainstorm-master/src/experiment_engine.py", line 133, in run_experiment
run_metadata=None,
File "/Users/apple/brainstorm-master/src/experiment_engine.py", line 184, in train_batch_by_batch
joint_loss, joint_loss_names = exp.train_on_batch()
File "/Users/apple/brainstorm-master/src/transform_models.py", line 589, in train_on_batch
loss_vals = self.transform_model.train_on_batch(X, Y)
File "/Users/apple/anaconda3/lib/python3.6/site-packages/keras/engine/training.py", line 1826, in train_on_batch
check_batch_axis=True)
File "/Users/apple/anaconda3/lib/python3.6/site-packages/keras/engine/training.py", line 1411, in _standardize_user_data
exception_prefix='target')
File "/Users/apple/anaconda3/lib/python3.6/site-packages/keras/engine/training.py", line 153, in _standardize_input_data
str(array.shape))
ValueError: Error when checking target: expected flow to have shape (None, 160, 192, 224, 3) but got array with shape (1, 160, 192, 224, 1)

@Liiiii2101 did you try my suggestion of using --model flow-fwd instead of --model flow-bidir? I will be deprecating support for flow-bidir since it does not fit on many standard GPUs.

Hi, Amy.
I am also facing OOM issues even after using --model flow-fwd.
I have only 8GB on my GPU.
Stats:
Limit:<PHONE_NUMBER>
InUse:<PHONE_NUMBER>
MaxInUse:<PHONE_NUMBER>
NumAllocs:                    1699
MaxAllocSize:<PHONE_NUMBER>
Is there any workaround on this?
Thanks!

Hi, Amy! After using --model flow-bidir, I still got the same error:
Traceback (most recent call last):
File "main.py", line 355, in
test_every_n_epochs=test_every_n_epochs)
File "/Users/apple/brainstorm-master/src/experiment_engine.py", line 133, in run_experiment
run_metadata=None,
File "/Users/apple/brainstorm-master/src/experiment_engine.py", line 184, in train_batch_by_batch
joint_loss, joint_loss_names = exp.train_on_batch()
File "/Users/apple/brainstorm-master/src/transform_models.py", line 589, in train_on_batch
loss_vals = self.transform_model.train_on_batch(X, Y)
File "/Users/apple/anaconda3/lib/python3.6/site-packages/keras/engine/training.py", line 1826, in train_on_batch
check_batch_axis=True)
File "/Users/apple/anaconda3/lib/python3.6/site-packages/keras/engine/training.py", line 1411, in _standardize_user_data
exception_prefix='target')
File "/Users/apple/anaconda3/lib/python3.6/site-packages/keras/engine/training.py", line 153, in _standardize_input_data
str(array.shape))
ValueError: Error when checking target: expected flow to have shape (None, 160, 192, 224, 3) but got array with shape (1, 160, 192, 224, 1)

Hi Amy,
When I try to train transform models I have used all these three commands, including --model flow-fwd, -bck and -bidir, but there is a similar error:
Traceback (most recent call last):
File "main.py", line 357, in
test_every_n_epochs=test_every_n_epochs)
File "/home/taozhong/下载/brainstorm-master/src/experiment_engine.py", line 133, in run_experiment
run_metadata=None,
File "/home/taozhong/下载/brainstorm-master/src/experiment_engine.py", line 184, in train_batch_by_batch
joint_loss, joint_loss_names = exp.train_on_batch()
File "/home/taozhong/下载/brainstorm-master/src/transform_models.py", line 589, in train_on_batch
loss_vals = self.transform_model.train_on_batch(X, Y)
File "/home/taozhong/.local/lib/python3.6/site-packages/keras/engine/training.py", line 1826, in train_on_batch
check_batch_axis=True)
File "/home/taozhong/.local/lib/python3.6/site-packages/keras/engine/training.py", line 1411, in _standardize_user_data
exception_prefix='target')
File "/home/taozhong/.local/lib/python3.6/site-packages/keras/engine/training.py", line 153, in _standardize_input_data
str(array.shape))
ValueError: Error when checking target: expected flow to have shape (None, 160, 192, 224, 3) but got array with shape (1, 160, 192, 224, 1)

Hi, Amy! After using --model flow-bidir, I still got the same error:
Traceback (most recent call last):
File "main.py", line 355, in
test_every_n_epochs=test_every_n_epochs)
File "/Users/apple/brainstorm-master/src/experiment_engine.py", line 133, in run_experiment
run_metadata=None,
File "/Users/apple/brainstorm-master/src/experiment_engine.py", line 184, in train_batch_by_batch
joint_loss, joint_loss_names = exp.train_on_batch()
File "/Users/apple/brainstorm-master/src/transform_models.py", line 589, in train_on_batch
loss_vals = self.transform_model.train_on_batch(X, Y)
File "/Users/apple/anaconda3/lib/python3.6/site-packages/keras/engine/training.py", line 1826, in train_on_batch
check_batch_axis=True)
File "/Users/apple/anaconda3/lib/python3.6/site-packages/keras/engine/training.py", line 1411, in _standardize_user_data
exception_prefix='target')
File "/Users/apple/anaconda3/lib/python3.6/site-packages/keras/engine/training.py", line 153, in _standardize_input_data
str(array.shape))
ValueError: Error when checking target: expected flow to have shape (None, 160, 192, 224, 3) but got array with shape (1, 160, 192, 224, 1)

Hi Amy,
When I try to train transform models I have used all these three commands, including --model flow-fwd, -bck and -bidir, but there is a similar error:
Traceback (most recent call last):
File "main.py", line 357, in
test_every_n_epochs=test_every_n_epochs)
File "/home/taozhong/下载/brainstorm-master/src/experiment_engine.py", line 133, in run_experiment
run_metadata=None,
File "/home/taozhong/下载/brainstorm-master/src/experiment_engine.py", line 184, in train_batch_by_batch
joint_loss, joint_loss_names = exp.train_on_batch()
File "/home/taozhong/下载/brainstorm-master/src/transform_models.py", line 589, in train_on_batch
loss_vals = self.transform_model.train_on_batch(X, Y)
File "/home/taozhong/.local/lib/python3.6/site-packages/keras/engine/training.py", line 1826, in train_on_batch
check_batch_axis=True)
File "/home/taozhong/.local/lib/python3.6/site-packages/keras/engine/training.py", line 1411, in _standardize_user_data
exception_prefix='target')
File "/home/taozhong/.local/lib/python3.6/site-packages/keras/engine/training.py", line 153, in _standardize_input_data
str(array.shape))
ValueError: Error when checking target: expected flow to have shape (None, 160, 192, 224, 3) but got array with shape (1, 160, 192, 224, 1)

When I update my keras2.1.1 to 2.1.2, this problem was solved. But that happened another error:
7] Stats:
Limit:<PHONE_NUMBER>2
InUse:<PHONE_NUMBER>
MaxInUse:<PHONE_NUMBER>
NumAllocs:                    1623
MaxAllocSize:<PHONE_NUMBER>
......
[[{{node loss/vm2_cc_fwd_loss_1/Mean_5/_523}} = _Recvclient_terminated=false, recv_device="/job:localhost/replica:0/task:0/device:CPU:0", send_device="/job:localhost/replica:0/task:0/device:GPU:0", send_device_incarnation=1, tensor_name="edge_16989_loss/vm2_cc_fwd_loss_1/Mean_5", tensor_type=DT_FLOAT, _device="/job:localhost/replica:0/task:0/device:CPU:0"]]
Hint: If you want to see a list of allocated tensors when OOM happens, add report_tensor_allocations_upon_oom to RunOptions for current allocation info.
Even when I tried all these three model type (-fwd, -bck and bidir) it still happen.

Hi, Amy.
I am also facing OOM issues even after using --model flow-fwd.
I have only 8GB on my GPU.
Stats:
Limit:<PHONE_NUMBER>
InUse:<PHONE_NUMBER>
MaxInUse:<PHONE_NUMBER>
NumAllocs: 1699
MaxAllocSize:<PHONE_NUMBER>
Is there any workaround on this?
Thanks!

Hi,
Have you solved this problem?

@jacobxchan @TaoZhong11 I trained my models with a 12GB GPU (sorry, I need to update the requirements in the README). With 8GB, you will likely need to spatially downsample your training volumes.

@jacobxchan @TaoZhong11 I trained my models with a 12GB GPU (sorry, I need to update the requirements in the README). With 8GB, you will likely need to spatially downsample your training volumes.

I crop the training volume to 144x148x188 and run it in 1080ti, 12G and this errors still happen.

@jacobxchan @TaoZhong11 I trained my models with a 12GB GPU (sorry, I need to update the requirements in the README). With 8GB, you will likely need to spatially downsample your training volumes.

I further downsample  the data and the model finally work. Thank you for your excellent work!


Revert "Add organisations to statutory instruments"

Reverts alphagov/govuk-content-schemas#777
These should have been placed in edition_links

@steventux Did you intend to also put them in the right place here?  Or do they already exist in that place in the schema?


InfoQ Software Architecture and Design Trends Report - April 2024 / InfoQ

URL: https://www.infoq.com/articles/architecture-trends-2024
Review (1-2 sentences): Have you already heard about architecture as a team sport? About cell-based architecture? Review the latest innovations and trends in the design trends report ;)

Published in Digest #36.


spago: 0.20.1 -> 0.20.2

Motivation for this change
New version of spago was released: https://github.com/purescript/spago/releases/tag/0.20.2
Things done

[X] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[ ] macOS
[X] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

cc @f-f

@ofborg build spago


[Menu] Menu does not lock scroll on some iOS devices

[x] The issue is present in the latest release.
[x] I have searched the issues of this repository and believe that this is not a duplicate.

Current Behavior 😯
When using Menu and opening it on some iOS devices, the browser does not lock the scroll so you can scroll over the site with the floating Menu open and fixed on the original position.
I noticed the problem doing QA on a project. It works fine on latest version of Chrome and Safari on macOS, but it is present using Safari on an iPhone 8.
Expected Behavior 🤔
When a Menu is opened, site scrolling should be be disabled.
Steps to Reproduce 🕹
The problem could be recreated opening the official docs using an iPhone 8.
On the next image, the menu is open by clicking on the "Open Menu" button and then scrolling down.

Duplicate of #23606

Duplicate of #23606


point to dist version

otherwise you have to download and install the package.json dependencies

Thanks, I missed that.


Not working with Dioxus Fullstack

I'm uncertain to what is causing this but wanted to report it. When dioxus full stack is in use with wasi-sol the app will not run. This can be replicated just by updating the dioxus example in a couple of places and then running the dev server.
Cargo.toml
[package]
name = "wasi-sol-example"
version = "0.1.0"
edition = "2021"

[dependencies]
console_error_panic_hook = "0.1.7"
log = "0.4.21"
wasi-sol = { path = "../../", features = ["dio"] }
wasm-logger = "0.2.0"
dioxus = { version = "0.5", features = ["web", "fullstack" ] }

[features]
default = []
server = ["dioxus/axum"]
web = ["dioxus/web"]

main function in main.rs
fn main() {
    console_error_panic_hook::set_once();
    wasm_logger::init(wasm_logger::Config::default());

    #[cfg(feature = "server")]
    dioxus::fullstack::launch::launch(app, Vec::default(), dioxus::fullstack::Config::default());
}

Then run the following command
dx serve --platform fullstack --hot-reload

You'll get the following output along with no app running
Dioxus @ v0.5.4 [12:34:46]

    > Hot Reload Mode: RSX
    > Watching: [ src, assets, Cargo.toml, Dioxus.toml ]
    > Custom index.html: None
    > Serve index.html on 404: True

    > Build Features: [ server ]
    > Build Profile: Debug
    > Build took: 83138 millis

Listening on <IP_ADDRESS>:8080
panicked at /Users/painedpineapple/.cargo/registry/src/index.crates.io-6f17d22bba15001f/wasm-bindgen-0.2.92/src/lib.rs:1017:1:
function not implemented on non-wasm32 targets
panicked at /Users/painedpineapple/.cargo/registry/src/index.crates.io-6f17d22bba15001f/dioxus-fullstack-0.5.2/src/render.rs:97:60:
called `Result::unwrap()` on an `Err` value: RecvError(())

Thanks for the experiments, @painedpineapple!
Currently, the crate only supports client-side rendering, meaning all components are rendered in the browser. Adding support for server-side rendering would be awesome for performance/seo reasons.
PRs are welcome as usual, so feel free to contribute if you'd like.


Initializing a temporary file in the user system with un-predictable …

Security Hotspot:
Creating a temporary file in the user system with a predictable name.
Explanation:
The temporary folders are generally created in the System. If the files saved by the application have predictable names, then an attacker may try to inject the virus into the system by creating a similar file before the application does. Moreover, these folders are not protected, it will be a very flexible attack for a hacker to launch.
Solution:
The file names should be random and complex for an attacker to predict easily.

The change does not create a random filename, though... Not correct.


Bug: YAML Title rule incorrectly considers #tag after header as a part of the title

Describe the Bug
YAML Title rule incorrectly considers #tag after header as a part of the title
How to Reproduce
Markup
# title
#tag

YAML Title rule.
Expected Behavior
Expected output if applicable:
---
title: title
---
# title
#tag

Actual Behavior
Expected output if applicable:
---
title: title
#tag
---
# title
#tag

Thanks for reporting this. It should be an easy fix layer on today unless you already have a fix.


Twitter bot issue:  should only tweet opportunities published on prod

Currently the twitter bot is tweeting opportunities from both prod and dev.
This should be changed so that the twitter bot only tweets opportunities on prod and never the ones on dev

Dev should no longer be tweeting out any opportunites 👍

Thanks so much @markwooff !!


VDS-1572: Update webstore

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.6 out of 7 committers have signed the CLA.:white_check_mark: trueboroda:white_check_mark: ilyawzrd:white_check_mark: asvishnyakov:white_check_mark: AleksandraBalabanova:white_check_mark: pushnitsa:white_check_mark: omnidark:x: vc-civc-ci seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Update fiwalk Guide with Running fiwalk with the ClamAV Plugin Quick Start Guide Instructions

Add Running fiwalk with the ClamAV Plugin to Scan Disk Images for Viruses and Malware from Quickstart Guide to Fiwalk Guide.

Branched and ready for review. There is a comment in there about an issue formatting a code block that I could not figure out.


Error in authorization rules (entity-rule mismatch) does not block the server to start

If we have a mismatch in entities and rules, this error is thrown and the whole db-authorization plugin initialization fails.
This is expected, but this should block the server to start so the user should be able to see the message and fix the authorization configuration.
What it actually happens is that the server starts, no error is printed but the whole authorization plugin is not configured and as result no authorization check is in place.
This is a regression, and it's probably due to the fact that we are now running this initialization on onReady hook.

After some investigation, we found out that the problem is in fastify/avvio (when  pluginTimeout: 0) and we fixed it here.


ERR_UNKNOWN_FILE_EXTENSION

`
$ bin/dev.js hello
(node:80707) [ERR_UNKNOWN_FILE_EXTENSION] TypeError Plugin: oclif-hello-world [ERR_UNKNOWN_FILE_EXTENSION]: Unknown file extension ".ts" for /Users/Downloads/hello-world-esm-main/src/commands/hello/index.ts
module<EMAIL_ADDRESS>task: toCached
plugin: oclif-hello-world
See more details with DEBUG=*
(Use node --trace-warnings ... to show where the warning was created)
(node:80707) [ERR_UNKNOWN_FILE_EXTENSION] TypeError Plugin: oclif-hello-world [ERR_UNKNOWN_FILE_EXTENSION]: Unknown file extension ".ts" for /Users/Downloads/hello-world-esm-main/src/commands/hello/world.ts
module<EMAIL_ADDRESS>task: toCached
plugin: oclif-hello-world
See more details with DEBUG=*
Say hello to the world and others
USAGE
$ oex hello COMMAND
`
oclif-hello-world/0.1.1 darwin-arm64 node-v19.8.1

This template is not expected to work until https://github.com/oclif/core/pull/759 is merged


Consider using CSS containment

CSS containment should fit extremely well with the way CodeMirror lays out the editing surface, possibly providing a big performance boost.
CSS containment is added with the new contain property, which can have values of size || layout || style || paint.

https://developers.google.com/web/updates/2016/06/css-containment
https://developer.mozilla.org/en-US/docs/Web/CSS/contain

layout isolates an element's layout so that nothing outside it can effect its layout and vice-versa. It reduces the need to check other elements when layout changes. I think this is safe as all text elements in CodeMirror don't appear to have styling that would effect either other's layout - no margins at least.
paint clips an element so neither it of its children can paint outside its bounds. I think this is safe to apply to text elements too.
style means no properties like counters can be effected by the element. Its probably a minor win.
size means you have to provide an explicit size. You might be able to use this for pre.CodeMirror-line`, but won't be able to for spans.
content is layout + paint + style, and I think is safe for most elements.
strict is content + size, and hard to apply everywhere.
I tried adding containment to a few rules in the online demo and these additions seemed to work fine:
.CodeMirror-cursor {
  contain: strict;
}
.CodeMirror pre {
  contain: content;
}
.CodeMirror pre span {
  contain: content;
  display: inline-block;
}

The more elements you can apply containment to the better.
I'm not sure if you have a benchmark you can try, but from talking to Chrome engineers about things like rendering to canvas or WebGL, using containment should get similar perf benefits without having to implement your own layout system, and ti should be really easy to add.

If you want to do the footwork of setting up some benchmarks and proving that it makes a significant difference, that would be interesting. I think paint containment might cause some problems for people — absolutely positioned bookmark widgets sticking out of their containing lines can be useful, but if there's a big performance win, it might be worth making that harder anyway.

@marijnh I probably don't have the time for this anytime soon, I was just investigating different code editing options and noticed how CodeMirror could likely benefit from containment.
When I get back around to my project that requires editing, I'll loop back on this and see if I can contribute.

I think paint containment might cause some problems for people — absolutely positioned bookmark widgets sticking out of their containing lines can be useful, but if there's a big performance win, it might be worth making that harder anyway.

It would be easy to temporarily opt a lit out of paint containment by applying a class. I'm not sure how the extension API works, but if an extension can apply classes to lines and spans, this won't be difficult at all. A slightly more pure alternative would be to put widgets on a different layer, not container by lines at all.


issue/4271-endpoint-partial-compiles (#4348)

Pull request opened by the merge tool on behalf of #4348

Merged into branches iso5 in 676d030d1d31a383bcff79654c862283cf40ebc2


Add exclude-from option

I've added the exclude-from option, allowing to specify excludes in a text file, i'd like to see it in the main repository so I can use it from composer

Awesome! Thank you :)

You're welcome