tips lag with vim-surround in normal mode

after changing parameter " to `

and move cursor

is "`" valid?
Could you also send the language and LSP you using, minimum vim configure and the code snippet you are using so I can reproduce this issue.

laguage is go, and this is lsp config:
local capabilities = vim.lsp.protocol.make_client_capabilities()
capabilities.textDocument.completion.completionItem.snippetSupport = true
capabilities.textDocument.completion.completionItem.resolveSupport = {
  properties = {
    'documentation',
    'detail',
    'additionalTextEdits',
  }
}

local nvim_lsp = require'lspconfig'

local opts = {
    tools = {
        autoSetHints = true,
        hover_with_actions = true,
        runnables = {
            use_telescope = false
        },
        inlay_hints = {
            show_parameter_hints = false,
            parameter_hints_prefix = "",
            other_hints_prefix = "",
        },
    },
    capabilities = capabilities,

    -- all the opts to send to nvim-lspconfig
    -- these override the defaults set by rust-tools.nvim
    -- see https://github.com/neovim/nvim-lspconfig/blob/master/CONFIG.md#rust_analyzer
    server = {
      on_attach = function(client, bufnr)
        require'aerial'.on_attach(client)
      end
      }, -- rust-analyer options
}
nvim_lsp.gopls.setup{
--	on_attach = on_attach_vim,
  on_attach = function(client, bufnr)
    require'aerial'.on_attach(client)
  end,
	capabilities = capabilities,
	cmd = {"gopls", "serve"},
	settings = {
		gopls = {
			analyses = {
				unusedparams = true,
			},
			staticcheck = true,
		},
	},
}

require "lsp_signature".on_attach({
  bind = true, -- This is mandatory, otherwise border config won't get registered.
  hint_prefix = "",
--  use_lspsaga = true,
  hi_parameter = "Search",
  fix_pos = true,
--  floating_window = false,
  extra_trigger_chars = {"(", ","},
  handler_opts = {
    border = "none",
  }
})
require('rust-tools').setup(opts)

tips can gone away after leaving insert mode, even if  i type the wrong arguments.

Hi, @y1rn
Thanks for your update. I think I know what happened. Would you check if the latest version works for you?

Hi, @y1rn
Thanks for your update. I think I know what happened. Would you check if the latest version works for you?

still lag

Strange, But I can not longer reproduce
My Config
Plug 'neovim/nvim-lspconfig'

Plug 'ray-x/lsp_signature.nvim'

call plug#end()

lua <<EOF
local capabilities = vim.lsp.protocol.make_client_capabilities()
capabilities.textDocument.completion.completionItem.snippetSupport = true
capabilities.textDocument.completion.completionItem.resolveSupport = {
  properties = {
    'documentation',
    'detail',
    'additionalTextEdits',
  }
}

local nvim_lsp = require'lspconfig'

local signature_config = {
  log_path = "/Users/ray.xu/tmp/sig.log",
  debug = true,
  bind = true, -- This is mandatory, otherwise border config won't get registered.
  hint_prefix = "",
  hi_parameter = "Search",
  fix_pos = true,
  extra_trigger_chars = {"(", ","},
  handler_opts = {
    border = "none",
  }
}

nvim_lsp.gopls.setup{
  on_attach = function(client, bufnr)
    require "lsp_signature".on_attach(signature_config)
    -- require'aerial'.on_attach(client)
  end,
  capabilities = capabilities,
  cmd = {"gopls", "serve"},
  settings = {
    gopls = {
      analyses = {
        unusedparams = true,
      },
      staticcheck = true,
    },
  },
}
EOF
set mouse=a
colorscheme evening

And the screen recording:
https://user-images.githubusercontent.com/1681295/126102940-00d99921-05f9-4dad-81f2-fd5e763acbb1.mov

as my mentioned, my case with using vim-surround in normal mode, just click cs"` in normal mode, not in insert mode

as my mentioned, my case with using vim-surround in normal mode, just click cs"` in normal mode, not in insert mode

https://user-images.githubusercontent.com/1681295/178719102-e08a4666-ba94-4b67-b378-ac81ef1405df.mp4
This one should have been fixed for a while. The signature should not be triggered when changing the souranding


[BUG] Wingetui installer  puts shortcut in wrong place

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 tested that this issue has not been fixed in the latest (beta or stable) release.
[X] I have checked the FAQ section for solutions
[X] This issue is about a bug (if it is not, please use the correct template)

Describe your issue
It puts shortcut in "C:\ProgramData\Microsoft\Windows\Start Menu"
instead of C:\ProgramData\Microsoft\Windows\Start Menu\Programs
Steps to reproduce the issue
No response
WingetUI Log
[wigetui log.txt](https://github.com/marticliment/WingetUI/files/13198617/wigetui.log.txt)

Package Managers Logs
[package manager log.txt](https://github.com/marticliment/WingetUI/files/13198618/package.manager.log.txt)

Relevant information
No response
Screenshots and videos

This issue was fixed on a previous commit


The weights of GroundingDINO-L

Thanks a lot for the extraordinary work!
Can you provide the weights of GroundingDINO-L (swin-large) ?

The checkpoint will be provided later due to the company's procedures.

Hi,
I am also eager to test GroundingDINO-L as the reported results are quite amazing.
Any news regarding when the weights are expected to be published?

Great work! I'm also interested in the GroundingDINO-L. Are there any updates on when it will be released? Thank you!

Are there any updates on when it will be released? thanks！

The checkpoint will be provided later due to the company's procedures.

Any updates here?

Any update here ?

Any update here ?

Any update here ?

Are there any updates on when it will be released? thanks！

Any update here ?

Any updates here?


[Issue] azure-sdk bot bot miss registration to winget-pkg repository

[x] Make sure you've installed the latest version using instructions in the wiki

Describe the bug
Developer tool can't upgrade with winget upgrade microsoft.azd .
https://github.com/microsoft/winget-pkgs/blob/master/manifests/m/Microsoft/Azd/1.4.400/Microsoft.Azd.installer.yaml
azure-sdk bot miss registration to winget-pkg manifest. Every version exists same miss.
miss(current)
Installers:
- Architecture: x64
  InstallerUrl: https://github.com/Azure/azure-dev/releases/download/azure-dev-cli_1.4.*3*/azd-windows-amd64.msi

correct
Installers:
- Architecture: x64
  InstallerUrl: https://github.com/Azure/azure-dev/releases/download/azure-dev-cli_1.4.*4*/azd-windows-amd64.msi

winget's installurl must match about PackageVersion's release artifact.
Expected behavior
InstallerUrl's binary must match PackageVersion release.

Hey @kkamegawa, thanks for opening an issue! This is by-design as MSI's versioning behaviors predate SemVer and MSI does not support SemVer. The result is an algorithm for finding the "patch" or "revision" number of the MSI based on a given SemVer. If MSI supported SemVer this would not be necessary.
The algorithm for determining the MSI version given the SemVer version of the tool is expressed here: https://github.com/Azure/azure-dev/blob/main/eng/scripts/Get-MsiVersion.ps1#L48-L53
An example execution shows that 1.4.3 maps to 1.4.400
.\Get-MsiVersion.ps1 -CliVersion 1.4.3
1.4.400

This enables us to release SemVer versions like 1.4.3-beta.1 and get MSI versions which will correctly upgrade over older versions (e.g. 1.4.2) and be superseded by newer versions (e.g. 1.4.3):
.\Get-MsiVersion.ps1 -CliVersion 1.4.2
1.4.300

# Installs over 1.4.2
.\Get-MsiVersion.ps1 -CliVersion 1.4.3-beta.1
1.4.301

# Installs over 1.4.3-beta.1
.\Get-MsiVersion.ps1 -CliVersion 1.4.3
1.4.400

Thanks again for reporting! I'm going to close this issue for now but please re-open if you have more questions.


Uncaught TypeError: vtxdata.defCellAttr.forEach is not a function

I'm getting this in all browsers presently.
Uncaught TypeError: vtxdata.defCellAttr.forEach is not a function
    at initDisplay (vtxclient.js:4715:27)

My setup is extremely simple.  Nginx https -> Enigma BBS ws://
Am I missing something?
Thanks for looking!

Seeing that this is not fixed still almost 2 years later, this might help someone else. These changes may or may not need to be made based on what you want, but it might give a working web terminal, or at least give someplace to start.
Here is what I changed in www/assets/vtx/vtxclient.js
Line 4714: if (Array.isArray(vtxdata.defCellAttr)) {
Line 4734: if (Array.isArray(vtxdata.defCrsrAttr)) {
Then in the vtxdata.js
var vtxdata = {
     sysName: "bbs title",
     telnet: 1,
     autoConnect: 0,
     wsConnect: "ws://<IP_ADDRESS>:8810",
     wsProtocol: "telnet",
     wsDataType: "arraybuffer",
     term: "ansi-bbs",
     codePage: "CP437",
     fontName: "UVGA16",
     fontSize: "24px",
     crtCols: 80,
     crtRows: 25,
     crtHistory: 500,
     xScale: 1,
     initStr: "",
  //   defPageAttr: [],
  //   defCrsrAttr: ['thick', 'horizontal'],
  //   defCellAttr: [],
     defCellFG: 0xFF,
  //   defCellBG: 0x00,
  //   defPageBG: 0x00,
  //   defPageBorder: 232,
 };


Use cell re-use based disposal bags for bindings on cells

Instead of de-init based ones - fixes #591
Gonna take a quick stab at  tests now, if in a few hours there are none, that's probably cause I moved back to eigen auctions work.

Interesting! The fix works for sure, but I'm curious why the problem is there in the first place. I may take a look at the flatMapTo (which is of my own invention, so).

:+1:

:raised_hands:


TSAN error triggered by std::string race in FakeRawConnection::write in builds from Centos docker container

This is the same as #944. This is still happening in builds in the centos docker container.
It appears this is an issue with the centos devtoolset toolchain.
See:
https://bugzilla.redhat.com/show_bug.cgi?id=1546704
https://gcc.gnu.org/onlinedocs/libstdc++/manual/using_dual_abi.html

I've found no way to fix this without recompiling the standard library. Closing.

This is supposed to be fixed in CentOS 8, when it is released.

@ggreenway if possible can you specify how you fixed it in case anyone else wants to also fix it?

I stopped building on CentOS entirely.


Add webpage + add panelbear analytics

Minor commits:

Adding my webpage to the list of academic pages
Adding panelbear analytics option. Changes to the config file (analytics id entry, enable entry). Script added to head.html.

Thanks for your contribution!

Thanks for your contribution!


Text is dark while typing in insert mode

Hello, i'm using the neovim extension on vscode. When i'm typing out code in insert mode, the text is dark, and only becomes visible once i exit insert mode. Here's a short video demonstrating what i mean:
https://user-images.githubusercontent.com/75761281/149023714-9dc0c207-26d7-42c4-9380-cba20d80606c.mp4
You can see how this can be frustrating, and makes it difficult to code with the neovim extension
Operating system: Fedora Linux
Neovim version: NVIM v0.6.1

Hello, having the same issue.
OS: Garuda Linux
Neovim version: NVIM v0.6.1

@querty-300 are you using an init.vim file? When I removed some stuff from it, the issue disappeared... I think it was somethign with Coc...

Yes, try with empty init.vim


Adds support to adoCodespacesAuth.tenantID extension setting for conf…

Provides the 'adoCodespacesAuth.tenantID' extension setting to configure the Entra ID tenant to sign in to.
Address #14

Mostly looks good functionally. Just some minor code changes and we should be good to go.
Can you also update the package.json version (probably a minor bump to 1.2.0) and add a line in the README.md to explain this configuration? Thanks!

Thanks @liguori. I will do a quick test from my side and then release it.

When testing this, it seems to not work when requesting token. It authenticates fine initially, but when you request a token with the command ado-auth-helper get-access-token or with git fetch it just hangs.
There are two reasons from what I can see. The PR breaks the default assumption that getAccessToken when called without scopes uses the ADO scope. So when called from socket server handler it doesn't get the default scope now. Another reason might be because we need to pass the tenant in the calls to getAccessToken function when it's called from socket server handler.
@liguori can you create a followup PR. Maybe move the logic for getting tenant from configuration to the getAccessToken function and also fix the default scope assumption.
You can test it by updating the version number, running yarn vsce:package, download the generated vsix bundle and then open a codespace, search for "vsix" and install the local download

Then signout, add the tenant id config and then signin to microsoft account. Run ado-auth-helper get-access-token to test if it works.

It's late for me today so I am gonna go, but I can also take a stab at fixing this tomorrow if you want.


Q: How does GitVersion handle proxy servers?

I've got a problem with connecting from a single PC - GitVersion is trying to connect via a proxy server.
The issue I have is in TeamCity, so I've added a TEAMCITY_VERSION env variable to trick running normally from the command line.  The code fails in GitHelper.NormalizeGitDirectory, with a 407 (proxy authentication required):
An unexpected error occurred:
LibGit2Sharp.LibGit2SharpException: Request failed with status code: 407
   at LibGit2Sharp.Core.Ensure.HandleError(Int32 result)
   at LibGit2Sharp.Core.Proxy.git_remote_fetch(RemoteSafeHandle remote, Signatur
e signature, String logMessage)
   at LibGit2Sharp.Network.DoFetch(RemoteSafeHandle remoteHandle, FetchOptions o
ptions, Signature signature, String logMessage)
   at LibGit2Sharp.Network.Fetch(Remote remote, FetchOptions options, Signature
signature, String logMessage)
   at GitVersion.GitHelper.NormalizeGitDirectory(String gitDirectory, Authentica
tion authentication) in c:\BuildAgent\work\68ea6bfb44e5afb\GitVersionCore\BuildS
ervers\GitHelper.cs:line 26

I've intercepted gitversion.exe, and it's connecting to an IP of a genuine local proxy, but I can't find any occurence in the registry, in Internet Options or in any of git's config (system, global or local).
I created a very simple console app with the following libgit2sharp code to see if I could replicate the issue:
var dir = Environment.CurrentDirectory;

Console.Write("Fetching origin...");

using (var repo = new Repository(dir))
{
    var remote = repo.Network.Remotes.First();
    repo.Network.Fetch(remote);
}

Console.WriteLine("done.");

And... it works fine (it connects to github.com direct, no proxy).
Is there something obvious I'm missing?  Any suggestions appreciated.  I've been going over GitVersion's source for a while, but I can't see what would cause the difference between my console app and this!

Solved, so I'll close.  In case anyone needs for future reference:
Strangely, I could replicate the issue if you changed my console app from the default .NET Framework 4.5.1 to match GitVersion's .NET Framework 4.
Some more digging, and libgit2 uses WinHTTP for its proxy configuration... and I found from when time began a default proxy set, which can be viewed using:
netsh winhttp show proxy
and, cruicially, cleared using:
netsh winhttp reset proxy


firejail: Disable X sandboxing

Mitigates CVE-2017-5180 (local root exploit).
@hashbang/administrators Review, merge and deploy needed ASAP, this is an ongoing security issue.

Deployed to production.
Still running in an issue on to1, so the diff was applied manually there.


Include vout when copying transaction ID from coin selection

Fixes #432
I think it makes sense to just add the vout to the existing function because I can't imagine a situation where a user in the coin selection dialog would want just the transaction ID rather than the specific outpoint, and they can just delete it from the end anyway.

ACK c4ea11c919a49322578db549f31c6e68a2560540
FWIW we used to append the vout to the id when copying from the normal transaction list as well, a long time ago, but it always got a lot of flak from confused users. But I think it's fine here, coin control is an advanced feature anyway.

Thanks for the reviews, will address the nits shortly.

Would it be beneficial to apply this functionality to the context menu action under the transaction tab?

IMO no, because unlike the coin selection which deals with UTXOs, transactions can have multiple outputs and most non-technical users don't understand the idea of outputs anyway, so it would just be confusing.

Addressed nits

Concept ACK.


Removed section on using gaetway for spoke-to-spoke connectivity

@MikeWasson this fixes the issue on the email you forwarded me

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/reference-architectures/hybrid-networking/hub-spoke.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Add authentication header & remove unnecessary survey calls

Motivation and Context

Call to action service was being done without an authentication header
resulting in 401s on event update. This pr adds the header into the
request. Also a number of unnecessary calls are being made to survey
service, changed the logic order so the calls are not being made
unless it is needed.
What has changed

Added authentication header to action service call
Changed order for calling survey service to after the event tag check.
How to test?

mvn test
ATs
Links

@benjefferies forgot to update the unit tests to check that the HTTPEntity is being passed in, all done now 👍


Fix set-default command and remove useless populator client

It is now possible to set a default cluster using its name or its identifier:

This was done :

to help people to set a default cluster if a name is too complex
to avoid weird situation where a cluster doesn't have a name

Done ! Do you think we can merge this pull request ?

Yes, this can be merged! :)


Password will not Rotate and Name does not Update in CloudLaps after Re-image and Rename

After re-imaging and renaming a computer the computer name will not update in Cloudlaps and password will not rotate.  CloudLaps seems to hold on to the old computer name.  The computer was removed from AAD and Intune before being re-imaged and renamed.  Does anyone know where CloudLaps stores the computer names so I can remove the old one?

Stores in the key vault. You'll have to give yourself permissions to be able to view those and then purge old ones.

I think I may be running into this too. I do a lot of re-imaging using the same device/serial as part of testing deployment scenarios. But I have seen it work sometimes so that makes me wonder if there's something else causing the issue. I saw another post about the rotation frequency set in the function app - could it be that if I'm re-imaging multiple times daily?
Event ID 14 - CloudLAPS: Forbidden, password was not allowed to be updated.
Otherwise, working great! what a cool tool.

OK, confirmed, in my case, it was not able to rotate the password for that re-imaged device because it had not been enough days since the last time it rotated for that same device. CloudLAPS uses the serial number to store the password in the keyvault so from CloudLAPS perspective, it was the same exact device even though it had been re-imaged. I set the "UpdateFrequencyDays" setting in the function app to "0" and it was able to rotate.
@rodawing

Hi
We also just discovered this and want to try and work out how to clear the timestamp or clear the serial number from the keyvault
What we had with the old LAPS is when imaging a device and the LAPS install script was executed. We added a powershell command to clear the Active Directory timestamp for that computer name
This allowed LAPS to update the password instantly
So a newly imaged device would have its LAPS account created and password updated
Does anyone know how to do similar with the keyvault in Azure AD ?

Did you try changing the function app UpdateFrequencyDays to 0? I haven't had any issues after changing it to 0. The password changes every time the baseline runs but that's OK, I'd be looking up the password and copying and pasting it anyway so it's no extra work. Even devices that I re-image over the top get a LAPS password assigned almost immediately since baselines run soon after workplace join/autopilot. Not sure what else you might be running into. If that doesn't work, can you describe the exact problem you're running into?

Did you try changing the function app Confirm-SecureBootUEFI to 0? I haven't had any issues after changing it to 0. The password changes every time the baseline runs but that's OK, I'd be looking up the password and copying and pasting it anyway so it's no extra work. Even devices that I re-image over the top get a LAPS password assigned almost immediately since baselines run soon after workplace join/autopilot. Not sure what else you might be running into. If that doesn't work, can you describe the exact problem you're running into?

Hi Thanks for the reply
No I havent yet
This might work ok with our on prem machines (domain joined) as we use a configuration baseline that is set to run once a day ( so the same frequency as I set in the function app )
But not sure how this will work with our Azure-Joined machines. Would it stop updating completely if its set to 0 on those azure-joined devices?
Worth a test though
Wanted a more permanent and automated solution to the imaging issue as we re-image many machines per day

It never even occurred to me that this would also work for on-prem, AD-joined devices. I think most people are using this with Intune, do you have Intune + AAD or just AAD? Or SCCM? Or anything that can run some kind of script regularly on an AAD-joined device?
If you set that setting to 0, it just updates the password in the KV every time the script runs. That solved my problem because I also re-image constantly throughout the day when I'm testing things and I was running into an error than prevented the password from being updated in the KV for a serial number that was updated less than X days ago.

It never even occurred to me that this would also work for on-prem, AD-joined devices. I think most people are using this with Intune, do you have Intune + AAD or just AAD? Or SCCM? Or anything that can run some kind of script regularly on an AAD-joined device?
If you set that setting to 0, it just updates the password in the KV every time the script runs. That solved my problem because I also re-image constantly throughout the day when I'm testing things and I was running into an error than prevented the password from being updated in the KV for a serial number that was updated less than X days ago.

CloudLAPS was that fantastic to us we wanted to make it work with our on prem environment and evetually let it replace old LAPS
We have so many users working at home now since COVID hit and hardly any of the old LAPS passwords are updating
so we combined both detection and remediation scripts and threw it in SCCM configuration item / baseline to run once a day
It works ok. But just a few of issues I would like to sort out. Like the newly imaged machine issue I am writing here about
We have two environments ( currently in testing and development is our AAD ) and we use SCCM for our older environment
We use the configuration basline via SCCM to execute the CloudLAPS scripts once a day to our old environment
Changing the update frequency to 0 will work ok with our old domain joined environment as we use SCCM to run the script on a daily basis
But unsure how it will trigger on our AAD environment

Are you moving to AAD and trying to leave AD/SCCM behind with no connectivity between the two? SCCM can work with AAD and for internet clients, could you use an SCCM cloud management gateway? (https://docs.microsoft.com/en-us/mem/configmgr/core/clients/manage/cmg/overview) Or better yet, could you just use Intune? I have a similar problem, I have Intune but Intune doesn't manage servers so I'm not sure how I could do LAPS (or any app deploy or running scripts/config) on straight AAD-joined server OS. :/

We have two environments ( currently in testing and development is our AAD ) and we use SCCM for our older environment
We use the configuration basline via SCCM to execute the CloudLAPS scripts once a day to our old environment

so where does the password go? Does it go to same portal? TBH with you I never though about this from on-prem perspective before.

We have two environments ( currently in testing and development is our AAD ) and we use SCCM for our older environment
We use the configuration basline via SCCM to execute the CloudLAPS scripts once a day to our old environment

so where does the password go? Does it go to same portal? TBH with you I never though about this from on-prem perspective before.

It goes to the same loction to the Azure Function app keyvault. It behaves the same way except we get SCCM to deploy execute the script

Are you moving to AAD and trying to leave AD/SCCM behind with no connectivity between the two? SCCM can work with AAD and for internet clients, could you use an SCCM cloud management gateway? (https://docs.microsoft.com/en-us/mem/configmgr/core/clients/manage/cmg/overview) Or better yet, could you just use Intune? I have a similar problem, I have Intune but Intune doesn't manage servers so I'm not sure how I could do LAPS (or any app deploy or running scripts/config) on straight AAD-joined server OS. :/

I just set the u pdate frequency to 0. But I think this will cause the Azure AD devices to never rotate the password ever
On https://msendpointmgr.com/cloudlaps/#12-function-app-documentation
it states the following:
UpdateFrequencyDays
Password rotation (described more in detail in the solution overview documentation above) will only be allowed if the most recent password rotation occurred in the past, more than the specified amount of days configured in this application setting. The default value is 3, which for instance only allows the password rotation process from a device to happen every 3 days. Change this value to a desired amount of days that meets the organization requirements. A value of 0 is not supported.**_


refactor: allow service to abort on some errors

Signed-off-by: Gregory Hill<EMAIL_ADDRESS>See the comment on this PR: https://github.com/interlay/interbtc-clients/pull/389

On a side note, because we removed the checks in service to always restart on any error we will be stuck in a loop now if the wrong Bitcoin network is configured. We should distinguish between recoverable and unrecoverable errors for which this is the latter.

Specifies Abort and Retry variants on the service::Error enum to allow the consumer to propagate a hard-error to the ConnectionManager.
On a related note, I'm not sure there is any justification for the service logic to have its own crate since it is only ever consumed by the vault binary. I wonder if we should fold it in?

Would it make sense to use backoff::Error?

On a related note, I'm not sure there is any justification for the service logic to have its own crate since it is only ever consumed by the vault binary. I wonder if we should fold it in?

Yea makes sense to me, but it'd have pretty low prio

Would it make sense to use backoff::Error?

I would like to but then we would have backoff::Error<service::Error<Error>>, not worth it right now but definitely something to consider when we fold the service crate into the vault bin.

Would it make sense to use backoff::Error?

I would like to but then we would have backoff::Error<service::Error<Error>>, not worth it right now but definitely something to consider when we fold the service crate into the vault bin.

Does the service::Error still need to be generic when we have backoff::Error though? You only made it generic in this pr. backof::Error<service::Error> doesn't look bad to me


error on defferedMountedTo, _ is not defined

Hi,
I have an issue when map is mounted :
deferredMountedTo: function deferredMountedTo(parent) {
this.$tileLayer.addTo(parent);
=> _.forEach(this.$children, function (child) {
child.deferredMountedTo(that);
});
}

I don't understand why lodash is not present.

I do this dirty fix in my app
 window._ = require('lodash');

Hi,
Thanks for having raised that issue.
The problem was that lodash was not requiered correctly on all component so you could end up having lodash not defined when using some components only.
Lodash was only used for few forEach so I removed it as a dependancie

Thanks


[bugfix] add missing parameters.

Summary

Add missing parameters. (slug, order)
If you want to search ainft object by slug in order by creation time:

const { ainftObjects } = await ainft.nft.searchAinftObjects({ slug: 'test-slug', order: 'asc' });

Thank you for quick review! @platfowner now merging...


TreeError [CommentsTree] Data tree node not found: [object Object]

Does this issue occur when all extensions are disabled?: Yes (-ish; I have a single extension populating some comments for reproducing this)

VS Code Version: 1.75.0
OS Version: macOS Ventura 13.2

Steps to Reproduce:

Populate Comments API with some comment threads.
Collapse the comments shown for the first file in the Comments panel.
Move focus to the overall comments tree view by clicking somewhere in the empty space below.
Hide the Comments panel e.g. by clicking on Problems.
Open the Comments panel again.

  ERR TreeError [CommentsTree] Data tree node not found: [object Object]: Error: TreeError [CommentsTree] Data tree node not found: [object Object]
    at CommentsList.getDataNode (vscode-file://vscode-app/Users/hermannloose/GitHub/vscode/out/vs/base/browser/ui/tree/asyncDataTree.js:506:23)
    at vscode-file://vscode-app/Users/hermannloose/GitHub/vscode/out/vs/base/browser/ui/tree/asyncDataTree.js:461:50
    at Array.map (<anonymous>)
    at CommentsList.setFocus (vscode-file://vscode-app/Users/hermannloose/GitHub/vscode/out/vs/base/browser/ui/tree/asyncDataTree.js:461:36)
    at CommentsPanel.refresh (vscode-file://vscode-app/Users/hermannloose/GitHub/vscode/out/vs/workbench/contrib/comments/browser/commentsView.js:206:35)

Note that this issue does not happen if the first comment on the first file remains visible (it will get focused) or if focus remains on some file row in the Comments panel before switching away and back (focus will remain on that row).

Thanks for creating this issue! It looks like you may be using an old version of VS Code, the latest stable release is 1.74.3. Please try upgrading to the latest version and checking whether this issue remains.
Happy Coding!

@hermannloose Have you tried on the very latest 1.75? I'm not able to repro the issue and I made a change that may have fixed this last week: https://github.com/microsoft/vscode/issues/171711

@alexr00 This was on a fresh sync of main from today: da0309f2514ac22ac82dcd080cfc764fe2a1f434

Any idea what you're doing differently?

Here is a short overview of the steps. Ignore the "changes to thread state" logging, that's noise from my test extension, it doesn't do anything but log inside a setInterval().

Right now it seems to robustly reproduce also via:

Collapse All in Comments panel.
Switch to Problems.
Switch back to Comments panel.

i.e. without requiring clicking in the empty space to change focus. But then I recall that wasn't always happening consistently earlier, I think.
I also just checked whether it depends on whether the first file in the list is the one that's currently open in the editor, but I get the same error whether it's open or not.

I tried running out of source to get the to the same commit as you and I can't repro the issue there either. I can see where it would happen though. This gets called every time the Comments view goes from not-visible to visible:
https://github.com/microsoft/vscode/blob/4db912c6c15c667db70aaeb8dfd082e3a0c6a30e/src/vs/workbench/contrib/comments/browser/commentsView.ts#L394-L400
But I would only expect this error to occur if the tree node we're trying to focus has been disposed, which I don't think it should be after the change for https://github.com/microsoft/vscode/issues/171711.
Can you confirm whether you have this commit in particular? https://github.com/microsoft/vscode/commit/db4258b1b6d127dd33ecfbe3bd3dd8eb16ef2c2d#diff-551bd3c0a95064d0b24d696cd5c1a83f0609ea66de94a7a714b15a5477b85f65

Now I don't trust myself to have been building VS Code correctly before ...
I noticed that it wasn't picking up debug logging that I put in and instead of running yarn and then ./scripts/code.sh I tried running yarn watch and ./scripts/code.sh in another terminal and now I cannot reproduce the problem and some other weird warnings I had been seeing have gone away. 🙈 I probably was holding this all wrong and running some older code still?

👍 thanks for confirming that the issue is resolved!


Update index.md

Why:
Closes:

What's being changed (if available, include any code snippets, screenshots, or gifs):

Check off the following:

[x] I have reviewed my changes in staging (look for the "Automatically generated comment" and click the links in the "Preview" column to view your latest changes).
[x] For content changes, I have completed the self-review checklist.

I have capitalize the word P in profile

I have capitalize P in profile.


Forbidden request status transition from aborted-archived to closed-out

Impact of the bug
ReqMgr2
Describe the bug
While debugging MSRuleCleaner, we have found a strange workflow status transition for:
https://cmsweb.cern.ch/reqmgr2/fetch?rid=cmsunified_ACDC4_Run2017H-v1_LowEGJet_09Aug2019_UL2017_LowPU_200901_093632_7002
which should be forbidden, according to the allowed status transition:
https://github.com/dmwm/WMCore/blob/master/src/python/WMCore/ReqMgr/DataStructs/RequestStatus.py#L55
How to reproduce it
We can try it via RESTful API.
Expected behavior
We need to always validate the status transition and make sure it complies with the allowed status transition defined in the ReqMgr2 DataStructs (no matter how the status transition is triggered, via web UI or RESTful API).
I'm also in contact with Hasan to see why Unified did such transition and whether there is any valid use case for that.
Additional context and error message
logs are likely no longer available.

Just a minor update. I heard back from Hasan and Unified is making this workflow transition with the cascade option, here:
https://github.com/CMSCompOps/WmAgentScripts/blob/master/reqMgrClient.py#L1002
This should help us reproducing this error.

Just a minor update. I heard back from Hasan and Unified is making this workflow transition with the cascade option, here:
https://github.com/CMSCompOps/WmAgentScripts/blob/master/reqMgrClient.py#L1002
This should help us reproducing this error.


Unable to use negative indices in Array.remove

Godot version: 3.2.3

OS/device including version: Windows 10 x 64

Issue description:
When you create an Array and later use Array.remove(index), index must be positive, althouth Arrays can be indexed using negative indices. The debugger message in the minimal reproduction project is: E 0:00:00.895   remove: Index p_index = -1 is out of bounds (size() = 3).  <C++ Source>  ./core/cowdata.h:161 @ remove()   Spatial.gd:7 @ _ready()
Maybe other methods have the same problem.
Steps to reproduce:
Create an Array and try to remove one element using a negative index with Array.remove
Minimal reproduction project:

Github.zip

This needs to be implemented. See also https://github.com/godotengine/godot/pull/37209.


Started work on adding MathPower command, see #367

@JoshuaKGoldberg, I started to work on #367. It seems, I've done everything I found to add a new command. However, tests error at me and I don't know why. Your help is deeply appreciated. Thanks!

Thanks! Definitely missed it, sorry.
However, it still throws the following (though I didn't change the addition operator at all):
D:/Repos/Git/gls/test/ComparisonTestsRunner.ts
ERROR: 84:68  restrict-plus-operands  Operands of '+' operation must either be both strings or both numbers

The error in ComparisonTestsRunner.ts looks like a bug in TSLint. Very odd... You can ignore it, and run gulp src:tsc && gulp test to build+test instead.
Now that you've imported the file in CommandsBagFactory.ts you'll also need to add it to the commands being created in the forContext method. They're in alphabetical order :)

@JoshuaKGoldberg, do I have to change anything else here? Thanks!

@Oire yes - right now the Travis CI build is failing because of an error in the Python test files. You can run gulp locally to verify whether everything's working (see the newly rewritten docs/development.md).
Looking through the raw log from https://travis-ci.org/general-language-syntax/GLS/jobs/286767285, it's failing on test/integration MathPower math power Python because the test case is missing from math import pow.

LGTM; thanks for the contribution!

Thanks a lot! :)


Added typer to module

Closes #3
Added the base of the code that will be needed to implement the CLI.

Adding in a unit test before merging


Write tests for actionCreators/resourceHelpers

Need unit tests for:

addEmptyResource
addResourceFromDataset
chooseURI
newSubject
newPropertiesFromTemplates

Closing, since actionCreators/resourceHelpers are covered by actionCreators/resources tests.


Installer error in a electrion-vue project with yarn add

I'm trying to install this in a electron-vue project using
yarn add diskusage
I'm gotting this error:

error D:\Projetos\ProjetoX\myproject-installer\node_modules\diskusage: Command failed.
Exit code: 1
Command: node-gyp rebuild
Arguments:
Directory: D:\Projetos\ProjetoX\myproject-installer\node_modules\diskusage
Output:
D:\Projetos\ProjetoX\myproject-installer\node_modules\diskusage>if not defined npm_config_node_gyp (node "C:\Program Files\nodejs\node_modules\npm\bin\node-gyp-bin\....\node_modules\node-gyp\bin\node-gyp.js" rebuild )
else (node "" rebuild )
gyp info it worked if it ends with ok
gyp info using<EMAIL_ADDRESS>gyp info using<EMAIL_ADDRESS>| win32 | x64
gyp ERR! configure error
gyp ERR! stack Error: Command failed: C:\Users\pedro.henrique\AppData\Local\Programs\Python\Python37-32\python.EXE -c import sys; print "%s.%s.%s" % sys.version_info[:3];
gyp ERR! stack   File "", line 1
gyp ERR! stack     import sys; print "%s.%s.%s" % sys.version_info[:3];
gyp ERR! stack                                ^
gyp ERR! stack SyntaxError: invalid syntax
gyp ERR! stack
gyp ERR! stack     at ChildProcess.exithandler (child_process.js:295:12)
gyp ERR! stack     at ChildProcess.emit (events.js:203:13)
gyp ERR! stack     at maybeClose (internal/child_process.js:1021:16)
gyp ERR! stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:283:5)
gyp ERR! System Windows_NT 10.0.17134
gyp ERR! command "C:\Program Files\nodejs\node.exe" "C:\Program Files\nodejs\node_modules\npm\node_modules\node-gyp\bin\node-gyp.js" "rebuild"
gyp ERR! cwd D:\Projetos\ProjetoX\myproject-installer\node_modules\diskusage

I'm using:
Windows 10
Node 12.7.0
Any idea what is causing, i already tryed to delete my node_module.
Thx.

This looks like an issue with your Python installation. I have Python 2.7.12 installed on my development machine, which works with node-gyp and node-diskusage, so perhaps try installing a different Python version, or re-installing your current Python version.

I had two versions installed of Phyton
2.5.4 64 Bits and 3.7.4 32 Bits
I removed the 64 bits version and reinstalled the last one.
I'm gotting the same error.

it's an issue with an older version of node-gyp, which is included in npm-lifecycle, which in turn is included in npm.
npm install -g npm should fix it

Closing, as a solution has been provided. Feel free to request a reopen if you're still encountering issues.


Fixes #293: streams.sink.polling.interval is possibly confusing and m…

Fixes #293
Removed all references to streams.sink.polling.interval

@mroiter-larus thank you!


ATS 10.0.0 Crashes with SEGMENTATION FAULT

While i am trying to stream a video content through the ATS I sometimes get a crash relating to this. This might be some issue related to the configuration but the only configuration i am doing here that is
map https://domainofsite.com/ http://ip_of_origin:8080/ @plugin=header_rewrite.so @pparam=/var/test/rules1.conf
rules1.conf

cond %{READ_RESPONSE_HDR_HOOK}
set-status 200
set-header access-control-allow-origin https://domain.site.base
set-header access-control-allow-methods OPTIONS, GET, POST
set-header access-control-allow-credentials true
set-header access-control-allow-headers Authorization, Origin

(gdb) bt
#0  response_get (this=<optimized out>) at /home/nischal/trafficserver/proxy/hdrs/HTTP.h:1388
#1  HttpTransact::HandleCacheOpenReadHit (s=0x2b2559e43ee0) at HttpTransact.cc:2803
#2  0x000000000057a911 in HttpSM::call_transact_and_set_next_state (this=0x2b2559e43de0, f=<optimized out>) at HttpSM.cc:7534
#3  0x000000000058db41 in HttpSM::state_api_callout (this=0x2b2559e43de0, event=<optimized out>, data=<optimized out>) at HttpSM.cc:1620
#4  0x0000000000599f8b in HttpSM::set_next_state (this=0x2b2559e43de0) at HttpSM.cc:7576
#5  0x000000000057d6d5 in HttpSM::do_hostdb_lookup (this=this@entry=0x2b2559e43de0) at HttpSM.cc:4315
#6  0x000000000059a172 in HttpSM::set_next_state (this=0x2b2559e43de0) at HttpSM.cc:7877
#7  0x000000000058db41 in HttpSM::state_api_callout (this=0x2b2559e43de0, event=<optimized out>, data=<optimized out>) at HttpSM.cc:1620
#8  0x0000000000599f8b in HttpSM::set_next_state (this=0x2b2559e43de0) at HttpSM.cc:7576
#9  0x000000000058db41 in HttpSM::state_api_callout (this=0x2b2559e43de0, event=<optimized out>, data=<optimized out>) at HttpSM.cc:1620
#10 0x0000000000599f8b in HttpSM::set_next_state (this=0x2b2559e43de0) at HttpSM.cc:7576
#11 0x00000000005902b3 in HttpSM::state_cache_open_read (this=0x2b2559e43de0, event=1102, data=<optimized out>) at HttpSM.cc:2672
#12 0x0000000000591988 in HttpSM::main_handler (this=0x2b2559e43de0, event=1102, data=0x2b2559e45cb0) at HttpSM.cc:2736
#13 0x00000000006135e3 in handleEvent (data=0x2b2559e45cb0, event=1102, this=0x2b2559e43de0) at /home/nischal/trafficserver/iocore/eventsystem/I_Continuation.h:219
#14 HttpCacheSM::state_cache_open_read (this=0x2b2559e45c48, event=<optimized out>, data=<optimized out>) at HttpCacheSM.cc:118
#15 0x00000000007746cf in handleEvent (data=0x2b2574056c40, event=1102, this=0x2b2559e45c48) at /home/nischal/trafficserver/iocore/eventsystem/I_Continuation.h:219
#16 CacheVC::callcont (this=this@entry=0x2b2574056c40, event=event@entry=1102) at P_CacheInternal.h:650
#17 0x000000000076f45f in CacheVC::openReadStartEarliest (this=0x2b2574056c40) at CacheRead.cc:996
#18 0x000000000074a54d in CacheVC::handleReadDone (this=0x2b2574056c40, event=<optimized out>, e=<optimized out>) at Cache.cc:2289
#19 0x000000000074df89 in AIOCallbackInternal::io_complete (this=0x2b2574056dd0, event=<optimized out>, data=<optimized out>) at /home/nischal/trafficserver/iocore/aio/P_AIO.h:122
#20 0x00000000008a195f in handleEvent (data=0x1c57440, event=1, this=<optimized out>) at I_Continuation.h:219
#21 EThread::process_event (this=0x2b254d6b1010, e=0x1c57440, calling_code=1) at UnixEThread.cc:153
#22 0x00000000008a1f06 in EThread::process_queue (this=this@entry=0x2b254d6b1010, NegativeQueue=NegativeQueue@entry=0x2b254e2f0330, ev_count=ev_count@entry=0x2b254e2f032c,
    nq_count=nq_count@entry=0x2b254e2f0328) at UnixEThread.cc:188
#23 0x00000000008a291d in EThread::execute_regular (this=this@entry=0x2b254d6b1010) at UnixEThread.cc:244
#24 0x00000000008a3732 in EThread::execute (this=0x2b254d6b1010) at UnixEThread.cc:349
#25 0x00000000008a0e32 in spawn_thread_internal (a=0x1c52f40) at Thread.cc:79
#26 0x00002b2549f5fea5 in start_thread () from /lib64/libpthread.so.0
#27 0x00002b254ac96b0d in clone () from /lib64/libc.so.6
(gdb)

Can anybody point me towards any reason why this might occur ?

Demangled Error using c++filt
`
/opt/trafficserver/bin/traffic_server(crash_logger_invoke(int, siginfo_t*, void*)+0x8e)[0x4ea38e]
/lib64/libpthread.so.0(+0xf630)[0x2b47b3ca9630]
/opt/trafficserver/bin/traffic_server(HttpTransact::HandleCacheOpenReadHit(HttpTransact::State*)+0x22)[0x5e54b2]
/opt/trafficserver/bin/traffic_server(HttpSM::call_transact_and_set_next_state(void ()(HttpTransact::State))+0x1e1)[0x57a911]
/opt/trafficserver/bin/traffic_server(HttpSM::state_api_callout(int, void*)+0x811)[0x58db41]
/opt/trafficserver/bin/traffic_server(HttpSM::set_next_state()+0x5eb)[0x599f8b]
/opt/trafficserver/bin/traffic_server(HttpSM::do_hostdb_lookup()+0x255)[0x57d6d5]
/opt/trafficserver/bin/traffic_server(HttpSM::set_next_state()+0x7d2)[0x59a172]
/opt/trafficserver/bin/traffic_server(HttpSM::state_api_callout(int, void*)+0x811)[0x58db41]
/opt/trafficserver/bin/traffic_server(HttpSM::set_next_state()+0x5eb)[0x599f8b]
/opt/trafficserver/bin/traffic_server(HttpSM::state_api_callout(int, void*)+0x811)[0x58db41]
/opt/trafficserver/bin/traffic_server(HttpSM::set_next_state()+0x5eb)[0x599f8b]
/opt/trafficserver/bin/traffic_server(HttpSM::state_cache_open_read(int, void*)+0x3b3)[0x5902b3]
/opt/trafficserver/bin/traffic_server(HttpSM::main_handler(int, void*)+0x198)[0x591988]
/opt/trafficserver/bin/traffic_server(HttpCacheSM::state_cache_open_read(int, void*)+0x343)[0x6135e3]
/opt/trafficserver/bin/traffic_server(CacheVC::callcont(int)+0xaf)[0x7746cf]
/opt/trafficserver/bin/traffic_server(CacheVC::openReadStartHead(int, Event*)+0x605)[0x7700c5]
/opt/trafficserver/bin/traffic_server(CacheVC::handleReadDone(int, Event*)+0x26d)[0x74a54d]
/opt/trafficserver/bin/traffic_server(AIOCallbackInternal::io_complete(int, void*)+0x39)[0x74df89]
/opt/trafficserver/bin/traffic_server(EThread::process_event(Event*, int)+0x30f)[0x8a195f]
/opt/trafficserver/bin/traffic_server(EThread::process_queue(Queue<Event, Event::Link_link>, int, int*)+0x186)[0x8a1f06]
/opt/trafficserver/bin/traffic_server(EThread::execute_regular()+0x18d)[0x8a291d]
/opt/trafficserver/bin/traffic_server(EThread::execute()+0x1c2)[0x8a3732]
/opt/trafficserver/bin/traffic_server[0x8a0e32]
/lib64/libpthread.so.0(+0x7ea5)[0x2b47b3ca1ea5]
/lib64/libc.so.6(clone+0x6d)[0x2b47b49d8b0d]
`

Haven't got the issue after 156528d till now

Haven't got the issue after 156528d till now

Just to clarify: with 156528d you no longer see a crash? Or there was a period after using that commit that you didn't see the crash, but now you have seen it again?

@bneradt  I mean that i was using 2aa1d3106398eb082e5a454212b0273c63d5f69d commit to build the ATS and another more recent one in which i was getting the error . But as i saw there were some changes in the exact file i was getting errors i.e. HttpTransact.cc i updated the version to 156528d . After this i have not got any errors till now for 3 days relating to this.

@bneradt I mean that i was using 2aa1d31 commit to build the ATS and another more recent one in which i was getting the error . But as i saw there were some changes in the exact file i was getting errors i.e. HttpTransact.cc i updated the version to 156528d . After this i have not got any errors till now for 3 days relating to this.

OK, great. Glad that patch seems to address the crash. Good find. I'll go ahead and close this issue then. Feel free to re-open if a crash with that stack resurfaces for you.
Thank you @NischalBhandari .


Remove dep on signalfx-metrics

Remove dependency on signalfx-metrics. Is this actually needed any more in the 1.x line?

signalfx-metrics is used to send metrics from micrometer to splunk, it is requires in 1.x and can not be removed.


Overhaul REPL, add support for zebra input

This change really only touches the repl and the pretty printing, not anything that we're running in production.
I started adding support for zebra to the repl and got a bit carried away overhauling it. I think the result is a lot nicer, try building it and play around yourself.
Not sure that this PR is reviewable, but again, nothing critical has changed so I'm not worried about introducing any production bugs here.
/cc @tranma @amosr

I reviewed the list of changed files....
👍


Count how many times a monster has attacked you

I'm always frustrated when I'm doing Vorkath and usually I can easily count how many times he has attacked and when he will go into poison phase but I always have trouble the first phase because I'm focusing on speccing with my BGS.
I'd like a counter (really anywhere on the screen) that counts how many time the monster has attacked me. This would be extremely helpful for people trying to learn bosses with phases as well as people trying to get more efficient.
I think you should have the ability to configure what monsters will have their attacks counted. Maybe the monsters that are tagged have their attacks counted for instance.

Duplicate of #2447


Add guideline for built-in image usage

Add guideline for built-in image usage

Codecov Report

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

@@            Coverage Diff            @@
##             master     #108   +/-   ##
=========================================
  Coverage          ?   64.35%
=========================================
  Files             ?       45
  Lines             ?     1400
  Branches          ?      153
=========================================
  Hits              ?      901
  Misses            ?      444
  Partials          ?       55

Continue to review full report at Codecov.

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


Food Products Model

As a customer, you must look at a list of foods from which to select one or more meals. Create and implement a website's user interface.

[x] Name of Food
[x] Cost of Food
[x]  Favorite button
[x] Thumbnail of Food
[x] Category

Complete model
Users may easily look at products using the mouse or their fingers


date handling: failing tests, chronic causes different behavior

I got test failures when i ran, caused by issues in the Date handling.
The tests are providing U.S. formatted dates (e.g. 5/1/2010) but Date.parse expects Day/Month/Year format like is used in the E.U.
bypassing that test, the 'today' in test_date_formatting also caused breakage.
Both of these were fixed by installing the chronic gem.
I don't know the history behind this, so maybe this is intentional, but IMHO, an optional dependency shouldn't change existing functionality.  Perhaps another way to handle could be a way to register a field (e.g. "%m/%d/%Y", "%Y/%m/%d", etc) for use with Time.strptime, though that would break the relative time feature that Chronic adds.

I second this. As chronic is optional 6d45d0ea19af0d6e0242ad1f8fe35aeeabfda0ca test should be passing without it being installed.
I’ve filed separate https://github.com/ManageIQ/trollop/issues/86 before I’ve noticed this issue already mentions problem with failing tests.

This is tricky.
We need the gem installed to run the tests.
So it will be tricky to run tests that fail if the gem is not installed
There is the american_date gem that swaps that date order as well.
Many programming languages (and government agencies) have confusion around %m/%d/%y vs %d/%m/%y

@kbrock ic, perhaps an explicit message that tests are failing due to a missing gem would suffice. Trollop 1.2.1 was masked in Gentoo with recent Ruby versions simply because maintainer didn’t investigate deep enough.

#parse_data_parameter tries to load chronic and if it fail to load, then the Date behavior of Trollop is "normal".  Maybe the tests either need to have chronic as a hard dependency, or bypass the tests that would fail if chronic was not available.
Again, i'm not really a big fan of the behavior changing based on existence of another installed gem - in a group environment (e.g. a University), someone may install a gem (chronic) into a "global" Ruby install and then all of the sudden command line tools that use Trollop would start behaving oddly.  That would be pretty hard to track down. :confused:


Netlix-movie recommendation

@khusheekapoor
i would to do this issue
i am a GSSOC'23 contributor
the dataset i am gonna use is :-https://www.kaggle.com/code/laowingkin/netflix-movie-recommendation

@khusheekapoor
i would to do this issue
i am a GSSOC'23 contributor
the dataset i am gonna use is :-https://www.kaggle.com/code/laowingkin/netflix-movie-recommendation
@khusheekapoor

Hi, I am excited to contribute to the Netflix movie recommendation project! With experience in data analysis and recommendation systems, I'm eager to develop effective algorithms for personalized movie suggestions.
Kindly consider assigning me this task to enhance the movie-watching experience for Netflix users.

@lcs2022026   - you can go ahead! We are assigning you 21 days for this project, after which it will be assigned to someone else if not completed. All the best!
Name the file as: algorithm_dataset.ipynb and link it in the readme of the labeled directory as algorithm - dataset.
@tushtithakur - since we are following the first-come-first-serve policy, we will not be able to assign you this issue. However, you can create another issue and use the same algorithm (ANN) on a different dataset.


Add resampled_resolutions parameter in getting_started notebook

Fixes #324

Added resampled_resolutions parameter in axon_segmentation function in section 2.2 of 00-getting_started.ipynb.
Added description of the parameter for the user.

@mariehbourget I've been trying to review this PR for the last hour, however I've encountered a bad Jupyter Notebook bug (https://github.com/jupyter/notebook/issues/2563). I'm certain it has nothing to do with this PR, however I can't test the notebook as Jupyter isn't seeing my venv python installation. I'll keep you up to date, hopefully I can resolve this issue soon and continue reviewing.


2.10 C2 no longer powers off

After the latest update my C2 no longer powers down when idle or shutoff.
When running the test I get no errors, but the screen does not shut off.
The screen starts with the computer.

Hi @atledreier, it sounds like the app can not properly communicate with the TV. The most typical reason for your issue is that the IP of the TV has changed. Please verify that the configuration is still correct.
If this does not resolve your issue please enable the logging and paste the log and config.json on pastebin or similar and paste the link here and I will have a look

After the latest update my C2 no longer powers down when idle or shutoff. When running the test I get no errors, but the screen does not shut off.
The screen starts with the computer.

This is exactly what happened when LG firmware 03.30.30 was released I think. If your firmware is 03.30.30 or newer you need to run LGTV Companion build 2.0 or newer. My guess is you are using build 1.x of LGTVCompanion

It was the IP. Somehow my static IP rule had changed in my router.
All good now, thanks for the tip! :-)


etcd not mentioned as Cluster Plane component Kubernetes Components

Hi all!
In the overview section Kubernetes Components, it is mentioned that kube-apiserver, kube-scheduler, kube-controller-manager, etc are control plane component but it has not been mentioned for etcd component.
I think it can lead to slight confusion.

We should perhaps explain that etcd is not part of the control plane per se; it is in fact the backing store for one of the control plane components (kube-apiserver)
/language en

etcd
Consistent and highly-available key value store used as Kubernetes' backing store for all cluster data. It is not particularly a control plane component but is mainly used as a separate coordination service.
If your Kubernetes cluster uses etcd as its backing store, make sure you have a back up plan for those data.
You can find in-depth information about etcd in the official documentation.

/triage accepted
@Shivam164 @sftim . As per the discussion above this addition of line will be helpful.
Is there anything else in your mind?

/assign


re-core

Now that we are going to have support for React16 and usage of react-scripts, I think it would be a good moment to modularize and stabilice the project.
My propose is to create a re-core project that's a streamlined version of re-start, that only has the barebones to do a multi-platform project. No redux, no redux-router, nothing: just only the react-native dependencies for each platform and the scripts to generate the different versions. Later, re-start would depend on this re-core project and still offer the current basic template. This would allow people to create their own templates that fit best to their workflows, for example I would do one that includes react-native-material-ui and redux-offline already configured :-)

That's a great idea. Working on something similar. Will share with you tomorrow probably and have your suggestions on it 😄

hey @piranna, check this out.
re-base will be the base template (what you wanted to do with re-core), based on which we can create more templates with different permutations and combinations of opinions and libraries and bootstrap any new project with a template that best fits the requirements.
So re-start will be a collection of opinionated templates that work on all available platforms.
Next step would be to create a CLI, such that we can mix and match different templates at the time of bootstrapping a project because more the number of templates, difficult it would be to maintain this project.
cc: @ankitpopli1891, what are your views on this?

It would be great if you could test a couple of templates:
react-native init projectName --template re-start/re-base
cd projectName
node scripts/additionalDependencies.js

its still a work in progress, and I'll be working on rest of the templates tomorrow.

re-base will be the base template

Honestly, the joke with the names starting with re- is going out of our hands... X-D Anyway, it needs some clean-ups, but it's exactly what I was talking about, good job :-)

It fails to install :-(
Fetching template react-native-template-re-start/re-base...
npm ERR! Error while executing:
npm ERR! /usr/bin/git ls-remote -h -t<EMAIL_ADDRESS>npm ERR!
npm ERR! ERROR: Repository not found.
npm ERR! fatal: Could not read from remote repository.
npm ERR!
npm ERR! Please make sure you have the correct access rights
npm ERR! and the repository exists.
npm ERR!
npm ERR! exited with error code: 128

Wouldn't it be better to move re-base to it's own project?

re-base and re-start are 2 different templates. You have to select one.
Try out:
react-native init projectName --template re-start
cd projectName
node scripts/additionalDependencies.js

or
react-native init projectName --template re-base
cd projectName
node scripts/additionalDependencies.js

re-base and re-start are totally isolated projects. They are just managed in a single repository.
Something like react-router does with the help of lerna.

react-native init projectName --template re-base
cd projectName
node scripts/additionalDependencies.js

Ok, it worked :-)
Issues:

[ ] package.json gets minified in a single line instead of being indented and formatted.
[ ] registerComponent() of index.windows.js uses everywhere instead of the project name as index.js does
[ ]  port of Electron app is hardcoded to 3000 that could be being used for other process, use a random port (port zero and fetch the actual one being use).
[ ] title of HTML page is hard-coded to re-start instead of the project name. I think I've read by using HelloWorld it automatically gets replaced when using a template
[ ] What's the purposse of /src/index.js? So far it seems useless to me having /index.js...
[ ] move scripts/additionalDependencies.js to an independent project as requested at #40
[ ] remove react-redux, react-router, react-router-dom, react-router-native, redux and redux-thunk dependencies from package.json
[ ] Is the package.json main field needed to be set to index.electron.js so Electron can correctly run? If not, I think it's better fo remove the entry so the default will be index.js
[ ] electron script is running the web version in background. I think index.electron.js script should start and finish its own webserver
[ ] What's the purposse of app.json file?

[ ] dependencies versions are hardcoded instead of setted by ranges. Don't know if this is on purposse to be sure they are compatible between them...
[ ] integrate dependencies.json and devDependencies.json in builder package.json by using https://github.com/piranna/buildDependencies
[ ] add simple test to just generate the bundles for each platform and be sure they are working
[ ] add TravisCI support
[ ] add support for https://greenkeeper.io/ powered by TravisCI and the tests so dependencies can be up to date

[ ] add eslint configuration and integrate it by default with jest (I propose default config two spaces and no semicolons :-P)

I like where this is going. I will be testing this out myself in the following days.
I wanted to see if it would be possible to set this up with Expo as well (so that yarn run web works but mobile side uses Expo). Has anybody done any testing with it?

Dependencies got upgraded:
 react                16.0.0  →  16.1.0
 react-dom            16.0.0  →  16.1.0
 react-native         0.50.1  →  0.50.3
 react-test-renderer  16.0.0  →  16.1.0

 electron       ^1.6.11  →   ^1.7.9
 react-scripts  ^1.0.14  →  ^1.0.17

[ ] babel-plugin-transform-react-remove-prop-types devDependency should be removed too, specially if it's not enabled by default.

Hi @amoghbanta, hope you are well. How is this going on? Is it ready to merge in master? I could be able to do myself the proposed changes but I was waiting to the branch getting merged first before doing so... I know I have permissions on this repo, but being your repo I think this kind of decission are better done by you, specially since you have the publish permissions on npm. What about doing an org there and put some of us with publish permissions? Adding us to this org would be a good idea, this would allow to maintain the project and do some improvements, like moving re-base and the other templates to independent projects and use them as dependencies or add TravisCI support and add tests, what do you think? :-)


add clientConfigSet and clientConfigList to State object

What this PR resolves:
Add missing method to set and list a client config from the jumpscale state.
Before this PR only get method was available.

Added docstring and tests.
Not happy with https://github.com/Jumpscale/core9/pull/139/files#diff-e0da9137610d6d4ee7e08c816274393aR11 though.
Cause if you run the test locally it will mess up your state.
I tried to change the j.core.state.configStatePath and j.core.state.configJSPath, but didn't managed to get a clean fake state. If you have any suggestion


unfortunately build is really difficult

Hi guys
As written in the readme, the advantages of using the development version are many, for example, wireguard integration, which I need a lot.
But unfortunately, it is really difficult to get a build from the system, should we have a build pipeline?

Yes I know. Building for Windows is especially hard. (Linux is easy just copy and paste the commands and the src/build_UNIX.md guide). SE does have a CI/CD pipeline on Azure where you can download the latest builds as exe files and install them like normal.

https://dev.azure.com/SoftEther-VPN/SoftEther VPN/_build?definitionId=6
This will take you to the project's Azure CI/CD Pipeline. Feel free to explore and you'll find the downloads.

Just follow our build instructions.


swc paths aren't compiled correctly for sources outside of the root folder

Describe the bug
Created a small project that has main apps directory and a shared folder.
When trying to use path aliasing to the outer shared folder the compiled code doesn't include the shared code at all.
(When using build using tsc, and the same paths everything works fine)
Below is a playground that demonstrates the issue.
Input code
No response
Config
{
  "$schema": "https://json.schemastore.org/swcrc",
  "module": {
    "type": "commonjs"
  },
  "jsc": {
    "target": "es2017",
    "parser": {
      "syntax": "typescript",
    },
    "baseUrl": "src",
    "paths": {
        "shared/*": ["../../../shared/*"]
    },
  },
  "minify": false
}

Playground link
https://github.com/johnny-fuse/SwcPlayground
Expected behavior
The compiled code structure should be the same when using tsc.
Actual behavior
No response
Version
1.3.40
Additional context
No response

any fix?

Hi, any update on this issue?

any fix now maybe?

Closing as this is not a bug.
npx swc src --out-dir dist is the command for compiling src, not your project.
TypeScript compiler has a project mode, and that's why depdendencies are compiled along with the input files you specified.


bench: compile again

Closes #18

nice, thank you

@dignifiedquire Can't build on stable,
error[E0554]: #![feature] may not be used on the stable release channel
 --> merkle/benches/crypto_sha512.rs:3:1
  |
3 | #![feature(test)]
  | ^^^^^^^^^^^^^^^^^

Can that be removed?


[PE-D][Tester C] 'find' Command Causes Interface to Become Blank When No Match is Found

Bug Report: find Command Causes Interface to Become Blank When No Match is Found
Title: find command causes contact book interface to remain blank after searching for a non-existent contact.

Steps to Reproduce

Launch LegacyLink.
Ensure there are contacts in the address book.
Enter the command: find Jack (assuming "Jack" is not in the contact book).

Expected Behavior

The application should display a message indicating that no matching contacts were found.
The current list of contacts, events and upcoming ones should remain visible.

Actual Behavior

The contact book interface becomes blank, showing no contacts.
Attempting to use list -p to restore the contact list does not work, and the interface remains empty.

Severity
severity.High

Type
type.FunctionalityBug

Why This Is a Problem

Data Accessibility Issue: Users lose access to their contact list and events, and upcoming events which is crucial for managing family connections and brings inconvenience.
Frustration for Users: Users are unable to recover their contact list using available commands, leading to a poor user experience.
Workflow Disruption: This issue forces users to restart the application or other means/commands to regain access to their contacts and events, disrupting their workflow.

Suggestions for Improvement

Modify the find command to ensure the interface restores the previous list view when no matches are found.
Display a user-friendly message when no matches are found, such as:
"No contacts found matching 'Jack'. The full contact list is still available."

Labels: severity.High type.FunctionalityBug
original: bbryant824/ped#2

This is a feature.


Update UI Readme

PR Checklist

[x] A description of the changes is added to the description of this PR.
[ ] If there is a related issue, make sure it is linked to this PR.
[ ] If you've fixed a bug or added code that should be tested, add tests!
[ ] If you've added or modified a feature, documentation in docs is updated

Description of changes

Removed the warning message. See discussion.
Add a title, an intro and an image.
As an idea, I add a broken link - UI Documentation.
Update the Get started section.
Add a References section to notate the merged.
Commented out Authenticate and Login section.

Please let me know what you think.

@JuanPabloDiaz I merged in the Login UI PR so the README has been updated to include the login and authorization instructions 👍

@JuanPabloDiaz I merged in the Login UI PR so the README has been updated to include the login and authorization instructions 👍

Thanks for letting me know. I just update it


Is vault_public_subnet_ids required for an internal deployment?

The variable vault_public_subnet_ids can optionally be set, but is not required to be set.
The example/custom/main.tf demonstrates how to not set this variable.


unlockRewardForStaking function and msg.sender

It seems that the current version of the unlockRewardForStaking function transfers 73 000 000 tokens from the Distribution contract address to the bridge contract address.
The problem here is that token sending to the bridge contract will be done from the Distribution contract address. So, we won't be able to send the reward for staking tokens to zero address (burn them) on xDai side because we can't know the private key of the Distribution contract address. Here we should somehow pass our multisig address to the Transfer event (in its from field) or pass address(0) there, as I wrote about it earlier.

To be able to properly emit the Transfer event for the bridge, I suggest to replace the line
token.transfer(_bridgeAddress, stake[REWARD_FOR_STAKING]);

with
token.transfer(poolAddress[REWARD_FOR_STAKING], stake[REWARD_FOR_STAKING]);
token.transferFrom(poolAddress[REWARD_FOR_STAKING], _bridgeAddress, stake[REWARD_FOR_STAKING]);

This will emit the Transfer(poolAddress[REWARD_FOR_STAKING], _bridgeAddress, 73000000) event which will be caught by the erc-to-erc bridge, and the bridge will mint 73000000 tokens for the poolAddress[REWARD_FOR_STAKING] multisig on xDai side, and then the multisig on xDai side would call transfer(0, 73000000) to burn the tokens on xDai side.
Note that this solution requires two additional things:

Before calling the Distribution.unlockRewardForStaking the multisig must call the approve function with the following parameters:

spender = Distribution address
value = 73 000 000

We must initialize poolAddress[REWARD_FOR_STAKING] by our multisig address in the initialize function.

Fixed in https://github.com/poanetwork/dpos-token/commit/3309adc6a76f353c1f6258b7631cbe9aa3af5f68


License of mtl_vpu_v0.0.bin

There is a LICENSE.md at the top level directory which is an MIT license. Does it cover firmware/mtl_vpu_v0.0.bin as well which is a binary blob?

MIT does not apply to firmware binaries. I added the license only for firmware binaries in PR #11

The change has been merged. @anthonywong , I am closing the ticket as done. Please, reopen in case of doubt


OLS-186: Slightly better unit tests for logger

Description
Slightly better unit tests for logger
Type of change

[ ] Refactor
[ ] New feature
[ ] Bug fix
[ ] CVE fix
[ ] Optimization
[ ] Documentation Update
[ ] Configuration Update
[ ] Bump-up dependent library
[x] Unit tests

Related Tickets & Documents

Related Issue #OLS-186

Checklist before requesting a review

[x] I have performed a self-review of my code.
[x] PR has passed all pre-merge test jobs.
[ ] If it is a core feature, I have added thorough tests.

Testing

On CI
make test-unit

/lgtm

/approve


Bug report: the save action on spo page control set discards other changes

The page.save method needs to be updated to include all other page properties as well. Right now only CanvasContent1 gets updates, and other settings get discarded.

Good catch! That's clearly not the intention


[Subtask] Add create and delete CLI commands for Model

Describe the subtask
Part of an initiative to add Model entity support to CLI
Parent issue
#5959

@justinmclean I would also like to work on this issue.


[int64] fix up

@glennsl I have come to a problem when I try to use the Comparison module.
I tried to type the Container.Int64.compare to be t -> t -> Comparision.comparision (on a side note I'd like it to be Comparision.t). And then I had a situation that Container.Map.Make functor expects a module Map.OrderedType with compare:t->t->int. I tried to make a functor to convert between Comparision and Map.OrderedType but it seems it does not work. Do you have any suggestions? Thanks!

I think it would have to be a functor. Why didn't that work?
It's Comparison.comparison to be consistent with Result.result btw, but I don't feel strongly about keeping that.

@glennsl Hi I fixed the code per the comment, I will merge this in now.
After some thoughts and consideration I think we should move comparision.ml to bord.ml which represents a total ordering relationship and provide utilities for that. And then change compare functions in modules to use Bord.t. And then I will get back to change Bmap. Its key should be a instance of Bord.S which represents a set with a total order relationship between elements (it is not there yet).

Ok, sounds good to me!


Is it possible to configure export arguments?

Question
Output of python -c "import pydantic.utils; print(pydantic.utils.version_info())":
             pydantic version: 1.4
            pydantic compiled: True
                 install path: ~/.virtualenvs/structurizr/lib/python3.6/site-packages/pydantic
               python version: 3.6.9 (default, Nov  7 2019, 10:44:02)  [GCC 8.3.0]
                     platform: Linux-5.3.0-7642-generic-x86_64-with-Ubuntu-18.04-bionic
     optional deps. installed: []

I have an application where my models should always be exported with by_alias=True and exclude_unset=True. Similarly to what is proposed in #660, I was looking to add these to the models' Config but that's not possible at the moment.
What is the right way to achieve this? Should I override the dict and json methods as below?
from typing import Set

from pydantic import BaseModel, Field

class ModelTest(BaseModel):

    id: str = Field("")
    origin_id: str = Field("", alias="originId")
    tags: Set[str] = Field(())

    def dict(self, **kwargs):
        kwargs.setdefault("by_alias", True)
        kwargs.setdefault("exclude_unset", True)
        return super().dict(**kwargs)

    def json(self, **kwargs):
        kwargs.setdefault("by_alias", True)
        kwargs.setdefault("exclude_unset", True)
        return super().json(**kwargs)

Hi @Midnighter ! As it is indeed unavailable in the Config, I would personally go with a custom BaseModel that I would use in my whole application.
class MyBaseModel(BaseModel):
    def dict(self, by_alias=True, exclude_unset=True, **kwargs):
        return super().dict(by_alias=by_alias, exclude_unset=exclude_unset, **kwargs)

    def json(self, by_alias=True, exclude_unset=True, **kwargs):
        return super().json(by_alias=by_alias, exclude_unset=exclude_unset, **kwargs)

class ModelTest(MyBaseModel):
    id: str = Field("")
    origin_id: str = Field("", alias="originId")
    tags: Set[str] = Field(())

m = ModelTest(originId='myid')
print(m.dict())
# > {'originId': 'myid'}
print(m.dict(exclude_unset=False))
# > {'id': '', 'originId': 'myid', 'tags': ()}
print(m.json())
# > {"originId": "myid"}

This way it allows you to easily change the default values and maybe one day use a more "native" way.

Okay, thank you :smiley:

Why was this issue closed?
@PrettyWood 's response sounds like a great workaround. But can I still write a PR to change pydantic so future users don't need the workaround?
i.e. so that this MWE works:
from pydantic import BaseModel
from typing import Optional

class MyModel(BaseModel):
    x: Optional[int]
    class Config:
        exclude_none = True

assert MyModel().dict(exclude_none=False)['x'] == None
assert 'x' not in MyModel().dict()


The new operators are missing

This article needs to be updated as per "?." operator precedence and order of evaluation.  (We need not guess that it is equivalent to the "." operator)

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

ID: ddc4f59b-159f-5038-fc24-4e78c7dff783
Version Independent ID: f0015080-14af-58f1-15bb-c2691ea4186f
Content: Precedence and Order of Evaluation
Content Source: docs/c-language/precedence-and-order-of-evaluation.md
Product: visual-cpp
GitHub Login: @mikeblome
Microsoft Alias: mblome

@avifarah Can you please clarify? Do you mean the ? :  ternary operator?

I do not mean the ternary operator but the Elvis operator, the "?." (also called the Safe Navigation Operator).  So for example if we have a class instance, x and a member m, then x.m is the correct way to access m.  However if x can be null then a safe way to access m is x?.m. Now if x is not null then x?.m will operate as x.m did, but if x is null then x?.m will not throw an exception instead x?.m will return null.  Please note that this Elvis operator is not the only missing operator.  Thank you for paying attention to my comment.

@avifarah You may be looking in the wrong place. This is the C++ documentation, not the C# documentation. If you're looking for the ?. C# language feature, then you want to look at the Null-conditional operator documentation. C++ has no such operator. (I was curious if it had been added to C++20, but the latest draft standard mentions no such thing.) The gcc compiler does support a non-standard "Elvis operator" ?: with an absent second operand, but MSVC does not.
I'm closing this issue as not relevant to this document. I'm a little sorry the C++ language does not accommodate you; it looks like a feature that could simplify some code.

@avifarah You know what, I was looking at the wrong article, the one for C++ instead of C. However, the ?. operator isn't supported by the C compiler, either. It's not part of the C11/C17/C18 language standard, nor C99, which is approximately what the MSVC C compiler supports.


feat(batch): support hash based lookup join

I hereby agree to the terms of the Singularity Data, Inc. Contributor License Agreement.
What's changed and what's your intention?
PLEASE DO NOT LEAVE THIS EMPTY !!!
Please explain IN DETAIL what the changes are in this PR and why they are needed:

Original lookup join is inefficient when a large number of keys need to lookup, because its nested loop implementation. Hash based implementation can be 1000% faster than original one in some scenarios.
This PR implement the hash based lookup join by reusing the hash join codes.
It can speed up #5555 case from 10s + to 100ms+.

Checklist

[x] I have written necessary rustdoc comments
[x] I have added necessary unit tests and integration tests
[x] All checks passed in ./risedev check (or alias, ./risedev c)

Refer to a related PR or issue link (optional)
#5555

This can improve performance when build side is quite large, but building hash table may affect latency for short queries. Did you test it in sysbench?

Another thing is the optimizer thinks lookup join keeps build side input's order, and this may change it. cc @st1page

Another thing is the optimizer thinks lookup join keeps build side input's order, and this may change it. cc @st1page

Hash table is built by the lookup side of lookup join, so we can still preserve build side order.

This can improve performance when build side is quite large, but building hash table may affect latency for short queries. Did you test it in sysbench?

Yes, it make no difference for sysbench single key index lookup latency. Lookup join latency is dominated by RPC.

BTW, I think 1000 records for an index lookup is quite normal, since we have no statistics about index columns and users  can build an index on whatever columns they like.


DNS instances don't get updated when a new bosh-dns-alias is added

Describe the bug
After adding a new bosh-dns-alias using the following ops-file, the quarks-operator didn't update the generated CNAMEs:
- type: replace
  path: /addons/name=bosh-dns-aliases/jobs/name=bosh-dns-aliases/properties/aliases/-
  value:
    domain: service-discovery-controller.service.cf.internal
    targets:
    - deployment: kubecf
      domain: bosh
      instance_group: diego-cell
      network: default
      query: '*'

To Reproduce
Use the ops-file above after an initial kubecf installation and check that the DNS entries don't get updated.
Expected behavior
I should be able to resolve the newly added name.
Environment

quarks-operator version 5.2.0+0.g22cafc90.

I believe this is a duplicate of https://github.com/cloudfoundry-incubator/quarks-operator/issues/1043
Can you retry with the newest operator?


Fail to change the public read access for one Premium blob container

Storage Explorer Version: 1.16.0-dev
Build: 20200916.1
Branch: main
Platform/OS: Windows 10 / CentOS 7.6.1810 (Core) / MacOS Catalina
Architecture: ia32/x64
Regression From: Not a regression
Steps to reproduce:

Expand one premium storage account -> Blob Containers.
Create a blob container -> Right click it -> Click 'Set Public Access Level...'.
Select the option 'Public read access for container and blobs' on the popped dialog -> Click 'Apply'.
Check the result.

Expect Experience:
Succeed to change the public read access.
Actual Experience:
Fail to change the public read access.

Error details:
"message": "One of the headers specified in the request is not supported.\n
More Info:
This issue also reproduces when selecting the option 'Public read access for blobs only'.

Did some research. Public access is not supported for premium premium gpv1 and premium gpv2 accounts. You can confirm this in the portal:

We'll switch to hiding the action in that case.

Did some research. Public access is not supported for premium premium gpv1 and premium gpv2 accounts. You can confirm this in the portal:

We'll switch to hiding the action in that case.

Fix merged into main.

Fix merged into main.


Provide a more comprehensive API analysis by using apis of .NET site

Intent for this US is to incorporate the new apisof.net added with Binary Analysis more deeply into analysis of source code w/in UA

Closing as duplicate of #1109

@terrajobst would love to get some more detail/substance/suggestions into this issue based on what you know of binary analysis, etc. and what we could do while analyzing a project & source code. (e.g. utilize nupkg ref detection & recommendation, etc.)
once we have that list enumerated here I can start chewing off it in the code.


Implement --cn flag to override headers.cn

r @jakev @abhinav

See #119 for motivation for --cn fake --sneaky usage.

LGTM

We do have some services currently whitelisting "tcurl" in particular for health checks soooo maybe break it into 2 diffs? First to add the option, update health checks to continue identifying as tcurl, then change default behavior.

Agreed; let's avoid changing the health check behaviour. it should probably be "sneaky" by default.

Backed out the appended commits and amended to “be sneaky by default”

Shouldn't this be documented in the command line help?


feat: Split mock services into 2: Customer Service and External Data Service

TODOs:

Document service responsibilities in README
Document plans for bot-pattern customer service approach in README
Ensure no conflicting ports in tests

Going to wait for #13510 to complete first, as the two will definitely conflict, and that PR is already well into the review stage and probably makes sense to complete first anyways 🙂

One last piece of feedback, and with this I've looked at all the files 😄 . Current shape looks good overall.

Thank you for reviewing!!


[Item Relationship] R24.5 Release Checks

[!IMPORTANT]
Follow the guidance on how to use the templates.

RELEASE

HELM Charts 7.1.2
Release 5.1.2

Release Info
Please provide information on what you want to be included in the Eclipse Tractus-X release.
If you are not owner of this issue, please provide the information as comment to the issue.
Make sure to assign this issue to expert(s) for their approval, as soon as you have finished preparation. Multiple assignees allowed; they will un-assign themselves once review completed.
Version to be included in Eclipse Tractus-X release: 7.1.1
Leading product repository: https://github.com/eclipse-tractusx/item-relationship-service
Link

relates to https://github.com/eclipse-tractusx/item-relationship-service/issues/511

Compliance Verifications
This issue tracks all checks, that need to be performed for a product release in Eclipse Tractus-X.

[x] Interoperability checks performed
[ ] Data Sovereignty checks performed
Data Sov Tests in progress: Final alignment on Wednesday. https://jira.catena-x.net/browse/TEST-1915

General Checks

[ ] Tractus-X Release Guidelines(TRGs) fulfilled https://github.com/eclipse-tractusx/item-relationship-service/issues/609
[x] UUX Checks  Compliant with the Style Guid

[x] AppWalkThrough on 8.5.2024 @jjeroch
[x] UUX Review of app
[x] Provided user accounts for Trace-X Instances Trace-X and BMW
[x] Fix UUX topics

[!NOTE]
Note: most criteria for documentation and security are now covered in TRGs

Security Checks

[x] Thread Modelling Analysis passed

Test Results

[ ] E2E Integration Test passed
https://jira.catena-x.net/browse/TEST-1887 Done
https://jira.catena-x.net/browse/TEST-1892 Done
https://jira.catena-x.net/browse/TEST-1888 Done
Findings:
no findings
[x] User Journey approved

Helpful Links

Eclipse Tractus-X openAPI specs on SwaggerHub
https://confluence.catena-x.net/display/PL/Release+Management+%3D>+24.05+ONE+PAGER

@jjeroch  could you please confirm, that no uux review is required for the IRS. Thank you very much.

User Journey is approved from BO Perspective.

Good morning @guenterban, @bs-jokri, @pablosec,
in both products (Trace-X FOSS and IRS) there are no relevant system-related changes compared to R23.12 and R24.3. For this reason, we request the release of the QGC "Threat Modeling Analysis passed" based on version R23.12 & R24.3.
If you come to a different conclusion, we will be happy to provide you with dates for the threat modeling analysis.
Sig-Release:
Trace-X: https://github.com/eclipse-tractusx/sig-release/issues/611
IRS: https://github.com/eclipse-tractusx/sig-release/issues/670
Thank you very much
IRS and Trace-X Team

Good morning @almadigabor,
would you please again support the Team IRS regarding TRG checks for the upcoming release R24.5?
Thank you very much
Martin

Threat Modeling Analysis approved

is there a sovereignty check already conducted considering these criteria https://confluence.catena-x.net/x/NTeJBg ? I'd expect as you are using the edc - the only relevant part is the use of the correct BPN for accessing or providing data on the one side and the check for company-rules for accepting offers on the consumer side... I could nevertheless not validate that assumption

I've opened the TRG checks issue. https://github.com/eclipse-tractusx/item-relationship-service/issues/609

is there a sovereignty check already conducted considering these criteria https://confluence.catena-x.net/x/NTeJBg ? I'd expect as you are using the edc - the only relevant part is the use of the correct BPN for accessing or providing data on the one side and the check for company-rules for accepting offers on the consumer side... I could nevertheless not validate that assumption.

[x] The provider of the connector MUST ensure the consumption of data via the DSP protocol adhering to the released version of CX-0018 standard. For example, using the Eclipse Data Space Connector on Tractus X.
[x] The application MUST ensure that the consumer of the data is uniquely identifiable through its associated BPN-L.
[x] This BPN-L must be used by this application to identify the data.
[x] The application MUST use the BPN-L of consumer of the data when consuming data via the DSP connector (e.g. EDC or managed EDC).
[x] A user of an application MUST be enforced to act within the constraints of the framework agreement by appropriate means (legal, technical,...).
[x] To agree to one or multiple contract offers for a queried data asset, the selection of the appropriate offer MUST be done through the consuming application, ensuring to follow company-defined rules and regulations for a user of that application.

E2E Tests were successfully performed. INT Tests need to be documented/updated. Data Sov Tests outstanding. Thank You!

QG review 14-May:
data sov in progress; no blockers expected
TRG completion expected 14-May; successful collaboration across products & with SystemTeam
minor version expected for final release (5.1.3 & 7.1.3)

Tractus-X Release Guidelines(TRGs) fulfilled https://github.com/eclipse-tractusx/item-relationship-service/issues/609
Release Checks approved ✅

@vialkoje for IRS only data sov conclusion is outstanding. Are you fine with @mkanal `s comments and can you approve?

which uses the SAP DIM to only consu

@vialkoje irs is a data consumer. We consume complete data via the EDC which uses the SAP DIM to only consume data for BPNLs.

@mkanal : could you shortly explain on "how are the company rules put into the IRS by the user/admin of the IRS ? What happens if there are multiple acceptable policies ? -> what is the heuristic for selecting the right one and will the consuming party be aware about it ? thanks for clarifying !

As IRS is implicitly tested via Trace X App E2E tests passed successfully.
INT Tests were performed "Best Effort" and documented. Due to Testdata Issues some tests were instable and focus was on Unit/Component and Moduletests.
Testmanagement approval given.

@vialkoje , @cbrugg , @mkanal as aligned referencing here notes / feedback from E2E Tests for Data Sovereignty.
No blocking issues for current Rel 24.05 but main items to be considered for next release(s):

Note: additional tests (from customer perspective) not possible as no frontend available yet (checks only in Insomnia in connection with EDC)
current concept only handles one standard policy per DC BPN, not possible to configure separately for different partner BPNs (e.g. wrt "ContractReference" Constraint)
no validation of the Schema "profile": "cx-policy:profile2405", can currently use old / outdated policy namespaces without error --> planned interface with Policy Hub, requires check of dev dependencies

current concept only handles one standard policy per DC BPN, not possible to configure separately for different partner BPNs (e.g. wrt "ContractReference" Constraint)

what happens if there are multiple acceptable offers ?
can you shortly describe how the acceptability of an offer is configured by the consuming company ?

@vialkoje

what happens if there are multiple acceptable offers ?

IRS first filters the EDC offers for the wanted asset (e.g. Digital Twin Registry or Submodel Data) if there are multiple offers for the search, IRS will start a negotiation with all of them.
During the negotiation initiation, IRS then validates the policy of the offer with the policy configured for this BPN in IRS. If this policy matches, IRS starts the EDC negotiation, if not the negotiation will fail on IRS side.
In case one of the offers has a matching policy, IRS will use this offer, if none of the offers have matching policies, IRS will abort the negotiation. If more than one offer has a matching policy, IRS will take the first successful response.

can you shortly describe how the acceptability of an offer is configured by the consuming company ?

We don't configure directly which policy to use. The accepted policy for a BPN is added to the IRS policy store (either by configuration or via IRS policy store API).
The policies for the BPN of the provider are then validated with the policy of the offer. If the policy of the offer matches one of the policies configured for the BPN, IRS accepts the offer.

ready to approve QG as soon as data sov is closed-out via @ds-jhartmann , @mkanal and @vialkoje

Configuration of acceptable policies in the IRS has to be communicated to the consuming entity - along with the fact to arbitrarily chose the first acceptable offer if multiple exist. Please point me to the documentation where this is described. Technically everything is implemented. Expert approval granted.

Thank you @vialkoje, we will address this and make our documentation clearer on this matter.
@RolaH1t all QG checks are done and approved. I kindly ask for the final approval.

Congrats:
Quality Gate fully approved with all necessary pre-requisites fulfilled!
Thanks for all your efforts,
Roland

Hello @mkanal
Since the feature is a 24.05 feature and the development phase for 24.08 is coming to an end, we need a status on the feature. Can you please update the status?

[ ] Currently you are assigned (Responsible) → Is this correct? If not, please assign the correct contact person
[ ] Please check whether the status (backlog, work in progress ...) is set correctly
[ ] Please comment on the current status of the feature
[ ] Are all SubTasks (issues from other repositories that deal with the feature) linked? → The easiest way is to mention the feature here in the issue (via the ID) so we can see which teams/repositories are involved.
[ ] Is there a spillover planned?

If you need any clarification, please get in touch, thank you very much.
Stephan

Congrats:
Quality Gate fully approved with all necessary pre-requisites fulfilled!
Ticket will be closed.


Git guild/master

Added benefits for section 5

Testing #12
Issues
This version of benefits.md includes HTML, which will interfere with Hugo's Markdown processing.
Screenshots
Using the changes from #12, I rendered the homepage with Hugo.
HTML included in benefits.md:

HTML removed from benefits.md:

Hugo supports shortcodes within .md content files. That may be the way to go.

HTML should not be required, I am new to MD and trying to follow the conventions already in the skeleton files, but feel free to make any required changes to layout.

The MD formatting looks good, as does the copy. It's a learning process with Hugo and MD for both of us. Saw a bit of HTML in the file and noticed that Hugo only parsed the Markdown properly when the HTML was removed, which led me to the Hugo documentation. Hope it was of some interest, at least.

Yes, was just reading the documentation you sent. Very interesting. Also trying to figure out hugo :-).

@indieinc thank you for the commits! Please be aware that @glenkow and other non-technical people are helping us, including using the unfamiliar Github and Markdown. Not that you haven't been, but please be respectful of people's roles, and specialities. We're here to compliment each other. :+1:


Windows install error

I ran the pip install wikmd install command and got no errors. I then ran python -m wikmd.wiki to start the server and got the following git error.
g:\wikmd>python -m wikmd.wiki
Traceback (most recent call last):
  File "C:\Program Files\Python312\Lib\site-packages\git\__init__.py", line 140, in <module>
    refresh()
  File "C:\Program Files\Python312\Lib\site-packages\git\__init__.py", line 127, in refresh
    if not Git.refresh(path=path):
           ^^^^^^^^^^^^^^^^^^^^^^
  File "C:\Program Files\Python312\Lib\site-packages\git\cmd.py", line 455, in refresh
    raise ImportError(err)
ImportError: Bad git executable.
The git executable must be specified in one of the following ways:
    - be included in your $PATH
    - be set via $GIT_PYTHON_GIT_EXECUTABLE
    - explicitly set via git.refresh()

All git commands will error until this is rectified.

This initial message can be silenced or aggravated in the future by setting the
$GIT_PYTHON_REFRESH environment variable. Use one of the following values:
    - quiet|q|silence|s|silent|none|n|0: for no message or exception
    - warn|w|warning|log|l|1: for a warning message (logged at level CRITICAL, displayed by default)
    - error|e|exception|raise|r|2: for a raised exception

Example:
    export GIT_PYTHON_REFRESH=quiet

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "<frozen runpy>", line 198, in _run_module_as_main
  File "<frozen runpy>", line 88, in _run_code
  File "C:\Program Files\Python312\Lib\site-packages\wikmd\wiki.py", line 29, in <module>
    from wikmd.git_manager import WikiRepoManager
  File "C:\Program Files\Python312\Lib\site-packages\wikmd\git_manager.py", line 6, in <module>
    from git import GitCommandError, InvalidGitRepositoryError, NoSuchPathError, Repo
  File "C:\Program Files\Python312\Lib\site-packages\git\__init__.py", line 142, in <module>
    raise ImportError("Failed to initialize: {0}".format(_exc)) from _exc
ImportError: Failed to initialize: Bad git executable.
The git executable must be specified in one of the following ways:
    - be included in your $PATH
    - be set via $GIT_PYTHON_GIT_EXECUTABLE
    - explicitly set via git.refresh()

All git commands will error until this is rectified.

This initial message can be silenced or aggravated in the future by setting the
$GIT_PYTHON_REFRESH environment variable. Use one of the following values:
    - quiet|q|silence|s|silent|none|n|0: for no message or exception
    - warn|w|warning|log|l|1: for a warning message (logged at level CRITICAL, displayed by default)
    - error|e|exception|raise|r|2: for a raised exception

Example:
    export GIT_PYTHON_REFRESH=quiet

OK, I installed git and it seems to be working but localhost url is not in the docs.

@tyeeman Could you check the latest version of wikmd on pypi using pip? 81c7d086268 should fix this.

Yes, it works, thanks.


Shockolate

Project link: https://github.com/Interrupt/systemshock
Project license: GPLv3
Does it work on 64-bit only Linux: YES
Is source code available?: YES
Is source code stored in Git?: YES
List of Steam AppIds of supported games:

https://steamdb.info/app/410710/

@ShFil119 Does Shockolate project work for enhanced, or only classic version on steam?

iirc you're better off using classic edition assets, as EE reorganizes them in a way not recognized by the original source that shockolate is based on

Tracking progress for classic in https://github.com/luxtorpeda-dev/packages/pull/190

Tracking progress for classic in https://github.com/luxtorpeda-dev/packages/pull/190


zero-diff changes in re-gridding (re-tiling) restarts

GEOSldas needs to have access to "getids" subroutines, which have been put into a module.
Also contains some clean-up and renaming.
Addresses issue #225

Merging since regridding worked and was 0-diff


fixed #1216

Pull Request Test Coverage Report for Build 5186

1 of 3 (33.33%)  changed or added relevant lines in 1 file are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage increased (+0.0007%) to 63.535%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

sharding-sql-test/src/main/java/io/shardingsphere/test/sql/SQLCasesLoader.java
1
3
33.33%

Files with Coverage Reduction
New Missed Lines
%

sharding-core/src/main/java/io/shardingsphere/core/parsing/parser/clause/InsertValuesClauseParser.java
1
96.72%

Totals

Change from base Build 5185:
0.0007%

Covered Lines:
8048

Relevant Lines:
12667

💛  - Coveralls