0.9.5-RC3 was working for Consumer, later releases return Null messages

Description
We're using Kafka <IP_ADDRESS> to populate a topic that we've been reading from without issue, using the 0.9.5-RC3 client. However, after upgrading to 0.11.0, the consumer receives no messages at all, behaving like the topic is not recognized at all (Consume() returns null, and no errors are thrown - as if the topic name is wrong...). Trying the 0.9.5 Client resulted in a similar failure. We can see this without using any of our code (SimpleConsumer examples for 0.11.0 and 0.9.5 don't work and 0.9.5-RC3 works).
How to reproduce
We can reproduce by just running the current 0.9.5-RC3 and latest (0.11.0) SimpleConsumer examples and connecting to a Kafka <IP_ADDRESS> topic.
Checklist
Please provide the following information:

[ ] Confluent.Kafka nuget version: 0.9.5+
[ ] Apache Kafka version: <IP_ADDRESS>
[ ] Client configuration: Windows Service running on .NET 4.5 (Visual Studio 2013), also tried with VS2017 + .NET 4.6
[ ] Operating system: Windows 7 Client

Can't think of what could be the problem. Can you set the "debug" configuration property to "all" and paste the output here.

Thanks for the prompt response - here are the debug:all logs for the simple consumer example using 0.11.0 - I suppose it looks like it does find the topic (?):
out_0110.txt
For comparison, here are the logs for 0.9.5-RC3:
out_095RC3.txt
When running the simple consumer example in 0.11.0, no additional output besides the log is created, while 0.9.5-RC3 prints out the message content.

There are lots of PROTOERR's in there... Maybe try setting api.version.request to false and broker.version.fallback to 0.9.0. Can you share details of the configuration of the producer that produced the messages? It looks like something is going wrong under the covers.. I've not seen this before.
4|2017-09-20 09:19:31.031|rdkafka#consumer-1|PROTOERR| [thrd:dnfbkafka05:9092/5]: dnfbkafka05:9092/5: Protocol parse failure at 1048290/1048621 (rd_kafka_msgset_reader_msg_v0_1:464) (incorrect broker.version.fallback?)
4|2017-09-20 09:19:31.031|rdkafka#consumer-1|PROTOERR| [thrd:dnfbkafka05:9092/5]: dnfbkafka05:9092/5: expected 595 bytes > 331 remaining bytes

It looks like it is indeed fetching messages from the broker:
...
7|2017-09-20 09:19:33.139|rdkafka#consumer-1|FETCH| [thrd:dnfbkafka05:9092/5]: dnfbkafka05:9092/5: Fetch topic ie-trans-out1 [0] at offset 138217863 (v2)
7|2017-09-20 09:19:33.139|rdkafka#consumer-1|FETCH| [thrd:dnfbkafka05:9092/5]: dnfbkafka05:9092/5: Fetch 1/1/4 toppar(s)
7|2017-09-20 09:19:33.139|rdkafka#consumer-1|SEND| [thrd:dnfbkafka05:9092/5]: dnfbkafka05:9092/5: Sent FetchRequest (v2, 72 bytes @ 0, CorrId 257)
7|2017-09-20 09:19:33.430|rdkafka#consumer-1|RECV| [thrd:dnfbkafka05:9092/5]: dnfbkafka05:9092/5: Received FetchResponse (v2, 152279 bytes, CorrId 257, rtt 281.00ms)
7|2017-09-20 09:19:33.430|rdkafka#consumer-1|FETCH| [thrd:dnfbkafka05:9092/5]: dnfbkafka05:9092/5: Topic ie-trans-out1 [0] MessageSet size 152234, error "Success", MaxOffset 138218113, Ver 2/2
7|2017-09-20 09:19:33.430|rdkafka#consumer-1|CONSUME| [thrd:dnfbkafka05:9092/5]: dnfbkafka05:9092/5: Enqueue 250 message(s) on ie-trans-out1 [0] fetch queue (qlen 250, v2, last_offset 138218112)
7|2017-09-20 09:19:33.430|rdkafka#consumer-1|FETCH| [thrd:dnfbkafka05:9092/5]: dnfbkafka05:9092/5: Fetch topic ie-trans-out1 [0] at offset 138218113 (v2)
7|2017-09-20 09:19:33.430|rdkafka#consumer-1|FETCH| [thrd:dnfbkafka05:9092/5]: dnfbkafka05:9092/5: Fetch 1/1/4 toppar(s)
7|2017-09-20 09:19:33.430|rdkafka#consumer-1|SEND| [thrd:dnfbkafka05:9092/5]: dnfbkafka05:9092/5: Sent FetchRequest (v2, 72 bytes @ 0, CorrId 258)
7|2017-09-20 09:19:33.526|rdkafka#consumer-1|RECV| [thrd:dnfbkafka05:9092/5]: dnfbkafka05:9092/5: Received FetchResponse (v2, 40856 bytes, CorrId 258, rtt 109.00ms)
7|2017-09-20 09:19:33.526|rdkafka#consumer-1|FETCH| [thrd:dnfbkafka05:9092/5]: dnfbkafka05:9092/5: Topic ie-trans-out1 [0] MessageSet size 40811, error "Success", MaxOffset 138218180, Ver 2/2
7|2017-09-20 09:19:33.526|rdkafka#consumer-1|CONSUME| [thrd:dnfbkafka05:9092/5]: dnfbkafka05:9092/5: Enqueue 67 message(s) on ie-trans-out1 [0] fetch queue (qlen 67, v2, last_offset 138218179)
7|2017-09-20 09:19:33.526|rdkafka#consumer-1|FETCH| [thrd:dnfbkafka05:9092/5]: dnfbkafka05:9092/5: Fetch topic ie-trans-out1 [0] at offset 138218180 (v2)
7|2017-09-20 09:19:33.526|rdkafka#consumer-1|FETCH| [thrd:dnfbkafka05:9092/5]: dnfbkafka05:9092/5: Fetch 1/1/4 toppar(s)
7|2017-09-20 09:19:33.526|rdkafka#consumer-1|SEND| [thrd:dnfbkafka05:9092/5]: dnfbkafka05:9092/5: Sent FetchRequest (v2, 72 bytes @ 0, CorrId 259)
7|2017-09-20 09:19:33.576|rdkafka#consumer-1|RECV| [thrd:dnfbkafka05:9092/5]: dnfbkafka05:9092/5: Received FetchResponse (v2, 5529 bytes, CorrId 259, rtt 47.00ms)
7|2017-09-20 09:19:33.577|rdkafka#consumer-1|FETCH| [thrd:dnfbkafka05:9092/5]: dnfbkafka05:9092/5: Topic ie-trans-out1 [0] MessageSet size 5484, error "Success", MaxOffset 138218189, Ver 2/2
7|2017-09-20 09:19:33.577|rdkafka#consumer-1|CONSUME| [thrd:dnfbkafka05:9092/5]: dnfbkafka05:9092/5: Enqueue 9 message(s) on ie-trans-out1 [0] fetch queue (qlen 9, v2, last_offset 138218188)
7|2017-09-20 09:19:33.577|rdkafka#consumer-1|FETCH| [thrd:dnfbkafka05:9092/5]: dnfbkafka05:9092/5: Fetch topic ie-trans-out1 [0] at offset 138218189 (v2)
...

So it is a bit weird if you don't see any messages in your consumer.
Can you share your client configuration and your consumption loop code?

@mhowlett
The PROTOERR are actually no errors but partial receives, it is only debug=protocol that turns them into error logs (which is weird, yes). The partial messages are an optimization in the broker to allow sendfile() usage which the client is supposed to handle (and so it does)

ahh ok.
he notes the problem occurs with the SimpleConsumer example code which uses the deserializing consumer. Perhaps try using the non-deserializing consumer. I'm not sure why this would work, but it will at least narrow down the problem a little.

You're right - I think the issue is in the StringDeserializer. Running the following code:
'''c#
public static void Main(string[] args)
{
string brokerList = args[0];
var topics = args.Skip(1).ToList();
        var config = new Dictionary<string, object>
        {
            { "group.id", "simple-csharp-consumer" },//{"debug", "all"},
            { "bootstrap.servers", brokerList }
        };

        using (var consumer = new Consumer(config))
        {
            consumer.Assign(new List<TopicPartitionOffset> { new TopicPartitionOffset(topics.First(), 0, 0) });

            while (true)
            {
                Message msg;
                if (consumer.Consume(out msg))
                {
                    Console.WriteLine($"Topic: {msg.Topic} Partition: {msg.Partition} Offset: {msg.Offset} {msg.Value.Count()}");
                }
            }
        }
    }
}

'''
Produces the following output:
out.txt
So, I can see the byte arrays are not empty and getting through. Our messages are JSON strings, so this makes sense... Did something change about the deserializer from 0.9.5-RC3 onwards?

you'll probably get events on OnConsumeError - the strings are probably not valid utf8.

previously the de-serializers would throw an exception if there was an error decoding (can't recall exactly when that changed). it may be appropriate to specify a different string encoding in the deserializer, or write your own deserializer.
feel free to provide more information about / dump of the encoded message if you think it's reasonable that something other than what currently happens should happen.

Hmm, on the producer side, we're using the org.apache.kafka.common.serialization.{Serde, Serdes} (in Scala):
object AclArchiveDestination extends KafkaStreamDestination[JLong, String]{
  override def deliverStream: (PrefixedProperties) => (KafkaStreams) => (KStream[JLong, String]) => Unit =
    props => streamsController => stream => {
      val longSerde: Serde[JLong] = Serdes.Long()
      val stringSerde: Serde[String] = Serdes.String()

      stream.to(longSerde, stringSerde, props.getProperty("stream.output.topic"))
      startStream(streamsController)
    }
}

That should be UTF8 by default, looking at org.apache.kafka.common.serialization:
public class StringSerializer implements Serializer<String> {
    private String encoding = "UTF8";

    public StringSerializer() {
    }
....

Do we have to explicitly specify it now?

oh, your keys are long? the problem is there then I think - if you're using the simple consumer example as is, it expects null keys. if it gets any data at all, that will cause the Consumer to give consume errors.

Yes, this is it! We were keeping our consumer agnostic to key type and passing a Null deserializer for the key. This used to work in 0.9.5-RC3, but as you mentioned, it looks like we now need to match the consumer key deserializer with the produced type, even if we do not use the key at all. Not sure if that was a desired effect of the changes from 0.9.5-RC3, but we can move forward - changing the consumer  to use a long deserializer resolved our issue. Many thanks!

I think we should add an Ignore deserializer type (alternatively parameterize the null deserializer constructor, though I think that would be worse).


need to catch up to 2017 news and hits

Got inspired from http://christopheradams.io @christopheradams and updated this site with the new video. Need to look back and find articles and update this site to latest.

nuked


s3fs: 1.88 -> 1.89

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/s3fs-fuse/versions.
meta.description for s3fs is: "Mount an S3 bucket as filesystem through FUSE"
meta.homepage for s3fs is: ""
meta.changelog for s3fs is: ""
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 1.89 with grep in /nix/store/bis5ks61aqarc4b04ybnyfxpk1cznb6b-s3fs-fuse-1.89

directory tree listing: https://gist.github.com/b4ac59da3bb0a55f3e9316fbca0fa587

du listing: https://gist.github.com/1f7a750c31859ed7dc2e1db64173e056

Rebuild report (if merged into master) (click to expand)

4 total rebuild path(s)

1 package rebuild(s)

1 x86_64-linux rebuild(s)
1 i686-linux rebuild(s)
1 x86_64-darwin rebuild(s)
1 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
s3fs

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/bis5ks61aqarc4b04ybnyfxpk1cznb6b-s3fs-fuse-1.89 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A s3fs https://github.com/r-ryantm/nixpkgs/archive/1b5aea9a7afd24060ec2188a48cb5518eda9ddcd.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/bis5ks61aqarc4b04ybnyfxpk1cznb6b-s3fs-fuse-1.89
ls -la /nix/store/bis5ks61aqarc4b04ybnyfxpk1cznb6b-s3fs-fuse-1.89/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

s3fs

Maintainer pings

Result of nixpkgs-review pr 114428 at 1b5aea9a run on aarch64-linux 1

1 package built:

s3fs

2 suggestions:

warning: maintainers-missing
Package does not have a maintainer. Consider adding yourself?
Near pkgs/tools/filesystems/s3fs/default.nix:27:3:
   |
27 |   meta = with lib; {
   |   ^

warning: unclear-gpl
gpl2 is a deprecated license, check if project uses gpl2Plus or gpl2Only and change meta.license accordingly.
Near pkgs/tools/filesystems/s3fs/default.nix:29:5:
   |
29 |     license = licenses.gpl2;
   |     ^


Find older pods data using PromQL

Use PromQL to find all pods, related to the K8SObject.

Approved by @arikalon1 in https://github.com/robusta-dev/krr/pull/35


Redesign to allow storage methods customization

This is a major redesign proposal. I will try to point out the changes:

Samples were restructured - shared code has been extracted to shared project, 2 new sample projects for blazor server version.

Store handling has been extracted to classes that implement IStoreHandler. In the project there are 2 classes that implement it:

JsonStoreHandler.cs - this is the previous code that was in PersistMiddleware and was responsible for state to json serialization and json to state deserialization; it contains a store of type IStringStateStorage (sample implementation is in sample shared project LocalStateStorage.cs)
DictionaryStoreHandler (the name probably should be changed to more generic) - this store just passes the state to store of type IObjectStateStorage (sample implementation is in sample shared project InMemoryStateStorage.cs)

Store handlers and stores are added to DI in Startup.cs and Program.cs (depending on the hosting model)

White/black lists are based on dictionaries. Also consumers can use IPreserveState / ISkipPreserveState marker interfaces on states, to apply desired persist behaviour. The model was kept - either use exclusion approach (default) where ISkipPresreveState & black list is checked or use inclusion approach where IPresreveState & white list is checked. I left the white/black lists for 2 reasons:
a) mandatory black list
b) it is much easier to extend the package in the future and open ability to include/exclude state during program flow

There are couple of simple examples of how the include/exclude approach is working - play around with the commented section in Startup.cs/Program.cs .

InitializePersistMiddlewareAction.cs and reducer has been removed, as I do not see any need for it any longer.

In server version, before state is restored from storage, the initial values are visible (fraction of a second, but this is noticable). This can be solved with injecting preserving store to feature and provide initial value from the store (if exists). But because this is not yet really sanctioned as a correct approach, I kept it out of this solution.

Just adding some more info here about initial state.. this is how redux-persist initialization looks like.. @@INIT to default state, a PERSIST state where it initializes the persist middleware and a rehydrate state.  Some of the reasons for how they do things might not apply but there also may be good ideas worth stealing there.

https://github.com/rt2zz/redux-persist

I had a look through it looks like a lot of work!
Would it be ok with you to use attributes instead of adding the empty interface IPersistState to the states?
I like the idea of not imposing requirements on the state objects.
Also, changing from ISyncLocalStorageService to  ILocalStorageService seems to have caused some side effects in the WASM sample where it's not drawing correctly.. changing it back seems to fix it so I checked in this change:

Right....my bad. Seems that the code for pages cannot be really shared then. When running server version, async has to be used, otherwise it will not work. Unless you tell me what kind of issues you were experiencing - maybe they can be fixed without sacrificing the sharing of components?

It seems like it should be possible to make it work on both I wasn't aware that it has to be async for the server version.   I'll have another go at getting the client to refresh OK async.  You can see the issue in the screenshot showing on my previous comment.

Yeah, I have it on mine as well. What I found out is basically that the last SubscribeToAction fails to render. If you change the order, you will notice that.

So, if you add StateHasChanged(), then it is going to show it...
        protected override async Task OnAfterRenderAsync(bool firstRender)
        {
            if (firstRender)
            {
                CounterFromStore["Counter"] = await localStorage.GetItemAsStringAsync("Counter");
                CounterFromStore["CounterInclude"] = await localStorage.GetItemAsStringAsync("CounterInclude");
                CounterFromStore["CounterExclude"] = await localStorage.GetItemAsStringAsync("CounterExclude");

                SubscribeToAction((Action<IncrementCounterAction>)(result => PrintStateDirectly("Counter")));
                SubscribeToAction((Action<IncrementCounterIncludeAction>)(result => PrintStateDirectly("CounterInclude")));
                SubscribeToAction((Action<IncrementCounterExcludeAction>)(result => PrintStateDirectly("CounterExclude")));
                StateHasChanged();
            }
            await base.OnAfterRenderAsync(firstRender);
        }

Good call.. I reverted my change and added that.


Added the case of using Azure cloud shell

Even though this document has a chapter on using Azure Cloud Shell at the beginning, it was missing a note on using Azure Cloud Shell.
The text I added is based on the following.

[docs.microsoft.com] Quickstart for Bash in Azure Cloud Shell : Select the Bash environment
https://docs.microsoft.com/en-us/azure/cloud-shell/quickstart#select-the-bash-environment

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

@wesmc7777, it is not in progress.
If possible, could you proceed with the merge?
Do I need to do anything else?

#sign-off

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


cannot get duration for some facebook URLs (have some warnings)

[x ] I'm reporting a broken site support issue
[ x] I've verified that I'm running youtube-dl version 2019.05.20
[ x] I've checked that all provided URLs are alive and playable in a browser
[ x] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[ x] I've searched the bugtracker for similar bug reports including closed ones
[ x] I've read bugs section in FAQ

Verbose log
PS C:\> .\youtube-dl.exe https://www.facebook.com/1159565407414586/videos/341386689906120/ --verbose --get-title --get-duration
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['https://www.facebook.com/1159565407414586/videos/341386689906120/', '--verbose', '--get-title', '--get-duration']
[debug] Encodings: locale cp1252, fs mbcs, out cp437, pref cp1252
[debug] youtube-dl version 2019.05.20
[debug] Python version 3.4.4 (CPython) - Windows-10-10.0.17763
[debug] exe versions: ffmpeg N-93129-g9e1e521393, ffprobe N-92435-g759589b55e
[debug] Proxy map: {}
WARNING: unable to extract thumbnail URL; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
[debug] Default format spec: bestvideo+bestaudio/best
Peak Design have redesigned the tripod from the ground up! ✅ Sleek, ergonomic...

Description
I want to get title and duration from that URL but get an error. Title I have got in last string but no duration.

Try -F
youtube-dl.exe -F https://www.facebook.com/1159565407414586/videos/341386689906120/ --verbose --get-title --get-duration

the same:
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['-F', 'https://www.facebook.com/1159565407414586/videos/341386689906120/', '--verbose', '--get-title', '--get-duration']
[debug] Encodings: locale cp1252, fs mbcs, out cp437, pref cp1252
[debug] youtube-dl version 2019.05.20
[debug] Python version 3.4.4 (CPython) - Windows-10-10.0.17763
[debug] exe versions: none
[debug] Proxy map: {}
WARNING: unable to extract thumbnail URL; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.


How to keep model in high alert state for faster inference?

I'm running insanely-fast-whisper in environment where low latency is crucial. Once .wav file is created, it needs to be transcribed immediately. Everytime I run:
D:\InsanelyFastWhisper>insanely-fast-whisper --file-name myoutputfile_17245952653c006.wav --device-id 0
Special tokens have been added in the vocabulary, make sure the associated word embeddings are fine-tuned or trained.
🤗 Transcribing... ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 0:00:00
Voila!✨ Your file has been transcribed go check it out over here 👉 output.json

I see some pause in this part: Special tokens have been added in the vocabulary, make sure the associated word embeddings are fine-tuned or trained.
And then Transcribing runs pretty fast. I assume the model is being loaded first, so it takes time.
How do I keep it in the high alert state so that transcription would run faster upon feeding of the new .wav file?
Thanks!

Don't use the CLI and use the transformers python way.


How to disable "sticky" feature

I love excalidraw, but there is this very annoying feature.
When I draw a line, then write some text next to it, the text is now "stick" with the line.

Whenever I move the text, that line stretches as well.
Is there a way to disable this? If not, can this be added as a feature?

Closing this as I have created a new issue where we will be discussing this and improve the overall UX for arrow binding https://github.com/excalidraw/excalidraw/issues/6685


Fix a version that refer for k8s.io/apimachinery

https://github.com/Peripli/service-broker-proxy-k8s/blob/master/Gopkg.lock#L666
The proposal is not to refer "master" branch (as it can bring instabilities) but to refer a stable version. For instance version 1.10.9.

Fixed with https://github.com/Peripli/service-broker-proxy-k8s/pull/56


FIX: Memory leak + infinity future

Described issue in #14 .
Added getter for isClosed to check whenever connection was closed.
Closes #14

I've noticed also that you upgrade minimum version of web_socket_channel, but not every package has upgraded it as well, wouldn't it better to support various versions? 👀  I'll provide additional commit for that, let me know if thats ok.

Previously it was >2.4.0 so to maintain previous version, I've provided '>=2.4.0 <4.0.0' up to version 4 since it may have some braking changes in future.

hey @PcolBP thanks for the PR,
isClosed should start at true, that's correct, adding a try/catch around isReady is also a good idea !
The getter is handy aswell
Tell me if i'm wrong but I think there is no need for _hasInitializationException since if start() throws, isClosed is never set to false so calling close would do nothing :
Future<void> close() async {
    if (_isClosed) return;
...

We have to close as well controller so since it will leave all the time if not closed.
I'll change isClosed on true, you're.
If we would like to remove _hasInitializationException we should make make controller initialisation later or make it nullable from the start 🤔

ah yes you are right, maybe we could make _outputTranscriptStream a late field, and instantiate it after a succesful _wsChannel.ready ?

It depends on the users use case but obviously it would be the breaking change since behaviour would change

yeah I would like to keep the API is it is, all things considered your changes look good, but I'm not sure that close() would work as expected

Never mind LGTM. only thing is _isClosed should be instantiated to true, since by default the stream is open


[Release v1.14.0-alpha.1] Update cmd/cmctl's go.mod to v1.14.0-alpha.1

This PR cmd/cmctl's go.mod to v1.14.0-alpha.1 as part of the release process.
To the reviewer: the version changes in go.mod must be reviewed.
NONE

/hold

/lgtm
/approve

/unhold since the staging build succeeded: https://console.cloud.google.com/cloud-build/builds/49e0f88a-3c6a-4fcd-a2b4-d5b65a9e6492?project=cert-manager-release

/unhold


debug.startDebugging cannot find preLaunchTask

I am trying to create an extension that will be used for debugging Python code running in an embedded interpreter in another software. I have registered a command which is used to start the debugging process like this.
	vscode.debug.startDebugging(undefined,             {
		"name": "Remote Debug",
		"type": "python",
		"request": "attach",
		"connect": {
			"host": "localhost",
			"port": 5678
		},
		"preLaunchTask": "my-debug"
	});

The task is configured in my package.json like this
		"taskDefinitions": [
			{
				"type": "my-debug",
				"properties": {
					"isBackground": true
				}
			}
		],
		"problemMatchers": [
			{
				"name": "debug-problem-matcher",
				"owner": "custom",
				"pattern":[
					{
						"regexp": "."
					}
				],
				"background": {
					"activeOnStart": true,
					"beginsPattern": ".",
					"endsPattern": "Ready to attach the debugger"
				}
			}
		]

and this is the function I use to activate the Task
function registerTaskProvideDebug(context: vscode.ExtensionContext) {
    vscode.tasks.registerTaskProvider("my-debug", {
        provideTasks(token?: vscode.CancellationToken) {
            let shellCommand = .....;

            var execution = new vscode.ShellExecution(shellCommand);
            var problemMatchers = ["debug-problem-matcher"];
            return [
                new vscode.Task({type: "my-debug"}, vscode.TaskScope.Workspace,
                    "my-debug", "my-extension-name", execution, problemMatchers)
            ];
        },
        resolveTask(task: vscode.Task, token?: vscode.CancellationToken) {
            return task;
        }
    }
    );
}

However when I run my command I get the message Could not find the task "my-debug". The task runs find from the Terminal > Run Task. I have also tried to set it as "my-extension: my-debug" in startDebugging with no success.
Am I doing something wrong, is this a bug or it just this isn't possible to be done?

I need to update the task documentation to account for user tasks, but there's a note here that says tasks only work when there's a folder open: https://code.visualstudio.com/Docs/editor/tasks
From a tasks perspective, if no folder is open and an extension contributed task is passed into getTask that task just doesn't exist. The behavior in the issue sounds by design to me.


Add missing ABSL_DLL for a few functions

The throw_bad_optional_access and ThrowBadVariantAccess are used in a few inlined functions and should have the ABSL_DLL prefix, otherwise linking with absl.dll will fail with errors like:
error LNK2019: unresolved external symbol "void __cdecl absl::
optional_internal::throw_bad_optional_access(void)" (?throw_bad_optional_access@opt
ional_internal@absl@@YAXXZ) referenced in function "public: class base::TimeDelta &
 __cdecl absl::optional<class base::TimeDelta>::value(void)& " (?value@?$optional@V
TimeDelta@base@@@absl@@QEGAAAEAVTimeDelta@base@@XZ)\r\n'

Thank you. This should be merged soon.


Cross-browser Blockstack state being reset

Not sure if this is a known issue, but I noticed some discrepancies between different browsers on Blockstack
Safari doesn't have access to my Blockstack User Data:

But Chrome does:

On Chrome, the blogs are fetched:

But on Safari, the blogs are not fetched:

Posting a blog on Safari will not appear on Chrome.
New Blog post made on Safari:

Doesn't appear on Chrome:

Chrome is set to the default Blockstack Browser on my computer, so I'm assuming that probably has something to do with Safari not being able to access my Blockstack user/storage info. However, it might be confusing if users do something on a Blockstack App using Safari, but don't see the changes when accessing the app using a different browser.

I just tested this on Firefox, and the blog posts I made on Safari show up on Firefox and vice versa. It's just Chrome that doesn't share data with Safari & Firefox.
Firefox also doesn't have access to my Blockstack account data. I'm assuming this is because Chrome is set to my default Blockstack browser.
Blogs written on Safari are shown on Firefox

This issue was actually being caused by something else. Opening a separate issue


improve Navigator 1.0 connectivity

For example, developer could choose if pushing a new page should influence path. That page must be BeamPage so it needs an optional addToPath default false (or maybe true?).
Basically, a reverse process of what is now happening: path being transformed into a list of BeamPage

Ok, let's do this.
In general, update will go both ways: building pages from state and taking URI from pages when they are built. In fact, it's better that the last BeamPage gives URI. Then arbitrary pops are solved nicely.
@AdamBuchweitz thanks for reminding me of this


No issue porting json raw tx result adapter

What kind of change does this PR introduce?

[ ] bug fix
[x] feature
[ ] docs
[ ] update

What is the new behavior?
Adapting to lbd domain tx-result events
Remained

Add more adapting events

Sorry this is duplicated with #77


fix: Monaco editor links opened in blank window in electron

Description
Monaco editor has option config to auto detect urls in text. ctrl + click on it will call vscode.open() -> window.open() open url.
Our current electron setting can handle open in browser, but<EMAIL_ADDRESS>seems has issue accept correct arguments. then we will see a empty page, case the accepted arguments url is about:blank.
Alternative solution
If we upgrade<EMAIL_ADDRESS>the electron opened window would be correctly, but still failed to open in default browser.
Current solution
So before find a better way, current fix is disable link detecting in BaseEditor.text for electron environment.

Task Item
close #4070

Screenshots
Before (electron):

After (electron):

Coverage remained the same at 55.739% when pulling f53d86dff3c1f53a8b40c142fcc2c2e2498f8443 on zhixzhan/electron-open-link into 8ad58bd1b4a2989c175e48b0f3a428fc4621f3db on main.


Start Using SAP HANA 2.0, express edition (Virtual Machine Method)

Hello,
2 suggestions for improvement:

Probably at the beginning of step 2, you could advise people how to change their keyboard layout (since not everyone has the provided default English(US) keyboard, and so e.g. the character '-' may not be input correctly, and so on. The spell is:
sudo yast2 keyboard
Then use arrow keys and tab keys to setup the keyboard appropriately.

In my VM (running on VMWare Player on Windows 10), there is no scrollbar or window resize possibilities, so it may be difficult to confirm that on running
HDB info we can see that all the needed services are running. A workaround is this:
HDB info > hdbinfo.txt
vi hdbinfo.txt  (then use e.g. :q to exit from vi, since many people will not know that and could be stuck inside vi :) ).

Hope these are useful suggestions. Have a nice day and best regards,
-dylan drummond

Thank youvery much for the input, Dylan!


test

Signed-off-by: joshvanl<EMAIL_ADDRESS>

@JoshVanL I think something must have gone wrong when the jobs configmap was updated in https://github.com/jetstack/testing/pull/591#issuecomment-933631703
When I look at the job-config configmap in the Prow build cluster, there is no spiffe jobs at all.
I wonder if making some small change against this config (i.e specifying the branch could kick the update). I think it may have gotten confused when the file was renamed.
Otherwise, I see this repo appear in tide's and hooks' logs, it just doesn't seem to have any tests registered against it

I see that the configmap now has the spiffe jobs
/test all

wow actually working, I didn't expect that

/close


proper styling

Create a sidebar with controls and a main content page with everything else.
The sidebar should be minimal to take away as little as possible from the actual visualization

something like this:


Extension is no longer tracking time

Ever since the most recent update to the extension, it has stopped tracking time. I have tried removing it and configuring it all again, but the result is the same.
Are there any logs that I can look at to see why nothing is being logged?  It worked well for months.

Just had this error get spat out in my console from one of the sites its supposed to be tracking

Uncaught (in promise) Error: Extension context invalidated.
    at wakatimeScript.js:1:371252
    at wakatimeScript.js:1:371211

I see "Cross-Origin Request Blocked" in the logs. I believe that https://github.com/wakatime/browser-wakatime/commit/88cd9943074d468359de98c551c6f964b0469191 caused the issue. That commit removed <all_urls> from permissions. <all_urls> was valid in manifest v2, but is now moved to host_permissions in manifest v3. So it is invalid as said in the commit on Chrome, but not on Firefox, which is still using manifest v2. Now on Firefox, the extension does not have permission to send cross-origin requests to the WakaTime server.
However, adding back <all_urls> would pose potential security risks. It would be better to only declare permissions for the WakaTime server. But in order to support custom servers, it should request permissions during runtime when the user sets the API URL in the settings.
By the way, I think we can use manifest v3 for Firefox to be consistent with Chrome.

By the way, I think we can use manifest v3 for Firefox to be consistent with Chrome.

#161 was opened just 2 days after Firefox 109 with MV3 support was released. I guess that was the reason to keep using MV2 for Firefox. Now it makes sense to migrate to MV3 for Firefox.

@ouuan Is there any workaround to get this working for now? Surely I'm not the only one experiencing this issue?

@ouuan Is there any workaround to get this working for now? Surely I'm not the only one experiencing this issue?

First, I want to make sure whether you are using the official WakaTime server (https://api.wakatime.com/) and whether you are using Chrome or Firefox. According to your screenshot, it seems that you are using Chrome. However, based on my experiments, the CORS request failure only happens for Firefox and custom WakaTime servers.
For further troubleshooting:

Check the "Site access: Allow this extension to read and change all your data on websites you visit" option on the extension details page.
Do you always see the "Extension context invalidated" error? I only see this error when I reinstall the extension.
Go to chrome://inspect/#service-workers and click "inspect" of jnbbnacmeggbgdjgaoojpmhdlkkpblgi. See the logs there.

@ouuan Thank you for getting back to me.

I am using https://api.wakatime.com/api/v1:

I am using Google Chrome Version 130.0.6723.117 (Official Build) (64-bit)
Permissions
Read and change all your data on all websites

"Extension context invalidated" It was something that I noticed shortly after logging this in GitHub and I had just reinstalled the extension as a last resort to try and get it working again.
If I click on just the service worker there is nothing, it is blank:

If I click on the other link under the service worker I get the below:

Is there anything else I can look at to find the issue?

3. Permissions
Read and change all your data on all websites

What about the "Site Access" under "Permissions"? You can change the permissions there.

5. If I click on just the service worker there is nothing, it is blank:

Also check the Network tab. Switch between tabs and go back to the service worker inspector to see if heartbeats are sent.

I have this:

Can't find anything regarding Site Access under:
chrome://settings/content/siteDetails?site=chrome-extension://jnbbnacmeggbgdjgaoojpmhdlkkpblgi
Checking the network tab whilst flipping through a few sites I get the below:

This is all being performed whilst using Chrome on Windows.  I thought I should give it a try using Chrome on Mac and its the same result.
I also tried using Chrome in Incognito mode with only the Wakatime extension enabled to remove any doubt of a conflicting extension, and it's the same outcome.

Try clicking the extension icon. It should send requests to the WakaTime server to get and display the stats if working normally:

This always shows me my current time from when I work and track time through VSCode. And when I inspect the extension and then click on the icon I get the same network result as per my previous screenshot.
When I reinstalled the extension, I also removed all of my included sites and tested with one site to track thinking that maybe one of my configured sites was causing an issue but even with just using once site, no tracking came through.  I tried with a few sites individually to get a good sample set of tests and none of them worked.
Investigating a bit further, in the previous version when it did work.  When you added a new site you needed to add "@@" after the URL to track so that it came up in Wakatime under the category/name you wanted.  I just checked now, and the example provided no longer includes the "@@" perhaps this is my problem?

@ouuan Alright, it's tracking it now, but it's not logging it under the correct Project. Now that the "@@" functionality has been removed... I see that I have to add it under projects in order to get it to correctly log under the right project as it was logging it under a completely irrelevant project name.
Looks like there was no issue with the extension, just configuration changes that I was unaware of after the most recent update.
I just double-checked the GitHub page and there isn't any reference to this configuration.
Anyway, glad its working now.
Thanks for the help.

Fine, so we are encountering different issues. I think you can close this issue as this thread is now off-topic for my issue on Firefox.

Good luck!


PR: Change for file save on split editor when tabs are moved

Fixes #5703.
This affected save(), save_as(), and save_copy_as() directly.  Also added changes for set_todo_results() and set_analysis_results() as they were affected by the issue of using the same index across editor stacks.

Hello @csabella! Thanks for submitting the PR.

In the file spyder/widgets/tests/test_save.py, following are the PEP8 issues :

Line 25:1: E302 expected 2 blank lines, found 1
Line 41:1: E302 expected 2 blank lines, found 1

Thanks for your contribution @csabella! It looks good to me, except for a minor comment I left. Please also fix the style issues reported by pep8speaks.

I think the reason why Pierre (Spyder's original author) didn't implement the functionality to move tabs in the Editor before was because of all the issues that would arise with split panes.
But thanks to the great work of Cheryl and @jnsebgosselin, now this is a thing from the past!!


FormData helper function

Describe the problem
We have currently no connection between forms and form actions. (I know this is a tricky topic)
It might be cool to improve the DX here, because forms are such an important part of the resilient application story of SvelteKit (works with and without JS)
If I am not mistaken, we should be able to extract some information on the route level.
This would enable the following patterns:

extract (typed) objects from FormData with a helper function
possibly guard against parsing errors
show warning in .svelte files if forms are lacking required input fields
show warnings, if form action is not available (e.g. '/upsi' does not have form action 'help'
possibly a thousand other cool things I can't think of right now

Things to think about:

files should probably be extracted separately
it might be good to have the possibility to mark fields as optional with something like sveltekit-form-input-optional or make use of required
this would only work in routes where there is a form present in the +.page.svelte file and the

Describe the proposed solution
Basic example of a helper function:
<!-- /test/+page.svelte -->
<form method="POST" enctype="multipart/form-data">
  <input type="text" name="text" id="text">
  <button type="submit">submit big file</button>
</form>

// /test/+page.server.js
export const actions = {
    default: async ({request}) => {
        const data = await proccesFormdata(request)
       // have a nicly processed data dependent on the request
       // do easy validation, for example with zod
    }
};

I know that form actions can be called from different routes, but maybe we can check that at compile time as well?
The example below could maybe show a warning like Form Action for '/test' also expects field with name="text"
<!-- /other/+page.svelte -->
<form action="/test" method="POST" enctype="multipart/form-data">
  <input type="text" name="upsi" id="text">
  <button type="submit">submit big file</button>
</form>

Alternatives considered
No response
Importance
would make my life easier
Additional Information
No response

@dummdidumm ok, I agree with your viewpoint.
I'll close it in favor of leaving that part up to third parties.
Maybe we'll get some sort of form-renderer in superforms, that takes in a zod schema 😊

@dreitzner the Zod schema can be introspected to a great degree, I'm using it to create constraints for the fields based on min/max values, etc, so you could use it to generate input fields based on a schema on the client.
I had that thought for a while, there is even some simple metadata available, but I'll scrap that for 1.0 and will let the Zod schema be the metadata. That should make it possible to build a very flexible form-renderer on top of Superforms, or even without it.


JSDoc documentation for components under lib/

Some components already have JSDoc comments, others don't
This task involves setting up the jsdoc NPM package and writing a compilation script for generating HTML documentation pages; maybe we can store these files on GitHub pages using a gh-pages branch served @ csss.github.io/csss-raspberry?

I don't like the generated JSDoc html 🤡


How much system resources does splash need?

In my earlier ages software used to come with "system requirements". Nowadays that hardware is cheap nobody cares any more. But some people need to optimize their stuff so please give me some numbers. For example in comparison to chrome/firefox. Currently Im using Selenium with HTMLUnit and in comparsion to headless chrome/friefox it needs about 1/4 of memory. How about splash (in combination with scrapy as this seems to be the way to go)?
Thanks

Have you seen https://splash.readthedocs.io/en/stable/faq.html#how-to-run-splash-in-production ?

that is great! thanks! Im still evaluationg. so what is the assumption here, how many session could I squeeze in when I limit the memory to 500MB? As I said selenium + htmlunitdriver can do 4 in parellel.

I believe that highly depends on the specific webpages you crawl. I suggest you test for yourself.

Splash is running a real browser engine (webkit, which is used in Safari), so in general it should be comparable to Chrome/Firefox in terms of memory and CPU usage. But as @Gallaecio pointed out, the usage depends heavily on the website. Also the will be some fixed overhead + some overhead for each tab.

really, full fletched browser engine? but memory consumption should still be a lot lower than a headless chrome since it runs the engine on server thus can handle multiple requests (in parallel?) and only renders js right?

If you are running splash on some server, and not interested in memory consumption of this server, but only interested in memory consumption of the splash process which talks to this server, then it would be quite low indeed, and per-session overhead is very low (depends on the spider).

no I do care about memory consumption. why is there no estimate? please someone give me some numbers.


Re-enable support for running metrics via (grafana/prometheus) on macos

Is your feature request related to a problem? Please describe.
In a previous PR here, changes were made to allow ability to run metrics infrastructure (grafana /prometheous) locally on macos.
Main changes in that PR involved not using network_mode since it is not supported in MacOS, see here where it is stated:

The host networking driver only works on Linux hosts, and is not supported on Docker Desktop for Mac, Docker Desktop for Windows, or Docker EE for Windows Server

The workaround on MacOs is to use port mapping instead.
The PR also introduced using host.docker.internal instead of localhost as localhost does not work with Docker for Desktop for Mac.
In a follow up PR here that de-duplicated files and re-organised the dashboard, the possibility to set needed endpoints via environment variable was introduced. Unfortunately this breaks the ability to run the dashboard on MacOs as it assumes network_mode: host works on MacOS
Describe the solution you'd like
Make changes to make it possible to run metrics via (grafana/prometheus) on macos again.
Ensure little duplication of configuration files as possible.

Sorry for breaking compatibility! :pray: Happy to review and merge changes to facilitate your DX


-returning tasklist

FIXED return tasklist was missing in batch.Export.imagecollection.toCloudStorage

I am sorry for the super late reply @hubert-thieriot, thanks! I'll merge right now


Uri setters

If I have code like this:
void main() {
   var u = Uri();
   u.host = 'dart.dev';
}

I get this result:
a.dart:8:6: Error: The setter 'host' isn't defined for the class 'Uri'.
 - 'Uri' is from 'dart:core'.
Try correcting the name to the name of an existing setter, or defining a setter
or field named 'host'.
   u.host = 'dart.dev';
     ^^^^

I think all the Uri properties should have setters, otherwise people are forced
to set them at time of object declaration.

Example here from another language:
https://developer.mozilla.org/docs/Web/API/URL/host
can get and set

Dart URIs are immutable. That makes it safe to share them, instead of everybody who stores a URI needing to copy them defensively.
If you want to change the host, you can use the replace function: u = u.replace(host: 'dart.dev');.
We have no plan to change the Uri class to be mutable.

It seems like the negatives would outweigh the positives of doing that. Ive never seen any programming language give back and object that you can even modify. Most languages leave that up to the user to do via freeze or similar. Giving back frozen objects by default would seem wasteful, as the user ends up having to create copies of the entire object, any time they want to make any change.

Immutable objects are actually quite widespread in object oriented languages. Because objects have identity, any mutation will immediately affect all other code that has access to the same object. Almost any language which have a String class also makes it impossible to change the content of an existing string. That's not because String is a "primitive type" (it rarely is), but because strings are objects that you very often share with other code. If you use a string as a map key, and someone else is able to change its contents, they break your map.
When you have shared mutable objects, you need some strategy to defend yourself:

Hopes and prayers. Just assume that nobody changes the object, and 99% of the time you're right.
Defensive copying. If you want to keep an object longer than just a single synchronous call, you make your own copy of it. (If the language supports copy-on-write, that might make it less expensive, but most don't). If you pass an object to someone else, you may want to make defensive copy first, to protect your own object. (And if you're really lucky, both ends do defensive copying).
Freezing. Isn't that just a JavaScript thing, and only because they provided it as a language feature? It's a bad API because that is itself a change to the object, one that might cause some other code to crash. Also, if half the the API stops working at some point, it's not particularly dependable. Do you need a way to check whether an object has been frozen, and then a way to get a thawed copy to work with?
Extensive invariant documentation. Every method documents whether it changes an object or not, and whether it requires its arguments to be unchanging. Then users are expected to satisfy that. See also: The first item.

Immutable objects avoid that issue entirely. They can be freely shared. Since Dart has asynchronous functions, which means that unpredictable other code can run during a single function code, it's even more important that you can trust your objects to stay they same.
Immutable objects are often data objects, entirely defined by the values you can extract from them.
The issue is then that if you need a different object, you need to create a new one. You have less reuse, but more security. Efficient generational garbage collection attempts to reduce the cost of creating new objects, but you still need a way to do it.

Builders. The immutable type has an associated mutable builder type, and you create a builder from the original, mutate the builder, then create a new immutable result from the builder. This separates mutation for use, but costs an extra class.
replace methods. Like uri.replace(scheme: "https"), a way to directly create a new object from the old one, but with some changes applied. This is more functional, less imperative, than the builders.
No help. Just use a constructor with values extracted from the old object. More verbose, but you can do anything.

The Dart SDK has no real builders. We have plenty of immutable classes (String, DateTime, Duration and Uri are some of the most prevalent), and some of them have update methods. Well, it's really just Uri, but we've considered adding it on DateTime and Duration as well. It's just a breaking change to modify their API.
A class like Future also has an immutable API, the API has no way to modify the future. Because of that Futures can safely be shared with anybody. It's not a fully immutable class because it has state that changes, but that's not part of the API that you share along with the object.
So, all in all, I think making DateTime immutable is a prudent design choice for a language like Dart. We should add a replace function to it, though.

Thanks for the detailed response.
I dont really have the energy to respond to all those points, but I will add a
little more.
Dart is not the first, or the only language that has asynchronous support. Both
Go (via the go keyword), and JavaScript (via async/await) have that as well
for just two examples. And in both cases, they offer mutable URL objects.
Freezing isnt just with JavaScript. Python offers two sets of objects as one
approach (set and frozenset), and Ruby has the freeze instance method for
existing objects.


Error:dynamic module does not define init function (initlibrealsense)

When I import librealsense, the error happened, how to fix it, please?
import librealsense
ImportError: dynamic module does not define init function (initlibrealsense)

Hi @geoffzhang
You should be doing import pyrealsense2 - see more information here

[Realsense Customer Engineering Team Comment]
hi @geoffzhang,
would like to check some basic background for your question.
what OS you are using?
which Realsense camera product you are using?
seems you are using python, please let me know if this is wrong,
thanks.

@dorodnic @RealSense-Customer-Engineering Thanks for your correspondings. Yes, I am using Python and my issue has not resolved.
First, my camera is realsense sr300.
The OS I used is ubuntu 16.04.
The version of librealsense I installed is 'librealsense-1.12.1'.
And the 'rs-capture' can run successfully.
The pyrealsense is installed by 'pip'.
second, my issues:
1、when I import pyrealsense, two warnings will appear:

import pyrealsense as rs
/home/geoff/anaconda2/lib/python2.7/site-packages/pyrealsense/extlib.py:26: UserWarning: librealsense not found.
warnings.warn("librealsense not found.")
/home/geoff/anaconda2/lib/python2.7/site-packages/pyrealsense/extlib.py:33: UserWarning: rsutilwrapper not found.
warnings.warn("rsutilwrapper not found.")
2、when I run Service(), an error will appear:
rs.Service()
Traceback (most recent call last):
File "", line 1, in
File "/home/geoff/anaconda2/lib/python2.7/site-packages/pyrealsense/core.py", line 27, in init
self.start()
File "/home/geoff/anaconda2/lib/python2.7/site-packages/pyrealsense/core.py", line 34, in start
lrs.rs_create_context.restype = ctypes.POINTER(rs_context)
AttributeError: 'NoneType' object has no attribute 'rs_create_context'
3、when I import pyrealsense2
import pyrealsense2
Traceback (most recent call last):
File "", line 1, in
ImportError: No module named pyrealsense2.

I hope to get your help, thanks.

I don't think we will be able to offer much insight on pyrealsense.
In order to use pyrealsense2 you need to install librealsense2 - it also supports SR300.


Error validating CycloneDX format in version 1.6

Hello,
I think there are various errors when validating a CycloneDX format in version 1.6.
Requirements:
$ trivy --version
Version: 0.56.1
[...]
$ cyclonedx-cli --version
0.27.1+6c0e6c8f45d446ed4a28891040592e4ae4a39c3c

Steps for reproduction:
$ trivy image --format cyclonedx --output /tmp/playwright.json mcr.microsoft.com/playwright/java:v1.47.0-noble-amd64
2024-10-14T14:53:21+02:00	INFO	"--format cyclonedx" disables security scanning. Specify "--scanners vuln" explicitly if you want to include vulnerabilities in the "cyclonedx" report.
2024-10-14T14:53:22+02:00	INFO	Detected OS	family="ubuntu" version="24.04"
2024-10-14T14:53:22+02:00	INFO	Number of language-specific files	num=1
$ cyclonedx-cli validate --input-file /tmp/playwright.json
Validation failed:
Value is "object" but should be "array"
http://cyclonedx.org/schema/bom-1.6.schema.json#/properties/tools/oneOf/1
On instance: /metadata/tools:
[...]
Unable to validate against any JSON schemas.
BOM is not valid.

Analysis
Value is "object" but should be "array"
http://cyclonedx.org/schema/bom-1.6.schema.json#/properties/tools/oneOf/1
On instance: /metadata/tools:
{
      "components": [
        {
          "type": "application",
          "group": "aquasecurity",
          "name": "trivy",
          "version": "0.56.1"
        }
      ]
    }

tools array is deprecated. Tools is object now - https://cyclonedx.org/docs/1.6/json/#tab-pane_metadata_tools_oneOf_i0
Required properties ["id"] are not present
http://cyclonedx.org/schema/bom-1.6.schema.json#/oneOf/0
On instance: /components/510/licenses/0/license:
{
            "name": "GPL-3.0"
          }

License can be id or name - https://cyclonedx.org/docs/1.6/json/#tab-pane_components_items_licenses_oneOf_i0_items_license_oneOf_i1
Value should have at most 1 items
http://cyclonedx.org/schema/bom-1.6.schema.json#/oneOf/1
On instance: /components/313/licenses:
[
        {
          "license": {
            "name": "BSD-3-Clause"
          }
        },
        {
          "license": {
            "name": "ISC"
          }
        },
        {
          "license": {
            "name": "GPL-3.0"
          }
        }
      ]

Licenses is array - why should it be one item? https://cyclonedx.org/docs/1.6/json/#components_items_licenses
Thank you for your attention to this ticket.

@simonfrancaix When I ran it, I had two instances of empty licenses
        {
          "license": {}
        },

which violate the specification as id or name is required. If I remove these two, it validates successfully.
However, what I don't understand why the validation produces so many misleading/wrong error messages.

It seems to report also the oneof cases against which it could not validate, even though another case succeeded, e.g.
the case 1 for tools (legacy tools), it can't validate
http://cyclonedx.org/schema/bom-1.6.schema.json#/properties/tools/oneOf/1
against it, but it can perfectly fine validate against the case 0 (the new tools).
Similarly, it can't in most cases validate against the LicenseChoice case 1 (license expression), but it can in most cases validate against the case 0 (licenses list).

Hi @andreas-hilti
I think this is a widespread issue that is also affecting v1.5
Can you take a look at this thread: https://github.com/CycloneDX/cyclonedx-maven-plugin/issues/579

@simonfrancaix @crimsonvspurple The number of (misleading) validation messages should be significantly reduced with release 0.27.2, and thus it should highlight the actual issue much better. Please give it a try.

Result of 0.27.2: https://github.com/crimsonvspurple/sbom-test/blob/master/validation_result.txt


Smooth animation

Is it possible to move pedestrians smoothly?

In my case there are only data points for every second. You could interpolated the value in-between. However, I am not sure what the impact on performance would be.


My Reservations

Well done @bigmosi, you've done a tremendous job. However, can you explain the purpose of the .idea folder?

@teckim thanks for having concern about the folder actually I don't understand how the .idea folder comes about, may come after running the npm install.


More force?

Firstly, this is amazing and thank you so much for providing it. I was wondering if there is a way to increase the force on the fingers. So it would bend at a higher threshold of force.

The bending of the fingers happens through the ConfigurableJoints on the hands. You should be able to increase the force in it to make the bending faster. To modify all fingers at once, you can search in the scene hierarchy for ‘t:ConfigurableJoint’. That will list only the things in the scene with that component.

Based on my tinkering I think the drives (x drive, y drive, z drive) are what does it on the configurable joint. I do think it would still be easier if this was just a field on your Hand Emulator script, because I have more configurable joints than just your hand so it's still quite tedious to manually select them all and change the drives. I can make a pull request if you'd like.

Go for it! I'd really appreciate the help. If you don't get a chance to, I can also implement it if you want.


Should provide option for "Set next statement" to set the current debug point.

Should be able to set the current statement to execute when in the debugger from the context menu like you can do in Visual Studio. This allows you to skip code that you don't want to execute which is often valuable when it is time consuming to have to change the code, maybe rebuild and restart the debugger. Would go nicely with 4993.

Any news on this topic?

@jawn for what language/debugger?

Tried it in Powershell.
Also interested in C#/javascript, don't know if it's a gap there as well.

We will add the (generic) frontend UI for this feature only if enough debugger backends are actually able to support this. So far interest for the feature and support from the debugger extension authors is small.

@weinand thanks for the clarification!

@weinand As far as I know, PowerShell's debugger does not support this feature (though it could in the more distant future).

Hi guys, when do you think this will be implemented? I find this feature very helpful when debugging code

+1 for C# or should this issue be created in C# plug-in?

This feature request will not be considered in the next 6-12 months roadmap and as such will be closed to keep the number of issues we have to maintain actionable. Thanks for understanding and happy coding!


Recording Mode Update Required

I have a new Hubitat setup and I've been testing this driver you have here.  Enjoying it so far, but I found one issue I wanted to share.  With my setup I have to change one of the Recording Mode values in the unifiProtectCamera driver for it to work correctly.  Without this change you are unable to set the mode back to "detections".
The word "motion" in this function does not work in my setup, it must be changed to "detections".  Maybe you could just add that value to the array if "motion" is still valid in other use cases.
`def setRecordingMode(mode)
{
if(!(["always", "motion", "never"].contains(mode)))
{
log.debug "unsupported recording mode (${mode})"
return
}
try
{
    getParent()?.httpExecWithAuthCheck("PATCH", parent?.genParamsMain(getCameraBaseUrl(), new groovy.json.JsonOutput().toJson([recordingSettings: [mode: mode]])), true)
}
catch (Exception e)
{
    log.debug "setRecordingMode() failed: ${e.message}"
}

}`
My current setup where this change is required
CloudKey Gen2
Unifi OS: 2.4.10
Network: 7.1.66
Protect: 2.0.1
Thank you for taking the time to build and share this driver with the community!

Good catch -- I made a change to add "detections" and also left "motion" as supported, since that name worked on older versions.
I didn't change the version in HPM, so if you installed
https://github.com/tomwpublic/hubitat_unifiProtect/commit/2e37b25a058cf6c69f1baebc034655edf3d19c38


Android, Lightmode, Custom setup dialog box with cancel visible on screen after entering password

@bmuramatsu I'm not sure I follow, could you provide more information on this issue?

Thanks! This will be fixed in #321


Getting issue 'Failed to resolve dependencies'

Not able to build and run this sample, getting error related to dependencies:
'because mobile-buy-sdk-ios >=5.1.0 contains incompatible tools version and root depends on mobile-buy-sdk-ios 5.1.0..<6.0.0, version solving failed.'
Is there any issue related to iOS Mobile Buy SDK version?
Cheers!

Yeah I am getting the same issue, any luck with fixing it @hemant89 ?

Facing the same issue above, when I try building the sample application, the build is failing and I'm getting:
'because mobile-buy-sdk-ios >=5.1.0 contains incompatible tools version and root depends on mobile-buy-sdk-ios 5.1.0..<6.0.0, version solving failed.'
Would we have to manually update the version of the mobile-buy-sdk-ios in the project? If so how would you go about this?
Any progress or workarounds on this issue? @Mako96 @hemant89


Fix api_version deprecation notes

There was a typo with the digits switched. The correct api version is 0.1.6.

We seem to currently have a failing test:
https://travis-ci.com/oeway/ImJoy/builds/138970410#L701-L702

Thank you.
The failed test likely caused by the web-python loading time exceeds the 20s limit, e.g. because of slow connection on the CI machine. It happens before and typically resolved by restarting the build. Let's try to increase the timeout value.

Ok, so it seems we solved it. Let's merge it.


Warn if not possible to set >1500 MTU on vxlan-vport

This check started to look a bit as an overkill to me, mainly because we need to specify exact port for vxlan-vport.
Fixes #1853

I'd suggest to add a note about the problem to https://www.weave.works/docs/net/latest/features/#fast-data-path

also, 150 lines of code just to print a warning for a condition that, to the best of our knowledge, has puzzled a single user, seems OTT.

I expect to see more puzzled users once 4.3 / 4.4 is widespread.

very, very few users set a WEAVE_MTU.

Re-opening against 1.6


Stem Player site returning 401 request on track retrieve

Installed the Stem Player emulator to Tampermonkey, tried downloading tracks from the platform page with this error
{"error":{"code":401,"error":"unauthorized","message":"JWT must be provided"}}

this is the same as issue #2

Fixed it, please update


Decorator for classes

A simple syntax decorator like this would vastly improve this library with how such features are moving towards in modern Python ecosystems.
import pydantic
from uuid import UUID
from pydantic_partial import optional

class Something(pydantic.BaseModel):
    name: str
    age: int

# All these fields are optional
@optional
class SomethingOptional(Something):
    pass

class SomethingId(Something):
    id: UUID

If someone can show me a direction on how to do this, I'm happy to support this ticket.

You could just use:
class SomethingOptional(
    create_partial_model(Something),
):
    pass

...which would give you even more control. Does this solve the issue for you?

Hmm, more control perhaps.
But I think the syntactic sugar is a really nice feature for a generalized partial. IMO it is much more readable as a decorator.

I did think about this again when doing my updates to support pydantic v2. My conclusion ist, that I don't really see any benefit in using decorators. In contrast I think this will create some strange use cases.
For example this would be valid:
@optional
class Something(pydantic.BaseModel):
    name: str
    age: int

Which would lead to the question why it was not defined as this in the first place:
class Something(pydantic.BaseModel):
    name: str | None = None
    age: int | None = None

I know the use case above was going to only use @optional on child classes, still you could do something like this:
@optional
class SomethingOptional(Something):
    additional_field: str

...which would lead to additional_field also being additional_field: str | None = None while the code you wrote kind of looks otherwise.
Also having a decorator would actually mean to create a new class from SomethingOptional, make all fields optional and then give the new class the name SomethingOptional again. I don't really like it being done this way.
So... I will not support creating partial models using a decorator. You may still just implement this yourself, shouldn't be that hard to do. 😉


(JS) For loop, not recognized in the tests

Hello freeCodeCamp team,
I am stuck in a problem in "basic Javascript", it seems that the tests expect a for loop, but do not accept the one I propose. I tried to delete the cache but it still does not work. In localhost the loop works without problems
Link: https://learn.freecodecamp.org/javascript-algorithms-and-data-structures/basic-javascript/iterate-with-javascript-for-loops/

Browser Name: Google Chrome
Browser Version: 67.0.3396.87 (Build oficial) (64 bits)
Operating System: Windows 10

@gburu98 Most probably it's failing because you deleted the example of the pre-written code. Reset the code and try again. I don't know why that pre-written code is required to pass this test.

@mpmaan Thank you very much. You were very helpful!!!

@gburu98 Please close this issue is you have no further doubt.


Added support for nbconvert via console

Thank you for the great plugin. The lines I've added implement a feature to direct call nbconvert from console and set docx as output format. The gui is not needed in this case.

New commit with removed failures (code style reworked)


Documentation licensing

Since the documentation examples are intended to be used as copy-paste, and the code is already licensed, why using attribution license for examples and pieces of code instead of CC0?

I'd start by reviewing issues #16470 and #16487 — for the most part this has to do with the need to contact every single individual contributor who has ever touched the docs to change which is an extremely painful process for a project of this size as I understand it.

Closing out since there are no planned changes here.
The docs content itself is under a different license than the code, yes, but the code included there is still under that MIT license as far as I know. However, as I'm no lawyer, if someone has a more authoritative source, I'd happily revisit.

@mdo to clarify,  are the examples are MIT licensed?
The README states "Code released under the MIT License.",  which seems to be a resounding yes...
But then "Docs released under Creative Commons." ... hmph.
The examples are under docs/,  but they are Code... which is it?


BigNumber conversion

Great work on the libs, it is a pleasure using them. I am running into one thing and that is the BigNumbers conversion. When I start doing arithmetic, say fe. adding 0.1 to a 200 stored as a Number type in dynodb, I get "200.1" as a string the following will become "200.1.1" etc etc.
As a quick solution, as I store deeply nested data, I just do a deepwalk converting all BigNumbers to regular number and all is normal again except for the fact I don't like to do a deep-walk at the edges of my system.
Is this a case foreseen, and by design, or not, and what is a way to go around that?

Thanks!  That doesn't like fun, and definitely isn't the intended behaviour.  Can you show an example?
 I have another update pattern, I am not using update-time!. I am doing a get-item! and put-item! and doing the increments in clojurescript (nodejs). As hildebrand converts to BigNumber when pulling data out of dynamo, I get to work with BigNumber in the clojurescript domain which does not handle the arithmetic correctly.
Example (out of my head):
(def item (hildebrand.core/get-item! .....))
;; => {:numeric-field BigNumber 200 }

(update item :numeric-field + 0.1)
;;=> {:numeric-field String "200.1"}
;; after the first arithmetic the field gets converted to a string
;; as there is no protocol on BigNumber to handle arithmetic

(update item :numeric-field + 0.1)
;;=> {:numeric-field String "200.1.1"}

Sorry, I'm a little slow on the uptake, I was travelling - I didn't know anyone other than me was using the cljs support .  Working on browser support at the moment.
Yeah, so as you mentioned, no mechanism for handling arithmetic via + using bignumber.js - you'd have to do something like:
(update item :numeric-field #(.plus % 0.1))
I don't know there's any real way of changing that, either in Javascript or Clojurescript, without changing the approach to handling numbers.  If your numbers are small, and small things only ever happen to them, I can see the ubiquitous use of bignumber being tedious - however there would also be problems if we chose when to return bignumbers, and when not.  If this is a pain for you, I could add a feature to allow either the "credentials" map (for all requests performed with that map), or the optional final map arg to the Dynamo functions (for per-operation behaviour) to contain a key which will convert all incoming numbers from Dynamo into Numbers, throwing an error if any of them are unrepresentable due to precision.

Well like they say, I am the other guy :-) Your solution would cover my usecase, credentials and potential override per invocation would be the most flexible solution I would also like to have a "force" option as I want to be able to ignore the errors in the "unrepresentable case". In this case I just want my numbers as js/Number. :+1:


K8SPG-554: add .spec.tlsOnly field

https://perconadev.atlassian.net/browse/K8SPG-554
DESCRIPTION
This PR adds a .spec.tlsOnly field that converts all HBA records with host or hostnossl origins to hostssl. This behavior matches the tlsOnly option in Percona Operator for PostgreSQL v1.
Note: the field name is not final and should be discussed
CHECKLIST
Jira

[ ] Is the Jira ticket created and referenced properly?
[ ] Does the Jira ticket have the proper statuses for documentation (Needs Doc) and QA (Needs QA)?
[ ] Does the Jira ticket link to the proper milestone (Fix Version field)?

Tests

[ ] Is an E2E test/test case added for the new feature/change?
[ ] Are unit tests added where appropriate?

Config/Logging/Testability

[ ] Are all needed new/changed options added to default YAML files?
[ ] Are all needed new/changed options added to the Helm Chart?
[ ] Did we add proper logging messages for operator actions?
[ ] Did we ensure compatibility with the previous version or cluster upgrade process?
[ ] Does the change support oldest and newest supported PG version?
[ ] Does the change support oldest and newest supported Kubernetes version?

Test name
Status

custom-extensions
passed

custom-tls
passed

demand-backup
passed

finalizers
passed

init-deploy
passed

monitoring
passed

one-pod
passed

operator-self-healing
passed

pitr
passed

scaling
passed

scheduled-backup
passed

self-healing
passed

sidecars
passed

start-from-backup
passed

tablespaces
passed

telemetry-transfer
passed

upgrade-consistency
passed

upgrade-minor
passed

users
passed

We run 19 out of 19

commit: https://github.com/percona/percona-postgresql-operator/pull/998/commits/ce5136e1dd3771351419a4ce78c39403a0f9e4f7
image: perconalab/percona-postgresql-operator:PR-998-ce5136e1d


Shadows in Hololens 2

Describe the bug
Hi all. I am a newbie at designing apps in hololens 2 so i'll give it a try
Using a point light source renders shadows between objects in Unity but shadows do not render
Expected behavior
I expected shadow casting from one object to another
Screenshots

Your setup (please complete the following information)

Unity Version 2021.3.18f1
MRTK Version Foundation 2.8.3

Target platform (please complete the following information)

HoloLens 2

Thank you

@nmitrak, thank you for your issue.  Unfortunately, this is not directly related to MRTK.   However, I will convert this to a topic in our discuss forums.  Perhaps someone there can assist.


Add a flag to check for tool updates, without installing them

Currently the rokit update command automatically updates the manifest to the latest versions of tools - it would be nice to have a flag to only check for updates, without modifying the manifest.

I'd be happy to take a look at this on the weekend, I'll probably call it something like rokit check unless you've got some other ideas?

I think rokit check on it's own may be a bit unclear what exactly it is checking. Is it checking that the manifest is okay? Is it checking for corrupted tools? etc...
How about rokit update --check?


FIX: Fix recon

@rkmaddox @maddycapp27 because trackers are identified by their memory location, it is actually possible for tracker names to repeat in the unlucky case where the same memory address is used by multiple trackers (e.g., one is instantiated after another is garbage collected). I hit such a use case recently. This fixes the reconstruction.
Should we also fix the naming, e.g. by also tagging with the time of instantiation (to the millisecond), too?

@maddycapp27 any thoughts on this one?

@larsoner Sorry for the delay! I don't see any harm in tagging with more info but you would be a better judge. Could the time tag replace the ID number or would it be another data line?

If time is only recorded to the millisecond, then we still ned the other id. It's entirely possible that two trackers could be created during the same millisecond.

I'm thinking both time and id

Turns out time.time() is good to the microsecond, but I still made it use id-time now instead of just id.
Okay with you @rkmaddox @maddycapp27 ?

Sounds good to me

@drammock do you have time to look?

I can review later this morning.
On December 11, 2017 8:46:18 AM PST, Eric Larson<EMAIL_ADDRESS>wrote:

@drammock do you have time to look?

tests pass in both py2 and py3 on my Linux, and both tracker_dealer.py and tracker_dealer_doublesided.py examples work.  Only thing that sticks out at me is that the example scripts don't end with a plt.show().  The tracker IDs in the verbose output are annoyingly long to identify by eye, but at least they won't collide now.

Thanks for the quick look @drammock

Not to beat a dead horse, but if we wanted to ensure uniqueness, but also make them more readable, we could hash the id-time tag.

+1 by me once someone is motivated to do it (i.e., not a high priority)

For whomever does it in the future:
import hashlib

hashlib.md5('foo').hexdigest()
# outputs 'acbd18db4cc2f85cedef654fccc4a4d8'
hashlib.md5('fool').hexdigest()
# outputs 'dd7b696b96434d2bf07b34f9c125d51d'

I thought of that too, but I didn't think that md5 hashes were much of an
improvement in readability... probably somewhat quicker to make the "are
these two different" judgement at a glance, but still hard to make the "is
this the same one I was looking at several seconds ago?" judgement.
On Mon, Dec 11, 2017 at 11:50 AM, rkmaddox<EMAIL_ADDRESS>wrote:

For whomever does it in the future:
import hashlib
hashlib.md5('foo').hexdigest()# outputs 'acbd18db4cc2f85cedef654fccc4a4d8'
hashlib.md5('fool').hexdigest()# outputs 'dd7b696b96434d2bf07b34f9c125d51d'
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/LABSN/expyfun/pull/339#issuecomment-350837611, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABugUwyKXuzqbMhc0PZU6i7UkCRyslWDks5s_YeTgaJpZM4QvZpE
.

Agree, though it would at least allow eyeballing that wouldn't depend on the last few characters.
If we are OK with one in 4.3 billion chance of collision: https://github.com/zacharyvoase/humanhash.


Use string ID instead of integer

Details
Currently, almost all of the IDs are integers like this:
1 shopkeeper
2 Zeome
3 putit
4 red putit
...

Use string ID for them.
To keep high performance, it is better to use symbol like Ruby if possible.

In PR #1369 you can see that some code is still using chara_find(203) with hardcoded magic numbers. This is moyer's legacy ID. His string ID is moyer_the_crooked. Is there a function that works with strings instead? Ideally it would be as simple as chara_find("moyer_the_crooked").

@Vorlent

Is there a function that works with strings instead?

Currently, there is no handy function doing so, but you can write a wrapper function like this.
int chara_find(const std::string& chara_id)
{
    // Note: if `chara_id` not found, `ensure()` throws an exception.
    return chara_find(the_character_db.ensure(chara_id).legacy_id);
}


Generate sitemap

Hi!
Can you implement sitemap to this project? I'm not familiar with Deno so I don't know how.
Thank you!

Do you mean a human-readable sitemap

site map - Wiktionary, the free dictionary

or a machine-readable sitemap.xml?

Sitemaps - Wikipedia

Or maybe both?

Open for contributions.


[ConstraintSystem] Strengthen dependent member type checks while infe…

…rring bindings
Look through specifier (inout, l-value) and optional types while
checking for presence of dependent member types to avoid inferring
incorrect bindings (which could lead to infinite recursion in the
solver).
Resolves: SR-13856
Resolves: rdar://problem/71383770

@swift-ci please test source compatibility

@swift-ci please smoke test

@swift-ci please smoke test macOS platform


[BugFix] fix sink operator dcheck fail occasionally

Why I'm doing:
What I'm doing:
Fixes https://github.com/StarRocks/StarRocksTest/issues/5465
The problem is quite strange for the d-checked variable (in the core) satisfies the condition.
One suspicious point is that the variable is an atomic. So I made a change and wish to give it a try.
What type of PR is this:

[x] BugFix
[ ] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Does this PR entail a change in behavior?

[ ] Yes, this PR will result in a change in behavior.
[x] No, this PR will not result in a change in behavior.

If yes, please specify the type of change:

[ ] Interface/UI changes: syntax, type conversion, expression evaluation, display information
[ ] Parameter changes: default values, similar parameters but with different default values
[ ] Policy changes: use new policy to replace old one, functionality automatically enabled
[ ] Feature removed
[ ] Miscellaneous: upgrade & downgrade compatibility, etc.

Checklist:

[x] I have added test cases for my bug fix or my new feature
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

[ ] This is a backport pr

Bugfix cherry-pick branch check:

[x] I have checked the version labels which the pr will be auto-backported to the target branch

[ ] 3.2
[ ] 3.1
[ ] 3.0
[ ] 2.5

@Mergifyio backport branch-3.2

https://github.com/Mergifyio backport branch-3.1

https://github.com/Mergifyio backport branch-3.0


Latest version of Eventviewer is not capturing all from MS Mouse

I have the Microsoft Sculpt Comfort Bluetooth Mouse (H3S-00001) and in the previous version of Karabiner/Eventviewer all the mouse actions were captures, such as swipe up/down on the Windows symbol, and the clicks on the scroll wheel. In Karabiner Elements/Eventviewer it's just the pressing of the blue Windows button that is captured. I have also disabled "Hide ignorable events."
I see some others have posted JSON with the scroll wheel events so maybe I can recover some of the functionality.
Thanks!

I have the same mouse as yours, I tried to remove the mouse from bluetooth and add it back and the side button now works.


⚠ BeyondHD has degraded performance

In 250b5b6, BeyondHD ($BHD) experienced degraded performance:

HTTP code: 200
Response time: 10620 ms

Resolved: BeyondHD performance has improved in c92bf8b after 8 minutes.


Maven: support PMD copy-paste detector

Question, Bug, or Feature?
Type: Feature
Enter Task Name: Maven/Gradle
Environment

Server - Azure Pipelines
Agent - Hosted

Issue Description
Running PMD is already supported for Maven/Gradle.
Please also support CPD: https://pmd.github.io/latest/pmd_userdocs_cpd.html

Hi @Trass3r, thanks for reporting! We are working on more prioritized issues at the moment but will get back to this one soon.

It's still a valid feature request.

Still a valid feature request.


Use single hash tables for compoment insertion/removal edges to improve spatial locality.

Out of curiosity, any changes to the benchmarks? We're probably not exercising cases where this would matter (very high cache pressure, perhaps), but maybe it registers anyway.

I added a benchmark stressing insertion and removal but sticking around a steady state of attached components. This yields
test insert_remove                  ... bench:         356 ns/iter (+/- 225)

before and
test insert_remove                  ... bench:         330 ns/iter (+/- 123)

after these changes. I'd say close to noise with a positive tendency?

Thanks for investigating!


Can not filter branch of Webhook

I used GitLab WebHook as trigger of my project. I found it build any branch I pushed. Do we have a solution to filter branch? Like just start build when I pushed on "develop" branch ?

Did you find resolve this issue? Looking at code it seems there's no filtering.


No mentions about licenses prices (free or payed)

On another page about External Users I read:

"The licensing requirements differ. To learn more about licensing, see Azure AD External Identities licensing and the SharePoint Online external sharing overview. "

On this page, there are no mentions of licenses payments:

Paid B2B?
1:5 ratio (obsolete)?
Login count (MAU)?

I want to share a SharePoint some lists and files with external users for editing but I cannot CLEARLY understand if I must pay somethings for these external users' collaborations.

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

ID: 6cadc974-5434-4647-6480-996c1e40a2f8
Version Independent ID: 7953c446-e5ac-6a46-4ab4-7acd4b75f3b0
Content: External sharing overview - SharePoint in Microsoft 365
Content Source: SharePoint/SharePointOnline/external-sharing-overview.md
Service: sharepoint-online
GitHub Login: @MikePlumleyMSFT
Microsoft Alias: mikeplum

@m4ss1m0g Thank you for submitting feedback and contributing to the docs. We are currently investigating this.

Hello @scanum
do you have any news regarding @m4ss1m0g 's feedback?
For me it is not clear what is the difference (regarding licensing) between using AAD B2B invitation manager and SharePoint invitation manager? I know it from a technical perspective as it is well described at https://docs.microsoft.com/en-us/sharepoint/sharepoint-azureb2b-integration, but it is hard to understand on licensing perspective.


[ENH] Add sparse index

Description of changes
Summarize the changes made by this PR.

Improvements & Bug fixes

...

New functionality

...

Test plan
How are these changes tested?

[ ] Tests pass locally with pytest for python, yarn test for js, cargo test for rust

Documentation Changes
Are all docstrings for user-facing APIs updated if required? Do we need to make documentation changes in the docs repository?

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#1825  👈
#1823
#1822
#1821
#1820
#1819
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @HammadB and the rest of your teammates on  Graphite


Add expose ports through services for VMs, extra ports and services d…

Add capability of exposing ports for Kubevirt VMs
Description
Added methods to create services through which ports can be exposed for VMs, each VM will have one service for each type through which multiple ports can be exposed.
Status

done, ready for review

Checklist (tick everything that applies)

[x] Code linting (required, can be done after the PR checks)
[ ] Documentation
[x] Tests
[x] ICLA (required for bigger changes)

Thanks for the contribution.
I added some comments in-line.

Thanks for pointing out those issues. They have been fixed!

Thanks for addressing the feedback - I merged changes into trunk.


Specify browser compatibility in the docs

Be sure that it works fine at modern browsers.
So the main problem is IE. You shouldn’t need to support anything below IE 8 nowadays. But IE 6 to 8 also work fine.

I meant you should document that exactly, a version for each specified browser, not to answer in this issue.


Later Versions

Can the page be updated to state if this is the minimum version that needs to be installed. Or if it is THE supported version and moving to a later version will leave you unsupported as per the note "Microsoft reserves the right to reject support cases where a non-certified device is connected to the Phone System via Direct Routing."

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

ID: 1b1a1c01-10a9-7f69-1c7b-1a566b0f5112
Version Independent ID: 329ee759-40fc-98fb-b91e-87786c63024b
Content: List of Session Border Controllers certified for Direct Routing
Content Source: Teams/direct-routing-border-controllers.md
Service: msteams
GitHub Login: @CarolynRowe
Microsoft Alias: crowe

@TobieFysh Thank you for submitting feedback and contributing to the docs. We are currently investigating this.

Hello @TobieFysh
I don't think this item would be updated anytime soon but we could ask the author...
@scanum Thank you

@TobieFysh Thanks for submitting feedback. We will go ahead and update this in the future.


fix(cli): highlight insert into ... values

Resolves: #1478

Nice, I appreciate the pointer @universalmind303


Disable on mobile?

Hi,
Just wondering if there is some easy way of disabling on mobile?
Or is the way to do a user agent check?
Example:
if (!isMobile) Scrollbar.init()

http://www.letmegooglethat.com/?q=how+to+detect+user+agent


Feature request385

Implemented a new menu on PVRRecordings view:
Group Sorting:
( ) By name
-> Sorts the recordings by title (asc) and starttime (desc). Newest per title are thus visible on top
( ) By date added
-> Sorts the recordings by starttime (desc) only, newest at the top.
( ) Unsorted
-> Shows the recordings in same order as before as delivered by Kodi.
Additional Filter Checkbox
[ ] Hide watched
-> This is a heuristic that tries to hide recordings already watched:
If recording.playcount is greater than 0, hide.
If resume stamp is greater than tv show duration, hide.
Tv show duration is typically smaller than total recording length as there are some minutes before and after the tv show recorded.

This implements #385

To be honest, i don't have my PVR setup working, so i can't test this, but the code looks good, so i'm going to trust you and merge this.
Could you just check the (small) comments i made, and improve them.

Merging this, though there's still a redundant check...


cop: remove Copy trait of RpnFnCallPayload

Signed-off-by: H-ZeX<EMAIL_ADDRESS>What have you changed? (mandatory)

remove Copy trait of RpnFnCallPayload

What are the type of the changes? (mandatory)

Engineering (engineering change which doesn't change any feature or fix any issue)

How has this PR been tested? (mandatory)
No.
Does this PR affect documentation (docs) or release note? (mandatory)
No.
Does this PR affect tidb-ansible update? (mandatory)
No.
Refer to a related PR or issue link (optional)
https://github.com/tikv/tikv/pull/4973#issuecomment-506310744

/test

Why is it removed?

@breeswish Because the RpnFnCallPayload is very large(24B, include the padding), pass it by reference is much better.

I think this doesn't mean that Copy should not be implemented. In fact Copy is still preferred over Clone if you really want a copy of the data.

Maybe a complete removal of RpnFnCallPayload can be considered. RpnFnCallPayload is introduced only to group parameters necessary when evaluating a function. Previously we needed to write out all these necessary parameters when introducing a UDF, no matter whether they are really needed or not, which is pretty verbose. Now we have #[rpn_fn] and the signature can be pretty clean in most of the cases when these raw parameters in RpnFnCallPayload are not needed.

@sticnarf Please also take a look

Yes, I agree we can consider removing the payload.
The only reason why I suggest removing Copy is to make clippy happy...

Closing for https://github.com/tikv/tikv/pull/5007 since it will remove Copy (and Clone).


Support spring data redis using  jedis as connectors

Spring-boot-starter-data-redis use lettuce as connectors by default, If I switch it to jedis：
<dependency>
      <groupId>org.springframework.boot</groupId>
      <artifactId>spring-boot-starter-data-redis</artifactId>
      <exclusions>
        <exclusion>
          <groupId>io.lettuce</groupId>
          <artifactId>lettuce-core</artifactId>
        </exclusion>
      </exclusions>
    </dependency>
    <dependency>
      <groupId>redis.clients</groupId>
      <artifactId>jedis</artifactId>
    </dependency>

redis operations cannot be traced.

We're going with Lettuce for now. If there is any additional interest in Jedis we'd need more interest from the community. As always, PRs are more than welcome!


Update Notebooks

Primary Changes:

Placed all functionality for both classes in 1 notebook.
Notebook order is as follows: Parse EVL/EVR -> Look at DataFrame / manipulate it -> Load and plot Sv -> Show masking functionality -> Show saving functionality.
All data now is being grabbed from the feature branch. I'll change it to the main branch right before I merge this branch.
I kept in the old notebooks just as a reference. I will remove them before merging.

Secondary Changes:

Made use of select_region earlier in r2d.mask to simplify both region_id selection and mask_labels creation (when passed in mask_labels is None)
Modified EVR trawl region to make it smaller
Fix region plot bug where dataframe was being passed into close_region even though close_region doesn't take in dataframes
Added kwaargs to to_csv

Index value should be something the user wishes to pass in

@valentina-s Thanks for the review! I added Echoregions to ReviewNB incorporated most of your changes with just a few comments of my own:
Getting Started

manual annotations -> turns out some annotations from Echoview are automatic annotations. Maybe we can say directly "Echoview" annotations (since this is the focus now)
Mention Echoview earlier

In the previous revision of the description, Wu-Jung made changes that were less Echoview-focused so that this package was not locked into Echoview. I think I agree with these changes too since the annotations don't necessarily need to come from EVL EVR, for instance we can create annotations with our model (in this case it would not be manual though 😆).
Regions2d and Lines

bottom_contours: I usually think of contour as something closed (although I know it is is not necessary). I wonder if bottom boundary is better or maybe there is another term?

bottom_contours (often contour is through to be closed)

Perhaps in this PR, we can sort out this naming issue. I think bottom boundary does not reflect what the data we actually get is: points. Perhaps bottom boundary points? Or bottom boundary vertices? I am also thinking of changing the region contour name with something like region boundary points/vertices.

@valentina-s Thanks for the review! I added Echoregions to ReviewNB incorporated most of your changes with just a few comments of my own:
Getting Started

manual annotations -> turns out some annotations from Echoview are automatic annotations. Maybe we can say directly "Echoview" annotations (since this is the focus now)
Mention Echoview earlier

In the previous revision of the description, Wu-Jung made changes that were less Echoview-focused so that this package was not locked into Echoview. I think I agree with these changes too since the annotations don't necessarily need to come from EVL EVR, for instance we can create annotations with our model (in this case it would not be manual though 😆).
Regions2d and Lines

bottom_contours: I usually think of contour as something closed (although I know it is is not necessary). I wonder if bottom boundary is better or maybe there is another term?

bottom_contours (often contour is through to be closed)

Perhaps in this PR, we can sort out this naming issue. I think bottom boundary does not reflect what the data we actually get is: points. Perhaps bottom boundary points? Or bottom boundary vertices? I am also thinking of changing the region contour name with something like region boundary points/vertices.

Thanks for adding NBViewer @ctuguinay, it was very nice to use it! I went through the notebooks again and marked some small things which you can see in the viewer.

regarding the annotations: maybe we just call them annotations (no need to say manual, often that is assumed anyway). And the first time you introduce Echoview software, provide a link to it

for the bottom: how about just bottom points: Echoview uses this terminology here and there discussing how a point is part of the bottom?

Could you just change Echoregion's to Echoregions' since the "s" is part of the package name (in both notebooks). Otherwise it all looks good and the notebooks run fine.


Bootz FP

Added the bootz FP testcases
Have attached the logs for one sample case
Uploading Valid_configuration.txt…

Pull Request Functional Test Report for #2324 / 8a445c593d6cd47d664bcea7d12d1d71738e1337
No tests identified for validation.
Help


Porting webpacker config files to plain webpack

Hi @nikushi! Funnily enough I started on a gem just like yours at https://github.com/dbalatero/webpack-lite a while back. I saw your article https://medium.com/@nikushi/replacing-rails-webpacker-with-vanilla-webpack-54b7ea90d913 and got curious about your project. I like where it's at so far!
I'm curious about a few things before diving into trying this gem:

How do you propose porting a Webpacker config from a production app to plain Webpack? My thought was to actually dump out the final Webpacker config to a file and then make surgical edits to it to bring it back to parity with what Webpacker provides behind the scenes.

Is there any interest in replicating the compile-on-first-request functionality that Webpacker has? Or does that become irrelevant as long as you:

run webpack-dev-server in development mode
run webpack-dev-server in test mode
pre-compile your JS assets in CI before running tests

Hi @dbalatero,  thank you for the feedback!
Extracting a config from Webpacker: The gem does not have such function. It would  better for Webpacker to have it. FYI someone tried eject function https://github.com/rails/webpacker/pull/1916. This pr may help you.
The next, about Hot Module Replacement: I proposed in my blog post using rack-proxy to enable HMR. I think that installing such rack middleware can be supported in my gem. So, how about starting/stopping webpack-dev-server along with rails? My thought is that webpack-dev-server should start manually separated from rails s. Foreman may be a choice If you want to start both together.
The last is about the test mode. I run npm run build sometimes manually in my project, when it is needed before testing request specs, though that takes labor. CI is the same. This workflow can be improved, but no ideas.

I am going to think about hot reload in development mode and pre-compile in test mode.

FYI: I'm trying pre-compile in test by using rspec's before :suite hook at #11.

@nikushi cool! I'd be down to work on a PR to add rack-proxy to enable webpack-dev-server/HMR passthrough. I agree that webpack-dev-server should be started alongside Rails manually.

The webpack plugin for browsersync is pretty handy in this regard. You get a reload whenever a js, rb or erb file is changed. Not for everyone, but this flow is nice to work with once you have it set up.
Basicly, webpack-dev-server is proxys requests based on url. Requests to /javascripts/webpack/ (my output folder) get proxyed to https://localhost:8081/* (live wpd server output). All other rails requests get dutifully passed to the 3000 port like normal.
const jsonPlaceholderProxy = proxy('/javascripts/webpack/', { //change to match your output folder
    target: 'https://localhost:8081',
    changeOrigin: true, // for vhosted sites, changes host header to match to target's host
    logLevel: 'debug',
    secure: false,
    ssl: {
      key: fs.readFileSync('./ssllocal/server.key', 'utf8'),
      cert: fs.readFileSync('./ssllocal/server.crt', 'utf8')
    },
    pathRewrite: {
      '^/javascripts/webpack/(.*)': '/$1' // rewrite path
    }
  });
}
// later in plugins (there are better ways to detect WDS, but we just set explicit flags)
process.env.WPD_HMR !== 'false' ? new BrowserSyncPlugin(
    // BrowserSync options
    {
      host: `${subdomain}.domain.me`, // or localhost
      port: 3002,
      watchOptions: {
        ignoreInitial: true,
        ignored: '*.txt'
      },
      files: ['./app/**/{*.erb,*.rb}'],
      open: false,
      proxy: {
        target: `https://${subdomain}.domain.me:3000/`, // or localhost
        middleware: [jsonPlaceholderProxy]
      },
      https: { key: './ssllocal/server.key', cert: './ssllocal/server.crt' }
    },
    { reload: false } // prevent BrowserSync from reloading the page and let Webpack Dev Server take care of this
  ) : noop()

This is a departure from other solutions, but at least it works with HMR and style injection. Plus its nice to have erb files reload the browser when saved. Hope this is useful.

pre-compile in test mode.

Now you are able to use this function with v0.3.0 🚀.