tensorpack/examples/DeepQNetwork/DQNModel.py

        with tf.variable_scope('target'), \
                collection.freeze_collection([tf.GraphKeys.TRAINABLE_VARIABLES]):
            targetQ_predict_value = self._get_DQN_prediction(next_state)    # NxA

Why is collection.freeze_collection here? I consulted the freeze_collection, and it is said as follows:
    """
    Args:
        keys(list): list of collection keys to freeze.
    Returns:
        a context where the collections are in the end restored to its initial state.
    """

I don't quite understand why it is done here. I am very grateful if you would tell the reasons. Thanks!

Variables in this scope don't need to be trained. So they shouldn't be added to the TRAINABLE_VARIABLES collection.

Thanks.
So what is done here is that these variables might be trained, but within that context, these variables would finally be restored to its original value. So anyway, implicitly, they are not trained, right?
I still have a doubt. In the code, there are the following:
best_v = tf.reduce_max(targetQ_predict_value, 1)    # N,
or
best_v = tf.reduce_sum(targetQ_predict_value * predict_onehot, 1), depending on whether the DQN variant is double or not. But I feel the point locates at the following code:
target = reward + (1.0 - tf.cast(isOver, tf.float32)) * self.gamma * tf.stop_gradient(best_v)
We can notice the tf.stop_gradient operation. So is it still necessary to have the collection.freeze_collection when creating the target network? I mean the variables within the target network are all stop_gradient to some extent.
So my new question is whether collection.freeze_collection is still necessary.
I am not questioning the correctness of the code. I know it is correct. I just want to make sure I understand the underlying logic. Thanks a lot!

No. These variables will not be trained. In tensorflow, only variables added to this collection tf.GraphKeys.TRAINABLE_VARIABLES will be trained by default.
Your observation is correct. You can remove freeze_collection and it will work the same. But tensorpack will print warnings when some "TRAINABLE_VARIABLES" have no gradient, which is usually a bug. freeze_collection avoid this warning message.

Good! Thanks a lot. Your words are very clear.
Thanks!

To simplify the code I removed the freeze_collection part in 9850edf5342a27f. It also fixed an import bug which caused crashes in evaluation.

You are so helpful. Thanks a lot!


Do extended test of base games running using release_version=0

Once we deploy the Artifacts expansion to production, all new tables will be using release_version=1, however, all ongoing games will continue to run using release_version=0. Since they use wildly different code paths, it's important that some of our alpha games spend time using release_version=0 (of course, games using the expansion would still need to run using release_version=1).
Let's try this in innovationdev for a while before pushing this to the alpha.

This spent some time in alpha and no problems were reported! I'm feeling better about the stability of the game for when this is pushed to public release.


Log Implemented

Natural log function implemented in the language by adding the relevant case to the evaluatePrimOp function. Seems to work as intended

Can you rebase so this pull request only has the log commits?
Thanks for all your contributions :)
On Nov 23, 2017 22:32, "maymoo99"<EMAIL_ADDRESS>wrote:

Natural log function implemented in the language by adding the relevant
case to the evaluatePrimOp function. Seems to work as intended
You can view, comment on, or merge this pull request online at:
https://github.com/hakaru-dev/hakaru/pull/128
Commit Summary

chi distributions added to stdlib
Merge pull request #1 from hakaru-CS4ZP6/mahmoud
adding stuff
adding python script for plotting histograms
Merge pull request #2 from hakaru-CS4ZP6/justin
pascal and other related distributions
histogram script fix
Merge branch 'master' into mahmoud
Merge pull request #3 from hakaru-CS4ZP6/mahmoud
Merge pull request #4 from hakaru-dev/master
Some new distributions
Merge branch 'master' into mahmoud
Merge pull request #5 from hakaru-dev/master
Working on uniform transforms
Merge branch 'master' into mahmoud
Adding factorial/choose
Adding bunch of discrete distributions. Some rearranging.
Fixes for PR
Commenting in stdlib and some rearranging
Commenting Uniform Transforms
Revert so I can do incremental PLs
More revert
Added evaluatePrimOp for log function

File Changes

M haskell/Language/Hakaru/Sample.hs
https://github.com/hakaru-dev/hakaru/pull/128/files#diff-0 (6)

Patch Links:

https://github.com/hakaru-dev/hakaru/pull/128.patch
https://github.com/hakaru-dev/hakaru/pull/128.diff

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/hakaru-dev/hakaru/pull/128, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAAhUZ9FQSUNATR5s4M9W3QRr1en7L2oks5s5eR2gaJpZM4QpMXl
.


add quick action for view Video One frame one step one click

add quick action for view Video One frame one step one click
I have 4k30hz 1sec 30frame. I want to select 1 frame output to photo …thanks

Not a quick action, but now part of the customizable playback control buttons (like play, skip 10s, etc.)


[JsonPatch]: Move with "-" index is not working

Nuget Package
JsonPatch.Net
Package Version
3.2.0
Operating System
Windows
.Net Target (if relevant, please specify the version in the description)
.Net (5 or after)
A clear and concise description of the bug.
When using the "-" operator in a json pointer for an array object, I want to have the "from" value added at the end of the "path" array.
This is also described here: https://datatracker.ietf.org/doc/html/rfc6902#appendix-A.16
What did you expect?
Value added to the end of the array.
Please add test code or steps to reproduce the behavior.
    [TestMethod]
    public void MoveShouldMoveItemAround()
    {
        // Arrange
        var model = new { Items = new[] { 1, 2, 3, 4, 5 } };
        var jsonModel = JsonSerializer.SerializeToNode(model);

        var doc = new global::Json.Patch.JsonPatch(
            PatchOperation.Move(
                JsonPointer.Parse("/Items/2"),
                JsonPointer.Parse("/Items/-")));

        // Act
        var result = doc.Apply(jsonModel);

        // Assert
        result.Error.Should().BeEmpty();
        result.Result!["Items"]!.AsArray()[^1].Should().BeEquivalentTo(3);
    }

returns: Expected result.Error to be empty, but found "Target path /Items/- could not be reached.".
Is there any other information you'd like to share regarding this bug?
Looking into the implementation, this line should be the cause of this bug:
		[...]
			if (lastPathSegment.Length == 0 && lastPathSegment[0] == '-')
				index = arrTarget.Count;
		[...]

https://github.com/json-everything/json-everything/blob/cf4faf3928287a5eacd611df3dec3ffea70edb3c/src/JsonPatch/MoveOperationHandler.cs#L64
This is always false, because if the length is null, it can never also be '-'.
Code of Conduct & Contributor Guidelines

[X] I agree to follow this project's Code of Conduct and Contribution Guidelines.

Edit: a similar issue should also occur in copy:
https://github.com/json-everything/json-everything/blob/cf4faf3928287a5eacd611df3dec3ffea70edb3c/src/JsonPatch/CopyOperationHandler.cs#L44

Yeah, looks like I have flipped logic in my equality.  Happens occasionally. 😆

Thanks for the quick fixes.
I had just mentioned it in an edit, but the same issue also occurs for the copy operation:
https://github.com/json-everything/json-everything/blob/31afd0e387a187816c26c67eb7cb77227ef9d1df/src/JsonPatch/CopyOperationHandler.cs#L44

Dangit. Missed that.  If you want to set up a PR, I'd be happy to merge it.  Please bump the patch version and add a release note.


Error stats total instructor

In Stats -> Totals, the values of the columns "Instructor 90 days" and "Instructor Last 12 months" are reversed. "90 days" shows the "Last 12 months" value and vice versa.

looks like this issue was even in the previous versions... will investigate


Updating sg egress to use input variable

what
Egress was hardcoded to true when there was an input for it.
why
Full egress is not always warranted.
references

Link to any supporting github issues or helpful documentation to add some context (e.g. stackoverflow).
Use closes #123, if this PR closes a GitHub issue #123

@Nuru @nitrocode Can I get eyes on this?

@nitrocode @Nuru Can I get eyes on this?

@aknysh
Done, Sorry thought I had done that already!


memory leak if binding failed

Describe the bug (描述bug)
memory leak when binding failed
To Reproduce (复现方法)
bind a used port
Expected behavior (期望行为)
Versions (各种版本)
OS: Ubuntu 18.04
Compiler: clang + ASan
brpc: 0.9.0
protobuf: 3.0.0
Additional context/screenshots (更多上下文/截图)
E0428 03:47:30.283096 117030 /home/zyh/incubator-brpc/src/brpc/server.cpp:956] Fail to listen <IP_ADDRESS>:8301
E0428 03:47:30.283232 117030 /home/zyh/braft/example/atomic/server.cpp:508] Fail to start Server

=================================================================
==117030==ERROR: LeakSanitizer: detected memory leaks

Direct leak of 56 byte(s) in 1 object(s) allocated from:
    #0 0x6d14ad in operator new(unsigned long) (/home/zyh/braft/example/atomic/atomic_server+0x6d14ad)
    #1 0xca1961 in brpc::Server::StartInternal(in_addr const&, brpc::PortRange const&, brpc::ServerOptions const*) /home/zyh/incubator-brpc/src/brpc/server.cpp:783
    #2 0xca330e in brpc::Server::Start(butil::EndPoint const&, brpc::ServerOptions const*) /home/zyh/incubator-brpc/src/brpc/server.cpp:1063
    #3 0xca34b2 in brpc::Server::Start(int, brpc::ServerOptions const*) /home/zyh/incubator-brpc/src/brpc/server.cpp:1081
    #4 0x6d5038 in main /home/zyh/braft/example/atomic/server.cpp:507:16
    #5 0x7fa2b2449bf6 in __libc_start_main /build/glibc-S9d2JN/glibc-2.27/csu/../csu/libc-start.c:310

SUMMARY: AddressSanitizer: 56 byte(s) leaked in 1 allocation(s).

server.ReleaseKeytableForUT();

这个方法是哪来的，源码里没找到呀

server.ReleaseKeytableForUT();

这个方法是哪来的，源码里没找到呀

我修改了下描述。“我的版本是内部改过一些的版本，如果没有这个方法的可以自己加下。”

Thanks @Lhfcws
So according to your comments, the leak is true, but we can ignore it, right?
Is there a better elegant way to solve this, like smart pointer or something else?

Thanks @Lhfcws So according to your comments, the leak is true, but we can ignore it, right?
Is there a better elegant way to solve this, like smart pointer or something else?

Obviously smart pointer can solve this issue functionally, but I have no idea how it may affect the performance.  Maybe some other maintainers or core developers could tell us the tradeoff when they decide to leave this mem-leak issue ignored.

Based on this issue, may I request a CVE for this issue? It'll help end-users know that they should update and benefit our research.


Could not find a declaration file for module 'react-native-haptic-feedback'

version change
  ◉ react-native-haptic-feedback                 2.2.0  →    2.3.1

error
src/packages/Haptic/Haptic.ts:1:42 - error TS7016: Could not find a declaration file for module 'react-native-haptic-feedback'. '/Users/e/Documents/a/b/node_modules/react-native-haptic-feedback/lib/commonjs/index.js' implicitly has an 'any' type.
  There are types at '/Users/e/Documents/a/b/node_modules/react-native-haptic-feedback/lib/typescript/module/src/index.d.ts', but this result could not be resolved under your current 'moduleResolution' setting. Consider updating to 'node16', 'nodenext', or 'bundler'.

1 import { trigger as hapticTrigger } from 'react-native-haptic-feedback';
                                           ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

Found 1 error in src/packages/Haptic/Haptic.ts:1

code
import { trigger as hapticTrigger } from 'react-native-haptic-feedback';

export const Haptic = {
  trigger: () => {
    hapticTrigger('impactLight', {
      enableVibrateFallback: true,
      ignoreAndroidSystemSettings: false,
    });
  },
};

I'm encountering the same thing when I upgraded to 2.3.1

Set your ts-config.ts compilerOptions moduleResolution and module to "NodeNext"
Guess the reason is 2.3.1 package.json claims "exports" property

Also getting this :/

Released as v2.3.2 . Sorry, didn´t have time before


Use implicit IV for encryption

Problem
The encryption IV can be derived from other fields in message header. It'll save 8 bytes per message.
Summary of Changes
Use Message ID (monotonically increasing number per peer connection), and Source Node ID to derive the IV.

@emargolis, @dougsteed, @roydsouza

Marking as changes requested: usage of OR and an unfortunate node ID weakens message id usage (although only for very large numbers).
Specific examples (made up large to force errors):
NodeId 0xFF
makes messageId 0x0000000100000000
be identical to messageid 0x0000000200000000 (or 0 for that matter).
We should not wrap 32 bit that frequently, but this should be XOR or not used at all. Even with XOR if keys are reused I worry that one node id inverts the id of another ... so why bother. Do we have anyone to consult about cryptography (not my domain)

Won't it make the IV 0x000000FF00000000?

It is of course extremely important that the same IV not ever be repeated for a different message with the same key. Provided this can be guaranteed through the combination of message id/node id this will be fine. Need to be careful if there is any scenario where the message id space could somehow be reset with the same key being used.

Apparently my main misunderstanding was that messageid is 32-bit. This was not obvious from uint64_t iv = GetMessageId() which seemed like message id would be 64 bit,starting my whole dillema.
Discussed with @pan-apple to try to make it clearer (assert/comment/xor/anything that makes me read the code and claim it is obviously correct).

It is of course extremely important that the same IV not ever be repeated for a different message with the same key. Provided this can be guaranteed through the combination of message id/node id this will be fine. Need to be careful if there is any scenario where the message id space could somehow be reset with the same key being used.

@dougsteed , I agree. @andy31415 and I talked offline, and main confusion was due to size of message ID. Currently it is 4 bytes, but uint64_t IV = header.GetMessageId(); made it look like it is 8 bytes long. I'll add a comment and update the code to make sure we are using 4 bytes of message ID here (by & it with 0xffffffff)

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Get rid of request module

request has been fully deprecated since 11th Feb and it was in maintenance mode since March 2019.
I think it's time FCA switches to another http library, like node-fetch, which is more lightweight and can do all the same things request does.

yeah i've noticed that. will try to change that module.


Template are totally out of date

Hi,
Everyone seems your template are out of date .. not sure job "logsearch still exist".
Does anyone have a uptodate template ?
Thanks

@shinji62, sorry - we should really remove the templates/ folder, since we don't use it.
Take a look at https://github.com/logsearch/logsearch-boshrelease/tree/develop/examples


Use normalized path for static files.

Currently, filePublicPath is calculated off of the unnormalized version of the path, so "public/" doesn't match anything on Windows systems.  This leads to reply.sendFile trying to read from a file that likely does not exist, since it tries to use path.join(process.cwd(), "public") as the root.

thanks!


Experiments API Guidelines

Before myself and George and implement Experiments API we need to get some concrete implementation details, my main area of uncertainty is how the Experiments API will integrate with the rest of p2plab.
So far I have the following notes on what the Experiments API needs to do:

A higher level API allowing configuration of a variety of factors to be used in scenarios such as :

Disk Size / Disk Type
Memory Size / Memory Type
CPU Count / CPU Type

Ability to introduce various types of effects into the benchmarks such as:

Network throttling
Response delays (ie, before responding wait X miliseconds)

While looking through the codebase I noticed experiment.go so this leads me to think that the Experiments API doesn't need to be written from scratch, and instead modified to allow the aforementioned functionality in an automation-capable way. If this is true, how this integrates with the rest of p2plab is a bit unclear

From my drafts, this was roughly how I imagined it to work. Since clusters and scenarios can be described declaratively in a JSON, I was thinking of leverage golang templates in order to interpolate values:
{
	"inputs": [
		{
			"myType": "t3.micro"
		},
		{
			"myType": "t3.medium"
		}
	],
        "cluster": {
                "groups": [
                        {
                                "size": 1,
                                "instanceType": "{{myType}}",
                                "region": "us-west-2"
                        },
                        {
                                "size": 2,
                                "instanceType": "{{myType}}",
                                "region": "us-west-2",
                                "labels": ["neighbors"]
                        }
                ]
        },
	"scenario": {
        	"objects": {
        		"golang": {
        			"type": "oci",
        			"source": "docker.io/library/golang:latest"
        		}
        	},
        	"seed": {
        		"neighbors": "golang"
        	},
        	"benchmark": {
        		"(not 'neighbors')": "golang"
        	}
	}
}

Under the hood, the experiment backend is just calling executing the templates, provisioning a cluster for each input and running the benchmarks in parallel. Then we should have some way of aggregating the results into some analyzable form like CSV perhaps.

Thanks this helps to clear up some confusion. I have one question however to make sure I understand yoru comment correctly

provisioning a cluster for each input and running the benchmarks in parallel

According to this, based on the example JSON you gave we would create 2 clusters, 1 cluster of 1 node of type t3.Micro, and 1 cluster of 2 nodes of type t3.Medium. These clusters would be created simultaneously, and the "(not 'neighbors')": "golang" benchmark would be ran against both clusters are the same time.

Then we should have some way of aggregating the results into some analyzable form like CSV perhaps.

Makes sense, I guess we can just output the current benchmark format into a CSV file.

According to this, based on the example JSON you gave we would create 2 clusters, 1 cluster of 1 node of type t3.Micro, and 1 cluster of 2 nodes of type t3.Medium.

The cluster definition can be made from heterogeneous groups. They are still part of the same cluster under test.
According to this example JSON, we would create 2 clusters, 1 cluster of 3 nodes of type t3.micro (with 2 nodes labeled neighbors), and 1 cluster of 3 nodes of type t3.medium (with 2 nodes labeled neighbors).

Ah that clears things up, thanks! :rocket:

This is also just my rough brainstorming how this can work, feel free to propose an alternative model. At a high level, we want to range over some changing values, and run benchmarks over that. The values that can change can range from AWS properties (instance type, ebs volume type), IPFS tunables (datastore, libp2p tunables, chunkers), and scenario (data size, how many leechers to seeders, cross region versus not).

Will do, at the moment I can't think of any alternative model that would provide significant benefits. The only thing I can think of as an addition to the model you proposed, is an integration with something like toxiproxy so we can simulate artificial network delays.
For example the groups configuration could be adjusted to the following, enabling a 100ms delay to downstream connections in the "size 1 group".
        "cluster": {
                "groups": [
                        {
                                "size": 1,
                                "instanceType": "{{myType}}",
                                "region": "us-west-2"
                                 "toxiproxy": {
                                     "latency_down": "100ms"
                                  }
                        },
                        {
                                "size": 2,
                                "instanceType": "{{myType}}",
                                "region": "us-west-2",
                                "labels": ["neighbors"]
                        }
                ]

Cue looks really cool, I will do some research about it.

Ok I have a basic "cue parser" package that can take cue files, and generate metadata.ExperimentDefinition from them. This is very basic, and enables generating the kind of output seen here.
this is the parser golang package and this is the basic cue module. Of particular interest is the "p2plab cue definitions"  and the example cue file using these definitions
The parser package returns a wrapped cue.Instance type called P2PLabInstance that exposes some helper methods for working with the cue instance, as well as a function to return a metadata.ExperimentDefinition from a P2PLabInstance.
The cue parser package is a bit more verbose than it needs to be, however this is due to me not being familiar with the cue API. As such I can think of two main options for what to do next:
Option 1

Submit the PR to merge the cue definitions and parser package as is
Refine the parser package + cue definitions after (if need be)

Submit another PR  (if need be)

If we believe the cue integration is good enough as is, proceed with integrating the cue parser into the experiments API for basic integration (ie, replacing the json files used now with cue files)
After basic integration, enable more complex usage of cue, more advanced terraform builds, enable customizing ipfs/libp2p tunables, etc...

The advantage of this option is a more iterative approach, allowing for smaller individual changes, but more overhead.
Option 2

Get feedback on the cue files, and see what needs to change what needs to be refactored
Enable basic cue integration in the experiments API (ie, replacing the json files used now with cue files)
After basic integration open a PR to merge the functionality in
Fine tune cue integration + enable more complex usage of cue, more advanced terraform builds, enabling customizing ipfs/libp2p tunables, etc...

The only main advantage I can think of with this option is less overhead

Hey @bonedaddy, great work, I looked over the links you gave me and it looks pretty good. As an added bonus, this provides us schema validation over the input users send up too.
A few things:

Experiment should have a list of [](cluster, scenario), I'm not sure what to call this tuple. A Trial? https://github.com/RTradeLtd/p2plab/blob/cue/cue/cue.mod/p2plab.cue#L30-L33
I'm not too familiar with cue myself, but why do we need to deal with cue.Value? Can't we just compile it down to JSON and deal with JSON from there?
Can you show an example of a cue definition that has multiple trials? The current example is still only one trial, so we can't make a comparison with the Golang template example.

Let's go with Option 1, I can comment more in the PR once you have it up.

Sure let me implement the trial stuff and I'll open a PR.

Experiment should have a list of [](cluster, scenario), I'm not sure what to call this tuple. A Trial? https://github.com/RTradeLtd/p2plab/blob/cue/cue/cue.mod/p2plab.cue#L30-L33

Trial works, but it might be more fitting to call it Trials for experiments which might have more than 1 trial. But either works.

Can you show an example of a cue definition that has multiple trials? The current example is still only one trial, so we can't make a comparison with the Golang template example.

Yup!

I'm not too familiar with cue myself, but why do we need to deal with cue.Value? Can't we just compile it down to JSON and deal with JSON from there?

I believe (unless I'm using the API incorrectly) that cue.Value is what allows you to access the values of fields, and marshal them to JSON. When compiling cue source you get a cue.Instance type. This type has a Lookup function which allows you to lookup the value of a field which gets returned as a cue.Value type. This type has a function MarshalJSON that lets you access the JSON representation of the data you want. Additionally cue.Value also has a Lookup function that allows you to lookup addition fields underneath the current field to get their values.
So essentially the workflow (as far as I can tell) is as follows:

Compile cue source code
Lookup the fields you want
Marshal the returned cue.Value
Unmarshal into your golang struct/type

Unfortunately the documentation about using cue from Go is a bit sparse, so I've mostly been piecing together the results from trial+error, and some other code bases I've found. While on the cue slack someone me
ntioned that when you use cue export you're using this internal library and that it might be useful to copy+paste this internal librar
y into a non-internal package to more easily access the data you want from go code, however I have to do a bit more reading on cue first before determining if that is the appropriate way to use cue.

Now that we've got cue integrated, I think the next best step would be enabling experiment definition from cue source files, submitting that in a PR, and once that's done george and I can start work on the API side of things

Now that we can persist the experiment definitions to disk, @xiegeo and I will brainstorm a plan for implementing the Experiments API and then we can evaluate the brainstorm and see what sucks and what doesn't, followed by implementing it.
@hinshun one question I have is where should the experiments API go? For example labd, and labapp top-level directories have a child directory inside of them with the api suffix. Based on this, I'm assuming the experiments top-level directory should have a child directory named something like experimentapi? That seems a bit wordy though so I'm wondering if something like expapi would be better?

@bonedaddy I prefer keeping experimentapi for consistency's sake.
Here's the high-level changes we need to make:

Implement labd/routers/experimentrouter/router.go. Creating an experiment is effectively running the experiment. I imagine the metadata.Experiment will need a new field to contain the results. I also prefer if you kept the router light, and write the logic that actually runs experiment in the experiments package.
Ensure the labd/controlapi/experiment.go is all hooked up.
Ensure the experiment.go interface for ExperimentAPI is correctly matching the implementation in labd/controlapi/experiment.go.

For running the experiment:

[Client-side] Parse and execute cue locally, and send up JSON of the experiment definition.
[Server-side] Create experiment with definition in metadata db.
[Server-side] Create goroutines and run benchmarks with [cluster, scenario] in parallel.
[Server-side] Collect results and write to experiment in metadata db.
[Server-side] Send metadata.Experiment back to client.
[Client-side] Decide how we want to present the data. Perhaps the unixy thing to do here is to present as a CSV by default, and then we can write it to a file for future processing, or pipe it into some program to chart it.

@hinshun  Ah shoot missed the notification, thanks for the response. Good to know there's already a placeholder should make this much simpler.

I also prefer if you kept the router light, and write the logic that actually runs experiment in the experiments package.

Good idea, i took a brief look at the existing router and I don't think I'll need to make many changes.

Ensure the labd/controlapi/experiment.go is all hooked up.

By this do you mean that its hooked up to labd/routers/experimentrouter/router.go?

For running the experiment:

Your steps make sense. I think presenting as a CSV file by default is a good idea.
I'll get started on this and let you know if i run into any problems

Ensure the labd/controlapi/experiment.go is all hooked up.

By this do you mean that its hooked up to labd/routers/experimentrouter/router.go?

Yup. Just the client code to match the server routes.

Cool, makes sense. Will take a stab at this today and let you know if i run into any issues

@hinshun I think this can be closed now? I'll update the first comment to include a link to the commented out code in nodes/session.go


arch: riscv: stacktrace: allow different number of frames

Set max frames to:

CONFIG_EXCEPTION_STACK_TRACE_MAX_FRAMES on exception
CONFIG_ARCH_STACKWALK_MAX_FRAMES on arch_stack_walk()

Fixes #74713

https://github.com/zephyrproject-rtos/zephyr/blob/dcf42917c550714d2457947538b9e29d083e872e/arch/riscv/core/stacktrace.c#L17-L18
An error is raised when CONFIG_EXCEPTION_STACK_TRACE_MAX_FRAMES equals to the CONFIG_ARCH_STACKWALK_MAX_FRAMES in the MAX() macro

To be honest, this sounds like more of a bug in our MAX() implementation.
#define MAX(a, b) (((a) > (b)) ? (a) : (b))
#define MIN(a, b) (((a) < (b)) ? (a) : (b))

Should be
#define MAX(a, b) (((a) >= (b)) ? (a) : (b))
#define MIN(a, b) (((a) <= (b)) ? (a) : (b))

in <zephyr/sys/util.h>

To be honest, this sounds like more of a bug in our MAX() implementation.
#define MAX(a, b) (((a) > (b)) ? (a) : (b))
#define MIN(a, b) (((a) < (b)) ? (a) : (b))

Should be
#define MAX(a, b) (((a) >= (b)) ? (a) : (b))
#define MIN(a, b) (((a) <= (b)) ? (a) : (b))

in <zephyr/sys/util.h>

Wouldn't this make the error to be something like 8 >= 8 is always true...?

Closed in favor of #77067


To this day I still have not received any confirmation on the wallet.

When I installed 3 files:
hexxcoin-cli
hexxcoind
The avast software reports a file containing malicious information.
When booting hexxcoin-qt it displays the bottom corner:
no block source available ........ 1 year (s) and 27 week (s) behind.
I have a transfer order hxx check in block:
Details for Address
Address HELptHjeSeaNDBuztUq6MPxJCxEZajARARat
Balance 36.57242089 HXX
Rich List Rank 1218 với khối 327531 với 36.57242089 HXX
Guesstimated Wallet none
Received 36.57242089 HXX in 1 transactions
Sent 0.0 HXX in 0 transactions
Transactions
Hash Block Amount / Amount
80954f36d145b189 ...<PHONE_NUMBER>-08-29 14:13:52 + 36.57242089 HXX 36.57242089 HXX
To this day I still have not received any confirmation on the wallet.
Should I remove and reload my wallet?
I use win 10.
Information wallet.
v<IP_ADDRESS>-0adefaa-travis
/Hexx<IP_ADDRESS>/
Berkeley DB 4.8.30: (April 9, 2010
Sun Sep 2 10:33:43 2018
main
16 (In: 0 / Out: 16) 16 (In: 0 / Out: 16)
current number of blocks: 10119
...
I'm afraid of losing my coin. Please help me solve the problem with my thanks.

Install the chainfiles
Stop/quit your running wallet
Navigate to your hexxcoin application folder
(Windows) -> C:\Users\Username\AppData\Roaming\hexxcoin
(Mac) -> /Users/Username/Library/Application Support/hexxcoin
(Unix) -> /.hexxcoin
Always backup your wallet.dat file from the application folder
Remove/delete all folders and files from the application folder EXCEPT wallet.dat (and xnode.conf if you run an Xnode)
Download the latest chainfiles -> https://github.com/hexxcointakeover/hexxcoin/releases/download/chain/chainfiles.zip
Extract/unzip the chainfiles.zip file and copy/paste chainstate, blocks and db.log in the application folder:
Restart your wallet

Thank you. I have done the tutorial and have synced the data.
But still have not received hexx coin.
I checked the block "Not yet redeemed"
"
Details for Transaction
Hash 80954f36d145b1899cbe942cc90f28119e3801f3f23585dc2897c6001abeff1d
Block Height 325606: 1 (2828 confirmations)
Block Date / Time 2018-08-29 14:13:52
Total Output 59.02846454 HXX Giá trị 0.000373 HXX
Inputs / Outputs
Raw Transaction
Inputs
Index Previous output Address Amount
0 in 325583 HPfNjAYaVraFwDGsr6Z758AZ5Tiq4eT3FA 26.78748566 HXX
1 in 32fps:
Outputs
Index Redeemed in Address Amount
0 Not yet redeemed HELPTHeSeaNDBuztUq6MPxJCxEZajARat 36.57242089 HXX
.
What should I do? Please can help!


Only use the needed vendor libraries

Generated projects should only use the vendordeps that it needs. It's not really feasible to expect every team to have vendor setups installed locally for things they don't use. Deleting the offending vendordep is a workaround, but not a solution

The vendordeps should auto-install, so this shouldn't be necessary; it still might be an okay improvement, I'm unsure if it's worth the extra duplication.

FYI we're using an ADIS16448 so we added the JSON to vendordeps. Some library being brought in by navx_frc.json and REVRobotics.json was preventing the ADIS from being discovered ("can not find ADIS16448"). Deleted the superfluous JSON, rebuilt, and it's working.


VS2022 Error: The specified executable is not a valid application for this OS platform

Hi,
I am trying to run Snyk on VS2022 and I am getting the error: "The specified executable is not a valid application for this OS platform"
Environment details:
VS2022 17.2.6
Windows 10 21H2
Snyk 1.1.24
Logs

2022-07-20 12:40:03.894 [07052] INFO [01] SnykService               Initialize Snyk services
2022-07-20 12:40:03.921 [07052] INFO [01] SnykTasksService          SnykTasksService initialized
2022-07-20 12:40:03.924 [07052] INFO [01] SnykSolutionService       Enter InitializeSolutionEvents method
2022-07-20 12:40:03.924 [07052] INFO [01] SnykSolutionService       Leave InitializeSolutionEvents method
2022-07-20 12:40:03.924 [07052] INFO [01] SnykService               Leave SnykService.InitializeAsync
2022-07-20 12:40:03.924 [07052] INFO [01] SnykVSPackage             Get SnykService as ServiceProvider.
2022-07-20 12:40:03.924 [07052] INFO [01] SnykVSPackage             Start InitializeGeneralOptionsAsync.
2022-07-20 12:40:03.926 [07052] INFO [01] SnykVSPackage             Call GetDialogPage to create. await JoinableTaskFactory.SwitchToMainThreadAsync().
2022-07-20 12:40:03.926 [07052] INFO [01] SnykVSPackage             GeneralOptionsDialogPage not created yet. Call GetDialogPage to create.
2022-07-20 12:40:03.938 [07052] INFO [01] SnykVSPackage             Call generalOptionsDialogPage.Initialize()
2022-07-20 12:40:03.942 [07052] INFO [01] SnykService               Initialize Analytics Service...
2022-07-20 12:40:03.950 [07052] INFO [01] SnykService               analytics enabled = true, endpoint = https://snyk.io/api/user/me
2022-07-20 12:40:03.959 [07052] INFO [01] SnykAnalyticsService      Initializing analytics service instance
2022-07-20 12:40:03.963 [07052] INFO [01] SnykAnalyticsService      Loading Iteratively
2022-07-20 12:40:04.098 [07052] INFO [01] SnykAnalyticsService      Analytics set to enabled
2022-07-20 12:40:04.098 [07052] INFO [01] SnykAnalyticsService      Analytics service instance initialized
2022-07-20 12:40:04.098 [07052] INFO [01] SnykService               Analytics service initialized
2022-07-20 12:40:04.102 [07052] INFO [01] SnykAnalyticsService      Identifying user
2022-07-20 12:40:05.773 [07052] INFO [01] SnykAnalyticsService      Identifying segment client. User-Id = 5158cd16-7515-496b-acdf-06fc0917e2ee, Anonymous Id = 3fc722db-aa4f-42c7-81f9-a7b5fe7ff867, Vs Version: Visual Studio 2022
2022-07-20 12:40:05.781 [07052] INFO [01] SnykAnalyticsService      Analytics identification completed
2022-07-20 12:40:05.784 [07052] INFO [01] SnykVSPackage             Initialize Commands()
2022-07-20 12:40:05.791 [07052] INFO [01] SnykVSPackage             Initializing tool window
2022-07-20 12:40:05.793 [07052] INFO [01] SnykVSPackage             ToolWindow is not initialized. Call await JoinableTaskFactory.SwitchToMainThreadAsync().
2022-07-20 12:40:05.793 [07052] INFO [01] SnykVSPackage             Call FindToolWindow().
2022-07-20 12:40:05.955 [07052] INFO [01] SnykVSPackage             Check ToolWindow is not null Snyk.VisualStudio.Extension.Shared.UI.Toolwindow.SnykToolWindow.
2022-07-20 12:40:05.955 [07052] INFO [01] SnykVSPackage             Initialize ToolWindow.Content. Call await JoinableTaskFactory.SwitchToMainThreadAsync().
2022-07-20 12:40:05.955 [07052] INFO [01] SnykVSPackage             Call ToolWindow.Content.
2022-07-20 12:40:05.955 [07052] INFO [01] SnykVSPackage             Leave InitializeToolWindowControlAsync() method
2022-07-20 12:40:05.958 [07052] INFO [01] SnykVSPackage             Before call toolWindowControl.InitializeEventListeners() method.
2022-07-20 12:40:05.959 [07052] INFO [01] SnykToolWindowControl     Enter InitializeEventListenersAsync() method.
2022-07-20 12:40:05.959 [07052] INFO [01] SnykToolWindowControl     Initialize Solultion Event Listeners
2022-07-20 12:40:05.959 [07052] INFO [01] SnykToolWindowControl     Initialize CLI Event Listeners
2022-07-20 12:40:05.960 [07052] INFO [01] SnykToolWindowControl     Initialize Download Event Listeners
2022-07-20 12:40:05.961 [07052] INFO [01] SnykToolWindowControl     Leave InitializeEventListenersAsync() method.
2022-07-20 12:40:20.500 [07052] INFO [01] SnykTasksService          Enter Download method
2022-07-20 12:40:20.500 [07052] INFO [01] SnykTasksService          Start run task
2022-07-20 12:40:20.507 [07052] INFO [32] SnykCliDownloader         Enter GetLatestReleaseInfo method
2022-07-20 12:40:20.508 [07052] INFO [32] SnykCliDownloader         Get latest CLI release info
2022-07-20 12:40:22.020 [07052] INFO [01] SnykCodeClient            Create http client with with url https://deeproxy.snyk.io/.
2022-07-20 12:40:32.618 [07052] INFO [01] SnykTasksService          Enter Scan method
2022-07-20 12:40:32.930 [07052] INFO [01] SnykTasksService          Start scan task
2022-07-20 12:40:35.607 [07052] INFO [31] SnykSolutionService       Solution folder from is C:\Source\MyApplicationFolder\API
2022-07-20 12:40:35.607 [07052] INFO [31] SnykSolutionService       Result solution folder from is C:\Source\MyApplicationFolder\API
2022-07-20 12:40:35.615 [07052] INFO [31] OssService                Custom Endpoint is null
2022-07-20 12:40:35.615 [07052] INFO [31] OssService                Organization null
2022-07-20 12:40:35.616 [07052] INFO [31] OssService                Ignore Unknown CA is false
2022-07-20 12:40:35.622 [07052] INFO [31] SnykUserStorageSettingsService Enter GetAdditionalOptions method
2022-07-20 12:40:35.654 [07052] INFO [23] SnykSolutionService       Solution folder from is C:\Source\MyApplicationFolder\API
2022-07-20 12:40:35.654 [07052] INFO [23] SnykSolutionService       Result solution folder from is C:\Source\MyApplicationFolder\API
2022-07-20 12:40:35.655 [07052] INFO [23] OssService                Additional Options is
2022-07-20 12:40:35.660 [07052] INFO [23] SnykUserStorageSettingsService Enter GetIsAllProjectsEnabled method
2022-07-20 12:40:35.680 [07052] INFO [36] SnykSolutionService       Solution folder from is C:\Source\MyApplicationFolder\API
2022-07-20 12:40:35.680 [07052] INFO [36] SnykSolutionService       Result solution folder from is C:\Source\MyApplicationFolder\API
2022-07-20 12:40:35.681 [07052] INFO [36] OssService                Is Scan All Projects is true
2022-07-20 12:40:35.681 [07052] INFO [36] OssService                Solution path is C:\Source\MyApplicationFolder\API
2022-07-20 12:40:35.681 [07052] INFO [36] OssService                Start scan
2022-07-20 12:40:35.685 [07052] INFO [36] SnykCli                   Path to scan C:\Source\MyApplicationFolder\API
2022-07-20 12:40:35.686 [07052] INFO [36] SnykCli                   CLI path is C:\Users\MyUser\AppData\Local\Snyk\snyk-win.exe
2022-07-20 12:40:35.690 [07052] INFO [36] SnykCli                   Enter BuildArguments method
2022-07-20 12:40:35.690 [07052] INFO [36] SnykUserStorageSettingsService Enter GetAdditionalOptions method
2022-07-20 12:40:35.735 [07052] INFO [08] SnykSolutionService       Solution folder from is C:\Source\MyApplicationFolder\API
2022-07-20 12:40:35.735 [07052] INFO [08] SnykSolutionService       Result solution folder from is C:\Source\MyApplicationFolder\API
2022-07-20 12:40:35.736 [07052] INFO [08] SnykUserStorageSettingsService Enter GetIsAllProjectsEnabled method
2022-07-20 12:40:35.757 [07052] INFO [27] SnykSolutionService       Solution folder from is C:\Source\MyApplicationFolder\API
2022-07-20 12:40:35.757 [07052] INFO [27] SnykSolutionService       Result solution folder from is C:\Source\MyApplicationFolder\API
2022-07-20 12:40:35.758 [07052] INFO [27] SnykCli                   Result CLI options --json test --all-projects
2022-07-20 12:40:35.759 [07052] INFO [27] SnykCli                   Leave BuildArguments method
2022-07-20 12:40:35.760 [07052] INFO [27] SnykCli                   Token added from Options
2022-07-20 12:40:35.762 [07052] INFO [27] SnykCli                   Start run console process
2022-07-20 12:40:36.285 [07052] EROR [27] SnykConsoleRunner         Cli running process error.
System.ComponentModel.Win32Exception (0x80004005): The specified executable is not a valid application for this OS platform.
at System.Diagnostics.Process.StartWithCreateProcess(ProcessStartInfo startInfo)
at Snyk.VisualStudio.Extension.Shared.CLI.SnykConsoleRunner.Execute()
2022-07-20 12:40:36.537 [07052] INFO [27] SnykCli                   Start convert console string result to CliResult and return value
2022-07-20 12:40:36.540 [07052] INFO [27] OssService                Scan finished. Is successful: false
2022-07-20 12:40:36.540 [07052] EROR [27] SnykTasksService          Oss scan exception
Snyk.VisualStudio.Extension.Shared.Service.OssScanException: Exception of type 'Snyk.VisualStudio.Extension.Shared.Service.OssScanException' was thrown.
at Snyk.VisualStudio.Extension.Shared.Service.OssService.d__7.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at Snyk.VisualStudio.Extension.Shared.Service.SnykTasksService.d__74.MoveNext()
2022-07-20 12:40:41.900 [07052] INFO [01] SnykTasksService          Enter Download method
2022-07-20 12:40:41.900 [07052] INFO [01] SnykTasksService          Start run task
2022-07-20 12:40:41.901 [07052] INFO [30] SnykCliDownloader         Enter GetLatestReleaseInfo method
2022-07-20 12:40:41.901 [07052] INFO [30] SnykCliDownloader         Get latest CLI release info

Hoping you can assist.
Thanks for the great tool!

Hi @tinonetic !
The release had nothing to do with it. Did this error prevent you from running scans previously and now it doesn't?

Hi, since GitHub issues are not officially monitored by Snyk support, we will be disabling "Issues" tab on this repository very soon.
Please raise the current issue directly by contacting Snyk Support on the web or emailing us at<EMAIL_ADDRESS>


Add  target platform

Add  new  target platform to windows 10 x86 and x64  architectures , which is  really for  windows 10 enterprise customers. Thanking you

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

ID: ec8e958d-60a7-ef70-4d61-d8962982007d
Version Independent ID: f6024c93-22d8-8ab6-1c69-469f93f3d3cc
Content: Internet Explorer Administration Kit (IEAK) information and downloads
Content Source: browsers/internet-explorer/ie11-ieak/ieak-information-and-downloads.md
Product: ie11
GitHub Login: @shortpatti
Microsoft Alias: pashort

Target OS has been added and it will show later today.

duplicate of #1902


I can't see accessories/devices on iphone13pro(16.3) and iwatch(9.3) but on iPad(15.7.1) yes :-/ Why?

I have no idea about this, at least one of the apple device see the accessory, means homebridge is working.maybe you can fire an issue on apple fourm.

Yesterday's ios update solved my problem , now i can see all devices from Tuya app ;-)

great :)


Tc.Prober doc config; TcOpen application article

Sorry for this huge PR; multiple commits

Tc.Prober has been moved to TcOpen repo; the docu config added here
Brief description of concepts TcoCore and TcOpen application

@TomKovac should contain for https://github.com/Inxton/TcOpen/pull/9 so we can test gh pages


pkill in roles/reset/tasks/main.yml assumes the path for the k3s data directory ends with "k3s"

k3s_server_location was introduced in #133 and can be used to change the location of the data directory (default: /var/lib/rancher/k3s).  In roles/reset/tasks/main.yml, pkill is used to kill k3s processes with a pattern that begins with "k3s/data".  "k3s" is part of what can change with k3s_server_location.
If one changes the location and the path does not end with "k3s", the containerd-shim processes will not be killed.

Resolved with https://github.com/k3s-io/k3s-ansible/pull/230


v1.0.0:SD_WU_6: File Delete option should not be available from shared read only view.

Users get an option to delete a selected file from the shared view but the selected file is not deleted as it should be. This option must not be available from shared read only view.

This means that the user could be at risk of deleting files while in read-only mode, which is going to be seriously frustrating for them. GIven that, I'd call this a bug rather than an enhancement.

Also, the view for the Share shows the icons for "Create New File" and "Create New Directory" when the Share is read-only.  It does not let you create files or folders (correctly) but having the icons there is wrong.

Tested on scidrive 2:
the issue has not been fixed.

For read-only shares these actions will return error, no files will be deleted or created. For now there's no way for the portal to know if a share is read-only or not.


[SPARK-30798][SS][FOLLOW-UP] Scope Session.active in IncrementalExecution

What changes were proposed in this pull request?
The optimizedPlan in IncrementalExecution should also be scoped in withActive.
Why are the changes needed?
Follow-up of SPARK-30798 for the Streaming side.
Does this PR introduce any user-facing change?
No.
How was this patch tested?
Existing UT.

cc @cloud-fan @HyukjinKwon

Test build #124589 has started for PR 28936 at commit 5ca38fd.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/29210/
Test PASSed.

Test build #124589 has finished for PR 28936 at commit 5ca38fd.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/124589/
Test FAILed.

retest this please

Test build #124595 has started for PR 28936 at commit 5ca38fd.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/29216/
Test PASSed.

Test build #124595 has finished for PR 28936 at commit 5ca38fd.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/124595/
Test PASSed.

Thanks, @xuanyuanking and @cloud-fan . I created SPARK-32126 and update the title.
Merged to master/3.0.

Sorry for the late.
Agree we need a new Jira for this. I should create it by myself, thanks for your help! @dongjoon-hyun


Optionally prevent non-admins from accessing /_all_dbs

This is a leftover from the 1.x -> 2.x transition. Adds a config option (bike shed away pls) to block non-admins from accessing /_all_dbs, which we should set to true in 3.0.0

+1 otherwise.

+1, very tidy.


support dataChangeLastModifiedBy display after editing in a namespace

support dataChangeLastModifiedBy display after editing in a namespace, not only in the edit history.

Coverage decreased (-0.08%) to 53.703% when pulling 851c814c900e1d479be906d68dd4a444a688d131 on last_modified_by into ff392573ef64dc3144051cd41a7fb09551ae733a on master.

Codecov Report

Merging #2680 into master will decrease coverage by 0.08%.
The diff coverage is 16.66%.

@@             Coverage Diff              @@
##             master    #2680      +/-   ##
============================================
- Coverage     50.23%   50.15%   -0.09%
+ Complexity     2061     2060       -1
============================================
  Files           413      413
  Lines         12644    12666      +22
  Branches       1292     1294       +2
============================================
  Hits           6352     6352
- Misses         5844     5864      +20
- Partials        448      450       +2

Impacted Files
Coverage Δ
Complexity Δ

...p/framework/apollo/portal/service/ItemService.java
62.8% <0%> (-0.53%)
16 <0> (ø)

...k/apollo/biz/utils/ConfigChangeContentBuilder.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

...p/framework/apollo/portal/api/AdminServiceAPI.java
12.57% <0%> (-0.25%)
1 <0> (ø)

...apollo/adminservice/controller/ItemController.java
7.89% <0%> (-0.93%)
2 <0> (ø)

...trip/framework/apollo/portal/entity/bo/ItemBO.java
81.25% <100%> (ø)
8 <1> (ø)
:arrow_down:

...mework/apollo/portal/service/NamespaceService.java
66.23% <42.85%> (-1.34%)
26 <0> (ø)

.../apollo/internals/RemoteConfigLongPollService.java
79.01% <0%> (-1.24%)
27% <0%> (-1%)

Continue to review full report at Codecov.

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

Do I need to fix codecov/patch?


Fix small typo in README.md in installing flipper-active_record part.

I found a typo when trying to install the gem, especially when installing the flipper with an active record adapter. So this PR is just to fix the typo :)

@philiplambok thanks!
Also there is going to be a new release out very soon that reflects these docs. The latest gem and the docs here don't line up currently. Just an FYI.
If you don't mind, I would love to know what you are using flipper for as well and how it is working out for you (what went well, what was hard other than what you mentioned here).
If you aren't comfortable leaving that information here in a public comment, feel free to email me directly<EMAIL_ADDRESS>I'm just always curious. :)


[Known][Waiting] Drag and Drop files doesn't work on Linux

Similar to https://github.com/dan0v/xdelta3-cross-gui/issues/2
The drag and drop (files/folder) from Files window to the app doesn't work on Linux. It works normally on Windows and MacOS.

Is there any know work around for this please

Fwiw I've had a crack at this today. I did have some limited success in implementing the "Connect" callback, but couldn't seem to wire everything up completely. I don't think it will be too bad - given a lot of the boilerplate is already there. However, I was basically hacking away at stuff by spawning a function to "RunOnGlibThread" on the MainWindow, and trying to stuff it all in there as a PoC... and it all just felt incredibly dirty and hacky, and I figure it is best to leave it to those who actually know what they're doing.
It would be really nice to get this sorted though, so we have all 3 platforms working with drag and drop. I've attached a really simple python script which I was using on KDE/Wayland with success to demo the APIs in use. There's no way I'd be able to take it further than this though, as I just don't have the skillset to sort it, sorry.
#!/usr/bin/env python3

import gi
gi.require_version('Gtk', '3.0')
from gi.repository import Gtk, Gdk, GObject

class DragDropWindow(Gtk.Window):

    def __init__(self):

        Gtk.Window.__init__(self, title='Drag and drop test')
        self.set_default_size(400, 400)

        # Set up drag and drop
#       self.connect("drag-motion", self.motion_cb)
#       self.connect("drag-drop", self.drop_cb)
        self.connect("drag-data-received", self.got_data_cb)
#       self.drag_dest_set(0, [], 0)
        targets = Gtk.TargetList.new([])
        targets.add_uri_targets(2)
        self.drag_dest_set_target_list(targets)
        self.drag_dest_add_text_targets()

#      { "STRING",        0, TARGET_STRING },
#      { "text/plain",    0, TARGET_STRING },
#      { "text/uri-list", 0, TARGET_URL },

#    def motion_cb(self, wid, context, x, y, time):
#        Gdk.drag_status(context,Gdk.DragAction.COPY, time)
#        return True

#    def drop_cb(self, wid, context, x, y, time):
#        wid.drag_get_data(context, context.list_targets()[-1], time)

    def got_data_cb(self, wid, context, x, y, data, info, time):

        print("got cb")

        if info == 0:
            text = data.get_text()
            datatype = data.get_target()
            print("Received text: %s" % text)
            print("Data type:     %s" % datatype)

        elif info == 2:
            print("Path/s %s" % data.get_uris())
            print("Size %s" % data.get_length())
            print("datatype/mime %s" % data.get_target())

        context.finish(True, False, time)

win = DragDropWindow()
win.connect("destroy", Gtk.main_quit)
win.show_all()
Gtk.main()

Lack of drag drop on linux is actually a surprise for me. I thought it's done long ago. I think that this feature is mandatory. Priority is the same as clipboard support.

Still no drag and drop support on Linux!? It's a fairly basic feature, although perhaps not trivial to implement... Still, perhaps it would be better to say "limited Linux support" and list the limitations since Avalonia is touted as a "cross-platform UI framework" with no mention that Windows is the primary platform, and the support for the other platforms vary. Still very cool and very useful, but this is a bit of a disappointment and it comes after investing quite a bit into it just to realize that there's no drag and drop on Linux, and doesn't appear to be a very high priority either. The original post here is now over 2 years old...
(Also, it can be noted slightly off-topic that copy/paste support is also sketchy on macOS and Linux and needs a bit of client OS-dependent code to work around the limitations, but at least it's possible to make it work with reasonable effort)

@xecrets I understand your needs, but the thing is that Linux has several different window managers, all working differently. We are still trying to figure out a proper API for it. #11011
As the issue is labeled with help-wanted, if you or anyone else want to take a look into providing DnD support, a PR is welcome.

@timunie - Thanks for explaining. I do realize there's a challenge, and would love to contribute - but it's simply way above my pay-grade to write Linux-specific implementations of this sort, for perhaps also three window managers too to boot! I have considered it, especially for the somewhat related issue of copy/paste, where more of the scaffolding seems to be in place. But then again, I really have no idea of how the different window managers differ.
As the issue is indeed non-trivial, and it's likely to take some time, it would be nice if the documentation would mention supported platforms, and if there was like a table of what feature groups are supported on what platforms. As the number of platforms increase, I'm guessing this will become even more valuable.
For now, I've implemented OS-specific wrappers of IClipboard for macOS and Linux (Ubuntu/Gnome) that handles DataFormats.Text and DataFormats.Files at least well enough for my current needs. As for DnD I'm now in the process of having OS-specific text prompts and documentation that simply avoids mentioning DnD for Linux... By the way, isn't it a little strange to have an incomplete IClipboard implementation - and then adding an [NotClientImplementable] attribute to the interface ;-) ?

By the way, isn't it a little strange to have an incomplete IClipboard implementation - and then adding an [NotClientImplementable] attribute to the interface ;-) ?

No. This exactly allows us to improve / implement it without breaking changes. If we would have make it implementable, we couldn't add it during 11.x timeframe. I also think if someone wants to implement it, Avalonia should be fixed. Not a hack-fix in the App.

I looked into how X11 Drag & Drop (XDND) could be added into Avalonia.
The current IDataObject interface critically assumes that Get is sync. In XDND a client can read the mime types in sync but to receive the actual content or file, the receiver calls XConvertSelection, the sender responds to SelectionRequest, the receiver receives SelectionNotify and can then read the property that actually holds the content or file.
The workaround I could think off here is to assume that the library user just accepts the known mime type, request it, receive the contents and only then emit the event.

The current IDataObject was copy-pasted from WPF without any thought about X11 and it got merged because we didn't have "the proposed API needs to be supported across all platforms before merging" policy back in the day.
My current idea about fixing the mess is to have a dedicated X11 thread just for clipboard and DND.

If someone is still interested in this topic, I found this on the avalonia docs that does not allow to drag and drop things but at least permits to import a specified file.

Same here. We implemented Drag&Drop support for our application and later we noticed it's not working for the Linux platform. I hope it's implemented soon.

Is there any progress/status update on this? Drag-n-Drop still seems to be missing today for Linux, which is not great for UX for many apps.


shims: correct the check for newer SLPI enumerators

The previous check did not work as the enumertors are not defines but
rather a proper enumerator.  This adds a build time check for the
support and conditionalises the compilation.

CC: @rokhinip
Unfortunately, the CI hosts are updated outside of my control by Microsoft.  They bumped the SDK, so we need this for the release.

@swift-ci please test

Sounds good.


update README.md content to be more informative

1. Why is this change necessary?
The current readme contains a lot of marketing information that is better suited for the website, not a technical readme.
2. What does this change do, exactly?
As a first step, this update declutters the content of the readme and focuses more on it being a guideline for how to interact with the software and community.
5. Checklist

[x ] I have rebased my changes to remove merge conflicts
[x ] I have read the contribution requirements and fulfil them.

copilot:summary
copilot:walkthrough

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Niklas Dzösch seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

Was merged internally in d190cf9593e54eb1043b6225df4b3a5df1d36415 already, so closing this PR.


Design Meeting Notes, 12/1/2017

Partial generic type arguments
#10571

Partial inference for generic type arguments

Today we either infer all or none

Gets a little bit confusing for defaults

Today, if you have a parameter list like in f<T, U, V = never>(x: T, y: U, z: V): void, a type argument list like in f<string, boolean>("", true, 100) will infer never for V and cause an error.

Argument of type '100' is not assignable to parameter of type 'never'.

This would technically be a breaking change.

But might be hard to actually encounter.
Though, type parameter defaults with any might be a case.

So idea: always do inference, and maybe do an optimization when all type parameters are "fixed".
Should you be able to mix positional/named?

Either

Only positional & only named
Positional followed by named

#19205

Comes up with Vue's <Data, Computed, Methods, PropNames>

Want to be able to add Props, but it comes last.
Even if it didn't come last, people want to be able to specify these.

By the time you have 5 type arguments, you really won't be able to reason these.

Do we want to do this even if associated types might be another option?

Seems like we could still do this and tackle associated types later.

Syntax

Technically an invalid left-hand side, should be fine.

Conclusions

Partial inference

Do it, infer in place of default arguments, and continue to use the default when no inferences are available.

Named type parameters

Positional followed by named

Extracting returned/constructed types
#6606
#20350

Undergoing overload resolution is problematic.

Implementation is super complex and you run into several circularity problems.

Could just give the union, but what abut type parameters?
What about the "zero-order" problem? In other words, no overloads.

This is one of the most common cases.

Does this solve things cases like promisify?

No, you need variadic kinds.

Would match types take care of this?

They wouldn't be able to decompose overloads.

What about a zero-order typeof?

What does that even mean?

How's this different from const x: typeof x?

We can short-circuit more easily based on the symbol thanks to the restricted forms.

Just want to say that I'm very excited about partial inference of generic type arguments :)
I just ran into this limitation a few minutes ago, and it's nice to see that there is already active planning for it.


Populate branch comboboxes using RPC

Define a new widget, BranchSelect which is automatically populated with the list of branches obtained via RPC. Fixes #27.
I also started defining an RpcWrapper namespace whose job is to wrap the RPC methods, do the conversions to/from JSON, and report errors. This seems like it would be a good abstraction to have though it currently only wraps listbranches. For the sake of keeping things organised it would probably be a good idea to either move all the RPC calls into it, or just kill it (and move the listbranches stuff into BranchSelect). Thoughts?

Hey contact me on slack next time you get the chance!


Fix typo in README.md

Correct name of condition warningCountAbsoluteSpecifiedScoreAndHigher

Thank you for @ampleyfly for this pull request, we intend to merge this soon.


Make container position: relative

Allows for better/easier positioning of elements within (e.g., a toggler in a navbar). Alternative is to add a relative positioning utility, but I think this is more intuitive.

Just as a reminder:
That change makes it necessary to place elements, which should be absolute to e.g. the body or main wrapper need to be placed outside of the container. Sometimes this might not be possible due to the logic of a framework or system.


Does not connect to open wifi

There is no place in the flask site to indicate the AP is open. It will accept any password but will not connect to the AP.
I am assuming this would require:

update the template to add a check box for open access points
update the wpa supplicant template to add a "wpa_psk NONE" (I think this is the command for open wifi in supplicant, I will have to double check)

Anything I missed? I can take a shot at this if that these are the only changes needed.

Have you resolved this issue? having the same problem. any help?


CVE-2022-2068 c_rehash vuln in bullseye

Current image (2.4.54) with bullseye as a base has the CVE-2022-2068 vulnerabilities in it.  Could a build be created that incorporates the fix for that?
https://security-tracker.debian.org/tracker/CVE-2022-2068

It'll get updated soon on its regular monthly cadence. You can also manually update the package in your image if you wanted in the meantime https://github.com/docker-library/python/issues/728#issuecomment-1132025905
Background:

Tags in the [official-images] library file[s] are only built through an update to that library file or as a result of its base image being updated (ie, an image FROM debian:buster would be rebuilt when debian:buster is built).
-https://github.com/docker-library/official-images/tree/2f086314307c04e1de77f0a515f20671e60d40bb#library-definition-files

Official Images FAQ:

Though not every CVE is removed from the images, we take CVEs seriously and try to ensure that images contain the most up-to-date packages available within a reasonable time frame
- https://github.com/docker-library/faq/tree/6138d05aabf61563606d86f98d0ccbd99f162b33#why-does-my-security-scanner-show-that-an-image-has-cves

Since our build system makes heavy use of Docker build cache, just rebuilding the all of the Dockerfiles won't cause any change. So we rely on periodic base image updates.

We strive to publish updated images at least monthly for Debian and Ubuntu. We also rebuild earlier if there is a critical security need. Many Official Images are maintained by the community or their respective upstream projects, like Alpine and Oracle Linux, and are subject to their own maintenance schedule.
- from the same FAQ link


The wrong parameter was being sent through to a method.

I updated the method call to send through a course module id

👍


Sample data for predictive algorithms

@rmgarnett asks:

Can you help me find NEE/GPP/Reco data for a flux site that has an open access use policy for development of prediction algorithms? you gave me earlier.  It'd be nice to include such data in this blog post.  Any chance you have some similar data I could use to recreate similar plots?

Here for Mead, Nebraska http://ameriflux-data.lbl.gov:8080/SitePages/siteInfo.aspx?US-Ne3
There are lots of different ways in which the data were 'gap filled'. I chose the ones that use ANN because it sounds fancy. If you want to filter by only the true observed data, you can filter where [NEE_or_f ANNqc == 0]

data
readme explaining variables
conditions of use (basically, contact data provider prior to publication):

R code:
library(data.table)

readme <- readLines(con = "ftp://cdiac.ornl.gov/pub/ameriflux/data/Level4/Sites_ByName/L4_README.txt")
writeLines(readme)

x <- fread("ftp://cdiac.ornl.gov/pub/ameriflux/data/Level4/Sites_ByName/Mead_Rainfed/AMF_USNe3_2005_L4_h_V002.txt")
x[x==-9999]<-NA

## show that NEE == GPP - Reco

ggplot(data = x, aes(x = GPP_or_ANN - Reco_or, y = -NEE_or_fANN)) +
  geom_point()

summary(lm(NEE_or_fANN ~ I(GPP_or_ANN - Reco_or), data = x))

## subset original data
x_original <- x[NEE_or_fANNqc == 0]

write.csv(x[, list(DoY, GPP = GPP_or_ANN, NEE = NEE_or_fANN, Reco = Reco_or)], "~/mead_nebraska_gapfilled.txt", row.names = FALSE)

write.csv(x_original[, list(DoY, GPP = GPP_or_ANN, NEE = NEE_or_fANN, Reco = Reco_or)], "~/mead_nebraska_original.txt", row.names = FALSE)

I've attached the filtered (original) and gap filled data from the final step with column names DoY, GPP, NEE, and Reco:

mead_nebraska_original.txt
mead_nebraska_gapfilled.txt

Here is a plot of the gap filled data:
library(ggthemes)
ggplot(data = x[DoY > 100 & DoY < 150]) +
  geom_point(aes(x = DoY, y = GPP_or_ANN), alpha = 0.5, color = 'black', size = 0.1) +
  geom_point(aes(x = DoY, y=Reco_or), color = 'red', alpha = 0.5, size = 0.1) +
  geom_point(aes(x = DoY, y=NEE_or_fANN), color = 'green', alpha = 0.5, size = 0.1) +
  ggtitle("2004 GPP (black), Reco (red), and NEE (green) at Mead, NE")+
  xlab("Day of Year") +
  ylab("umolCO2 m-2 s-1") +
  theme_tufte()


No published analyzer available for language version 2.14.0

Thank you for taking the time to file an issue!
This tracker is for issues related to:

Analyzer
Common Front End (CFE) and kernel
Dart core libraries ("dart:async", "dart:io", etc.)
Dart VM
dart2js
dev_compiler

Some other pieces of the Dart ecosystem are maintained elsewhere. Please
file issues in their repository:

Dart language: https://github.com/dart-lang/language
Dart website: https://github.com/dart-lang/site-www
Flutter: https://github.com/flutter/flutter
Linter: https://github.com/dart-lang/linter
dart format: https://github.com/dart-lang/dart_style

If you aren't sure, file the issue here and we'll find the right home for it.
In your issue, please include:

Dart SDK Version (dart --version)
Whether you are using Windows, MacOSX, or Linux (if applicable)
Whether you are using Chrome, Safari, Firefox, Edge (if applicable)

Missing some or all of the above might make the issue take longer or be
impossible to act on.
If you simply have a question, consider starting with Stack Overflow:
https://stackoverflow.com/questions/tagged/dart
For a real-time response, consider our Gitter channel:
https://gitter.im/dart-lang/sdk

Duplicate of https://github.com/dart-lang/sdk/issues/46658


Desktop apps

Use go-astilectron to bundle adash as a desktop GUI apps for Windows, Mac and Linux.

Experimental support for this is on the electron branch.  Current state: it works, but downloads stuff to vendor/ that conflicts with our Go module config, and also can't use the auth token because it doesn't seem to be storing cookies.


[build] Log code coverage info

Summary

We want to know how we perform for each commit in terms of code coverage and learn the trend such that we can seek for improvement and bump/lower the requirements if needed.
It'd be a lot work to visualize the trend so for the time being, we log the coverage data by parsing the XML reports provided by Jacoco and diffCoverage so dev can see their coverage info more easily.
How was this PR tested?

./gradlew jacocoTestCoverageVerification diffCoverage --continue
Example outputs:

> Task :clients:venice-push-job:jacocoTestReport
Branch coverage: 49.84% // coverage info for this submodule

> Task :all-modules:diffCoverage
Fail on violations: true. Found violations: 0.

> Task :clients:diffCoverage
Fail on violations: true. Found violations: 0.

> Task :clients:logDiffCoverage
Branch coverage: 75.0% // coverage info for my current commit/diff

Does this PR introduce any user-facing changes?

[x] No. You can skip the rest of this section.
[ ] Yes. Make sure to explain your proposed changes and call out the behavior change.

Should we add cloud-based CC? I've tried it: https://github.com/sushantmane/venice/pull/5#issuecomment-1457587145. Many teams at LI are also using it.

Should we add cloud-based CC? I've tried it: sushantmane#5 (comment). Many teams at LI are also using it.

@sushantmane  I am going to merge this PR first for local coverage report. I've managed to get the Codecov working on my personal repo but needs some tweaking to make it work for venice, e.g. multiple project setup, branch coverage checking, etc.

Should we add cloud-based CC? I've tried it: sushantmane#5 (comment). Many teams at LI are also using it.

@sushantmane  I am going to merge this PR first for local coverage report. I've managed to get the Codecov working on my personal repo but needs some tweaking to make it work for venice, e.g. multiple project setup, branch coverage checking, etc.

Let me know if you need help


[doc] Extend documentation for detection status

The image has been uploaded under vue-cli directory, but no other .vue files changed. Is this going to appear in the GUI?


Add some model supports

can add more or some models supports (posible)? like as:
-3d studios (.3ds)
-genesis 3d (.act)
-quake 2 (.md2) (my favourite)

For the record, oldschool model formats will not be supported officially by Godot, but nothing prevents a third party from creating a module for it.


API: /delete endpoint

So I added a /api/v2/admin/delete endpoint and it works as per my manual testing.
We can now delete in time-ranges. And the endpoint now has the compulsory parameters match, start, end.
Per the discussion in #2760 I am thinking of putting all admin endpoints under /api/v2/admin and protecting it with a toggle-flag.
This PR is just to get the discussion started (and mostly will be redone):

Is the path okay?
The new file /v2/api.go copies most of the functions and datastructures from /v1/api.go. Should I export them instead? Or broadly, how should the new API code be structured.
For deletes: what should be the response be? As we can delete time-ranges and would be updating tombstones, I am not able to see what useful info we can provide here. Would 200 OK suffice with no body?

cc @fabxc @brian-brazil @grobie @juliusv @beorn7 @SuperQ

This change is

If we're going to put admin stuff under this path, are we going to switch e.g. the alertmanager to also have a /api/v2/admin/reload ?

Eventually yes. Time to start thinking about a properly consistent API
design again.
On Sat, May 27, 2017 at 6:58 PM Brian Brazil<EMAIL_ADDRESS>wrote:

If we're going to put admin stuff under this path, are we going to switch
e.g. the alertmanager to also have a /api/v2/admin/reload ?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/prometheus/prometheus/pull/2777#issuecomment-304464029,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEuA8rxDqNTTRNFOC92-u7t9PVqZbaGDks5r-FZFgaJpZM4NoaDa
.

So when we next have a new API, init scripts will break?
I'm not sure we want all the admin stuff under the same API as the end-user stuff from a manageability standpoint.

Health and readiness endpoints should probably remain top-level concerns and never be more complex than returning 200 if things are okay. @beorn7 any further thoughts on standardizing that across our components yet?
@brian-brazil did you think about anything other than health and readiness? I think reload falls under the /v2/admin category though. APIs are versioned and generally not removed without a long enough overlap for migration.

I was mainly thinking about the lifecycle stuff, as it'd be good to have that consistent across binaries.

As discussed in https://github.com/prometheus/pushgateway/issues/105 , it would be great to have consistency for health and liveness checks across all our binaries. API versioning is for the opposite: Separate incompatible HTTP calls, and that even between different versions of the same binary.
I assume @brian-brazil is referring to reload and quit when talking about lifecycle stuff. Which gives us the following endpoints to be outside of API versioning but behind /-/... instead:

/-/healthy
/-/ready
/-/quit
/-/reload

Any additions or corrections?

Closing in favour of #2821


questions regarding use of Eureka for a real Robot

Hi,
I have following questions regarding use of Eureka for a real Robot:

What files are the better place to add real Robot SDK calls to drive real robot?
Are there any plans to integrate on-device LLM like llama instead of cloud access of GPT-4 or 3.5 to have near Real time Robot control?
Optionally, are there any plans to add four legged Robot similar to Spot in the simulator?

The best place to add real Robot SDK calls to drive a real Robot in Eureka depends on the specific SDK and the desired functionality. However, some common places include:
The RobotController class. This class provides a high-level interface for controlling a Robot, and it is a good place to add SDK calls for basic functionality such as moving the Robot around, manipulating objects, and collecting sensor data.
The RobotActuator and RobotSensor classes. These classes provide lower-level interfaces for controlling individual actuators and sensors on the Robot. They can be used to add SDK calls for more complex functionality, such as controlling the Robot's gripper or performing precise navigation.
The RobotWorld class. This class represents the Robot's world, and it can be used to add SDK calls for interacting with the environment, such as detecting obstacles or planning paths.
Integration with on-device LLM
There are currently no plans to integrate on-device LLMs such as LaMDA into Eureka. However, this is a possibility that the team is considering for the future.
Four-legged Robot in the simulator
There are also currently no plans to add a four-legged Robot similar to Spot to the Eureka simulator. However, this is something that the team is open to considering in the future.
In the meantime, there are a number of other Robot simulators available that support four-legged Robots, such as PyBullet and SimSpark. These simulators can be used to develop and test Robot control algorithms before deploying them to real Robots.


Basic private key and account management

We should implement some basic key/account management logic.

[ ] Key creation (recovery phrase? test accounts probably don't need password protection)
[ ] Account switching
[ ] Key/account naming and metadata so it makes sense

The next stage would be to allow collaboration and be a good local keychain/wallet.
What are some useful functions for storing/accessing keys?

We should probably bundle with something like https://github.com/rustyx/keythereum-node/blob/master/index.js so we don't need to depend on keythereum library. I guess we are okay for now though.


Am not getting relationship data back from api for Secrets & Configmaps

Steps to reproduce (least amount of steps as possible):

Create a Secret and a Config map
Create a deployment that references that secret & Config map in the environment variables section
Go to secret & config map detail pages and look at related resources

Result:  Nothing show up in related resource.  Should show deployment

Able to reproduce this on 2.5.5:

From cluster explorer --> Secrets --> Add secret [add key-value pair]
From cluster explorer --> config maps --> Add configmap [add key-value parir]
Now from deployments --> Create deployment --> add image, environment variables [add variables] select the config map and secret created in the above step
Once the deployment is active --> secrets/config maps --> click on the added secret/config map and in related resources tab no workloads seen
On dashboard:

On ember UI:

Verified the issue on : v2.5-bdec2f2a84e6dc5bca61b727ca5306174c816f33-head
From cluster explorer --> Config maps --> create a config map with key/value pair
Create a secret key/value pair
From the deployments --> create a workload with env variables --> configmap/secret key attached created from the above step
Now if we verify the cluster explorer --> Config maps/secrets --> click on the created configmap --> related resources has the workload

Re-opening the issue for master-head PR

This doesn't need a PR for master-head unless it is broken.
Wrangler is updated fairly frequently on master-head automatically.
Can you test this out @anupama2501 ?


Fixed build problem and crashing SearchActivity after orientation change

Description (required)
Resolves https://github.com/commons-app/apps-android-commons/issues/2125
Resolves https://github.com/commons-app/apps-android-commons/issues/2093
What changes did you make and why?
https://developer.android.com/guide/topics/manifest/activity-element#config
"If your application targets Android 3.2 (API level 13) or higher, then you should also declare the "screenSize" configuration, because it also changes when a device switches between portrait and landscape orientations." - from docs.

Build problem fix a duplicate of #2028

Can you make this one just target #2125?

@domdomegg done.

Change seems okay, tested on API levels 25 and 28. Don't know if it fixes the issue as was unable to recreate.
Given the documentation, think it's still worth merging even if bug irrelevant.

This does fix the issue - although I believe the 'true' issue might be deeper. Merging this as does prevent the crash and also recommended by documentation.


Bot Doesn't work with categories.

NadekoBot is immediately not reading and voice or chat channel within a discord that has a category in it. Categories were recently added in a new discord update so I didn't think it would work with anything with thin that category but the bot is just not reading ANY channel in that discord even channels that aren't in a category.
Tested:
I deleted all added Categories in my discord. then the bot was able to read all Voice and chat channels.
I then added a single Category and added a couple of channels into it. The bot was not able to read any of the Voice or chat channels.

Are you running the latest version? Make sure to update Nadeko. It was fixed in 1.8 I believe.

I updated it a couple of times including the stable and the latest builds neither of them worked

I haven't tried reinstalling it completely I only updated but I can try that

Update your bot to 1.8+
If you're on linux you need to use this link
first, backup your current installation just in case
cd ~ && wget -N https://github.com/Kwoth/NadekoBot-BashScript/raw/1.4/linuxAIO.sh && bash linuxAIO.sh

Reinstall prerequisites
Download nadeko (first option)
Run nadeko


add filters

Server code for PR ga4gh/schemas#777 adding fields for variant filters.

closes #1491

Codecov Report

Merging #1577 into master will increase coverage by 0.03%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #1577      +/-   ##
==========================================
+ Coverage   85.91%   85.95%   +0.03%
==========================================
  Files          34       34
  Lines        7489     7510      +21
  Branches      938      943       +5
==========================================
+ Hits         6434     6455      +21
  Misses        857      857
  Partials      198      198

Impacted Files
Coverage Δ

ga4gh/server/datamodel/variants.py
94.44% <100%> (+0.16%)
:white_check_mark:

Continue to review full report at Codecov.

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


feat: Add codable identifier arrays

Description
Adds a route to Kitura to return an array of tuples of identifiers and codable objects. These are then sent as JSON as an array of dictionaries
Motivation and Context
This change was requested for use with the ORM when performing a get all function and it allows you to return the results in order which is needed for using .orderedby in SQL
How Has This Been Tested?
The tests will fail until this pull request to KituraContracts is merged/
This has been tested using swift with an updated Kitura contracts.
A new test has been added for this change.

Codecov Report

Merging #1223 into master will decrease coverage by 0.39%.
The diff coverage is 57.14%.

@@            Coverage Diff            @@
##           master    #1223     +/-   ##
=========================================
- Coverage   89.39%   88.99%   -0.4%
=========================================
  Files          38       38
  Lines        2234     2262     +28
=========================================
+ Hits         1997     2013     +16
- Misses        237      249     +12

Flag
Coverage Δ

#Kitura
88.99% <57.14%> (-0.4%)
:arrow_down:

Impacted Files
Coverage Δ

Sources/Kitura/CodableRouter.swift
76.15% <57.14%> (-2.53%)
:arrow_down:

Continue to review full report at Codecov.

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

@ianpartridge Now The Kitura contracts has been tagged and this has been rebased to included the refactoring/error commit Would you be able to review?


Adding CryptoJobMetrics and Tracker to track detailed crypto job metrics

This patch adds detailed cryptoJob metrics like request queuing time, processing time, response waiting time and so on.
Coding style applied
SLA: 20 mins

Codecov Report

Merging #821 into master will decrease coverage by <.01%.
The diff coverage is 97.05%.

@@             Coverage Diff              @@
##             master     #821      +/-   ##
============================================
- Coverage     66.77%   66.77%   -0.01%
+ Complexity     4157     4151       -6
============================================
  Files           354      354
  Lines         27217    27281      +64
  Branches       3440     3440
============================================
+ Hits          18175    18216      +41
- Misses         8106     8123      +17
- Partials        936      942       +6

Impacted Files
Coverage Δ
Complexity Δ

.../main/java/com.github.ambry.router/DecryptJob.java
97.22% <100%> (+0.25%)
5 <4> (ø)
:arrow_down:

.../com.github.ambry.router/GetBlobInfoOperation.java
82.71% <100%> (+0.21%)
34 <0> (ø)
:arrow_down:

....github.ambry.router/NonBlockingRouterMetrics.java
89.58% <100%> (+1.5%)
26 <0> (ø)
:arrow_down:

.../main/java/com.github.ambry.router/EncryptJob.java
97.36% <100%> (+0.22%)
5 <1> (ø)
:arrow_down:

...ain/java/com.github.ambry.router/PutOperation.java
90.02% <75%> (+0.06%)
71 <0> (ø)
:arrow_down:

...java/com.github.ambry.router/GetBlobOperation.java
90.6% <92.85%> (-0.17%)
32 <0> (-1)

...c/main/java/com.github.ambry.network/Selector.java
78.48% <0%> (-3.59%)
44% <0%> (-1%)

.../main/java/com.github.ambry.store/HardDeleter.java
80% <0%> (-2.26%)
41% <0%> (-2%)

.../src/main/java/com.github.ambry.store/Journal.java
89.65% <0%> (-1.73%)
19% <0%> (-1%)

...in/java/com.github.ambry.store/BlobStoreStats.java
71.05% <0%> (-0.96%)
82% <0%> (-1%)

... and 2 more

Continue to review full report at Codecov.

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


Update required to support terraform 12.*

This modules does not work with Terraform 12.*

Yes. I have the same problem. Error:
Error: Incorrect attribute value type

  on .terraform/modules/storage-bucket/SweetOps-terraform-google-storage-bucket-47591c5/main.tf line 36, in resource "google_storage_bucket_acl" "default":
  36:   role_entity = [

Inappropriate value for attribute "role_entity": element 0: string required.

https://github.com/SweetOps/terraform-google-storage-bucket/pull/13

has been fixed in the new module version https://github.com/SweetOps/terraform-google-storage-bucket/releases/tag/0.3.0


feat: support bundled PostgreSQL

As title, two breaking changes introduced:

tags.bundle won't enable etcd anymore
postgresql will be used for specifying values of the bundled bitnami/postgresql, original one is renamed to postgresqlClient

In that, the next chart version will be 0.2.0.

cc. @WanYixian


Slot props not defined in Typescript

Hello!
I've been tinkering with this library today and it's been great! One thing I've run into is that the packaging that's been done so far doesn't generate typings for the various slotProps that are used; i.e., the slotProps for ListBoxOption is {} instead of correctly picking up the active/selected/disabled props.
Root cause here is spreading the slotProps onto the slot, instead of defining them manually. Looks like if you did something like the below, sveltekit package picks up the types correctly
<slot active={slotProps.active} ...etc/>

Yep, this is due to this issue: https://github.com/sveltejs/language-tools/issues/1291
It's already been fixed, we're just waiting for a new release of the language tools which contains the fix. Not sure when that will be but hopefully it'll be soon.

Ohhh perfect 👍 Glad they fixed it at the tool source! Closing out the issue

Going to leave this open until the fix is released, actually, for reference for other people who run into this.

The fix seems to be released in svelte2tsx-0.4.13. Can someone else confirm this?

<EMAIL_ADDRESS>is still missing its slot definitions. Should we rebuild the project ourselves with the latest svelte2tsx and report back?

This should be fixed in the latest release now.


Run Gemma models with ollama-python

Description of the feature request:
In Gemma cookbooks, we already have a notebook named Run_with_Ollama.ipynb. In this notebook, we find the guide on how to run the Gemma family of models with Ollama. But now recently we have the ollama-python package here with which we can run gemma models too. We can therefore think of this feature request as a sequel to our current notebook.
What problem are you trying to solve with this feature?
The simplest method for integrating Python 3.8+ projects with [Ollama] is to use the Ollama Python library. Thus, using the above-described thought notebook, we can demonstrate how to use the Ollama Python library to run the Gemma family of models. It offers a wide range of options for chatting with models, including standard chat messages and streaming responses. It also offers custom client options and an async client. People can further benefit from these features that are to be displayed in the notebook when working on their projects.
Any other information you'd like to share?
This is the very intro work on this notebook for supporting my thought, which further to be enhanced before making the actual pull request.
Notebook Link: https://colab.research.google.com/drive/1knkSItPepE6V0KhhTimbG14nzHz20Mei?usp=sharing
cc: @windmaple

This sounds good. Can you send a PR based on your intro notebook? Happy to merge it.

PR has been merged, Closing this issue. Thanks, Sitam!


BigInt becomes Number if queried with $queryRaw

Bug description
When I try to $queryRaw() a BigInt column, it becomes Number, resulting in loss of precision for greater number than Number.MAX_SAFE_INTEGER.
// This user has ID of `BigInt(1)` with schema written below.
await prisma.user.create({
  data: { name: 'dummy' },
})

const user1 = (await prisma.user.findFirst()) as User
// This output `true`, as expected
console.log({ user: user1, isSame: user1.id === BigInt(1) })

const [user2] = await prisma.$queryRaw<User[]>`SELECT id, name FROM "User"`
// However, this output `false`, as `typeof user2.id === 'number'`
console.log({ user: user2, isSame: user2.id === BigInt(1) })

How to reproduce

I created a minimal reproduction repository of this issue.
$ git clone https://github.com/yukidaruma/prisma-bigint-rawquery
$ cd prisma-bigint-rawquery
$ npm i
$ docker-compose up -d
$ npx prisma migrate dev --name bigint_rawquery_test
$ npm run dev

> dev
> ts-node ./script.ts

{ user: { id: 1n, name: 'dummy' }, isSame: true } // Using `findFirst()` method
{ user: { id: 1, name: 'dummy' }, isSame: false } // Using `$queryRaw()` method

Expected behavior
I expect $queryRaw() to handle BigInt column properly, returning BigInt value instead of Number value.
{ user: { id: 1n, name: 'dummy' }, isSame: true } // Using `findFirst()` method
{ user: { id: 1n, name: 'dummy' }, isSame: true } // Using `$queryRaw()` method

Prisma information

// Can be found at: https://github.com/yukidaruma/prisma-bigint-rawquery/blob/main/prisma/schema.prisma

datasource db {
  provider = "postgresql"
  url      = env("DATABASE_URL")
}

generator client {
  provider = "prisma-client-js"
}

model User {
  id   BigInt @id @default(autoincrement())
  name String
}

Environment & setup

OS: Windows
Database: PostgreSQL 13.3 (with postgres:13.3-alpine Docker image)
Node.js version: v16.1.0

Prisma Version
Environment variables loaded from prisma\.env
prisma               : 2.23.0
@prisma/client       : 2.23.0
Current platform     : windows
Query Engine         : query-engine adf5e8cba3daf12d456d911d72b6e9418681b28b (at<EMAIL_ADDRESS>Migration Engine     : migration-engine-cli adf5e8cba3daf12d456d911d72b6e9418681b28b (at<EMAIL_ADDRESS>Introspection Engine : introspection-core adf5e8cba3daf12d456d911d72b6e9418681b28b (at<EMAIL_ADDRESS>Format Binary        : prisma-fmt adf5e8cba3daf12d456d911d72b6e9418681b28b (at<EMAIL_ADDRESS>Default Engines Hash : adf5e8cba3daf12d456d911d72b6e9418681b28b
Studio               : 0.393.0

Thanks for the detailed reproduction, I can confirm this bug in 2.25.

I observe the same kind of issue with Decimal e.g.
await prisma.capacity.createMany({
    data: [
      { year: 2016, data: new Decimal(1.1) },
      { year: 2017, data: new Decimal(1.8) }
    ],
  })

  const capacityTotal = await prisma.capacity.aggregate({
    _sum: {
      data: true
    }
  })
  console.log(capacityTotal) // { _sum: { data: 2.9 } }

  const rawCapacityTotal = await prisma.$queryRaw`SELECT SUM(data) FROM public."Capacity"`
  console.log(rawCapacityTotal) // [ { sum: 2.9000000000000004 } ]

Is this a bug that was introduced, or has correct handling of this always been missing from the implementation? It seems like a critical problem for any real level of use of raw queries. It has me very concerned about whether I need to strip out Prisma from my project and use something else.

Is anything in the works for a fix for this?

Too bad I had to find out about this after migrating quite some logic to Prisma in dev. Any chance that there will be a fix for this soon? This is critical.

Hey,
This issue was fixed by https://github.com/prisma/prisma-engines/pull/2847. It will be available in the next release under the improvedQueryRaw feature flag.
Beware that enabling improvedQueryRaw is a breaking change. Release notes will contain information as to how to upgrade (so will the documentation).
Thanks for reporting 🙏