Lights - GameObject linkage

Now Lights use the transform components from gameObjects to draw themselves and its aim.
Added two functions to the ComponentTransform class to get the Forward vector from the global and local matrices

Source/DataModels/3DModels/Mesh.cpp when you do things like p1.position = float4 it will be best to set those variables through seter. I doubt if things like pointLights.size() should be .GetSize() idk.
Source/DataModels/Components/ComponentSpotLight.cpp in ComponentSpotLight idk if it's github but can you put color variable under radius variable?
Source/DataModels/Components/ComponentSpotLight.h same here.

I answer point by point in the same order respectively:

pl is an struct, not an class object that's why i initiate them like this. Furthermore, they are here just to test if lights bind correctly with the shaders, on the final engine this values will be initialized by using the already existing getters from the different componentLight classes. On the next merge, this class will be removed so any changes done here are just for merging purposes later during development of the engine.
Yes i continued the line under the radius variable for verbosity purposes (not exeeding the 120 limit and it's preferred to arrange the variables on the same tab space)
Same as previous point.


Website Down and Bot Down?

Not able to reach the website and the bot no longer works in my dis channel

Correct, the ISP of RuneInfo is having some trouble rewiring thier building.
If you join the RuneInfo discord channel you can get more info, but that's optional.

The building where the bot and website are being hosted is being completely re-fibered by the ISP (due to issues experienced earlier in the year with mice chewing on exposed fiber lines). While this was supposed to be a two day job (last thursday/friday) there have been a number of hurdles further delaying my specific fiber line (as well as others in the same building). Today I was informed that the subcontractor made all of the fiber runs 6' too short (~2 meters), and the splice tech ran out of fiber on his spool while trying to make up for all the short runs. The splice tech is supposed to be on another job tomorrow, so he was unsure if he would be able to return tomorrow to finish or if it would be Monday before he can return. I apologize for the down time, but there isn't anything I can do about it at this time.
For further updates, please join the official discord via the link in the readme.


Please confirm these before moving forward

[X] I have searched for my issue and not found a work-in-progress/duplicate/resolved issue.
[X] I have not been informed if the issue is resolved in a preview version of the winget client.

Category of the issue
Other
Brief description of your issue
When searching for UltraEdit all is fine:
winget search ultraedit
Name      Id                                 Version   Source
--------------------------------------------------------------
UltraEdit IDMComputerSolutions,Inc.UltraEdit <IP_ADDRESS> winget

When using the Package ID to show detailed information it fails:
winget show IDMComputerSolutions,Inc.UltraEdit

An unexpected error occurred while executing the command:
Download request status is not success.
0x80190194 : Saknas (404).

When trying to export a list of installed apps it fails if UltraEdit is installed and no file is written to disk:
winget export -o C:\temp\InstalledApps.json --accept-source-agreements

2024-05-03 09:29:31.228 [YAML] Detected UTF-8
2024-05-03 09:29:31.229 [CLI ] Installed package is available. Package Id [IDMComputerSolutions,Inc.UltraEdit], Source [Microsoft.Winget.Source_8wekyb3d8bbwe]
2024-05-03 09:29:31.230 [REPO] Downloading manifest
2024-05-03 09:29:31.230 [CORE] WinINet downloading from url: https://cdn.winget.microsoft.com/cache/manifests/i/IDMComputerSolutions,Inc/UltraEdit/<IP_ADDRESS>/03fc
2024-05-03 09:29:31.846 [CORE] Download request failed. Returned status: 404
2024-05-03 09:29:31.846 [FAIL] C:\__w\1\s\external\pkg\src\AppInstallerCommonCore\Downloader.cpp(148)\WindowsPackageManager.dll!00007FF9936891E2: (caller: 00007FF9938150D7) Exception(3) tid(4e20) 80190194 Saknas (404).

    Msg:[Download request status is not success.]
2024-05-03 09:29:31.846 [REPO] Downloading manifest failed, waiting a bit and retrying: https://cdn.winget.microsoft.com/cache/manifests/i/IDMComputerSolutions,Inc/UltraEdit/<IP_ADDRESS>/03fc
2024-05-03 09:29:32.356 [CORE] WinINet downloading from url: https://cdn.winget.microsoft.com/cache/manifests/i/IDMComputerSolutions,Inc/UltraEdit/<IP_ADDRESS>/03fc
2024-05-03 09:29:32.520 [CORE] Download request failed. Returned status: 404
2024-05-03 09:29:32.520 [FAIL] C:\__w\1\s\external\pkg\src\AppInstallerCommonCore\Downloader.cpp(148)\WindowsPackageManager.dll!00007FF9936891E2: (caller: 00007FF9938150D7) Exception(4) tid(4e20) 80190194 Saknas (404).

    Msg:[Download request status is not success.]
2024-05-03 09:29:32.520 [CLI ] Caught wil::ResultException: C:\__w\1\s\external\pkg\src\AppInstallerCommonCore\Downloader.cpp(148)\WindowsPackageManager.dll!00007FF9936891E2: (caller: 00007FF9938150D7) Exception(4) tid(4e20) 80190194 Saknas (404).

    Msg:[Download request status is not success.]

Could this be because the Package Id has a comma in it?
Steps to reproduce

Install UltraEdit
Run winget show IDMComputerSolutions,Inc.UltraEdit

Actual behavior
An unexpected error occurred while executing the command:
Download request status is not success.
0x80190194 : Saknas (404).

Expected behavior
It would give a detailed description of the application
Environment
winget --info
Windows Package Manager v1.7.11132
Copyright (c) Microsoft Corporation. All rights reserved.

Windows: Windows.Desktop v10.0.19045.4291
System Architecture: X64
Package: Microsoft.DesktopAppInstaller v1.22.11132.0

Winget Directories
-----------------------------------------------------------------------------------------------------------------------
Logs                               %LOCALAPPDATA%\Packages\Microsoft.DesktopAppInstaller_8wekyb3d8bbwe\LocalState\Diag…
User Settings                      %LOCALAPPDATA%\Packages\Microsoft.DesktopAppInstaller_8wekyb3d8bbwe\LocalState\sett…
Portable Links Directory (User)    %LOCALAPPDATA%\Microsoft\WinGet\Links
Portable Links Directory (Machine) C:\Program Files\WinGet\Links
Portable Package Root (User)       %LOCALAPPDATA%\Microsoft\WinGet\Packages
Portable Package Root              C:\Program Files\WinGet\Packages
Portable Package Root (x86)        C:\Program Files (x86)\WinGet\Packages
Installer Downloads                %USERPROFILE%\Downloads

Links
---------------------------------------------------------------------------
Privacy Statement   https://aka.ms/winget-privacy
License Agreement   https://aka.ms/winget-license
Third Party Notices https://aka.ms/winget-3rdPartyNotice
Homepage            https://aka.ms/winget
Windows Store Terms https://www.microsoft.com/en-us/storedocs/terms-of-sale

Admin Setting                             State
--------------------------------------------------
LocalManifestFiles                        Disabled
BypassCertificatePinningForMicrosoftStore Disabled
InstallerHashOverride                     Disabled
LocalArchiveMalwareScanOverride           Disabled

Screenshots and Logs
No response

I was able to reproduce this output:
winget show seems have extra trouble with UltraEdit:
PS C:\ManVal> winget show IDMComputerSolutions,Inc.UltraEdit
An unexpected error occurred while executing the command:
Download request status is not success.
0x80190194 : Not found (404).
PS C:\ManVal> winget show "IDMComputerSolutions,Inc.UltraEdit"
An unexpected error occurred while executing the command:
Download request status is not success.
0x80190194 : Not found (404).
PS C:\ManVal> winget show "UltraEdit"
An unexpected error occurred while executing the command:
Download request status is not success.
0x80190194 : Not found (404).
PS C:\ManVal> winget show "IDMComputerSolutions,Inc"
No package found matching input criteria.
PS C:\ManVal> winget show PowerShell
Multiple packages found matching input criteria. Please refine the input.
Name       Id                   Source
---------------------------------------
PowerShell 9MZ1SNWT0N5D         msstore
PowerShell Microsoft.PowerShell winget

But winget list had no problems:
PS C:\ManVal> winget list IDMComputerSolutions,Inc.UltraEdit
No installed package found matching input criteria.
PS C:\ManVal> winget list "IDMComputerSolutions,Inc.UltraEdit"
No installed package found matching input criteria.
PS C:\ManVal> winget list "UltraEdit"
No installed package found matching input criteria.
PS C:\ManVal> winget list "IDMComputerSolutions,Inc"
No installed package found matching input criteria.

And neither did the PowerShell cmdlets:
PS C:\ManVal> Get-WinGetPackage IDMComputerSolutions,Inc.UltraEdit
PS C:\ManVal> Find-WinGetPackage IDMComputerSolutions,Inc.UltraEdit
PS C:\ManVal>

I came across this winget export behaviour today when some clients on our network/domain had a bunch of 2:s (not needed on the client)/not showing in Software Center when it came to all our in SCCM deployed available WinGet applications because of winget export failing to create the list needed by our detection script: https://github.com/Romanitho/Winget-Install/blob/main/winget-detect.ps1.
And it failed because the clients have UltraEdit installed!
In my opinion winget export is flawed when it instead of just skipping the failed download request and move on to the next quits without saving the json to a file.
Should I open an Issue in winget repo too?

Solution pending:
https://github.com/microsoft/winget-pkgs/issues/152915


entity.get(str) should return None when entity does not exists

voila

This should be fixed now. :)


Deployment commit for prod-stable

Merged main branch to prod-stable.
Use latest commit to update namespace ref in app-interface repo. Don't use merge commit, SHAs must be unique when images are created for each branch.

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

Project coverage is 76.96%. Comparing base (2dfe845) to head (248bcc3).

Additional details and impacted files
@@             Coverage Diff              @@
##           prod-stable    #1262   +/-   ##
============================================
  Coverage        76.96%   76.96%
============================================
  Files               68       68
  Lines              673      673
  Branches           120      100   -20
============================================
  Hits               518      518
- Misses             139      155   +16
+ Partials            16        0   -16

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


✨(translations) add translation feature

Purpose
Translate widget properties, submit button and edit button
Proposal

Add translation object as prop of vernacontext
Add translation management on widgets configuration names
Add translation managed for enum and items that handle both add or remove elements

I'll had feature so it works with selector too


Demo App: Component - AI Image Analyzer

Take an image and perform a series of prompts/transforms/etc to achieve the result. This will be different per application so flexibility or ease of customization is preferable.
We can use Ollama to run with the llava set of models using the chat interface. I used my corgi hot dog as a test:
ollama run llava "Respond only with boolean, does this image contain an animal ./loki_sploot.jpg"
Added image './loki_sploot.jpg'
 Yes

ollama run llava "Respond only with boolean, does this image contain an animal: ./hot_dog.jpg"
Added image './hot_dog.jpg'
 No

import ollama from 'ollama'

const res = await ollama.chat({
	model: 'llava',
	messages: [{
		role: 'user',
		content: 'Respond only with boolean, does this image contain an animal'
		images: ['./loki_sploot.jpg']
	}]
})

ref: https://ollama.com/blog/vision-models

i'm taking a stab at this


there is a coccurency problem that lead tcp connections leak.

Search before asking

[X] I had searched in the issues and found no similar issues.

What happened
#15954   there are detailed infomation here.
there are too many <IP_ADDRESS>:50052 connections, and they lead tcp connection leak.  i think it also has the same problem at 3.1.x， but i didn't test it.
What you expected to happen
no
How to reproduce
no
Anything else
No response
Version
3.1.x
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

How many connections build on alert-server? Is this equal to the number of tasks running after restarting the service? How many worker-servers do you have?

There are three worker servers.but their number doesn't always equal the number of tasks as connection leaks don't consistently occur. The number of connections can vary and it primarily depends on the length of time the server has been running.

Mybe we need to further check the problem, print the connection port at the close channel, and check whether the ports in the system include closed ports.Analyze whether the closing method is called but the channel is not closed。

look at here.the connection is runnig here for many days(I obersevered many connections that as same as this).because the channels(a map that store all the channels) lose the quote of the channel forever, it can't close because we cann't  get the channel anymore as the channel losed  when channels.put(host,channel)  called.

 look at here.the connection is runnig here for many days(I obersevered many connections that as same as this).because the channels(a map that store all the channels) lose the quote of the channel forever, it can't close because we cann't get the channel anymore as the channel losed when channels.put(host,channel) called.

When tasks are concurrent (tasks running on the same worker-server), the tasks are A and B. Concurrency occurs when creating channels. NettyRemotingClient.createChannel is called. A creates channelA, B creates channelB, and A uses channelA to send messages, B uses channelB to send messages, but when A closes channel, it takes the channel by host,so get the channel of channelB, and channelB is closed. channelA is free.
当任务出现并发（同一个worker-server运行的任务），任务为A，B，在创建channel的时候出现了并发，调用NettyRemotingClient.createChannel，A创建了channelA，B创建了channelB，A用channelA发送消息，B用channelB发送消息，但是A在关闭的时候根据host拿channel，关闭的是channelB。channelA就游离了。

I have reproduced this issue. In version 3.2.x, it appears that the channel is reused and simply not closed. Keeping the connection was stored in a map seems to be intended for reusing the connection, but closing it after each use negates the purpose of reusing it and only adds complexity. We can add a logic to a handler to close the connection if no data has been transferred for a while, extending this period, for example, over 24 hours. However, upon a preliminary review of the code, versions 3.0 and 3.1 also seem to have this issue.

复现了这个问题。我看3.2.x好像就是复用channel,干脆不关闭了，将连接保存在map中应该是为了复用连接，但是每次调用就关闭根本就没有复用的意义了，反而引入了复杂度（可以加一个逻辑到一个handler里面，如果超过多久没有真正传输数据就关闭，将这个时间调大一点，比如超过24小时）.     但3.0和3.1初看代码都会有这个问题.

I will test and observe it at version of 3.1.9, if there is a problem too,  I'll submit a pr to fix this.

I will test and observe it at version of 3.1.9, if there is a problem too, I'll dicuss it here and then submit a pr to fix this.
+1

Good catch, this is due to the concurrent problem in NettyRemotingClient, if multiple operations belong to one host come at the same time, might create multiple channels. I submit https://github.com/apache/dolphinscheduler/pull/16021 to fix this.

Does it need a container to store the channel that will be deleting.  like below：


Camera is not yet ready

Issue Description
I got an error when I called:
const image = await cameraRef.current.capture({
  saveToCameraRoll: false
});

Possible Unhandled Promise Rejection (id: 3):
Error: Camera is not ready yet.

Steps to Reproduce / Code Snippets / Screenshots

npm i -S react-native-camera-kit
Add the following code to your app's view

// somewhere:
const cameraRef = React.useRef();

// down below that

const takePicture = React.useCallback(async () => {
  const image = await cameraRef.current.capture({
    saveToCameraRoll: false
  });

  console.log(image);
}, []);

// on return
<CameraKitCamera
  ref={cameraRef}
  style={{
    flex: 1,
    backgroundColor: 'white'
  }}
/>

Run app with npm run android
Call the takePicture either with timeout or using TouchableOpacity

Environment

React Native Navigation version: None
React Native version: 0.63.2
Platform(s) (iOS, Android, or both?): Android only
Device info (Simulator/Device? OS version? Debug/Release?): Realme 6 Pro

Hey @aprilmintacpineda, can you try this again with the latest release?

this is not fixed, iam using lates release


Fixed #33132 -- Circular contexts when rendering Form with BoundFields.

Ticket #33132
I've tried to add a test that shows the circular reference in the Context. However, while it shows some reduction in the depth of the context it isn't circular 🤔  I see the test returns about 20 render calls before this patch and reduces to one with it.
In additon to form I think we can also remove field and limit the scope as mentioned on the ticket.
I've not yet reviewed the docs.

It's for me on Python 3.8 🤔

I tried again today and I am seeing a recursion error 👍
I had a look at the docs and I don't think any changes needed. However, I think the label_tag() docs could do with improving. We should explain the context available in the template and I find the whole section a little hard to follow. However, to tackle that needs more energy and creative juices I have right now and is slightly unrelated to this? I'm thinking about creating a separate ticket?

@smithdc1 Thanks for updates :+1:

I had a look at the docs and I don't think any changes needed. However, I think the label_tag() docs could do with improving. We should explain the context available in the template and I find the whole section a little hard to follow. However, to tackle that needs more energy and creative juices I have right now and is slightly unrelated to this? I'm thinking about creating a separate ticket?

IMO, there is no need for a separate ticket, we can Refs #31026 -- ... the original one.

buildbot, test this please.


Did not find any episode values

https://github.com/facebookresearch/Horizon/blob/0e7fe9d742c408e1b42803b42e104efbc56bae5b/ml/rl/training/evaluator.py#L926
I have hit this condition a few times during runs and was wondering if there were docs to explain what it means.
Thanks!

One thought: after running the spark transformation process, my output does not have reward_timeline while the example data does.  I also see that reward_timeline was removed.
https://github.com/facebookresearch/Horizon/commit/e3abf433f8bbeca3140e1a5422568a778b9a3887
This may not be relevant.

Also related, my data does not have a episode_value parameter after spark transformation while the example data for cart pole does.

My mistake - needed to set addTerminalStateRow to true for my application (rewards are delayed until the very end).  Sorry to bother!


importing swagger deletes requests from collection

Describe the bug
Collection requests getting deleted if using same swagger file to import into another collection
To Reproduce
Steps to reproduce the behavior:

~Go to https://editor.swagger.io/~ See https://github.com/Kong/insomnia/issues/3453#issuecomment-877886904
~Click on File -> Convert and save as json~
Open latest Insomnia
Import json file as new new request collection
Go into import and import the same json again, but into a new separate collection
Bug happens, first collection looses all requests.

Expected behavior
Importing same swagger into a new collection does not remove the requests from any other collections. There are cases where I want to import same swagger spec into multiple collections.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: Win10
Installation Method: via downloading installer from page
App Version 2021.3.0

Thank you for the clear reproduction steps.  Can confirm.

I saw this was assigned to me but presumed it was an automated process. Is there anything else I should do apart from reporting this issue? It seems it was replicated by other people and is indeed a bug, and not a fun one at that.

Note, in order to reproduce this you need a spec which has methods with their operationId defined in the spec. The default spec at https://editor.swagger.io no longer has that. Use https://petstore.swagger.io/v2/swagger.json to reproduce.

Closing as duplicate of https://github.com/Kong/insomnia/issues/2882
Also related to https://github.com/Kong/insomnia/issues/4283


Should watch be called initially

Hello!  Thank you for providing this great framework.  I have a question.  When looking at this example: http://jsfiddle.net/lega911/o1qvt257/ the watch callback is called after the initial load even when no changes have happened to the 'filter' scope value.  Is this normal behavior?  Is there a way to prevent a watch from being called initially?
Thanks.

Hello!
Yes, it's normal behavior, Angular.js works similar, old versions didn't fire callback initially, but it's used in 90% cases.
examples how to avoid it:

http://jsfiddle.net/lega911/umcheq8q/
http://jsfiddle.net/lega911/6dae9hxm/   (prevValue === undefined at start)
http://jsfiddle.net/lega911/3xkp7qxc/    (w.refresh()  - updates inner value, that $digest uses for comparing, you can use it any time)

Wow. Thank you for this! I had implemented something like #1 for my use case, but I didn't know about the refresh() method.
Is it acceptable to ask another question on the Issue tracker or is there another place where the community is active that is preferred?
Thanks for taking the time to answer me!

Any place is ok. I observe github, G+ and email.


Rename pt-br.json to pt-BR.json

Correction from pt-br to pt-BR to comply with international standards

This is extremely important for Brazilian users, as using pt-br instead of pt-BR makes the theme in English.

Thanks! Merged 👍


Visible elements

How is it possible to get all visible HTML elements on book page using this library?
Thanks!

I would like to know this as well.


Windows Node UNEXPECTED_KERNEL_MODE_TRAP vfpext.sys

Is this a request for help?:
Yes

Is this an ISSUE or FEATURE REQUEST? (choose one):
Issue

What version of acs-engine?:
0.21.1

Orchestrator and version (e.g. Kubernetes, DC/OS, Swarm)
K8s 1.10.7
Windows Server 2016 17134
What happened:
Blue screen of death OS crash when scheduling a large number of pods simultaneously.
Your PC ran into a problem and needs to restart.
If you call a support person, give them this info:
UNEXPECTED_KERNEL_MODE_TRAP

vfpext.sys

0x0000000000000008
0xFFFFAD8068589EF0
0xFFFFC2057182AFA0
0xFFFFF80F422CA66F

How to reproduce it (as minimally and precisely as possible):
ACS template and Dockerfiles below. It only appears to happen with this particular Dockerfile below (which seems weird, I have other Dockerfiles that don't cause crashes). Scale deployment up to 10 replicas and wait for crash.
{
    "apiVersion": "vlabs",
    "properties": {
        "orchestratorProfile": {
            "orchestratorType": "Kubernetes",
            "orchestratorRelease": "1.10",
            "kubernetesConfig": {
                "addons": [
                    {
                        "name": "kubernetes-dashboard",
                        "enabled": false
                    }
                ],
                "apiServerConfig": {
                    "--token-auth-file": "/etc/kubernetes/token.csv"
                }
            }
        },
        "masterProfile": {
            "count": 1,
            "dnsPrefix": "",
            "vmSize": "Standard_B2ms",
            "customFiles": [
                {
                    "source": "",
                    "dest": "/etc/kubernetes/token.csv"
                }
            ]
        },
        "agentPoolProfiles": [
            {
                "name": "windowspool",
                "count": 1,
                "vmSize": "Standard_B2ms",
                "availabilityProfile": "AvailabilitySet",
                "osType": "Windows",
                "osDiskSizeGB": 127,
                "storageProfile": "StorageAccount"
            }
        ],
        "windowsProfile": {
            "adminUsername": "",
            "adminPassword": "",
            "windowsPublisher": "MicrosoftWindowsServer",
            "windowsOffer": "WindowsServerSemiAnnual",
            "windowsSku": "Datacenter-Core-1803-with-Containers-smalldisk"
        },
        "linuxProfile": {
            "adminUsername": "",
            "ssh": {
                "publicKeys": [
                    {
                        "keyData": ""
                    }
                ]
            }
        },
        "servicePrincipalProfile": {
            "clientId": "",
            "secret": ""
        }
    }
}

Base Dockerfile:
FROM microsoft/dotnet-framework@sha256:c2586af354bc76c2f431e3d6584bd38c277b896a6a5b07dc0042b4ff7480956f

RUN powershell -Command \
    New-ItemProperty -Path HKLM:\SYSTEM\CurrentControlSet\Services\Dnscache\Parameters -Name MaxNegativeCacheTtl -PropertyType DWord -Value 0

COPY ./helpers /helpers

App Dockerfile:
ARG REGISTRY

FROM ${REGISTRY}/**my image and tag derived from above Dockerfile**

ARG CONFIGURATION

ENTRYPOINT powershell.exe C:\helpers\install-certs.ps1 && C:\application\myapp.exe

COPY ./bin/${CONFIGURATION} /application

Anything else we need to know:
I have a QA env where I can fairly reliably repro the issue if anyone wants me to test anything.

This is fixed in an update that's not part of the VM yet. You can work around it using this in your apimodel:
{
  "apiVersion": "vlabs",
  "properties": {
    "orchestratorProfile": {
      "orchestratorType": "Kubernetes",
      "orchestratorRelease": "1.11"
    },
    "masterProfile": {
      "count": 1,
      "dnsPrefix": "plang0607a",
      "vmSize": "Standard_D2_v3"
    },
    "agentPoolProfiles": [
	    {
		"name": "windowspool",
		"count": 2,
		"vmSize": "Standard_D2_v3",
		"availabilityProfile": "AvailabilitySet",
		"osType": "Windows",
		"osDiskSizeGB": 127,
		"extensions": [
	    {
			"name": "windows-patches"
		    }
		]
	    },
	    {
		"name": "linuxpool",
		"count": 2,
		"vmSize": "Standard_D2_v2",
		"availabilityProfile": "AvailabilitySet",
		"osType": "Linux"
	    }
    ],
    "windowsProfile": {
	    "adminUsername": "",
	    "adminPassword": "",
	    "windowsPublisher": "MicrosoftWindowsServer",
	    "windowsOffer": "WindowsServerSemiAnnual",
	    "windowsSku": "Datacenter-Core-1803-with-Containers-smalldisk"
     },
    "extensionProfiles": [
          {
	    "name": "windows-patches",
	    "version": "v1",
            "extensionParameters": "'https://privatepatches.blob.core.windows.net/hotfix/Windows10.0-KB999999-x64-InstallForTestingPurposesOnly.exe?se=2018-09-17T00%3A09%3A48Z&sp=r&sv=2017-07-29&sr=b&sig=SoWlPWDCIsyuO2xKvPRgeK90U/9k/K6B4wkdFT3Jgpg%3D', 'http://download.windowsupdate.com/c/msdownload/update/software/secu/2018/08/windows10.0-kb4343909-x64_f931af6d56797388715fe3b0d97569af7aebdae6.msu'"
    }
    ],
    "linuxProfile": {
      "adminUsername": "azureuser",
      "ssh": {
        "publicKeys": [
          {
            "keyData": ""
          }
        ]
      }
    },
    "servicePrincipalProfile": {
      "clientId": "",
      "secret": ""
    }
  }
}

If that doesn't work - please open a support ticket and they can provide a private patch.
cc @daschott

Hi @PatrickLang ,
I have had the same issue today. To apply your suggestion above do I need to completely delete the cluster and deploy a new one or is there some other process to this?
Thanks for your help

@tonyschollum you need to open a support ticket with Microsoft/Azure, else there should be a public KB coming out soon (no earlier than mid-November unfortunately.) Will loop back here once it is posted.
Either option does not require you to delete your cluster. You will be required to install an update and reboot each one of your nodes however. If you have a new cluster you are trying to create, you can also apply the patches automatically using Windows Patching Extension

If you have a fix from Microsoft support, you can follow the steps here to get access to your existing nodes with Remote Desktop. Then you can post the fix in a storage account, download and install it on the Windows nodes
https://github.com/Azure/acs-engine/blob/master/docs/kubernetes/troubleshooting.md#connecting-to-windows-nodes
Using the Windows patching extension is definitely easier for new clusters.

Ah yeah unfortunately the msu doesn't successfully install but thanks for the help.
I did notice that Datacenter-Core-1809-with-Containers-smalldisk is now on the list of published images and is the equivalent build of Windows 2019. Thinking this would have the fix?
Only issue currently against that SKU is:
Events:
  Type     Reason                  Age               From                   Message
  ----     ------                  ----              ----                   -------
  Normal   Scheduled               5m                default-scheduler      Successfully assigned default/ndmdotnet-admintool-5fb787d6f6-zws7l to 16822k8s9010
  Warning  FailedCreatePodSandBox  3s (x24 over 5m)  kubelet, 16822k8s9010  Failed create pod sandbox: rpc error: code = Unknown desc = failed pulling image "kubletwin/pause": Error response from daemon: repository kubletwin/pause not found: does not exist or no pull access

No, this issue still persists on Windows Server 1809 until a hotfix is released (ETA late January). The official hotfix is available
now for WS1803: https://support.microsoft.com/en-us/help/4467682/windows-10-update-kb4467682
Can you try it? What errors are you seeing?
For DataCenter-Core-1809, you are correct I also noticed an issue in acs-engine because of the move to the new container registry. You need to build kubletwin/pause image by editing the Dockerfile in C:\k to annotate the
mcr.microsoft.com/windows/nanoserver:1809 image
and build it as docker build -t kubletwin/pause .
Once you move to azure-CNI 1.0.14, please also double-check that the service CIDR in the 10-azure.conflist isn't set to <IP_ADDRESS>/8 but <IP_ADDRESS>/16. You can check the service CIDR using
kubectl cluster-info dump | grep -i service-cluster-ip-range
If you're still having trouble with DNS, one workaround is to remove the entry for the 163.x.y.z nameserver. For some reason it isn't reaching it on my cluster. I'm not from the Azure networking team but I found that worked for all the typical networking flows then.

@y325A can you also please use https://support.microsoft.com/en-us/help/4467682/windows-10-update-kb4467682 and confirm whether the issue is resolved?

@daschott thanks so much for your guidance just out of curiousity I was looking at the changelog for https://support.microsoft.com/en-us/help/4467682/windows-10-update-kb4467682 do you know which line in particular talks about this problem as don't see it listed.

I don't think it's explicitly written there, the changelogs aren't exhaustive and don't always reflect all the fixes unfortunately... But looking at the bug this is the official KB article that contains a complete patch for the BSoD referenced by @y325A here.

Ah @daschott your awesome, the msu you referenced does work at least so far and haven't seen the issue for the past hour or so, going to leave on overnight and test. Appreciate you taking the time :D

Hey @daschott I think acs-engine now defaults to 1809 so to be able to apply the patch since for 1809 won't be available for a month, should I just revert the generated azuredeploy to call the older 1803 image?

Hey @daschott @PatrickLang the update msi patch is no longer hosted. Do you know how I can get this update again? Not sure why it was removed.

@sylus, yes it seems like the KB was pulled down because there were some other changes included in the package which introduced regressions. Can you use the new 12B release, it should contain all the fixes from the previous KB without regressions:
https://support.microsoft.com/en-us/help/4471324

Ah thanks so much @daschott your awesome :)

Alright, closing since https://support.microsoft.com/en-us/help/4471324/windows-10-update-kb4471324 is released.


Cyclically rotate an array by one

❗Write Code in Only One Language ( CPP , C , Java , Python )
Give File Name :  cyclically_rotate  ✅ In respective language folder
📑 DESCRIPTION
Cyclically rotate an array by one

Given an array, rotate the array by one position in clock-wise direction.
Your Task:
You don't need to read input or print anything. Your task is to complete the function rotate() which takes the array A[] and its size N as inputs and modify the array in place.
Example 1:
Input:
N = 5
A[] = {1, 2, 3, 4, 5}
Output:
5 1 2 3 4

Example 2:
Input:
N = 8
A[] = {9, 8, 7, 6, 4, 2, 1, 3}
Output:
3 9 8 7 6 4 2 1

Constraints:
1<=N<=105
0<=a[i]<=105

assign this is to me i am from RCPIT

@Mr-Yash-beldar let me solve this..

@Mr-Yash-beldar Hi,can you please assign this issue to me,thanks!

@Mr-Yash-beldar solved check PR #108


Improve error messages related to shader_type

The list of allowed shader types is now displayed if any shader_type-related error is emitted.
This makes it easier to remember which shader types are allowed when creating a new shader.
Preview

Thanks!


How do I iterate through FirebaseListObservable<[]> object

Version info
Angular: 2.1.0
Firebase: 3
AngularFire: 2.0.0-beta.6
Other (e.g. Ionic/Cordova, Node, browser, operating system): Ionic
How to reproduce these conditions
Steps to set up and reproduce
Say I subscribe to a list of items:
this.items = this.af.database.list('items');

And I add to these items in another function:
this.items.push({ im: 'cool' });

Then in another function I'd like to iterate through my items:
this.items.forEach(item => {
    console.log('Item:', item);
});

Expected behavior
I expect console.log(item) to print out $ Item: { im: 'cool' } but instead it prints out an array for each time I added to the items list.
Actual behavior
The whole array is printed multiple times:
$ Item: []
$ Item: [{ im: 'cool' }]
$ Item: [{ im: 'cool' }, { again: 'yep' }]

Thanks for using the issue template! Let me take a look and see what the intended behavior here is.

Hello,
I had the same issue with
this.af.database.list('/items', {
    query: {
                orderByChild: 'im',
           equalTo: 'cool'
               }
}).subscribe(snapshot => { console.log(snapshot );});
Expected behavior
I expect console.log(snapshot ) to print out :
 [{ im: 'cool' }]
Actual behavior
What is actually printed out
[]
[{ im: 'cool' }]
I added then something like :
.subscribe(snapshot => {
if(snapshot.length > 0){
console.log(snapshot );
}
});

This is intended behaviour.
Observable.forEach() is an alias for Observable.subscribe(), according to the rxjs documentation
In order to iterate through a FirebaseListObservable:
// this.items is an Observable. You could use .forEach here too, but I find that confusing.
this.items.subscribe(items => {
    // items is an array
    items.forEach(item => {
        console.log('Item:', item);
    });
});

So even if you use the subscribe method, I'm still getting the same issue. Perhaps this is just how the subscribe process works.
Say I have a list of 2 items. My subscribe method is called 3 times (even when using subscribe):
$ Item: []
$ Item: [{ im: 'cool' }]
$ Item: [{ im: 'cool' }, { again: 'yep' }]

Is there a more proper way of querying a the list of data and getting back all the data at once, instead of in pieces?

@sean-hill , It is what I am trying to explain in my previous post, I am having the same issue with the subscribe method. As @Maistho told us,

Observable.forEach() is an alias for Observable.subscribe()

As I understand it, Observables are asynchronous so there is no way to retrieve only once your list as the complete list using subscribe() or forEach() .
So I see 2 possibility,
1- You deal with the multiple console.log('Item:', item);
2- It seems to be possible to use maps : .map .flatMap as a workaround,
cf : http://stackoverflow.com/questions/38259038/getting-data-out-of-firebase-object-with-angular2-angularfire2-and-typescript

@sean-hill Do you get the data back in chunks of one Object at a time even if it is available in the firebase database to begin with?
The returned array should always contain all items that have been added to the firebase database at that location. (sometimes it might be empty even though it's not? It works that way with auth)

@Maistho that is correct. I'll try and make a jsfiddle that shows the issue.

@Maistho take a look at this repo and follow the instructions to replicate. I tried to host it with firebase, but the ng build command wasn't working with the angular cli.

@Maistho take a look at this repo and follow the guide in the README to replicate the issue.

@sean-hill That does seem like a bug. Weird, I have not experienced that myself.
Inspecting the websocket shows me that the data only arrives once, and as far as I can tell from this code, the observable should not trigger next multiple times.
Try setting the subscription in the constructor or ngOnInit and see if the same issue appears again.
Also, I see that you have included a newer version of firebase in your package.json, maybe try removing that (it's included with angularfire2) as well?

@Maistho I removed the firebase dependency, which I installed from the Installation Instructions:
$ npm install firebase angularfire2 --save

Still had the same problem. I'll try setting up the subscription in the constructor. My use case though for this flow was this. I wanted to create a list of items, and then copy that list of items and push them all into a separate collection. AKA, the first list of items was a template, and then I would take those items and turn them into actual commands that a process would run. Perhaps it's not a firebasey why of doing things 😃

I can't reproduce anything unexpected by creating a list and calling push(). The initial list is loaded with no data, when an item is added, the subscription is updated with the new value, and so on. Since there's not enough code in the repro here to recreate the conditions you've described (just some random snippets described as "in another function" I'm going to close this.
Feel free to resubmit with a minimal, verifiable, complete, example, such as this one that demonstrates some unexpected behavior and I'll be happy to continue the discussion.

@katowulf did you take a look at the example repo I posted above? It shows the exact issue when you open the console along with the code base that causes it.

@katowulf in addition to the repo I already provided above, I just forked your plunker to show the issue also. View it here. Just open your console and click "View all".

Thanks! Turns out this is a dup of #574


Blur the background when the popup is active

Either blur, or darken, but so that you can see that it's not clickable

Awful browser support (https://caniuse.com/css-backdrop-filter), but added it.


Get product by id: response status codes fixed

Description
This PR is updated the endpoint get/products/:id respond with correct response message and status code.
Fixes # (191)
How to test?
Pull the branch, run: npm run dev
Check on the swagger.
For ex: id =5, Status code:200 and response: product with 5
id =ab, Status code:400 and response: invalid id
id =4ab/ab4, Status code:400 and response: invalid id
id =567, Status code:404 and response: product with id 567 doesn't exist.
status code:500 for errors of server(if)
Checklist

[ ✔] I have performed a self-review of my code
[✔ ] I have followed the name conventions for CSS Classnames and filenames, Components names and filenames, Style filenames, if you are in doubt check the the project README.MD and here https://github.com/HackYourFuture-CPH/curriculum/blob/master/review/review-checklist.md
[✔ ] I have commented my code, particularly in hard-to-understand areas, if you code was simple enough mark the box anyway
[✔ ] I have made corresponding changes to the documentation, if you code was simple enough mark the box anyway
[ ✔] This PR is ready to be merged and not breaking any other functionality

I didn't, but let me check that. Something might happened while merging.
On Wed, 23 Feb 2022, 20.50 FlorentinaPetica, @.***>
wrote:

@.**** commented on this pull request.
In src/server/api/controllers/products.controller.js
https://github.com/HackYourFuture-CPH/fp-class19/pull/192#discussion_r813257460
:

@@ -78,19 +57,48 @@ const getProductById = async (id) => {

     'is_on_discount',

     'discount_percent',

     'picture',

 )

 .where({ id });

if (products.length === 0) {

 throw new Error(`incorrect entry with the id of ${id}`, 404);

)

.where('products.is_on_discount', '=', '1');

+const getProductById = async(prod_id) => {

const id = parseInt(prod_id, 10);

getProductsById function is working as intented. 👍
In src/server/api/controllers/products.controller.js
https://github.com/HackYourFuture-CPH/fp-class19/pull/192#discussion_r813261410
:

 .select(

 'id',

 'name',

 'price',

 'color',

 'size',

 'is_available',

 'stock_amount',

 'is_on_discount',

 'discount_percent',

 'picture',

)

.where('products.is_on_discount', '=', '1');

This function getDiscountProducts looks a little incomplete. It doesn't
have any error handeling. Did you deleted something?
—
Reply to this email directly, view it on GitHub
https://github.com/HackYourFuture-CPH/fp-class19/pull/192#pullrequestreview-891596105,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ATK25BRLJQNRHQNYMTCOLVDU4U3B3ANCNFSM5PEZ4UFQ
.
You are receiving this because you were assigned.Message ID:
@.***>


useParam: implement setParams on native

currently, the behavior on iOS/Android is to start with the screen parameter, and then use react state after that. this doesn't make much sense. we should use react navigation params the whole time, by using setParams under the hood.

Hi Fernando.
As this been solved?
Currently testing my app on andriod. And when using params, it only uses the first one selected. Routing to that page still returns the same param even with a different slug.
But all works as expected on web.
I use "solito": "latest"

hey, not quite yet. i know the solution but haven’t had time to fix it with work

Alright, thanks man!

You can test this now on 0.0.29-alpha.1:
yarn add solito@canary


Add integrated test for distinct syntax

Pull Request Test Coverage Report for Build 6129

7 of 8 (87.5%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.04%) to 72.73%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

sharding-core/src/main/java/io/shardingsphere/core/parsing/antlr/filler/impl/SelectClauseFiller.java
3
4
75.0%

Totals

Change from base Build 6128:
0.04%

Covered Lines:
9396

Relevant Lines:
12919

💛  - Coveralls


JobPost call failed when mapping the response to the DTO

When calling GetJob from Hippo, the response was always null. While debugging the Client I noticed that the mapping of the Job object was failing due to some object not expecting null values. Is the API Client generated from the latest Nomad release?

The API client was generated using the same openapi.yaml file found in the nomad-openapi project, which was recently bumped up to 1.3.0-beta.1 compatibility.

Hmm... Looking at the EmitDefaultValue property, if the default value is set (i.e. null in this case), it shouldn't be emitted when serialized to JSON.
https://docs.microsoft.com/en-us/dotnet/api/system.runtime.serialization.datamemberattribute.emitdefaultvalue?view=net-6.0

That being said, the docs do say

Note

Setting the EmitDefaultValue property to false is not a recommended practice. It should only be done if there is a specific need to do so (such as for interoperability or to reduce data size).

So maybe the autogenerated client is not following best practices.

The build warnings may indicate something wrong with the autogenerated code:

/home/bacongobbler/code/nomad-dotnet/src/Fermyon.Nomad/Model/CSIVolume.cs(514,22): warning CS8073: The result of the expression is always 'true' since a value of type 'DateTime' is never equal to 'null' of type 'DateTime?' [/home/bacongobbler/code/nomad-dotnet/src/Fermyon.Nomad/Fermyon.Nomad.csproj]

Hmm... Here's one that may explain the issue: https://github.com/hashicorp/nomad-openapi/pull/86

Would you mind signing off on the commits following the guide from the DCO bot?
https://github.com/fermyon/nomad-dotnet/pull/1/checks?check_run_id=6355873060


Add Stylus support

Summary
Adds syntax highlighting for Stylus code and files.
Checklist

[ ] Added or updated unit tests (required for all new features)
[ ] Has server changes
[ ] Has redux changes
[ ] Has UI changes
[x] Includes text changes and localization file (.../i18n/en.json and .../webapp/i18n/en.json) updates
[ ] Touches critical sections of the codebase (auth, posting, etc.)

adding @hmhealey to review this PR as this might need to be included in RN as well

@enahum I will remove the changes as soon as possible. Just curious: what's wrong with these localizations?

Just curious: what's wrong with these localizations?

No need to reply, I found the reason myself!

can you remove the changes in every localization file but en.json ?

Done.


POST request timeout and BadRequestError: request aborted

Hello, thank you for that module.
I'm using cors-anywhere as ExpressJS middleware for GET and POST requests. GET requests working as expected, but POST request can not be parsed by target test server and body parser always gives: BadRequestError: request aborted.
Comments in example code.
Cors anywhere middleware:
var app = require('express')();
var corsAnywhere = require('cors-anywhere');

var port = 3330;

let proxy_cors = corsAnywhere.createServer({
  originWhitelist: [], // for test purpose
  requireHeaders: [], // for test purpose
  removeHeaders: [], // for test purpose
  httpProxyOptions: {
    secure: false
  },
});

app.get('/proxy/:proxyUrl*', async (req, res) => {
  req.url = req.url.replace('/proxy/', '/');
  proxy_cors.emit('request', req, res);
}); // WORKING

app.post('/proxy/:proxyUrl*', async (req, res) => {
  req.url = req.url.replace('/proxy/', '/');
  proxy_cors.emit('request', req, res);
}); // NOT WORKING, BECAUSE SOMETHING SENT WRONG

app.listen(port, function () {
    console.log('Server started on port:', port);
});

Test ExpressJS server:
var app = require('express')();
var bodyParser = require('body-parser');

var port = 3333;

app.use(bodyParser.json());
app.use(bodyParser.urlencoded({extended: false}));

app.get('/test', function (req, res) {
    // WORKING FINE
    res.json({status: 'ok'});
});

app.post('/test', function (req, res) {
    // DIRECT CALL (without cors-anywhere) - WORKS
    // CORS-ANYWHERE: REQUEST NEVER GET HERE AND AFTER TIMEOUT, BODYPARSER GOT: BadRequestError: request aborted
    res.json({status: 'ok'});
});

app.listen(port, function () {
    console.log('Server started on port:', port);
});

Works for me with Node v14.13.1, cors-anywhere 0.4.3, express 4.17.1, and curl http://<IP_ADDRESS>:3330/proxy/http://<IP_ADDRESS>:3333/test -dparam=value
How can this issue be reproduced?

Hi, i encounter the same issue.
i tried to do a POST request with a JSON body, i returned timeout.
i've no problem when doing a GET request or a POST request without body.
here's my example of POST request:
POST http://localhost:3100/proxy/https://postman-echo.com/post
JSON body: { "test": "anything" }


defines CPU orchestration APIs

Signed-off-by: Joseph<EMAIL_ADDRESS>Ⅰ. Describe what this PR does
Ⅱ. Does this pull request fix one issue?

Ⅲ. Describe how to verify it
Ⅳ. Special notes for reviews

/hold
let's wait for #209 being merged.

/lgtm
/approve

/hold

/unhold

/lgtm


Readme

I think we should include links to famous.org examples near the top (ala React).  Shd I just submit PR?
https://github.com/facebook/react/blob/master/README.md

Absolutely!


Update tooltips and popover design according to UI Kit v5

Related issues
Closes #1594
Description
Update tooltips and popovers design.
Motivation & Context
Compliance to UI Kit v5.
Types of change

New feature (non-breaking change which adds functionality)

Live previews
Tooltips: https://deploy-preview-1803--boosted.netlify.app/docs/5.3/components/tooltips/
Popovers: https://deploy-preview-1803--boosted.netlify.app/docs/5.3/components/popovers/#four-directions
Checklist
Contribution

[x] I have read the contributing guidelines

Accessibility

[x] My change follows accessibility good practices; I have at least run axe

Design

[x] My change respects the design guidelines defined in Orange Design System
[x] My change is compatible with responsive display

Development

[x] My change follows the developer guide
(NA) I have added JavaScript unit tests to cover my changes

Documentation

(NA) My change introduces changes to the documentation and/or I have updated the documentation accordingly

Checklist (for Core Team only)

[x] My change introduces changes to the migration guide
(NA) My new component is added in Storybook
[x] My new component is compatible with RTL
[x] Manually run BrowserStack tests
[x] Manually test browser compatibility with BrowserStack (Chrome >= 60, Firefox >= 60 (+ ESR), Edge, Safari >= 12, iOS Safari, Chrome & Firefox on Android)
[ ] Code review
[ ] Design review
[ ] A11y review

After the merge

[ ] Manually launch Percy tests

Possible graphic issue on laptop screen (but not when I resize my other screen to the size of my laptop screen):

Some elements after the spec review:

Can rollback the modifications regarding the max-width that could stay 196px if the logic in Boosted is right. Otherwise, it can be set to 200px as done by Bootstrap.
Top padding of tooltip and multiline tooltip is the same: 10px
Arrow's size is 5px (including the border)

Small update with the review above. Small space between the element and the popover, I don't know if it can be fixed:

Possible graphic issue on laptop screen (but not when I resize my other screen to the size of my laptop screen):

Should be fixed by https://github.com/Orange-OpenSource/Orange-Boosted-Bootstrap/pull/1803/commits/306ed534979cacaceddf3e1b283f1befd46fcb69

A remaining graphic buh has been submitted to Bootstrap: Popover arrow while zooming #38200

Design review of the Tooltips and Popovers styles
For both the Tooltips and Popovers, the pointer sizes, outline thickness, outline colour, padding, spacing and font sizes are all good.
Changes: Include a 5px space between the tip of the pointer and the object it is pointing to. In the examples, the pointers are touching the objects.

Design review update
Spacing between the pointer tip and the object is now correct.

Is it an optical effect or the width of the border's arrow is missing 1 or 2 px? macOS + Firefox here

Might have found a solution by playing with ::before, the left/right values, and the border widths. But let's tackle it after having merged this PR. It's gonna be easier to review and to modify.

Decided in daily:

Keep this "Custom popover" section.
Restore the "Custom button" one (to do in an other PR)
Add a design callout message (in this PR if not already done in PR #1614)
Keep the custom padding-bottom of 10px (instead of 20px specified by Orange designers) that illustrates how this component can be custom
To emphasize the customization, the orange color used for the borders and the background title part will be replaced by an other one (we thought of using Bootstrap's purple). Depending on the purple, the text color might be changed to white instead of black to ensure a satisfying text/background color contrast. We intend to use this same color on each custom variant of Boosted component. A PR will be made to Bootstrap to ask them to do the same.


Allow installation in laravel 9 and small fix in the pricelist response

Two small changes:

Laravel 9 has psr/log version 3, so I can't install the package by default on laravel
The Pricelist response has a prices key which should be used to parse the collection

Could you update the test for this as well? It's CustomersApiListPricesTest

Done!


Add new default dashboard templates for routers and switches

Relevant information
Adds new default dashboard templates for routers and switches.
Checklist

[x] I've read the guidelines and understand the acceptance criteria.
[x] The value of the attribute marked as identifier will be unique and valid.
[x] I've confirmed that my entity type wasn't already defined. If it is I'm providing an
explanation above.

Hello, I'm getting a validation-related error that seems to be related to the underlying Node version -- the GitHub action needs to be updated. https://github.com/newrelic/entity-definitions/actions/runs/3472451492

Hi, wondering if someone can take a look at this stuck validation. I tried to fix the node version-related error I was seeing earlier but updating the github action code, and now that action is stuck.


Add support for Rails 6

This PR adds support for Rails 6 to the gemspec.
What do you think about removing the upper dependency or moving it to be less than  < 7?
I imagine the asset pipeline will be around until then, as I haven't seen any plans announced yet to fully remove it from Rails just yet, and imagine they would only do so after deprecating it (maybe in Rails 6.1?)

@kylefox If you'd like, I can help maintain the gem? ❤

Hey guys, I've merged it into master. I'm moving over the next few weeks but will work on getting it so I can publish a new version to RubyGems, in the meantime, check it out from master and let us know if there are any more issues.
cc @AnthonyClark

Hi friends! I've just pushed 2.1.0 to RubyGems with support for Rails 6. Thanks for the patience!

Thank you @king601 !


Update-1

Requirements

Filling out the template is required. Any pull request that does not include enough information to be reviewed in a timely manner may be closed at the maintainers' discretion.
All new code requires tests to ensure against regressions

Description of the Change

I corrected the language parts. First, I prepared the languages in i18n.
I changed the code in "Articles in this category". This requires logic knowledge, please see here.
I added the path for category pages. Its name is "breadcrumbs.html".
I installed the newest version of Simple-Jekyll-Search. This is a javascript code, so you don't need to change it in the future. This code must be at the bottom of base.html due to DOM. Please don't touch.

Alternate Designs
Simple-Jekyll-Search was chosen for lighter and faster search results. The deficiencies are fixed
Benefits
Languages have been edited, categories have been created, the search part has been edited. All errors are fixed.
Possible Drawbacks
Since the posts are not in Jekyll format (see here, they are not in _posts, etc.), you must manually edit the search.json section.
Verification Process
 I added and integrated languages. Result: Works fine.
--> I added the search bar. Result: Works in separate languages (English - Russian).
--> I added and integrated languages. Result: Works fine.
--> I added the path for category pages. Result: It displays.
--> I changed the code in the "Articles in this category". Result: It was not working, now it is working very well.
Applicable Issues
The search works fine, but because of a bug, it outputs an error for no reason. There is no problem, it just writes an error on the console.

@fsuaterdogan, please review this remarks

In _data/i18/en.yml

12 row should be changed to "searchbar: What can we help you with?"

In  _data/i18/ru.yml

9 row "чем" should be changed to "Чем"

12 row should be changed to "searchbar: Чем мы можем вам помочь?"

17 row should be changed to categs:"Категории"

24 row should be changed to "askaquestion: Задать вопрос"

27 row should be renamed to "about: О нас"

28 row should be renamed to "jobs: Вакансии"

When this remarks will be fixed I test how search does work.


Enable key-ration ability for piped key

What this PR does / why we need it:
Which issue(s) this PR fixes:
Fixes #1102
Does this PR introduce a user-facing change?:

Enable key-ration ability for piped key

"key-rotation ability" may be more appropriate for the commit message.

Thank you.
/approve


fixing jhu to be hutch and adding candace as contributor

After talking with Ira, he thought since the majority of the time that I worked on this course was at Hutch, it makes sense to publish there. The sample attribution was therefore out-of-date.
Adding Candace as a contributor author.

@cansavvy  what do you think?


Masking based on columns not rows

When looking at the utils/shap_helpers.py I see that a mask is defined. However, if I am right, the mask is defined on the number col columns while it should be the number of row.

I would propose the followoing change
X.shape[1]

Would become
X.shape[0]

Also:
There is a typo in model_interpret.py on row 99.

Should be neither

@Matgrb @timvink I saw I messed-up the pipeline (sorry about that). What is the best option? Revert the previous merge and re-open this issue or create a new issue?

No worries Gilles!
If you would like to fix the pipeline and test the fix, you can make a PR that implements it and see if tests succeed :)


feat(FieldTheory/KrullTopology): topology on finite Galois group is discrete

From the FLT project.

Whoops, wrong button
bors r-


Crashes on hotreload enviroment.

Hi! While trying to use rlImGui on a hotreloadable enviroment, where the game code is loaded dynamically, the following line triggers an assert in glfw on call to glGetFramebufferSize.
https://github.com/raylib-extras/rlImGui/blob/d89083021bbf9eadefb5a307154f99e421c4901f/rlImGui.cpp#L94
Assertion failed: window != NULL, file ${LOCAL_PATH}/glfw/src/window.c, line 694

The solution was replacing the call to glfwGetFramebufferSize with:
width = rlGetFramebufferWidth();
height = rlGetFramebufferHeight();

Currently is working fine on my specific project, so I don't know if this is an actual issue or not.
Keep the good work. ❤

I have pushed these change sup, thank you.

I am reverting this change, it does not work properly with resizable windows. That is more important than supporting a hot reload system.


Webpack 2 Support

Hi, thanks for this plugin! I haven't tried it with webpack 1 but at least with webpack 2 I can't use it. My conf:
new ExternalsPlugin({
    type: 'amd',
    include: process.cwd() + '/bower_components'
}),

10% building modules
Fatal error: Cannot read property 'request' of undefined

https://github.com/Morhaus/webpack-externals-plugin/blob/master/index.js#L19 seems to be it. Time to release a webpack 2 version of the plugin?

I created this repo meanwhile, as I needed it for a project of mine. If you merge this I can delete mine.

also have this issue
/Users/steve/Projects/mern-starter/node_modules/webpack-externals-plugin/index.js:19
              data.dependency.request,
                             ^

TypeError: Cannot read property 'request' of undefined
    at /Users/steve/Projects/mern-starter/node_modules/webpack-externals-plugin/index.js:19:30
    at applyPluginsAsyncWaterfall (/Users/steve/Projects/mern-starter/node_modules/webpack/lib/NormalModuleFactory.js:94:13)
    at /Users/steve/Projects/mern-starter/node_modules/tapable/lib/Tapable.js:268:11
    at NormalModuleFactory.params.normalModuleFactory.plugin (/Users/steve/Projects/mern-starter/node_modules/webpack/lib/CompatibilityPlugin.js:52:5)
    at NormalModuleFactory.applyPluginsAsyncWaterfall (/Users/steve/Projects/mern-starter/node_modules/tapable/lib/Tapable.js:272:13)
    at resolver (/Users/steve/Projects/mern-starter/node_modules/webpack/lib/NormalModuleFactory.js:69:10)
    at process.nextTick (/Users/steve/Projects/mern-starter/node_modules/webpack/lib/NormalModuleFactory.js:194:7)
    at _combinedTickCallback (internal/process/next_tick.js:73:7)
    at process._tickCallback (internal/process/next_tick.js:104:9)


Prepare release 0.13.2

@rgommers Feel free to tag the release when you have a minute.

Release is up now.

@dnicolodi I replaced the 0.13.2 tag you pushed with a GPG-signed one also containing the changelog entries: https://github.com/mesonbuild/meson-python/releases/tag/0.13.2. This is documented in https://meson-python.readthedocs.io/en/latest/contributing/release-process.html.
I'll open a PR to tweak the PyPI upload instructions, because PyPI no longer supports GPG signature upload, and in fact emails you that they were discarded if you try.

I didn't intend to push a tag. I don't think Github allows to push tags via merge requests.

No worries at all. I thought the same - no tags via PRs, so it's a little unclear what happened. It's even possible that I did it, and that since it was an unsigned tag the GitHub UI reported it as coming from you because it was attached to one of your commits.


Better rundb registration

When strax registers new rundb entries, the host field https://github.com/XENONnT/straxen/blob/4c59975e39fb44bd70a426e949c061511ba54eef/straxen/rundb.py#L158 should probably refer to the hostname alias ('dali') rather than the full hostname ('dali-login1.etc.etc').
We should also include the lineage hash of the data (e.g. in the meta field under lineage_hash) rather than just the lineage. This will make searching a lot easier.

Since this is now handled mostly by rucio I'm closing this issue


[UI Enhancement] Elasticsearch exception type=index_not_found_exception

Please answer these questions before submitting your issue.

Why do you submit this issue?
[ ] Question or discussion
[ ] Bug
[ ] Requirement
[x] Feature or performance improvement

Requirement or improvement

Please describe your requirements or improvement suggestions.

When using ElasticSearch as the storage backend, no data on the date selected by the UI will cause oap-server to throw a large number of index_not_found_exception exceptions, and the UI interface will not display any error prompts.

Therefore, should the UI date selection component restrict the selectable dates to ensure that the user selects a date without data and the aop server cannot return data?

@EvanLjp Could you recheck? I remember we ignore the case index not exist in the codes.

I would recheck it today, thx for your notice @mritd

@EvanLjp Thanks for your reply, my operating environment:
OS: Kubernetes
oap-server: apache/skywalking-oap-server:8.2.0-es7
skywalking-ui: apache/skywalking-ui:8.2.0
Elasticsearch: docker.elastic.co/elasticsearch/elasticsearch:7.9.2

@mritd I'm sorry that I only added the codes in the ES6 client and ignored the ES7 client. Thx a lot.

@EvanLjp Okay, I will wait 8.3.0 fixed. 😁


[Remote Store] Add segment transfer timeout dynamic setting

Description

Segment uploads for Remote Store happen without any timeout and can hold the thread on latch.await() for ever if there is an error which is swallowed silently by the code.
Since the future is never completed, the shard is stuck and is never able to retry the upload
This leads to remote time lag to increase and never come down until we restart the process
This PR adds the timeout to segment uploads to recover automatically from such situations after the timeout.

Related Issues
Resolves #[Issue number to be closed when this PR is merged]

Check List

[ ] New functionality includes testing.

[ ] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[ ] API changes companion pull request created.
[ ] Failing checks are inspected and point to the corresponding known issue(s) (See: Troubleshooting Failing Builds)
[ ] Commits are signed per the DCO using --signoff
[ ] Commit changes are listed out in CHANGELOG.md file (See: Changelog)
[ ] Public documentation issue/PR created

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Please add changelog entry, fix the gradle build and complete the check-list in PR description.

Please add changelog entry, fix the gradle build and complete the check-list in PR description.

Thanks @sachinpkale for approving. I've added the necessary details and addressed your comment.


Rancher Component Framework

Detailed Description
Develop and publish a Rancher Component framework for other products to use during development.
Context
UIs for Rancher have expanded to support more than a single product. As more products are developed under the Rancher umbrella, it would be beneficial to have a component framework to share across different products. For example, Rancher Desktop currently duplicates components by copying them over from Rancher Dashboard; it would be beneficial to reduce duplication across products by importing packages to consume components.

Associated epic #5108

Closing this in favor of epic #5108


fix(mysql): fix mysql open area to transfer param to a file

关联研发信息
dev_branch: fix/mysql_open_area_transfer_param/#5662
target_branch: v1.5.0


psd welch calculated incorrectly at frequency=0

Description of the problem
the mne function psd_array_welch uses the scipy.signal.spectrogram function for each segment. The latter function has the option detrend which is set as default to 'constant'. This signifies that the mean is subtracted when calculating the psd, which is most probably not desired and not documented in mne. I do not believe it would make sense as a default behavior. I suggest that the detrend option be set to "False" when calling spectrogram from mne, or at least given a choice.
Steps to reproduce
import mne
import numpy as np
import matplotlib.pyplot as plt
import scipy.fft as sfft

n = 256
x = np.random.rand(n)-0.4 # not centered
fs = 100
psd1, f = mne.time_frequency.psd_array_welch(x, fs, window='boxcar')
dft = sfft.rfft(x)
psd2 = np.abs(dft * np.conjugate(dft)) / n /fs
psd2[1:] *= 2

plt.figure()
plt.plot(f, psd1, label='mne')
plt.plot(f, psd2, label='mine')
plt.legend()
plt.show()

print(psd1[:5])
print(psd2[:5])

output:
[1.50992908e-33 2.62355362e-03 1.04821458e-04 1.20104558e-03
 7.80199506e-04]
[0.03302601 0.00262355 0.00010482 0.00120105 0.0007802 ]

Link to data
No response
Expected results
psd1[0] should be the same as psd2[0]

Actual results
psd1[0] and psd2[0] are different
Additional information
Platform             macOS-13.3.1-arm64-arm-64bit
Python               3.10.11 | packaged by conda-forge | (main, May 10 2023, 19:01:19) [Clang 14.0.6 ]
Executable           /Users/chapon01/Programs/miniconda3/envs/mne/bin/python
CPU                  arm (12 cores)
Memory               32.0 GB
Core
├☑ mne               1.4.0
├☑ numpy             1.23.5 (OpenBLAS 0.3.21 with 12 threads)
├☑ scipy             1.10.1
├☑ matplotlib        3.7.1 (backend=module://matplotlib_inline.backend_inline)
├☑ pooch             1.7.0
└☑ jinja2            3.1.2
Numerical (optional)
├☑ sklearn           1.2.2
├☑ numba             0.56.4
├☑ nibabel           5.1.0
├☑ nilearn           0.10.1
├☑ dipy              1.7.0
├☑ openmeeg          2.5.6
├☑ pandas            2.0.1
└☐ unavailable       cupy
Visualization (optional)
├☑ pyvista           0.39.0 (OpenGL 4.1 Metal - 83.1 via Apple M2 Pro)
├☑ pyvistaqt         0.0.0
├☑ ipyvtklink        0.2.2
├☑ vtk               9.2.6
├☑ qtpy              2.3.1 (PyQt5=5.15.6)
├☑ pyqtgraph         0.13.3
├☑ mne-qt-browser    0.0.0
└☐ unavailable       ipympl
Ecosystem (optional)
└☐ unavailable       mne-bids, mne-nirs, mne-features, mne-connectivity, mne-icalabel, mne-bids-pipeline

Agreed detrend=False is better so DC is actually estimated, PR welcome!

ok, I'll give it a try.
It seems that same issue is present for the multitaper psd calculation, but I am not directly seeing where it is happening in the code.

inside _mt_spectra:
x = x - np.mean(x, axis=-1, keepdims=True)
Not sure why this is there?

It makes me think removal is intentional and required. Maybe we do need to remove it... I'd have to go back to some signals textbook to remember. It's possible that including DC when using a window function (which we do in Welch and use several in MT) is not a good idea.

I can see how there could be some leakage from the DC if ones pads the signal with 0s. But I believe it depends on the usecase. Sometimes one wants to know the DC. Sometimes not. Maybe would make more sense to have it as an option rather than set to removing the mean.

Yeah a remove_dc=False default kwarg makes sense!


feat: add provide function

PR Checklist
Please check if your PR fulfills the following requirements:

[x] The commit message follows our guidelines: https://github.com/NG-ZORRO/ng-zorro-antd/blob/master/CONTRIBUTING.md#commit
[x] Tests for the changes have been added (for bug fixes / features)
[x] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[x] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[x] Documentation content changes
[ ] Application (the showcase website) / infrastructure changes
[ ] Other... Please describe:

What is the current behavior?

Issue Number: #7953
What is the new behavior?
See: #7953
Does this PR introduce a breaking change?
[ ] Yes
[x] No

Other information

I think provideNz or provideNgZorro or provideZorro will be better naming.
Because it does not rule out that we will have such a function signature:
provideNz(
  config,
  withAFeature(),
  withAFeature()
)

In other words, not only configurations can be added here.

I think provideNz or provideNgZorro or provideZorro will be better naming.
Because it does not rule out that we will have such a function signature:
provideNz(
  config,
  withAFeature(),
  withBFeature()
)

In other words, not only configurations can be added here.

This proposal is based on the package dimension.
like:

@angular/common/http

provideHttpClient

@angular/platform-browser

provideClientHydration

@angular/platform-browser/animations

provideAnimations
provideNoopAnimations
API List more...

@simplejason 你好，麻烦看下这个PR是否可以合入。

目前有可能保留原有的使用方式但标注 deprecated 吗？看上去用户升级后会出现 break change

目前有可能保留原有的使用方式但标注 deprecated 吗？看上去用户升级后会出现 break change

现在只是提供了一系列provide的function，原使用方式目前不影响，provide的函数是为了日后取消了module而做准备

这里不会出现 breaking change，我们应该移除这个 label 😊

@MunMunMiao  provide function 和原来的使用方式是等价的，我觉得 ng-add schematic 现在应该可以默认生成 provide function 形式的代码了


Support base.split()/unsplit()

Ref:
https://r-coder.com/split-r/

Should use group_by/group_split instead.


What are routes inside config

I am trying to clear up the package and am not sure if I can safely remove routes, it seems that they are specified there for database purposes and I am not using database. I wanted to find out what their purpose is and how they work?

the routes in server are used to fetch data and give it to the react components - so if you have no db they can be removed.
Easiest thing to start with is to comment out the routesConfig(app) line in server/index.js (there is a comment about exactly this). You may also need to make some changes to server.jsx since it currently gets data from the routes
Please re-open or comment if you need any more help


WebException: The request was aborted: Could not create SSL/TLS secure channel.

Apparently there are some issues when trying to access websites with https. My code is:
var engine = EngineBuilder.New()
                     .UseJint()// Enable JavaScripts execution.
                     .Build(); // Builds the Optimus engine.

//Request the web page.
var page = await engine.OpenUrl(https://www.mysite.com);

When executing OpenUrl() I get the exception "WebException: The request was aborted: Could not create SSL/TLS secure channel."
Even if I add the lines
ServicePointManager.Expect100Continue = true;
ServicePointManager.SecurityProtocol = SecurityProtocolType.Tls12;

Then I get a deadlock
Any thoughts or workarounds for this?

Hm. What is your OS?

Oh I forgot to mention, I’m using VS2017 in W10 x64

You should configure httpClientHandler with own certificate validation method:
var engine = EngineBuilder.New()
	.ConfigureResourceProvider(res => res.Http(h => h.ConfigureClientHandler(c =>
		c.ServerCertificateCustomValidationCallback = (a1, a2, a3, a4) => true)))
	.Build();
`


Rename 'id' parameter into 'commentId' for the '/comments' endpoint

Initially suggested by @aslakknutsen during the review of https://github.com/almighty/almighty-core/pull/716 but apparently lost when I rebased the commits: replace the id parameter into commentId in the comments endpoint defined in https://github.com/almighty/almighty-core/blob/master/design/comments.go#L98

thanks, @kwk !


Feature to add Budget Goals

Feature to add Budget Goals

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:
:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
see 1 file with indirect coverage changes

🚨 Try these New Features:

Flaky Tests Detection - Detect and resolve failed and flaky tests
JS Bundle Analysis - Avoid shipping oversized bundles


elasticdump not quite working

Hey there!
I tried installing elasticdump and it was fine after the first line, but wouldn't let me run './bin/elasticdump/'. I tried running 'elasticdump --all=true --input=...etc' but it said 'elasticdump: command not found'. Is there something I could have done wrong?

Can you share your solution for future readers?

Whoops, sorry, yup, will do now:
I ran the line 'ln -s /usr/bin/nodejs /usr/bin/node', as I am using a linux machine, so I think running this should make the shell script work. then i realised that whenever I wanted to executing something like 'elasticdump --all=true --input=...etc', i have to replace 'elasticdump' with './bin/elasticdump/', so that it would work! Hope this helps!


Combine Volunteer Profile

The program interest section of the volunteer profile needs to be combined with the table that shows a volunteer's upcoming events.

 CELTS #81: Combine Volunteer Profile


Change Github action example to always log out

If logging out is a necessary step, then it follows that we should always log out regardless of whether the previous steps were successful or not.

@jorinschramm-truesec : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Thanks @jorinschramm-truesec! That's a great point.

#sign-off

#sign-off


[5.4] Eloquent Collection keyBy() consistency

Eloquent returns it's own wrapper on top of Base Collection, which leads to inconsistent behaviour.
Role::find(1)->fields->keyBy('name')->only('phone')
Returns empty collection because only() calls getDictionary() which covers previously key'd collection with ones returned from model getKey().
This change revokes base collection in Illuminate\Database\Eloquent\Collection::keyBy so the prior has sense with future only(), except() and others calls.

It should definitely not return a base collection. I think you are wanting it to behave that way because you are using a collection as a wrapper around a single object structure. More details about ->fields would give us more info though.

@taylorotwell fields is a has many relation objects collection - eloquent collection.
to achieve Role::find(1)->fields->keyBy('name')->only('phone') now I need to first wrap fields using collect() helper function and use keyBy, only from there.

calling keyBy on Eloquent Collection has no sense, because it will always use getDictionary().

Can you please explain in more detail? What calls getDictionary?

Current implementation returns unexpected empty collections because of it.

Because of what?

keyBy on eloquent collection returns eloquent collection, then if I call only() on eloquent collection , it will use it's concrete implementation which will eventualy, return $dictionary = Arr::only($this->getDictionary(), $keys);, where getDictionary() creates newly created array where keys are no longer ones created using keyBy() but ones that are returned by Model getKey().
Because it returns items from collection based not on keys it has, but always on primary keys (resolved by getKey on model).
Role::find(1)->fields->keyBy('name') returns eloquent collection indexed by value from name attribute. Next calling ->only('Joseph') returns empty collection because keys are primary keys, not name values.
Uff

@aambrozkiewicz if you want to call only on it this way, you should call toBase yourself:
Role::find(1)->fields->keyBy('name')->toBase()->only(...);

It makes no sense to change what keyBy returns.

@JosephSilber yes, that's what I need but wasn't obvious for me at first. Thanks!


[Easter Egg] Fixed Alert.js

Removed Line 26 in https://github.com/appwrite/todo-with-react/blob/main/src/pages/Alert/Alert.js as shown in the image below

@uravgkarthik  Thanks for your contributions, reach out to us on Discord to claim your swag!


Provide the full CMSSW exit code from the job report (if possible)

We currently use condor_chirp to report the Linux-provided exit code from CMSSW:
https://github.com/dmwm/WMCore/blob/0615bd977edd4ad103db7ea24f461aa794c9a104/src/python/WMCore/WMSpec/Steps/Executors/CMSSW.py#L236
However, this is an 8-bit exit code: CMSSW's framework job report provides a more detailed exit code.
If the exit code is non-zero, the framework job report exists and is parseable, we should provide the error code from CMSSW instead of the one from Linux.

Note that if we have a non-zero exit code and the framework job report parsing throws an exception, then we should suppress the exception and utilize the Linux exit code instead when constructing the CmsRunFailure WMExecutionFailure.

++1 for suppressing that exception which always makes it look WMCore failed when it was simply the user application which did it !


Adding Jean-Marc Meessen as co-maintainer to pipeline-metadata-utils

Description
In order to properly help GSoC mentor to workout the release process for this component, a temporary access as maintainer is required to https://github.com/jenkins-infra/pipeline-metadata-utils.
CC: @kwhetstone @abayer
Submitter checklist for adding or changing permissions

Always

[x] Add link to plugin/component Git repository in description above

When adding new uploaders (this includes newly created permissions files)

[x] Make sure to @mention an existing maintainer to confirm the permissions request, if applicable
[x] Use the Jenkins community (LDAP) account name in the YAML file, not the GitHub account name
[ ] Make sure to @mention the users being added so their GitHub account names are known if they require GitHub merge access (see below).
[x] All newly added users have logged in to Artifactory at least once

Reviewer checklist (not for requesters!)

[ ] Check this if newly added person also needs to be given merge permission to the GitHub repo (please @ the people/person with their GitHub username in this issue as well). If needed, it can be done using an IRC Bot command
[ ] Check that the $pluginId Developers team has Admin permissions while granting the access.
[ ] In the case of plugin adoption, ensure that the Jenkins Jira default assignee is either removed or changed to the new maintainer.
[ ] If security contacts are changed (this includes add/remove), ping the security officer (currently @Wadeck) in this pull request. If an email contact is changed, wait for approval from the security officer.

There are IRC Bot commands for it

@abayer is the last one to release the plugin component, so I will proceed with the merge.
@jmMeessen note that this is a jenkins-infra repository, so Jenkins GitHub admins or the infra bot cannot add you to the team with permissions. You might have to ask @dduportal to get the access


App does not appear to work on 14" Macbook Pro

I start the app but there are no changes to the Menu bar. Does this work on the laptops with smaller screens?

You need to manually move the icons on the left of the | (bar) icon. It works on my MacBook Pro 14 M2 Pro exactly as on my MacBook Air M1

Probably the arrow icon is under the notch, change the resolution to be able to move it to the desired place.

I am facing the same issue (moved to a 14" MBP with notch just 2 days ago). Can the design be changed to open a drawer/bar below the main menu bar? That would solve the notch issue

me too

Doesn’t work on my 14' MBP too. I can’t find the vertical line to set icons to hide

Doesn’t work on my 14' MBP too.

Same here. (MBPro 14", Ventura 13.4.) I can see the caret icon (<) but there is no vertical line/pipe icon (|).

Same here. (MBPro 14", Ventura 13.4.) I can see the caret icon (<) but there is no vertical line/pipe icon (|).

Same for me on a 14 inch MBP. The caret is showing, but the pipe icon is not there. I have removed all other icons, but it is still not showing.

Same here. MBP 14", Ventura 13.5. Can't see the vertical line icons.

Same on MBA M2 13", Ventura 13.4.1. Missing vertical line.

I also had issues with a Macbook Pro 14" (M3 Pro). I was toggling with the caret and using the command (⌘) to move icons, but nothing seemed to move the needle as I could never see the vertical pipe. I was just about to uninstall the tool when I saw the note about right-clicking the caret in the menu. That was the missing piece!
Thank you @franciscolourenco!


Not letting me create new files with my default template

I just noticed that when this plugin is enabled then if I am creating new notes either by typing in the quick switcher to create a new note or by clicking on the link so when I am creating new note it is not creating it with my own default template of new note creation. It is only creating new note with "view-count" field in the property. When I disable this plugin then my new notes are creating with default template. Please resolve this issue.

Interesting behavior. What plugin are you using for templates? The core
plugin from Obsidian or something like templater?
On Fri, Apr 19, 2024, 7:23 PM Gourav1910 @.***> wrote:

I just noticed that when this plugin is enabled then if I am creating new
notes either by typing in the quick switcher to create a new note or by
clicking on the link so when I am creating new note it is not creating it
with my own default template of new note creation. It is only creating new
note with "view-count" field in the property. When I disable this plugin
then my new notes are creating with default template. Please resolve this
issue.
—
Reply to this email directly, view it on GitHub
https://github.com/trey-wallis/obsidian-view-count/issues/10, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AJTQYWZSKRPAAU7DOJBDJUTY6G7S3AVCNFSM6AAAAABGQBVCPOVHI2DSMVQWIX3LMV43ASLTON2WKOZSGI2TIMZQHA3DQMA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Also could you give me more info
What operating system
what version of Obsidian
What version of the view count plugin
On Fri, Apr 19, 2024, 7:30 PM Trey Wallis @.***> wrote:

Interesting behavior. What plugin are you using for templates? The core
plugin from Obsidian or something like templater?
On Fri, Apr 19, 2024, 7:23 PM Gourav1910 @.***> wrote:

I just noticed that when this plugin is enabled then if I am creating new
notes either by typing in the quick switcher to create a new note or by
clicking on the link so when I am creating new note it is not creating it
with my own default template of new note creation. It is only creating new
note with "view-count" field in the property. When I disable this plugin
then my new notes are creating with default template. Please resolve this
issue.
—
Reply to this email directly, view it on GitHub
https://github.com/trey-wallis/obsidian-view-count/issues/10, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AJTQYWZSKRPAAU7DOJBDJUTY6G7S3AVCNFSM6AAAAABGQBVCPOVHI2DSMVQWIX3LMV43ASLTON2WKOZSGI2TIMZQHA3DQMA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Also could you give me more info What operating system what version of Obsidian What version of the view count plugin
…
On Fri, Apr 19, 2024, 7:30 PM Trey Wallis @.> wrote: Interesting behavior. What plugin are you using for templates? The core plugin from Obsidian or something like templater? On Fri, Apr 19, 2024, 7:23 PM Gourav1910 @.> wrote: > I just noticed that when this plugin is enabled then if I am creating new > notes either by typing in the quick switcher to create a new note or by > clicking on the link so when I am creating new note it is not creating it > with my own default template of new note creation. It is only creating new > note with "view-count" field in the property. When I disable this plugin > then my new notes are creating with default template. Please resolve this > issue. > > — > Reply to this email directly, view it on GitHub > <#10>, or > unsubscribe > https://github.com/notifications/unsubscribe-auth/AJTQYWZSKRPAAU7DOJBDJUTY6G7S3AVCNFSM6AAAAABGQBVCPOVHI2DSMVQWIX3LMV43ASLTON2WKOZSGI2TIMZQHA3DQMA > . > You are receiving this because you are subscribed to this thread.Message > ID: @.***> >

Operating system is windows 11. And also it is creating the same problem in android app. I checked at both places when I disable the plugin then the note creation process in inputting my default template.
Obsidian version is the latest version i.e. 1.5.12
The version of view could plugin is 1.2.0

Also could you give me more info What operating system what version of Obsidian What version of the view count plugin

Operating system is windows 11. And also it is creating the same problem in android app. I checked at both places when I disable the plugin then the note creation process in inputting my default template.
Obsidian version is the latest version i.e. 1.5.12
The version of view could plugin is 1.2.0

Interesting behavior. What plugin are you using for templates? The core plugin from Obsidian or something like templater?

I am using templater plugin.

@Gourav1910 Could you post a video on this behavior? I am not having issues with this. Please see this video as a reference.
https://github.com/trey-wallis/obsidian-view-count/assets/40307803/986d9075-84ac-4ac3-940c-e31a2ea9d090

In the video, it shows that templater is adding frontmatter and markdown note content and that the view count is successfully appended without clearing anything.
In your comment you said that you used the quick switcher or clicked on a link and a template was applied. It sounds then that you have a default template that is being applied. How do you configure the plugin to do that?

In the video, it shows that templater adds frontmatter and content and the view count is successfully appended without clearing anything.
In your comment you said that you use the quick switcher or click on a link and a default template is applied. How do you configure the templater plugin to do that?

I saw your video and that's not how I am creating note. So there is an option in obsidian core settings which let us select in which folder every single new note should be created.
(Settings -> Options -> File and links -> Default location for new note -> "Default folder for new note")
Then in templater plugin there is an another option of folder notes in which we can decide in particular folders if we create note then which template it should apply automatically.
So I have my "To start note template" which I have selected to create every single note in "Notes" folder.
Now whenever I create new note using quick switcher or by clicking on any uncreated note link then the obsidian core settings automatically create it in "Notes" folder (because I have chosen it in settings) and because the note is creating in "Notes" folder so the templater plugin settings automatically apply my "To start note template".
But when I am turning view count plugin on then it is skipping or overwriting my default template and generating new note without any template except the "view-count" field in the properties.
And I checked this behaviour both in Android mobile app as well as windows 11 app. At both places when I turn off view count plugin then my template setup is working fine on creating new note using quick switcher or clicking on uncreated link. And when I turn it on then it creates a blank new note and put the view count property field only.

@trey-wallis hi I have little update on the issue. I just tried it again and I found out that sometimes it is creating file with my default template and sometimes it doesn't. I just created five notes. Out of which 2 times it didn't put the template and three times it did put the template. I think there is some issue in the timing of when view count adds it property field. Can you give it some delay so that it don't interfere the process of creating the note and add its property later. (I mean that's what I understood but the issue could be something else as well.)

I tried following your guide, however I don't know how to setup templater to apply a template on note creation. How do I do that?

Then in templater plugin there is an another option of folder notes in which we can decide in particular folders if we create note then which template it should apply automatically.

I tried following your guide, however I don't know how to setup templater to apply a template on note creation. How do I do that?

Then in templater plugin there is an another option of folder notes in which we can decide in particular folders if we create note then which template it should apply automatically.

Here is a video I have recorded. https://drive.google.com/file/d/14SlvJb2vHyER_jdXqrfzb-FNcrWwB3hD/view?usp=drivesdk
Two steps to do that:

First update the setting in obsidian that which is the default place of note creation.
Then go to templater plugin settings and add a folder template. Folder template is an option in templater settings that if we directly creating a note in that particular folders then which template to apply automatically.

Since our all the notes is creating by default in our specific folder (which we have setup in step 1) so templater plugin automatically apply our defined template of that folder in every new note creation. I have shown it all in the linked video.

I was able to reproduce it. Thanks!
That is very weird behavior. Templater is abnormally slow. I think adding a delay is the best option. I will add this option to the settings. Thanks

Please upgrade to v1.2.1 and toggle the Enable Templater delay option in the settings

Let me know if things are working

Let me know if things are working

Yes now it is working fine. I have one more request. As we can know how many times we have opened any particular note with the help of this plugin. Now I want to use that data to list out the notes that are in trending this week. Means which are the top ten notes I am opening more frequently in this week. And it should be dynamically update the list.
I have tried to achieve this using dataview query but there is no way for dataview to know which note is opened most in the past 7 days. It only listing out the most opened note of all time by looking at the view-count property field value.
So is there a way we can tell dataview that these are the notes we have opened the most in past 7 days or past 30 days.

I just opened a new issue. I will close this one. Thanks

Just want to raise this issue and found out that has been solved!
But... My Template is kinda complex and needs wait for 2~3 seconds to apply.. So I'm wondering could you please add an option to set the delay time?

@Moyf Yes, I can add that. I will open this as a new issue.

@Gourav1910 I have updated the setting in 1.2.2. You will need to go to the settings and turn it back on.


Code Quality (part 3)

SonarQube analysis reported 17 issues

 2 critical
 12 major
 3 minor

Watch the comments in this conversation to review them.

SonarQube analysis reported 7 issues

 2 critical
 5 major

Watch the comments in this conversation to review them.


Toggle display of hidden characters on diff page

Time for something new...
I added a toggle switch to the diff page that allows you to show otherwise hidden characters.

CR end-of-line
Spaces
Tabs
Byte-order-mark

Closes https://github.com/abapGit/abapGit/issues/2124
Closes https://github.com/abapGit/abapGit/issues/3676
Improves https://github.com/abapGit/abapGit/issues/2804
Example 1:

After toggle: Shows spaces and CR

Example 2:

After toggle: Shows that local file has BOM, but remote does not

Example 3:

After toggle: Show the remote file contains tabs

It works beautifully with one exception: Lines will not contain any regular spaces and therefore not wrap into the next line automatically. So longer lines might not be visible as you can see here:

Is there a CSS way to force wrapping (inside a TD cell)? Or should we force width of the table and add a horizontal scrollbar?

I guess the scrollbar issue only occurs with "hidden characters" enabled?

Yes. The default without hidden chars remains as is


fix: improve socket messages delivery

This PR completely refactors how messages are excanged between client/server. This new approach should make client more reliable
Fixes #2975

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 184 (0.0%)  changed or added relevant lines in 8 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.1%) to 23.699%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

genereteDocs.ts
0
1
0.0%

lib/ZwaveClient.ts
0
2
0.0%

lib/SocketManager.ts
0
3
0.0%

src/apis/ConfigApis.js
0
8
0.0%

src/lib/logger.js
0
8
0.0%

src/lib/instanceManager.js
0
26
0.0%

src/mixins/InstancesMixin.js
0
31
0.0%

app.ts
0
105
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.1%

Covered Lines:
3818

Relevant Lines:
17015

💛  - Coveralls


Lazy subset shape, MultiPolygon crossing meridian, DataArray in subset_shape.

Pull Request Checklist:

[ ] This PR addresses an already opened issue (for bug fixes / features)

This PR fixes #xyz

[x] Tests for the changes have been added (for bug fixes / features)
[ ] Documentation has been added / updated (for bug fixes / features)
[x] HISTORY.rst has been updated (with summary of main changes)
[ ] bumpversion (minor / major / patch) has been called on this branch
[ ] Tags have been pushed (git push --tags)

What kind of change does this PR introduce?

I  was using data through open_dap and realized that subset_shape wasn't lazy. The last part was doing a dropna() which triggers a download of all the data, not cool.
Then while playing with my fix, I realized that passing a DataArray to the method made it fail. Because of an iteration over ds.data_vars.
Furthermore, I realized that my fix that allowed passing a GeoDataFrame directly to the method was also broken 'cause wrap_lons_and_split_at_greenwich is modifying the thing in place.
At last, this in place thing was discovered because the decorator doesn't accept MultiPolygon geometries. Fixed that too.

Other information:

In order for the tests to pass, I had to modifiy a value in test_subset.py::TestSubsetShape.test_wraps. Why? Mystère et boule de gomme.

Pull Request Test Coverage Report for Build 1583

11 of 15 (73.33%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.1%) to 87.865%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

xclim/subset.py
11
15
73.33%

Totals

Change from base Build 1582:
-0.1%

Covered Lines:
1984

Relevant Lines:
2258

💛  - Coveralls

I'm considering bringing the coverage up on these utilities (and dealing with the crufty CRS handling). Would it be possible to test a few of your new case conditions?


Separate public and private use statements

Recently we discussed a way to separate the public and private import statements to make the code more clear and prevent rustfmt joining them all together.
Separate public exports using a code block and #[rustfmt::skip]. Has the nice advantage of reducing the number of #[doc(inline)] attributes also.

Modules first, as they are part of the project's structure.
Private imports
Public re-exports (using rustfmt::skip to prevent merge)

Use the format
mod xyz;
mod abc;

use ...;

pub use {
    ...,
};

This patch does not introduce any changes to the rendered HTML.

Based on discussion here: https://github.com/rust-bitcoin/rust-bitcoin/discussions/2088


Enable Location Updates issue

Adding the code for location updates to the submit button (btnDateRange) causes the app to hang when the button is pressed.

Hi @diver7,
Can you let us know if you're using the latest version of the app?
As of this message, that is version V2.20200312.1708 (deployed 5pm PST on 3/12):
https://github.com/microsoft/powerapps-tools/commit/a084ebe04d15f506edb032bcc8bda0b7dbead52a
By 'hangs' do you see dots going across the top of the screen?

No sure where to find that version number you stated but I downloaded the package for the app from GitHub on 3/13/2020 at 2:51PM CDT. So I should have the latest.
When you tap the Submit button it turns grey to acknowledge it has been hit but then nothing happens. Nothing on the screen changes and there are no progress indicators.

I found the issue. I was copying the location code from here. I went back and just uncommented the two lines of code in the original app code for the submit button and it worked fine.
            // To implement location, add a comma to the line above and uncomment the lines below for latitude and longitude.
             Latitude: Text(Location.Latitude),
             Longitude: Text(Location.Longitude)

Looks like the article on docs.microsoft.com just needs to be updated with the latest code.


Use https in SVG URL

Use https since that is what is redirected to.

Description

Code sample

This MR should be closed. This goes against the right usage of XML namespaces. See https://stackoverflow.com/questions/30707609/xml-namespace-uri-with-https#30761004 for reference.