Addressing Scrolling Object Collection related issues

Overview
This PR addresses the following issues:

When duplicating prefab GameObjects, the OnDisable function in the SOC script is called which in the end tries to destroy the instanced material managed by Material Instance. However, in the case where TextMeshPro is present in the prefab, TMP has its own material life cycle management and thus causes problems (i.e. missing material/pink and error message in the console).
Sometimes removing GameObjects inside the SOC container (followed by some other operations) leads to complaints about missing reference etc. from the UpdateRenderers function in the ClippingPrimitive script. This PR adds null check to guard against that case.

Changes

Fixes: #9481

/azp run

Does this replace / work with https://github.com/microsoft/MixedRealityToolkit-Unity/pull/9193?

Thanks for fixing this Max! I have tried locally and it seems to fix the pink material issue.
 It's coming from TMP. But again, we probably won't be able to sort all issues triggered from other components managing material lifecycle and competing with MaterialInstance.

@vaoliva Thanks for testing and reviewing! I merged this PR to get it into the preview released today. I was not able to repro the issue you mentioned, so could you maybe share some more details/a gif that I can investigate?
@Cameron-Micka I have a question for you: would it be OK if we just always set the auto destroy flag to false in the OnDisable function of SOC in edit mode? The current PR seems more like a patch to a specific case than a thorough solution. Although I have yet to be able to repro the scenario described above but that leads me to think whether we should just never try to destroy materials in that OnDisable function so that we do not need to cover all the potential edge cases individually.

@Cameron-Micka Just tested as you suggested and the number was indeed staying steady even when setting the auto destroy flag to false. Seems that we can go down this route if necessary. Just one more question: why are we concerned about material leaking in the editor in the first place? My impression is that is something we need to worry in player but not necessarily in editor.

why are we concerned about material leaking in the editor in the first place? My impression is that is something we need to worry in player but not necessarily in editor.

Great question! Since the clipping primitive feature works in editor (to allow the effect to be seen without playing) the component creates new materials and adds them to the scene. If we don't clean up those material we could run the risk of leaking them into the scene file.
Fortunately the editor warns you when you try to do renderer.material at edit time. But not we you explicitly new up a material.


refactor: Use mobilestack reference in android builds

Description
Makes the Android builds more brand agnostic.
Test plan

Related issues

Fixes #[issue number here]

Backwards compatibility

Network scalability
If a new NetworkId and/or Network are added in the future, the changes in this PR will:

[ ] Continue to work without code changes, OR trigger a compilation error (guaranteeing we find it when a new network is added)

This seems to fail on the nightly release that I tested due to the build looking for celo-release-key.keystore, despite having removed all apparent references of it from the repo... not sure why this is happening.

The nightly release test passed! https://valora-app.slack.com/archives/C02D08P412Q/p1727385960907899


remove pre-process line

What kind of changes does this PR include?

New or updated content

Description

Closes #1918

Removes misleading line about SCSS in is:inline styles. Waiting for confirmation that there isn't other "pre-processing" happening.

We got our confirmation:

is:inline means not processed by Astro at all
that's true when used on a style or script or slot
it means "leave this alone"
so no, sass will not work on it


[WIP] ProteoRE  ID_converter tool

Proteore id_converter tool
version 20210601
for review, thank you

I still have an error in the tool test here.
The error is :
parameter 'mapping_file': requires a value, but no legal values defined
But in the xml file, in the test lines, mapping_file is affected, so I don't understand ...
If someone could help here please ???
thansk a lot

@combesf can you have a look at this tool: https://github.com/bgruening/galaxytools/tree/master/tools/general_mapping
Its more generic I think and maybe you can use that tool instead?

can you have a look at this tool: https://github.com/bgruening/galaxytools/tree/master/tools/general_mapping
@bgruening : not sure that these tools are exactly the same; btw I was unable to find it in the TS so that we can install it on our dev-instance for evaluation purpose, could provide us with the exact name or a link ? is this tool deployed in usegalaxy.eu ?

Hi @yvandenb I am not sure where the general mapping tool is but I think the 'uniprot_rest_interface' Uniprot tool can do the ID conversions for many different IDs and organisms: https://toolshed.g2.bx.psu.edu/repository?repository_id=c8774310981b07c5 It's also installed on the EU server.

@combesf, @yvandenb  I never published the tool in the toolshed, but at this time I wanted to solve the exact same problem as you, but in a more general way: https://github.com/bgruening/galaxytools/blob/master/tools/general_mapping/readme.md
In essence, an admin can create config files, of arbitrary mapping files. One tool that can map IDs for proteomics and physics and so on...


Is validation correct??

I reproduced your code getting similar results.
I have some doubts on the validation strategy.
The following line made me think that the code was selecting different sets of rows for train and validation:
for train_indices, val_indices in kf:

On the other hand, looking at the complete snippet:
for train_indices, val_indices in kf:
    # split the data into train and validation
    train_dataset = dataset[train_indices,:]
    val_dataset = dataset[train_indices,:]

Actually val_indices is not used in any other part of the code, and in reality looking at the above code it appears that we are using the same set for training and validation.
Am I missing something?

@smartinsightsfromdata, thanks for finding the bug! Like you said just the matter of properly setting val_indices:
val_dataset = dataset[val_indices,:]
I have addressed it in @4002fc3.


14bit grayscale image cannot be correctly compressed in lossless mode

Describe the bug
For 16bit grayscale PGM(with 65535 in header), everything works fine, the idenfity result is identical for src and target.
But for 14bit grayscale PGM(with 16383 in header), this issue happens.
To Reproduce
I have a 14-bit grayscale PGM src.pgm, and I converted it to lossless JXL by using cjxl -d 0 src.pgm target.jxl.
The problem is the brightness of these two are slightly different.
By using identify -verbose, I get the Channel statistics:
src.pgm
  Channel statistics:
    Pixels: 24337152
    Gray:
      min: 0  (0)
      max: 15871 (0.968748)
      mean: 642.471 (0.0392157)
      median: 229 (0.0139779)
      standard deviation: 1242.43 (0.0758363)
      kurtosis: 15.1143
      skewness: 3.73396
      entropy: 0.759787

target.jxl
  Channel statistics:
    Pixels: 24337152
    Gray:
      min: 0  (0)
      max: 15871 (0.96875)
      mean: 642.457 (0.0392148)
      median: 228.99 (0.0139773)
      standard deviation: 1242.42 (0.0758362)
      kurtosis: 15.1143
      skewness: 3.73397
      entropy: 0.759787

One more thing, I also decode the compressed jxl file to decoded.pgm.
The pixel values in decoded.pgm are not identical to src.pgm.
Expected behavior
It should like the 16bit grayscale image, the two statistics from identify should be identical.
Environment

OS: MacOS
Compiler version: Apple clang version 15.0.0
CPU type: M2
cjxl/djxl version string: Both v0.10.2 and main

For anyone running into this issue:, you need to specify the buffer bit depth for both the encoder and decoder in order to perfectly reconstruct the original image:
const JxlBitDepth bit_depth = {JXL_BIT_DEPTH_FROM_CODESTREAM};
JxlEncoderSetFrameBitDepth(frame_settings, &bit_depth);

const JxlBitDepth bit_depth = {JXL_BIT_DEPTH_FROM_CODESTREAM};
JxlDecoderSetImageOutBitDepth(dec.get(), &bit_depth);


Guide "Auth Clients" - Error in Example Code

I've just copied the onAuthSuccess($client) function in the yii2 user guide.
I have tried to signup with google. Below the example response
array (size=9)
    'id' => string 'https://www.google.com/accounts/o8/id?id=Example' (length=80)
    'namePerson/first' => string 'Max' (length=3)
    'namePerson/last' => string 'Mustermann' (length=7)
    'pref/language' => string 'de' (length=2)
    'contact/email' => string<EMAIL_ADDRESS>(length=21)
    'first_name' => string 'Max' (length=3)
    'last_name' => string 'Mustermann' (length=7)
    'email' => string<EMAIL_ADDRESS>(length=21)
    'language' => string 'de' (length=2)

The copied example code are some mistakes I think.
The code proofs in the "else //signup" if the attribute "$attributes[username]" is set and a few rows below it tries to get "$attributes[login]".
Both entries are not in the google response above.
And something is wrong I think. The if condition in the else //signup part
if (isset($attributes['email']) && isset($attributes['username']) && User::find()->where(['email' => $attributes['email']])->exists()) {

The third condition should be
 && !User::find()->where(['email' => $attributes['email']])->exists()) {

And the else blog is missing. So if the User does exist try to login or throw exception or do somthing like this.

This is an example, it can not work in all cases.


rst 1471 welsh paper forms notice

This PR adds a notice in the digital forms for the users, about the service being currently available only in English.
There are plans to introduce a Welsh version in 2019, so this is an interim solution.

Approved

Approved

Approved @leoapost

Approved @leoapost

Approved @leoapost


Timeout on Initialization shows message then perpetual loading

Steps to Reproduce
When the initialization happens (when onboarding or opening the app), it can timeout, displaying the following;

This timeout message goes away and seems to go into perpetual initializing.
This seems to happen more in Sauce Labs than in the real world. There seems to be some extra latency when including the SL environment.
Expected Result
This is what needs to be nailed down. There was a conversation on RC about what should happen. There may need to be more conversation on this. This is what @knguyenBC said.

The app should allow people to continue to enter into the app. The app should not, in any way, bar people from entering their wallet. There’s already a “big brother” sense looming around the app and we want to make sure we tamp it down as much as possible. Also, there are things they can do, such as change anything in settings, delete credentials, simply review their credentials etc. If they try to continue to use their wallet for a proof request or credential offer, it should provide them another error. If anything, this error you posted Sheldon, might be better off removed as a user facing error as they’re not able to do anything to correct it.

Environment
iOS 15.7
Haven't got a timeout on Android to see if the same thing happens.
Workaround
Close and reopen the app is usually successful
Severity
Medium
Priority
Medium
Test Impact
The timeout is not formally tested yet, but it occasionally happens and causes a test failure in the test pipeline. It can impact any and all tests.

Discussed at standup. Team huddle required. Ticket might be divided into multiple tickets as this task is not trivial.

we will improve the error handling and how it it displayed while also investigating the root cause


I want to apply mpEDMD on I system I have not identify yet

mpEDMD does not need the system to be known, it is purely data-driven. If you have a more specific question, I am more than happy to look into it?


Integrate pep8 speaks?

Should we integrate pep8 speaks into Elephant? The tool checks for pep8 compatibility for ongoing pull requests.
https://pep8speaks.com/
https://github.com/OrkoHunter/pep8speaks

@apdavison I tried, but I cannot do this, since I am not owner of NeuralEnsemble. Could you please install the pep8 plugin when you get the chance? Thanks!

Done.


Update documentation for the spring-jms-service-bus/spring-jms-service-bus-topic-sample/ project that was missing relevant info

Purpose

Update description for the spring.jms.servicebus.topic-client-id application property used to run the sample

Does this introduce a breaking change?

[ ] Yes
[x] No

Pull Request Type
What kind of change does this Pull Request introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[x] Documentation content changes
[ ] Other... Please describe:

Docs Build status updates of commit 2a7bf84:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

Line 0, Column 0: [Error-download-failed] Download failed for file 'https://ops/buildconfig/?name=AzureJava&locale=en-us&repository_url=https://github.com/Azure/azure-docs-sdk-java&branch=main&xref_endpoint=&xref_query_tags='. Try closing and reopening the PR. If you get this Error again, file an issue.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 2a7bf84:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

Line 0, Column 0: [Error-download-failed] Download failed for file 'https://ops/buildconfig/?name=AzureJava&locale=en-us&repository_url=https://github.com/Azure/azure-docs-sdk-java&branch=main&xref_endpoint=&xref_query_tags='. Try closing and reopening the PR. If you get this Error again, file an issue.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Hey @trycatchkamal , thanks for your contribution!
Unfortunately, we probably won't directly take this for a couple of reasons.

The spring related content is actually being removed from this repo. Reference #2067 #2071
The readme you're adjusting here is actually copied out of the source repo when we publish each package. Any changes made directly here will just get flattened the next time the readme gets deployed during a release.

@chenrujun can you please take a look and pull this contribution over to the relevant readme if you find it valuable.
@trycatchkamal this ToC is where you will find the spring content going forward.
Also, don't worry about the failing openpublishing.build check. I've filed an issue and am working with the docs.ms folks to get these green.

Hi, @yiliuTo, please take a look.

Hi @trycatchkamal thanks for the contribution, and as @scbedd explained, the source of this README is in https://github.com/Azure/azure-sdk-for-java/blob/main/sdk/spring/azure-spring-boot-starter-servicebus-jms/README.md. You could migrate this pr to repo https://github.com/Azure/azure-sdk-for-java/, and PRs are welcome!

Since the PR is not active, we closed it for now. Please let us know if it is still in use.


update project for 6.0 release

updating this project for the latest release

Obsolete - already updated.


Problems running GCN in a loop

When i try to put the GCN model construction in a loop (to test various randomly generated hyperparameters), I get the following error: ValueError: Initializer for variable graph_convolution_22/kernel/ is from inside a control-flow construct, such as a loop or conditional. When creating a variable inside a loop or conditional, use a lambda as the initializer
Has anyone come across this error?

I know this is almost a year late but it might be fixed by:
from keras import backend as K
K.clear_session()


Create VM interface

Issue summary

Need to create the VM object
https://filecoin-project.github.io/specs/#systems__filecoin_vm
Other information and links

I'm confused, you just linked the VM system, what specifically are you talking about?

Closing since this isn't relevant now


Add some metadata to the servers

It's pretty useful to be able to query the package_name when it's applicable. This will allow for some flexibility and easy to implement features, here are some of them:

very useful for debugging, since some servers may have multiple distributions, see: https://github.com/hrsh7th/vscode-langservers-extracted
ability to reuse a server that already exists on path
allow installing/updating globally, but still without requiring sudo, i,e: pip install --user and so on
you can uninstall a package by name instead of nuking the folder
use the package full name in the info panel instead of the abbreviated name used by lspconfig. (since you actually need npm install bash-language-server for bashls)
allow an easier way to fetch metadata for some fancy LspInstallInfo per server

Let me know what you think :)

Hey! This is definitely on my mind as well. I do want to be careful adding certain features, especially those that incur a lot of added complexity. It'd take a lot of convincing and then some more for me to consider allowing global/system-wide or even user-local installations (really, anything outside of nvim-lsp-installer's dedicated file system boundaries). But maybe I misunderstand your 2nd and 3rd points?

you can uninstall a package by name instead of nuking the folder

I'm not following this one, could you perhaps give an example?

use the package full name in the info panel instead of the abbreviated name used by lspconfig. (since you actually need npm install bash-language-server for bashls)

Yeah some of the lspconfig names are pretty opaque in terms of what's actually installed. I definitely think it makes sense to be more transparent with what'll actually end up being installed. I have an idea where this kind of metadata would quite easily be able to be inferred from each server's installation steps. For example, a server that has a npm.packages { "typescript-language-server", "typescript" } installer configured, one could easily infer that:

a Node.js runtime with npm installed is required to install this server
npm package typescript-language-server will be installed
npm package typescript will be installed

allow an easier way to fetch metadata for some fancy LspInstallInfo per server

Yeah, I'm wondering what would be interesting to display here. I recently added a relative timestamp showing when this server was installed. I think this is the cheapest outlet to experiment with :)
All in all, there's only really two features remaining before I feel like this plugin is more or less feature complete, and that is 1) a nicer API to ensure that a set of provided servers are installed (easy to impl), and 2) functionality that allow you to upgrade servers when upgrades are available (either through some UI notification, or automatically behind the scenes). Anything else feels somewhat redundant

I actually retract the idea of user-level install because you don't want to be dealing with people who didn't setup their package managers correctly. In a perfect world, the user would have configured all the correct paths for their tools, but that's sadly not the case: https://github.com/LunarVim/LunarVim/issues/1468, https://github.com/LunarVim/LunarVim/pull/1463 😢

Regarding the other points, let's take this example.
:LspInstall sumneko_lua pyright yamlls jedi_language_server tsserver
Here's what I got:
lsp_servers
├── jedi_language_server
│   └── venv
├── python
│   ├── node_modules
│   ├── package.json
│   └── package-lock.json
├── sumneko_lua
│   ├── [Content_Types].xml
│   ├── extension
│   └── extension.vsixmanifest
├── tsserver
│   ├── node_modules
│   ├── package.json
│   └── package-lock.json
└── yaml
    ├── node_modules
    ├── package.json
    └── package-lock.json

It might be good if jedi were to use the python folder as well. But the is currently not attainable, because the pyright ninstaller doesn't know how search for pyright anymore. It will resort to a more aggressive approach by removing the root folder, in this case python, completely instead of relying on the package manager to clean it up.
https://github.com/williamboman/nvim-lsp-installer/blob/31f77fc8d638001a1dc548e45580714b9a63d3f2/lua/nvim-lsp-installer/server.lua#L105

All in all, there's only really two features remaining before I feel like this plugin is more or less feature complete, and that is 1) a nicer API to ensure that a set of provided servers are installed (easy to impl)

This is why I feel this such an easy fix/feature. Consider this, which will be even easier internally:
fd 'package.json' --max-depth=2 | xargs dirname | xargs -I {} npm ls -C {}

# 	  results

#	  python@ /home/hatsu/.local/share/nvim/lsp_servers/python
#	  └──<EMAIL_ADDRESS>
#	  tsserver@ /home/hatsu/.local/share/nvim/lsp_servers/tsserver
#	  ├──<EMAIL_ADDRESS>#	  └──<EMAIL_ADDRESS>
#	  yaml@ /home/hatsu/.local/share/nvim/lsp_servers/yaml
#	  └──<EMAIL_ADDRESS>
Do you see how it would make things just a slightly bit easier to display and validate that we're using v0.22.0 of yaml-language-server. You can now do npm rm yaml-language-server and then be free to remove the folder afterwards.
Other observations:

Again, this is not really specific to node modules, it just so happens that most servers are written in node, for better or for worse.  Hopefully there's some fast Rust tool that's good at dealing with nodejs mess :)
This will allow for less hacky ways to pin a version of language-server, use a different source/fork or any other useful options that can be passed to the package manager.

It might be good if jedi were to use the python folder as well. But the is currently not attainable, because the pyright ninstaller doesn't know how search for pyright anymore. It will resort to a more aggressive approach by removing the root folder, in this case python, completely instead of relying on the package manager to clean it up.
[...]
Do you see how it would make things just a slightly bit easier to display and validate that we're using v0.22.0 of yaml-language-server. You can now do npm rm yaml-language-server and then be free to remove the folder afterwards.

I'm not following why you'd either a) want to put this kind of "semantic" state in the file system hierarchy in the first place, and b) why exactly where servers end up being installed is a concern for the (average) end user (other than the guarantee that it's inside the data stdpath [i.e., not system-wide, globally, or user-local]).
The fact that there exist a python directory name (which is allocated to pylsp iirc) is just a legacy thing. In fact soon the default behavior will be that these directories will take the same name as their corresponding lspconfig name (apart from the existing servers with non-matching names, to avoid breaking changes).
Don't you think it'd be much easier to contain this kind of state/logic in the Lua layer? Or I might still be misunderstanding.

Do you see how it would make things just a slightly bit easier to display and validate that we're using v0.22.0 of yaml-language-server. You can now do npm rm yaml-language-server and then be free to remove the folder afterwards.

Perhaps from a shell scripting perspective. I think the same can easily be implemented in a platform-agnostic manner if each server was responsible for reporting such metadata (so for npm-based installations we'd spawn subprocesses that report back a single package version). This could easily be parallelized with libuv.

Again, this is not really specific to node modules, it just so happens that most servers are written in node, for better or for worse.  Hopefully there's some fast Rust tool that's good at dealing with nodejs mess :)
[...]
This will allow for less hacky ways to pin a version of language-server, use a different source/fork or any other useful options that can be passed to the package manager.

If my quick check was correct, 23/50 servers are currently installed via npm, so it's actually not even a majority. But yeah I think the extra capabilities each server would offer (versioning, pinning, whatnot) will be entirely constrained by how it's installed in the first place. npm-based distributions can be heavily standardized whereas servers that require custom build scripts will not.

I've started experimenting with this now, preview: https://asciinema.org/a/Bj6Q03SweLj6gJ7EGNqIr4beU. This is actually pretty easy to implement thanks to the state-driven UI framework that is in place. The question is what's interesting to show.. I'm thinking the following, for starterss:
For installed servers:

Installation date
Installed version
Latest available version (perhaps also do some diffing and highlight this if it differs from the currently installed version)

For uninstalled servers:

Version that would be installed
What exactly will be installed (this one I might postpone til a later iteration)

Anything else that might be interesting to display?

This looks like a great feature. Although obvious, how about displaying the full installation path (the LSP server directory or the cmd executable) so that one can navigate if they'd want some troubleshooting?

That makes sense too I think! I'd probably opt for the server directory itself rather than the full command. The full command should be available via :LspInfo already, I also hope to be able to remove these cmd overrides from this plugin altogether in the future:tm:.

I've started experimenting with this now, preview: asciinema.org/a/Bj6Q03SweLj6gJ7EGNqIr4beU.

I'm actually working on implementing something similar just now

Demo

There was a small problem where it would try to attach the server immediately if I had the a file with the associated filetype opened.

Race condition(?)

The solution was disabling vim.cmd [[ do User LspAttachBuffers ]].

That's so cool 😎 ! However this seems to be a tighter integration between LunarVim and nvim-lsp-installer, and not so much about extending the feature set of nvim-lsp-installer - or am I missing something 🤔 ?
Regarding the race condition - this might be because the .setup() function of the server is called before the server is successfully installed? vim.cmd [[ do User LspAttachBuffers ]] will essentially only trigger lspconfig to attach all existing buffers to potential server candidates that have been registered with lspconfig (i.e. through .setup())

I also have plans on adding something similar to what you're doing actually! I just pushed a somewhat finished version of it to ensure-installed-servers

Oh my bad, I misread the command name and thought it was an auto-installer!
The two features that I would like to see, and can even help with, are:

allow dismissing the split quickly and also allow the use of an actual popup
it has been mentioned before, but I'd like to see some kind of feedback from the installer that is not coupled to the UI state.

I feel like the last point might be tricky, but I hope not.

allow dismissing the split quickly and also allow the use of an actual popup

Yeah now most interactions with the plugin will be centered around the UI window. Implementing a floating window instead of a regular vertical split should actually be pretty straight forward. The code is a bit so-so and not really considered stable yet, but this is the part responsible for opening the window. I'd like to avoid having options for everything so I think either a floating window or vertical split window should be supported, not both. I don't have any strong opinions so happy to receive PR changing to floating window!

2. it has been mentioned before, but I'd like to see some kind of feedback from the installer that is not coupled to the UI state.

This should be possible today. The drawback is that if you bypass the APIs that are attached to the UI window, the UI window will become out of sync with any operations done outside of it. What you're interested in would be Server:install_attached(). Here's how the UI calls that function, for reference.

edit: Everything's technically possible™, so we could make sure to funnel everything through some centralized state management while allowing for 3rd party use cases (basically lift the state management from the status-win/init.lua module and spread it globally across the plugin). I'm not too compelled to make such a change at this point in time though, I'd like things to stabilize a bit more.

Yeah that's understandable, take a look at https://github.com/williamboman/nvim-lsp-installer/pull/98 and see if it's good enough.

Some more progress on this: https://asciinema.org/a/fmqCxtLUR4HrqLcbteNaMGQqR

Some more progress on this: asciinema.org/a/fmqCxtLUR4HrqLcbteNaMGQqR

How's the performance? I messed around a bit with yarn which seems slightly faster in fetching data, but the initial setup is way too annoying compared to npm. I can make a PR for it if you'd like to see/support it.

Do you mean in terms of latency? It's perfectly fine for me, although the "latest version" is network-bound so ymmv. What happens is that when you expand a server it'll gather each individual data point separately in a subprocess via libuv, so in terms of perceived performance I don't think yarn vs npm will matter

Do you mean in terms of latency? It's perfectly fine for me, although the "latest version" is network-bound so ymmv. What happens is that when you expand a server it'll gather each individual data point separately in a subprocess via libuv, so in terms of perceived performance I don't think yarn vs npm will matter

Good to know! I've heard conflicting tales about the superiority of yarn. However, I could not get yarn init to shut up about either the license file or the default settings. It's made to be used interactively in the first place, smh..

Hehe maybe you need some NON_INTERACTIVE env set 🤷♂. Another reason to stick with npm is that it's generally shipped together with Node, whereas Yarn is less so (maybe this has changed). I do want to enable the possibility to run server installations via Yarn instead of npm, as npm has a tendency to cause issues when on corp VPNs/proxies. But that's a separate topic

Note to self: I realized for node packages we could probably bypass npm entirely through something like:
local package_json = vim.json.decode(fs.read_file(path.concat { server.root_dir, "node_modules", package, "package.json" })
package_json.version

And for fetching latest versions https://api.npms.io might be more interesting than going through npm cli, like:
local package = vim.json.decode(fetch(("https://api.npms.io/v2/package/%s"):format(package_name)))
local latest_version = package.collected.metadata.version

And for fetching latest versions https://api.npms.io might be more interesting than going through npm cli, like:
local package = vim.json.decode(fetch(("https://api.npms.io/v2/package/%s"):format(package_name)))
local latest_version = package.collected.metadata.version

I recently learned that npm view fetches data from the online registry, which makes it slow. Is that the reason why you don't want to go through local npm calls?

Note to self: I realized for node packages we could probably bypass npm entirely through something like:
local package_json = vim.json.decode(fs.read_file(path.concat { server.root_dir, "node_modules", package, "package.json" })
package_json.version

Can you check this can be help for dont use the api :)
https://stackoverflow.com/questions/10972176/find-the-version-of-an-installed-npm-package


UI freezes while using Web worker

Hi guys,
I'm using Drift with Flutter Web. I have a really big list of objects that I need to insert in database(IndexedDB) without UI freeze. I configured my project according to flutter_web_worker_example (add support of web worker), but when I inserting list in database via web worker, UI freezes. (Please see attached video file)
I created a example with described issue at drfit_webworker_poc
Database and example of insert action
@DriftDatabase(tables: [MachineTable])
class AppDatabase extends _$AppDatabase {
  AppDatabase(QueryExecutor executor) : super(executor);
  AppDatabase.connect(DatabaseConnection c) : super.connect(c);

  @override
  int get schemaVersion => 1;

  Future<void> insertMachines(List<Machine> machines) {
    return batch((batch) {
      batch.insertAll(
          machineTable,
          machines,
          mode: InsertMode.insertOrReplace);
    });
  }
}

Configuration of web workers in database
class PlatformInterface {
  static QueryExecutor createDatabaseConnection(String databaseName) {
    return LazyDatabase(() async {
      return _connectToWorker(databaseName).executor;
    });
  }

  static DatabaseConnection _connectToWorker(String databaseName) {
    final worker = SharedWorker(kReleaseMode ? 'worker.dart.min.js' : 'worker.dart.js', databaseName);
    return remote(worker.port!.channel());
  }
}

Example of performance test that executes in application
driftDbPerformanceTest() async {
  print("Start drift db performance test");
  final database = AppDatabase(Platform.createDatabaseConnection('sample'));

  await Future.delayed(const Duration(seconds: 3));
  final stopwatch = Stopwatch()..start();

  final json = jsonDecode(await rootBundle.loadString('jsons/basic_old_response.json'));
  final machines = (json as List<dynamic>).map((e) => Machine.fromJson(e)).toList();
  print("finish machines serialization json(length: ${machines.length}) in milliseconds ${stopwatch.elapsed.inMilliseconds}");

  stopwatch.reset();
  await database.insertMachines(machines);
  print("insert machines using drift(length: ${machines.length}) in milliseconds ${stopwatch.elapsed.inMilliseconds}");
}

https://user-images.githubusercontent.com/103174674/197016244-62eeca7c-c684-47a2-8827-fd6d1838809c.mp4

Thanks for the detailed issue. I didn't check this yet, but I suppose this is the cost of creating and serializing all the SQL statements to send to the worker (this still happens on the main thread).
I have some pending investigations I need to do on performance (https://github.com/simolus3/drift/discussions/2064 is also in my backlog). If it's possible, you could probably reduce the lag if it's possible to run the entire insert operation on a background worker (so that the worker loads the data and inserts it)?

@simolus3 Thanks for replying back!
Can you please provide an example of running the entire insert operation on a background worker ?

It's not that easy unfortunately.
In general, you want something like Flutter's compute but with web workers. I'm not sure if a package exists to make this easier already. Otherwise, you're already creating the database here. So the same file could create an instance of your AppDatabase class and insert some data into it.
If you want to load data from an URL, you'd have to find a way to inform the worker about that URL. You could use onConnect and messagePorts to establish a communication channel between the worker and the main tab. This is also what drift is doing internally to relay database operations through the worker. You could use this communication channel to send commands like "download this URL, then insert the resulting rows into the database" to the worker. The worker would then execute that chunk of work.

@simolus3 Thanks! I'll try to implement background worker in example

@simolus3 Thanks! I'll try to implement background worker in example

I have same problem.
If you can implement it, let me know, thanks.


Uncaught TypeError: editor.transact is not a function

[Enter steps to reproduce:]

...
...

Atom: 1.21.2 ia32
Electron: 1.6.15
OS: Microsoft Windows 10 Pro
Thrown From: sequential-number package 0.5.0
Stack Trace
Uncaught TypeError: editor.transact is not a function
At C:\Users\Oscar\.atom\packages\sequential-number\lib\sequential-number.coffee:50

TypeError: editor.transact is not a function
    at Object.exec (/packages/sequential-number/lib/sequential-number.coffee:50:14)
    at /packages/sequential-number/lib/sequential-number.coffee:11:34
    at Function.module.exports.Emitter.simpleDispatch (~/AppData/Local/atom/app-1.19.3/resources/app/node_modules/scrollbar-style/node_modules/event-kit/lib/emitter.js:25:14)
    at SequentialNumberView.module.exports.Emitter.emit (~/AppData/Local/atom/app-1.19.3/resources/app/node_modules/scrollbar-style/node_modules/event-kit/lib/emitter.js:125:28)
    at SequentialNumberView.module.exports.SequentialNumberView.handleKeyup (/packages/sequential-number/lib/sequential-number-view.coffee:61:8)

Commands
     -3:00.8.0 intentions:highlight (input.hidden-input)
     -3:00.1.0 sequential-number:open (input.hidden-input)
     -2:48.5.0 command-palette:toggle (atom-notification.fatal.icon.icon-bug.native-key-bindings.has-detail.has-close.has-stack)
     -2:44.8.0 core:confirm (input.hidden-input)
     -2:44.8.0 settings-view:open (atom-notification.fatal.icon.icon-bug.native-key-bindings.has-detail.has-close.has-stack)
     -0:23.7.0 intentions:highlight (input.hidden-input)
  6x -0:23.3.0 editor:duplicate-lines (input.hidden-input)
 10x -0:22.5.0 core:move-up (input.hidden-input)
     -0:21.1.0 intentions:highlight (input.hidden-input)
 19x -0:21 find-and-replace:select-next (input.hidden-input)
     -0:19 intentions:highlight (input.hidden-input)
     -0:18.7.0 sequential-number:open (input.hidden-input)
     -0:16.4.0 core:confirm (input.hidden-input)
     -0:00.9.0 command-palette:toggle (input.hidden-input)

Non-Core Packages
activate-power-mode 1.4.0
aligner-javascript 1.1.0
atom-alignment 0.13.0
atom-beautify 0.29.23
atom-latex 0.8.2
atom-material-syntax 1.0.2
atom-material-ui 1.3.10
atom-python-run 0.7.3
autocomplete-python 1.8.63
busy-signal 1.4.1
git-control 0.9.0
intentions 1.1.2
language-abaqus 0.2.0
language-latex 1.0.0
language-tex 0.4.0
latex 0.49.0
linter 2.1.4
linter-spell 0.13.2
linter-spell-latex 0.9.2
linter-ui-default 1.2.3
minimap 4.28.1
minimap-git-diff 4.3.1
minimap-hide 0.3.0
python-indent 1.0.2
python-tools 0.6.9
sequential-number 0.5.0
split-diff 1.4.0
sync-settings 0.8.1

Hi @soraxas Thank you for report :+1:
Can you tell me what kind of input to the Input Panel will cause an error?
Thanks.

Hi @tsuyoshiwada sorry for not including the procedures that cause the issue before.
It wasn't any special input, was just 1 , 1 + 1, or any other pure numbers. That was when I only installed the package for the first time and tried it out. After a few restart (restarting only Atom, not the OS) it still throws the same error. I was in a hurry into finishing a project before, so I just went ahead and used another similar package for incrementing multiple numbers.
But today after seeing your comment I reinstalled it again and it works perfectly without any error as of yet. I suppose there was something wrong when it was installed initially. Sorry, I couldn't reproduce it again for you, it's all good now. I will close the issue.

Hi @soraxas, Thanks!
Please report again if there are problems :smiley:


the command "make prettier" never finish

Please do not submit support requests or "How to" questions here. For that, go to StackOverflow.
What you were expecting:

I want to fomat my code.
I launch "make prettier" on the root folder
Nothing happen.
What happened instead:

Steps to reproduce:

Related code:

insert short code snippets here

Other information:

Environment

Admin-on-rest version:
Last version that did not exhibit the issue (if applicable):
React version:
Browser:
Stack trace (in case of a JS error):

There is also a problem on the pattern matching
Its specified "packages/ra-admin/src//*.js" than "packages//src/**/*.js"

Its seem to be a problem on my code.

Fixed by #1711


k4run --help not working due to inputFiles check

This kills k4run before the help message can be printed :/
https://github.com/key4hep/CLDConfig/blob/a9c7ee5c82bf4df45355258eeb2d1d1598f595a0/CLDConfig/CLDReconstruction.py#L78-L80

fixed by #29


ci(tmt): run the tests on Testing Farm

TODO:

[ ] Write new tests or update the old ones to cover new functionality.
[ ] Update doc-strings where appropriate.
[ ] Update or write new documentation in packit/packit.dev.
[ ] ‹fill in›

Fixes
Related to
Merge before/after

RELEASE NOTES BEGIN
Packit now supports automatic ordering of ☕ after all checks pass.
RELEASE NOTES END

/packit test


hi, now grammar does not support ?

hi, now grammar does not support ?
https://github.com/ncbray/wassembler/blob/master/demos/simple.wasm

Yes.  This is old, and has not been updated.  It helped get web assembly started.  Other compilers can now be used.


[AutoPR sqlvirtualmachine/resource-manager] chore: jsonfmt sqlvirtualmachine

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/6543

This PR has been merged into https://github.com/Azure/azure-sdk-for-java/pull/4266


RabbitMQ Logging: Small Regex Parser Fix

We were getting some errors in the logging-module logs concerning an invalid timestamp format and according to

Was able to validate changes on a GCE VM
Before:
[2022/06/29 15:27:28] [error] [parser] cannot parse '2022-06-29 15:27:26.103510+00:00' after %L
[2022/06/29 15:27:28] [ warn] [parser:rabbitmq.rabbitmq.rabbitmq] invalid time format %Y-%m-%d %H:%M:%S.%L+%Z for '2022-06-29 15:27:26.103510+00:00'

Afterwards the error log does not show up in logging-module.log
Also added a mapping for the warning severity:
Before:
jsonPayload: {
message: "Failed to add user 'otelu6': the user already exists"
process_id: "0.1219.0"
severity: "warning"
}

Kokoro failures are unrelated
<failure message="Failed" type="">
third_party_apps_test.go:582: Test logs: https://console.cloud.google.com/storage/browser/ops-agents-public-buckets-test-logs/prod/stackdriver_agents/testing/consumer/third_party_apps/test/sles15/572/20220701-061725/logs/TestThirdPartyApps_sles-15_hbase third_party_apps_test.go:595: Instance Log: https://console.cloud.google.com/logs/viewer?resource=gce_instance%2Finstance_id%2F2585725234817747588&project=stackdriver-test-143416
</failure>


Downloaded file does not match reported size

I have a PNG image file in my OneDrive.
The API and the Web UI both report that the file is sized 21.9K
When I download the file (via either the UI, or the API), the downloaded file is 8.5K.
When trying to upload the 8.5K file back to OneDrive, it is a gain being reported as a 21.9K file.
I've checked that the file doesn't contain any alternative streams.
You can get the file here: https://onedrive.live.com/redir?resid=82a4826daa392c28!4988&authkey=!AF48zYh_qglwvVg&v=3&ithint=photo%2Cpng

Hi @saguiitay, this is unfortunately an edge case that has existed in OneDrive for some time. I'll redirect you to this answer of Ryan's on StackOverflow for details, and I'll reiterate that we're definitely aware of the pain this issue causes and we're making good progress towards its resolution.

I'm actually facing a similar issue for a DOCX file:

Id: 82A4826DAA392C28!4864
Name: Document 1.docx
Size: 15173

When I download the file (either via the API, or via the web interface), the file size is 12209 bytes.

Still facing this issue here. Any suggestion or news?

i am seeing this on a lot of my files - how come this is still an issue?

While the issue is trivial at face value, there were complex constraints in the system that unfortunately complicated the path to resolution. Having said that, we've made good progress on relaxing those constraints so we're now in a position we can directly tackle the issue at hand. Hopefully it won't be too much longer!
This is our oldest open issue and we're all very much eager to see it gone :)

Just facing this issue with a .docx file and causing some failures. Do you have an ETA for the fix?

I think we might have just started experiencing this issue today. After uploading a plain text file, the reported file size is 0. However, when using the download URL to download the file, its content is what was expected (non-empty). Our tests started reporting this issue almost consistently starting today.

@dluces the issue here is that files are reported to be larger than they actually are, so what you're seeing is something different entirely. There was an issue that occurred today that resulted in some files being recorded as being 0-bytes, but it's now resolved and any files that were put into this state will be repaired in the next day or so.

@ificator sounds good, thank you!

This issue should finally be resolved - please let us know if you continue to see file sizes being reported as larger than they actually are.

Still encountering the same issue at file 443A255D801D1655!300: The size displayed in onedrive.live.com and the size reported by the API is around 40 KiB, however the downloaded size is just 12 KiB @ificator
Thank you very much

The fix definitely applied to all new / updated files, but I'm not sure if we're expecting people to manually fix existing items in this state by reuploading - I'll follow up and see what the expectations of the fix were.
One option you could try in the mean time to avoid reuploading is to use the copy API - that'll essentially reupload but without you incurring any bandwidth costs. Obviously not ideal, but it's the best I can think of while I follow up.

Thank you for your answer, my current workaround is to ignore the size values and treat them as all the same :joy: The copy API modifies the id of items and it may not be suitable for me


feat: [example] add rn67 example and refactor kotlin

Support React Native 65+
Adds React Native 67 example
Add client timeout parameter to module initilaization

@kaloudis Can you try this branch and let me know .
The client timeout is now a parameter you can pass at initialization and defaults to 60 seconds.

@marcosrdz Can you check if this resolves #38 for you guys ? @kaloudis says it's good, so if it works for you guys i'll merge and publish a release.


mruby full support?

You should definitely take a look at go-mruby project. It will allow you to easy implement full ruby syntax at the same time achieving your goal to use go libs.
https://github.com/mitchellh/go-mruby

@luisbebop Thanks for sharing!!


Change error code for missing new for classes

https://github.com/dudykr/stc/blob/e59631e58fb04578142befb6e5dbc7ef1d52f9db/crates/stc_ts_type_checker/tests/conformance/expressions/functionCalls/forgottenNew.ts#L1-L5
The code above should fail with TS2348, but currently it failes with TS2349

It has the context of tried to calculate return type, which is used at
https://github.com/dudykr/stc/blob/e59631e58fb04578142befb6e5dbc7ef1d52f9db/crates/stc_ts_file_analyzer/src/analyzer/expr/call_new.rs#L1973-L1977
But error is NoCallablePropertyWithName, which is TS2349, and

So we should modify
https://github.com/dudykr/stc/blob/e59631e58fb04578142befb6e5dbc7ef1d52f9db/crates/stc_ts_file_analyzer/src/analyzer/expr/call_new.rs#L821-L825
to use different error variant.

Hey! I'd like to work on this issue.

Thanks!

Just to be sure: to accomplish this we need to create new error type in crates/stc_ts_errors/src/lib.rs, map it to correct code (TS2348), and then change the error in crates/stc_ts_file_analyzer/src/analyzer/expr/call_new.rs. Is that correct?

Yeap, exactly

Great, thanks for confirming! Is there any convention for errors' naming?

No, but I think we need to refactor them in the future

From the debugging I see that we are getting this error
https://github.com/dudykr/stc/blob/e59631e58fb04578142befb6e5dbc7ef1d52f9db/crates/stc_ts_file_analyzer/src/analyzer/expr/call_new.rs#L816 instead of https://github.com/dudykr/stc/blob/e59631e58fb04578142befb6e5dbc7ef1d52f9db/crates/stc_ts_file_analyzer/src/analyzer/expr/call_new.rs#L821
Is this behaviour correct? Did you mean to change the former error?

This is incorrect, because NoNewSignature should be converted to another error.
It should be NoConstructorPropertyWithName I guess

And NoConstructorPropertyWithName should have error code of TS2348

The problematic part is the conversion of NoNewSignature to NoCallablePropertyWithName.
I think it should be NoConstructablePropertyWithName instead

I see. Is that okay if the test won't pass after the changes though?

Reducing matched_errors is  fine but increasing extra_errors or panic is not fine


Please support .NET Core

Please support .NET Core beside .NET Framework 4+

Sure. I can't say specific dates, but plans for this are already voiced earlier here: https://github.com/3F/MvsSln/issues/5#issuecomment-480867917
Of course anyone else can speed up the process via PR. Or please wait for my time.

Sure. I can't say specific dates, but plans for this are already voiced earlier here: #5 (comment)
Of course anyone else can speed up the process via PR. Or please wait for my time.

I am not sure it can be as a PR because you should change your .NET Framework target and it would be a breaking change for users. (They should update .NET Framework 4 to .NET Framework 4.7.2)
I checked your library it works great with these configs.
<TargetFrameworks>netcoreapp2.1;net472</TargetFrameworks>
<ItemGroup>
    <PackageReference Include="Microsoft.Build" Version="16.0.461" />
</ItemGroup>

I compiled it successfully but I should be for vNext (MvsSln 3.0).
What do you think?

@HamedFathi No. We need to provide both because we have no plans to drop support .netfx 4.0 at least this year.
That is, I can accept only changes to build separate platforms at the same time. This can be achieved via vssbe-scripts (this project already controlled by this) and/or additional msbuild targets.
about minimal .netfx 4.6 see here:

https://github.com/3F/MvsSln/issues/5#issuecomment-410464661
https://github.com/3F/MvsSln/issues/1#issuecomment-410466850

I think it is possible to support both variants. It is a little tricky:
 <ItemGroup>
   <Reference Include="System.XML" />
   <Reference Include="Microsoft.Build" Condition="'$(TargetFramework)' == 'net40'" />
   <Reference Include="Microsoft.CSharp" Condition="'$(TargetFramework)' == 'net40'" />
 </ItemGroup>

and for the packages
  <ItemGroup Condition="'$(TargetFramework)' != 'net40'">
    <PackageReference Include="Microsoft.Build" Version="16.0.461" />
    <PackageReference Include="Microsoft.CSharp" Version="4.5.0" />
  </ItemGroup>

This way, you can address multiple target frameworks (just replace net472 by net40) in @HamedFathi 's comment and hence can create a nuget package for both frameworks.
I would be really excited to use this package for .NET core based applications.

I think it is possible to support both variants.

Here's my latest Choose/When complete example together with vssbe scripts:
https://github.com/3F/vsSolutionBuildEvent/blob/master/vsSolutionBuildEvent/vsSolutionBuildEvent.csproj#L166
That allows more complex separate distribution even for vsix and nuget packages, like this:

https://github.com/3F/vsSolutionBuildEvent/releases/tag/0.12.11

But again, please use PR or wait for my time. I have other priority tasks at this moment, and just 2 hands.
Thanks for using.

Good news! I just released related vsSolutionBuildEvent project. And now I've plan to review current issue this or next week.
Also, I think about the same scheme like in https://www.nuget.org/packages/E-MSBuild/ where package dependencies:
.NETFramework 4.0          <<<<<<<<<
    Varhead (>= 1.14.0)

.NETStandard 2.0
    Microsoft.Build (>= 15.9.20)     <<<<<<<<<<<
    Microsoft.Build.Framework (>= 16.0.461)
    Varhead (>= 1.14.0)

Actual PR https://github.com/3F/MvsSln/pull/21 still is in progress and depends on this issue:
https://github.com/NuGet/Home/issues/8644
Join discussion or follow the news.

After short discussion with our team, I decided to do the following: #21 ( 'Update 1' section )
I'll update this PR very soon.

Everybody please note the following.
In fact,

modern Microsoft.Build assemblies are much more closely integrated with Visual Studio and much more difficult to maintain independently.

In our wiki I added a short article about Custom MSBuild Resolver.
Let me know if you have some misunderstanding. Here or in new issue.
PR is ready to merge. Let me just check something before it. And please feedback!


Filter form UI revamp

As ...
Nelly - Network Engineer
I want ...
An extensible and easy-to-use filter form UI in all object list views
So that ...
I have a better experience when trying to filter results which allows me to use all possible filtering options
I know this is done when...
Requirements

Not display all permutations of fields and lookup expressions simultaneously (by default)
Provide the user an ability to select any field and lookup expression combination supported by the FitlerSet to filter the table by
Provide the user with an ability to edit, view current filter arguments
Provide the user the ability to specify an arbitrary number of filters

Out of scope

Filtering arbitrary by nested lookup

Optional - Feature groups this request pertains to.

UI/UX

Database Changes
None
External Dependencies
None?
See #1608

Is "saving" a set of filters, or setting them as per-user defaults for a particular object-list, in scope?

Is "saving" a set of filters, or setting them as per-user defaults for a particular object-list, in scope?

No

Let's see if we can also get an answer to #2162 before this.

Closed with #2348.
Great work @timizuoebideri1!


fix: updates flask-openid to 1.3.0

this resolves issue #72 and allows the image to
be built locally
Issue #, if available:
#72
Description of changes:
Updates flask-openid to 1.3.0 allowing the image to be built locally with ./mwaa-local-env build-image
Tested with
Ran ./mwaa-local-env build-image and got a successfully built image that showed the UI.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

I'll create the PR to the 2.0.2 branch, not to main.


add: missed delimiter

great catch @neonlight911 - thank you for submitting this PR and thank you for your patience.


[core] Merge HUE-3102 to pysaml2 v4.9.0

What changes were proposed in this pull request?

re-apply HUE-3102 to pysaml2 v4.9.0

How was this patch tested?
End-to-end with SAML2 IDP
Please review Hue Contributing Guide before opening a pull request.

@ranade1
I generated a self-signed certificate and use following config in safety valve.
========================================================
[desktop]
redirect_whitelist="^/.$,^https://dev-12345678.oktapreview.com/.$"
[[auth]]
backend=libsaml.backend.SAML2Backend
[libsaml]
xmlsec_binary=/usr/bin/xmlsec1
metadata_file=/opt/cloudera/security/okta_idp_metadata.xml
key_file=/opt/cloudera/security/key.pem
cert_file=/opt/cloudera/security/cert.pem
key_file_password_script="/opt/cloudera/security/keypass.sh key_pass"
username_source=nameid
name_id_format="urn:oasis:names:tc:SAML:1.1:nameid-format:unspecified"
entity_id=yc-c7i5s
base_url=https://yc-c7i5s-1.xxxxx.xxxx:8889
authn_requests_signed=true
========================================================

Discussed with Romain. Erick mentioned a PR to pysaml2 in HUE-3102 commit, but it was rejected because of named pipe won't work on Windows. We will fork pysaml2 and apply patch in gethue.


[Camera] Camera View Tab overlaps with Game view

When moving Camera View Tab to the same node as Game View Tab, the cameras overlap causing the issue below:

This overlap happens with any of the two tabs selected.
Ideally, the Camera View would be hidden when in Game View, and Game View would be hidden when in Camera View.
The Camera works fine in other scenarios:

I tried various reasonable ways over yesterday to avoid writing to the texture of the camera view, so as not to create additional data. However, all attempts failed. I'll try to upload a fix with texture writing today

Sounds good

I think it fixed. But it's a floating bug. @naomihub can you confirm that it's fixed for you too?

Overlapping is fixed. This issue can be closed


how lists of response works?

can you explain - in a default setup how a long list of responses will change the possible weighty of response  from say doing them in separate lsits. That is how does a list of responses weight the setup to move through to those responses. I am asking mainly to understand how to author these, that is when to keep a long list going as opposed to making another list.
So then the difference between do this:
chatterbot.train([
"How are you?",
"I am good."
])
chatterbot.train([
"That is good to hear.",
"Thank you",
])
-- compared to do them in a list:
chatterbot.train([
"How are you?",
"I am good.",
"That is good to hear.",
"Thank you",
"You are welcome.",
])
thanks, Steve

Given the first example with two short lists, the chat bot would not make a connection between "I am good." and "That is good to hear.". That is, it would not know that "I am good." has "That is good to hear." as a possible response.
In the single longer list, each statement will be learned as a response to the previous.

Thanks, ok that came off as a dumb question, I was looking for the next level of detail or better yet I should have just explained my issue.  we have a chatterbot setup working with slack (wonderful program and setup! - thanks) and in our code the ability to correct on the fly. So this way, as one current approach, I can start with nothing, then start with a question and after that  answer live just use correction as the better answer. And built up a conversation that way. What the issue is, I appear to be  getting too many latter matching responses to my input questions then my answers. So a never said before "who are you" matches to the question "are you a bot" rather than the answer "yes I am a AI chatbot, my name is ivbot". It hit me that this live correction method just creates two phrase length Q then A lists rather than the longer conversation lists and that might be why this is happening (which means I should abandon this technique and go to the more conventioanl premaking lists) - so that drove my original question. It could be that I have a different mistake in my code that is causing this too. I am using
logic_adapter="chatterbot.logic.BestMatch",
statement_comparison_function=synset_distance,
any suggestions appreciated.
thanks, steve

Hmm, maybe you could set get_most_frequent_response:
response_selection_method=get_most_frequent_response

Making this change would cause your bot to select the response that occurs most frequently. If your correcting methods incremented the occurrence count of the correct response, it should cause that response to be selected the next time.


ImageCache.clear should maybe be called when views are detached from the engine

Currently, when a view is detached from the FlutterEngine, the rasterizer is torn down, and after https://github.com/flutter/engine/pull/33890, the skia resource cache is cleared as well. However, the image cache isn't currently and it's not as clear what the correct behavior should be.
When the engine is retained but the view is no longer attached (such as in hybrid use cases), it's unclear what the user's intentions are.
1- the application may be weaving in and out of Flutter views, in which case it may be more beneficial to keep the image cache
2- the application may not show a Flutter UI for a long time or when it does show a Flutter UI again, it would be for a different screen with minimal re-use of the previous screen's cache
For case 2, it would be great to have this as a global configurable (perhaps as an option in the embedding) such that it would require no additional wiring to not leave resident memory in the process when no Flutter screens are shown.
For case 1, the solution could be as simple as letting the user decide whether to listen to listen to https://api.flutter.dev/flutter/widgets/WidgetsBindingObserver-class.html and call ImageCache.clear when detached. But we should definitely have clear docs for this since it would otherwise be very difficult to discover.
@dnfield @zanderso

I think what's probably more important here is making it easier for users to discover what images (and perhaps pictures) are resident in memory, particularly those in the image cache, and how much memory they're taking. I vaguely remember devtools making this somewhat possible, but I haven't looked at that recently.
@jacob314 might remember, or @polina-c

I'm hesitant to change or add behaviore related to clearing the image cache though, since I think application developers are in a better position to know whether/when they need to do that - and they have no good way to recover from an incorrectly cleared memory cache, whereas in low memory situations the image cache will get cleared anyway (except, currently, in the FlutterFragment case because of #105346)

Per Dans comment: The cache is purged in low memory situations and developers have signals already to affect clearing the cache themselves. Closing this as WAI.


Can't make a promise resolve to a class if it extends EventEmitter and has a constructor

I'm stumped.
/* @flow */
// jshint ignore:start

import {EventEmitter} from 'events';

class Bar extends EventEmitter {
	constructor() {
		super();
	}
}

var pr = new Promise((resolve, reject) => {
	resolve(new Bar());
});

$ flow --version
Flow, a static type checker for JavaScript, version 0.14.0
$ flow

/private/tmp/a/foo.js:6:7,9: Bar
This type is incompatible with
/private/tmp/flow/flowlib_5929094/core.js:352:1,379:1: Promise

Found 1 error

Changing the promise to resolve to anything else (like new Object() or even new EventEmitter()) makes the error go away.
Removing Bar's constructor makes the error go away.
Making Bar not extend anything (and removing super()) or making it extend anything else like Object makes the error go away.

This is not the first time that Promise and EventEmitter have quarreled! I wonder if this is related to #545?

Yeah, looks like the same issue. I tried a similar test where I inlined the declarations of EventEmitter and events$EventEmitter into my file exactly as they were in Flow's node.js definitions file, and the error would go away, so I'm pretty confused. Maybe it has to do with Bar extending a class that was imported from elsewhere and declared in an external definitions file.

I tried a similar test where I inlined the declarations of EventEmitter and events$EventEmitter into my file exactly as they were in Flow's node.js definitions file, and the error would go away

Thanks @AgentME , this seems like a great workaround.

...but it doesn't work.

I did that just to probe what exactly triggered the bug. If you want to work around it and make a file type-check, try casting some things to any. This or something like it ought to work:
var pr: Promise<Bar> = new (Promise: any)((resolve, reject) => {
    resolve(new Bar());
});


Cypress: PSoC-6: Add SCB[spi] driver

Add master spi driver for PSoC-6. Initial config is added for cy8ckit_062_ble board.

buildkite errors are non related.

Rebase
Update west.yml with new hash value
Rework device_pm_control_nop in favor of NULL


EV3 programms do not run

Describe the bug

NEPO programs downloaded on EV3  (firmware v1.10E) C4ev3 via USB do not run. The programs immediately exit.

To Reproduce
Steps to reproduce the behavior:

Program a Nepo program for EV3
Connect the EV3 via USB and download the program to it via USB (C4ev3)
Start the program on the EV3
Program starts and immediately exits.

Device information

Type [Desktop]
OS: Windows 10, OSX 14.11
Browser Chrome, Firefox, Safari

Additionally, the EV3 often loses connection.

Kopiert aus einem anderen Ticket (dort als Kommentar):
charlie-chaperon commented last week
This problem appeared to me just recently (15th november 23) during my weekly robotics workshop. I investigated by using several platforms and I think I could locate the problem.
Tested systems
several versions of Win 10 Enterprise and Pro - 22H2 - Builds 19045.3636 & 19045.3208 & 19045.3570
Ubuntu Linux 22.04.3 LTS
Debian 12
tested browsers
Firefox 119.0 & 115.3.1.esr
Chrome 119.0.6045.124 and other recent versions
recent Brave browser
If you need more precise information about the systems, let me know. I stopped writing down details when I found the potential cause.
What does work, what does not?
Open Roberta lab 5.2.6 + "Save as" = works on none of the systems tested (Win10)
Open Roberta Lab 5.2.6 + upload-c4ev3 script (Linux) = does not work, error message during script
Lego Mindstorms app (1.5.2 & 1.5.0) on Win10 = works without problems
Open Roberta lab 5.2.6 + ev3dev uplaod via Wifi / internet = works without problems
uploading old (april 2023) uf2 programs = works on all linux (via script) and Win10 systems (via send to and direct copy to ev3)
last known working setup = 8th november 23 on Win10
I started a weekly robotics workshop where we used Open Roberta Lab. It worked without the issue until Wednesday 8th. On my workshop this wednesday (15th), none of the laptops and/or EV3 bricks did work. When I checked the issue at home, I realized that my other EV3 bricks which had no problems before showed the same problem [see tested systems].
symptoms
The usual upload process (save the program file from browser to brick) does not work. I don't get the beeping sound to confirm the upload. Although there is a "NEPOprog" visible after download (also on systems that did not have any present before), it seems to be transferred incorrectly. Instead of being placed inside a seperate folder (like NEPOprog/NEPOprog.uf2) it seems to be located directly at BrkProg_SAVE , next to the demo program, when I look for it via the EV3 screen. When I try to start the program (NEPOprog) the Mindstorms screen appears, but immidiately stops, so i can't even read the output.
When using the upload-c4ev3 script in linux, i get the following output:
linux console:
:~/Downloads$ upload-c4ev3 NEPOprog.uf2
Extracting UF2 file
UF2 file successfully extracted
Uploading program files ...
(ev3duder output follow)
USB connection established.
upload was successful.
USB connection established.
upload was successful.
USB connection established.
upload was successful.
USB connection established.
upload was successful.
(end of ev3duder output)
(ev3duder output follow)
USB connection established.
../prjs/BrkProg_SAVE/NEPOprog.elf: /lib/libc.so.6: version GLIBC_2.33' not found (required by ../prjs/BrkProg_SAVE/NEPOprog.elf) ../prjs/BrkProg_SAVE/NEPOprog.elf: /lib/libc.so.6: version GLIBC_2.34' not found (required by ../prjs/BrkProg_SAVE/NEPOprog.elf)
../prjs/BrkProg_SAVE/NEPOprog.elf: /lib/libc.so.6: version GLIBC_2.32' not found (required by ../prjs/BrkProg_SAVE/NEPOprog.elf) exec` status unknown.
(end of ev3duder output)
Program files uploaded
Done!
Wait for the robot to beep and then start the program
EV3 screen (while uploading):
executing shell cmd
This message is shown ~ half a second
suggested problem
It looks like a glibc related issue to me. My linux systems use glibc 2.36-9+deb12u3 or 2.35-0ubuntu3.4 respectively, but I assume my laptop's version is not the problem. Since the problem appeared just recently (Open Roberta Lab was released novemeber 14th) and I had no problems before, I suppose this happened due to recent changes.
workaround
Uploading older uf2 programs (april 2023) that I had still around worked just fine with the same laptops and without the error message. Using Open Roberta Lab 5.2.5 might just work.
to do
check if older version are affected

It seems that the C4EV3 compiler no longer works on a current Ubuntu version (we recently upgraded to 22.04). Since this project (the C4EV3 compiler) was a student project, we can't hope for updates but are currently working on a workaround here. But that will take some time. As soon as I have new information, I will let you know here.

I tested the uplaod with Lab version 5.2.8 and Windows 10 as well as Linux work again.
Thanks a lot!

solved by redeployment


Add fn as_bytes to PyByteArray

I understand the rational for to_vec (https://github.com/PyO3/pyo3/issues/373#issuecomment-512332696) but in my use case it won't never happen (I just need to read data and nothing will modified them).
So it'd be nice to avoid a useless copy here.

maybe it would work to return &Cell<[u8]>, since the Python value is both shared and mutable

&Cell<[u8]> would work, as far as I can tell, since it is both shared and mutable, and doesn't implement Send/Sync, so wouldn't be able to be passed into allow_threads, preventing access while the GIL is unlocked.
However, if Python interpreters are allowed to move the backing buffer during a garbage collection (or similar), then returning any reference wouldn't be safe, unless the buffer was somehow pinned. Not sure if interpreters are allowed to do that.

I think the bytearray can be resized at any point in python code just by using +=  operator, so returning a reference to data in the bytearray is definitely unsafe.
We can add pub fn data() -> *mut u8 which returns a pointer to the start of the backing buffer. It would need to have documented that the backing buffer may be relocated during any Python code execution so it's up to the user to tread carefully.
This would allow @calixteman to do what he wants via the unsafe api std::slice::from_raw_parts.

We can add pub fn data() -> *mut u8 which returns a pointer to the start of the backing buffer.

:+1:  but I'm also for unsafe fn as_bytes(&self) -> &[u8]. It would be convenient.


feat(system-update): remove datahub-update requirement on schema reg

When running DataHub without confluent schema registry, the datahub system-update process must function. These changes allow a mock schema registry client to remove this requirement.
Checklist

[ ] The PR conforms to DataHub's Contributing Guideline (particularly Commit Message Format)
[ ] Links to related issues (if applicable)
[ ] Tests for the changes have been added/updated (if applicable)
[ ] Docs related to the changes have been added/updated (if applicable). If a new feature has been added a Usage Guide has been added for the same.
[ ] For any breaking change/potential downtime/deprecation/big changes an entry has been made in Updating DataHub

The flaw in this initial design fails to address the consumer side. Need to handle both and in both cases we'll depend on the mock schema registry. Updates are in the works.


Сrash  if ship when  assembling    have Mekanism

https://paste.dimdev.org/huwexonewi.mccrash

I think I've fixed it in build 448. Can you verify its fixed?

50/50
Cables from Mekanism are transparent. Ic2 Cales i will test tomorrow

Crashes this tipe not found ^-^ ))

Nice! Another bug fixed.


MikroORM.init throws an error if driverOptions is undefined

Describe the bug
MikroORM.init throws TypeError: Cannot read properties of undefined (reading 'context') error if I set undefined to driverOptions.
If driverOptions is not set, no error is thrown.
Stack trace
C:\Users\Bob\Desktop\mikro-orm-issue\node_modules\@mikro-orm\sqlite\node_modules\@mikro-orm\knex\AbstractSqlConnection.js:106
        if (driverOptions.context?.client instanceof knex_1.knex.Client) {
                          ^
TypeError: Cannot read properties of undefined (reading 'context')
    at SqliteConnection.createKnexClient (C:\Users\Bob\Desktop\mikro-orm-issue\node_modules\@mikro-orm\sqlite\node_modules\@mikro-orm\knex\AbstractSqlConnection.js:106:27)
    at SqliteConnection.connect (C:\Users\Bob\Desktop\mikro-orm-issue\node_modules\@mikro-orm\sqlite\SqliteConnection.js:11:28)
    at async SqliteDriver.connect (C:\Users\Bob\Desktop\mikro-orm-issue\node_modules\@mikro-orm\core\drivers\DatabaseDriver.js:41:9)
    at async MikroORM.connect (C:\Users\Bob\Desktop\mikro-orm-issue\node_modules\@mikro-orm\core\MikroORM.js:59:28)
    at async Function.init (C:\Users\Bob\Desktop\mikro-orm-issue\node_modules\@mikro-orm\core\MikroORM.js:48:13)

To Reproduce
import { MikroORM } from "@mikro-orm/core";
import { SomeEntity } from "./someEntity";

export const someFunc = async () => {
  const orm = await MikroORM.init({
    type: "sqlite",
    entities: [SomeEntity],
    dbName: "./db.sqlite",

    driverOptions: undefined, // This line causes error. If this line is deleted, no error is thrown.
  });

 ...
};

Expected behavior
No error
Additional context
It seems<EMAIL_ADDRESS>does not have this issue.
Versions

Dependency
Version

node
16.15.0

typescript
4.7.4

mikro-orm
5.2.0

your-driver
sqlite

It needs to be defined, passing undefined is not supported, just like with all the other optional properties with default value. Why would you want to do that? "empty value" for driverOptions is an empty object, not undefined.


Initial zoom error

On first load the image is zoomed in one level when it should load zoomed all the way out.
Problem might be with index. js file line 20

is this still an issue?


117's HD Plugin with Night Time Feature

This feature would allow users to enable a night time feature for all of Gielinor that would turn on either manually by the user or automatically depending on the users time zone. (Similar to WoW)
I think it would be super cool to see iconic areas such as Varrock, Falador, etc. and what they would look like at night lit by the moon and fire.

Issues and suggestions for hub plugins must be made on that plugin's repository; they are not maintained by us. The repo is usually accessible from the Support link on the plugin.


⚠ Bofrost.de has degraded performance

In a6c3541, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1117 ms

Resolved: Bofrost.de performance has improved in caf0c54.


Is there any way to make it sortable?

Is there any way to make the added item sortable?

How do you propose the items to be sorted? which keys should be used?
Please give me a use case for when this would be valuable


Ignore pom artifact types

One of my projects has a dependency that looks like:
<dependency>
	<groupId>com.github.fommil.netlib</groupId>
	<artifactId>all</artifactId>
	<version>1.1.2</version>
	<type>pom</type>
</dependency>

This ends up going into the .ensime file along with all the dependency
jars, and then causes ensime to throw an error because it's not a zip
file. So let's just avoid putting anything into the the source roots
unless it's a jar type file according to the artifact data.

 You should sign our Contributor License Agreement in order to get your pull request merged.

lol, yeah, sorry

Hey, @fommil , since this dependency was your fault in the first place :smile:, you want to either merge this in or tell me if you have a better solution?


Spanish Localization

As a fellow Spanish enjoyer (I'm Latin-American), I would like to deliver the Spanish localization for the Jupyter Extension for Visual Studio Code. How does that sound?

Seems like the right approach is to use the VSCode localization pipeline, which can be achieved via: https://github.com/microsoft/vscode-engineering/wiki/Extensions-Localization
We already have an issue to do that: https://github.com/microsoft/vscode-jupyter/issues/9676
So, I will close this issue.


[GHSA-78xj-cgh5-2h22] NPM IP package vulnerable to Server-Side Request Forgery (SSRF) attacks

Updates

Affected products

Comments
ip package fixed the CVE in 2.0.1

Hey @NoFr1ends, I wrote a competing PR (sorry for not checking before) that correctly shows the fixed versions (as 1.1.9 is also a fixed version). Please have a look and consider closing your PR in favor of mine.

Closing this in favor of https://github.com/github/advisory-database/pull/3553.


Perform refactoring pass on UsdNotificationHandler

The UsdNotificationHandler needs a refactoring pass performed to clean it up a bit and make the code a bit more self documenting. There are two primary issues that need to be resolved, outlined below, but whoever performs this work should use judgement to decide if there are other improvements that could be made.

It's probably time to create a container class for ChangedPrims rather than using a pure vector. We have several cases where we are just passing in boilerplate parameters: in particular the propertyName parameter is only used by changed prims rather than added or removed prims. Abstracting adding entries to the collection would allow for self-documenting improvements, ergonomic improvements, and simplify where we need to handle logging.
We are also creating a fair number of unnecessary temporary objects in our usages of emplace_back that we should clean up. Clang-tidy is flagging these and the reasoning is outlined best by this article. Effectively the way we're using emplace_back right now doesn't fully provide us with all the benefits and is using more cycles than what is necessary.

For point 2 -- C++20 added support for emplace_back for struct member initialization. Once we're able to switch to C++20 we should make that change. This affects a few areas of the code, not just UsdNotificationHandler.
E.g. this code compiles in C++20 but not C++17


bug: WalletConnectConnector does not respect FallbackProvider from configureChains

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
I noticed today that when the Polygon Mumbai public/default RPC (https://matic-mumbai.chainstacklabs.com/) was down, despite configuring higher priority RPC nodes with configureChains, the failing public RPC node was the only node that was contacted for estimating gas or sending transactions.
Expected Behavior
The connector should either:

respect the FallbackProvider created by configureChains
provide a method for specifying fallback RPC nodes
provide a method for specifying a single RPC node

Steps To Reproduce
No response
Link to Minimal Reproducible Example (CodeSandbox, StackBlitz, etc.)
No response
Anything else?
No response

This is a bit tricky as you can only pass one RPC URL per chain to WalletConnect, and I don't think WalletConnect's EIP-1193 Provider has fallback functionality.
Let's move this to discussions and we can keep track of it there.


Beta2: IE 11 'Promise' is undefined

html2canvas beta2 seems to work in FireFox, and even in chrome, but doesn't seem to work in IE.
When running I will get the error: "'Promise' is undefined"
The promise started here is in Line: 615, Column: 9
I tried rerunning the the npm install and the grunt but this didn't work.
I can't seem to find where to resolve this in the code. It does seem that alpha1 does work with this promise (but not with other things I seem to need which are in beta2)
regards,
Bart

As mentioned in the readme, to support browsers without native Promise support, you'll need to include some Promise polyfill before including html2canvas. The reason html2canvas doesn't bundle one by default is that many applications might not need one and the type of Promise polyfill that gets included should be left up to the application.


Failed to receive SOCKS5 connect request ack

I am using Windows 7 machine to run powershell.exe. The Win 7 machine does have Internet access as well so it can connect out on the internet.

On the win7 client where socks proxy is running, I can see the connection from the kali machine from which the curl session above was run.

I have attached a Wireshark capture as well between kali (<IP_ADDRESS>) and win7 (<IP_ADDRESS>) where socks proxy is running.
Invoke-SocksProxy.wiresharkcap.pcapng.zip

Will investigate on Monday. In the meantime, you can try other clients. Proxychains should work. Or eventually socks4 with curl.

Cant reproduce that issue sorry. Are you sure your windows host is connected to the internet and can resolve google.com?
If your host is using a corporate proxy, this wont work natively.

I identified the issue on windows 7. The Ipv4.New() constructor does not exist. Will patch and update.

Hi @p3nt4
I am seeing the connections on the attacker side simply hanging indefinitely. I have the two machines - kali (<IP_ADDRESS>) and win7 (<IP_ADDRESS>)- both in a host-only adapter network connection in Virtualbox.
The hanging response is same whether I use -x socks5://: OR --socks5-hostname :.

Even Proxychains just hangs...

Ok, let's debug it:
Add:
$_ | Out-File "Error.log" -Append

In the catch of the socks connection manager block.
And please give me the output of the error.

Hey @p3nt4
So, I have done as you requested, and there was nothing written to the error log. So I have added extra Debug messages to see what is happening throughout the script. The modified script is below.
The output is shown below that is displayed within 'Normal.log' - so it appears that the output is trapped within the while loop.

`<#
.SYNOPSIS
Powershell Socks5 Proxy
Author: p3nt4 (https://twitter.com/xP3nt4)
License: MIT
.DESCRIPTION
Creates a Socks proxy using powershell.
Supports both Socks4 and Socks5 connections.
This is only a subset of the Socks 4 and 5 protocols: It does not support authentication, It does not support UDP or bind requests.
New features will be implemented in the future. PRs are welcome.
.EXAMPLE
Create a Socks proxy on port 1234:
Invoke-SocksProxy -bindPort 1234
Create a simple tcp port forward:
Invoke-PortFwd -bindPort 33389 -destHost <IP_ADDRESS> -destPort 3389
#>
[ScriptBlock]$TcpConnectionMgr = {
param($vars)
$srvConnection=$vars.srvConnection
$cliConnection=$vars.cliConnection
$Script = {
param($vars)
$vars.inStream.CopyToAsync($vars.outStream)
}
$cliStream = $cliConnection.GetStream()
$srvStream = $srvConnection.GetStream()
$vars2 = [PSCustomObject]@{"inStream"=$cliStream;"outStream"=$srvStream}
$PS = [PowerShell]::Create()
$PS.AddScript($Script).AddArgument($vars2) | Out-Null
[System.IAsyncResult]$AsyncJobResult = $null
$vars3 = [PSCustomObject]@{"inStream"=$srvStream;"outStream"=$cliStream}
$PS2 = [PowerShell]::Create()
$PS2.AddScript($Script).AddArgument($vars3) | Out-Null
[System.IAsyncResult]$AsyncJobResult2 = $null

try
{
    $AsyncJobResult = $PS.BeginInvoke()
    $AsyncJobResult2 = $PS2.BeginInvoke()
    while($cliConnection.Connected -and $srvConnection.Connected){
        sleep -m 100;
    }
}
catch {
}
finally {
    if ($cliConnection -ne $null) {
        $cliConnection.Close()
        $cliConnection.Dispose()
        $cliConnection = $null
    }
    if ($srvConnection -ne $null) {
        $srvConnection.Close()
        $srvConnection.Dispose()
        $srvConnection = $null
    }
    if ($PS -ne $null -and $AsyncJobResult -ne $null) {
        $PS.EndInvoke($AsyncJobResult) | Out-Null
        $PS.Dispose()
    }
    if ($PS2 -ne $null -and $AsyncJobResult2 -ne $null) {
        $PS2.EndInvoke($AsyncJobResult2) | Out-Null
        $PS2.Dispose()
    }
}

}
[ScriptBlock]$SocksConnectionMgr = {
param($vars)
function Get-IpAddress{
param($ip)
IF ($ip -as [ipaddress]){
return $ip
}else{
$ip2 = [System.Net.Dns]::GetHostAddresses($ip)[0].IPAddressToString;
}
return $ip2
}
$client=$vars.cliConnection
$TcpConnectionMgr=$vars.TcpConnectionMgr
$buffer = New-Object System.Byte[] 32
try
{
"Entering SocksConnectionMgr Try Block" | Out-File "Normal.log" -Append
$cliStream = $client.GetStream()
$cliStream.Read($buffer,0,2) | Out-Null
$socksVer=$buffer[0]
if ($socksVer -eq 5){
"Entering First If Statement Block" | Out-File "Normal.log" -Append
$cliStream.Read($buffer,2,$buffer[1]) | Out-Null
for ($i=2; $i -le $buffer[1]+1; $i++) {
if ($buffer[$i] -eq 0) {break}
}
if ($buffer[$i] -ne 0){
$buffer[1]=255
$cliStream.Write($buffer,0,2)
}else{
$buffer[1]=0
$cliStream.Write($buffer,0,2)
}
$cliStream.Read($buffer,0,4) | Out-Null
$cmd = $buffer[1]
$atyp = $buffer[3]
if($cmd -ne 1){
$buffer[1] = 7
$cliStream.Write($buffer,0,2)
throw "Not a connect"
}
"Middle" | Out-File "Normal.log" -Append
if($atyp -eq 1){
$ipv4 = New-Object System.Byte[] 4
$cliStream.Read($ipv4,0,4) | Out-Null
$ipAddress = New-Object System.Net.IPAddress(,$ipv4)
$hostName = $ipAddress.ToString()
}elseif($atyp -eq 3){
$cliStream.Read($buffer,4,1) | Out-Null
$hostBuff = New-Object System.Byte[] $buffer[4]
$cliStream.Read($hostBuff,0,$buffer[4]) | Out-Null
$hostName = [System.Text.Encoding]::ASCII.GetString($hostBuff)
}
else{
$buffer[1] = 8
$cliStream.Write($buffer,0,2)
throw "Not a valid destination address"
}
$cliStream.Read($buffer,4,2) | Out-Null
$destPort = $buffer[4]256 + $buffer[5]
$destHost = Get-IpAddress($hostName)
if($destHost -eq $null){
$buffer[1]=4
$cliStream.Write($buffer,0,2)
throw "Cant resolve destination address"
}
"[] Destination Host $destHost"| Out-File "Normal.log" -Append
$tmpServ = New-Object System.Net.Sockets.TcpClient($destHost, $destPort)
"[] Destination Host $destHost connected on $destPort, with tmpSrv $tmpServ"| Out-File "Normal.log" -Append
if($tmpServ.Connected){
$buffer[1]=0
$buffer[3]=1
$buffer[4]=0
$buffer[5]=0
$cliStream.Write($buffer,0,10)
$cliStream.Flush()
$vars = [PSCustomObject]@{"cliConnection"=$client;"srvConnection"= $tmpServ}
$PS3 = [PowerShell]::Create()
"[] Creating TCP Conection Manager"| Out-File "Normal.log" -Append
$PS3.AddScript($TcpConnectionMgr).AddArgument($vars) | Out-Null
"[] TCP Conection Manager created"| Out-File "Normal.log" -Append
[System.IAsyncResult]$AsyncJobResult3 = $null
$AsyncJobResult3 = $PS3.BeginInvoke()
"[] AsyncJobResult3 invoked"| Out-File "Normal.log" -Append
while($client.Connected -and $tmpServ.Connected){
"[] Looping and waiting"| Out-File "Normal.log" -Append
sleep -m 100;
}
"[] Completed Async Script"| Out-File "Normal.log" -Append
}
"Middle2" | Out-File "Normal.log" -Append
else{
$buffer[1]=4
$cliStream.Write($buffer,0,2)
throw "Cant connect to host"
}
}elseif($socksVer -eq 4){
"Else If statement" | Out-File "Normal.log" -Append
$cmd = $buffer[1]
if($cmd -ne 1){
$buffer[0] = 0
$buffer[1] = 91
$cliStream.Write($buffer,0,2)
throw "Not a connect"
}
$cliStream.Read($buffer,2,2) | Out-Null
$destPort = $buffer[2]*256 + $buffer[3]
$ipv4 = New-Object System.Byte[] 4
$cliStream.Read($ipv4,0,4) | Out-Null
$destHost = New-Object System.Net.IPAddress(,$ipv4)
$buffer[0]=1
while ($buffer[0] -ne 0){
$cliStream.Read($buffer,0,1)
}
$tmpServ = New-Object System.Net.Sockets.TcpClient($destHost, $destPort)
if($tmpServ.Connected){
"Sub-If statement" | Out-File "Normal.log" -Append
$buffer[0]=0
$buffer[1]=90
$buffer[2]=0
$buffer[3]=0
$cliStream.Write($buffer,0,8)
$cliStream.Flush()
$vars = [PSCustomObject]@{"cliConnection"=$client;"srvConnection"= $tmpServ}
$PS3 = [PowerShell]::Create()
$PS3.AddScript($TcpConnectionMgr).AddArgument($vars) | Out-Null
[System.IAsyncResult]$AsyncJobResult3 = $null
$AsyncJobResult3 = $PS3.BeginInvoke()
while($client.Connected -and $tmpServ.Connected){
sleep -m 100;
}
}
"Hi" | Out-File "Normal.log" -Append
}else{
throw "Unknown socks version"
}
}
catch {
$_ | Out-File "Error.log" -Append
}
finally {
if ($client -ne $null) {
$client.Close()
$client.Dispose()
$client = $null
}
if ($PS3 -ne $null -and $AsyncJobResult3 -ne $null) {
$PS3.EndInvoke($AsyncJobResult3) | Out-Null
$PS3.Dispose()
}
}
}
function Invoke-SocksProxy{
param (
        [String]$bindIP = "<IP_ADDRESS>",

        [Int]$bindPort = 1080

 )
try{
    $listener = new-object System.Net.Sockets.TcpListener([System.Net.IPAddress]::Parse($bindIP), $bindPort)
    $listener.start()
    write-host "Listening on port $bindPort..."
    while($true){
        $client = $listener.AcceptTcpClient()
        Write-Host "New Connection from " $client.Client.RemoteEndPoint
        $vars = [PSCustomObject]@{"cliConnection"=$client;"TcpConnectionMgr"=$TcpConnectionMgr}
        $PS3 = [PowerShell]::Create()
        $PS3.AddScript($SocksConnectionMgr).AddArgument($vars) | Out-Null
        [System.IAsyncResult]$AsyncJobResult3 = $null
        $AsyncJobResult3 = $PS3.BeginInvoke()
    }
 }
catch{
    write-host $_.Exception.Message
}
finally{
    write-host "Server closed."
    if ($listener -ne $null) {
              $listener.Stop()
       }
    if ($client -ne $null) {
        $client.Close()
        $client.Dispose()
        $client = $null
    }
    if ($PS3 -ne $null -and $AsyncJobResult3 -ne $null) {
        $PS3.EndInvoke($AsyncJobResult3) | Out-Null
        $PS3.Dispose()
    }
}

}
function Invoke-PortFwd{
param (
        [String]$destHost,

        [Int]$destPort,

        [String]$bindIP = "<IP_ADDRESS>",

        [Int]$bindPort

 )
$destIp = Get-IpAddress $destHost
$listener = new-object System.Net.Sockets.TcpListener([System.Net.IPAddress]::Parse($bindIP), $bindPort)
$listener.start()
write-host "Listening on port $bindPort..."
while($true){
    $client = $listener.AcceptTcpClient()
    Write-Host "New Connection"
    $serv = New-Object System.Net.Sockets.TcpClient($destIp, $destPort)
    $vars = [PSCustomObject]@{"cliConnection"=$client;"srvConnection"= $serv}
    $PS3 = [PowerShell]::Create()
    $PS3.AddScript($TcpConnectionMgr).AddArgument($vars) | Out-Null
    [System.IAsyncResult]$AsyncJobResult3 = $null
    $AsyncJobResult3 = $PS3.BeginInvoke()
}
if ($listener -ne $null) {
          $listener.Stop()
   }
if ($PS3 -ne $null -and $AsyncJobResult3 -ne $null) {
    $PS3.EndInvoke($AsyncJobResult3) | Out-Null
    $PS3.Dispose()
}
write-host "Connection closed."

}
function Get-IpAddress{
param($ip)
IF ($ip -as [ipaddress]){
return $ip
}else{
$ip2 = [System.Net.Dns]::GetHostAddresses($ip)[0].IPAddressToString;
Write-Host "$ip resolved to $ip2"
}
return $ip2
}
export-modulemember -function Invoke-SocksProxy
export-modulemember -function Invoke-PortFwd
`

I dont see any abnormal behaviour here. That's surprising.
What version of .Net are you using?


yearly cert bump

Followed the steps in https://github.com/stackrox/scanner/blob/master/scripts/cert/README.md
Also added a yearly reminder on Slack. Hoping we don't need to do this anymore, though... :)

@RTann can you confirm the osci failure isn't related to this change? Doesn't seem so to me, but wanted to double-check before giving an approval

Yeah @BradLugo from what I can tell the failures are a combo of things addressed in Dave's PR https://github.com/stackrox/scanner/pull/1737. Those nils are due to the timeout. Going to merge


Is there a way to make the formatter ignore reader macros?

I use the #+(or) reader macro quite a bit to comment out expressions and the auto-formatter is formatting them like regular comments.  Any chance the formatted could ignore them?

Yes, that should be doable. Do you have an example of what it's doing versus what it should do?

Totally.  Here are some examples of how it should look:
(defun test (a &optional b)
  #+(or)
  (+ 1 1)
  (+ 2 2))

or
(defun test (a &optional b)
  #+nil
  (+ 1 1)
  (+ 2 2))

or
(defun test (a &optional b)
  #+todo
  (+ 1 1)
  (+ 2 2))

and here is how it looks:
(defun test (a &optional b)
  #+
  (or)
  (+ 1 1)
  (+ 2 2))

and
(defun test (a &optional b)
  #+nil
  (+ 1 1)
  (+ 2 2))

and
(defun test (a &optional b)
  #+todo
  (+ 1 1)
  (+ 2 2))

You'll notice the ones on the atoms work as expected.  Some people use #+nil or #+todo instead of #+(or) for this same purpose, so I included all 3 examples.  The value after #+ can be anything that isn't defined as a feature.  I like #+(or) because it is easy to switch to #+(and) if I want to enable it temporarily.  When Slime encounters that particular reader macro, it will actually set the color of the s-expression that follows it to the comment color, which is cool, but probably not necessary.  If it kept the #+ along with the following s-expression or atom immediately after it, that would do it for me.
Sorry if I made this more confusing.  I tend to have an easier time writing things if I have a few examples, so I included more than one.  Let me know if I made it worse.

Ok, thanks. That's what I thought it was probably doing. It's treating the open parens as a separator and splitting them. I was going to ask about coloring what's after it as a comment. I can add that, too.

Should be fixed in version 0.0.9.

Thanks for the quick turnaround on this!


Spectrogram example notebook

Adding an example notebook for audio processing with spectrogram augmentation

@SundarRajan28 also please add a readme to state pre reqs like librosa, matplotlib modules etc.

@LakshmiKumar23 Can you share how you are building the example notebook so that I can replicate the issue? I have not encountered this error.

@LakshmiKumar23 Can you share how you are building the example notebook so that I can replicate the issue? I have not encountered this error.

@SundarRajan28 I am running it on my vscode

@rrawther I have resolved the review comments

@LakshmiKumar23: Can you please see if review comments are addressed and approve if it is ready to be merged


Discrepancy in front-end and back-end text for VE

If you look at the front end of the VE chapter under "An illustrative example" (as far as I can tell), you see in the first line "...simplicity that we a \n given a chain..."; but the text in the source clearly has "....we are given a chain....".
Is it just me or is there something going on here?

This was because the Markdown files get compiled into HTML for output. Previously, someone had to manually re-compile the HTML every time the Markdown files were changed, and it looks like someone forgot to do that. However, we have since changed this repo to automatically recompile, so this shouldn't be an issue anymore!


Feature: this.skipDescendants()

Yeah, seems like this.block().
E.g.
var treedata = [{
  a: {
    block: true,
    ab: {
      name: "ab leaf"
    }
  },
  b: {
    stop: true,
    bc: {
      name: "bc leaf",
    }
  },
  c: {
    cd: {
      name: "cd leaf"
    }
  }
}]

result = traverse(treedata).map(function (item) {
  item.visited = true;
  if (item.block) {
    this.block();
  } else if (item.stop) {
    this.stop();
  }
});

console.log(JSON.stringify(result, null, 2));

Output:
[
  {
    "a": {
      "block": true,
      "ab": {
        "name": "ab leaf"
      },
      "visited": true
    },
    "b": {
      "stop": true,
      "bc": {
        "name": "bc leaf"
      },
      "visited": true
    },
    "c": {
      "cd": {
        "name": "cd leaf"
      }
    },
    "visited": true
  }
]

This issue can probably be renamed to "Document this.block and this.stop"

It would be nice if this.block and this.stop was documented, even with this example :)
These are useful when you want to traverse one structure (without modifying it) and copy selective data into a fresh empty object.
I've only tested this.block which works fine :)

https://github.com/substack/js-traverse/issues/26


Remove independent errs package

If we want external code use errors.Is(), will be enough to have special errors inside ingestion package, not globally.

done with https://github.com/wtask-go/mixpanel/pull/7