.NET Core strategy is good, but the implementation is totally wrong

HI asp.net team,
I looked at the Web Application from ASP.NET 5 Templates and watched some videos about .NET Core and honestly I am disappointed. I think that your implementation of ASP for multiplatform is wrong. You did not look at how others (like Java or PHP) have solved the  multiplatform development.

About the “dependencies” section in project.json: Why do I have to know for instance that the MVC is in “Microsoft.AspNet.Mvc” package with some version and it must be included if I want MVC? What additional   knowledge gives me this? The “dependencies” section has to have only one line where the .NET Core is (NET_CORE_HOME="path_to_some_version_of_net_core")  and relative paths to third party libs.
About the “frameworks” section in project.json: Why do you need this section? The framework for my app has to be only one. Now there is a package version, a .Net Core version  and a framework  version.  Why did you make it so difficult for managing? Now I have to consider for instance that my app supports   .Net Core 1, MVC version xxxx (and not version zzzz) and only framework dnx451. Isn’t it simpler to say that my app supports .Net Core 1?
Why do I need Kestrel? Kestrel is a cross-platform web server. Can I use it in production? I suppose no (maybe in two years).  What I need is modules and a config files for other well-known cross-platform web servers (like Apache).
Why all the .Net Core packages and runtimes are included when I publish an app? Suppose I have 20 apps on my server. I would have 20 copies of the same .Net Core environment?! This is a totally wrong decision.
About the development process: How are you going to explain to beginner developers the adding/configuring services and DI patterns? You have made the development so difficult to understand, that a beginner has to first learn other frameworks and then eventually .Net Core. Your “Hello World” sample is about 40 lines of not understandable code. In PHP it is ONLY one line. What would a beginner prefer?!

Niki

About the “dependencies” section in project.json: Why do I have to know for instance that the MVC is in “Microsoft.AspNet.Mvc” package with some version and it must be included if I want MVC? What additional knowledge gives me this? The “dependencies” section has to have only one line where the .NET Core is (NET_CORE_HOME=) and relative paths to third party libs.

how would you know which dependencies to fetch from NuGet if you don't specify it somewhere?

About the “frameworks” section in project.json: Why do you need this section? The framework for my app has to be only one. Now there is a package version, a .Net Core version and a framework version. Why did you make it so difficult for managing? Now I have to consider for instance that my app supports .Net Core 1, MVC version xxxx (and not version zzzz) and only framework dnx451. Isn’t it simpler to say that my app supports .Net Core 1?

the upcoming netstandard solves that IMO.

Why do I need Kestrel? Kestrel is a cross-platform web server. Can I use it in production? I suppose no (maybe in two years). What I need is modules and a config files for other well-known cross-platform web servers (like Apache).

Kestrel is based on libuv which is also used in node.js. Kestrel itself has the same maturity level as ASP.NET - both will be V1 products. So the decision is up to you whether you generally want to use V1 products.

Why all the .Net Core packages and runtimes are included when I publish an app? Suppose I have 20 apps on my server. I would have 20 copies of the same .Net Core environment?! This is a totally wrong decision.

AFAIK there will be (or maybe already is - I'm not sure) a way to install a predefined set of .NET Core assemblies on a server. dotnet/cli is the place to look for that.

About the development process: How are you going to explain to beginner developers the adding/configuring services and DI patterns? Your “Hello World” sample is about 40 lines of not understandable code. In PHP it is ONLY one line. What would the beginner prefer?!

check cli-samples for minimal samples.

Why all the .Net Core packages and runtimes are included when I publish an app? Suppose I have 20 apps on my server. I would have 20 copies of the same .Net Core environment?! This is a totally wrong decision.

I'll ask the reverse question. Let's say you have a server with 20 web apps hosted on it developed at a different time. You want to create an app with the latest framework.
The last deployment model followed the one you mentioned. One global framework. Issue? The last framework has lots of changes and you don't want to re-test all the 20 other apps to see if something broke. Your company doesn't have the budget to do that.
This new model will allow you to just ship the framework with the apps. Space is cheap. Managing multiple frameworks is expensive.

Why do I need Kestrel? Kestrel is a cross-platform web server. Can I use it in production? I suppose no (maybe in two years). What I need is modules and a config files for other well-known cross-platform web servers (like Apache).

You don't NEED Kestrel. It's just that for V1 it's the only one you have. As for sample configurations, there is already requests to have those. If yours isn't there, please do open an issue.
Nginx: https://github.com/aspnet/Home/issues/1170

About the development process: How are you going to explain to beginner developers the adding/configuring services and DI patterns? Your “Hello World” sample is about 40 lines of not understandable code. In PHP it is ONLY one line. What would the beginner prefer?!

Proper documentation and tutorials are currently missing. Not surprising. Everyone has been focused on either delivering RC2 or waiting for the RC2 before doing anything major.
This will come in time but the main solution is going to be plenty of templates. Dependency injection takes 5 minutes to explain in person. One good article when you are online.

About the “frameworks” section in project.json: Why do you need this section? The framework for my app has to be only one. Now there is a package version, a .Net Core version and a framework version. Why did you make it so difficult for managing? Now I have to consider for instance that my app supports .Net Core 1, MVC version xxxx (and not version zzzz) and only framework dnx451. Isn’t it simpler to say that my app supports .Net Core 1?

Compared to what?
NodeJS has package version, runtime version, and flags on which to run.
Java has runtime version, framework versions (Sparks/Grails/etc...) and JAR versions.
Let's face it. You have multiple parts. You have a runtime, you have a framework and then you have packages. dependencies represent the packages, frameworks represent the supported framework version and the runtimes are what is going to be running your app. That runtime needs to know if your application is actually going to be able to run on it.
Am I missing something? Please correct me if I'm wrong. If other frameworks are making it implicit, I'd rather have it explicit. Less chance to find out that you were trying to run new packages on an outdated runtime or have to copy/paste dependencies or my app just assume that it knows better.
Every framework is going to run into those issues. Runtime version. Framework version. Package version.

About the “dependencies” section in project.json: Why do I have to know for instance that the MVC is in “Microsoft.AspNet.Mvc” package with some version and it must be included if I want MVC? What additional knowledge gives me this? The “dependencies” section has to have only one line where the .NET Core is (NET_CORE_HOME="path_to_some_version_of_net_core") and relative paths to third party libs.

And that assumes that you only have one runtime and one framework and that your runtime is managing those dependencies by whether they exist in a specified path or not.
Packages are managed by Nuget which they are going to install in a standard location. Easier to deploy and easier to maintain. Those packages are zipped and extracted at a location on disk which are then going to be able to be picked up by .NET Core and loaded up.
The runtime will be included in PATH to be sure that we can run dotnet or dnx (depending on framework version you are using). If you need another version of the framework, you can upgrade/install it directly from the command line.
As for knowing which version of AspNet.MVC? What about upgrading the framework? You need to specify it somewhere if you want to upgrade it. No?

I want to use .Net Core as simple as possible. I think that .Net Core should be a simple zip file with all  first party libs (Mvc, StaticFiles, Razor, …). I don’t care which version is MVC or Razor.  Next step is to extract the framework somewhere on my server and to say to my app to use it (NET_CORE_HOME="path_to_some_version_of_net_core"). That’s it. No package version, no framework versions (dnxcore50), no global framework. With this solution I can VERY easily test my app with every version of .Net Core. No one ships  it’s framework with the apps. See PHP or Java or Python or Perl? No one. You will be the first (and the last).
About the Kestrel. For me it is useless, because it could not be used in production. It does not have security, load balancing... A better approach is to be developed modules for other web servers (like PHP has done for Apache).
About the development process: Let’s see what we have in one simple web application in .Net Core: 1. Dependency injection. 2. MVC as  framework. 3. Adding/configuring services. 4. Lambda expressions.  You can’t explain them for 5 minutes.  The beginner has to know all this in order to test something. In PHP you just start working on your project. I ask again. What would the beginner prefer?
I think that Nuget  should be a simple package manager for importing third party libs to my project, not to the .Net Core.  Nuget should download a lib, installs it somewhere relative to my project and makes some configuration work (in project.json). That’s it.  Why did you make .Net Core to depend so much from Nuget. What if I don’t want to use it.

I don’t care which version is MVC or Razor.

Now you don't. Until you want the next version. A user shouldn't have to upgrade framework if it just wants the next version of the web tools. It's why they are there. They are shipped outside the actual runtime. They are a package with their own release cycle.

No one ships it’s framework with the apps. See PHP or Java or Python or Perl? No one. You will be the first (and the last).

True. As mentioned before, optimizations will be in place. My bet? It will simplify deployment and you are literally the first person I hear that complaint about that specific feature.

About the Kestrel. For me it is useless, because it could not be used in production. It does not have security, load balancing...

It's meant to host the app but not to be a production web server. Even in the Windows world, we are not exposing Kestrel directly to the web. IIS will take care of that. On your end? You should use NGINX to secure your front-end.

About the development process: Let’s see what we have in one simple web application in .Net Core: 1. Dependency injection. 2. MVC as framework. 3. Adding/configuring services. 4. Lambda expressions. You can’t explain them for 5 minutes. The beginner has to know all this in order to test something. In PHP you just start working on your project. I ask again. What would the beginner prefer?

You are confusing some concept. 1 and 3 is dependency injection which is your initial point. Now you are adding MVC which is pretty similar to what is already existing. It's easy to ramp up and easy to understand. Point 4 is about C# not ASP.NET Core. Is your point still valid for nodejs functions, Java arrow methods? Because they are very similar.
ASP.NET Core with the starter template? I can get a student up and running within the hour and have him productive and create APIs backed by database within a day of work.

Why did you make .Net Core to depend so much from Nuget. What if I don’t want to use it.

What do you want to use instead? Copy/pasting of dependencies is prone to error and people never update them once they are included. That is a security risk. What do you recommend?

I don't know where this "kestrel cannot be used in production" comes from, since beta8 I guess it is how you run a .net core app. As far as I know, the team is committed to make kestrel a first class citizen in the modern web. However kestrel is very node alike. It is not meant to be a general purpose web server. For that matter you can use IIS, Nginx or Apache to front your kestrel exactly as you would do with node.

I think the questions fundamentally misses the point of what .NET Core and ASP.NET Core are. There are many similarities in .NET Core to the way Ruby and Node works, for example. I recommend @nikigter invest some time to understand how these technologies all work, these questions are answered in detail in the docs, videos, etc. I don't see a point to discuss this all over again.

I agree with @giggio.
And since we're so close to RC2 and a month away from RTM, I suggest we close this issue. Nice discussion but "implementation is totally wrong" cannot be resolved. Everything mentioned can be flagged as "by design".

Please, don’t understand me wrong, but the model where you ship the framework with the apps and that packages are managed by Nuget is not good.
If you compare a development and installation process of a Web application written in PHP and .Net Core the winner will be definitely PHP:

In PHP you just start working on your project. In .Net Core you have to think about package versions, Nuget, frameworks, project.json configurations staff
In PHP the whole  web application contains 10 files (for instance). In .Net Core you have your project (let’s say 5  files) plus 713 files and 842 folders for the framework! This is crazy.

I think that projects in .Net Core are more difficult and expensive to maintain. A lot of small and middle companies are not going to use it. That’s it.
Niki
P.S.
I agree with @giggio and @MaximRouiller too. Let’s close this issue.

Comparing a new model in pre-release candidate to a legacy programming language that is rooted in completely unrelated technology doesn't seem like the best correlation of usefulness.

@ciel maybe you are right. But. The more I deep drive into .Net Core the more I realize that for the asp team will be very difficult (or impossible) to convince web developers to use it as a primary development framework.

http://blog.codinghorror.com/php-sucks-but-it-doesnt-matter/
Jeff Atwood (stackoverflow) seems to disagree with you, and I trust his judgement a great deal more.
.NET core doesn't need to convince programmers of anything. It's not complicated or difficult. Everything you outline as a con is just the usual stress of edge development. You see exposed nerves because you're working on internals and guts. The final product will have 0 of that. Even package management will be a non-issue.

@ciel  in the last post I pointed   PHP only as a sample. That’s true for every other.
Let’s say you have 10 different simple projects for processing log files. In other languages (not only in PHP) you would have 10 source files. In .Net Core there would be 7130 files and 8420 folders because of the frameworks!  Correct me if I am wrong. Guess what would be the customer’s reaction and decision when he sees so many files and folders on his server. He doesn’t care that the asp team has created the best ever development framework and how simple it is to work with it. @MaximRouiller I don’t see here simplicity. The problem is that you are talking about only from the development’s point of view. See how the final .Net Core project looks like  from the customer’s point of view.

You're completely wrong.
On May 16, 2016 1:01 AM, "nikigter"<EMAIL_ADDRESS>wrote:

@ciel https://github.com/ciel in the last post I pointed PHP only as a
sample. That’s true for every other.
Let’s say you have 10 different simple projects for processing log files.
In other languages (not only in PHP) you would have 10 source files. In
.Net Core there would be 7130 files and 8420 folders because of the
frameworks! Correct me if I am wrong. Guess what would be the customer’s
reaction and decision when he sees so many files and folders on his server.
He doesn’t care that the asp team has created the best ever development
framework and how simple it is to work with it. @MaximRouiller
https://github.com/MaximRouiller I don’t see here simplicity. The
problem is that you are talking about only from the development’s point of
view. See how the final .Net Core project looks like from the customer’s
point of view.
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/aspnet/Home/issues/1432#issuecomment-219354923

Alright... if I remember correctly... multiple runtimes can be installed on the machine. Meaning multiple frameworks. That also mean a different amount of packages. All of those being centralized.
If I said anything different than that, I'm sorry. I was talking on the point of view of an older beta. But now, everything is centralized. Multiple apps with different runtimes will need a server with all those runtimes installed (including their packages).
Yes, it's a lot of files. But let project ourselves 5 years in the future. It will allow an insane amount of variation on a single server. Need to host 25 applications each built slightly differently on 5 different version of the framework? You only need a server with those 5 runtimes installed.
From my client's point of view? I don't need to pay to have regression testing on those 25 applications. I don't need to have IT guys testing multiple frameworks install to ensure that nothing was broken. My client is saving money. My client won't care that I use a tad more HD space. He will care that he doesn't have to spawn 5 different VMs (one per runtime) just to ensure nothing was broken.
That my friend, is value.

Multiple apps won't need different servers. Packages and runtimes are project-local, not machine-local.  You don't need your targeted files in the GAC anymore. There is no regression testing needed, and there is literally nothing about the .NET Core strategy that is like you're claiming.
Your application has a folder. Inside of it are the dependencies. It's really that simple.

Why would you need your IT staff to test multiple frameworks? Just because you can write cross-platform doesn't mean you have to. Make your application and target a platform, then market it for that platform. You simply have options other than 'Windows' now.
You have to do all that testing in php too, you know. php treats Windows servers very differently than Apache servers.
I feel like your argument is grasping at straws and pretty uninformed. Have you actually built and run a .NET Core application?

Why in the world would one client be running your application in 5 places, on 5 different platforms? That's such a convoluted strawman argument I can't even think of a rebuttal. That will literally never happen.

@ciel I was talking in past experience of managing hosting .NET servers for clients.
If you are to upgrade a framework, you'll need to ensure that nothing is broken in either your application you developed for your client or for anything you bought from different vendors. So any upgrade to an existing machine is a pain. The new model solve that.
I've never talked about running PHP. ;-)
I've built the apps. I haven't had anything running in the past few months so... yeah. Memory fades and my grasp on some things might be off a bit.
Not 5 places in 5 platforms. One server. 25 web apps built a different time with different runtime requirements. That is very common. Yet again, ASP.NET Core solves that issue.

If we're running a containerised microservices architecture, where you redeploy apps instead of updating them, why would you have so much technical debt in the form of different runtime environments?
Aren't we moving rapidly away from this monolithic model that requires we support multiple runtime versions? One runtime, agile microservices, containers.
Hopefully, we will be able to deploy .net core using a range of package managers, including apt, yum, snappy, docker. And, be able to use PostgreSQL with NGINX and a host of other apps in our open source stacks.
Maybe we want to use BuildRoot to create a 10MB linux OS to run in a Docker container (or Rkt, or LXD) that has all the components we need just to run our app. Add .net core to that tiny container and you've got something quite different to the typical .net environment running under Windows. Deploy your C# app into that and be ready to destroy the container when you want to update the code.
One container, one runtime, one app, one version instance. Or, in the case of LXD (a full system container): one container, one runtime, one app stack, one version instance. It's an entirely new world for .net on Linux.

@nikigter You are comparing pho 'echo "hello" vs whole modern web framework asp.net core. Maybe try to compare it to zend or simphony php framework, then you will understand that .net core is amazing and very productive.

For one think I must thank the OP, that I laughed so hard...

Sorry to revive a dead thread but this has to be said on top of all this (for people who happen across this thread it may be helpful).
The OP is making comparisons between a framework (.net core and asp.net core) against languages (PHP is the main example used here). In PHP you could also use frameworks such as laravel and get just as many files and "complications", just as you can use one or two files to create a base c# application. Do not confuse the two when making comparisons, it's somewhat like apples and pairs.


Attach files to email from Box.com

This should be added to the Mail helper method, where the file that is attached can be sitting in Box.com storage.
In most circumstances, for Spam detection reasons, it would be better to link to the file- however, there are legitimate use cases for attaching a file. Example: Receipt PDFs

/request

Thank you @JSmurf! It's all yours :)

Since there has been no activity on this issue since March 1, 2020, we are closing this issue. Please feel free to reopen or create a new issue if you still require assistance. Thank you!


Update introspection and debugging page with deployments

Only nits; LGTM otherwise. Feel free to apply the tag once comments are addressed.

Thanks Janet.

Docs look good to me. Thanks for the tech review, Patrick. :)


Replacing ForneyLab methods with native Julia methods if possible

It is a good practice to stick to native Julia methods as much as possible so that the current inference algorithms in ForneyLab can easily interface with other Julia packages. For example: cholinv method of ForneyLab sometimes prevent the flow of Flux variables through FFG but when it is replaced with inv() method of Julia, the problem disappears.

While I agree that it is better to reuse native Julia methods, simply replacing cholinv with inv is not the preferred solution here. Replacing will lead to a significant performance decrease, because larger matrices can lose PD-ness due to numeric instabilities. A better solution was proposed in #30, where PD matrices are assigned a separate abstraction.

Will close in favor of #30


[spark] Make show table extended compatible with spark3.4-

Purpose
Make show table extended compatible with spark3.4-
Tests

API and Format

Documentation

+1 and thanks, i forgot to cover the lower spark version. @Zouxxyy


fix(models): Add pk to make model  serializable by django

To make data class serializable using drf model serializers for identity overrides, else we can't use model serializer and all the good things that comes with it, like: verifying the feature exists or not etc.

I don't think we should be using ModelSerializer for engine models. They are not Django models. I think we'd end up with unexpected behaviour if we try and shoehorn our models into these serializers.


Feature Request:  Advanced Settings

Hi,  Thanks for adding this Extension to 3D Slicer.
Under Outputs could you create GUI for these two options?
--statistics: This will generate a file statistics.json with volume (in mm³) and mean intensity of each class.
--radiomics: This will generate a file statistics_radiomics.json with radiomics features of each class. You have to install pyradiomics to use this (pip install pyradiomics).
As per https://github.com/wasserth/TotalSegmentator.
Thanks for all of your insights over the years.  Brad

Thanks for the suggestions. These features of TotalSegmentator are not exposed in Slicer because much improved versions of these features are already available:

You can get more detailed statistics in Segment Statistics module.
You can run radiomics using SlicerRadiomics extension.


Auto-generate gogo annotations for api_v3

Which problem is this PR solving?

Part of #4150

Description of the changes

Upgrade to new IDL that removes gogo annotation from api_v3/query.proto
Add a pre-processor to inject annotations during build
Fix tests
Add start/end time to GetTrace test (which was failing in #6228)

How was this change tested?

Checklist

[ ] I have read https://github.com/jaegertracing/jaeger/blob/master/CONTRIBUTING_GUIDELINES.md
[ ] I have signed all commits
[ ] I have added unit tests for the new functionality
[ ] I have run lint and test steps successfully

for jaeger: make lint test
for jaeger-ui: yarn lint and yarn test

Test this change in this draft PR: https://github.com/jaegertracing/jaeger/pull/6237
Moving gogoproto related lines out is a great idea, since original proto file could be used by other programming languages

@rim99 please approve this change and I will merge it.

@yurishkuro woud you like to regenerate file cmd/query/app/internal/api_v3/query_service.pb.go after idl renaming num_traces -> search_depth? Or it will be handled by some automated flows?
Because to manage all test passed, I had to make this change:
https://github.com/jaegertracing/jaeger/pull/6237/commits/d6eeed8f8d36381a77e1c283fc419dafd32b8b06#diff-9f4e53eb6033f5e7d7af97a3fd7328e26bd3e5e948e01276cb85b3abfc0b2432

@yurishkuro woud you like to regenerate file cmd/query/app/internal/api_v3/query_service.pb.go after idl renaming num_traces -> search_depth? Or it will be handled by some automated flows?
Because to manage all test passed, I had to make this change:
https://github.com/jaegertracing/jaeger/pull/6237/commits/d6eeed8f8d36381a77e1c283fc419dafd32b8b06#diff-9f4e53eb6033f5e7d7af97a3fd7328e26bd3e5e948e01276cb85b3abfc0b2432


Web client login - some people can use their ranger handle, some people need their full email address

"I wanted to let you know that I just went to login to the IMS to kick the tires, and the login system asks for your Ranger Handle, but my handle wouldn’t work to login. I had to use the email address I use to login to the Ranger Clubhouse to make that work. So perhaps the login page should ask for an email address, as it does on the clubhouse?" -Tribeca
"The online training login was a little tricky.  It asked for your ranger handle and password, but instead you needed to enter your email and password." -Fortress
Note that just my login worked, I didn't need to enter an email.  -Librarian

My handle works as 'Zeitgeist' but not as 'zeitgeist'.   My email address works as<EMAIL_ADDRESS>and not as<EMAIL_ADDRESS> maybe we just tell folks that capitalization matters.

capitalization matters.


Error loading Keras model with custom layer in JSON format

Please make sure that this is a bug. As per our GitHub Policy, we only address code/doc bugs, performance issues, feature requests and build/installation issues on GitHub. tag:bug_template
System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): No
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): OSX Mojave
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device:
TensorFlow installed from (source or binary): binary
TensorFlow version (use command below): 1.12.0
Python version: 2.7
Bazel version (if compiling from source):
GCC/Compiler version (if compiling from source):
CUDA/cuDNN version: N/A
GPU model and memory: N/A

You can collect some of this information using our environment capture script
You can also obtain the TensorFlow version with
python -c "import tensorflow as tf; print(tf.GIT_VERSION, tf.VERSION)"
Describe the current behavior
I was following the Keras tutorial on TF website and I was trying out my custom layer, which was basically the same as in the tutorial.
class MyLayer(layers.Layer):

  def __init__(self, output_dim, **kwargs):
    self.output_dim = output_dim
    super(MyLayer, self).__init__(**kwargs)

  def build(self, input_shape):
    shape = tf.TensorShape((input_shape[1], self.output_dim))
    # Create a trainable weight variable for this layer.
    self.kernel = self.add_weight(name='kernel',
                                  shape=shape,
                                  initializer='uniform',
                                  trainable=True)
    # Be sure to call this at the end
    super(MyLayer, self).build(input_shape)

  def call(self, inputs):
    return tf.matmul(inputs, self.kernel)

  def compute_output_shape(self, input_shape):
    shape = tf.TensorShape(input_shape).as_list()
    shape[-1] = self.output_dim
    return tf.TensorShape(shape)

  def get_config(self):
    base_config = super(MyLayer, self).get_config()
    base_config['output_dim'] = self.output_dim
    return base_config

  @classmethod
  def from_config(cls, config):
    return cls(**config)

model = tf.keras.Sequential([
    MyLayer(10),
    layers.Activation('softmax')])

# The compile step specifies the training configuration
model.compile(optimizer=tf.train.RMSPropOptimizer(0.001),
              loss='categorical_crossentropy',
              metrics=['accuracy'])

dataset = get_dataset(*get_train_data()) # custom functions

val_dataset = get_dataset(*get_val_data())

test_dataset = get_dataset(*get_test_data())

# Trains for 5 epochs.
model.fit(dataset, steps_per_epoch=30, epochs=5, callbacks=callbacks, validation_data=val_dataset)
model.evaluate(test_dataset, steps=30)

model.save_weights('./weights/model1')
json_string = model.to_json()

pprint(json.loads(json_string))

fresh_model = tf.keras.models.model_from_json(json_string, custom_objects={'MyLayer':MyLayer})
fresh_model.load_weights('./weights/model1')

fresh_model.compile(optimizer=tf.train.RMSPropOptimizer(0.001),
              loss='categorical_crossentropy',
              metrics=['accuracy'])

fresh_model.evaluate(test_dataset, steps=30)

However, I got the following error when I tried to run it and I could not find any related issues online.
Traceback (most recent call last):
  File "/Users/ray/miniconda3/envs/ml/lib/python2.7/site-packages/IPython/core/interactiveshell.py", line 2878, in run_code
    exec(code_obj, self.user_global_ns, self.user_ns)
  File "<ipython-input-2-67ef4d13fb1e>", line 1, in <module>
    runfile('/Users/ray/WorkSpace/machinelearning/tensorflow-learn/custom_layer.py', wdir='/Users/ray/WorkSpace/machinelearning/tensorflow-learn')
  File "/Applications/PyCharm.app/Contents/helpers/pydev/_pydev_bundle/pydev_umd.py", line 197, in runfile
    pydev_imports.execfile(filename, global_vars, local_vars)  # execute the script
  File "/Users/ray/WorkSpace/machinelearning/tensorflow-learn/custom_layer.py", line 86, in <module>
    metrics=['accuracy'])
  File "/Users/ray/miniconda3/envs/ml/lib/python2.7/site-packages/tensorflow/python/training/checkpointable/base.py", line 474, in _method_wrapper
    method(self, *args, **kwargs)
  File "/Users/ray/miniconda3/envs/ml/lib/python2.7/site-packages/tensorflow/python/keras/engine/training.py", line 600, in compile
    skip_target_weighing_indices)
  File "/Users/ray/miniconda3/envs/ml/lib/python2.7/site-packages/tensorflow/python/keras/engine/training.py", line 134, in _set_sample_weight_attributes
    self.output_names, sample_weight_mode, skip_target_weighing_indices)
AttributeError: 'Sequential' object has no attribute 'output_names'

Describe the expected behavior
If the tutorial was correct, the code should run without any bugs and the evaluation results should be the same after loading.
Code to reproduce the issue
Provide a reproducible test case that is the bare minimum necessary to generate the problem.
Other info / logs
Include any logs or source code that would be helpful to diagnose the problem. If including tracebacks, please include the full traceback. Large logs and files should be attached.

@larry0123du Can you provide the minimal full code sample that could reproduce the error? The code you pasted is not complete with get_dataset and callbacks missing.

Sure, my get_dataset is as follows:
import tensorflow as tf
import numpy as np

def get_train_data():
    # training data
    data = np.random.random((1000, 32))

    data = tf.cast(data, tf.float32)

    labels = np.random.random((1000, 10))

    labels = tf.cast(labels, tf.float32)

    return data, labels

def get_val_data():
    # validation data
    val_data = np.random.random((100, 32))

    val_data = tf.cast(val_data, tf.float32)

    val_labels = np.random.random((100, 10))

    val_labels = tf.cast(val_labels, tf.float32)

    return val_data, val_labels

def get_test_data():
    # test data
    test_data = np.random.random((100, 32))

    test_data = tf.cast(test_data, tf.float32)

    test_labels = np.random.random((100, 10))

    test_labels = tf.cast(test_labels, tf.float32)

    return test_data, test_labels

def get_dataset(data, labels):
    dataset = tf.data.Dataset.from_tensor_slices((data, labels))
    dataset = dataset.batch(32)
    dataset = dataset.repeat()
    return dataset

and the callback is
callbacks = [
    tf.keras.callbacks.EarlyStopping(patience=2, monitor='val_loss')
]

Let me know if you need anything else!


React mixin immutable support

Can you add support of immutablejs objects?

Alas, no, I do not use React.

Ok, I will try implement that later or add example with immutable data

Dear all! Meteor is moved to the separate repository. If this issue is still actual, please create it there once again and this rojects needs a maintainer:

ES2015+: https://github.com/SortableJS/react-sortablejs 🆘
mixin: https://github.com/SortableJS/react-mixin-sortablejs


[OAP-cache][OAP-1811][POAE7-486]add sbin folder

1:start_plasma_server.sh
2:start_redis_server.sh
What changes were proposed in this pull request?
(Please fill in changes proposed in this fix)
How was this patch tested?
(Please explain how this patch was tested. E.g. unit tests, integration tests, manual tests)
(If this patch involves UI changes, please attach a screenshot; otherwise, remove this)

@jikunshang @yma11 @winningsix @yeyuqiang Please take a review ,thanks.

#1811


chart does not update when select token on the right list.

chart update is not working.
When new token selected.

solved


Remove table of contents from README

Github now has autogenerated table of contents:

Should we remove manual one to avoid maintaining it? Or it's nicer to have both?

Uuuu, github steps up their game 😄  I'm neutral. I guess we can keep it for now, doesn't seem to take much of effort to maintain it at this points and looks pretty good :)

Alright, let's keep it. I actually just noticed it, it's not obvious it's there.


Updating sha256 of python and scala

Checklist

[x] I have filed an issue about this change and discussed potential changes with the maintainers.
[ ] I have received the approval from the maintainers to make this change.
[x] This is not a stylistic, refactoring, or cleanup change.

Issue number: #2343
Description of this change
Build is failing on
ERROR: /home/builduser/workspace/intellij-plugins-ci/intellij-check-and-notify-plugins-state/intellij/third_party/python/BUILD:7:13: //third_party/python:python_internal depends on @python_2020_1//:python in repository @python_2020_1 which failed to fetch. no such package '@python_2020_1//': java.io.IOException: Error downloading [https://plugins.jetbrains.com/files/7322/88054/python-ce-201.7846.93.zip] to /home/builduser/.cache/bazel/_bazel_builduser/1c03a469a8e977776a02273bfa495afd/external/python_2020_1/python-ce-201.7846.93.zip: Checksum was 648438d26e85072f90a62f9f5b9f9b983a49f3cb752e87d01f915cb6a03644b9 but wanted a3d7217794619dfa0432d922218dd875f49658939c6e5a54c01f99b333d35e12
and
Error in download_and_extract: java.io.IOException: Error downloading [https://plugins.jetbrains.com/files/1347/76628/scala-intellij-bin-2020.1.7.zip] to /home/builduser/.cache/bazel/_bazel_builduser/1c03a469a8e977776a02273bfa495afd/external/scala_2020_1/scala-intellij-bin-2020.1.7.zip: Checksum was 626b9c9bc2f90d64498524bd7138558edacf50bbc72a02f882401118a1fc1403 but wanted bbc019e7cde3baf21b11f0abde21f42d4606ca5ca9100b3bdace3292f70cceef
Manually downloading from those URL reveals that the content has indeed changed :-/

@philwo  - question about BK - do we run on ephemeral machines or do builds share the same repository cache? If it's the latter - this can explain why builds are not failing in master right now.

@chaoren @vsiva ... any ack?

do we run on ephemeral machines or do builds share the same repository cache

Linux machines are ephemeral and only run one CI job, then are deleted. macOS machines are persistent and the repository cache is being kept and reused over time.

Thanks for this PR. I imported it to our internal source of truth. #2371 is the re-export.
I think I can clarify the build situation: We only build the intellij-beta and intellij-ue-beta versions on CI. Those version variables currently point to 2020.3 for which we already fixed the checksums when the build failed.

Closing this PR as #2371 was merged.


[WIP] Update the ROS2 cpp code generator to define nodes as components

@ipa-hsd is this still WIP?


Update required dependencies to latest version

Update required dependencies to the latest version

Is there any reason for this not to be merged in yet?

This is already fixed


Streamline _create_overpass_query method

Hello, first time contributor here
Is your feature proposal related to a problem?
Yes. The _create_overpass_query method creates a larger than necessary query string. In it are three main issues that cause a significant amount of redundancy: 1) each tag-value pair is explicitly called one by one, 2) each tag-value is called three times (node, way, relation), and 3) the polygon coordinate string is replicated for every tag-value pair.
Describe the solution you'd like to propose
The following changes address the above three deficiencies:

multiple values for the same tag should be combined together into a single component, as per Overpass documentation https://dev.overpass-api.de/overpass-doc/en/criteria/per_tag.html
Example:
way[highway~"^(trunk|primary|secondary|tertiary)$"]

The node, way, and relation trifecta can be replaced with the single query statement nwr as per Overpass issue https://github.com/drolbr/Overpass-API/issues/453#issuecomment-378718323

The polygon can be globalized and referenced multiple times, rather than explicitly sending the same coordinate sequence for each query component, as per Overpass documentation https://dev.overpass-api.de/overpass-doc/en/criteria/union.html
Example:
nwr({{bbox}})->.all;
(
nwr.all[amenity=atm];
nwr.all[atm=yes];
);

A clear and concise description of what you want to change in OSMnx and how you plan to implement it.
Replace the guts of the _create_overpass_query method to take advantage of the above. On an example query of mine the query size shrank from 20,625 bytes to 770 bytes (see attached files pre.txt and post.txt). osmnx lint and unit tests all pass in my forked copy of the repo.
pre.txt
post.txt
Provide a complete minimal working example of code to demonstrate your use case. Add any additional code snippets, context, or screenshots about the feature proposal to demonstrate its importance and feasibility.
def _create_overpass_query(polygon_coord_str, tags):
    """
    Create an overpass query string based on passed tags.

    Parameters
    ----------
    polygon_coord_str : list
        list of lat lng coordinates
    tags : dict
        dict of tags used for finding elements in the selected area

    Returns
    -------
    query : string
    """
    # create overpass settings string
    overpass_settings = _make_overpass_settings()

    # make sure every value in dict is bool, str, or list of str
    error_msg = "tags must be a dict with values of bool, str, or list of str"
    if not isinstance(tags, dict):  # pragma: no cover
        raise TypeError(error_msg)

    # make sure every value in dict is bool, str, or list of str
    error_msg = "tags must be a dict with values of bool, str, or list of str"
    if not isinstance(tags, dict):  # pragma: no cover
        raise TypeError(error_msg)

    # add node/way/relation query components one at a time
    components = []
    for key, value in tags.items():

        if isinstance(value, bool) and value:
            # if bool (ie, True) just pass the key, no value
            tag_str = f"[{key}]"

        elif isinstance(value, str):
            tag_str = f"[{key}='{value}']"

        elif isinstance(value, list):
            if not all(isinstance(s, str) for s in value):  # pragma: no cover
                raise TypeError(error_msg)

            if len(value) == 1:
                tag_str = f"[{key}='{value[0]}']"
            else:
                tag_str = "[{0}~'^({1})$']".format(key, "|".join(value))

        else:  # pragma: no cover
            raise TypeError(error_msg)

        components.append(f"nwr.all{tag_str};")

    # finalize query and return
    components = "".join(components)
    query = f"{overpass_settings};nwr(poly:'{polygon_coord_str}')->.all;({components});(._;>;);out;"

    return query

Hello, for this function, is there any way to request all OSM data in the given poly?
I am looking for a way to just ask for all OSM data in a given polygon in the current release version, but the tags seems mandatory, and if an empty dict then there is no error, but no data is returned.

I wrote a similar patch, in issue https://github.com/gboeing/osmnx/issues/755, but unfortunately I didn't have time to fully test it. The OSMnx author was concerned that the query might not be a 1-to-1 match:

I'd be open to reviewing a PR from you for this. Should also demonstrate that you're getting a perfect match between the results from the current method vs the new one you're proposing, across several test queries.

He's likely going to have a similar concern about this one. Could you provide some testing that you get the same results across old and new queries?

@anditi-lidar I don't see why it wouldn't be possible to allow an empty tags dictionary to indicate all OSM data be retrieved. I'll take a look.
@nickodell Ah, I didn't see that as I only looked at open issues prior to opening this one. I thought your brief analysis was well done. I would be willing to do a side-by-side comparison of the revised query in the context of my own validation dataset of about 30k POI in USA and Japan. IMO, using 'nwr' instead of the three separate node/way/relation query is a no-brainer and shouldn't require as much validation as the area and tag-value compression.

@derinwalters
That would be great.

@derinwalters thanks for suggesting this. And thanks @nickodell for your comments. Yes I'm open to a reviewing a PR. The key thing, as Nick mentioned, will be comprehensively demonstrating there is no change in query results from the old way to the new way. For the end user we must guarantee consistent deterministic results between versions. Let me know if you're willing to take this enhancement on. I'm grateful for your proposal.

Also, please note that I'm on international travel with limited connectivity, so please forgive sporadically long response times.

@gboeing yessir, I'll take a crack at it

One thought that occurred to me and is worth documenting here: this change will cause cache invalidation upon release. This seems acceptable but worth noting.

Alrighty, I ran two sets of queries covering 4500 km^2 around Tokyo and San Francisco with 1) osmnx 1.2.2 and 2) the proposed change to _create_overpass_query above. Each returned 463488 elements (45MB) and the diff between the raw json responses was clean.
Query string byte size down from 1MB to 41KB
Query-to-response time delta of downloader.py{overpass_request} down from 5711 seconds to 4450 seconds. Yes, I know this also includes delay due to waiting on Overpass slot availability, but seems like a reasonable enough proxy.
Raw queries and json responses, along with a diff of the responses, attached as Archive.zip
Archive.zip
Many of the queries looked like this:
out:json][timeout:1800];nwr(poly:'36.856085 -118.345039 36.947997 -118.345039 36.947997 -118.459417 36.856085 -118.459417 36.856085 -118.345039')->.all;(nwr.all[aerialway~'^(cable_car|chair_lift|gondola|mixed_lift|station)$'];nwr.all[amenity~'^(cafe|community_centre|drinking_water|place_of_worship|public_bath|ranger_station|restaurant|shelter|shower|toilets|vending_machine|water_point|watering_place)$'];nwr.all[ford='yes'];nwr.all[geological~'^(volcanic_caldera_rim|volcanic_vent)$'];nwr.all[highway~'^(bus_stop|rest_area|services|trailhead)$'];nwr.all[leisure='park'];nwr.all[mountain_pass='yes'];nwr.all[natural~'^(beach|cave_entrance|fumarole|geyser|hot_spring|peak|saddle|spring|volcano|water|wetland)$'];nwr.all[public_transport='station'];nwr.all[shop='convenience'];nwr.all[tourism~'^(alpine_hut|camp_site|guest_house|hostel|hotel|information|viewpoint|wilderness_hut)$'];nwr.all[water~'^(lagoon|lake|pond|reservoir|river)$'];nwr.all[waterway~'^(dam|river|stream|waterfall)$'];);(._;>;);out;

| OSMNX 1.2.2 |   |   |   | PROPOSED |   |   |
-- | -- | -- | -- | -- | -- | -- | -- | --
MGRS | Qbytes | Qseconds | Relems | Rbytes | Qbytes | Qseconds | Relems | Rbytes
10SEG94 | 27026 | 29 | 20546 | 2005616 | 1003 | 23 | 20546 | 2005616
10SEG95 | 27026 | 17 | 10986 | 1099060 | 1003 | 137 | 10986 | 1099060
10SFG04 | 27026 | 122 | 10947 | 989915 | 1003 | 125 | 10947 | 989915
10SFG05 | 27026 | 100 | 10448 | 934994 | 1003 | 8 | 10448 | 934994
10SFG15 | 27026 | 11 | 12719 | 1137548 | 1003 | 8 | 12719 | 1137548
10SFG16 | 27026 | 14 | 12150 | 1087687 | 1003 | 13 | 12150 | 1087687
11SLA57 | 27026 | 10 | 6999 | 640677 | 1003 | 5 | 6999 | 640677
11SLA66 | 27026 | 238 | 6144 | 571585 | 1003 | 97 | 6144 | 571585
11SLA67 | 27026 | 110 | 8175 | 758551 | 1003 | 7 | 8175 | 758551
11SLA68 | 27026 | 9 | 9995 | 937554 | 1003 | 20 | 9995 | 937554
11SLA76 | 27026 | 10 | 15541 | 1500183 | 1003 | 25 | 15541 | 1500183
11SLA77 | 27026 | 108 | 10547 | 976668 | 1003 | 6 | 10547 | 976668
11SLA78 | 27026 | 110 | 7993 | 754440 | 1003 | 20 | 7993 | 754440
54SUD78 | 26171 | 121 | 857 | 93288 | 998 | 86 | 857 | 93288
54SUD79 | 26171 | 259 | 4784 | 508375 | 998 | 8 | 4784 | 508375
54SUD89 | 26171 | 124 | 2385 | 244924 | 998 | 87 | 2385 | 244924
54SUE10 | 26171 | 352 | 12292 | 1207789 | 998 | 14 | 12292 | 1207789
54SUE11 | 26171 | 213 | 11617 | 1097582 | 998 | 242 | 11617 | 1097582
54SUE20 | 26171 | 134 | 7456 | 776832 | 998 | 19 | 7456 | 776832
54SUE21 | 26171 | 353 | 6540 | 648404 | 998 | 124 | 6540 | 648404
54SUE22 | 26171 | 421 | 8416 | 812698 | 998 | 5 | 8416 | 812698
54SUE26 | 26171 | 211 | 9734 | 914688 | 998 | 128 | 9734 | 914688
54SUE27 | 26171 | 14 | 5503 | 510248 | 998 | 5 | 5503 | 510248
54SUE28 | 26171 | 17 | 14684 | 1425217 | 998 | 9 | 14684 | 1425217
54SUE32 | 26171 | 152 | 10402 | 1017586 | 998 | 132 | 10402 | 1017586
54SUE33 | 26171 | 17 | 15669 | 1559011 | 998 | 8 | 15669 | 1559011
54SUE34 | 26171 | 23 | 24422 | 2431862 | 998 | 8 | 24422 | 2431862
54SUE35 | 26171 | 131 | 9365 | 880887 | 998 | 236 | 9365 | 880887
54SUE36 | 26171 | 214 | 16388 | 1519745 | 998 | 12 | 16388 | 1519745
54SUE37 | 26171 | 15 | 7928 | 852064 | 998 | 123 | 7928 | 852064
54SUE38 | 26171 | 421 | 7491 | 721721 | 998 | 73 | 7491 | 721721
54SUE47 | 26171 | 45 | 11690 | 1133578 | 998 | 53 | 11690 | 1133578
54SUE60 | 26171 | 268 | 8732 | 938020 | 998 | 153 | 8732 | 938020
54SUE61 | 10121 | 13 | 6271 | 568879 | 556 | 101 | 6271 | 568879
54SUE70 | 26171 | 131 | 13544 | 1417304 | 998 | 15 | 13544 | 1417304
54SUE71 | 26171 | 20 | 15831 | 1708717 | 998 | 217 | 15831 | 1708717
54SUE72 | 10121 | 82 | 5815 | 552424 | 556 | 145 | 5815 | 552424
54SUE73 | 10121 | 42 | 5537 | 509953 | 556 | 447 | 5537 | 509953
54SUE80 | 26171 | 16 | 2789 | 296418 | 998 | 9 | 2789 | 296418
54SUE83 | 10121 | 148 | 9561 | 890540 | 556 | 284 | 9561 | 890540
54SUE84 | 10121 | 288 | 11356 | 1110164 | 556 | 290 | 11356 | 1110164
54SUE85 | 10121 | 156 | 9591 | 956823 | 556 | 152 | 9591 | 956823
54SUE94 | 26171 | 264 | 26821 | 2984093 | 998 | 274 | 26821 | 2984093
54SUE95 | 10121 | 148 | 10323 | 983241 | 556 | 17 | 10323 | 983241
54SUE96 | 10121 | 12 | 6504 | 603358 | 556 | 480 | 6504 | 603358
TOTAL | 1060410 | 5711 | 463488 | 45270911 | 41439 | 4450 | 463488 | 45270911

@anditi-lidar I'm going to pass on making the change you mentioned for "tagless" queries, because it would create some potentially confusing behavior with the geometries.py{_create_gdf} method that removes elements without any associated tags.
                # add elements that are not nodes and that are without tags or
                # with empty tags to the untagged_element_ids set (untagged
                # nodes are not added to the geometries dict at all)
                if (element["type"] != "node") and (
                    ("tags" not in element) or (not element["tags"])
                ):
                    untagged_element_ids.add(unique_id)

@anditi-lidar I'm going to pass on making the change you mentioned for "tagless" queries, because it would create some potentially confusing behavior with the geometries.py{_create_gdf} method that removes elements without any associated tags.

Yeah no worries. I was pushing my luck anyway trying to piggyback a feature request onto this issue. ;-)

Thanks @derinwalters this is looking good. Would you like to open a PR? We can discuss a couple more tests for thoroughness there.

@gboeing PR submitted


Updating Windows runner version

#659

Codecov Report

Merging #665 (78e17d0) into main (e59b9fe) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #665   +/-   ##
=======================================
  Coverage   90.24%   90.24%
=======================================
  Files          78       78
  Lines        4543     4543
  Branches      839      839
=======================================
  Hits         4100     4100
  Misses        252      252
  Partials      191      191

Flag
Coverage Δ

unittests
90.24% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Should be good.
@ksaur @interesaaat
Can you review it? Thanks!

Why there is still 2019 in the CI/CD pipeline? It looks that it is stuck there.

LGTM but I am not sure if the windows 2019 checks will also block on main.

Ok solved.


multiple class_name on same script

Godot version:
3.1 flathub

OS/device including version:
Fedora 29

Issue description:
In the same script class_name can be written multiple time without triggering any warning or error

Steps to reproduce:

Create a script
write multiple class_name name

Minimal reproduction project:
not applicable

Script should allow only one class_name

Working on it!

Isn't this removing a possible feature? Multiple names for the same class. Think Vector3 and Color... they're both the same, so why not combine them into one?

It's not working like that right now.
Eventually, we should have a single class_name with multiple name like:
class_name vector, color, ...
On Tue, 16 Apr 2019, 12:50 pm Jimmio92<EMAIL_ADDRESS>wrote:

Isn't this removing a possible feature? Multiple names for the same class.
Think Vector3 and Color... they're both the same, so why not combine them
into one?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/godotengine/godot/issues/27959#issuecomment-483776778,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AQZixjbwoB1y6NYY9IcaSSZhMgnhE8Nhks5vhg12gaJpZM4csoOC
.


Chore/add tugraph assets

初始化 TuGraph-DB 的 G6VP 资产

 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.


CLI installation instructions

A section should be added to the README for installation instructions.
Via Cargo:
cargo install dotenvy --bin dotenvy --features cli
And possibly also make it available in package managers.

Additionally, Github Actions workflow could be added to include in Github Releases (like zenv does, but has not seen any development in years).


Is there a way to use the AdMob test ads/test devices with this plugin?

What do you mean? If you integrate the plugin into the app, it just works

Is this solved?
I'm closing for now but open it again if you still need help with the test ads.


Fix crash when modifying a translucent entity outline.

The code for removing items wasn't covered in the unit tests and we were missing an array indexer. New test crashes in master, passes in this branch.

Looks good.


Fix DB::getTablePrefix() issue

Issue Reference
Request: GET
Url: domain/admin/customers/customers/search?query=test
Response
Status Code: 500
error message:  "SQLSTATE[42S22]: Column not found: 1054 Unknown column '__msh__first_name' in 'where clause' (Connection: mysql, SQL: select count(*) as aggregate from __msh__customers where email like %test% or CONCAT(__msh__first_name, " ", __msh__last_name) like %test%)",

Thanks for the contribution.

Please run vendor/bin/pint on your project.

@Sakanweb Any update?

@devansh-webkul done


is this suitable for primary school students who is familiar with scratch?

Asked by Juliana:

if the June session is suitable for students without Python knowledge, but are proficient with Scratch.
Are primary school kids suitable for the session?

No, on https://github.com/melvinzhang/ai-python-course/blob/master/README.txt is says "Prior knowledge assumed: primitive data types, list, function, concept of
object, loop, conditionals, string manipulation"


MPI_SIZEOF is Fortran-only

From MPI 4.1 pp 777
Rationale. This function is not available in other languages because it would not be useful. (End of rationale.)
Should we remove it from the mpi.h header here? Or maybe comment it out with a note that it is Fortran-only?

We deprecated it already so I'm not sure it even matters.

I'll submit a PR with the removal.


Client-create Task

create Task as popup? Suggestions? @erikhofer  you wanted something like fast access for creating tasks to the inbox ?

Tasks should be added directly to the task list as an empty line, without a popup.
We can postpone quick entry to inbox for now.

what about the difference between creating an "project" (root task), extra button or logic ?

A root task is not a project. There should be two buttons:

Create task → create task at the same level as the selected task (or root task, if no task is selected)
Create subtask → create task as child of the selected task (disabled if no task is selected)


Move Merge Functionality for Entities into separate Page/View

Currently, merge functionality of two entities is implemented as part of an entities detail view.
Suggestion is to create a separate page from which you can select two entities that should be merged together.

Implemented in 5f21c4dfb1a428ff076006029f6474c569c9fa1a


Fix Live pipeline for Metrics Advisor

API change check
API changes are not detected in this pull request.


Error codes

Ticket

[CAS-164]

Description

Add error codes for TwilioException
Fix README reference to change iOS constant

Commit message

feat: Add error codes to TwilioException

Testing

[x] Added unit tests
[x] Ran unit tests successfully
[ ] Added documentation for public APIs and/or Wiki

          1 Warning

:warning:
The Pull Request title does not follow the convention [00000] PR Title text

  Generated by :no_entry_sign: Danger

          1 Warning

:warning:
Size impact

ABI
APK Size Impact

x86
2.1MB

x86_64
2.1MB

armeabi-v7a
2.1MB

arm64-v8a
2.1MB

universal
2.1MB

  Generated by :no_entry_sign: Danger

          1 Warning

:warning:
IPA size: 1.9 MB
IPA without SDK size: 1.9 MB
App size: 7.1 MB
TwilioPasskeysAuthentication.framework: 6.1 MB, Impact: 85.4%

  Generated by :no_entry_sign: Danger

          1 Warning

:warning:
The Pull Request title does not follow the convention [00000] PR Title text

  Generated by :no_entry_sign: Danger

          1 Warning

:warning:
Size impact

ABI
APK Size Impact

x86
2.1MB

x86_64
2.1MB

armeabi-v7a
2.1MB

arm64-v8a
2.1MB

universal
2.1MB

  Generated by :no_entry_sign: Danger

🧛 shared module Code Coverage: 92.42%
Coverage of Modified Files:

File
Coverage

CredentialExceptionExtension.kt
91.95%

PasskeyPayloadMapper.kt
100.00%

TwilioException.kt
90.18%

TwilioPasskeys.kt
97.18%

Modified Files Not Found In Coverage Report:
AndroidTwilioPasskeyTest.kt
CredentialExceptionExtensionTest.kt
NSErrorExtension.kt
NSErrorExtensionTest.kt
PasskeyPayloadMapperTest.kt

Codebase cunningly covered by count Shroud 🧛

  Generated by :no_entry_sign: Danger

          1 Warning

:warning:
IPA size: 1.9 MB
IPA without SDK size: 1.9 MB
App size: 7.1 MB
TwilioPasskeysAuthentication.framework: 6.1 MB, Impact: 85.4%

  Generated by :no_entry_sign: Danger

          2 Warnings

:warning:
Big PR

:warning:
The Pull Request title does not follow the convention [00000] PR Title text

  Generated by :no_entry_sign: Danger

          2 Warnings

:warning:
Big PR

:warning:
The Pull Request title does not follow the convention [00000] PR Title text

  Generated by :no_entry_sign: Danger

          1 Warning

:warning:
Size impact

ABI
APK Size Impact

x86
2.1MB

x86_64
2.1MB

armeabi-v7a
2.1MB

arm64-v8a
2.1MB

universal
2.1MB

  Generated by :no_entry_sign: Danger

🧛 shared module Code Coverage: 92.42%
Coverage of Modified Files:

File
Coverage

CredentialExceptionExtension.kt
91.95%

PasskeyPayloadMapper.kt
100.00%

TwilioException.kt
90.18%

TwilioPasskeys.kt
97.18%

Modified Files Not Found In Coverage Report:
AndroidTwilioPasskeyTest.kt
CredentialExceptionExtensionTest.kt
NSErrorExtension.kt
NSErrorExtensionTest.kt
PasskeyPayloadMapperTest.kt
README.md

Codebase cunningly covered by count Shroud 🧛

  Generated by :no_entry_sign: Danger

          1 Warning

:warning:
IPA size: 1.9 MB
IPA without SDK size: 1.9 MB
App size: 7.1 MB
TwilioPasskeysAuthentication.framework: 6.1 MB, Impact: 85.4%

  Generated by :no_entry_sign: Danger

          2 Warnings

:warning:
Big PR

:warning:
The Pull Request title does not follow the convention [00000] PR Title text

  Generated by :no_entry_sign: Danger

          1 Warning

:warning:
IPA size: 1.9 MB
IPA without SDK size: 1.9 MB
App size: 7.1 MB
TwilioPasskeysAuthentication.framework: 6.1 MB, Impact: 85.4%

  Generated by :no_entry_sign: Danger

          1 Warning

:warning:
Size impact

ABI
APK Size Impact

x86
2.1MB

x86_64
2.1MB

armeabi-v7a
2.1MB

arm64-v8a
2.1MB

universal
2.1MB

  Generated by :no_entry_sign: Danger

🧛 shared module Code Coverage: 92.42%
Coverage of Modified Files:

File
Coverage

CredentialExceptionExtension.kt
91.95%

PasskeyPayloadMapper.kt
100.00%

TwilioException.kt
90.18%

TwilioPasskeys.kt
97.18%

Modified Files Not Found In Coverage Report:
AndroidTwilioPasskeyTest.kt
CredentialExceptionExtensionTest.kt
NSErrorExtension.kt
NSErrorExtensionTest.kt
PasskeyPayloadMapperTest.kt
README.md

Codebase cunningly covered by count Shroud 🧛

  Generated by :no_entry_sign: Danger

          2 Warnings

:warning:
Big PR

:warning:
The Pull Request title does not follow the convention [00000] PR Title text

  Generated by :no_entry_sign: Danger

🧛 shared module Code Coverage: 92.42%
Coverage of Modified Files:

File
Coverage

CredentialExceptionExtension.kt
91.95%

PasskeyPayloadMapper.kt
100.00%

TwilioException.kt
90.18%

TwilioPasskeys.kt
97.18%

Modified Files Not Found In Coverage Report:
AndroidTwilioPasskeyTest.kt
CredentialExceptionExtensionTest.kt
NSErrorExtension.kt
NSErrorExtensionTest.kt
PasskeyPayloadMapperTest.kt
README.md

Codebase cunningly covered by count Shroud 🧛

  Generated by :no_entry_sign: Danger

          1 Warning

:warning:
IPA size: 1.9 MB
IPA without SDK size: 1.9 MB
App size: 7.1 MB
TwilioPasskeysAuthentication.framework: 6.1 MB, Impact: 85.4%

  Generated by :no_entry_sign: Danger


Add ability to swap which folder is "left" and "right" with a simple button click

I use Beyond Compare 4 a good bit, and I like the option it provides to swap left vs. right folders.
See the far right button?

Is it possible to implement something like this in the VSCode extension? I realize since the "left" folder comes from the workspace this might not be possible.

Thank you!
It can be possible with the right changes in the code ;)
A workaround for now is open the "right" folder in vscode and compare it to the left..

I pre-released v0.11.0. Please install it and verify the feature is working as expected.
Thanks!


ci: Upgrade actions that use EOL Node v16

Addresses: https://qctrl.slack.com/archives/CF98D2868/p1715565148601279
Changes proposed in this pull request:

Update actions that use Node v16 since it has reached end of life.

No changes?

No changes?

Master already had these changes merged in from this lol https://github.com/qctrl/open-controls/pull/295


remove pre-release step in app-smoke-test.yml

Error
›   Error: loadConfig is not a function  at create app with no extensions step https://github.com/adobe/aio-e2e-tests/actions/runs/5514397331/jobs/10053607346
Source:
https://github.com/adobe/aio-e2e-tests/blob/master/.github/workflows/app-smoke-test.yml
This test workflow uses the master branch of the app plugin.
Investigation
This is because of the pre-release step which may pull in breaking updates: https://github.com/adobe/aio-e2e-tests/blob/4900c4cd29f66be8e7ef4f44e3dc27986d1bd0e0/.github/workflows/app-smoke-test.yml#L26-L32
Fix:
Remove the Update package.json with an npm pre-release version step.

fixed in #99


[HELP] invalid middleware type or middleware does not exist

Hello, I've been trying to setup the crowdsec bouncer plugin but traefik is unable to load the middleware apparently. It's probably some misconfiguration problem but I went over it multiple times and I can't figure it out.
This is part of my static traefik.yaml
entryPoints:
  websecure:
    address: :443
    http:
      middlewares:
        - crowdsec@file

experimental:
  plugins:
    enabled: true
    bouncer:
      moduleName: github.com/maxlerebourg/crowdsec-bouncer-traefik-plugin
      version: v1.2.1-rc1

This is my middleware rule, in a dynamic configuration file:
http:
  middlewares:
    crowdsec:
      plugin:
        bouncer:
          enabled: true
          crowdsecMode: "live"
          crowdsecLapiHost: "crowdsec:8080"
          crowdsecLapiKey: /

From the logs it looks like the configuration is correctly loaded:
level=debug msg="Configuration received: {"http":{"middlewares":{"crowdsec":{"plugin":{"bouncer":{"crowdsecLapiHost":"crowdsec:8080","crowdsecLapiKey":/,"crowdsecMode":"live","enabled":"true"}}}
though traefik fails to apply the middleware to any host:
 level=error msg="invalid middleware \"crowdsec@file\" configuration: invalid middleware type or middleware does not exist" entryPointName=websecure routerName=websecure-service@docker
What am I missing? The only difference with the suggested configuration is that I put the middleware in the static configuration as a global one for my entry point, as I do with many other middlewares. Is that not supposed to be supported?

Are you sure that your dynamic conf is loaded ? Your CrowdsecLapiKey is not just / ?
Show me log from your crowdsec instance and put our plugin in debug mode, maybe you will have further info.

Perfect, I close the issue :)
Don't hesitate to Star the project, to gain visibility and support the project


[SPARK-49830] Add required fields for sparkApplicationSentinel

What changes were proposed in this pull request?
Add the required fields for sparkApplicationSentinel
Why are the changes needed?
Without this, when enable sparkApplicationSentinel, the operator got ERROR
Caused by: io.fabric8.kubernetes.client.KubernetesClientException: Failure executing: PUT at: [https://<IP_ADDRESS>:443/apis/spark.apache.org/v1alpha1/namespaces/spark-4/sparkapplications/spark-app-sentinel](https://<IP_ADDRESS>/apis/spark.apache.org/v1alpha1/namespaces/spark-4/sparkapplications/spark-app-sentinel). Message: SparkApplication.spark.apache.org "spark-app-sentinel" is invalid: spec.runtimeVersions: Required value. Received status: Status(apiVersion=v1, code=422, details=StatusDetails(causes=[StatusCause(field=spec.runtimeVersions, message=Required value, reason=FieldValueRequired, additionalProperties={})], group=spark.apache.org, kind=SparkApplication, name=spark-app-sentinel, retryAfterSeconds=null, uid=null, additionalProperties={}), kind=Status, message=SparkApplication.spark.apache.org "spark-app-sentinel" is invalid: spec.runtimeVersions: Required value, metadata=ListMeta(_continue=null, remainingItemCount=null, resourceVersion=null, selfLink=null, additionalProperties={}), reason=Invalid, status=Failure, additionalProperties={}).

Does this PR introduce any user-facing change?
no
How was this patch tested?
test locally, modify the values.yaml with sparkApplicationSentinel enabled
Was this patch authored or co-authored using generative AI tooling?
no

Merged to master.


T308888: Possibly remove deploy-hook

I removed images/deploy-hook/ and the files in it, the file paws/templates/deploy-hook.yaml, and the parts of deploy-hook in paws/values.yaml

Does this work?


Export human-readable object names for geometric data

For the name nodes that are involved in mouse picking (and later the object hierarchy) we're using the geometry id's, but these names appear to be codes rather than proper names. Find out if there's a better way...

This is done, but not too happy with the names in the test model, need a better test model?


Private Key

Once the key generated, how do I get private key to store it?!

If you want to store a private key in a file, you can use the key.Export method as follows:
var algorithm = SignatureAlgorithm.Ed25519;

var creationParameters = new KeyCreationParameters
{
    ExportPolicy = KeyExportPolicies.AllowPlaintextArchiving
};

// create a new key
using (var key = new Key(algorithm, creationParameters))
{
    // export it
    var blob = key.Export(KeyBlobFormat.NSecPrivateKey);

    File.WriteAllBytes("myprivatekey.nsec", blob);
}

Later, you can re-import the key using the Key.Import method as follows:
var algorithm = SignatureAlgorithm.Ed25519;

var blob = File.ReadAllBytes("myprivatekey.nsec");

using (var key = Key.Import(algorithm, blob, KeyBlobFormat.NSecPrivateKey))
{
    algorithm.Sign(key, ...);
}

The format parameter specifies the format to use when exporting the private key. Currently, the formats NSecPrivateKey, PkixPrivateKey, and RawPrivateKey are available. Note that all of them export the private key in plaintext without any protections, which is why the original key needs to be generated with the AllowPlaintextArchiving flag.
Does that answer your question?


Clean up compile warnings during build

There's actually only one warning when makeing the lib, it just repeats itself several times:
/Users/markoates/Repos/allegro_flare/include/allegro_flare/gui/gui_screen.h:33:9: warning: 'UIScreen::mouse_axes_func' hides overloaded virtual function [-Woverloaded-virtual]
   void mouse_axes_func();
        ^
/Users/markoates/Repos/allegro_flare/include/allegro_flare/gui/widget.h:78:17: note: hidden overloaded virtual function 'UIWidget::mouse_axes_func' declared here: different number of
      parameters (4 vs 0)
   virtual void mouse_axes_func(float x, float y, float dx, float dy);
                ^

Fixed in https://github.com/MarkOates/allegro_flare/commit/181d57f13f852152f1707530aa564e06f751d380


How can I use this Dataset to train a LLM on Huggingface?

How can I use this Dataset to train a LLM on Huggingface?

If you want to train an LLM to generate user prompts, maybe you can start with some small open source LLM models (e.g., Llama 2) and fine tune them on DiffusionDB prompt text?


Editk8demo

Added fixes for ACI connector which includes a canary build of the connector. Also scoped the service principle differently. @rgardler I took out some of your nginx stuff because it wasn't giving me an ip even though it was running - we don't support kubectl get logs and some other ip getting things but I was able to get it in a json format output but I need some help with the correct json string because it's slightly different from what you have.
This is what I deleted : Since we need to ensure our Public IPs have been assigned before
proceeding, and because we need the IP number later we'll run a loop
to grab the IP once assinged. This is a little cumbersome but great if
you want to script things. If you are doing this manually you can use
kubectl get service --wait to display changes as they happen.
NGINX_IP=""
while [ -z $NGINX_IP ]; do sleep 10; NGINX_IP=$(kubectl get service vamp -o jsonpath="{.status.loadBalancer.ingress[*].ip}"); done
kubectl get pods -o wide

Now we have our IP:
echo $NGINX_IP
Take a look...
xdg-open $NGINX

@rbitia,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

If you want this merged you'll have to fix the conflicts above.


[FUNCTIONALITY] - Show more

Description:
Create a show more/ show less function for our cards, animation is not necessary
Acceptance:

Show more / Show less link
Should expand and contract

Tests:

When clicking on show more, the extra cards are displayed
When clicking on show less, the cards are contracted

Made a mistake and committed to main, no PR for this issue.


Tree view won't scroll

I am unable to get the index area to scroll downward when the number of topics extends below the bottom of the browser window.  This just started happening in the last few days.
In the attached picture, the number of topics under the "Sequelize" topic is much more than 5, but I can't make them visible.

The stickiness of the navigation was being added by the mkdocs template, we needed to add the javascript bit that applies navigation stickiness, initially and on resize, based on the window height vs nav height.
This bug was related to another fix to the nav menu I just made, which removed a mkdocs specific fix to the nav. I just deployed new js that allows scrolling + sticky menus on mkdocs.

Works for me now, thanks!


Add documentation about required Kubernetes roles

Not every team will be able to get cluster admin privileges.

Not every team will be able to get cluster admin privileges.

Are you able to run moon inside kubernetes without default service account having admin, and there currently is just a lack of documentation? Or do you need to have the default service account that moon uses have admin privilages and you are creating documentation on what you need to get from your cluster admin?  Just trying to figure out what this change will provide.


[Doc] Missing doc for ConvTranspose

I was fairly surprised to not found a conv transpose op on flax: https://flax.readthedocs.io/en/latest/flax.linen.html#linear-modules.
After investigation, it seems that the operation exists in the code, but do not appear on the doc.
https://github.com/google/flax/blob/b112b1128bd275bc5380b8370887f3c791209712/flax/linen/linear.py#L290
Is it expected ?

No, that isn't expected, thanks for catching this! I will add it soon.

I believe this can be closed: docs


Run as non-root user

Set the app to run as a non-root user.

Is that why I get this running as non-root?

ERROR: datastore for scope "global" is not initialized

@Ed42 No that come from the overlay network. You need to initialize your Docker daemon as a Swarm for it to work. I don't know if you can run it without the Swarm, the doc makes it feel optional.

Run as non root users:  https://github.com/dockersamples/atsea-sample-shop-app/commit/2a40c44afc9cc916704593d2ca8ce52304ed1261


Form explainer fix: IE8 highlights

Changes

Update definitions highlight filters to use new teal

Additions

Add .has-attention (and hover state?) highlight filters for IE8

Preview

Testing
This will need to be tested in Saucelabs. Open loan estimate in IE8 & see darker highlight on image map when associated explanation is hovered over/clicked.
Notes
I'm having trouble seeing the hover state for the image map links. Not sure if this is Saucelabs or another bug.
Review
@cfarm

:+1:


Updates a number of references to "main"

One step towards updating default branch name to main.

One step towards updating default branch name to main.


Add color to each tags

Hi @michalsn,
Thank you! for creating this great library.
I have a question,
How to color each tag when displayed on the frontend?
its like the github label, if the tag name is bug the color is red, if the label is enhancement the color is purple, and so on.

As for displaying the tags themselves, if the tags are predetermined, you can simply add a "color" field to the table.


Sample code of the "Guaranteed Delivery with Failover" page doesn't work as expected

Description

When the ForwardMessageProcessor calls the target endpoint SimpleStockQuoteService, the SOAPAction urn:getQuote header is not sent with the request causing a failure in the backend.

Steps to reproduce

The steps to be followed to reproduce the issue.

Affected version(s)

The versions that are affected by the issue.

Related issue(s) (optional)

Any related issues such as sub tasks and issues reported in other repositories (e.g., component repositories), similar problems, etc.

Suggested label(s) (optional)

Optional comma-separated list of suggested labels. Non committers can’t assign labels to issues, and thereby, this will help issue creators who are not a committer to suggest possible labels.

Suggested assignee(s) (optional)

Optional comma-separated list of suggested team members who should attend the issue. Non committers can’t assign issues to assignees, and thereby, this will help issue creators who are not a committer to suggest possible assignees.

Fixed with #684


Ensembl repeat download

( https://jira.sanger.ac.uk/browse/TOLIT-433 )
The test and test_full profiles are implemented and tested on GitHub. test corresponds to using command-line parameters, test_full to using a sample-sheet.
$ nextflow run src/ensemblrepeatdownload/ -profile test,singularity --outdir ensemblrepeatdownload
(...)
[87/8c6a05] process > SANGERTOL_ENSEMBLREPEATDOWNLOAD:ENSEMBLREPEATDOWNLOAD:DOWNLOAD:ENSEMBL_GENOME_DOWNLOAD (GCA_907164935.1)       [100%] 1 of 1 ✔
[18/6bde73] process > SANGERTOL_ENSEMBLREPEATDOWNLOAD:ENSEMBLREPEATDOWNLOAD:PREPARE_FASTA:TABIX_BGZIP (GCA_907164935.1.masked.ens... [100%] 1 of 1 ✔
[a9/7eeace] process > SANGERTOL_ENSEMBLREPEATDOWNLOAD:ENSEMBLREPEATDOWNLOAD:PREPARE_FASTA:CUSTOM_GETCHROMSIZES (GCA_907164935.1.m... [100%] 1 of 1 ✔
[a4/1615f3] process > SANGERTOL_ENSEMBLREPEATDOWNLOAD:ENSEMBLREPEATDOWNLOAD:PREPARE_FASTA:SAMTOOLS_DICT (GCA_907164935.1.masked.e... [100%] 1 of 1 ✔
[aa/d9c2d6] process > SANGERTOL_ENSEMBLREPEATDOWNLOAD:ENSEMBLREPEATDOWNLOAD:PREPARE_REPEATS:REPEATS_BED (GCA_907164935.1.masked.e... [100%] 1 of 1 ✔
[ee/4b65ac] process > SANGERTOL_ENSEMBLREPEATDOWNLOAD:ENSEMBLREPEATDOWNLOAD:PREPARE_REPEATS:TABIX_BGZIP (GCA_907164935.1.masked.e... [100%] 1 of 1 ✔
[8e/ce840d] process > SANGERTOL_ENSEMBLREPEATDOWNLOAD:ENSEMBLREPEATDOWNLOAD:PREPARE_REPEATS:TABIX_TABIX_CSI (GCA_907164935.1.mask... [100%] 1 of 1 ✔
[2e/3df124] process > SANGERTOL_ENSEMBLREPEATDOWNLOAD:ENSEMBLREPEATDOWNLOAD:PREPARE_REPEATS:TABIX_TABIX_TBI (GCA_907164935.1.mask... [100%] 1 of 1 ✔
[0e/9adaa0] process > SANGERTOL_ENSEMBLREPEATDOWNLOAD:ENSEMBLREPEATDOWNLOAD:CUSTOM_DUMPSOFTWAREVERSIONS (1)                          [100%] 1 of 1 ✔
$ tree ensemblgenedownload/
ensemblrepeatdownload/
├── pipeline_info
│   ├── execution_report_2022-09-30_09-20-20.html
│   ├── execution_timeline_2022-09-30_09-20-20.html
│   ├── execution_trace_2022-09-30_09-20-20.txt
│   ├── pipeline_dag_2022-09-30_09-20-20.html
│   └── software_versions.yml
└── repeats
    └── ensembl
        ├── GCA_907164935.1.masked.ensembl.bed.gz
        ├── GCA_907164935.1.masked.ensembl.bed.gz.csi
        ├── GCA_907164935.1.masked.ensembl.bed.gz.tbi
        ├── GCA_907164935.1.masked.ensembl.fa.dict
        ├── GCA_907164935.1.masked.ensembl.fa.gz
        ├── GCA_907164935.1.masked.ensembl.fa.gz.fai
        ├── GCA_907164935.1.masked.ensembl.fa.gz.gzi
        └── GCA_907164935.1.masked.ensembl.fa.gz.sizes
$ nextflow run src/ensemblrepeatdownload/ -profile test_full,singularity --outdir ensemblrepeatdownload
(...)
[9c/efebb2] process > SANGERTOL_ENSEMBLREPEATDOWNLOAD:ENSEMBLREPEATDOWNLOAD:PARAMS_CHECK:SAMPLESHEET_CHECK (samplesheet.csv)         [100%] 1 of 1 ✔
[c2/c5483a] process > SANGERTOL_ENSEMBLREPEATDOWNLOAD:ENSEMBLREPEATDOWNLOAD:DOWNLOAD:ENSEMBL_GENOME_DOWNLOAD (GCA_905163415.1)       [100%] 4 of 4 ✔
[b1/8a5a98] process > SANGERTOL_ENSEMBLREPEATDOWNLOAD:ENSEMBLREPEATDOWNLOAD:PREPARE_FASTA:TABIX_BGZIP (GCA_905163415.1.masked.ens... [100%] 4 of 4 ✔
[70/89d012] process > SANGERTOL_ENSEMBLREPEATDOWNLOAD:ENSEMBLREPEATDOWNLOAD:PREPARE_FASTA:CUSTOM_GETCHROMSIZES (GCA_905163415.1.m... [100%] 4 of 4 ✔
[30/912674] process > SANGERTOL_ENSEMBLREPEATDOWNLOAD:ENSEMBLREPEATDOWNLOAD:PREPARE_FASTA:SAMTOOLS_DICT (GCA_905163415.1.masked.e... [100%] 4 of 4 ✔
[bc/bab3cb] process > SANGERTOL_ENSEMBLREPEATDOWNLOAD:ENSEMBLREPEATDOWNLOAD:PREPARE_REPEATS:REPEATS_BED (GCA_905163415.1.masked.e... [100%] 4 of 4 ✔
[8c/9658b8] process > SANGERTOL_ENSEMBLREPEATDOWNLOAD:ENSEMBLREPEATDOWNLOAD:PREPARE_REPEATS:TABIX_BGZIP (GCA_905163415.1.masked.e... [100%] 4 of 4 ✔
[15/c51007] process > SANGERTOL_ENSEMBLREPEATDOWNLOAD:ENSEMBLREPEATDOWNLOAD:PREPARE_REPEATS:TABIX_TABIX_CSI (GCA_905163415.1.mask... [100%] 4 of 4 ✔
[64/18be16] process > SANGERTOL_ENSEMBLREPEATDOWNLOAD:ENSEMBLREPEATDOWNLOAD:PREPARE_REPEATS:TABIX_TABIX_TBI (GCA_905163415.1.mask... [100%] 4 of 4 ✔
[b2/176ef3] process > SANGERTOL_ENSEMBLREPEATDOWNLOAD:ENSEMBLREPEATDOWNLOAD:CUSTOM_DUMPSOFTWAREVERSIONS (1)                          [100%] 1 of 1 ✔

Input parameters are the assembly accession, and the Ensembl species name.
PR checklist

[x] This comment contains a description of changes (with reason).
[ ] If you've fixed a bug or added code that should be tested, add tests!
[ ] If you've added a new tool - have you followed the pipeline conventions in the contribution docs
[x] Make sure your code lints (nf-core lint).
[x] Ensure the test suite passes (nextflow run . -profile test,docker --outdir <OUTDIR>).
[x] Usage Documentation in docs/usage.md is updated.
[x] Output Documentation in docs/output.md is updated.
[x] CHANGELOG.md is updated.
[x] README.md is updated (including new tool citations and authors/contributors).

Last but not least, this pipeline downloads the masked genomes from Ensembl and sets up the files on disk exactly the same way as the insdcdownload pipeline, using the same shared subworkflows.

I will address all your comments. Since they were the same as in other pipelines, I wanted to make sure I was approaching them the right way.
sanger-tol/ensemblgenedownload and sanger-tol/insdcdownload have now been released, and I'll fix this one next sprint

OK. Should I still do a release on Zenodo like the other ones ?
The main reason I split the Ensembl download (and made a different one from INSDC download) is that I was not happy with how the sample-sheet would look like. There are two many fields that are just for one of the three. But perhaps I should just drop support for sample-sheet ?
I was also thinking: my problem is with the CSV format itself. But some nf-core pipelines (and our genome assembly pipeline) use YML as input. YML may be more amenable to having different sections and multiple entries per section ?


[Snyk] Upgrade morgan from 1.6.1 to 1.10.0

Snyk has created this PR to upgrade morgan from 1.6.1 to 1.10.0.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 7 versions ahead of your current version.

The recommended version was released on 4 years ago.

Issues fixed by the recommended upgrade:

Issue
Score
Exploit Maturity

Arbitrary Code InjectionSNYK-JS-MORGAN-72579
661
Proof of Concept

Release notes

Package name: morgan

1.10.0 - 2020-03-20
Add :total-time token
Fix trailing space in colored status code for dev format
deps<EMAIL_ADDRESS>
deps<EMAIL_ADDRESS>

deps<EMAIL_ADDRESS>
Replace internal eval usage with Function constructor
Use instance methods on process to check for listeners

deps<EMAIL_ADDRESS>
Fix res.writeHead patch missing return value

1.9.1 - 2018-09-11
Fix using special characters in format
deps<EMAIL_ADDRESS>
perf: remove argument reassignment

1.9.0 - 2017-09-27
Use res.headersSent when available
deps<EMAIL_ADDRESS>
Use safe-buffer for improved Buffer API

deps<EMAIL_ADDRESS>deps<EMAIL_ADDRESS>
Remove unnecessary Buffer loading

1.8.2 - 2017-05-24
deps<EMAIL_ADDRESS>
Fix DEBUG_MAX_ARRAY_LENGTH
deps<EMAIL_ADDRESS>

1.8.1 - 2017-02-11
deps<EMAIL_ADDRESS>
Fix deprecation messages in WebStorm and other editors
Undeprecate DEBUG_FD set to 1 or 2

1.8.0 - 2017-02-05
Fix sending unnecessary undefined argument to token functions
deps<EMAIL_ADDRESS>deps<EMAIL_ADDRESS>
Allow colors in workers
Deprecated DEBUG_FD environment variable
Fix error when running under React Native
Use same color for same namespace
deps<EMAIL_ADDRESS>

perf: enable strict mode in compiled functions

1.7.0 - 2016-02-19
Add digits argument to response-time token
deps<EMAIL_ADDRESS>
Enable strict mode in more places
Support web browser loading

deps<EMAIL_ADDRESS>
perf: enable strict mode

1.6.1 - 2015-07-04
deps<EMAIL_ADDRESS>

    from morgan GitHub release notes

[!IMPORTANT]

Check the changes in this PR to ensure they won't cause issues with your project.
This PR was automatically created by Snyk using the credentials of a real user.
Max score is 1000. Note that the real score may have changed since the PR was raised.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:

🧐 View latest project report
📜 Customise PR templates
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

Archipelo Digest
The branch digest for this pull request is ready: snyk-upgrade-945b579be02e1c861dd2e800a92610f1
This branch has 0% commit with provenance.

Metric
Value

Copy-paste event
0

Code leak
0

Downloads
0

AI coding
0


Office apps does not appear in the power Run

I have office 2019 on my device and I can search it on the Windows Search. However, none of those apps shows up in Power Run. I have even dropped an image showing the same.

Can you search for Word in the Windows Search?

I can't repro this issue:

sure

are you using office 365?

@CookieByte2057
Well then the PT team will look into it.
I am also using Office 2019
(Not a huge fan of subscriptions)

Ok, I'll see, Thanks BTW :-)

@CookieByte2057 Why did you close this issue?

I think it's a local issue by the indexing. We can't do much on that

@Aaron-Junker
What can I do from my side to solve this issue?

There's a way to clear the cache. But I forgot how.
@enricogior Are you know hiw to do this?

@CookieByte2057, What version of PowerToys are you running?  During PT Run startup now in atleast 0.23, we reindex everything
How did you install office?

@crutkas
I have PowerToy version 0.23.2.
Well, Office came preinstalled on my pc.

Hi @CookieByte2057
Can you please zip and share this folder? - C:\Users\USERNAME\AppData\Local\Microsoft\PowerToys\PowerToys Run\Logs, replace USERNAME with the system user name.
It might contain some logs as to why the app is not being loaded. Thank you.

<IP_ADDRESS>.zip

@CookieByte2057 Nothing odd seems to stand out in the logs. There are no exceptions against packaged apps.
Could you try to search for any other packaged applications like terminal, office or spotify? Easy way to detect packaged apps is that they have a panel when you search for them in the start menu. When you search for them on PT Run, the subtitle must say Packaged Application instead of just Application.

on PT Run -

@CookieByte2057, Can you share the windows version that you are on? Please press Win+R and enter winver.
You should get a window with the version. Would be great if you could share a screenshot of that window.
Eg:

You should not be able to see the open file location under word, unlike the screenshot below. Please confirm that you CANNOT see the open file location option. This is to confirm my thoughts on the application type.

Finally, can you open powerShell and execute the following command (simply copy-paste it and press enter)?
cd Desktop; Get-AppxPackage -User $env:UserName > packages.txt
A packages.txt file would be created on your desktop, can you please share that here? The above command simply lists all the packaged apps for the current user. If should ideally contain word, excel, powerpoint etc. If they do, then there is something wrong on the powertoys side, if they don't then the way those applications are configured is different from the other packaged apps.
Thank you for your time and all your responses!

packages.txt

Hey @alekhyareddy28 ,
Even I searched for Word, Excel and PowerPoint in the packages list but I didn't found any of these apps' package, however, I did found OneNote package and I am able to search for it in the PT Run.

@CookieByte2057 we're looking into this issue to see why the some office apps don't show up.
I think we should keep the issue open unless it's resolved.

ok

Why did you close this @CookieByte2057?


refactor and ensure to trim searchBox.value

wrap read/write access to this.searchBox.value, and use get/set
function respectively
when reading, trim the searchBox value when appropriate

Is this guided by the 2 instances where we use query.trim(), one at isRecommendedExtensionsQuery and other at RecommendedExtensionsView.show()?
Or anything else as well?
If its just the 2 instances, then the trim() in the second case is actually not required. We can remove that regardless of any other change.
Which leaves us with the single instance where query.trim() is being used. A refactor just for this single instance is an overkill.
Let me know if there was any other reasons for this refactoring.

Or anything else as well?
Yes for sure, I had observed that when typing spaces into the search box, the query was submitted as is, also the delay interval  in triggerSearch matters when the input only contains white spaces.
Then the two trim()s you mentioned becomes unnecessary and can be removed as a result, no the reason. But overall this is indeed not a big deal, just in case though.

@ryenus Sorry for not getting back earlier. Got caught up with work and then vacation :)
I agree that this helps the case of input containing only white spaces but since that case in itself is not part of the "normal" workflow, I wouldnt take this PR unless there are other benefits for this change.


Missing Identifier.mom file during build

During the build phase, the application is unable to find the Identifier.mom file, causing the build to fail.
The error message displayed is:
Resource "/Users/conrad/dev/JNCTN-UX/apps/jnctn-mobile/ios/build/Build/Products/Debug-iphonesimulator/WalletLibrary/WalletLibrary.framework/Identifier.mom" not found. Run 'pod install' to update the copy resources script.
To Reproduce
Steps to reproduce the behavior:

Clone the repository
In the Podfile set use_frameworks! :linkage => :static
Run pod install
Build the project
See error

Expected behavior
The Identifier.mom file should be correctly located and the build should succeed without any issues.
Workaround
Currently, a workaround for this issue involves adding a new "Run Script" phase in the build phases of the WalletLibrary target using a post_install hook in the Podfile.
Here is the script:
    installer.pods_project.targets.each do |target|
      # BEGIN support for WalletLibrary
      if target.name == 'WalletLibrary'
        target.build_configurations.each do |config|
          config.build_settings['LD_RUNPATH_SEARCH_PATHS'] = ['$(inherited)', '@executable_path/Frameworks']
          phase_name = "Copy Identifier.mom for WalletLibrary"
          script_phase = target.shell_script_build_phases.find { |bp| bp.name == phase_name }
          unless script_phase
            script_phase = target.new_shell_script_build_phase(phase_name)
            script_phase.shell_script = <<-SCRIPT
              SOURCE_PATH="${TARGET_BUILD_DIR}/WalletLibrary.framework/VerifiableCredentialDataModel.momd/Identifier.mom"
              DEST_PATH="${TARGET_BUILD_DIR}/WalletLibrary.framework/Identifier.mom"
              mkdir -p "$(dirname "$DEST_PATH")"
              if [ -f "$SOURCE_PATH" ]; then
                  cp "$SOURCE_PATH" "$DEST_PATH"
              else
                  echo "File $SOURCE_PATH does not exist."
              fi
            SCRIPT
          end
        end
      end
      # END support for WalletLibrary
    end

This script will copy the Identifier.mom file from its generated location to the expected location, resolving the build error.
This is only a workaround, and the issue should ideally be addressed in the core project so that manual intervention is not required.
Environment
WalletLibrary: 0.0.1-beta.2
OS: macOS 13.4
Xcode version: 14.3
Cocoapods version: 1.11.3

Thank you for this work around. We currently have it in the roadmap to make the Identifier layer pluggable, hopefully fixing this issue. We are still working on the priority for this feature. Will update the issue when this feature is added.

Based on testing just now, this patch is still required for v1.0.0


Use report selections for spear check

graag niet meer sturen.
Michel Ale
Avatarum
0653570131
Op 29 december 2023 20:50:36 +01:00, schreef René Brummel @.***>:

Merged #32 https://github.com/RedAndBundle/Cust.SpearTech/pull/32 into main.
—
Reply to this email directly, view it on GitHub https://github.com/RedAndBundle/Cust.SpearTech/pull/32#event-11358452370, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABVGCWCKSEYRO3GWMBVDIKTYL4NIZAVCNFSM6AAAAABBG6OZUGVHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJRGM2TQNBVGIZTOMA.
You are receiving this because you are subscribed to this thread.Message ID: @.***>

graag niet meer sturen.
Michel Ale
Avatarum
0653570131
Op 29 december 2023 20:50:30 +01:00, schreef René Brummel @.***>:

You can view, comment on, or merge this pull request online at:
https://github.com/RedAndBundle/Cust.SpearTech/pull/32
Commit Summary

fa82ffe https://github.com/RedAndBundle/Cust.SpearTech/pull/32/commits/fa82ffe968ebeba8493b3ab45649dcc2331a7165 Use report selections for spear check
File Changes
(9 files https://github.com/RedAndBundle/Cust.SpearTech/pull/32/files)

D SpearTech.ForNAV.Check/BankAccount/AppSourceCop.json https://github.com/RedAndBundle/Cust.SpearTech/pull/32/files#diff-e20c428f8c365662df7ae113afab5ce8008e171c66bfa808f58768e5548342b6 (3)

M SpearTech.ForNAV.Check/Check/RunCheck.Codeunit.al https://github.com/RedAndBundle/Cust.SpearTech/pull/32/files#diff-6fa902815271dcc96d8d4c81c9cfa320bacf21f8fe10326ff8c877719b63a355 (26)

M SpearTech.ForNAV.Check/Check/RunCheckBatch.Table.al https://github.com/RedAndBundle/Cust.SpearTech/pull/32/files#diff-cb9bb2df62d5ae8e37f70177a67fde5bfbbefecd94d08cad0c8e9dd2e5f1af19 (2)

A SpearTech.ForNAV.Check/Setup/ReportSelUsageBank.EnumExt.al https://github.com/RedAndBundle/Cust.SpearTech/pull/32/files#diff-58f8b4cefe6e7904567334445c7e3606a3e3b259bf10141f9e961a008211913d (7)

A SpearTech.ForNAV.Check/Setup/ReportSelection.Codeunit.al https://github.com/RedAndBundle/Cust.SpearTech/pull/32/files#diff-31cb0e1636c82711c9ad50185c442efa4cd2c759a21dd1b8dda6868085f7dfae (20)

A SpearTech.ForNAV.Check/Setup/ReportSelectionUsage.EnumExt.al https://github.com/RedAndBundle/Cust.SpearTech/pull/32/files#diff-23ebf0c7e5691f3dbafb81651b108715503b7bf12388813612b2db7fabb4fb88 (7)

M SpearTech.ForNAV.Check/Setup/Setup.table.al https://github.com/RedAndBundle/Cust.SpearTech/pull/32/files#diff-63f51c9569e20bbc0a02b6f96c3aa3298e8eec1b9fc7befed620a2c4a189f82b (10)

M SpearTech.ForNAV.Check/Translations/SpearTech.ForNAV.Check.g.xlf https://github.com/RedAndBundle/Cust.SpearTech/pull/32/files#diff-eaf59564a51c9cd5f58aa52e059f612f14f5ad2031e78755b46624f02577dccd (15)

M SpearTech.ForNAV.Check/app.json https://github.com/RedAndBundle/Cust.SpearTech/pull/32/files#diff-d64e444db6069a31aeb9a412509aebb4573e05ffbc84dc21f06f87f161081b76 (2)
Patch Links:

https://github.com/RedAndBundle/Cust.SpearTech/pull/32.patch

https://github.com/RedAndBundle/Cust.SpearTech/pull/32.diff

—
Reply to this email directly, view it on GitHub https://github.com/RedAndBundle/Cust.SpearTech/pull/32, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABVGCWCLS5HVOXVWW2DY2E3YL4NINAVCNFSM6AAAAABBG6OZUGVHI2DSMVQWIX3LMV43ASLTON2WKOZSGA3DAMZWGQZTGNY.
You are receiving this because you are subscribed to this thread.Message ID: @.***>


send faux email address when creating registry credential

send faux email address when creating registry credential + logging enhancements

OK, the logging change looks good, but there are a few things:

The tests fail I'm guessing because of the email change.
main_test.go needs to be updated to use the mocks package from rancher not objectpartners
We should cut over to trash.yml for deps management
Convert this to dapper/makefile from go-skel.

All Rancher projects follow a similar pattern. You should be able to type make deps,test,build,package,release,ci etc. Releasing this based on someone elses flow is a bit painful.
Also, for future projects, please fork and merge in from your repo. We do not typically do branches in our main rancher/* projects unless its for release management.

This needs the Dockerfile.dapper in the root directory from go-skel.
When its ready you should be able to:
make deps
make test
make build
make package
and it should all work.


Optimizing CSS border rules

Changes

Added test script on package.json to run tests
Removed logs from current tests
Optimize border rules when all sides are defined with the same values.
border: 1px solid #000  becomes {borderWidth: 1, borderStyle: 'solid', borderColor: '#000'}

Hey thanks for the PR!


task switch not allowed from inside gc finalizer

Maybe we need to do something like https://github.com/JuliaLang/julia/pull/33698?
julia> with_logger(TerminalLogger()) do
           @info "hello" sticky=true
       end

julia> @async GC.gc()

error in running finalizer: ErrorException("task switch not allowed from inside gc finalizer")
Task (done) @0x00007f3689b89600

Oh yeah that's nasty :grimacing:
I think we need to improve with_logger so that it sends some kind of "attach" and "detach" events to the logger. (Cf Transducers.start / Transducers.complete)

In the meantime would it work to use Core.write to synchronously reset the scrolling region from the finalizer?

I think we need to improve with_logger so that it sends some kind of "attach" and "detach" events to the logger.

Yeah, I think that's a good idea. I was thinking a similar API for with_logger and global_logger (on_uninstall or something).

use Core.write

This sounds like a nice hack (at least for now).

Oh, I think I misunderstood your suggestion. We could certainly just use @async from the finalizer (ensuring it calls showsticky directly to avoid prolonging the StickyMessages lifetime?)

Both Core.write and @async sound OK to me.  I guess I prefer @async as it's a public API.

Ok, how about #7?  I wasn't sure it made sense to use empty! because that prolongs the life of the sticky messages object. Though I guess I'm prolonging the life of the IO which is held by the StickyMessages which is just as weird in principle. Either way, finalizers are definitely a poor tool for this. (I think that's much clearer to me now than when I originally wrote this code...)

#7 looks good to me. It's nice that showsticky does not need StickyMessages to do this.


Texture Licenses

Readme currently includes:

All other textures by demon_boy github, except those from default game.

Could you please say which license your textures are under?
This is a really interesting mod, but without knowing the texture license it's difficult to do anything with it.

Thank you - and yes, I had tried looking there for them as well with no luck.


why backgroundposter's run methond use loop

A new event might have come in. This is to reduce the number of times the background poster thread is stopped and restarted (which is expensive).
-ut


Nix: Parity with XDPH 9257b0c

Same changes as https://github.com/hyprwm/xdg-desktop-portal-hyprland/pull/75, where applicable.

@fufexan


KOGITO-4215 - add explainability-core test-jar to bom

JIRA: https://issues.redhat.com/browse/KOGITO-4215
This PR aims to add the explainability-core test-jar to the bom.
Related PR on kogito-apps: https://github.com/kiegroup/kogito-apps/pull/605
Many thanks for submitting your Pull Request :heart:!
Please make sure that your PR meets the following requirements:

[x] You have read the contributors guide
[x] Pull Request title is properly formatted: KOGITO-XYZ Subject
[x] Pull Request title contains the target branch if not targeting master: [0.9.x] KOGITO-XYZ Subject
[x] Pull Request contains link to the JIRA issue
[x] Pull Request contains link to any dependent or related Pull Request
[x] Pull Request contains description of the issue
[x] Pull Request does not include fixes for issues other than the main ticket

Not needed anymore

Not needed anymore