Unity 2017.2 compatibility issue

Unity keeps spamming error :
MissingFieldException: Field 'UnityEditor.PreviewRenderUtility.cameraFieldOfView' not found.
When icon for brush needs to be drawn. Seems to be fairly quick fix ( I think ) but I can't attempt to build dlls. Please consider fixing this.. This is only issue that I can see for Unity 2017 at the moment. (that gets in a way of using it nicely)
Full error code :
MissingFieldException: Field 'UnityEditor.PreviewRenderUtility.cameraFieldOfView' not found.
Rotorz.Tile.Editor.BrushPreviewRenderUtility.BeginPreviewScene ()
Rotorz.Tile.Editor.BrushPreviewRenderUtility.CreateStaticTexture (Rotorz.Tile.Brush brush, Int32 width, Int32 height)
Rotorz.Tile.Editor.BrushUtility.CreateBrushPreview (Rotorz.Tile.Brush brush, Int32 width, Int32 height)
Rotorz.Tile.Editor.BrushEditor.RenderStaticPreview (System.String assetPath, UnityEngine.Object[] subAssets, Int32 width, Int32 height)
Rotorz.Tile.Editor.AssetPreviewCache.GenerateAssetPreview (UnityEngine.Object targetObject)
Rotorz.Tile.Editor.AssetPreviewCache.GetAssetPreview (UnityEngine.Object targetObject)
Rotorz.Tile.Editor.RotorzEditorGUI.DrawBrushPreviewHelper (Rect output, Rotorz.Tile.Editor.BrushAssetRecord record, Boolean selected)
Rotorz.Tile.Editor.RotorzEditorGUI.DrawBrushPreviewWithoutFallbackLabel (Rect output, Rotorz.Tile.Editor.BrushAssetRecord record, Boolean selected)
Rotorz.Tile.Editor.BrushListControl.DrawList (UnityEngine.Event e, Rect position, Rotorz.Tile.Brush selectedBrush)
Rotorz.Tile.Editor.BrushListControl.Draw (Rect position)
Rotorz.Tile.Editor.BrushListControl.Draw (Boolean showTabs)
Rotorz.Tile.Editor.BrushPaletteWindow.DrawBrushesGUI ()
Rotorz.Tile.Editor.BrushPaletteWindow.DoGUI ()
Rotorz.Tile.Editor.RotorzWindow.OnGUI ()
System.Reflection.MonoMethod.Invoke (System.Object obj, BindingFlags invokeAttr, System.Reflection.Binder binder, System.Object[] parameters, System.Globalization.CultureInfo culture) (at /Users/builduser/buildslave/mono/build/mcs/class/corlib/System.Reflection/MonoMethod.cs:222)
Rethrow as TargetInvocationException: Exception has been thrown by the target of an invocation.
System.Reflection.MonoMethod.Invoke (System.Object obj, BindingFlags invokeAttr, System.Reflection.Binder binder, System.Object[] parameters, System.Globalization.CultureInfo culture) (at /Users/builduser/buildslave/mono/build/mcs/class/corlib/System.Reflection/MonoMethod.cs:232)
System.Reflection.MethodBase.Invoke (System.Object obj, System.Object[] parameters) (at /Users/builduser/buildslave/mono/build/mcs/class/corlib/System.Reflection/MethodBase.cs:115)
UnityEditor.HostView.Invoke (System.String methodName, System.Object obj) (at C:/buildslave/unity/build/Editor/Mono/HostView.cs:285)
UnityEditor.HostView.Invoke (System.String methodName) (at C:/buildslave/unity/build/Editor/Mono/HostView.cs:278)
UnityEditor.HostView.InvokeOnGUI (Rect onGUIPosition) (at C:/buildslave/unity/build/Editor/Mono/HostView.cs:245)

It seems that Unity renamed and encapsulated some members of their PreviewRenderUtility class. Unity's automatic upgrade feature seemed to detect and resolve the problem automatically so that workaround should work in your project as well.
This issue should be resolve by the changes made in commit 391b906.

For some reason, Unity didn't upgrade this for me... dll release with the fix is really appreciated! Thanks.

I have tried to install the package to see if I can create dlls from it... but I just can't get passed the<EMAIL_ADDRESS>Permission denied (publickey)." error... I have solved the authentication issue.
I can get to :
$ ssh -T<EMAIL_ADDRESS>Hi Pixellore! You've successfully authenticated, but GitHub does not provide shell access.
But... I get :
npm install --save rotorz/unity3d-tile-system
npm ERR! Error while executing:
npm ERR! C:\Program Files\Git\cmd\git.EXE ls-remote -h -t<EMAIL_ADDRESS>npm ERR!
npm ERR<EMAIL_ADDRESS>Permission denied (publickey).
npm ERR! fatal: Could not read from remote repository.
npm ERR!
npm ERR! Please make sure you have the correct access rights
npm ERR! and the repository exists.
npm ERR!
npm ERR! exited with error code: 128

If I manually run : "C:\Program Files\Git\cmd\git.EXE" ls-remote -h -t<EMAIL_ADDRESS>I get something back.. :
85f866ccde94a1afa2eaa7aa5dee385bf711e2d2        refs/heads/master
c6565cf6805f72b191c3e8505609b32b4e95f1cf        refs/tags/v1.0.0
85f866ccde94a1afa2eaa7aa5dee385bf711e2d2        refs/tags/v1.0.0^{}
But obviously it didn't complete the installtion.

@Pixellore hmm if you've setup an SSH key on your Github account and on your local machine; I don't see why npm wouldn't be picking it up. I am using npm version 5.4.0; could you retry using that version to see if the problem persists.
npm install -g<EMAIL_ADDRESS>

Note - Installing the packages in this way will not work very well in an existing project
because Unity does not maintain asset links with scripts when switching from DLL / loose
source distributions. Installing the packages this way is of course useful if you want to build
the DLLs yourself.

I will build the DLLs sometime today for you and add to the releases page.

I think it has got to do with read/write access to git when I don't have the write access and trying to access it by ssh instead of https.
C:\Program Files\Git\cmd\git.EXE ls-remote -h -t<EMAIL_ADDRESS>I don't think I can do this because I don't have the write access.. but I can with https. I was looking whole day for this issue and there are few people having the same issue over ssh. I tried to find way to force npm to access git by https (dependencies) but I was unsuccessful.
https://github.com/npm/npm/issues/3830

Please try with v5.4.0; I would be interested to know if that works for you.

Oh, yeah, I just did and it does seem to work! Interesting... something must have changed for the recent version of npm.. But please do make dll for me.. I will do my best to try and learn how to make dll from now, but there is no promise that I will succeed in that area too. XD

Sorry man.. I was going to post this quick. but just tried the fixed version and got this :
Assertion failed: System is already interested in this transform, SetSystemInterested may only be called once.
UnityEditor.PreviewRenderUtility:BeginStaticPreview(Rect)
Rotorz.Tile.Editor.BrushPreviewRenderUtility:CreateStaticTexture(Brush, Int32, Int32) (at Assets/Plugins/Packages/@rotorz/unity3d-tile-system/Editor/Brush/BrushPreviewRenderUtility.cs:289)
Rotorz.Tile.Editor.BrushUtility:CreateBrushPreview(Brush, Int32, Int32) (at Assets/Plugins/Packages/@rotorz/unity3d-tile-system/Editor/Brush/BrushUtility.cs:155)
Rotorz.Tile.Editor.BrushEditor:RenderStaticPreview(String, Object[], Int32, Int32) (at Assets/Plugins/Packages/@rotorz/unity3d-tile-system/Editor/Brush/BrushEditor.cs:93)
UnityEditor.AssetDatabase:ImportAsset(String)
Rotorz.Tile.Editor.BrushUtility:RefreshPreviewIncludingDependencies(Brush) (at Assets/Plugins/Packages/@rotorz/unity3d-tile-system/Editor/Brush/BrushUtility.cs:190)
Rotorz.Tile.Editor.OrientedBrushDesigner:InsertNewVariation(BrushOrientation, Int32, Object) (at Assets/Plugins/Packages/@rotorz/unity3d-tile-system/Editor/Brush/Designer/OrientedBrushDesigner.cs:1663)
Rotorz.Tile.Editor.OrientedBrushDesigner:AcceptPendingVariations(BrushOrientation) (at Assets/Plugins/Packages/@rotorz/unity3d-tile-system/Editor/Brush/Designer/OrientedBrushDesigner.cs:1518)
Rotorz.Tile.Editor.OrientedBrushDesigner:DrawOrientationTrackBackground(Int32, Rect, BrushOrientation) (at Assets/Plugins/Packages/@rotorz/unity3d-tile-system/Editor/Brush/Designer/OrientedBrushDesigner.cs:658)
Rotorz.Tile.Editor.OrientedBrushDesigner:DrawOrientationListBackground(Rect) (at Assets/Plugins/Packages/@rotorz/unity3d-tile-system/Editor/Brush/Designer/OrientedBrushDesigner.cs:432)
Rotorz.Tile.Editor.OrientedBrushDesigner:Section_Orientations() (at Assets/Plugins/Packages/@rotorz/unity3d-tile-system/Editor/Brush/Designer/OrientedBrushDesigner.cs:323)
Rotorz.Tile.Editor.OrientedBrushDesigner:OnGUI() (at Assets/Plugins/Packages/@rotorz/unity3d-tile-system/Editor/Brush/Designer/OrientedBrushDesigner.cs:115)
Rotorz.Tile.Editor.DesignerView:Draw() (at Assets/Plugins/Packages/@rotorz/unity3d-tile-system/Editor/Window/Designer/DesignerView.cs:425)
Rotorz.Tile.Editor.DesignerWindow:OnGUI_DesignerView() (at Assets/Plugins/Packages/@rotorz/unity3d-tile-system/Editor/Window/Designer/DesignerWindow.cs:674)
Rotorz.Tile.Editor.DesignerWindow:DoGUI() (at Assets/Plugins/Packages/@rotorz/unity3d-tile-system/Editor/Window/Designer/DesignerWindow.cs:436)
Rotorz.Tile.Editor.RotorzWindow:OnGUI() (at Assets/Plugins/Packages/@rotorz/unity3d-tile-system/Editor/RotorzWindow.cs:194)
UnityEngine.GUIUtility:ProcessEvent(Int32, IntPtr)
Another hill to climb!

This seems like a bug in Unity 2017.2.

Just tried with Unity <IP_ADDRESS>b5 and it does the same .. damn it. Will try 2017.1

I reported this issue to Unity (Case 962008).

The variations that are shown in the brush designer are taken from the prefabs. The tile produced by a brush is something that doesn't exist until the brush is used. So the brush preview is generated by constructing a 1x1 tile system, painting the tile, rendering the preview, caching the preview and then using that as the preview for the brushes palette.

Understood. I have desperately tried to make it work only for the orientation brush, (becasue they are the only ones I am using) by : Editing brusheditor
public override Texture2D RenderStaticPreview(string assetPath, Object[] subAssets, int width, int height)
{
/*
var brush = target as Brush;
        var brushDescriptor = BrushUtility.GetDescriptor(brush.GetType());
        if (brushDescriptor != null && brushDescriptor.CanHavePreviewCache(brush)) {
            return BrushUtility.CreateBrushPreview(target as Brush, width, height);
        }
        else {
            return null;
        }
        */
        try
        {
            var brush = target as Brush;

            var brushDescriptor = BrushUtility.GetDescriptor(brush.GetType());
            if (brushDescriptor != null && brushDescriptor.CanHavePreviewCache(brush))
            {
                var orientatedBrush = target as OrientedBrush;
                if (orientatedBrush != null)
                {
                    var ori = orientatedBrush.DefaultOrientation;
                    if (ori != null)
                    {
                        if (ori.VariationCount > 0)
                        {
                            var obj = ori.GetVariation(0);
                            if (obj != null)
                            {

                                if (!AssetPreview.IsLoadingAssetPreviews())
                                {
                                    Texture2D texture2D = AssetPreview.GetAssetPreview(obj);
                                    if (texture2D != null)
                                    {
                                        return texture2D;
                                    }
                                }
                            }
                        }
                    }
                }
            }
        }
        finally
        {

        }
        return null;
    }
}

And it does work for the first time, but then when I refresh the preview, it crashes Unity out right. :(
I guess I will need to wait for the bug to be fixed, but you know Unity... No way to tell when that is going to happen.

I have been able to reproduce the issue my end... however Unity are currently unable to reproduce the issue. They are claiming that the preview is being generated without any errors in the latest beta. I am wondering if maybe they are testing on OS X rather than Windows. Are you able to test on OS X?

Just a note; Github issues use the following syntax to markup code listings:
```{language}
```

for example,
```csharp
public sealed class Foo
{
}
```

becomes:
public sealed class Foo
{
}

No.. I am unable to test it on OS X. The issue that you are talking about, is that the AssetPreview or the original bug we have discovered?

The assertion error.
The AssetPreviewUtility is not designed to be used in the way that you are using it above. It should be used when drawing the preview to the GUI. Unity is probably crashing because the preview utility returns null over and over again whilst the preview is loaded from cache on a different thread.
If you want to use AssetPreviewUtility then you will need to register a different BrushDescriptor for oriented brushes which draws the preview using that instead.
Your implementation would return false for CanHavePreviewCache and would use AssetPreviewUtility inside its implementation of DrawPreview. When you have more than (I think it's 10) previews visible you may notice that they all start flickering. This is because AssetPreviewUtility keeps fewer previews than are visible in memory.
Copying the following may be a good starting point for you for your custom preview drawing logic: https://github.com/rotorz/unity3d-tile-system/blob/master/assets/Editor/Brush/Descriptor/OrientedBrushDescriptor.cs
You can then register your custom implementation with something like this:
[InitializeOnLoad]
public class MyCustomBrushDescriptor : BrushDescriptor
{
    static MyCustomBrushDescriptor()
    {
        // This will override the oriented brush descriptor.
        BrushUtility.RegisterDescriptor(new MyCustomBrushDescriptor());
    }

    public MyCustomBrushDescriptor()
        : base(typeof(OrientedBrush), typeof(OrientedBrushDesigner), typeof(AliasBrushDesigner))
    {
    }

    // ...
}

I was able to get non crashing version by doing the below : copying the texture from one texture2d to another. I know this isn't probably the way to go... but.. for for some interest sake.
public override Texture2D RenderStaticPreview(string assetPath, Object[] subAssets, int width, int height)
        {
            /*
            var brush = target as Brush;

            var brushDescriptor = BrushUtility.GetDescriptor(brush.GetType());
            if (brushDescriptor != null && brushDescriptor.CanHavePreviewCache(brush)) {
                return BrushUtility.CreateBrushPreview(target as Brush, width, height);
            }
            else {
                return null;
            }
            */
            try
            {
                var brush = target as Brush;

                var brushDescriptor = BrushUtility.GetDescriptor(brush.GetType());
                if (brushDescriptor != null && brushDescriptor.CanHavePreviewCache(brush))
                {
                    var orientatedBrush = target as OrientedBrush;
                    if (orientatedBrush != null)
                    {
                        var ori = orientatedBrush.DefaultOrientation;
                        if (ori != null)
                        {
                            if (ori.VariationCount > 0)
                            {
                                var obj = ori.GetVariation(0);
                                if (obj != null)
                                {
                                    if (!AssetPreview.IsLoadingAssetPreviews())
                                    {
                                        GameObject go = obj as GameObject;
                                        Texture2D texture2D = AssetPreview.GetAssetPreview(go);
                                        texture2D.Apply();

                                        Color32[] pix = texture2D.GetPixels32();
                                        Texture2D destTex = new Texture2D(texture2D.width, texture2D.height);
                                        destTex.SetPixels32(pix);
                                        destTex.Apply();

                                        if (destTex != null)
                                            return destTex;
                                    }
                                }
                            }
                        }
                    }
                }
            }
            finally
            {

            }
            return null;
        }

    }

You can copy it with:
var duplicatedTexture = Instantiate(previewTexture);

The trouble is that you will often get null from AssetPreview.GetAssetPreview because if the preview hasn't already been cached it will return null until it's retrieved. Unity do this to avoid freezing the UI thread; previews just pop up when they are ready.

Thanks for the tip.
I would like to just try and build dll for my current use only without doing pull request on the git.  I tried to follow the instruction on the forum, but it's been a up hill battle for me. This is even harder than I thought.. I just need to be able to build dlls without contributing to the git because the method I want to use isn't proper one and I just need dlls until Unity fixes the bug. Is there any simplier way to create dll just for myself?
I tried to run : node ./scripts/prepare-solution.js
On folder such as : E:\Pixellore\Rotorz\node_modules@rotorz\unity3d-tile-system
but it gives me error :
E:\Pixellore\Rotorz\node_modules@rotorz\unity3d-tile-system>node ./scripts/prepare-solution.js
module.js:529
throw err;
^
Error: Cannot find module 'glob'
at Function.Module._resolveFilename (module.js:527:15)
at Function.Module._load (module.js:476:23)
at Module.require (module.js:568:17)
at require (internal/module.js:11:18)
at Object. (E:\Pixellore\Rotorz\node_modules@rotorz\unity3d-tile-system\scripts\prepare-solution.js:8:14)
at Module._compile (module.js:624:30)
at Object.Module._extensions..js (module.js:635:10)
at Module.load (module.js:545:32)
at tryModuleLoad (module.js:508:12)
at Function.Module._load (module.js:500:3)
So I don't have sln file to call MSBuild manually...

Ok, no worries.. I have got around building dll problem. What I did was to just add OrientedBrushEditor.cs script with the current dlls. OrientedBrushEditor just has the above code to spit out the preview image. I am only working with the orientedbrushes so this will do for me until Unity fixes the bug. Thanks a lot for working with me till now. It's been really useful!

Unity have managed to reproduce the issue and have notified their developers about it.

Excellent. It should show up in the issue tracker soon then.. I will keep my eyes on it.


Azure: Remove a need to provide secret while deploying CAA

In this new approach of not needing to have secrets, we will still create an identity like we do here but instead of passing the secret AZURE_CAA_CLIENT_SECRET to the daemonset, we will assign an identity to the worker nodes or to the CAA daemonset (workload) itself.
Current Approach
In current approach we create a service principal and use the client_id, client_secret produced out of it and give it to the CAA daemonset.
Advantages

Simpler works on any type of kubernetes be it a bring-your-own kubernetes or on AKS (of course it has to be on Azure).

Disdvantages

Have to manage a secret.
Does not work in CI set up we are trying to build.

Assign identity to the VM
Here we will create an identity and assign it to the worker nodes. So any request originating from the worker node will have permissions to create peer pod VM.
Advantages

No need to manage a client_secret.
It will work with CI.
Regardless of the k8s cluster type this will work. It will work for AKS and any DIY k8s setup on Azure.

Disadvantages

Obviously this allows "any" workload to create peer pod vm in the same RG.

Assign identity to the workload
Here you can see how one can assign identity to the k8s workloads to be able to communicate with the Azure APIs.
Advantages

No need to manage a client_secret.
It will work with CI.
Only the CAA pods will have that permission to create peer pod VMs.

Disadvantages

Won't work outside AKS, unless we figure out a way to provision an OIDC issuer on any k8s cluster running on Azure.

We can support multiple approaches but we have to choose one to ensure things work on CI and for users.

This is a great idea. We can support both the options - viz assigning identity to CAA (and peerpod-ctrl - since this does cleanups for VMs if required) and passing client_secret.

So there is a way to assign identity to self-managed clusters on Azure: https://azure.github.io/azure-workload-identity/docs/topics/self-managed-clusters.html
I haven't tried it so not sure how this will be translated into our CAA deployments.

Could you use https://kubernetes.io/docs/tasks/configure-pod-container/configure-service-account/#service-account-token-volume-projection with your azure clusters? This would be a more cross-provider friendly mechanism?

Could you use https://kubernetes.io/docs/tasks/configure-pod-container/configure-service-account/#service-account-token-volume-projection with your azure clusters? This would be a more cross-provider friendly mechanism?

@jtumber-ibm I don't I understand what you mean here. Service account tokens allow you to talk to the k8s api server, so using the k8s RBAC one can define what's allowed.
But here I am talking about how do we allow an application to talk to the Azure API without needing any secret? AKS has a mechanism called workload identity, which is similar to k8s service account token. On the Azure side you define what a particular "workload" on AKS can do with azure APIs and bam, you don't need secret.
My concern was how do we get this working for other DIY k8s clusters on Azure?!

Sorry if I wasn't clear. Does Azure not provide a Service Account token to IAM role token endpoint?

Yes it does but I know this works with the AKS, but not sure how it will work with DIY clusters on Azure. There seems to be a way to do that with DIY clusters, but I haven't tried it myself.


added the reqirements in the repo to anticipate any unwanted errors duri...

...ng installation by separately installing all the required packages.

Not sure I understand the issue. Either way not going to version control the dependencies in this repository. Feel free to open an issue for any install bugs you are running into and we can look into it.


Support all template-lint extensions

ember-template-lint support mjs, so we should key off that, when possible -- I also added cjs because it's spiritually equiv to js in v1 apps right now.
Without this change, we can see ember-template-lint working in the terminal, but not in editors.

how should this be tested? :thinking: (I'm new here)

@NullVoxPopuli testing same as debugging - https://github.com/ember-tooling/vscode-ember/blob/master/DEBUGGING.md
In short, we need to bundle LS and check how it's works in VSCode extension (or other env's)

I think something may have been lost in the repo move, perhaps -- there is no Launch Client :sweat_smile:

@NullVoxPopuli configs for debugging located here: https://github.com/ember-tooling/vscode-ember/blob/master/.vscode/launch.json#L35

hm, why isn't my breakpoint bound?

Looks like debugger not connected to LS, or LS not running (try do some code action to initialise LS)

I think I can confirm this works

still struggling with getting any breakpoint to hit tho :sweat_smile:
I want to try Promise.race

@NullVoxPopuli thank you for contribution, fix should be available in VScode - https://github.com/ember-tooling/vscode-ember/releases/tag/v3.0.57


Support unicode chars in xbmc.translatePath and fix tests

This fixes https://github.com/emilsvennesson/script.module.inputstreamhelper/issues/333

I think mnt_path() should return to_unicode(mount_path), then it should actually fix the issue.

So the paths that I'd add to_unicode to are:
backup_path
mount_path
temp_path

I'll take another look at the other path functions, but the root cause is xbmc.translatePath in Kodi 18.x (Python 2.7) which needs a bytestring as input and returns a bytestring as output.
So, I ensured that the translate_path wrapper function does the necessary conversions in Python 2.7 with to_unicode and from_unicode.

Right, I forgot you added a test for a non-ascii temp_path already.
But translate_path already had a to_unicode before, so the output should already have been unicode, yet still mount_path is a bytestring.

Are you sure the output of xbmc.translatePath is a "normal" bytestring and not something kodi specific, which wouldn't be recognized by isinstance(text, bytes)?

Are you sure the output of xbmc.translatePath is a "normal" bytestring
Yes, it's a bytestring in Kodi 18.x Python2

Testing unicode was unreliable because unicode_literals was imported in the tests. I'm trying to fix this now. Thinking how to mimic Kodi 18 and 19 behaviour with addon_settings in xbmcextra.py


Update OpenAiSolutionResponse.php

The goal is to fix this issue.

Fixed by https://github.com/spatie/error-solutions/pull/9


Breaks in Meteor using IE8 due to export.js

In meteor/export.js:
// moment.js makes `moment` global on the window (or global) object, while Meteor expects a file-scoped global variable
moment = this.moment;
delete this.moment;

the delete this.moment call causes an error in IE8 and breaks Meteor applications.
Should be surrounded with a try/catch as per this StackOverflow

I had a same problem.

Fixed in 606d7cfafa6ac72


Build Project Failed after add dependency

After adding these dependency to my project
libs.version.toml :
compass = "0.1.8"
compass-geocoder = { module = "dev.jordond.compass:compass-geocoder", version.ref = "compass" }
compass-geocoder-mobile = { module = "dev.jordond.compass:compass-geocoder-mobile", version.ref = "compass" }
compass-geocoder-web-googlemaps = { module = "dev.jordond.compass:compass-geocoder-googlemaps", version.ref = "compass" }

build.gradle.kts :
val commonMain by getting {
            dependencies {
                api(libs.decompose.router)
                implementation(libs.compass.geocoder)
                implementation(libs.compass.geocoder.web.googlemaps)
..

val mobileMain by creating {
            dependsOn(commonMain)
            androidMain.dependsOn(this)
            iosMain.dependsOn(this)
            dependencies {
                implementation(libs.compass.geocoder.mobile)
            }
        }

build project was failed with this log report

Execution failed for task ':composeApp:transformIosMainCInteropDependenciesMetadataForIde'.

Could not resolve all files for configuration ':composeApp:iosMainResolvableDependenciesMetadata'.
Could not find dev.jordond.compass:compass-geocoder-googlemaps:0.1.8.
Required by:
project :composeApp

perhaps anybody know the cause or fix for this issue

I think it is typo on docs
compass-geocoder-web-googlemaps = { module = "dev.jordond.compass:compass-geocoder-googlemaps", version.ref = "compass" }
should be
compass-geocoder-web-googlemaps = { module = "dev.jordond.compass:compass-geocoder-web-googlemaps", version.ref = "compass" }

You're correct, there was a typo in the docs for the dependency. I've committed the change.
Compass is still under development, so there's gonna be small things like this 😅.


fix uninitialized constant in lib/valid_email/validate_email.rb

This file needed a require 'mail' to eliminate the exception.

@hallelujah Ping :-)

+1
@tmornini, I would also remove require 'mail'in the following places:

ban_disposable_email_validator.rb
email_validator.rb
mx_validator.rb

Adding require 'mail' to validate_email.rb makes these lines unnecessary.

@merhard Are you sure the specs will run OK?
I'm a big fan of every file requiring everything it needs. Would have solved this issue, right? :-)

@tmornini 100% (I cloned your fork, made those changes, and ran the specs myself to confirm). If you look at the code in those files, they do not use anything from the Mail gem. require 'mail' is only there to make ValidateEmail work, which your pull request has fixed.

@merhard Thanks! Forgive my ignorance, but can you join into this PR with an additional commit? If so, please do so!

@hallelujah :+1:

Ping...

@hallelujah Ping!

@hallelujah PIng...


Error with dbt_project.yml when using extension

I have been getting this error after updating to dbt version 1.0.0:
YAMLSemanticError: Map keys must be unique; "+tags" is repeated
Earlier it was running seamlessly and the same project doesn't have issues running with DBT cloud.

Hey shardkul! Apologies for the late response, I haven't been checking this as much lately.
That error is popping up due to a library we use in the extension to parse the YAML of the dbt_project.yml file. It is caused by having two or more +tags at the same hierarchy level. Could you check whether this is true and remove the redundant +tags? That should solve it.


✗ Error uploading to Backblaze B2  InvalidAccessKeyId: Malformed Access Key Id at throwDefaultError

I can't figure out how to set .env for backblaze, is access key the same thing as application key?
In backblaze I see:
keyID: xxxxxx
keyName: xxxxxx
applicationKey: xxxxxxxxxxxxxx
and in my .env my best guess is this which gives an error:
BACKBLAZE_ACCESS_KEY_ID=(keyID goes here?)
BACKBLAZE_SECRET_ACCESS_KEY=(applicationKey goes here?)

yes
keyId is BACKBLAZE_ACCESS_KEY_ID
applicationKey is BACKBLAZE_SECRET_ACCESS_KEY
try generating a new application key, not the default provided master app key


[concurrency] Implement the Task allocator as bump-pointer allocator.

rdar://problem/71157018
TODO: This is still not finished yet. I need to polish it a bit.

Cool! I was just implementing an allocator in the Swift runtime for use with automatic differentiation — I used llvm::BumpPtrAllocator and it worked fine for me. Is there a reason to implement a new allocator instead of using llvm::BumpPtrAllocator? If so, perhaps AutoDiff can adopt this implementation as well.

@rxwei There are several differences, most importantly that the task allocator also can free memory (in a stack discipline).
Also, the task allocator uses "inlined" memory for the first slab, which avoid malloc at all if only a small amount of memory is needed.

@rxwei There are several differences, most importantly that the task allocator also can free memory (in a stack discipline).
Also, the task allocator uses "inlined" memory for the first slab, which avoid malloc at all if only a small amount of memory is needed.

These features would be needed for AutoDiff as well. I’m interested in using this to replace the allocator in #34886. Do you think it can be moved to a header file that we can import?

@swift-ci benchmark

Performance: -O

Regression
OLD
NEW
DELTA
RATIO

UTF8Decode_InitFromBytes_ascii_as_ascii
449
533
+18.7%
0.84x (?)

Improvement
OLD
NEW
DELTA
RATIO

UTF8Decode_InitFromData_ascii_as_ascii
800
642
-19.7%
1.25x (?)

Code size: -O
Performance: -Osize

Improvement
OLD
NEW
DELTA
RATIO

FlattenListLoop
2503
1631
-34.8%
1.53x (?)

FlattenListFlatMap
5790
4143
-28.4%
1.40x (?)

Code size: -Osize
Performance: -Onone
Code size: -swiftlibs

How to read the data
The tables contain differences in performance which are larger than 8% and
differences in code size which are larger than 1%.
If you see any unexpected regressions, you should consider fixing the
regressions before you merge the PR.
Noise: Sometimes the performance results (not code size!) contain false
alarms. Unexpected regressions which are marked with '(?)' are probably noise.
If you see regressions which you cannot explain you can try to run the
benchmarks again. If regressions still show up, please consult with the
performance team (@eeckstein).

Hardware Overview
  Model Name: Mac Pro
  Model Identifier: MacPro6,1
  Processor Name: 12-Core Intel Xeon E5
  Processor Speed: 2.7 GHz
  Number of Processors: 1
  Total Number of Cores: 12
  L2 Cache (per Core): 256 KB
  L3 Cache: 30 MB
  Memory: 64 GB

@swift-ci smoke test

@rxwei makes sense. I moved it to a separate file.

@swift-ci smoke test linux

I pushed a new version, which now does not use a pre-allocated first slab - to keep the task allocation small.
We can improve this in folllow-up work, e.g. by providing stack-allocated space or space allocated with the parent task's allocator.

@swift-ci test

Build failed
Swift Test OS X Platform
Git Sha - 8220224b5b854d4c0e7357ef268d67099732ac12

Build failed
Swift Test Linux Platform
Git Sha - 8220224b5b854d4c0e7357ef268d67099732ac12

@swift-ci test

Build failed
Swift Test Linux Platform
Git Sha - 635a89ca02067344409c76afe16c07e9181efac0

Build failed
Swift Test OS X Platform
Git Sha - 635a89ca02067344409c76afe16c07e9181efac0

@swift-ci test


Feature/spark/support mongostream source

Problem
Add support for sending events when reading from mongo db using spark streaming.
Closes: #2886
Solution
Mongo streaming visitor + tests.
One-line summary:
Mongo streaming visitor + tests.
Checklist

[ ] You've signed-off your work
[ ] Your pull request title follows our guidelines
[ ] Your changes are accompanied by tests (if relevant)
[ ] Your change contains a small diff and is self-contained
[ ] You've updated any relevant documentation (if relevant)
[ ] Your comment includes a one-liner for the changelog about the specific purpose of the change (not required for changes to tests, docs, or CI config)
[ ] You've versioned the core OpenLineage model or facets according to SchemaVer (if relevant)
[ ] You've added a header to source files (if relevant)

SPDX-License-Identifier: Apache-2.0
Copyright 2018-2024 contributors to the OpenLineage project

@pawel-big-lebowski Spark 3.5 tests were failing due to outdated version of the connector, I bumped it :)

@Imbruced would you mind resolving conflict?


[Docs] Add uprade hive table doc

Why are the changes needed?

Add uprade hive table doc to guide user.
Brief change log

Add uprade hive table doc in table-management.md
Modify flink-dim.md

How was this patch tested?

[x] Run test locally before making a pull request

Documentation

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

cc @zstraw  @shendanfengg


[Request] Add nullability annotations where possible

This again is meant to help with kotlin method imports, but is helpful for java as well.
Where possible, ambiguous inputs should be annotated with @Nullable or @NonNull.
A notable part is the payload, but I will check with my builds when I have time to see which spots kotlin will recognize as potentially null.
This is up for discussion, as you may not want annotations everywhere, but I can submit a PR for this if this is approved.

@AllanWang that's already the case for the core module as you can see here:
https://github.com/mikepenz/FastAdapter/blob/develop/library-core/src/main/java/com/mikepenz/fastadapter/package-info.java
everything is nonnull by default.
It might be missing for some extension module

Okay that is great. I recently saw the warning in my lint and was wondering whether it was enforced. Good to know that it is.

I'm closing this for now, but given that this change was made in July and I've used FastAdapter after, that annotation may not have been compatible with the auto import.
(Pressing ctrl + o will open a list of overridable methods, and Kotlin will set parameters as nullable if it isn't explicitly declared otherwise)
If this is still an issue I will reopen it

@FabianTerhorst does this work for you? I tried enabling the flag and importing bindView still results in a null payload.
Relevant post


Port updates for removing EULA acceptance checkbox from Arc deployments

Don't like doing this as one large PR but wanted to directly cherry pick the commits which came with some extra stuff that isn't strictly necessary for this.
The main change here is that we're removing the checkbox acceptance of the azdata EULA - instead using the azdata extension to handle that. The extension does its own prompting of EULA acceptance and so if that has already been done then we'll just skip straight to deployment - no extra steps are needed to accept it each time like currently.
@alanrenmsft If you have time to look over the changes to resource-deployment that'd be great. The changes there should all either be contained to the new functionality or only slightly modifying the existing functionality so I have pretty high confidence this isn't going to break existing deployments but an extra pair of eyes here wouldn't hurt.

Pull Request Test Coverage Report for Build 259999561

83 of 258 (32.17%)  changed or added relevant lines in 20 files are covered.
7 unchanged lines in 4 files lost coverage.
Overall coverage decreased (-0.06%) to 38.891%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

extensions/resource-deployment/src/services/apiService.ts
2
3
66.67%

extensions/resource-deployment/src/ui/deployClusterWizard/pages/azureSettingsPage.ts
0
1
0.0%

extensions/resource-deployment/src/ui/deployClusterWizard/pages/clusterSettingsPage.ts
0
1
0.0%

extensions/resource-deployment/src/ui/deployClusterWizard/pages/serviceSettingsPage.ts
0
1
0.0%

extensions/resource-deployment/src/ui/dialogBase.ts
0
1
0.0%

extensions/resource-deployment/src/ui/notebookWizard/notebookWizard.ts
0
1
0.0%

extensions/resource-deployment/src/ui/notebookWizard/notebookWizardPage.ts
0
1
0.0%

extensions/resource-deployment/src/ui/wizardBase.ts
0
1
0.0%

extensions/resource-deployment/src/ui/deploymentInputDialog.ts
0
2
0.0%

extensions/resource-deployment/src/services/tools/toolBase.ts
1
4
25.0%

Files with Coverage Reduction
New Missed Lines
%

extensions/azdata/src/azdata.ts
1
48.84%

extensions/resource-deployment/src/services/tools/azdataTool.ts
1
44.12%

extensions/resource-deployment/src/ui/dialogBase.ts
1
15.38%

extensions/resource-deployment/src/ui/modelViewUtils.ts
4
4.4%

Totals

Change from base Build 259930564:
-0.06%

Covered Lines:
19188

Relevant Lines:
45021

💛  - Coveralls


Enter key on selected suggestion

Hello, how to let the user select a suggestion with the keyboard (down key) and select the suggestion if the user press the enter key ?

Hi @Daaaaad I think the default option are already providing you so no? See this codepen: http://codepen.io/Algolia/pen/OXgkxv

You're right @vvo, but my template contains a link and I would like the user to be redirected to this link when pressing the enter key... I guess I have to to custom code for that, right ?

Ok I see what you want, you can use that: https://github.com/algolia/autocomplete.js/issues/85#issuecomment-225701216
We may provide an easier way to do so soon, https://github.com/algolia/autocomplete.js/issues/85
Let me know

It works, perfect ;)

awesome!


Issues when run command freeswitch_exporter

I got this error when run command:
freeswitch_exporter config=/root/esl.yml
Traceback (most recent call last):
File "/usr/local/bin/freeswitch_exporter", line 7, in
from freeswitch_exporter.cli import main
File "/usr/local/lib/python2.7/dist-packages/freeswitch_exporter/cli.py", line 7, in
from freeswitch_exporter.http import start_http_server
File "/usr/local/lib/python2.7/dist-packages/freeswitch_exporter/http.py", line 59
response = Response(f"Module '{module}' not found in config")
^
SyntaxError: invalid syntax
Could you help me how to run command correctly. I'm new in python.
Thank you !

Hi, could you please post your @/root/esl.yml@ (without the password obviously).

Oh, wait. This is not a config issue. It seems you are using Python 2. prometheus-freeswitch-exporter is Python 3 only.

It worked when i installed python 3.8 from source :)
Thank you for your support.


[Storage] Bucket lock samples

Samples

[x]  Tested samples
[x]  Define a retention policy
[x]  Remove a retention policy if not locked
[x]  Lock a retention policy
[x]  Set temporary hold
[x]  Release temporary hold
[x]  Set event-based hold
[x]  Release event-based hold
[x]  Set default event-based hold
[x]  Release default event-based hold
[x]  View default event-based hold
[x]  View bucket retention policy metadata
[x]  View metadata for object
[x]  Updated README.md
[x]  Updated CLI
[x] enable feature for test projects.

Summoning @yoshi-ruby to review my PR. Anyone have a moment to take a look?

Summoning @GoogleCloudPlatform/yoshi-ruby to review my PR. Anyone have a moment to take a look?

@frankyn I'll review this first thing tomorrow morning!

thanks both!

Responded to feedback, thanks @quartzmo!

Thank you @quartzmo and @blowmage! Super appreciate it! Now I need to figure out what happened with Kokoro CI, looks like a permission issue :/

Kokoro might take a bit of time to get back to normal. So, if tests are passing locally, I would submit.

Thanks @tbpg, do you know if this is related to the new trampoline release?

Also, I'm blocked until the library is released to rubygems.

do you know if this is related to the new trampoline release?
I think so.

Thanks @tbpg. I will merge next week, hopefully it will be rolled back by then.

Thanks y'all. Merging now.


kubernetes.io/ingress-bandwidth doesn't seem to work reliably

I'm seeing unexpected behavior when using the kubernetes.io/ingress-bandwidth annotation on my pod. Here's my networking config:
/etc/cni/net.d/10-canal.conflist
{
  "name": "k8s-pod-network",
  "cniVersion": "0.3.1",
  "plugins": [
    {
      "type": "calico",
      "log_level": "info",
      "datastore_type": "kubernetes",
      "nodename": "<mynodehost>",
      "mtu": 1450,
      "ipam": {
          "type": "host-local",
          "ranges": [
              [
                  {
                      "subnet": "usePodCidr"
                  }
              ]
          ]
      },
      "policy": {
          "type": "k8s"
      },
      "kubernetes": {
          "kubeconfig": "/etc/cni/net.d/calico-kubeconfig"
      }
    },
    {
      "type": "portmap",
      "snat": true,
      "capabilities": {"portMappings": true}
    },
    {
      "type": "bandwidth",
      "capabilities": {"bandwidth": true}
    }
  ]
}

My pod spec:
apiVersion: v1
kind: Pod
metadata:
  name: curl-client-35m
  annotations:
    kubernetes.io/ingress-bandwidth: 35M
spec:
  securityContext:
    runAsNonRoot: true
  containers:
  - name: curl-client
    image: curlimages/curl:7.78.0
    command: ["sh", "-c", "curl -sSL -w 'Download speed: %{speed_download} bytes/sec\n'  https://a-large-file-i-can-download-from-my-pod -o /dev/null"]
    securityContext:
      allowPrivilegeEscalation: false
      capabilities:
        drop:
        - ALL
      seccompProfile:
        type: RuntimeDefault

But here's the output I'm getting (from kubectl logs pod/curl-client-35m):
[root@my-node ~]# k logs --tail=10 -f pod/curl-client-35m
Download speed: 10257544 bytes/sec -> 82.060352 Mbps
[root@my-node ~]# k logs --tail=10 -f pod/curl-client-35m
Download speed: 4171448 bytes/sec -> 33.371584 Mbps
[root@my-node ~]# k logs --tail=10 -f pod/curl-client-35m
Download speed: 4417035 bytes/sec -> 35.33628 Mbps
[root@my-node ~]# k logs --tail=10 -f pod/curl-client-35m
Download speed: 4808109 bytes/sec -> 38.464872 Mbps
[root@my-node ~]# k logs --tail=10 -f pod/curl-client-35m
Download speed: 5672415 bytes/sec -> 45.37932 Mbps
[root@my-node ~]# k logs --tail=10 -f pod/curl-client-35m
Download speed: 10938176 bytes/sec -> 87.505408 Mbps
[root@my-node ~]# k logs --tail=10 -f pod/curl-client-35m
Download speed: 5281018 bytes/sec -> 42.248144 Mbps

It looks like sometimes this works as expected, since the middle speeds are all roughly 35Mbps. But the first and last few are all faster than 35Mbps, and I'm not sure why.

Opened in wrong repo, closing in favor of https://github.com/projectcalico/canal/issues/142

This might be linked to the way bandwidth limiting is implemented in Kubernetes at the moment.
When setting bandwidth limits, there are two values that can be configured:

bandwidth
burst

"Burst" allows the pod to use more bandwidth for a short period of time (or rather, it specifies an amount of data that won't be subject to traffic shaping).
Unfortunately, when limiting bandwidth by setting annotations, it will configure the "bandwidth" part, but the "burst" will still be pretty high.
 You can see that at the beginning of the transfer, there is a big burst; then it settles below the configured limit; and there is a little spike at the end (which is probably just because the measuring window is so small, so the value isn't very accurate).
I don't know exactly what part of Kubernetes transforms the annotations into instructions for the bandwidth plugin (perhaps it's Kubelet?) but that's where the problem would be.


Upgrade tests to use JUnit5

All tests were switched from JUnit version 4 to 5 (aka junit jupiter) in order to address issues with time-intensive tests running into timeouts in certain configurations. JUnit 5 offers a feature to set timeouts programmatically, i.e. flexibly according to the respective config.
The following changes were made to account for the (rather large) changes introduced in JUnit 5
Parameterized Tests
The inheritance-based architecture for parameterization built around AbstractSolverTests and JUnit 4's Parameterized runner had to be changed since test parameterization in JUnit 5 is handled using method-level annotations, see JUnit 5 docs.
Test configs (i.e. parameter sets) for all classes previoulsy extending AbstractSolverTests are now supplied via the static factory methods in at.ac.tuwien.kr.alpha.solver.RegressionTestConfigProvider. In order to not have to put the @ParameterizedTest and @MethodSource annotations on every method parameterized using this mechanism, I introduced a new meta-annotation @RegressionTest that bundles the two and has the correct supplier method for parameters fixed. Note that there is a special form of this annotation, @AggregateRegressionTest for use in AggregateTests that supplies configs both with and without sorting-grid based aggregate encodings enabled. (Default for standard RegressionTests is to use sorting-grid only)
Other changes due to JUnit 5
These are a few smaller things I stumbled across during migration which I thought would be useful to document here:

@Test(expected = SomeThrowable.class) was replaced with the method call assertThrows(Throwable, Executable), where the Executable is any lambda without params or return value.
Timeouts are now set programmatically in a fashion similar to the mechanism for exceptions. I created TestUtils#runWithTimeout as a convenient shorthand.
@Ignore is now called @Disabled.
assertEquals and other assertSomethings have a new parameter order in the version with custom error message, e.g. the junit4 method call assertEquals("Not equal", 1, 2) would be assertEquals(1, 2, "Not equal") in junit5.
@Before has a new name, i.e. setup methods run before each test are now annotated with @BeforeEach (there is also @BeforeClass).

@AntoniusW thanks for the detailed review! I applied your suggestions and addressed the remaining issues now, please have another look and merge if ok.

Looks all good. There are two open points and after deciding how to handle them, you @madmike200590, can merge this PR. One point is unnecessary wb.close(); inside try-with-resources blocks, the other a method I thought we said to move to another class. Thank you again for this contribution!


[WIP] Add "as you type" title suggestions to search/all

What
Enhance search on search/all with suggestions from titles that match the keywords user is entering.
Uses Accessible Autocomplete engine.
Why
Hypothesis: We think that users don’t always know what they’re looking for, so if we provide autocomplete suggestions, they will refine their search less often.
Performance

Strategies

we make use of a debounce function, so that we only trigger the lookup once 300ms have elapsed between keystrokes

we cache last shown suggestions, so if users and more characters that result in a matching query or if they delete characters, we serve those instead of fetching from the API

Trello card
Search page examples to sanity check:

https://finder-front-add-autoco-jp2dlr.herokuapp.com/search/all

Thanks to @kevindew and @sihugh for helping with CSP and Docker yarn stuff.

@kr8n3r are we OK to close this?


Dark mode fixes

First 3 commits first color styling issues in user profiles and the two searchbox versions (pills and legacy), see screenshots below.

Last commit refactors of the selectors to be nested under one singular parent body.night-mode selector instead of specifying it for every individual selector.

I merged the first commit.
The second and third seem to have some mixing, and also I think the pills look for night mode is wrong (it's ~impossible to see the pill's boundaries):

Nice, merged, thanks @synicalsyntax!  The night-mode CSS file is definitely cleaner after this.
@shubham-padia FYI, since this changes the styling on the search pills.


Thoroughly  test cache package

Signed-off-by: Tobias Guggenmos<EMAIL_ADDRESS>

Helps with #26


Explanation of buttons (on hover?)

Currently, the buttons in the automations list aren't self-explanatory:

I would expect at least a description on mouse hover over the icons. I really don't know what clicking each button will do.

That's a good idea. I'll try to add something on my next round of UI improvements!

Take a look at the new release v0.6.1

Hello i would like to contribute to the description on the buttons on hover

Hi @ChrizDuma I've already implemented a hover over for this in the last release.
But take a look at https://github.com/asosnovsky/Shortumation/wiki/Development and https://github.com/asosnovsky/Shortumation/issues?q=is%3Aopen+is%3Aissue+label%3A"good+first+issue"

@ChrizDuma if you want this issue here is a good starter https://github.com/asosnovsky/Shortumation/issues/146
you basically need to add a validator and cast it's type here https://github.com/asosnovsky/Shortumation/tree/main/webapp/src/types
then you need a sun editor condition node here https://github.com/asosnovsky/Shortumation/tree/main/webapp/src/components/ConditionEditor
then add sun as an option here https://github.com/asosnovsky/Shortumation/blob/416d88b965db3e562c0b07f5be06adb584005148/webapp/src/utils/automations.ts#L56
I would do this, but I am trying to get a few other things implemented first XD
fyi, for visual testing create a storybook for sun condition here https://github.com/asosnovsky/Shortumation/blob/416d88b965db3e562c0b07f5be06adb584005148/webapp/src/components/ConditionEditor/ConditionNodeBase.stories.tsx

if i may ask before proceeding, what language did you write this code in
On Monday, August 29, 2022 at 07:56:48 PM GMT+3, Ari Sosnovsky<EMAIL_ADDRESS>wrote:

@ChrizDuma if you want this issue here is a good starter #146
you basically need to add a validator and cast it's type here https://github.com/asosnovsky/Shortumation/tree/main/webapp/src/types
then you need a sun editor condition node here https://github.com/asosnovsky/Shortumation/tree/main/webapp/src/components/ConditionEditor
then add sun as an option here https://github.com/asosnovsky/Shortumation/blob/416d88b965db3e562c0b07f5be06adb584005148/webapp/src/utils/automations.ts#L56
I would do this, but I am trying to get a few other things implemented first XD
fyi, for visual testing create a storybook for sun condition here https://github.com/asosnovsky/Shortumation/blob/416d88b965db3e562c0b07f5be06adb584005148/webapp/src/components/ConditionEditor/ConditionNodeBase.stories.tsx
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>

@ChrizDuma Typescirpt+React for the UI, python for the backend, please see https://github.com/asosnovsky/Shortumation/wiki/Development#prequesites


[Problem] PreprocessPESR.std_delly_vcf: []

Instructions
I am very interesting to know more about GATKSVPipelineBatch.wdl.
I just plan to test GATKSVPipelineBatch.wdl with HG00096.final.bam and HG0129.final.bam.
During the test, I find an unusual condition.
The generated directory structure of Module00c.call-PreprocessPESR.call-StandardizeVCFs is just like below.
├── execution
│   ├── docker_cid
│   ├── glob-31de091d1940ddb276ce7b6086a74725
│   │   └── cromwell_glob_control_file
│   ├── glob-31de091d1940ddb276ce7b6086a74725.list
│   ├── manta.HG00096_unsorted.vcf
│   ├── manta.HG00129_unsorted.vcf
│   ├── rc
│   ├── script
│   ├── script.background
│   ├── script.submit
│   ├── std_000.manta.HG00096.vcf.gz
│   ├── std_001.manta.HG00129.vcf.gz
│   ├── stderr
│   ├── stderr.background
│   ├── stdout
│   └── stdout.background
├── inputs
│   ├──<PHONE_NUMBER>
│   │   └── HG00129.manta.vcf.gz
│   ├── 745429519
│   │   └── contig.fai
│   └── -841028741
│       └── HG00096.manta.vcf.gz
└── tmp.92bdb17f

The related code is just like this.
  output {
    Array[File] std_vcf = glob("std_*.vcf.gz")
  }

Log output:
rocessPESR complete. Final Outputs:
  "PreprocessPESR.std_melt_vcf": null,
  "PreprocessPESR.std_wham_vcf": null,
  "PreprocessPESR.std_delly_vcf": [],
  "PreprocessPESR.std_manta_vcf": []

I used a previous version of the code for a test, only used one BAM file, did not encounter this problem.
Do you have any good suggestions for solving this problem?

Thank you very much.

What's the contents of glob-31de091d1940ddb276ce7b6086a74725.list?
It looks like the task did successfully standardize the manta vcfs that were given to it as input, but cromwell didn't pick them up as inputs.
Did the entire workflow report any other errors in the logs?

This is unfortunately a result of the way Cromwell handles glob statements differently on local backends, which we aren't supporting. If anyone needs a workaround, they can always rewrite the WDL to avoid glob, ie by scattering over the VCF array and processing them individually.

This is unfortunately a result of the way Cromwell handles glob statements differently on local backends, which we aren't supporting. If anyone needs a workaround, they can always rewrite the WDL to avoid glob, ie by scattering over the VCF array and processing them individually.


frozen stages for backbone

for the frozen stages defined in ResNet.py :
def _freeze_stages(self):
¦   if self.frozen_stages >= 0:
¦   ¦   self.norm1.eval()
¦   ¦   for m in [self.conv1, self.norm1]:
¦   ¦   ¦   for param in m.parameters():
¦   ¦   ¦   ¦   param.requires_grad = False

¦   for i in range(1, self.frozen_stages + 1):
¦   ¦   m = getattr(self, 'layer{}'.format(i))
¦   ¦   m.eval()
¦   ¦   for param in m.parameters():
¦   ¦   ¦   param.requires_grad = False

for the common settings in configs.py use frozen_stages=1,
so why usually freeze thr first stage?

This is a typical convention originating from Detectron, which should yield better performance.

roger that. thx


Modify the path parameter to be by-value instead of a const reference.

Moreover, since we're going to make a copy of this parameter at Line 38 anyway, I think a by-value parameter is better than const reference.

A by-value parameter along with changing line 38 to a move assignment, yes.

Originally posted by @CaseyCarter in https://github.com/Azure/azure-sdk-for-cpp/pull/2595#discussion_r671440673
https://github.com/Azure/azure-sdk-for-cpp/blob/d90d5c761e90d9ff986dd434a3bddba06c9881e9/sdk/keyvault/azure-security-keyvault-keys/src/key_client.cpp#L53-L58
cc @Jinming-Hu

@CaseyCarter I am assuming you mean:
const std::vector<std::string>&& defaultPath -> const std::vector<std::string> defaultPath
And:
request.Path = defaultPath; -> request.Path = std::move(defaultPath);

@CaseyCarter I am assuming you mean:
const std::vector<std::string>&& defaultPath -> const std::vector<std::string> defaultPath

Almost! We need to remove the const here for the move below to have a useful effect: attempting to move from a const object generally results in a copy, and we can avoid that copy since we use this vector only to initialize request.Path.

And:
request.Path = defaultPath; -> request.Path = std::move(defaultPath);


[AutoRelease] t2-azurestackhci-2022-11-25-04826(can only be merged by SDK owner)

https://github.com/Azure/sdk-release-request/issues/3462
Live test success
https://dev.azure.com/azure-sdk/internal/_build?definitionId=1851

issue link:https://github.com/Azure/sdk-release-request/issues/3462


Date Range Picker not working properly when try to edit directly on input box

Describe the bug
When I try to edit the dates in date range picker directly in the input box without using the range picker the start date and end date is not getting updated rather it is getting reseted to old values.
To Reproduce

Go to date range picker
Select a date range (ex: 02/11/2023 - 15/12/2023)
Now try to edit the date directly on the input box (example change the end date from 15/12/2023 to 18/12/2024)
You will see that the date will not get updated it will be reseted again to 15/12/2023.

Expected behavior
It should change to 18/12/2024.
The expected behaviour is that the date should change even when we directly change that in the input box without changing it through date range picker
Screenshots
1.

@lechu24  That's because you are passing selected props startDate
And your startDate is new Date which is current date.
Try something like this and make sure you are selecting both date when props change.
<DatePicker
        dateFormat={dateFormat}
        locale={chosenLocale}
        selected={startDate}
        startDate={startDate}
        endDate={endDate}
        onChange={(date) => setStartDate(date)}
      />

    <DatePicker
      dateFormat={dateFormat}
      locale={chosenLocale}
      selected={endDate}
      onChange={(date) => {
        console.log(date);
        setEndDate(date);
      }}
    />

@aziyatali that's not really an answer. Your snippet if for a separate solution: using two date pickers to make a range.
The issue is with using the selectsRange prop. When using that prop you cannot set or change the entire range using the input. If you type in dates it'll set the start date but it won't recognize any further input past that.

I have the same issue. Seems like the problem is parsing logic preventing onChange from running unless it's a valid single date. Probably around here
Screenshot with a console.log after typing in the input field.
(https://github.com/Hacker0x01/react-datepicker/blob/f735e60e97da8e5f4066276da9cf601e85e9eeed/src/index.tsx#L604)

Any ideas?

See also #3596 and #3906 which report the same issue.


Bump Dependencies #12536430351

Bumps dependencies for product-is. Link : https://github.com/wso2/product-is/actions/runs/12536430351

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


when preload is activated 'ride resume' doesn't work good

In an email conversation with H looking for a bug for the 'ride resume' function for the Karro2, the conclusion was as follows.
When 'preload' for the Ki2 app is activated, the 'ride resume' function doesn't work properly. I.e. when you stop for lunch and power-off the device and then after lunch you start the device again you don't get the possibility of 'ride resume'. You have to start a new ride.
After the ride you then have two fitfiles.
Can 'ride resume' in combination with 'preload' be fixed?

Thanks for reporting.
The preload is basically starting the ride app and then closing it, that action is likely making the ride app assume that you don't want to resume the previous ride and ignores it.
I can try to look into that, but I don't expect a fix from our side. The preload is already there as a workaround for the SDK data elements not showing, so from our end this is tricky to fix. If you are still in contact with H please ask them nicely to fix the original problem in the first place so that we don't need the preload 🙂

Ok great 🙂 Thanks for the update.

This will be fixed by the new version using Karoo Extensions.


Randomize first_name of Profiles created by FactoryGirl

This one-line PR reduces false positives when testing sorting methods by randomizing the names created by the Profile factory.
(Background: The bug which caused #374 had been in the system for quite some time, but wasn't discovered because the sorting tests passed when they were supposed to fail. The reason for that is the way participant names are created in the Profile factory: in order to make them different from one another, the string 'Karl' is concatenated with the Profile's id. That however means that in most cases sorting by name and sorting by id has the exact same result. Tests checking if participants were correctly sorted by name may pass even if the sorting method didn't do anything at all.)

Coverage remained the same at 100.0% when pulling 447700796b68cff8ff29bf4c923575e652f78a20 on 374_DebugSortTest into 027294343ab7875ab3b50a97674de822e65dd707 on dev.

Coverage remained the same at 100.0% when pulling b0ce326cd81c795641a3df65ec4b3208f97979ba on 374_DebugSortTest into f10f84a1b121f9cfea462d8d68ecba83c1dd0f49 on dev.

Oh, it's even a test I wrote myself. How awkward :smile:
I've changed it to use the full name, which is still unique. I also repeated the tests with a modified factory to double check, no other test relies on the first_name to be unique.

Coverage remained the same at 100.0% when pulling 40ecfc6b0e46e8988bc0bcf5ff9f6b73d3b19cdc on 374_DebugSortTest into f10f84a1b121f9cfea462d8d68ecba83c1dd0f49 on dev.


1.3.0 update - cannot find module

After updating from 1.2.0 to 1.3.0 I am getting the following error:
Error: Cannot find module '/Users/chris/app/node_modules/extract-text-webpack-plugin/loader.js?{"omit":1,"extract":true,"remove":true}!style!css!postcss!resolve-url!sass?sourceMap!./lib/bootstrap.styles.loader.js!./no-op.js'
    at Function.Module._resolveFilename (module.js:325:15)
    at Function.Module._load (module.js:276:25)
    at Module.require (module.js:353:17)
    at require (internal/module.js:12:17)
    at Object.<anonymous> (webpack:///./no-op.js":1:1)
    at __webpack_require__ (webpack:///webpack/bootstrap ffe52b1d2cc71d90e522:19:1)
    at Object.module.exports.module.exports.forcePointer (webpack:///./~/bootstrap-loader/no-op.js:1:1)
    at __webpack_require__ (webpack:///webpack/bootstrap ffe52b1d2cc71d90e522:19:1)
    at Object.<anonymous> (webpack:///./~/bootstrap-loader/extractStyles.js:1:1)
    at __webpack_require__ (webpack:///webpack/bootstrap ffe52b1d2cc71d90e522:19:1)

Ideas? Thank you!

Hey, seems like the latest absolute path support breaks this case. I published 1.3.1, it is now optional and hidden behind a flag.
Let me know if it solves your case

1.3.1 works now. Thank you for the quick fix!


Reload configurations

How can I reload nginx to accept an updated configuration?

You could just restart the container.
docker restart <Container ID>

ha yeah sorry I realised this about 2 secs after asking the question. duh.


[RayJob] Enable job log streaming by setting PYTHONUNBUFFERED in job container

Why are these changes needed?
Previously, kubectl get logs job/my-rayjob and kubectl logs my-rayjob-xxxxx (getting the logs from the pod directly) would be empty until the job finished.
This PR sets PYTHONUNBUFFERED in the job container so that the logs can be viewed while the job is still running.
This improves the log viewing experience to match that of ordinary Kubernetes Jobs.
This PR adds unit tests, but adding an end-to-end test (e.g. submit a job that prints a log and then sleeps forever, and check that we can read the log) will require some followup work. The main issue is the following:

The RayJob sample YAML test framework waits for a job to "converge" (actually, to be SUCCEEDED) before running any Rules.  But for this test, we just want the job to start before running the Rule.

Related issue number

Closes #1374
Checks

[ ] I've made sure the tests are passing.
Testing Strategy

[x] Unit tests
[x] Manual tests
[ ] This PR is not tested :(

@kevin85421 Comments have been addressed, please take another look!

@kevin85421 updated the PR to not check if the env var was already set, as discussed offline, because we don't anticipate any use case for PYTHONUNBUFFERED=0.

Maybe we can use submitterTemplate.Spec.Containers[common.RayContainerIndex] to replace submitterTemplate.Spec.Containers[0]. You can either update it in the PR or open an issue as a good first issue.

Open a good first issue #1397.


Module Identification

If you install your fork with original module side by side, then both look almost identical in Xposed app, due to using the same label and icon. This patch address this by:

Changing the label to read "App Locale 2"
Changing the app icon color

Merge approved, thanks.


Incorrect password issue

@arunkumar413  I am getting this error while starting the backend! Any solution in mind ?
Originally posted by @MrCyberlord in https://github.com/arunkumar413/task-force/issues/2#issuecomment-2437193207

@MrCyberlord You need to point the configuration in the file server/DBConfig.js to your local machine postgres DB username and password.

Got it! Your solution has solved the issue.

@arunkumar413  your solution has solved the issue.

Thanks


net/http: never cancel r.Context() after Hijack

Fixes #32314

@odeke-em feel free to take over, I don't have the time to respond to the reviews

Hey @nhyor, I think you copied the test that I provided verbatim, it won't
have the trybot failures and also matches the style for code review :) Just
that change and it'll be perhaps good to go.
On Tue, Jun 11, 2019 at 6:42 AM Anmol Sethi<EMAIL_ADDRESS>wrote:

Closed #32315 https://github.com/golang/go/pull/32315.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/golang/go/pull/32315?email_source=notifications&email_token=ABFL3VZ656MJNHZNADRUMX3PZ6TVDA5CNFSM4HQYGVV2YY3PNVWWK3TUL52HS4DFWZEXG43VMVCXMZLOORHG65DJMZUWGYLUNFXW5KTDN5WW2ZLOORPWSZGOR5GO65Q#event-2404183926,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABFL3V27XU3UN2DDEFQ3CSDPZ6TVDANCNFSM4HQYGVVQ
.

*I think if you copy the test that I provided verbatim
Sorry was typing from my phone
On Tue, Jun 11, 2019 at 9:44 AM Emmanuel Odeke<EMAIL_ADDRESS>wrote:

Hey @nhyor, I think you copied the test that I provided verbatim, it won't
have the trybot failures and also matches the style for code review :) Just
that change and it'll be perhaps good to go.
On Tue, Jun 11, 2019 at 6:42 AM Anmol Sethi<EMAIL_ADDRESS>wrote:

Closed #32315 https://github.com/golang/go/pull/32315.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/golang/go/pull/32315?email_source=notifications&email_token=ABFL3VZ656MJNHZNADRUMX3PZ6TVDA5CNFSM4HQYGVV2YY3PNVWWK3TUL52HS4DFWZEXG43VMVCXMZLOORHG65DJMZUWGYLUNFXW5KTDN5WW2ZLOORPWSZGOR5GO65Q#event-2404183926,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABFL3V27XU3UN2DDEFQ3CSDPZ6TVDANCNFSM4HQYGVVQ
.


maven-compiler-plugin: disable forking + remove duplicated config

jboss-parent already defines source, target, warnings and deprecation
fork config removed == forking disabled (default is false)
meminitial and memmax are only needed when forking
with forking disabled, the performance gain is about 60%
for the compile-only builds

+1


Few queries

How do we start the server?
How to write 2 native C clients (sender/receiver) using the API to create a chat room and interchange messages between them?

Thanks for asking. We will update docs soon.


WSL2 rocminfo Issue

I've tried to install rocm according to the installation guide, to use my RX580 GPU with TensorFlow.
When I came to step 8, I got the following error:
/opt/rocm/bin/rocminfo

ROCk module is NOT loaded, possibly no GPU devices
Unable to open /dev/kfd read-write: No such file or directory
Failed to get user name to check for video group membership
hsa api call failure at: /src/rocminfo/rocminfo.cc:1142
Call returned HSA_STATUS_ERROR_OUT_OF_RESOURCES: The runtime failed to allocate the necessary resources. This error may also occur when the core runtime library needs to spawn threads or create internal OS-specific events.

My WinVer: 10.0.21292.0
WLS2 - Ubuntu: 20.04
Is there any way to use my RX580 GPU with WSL2 or should I do a separate installation for Ubuntu?

@namcho ,
As per the installation guide, the result of experimentation of  ROCm on Customize kernels over Windows is undefined. Kindly refer to the table given here.
We request you to kindly use Supported Linux Host for ROCm.

@namcho ,
As per the installation guide, the result of experimentation of  ROCm on Customize kernels over Windows is undefined. Kindly refer to the table given here.
We request you to kindly use Supported Linux Host for ROCm.


poezio: add setuptools as propagatedBuildInput

This unbreaks the otherwise-broken package (setuptools is needed for importing pkg_resources).
Presumably this change should also be backported to the 20.03 release--I could add a PR to that branch as well, but figure it's best to sort out a merge to master first.
Motivation for this change
Unbreaking broken package.
Things done

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

[x] NixOS
[ ] macOS
[ ] other Linux distributions

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

Hi.
Sorry I missed your PR earlier. I had the exact same fix integrated into master in 0fa981bfbec38e620f92d6b812f3edf0d7401be7.
I just backported it to 20.03 in b6cda510fae9d88666c3daaf0de23d8258f0aaab
Thanks for the MR.


[ci] Use a token to upload codecov reports

Fixes #68

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

Project coverage is 51.60%. Comparing base (bb9bb85) to head (27fca7e).
Report is 10 commits behind head on main.

:exclamation: Current head 27fca7e differs from pull request most recent head 5807e07. Consider uploading reports for the commit 5807e07 to get more accurate results

Additional details and impacted files

@@            Coverage Diff             @@
##             main      #70      +/-   ##
==========================================
+ Coverage   49.14%   51.60%   +2.45%
==========================================
  Files          15       15
  Lines         643      593      -50
==========================================
- Hits          316      306      -10
+ Misses        327      287      -40

see 12 files with indirect coverage changes
see 12 files with indirect coverage changes


Add 3.2.2-pshopify10

This PR adds v3.2.2-pshopify10 that has everything in v3.2.2-pshopify9-experimental2 and just one more thing:

https://github.com/Shopify/ruby/commit/4726799892ae6447557b20a4eb1be4d2f5c9c479: Implement GC.free_all_empty_pages

I know this is for temporary experiments. But we've had a logging problem on Core ever since Core switched the Ruby version from 3.2 (pshopify8) to 473f6f9e90933ba9c91378ad082447fd3e1c2c92 (pshopify8-experimental1). I don't believe that this could break logging, but I'd still like to double check if switching it back to a pshopify fixes the problem before wasting more time on this. Let me just include this in pshopify for now.

https://github.com/Shopify/ruby/commit/41c25c40a3e72ac797fab6ac094099121f27ce78: YJIT: Compile exception handlers
https://github.com/Shopify/ruby/commit/aaa1e3e57ea7437f703b0805fb6fd3599fa6bb2d: YJIT: Introduce RubyVM::YJIT.stats_string

I highly doubt it's the cause.


Nested Maker Transfer Events

Fix bug where marker transfer events can have multiple events listed in the array of attributes.

Split out the event stream so that the movement stream can be reran in test and prod without NYCB changing env vars. Also seems beneficial to have them separate anyway so they can be progressed and reset separately.

Here's the original record in question and how it made it into my local database when I ran the stream against prod.
https://explorer.provenance.io/tx/4FDDFF7A3877D2B6ABA3413611FFBB34C8C53F4EBC3099966CB03FC17050C3C6
{"type":"provenance.marker.v1.EventMarkerTransfer","attributes":[{"key":"amount","value":"\"9829400\""},{"key":"denom","value":"\"usdf.c\""},{"key":"administrator","value":"\"pb1l00u2u8vzd6wvxxtsn0kel3gn870g39v4vx2fn\""},{"key":"to_address","value":"\"pb1scq2uxs24afwfkqfghuuewf82za7ugth6u564e\""},{"key":"from_address","value":"\"pb1l00u2u8vzd6wvxxtsn0kel3gn870g39v4vx2fn\""},{"key":"administrator","value":"\"pb1l00u2u8vzd6wvxxtsn0kel3gn870g39v4vx2fn\""},{"key":"to_address","value":"\"pb1afmnu23g0fht9rdmxyjxrd06uqsr4vx62tfaqj\""},{"key":"from_address","value":"\"pb1l00u2u8vzd6wvxxtsn0kel3gn870g39v4vx2fn\""},{"key":"amount","value":"\"5900\""},{"key":"denom","value":"\"figure-technologies-inc.cs.stock\""},{"key":"amount","value":"\"200600\""},{"key":"denom","value":"\"usdf.c\""},{"key":"administrator","value":"\"pb1l00u2u8vzd6wvxxtsn0kel3gn870g39v4vx2fn\""},{"key":"to_address","value":"\"pb1evdjjcagllgyf23pzf7e2gkvfuakegsnv49yhj\""},{"key":"from_address","value":"\"pb1l00u2u8vzd6wvxxtsn0kel3gn870g39v4vx2fn\""}]}

digital_currency_consortium=# select * from coin_movement where txid_v2 ilike '4FDDFF7A3877D2B6ABA3413611FFBB34C8C53F4EBC3099966CB03FC17050C3C6%';
-[ RECORD 1 ]-------+-------------------------------------------------------------------
txid                |
from_addr           | pb1l00u2u8vzd6wvxxtsn0kel3gn870g39v4vx2fn
from_addr_bank_uuid |
to_addr             | pb1scq2uxs24afwfkqfghuuewf82za7ugth6u564e
to_addr_bank_uuid   | 9f4b267c-f8fd-9065-58ed-a2e70f9045a5
block_height        | 2991843
block_time          | 2021-11-10 16:50:29.855387+00
amount              | 9829400
denom               | usdf.c
type                | TRANSFER
created             | 2021-11-11 20:30:45.182553+00
txid_v2             | 4FDDFF7A3877D2B6ABA3413611FFBB34C8C53F4EBC3099966CB03FC17050C3C6-0
-[ RECORD 2 ]-------+-------------------------------------------------------------------
txid                |
from_addr           | pb1l00u2u8vzd6wvxxtsn0kel3gn870g39v4vx2fn
from_addr_bank_uuid |
to_addr             | pb1evdjjcagllgyf23pzf7e2gkvfuakegsnv49yhj
to_addr_bank_uuid   | a9659f0d-9141-3b34-9118-2f6f4d2e4c3e
block_height        | 2991843
block_time          | 2021-11-10 16:50:29.855387+00
amount              | 200600
denom               | usdf.c
type                | TRANSFER
created             | 2021-11-11 20:30:45.186462+00
txid_v2             | 4FDDFF7A3877D2B6ABA3413611FFBB34C8C53F4EBC3099966CB03FC17050C3C6-1


Instead of an attribute list like "PREF;WORK;FAX", the OSX contacts a…

…pp expects a list like that: "type=PREF;type=WORK;type=FAX". To be compatible to this format, I added a little bit of code to transform it like that.
According to the specs from http://tools.ietf.org/html/rfc6350#section-6.4.1, this is also closer to the default implementation. An example there looks like<EMAIL_ADDRESS>Can you confirm the vcard still works on other clients like MS Outlook on Windows or some other tools?
I also adjusted the tests.

Why is this on hold?

@mgrundkoetter May i ask why you look at the vcard version 4 specs while making a PR for the master branch?
Please take a look at the 2.0.0-dev branch which uses the vcard 4 specs.

Related to #205.


Revert "Honda Pilot tuning / lane centering fixes (#408)"

This reverts commit fdbf1c5938b70a9fa389cb6a117f123b6cbf5a2e.

seems to work well on 2018 and 2019 model, but not on 2017. See comment on #408


Draft: Allow custom results.json

Currently, users can let their tests create their own results.yaml. Looking at the code, I don't see anything that would fundamentally require it to be yaml-only, though I haven't familiarized myself with the inner-workings of tmt yet, so feel free to close this PR if it's stupid :smiling_face_with_tear:
Rationale:
When running tests written in Python, generating yaml files without creating custom yaml parser means importing external package, which means adding extra dependency. This is especially problematic for packages like pyYAML, ruamel, etc, as they are not pure-python and pre-built wheels are not available for all architectures (e.g. ppc64le, s390x).
with json, users can just use (built-in) json.dump(). Example:
import json

result = []

if some_setup_step():
    result.append({'name': '/some_setup', 'result': 'pass''})

#...test...

with open(f"{TMT_TEST_DATA}/results.json", "w") as f:
    json.dump(result, f)

Feature LGTM but should be documented.
We have short checklist https://tmt.readthedocs.io/en/stable/contribute.html#pull-requests, most of it is already done.

Apologies, I'm a bit new here. I've added something in the docs, but not sure what else from the checklist is applicable or how to check it :)

Thank you @martinhoyer . The doc change is enough IMO.
/packit test

/packit test

/packit test


Add a Gitter chat badge to README.md

envato/rack-ecg now has a Chat Room on Gitter
@madlep has just created a chat room. You can visit it here: https://gitter.im/envato/rack-ecg.
This pull-request adds this badge to your README.md:

If my aim is a little off, please let me know.
Happy chatting.
PS: Click here if you would prefer not to receive automatic pull-requests from Gitter in future.

added manually

http://www.badgerbadgerbadger.com/


feat: computed property support expression

What kind of change does this PR introduce? (check at least one)

[ ] Bugfix
[x] Feature
[ ] Code style update
[ ] Refactor
[ ] Build-related changes
[ ] Other, please describe:

Does this PR introduce a breaking change? (check one)

[ ] Yes
[x] No

If yes, please describe the impact and migration path for existing applications:
The PR fulfills these requirements:

[x] It's submitted to the dev branch for v2.x (or to a previous version branch), not the master branch
[ ] When resolving a specific issue, it's referenced in the PR's title (e.g. fix #xxx[,#xxx], where "xxx" is the issue number)
[x] All tests are passing: https://github.com/vuejs/vue/blob/dev/.github/CONTRIBUTING.md#development-setup
[x] New/updated tests are included

Other information:
Using computed property like lodash.get, so we don't need to care about the data whether is null and prevent runtime error.

I don't really see value in adding support for strings in getters, same can be achieved today already using Optional Chaining like this:
  computed: {
    d: data => data.a?.b?.c?.d,
  }

@nickmessing  thanks, i close this PR


Entries: Add build system as field

And move "uses CMake" there for example.

done in 6070bd53604c1702094ef1e69b2fd10f2c570aae
field in the "## Build" section


Echo

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
-0.001%

Covered Lines:
1094

Relevant Lines:
1212

💛  - Coveralls


Is this project stoped

just asking.
trying to find way to change my old raspberry into pineapple

Hi,
For now, yes, sorry

Thank you for reply, I'll try another way
Nemanja Nedeljković<EMAIL_ADDRESS>于2019年11月13日周三 上午11:01写道：

Hi,
For now, yes, sorry
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/nemanjan00/wifi-raspberry/issues/1?email_source=notifications&email_token=ACAEFHFFUZ6A2HE4A354AYTQTNU2DA5CNFSM4JMOXHC2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOED4WUKQ#issuecomment-553216554,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACAEFHCL354AYLM4PYP3DKLQTNU2DANCNFSM4JMOXHCQ
.

--
Mail From Mars.


add java resources in persian language

What does this PR do?
Add resource(s)
For resources
Description
Why is this valuable (or not)?
It helps to improve knowledge in Java
How do we know it's really free?
For book lists, is it a book? For course lists, is it a course? etc.
it is a course
Checklist:

[x] Read our contributing guidelines.
[x] Search for duplicates.
[x] Include author(s) and platform where appropriate.
[x] Put lists in alphabetical order, correct spacing.
[x] Add needed indications (PDF, access notes, under construction).
[x] Used an informative name for this pull request.

Follow-up

Check the status of GitHub Actions and resolve any reported warnings!

what's the problem in my commit?

The message from the linter is
 /home/runner/work/free-programming-books/free-programming-books/books/free-programming-books-fa_IR.md   41:1-4[5](https://github.com/EbookFoundation/free-programming-books/actions/runs/4796075037/jobs/8531413361?pr=9353#step:5:6):85  warning  Alphabetical ordering: swap l.42 and l.41  alphabetize-lists  remark-lint
I don't know how the linter wants Farsi to be ordered but just do what it asks.


Digimon World: Next Order [PCSH00261]

Game summary

Game name: Digimon World: Next Order
Game serial: PCSH00261
Game version: 1.02

Vita3K summary

Version: v0.1
Build number: 1075
Commit hash: https://github.com/vita3k/vita3k/commit/6142543

Test environment summary

Tested by: Zangetsu38
OS: Windows 10
CPU: AMD Ryzen 2700X
GPU: NVIDIA GTX 1080 Ti
RAM: 16 GB DDR4 3600 mhz

Issues

stop boot on try load mono-vita module
Unicorn error:

Unicorn error 0x7 at: start PC: 0x835022E0 error PC 0x84B9CBA0 LR: 0x9E8F

Log
vita3k.log

can u fix this game ?

can u fix this game ?

game is unbity engine, so is same state for all unity game

awww i really wanted compatibility for this ç.ç

Please make this game playable.. i really want to play this game.. if you can lah

Please make this game playable.. i really want to play this game.. if you can lah

it is currently ingame on my wip pr

Please make this game playable.. i really want to play this game.. if you can lah

it is currently ingame on my wip pr

Very sorry but the app is not responding when i play the game

And its force close

did newest vita3k (8 hours ago) can play this game now?
didn the latest version of this digimon is v1.05? (including 12 other digimon)
i will try boot it soon.

is this game ever going to be compatible?

did it changed or still boot only ?

Why i can't instal itTT


Remove redundant implementations of Capsule

After implementing ZeroMQ in capsule, we figured out that Django server is not getting used at all and since all communications between other modules and capsule is being handled by ZeroMQ,  we don't really need Django or Flask server.
There are 3 capsule implementations using different frameworks (ZeroMQ, Django and Flask). So to remove the confusion, we should remove Django and Flask implementation of capsule and only keep ZeroMQ implementation of Capsule.

Agree !!

I think we should let it remain since it provides the developers with the choice to go ahead with their choice of framework and help us expand our reach as well in different communities. So we could let those remain as well.

I don't think we are ever going to use these implementations, so keeping these doesn't make any sense.

hmm @iamtrask what do u say ? Shall we go ahead.

We definitely shouldn't leave all 3 in their current state, where we have to maintain identical functionality across all 3. If anyone wants to build the abstraction necessary to make it so that they all consume the same code......
...yeah we should probably go ahead.... ZeroMQ is going to be loads faster in the long run.


data-table fixed header and group header cannot work fine together

TuSimple/naive-ui version (版本)
2.19.7
Vue version (Vue 版本)
3.2.20
Browser and its version (浏览器及其版本)
Chrome 94.0.4606.54 (Official Build) (64-bit)
System and its version (系统及其版本)
Ubuntu 94.0.4606.54 (Official Build) (64-bit)
Node version (Node 版本)
Reappearance link (重现链接)
https://codesandbox.io/s/spring-smoke-8zjkl
Reappearance steps (重现步骤)

create group header with single header
set group header and single header all fixed left

Expected results (期望的结果)
the header set fixed: left should all stick in the left
Actual results (实际的结果)
scroll to right, the fixed left header not working properly

Remarks (补充说明)

@LYErin it's a bit more complex than you think. ref 37d7fc981b21d


wrong language on new repo

My repo https://github.com/depazsierra/zoiperExample is only set to "C" and the mainly language of the project is swift & objective - c

I'm guessing this is because of these two huge files. You can tell Linguist to ignore them using Linguist overrides.

👋 @depazsierra, I recommend @pchaigno's suggestion as a resolution here.

Thanks @pchaigno / @Caged !


logging output not showing on console when calling DSPy modules

I generally set up my logging following this pattern:
import logging
logging.basicConfig(format='%(levelname)s:%(message)s', level=logging.INFO)

then in my main ...
logger = logging.getLogger(__name__)
# here debug parameter is sent in through CLI
logger.setLevel(logging.DEBUG if args.debug else logging.INFO)

I would then make calls to logger.info(...) or logger.debug(...) within the code (passing in a reference to logger to functions called from main). Expectation is to see INFO outputs on console always and DEBUG when debug == True. This works with non-DSPy code (LCEL) but not when calling DSPy code, i.e. when calling DSPy code I don't see any console output and I have to replace them with print calls.
Is there a recommended strategy similar to the above that I could use for logging to work with DSPy code?

cc @KCaverly

We use structlog on the backend for dspy logging, and should have fully standard library compatible logs. We do set the basicConfig for logging ourselves (this makes it a bit easier as people are frequently looking for dspy output in notebooks), but I imagine this may be a conflict with your existing setup.
We do have a logger enabled for dspy, we can access like so:
dspy.logger.info("this is a logging info message")
dspy.logger.debug("this is a logging debug message")

This avoids us using name boilerplate everywhere, and opens up for us to track additional json data etc.

A best practice for libraries is to never touch the base logging (Root loggers or call basicConfig). This is because once logging is set, the logging.basicConfig does nothing unless force=True, which a lot of people might not have set.
I'd strongly recommend removing the call to logging.basicConfig from the library as it will cause people headaches going forward, trying to debug why their loggers have unexpected behaviours.

Makes sense to me, I think its an artifact of this being more of a demo repo than a framework. Ill put over a pr today to remove it.

Thank you, I will try dspy.logger.* as @KCaverly suggested and remove the logging.basicConfig from my calling code as @DavidGeorge528 suggested.
I ran some quick-and-dirty validation using the REPL and I notice that:

import dspy results in creation of openai_usage.log, azure_openai_usage.log and assertion.log.
dspy.settings.configure(log_openai_usage=False) results in no more logs being written to openai_usage.log following the command.
dspy.settings.configure(log_azure_openai_usage=False) works but logs continue being written to azure_openai_usage.log
I have to set the level in my code as dspy.logger.level = logging.INFO (or logging.DEBUG) then calls to dspy.logger.info / spy.logger.debug get written out to assertion.log

Ideally, though I think the openai and azure logs should be created when I call dspy.settings.configure(log_openai_usage=True) and spy.settings.configure(log_azure_openai_usage=True).
It would also be nice to have a setting where the log messages show up on console rather than assertion.log.
But I can live with the state as it is now. Thank you for all the help and guidance. @KCaverly your comment indicates that maybe you want to do something on this, but if not, please feel free to close (or let me know, I can close).

These extra log files are from an old way of managing logging, we will work through to remove them and keep it standardized.

@okhat a bit of background on why the 'basicConfig' calls were removed, and thus dspy no longer by default prints logs to console.

adding that calls to https://api.openai.com (i.e. plain vanilla openai not azure openai) end up getting logged to azure_openai_usage.log with the default settings. seems related.
fake_useragent not found, using default user agent.To get a realistic header for requests, `pip install fake_useragent`.

Successfully loaded faiss.


Add nix flake support

Allows idris2-lsp to be installed using nix (see README)

@ShinKage @michaelmesser Do you use nix? If yes, it would be nice to check this PR. If not, we can merge as it is.

Ok, fortunately the second point is easy to address; for example:
diff --git a/Idris2 b/Idris2
index dc1b538..ea093ff 160000
--- a/Idris2
+++ b/Idris2
@@ -1 +1 @@
-Subproject commit dc1b5387b8d54132a6707c4c62d7777748271db4
+Subproject commit ea093ffaed7b003d28e35cb3b1114ac4bce4603d
diff --git a/flake.nix b/flake.nix
index 207f608..366fb48 100644
--- a/flake.nix
+++ b/flake.nix
@@ -33,6 +33,11 @@
           echo "export idrisVersion : ((Nat,Nat,Nat), String); idrisVersion = $VERSION" >> src/IdrisPaths.idr
           echo 'export yprefix : String; yprefix="~/.idris2"' >> src/IdrisPaths.idr
           '';
+          # Perhaps counter-intuitively, we don't want to use the buildPhase provided by
+          # `buildIdris` becuase it will do a default build which will build Idris2, not
+          # the Idris2 API package. Fortunately, we can simply skip that buildPhase because
+          # `idris2 --install idris2-api` will build it as-needed in order to install it.
+          dontBuild = true;
         };

I did realize as I was switching back and forth between things that this PR predates the split of LSP-lib off into its own package. Definitely apologize this sat for so long. I hope that throwing the new package into the mix is no harder than the others built by this flake currently.

I don't mind merging this PR, but I don't use nix. Are there any other nix users who can comment on these changes? If not, we can just merge this before the new year :)

@andorp id rather not merge this as is. I had a few requests above including one that fixes this derivation for use on x86_64-darwin. My most recent realization was that this also won’t work due to the introduction of the LSP-lib which isn’t yet accounted for in the changes here.

Ok, just let me know when you think this is done.

I've opened a new PR that adds a flake here.
I made the fixes I proposed in this PR's comments, upstreamed the API package as suggested in a comment of the flake for this PR, and I built the relatively new LSP-lib dependency into the new flake.

This PR was the inspiration for https://github.com/idris-community/idris2-lsp/pull/210 but it has now been superseded by that PR.


hapmap 3 reference information

Hi Bjarni,
It is possible to get some information on the HapMap 3 reference data used for the --only-hm3 flag? I'm not using rs as SNPs IDs, so I would appreciate information on the data source for the HapMap 3 reference to ensure that I can merge my data properly.
Thank you :)
Best,
Agnete

Hi Agnete,
It’s described here:  https://www.broadinstitute.org/medical-and-population-genetics/hapmap-3 https://www.broadinstitute.org/medical-and-population-genetics/hapmap-3
More specifically, I used the same file as LDscore (Bulik-Sullivan et al., Nat Genet 2015) uses, i.e. https://data.broadinstitute.org/alkesgroup/LDSCORE/hapmap3_snps.tgz https://data.broadinstitute.org/alkesgroup/LDSCORE/hapmap3_snps.tgz
Best,
Bjarni

On 28 Aug 2019, at 13.09, agnetelundgaard<EMAIL_ADDRESS>wrote:
Hi Bjarni,
It is possible to get some information on the HapMap 3 reference data used for the --only-hm3 flag? I'm not using rs as SNPs IDs, so I would appreciate information on the data source for the HapMap 3 reference to ensure that I can merge my data properly.
Thank you :)
Best,
Agnete
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/bvilhjal/ldpred/issues/102?email_source=notifications&email_token=AAFDXI6CO3W7HOFZ4VK35ULQGZMH3A5CNFSM4IRDFAKKYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HH4RP3Q, or mute the thread https://github.com/notifications/unsubscribe-auth/AAFDXI7OFWPBMXILP6LGT3DQGZMH3ANCNFSM4IRDFAKA.

Thank you, Bjarni - this is very helpful!
Best,
Agnete


data management plan?

@mpinsky I've started a data management plan.  What do you think?
https://github.com/pinskylab/policies/blob/master/data-management.md

Malin updated


replace same tag default outbound handler

example config:
{
  "outbounds": [
    {
      "protocol": "vmess",
      "tag": "tagA"
    },
    {
      "protocol": "vmess",
      "tag": "tagA"
    }
  ]
}

first tagA outbound will be default outbound, the second outbound will replace first one in taggedHandler .
it looks weird.  like a orphan outbound.

#1435

I think v2ray-core should refuse to accept duplicate tags, or at least show a warning.

accept duplicate tag handler will make 'update handler config by tag / implement command.OutboundOperation'  easier.

Codecov Report

Merging #1437 (e80a349) into master (b960d3f) will decrease coverage by 0.20%.
The diff coverage is 50.00%.

@@            Coverage Diff             @@
##           master    #1437      +/-   ##
==========================================
- Coverage   39.59%   39.38%   -0.21%
==========================================
  Files         598      590       -8
  Lines       35064    34956     -108
==========================================
- Hits        13882    13768     -114
  Misses      19657    19657
- Partials     1525     1531       +6

Impacted Files
Coverage Δ

app/proxyman/outbound/outbound.go
48.23% <50.00%> (+0.08%)
:arrow_up:

app/router/command/errors.generated.go
0.00% <0.00%> (-100.00%)
:arrow_down:

transport/internet/udp/dispatcher.go
64.76% <0.00%> (-12.39%)
:arrow_down:

common/buf/reader.go
82.35% <0.00%> (-5.89%)
:arrow_down:

testing/servers/tcp/tcp.go
81.81% <0.00%> (-5.46%)
:arrow_down:

testing/scenarios/common.go
73.07% <0.00%> (-4.62%)
:arrow_down:

transport/internet/udp/hub.go
73.61% <0.00%> (-4.17%)
:arrow_down:

transport/internet/kcp/dialer.go
80.39% <0.00%> (-3.93%)
:arrow_down:

transport/internet/system_listener.go
48.07% <0.00%> (-3.85%)
:arrow_down:

proxy/freedom/freedom.go
42.45% <0.00%> (-3.78%)
:arrow_down:

... and 16 more

Continue to review full report at Codecov.

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

Thanks for your works.


[FEATURE REQ] Improvements for console output when scanning directories with modules

Please describe the feature or suggestion.
Currently, when using the CLI to scan a directory that contains templates with modules, the console output can be improved in a couple ways:

Remove output for templates scanned that have no further results to output. This can happen if all of a template's results are in modules/other templates and has no direct results of its own. All of the template's results would be output under other templates that actually contain the results, not a reference to them.
Properly reflect the number of files scanned when scanning a directory with modules. The tool will potentially scan more templates than are in a directory if one of the scanned templates reference a template outside of that directory. Bicep modules use relative paths for modules so "../../../../someTemplate.json" is a valid path for a module.

Additional context
No response

About the first item, we should also evaluate the value of knowing how many rules passed, if we want to keep that information then we would still need to mention each file


TS SDK development should use eval with exec-path

https://googlecontainertools.github.io/kpt/guides/producer/functions/ts/quickstart/
Instead of using source and sink:
kpt fn source $CONFIGS |
  node dist/label_namespace_run.js -d label_name=color -d label_value=orange |
  kpt fn sink $CONFIGS

It should be possible to --exec-path
Option 1:  Make *_run.js files executable
https://kpt.dev/sdk/ts-guide?id=debug-and-test-the-function
Option 2: Make exec-path take additional arguments
Something like:
kpt fn eval $CONFIGS --exec-path node dist/label_namespace_run.js -- label_name=color label_value=orange

For some prior art, see -exec flag in Linux find command.
Option 2 is a more general solution applicable to other toolchains, so it is preferred.

I found that there are no direct way to support multiple values to a single flag with cobra. cobra will only accept one value for a flag. So if we want to support exec path plus arguments, we have 2 options:

Use quotes around the commands: kpt fn eval $CONFIGS --exec "node dist/label_namespace_run.js" -- label_name=color label_value=orange
Use StringSlice: kpt fn eval $CONFIGS --exec node,dist/label_namespace_run.js -- label_name=color label_value=orange

Option 1 is better. Should make sure it's documented in reference doc.