Add AirbyteServer block [breaking-changes]

Adds the AirbyteServer block to handle nginx authentication for OSS instance in AirbyteClient and store config info about the instance running airbyte.
This PR alters the tasks in this collection to accept an AirbyteServer instead of the individual fields, so these are breaking changes.
This PR also adds a deprecation warning to export_configuration, as the underlying API endpoint was removed from OSS airbyte as of airbyte v0.40.7

Closes #39
Closes #21
Example

Checklist

[ ] This pull request includes tests or only affects documentation.
[ ] Summarized PR's changes in CHANGELOG.md

Before I forget, I think we should add a changelog with the usual items and also highlight breaking changes in the first line.


Import PureData Files using only WebPd (pdfu) parser

(and so WebPd will be required, but that's fine)
follower of #220 and #231

sebpiq/WebPd#86 is, probably, required for this issue

Implemented sebpiq/WebPd#86 in shamansir/WebPd@3ed8936

It's done, basically, few minor issues occurred, I'll report them separately.


FAQ page content

determine if this is necessary and then implement it if it is

We think that Pico Accordions are a good fit for this page.

@Archie-G will write the content for this and then someone can convert it you svelte/pico code

KendoClubWebsiteFAQ_AG20240616.docx
KendoClubWebsiteFAQ_AG20240616.pdf
I did my best to shorten everything. See what you all think. Also please correct any English mistakes/typos I made lol.


Issue with Javascript Interop in Server Side Blazor Project

Currently I am trying to access the play() method on an HTML audio tag.
<audio id="timeOverPlayer" preload="auto"><source src="@settings["Time Over Sound"]" /></audio>
I've got a Script tag in my index.html file:
    <script>         window.playElement = function (element) {             document.getElementById(element).play();             return true;         }     </script>
And I'm trying to call this function from C# with the following static method which is an event handler (if that's important):
    public void OnFiveMinuteWarning(object sender, EventArgs args)
    {
        PlayAudio("fiveMinutePlayer");
    }

    public static Task<bool> PlayAudio(String elementName)
    {
        return JSRuntime.Current.InvokeAsync<bool>("playElement", elementName);
    }

When this happens I get no audio and no errors in the console. I can manually call the Javascript from the console, and the audio plays fine. If I add a Wait() on the PlayAudio method call, I get an error:

System.AggregateException
HResult=0x80131500
Message=One or more errors occurred.
Source=System.Private.CoreLib
StackTrace:
at System.Threading.Tasks.Task.Wait(Int32 millisecondsTimeout, CancellationToken cancellationToken)
at System.Threading.Tasks.Task.Wait()
at WebClock.App.Pages.Clocks.OnTimeOver(Object sender, EventArgs args) in C:\Users\samue\source\repos\BlazorClock\WebClock\WebClock.App\Pages\Clocks.cshtml:line 52
at WebClock.App.Services.Clock._timer_Elapsed(Object sender, ElapsedEventArgs e) in C:\Users\samue\source\repos\BlazorClock\WebClock\WebClock.App\Services\ClockService.cs:line 111
at System.Timers.Timer.MyTimerCallback(Object state)

Inner Exception 1:
JSException: Could not find 'playElement' in 'window'.
Error: Could not find 'playElement' in 'window'.
at http://localhost:37382/_framework/blazor.server.js:23:47556
at Array.forEach ()
at p (http://localhost:37382/_framework/blazor.server.js:23:47517)
at http://localhost:37382/_framework/blazor.server.js:23:48120
at new Promise ()
at e.beginInvokeJSFromDotNet (http://localhost:37382/_framework/blazor.server.js:23:48094)
at http://localhost:37382/_framework/blazor.server.js:16:11652
at Array.forEach ()
at e.invokeClientMethod (http://localhost:37382/_framework/blazor.server.js:16:11623)
at e.processIncomingData (http://localhost:37382/_framework/blazor.server.js:16:10229)

Not sure if this is just not being raised when async or if the error is because I'm Waiting on an async call. At this point I'm pretty puzzled.

As Steve said once wait not works in browser.
https://github.com/aspnet/Blazor/issues/1120#issuecomment-404663963

@highphilosopher the reason why the first case doesn't do anything was because you were returning a Task from the method but didn't execute it yet.
The reason why the second one runs is because the await keyword starts the task and waits for it to finish. I haven't tried it yet but if you don't particularly care if the async method finishes or not is that you could do PlayAudio("params").Start() on the first case though I do think that's not a good idea overall.

Thanks for answering, @RyoukoKonpaku!

@RyoukoKonpaku Actually when I try .Start() I get an error...
System.InvalidOperationException: 'Start may not be called on a promise-style task.'
@SteveSandersonMS
If I try to await the method call I still get the same error as I do with the Wait() method.
Note that it works fine when called from a button click in the HTML. It's the event firing call stack that fails to find the Javascript function. It seems like it's when the Server side comes back on a call stack that's not started from an async call.
I can even put the method back in the service... Call it from the button and it works. Call it from the event call stack and it fails.


DHCPv6: Static IP assignment should also offer to write DUID

I tried to assign a static IPv6 suffix to my hosts and failed.
Based on the hint given in https://apuntesderootblog.wordpress.com/2014/09/07/static-dhcpv6-leases-in-openwrt/ it started working after I manually added the DUID for the respective hosts (read from /var/hosts/odhcpd).
Based on https://en.wikipedia.org/wiki/DHCPv6 I would say this is not a shortcoming of odhcp but rather how it is supposed to work.
luci should therefore offer another field to select one of the DUIDs (possibly giving the currently assigned suffix as a hint for the user).

That seems to be fixed.


Add pod security enforce label to gardener created namespaces

How to categorize this PR?

/area security
/kind enhancement
What this PR does / why we need it:
This PR adds the pod-security.kubernetes.io/enforce label set to privileged for the garden and shoot control-plane namespaces. To comply with DISA Kubernetes STIGs we want to be able to configure our pod security policy form PodSecurityConfiguration to baseline https://kubernetes.io/docs/concepts/security/pod-security-standards/. Currently this is not possible since the garden and shoot control-plane namespaces contain pods that do not comply with baseline. This PR fixes this issue by setting namespace specific enforce level to privileged for those namespaces.
This PR is in sync with Kubernetes' STIG. See rule 254800 for more details.
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Release note:

Added pod security enforce level `privileged` label to `garden` and `shoot` control-plane namespaces.

Added pod security enforce level `baseline` label to `istio` and extension namespaces.

/cc @shafeeqes

What about https://github.com/gardener/gardener/blob/fe4b523dfc25d6d63b36f6b2541e212a82c31bc7/pkg/operator/controller/garden/reconciler_reconcile.go#L81-L90?

We can let the label be managed by the gardener-operator. I did not find a clear reason to do so since we would always want the label in the garden namespace, thus I did not add it to the operator. The change would require to move the line that sets the label to this if statement: https://github.com/gardener/gardener/blob/049227ca0f5a3e93970949a9f5eb81ab3987e9b7/pkg/gardenlet/controller/seed/seed/reconciler_reconcile.go#L312-L316
I can add it to the gardener-operator if you wanted, WDYT?

The value is: What if the garden cluster is not a seed cluster at the same time? Then there is no gardenlet which could set the label, hence gardener-operator has to do it itself.

Thank you. I did not take this into account. I will be adding it to the gardener-operator in my next commit.

What about the extension namespaces in the seed?
https://github.com/gardener/gardener/blob/5912d75b3e83abba453d1ea6647ba3f4d37c2682/pkg/gardenlet/controller/controllerinstallation/controllerinstallation/reconciler.go#L160-L169

What about the extension namespaces in the seed?

I checked the extension namespaces and found that their pods comply with the baseline policy. There is no need to set their policy to privileged and would be best if they comply with baseline for future releases as well.

What about the extension namespaces in the seed?

I checked the extension namespaces and found that their pods comply with the baseline policy. There is no need to set their policy to privileged and would be best if they comply with baseline for future releases as well.

Then we need to enforce baseline, right? By default, I think no policy is applied ie; it's privileged to run all pods.

Then we need to enforce baseline, right? By default, I think no policy is applied ie; it's privileged to run all pods.

Yes. I did not add it in my initial commit because the DISA STIGs rule 254800 only required us to configure the PodSecurity plugin of AdmissionConfiguration with a cluster level pod-security policy. It do agree that it would be better to set the label on all gardener created namespaces with appropriate policy.

There are also other namespaces related to istio in garden/seed clusters.

I did check the istio namespaces and they do comply with baseline as well. I will now add this policy to istio and the extension namespaces.

If only the garden namespace needs to run in privileged mode, then we should set baseline as default in the admission controller, https://kubernetes.io/docs/tasks/configure-pod-container/enforce-standards-admission-controller/#configure-the-admission-controller

If only the garden namespace needs to run in privileged mode, then we should enforce baseline as default in the admission controller, https://kubernetes.io/docs/tasks/configure-pod-container/enforce-standards-admission-controller/#configure-the-admission-controller

Only the garden and shoot control-plane namespaces need baseline.

Sorry, I did not get this. Did you set baseline as default already?

No, we will not set it to default to baseline, we only need to be able to set it to default to baseline. Currently I can set it to default to baseline the admission controller configuration in a shoot cluster. However I would have problems with the garden and shoot control-plane components which do not comply with baseline

No, we will not set it to default to baseline, we only need to be able to set it to default to baseline.

Okay, after this PR, you can do it, makes sense.

Currently I can set it to default to baseline in the admission controller configuration in a shoot cluster.

What do you mean? You mean setting the configuration in the Shoot spec?

What do you mean? You mean setting the configuration in the Shoot spec?

Yes. Sorry for not being descriptive enough.

What do you mean? You mean setting the configuration in the Shoot spec?

Yes. Sorry for not being descriptive enough.

Why would we want to do that? Isn't that the choice of the Shoot owner?

Why would we want to do that? Isn't that the choice of the Shoot owner?

Yes, I want the Shoot owners to be able to set it to baseline and for us to be able to set it for our clusters if we want to be compliant.

Setting it in the Shoot spec (for the Shoot kube-apiserver) only affects the Shoot cluster pods and not the seed cluster.

Setting it in the Shoot spec (for the Shoot kube-apiserver) only affects the Shoot cluster pods and not the seed cluster.

Yes, but we would be setting it in the Shoot spec of a seed cluster as well

Ah, You meant the Shoot spec of a ManagedSeed.

Yes, the pods in extension-provider-local do not comply with baseline, since they use hostPath volumes.

I had before tested with cert-seeds, extension-shoot-cert-service-w72tx, extension-shoot-dns-service-zcjw9, extension-shoot-lakom-service, extension-shoot-networking-filter
extension-shoot-networking-problemdetector, kupid-2ng6t, networking-calico-2ncjs, os-gardenlinux-577x5,
provider-aws, provider-gcp, reserve-excess-capacity, roles-seeds and found they all comply with baseline. I can remove my last commit and not set baseline to all extension namespaces or I can set privileged only for extension-provider-local?

Since we cannot make any assumptions about extension in gardener/gardener, I guess we have to set it to privileged in all cases.

There should be a way to set privileged only to the required extension namespace, i.e. provider-local. I do not think that we should set privileged for every extension namespace, esp if the extension does not need the privileged permissions. Ideally every extension should be able to set these permissions for itself. I am not sure if there is an easy way to do this 🤔

We can add an annotation pod-security-standart-enforce to ControllerRegistration resources which we would place into ControllerInstallation, check the annotation if it is set and add the appropriate label to the extension namespace if set, during reconciliation. We would place this annotation for provider-local to privileged and for the other extensions we would be able to select pod-security-standart-enforce that fits their components. WDYT @shafeeqes  @rfranzke ?

If there is no annotation, then we fall back to privileged.

I think that if there is no annotation then we should not set any explicit labels and leave the defaulting set via the kube-apiserver configuration take effect.

If there is no annotation, then we fall back to privileged.

I think that if there is no annotation then we should not set any explicit labels and leave the defaulting set via the kube-apiserver configuration take effect.

I thought the point was to have some label (even if it's privileged and it's the same as no label).

If there is no annotation, then we fall back to privileged.

I think that if there is no annotation then we should not set any explicit labels and leave the defaulting set via the kube-apiserver configuration take effect.

I thought the point was to have some label (even if it's privileged and it's the same as no label). We cannot also set the kube-apiserver configuration in all cases. For a managedseed, it's fine. But for eg, a GKE cluster, we cannot set this admission controller configuration in the kube-apiserver.

I do not think that explicit label is required at all times. I get your point that for non-managed seeds one cannot use the defaulting via the kube-apiserver configuration. In such case the labels should be put explicitly. But for the managed seed case empty != unprivileged since a configuration on the kube-apiserver can already be in place. I propose the following:

if annotation is present on the controllerinstallation then we apply it to the namespace as well
if annotation is missing then we leave the defaulting (if that is applicable)

Also this cannot be set for a GKE cluster today, but maybe this will not be the case tomorrow (or at least I hope that such configurations might be possible in the future)
WDYT, @shafeeqes ?

If there is no annotation, then we fall back to privileged.

I think that if there is no annotation then we should not set any explicit labels and leave the defaulting set via the kube-apiserver configuration take effect.

I thought the point was to have some label (even if it's privileged and it's the same as no label). We cannot also set the kube-apiserver configuration in all cases. For a managedseed, it's fine. But for eg, a GKE cluster, we cannot set this admission controller configuration in the kube-apiserver.

I do not think that explicit label is required at all times. I get your point that for non-managed seeds one cannot use the defaulting via the kube-apiserver configuration. In such case the labels should be put explicitly. But for the managed seed case empty != unprivileged since a configuration on the kube-apiserver can already be in place. I propose the following:

if annotation is present on the controllerinstallation then we apply it to the namespace as well
if annotation is missing then we leave the defaulting (if that is applicable)

Also this cannot be set for a GKE cluster today, but maybe this will not be the case tomorrow (or at least I hope that such configurations might be possible in the future)
WDYT, @shafeeqes ?

Sure.

Can you also document this for extensions, please? Perhaps add a new section to https://github.com/gardener/gardener/blob/master/docs/extensions/conventions.md and explain the annotation on the ControllerRegistration.

/retest


Logo on mobile view is no longer navigating the user back to the homepage

Summary of the bug
The logo on mobile view is no longer clickable. The logo used to be clickable and navigate users to the home page. Now that no longer works and the user struggles to navigate to home page.
🪜 How to reproduce
Tell us the steps to reproduce the problem:
Ensure your are viewing the site on mobile view.

Go to a page that is not the home page e.g. Accessibility
Click on the logo
See the error

🖥 📱 Device

Type: mobile

🧐 Expected behaviour
The logo should redirect the user to the home page.

@GCHQ-Developer-465 this seems to be working ok now - can you still recreate it?

Ticket now includes checking IcSideNavigation component as well.

@GCHQ-Developer-465 this seems to be working ok now - can you still recreate it?

Looks like this is working now. I will be testing the IcSideNavigation as @jd239 has mentioned below.

After more exploration with this I have come to the conclusion that this is no longer an issue. I tested the Side Navigation on a live application and it works as it should. Therefore have closed this issue.


Bug in "Complex Values" example?

This is either a bug, or I don't understand what's going on.
in ObjectSelect.js, the  element has this:
value={multiple ? value.map(JSON.stringify) : JSON.stringify(value)}
the first part of that statement, value.map(JSON.stringify), seems wrong to me, no? I can't quite figure out what that's supposed to be.
And is it the case that the second half of the statement is redundant, is JSON.stringify necessary? Isn't the value going to come through as an object literal?

Using an array for the values is a React thing.  See the bottom of https://facebook.github.io/react/docs/forms.html.  It might work to use an object as a value rather than a string, but the examples on that forms page all use strings, and also JSON.stringify will often take structurally equal objects to the same string.  (I'm not sure how bulletproof it is, but the example does claim to be "quick and dirty".)  So I think there is no bug.

Well, it was working, so in that sense, it wasn't a bug, but I tried without doing any of the json conversion, and it works just fine. I had been under the impression that the values should be strings. Thanks guys. :+1:


How to update current media metadata (user rating) without interrupting the playback?

I am using androidx.media3 to develop an audio player app which provides users with an option to rate currently playng media. So, inside my MediaSessionCallback I do this:
override fun onSetRating(
    session: MediaSession,
    controller: ControllerInfo,
    rating: Rating
): ListenableFuture<SessionResult> {

    val item = session.player.currentMediaItem

    item?.let {
        val metadata  = it.mediaMetadata.buildUpon().setUserRating(rating).build()
        val mediaItem = it.buildUpon().setMediaMetadata(metadata).build()
        session.player.setMediaItem(mediaItem, false)
    }

    return Futures.immediateFuture(SessionResult(SessionResult.RESULT_SUCCESS))
}

This works, but with the only caveat - it interrupts the audio for the moment. Please, help me to understand how user rating should work.

It's currently not possible to update the media item I'm afraid. I think you need to maintain these ratings in your app instead like in a map for each mediaId as a key or similar.
You need to do this anyway I think as you loose the updated media items once they are removed from the player. So at some point the rating need to be stored. It's probably best to provide the rating in the viewModel of the UI read it and update in from there instead of the field in the media item.

Thanks for your answer.
I guess I need to give more details of what I'm trying to achive and why I need to update current metadata. When the media is playing the service posts a mediastyle notification that has a custom action with a heart icon. This icon reflects current state of media rating - outlined if unrated and filled if rated. When user taps on that action he changes the rating and the notification needs to be updated.
I am using MediaSessionService class that has onNotificationUpdated method. I've overrided that method to build my custom notification, but this function is triggered only when currently playing media changes, specifically 'playWhenReady' field or 'mediaMetadata' field:
if (Util.areEqual(oldPlayerInfo.mediaMetadata, newPlayerInfo.mediaMetadata)
        && oldPlayerInfo.playWhenReady == newPlayerInfo.playWhenReady) {
      return;
    }
    updateNotificationIfNeeded(session);

So theoretically I can write my own code for that but it will not be the cleanest solution, because in that case I need to update the notification manually, for example like this:

Thanks for the detailed clarification. I understand your use case.
I marked this issue as an enhancement.
As a note for future implementation:
There are two things we may want to look into:

Apps need a way to invalidate the notification and trigger that a new notification is built and posted. This makes sure that apps can add custom actions (or information) that may change independently from the state of the player that actually triggers an auitomatic update of the notification. That's pretty much what we had with PlayerNotificationmanager.invalidate() in the old world.

In general we want to think of a way that app can update the metadata of the media item of the media source, that eventually is passed to onMediaMetadataChanged(). At least for the use case describe in this issue this would be the most natural way to bring in updated metadata and just let the service/notificationManager do it's usual task with the new metadata.

Point 2 is a bit more complicated because the media item has it's roots in the media source and is currently not supposed to change. Point 1 seems a valuable addition to the changes we are currently working to have better notification support.
I can't give you any time commitments for any of these points but I will make sure that 1) is part of the first stable release of Media3.

Hi,
for streaming radio apps it is also a problem, that is not possible to update the MediaSession, respectively the Notification for a MediaItem. Radio streams often publish "Currently Playing" metadata, which can be received in onMetadata() in the Player.Listener. If you want to surface the radio station metadata in the Notification, you need a way to update the Notification / MediaSession.
I think podcast apps that support chapters might run into the same issue.

@y20k  The metadata received via onMetadata should already update the notification because it's part of player.getMediaMetadata() (which is used for the notification).  If you think this isn't working in specific cases, could you file a new issue for this? This issue here is more updating the app-provided metadata in the MediaItem.

Hi @y20k
I was not aware Icecast metadata is already being put into MediaMetadata. That is great! ❤ No need to intercept the notification updates when new Icecast metadata arrives.
I think I still found an issue. An important part of Icecast metadata seems not to be used when creating MediaMetadata.
I am not sure, if is a bug, or intended. I created the issue #153 to document my findings.

Not sure I follow on the last few responses here. Is that to say that if a Live audio stream internally holds metadata through this IcyInfo standard it will get automatically picked up?
What about the use case where the stream itself doesn't have such metadata, but information is provided through other means and must be updated in the UI/Notification. For instance an API that returns the currently airing radio program along with a timestamp for when the next program is scheduled to start. Is there no way currently to schedule an update to the MediaMetadata if this is the mechanism through which live radio information is delivered? If not that seems like a pretty major shortcoming given that Media3 is geared around automatically setting up behaviours driven by MediaMetadata (e.g. the notification behaviour).
In the app I'm working on I'm also hoping to treat the MediaMetadata as the single source of truth for what to display on the app's own player UI as well.

Looks like the functionality described above (sourcing metadata about a live radio stream) could be achieved through a MediaSource and providing various windows? (Correct me if that is completely the wrong assumption or ballpark).
Though the use of media sources seems to be something that's limited to the ExoPlayer class itself and isn't exposed on the Player interface or MediaController, and playback of MediaItems needs to go through the MediaSession.Callback onAddMediaItems callback to populate the media URI given request metadata. There doesn't seem like an obvious entry point for making use of MediaSources

This will be possible via replaceMediaItem from 1.2.0.


报错：TypeError: Expected 'Iterator' as the return annotation for __iter__ of Dataset, but found thumt.data.iterator.Iterator

torch版本不对


compile error on macOS

On macOS with Apple clang version 14.0.0 I get the error
Consolidate compiler generated dependencies of target umfpack_int
[ 31%] Building C object CMakeFiles/umfpack_int.dir/UMFPACK/Source/umf_cholmod.c.o
/Users/fwein/Downloads/SuiteSparse-cmake/UMFPACK/Source/umf_cholmod.c:175:9: error: implicit declaration of function 'cholmod_analyze' is invalid in C99 [-Werror,-Wimplicit-function-declaration]
    L = CHOLMOD_analyze (S, &cm) ;
        ^
/Users/fwein/Downloads/SuiteSparse-cmake/UMFPACK/Source/umf_cholmod.c:24:29: note: expanded from macro 'CHOLMOD_analyze'
#define CHOLMOD_analyze     cholmod_analyze
                            ^
/Users/fwein/Downloads/SuiteSparse-cmake/UMFPACK/Source/umf_cholmod.c:175:9: note: did you mean 'cholmod_scale'?
/Users/fwein/Downloads/SuiteSparse-cmake/UMFPACK/Source/umf_cholmod.c:24:29: note: expanded from macro 'CHOLMOD_analyze'
#define CHOLMOD_analyze     cholmod_analyze
                            ^
/Users/fwein/Downloads/SuiteSparse-cmake/CHOLMOD/Include/cholmod_matrixops.h:122:5: note: 'cholmod_scale' declared here
int cholmod_scale
    ^
/Users/fwein/Downloads/SuiteSparse-cmake/UMFPACK/Source/umf_cholmod.c:175:7: warning: incompatible integer to pointer conversion assigning to 'cholmod_factor *' (aka 'struct cholmod_factor_struct *') from 'int' [-Wint-conversion]
    L = CHOLMOD_analyze (S, &cm) ;

The -Werror puzzles me, as it seems it could be avoided, however it is unclear how to avoid it. The compile call is
/Library/Developer/CommandLineTools/usr/bin/cc -DBLAS_UNDERSCORE -DDINT -Dumfpack_EXPORTS -I/Users/fwein/Downloads/SuiteSparse-cmake/AMD/Include -I/Users/fwein/Downloads/SuiteSparse-cmake/UMFPACK/Include -I/Users/fwein/Downloads/SuiteSparse-cmake/UMFPACK/Source -I/Users/fwein/Downloads/SuiteSparse-cmake/build/include/SuiteSparse/UMFPACK -I/Users/fwein/Downloads/SuiteSparse-cmake/build/CHOLMOD/Include -I/Users/fwein/Downloads/SuiteSparse-cmake/CHOLMOD/Cholesky -I/Users/fwein/Downloads/SuiteSparse-cmake/CHOLMOD/Core -I/Users/fwein/Downloads/SuiteSparse-cmake/CHOLMOD/Include -I/Users/fwein/Downloads/SuiteSparse-cmake/CHOLMOD/MatrixOps -I/Users/fwein/Downloads/SuiteSparse-cmake/CHOLMOD/Modify -I/Users/fwein/Downloads/SuiteSparse-cmake/CHOLMOD/Supernodal -I/Users/fwein/Downloads/SuiteSparse-cmake/build/include/SuiteSparse/CHOLMOD -I/Users/fwein/Downloads/SuiteSparse-cmake/SuiteSparse_config -I/Users/fwein/Downloads/SuiteSparse-cmake/build/include/SuiteSparse -I/Users/fwein/Downloads/SuiteSparse-cmake/build/include/SuiteSparse/AMD -O3 -DNDEBUG -arch arm64 -isysroot /Library/Developer/CommandLineTools/SDKs/MacOSX12.3.sdk -mmacosx-version-min=12.3 -fPIC -fvisibility=hidden -MD -MT CMakeFiles/umfpack_int.dir/UMFPACK/Source/umf_cholmod.c.o -MF CMakeFiles/umfpack_int.dir/UMFPACK/Source/umf_cholmod.c.o.d -o CMakeFiles/umfpack_int.dir/UMFPACK/Source/umf_cholmod.c.o -c /Users/fwein/Downloads/SuiteSparse-cmake/UMFPACK/Source/umf_cholmod.c

I don't know if this can be avoided by the project or if is an issue of the original sparse suite. The current sparse suite 6.0 beta compiles easily.

Could you provide a full reproducer? While I don't use macOS, the project's macos-12 Github workflow currently running AppleClang 13.1.6 is perfectly happy. Thus, at this point I must assume there's something funky with your setup.

You are right, it is an option.
When I enable WITH_CHECK, it fails. What does this option mean anyway? The cmake option help is not really helpful ("Enable the Check module").
For me this hint helped and I'm fine - thanks :)

BTW, all on but only WITH_CHOLESKY
does also not work
[ 19%] Building CXX object CMakeFiles/spqr.dir/SPQR/Source/spqr_1colamd.cpp.o
/Users/fwein/Downloads/SuiteSparse-cmake/SPQR/Source/spqr_1colamd.cpp:564:13: error: use of undeclared identifier 'cholmod_l_amd'
            cholmod_l_amd (AT, NULL, 0, (Long *) (Q1fill + n1cols), cc) ;
            ^
/Users/fwein/Downloads/SuiteSparse-cmake/SPQR/Source/spqr_1colamd.cpp:581:18: error: use of undeclared identifier 'cholmod_l_analyze_p2'
            Sc = cholmod_l_analyze_p2 (FALSE, AT, NULL, NULL, 0, cc) ;
                 ^
/Users/fwein/Downloads/SuiteSparse-cmake/SPQR/Source/spqr_1colamd.cpp:606:13: error: use of undeclared identifier 'cholmod_l_colamd'; did you mean 'cholmod_l_ccolamd'?
            cholmod_l_colamd (AT, NULL, 0, TRUE,
            ^~~~~~~~~~~~~~~~
            cholmod_l_ccolamd
/Users/fwein/Downloads/SuiteSparse-cmake/CHOLMOD/Include/cholmod_camd.h:61:5: note: 'cholmod_l_ccolamd' declared here
int cholmod_l_ccolamd (cholmod_sparse *, SuiteSparse_long *, size_t,
    ^
/Users/fwein/Downloads/SuiteSparse-cmake/SPQR/Source/spqr_1colamd.cpp:606:44: error: cannot initialize a parameter of type 'long *' with an rvalue of type 'int'
            cholmod_l_colamd (AT, NULL, 0, TRUE,
                                           ^~~~
/Users/fwein/Downloads/SuiteSparse-cmake/SPQR/Include/spqr.hpp:67:14: note: expanded from macro 'TRUE'
#define TRUE 1
             ^
/Users/fwein/Downloads/SuiteSparse-cmake/CHOLMOD/Include/cholmod_camd.h:62:23: note: passing argument to parameter here
    SuiteSparse_long *, SuiteSparse_long *, cholmod_common *) ;
                      ^
4 errors generated.

But again, this is on issue, just an observation.

The purpose of the Check module is explained in the official SuiteSparse guide:
https://github.com/sergiud/SuiteSparse/blob/5fa24ce1d67472e0e5cffd79d79b16000cd0f3c4/CHOLMOD/Doc/CHOLMOD_UserGuide.tex#L992-L998
I can reproduce the WITH_CHOLESKY=OFF build failure on my Linux machine. Let me look into the issue.

When I enable WITH_CHECK, it fails.

I'm not sure I can follow because the option is enabled by default. There is something else you have enabled. Please provide your exact CMake command-line.

 This is the only option I trigger.

I see, thanks. I can now reproduce the initial problem as well.

Both issues should be fixed in 5c1d50c26a05bccb58547bf550c4e13aef9fa1a9. Let me know if there any problems left.


Bit more information in First steps docs

In the README.md under First steps to configure your wagtail app, the documentation could be a bit more specific about where to put the example model. This is mentioned later, so maybe a change in ordering.
Also some more background about how the import_wordpress_data method works would be helpful.

Also some more background about how the import_wordpress_data method works would be helpful.

Question: Hi Matt, is this what you where looking for: https://github.com/torchbox/wagtail-wordpress-import/blob/main/docs/import_process.md


Is there a code for Finetune on CMU-MOSI here?

please tell me where the code for multimodal sentiment analysis is,thank you!

@sen0902, sorry that we have no plan to release the other codes and features.


TTRPC-based EventClient and EventServer implementation for Runtime v2

The firecracker-containerd project is creating a containerd shim using the v2 runtime model. We'd like to be able to use the EventServer and EventClient implementations but those currently require GRPC.
Our use case involves forwarding events that occur within a microVM over a vsock to a shim outside the VM, so the overhead of GRPC is unneeded. A TTRPC implementation of EventServer and EventClient would save that overhead and save us from maintaining both TTRPC and GRPC servers in our v2 shim (TTRPC would continue to be needed for the v2 TaskService).

This is a good solution to prevent a re-exec for accessing the HTTP grpc server.
+1 to this idea

Yea I looked at this the other day. It turns out the re-exec pipeline already does this. Why dont we just use the EventsClient from a shim ourselves?

The re-exec pipeline does this via the HTTP grpc stuff.  We want to keep that stack out of the shim because of memory usage and size.

Does anyone want to take on the development for this?

I will do it

Yes, closing.


Drawer not working after loggin back in the app

I am using redux in my react-native app with stack,tabs and drawer navigator.
my problem is that when I am going to logout in my and then I will login again.when I am going to click my humberger menu to open the drawer it will not open. I will drag first the drawer in order to function it back.
Here is my routes.js
const Credit = StackNavigator({
    Credit: {
      screen: Screen1,
      navigationOptions: ({ navigation }) => ({
        title: 'Credit',
        headerLeft: <Icon name="menu" size={30} color="white" onPress={ () => navigation.navigate('DrawerOpen') } />,
        headerTintColor: 'white',
        headerStyle:{
          backgroundColor: '#6200EE',
          elevation: 0,
         // showdowOpacity: 0
        },
        tabBarIcon: ({ tintColor }) => {
          return (
            <Icon
            name='ios-cash'
            type='ionicon'
            color={tintColor}
            size={26}

            />
          );
        }
      })
    }
  },{
    headerMode: 'none'
  });

Environment

Environment:
  OS: Windows 10
  Node: 8.9.1
  Yarn: 1.5.1
  npm: 5.6.0
  Watchman: Not Found
  Xcode: N/A
  Android Studio: Not Found

Packages: (wanted => installed)
  react: 16.3.1 => 16.3.1
  react-native: 0.55.3 => 0.55.3

Thank you in advance.

Make sure you close it when you logout.

@kelset ,
I added in my logout method
NavigationAction.navigate('DrawerClose');

still no luck  ?

sorry I accidentally closed, but this is still not resolve.

still I am struggling on this problem... can someone help me please.

This is related to https://github.com/react-navigation/react-navigation, not to React Native.

Post your code so we can help you


Installing queue backed by ZK

#We need to figure out the details here but here are some of the operations we need to support:
This is the API used during install call:
trait PackageQueueAdder {
  def add(
    pkg: PackageCoordinate,
    content: PackageQueueContents
  ): Future[PackageAddResult]
}

This is the API used by anyone interested in the install queue state. E.g. installer, statue API call:
trait PackageQueueReader {
  def readAll(): Future[Option[List[(PackageCoordinate, PackageQueueContents)]]]
  def read(package: PackageCoordinate): Future[Option[PackageQueueContents]]
}

This is the API used by the installer:
trait PackageQueueRemover {
  def remove(package: PackageCoordinate): Future[PackageRemoveResult]
}

final class ZooKeeperPackageQueue implements PackageQueueAdder with PackageQueueReader with PackageQueueRemover {
  override def add(...): ...
  override def read(...): ...
  ...
}

Migrated.


ModalView: on_dismiss runs before on_open if user dismisses popup before opening animation completes

The reason is that on_open only runs when opening animation completes, which can take some time if _anim_duration parameter is high.
Such a behavior seems counterintuitive, it's expected that on_open should always precede on_dismiss.
Also it worth noting that dismissing doesn't stop opening animation, so both animations run together.
To avoid these issues it's suggested not to dismiss the popup if opening animation is in progress and vice versa not to open if dismissing animation still runs.
Maintainer merge checklist

[ ] Title is descriptive/clear for inclusion in release notes.
[ ] Applied a Component: xxx label.
[ ] Applied the api-deprecation or api-break label.
[ ] Applied the release-highlight label to be highlighted in release notes.
[ ] Added to the milestone version it was merged into.
[ ] Unittests are included in PR.
[ ] Properly documented, including versionadded, versionchanged as needed.

Related issue https://github.com/kivy/kivy/issues/5540.

How about cancel animation and cancel "animation_done" callback ?


Doesn't "build" on Windows

"$ meteor add ostrio:templatehelpers
=> Errors while adding packages:
While building package ostrio:templatehelpers:
error: Couldn't build this package on Windows due to the following file with a colon -- ostrio:templatehelpers.js. Please rename and and re-publish the
package.                                                                                "
I'll add a pull request for this shortly

Hi @wildeyes thank you for submitting this issue. Fixed and published as v1.1.2, atmosphere
If you found this package useful, do not hesitate to star it at both GitHub and Atmosphere. Also you may like to [Tweet](https://twitter.com/share?url=https://github.com/VeliovGroup/Meteor-Template-helpers&text=Meteorjs, do logic in your templates with ease!) about it or share at Facebook


Cherry-pick Redshift dialect to our branch

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

68 of 72 (94.44%)  changed or added relevant lines in 4 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.06%) to 85.19%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/dialect/mod.rs
1
2
50.0%

src/dialect/redshift.rs
11
12
91.67%

src/tokenizer.rs
4
5
80.0%

tests/sqlparser_redshift.rs
52
53
98.11%

Totals

Change from base Build<PHONE_NUMBER>:
0.06%

Covered Lines:
8766

Relevant Lines:
10290

💛  - Coveralls


Fix #27 Path limit issues

As I encountered issue #27 , I tried to fix it.
I made a glorified search and replace in a script called ./autofix_paths.rb that we can remove after the PR is reviewed.
The resulting DSSAT seems to be working for me at  the moment, with the path issue fixed. I ran a batch file and got my results.
Notes:

The changes I made are from tag v<IP_ADDRESS> and not from v<IP_ADDRESS> (this has been rebased with develop now). The reason is the latest tag does not compile for me on Linux https://github.com/DSSAT/dssat-csm-os/issues/416 . If that issue gets fixed, I can likely re-run my  script.
Why are you * text eol=crlf automatically in the gitattributes? This is something that should be handled by the editors instead. And I'm sure even on windows, editors should be able to handle \n for newlines? This setup caused me issues on my side.
This is a bastard search and replace, I did my best as to be fairly specific, but this is likely that some variables are using more stack space than necessary.
I can't find any tests in this project at the moment? So I 'm not sure how to make sure I didn't break anything.

After fixing the compilation of the develop branch on linux in https://github.com/DSSAT/dssat-csm-os/pull/418 , I was able to rebase this branch with develop.
This still seems to be working. I got my results.

@fabiooliveira72 Can you put this in your queue for testing?

Hi @chporter!
Yes, for sure. I saw the PR. I will pass through this soon and test it.
I have another trip coming next week and Im finishing up the workshop things. Ill
Thank you.
Fabio Oliveira

Since new stuff was merged to the develop  branch, I rebased.

@fabiooliveira72 Is there any ETA for this testing?
Just wondering because if more things get merged then I have to fix conflicts.
Note that my dssat nix package got merged in Nixpkgs so anyone can now run dssat with no compilation/dependencies needed with nix run github:nixos/nixpkgs/nixos-unstable#dssat , this includes my patch for the path limit in this MR.

Hi @pcboy!
Thanks for your patience and checking this.
Yes, please keep updated your branch and resolve the conflicts. We need to delay the merge of this pull request because this is a major change that affect all the model. We have an upcoming DSSAT Release that we need to keep stable. I also have some updates for this pull request that I will merge with mine. The problem is the testing of all this.

@fabiooliveira72 Any news?
Clearly it's not really possible to make any significant changes to DSSAT since there are no unit tests.
Is there any plan to have some sort of test suite? Without a test suite, how do you even know DSSAT is working today I wonder?

Unit tests would be nice but completely inadequate for a large integrated model like DSSAT where a change to one module may impact other modules. We employ a whole-model, comprehensive, integrated approach to testing changes using a set of DSSAT input files which use the full range of options and capabilities, testing against measured data as well as previous stable model outputs.

@chporter Sure, that's an integrated test then. I was saying unit tests to refer to any automated test.
So where is this automated test suite that shows that the results are correct and don't break over time?

@pcboy Right now our testing is not automated. It's done by individuals with some offline tools and data. We really need a continuous integration system in github. Are you interested in helping to develop one?

@chporter I'm not really having any personal usage of DSSAT, so I'm not really willing to spend much personal time on it to be honest. I'm also far from understanding the details about this tool since I'm not a researcher, so likely not the best for this.
With that being said, this shouldn't be too complex to start. You simply need to do what you are doing manually, in code. It can be as simple as:

A folder tree like:

tests
└── test1
    ├── expected_outputs
    │   └── output1.txt
    └── inputs
        ├── DSSBatch.v48
        └── weather.WTH
        └── etc.etc

And test suite basically just run each test with the inputs and compare the result of dssat to the expected output.
I'm willing to help doing the "framework" to do that, then you only need to put the input data and expected output in the correct folders.
As soon as you have that, you at least can make sure that any change you do in the code, is not affecting the results, and if they do, you would have to manually change the test's expected output anyway. And that change would be reviewed in a PR.
Of course this is not perfect, but this would be a good first start and better than current situation. If something like this was present today, this PR would already be merged.


Icons sloooowwwww ...

Hy there.. spent a couple of hours setting up your wonderful icons, then did a test run ... I have to load about 2.000 and it takes about 2 minutes !!!

Can you provide more detail? If you are loading 2000 markers onto a Google Map it is going to be slow unless you do any clustering or optimisation.

Thanks for your reply Scott.
This is my problem: I have to represent about 2000 users and 1000 companies on the map.
I am using only two different icons (one for users and one for companies), but in 3 different backgrounds:

     Green for active

     Orange for expired

     Blue for never contacted

Is there anyway I can pre-create thes 6 (2 x 3) icons so that they load much faster??
(actually, I also need an icon for schools and one for teachers but there’s only about 10 of those so I don’t expect problems …)
At present, my code (just for the users) is :
I get a json reply with all the cnt objects and just create them like this …
function cnt (item) {
this.contactId  = item.contactid;

this.name       = item.name;

this.city       = item.city;

this.sede       = item.sede;

this.lat        = item.lat;

this.lng        = item.lng;

this.cntStat    = item.cntStat;

this.visible    = true;

this.marker     = null;

this.gLatLng    = new google.maps.LatLng(item.geoLat, item.geoLng);

this.markerLet  = '';

this.markerCol  = '';

this.explain    = '';

this.icon       = null;

this.iconUrl    = '';

switch (this.cntStat) {

    case 'V':   this.markerCol = 'FF0000';  this.explain = 'Current';   break;

    case 'X':   this.markerCol = 'FFCC00';  this.explain = 'Expired';   break;

    case 'N':   this.markerCol = 'FFFF00';  this.explain = 'Never';     break;

}

this.marker = new Marker({

    position:   new google.maps.LatLng(this.lat, this.lng)

,   map:        map

,   title:      this.name

,   icon: {

        path: MAP_PIN

    ,   fillColor: '#' + this.markerCol

    ,   fillOpacity: 1

    ,   strokeColor: ''

    ,   strokeWeight: 0

    }

,    map_icon_label: ''

});

/***

* STUDENT :  map-icon-university                    MAP_PIN

* TEACHER :  map-icon-courthouse                    SHIELD

* SCHOOL :   map-icon-museum                        SQUARE_PIN

* RED x Shenker

*

* COMPAYT:   map-icon-political                      ROUTE

*

*

*/

google.maps.event.addListener(

    this.marker

    ,   'click'

    ,   function () {

            for (i=0; i<cntArr.length; i++) {

                if (cntArr[i].marker == this) {

                    infoWindow.setContent(cntArr[i].getInfoContent());

                    infoWindow.open(map, this);

                   return;

                }

            }

        }

);

}
From: Scott de Jonge<EMAIL_ADDRESS>Sent: mercoledì 8 giugno 2016 01:36
To: scottdejonge/map-icons<EMAIL_ADDRESS>Cc: fdl333<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [scottdejonge/map-icons] Icons sloooowwwww ... (#54)
Can you provide more detail? If you are loading 2000 markers onto a Google Map it is going to be slow unless you do any clustering or optimisation.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/scottdejonge/map-icons/issues/54#issuecomment-224445763 , or mute the thread https://github.com/notifications/unsubscribe/AS4rUr812mNIbiRLzzUvKG7Ly_jV8ck0ks5qJgBKgaJpZM4IwG-O .  https://github.com/notifications/beacon/AS4rUvO8c1cCPX6FDcnXrNlCLZYoqsgQks5qJgBKgaJpZM4IwG-O.gif

That looks like it has a lot more to do with how you are managing 200 marker objects with Google Maps. The source SVG files are included in the repo and you are more than welcome to use them individually considering you only require 3 icons.
I would advise looking into marker clustering to optimise how many markers are being rendered in Google Maps. That should increase performance.


[Animation] Animation in cards

Details
I wants to add animation as we scroll them down.
Screenshots
No response
Type of Contribution

[ ] Update to an existing Animated Button
[ ] Adding a new Button
[ ] Resolving a bug
[ ] Proposal to the Repository
[ ] Changes related to documentation or README.md
[X] Other Changes

Checklist

[X] I have checked the existing issues
[X] I have read the Contributing Guidelines
[X] I am willing to work on this issue
[X] I am a GSSoC'23 contributor

@Akanksha-202 can you please assign this to me ?

@ojasaklechayt What are you going to use for implementing animations?

@ojasaklechayt What are you going to use for implementing animations?

I will do it with framer motion mostly

@ojasaklechayt  Cool!
Assigned to you!
All the best!


[Feature request] Extension method based widget API

Use case
Currently flutter-based code is heavy to read, especially if you need to write a fancy component with advanced logic, and even splitting it into multiple ones doesn't solve that.
Flutter widgets introduce two or more levels of indentation in code, which can be often seen as an unnecessary burden, especially after seeing if you need more than 3 levels of indentation, you’re screwed anyway, and should fix your program quote from Linux style guide.
Here is a library which tries to solve this by implementing extension method wrappers for widget constructors: https://github.com/ReinBentdal/styled_widget .
Proposal
It would be a nice idea to implement something like that in an autogenerated way where an additional tool scans code for widgets with, e.g. @GenerateExtensionWrapper(name: "extMethodName") metadata annotation.
This could allow generating exact same extension wrappers for people's custom widgets.

A code generate for this could live in an external package. We don't have plans to add something like this to the framework.


docs: remove irregular spaces

for some reason I found we have many 0xA0 spaces instead of 0x20. I guess they are not intentional.

I would love to know how you happened to find this!


Ansible cleanup + Grafana, Prometheus and node_exporter version bumps

Major changes:

Grafana 7.3.6 -> 7.5.4

Prometheus 2.23.0 -> 2.26.0

Node_Exporter 0.16.0 -> 1.1.2

All service restarts have daemon_reload on them - no longer needs it specifically done

Both the "monitor" and "prometheus-node-exporter" roles are now idempotent

We check the binary command --version
There is currently a slight discrepancy here between stdout and stderr (and the versions of Prometheus)
We no longer restart each of the services "because"

Slight adjustments to args on node_exporter to disable known unneeded collectors

Basic Readme added

Automatically provisions the Prometheus datasource into Grafana (this will seamlessly replace existing manually set datasource)

Easier to configure/test as state is more consistent
Sets groundwork so I can do Loki bits

BREAKING CHANGE: Adding the homedir to the prometheus user means the processes need to be stopped

Before running playbook, stop the prometheus and blackbox_exporter on the monitor instance. Pick one:
on instance: systemctl stop prometheus.service blackbox_exporter.service
ansible -i hosts -m command -a 'systemctl stop prometheus.service blackbox_exporter.service' monitor

Example of the idempotency - changes only recorded when changed. --check also now works.
Re-running the playbook on an already provisioned system:
Before:
$ ansible-playbook -i hosts playbook.yml -l monitor

glim-monitor      : ok=33   changed=10   unreachable=0    failed=0    skipped=0    rescued=0    ignored=0

(the changed here were re-downloading files and restarting services for no reason)
After:
$ ansible-playbook -i hosts playbook.yml -l monitor

glim-monitor      : ok=32   changed=0    unreachable=0    failed=0    skipped=10   rescued=0    ignored=0

The point being the 0 changed when re-running.

Confirmed changed=0, works well. Thanks!


IGNORE FOR NOW! WILL UPDATE WHEN READY TO REVIEW!

Description
This PR is made for bug fixes. Fixed a few bugs with the create store and the put request on the store. Zipcode not being accepted in the back end and state was showing city value.
Cors for put route on store was throwing error due to XHR error. Could not find a solution after working on it 3 days. Decided to put it a side a for now.
Type of change
Please delete options that are not relevant.

[x] Bug fix (non-breaking change which fixes an issues
[x] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[x] This change requires a documentation update

Checklist
Remove any items which are not applicable.

[x] I have performed a self-review of my own code
[x] My code follows the style guidelines of this project
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works .
[x] New and existing unit tests pass locally with my changes

Link to Trello card
https://trello.com/c/zAWnzYkV

Looks good! Thanks for fixing the CRUD functionality!


Translations, and added missing fields

Here are my changes, which are php 7.3 compatible:

moved template to php-extension, as it contains php code and should be handled that way by e.g. a webserver
removed "german template"
added a general translating object, so the default template can be used for all translations
better readable css code
fixed/added some missing fields
added seller, if this class is e.g. used to view incomming invoices
added notes
added payment terms which are needed to pay the bill

The default-template can now be used for incomming and outgoing invoices.
If changed the layout some bit, I hope it is ok for you, if not, leave me a note.
If it is planned to use this template for outgoing invoices and the seller should not be printed, I sugges we break this template down into incomming, outgoing, and default which shows everything.

Hi @mokraemer,
sorry. But there is a problem

HI @mokraemer,
there are also to many breaking changes for me:

You changed the signature of setTemplate
You renamed the detault.tmp to default.php (Why)
You changed a contract - I don't think that's a good idea.
You added Attributes

So long, Maybe it's a good idea to develop an idea together before you start coding... ;-)
Will not merge...


Fixed #28170 -- Fixed file_move_safe() crash when moving files to a CIFS mount.

https://code.djangoproject.com/ticket/28170

Everyone... @peterfarrell and I made the suggested changes above as well as added two tests to cover the modifications.  We'll update the Track ticket.

Are you still working on this? Did you see the test failure? You can squash commits when updating.

Yes, PyCon interrupted us.
Sent from my Verizon, Samsung Galaxy smartphone
-------- Original message --------
From: Tim Graham<EMAIL_ADDRESS>Date: 5/19/17 6:43 AM (GMT-08:00)
To: django/django<EMAIL_ADDRESS>Cc: "Farrell, Peter (SAA)"<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [django/django] Fixed #28170 -- Fixed file_move_safe() crash when moving files to a CIFS mount. (#8486)
Are you still working on this? Did you see the test failure? You can squash commits when updating.

You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/django/django/pull/8486#issuecomment-302706118, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ACuY5NxhJofKD33fQLMrPJXZApjVrLJsks5r7ZxegaJpZM4NW1oI.

If you have a chance to finish this by June 1, I'll include it in the 1.11.2 release.

Wil do. I’ll have time tomorrow to complete it.
--
Peter J. Farrell
Principal Software Engineer, Advanced Technology Development
Office of the Sergeant at Arms, United States Senate
From: Tim Graham<EMAIL_ADDRESS>Reply-To: django/django<EMAIL_ADDRESS>Date: Tuesday, May 30, 2017 at 2:28 PM
To: django/django<EMAIL_ADDRESS>Cc: "Farrell, Peter (SAA)"<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [django/django] Fixed #28170 -- Fixed file_move_safe() crash when moving files to a CIFS mount. (#8486)
If you have a chance to finish this by June 1, I'll include it in the 1.11.2 release.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/django/django/pull/8486#issuecomment-304982579, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ACuY5FA4soNH4eUP9sFbNUO5r6GdURL8ks5r_G3UgaJpZM4NW1oI.

@timgraham @djackson-saa Made all changes recommended by the reviewers, commits squashed, and all checks now pass. Ready for check-in.

@timgraham Thanks for the merge.

@timgraham - I echo the thanks from @peterfarrell


Support wasm32-unknown-emscripten Target

Feature Request
I have a dependency that only supports the wasm32-unknown-emscripten target (I think it requires exceptions, but I'm not sure).
Apart from dioxus-logger it seems that the target is compatible with the main parts of the echo system. Namely dioxus, dioxus-web and dioxus-router.
I can compile using cargo cargo build --target wasm32-unknown-emscripten, but it doesn't seem to work with dx serve.
Running the following command:
dx serve --target wasm32-unknown-emscripten

results in the the following in the target directory:
CACHEDIR.TAG  debug  wasm32-unknown-unknown

Implement Suggestion
The target flag doesn't seem to propagate properly to cargo build command.

After a quick search, it seems this hard coded target is the cause
https://github.com/DioxusLabs/dioxus/blob/b97e607aca0bd8a12971fcf403908addf9a45dd7/packages/cli/src/builder/cargo.rs#L48-L51

It seems like wasm-bindgen doesn't support emscripten, so this might be a dead end for now https://github.com/rustwasm/wasm-bindgen/issues/2722.

It seems like wasm-bindgen doesn't support emscripten, so this might be a dead end for now rustwasm/wasm-bindgen#2722.

Yeah looks like wasm-bindgen doesn't yet have a plan to implement this. I'll close this for now until there's more progress on their end :disappointed:


Upgrade template-infra v0.8.0 -> v0.12.2

Ticket
Part of FFS-1422.
Changes

Upgrade infra to v0.9.0
Upgrade template-infra v0.9.0 -> v0.10.0
Upgrade template-infra v0.10.0 -> v0.11.0
Upgrade template-infra v0.11.0 -> v0.11.1
Upgrade template-infra v0.11.1 -> v0.11.2
Upgrade template-infra v0.11.2 -> v0.12.0
Upgrade template-infra v0.12.0 -> v0.12.1
Upgrade template-infra v0.12.1 -> v0.12.2

Context for reviewers
This is a huge upgrade, as it captures roughly six months of work by a very
productive platform team. I've done my best to resolve all the conflicts in a
way that makes sense, but still we will have to be careful when deploying this.
This branch removes two customized fixes we made for infra bugs that were later
upstreamed:

The way we were accessing the build repository across environments
Fix for adding new environment variables on a new deploy

So we will have to be especially careful that the correct build repository is
used.
Testing
Testing/deployment plan:

See what the automated tests say here, fix any lint or security issues
Prior to merge, run all the make infra-update-... targets and audit what they intend to do
If they seem sensible, run them all for demo environment
Observe if any fixes are needed, and make them before merging
Then merge the code
Apply the terraform updates to production manually, as well, just to verify
that the behavior matches demo.

I have applied the changes in this PR, so I am going to merge and continue testing in our demo environment!


Implement a way for the node to clean up any changed state in the case of an exceeded deadline in the state transition

This has been handled by incorporating sync-by-height value feature for Rusk service (Rusk mock).


[TEST-USE] Fine Tuning

Environment (with MongoDB + 1 type of data + 1 model):

[x]  local cluster
[x]  development mode

DBs (development mode + 1 non-trivial type of data + 1 model):

[x]  MongoDB
[x]  SQLite
[x]  Postgres

Data Format (MongoDB + 1 model + development mode)

[x]  Chat
[x]  Prompt-Response
[x]  Prompt

Training options (MongoDB + 1 type of data + development mode)

[x]  ray
[x]  deepspeed
[x]  lora
[x]  qlora
[x]  Multi-GPUS

Load Options

[x]  Directly
[x]  Use checkpoint

At the functional level, the test has been completed (running a small max_steps test). You need to continue to run the real finetune task on data in different formats to test the effect after the model training is completed. This can be executed in parallel with other work.
Data Format (MongoDB + 1 model + development mode)

[ ] Chat
[ ] Prompt-Response
[ ] Prompt


Make library work with render

Currently we are doing nothing with rendered components. chai-enzyme is supporting them nicely. We should find a way to support rendered components.

Does anybody actually use the render method with enzyme? I'm never use it and wonder if it's really necassary to support..

Hi,
actually I do use render for presentational components. There's often cases I want to test child elements and render is a bit easier. Otherwise you need the sub components imported to be able test against them.
const Svg = () => <svg />
const Foo = () =>
<Svg />

With render:
expect(wrapper.find('svg').toBePresent().

With shallow:
import Svg from '...';

expect(wrapper.find(Svg).toBePresent().

Recently switched to Jest, so we might actually do more snapshots, but still.

That's good to know. Thanks for reporting. I think we'all be in an easier place to support this after #39 is done.

@pascalduez you can use find by name with shallow without a problem.
What you need to do is search for your component name and not your HTML output.
so your With shallow code will be:
expect(wrapper.find('Svg')).toBePresent();

@luizbon Thanks for the feedback, I'll give it another try.
Still, there are (few) cases where I'd like to test against "real" DOM elements, but  well.


Multiple bugfixes

Please refer to individual commits for bug fixes.

I had to rebase after Lib changes. Incorporated review comment by @Yongqu

LGTM


policy: Reject ingress rules with DNS policies

Fixes #8537

This change is

Let me know if this is what you had in mind. I'm not sure whether/how to detect the toFQDNs case since it won't be marshalled into the IngressRule struct and will end up not having any effect as a result.

test-me-please


Possible bug in machine field ?

My laptop broke so the discovery to this :)

Ah dang it!
That phrase should not show up... It should be replacing "To be filled by O.E.M." with an empty string whenever it finds it, but for some reason it's printing that instead. Can you check if the files that we're reading from aren't actually the cause so we can pinpoint where the issue is creeping up from?

Here you go !

I have no clue what is supposed to be in there but it seems that .replace probably isn't replacing multiple "To be filled by O.E.M" strings.

I have no clue what is supposed to be in there but it seems that .replace probably isn't replacing multiple "To be filled by O.E.M" strings.

Yup, I'll :mag_right: investigate the issue.

Uhm, I'm not exactly sure why this is occuring, file processing looks good as far as I can tell... :confused:

I'll try to debug this later since college is really hectic right now.

I'll try to debug this later since college is really hectic right now.

I bet, it was really stressful these couple weeks for me too, good luck with everything :smile:

Found the culprit!
Right around the end of linux::machine() we have:
if family == name && family == version {
            return Ok(family);
        } else if version.is_empty() || version.len() <= 15 {
            return Ok(new_product.into_iter().join(" "));
        }

        Ok(version)

First condition & second condition are not satisfied, so it ends up returning version, which in your case is "To be filled..."

I hate the diversity of the information that manufacturers provide or don't provide in this case. We'll have to add a new set of conditions or somehow simplify it and make it a lot more maintainable for the future.


[docs][joy-ui] Fix the Templates link on the Overview page

[x] I have followed (at least) the PR section of the contributing guide.

Closes https://github.com/mui/material-ui/issues/39085

Nice
@danilo-leal changes like, you can likely move without peer reviews, it feels simple enough to save the distraction, especially since you wrote the code in the first place
(I'm coming from ahrefs audit)

@oliviertassinari thank you! Appreciate the instructions!


What is mean NOBROWSER mode

Can you explain -n nobrowser mode?
I see the browser on a snap, but it's not chrome.

It's for flows which don't use the browser. For example, if you're just clicking on the screen and automating other programs (SAP, Word).

Then what browser I have seen in the snap?

Just tested that when I run it, I don't see a browser. Can you screenshot what browser you mean?

I'm confused. Who render HTML if no browser?

Hi Egor, sorry I was away the past few months - https://github.com/tebelorg/RPA-Python/issues/144
I see, TagUI runs on the PhantomJS engine for executing JavaScript. With -nobrowser, there is still a PhantomJS invisible browser, which is based on an older version of WebKit.
However, most modern websites are not tested with PhantomJS, so it is not the recommended way to automate websites headlessly. To automate websites without seeing browser, use -h or -headless to run Chrome using invisible mode. Let me know if you have questions!


Loosen check of capitalization of keyword names optionally

Problem
Rule 0302 checks whether every word of a keyword name starts with capital letter. This is too strict for two reasons:

For other languages than English might be natural to name things with only the first letter of their name being capital.
Even in the (English) documentation of RF, we can find many keywords with only the first letter of their names being capital.

Suggestion
I'd like the rule 0302 to be configurable: the default value of its parametr would check for every first letter being capital, as it does up to now. When the other value of the parameter will be specified, check for first letter of the first word only would be performed.

Thanks @MoreFamed for your request.
First of all, even if the official documentation contains keywords that do not have all words capitalized, it is advised to follow such convention. Such examples are rather overlooked.
Secondly, in Polish, for example, we use only capital letter at the beginning of the sentence, but since keyword names are not sentences and it is a programming language with some specific syntax and conventions, it's OK to make all words in a keyword capitalized, in my opinion. It shouldn't be dependent on any language - it's just a convention of Robot Framework language.
Moreover, people responsible for Robot Framework development say that this practice should be default and I agree with them.
Linters are the tools that need to enforce some rules and thanks to this, the code is more consistent and easy to read. The only option I can see here is that we can provide a configurable parameter that can enable user to choose if he/she wants to use all words capitalized, or only the first one, but I still need to think if this is the way we want to go.

@mnojek, thank you for your reaction. I understand your point of view, however, I'd like to add some notes.
First of all, I missed to stress that we do write our own keywords in Czech (and I guess rules or conventions of capitalization in Czech are very similar to that ones in Polish). In Czech, there is natural to name things "Like this", not "Like This". In other words, Names Like This, when written in Czech, make reading a script harder.
And that is connected with my second remark. Language of RF has been designed to mimic natural language to a vast extent. This is why, for example, it allows spaces and letters with diacritics to be in names of entities---unlike usual programming languages. It would be natural, in my opinion, to allow even capitalization of names be decided by conventions of the natural language names are written in and Robocop should check the compliance with this convention...
Please, let me know when you decide about this issue.

I agree that we could add flag for this rule allowing to 'loosen' check only for first word. For example check_only_first_word or similar.


ONNX dynamic_axes cause compile time errors in final CoreML model

🐞Describe the bug
If an ONNX model is created with dynamic_axes then the subsequent CoreML model surfaces
 Description of image feature 'input_image' has missing or non-positive width 0.

and
Input 'input_image' of layer '63' not found in any of the outputs of the preceeding layers.

Compilation errors in xcode.
Please note that this is regardless of whether flexible inputs/outputs are specified with:
    img_size_ranges.add_height_range((64, -1)) # or 64, 2045 etc.
    img_size_ranges.add_width_range((64, -1))  # or 64, 2045 etc.

During the onnx-coreml conversion step.
More details
If dynamic_axes are removed and the img_size_ranges are removed then the model operates correctly, though can only accept a single image size.
If dynamic_axes are removed, but the img_size_ranges are still used as above then the compilation errors (above) go away. But a runtime error is introduced:
Finalizing CVPixelBuffer 0x282f4c5a0 while lock count is 1.
[espresso] [Espresso::handle_ex_plan] exception=Invalid X-dimension 1/480 status=-7
[coreml] Error binding image input buffer input_image: -7
[coreml] Failure in bindInputsAndOutputs.

More details and additional discussion is available: https://stackoverflow.com/questions/61231340/input-input-image-of-layer-63-not-found-in-any-of-the-outputs-of-the-preceed
Specific Functions
The following was used to create the ONNX model, and then compile it to CoreML
def create_onnx(name):
    prior = torch.load("pth/" + name + ".pth")
    model = transformer.TransformerNetwork()
    model.load_state_dict(prior)

    dummy_input = torch.zeros(1, 3, 64, 64) # I wasn't sure what I would set the H W to here?

    torch.onnx.export(model, dummy_input, "onnx/" + name + ".onnx",
                      verbose=True,
                      opset_version=10,
                      input_names=["input_image"], # These are being renamed from garbled originals.
                      output_names=["stylized_image"], # ^
                      dynamic_axes={'input_image':
                                    {2: 'height', 3: 'width'},
                                    'stylized_image':
                                    {2: 'height', 3: 'width'}}
                      )

    onnx.save_model(original_model, "onnx/" + name + ".onnx")

def create_coreml(name):
    mlmodel = convert(
            model="onnx/" + name + ".onnx",
            preprocessing_args={'is_bgr': True},
            deprocessing_args={'is_bgr': True},
            image_input_names=['input_image'],
            image_output_names=['stylized_image'],
            minimum_ios_deployment_target='13'
            )

    spec = mlmodel.get_spec()

    img_size_ranges = flexible_shape_utils.NeuralNetworkImageSizeRange()

    img_size_ranges.add_height_range((64, -1))
    img_size_ranges.add_width_range((64, -1))

    flexible_shape_utils.update_image_size_range(
        spec,
        feature_name='input_image',
        size_range=img_size_ranges)

    flexible_shape_utils.update_image_size_range(
        spec,
        feature_name='stylized_image',
        size_range=img_size_ranges)

    mlmodel = coremltools.models.MLModel(spec)

    mlmodel.save("mlmodel/" + name + ".mlmodel")

System environment (please complete the following information):

coremltools version: 3.3
onnx-coreml version: 1.2
macOS version (if applicable): 10.15.4
python version: 3.7

@BorisKourt using flexible_shape_utils is right way to add different image size inputs.
How are you running you model? what's the input you are providing?

@bhushan23, thanks for your prompt reply. Here is the code that references or interacts with the model (Simplified, tell me if you need a particular part. I can also try to make a minimal reproducible example):
Load:
let bundle = Bundle(for: styleTransferModel.self)
let modelURL = bundle.url(forResource: modelName, withExtension:"mlmodelc")!
let theModel = try? styleTransferModel(contentsOf: modelURL)
var prediction: styleTransferModelOutput?

Prediction call:
guard let imageBuffer = CMSampleBufferGetImageBuffer(sampleBuffer) else { return  }
prediction = try theModel?.prediction(input_image: imageBuffer)

Note that all of this code works if the model is not flexible.
The size of the input image is 480x640 (in all cases.)

I have also found this issue at coremltools: https://github.com/apple/coremltools/issues/276 which seems to have a similar runtime error. Unfortunately it seems like a bit of a dead end, what do you think?


🐛 Problems creating rows and columns in the superset.

Bug description
🐛 I'm having trouble trying to create divisions on the screen. The superset appears to have limitations in this regard.
I can't, for example, create a column and add the number of elements and their divisions. It always breaks.
can you help me?

Do you mean in the dashboard editor, like you're not able to drag and drop the layout elements for rows or columns?

mismo problema, no deja diseñar correctamente, sería util poder ordenarlo


icc19 with mkldnn, cached memory and sockeye size weight

Description
(Brief description on what this PR is about)
Checklist
Essentials
Please feel free to remove inapplicable items for your PR.

[ ] The PR title starts with [MXNET-$JIRA_ID], where $JIRA_ID refers to the relevant JIRA issue created (except PRs with tiny changes)
[ ] Changes are complete (i.e. I finished coding on this PR)
[ ] All changes have test coverage:
Unit tests are added for small changes to verify correctness (e.g. adding a new operator)
Nightly tests are added for complicated/long-running ones (e.g. changing distributed kvstore)
Build tests will be added for build configuration changes (e.g. adding a new build option with NCCL)
[ ] Code is well-documented:
For user-facing API changes, API doc string has been updated.
For new C++ functions in header files, their functionalities and arguments are documented.
For new examples, README.md is added to explain the what the example does, the source of the dataset, expected performance on test set and reference to the original paper if applicable
Check the API doc at http://mxnet-ci-doc.s3-accelerate.dualstack.amazonaws.com/PR-$PR_ID/$BUILD_ID/index.html
[ ] To the my best knowledge, examples are either not affected by this change, or have been fixed to be compatible with this change

Changes

[ ] Feature1, tests, (and when applicable, API doc)
[ ] Feature2, tests, (and when applicable, API doc)

Comments

If this change is a backward incompatible change, why must this change be made.
Interesting edge cases to note here

Does the original CPU path fused RNN still work?

Also the other PR is merged. Please rebase code.

Also the other PR is merged. Please rebase code.
Done

Does the original CPU path fused RNN still work?
Yes

@TaoLv comments has been fixed now


Remove codeowners file

Change-type: patch
Signed-off-by: Miguel Casqueira<EMAIL_ADDRESS>

Please make sure to read the CONTRIBUTING document before opening the PR for relevant information on contributing to the documentation. Thanks!

Added a double approval 💣


Hot to add a property to a class?

Hi,
first of all: thank you for this project and the good work! I really like it.
I would like some help on how to add a property to a class, because I am not sure where I am going wrong. I followed the tutorial and got the DuplicateWithSuffixGenerator to work successfully in my project (it is a netstandard project).
What I want to do is the following:
    [GeneratePropertyForType(typeof(StringBuilder))]
    public class GenTest1
    {
        public GenTest1()
        {
            //If the code generation works, it should generate a property with type and name StringBuilder
            //this.StringBuilder = new StringBuilder();
        }
    }

This is the Attribute an IRichCodeGenerator definition I am using:
    [AttributeUsage(AttributeTargets.Class, Inherited = false, AllowMultiple = true)]
    [CodeGenerationAttribute(typeof(PropertyGenerator))]
    public class GeneratePropertyForTypeAttribute : Attribute
    {
        public Type PropertyType { get; }

        public GeneratePropertyForTypeAttribute(Type propertyType)
        {
            PropertyType = propertyType;
        }
    }
    public class PropertyGenerator : IRichCodeGenerator
    {
        private AttributeData AttributeData { get; }
        public string TypeOfGeneratedProperty { get; }

        public PropertyGenerator(AttributeData attributeData)
        {
            AttributeData = attributeData;
            TypeOfGeneratedProperty = AttributeData.ConstructorArguments[0].Value.ToString();
        }

        public Task<SyntaxList<MemberDeclarationSyntax>> GenerateAsync(TransformationContext context, IProgress<Diagnostic> progress, CancellationToken cancellationToken)
        {
            //not called
            return Task.FromResult(new SyntaxList<MemberDeclarationSyntax>());
        }

        public Task<RichGenerationResult> GenerateRichAsync(TransformationContext context, IProgress<Diagnostic> progress, CancellationToken cancellationToken)
        {
            try
            {
                var applyToClass = context.ProcessingNode as ClassDeclarationSyntax;
                var property = SyntaxFactory.PropertyDeclaration(SyntaxFactory.ParseTypeName(TypeOfGeneratedProperty), TypeOfGeneratedProperty.Split('.').Last())
                    .AddModifiers(SyntaxFactory.Token(SyntaxKind.PublicKeyword))
                    .AddAccessorListAccessors(SyntaxFactory.AccessorDeclaration(SyntaxKind.GetAccessorDeclaration).WithSemicolonToken(SyntaxFactory.Token(SyntaxKind.SemicolonToken)))
                    .AddAccessorListAccessors(SyntaxFactory.AccessorDeclaration(SyntaxKind.SetAccessorDeclaration).WithSemicolonToken(SyntaxFactory.Token(SyntaxKind.SemicolonToken)))
                    .NormalizeWhitespace();
                var result = new RichGenerationResult
                {
                    Members = new SyntaxList<MemberDeclarationSyntax>().Add(applyToClass.AddMembers(property))
                };
                return Task.FromResult(result);
            }
            catch (Exception e)
            {
                Console.WriteLine(e);
                throw;
            }
        }
    }

I know the generator is called because throwing an Exception in GenerateRichAsync shows up in build log, and the GenTest1...generated.cs file looks good:

// ------------------------------------------------------------------------------
//
//     This code was generated by a tool.
//
//     Changes to this file may cause incorrect behavior and will be lost if
//     the code is regenerated.
//
// ------------------------------------------------------------------------------
using CodeGenerator;
using System;
using System.Collections.Generic;
using System.Text;
[GeneratePropertyForType(typeof(StringBuilder))]
public class GenTest1
{
public GenTest1()
{
//this.StringBuilder = new StringBuilder();
}
public System.Text.StringBuilder StringBuilder
{
    get;
    set;
}

}

However, if I try to uncomment the access to the StringBuilder-Property, I get an error saying it does not exist (in VS). What am I missing?
Thank you for your time and input!
Edit: Sorted out my Syntax game, code generation itself looks fine now.

Did you declare the class as partial?

Did you declare the class as partial?

No I didn't. Thinking about your reply I think I didn't correctly understand how the CG process works. I thought I could "alter" the source code, but I have to think more in the lines of adding source files to the compilation unit, correct?
I'll try partial declaration. How would I go about multiple instances of my attribute on the same class, i.e. wanting to have multiple properties generated this way: would I have to generate a partial class for each property, or is there a way to "bundle" them?

Digging further, I am at least able to identify my problem. In my generated file (see above), there is no namespace, which most probably is the reason that the generated partial class is not found.
The generated file from the example contains the corresponding namespace.
The only difference is that I am using the IRichCodeGenerator interface. What do I have to do to make it add the namespace?

IRichCodeGenerator docs
Well for your case it seems you really don't need any of the Rich additions, so I'd advise you to just use ICodeGenerator instead.
Rich variant allows you to add using declarations, change namespaces etc. - but it's Result.Members is essentially a list of root compilation unit members. So, for your specific instance, you'll need to either create the namespace syntax needed and add your class as a member of it, or reuse the Namespace syntax in which your context.ProcessingNode is.

Thank you for the explanation. I read the documentation, but did not realize the effect of the differences. I switched back to ICodeGenerator , and everything is working now. The only reason for me to try the IRichCodeGenerator in the first place was because I missed making the class partial (result from incomplete understandig of how the code generation works), so I the rich generator was necessary when enhancing existing types. My proof of concept works now, thank you for your help! I intend to use it at work to simplify some IoC-Injection-Boilerplate-Code.
For the sake of completeness, here is my working code:
    [CodeGenerationAttribute(typeof(PropertyGenerator))]
    public class GeneratePropertyForTypeAttribute : Attribute
    {
        public Type PropertyType { get; }

        public GeneratePropertyForTypeAttribute(Type propertyType)
        {
            PropertyType = propertyType;
        }
    }
    public class PropertyGenerator : ICodeGenerator
    {
        private string TypeOfGeneratedProperty { get; }

        public PropertyGenerator(AttributeData attributeData)
        {
            TypeOfGeneratedProperty = attributeData.ConstructorArguments[0].Value.ToString();
        }

        public Task<SyntaxList<MemberDeclarationSyntax>> GenerateAsync(TransformationContext context, IProgress<Diagnostic> progress, CancellationToken cancellationToken)
        {
            try
            {
                var applyToClass = context.ProcessingNode as ClassDeclarationSyntax;
                if (!ClassIsPartial(applyToClass, progress))
                    return Task.FromResult(new SyntaxList<MemberDeclarationSyntax>());

                var property = SyntaxFactory.PropertyDeclaration(SyntaxFactory.ParseTypeName(TypeOfGeneratedProperty), TypeOfGeneratedProperty.Split('.').Last())
                    .AddModifiers(SyntaxFactory.Token(SyntaxKind.PublicKeyword))
                    .AddAccessorListAccessors(SyntaxFactory.AccessorDeclaration(SyntaxKind.GetAccessorDeclaration).WithSemicolonToken(SyntaxFactory.Token(SyntaxKind.SemicolonToken)))
                    .AddAccessorListAccessors(SyntaxFactory.AccessorDeclaration(SyntaxKind.SetAccessorDeclaration).WithSemicolonToken(SyntaxFactory.Token(SyntaxKind.SemicolonToken)));

                var partialClass = applyToClass
                                    .WithMembers(new SyntaxList<MemberDeclarationSyntax>().Add(property))
                                    .WithAttributeLists(new SyntaxList<AttributeListSyntax>())
                                    .WithModifiers(new SyntaxTokenList().Add(SyntaxFactory.Token(SyntaxKind.PartialKeyword)))
                                    .NormalizeWhitespace();
                var result = new SyntaxList<MemberDeclarationSyntax>().Add(partialClass);
                return Task.FromResult(result);
            }
            catch (Exception e)
            {
                Console.WriteLine(e);
                throw;
            }
        }

        private bool ClassIsPartial(ClassDeclarationSyntax applyToClass, IProgress<Diagnostic> progress)
        {
            if (!applyToClass.Modifiers.Any(SyntaxKind.PartialKeyword))
            {
                progress.Report(
                    Diagnostic.Create(
                        new DiagnosticDescriptor("CA1001", "Cannot apply attribute to non-partial class", "Make class partial", "Microsoft.Design", DiagnosticSeverity.Error, true)
                        , Location.Create(applyToClass.SyntaxTree, applyToClass.Span)));
                return false;
            }
            return true;
        }
    }`

(I think closing this should be ok, since it is resolved).


RUN Node on Node that was executed using "Run Node"

HKube micro-service
caching Service
Describe the bug
execute the following pipeline
when done "Run Node" on Node "Two" - new job started
on the new Job try to "Run Node" on Node Three - getting error :
Error response, status=500, message=node "two" is depend on node "one" which is not exists
{
    "name": "Athos-Cartesian",
    "nodes": [
        {
            "nodeName": "one",
            "algorithmName": "eval-alg",
            "input": [
<EMAIL_ADDRESS><EMAIL_ADDRESS>            ],
            "batchOperation": "cartesian"
        },
        {
            "nodeName": "two",
            "algorithmName": "eval-alg",
            "input": [
                "#@one"
            ]
        },
        {
            "nodeName": "three",
            "algorithmName": "eval-alg",
            "input": [
                "#@two"
            ],
            "batchOperation": "indexed"
        }
    ],
    "flowInput": {
        "one": [
            0,
            1,
            2,
            3,
            4,
            5,
            6,
            7,
            8,
            9
        ],
        "two": [
            10,
            11,
            12,
            13,
            14,
            15,
            16,
            17,
            18,
            19
        ]
    },
    "kind": "batch",
    "experimentName": "main",
    "options": {
        "ttl": 3600,
        "batchTolerance": 80,
        "progressVerbosityLevel": "info"
    },
    "priority": 3

}

tested on
systemVersion: "v2.0.55",
fullSystemVersion: "v2.0.55-1610028130079",

tested on
systemVersion: "v2.0.55",
fullSystemVersion: "v2.0.55-1610028130079",


ch: normalize loadScript paths to host app's url

This is to support relative paths on LoadScript calls.
with this PR, the command below will work just fine;
> out/Debug/ch test/es6/module-namespace.js

previously, it was requiring to be in test/es6 folder to succeed.

@jackhorton @tcare Thanks for the review.


(feat): Pass chaosUID as labels in chaos Job

Signed-off-by: shubhamchaudhary<EMAIL_ADDRESS>
What this PR does / why we need it:
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
Special notes for your reviewer:
Checklist:

[ ] Fixes #
[ ] Labelled this PR & related issue with documentation tag
[ ] PR messages has document related information
[ ] Labelled this PR & related issue with breaking-changes tag
[ ] PR messages has breaking changes related information
[ ] Labelled this PR & related issue with requires-upgrade tag
[ ] PR messages has upgrade related information
[ ] Commit has unit tests
[ ] Commit has integration tests

Please solve the BCH issue before Merging

Try to bundle getKubeConfig() and clients.GenerateClientSets(config) creation function into one, to reduce BCH.

Can you write a typed function around engineDetails to set ENV from? @shubhamchaudhary
It won't cause with BCH then.

Can you write a typed function around engineDetails to set ENV from? @shubhamchaudhary
It won't cause with BCH then.
Can we do refactoring things in different PR for the increment of the BCH score?


[REQUEST] Custom fields for pages

Please add custom fields for pages, this would be very useful in themes.

There is already custom fields
https://docs.bludit.com/en/content/custom-fields

Worked thanks.


[BUG] Storage Class is not created when installing using Helm?

Describe the bug (🐛 if you encounter this issue)
Installing Longhorn using Helm Chart 1.4.1 does not create a Storage Class.
To Reproduce
Just install Longhorn using Helm chart version 1.4.1 and execute:
kubectl get sc -A

returns no resources.
Expected behavior
Get a storage class created
Environment
Longhorn version: 1.4.1
Installation method (e.g. Rancher Catalog App/Helm/Kubectl): Helm
Kubernetes distro (e.g. RKE/K3s/EKS/OpenShift) and version: baremetal
Number of management node in the cluster: 1
Number of worker node in the cluster: 2
Node config
OS type and version: Ubuntu 22
CPU per node: 2
Memory per node: 4gb
Underlying Infrastructure (e.g. on AWS/GCE, EKS/GKE, VMWare/KVM, Baremetal): baremetal
More info:
I am deploying a completely new Kubernetes cluster, thus I don't think that I am doing anything wrong here.
values:
values:
  persistence:
    defaultClass: true
    defaultFsType: ext4
    defaultClassReplicaCount: 2
    defaultDataLocality: disabled
    reclaimPolicy: Reclaim
  ingress:
    enabled: true
    host: "longhorn.{{ domain_name }}"
    ingressClassName: nginx

Does anybody have idea why is this happening?

It was my fault, the issue was the I used wrong reclaim policy.


refactor: resolved unreachable footer issue

The issue occurred on the project page where the projects component implemented infinite scrolling, making the footer inaccessible. Previously, as soon as the footer was reached, new projects would load, continually pushing the footer out of view. Now, after adjusting the footer's height, this trigger has been deactivated, allowing the footer to be accessed.
GGsoc contribution

removed 404 error now , Consider this issue contribution as gssoc

removed 404 error now , Consider this issue contribution as gssoc

hii projectshut is not in gssoc24

Will it be considered in gssoc?
On Sat, 11 May, 2024, 7:26 pm Priyankar Pal, @.***>
wrote:

Merged #2507 https://github.com/priyankarpal/projectshut/pull/2507 into
main.
—
Reply to this email directly, view it on GitHub
https://github.com/priyankarpal/projectshut/pull/2507#event-12776530625,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A5BMCFSW7JIVPCSKFS3CDJDZBYPPNAVCNFSM6AAAAABHQFJUVOVHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJSG43TMNJTGA3DENI
.
You are receiving this because you were mentioned.Message ID:
@.***>

Will it be considered in gssoc?
…
On Sat, 11 May, 2024, 7:26 pm Priyankar Pal, @.> wrote: Merged #2507 <#2507> into main. — Reply to this email directly, view it on GitHub <#2507 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/A5BMCFSW7JIVPCSKFS3CDJDZBYPPNAVCNFSM6AAAAABHQFJUVOVHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJSG43TMNJTGA3DENI . You are receiving this because you were mentioned.Message ID: @.>

no


Restructure workspace and project

Restructure workspace and project
Piwik (workspace)
  PiwikTracker (folder)
    source code
  PiwikiOSDemo (project with an iOS build target)
    demo source code
  PiwikOXDemo (project with and OSX build target
    demo source code
  Pods (pod dependency targets)

Look at the AFNetworking organisation.
Need different demo project and/or test project for iOS and OSX

Implemented.


OverflowError: signed integer is less than minimum

when I run eval.py with my own img and model trained with yolact_base_config,  the info printed as follows:
Traceback (most recent call last):
File "eval_custom.py", line 1107, in
evaluate(net, dataset)
File "eval_custom.py", line 887, in evaluate
evalimages(net, inp, out)
File "eval_custom.py", line 626, in evalimages
evalimage(net, path, out_path)
File "eval_custom.py", line 603, in evalimage
img_numpy = prep_display(preds, frame, None, None, undo_transform=False)
File "eval_custom.py", line 246, in prep_display
cv2.rectangle(img_numpy, (x1, y1), (x2, y2), color, 1)
OverflowError: signed integer is less than minimum
I checked some training log, found that it seems than the training process is a liitle weird:
[1016]  830880 || B:<PHONE_NUMBER>2543161677282561227675926528.000 | C: 17793.001 | M: 66.328 | S: 1.117 | T:<PHONE_NUMBER>2543161677282561227675926528.000 || ETA: 15 days, 14:32:35 || timer: 0.305
Warning: Moving average ignored a value of inf
Computing validation mAP (this may take a while)...
Warning: Augmentation output an example with no ground truth. Resampling...
Calculating mAP...
   |  all  |  .50  |  .55  |  .60  |  .65  |  .70  |  .75  |  .80  |  .85  |  .90  |  .95  |

-------+-------+-------+-------+-------+-------+-------+-------+-------+-------+-------+-------+
box |  0.00 |  0.00 |  0.00 |  0.00 |  0.00 |  0.00 |  0.00 |  0.00 |  0.00 |  0.00 |  0.00 |
mask |  0.00 |  0.00 |  0.00 |  0.00 |  0.00 |  0.00 |  0.00 |  0.00 |  0.00 |  0.00 |  0.00 |
-------+-------+-------+-------+-------+-------+-------+-------+-------+-------+-------+-------+
[1017]  831140 || B:<PHONE_NUMBER>4195462406891206246221742080.000 | C: 17793.001 | M: 66.321 | S: 1.120 | T:<PHONE_NUMBER>4195462406891206246221742080.000 || ETA: 25 days, 17:03:19 || timer: 2.696
Warning: Moving average ignored a value of inf
[1017]  831150 || B:<PHONE_NUMBER>4195462406891206246221742080.000 | C: 17793.001 | M: 68.646 | S: 1.116 | T:<PHONE_NUMBER>4195462406891206246221742080.000 || ETA: 25 days, 16:33:23 || timer: 0.304
I guess the loss had diverged to infinite.
Am I right?

Yeah, your loss diverged. What does your training setup look like?

Yeah, your loss diverged.

can we assert on that and crash? Anything is going bonkers probably anyway then..

@breznak Yeah an assert checking for a couple iterations in a row of nonfinite loss makes a lot more sense and would probably help people understand what's going on.

Yeah, your loss diverged. What does your training setup look like?

en , in the recent epochs before 110, the model evals successfully in pictures, and the results are OK. the models saved after 110 epoch all have the problem mentioned above. my data is kind of that a instance have many pieces in an img which all the pieces are not connected  in an insatance. I gusess it is the problem of my data.

Should we consider reopening to track the "if diverged, stop", which is currently not implemented?

Should we consider reopening to track the "if diverged, stop", which is currently not implemented?

I have apply yolact to train some models. they all works well. If you saved your training log, it is easy to find divergence from log. You can pay more attention to more important aspects.


Cannot build " set_target_properties Can not find target to add properties to: ggml"

(.venv) ➜  build git:(master) ✗ cmake .. -DBUILD_SHARED_LIBS=ON -DCMAKE_BUILD_TYPE=Release
-- The C compiler identification is GNU 9.4.0
-- The CXX compiler identification is GNU 9.4.0
-- Check for working C compiler: /usr/bin/cc
-- Check for working C compiler: /usr/bin/cc -- works
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Detecting C compile features
-- Detecting C compile features - done
-- Check for working CXX compiler: /usr/bin/c++
-- Check for working CXX compiler: /usr/bin/c++ -- works
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Detecting CXX compile features
-- Detecting CXX compile features - done
-- Looking for pthread.h
-- Looking for pthread.h - found
-- Performing Test CMAKE_HAVE_LIBC_PTHREAD
-- Performing Test CMAKE_HAVE_LIBC_PTHREAD - Success
-- Found Threads: TRUE
-- CMAKE_SYSTEM_PROCESSOR: x86_64
-- x86 detected
CMake Error at CMakeLists.txt:190 (add_subdirectory):
  The source directory

    /home/user/Tools/06_MachineLearning/BERT/bert.cpp/ggml

  does not contain a CMakeLists.txt file.

CMake Error at CMakeLists.txt:201 (set_target_properties):
  set_target_properties Can not find target to add properties to: ggml

-- Configuring incomplete, errors occurred!
See also "/home/user/Tools/06_MachineLearning/BERT/bert.cpp/build/CMakeFiles/CMakeOutput.log".

I had to checkout the ggml submodule.... for those that missed it;
cd ./bert.cpp && git submodule update --init --recursive


Ability to override max_age for each signIn request

Currently max_age can only be specified as an option to UserManager, but we need to be able to specify it per signIn call
The reason for this is for implementing an OIDC re-authentication flow for specific feature of the application.
Desired behaviour:

User login to the application via OIDC provider - no max_age parameter should be sent as we want to have SSO
User uses the specific high risk feature, and we want to make sure that it is still the same user at the keyboard. So the app would do a signinRedirect or signinPopup, with max_age set to a relative small period (e.g. 5 minutes).

Did a bit of experiment, looks like the library does allow the override, it's just the TypeScript typing giving errors


Update Korean localization

Catch up to commit 8bad20e58c47 ("add option of disable auto update").

Thank you so much.


Code needs serious cleanup.

As of now, there is no unnecessary code. Closed. Open if code needs cleaning up again.