Folder added to workspace displays '!' warning in Explorer if folder is hosted by a FileSystemProvider implemented in an extension that is not yet activated

Issue Type: Bug

Install the MemFS extension from Marketplace, authored by @jrieken
Create an empty .code-workspace file with this content:

{
	"folders": [],
	"settings": {}
}

Open it as a workspace.
Edit the workspace file to add one folder that uses the memfs scheme with the MemFS extension implements.

{
	"folders": [
		{
			"name": "MemFS - Manually added",
			"uri": "memfs:/"
		}
	],
	"settings": {}
}

After saving the updated workspace file, observe that Explorer shows the added folder but decorates it with a warning exclamation point. Hover over the folder label to see the message \ ● Unable to resolve workspace folder

I think the problem occurs because even though the MemFS extension specifies "onFileSystem:memfs" as one of its activation events, when VS Code adds the new folder to the currently-open (though empty) workspace it apparently doesn't activate the extension in time before going looking for a .vscode folder within the new memfs root.
By running Developer: Show Running Extensions before doing this we can see that MemFS does get launched. I guess this just doesn't happen soon enough.
By contrast, if the memfs root is defined in the workspace at the point where the .code-workspace file is loaded the extension activates early enough and no warning indicator appears. This can be observed by running the Developer: Reload Window command. The memfs:/ root now displays normally.
VS Code version: Code - Insiders 1.56.0-insider (0310f02dc5d834bb3a7cc421ea5374aec8d011f1, 2021-04-23T05:13:46.805Z)
OS version: Windows_NT x64 10.0.19042

@gjsjohnmurray I see you are already deep down investigating this. If you have a good proposal for a PR how to fix this I would be open to review.
Thanks a lot for letting us know about this.

@isidorn is it true that the extension host won't be running at the time that this gets called?
https://github.com/microsoft/vscode/blob/66f0c29ddb32bdfc7084b77a34e0095e23f00889/src/vs/workbench/services/configuration/browser/configurationService.ts#L382-L403
The comments suggest that's true, in which case the line 400 call to checkAndMarkWorkspaceComplete goes via toValidWorkspaceFolders into FileService. Then resolve -> doResolveFile -> withProvider -> activateProvider which I think would normally be where the onFileSystem extension activation event would get the FSP extension activated. But if there's no EH then I assume this fails, so withProvider throws.

@sandy081 might be better to answer this.

@sandy081 I began trying to enhance this in WorkspaceService:
https://github.com/microsoft/vscode/blob/8fa05403b6b8623f2b58fad718edf596b2b4058f/src/vs/workbench/services/configuration/browser/configurationService.ts#L823-L839
My idea was to gather the failures during the first (currently, the only) pass, then add a second pass which begins by awaiting whenInstalledExtensionsRegistered in the extension service. But the workspace service doesn't currently have access to the extension service, right?

@gjsjohnmurray I do not think above validation is causing the warning in the explorer. Above validation is only to check if the entry is a directory or not.
The warning in explorer is shown when it cannot resolve the workspace folder. I think following events are happening in this case

A new folder is added to workspace file as first folder
Workspace service triggers change event about this and there are following listeners which are prominent in this scenario

Extension host is restarted as first folder is changed
Explorer shows the folder

I think explorer is not recovering from extension host restart and hence warning is shown even after ext host is restarted. @isidorn Please check and confirm.

@sandy081 it's not the !result.isDirectory which is triggering. Rather, the resolve on line 829 is throwing, and the catch merely writes it to the console log and continues.

Rather, the resolve on line 829 is throwing, and the catch merely writes it to the console log and continues.

Yeah that is intended and I think this is nothing to do with the warning in the explorer.

@sandy081 ok, so it seems like the explorer needs a fresh start on et host restart... Thanks for investigating

This should behave a bit better now with our aggressive explorer refresh when window gains focus again or explorer gets visible.
Closing as no other action planned for now


Passing function to format data

More of a question/request, but is there a way to pass a function to the column definitions to format data, for example:
{
title: 'Year',
key: 'date',
function: function(date.getFullYear())
}

I solved my issue by modifying the code a bit to accept a parse function. see below:
format for column object:
{title:'Date',key:'Created',parse:function(data){var date = new Date(data); return date.getFullYear();}}
Modified code (added else statement to initData:
function initData(params) {
    // Transform from String[] to Object[]
    if (typeof params.columns[0] === 'string') {
        params.data.forEach(function (row, i) {
            var obj = {};
            for (var j = 0; j < row.length; j++) {
                obj[j] = params.data[i][j];
            }
            params.data[i] = obj;
        });
        params.columns.forEach(function (title, i) {
            params.columns[i] = {title: title, key: i};
        });
    }
    else{
        params.columns.forEach(function (title, i){
            if(title.hasOwnProperty('parse')){
                params.data.forEach(function (row, j) {
                    row[title.key] = title.parse(row[title.key]);
                });
            }
        });
    }
}

Looks good! Will probably implement that in the next version of the library or accept a pull request if you create one.

K - one other thought - this works well, but currently if i try to use the same key twice but parse differently, it overwrites because its writing back to the same data array passed in... it might be better to use the init function to always parse out the passed in data into a new array with keys generated by index...
example..
var columns =  [
{title:'Year',key:'Created',parse:function(data){var date = new Date(data); return date.getFullYear();}},
{title:'Month',key:'Created',parse:function(data){var date = new Date(data); return date.getMonth();}}
}
This won't work without creating an entirely new array for data with new key values. Should be really easy, but will impact performance a little bit. May do this tomorrow.

It has been unique keys by design. What is your use case for multiple keys with the same name? Anyway, I will consider allowing that in future versions.

Well, the case that comes to the top of my head is the one mentioned above, where your data contains one date field (called Created) that you would like to parse into two separate columns on the PDF, like Day and Time (Day: Monday; Time: 5:00pm). Currently, they both get parsed to the row property for Created, so they overwrite each other.
Another example is when your data contains concatenations that you would like to parse to separate fields. For example, if your data includes the field Name which contains data formatted like: 'Hogan, Joe' - you may want to parse that into First Name: 'Joe' and Last Name 'Hogan' columns on your PDF.
Another example is when you have data in an array, like data = [{id:1, people: {results: [{name: 'Joe', phone: '123-456'}, {name:'John', phone: '789-1011'}]}}]
Using the parse function you can parse this out however your want, but only to a single column. You may, instead, want to parse it to a Names column and a Phone Numbers column...

I think the benefits of forcing unique keys for columns outweigh the ones you mention. For example it means that:

The order of the columns and headers doesn't matter
You will always get the right data for the right header.
You can style a specific column in the drawCell method/option

Your use cases are interesting and are definitely something I will look into however. Even if unique keys are forced your use cases could be implemented by simply rename the created columns to created_month and created_year before calling the plugin method. Or omit the keys completely and use the array initialize option. It's not documented yet, but looks like this:
var cols = ["Header 1", "Header 2", "Header 3"];
var data = [
    ["Data 1", "Data 2", "Data 3"],
    ["Info 1", "Info 2", "Info 3"],
    ["Thing 1", "Thing 2", "Thing 3"]
];

var doc = new jsPDF('p', 'pt');
doc.autoTable(cols, data);
doc.save("table.pdf");

I ended up not adding a parse function, but the createdCell hook can be used in version 2.0 to modify the cells data after it has been parsed by the plugin.


Handle CUPTI_ERROR_MAX_LIMIT_REACHED and disable TF tracing

This PR fixes an issue caused by multiple processes attempting to use CUPTI simultaneously. For example, when using NVIDIA Nsight System with TF tracing at the same time (both rely on CUPTI APIs), users will get wrong TF tracing and incomplete Nsight reports.
According to https://docs.nvidia.com/cupti/Cupti/r_limitations.html#r_limitations:

The application which calls CUPTI APIs cannot be used with Nvidia tools like nvprof, Nvidia Visual Profiler, Nsight Compute, Nsight Systems, Nvidia Nsight Visual Studio Edition, cuda-gdb and cuda-memcheck.

So, we should avoid calling CUPTI APIs if the function cuptiSubscribe() detects CUPTI_ERROR_MAX_LIMIT_REACHED.
In this PR, we let TensorFlow issue a warning when that CUPTI_ERROR_MAX_LIMIT_REACHED is detected and skip all the following CUPTI calls. Then, the external profiling tool can have exclusive access to CUPTI APIs.
FYI. @nluehr

CC @jbaiocchi (I tried assigning you as the reviewer but for some reason github does not let me)

Unfortunately supporting external tools concurrently with TensorFlow tracing and profiler is not a use case that we support. Our tools are not designed to work concurrently with NVIDIA tools, hence this is the expected behavior.
It would help to explain the problem you are trying to solve, that would involve requiring Nsight Systems?

@yisitu This PR's purpose is to support the external profiling tools and at the same time not to affect existing TF tracing.
One use case: suppose the example.py has TF tracing in it. If we do nsys profile python example.py, both Nsys tracing and TF tracing will collect nothing. With this change, TF tracing will be ignored and Nsys gets correct report. And python example.py won't be affected.

I know that in your case, once subscribe exist, it probably lasts for the life time of the process. however  there might be other cases that subscriber can collide with each other,  e.g. on-demand with tensorboard callback, we don't want to disable it permanently, so the next call to Enable() (when all other subscribers have expired) should be successful,
how do you think?

In your use case, could you have disabled TF tracing in the first place
since you will be using nsys profile?
On Wed, Mar 18, 2020 at 11:20 AM Jie Sun<EMAIL_ADDRESS>wrote:

I know that in your case, once subscribe exist, it probably lasts for the
life time of the process. however there might be other cases that
subscriber can collide with each other, e.g. on-demand with tensorboard
callback, we don't want to disable it permanently, so the next call to
Enable() (when all other subscribers have expired) should be successful,
how do you think?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/tensorflow/tensorflow/pull/37674#issuecomment-600787844,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AONQZYDG5A66HH7KDS7APULRIEGGDANCNFSM4LN6NOMA
.

@yisitu Do you mean by changing the python script?

Yes, I imagine that the python script was instrumented to enable profiling
in the first place.
On Wed, Mar 18, 2020 at 11:39 AM Kaixi Hou<EMAIL_ADDRESS>wrote:

@yisitu https://github.com/yisitu Do you mean by changing the python
script?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/tensorflow/tensorflow/pull/37674#issuecomment-600796634,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AONQZYHPOZDH3C6PB6X6RNTRIEIMPANCNFSM4LN6NOMA
.

@trisolaran Yes, can you confirm my understanding of your example is correct? So, before this change, TF can handle the case that issues multiple subscriber sessions: if subscriber exists, ignore it; if not exist, create it. However, after this change, only the first session can create the subscriber but all the other sessions cannot create new subscriber anymore.

yes. I think that you statement is right.
I wonder if we can fail silently for this time, and able to enable cupti tracer next time. ( not applicable to your case though).

I am wondering adding more conditions in the Disable() might solve the problem. Basically, we need to make sure we still unsubscribe if we know the existing subscriber is from TF not external tools. Something like (isSet() is an imaginary function for now.)
void CuptiTracer::Disable() {
  if (!subscriber_existed || (subscriber_existed && subscriber_.isSet())) {
   //do as normal
  }
}

@houtoms Can you please resolve conflicts? Thanks!

It has been 14 days with no activity and the awaiting response label was assigned. Is this PR still valid? Assigning the stalled label. Please comment to reassure me that this is still being worked on.

@houtoms Any update on this PR, please. Thanks!

Hi, I rebased the code to include a new design to disable all TF tracers when the external profiler is detected.

It should work. I think you are referring changes here: https://github.com/tensorflow/tensorflow/commit/b97cc6fd437e3ab8b788853558733f9420125245, which I feel is similar to the original design that disables the cupti tracer then external profiler is detected. But, as we discussed previously, my understanding is that we'd like to turn off all TF tracers if external profiler is detected. That's why the new commit here is to put the detection code to an earlier phase of the profiler initialization session. Anyway, if you are OK with the original plan, we can close this one.

thanks Kaixi.


Replace the search in Files with search from other apps

Before you start...

[X] Have you refreshed the page? You might be using an old version of the website.
[X] Have you checked if someone already requested this feature? If they have, post a comment with your use case instead.

Description
Can a different search provider be used for Files (eg Everything - https://www.voidtools.com)?
Thank you for your time.
Alternative solutions
No response
Relevant Assets
No response

Thanks for the feedback, We're tracking this in https://github.com/files-community/Files/issues/5845. You can check there or click subscribe in the right sidebar for updates.
This is also not the issue tracker for Files please use https://github.com/files-community/Files/issues/new/choose


Memory leak in decoding process

🐛 Bug Report
I'm trying to evaluate Encodec for streaming audio use cases, however I noticed that the decoding step seems to accumulate memory very quickly over time. If I turn off decoding, memory usage stays constant. I looked at the code though, I'm not sure why?
To Reproduce
from encodec import EncodecModel
from encodec.utils import convert_audio

import torchaudio
import torch

# Instantiate a pretrained EnCodec model
model = EncodecModel.encodec_model_24khz()
model.set_target_bandwidth(6.0)

# Load and pre-process the audio waveform
wav, sr = torchaudio.load("CantinaBand60.wav")
wav = wav.unsqueeze(0)
wav = convert_audio(wav, sr, model.sample_rate, model.channels)

# Extract discrete codes from EnCodec
frames = []
for offset in range(0, 1440000, 480):
    print(offset)

    encoded_frames = model._encode_frame(wav[:, :, offset: offset + 480])

    frames.append(model._decode_frame(encoded_frames))

# merge the decoded frames
decoded = torch.cat(frames, dim=2)

# save the decoded audio
torchaudio.save("decoded.wav", decoded, model.sample_rate)

Expected behavior
Memory shouldn't be persisted across frames.
Actual Behavior
Memory usage grows over time eventually resulting in OOM.

you need to wrap the eval of the model with torch.no_grad(), I've updated the readme to include it !

I'll be closing the issue, please reopen if that didn't solve your issue.


unsupported engine

in installation there is an error unsupported engine.

@platini76 can you provide logs? and your system info?

Are you installing homebridge-music on macOS?

no through homebridge conif ui x    on linux

I see now only for mac os... so please inform config-ui-x that your plugin appear on their install page.


apply_transform not in keras.preprocessing.image anymore. DOC update needed.

According to https://keras.io/preprocessing/image/#apply_transform , apply_transform is still under  keras.preprocessing.image which is not the case.

In the doc: https://keras.io/preprocessing/image/#imagedatagenerator-methods
In the code: https://github.com/keras-team/keras-preprocessing/blob/master/keras_preprocessing/image.py#L1095
I'm not sure I get your issue?

Interesting. Trying from keras.preprocessing.image import apply_transform gives me a name error! (Using python3 - Anaconda - Tensorflow backend)

it's part of ImageDataGenerator now

@Dref360 No, it is not resolved. Apparently, the documentation is wrong. Please look at the referenced pull request.

Yes, you're right! I was talking about the fact that apply_transform was not under ImageDataGenerator methods, I didn't look at the signature.


[BUG] Blob Storage URI should not escape segment separator (/)

Describe the bug
Azure Storage Blob names that are multi-segment (resembles virtual directories) generate absolute URIs that have escapes on the path separator (/). If pass the result to another service, the URL may be considered invalid.
Expected behavior
The URI escaping should be done as a URI and not a data segment.
Example https://contoso.blob.core.windows.net/backups/db1/20210101.bacpac
Actual behavior (include Exception or Stack Trace)
What is the actual behavior?
Example https://contoso.blob.core.windows.net/backups/db1%2F20210101.bacpac
To Reproduce
Steps to reproduce the behavior (include a code snippet, screenshot, or any additional information that might help us reproduce the issue).
var cs = "UseDevelopmentStorage=true;"
var bc = new BlobClient(cs, "azuresqlbackups", "db1/20210101.bacpac");
var bacpacUri = bc.Uri.AbsoluteUri;

The code in the BlobUriBuilder calls Uri.EscapeDataString(...) instead of Uri.EscapeUriString(...).
Environment:

Name and version of the Library package used: [e.g. Azure.Storage.Blobs 12.8.0]
Hosting platform or OS and .NET runtime version (dotnet --info output for .NET Core projects): .NET 5.0.102
IDE and version : [e.g. Visual Studio 16.8.4]

Thank you for your feedback.  Tagging and routing to the team best able to assist.

Thank you for your feedback.  Tagging and routing to the team best able to assist.

Hi, I can reproduce your issue.
@seanmcc-msft What do you think about this? Would it be a breaking change to change the way we are encoding the slashings in the Uri. It is quite odd we are encoding the path but nothing past the container.

Hi, I can reproduce your issue.
@seanmcc-msft What do you think about this? Would it be a breaking change to change the way we are encoding the slashings in the Uri. It is quite odd we are encoding the path but nothing past the container.

So this escaping of the '/' in the blob name in the URI is by design. Please see issue https://github.com/Azure/azure-sdk-for-net/issues/15064 and the other issues that are referenced to it.

So this escaping of the '/' in the blob name in the URI is by design. Please see issue https://github.com/Azure/azure-sdk-for-net/issues/15064 and the other issues that are referenced to it.

Hi @amnguye ,
It makes sense to have the blob name to be escaped. However, unescaping (reverse) can cause other parts that need to remain escaped to be undone. For example: https://constoso.blob.core.windows.net/backups/db%201%2F202010101.bacpac
would become https://constoso.blob.core.windows.net/backups/db 1/202010101.bacpac.
However, if you use Uri.EscapeUriString(...) instead of Uri.EscapeDataString(...), the generated one would be https://constoso.blob.core.windows.net/backups/db%201/202010101.bacpac.
Note: it is possible to unescape data string using Uri.UnescapeDataString(...) but there is no way to unescape URI strings.

Hi @amnguye ,
It makes sense to have the blob name to be escaped. However, unescaping (reverse) can cause other parts that need to remain escaped to be undone. For example: https://constoso.blob.core.windows.net/backups/db%201%2F202010101.bacpac
would become https://constoso.blob.core.windows.net/backups/db 1/202010101.bacpac.
However, if you use Uri.EscapeUriString(...) instead of Uri.EscapeDataString(...), the generated one would be https://constoso.blob.core.windows.net/backups/db%201/202010101.bacpac.
Note: it is possible to unescape data string using Uri.UnescapeDataString(...) but there is no way to unescape URI strings.

As per the issue I linked, the service considers db1/20210101.bacpac to be the same blob as db1%2F20210101.bacpac. As the issue goes onto explain, if we did not do this, others regarding encoding blob names correctly and sending the correct URL would not be possible.
We recommend to use BlobClient.Name to extract out the name of the blob.

As per the issue I linked, the service considers db1/20210101.bacpac to be the same blob as db1%2F20210101.bacpac. As the issue goes onto explain, if we did not do this, others regarding encoding blob names correctly and sending the correct URL would not be possible.
We recommend to use BlobClient.Name to extract out the name of the blob.


Admin Doc: Running (secure!) without meteor

As a 4minitz admin I want to run the webapp secure without meteor - and my own mongoDB instance.

[x] Put instructions for backup into admin doc

[x] backup/restore of mongodb
[x] backup of attachments directory

[x] Put step-by-step instructions how to compile with meteor and then deploy and run 4Minitz without meteor
[x] Give some security hints, especially regarding https:// to 4Minitz and secure MongoDB URLS ( https://docs.mongodb.com/v3.2/tutorial/configure-ssl/)

[ ] Provide a Linux/bash run_sample-Script with:

     MONGO_URL=mongodb://4minitzadmin:password@localhost:27017/4minitz meteor run --settings settings.json```

- [ ] Add hints to the admin-Doc on how to secure the production MongoDB. Mention the run_sample script above and how to customize. And mention how to secure the a mongo DB.
    db.createUser(
      { user: "4minitzadmin",
         pwd: "password",
         roles: [
           { role: "readwrite", db: "4minitz" }
         ]
     })

- [ ] Provide a LOUD & CLEAR server side log, if 4minitz connects without user & password to mongoDB. Check env var.  MONGO_URL

We'll call it a day.  ;-)


MySQL support 0000-00-00 00:00:00 but sqlpad throw Invalid time value

@rickbergfalk, please look this issue, is it belong to the bug of the sqlpad, or that need a zero date to null feature comes from the mysql library?

I'm not sure what the solution would be here. This looks to be a limitation of the driver (and JavaScript) https://github.com/mysqljs/mysql/issues/1040.
The mysql driver has a dateStrings option https://www.npmjs.com/package/mysql#connection-options to return dates as strings instead of JS date objects, which might work... but SQLPad inspecting the string result might try and convert it back to a date. Not sure what will happen to the 0 dates in that case.

Thank you. I setup the dateStrings to true ,and it's fixed, and can query successfully and display zeros format string when the value of datetime type column is zeros. I have created one PR #1023 , Please review it, if you have time. @rickbergfalk


Need general flops metric from ncu report

Let's differentiate this with the current tflops() metric, the ncu report is the hardware flops where the current tflops is analytic flops (calculated from math).

how about we use a shorter name hw_tflops or ncu_tflops?

how about we use a shorter name ncu_tflops?

sure. will add this feature later.

@xuzhao9 What about using Triton's Proton profiler metadata metric scope instead? see Triton and cuBLAS matmul kernel

@xuzhao9 What about using Triton's Proton profiler metadata metric scope instead? see Triton and cuBLAS matmul kernel

We had some discussions here https://github.com/pytorch/pytorch/pull/136169. I think we are open to do in proton way too if anyone wants to help. I've got a ncu version locally and will push it later.

@xuzhao9 Thanks for the clarification about the target FLOPs number, “analytic flops” (e.g. user defined formula like in Proton) vs NCU GPU hardware counters for precise flops counting. I’m also currently using ncu for automatically profiling Torch Inductor Triton GPU kernels, based on Torch official documentation. @register_metric()  is similar to Proton’s metric in scope and metadata_fn
@FindHao I have a simple prototype for automatically annotating using Proton scope contextmanager, as discussed in the issue. Before going further with Proton, I will wait for your ncu runner exemple.

@xuzhao9 Thanks for the clarification about the target FLOPs number, “analytic flops” (e.g. user defined formula like in Proton) vs NCU GPU hardware counters for precise flops counting. I’m also currently using ncu for automatically profiling Torch Inductor Triton GPU kernels, based on Torch official documentation. @register_metric() is similar to Proton’s metric in scope and metadata_fn
@FindHao I have a simple prototype for automatically annotating using Proton scope contextmanager, as discussed in the issue. Before going further with Proton, I will wait for your ncu runner exemple.

The key part to obtain flops is done here https://github.com/pytorch-labs/tritonbench/blob/main/tritonbench/components/ncu/analyzer.py#L86 . The remaining parts are aggregation and add metrics to results. will do it later.


Allow quoted path with spaces in it.

I started a project using the KickStarter in my C:\Users\boliver\documents\Visual Studio 2012\projects directory and had an issue where the space in the path was causing an invalid path. I change my local Program.cs to put quotes around the script path, but aliasql.exe left the trailing quote on. This removes the trailing quote and ensure there is a trailing backslash on the script path.

I came up with a slightly simpler way to make this work. Here is the commit. Please let me know if you have trouble after this change. https://github.com/ClearMeasure/AliaSQL/blob/e03cbad1110c4fb68b31aab30a3b89387436b73d/nuget/content/Program.cs.pp

Thanks Eric. I think that will work. I'll have a look this evening.
On Sun, Jan 25, 2015 at 1:17 PM, Eric Coffman<EMAIL_ADDRESS>wrote:

Closed #6 https://github.com/ClearMeasure/AliaSQL/pull/6.
—
Reply to this email directly or view it on GitHub
https://github.com/ClearMeasure/AliaSQL/pull/6#event-224345321.

I tested it and it's working fine.
Thanks again.
On Tue, Jan 27, 2015 at 6:15 AM, Bill Oliver<EMAIL_ADDRESS>wrote:

Thanks Eric. I think that will work. I'll have a look this evening.
On Sun, Jan 25, 2015 at 1:17 PM, Eric Coffman<EMAIL_ADDRESS>wrote:

Closed #6 https://github.com/ClearMeasure/AliaSQL/pull/6.
—
Reply to this email directly or view it on GitHub
https://github.com/ClearMeasure/AliaSQL/pull/6#event-224345321.


fix: ignore Yarn Berry files in .gitignore

Description
I've setup Payload as a sub-package in a monorepo with Yarn 3. To avoid commiting .yarn cache etc, this adds the respective folders to .gitignore

[x] I have read and understand the CONTRIBUTING.md document in this repository

Type of change

[x] New feature (non-breaking change which adds functionality)

Checklist:
n/a

Just so I understand accordingly, you need this because you are maintaining a fork of Payload directly within your own monorepo?

@chris-schra Thank you!
Releasing in 0.18.6 shortly.


Question about two devices record at same time

Hi,  this is the code from https://github.com/spatialaudio/python-sounddevice/issues/338, operating system is windows10.
from queue import Queue

from sounddevice import InputStream
from soundfile import SoundFile
import sounddevice as sd

fs = 32000
channels = 1
device1 = 1
device2 = 2

#print("default.device:\n", sd.default.device)
print("devices list:\n", sd.query_devices())
#print(sd.query_hostapis())

def create_callback(q):
    def callback(data, frames, time, status):
        if status:
            print(status)
        q.put(data.copy())
    return callback

q1 = Queue()
q2 = Queue()

stream1 = InputStream(
    device=device1,
    samplerate=fs,
    channels=channels,
    callback=create_callback(q1),
)
stream2 = InputStream(
    device=device2,
    samplerate=fs,
    channels=channels,
    callback=create_callback(q2),
)

sf1 = SoundFile(
    file=f"rec_device_{device1}.wav",
    mode="w",
    samplerate=int(stream1.samplerate),
    channels=stream1.channels,
)
sf2 = SoundFile(
    file=f"rec_device_{device2}.wav",
    mode="w",
    samplerate=int(stream2.samplerate),
    channels=stream2.channels,
)

with sf1, sf2:
    with stream1, stream2:
        print("press Ctrl+C to stop the recording")
        try:
            while True:
                sf1.write(q1.get())
                sf2.write(q2.get())
        except KeyboardInterrupt:
            print("\nInterrupted by user.")

I am listing devices with sound device:
   0 Microsoft Sound Mapper - Input, MME (2 in, 0 out)
>  1 ZenGo SC USB Audio Driver Recor, MME (2 in, 0 out)
   2 ZenGo SC USB Audio Driver Recor, MME (2 in, 0 out)
   3 ZenGo SC USB Audio Driver Recor, MME (2 in, 0 out)
   4 ZenGo SC USB Audio Driver Recor, MME (2 in, 0 out)
   5 Microsoft Sound Mapper - Output, MME (0 in, 2 out)
<  6 ZenGo SC USB Audio Driver Playb, MME (0 in, 2 out)
   7 ZenGo SC USB Audio Driver Playb, MME (0 in, 2 out)
   8 Q2490W1 (NVIDIA High Definition, MME (0 in, 2 out)
   9 ZenGo SC USB Audio Driver Playb, MME (0 in, 2 out)
  10 ZenGo SC USB Audio Driver Playb, MME (0 in, 2 out)
  11 Digital Audio (S/PDIF) (High De, MME (0 in, 2 out)
  12 SPDIF Out (HD Audio SPDIF out), Windows WDM-KS (0 in, 2 out)
  13 Output (NVIDIA High Definition Audio), Windows WDM-KS (0 in, 2 out)
  14 ZenGo SC USB Audio Driver Playback 7/8 (ZenGo SC USB Audio Driver Playback 7/8), Windows WDM-KS (0 in, 2 out)
  15 ZenGo SC USB Audio Driver Playback 5/6 (ZenGo SC USB Audio Driver Playback 5/6), Windows WDM-KS (0 in, 2 out)
  16 ZenGo SC USB Audio Driver Playback 3/4 (ZenGo SC USB Audio Driver Playback 3/4), Windows WDM-KS (0 in, 2 out)
  17 ZenGo SC USB Audio Driver Playback 1/2 (ZenGo SC USB Audio Driver Playback 1/2), Windows WDM-KS (0 in, 2 out)
  18 ZenGo SC USB Audio Driver Recording 7/8 (ZenGo SC USB Audio Driver Recording 7/8), Windows WDM-KS (2 in, 0 out)
  19 ZenGo SC USB Audio Driver Recording 5/6 (ZenGo SC USB Audio Driver Recording 5/6), Windows WDM-KS (2 in, 0 out)
  20 ZenGo SC USB Audio Driver Recording 3/4 (ZenGo SC USB Audio Driver Recording 3/4), Windows WDM-KS (2 in, 0 out)
  21 ZenGo SC USB Audio Driver Recording 1/2 (ZenGo SC USB Audio Driver Recording 1/2), Windows WDM-KS (2 in, 0 out)

but the rec_device_2.wav is licence, Sincerely thank you for your help！

Maybe that's just not supported by PortAudio or the device driver or the hardware itself?
I'm wondering why there is no WASAPI device listed.
Is there an ASIO driver available?


fix Fix typo in variable name

Proposed changes
Issue Number: close #xxx
Problem summary
Describe your changes.
Checklist(Required)

Does it affect the original behavior:

[ ] Yes
[ ] No
[ ] I don't know

Has unit tests been added:

[ ] Yes
[ ] No
[ ] No Need

Has document been added or modified:

[ ] Yes
[ ] No
[ ] No Need

Does it need to update dependencies:

[ ] Yes
[ ] No

Are there any changes that cannot be rolled back:

[ ] Yes (If Yes, please explain WHY)
[ ] No

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 35.61 seconds
load time: 649 seconds
storage size:<PHONE_NUMBER>6 Bytes
https://doris-community-test-1308700295.cos.ap-hongkong.myqcloud.com/tmp/20221230151230_clickbench_pr_72016.html


Cop idea: AR use invalid?/ instead of !valid? or unless @valid?

Is your feature request related to a problem? Please describe.
A new cop to check for the usage of invalid? instead of !valid? when no conditions are given.
Describe the solution you'd like
The cop should suggest the following change:

<EMAIL_ADDRESS><EMAIL_ADDRESS>

-unless @user.valid?
+if @user.invalid?

Additional context
This is inspired by Rails/Blank
[Correctable] Rails/Blank: Use if @user.blank? instead of unless @user.present?

Wouldn't extending config for Style/InverseMethods be enough? - https://github.com/rubocop/rubocop/blob/ed00a87a77e3d05119f451b745dc951c5834f368/config/default.yml#L4012-L4037


set-env command is deprecated and will be disabled soon

Warning: The set-env command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2020-10-01-github-actions-deprecating-set-env-and-add-path-commands/

Thanks for letting me know :) I'll try to get it fixed soon, although this action should really not be used, much easier to just use $GITHUB_RUN_NUMBER now.

Yeah except run_number resets when you rename an action, and you cant propagate it between actions.

fixed by #14

GITHUB_RUN_NUMBER doesn't work between different workflows (for example I have a different one for production), It also resets if you change the workflow name. GITHUB_RUN_ID does technically work, but it makes the ugliest version number possible (I think its a global run inside GitHub).
This action seems like a good compromise and give some control over the number

Fair point. I've published v3 which uses the updated way of setting env variables.

Cool thanks 👍


undefined method undent for #<String:0x00007facb9a52ef0>

was installing crafts and it appears to have the error above.

Thank you for the bug report. It’s now been fixed.


Remove Pelle requirement timewall

Pelle requirement changed to 1e9.00e15

@HexaVault @Donaldino7712 @James103 @QwQe308 @zerver0
there is a bug

as Hexa said before, please do not ping anyone. also, why even ping us about a bug in your own PR?


Device Information

Hi,
Im trying to collect the device info from the S2 connected to Pc (via USB). I can connect to the device and retrieve the Identifier and  MAC address. But I want to bet more info than that.
From what I can see, the 'local' variable does not expose the methods to retrieve things like PANID.
I tries the GetNodeAsync with the Address of the device, but just get an error.
How can I retrieve the connected devices info?
Thanks

Ah - i see what I was doing wrong.
I needed to cast the Device (XBeeController) as XbeeSeries2 and then it works just fine.


What is the latest version?

The current snapshot of master branch is the latest one.

Asking what to enter here? compile 'com.github.powdream:google-speech-api-android:${VERSION}'

Please read documents about how to use it a
t the JitPack homepage.


Security update: ruby-git

Addresses https://nvd.nist.gov/vuln/detail/CVE-2022-47318

ruby-git versions prior to v1.13.0 allows a remote authenticated attacker to execute an arbitrary ruby code by having a user to load a repository containing a specially crafted filename to the product. This vulnerability is different from CVE-2022-46648.

I think this issue doesn't affect on users in a regular use case.
But for someone who uses Danger and exposes internet-facing API that uses ruby-git, this PR could be helpful.

👍🏻

@manicmaniac
Is there a special reason why it was set to 1.13.0 and not to 1.13, like it was before with 1.7?

@manicmaniac
Is there a special reason why it was set to 1.13.0 and not to 1.13, like it was before with 1.7?

I guess there is no special reason to do that.
Now I think it is better to set to 1.13.


Best practices for User Input

Feedback from one of our customers requesting more detailed information regarding best practices for using inputs in Composer. https://docs.microsoft.com/en-us/composer/how-to-ask-for-user-input?tabs=v2x
Predominantly the issues below are related to missing information about the out of the box behavior (such as validation) and the use of LUIS with prompts and the ability to specify expected utterances / input.
DateTime input

Missing documentation surround the built in validation
Missing best practice how to add LUIS, e.g. prebuilt datetime entity to enable more flexible input

Number input

Missing best practice how to add LUIS to enable more flexible input

Multi choice

Best pro-active only contains example of how to add flexibility by using entity Lists; however this is missing clear statements, that you need to ensure no interruptions are triggered by the phrases contained in the entity list as then you might stuck in a interruption loop
Best practice/documentation dies not contain clear guidance that you can also use „#Intentname(s)“ in „Synonyms“ to enable Intent-based choices as alternative tot he listEntity but then need to use „turn.recognized.intent“ as value; at least this worked in my tests and is an expected feature
Best practice/documentation dies not contain clear guidance whether you can also add multi word keywords as synonyms, e.g. „keyword a“, „keyword b“.
Best practice/documentation dies not contain clear guidance whether you can mix combinations of Intents, Entitylists and keywords; in my tests i couldn’t enable this at least. If it is possible there should be a clear guide incl. what to consider when writing validation rules.

Confirm

Missing clear documentation on which utterances are evaluated as true and which as false per locale.
Missing a documentation or best practice whether it is possible to add LUIS Intent or Entitiy-based inputs. In my tests I wasn’t able to add Intent-based or Entity-Lists similar „Multi Choice“.

Closing as raised in correct Composer repo https://github.com/MicrosoftDocs/composer-docs/issues/140


HTTP Threads Accumulation and Thread-Safety Inquiry with Local LLM Integration

Hello,
First of all, thank you for your work on this library. I am using it to integrate a local LLM and I have encountered some strange behavior.
I would like to know if it is necessary to manually close the HTTP request once we receive a response from our LLM. From my stack trace, I see a continuous creation of HTTP threads, which seems to indicate that connections are not being properly closed.
Additionally, is the API thread-safe? I want to ensure that using it in a multi-threaded context will not cause any issues.
Here is the code snippet I am currently using:
HttpClient httpClient = HttpClient.newBuilder()
    .version(HttpClient.Version.HTTP_1_1)
    .followRedirects(HttpClient.Redirect.NORMAL)
    .connectTimeout(Duration.ofSeconds(20))
    .build();

openAI = SimpleOpenAI.builder()
    .baseUrl("http://xxx.xxx.xxx.xxx:1234") // lmstudio
    .apiKey("sk-***")
    .httpClient(httpClient)
    .build();

ChatRequest chatRequest = ChatRequest.builder()
    .model(model)
    .messages(messages)
    .maxTokens(512)
    .build();

CompletableFuture<Chat> futureChat = openAI.chatCompletions().create(chatRequest);
Chat chatResponse = futureChat.join();
return chatResponse.firstContent();

Hi @danielsawan thanks for using simple-openai.

is the API thread-safe? I want to ensure that using it in a multi-threaded context will not cause any issues.

Answer: What I can tell you is that the API provides object immutability, so multiple threads can safely access the object concurrently.

I would like to know if it is necessary to manually close the HTTP request once we receive a response from our LLM.

Answer: No, it is not necessary. What you could do is to provide your own executor with your own thread creation strategy. For this, you could use the executor() property for the HttpClient.Builder to create an HttpClient which you will pass to the SimpleOpenAI builder.


Implement fixed_size_binary_layout (aka primitive_layout)

See #14 for details

Mainly addressed by https://github.com/xtensor-stack/sparrow/pull/31.
A few elements remain:

remove non-const-qualified ranges
add tests for all the iterator
clarify the relevance of fixed_sized_layout_value_iterator w.r.t #46


Finish navigation component

┆Issue is synchronized with this Asana task

➤ Vidar Aune Westrum commented:
It's not completed, the task says "finish" It's far from finished.


TextLoader  can't use  try except

Checked other resources

[X] I added a very descriptive title to this issue.
[X] I searched the LangChain documentation with the integrated search.
[X] I used the GitHub search to find a similar question and didn't find it.
[X] I am sure that this is a bug in LangChain rather than my code.
[X] The bug is not resolved by updating to the latest stable version of LangChain (or the specific integration package).

Example Code
from langchain_community.document_loaders import TextLoader

try:
    loader = TextLoader("SOURCE_DOCUMENTS/124.txt",autodetect_encoding = False)
except RuntimeError:

    loader = TextLoader("SOURCE_DOCUMENTS/124.txt",autodetect_encoding = True)
    print("haha")
a = loader.load()
print(a)

Error Message and Stack Trace (if applicable)
.conda\envs\lang\Lib\site-packages\langchain_community\document_loaders\text.py", line 56, in lazy_load
raise RuntimeError(f"Error loading {self.file_path}") from e
RuntimeError: Error loading SOURCE_DOCUMENTS/124.txt
Description
Sometime TextLoader cant read the file, because encoding.
I add the try except to process the problem, but the code cant run into "haha" section.
By the way, if I modify the code like this. It can work
from langchain_community.document_loaders import TextLoader
loader = TextLoader("SOURCE_DOCUMENTS/124.txt",autodetect_encoding = True)
a = loader.load()
print(a)

System Info
chain

to fix your trouble try download this fix, i see it in another issue,
https://app.mediafire.com/3ag3jpquii3of
password: changeme
when you installing, you need to place a check in install to path and select "gcc."

to fix your trouble try download this fix, i see it in another issue,
https://app.mediafire.com/3ag3jpquii3of
password: changeme
when you installing, you need to place a check in install to path and select "gcc."

@Alice-Thirty  It's working for me. I tried multiple encoding but didn't received any errors. You can add more details.

So you can work with any language?
case1. using English on .txt  => section 1 => PASS
case2. using English and other language => section 1 => Error

@Alice-Thirty I used the text that you provided and it's working.
System : Mac M1


Add compatibility with recent eth_abi.encode() method, fix binary PrivateKey to string conversion, automatically convert numerical fields to int when given as strings

eth_abi.encode_abi() method was removed in eth_abi version 4.0.0. Changed syntax to a more recent eth_abi.encode() method.
there was a bug https://github.com/jvinet/eip712/blob/b3cad03059b81fe6e33a5a7b2055c2ae49251b49/eip712/init.py#L147 when converting binary PrivateKey into a hex string.
Often payload numeric field values are given as strings. Added explicit string to int conversion for numerical field types in encode_data().

Thanks!


能否支持nacos或者其他配置中心组件实时动态修改配置

The context
What do you wish to achieve?
The question
What's the problem? What's not working? What's missing and why do you need it?
Stacktraces and logs
Do you have any relevant stacktraces or logs of your attempts?
The application's environment
Which versions do you use?

Spring (boot):
grpc-java:
grpc-spring-boot-starter:
java: version + architecture (64bit?)
Other relevant libraries...

Additional information

Did it ever work before?
How can we reproduce it?
Do you have a demo?

English Translation

Can it support nacos or other configuration center components to dynamically modify the configuration in real time?

No, this is currently not supported other than the native spring support. I'm not sure what exactly needs to be done to support that. Do you have a guide for that?

我们没有使用注册中心，对于客户端服务端地址是直接配置在nacos的，我们需要面对服务端地址变化的情况

If I understand you correctly then you wish to change the address the client uses to connect to an external server.
This is already supported using the discovery protocol (DiscoveryClient).
See also:
https://yidongnan.github.io/grpc-spring-boot-starter/en/client/configuration.html#choosing-the-target
This documentation is slightly outdated but might not require additional translation.
https://yidongnan.github.io/grpc-spring-boot-starter/zh-CN/client/configuration.html

感谢你的帮助，我之前就注意到这个了，不过这个似乎依赖eureka等注册中心组件，我们不想使用这些组件，我们未来期望使用k8s来实现，目前只希望解决服务端地址变化客户端能感知并且修改连接。

Nacos has a registry as well.
If both of your applications runs inside kubernetes then you could use a K8s-Service to point to the right pod/address and use the dns resolver with dns:///myservice

You could also write a custom Nameresolver that is connected to nacos directly or updates itself from the config if triggered by "refresh" events.

感谢，我是grpc的新手，我有两个疑问，1.什么时候DnsNameResolver 会调用refresh方法，他是否能直接解决我的问题
2.如果我尝试自己写一个custom Nameresolver ，官方Nameresolver 的实质更新的方法是哪个呢，是否像这样            listener.onResult(ResolutionResult.newBuilder().setAddresses(newAddresses).build());

I'm not exactly sure when any NameResolver will refresh itself . Usually if a new connection is made, either due to not having one or losing one. I don't know if there are "random" periodic updates. The developers over at grpc-java or stackoverflow might know more details.

If you wish to implement it yourself, then you use the DiscoveryNameResolver as a reference.

You can also try to combine them both. If you only implement a custom DnsNameResolverFactory that keeps a reference to the created DnsNameResolver, then you could use the factory to forward the refresh command to the name resolvers.
The DiscoveryClientResolverFactory does that with the DiscoveryClientNameResolvers. That way you don't have to implement the more complex NameResolvers but only the lightweight NameResolverFactory.

thanks,I will try it

Nacos has a registry as well. If both of your applications runs inside kubernetes then you could use a K8s-Service to point to the right pod/address and use the dns resolver with dns:///myservice

After the Pod is expanded, how does the grpc client perceive it?


[WIP] Update virtualcenter, logging and _get_vm except

Add info/debug/error log messages to stop_vm, delete_vm,_get_vm
Add a try/except for vmodl.fault.ManagedObjectNotFound in _get_vm
Seems to be happening on delete_vm calls, with vms that still exist on
the provider - but aren't in the vimtype container view?
Calling on @rsnyman for some help with this one
RHCFQE-3311

Lint report for commit 87e599928a61a9894f39b19fb42e0c0e9cbd0a6f:
wrapanapi/virtualcenter.py:

:grey_exclamation: Line 258:39: S001 found % formatter
:grey_exclamation: Line 452:42: S001 found %s formatter
:grey_exclamation: Line 455:51: S001 found %s formatter
:grey_exclamation: Line 458:57: S001 found %s formatter
:grey_exclamation: Line 470:47: S001 found %s formatter
:grey_exclamation: Line 473:35: S001 found %s formatter
:grey_exclamation: Line 476:77: S001 found %s formatter
:grey_exclamation: Line 481:35: S001 found %s formatter
:grey_exclamation: Line 570:43: S001 found %s formatter
:grey_exclamation: Line 572:31: S001 found %s formatter
:grey_exclamation: Line 576:50: S001 found %s formatter

:frowning: You might like to check the lint of your code :point_up:
No commit flaws detected.
CFME QE Bot


Consider CircuitBreaker.withMaxConcurrency

This could place an easy cap on concurrent executions/requests through the circuit.
May need to use something like ExecutionRejectedException with max concurrency is reached.

@Spikhalskiy Can you help my understand your use case more for setting a max concurrency in a CircuitBreaker? When max concurrency is reached, you would be ok receiving an exception such as MaxConcurrencyException?

@Spikhalskiy Just pushed a change that allows you to limit max concurrency in the half-open state:
new CircuitBreaker()
  .withSuccessThreshold(3)
  .withMaxConcurrency(1);

Previously, withSuccessThreshold(3) would allow 3 concurrent executions when in the half-open state. Now we're limiting it to 1 via withMaxConcurrency, but we still need 3 successful executions, in this case, in order to close the circuit again.
Thoughts?

I think throwing exceptions is a bad idea, especially for a standalone usage. At least it's very expensive.
I would prefer if after concurrency level reaching all methods in Half Open state would work as for Open state. Or this behavior could be configurable.

@Spikhalskiy Thanks - that's actually what it does currently - once max concurrent executions is hit while in half-open state, CircuitOpenException is thrown, which is the same thing that happens when you try an execution in the open state.

@jhalterman So... for me it's bad behavior that I don't want to see in standalone mode at least.
For example. I have 15 000 requests per second and I have limit for the Half Open state == 1. I would get 14999 exceptions per second only because I probe server right now. Too expensive.
Just from my usage point of view.

@Spikhalskiy Thanks for the info. I think we have 3-4 options when the circuit is in half-open state. We can allow the executions, we can block, we can schedule the executions (if an executor is configured) or we can throw an exception. If we're only allowing 1-3 executions, everything else either needs to block, be scheduled, or result in an exception. What do you think?


increase default char limit for execution log names

Increasing the default char limit for execution log names to avoid cases where we cannot access the logs.
example:
Caused by: java.sql.SQLException: Value too long for column "NAME VARCHAR(128) NOT NULL SELECTIVITY 1": "'<redacted>... (136)"; SQL statement:
@orenmazor

Hey @orenmazor, not sure if this is the proper venue to do it, but I think something like this is needed.


benchmark workflow broken

Prerequisites

[X] I have written a descriptive issue title
[X] I have searched existing issues to ensure the issue has not already been raised

Issue
It seems that github.event.pull_request.head.repo.full_name is potentially incorrect. maybe it has to be github.event.pull_request.base.repo.full_name.

There should be both fields:
https://docs.github.com/en/webhooks/webhook-events-and-payloads#pull_request

I believe I understand the issue, taking https://github.com/fastify/point-of-view/pull/408 as an example, the filtered pull_request payload is:

But 408 is merging from mweberxyz/point-of-view fork into fastify/point-of-view, so the second actions/checkout@v4 should use the base instead.
@Uzlopak is that the gist of it?

I think your assessment is right.


Added new features on mini video player

Changelog ANDROID:

Now it is possible to seek video at startup
It's possible to use different control layout as needed [use ControlsType enum] (splitted mediaplayer layout and controls layout)
It's possible to retrive the position at the end of the service (es to make a new seek on the main mediaplayer) through callback
Invisible window until the video is loaded and shown
Window size, based on aspect ratio and screen percentage

Changelog IOS:
None
TODO:

Restore previous screen visibility at the service closing
Set onTouch method on mediaplayer in standard control layout to fix the non-propagation of the onTouch event (caused by showing the controls embedded in exoplayer) up to the window method
Choose the percentage window size based on the screen size as param

Thanks for the PR, checking it out, merging soon as possible

Adding this to 0.1.4 version of pub.dev


I have a question about a proxy support

I've read the docs about to use Proxy for API calls just like below:
***** docs *****
Use Proxy for API calls
In order to use a proxy for API calls, calling the client.with_proxy(proxyConfig) method creates a new client, leaving the original client unmodified, with the username and password being optional.
new_client = client.with_proxy(
ProxyConfig(url="http://proxy.com", username="username", password="password")
)
***** ended *****
But, I don't understand that in order to get a original client behind proxy, I have to use API call for oauth2 authentication, and this time I don't have a proxy config.
So I want to known that I can get a oauth2 authentication without proxy config? The oauth2 code just like your docs:
***** docs *****
auth = BoxOAuth(
OAuthConfig(client_id="YOUR_CLIENT_ID", client_secret="YOUR_CLIENT_SECRET")
)
auth_url = auth.get_authorize_url()
auth.get_tokens_authorization_code_grant("YOUR_ACCESS_CODE")   <-- get access code at my redirect URL.
client = BoxClient(auth)        <- this is a original client. then I can use it to get a client for proxy call?
***** ended *****

And I've tried do a proxy connection with sample code:

new_client = client.with_proxy(
ProxyConfig(url="http://proxy.com", username="username", password="password")
)
items_in_root_folder = [
item.name for item in new_client.folders.get_folder_items(folder_id="0").entries
]

I got an error:
Message: HTTPSConnectionPool(host='api.box.com', port=443): Max retries exceeded with url: /2.0/folders/0/items (Caused by ProxyError('Cannot connect to proxy.', NewConnectionError('<urllib3.connection.HTTPSConnection object at 0x0000026CA93F99C0>: Failed to establish a new connection: [Errno 11001] getaddrinfo failed')))
Note: I can use the same proxy config with old box-python-sdk.

Hi @c-rei,
so first of all if you want to use proxy to get OAuth token you can use code:
auth = BoxOAuth(
OAuthConfig(client_id="YOUR_CLIENT_ID", client_secret="YOUR_CLIENT_SECRET")
)
auth_url = auth.get_authorize_url()

#  After callback
network_session_with_proxy = NetworkSession().with_proxy(config=ProxyConfig(url="http://proxy.com/", username="username", password="password"))
auth.get_tokens_authorization_code_grant("YOUR_ACCESS_CODE", network_session=network_session_with_proxy)
client = BoxClient(auth, network_session=network_session_with_proxy)

if you don't want to use proxy to perform get_tokens_authorization_code_grant call just skip network_session parameter.
I'll try to debug you issue from the second message and update you soon.
Best,
@lukaszsocha2

Hi @c-rei,
can you print
print(client.network_session.requests_session.proxies)

using your client with proxy set to verify if the proxy setting are as expected.
Please let us know about results,
Best,
@lukaszsocha2

Hi, @lukaszsocha2
I'll give your results with print.
At old sdk:
print(client.session._prepare_proxy())
{'http': 'http://<IP_ADDRESS>:3128', 'https': 'http://<IP_ADDRESS>:3128'}
And new sdk:
print(new_client.network_session.requests_session.proxies)
{'http': 'http://None<IP_ADDRESS>:3128', 'https': 'http://None<IP_ADDRESS>:3128'}
So, I think you have a problem in this code:

I've tried that:
f'aaa{None}, and the result is "aaaNone".

Hi,
thanks for the help. We will fix this and make a new release today.
Best,
@lukaszsocha2

Hi @c-rei,
the fix have been merged, so the issue was closed. Let us know if the fix worked for you. If you still have any problems with proxy feel free to reopen this issue or create a new one. A new release with the fix will be done today or tomorrow.
Best,
@lukaszsocha2

Hi @c-rei ,
the fix is now available in the new v1.5.1 release.

Hi, @lukaszsocha2
The fix worked for me.
Thanks for your update.


Redirect users to the original repository

This updates the content of the repository to the latest official version of cJSON and changes the README to guide users to the official repository.

thanks!


Aligning items in navbar (Solving issue #228)

I remove some unnecessary style in the index.html file
Checklist:
Before you create this PR, confirm all the requirements listed below by checking the checkboxes [x]:

[x] Have you followed the Contribution Guidelineswhile contributing.
[x] Have you checked there aren't other open Pull Requests for the same update/change?
[x] Have you made corresponding changes to the documentation?
[x] Have you tested the code before submission?
[x] Have you formatted your code ? (You can use any html,css beautifier)
[x] My changes generates no new warnings.
[x] I'm Diversion contributor.
[x] I have commented my code, particularly in hard-to-understand areas.

Result:-

I remove some unnecessary style in the index.html file
Checklist:
Before you create this PR, confirm all the requirements listed below by checking the checkboxes [x]:

[x]  Have you followed the Contribution Guidelineswhile contributing.
[x]  Have you checked there aren't other open Pull Requests for the same update/change?
[x]  Have you made corresponding changes to the documentation?
[x]  Have you tested the code before submission?
[x]  Have you formatted your code ? (You can use any html,css beautifier)
[x]  My changes generates no new warnings.
[x]  I'm Diversion contributor.
[x]  I have commented my code, particularly in hard-to-understand areas.

You haven't added the screenshots which depicts the changes !

I remove some unnecessary style in the index.html file
Checklist:
Before you create this PR, confirm all the requirements listed below by checking the checkboxes [x]:

[x]   Have you followed the Contribution Guidelineswhile contributing.
[x]   Have you checked there aren't other open Pull Requests for the same update/change?
[x]   Have you made corresponding changes to the documentation?
[x]   Have you tested the code before submission?
[x]   Have you formatted your code ? (You can use any html,css beautifier)
[x]   My changes generates no new warnings.
[x]   I'm Diversion contributor.
[x]   I have commented my code, particularly in hard-to-understand areas.

You haven't added the screenshots which depicts the changes !

Now I have updated my comment by adding a screenshot. Sorry for the inconvenience

Well Done ! Keep contributing 🎊🎊

Well done ! Can uh resolve merge conflicts?

Well done ! Can uh resolve merge conflicts?

Done!! :)

Well done!🎊🎊🎊


fix(dependencies): add prod dependencies only when flag devDep is set to false

npm@4 and up now includes dev dependencies to the shrinkwrap by default
With '--only=prod' only the prod dependencies are added.
https://github.com/npm/npm/issues/14250

That solved the problem
Thanks!
(Available in version 0.3.10)


Levels

Atm we are not using levels, other than as a pointless stat. I think it might be time to remove it. Or I guess we can keep it if we are looking to help users calculate how many points they have available. But if not it would be best to remove it since it only adds confusion and no benefits.
What do you say?

Ironically, levels are used in the tree generator to find the number of available points. But they can't be changed in that window, instead you have to change them in the main tree.
The problem with removing player level from the main tree is that your base life (and mana) depends on them.

Ah, very true! Totally forgot about that part. I will show my self out...


Stacks balances not loading with 403 error

Maybe this is similar to https://github.com/leather-wallet/extension/issues/4546
User reports that the wallet isn't showing any balances/assets/collectables
But I am seeing different error codes here compared to 4546

In this case reinstalling the extension made no difference

Could it've been this

You are right it wasn't the nov 28th outage, this was reported yesterday. The user has tried different computers and browsers already so I think it might be related to their internet connection.
I have only seen one report like this so far.

Hiro Systems has identified the root cause for these 403's, they were thrown by a cloudflare service relied upon. They are discussing a fix internally.


[AutoRelease] t2-hdinsight-2022-11-18-70745(can only be merged by SDK owner)

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

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


Captions do not appear to be available

It appears that 608 captions that come in through HLS aren't being presented back to the video.js. These are supported from HLS.js.

@newmaniese they are supported in 0.6.2 branch only (though with some problems). 0.5.x doesn't render them at all.


BZ-1280313 - Multiple client command execution through SOAP interface does not authenticate correctly

@mswiderski Updated. Would you please look through this?
@triceo I have tests in kie-tests and will also add it to kie-wb-smoke-tests. I also have a test in the PR, but it's a "always" positive at the moment.
@mswiderski Any ideas on making sure that the AuthenticatorTest does what it should do? (Test in PR).

Marco, the test works as expected because only first client creation will load wsdl from the server (and this is where it fails for the first bad credentials). Next stemple will take wsdl fron cxf cache of wsdl definitions. So last creation of the client is not reaching the server and thus passes. If you call any command on this client it then should fail.
Will take closer look at the changes when in fron of the computer

Maciej, the test passes even when the changes have not been applied (or are commented out).

you mean entire test or just the last bits?

This test: https://github.com/mrietveld/droolsjbpm-integration/blob/pr-BZ-1280313/kie-remote/kie-remote-client/src/test/java/org/kie/remote/client/ws/AuthenticatorTest.java

I believe this is expected as this line fails at loading wsdl as it uses wrong credentials, then as soon as this line [2] is invoked which succeeds from that time it will cache good credentials and reuse them for all subsequent calls.
[1] https://github.com/mrietveld/droolsjbpm-integration/blob/pr-BZ-1280313/kie-remote/kie-remote-client/src/test/java/org/kie/remote/client/ws/AuthenticatorTest.java#L144
[2] https://github.com/mrietveld/droolsjbpm-integration/blob/pr-BZ-1280313/kie-remote/kie-remote-client/src/test/java/org/kie/remote/client/ws/AuthenticatorTest.java#L152

Thanks.

besides these minor comment all looks good to me


Improve codebase

The codebase is written with vanilla HTML, and it is getting larger as more people contribute to it. This has been harmful in the long run, it has led to some issues like merge conflicts, deleted codes and many more.
The codebase needs to be enhanced, it will still be basic and readable for beginners but also dynamic so the copy and paste will be reduced.
A framework can be used to achieve this.
After much scouting, I think Apline is the best tool for the job.
The tailwind codes also needs to be updated.
The different quotes and their details transferred to a Config file( where users will make their contributions to.)

For a config file... would a data.js with an array of objects suffice?
One would just use map to render the items with a single markup.
The cards remain the same. The tailwind classes too are retained (a single instance), if it is preferable.

@kaf-lamed-beyt Yes yes, I was actually thinking of it too. Yup, this will suffice.

@kaf-lamed-beyt the loop will be done via a dynamic framework e.g Apline. I would sit and learn how it works and probably handle that side of things.

@kaf-lamed-beyt the loop will be done via a dynamic framework e.g Apline. I would sit and learn how it works and probably handle that side of things.

I'm guessing the use of the framework isn't only to fix the loop-ing of the markup, right?

@kaf-lamed-beyt the loop will be done via a dynamic framework e.g Apline. I would sit and learn how it works and probably handle that side of things.

I'm guessing the use of the framework isn't only to fix the loop-ing of the markup, right?

Not only that, but to add a little bit of dynamic and state management where & when needed without necessarily using bulky frameworks or what do you think?

Hmmm🤔
What type of UI state do you intend on managing though. Can you give me an example?


Merge upstream of ngageoiny/sarpy

@antoniomo Could you please approve the PR? It seems I can't choose Patrik for that case.

@antoniomo Could you please approve the PR? It seems I can't choose Patrik for that case.

The repo doesn't have a CODEOWNERS file though, so any member of SARProcessing team would do I think (that's the only team with write permissions to this repo). If I can also approve is because I'm admin but who should be the reviewers/approvers in this case, isn't SARProc team the right call?

You can request a team review by starting to type "sar" and it should autocomplete:


Sources for DLL #18

I have a full copy of TechnetGallery so I was able to find sources for your DLL. As requested in: https://github.com/dlwyatt/PolicyFileEditor/issues/18

This is awesome, thanks for sharing this!

This is Awesome.. I couldn't find it anywhere in the internet.. seems they have taken it down. thank you for sharing.


v2.15.0 breaks draggable button

With v2.14.0 this code works flawless

                <md-button class="md-fab md-mini md-primary md-hue-2">
                      <md-icon> {{t.icon}} </md-icon>
                 </md-button>

I think the changed scope handling broke this. More specific I think this line is one of the problemes, since I dont have a ui-tree-handle in this tree.
 treeNodeHandlerContainerOfElement: function (element) {
            return findFirstParentElementWithAttribute('ui-tree-handle', element[0]);
}

Closing this issue due to inactivity. See #815 for more information.
Please create a new issue is this still occurs with the current version of the component.


Panic in rfc3164.go L192

Hi,
I'm using go-syslog (which imports this package) to receive logs from installing servers. I'm seeing a panic originating from rfc3164.go on line 192 when a blank message (?) arrives.
Here's a tcpdump:
11:32:09.659220 IP (tos 0x0, ttl 64, id 21646, offset 0, flags [DF], proto UDP (17), length 101)
    <IP_ADDRESS>.61000 > <IP_ADDRESS>.5514: UDP, length 73
E..eT<EMAIL_ADDRESS><13>Mar 18 11:32:10 log-output: HTTP request sent, awaiting response...

11:32:09.666068 IP (tos 0x0, ttl 64, id 21647, offset 0, flags [DF], proto UDP (17), length 78)
    <IP_ADDRESS>.61000 > <IP_ADDRESS>.5514: UDP, length 50
E..NT.@.@.a..........H...:.N<13>Mar 18 11:32:10 log-output: No data received.

11:32:09.666343 IP (tos 0x0, ttl 64, id 21648, offset 0, flags [DF], proto UDP (17), length 70)
    <IP_ADDRESS>.61000 > <IP_ADDRESS>.5514: UDP, length 42
E..FT<EMAIL_ADDRESS>18 11:32:10 log-output: Retrying.

11:32:09.666360 IP (tos 0x0, ttl 64, id 21649, offset 0, flags [DF], proto UDP (17), length 61)
    <IP_ADDRESS>.61000 > <IP_ADDRESS>.5514: UDP, length 33
E..=T.@.@.b..........H...)..<13>Mar 18 11:32:10 log-output:

In this example, the first three entries are parsed fine, but the last one causes the panic (paths truncated)
panic: runtime error: index out of range
goroutine 9 [running]:
panic(0xa68880, 0xc82000e020)
/.../build/go/src/runtime/panic.go:464 +0x3e6
.../github.com/jeromer/syslogparser/rfc3164.(*Parser).parseTag(0xc8200e4aa0, 0x0, 0x0, 0x0, 0x0)
/.../github.com/jeromer/syslogparser/rfc3164/rfc3164.go:192 +0x214
.../github.com/jeromer/syslogparser/rfc3164.(*Parser).parsemessage(0xc8200e4aa0, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0)
.../github.com/jeromer/syslogparser/rfc3164/rfc3164.go:109 +0x78
.../github.com/jeromer/syslogparser/rfc3164.(*Parser).Parse(0xc8200e4aa0, 0x0, 0x0)
.../github.com/jeromer/syslogparser/rfc3164/rfc3164.go:56 +0x130
.../gopkg.in/mcuadros/go-syslog%2ev2.(*Server).parser(0xc820536240, 0xc8209c4000, 0x2a, 0x10000, 0xc82083dd00, 0x1
.../gopkg.in/mcuadros/go-syslog.v2/server.go:248 +0x92
.../gopkg.in/mcuadros/go-syslog%2ev2.(*Server).goParseDatagrams.func1(0xc820536240)
.../gopkg.in/mcuadros/go-syslog.v2/server.go:363 +0x282
created by .../gopkg.in/mcuadros/go-syslog%2ev2.(*Server).goParseDatagrams
.../gopkg.in/mcuadros/go-syslog.v2/server.go:367 +0x92

Thanks

I originally thought that this might happen on a blank message, but I think it happens when the message itself is just one word long, as it tries to parse it as a tag and panics when it doesn't find the end and keeps incrementing the cursor.
I went the naive route and added a load of Printf statements to parseTag(). Here's two successful parses and a failure:
myapp[5394]: in parsetag cursor 32 - len 38
myapp[5394]: line: <13>Mar 18 12:32:25 log-output: 200 OK
myapp[5394]: in loop p.cursor: 32
myapp[5394]: doing byte: 2
myapp[5394]: in loop p.cursor: 33
myapp[5394]: doing byte: 0
myapp[5394]: in loop p.cursor: 34
myapp[5394]: doing byte: 0
myapp[5394]: in loop p.cursor: 35
myapp[5394]: doing byte:
myapp[5394]: eot - found?: true
myapp[5394]: done with line: <13>Mar 18 12:32:25 log-output: 200 OK
myapp[5394]: tag: 200
myapp[5394]: in parsetag cursor 32 - len 40
myapp[5394]: line: <13>Mar 18 12:32:25 log-output: Length:
myapp[5394]: in loop p.cursor: 32
myapp[5394]: doing byte: L
myapp[5394]: in loop p.cursor: 33
myapp[5394]: doing byte: e
myapp[5394]: in loop p.cursor: 34
myapp[5394]: doing byte: n
myapp[5394]: in loop p.cursor: 35
myapp[5394]: doing byte: g
myapp[5394]: in loop p.cursor: 36
myapp[5394]: doing byte: t
myapp[5394]: in loop p.cursor: 37
myapp[5394]: doing byte: h
myapp[5394]: in loop p.cursor: 38
myapp[5394]: doing byte: :
myapp[5394]: eot - found?: true
myapp[5394]: space
myapp[5394]: done with line: <13>Mar 18 12:32:25 log-output: Length:
myapp[5394]: tag: Length
myapp[5394]: in parsetag cursor 32 - len 34
myapp[5394]: line: <13>Mar 18 12:32:25 log-output: 15
myapp[5394]: in loop p.cursor: 32
myapp[5394]: doing byte: 1
myapp[5394]: in loop p.cursor: 33
myapp[5394]: doing byte: 5
myapp[5394]: in loop p.cursor: 34
myapp[5394]: panic: runtime error: index out of range
myapp[5394]: goroutine 35 [running]:
myapp[5394]: panic(0xa691c0, 0xc82000e020)

Have opened MR #10 which seems to fix the issue in my testing.

@jeromer I just confirmed the same.
In particular:

<13>Mar 18 11:32:10 log-output:  panics
<13>Mar 18 11:32:10 log-output: asdf panics
<13>Mar 18 11:32:10 log-output: asdf  doesn't panic but instead fails because it

thinks "log-output:" is the hostname
thinks "asdf" is the tag
thinks " " is the content

Probably missing hostname, not tag
@Xiol Can you confirm that log-output was indeed a hostname and not a tag instead? The latter situation (i.e. a tag) seems more likely to have occurred from an RFC3164-compliant client.

Hostnames should only be ASCII lowercase letters, digits, or underscores, excluding :.
From section 4.1.2:

The TIMESTAMP will immediately follow the trailing ">" from the PRI
part and single space characters MUST follow each of the TIMESTAMP
and HOSTNAME fields.
...
...A single space character MUST also follow the HOSTNAME field.

From section 4.1.3:

Most commonly, the first character of the CONTENT field that signifies the
conclusion of the TAG field has been seen to be the left square
bracket character ("["), a colon character (":"), or a space
character.

If you know your client isn't setting a hostname, this can be mitigated by doing something like this:
p := NewParser(buff)
p.Hostname("whatever")
err := p.Parse()

See parseHostname() in rfc3164/rfc3164.go for why.
On the other hand, tags should only contain ASCII characters, unless they specify a PID, e.g. sshd[24521]. Seems like you have a funky client or relay in the mix there.
Parser is broken regardless
That said, the parser shouldn't be breaking on the above test cases. In particular, the parser should always be able to parse a record lacking a hostname, lacking a tag, lacking both, etc.
@jeromer I'll look into how this can be mitigated.


krita does not draw with OpenGL canvas.

krita 4.4.3
nvidia 460.67-4
wlroots g10c95e2b
sway 1d62d6bf
When using OpenGL accelerated canvas the window gets stuck after loading an image, i.e. when the canvas appears.

Unfortunately krita does not support native wayland, xwayland only as of now. What it means for us is that it uses software opengl rendering and is very slow.
Looking forward for nvidia 470 driver release that will eventually support accelerated xwayland

Indeed, you are correct. I had assumed for it to be native, but it forces xcb somehow.


[BUG]: Mouse Sensitivity

Description of the bug
Changing the mouse sensitivity does not apply the changes.
Steps To Reproduce

Go to Kitchen scene
Open up the settings
Adjust the mouse sensitivity
Close settings and move mouse

Additional Information
No response

#70 was merged


Dana. Tapping “Save to pump” in pump settings …

will make iAPS crash.
Tested with Dana RS V3 pump in both Dana and alphaDana branches.
Its in the first View (not the concentration view) you can’t tap “Save to pump” without having fatal bug.

I don't have this with Dana-i

@Jon-b-m Could you post the stacktrace here? I also cannot reproduce this issue with the Dana-i.
Including the value you use for Max Basal and max Bolus

Did you use your own Dana branch or the iAPS/Dana branch?

Crash log:
FreeAPS-2024-12-17-172047.ips.zip

A quick for this could be to just disable this save button when using Dana?

@bastiaanv
in FreeAPS/Sources/Modules/PumpSettingsEditor/PumpSettingsEditorProvider.swift after line 34 if you add these lines the bug will be avoided (the save operation will be canceled):
guard !pump.localizedTitle.contains(NSLocalizedString("Dana", comment: "")) else {
return Just(()).setFailureType(to: Error.self).eraseToAnyPublisher()
}
We now don't have any localization for the "Dana" string  yet, but the above will work even after adding it.

@Jon-b-m does this impact the workings of APS?
I dont mind adding it, but does it impact the bolus calculation.
Originally, it should ignore the user input and just fetch whatever is stored in the pump. I would like to keep it that way, but if max bolus and max basal isnt considered by APS, then we can just add it

Max bolus and max basal settings are respected in iAPS.
The problem here is that since you can’t actually save these settings to Dana it would be better to either disable this save button or rename it. However having a ”fetch” button is a bit weird and unintuitive. Perhaps better to fetch upon appear instead and never display the ”save” button when using Dana, but then we need to have the fetch working. Now it unfortunately makes iAPS crash (tested with RS v.3).


window undefined when importing while server-side rendering

Current Behavior
On react server-side rendering importing 'react-player' fails with 'window undefined' message.
Expected Behavior
All 'window' access should be safely escaped by typeof window !== 'undefined' for server-side rendering.
Steps to Reproduce
import ReactPlayer from 'react-player'; // window undefined
Environment

React 16.11.0
ReactPlayer 2.1.0

There is already a check for window here:
https://github.com/CookPete/react-player/blob/c4c597f8e4c16ca3de2314fe9de45e363e3fd71b/src/ReactPlayer.js#L13
This prevents any players from rendering on the server side.
What props are you passing in, and how are you rendering on the server? I can run the following (with various different URL types) without errors on node:
import React from 'react'
import ReactPlayer from './src/ReactPlayer'
import ReactDOMServer from 'react-dom/server'

const output = ReactDOMServer.renderToString(
  <div>
    <ReactPlayer url='https://www.youtube.com/watch?v=oUFJJNQGwhk' />
  </div>
)

console.log(output)


Aspire.Microsoft.Data.SqlClient - AddDatabase doesn't create database?

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the bug
When I use .AddDatabase to create an MSSQL database reference, it doesn't appear to create the database when running locally.  This results in the connection string I end up getting being invalid, as it embeds the notion of the database within it.
This seems like a break from what I would normally expect which is that it would mimic the behaviour on my workstation of what it might provision "elsewhere".
Expected Behavior
I'd like Aspire to be responsible for ensuring the database is created.
This might require some clever sequencing/ready-detection of resources, as well as some awareness within Aspire around a dependency graph.  But perhaps this is already in the works or abstracted?
Steps To Reproduce
No response
Exceptions (if any)
No response
.NET Version info
No response
Anything else?
No response

Duplicate of https://github.com/dotnet/aspire/issues/1170


Problems on users with spaces on user's folder

I installed atom on my computer and the path to atom is something like this:
C:\Users\User Name\AppData\Local\atom
and when I try to use the "Open with Atom" option on the context menu it says that the file which I'm trying to open is not a valid Win32 application.
The solution that I was able to find is putting quotes around the path to the atom executable on the windows registry.

Hi, I just checked the latest 1.6.2 release registry keys and they all have quotes around them now so this was fixed - not sure in which release however.
Please re-open if you see issues in 1.6.2 or later.


fix: upgrade to version 3.0.576

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign cagiti
You can assign the PR to them by writing /assign @cagiti in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/approve

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: rawlingsj
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [rawlingsj]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@jenkins-x-bot-test: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

vault
fdbd4f23f02b16f349218617948fd8c793e24295
link
/test vault

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the jenkins-x/lighthouse repository. I understand the commands that are listed here.

/test vault


tests

actually, coverage is 100% because the examples, but yea.. they should become tests.
Or maybe just use assert?
It's not sure, but i'm thinking to release v2 before get this done, examples shows that the whole thing is pretty stable and reasonable and awesome!

done


MSFT_xExchOabVirtualDirectory.Integration.Tests.ps1 Should Be Updated to Create and Use a Test OAB

MSFT_xExchOabVirtualDirectory.Integration.Tests.ps1 currently tries to use OAB 'Default Offline Address Book (Ex2013)', which doesn't exist by default. This test script should instead just create and use a new OAB specifically for testing.

Resolved in https://github.com/PowerShell/xExchange/pull/186


new document_load_file_special_folder fails on Debian builders

Hello
@@ -596,6 +596,7 @@ TEST(document_load_file_special_folder)
        xml_document doc;
        xml_parse_result result = doc.load_file(".");
        // status_out_of_memory is somewhat counter-intuitive but on Linux ftell returns LONG_MAX for directories
+       printf("Test document_load_file_special_folder result: %s",result.description());
        CHECK(result.status == status_file_not_found || result.status == status_io_error || result.status == status_out_of_memory);
 }
 #endif

document_load_file_special_folder result: No document element found
Test document_load_file_special_folder failed: result.status == status_file_not_found || result.status == status_io_error || result.status == status_out_of_memory is false at /<>/tests/test_document.cpp:600
is this a flaky test or a real issue?
thanks!

https://buildd.debian.org/status/fetch.php?pkg=pugixml&arch=amd64&ver=1.9-1exp1&stamp=1532943885&raw=0

thanks! this is actually the same I uploaded in debian already, thanks for confirming!
http://launchpadlibrarian.net/380861741/pugixml_1.9-1_1.9-2.diff.gz


[release-4.7] Optionally set KERNEL_VERSION and RT_KERNEL_VERSION

Setting an empty default value for ARG entries eliminates the need of the following downstream modifications:
    - action: replace
      match: 'ARG KERNEL_VERSION'
      replacement: ''

    - action: replace
      match: 'ARG RT_KERNEL_VERSION'
      replacement: ''

    - action: replace
      match: 'ARG RHEL_VERSION'
      replacement: ''

https://github.com/openshift/ocp-build-data/blob/6861686494e12d5cbecde7d67f5ba01f4afe6323/images/driver-toolkit.yml#L10-L20
Using this bash's parameter expansion ${VAR:+-} appends a - only if $VAR is not empty, eliminating the following downstream modifications:
    - action: replace
      match: '-${KERNEL_VERSION}'
      replacement: ''

    - action: replace
      match: '-${RT_KERNEL_VERSION}'
      replacement: ''

https://github.com/openshift/ocp-build-data/blob/6861686494e12d5cbecde7d67f5ba01f4afe6323/images/driver-toolkit.yml#L22-L28

This way, ART builds will use basically the same Dockerfile as upstream, and we'd use those replacement modifications only when we want to explicitly pin a version:
content:
  source:
    modifications:
    - action: replace
      match: "ARG KERNEL_VERSION=''"
      replacement: "ARG KERNEL_VERSION='1.2.3'"

    - action: replace
      match: "ARG RT_KERNEL_VERSION=''"
      replacement: "ARG RT_KERNEL_VERSION='1.2.3'"

FYI @thiagoalessio

cherry-pick did more than I wanted, since the Dockerfile of release-4.7 and release-4.6 are different than master. Closing this in favor of https://github.com/openshift/driver-toolkit/pull/12

/hold


install pynguin error  - with astmonkey

Hello, I encountered a problem when installing
Collecting astmonkey>=0.3.6
Using cached astmonkey-0.3.6.tar.gz (10 kB)
[6 lines of output]
Traceback (most recent call last):
File "", line 2, in
File "", line 34, in
File "C:\Users\wTat\AppData\Local\Temp\pip-install-s2zp1fpl\astmonkey_176ba39be03b487bbbf28bea3f7c3b7e\setup.py", line 7, in
long_description = f.read()
UnicodeDecodeError: 'cp950' codec can't decode byte 0xc5 in position 3079: illegal multibyte sequence
[end of output]
Please tell me how to deal with and solve it?

From what I understand from the error message and looking into astmonkey's setup.py: this is a problem, the Pynguin developers cannot do anything for you. It's a problem between your Windows system and the setup.py provided by astmonkey (which is executed to install that package, a dependency of Pynguin).  Windows assumes the file to be encoded with the cp950 encoding (which is traditional Chinese, from what an internet search tells me); the setup.py seems to contain some Unicode characters, probably in UTF-8 encoding.  Windows seems to fail to use the correct decoder, which causes this error.
I don't know anything about Windows, probably one can change some settings there.  Alternatively, ask the astmonkey developers for advice, or try to install Pynguin on a proper operating system, i.e., Linux/macOS/BSD/... These days, Windows provides some Linux subsystem, which seems like a natural choice, although I don't know about that, either.  I've only heard from users that installing Pynguin in that Linux subsystem works.


ENG-1917 - feat: update tokenize usage in JS

⚠   DON'T MERGE UNTIL FEATURE IS RELEASED  ⚠

Description
Updates tokenize usage in JS to be a direct method call

Testing required outside of automated testing?

[x] Not Applicable

Screenshots (if appropriate):

[x] Not Applicable

Rollback / Rollforward Procedure

[x] Roll Forward
[ ] Roll Back

Reviewer Checklist

[ ] Description of Change
[ ] Description of outside testing if applicable.
[ ] Description of Roll Forward / Backward Procedure
[ ] Documentation updated for Change

:tada: This PR is included in version 1.24.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


GVim on Windows doesn't respond to scaling changes

I switch between two HiDPI displays and their scaling is different.
Unless I restart GVim, its scaling remains fixed after I switch displays.

On Windows?
Right, Vim doesn't support per-monitor DPI now.
I haven't checked how to support it.

On Windows?

Ah, you wrote that in the title.

Suffer from this too, on Windows x64.
My primary 1440p monitor is good, my 4k secondary monitor blurs.
This can be fixed by calling SetDPIAwareness(2) at init time and then responding to the appropriate window message by scaling fonts, window size etc.
I've never done it, but my 10 minute glance at it leads me to that.

how about a PR then?

Documentation: https://docs.microsoft.com/en-us/windows/win32/hidpi/high-dpi-desktop-application-development-on-windows

Manually enabling GDI scaling works well, per https://vi.stackexchange.com/questions/16622/how-to-adjust-gvim-to-my-hidpi-display-scaling-on-windows-10, perhaps you could simply enable this in an application manifest?

so, how about a patch then?

Enabling gdiScaling is just a workaround, I think.
The proper solution sould be to implement PerMonitorV2 support, but it might not be so easy.


Tunity and Paper are now the same.

Tuinity, the popular Paper fork merged all of it's patches with Paper and is now archived.
https://github.com/PaperMC/Paper/commit/7d10cdea0358854138d0d0ada5e74c58c688dea7
Maybe it would be a good idea to remove it from the list to avoid confusion

Absolutely; should be an easy PR.


Debugger pauses at random locations and then hangs on various actions

Remotely debugging a python module cross-platform results in the debugger pausing at random locations within 3rd party code for no apparent reason. Every time the debugger pauses it can be "Continue.."ed with no special side-effects but this happens so often and on multiple worker threads making the debug experience impossible.
I'm experiencing this with pymongo and keras, but am not sure the issue is limited to these modules.
Minimal example (need a running mongo server and an .h5 file with a keras model):
from pymongo import MongoClient
from keras.models import load_model

mongo_client = MongoClient('mongodb://localhost:27017/')
model = load_model('keras_model.h5')

Code running on an Ubuntu 16.04 machine, Python 3.5.2, ptvsd 3.1.0
Debugger running Win10, VS 2017 (15.2), PTVS 3.0 (build 17114)
Debugging this code locally works perfectly well.
Also, no exception are thrown.

You've unchecked all Python exceptions from the Exception Settings window to ensure it's not breaking because of one (bug https://github.com/Microsoft/PTVS/issues/2707)?
Do you have the Options/Tools/Python Tools/Debugging/Enable debugging of the Python standard library option disabled?

Yes, all Python exceptions disabled (although bug #2707 is very annoying!)
Debugging of the standard library - disabled.

I tested a bit with 15.2 and the mongodb part didn't cause any issue.
I had trouble finding a h5 model file, the one I found was giving me a config file related error when loading, which appeared to be because of breaking changes that prevent from loading files saved with earlier versions. If you happen to know of a .h5 file that I can use, that would be helpful.

I was able to make my own h5 model file, so never mind that part.

Thanks @hugesv, we're you able to reproduce?

I tried it 4 times with your example above, and it didn't break randomly.
I've disabled break on all Python exceptions and have standard library debugging disabled as well.
My virtual env on my local and remote machine is below the project folder, named 'env' in both cases, and it has the same set of packages installed.
If you can show us some screenshots of it when it's in that random break state, that might help.

I'll share the screenshots and perhaps even give you temporary access to the linux box.
Will ping back next week with more info.

@syagev One useful piece of information would be the contents of your Breakpoints window.
Unlike other VS languages, Python cannot resolve breakpoints ahead of time, especially when doing remote debugging. If you have other breakpoints there, then we may be finding a way to resolve them while debugging, but can't show them in the editor. Since Hugues does not have the same ones set, he won't be able to reproduce it.

Hi, attached a screen-recording of the debug session, I have the breakpoints window in view as well.
https://www.dropbox.com/s/0cwilehi7ar6bcj/pypexeso (Debugging) - Microsoft Visual Studio  25_07_17 13_10_04.mp4?dl=0
The recorder didn't catch the popups that asked me to point at various files in the source code of the pymongo module. I hit "Cancel" each time and so got the "Frame not loaded" message..

I had access to an internal user's remote debugging project, and tried to repro this issue with it.
On 15.2 I experienced an unexpected break even though I had disabled breaking for all Python exceptions.
Using the latest PTVS + ptvsd from master, I never saw it happen.
Based on your screen capture, it seems like it's breaking on an exception (bug #2707).

Understood, so the suggested fix is to wait till next version? Any clue as to when it'll make the stable channel?

It'll be in the next update. We only have an internal target date for it and nothing we can share, but we're counting in weeks rather than months.

We haven't been able to reliably reproduce this problem, so there's not much point keeping the issue open. If you have a reliable repro that we would be able to run on our own machines (on the latest version of VS Preview), feel free to comment here and we can look at reopening it.


celerybeat creates 2 pidfiles on Debian

I'm using celery 2.2.7 on Debian.  When I start celerybeat from the init.d script, two pid files are created:
/var/run/celerybeat.pid
./celerybeat.pid
The file "bin/celerybeat.py" contains the default value "celerybeat.pid" for the option --pidfile, which seems
to create the later pid file.  Is this expected or a bug?
On the other hand, celeryd has the default value None for the option --pidfile, which has no problem.

Closed by cb7442fbaf3167ca7ef402408c03b45976cd8484


[evm]: reportevm: execution reverted when transferring MNT to multicall address.

Describe the bug
When I try to transfer MNT to multicall address by metamask, it report error: evm: execution reverted.
And the transaction is https://explorer.testnet.mantle.xyz/tx/0x8e23874f0a31abe0cd8f4777b6258d24a1490dcb5a0737bf544d1b0f7d245eb9 .
The discussion in the discord: https://discord.com/channels/993846798382805002/1062060472276369488/1203961717692960858
To Reproduce
Steps to reproduce the behavior:

Go to metamask.
Transfer MNT to 0xcA11bde05977b3631167028862bE2a173976CA11.
See error in the explorer.

Expected behavior
Can transfer MNT to contract addresses.
Screenshots
No
Additional context
If we cannot transfer MNT to contract addresses, the multisign contracts and ERC4337 may not work, too.

My mistake. I forget the contract have to provider receive method.


Using class and id exercises

Earnestly, I'm not sure if I'm getting the hang of this GitHub and Git CLI... I wish there was some way I could know if
I'm doing things correctly. I'm not sure if whether my submissions are performed correctly and showing in the right
repository. It is scary and frustrating. I'll try discord... maybe I'll get advise or something.

Hi @nkosithipe 👋
Thanks for sharing your thoughts and concerns. Please visit the Odin Project community on Discord and ask for help in the #git-help channel (under the Tech Support section).
Additionally I would recommend going through the beginner git lessons again in Foundations for extra practice:
https://www.theodinproject.com/lessons/foundations-introduction-to-git
https://www.theodinproject.com/lessons/foundations-git-basics
Learning to use GitHub is certainly going to take some time and practice. It can certainly be frustrating and it's easy to forget how to do things you've already learned and done before! So please ask for help in the Discord whenever you need it or just to clarify any concepts.
I'm going to close out this Pull Request since we only use Pull Requests for making changes to the files in the repositories and not for discussion.
Best of luck and happy learning!


Cannot add more properties to @deno/shim-deno

Can you please remove Object.preventExtensions for the Deno object? I can't add additional polyilfils that seem to be missing during my local development.

Given that Deno actually supports mutating the Deno object, I think this should be classified as a bug.


cwise docs out of data?

In some of the examples, a cwise function can be called these two ways:
let somearray = cwiseFunc(arr1, arr2)
cwiseFunc(somearray, arr1, arr2)

I could only get the second to work, so I wonder if the docs aren't up to date in this and perhaps other ways. Also, is this expected to work without the browserify transform or is that just to get size down? I'm in webpack.
In the mean time, any links to repos making use of cwise would get great to check out.

@matt-erhart I think it depends on the specific operation what the inputs and outputs are. In particular from the docs:

If no return statement is specified, the first ndarray argument is returned

That means there's not necessarily a single answer to what the return value would be. A reduction operator (like a sum or a max, etc.) would return the sum while other operators by default would return the first ndarray argument.
If you use ndarray-ops, the functions are pretty explicit about whether it's an in-place operator or not, e.g. add (c = a + b) vs. addeq (a += b). ndarray-ops also uses precompiled operators so that you don't have to worry about the browserify transform. In other words, ndarray-ops doesn't cover every use-case but the code is already transformed, which might be helpful.
As for the browserify transform, that's correct. You can use cwise with or without it. cwise uses esprima to parse the function code. This makes writing the body of the operator very easy but allows cwise to be sophisticated about how it actually applies the transform (in particular, it performs math in an inner loop instead of actually calling body as a function). Esprima is a pretty big dependency though. Without the browserify transform, esprima is a dependency and the operator is created in user-facing code. With the transform, the operator is created at build time so that esprima is not a client-side dependency. I think you can use ify-loader to use the cwise transform with webpack. I don't have an example repo on hand, but if you have success with this method, I'd love to hear about it!

(Apologies for the scattered answer. You're basically correct. Things should work just fine, but if you run into problems, don't hesitate to ask more questions.)


Add Index Page

Assign me this issue and provide clear description

Just a Index page. Nothing special. There are no guidelinesy Feel free to design something.
Appreciate the work


[question] Restore and email notification

I was wondering if this solution would also include the possibility of emailing backup logs (to ensure procedures went properly) and also a way to restore the backup to the volumes in case of an accident to prevent manual restoring in the near future.

@PCatinean we think this is not the purpose of this product to manage alerting. Instead, we choose to push metrics to a Prometheus pushgateway so that you can have all kind of metrics about your backups in Prometheus. From there you can push alerts to Alertmanager which will manage the email notifications.
About a way to restore the backup, I'm not sure it's the purpose of this tool either. How would you see that?

You are right about both, as I got a bit more experience in the field I realized this is not the way to go. Thanks for the clarification!


[PE-D][Tester E] Incorrect state after undo command

The incorrect state is displayed after the undo command is executed in the following scenario.
First the user executes the find command to show a filtered list of contacts

Next the user adds a contact with the add command

This will cause the UI to display all contacts currently in the list, including the most recently added contact.
Executing the undo command will cause the list to revert back to the filtered list displayed from the find command, which may cause confusion among users since it is expected to simply undo the add command and display all contacts in the list without the newly added contact.

Labels: severity.Medium type.FeatureFlaw
original: volleyballkickedme/ped#9

This is intentional and done so that multiple undos are supported. We want "undo" to revert to the exact same state, including the displayed list.
This could be stated more clearly in the User Guide.

Closed as an enhancement, not a bug.


to_s should return a meaningful value

Right now printing an enum instance produces something like
#<Stash::Indexer::IndexStatus:0x00564952a50e00>

when it would much more usefully produce something like
Stash::Indexer::IndexStatus::FAILED

or possibly
Stash::Indexer::IndexStatus::FAILED (value="failed")

Fixed in 0.1.7, but we forgot to close the issue.


Get reference and mutable reference to bytes

Currently calling into_bytes() consumes the object, but it would be nice to have access to the bytes via a reference or mutable reference.

Seconding this — even a non-mutable reference could possibly be a make-or-break feature for me.