html/template: predefined escaper "html" disallowed in template

The Hugo build has started to fail for Go tip recently. Not sure exactly when, but it is green for Go 1.8.1.
ERROR 2017/04/12 19:54:30 Error while rendering "Tag2": html/template:_internal/_default/rss.xml:22:22: predefined escaper "html" disallowed in template

https://travis-ci.org/spf13/hugo/builds/221272152

https://golang.org/cl/37880, I presume.

@bep seems like this was intentional; see https://tip.golang.org/pkg/html/template/ and search for "ErrPredefinedEscaper".

Yes, well, a part of me may understand and even appreciate this change, the maintainer in me would have appreciated a WARNING of sorts instead of a hard fail or a way to handle that error in a less breaking way.
This is a breaking change, and with thousands of user defined templates out there, I don't see how I can fix/deprecate this behaviour without chaos/way too much support work for me and the other maintainers.

CC @stjj89

I discussed this change with @rsc on https://golang.org/cl/37880, and we decided that a explicit breaking change was the best way to resolve this security issue. A warning was not a viable option, since the Go compiler only reports errors (https://golang.org/doc/faq#unused_variables_and_imports), and a non-breaking change would silently modify the output of some templates, which might be harder to track down.
I understand the difficulty of fixing a large number of risky templates in a short period of time. I'm not familiar with your development process or continuous test infrastructure, though. Could you perhaps guarantee compatibility only for Go <1.8.1 until you have fixed these issues?

If this is a security issue, I would expect it to be part of a security patch release, not waiting n months for a Go 1.9 release.

I'm not familiar with your development process or continuous test infrastructure, though. Could you perhaps guarantee compatibility only for Go <1.8.1 until you have fixed these issues?

I'm not sure you understand the use case. I can probably fix the internal failing templates fairly fast, but there are 160 end user provided themes and many thousands of Hugo sites with their own set of templates that I don't control. But I will sure be on the receiving end of the support ticket asking what to do about:

predefined escaper "html" disallowed in template

Which, to be honest, I'm not even sure how to handle myself (but that is beside the point).
I will look more closely into this issue when I get some spare time but think it is fair to say that you would not have deliberately broken thousands of "real" Go programs with a muddy upgrade path like this one.

If this is a security issue, I would expect it to be part of a security patch release, not waiting n months for a Go 1.9 release.

You're right that this is a security issue, and should probably be resolved as per the Go Security Policy. @rsc @ianlancetaylor do either of you think it makes sense to roll this change back, and release it as according to the stated procedure?
This process gives you only 3 days advance notice before the fix lands in the Go repo, so I'm not sure if this would solve your problem with end-user-provided themes.

I'm not sure you understand the use case. I can probably fix the internal failing templates fairly fast, but there are 160 end user provided themes and many thousands of Hugo sites with their own set of templates that I don't control. But I will sure be on the receiving end of the support ticket asking what to do about:

predefined escaper "html" disallowed in template

Which, to be honest, I'm not even sure how to handle myself (but that is beside the point).

I would point your users to the ErrPredefinedEscaper documentation at https://tip.golang.org/pkg/html/template/#ErrorCode. It explains the error in greater detail than the error message, which is meant to be brief.

I will look more closely into this issue when I get some spare time but think it is fair to say that you would not have deliberately broken thousands of "real" Go programs with a muddy upgrade path like this one.

Sorry for the lack of clarity with this change. We can probably make this right by sending out an announcement explaining the issue and the upgrade path on the relevant mailing lists.

This is a change to tip, not to a release.  The security policy only applies to security issues in existing releases.  As far as I am aware, nobody is arguing that we should apply this change to an existing release.  So the security policy does not apply.
People using tip should expect it to break at any time.  That's why we make releases.  The comments about a lack of warning make clear that we need to highlight this issue in the Go 1.9 release notes, so that people can be aware of it when deciding when and whether to upgrade from 1.8 to 1.9.  No warning is required to break people using tip.
Assuming @bep is correct that this will break thousands of real Go programs, the real issues I see here are 1) should we make this change at all?  2) if we do decide to make this change, what can we do to make it easier for users to move forward?
There is clearly something missing from CL 37880: it does not change the documentation.  @stjj89 Can you update the docs for both text/template and html/template to make clear that these global template functions, while still present in text/template, are no longer available in html/template?  The html/template docs have explicit examples of how urlquery and html are used; to me those docs now seem actively misleading.
Would it be possible and reasonable for us to simply ignore a predefined escaper function that appears at the end of a pipeline in the expected contex?.  That is, if we are going to implicitly add | html anyhow, can we simply ignore an existing final | html?  That change, if possible, might mean that many existing templates continue to work without change.
@bep Can you give us some examples of real user templates that are now broken?
Thanks.

@bep Can you give us some examples of real user templates that are now broken?

A count from the Hugo themes repo (https://github.com/spf13/hugoThemes):
~/hugo/hugoThemes  master ✔                                                                                4d
▶ find . -name "*.html" -type f | xargs grep "| html" | wc -l
     235
▶ find . -name "*.xml" -type f | xargs grep "| html" | wc -l
     101

An extract:
./_script/hugoThemeSite/themeSite/static/theme/twentyfourteen/tags/templates/index.html:
{{ index .Params "disqus_url" | html }}
./_script/hugoThemeSite/themeSite/static/theme/twentyfourteen/tags/themes/index.html:
{{ index .Params "disqus_url" | html }}
./_script/hugoThemeSite/themeSite/static/theme/type/post/go-is-for-lovers/index.html:
{{ index .Params "disqus_url" | html }}
./_script/hugoThemeSite/themeSite/static/theme/vienna/post/goisforlovers/index.html:
{{ index .Params "disqus_url" | html }}
./academic/layouts/partials/share.html:         href="https://www.facebook.com/sharer.php?u={{ .Permalink | html }}"
./academic/layouts/partials/share.html:         href="https://twitter.com/intent/tweet?text={{ .Title | html }}&url={{ .Permalink | html }}"
./academic/layouts/partials/share.html:         href="https://www.linkedin.com/shareArticle?mini=true&url={{ .Permalink | html }}&title={{ .Title | html }}"
./academic/layouts/partials/share.html:         href="http://service.weibo.com/share/share.php?url={{ .Permalink | html }}&title={{ .Title | html }}"
./academic/layouts/partials/share.html:         href="mailto:?subject={{ .Title | html }}&body={{ .Permalink | html }}">
./bleak/layouts/partials/header.html:      var disqus_url = '{{if isset $doc.Params "disqus_url" }}{{ index $doc.Params "disqus_url" | html  }}{{ else }}{{ $doc.Permalink }}{{end}}';
./crisp/layouts/partials/comments.html:      var disqus_url = '{{if isset $doc.Params "disqus_url" }}{{ index $doc.Params "disqus_url" | html  }}{{ else }}{{ $doc.Permalink }}{{end}}';
./grid-side/layouts/partials/extra/disqus.html:        var disqus_url = '{{ with .GetParam "disqus_url" }}{{ . | html  }}{{ else }}{{ .Permalink }}{{ end }}';
./html5/layouts/partials/article/part/content.html:						{{ .RawContent | htmlEscape }}
./hugo-finite/layouts/partials/share.html:
./hugo-finite/layouts/partials/share.html:
./hugo-finite/layouts/partials/share.html:
./hugo-finite/layouts/partials/share.html:
./hugo-finite/layouts/partials/share.html:

Note that the above is the part I can control.  But then there are plenty of sites with their own set of templates, and believe me, they will cry for help.
I tried to add my own no-op html func myself, which would have solved my problem, because then I could apply my own deprecation policy.

Assuming @bep is correct that this will break thousands of real Go programs, the real issues I see here are 1) should we make this change at all? 2) if we do decide to make this change, what can we do to make it easier for users to move forward?

I still think we should make this change because of the security issues #19336, #19345, #19353. Moreover, the current escaper-merging logic violates the documented "Least Surprise Property", since it is unclear what sanitization takes place when the user second-guesses the contextual autoescaper. On the other hand, no actual issues have been found in the wild yet, and that preemptively fixing, so we could conclude that it isn't worth the trouble fixing this. I think it's better to err to the side of caution. @rsc and I discussed some options for moving forward in the CL (see here and here, and we concluded that the change is worth making, and that explicitly breaking people is better than silently breaking them.

See response below.

There is clearly something missing from CL 37880: it does not change the documentation. @stjj89 Can you update the docs for both text/template and html/template to make clear that these global template functions, while still present in text/template, are no longer available in html/template? The html/template docs have explicit examples of how urlquery and html are used; to me those docs now seem actively misleading.

Thanks for pointing this out. I will get to this after we decide on an agreeable solution.

Would it be possible and reasonable for us to simply ignore a predefined escaper function that appears at the end of a pipeline in the expected contex?. That is, if we are going to implicitly add | html anyhow, can we simply ignore an existing final | html? That change, if possible, might mean that many existing templates continue to work without change.

I like this idea. My only concern is that this "double standard" for using predefined escapers (i.e. not in the middle, but only at the end). Perhaps an announcement e-mail and updated documentation will be enough to clear this up?

To be clear, part of the plan here is to see what happens during release testing and maybe adjust the restrictions or even roll them back and try again.
@stjj89, allowing and ignoring |html where it has never had any effect at all sounds OK.


Updated EPM analysis notebook and solutions

The notebook has been updated according to movement v0.0.20.
I opted for teaching the functional syntax for filtering and kinematics (as opposed to  using the accessor), because it requires less explanation.
@lochhh I'd be grateful if you could go over it once to check for bugs and types.

Thanks @lochhh, all good calls!


[FLINK-33460][Connector/JDBC] Support property authentication connection.

Support property authentication connection.

hi, @snuyanzin @eskabetxe Could you help to have a review if you had the free time ?
Thank you~

hi, @Jiabao-Sun master, Could you help to have a review if you had the free time ?
Thank you~ 😄


check propagate status

alert user if the propagate status is not propagated, since image data isn't dynamically downsampled (@jovo )

http://brainviz1.cs.jhu.edu/ndv/kharris15apical/em,annos/


Ci Runners for Windows & Mac

CI for Windows and Mac Intel & Arm
Add build for lk_sandbox
wxWidgets binaries are installed from wxWidgets repo for Windows and brew for Mac. This is much faster than building it from source.

For sure the developers should be using the version from the wiki. But if we can build and test SAM with different versions of wxWidgets in a fraction of the time, it seems worthwhile to mix versions for CI.

I can fix the install from brew to 3.2.3 to reduce that version confusion.

However, it seems like downloading the binaries directly for wex and SAM isn't going to work. It looks SAM is using a static build of wx but the binary downloads will be dll builds of wx. Not sure how easy it is to flip that on and off, or whether it is worth it in order to reduce the amount of time it takes to run the SAM test on CI. If we could use the dll releases of wx, the SAM tests would be quite a bit faster given how long the wxWidgets build takes.


Add PerformanceWarning

From the feature in CuPy (cupy/cupy#1607).
background: #5444

LGTM. Found #6618 during review.

Jenkins, test this please.

Is utils appropriate? How about chainer.warnings?

Jenkins CI test (for commit 884506201a1b68ffa4daf9e07be2254d9b612967, target branch master) failed with status FAILURE.
(For contributors, please wait until the reviewer confirms the details of the error.)

chainer.utils is not very appropriate, but the module has experimental and nondeterministic too.  I'd like to move them to chainer.warnings_.

experimental and nondeterministic are functions, not warnings.
I think it's ok to leave them in utils.
As for PerformanceWarning, I'm fine with either of the following:

Put in chainer.warnings_ and alias as chainer.PerformanceWarning.
Put in chainer.warnings and use as chainer.warnings.PerformanceWarning.

Jenkins, test this please.

Jenkins CI test (for commit 11cfafeb977f3149163e3340745a6566fdbd4bca, target branch master) failed with status FAILURE.
(For contributors, please wait until the reviewer confirms the details of the error.)

Jenkins, test this please.

Jenkins CI test (for commit f8520d9a261e20333471013d04690befc3e74807, target branch master) failed with status FAILURE.
(For contributors, please wait until the reviewer confirms the details of the error.)

Jenkins, test this please.

Jenkins CI test (for commit f8520d9a261e20333471013d04690befc3e74807, target branch master) failed with status FAILURE.
(For contributors, please wait until the reviewer confirms the details of the error.)

Jenkins, test this please.

Jenkins CI test (for commit 8a5b2714316c5a9e25bb53fe3999afa467c67bfe, target branch master) failed with status FAILURE.
(For contributors, please wait until the reviewer confirms the details of the error.)

LGTM! Test failures are not related to this PR.


Adding the same card more than once doesn't stack

I.e. this should never happen:

Locally fixed by https://github.com/matanlurey/superlaser/commit/6a67613d9677fb1cbb0b0d225d04d66863ddf2f0, closing optimistically.


thebe integration

PR that updates myst-theme shared packages to add support for thebe
See also: https://github.com/executablebooks/myst-book-theme/pull/4
TODO:

[ ] better scroll handler @rowanc1
[ ] fix patch package - path.posix issue - @stevejpurves
[ ] do something about jest failures
[ ] bug - enabling compute on a notebook, clears ipywidgets outputs in the notebooks and in an article that embeds them

Wow, the Thebe support is super exciting. I can't wait to try out this new feature ✨


Ensure compiler daemons are reused with toolchains involved

@melix indicates that we may not reuse the compiler daemons between runs for incremental compilation when toolchains are in the mix. Ensure this is the case

In order to test-drive this change, we'll wait until the whole BT build is using toolchains so we can dog-food this. /cc @ljacomet

Closing as obsolete


Undefined options bug

Addressing #649, there was a gap in the resolveArguments logic that wouldn't let options be set if properties were undefined, as the original logic did not account for the case where properties wouldn't be defined.

Tested, this appears to work! Less complicated than I expected! Thanks!


Crash

There is a crash in [Harpy bundleID] method. Only 1 user out of 5k is affected so far.
Crashlytics: http://crashes.to/s/671bd5cc406

The bundleID method looks like this
- (NSString *)bundleID {
    return [NSBundle mainBundle].bundleIdentifier;
}

bundleIdentifier is a property Apple built into NSBundle within Foundation. This is one of those rare Apple-caused crashes.
The fact that it's 1/5000 means that it was a fluke, probably caused by the OS in some way.
Nothing I can do - sorry!


Entering  in variable name editor shouldn't close the editor

Entering a variable name containing a space is difficult as the name editor closes straight after pressing !   However, the space is there when you open it again, so this is rather a strange behaviour and inconsistent.  It does allow you to add another word to the end (and hence have a space in the variable name) but it's a awkward way to achieve this. There is an 'OK button' to press when done, so why close on  anyway?
Google Blockly is OK (on the same Chrome browser) -

Our backend is typescript and we don't support space in the variable name. This is by design.

OK, then why allow users to enter a ?  As I said, the  is appended to the variable name and upon rename, users can type another word and hence have a space in the variable name.  I suggest fixing to ignore  characters

{space within arrow brackets doesn't display}  OK, then why allow users to enter a space? As I said, the spaceis appended to the variable name and upon rename, users can type another word and hence have a space in the variable name. I suggest fixing to ignore characters

I would argue that the ticket is valid in the sense that having the modal close itself when the space key is pressed is a bit of an unexpected reaction.
Not only that, but it creates a variable name with the first word, there is no indication of what happened or why it happened, and so I found it slightly confusing the first time I encountered it.
Here is an example of a user would experience when trying to create a variable named my variable:

Blockly gets around this issue by replacing the spaces, and passing the variable name through a "unique name generator" function. I'm not saying MakeCode should do the same, but it would be useful if it was clear that spaces are not permitted.

I am reopening based on feedback.

This should be fixed now. Please verify


Updating Xamarin.Android version results into Embeddination errors

Steps to Reproduce

Use latest version or any version greater than v<IP_ADDRESS> of Xamarin.Android and perform Embeddination on a simple class.
Also, you might need to correct the folder structure of Xamarin.Android, according to Embeddination needs. I wanted to upload mine but its quite bigger than 10MB's. Get it from here and correct the folder structure.

Latest-folder-structure:

Xamarin.Android
-- include
-- lib
--- xamarin.android
---- xbuild
---- xbuild-frameworks

Folder-structure-Embeddination-needs:

Xamarin.Android
-- include
-- lib
---- xbuild
---- xbuild-frameworks

Expected Behavior
Embedination shouldn't fail.
Actual Behavior
Embedination fails.
Environment
MacOS, VS for Mac.

using System;

namespace Testembedination
{
    public class TEmbeded : Java.Lang.Object
    {
        public string test { get; set; }

    }
}

Build Logs

error MSB4018: The "GenerateJavaStubs" task failed unexpectedly.
/Users/xamarindev/sts-vasapp/Native/Android/EmbedinatedFramework/GenerateJavaStubs.proj(17,5): error MSB4018: System.NullReferenceException: Object reference not set to an instance of an object
/Users/xamarindev/sts-vasapp/Native/Android/EmbedinatedFramework/GenerateJavaStubs.proj(17,5): error MSB4018:   at Xamarin.Android.Tasks.ManifestDocument.Merge (System.Collections.Generic.List`1[T] subclasses, System.Collections.Generic.List`1[T] selectedWhitelistAssemblies, System.String applicationClass, System.Boolean embed, System.String bundledWearApplicationName, System.Collections.Generic.IEnumerable`1[T] mergedManifestDocuments) [0x00320] in <420d87f47a0d49a8b9818d1b84d19295>:0
/Users/xamarindev/sts-vasapp/Native/Android/EmbedinatedFramework/GenerateJavaStubs.proj(17,5): error MSB4018:   at Xamarin.Android.Tasks.GenerateJavaStubs.Run (Java.Interop.Tools.Cecil.DirectoryAssemblyResolver res) [0x0047d] in <420d87f47a0d49a8b9818d1b84d19295>:0
/Users/xamarindev/sts-vasapp/Native/Android/EmbedinatedFramework/GenerateJavaStubs.proj(17,5): error MSB4018:   at Xamarin.Android.Tasks.GenerateJavaStubs.Execute () [0x00223] in <420d87f47a0d49a8b9818d1b84d19295>:0
/Users/xamarindev/sts-vasapp/Native/Android/EmbedinatedFramework/GenerateJavaStubs.proj(17,5): error MSB4018:   at Microsoft.Build.BackEnd.TaskExecutionHost.Microsoft.Build.BackEnd.ITaskExecutionHost.Execute () [0x00023] in <99064ce461714f6a83a1f24370e1d7b8>:0
/Users/xamarindev/sts-vasapp/Native/Android/EmbedinatedFramework/GenerateJavaStubs.proj(17,5): error MSB4018:   at Microsoft.Build.BackEnd.TaskBuilder+<ExecuteInstantiatedTask>d__26.MoveNext () [0x00212] in <99064ce461714f6a83a1f24370e1d7b8>:0

Example Project (If Possible)

TestEmbedination.zip
TestComand.bat.zip

Updating to latest version of Xamarin.Android can fix many problems like I posted issue here. If someone here can support me to update the Xamarin.Android version to latest, I can contribute in updating the cake script to support the newer versions of Xamarin.Android. Currently the script needs few refractions for downloading and unzipping the new version of Xamarin.Android correctly. If you try to update the Xamarin.Android version using cake script, the build will fail since the folder structure is changed.

@emysa341 this is something we've been needing to do for a while now, I've been having to focus on Xamarin.Android itself lately.
The problem with updating the version of Xamarin.Android, is there is usually changes in the MSBuild tasks Embeddinator is using. The only good way to fix that is to add Embeddinator targets to Xamarin.Android itself and rely on those. Not something I've gotten around to yet.
You are originally trying to fix #655 , we need to track down the change in Xamarin.Android that fixed it. Do you know the version number or PR number that fixed it in Xamarin.Android? Java.Interop?
I might be able to make a small bump for Xamarin.Android in Embeddinator to fix this.

@jonathanpeppers I am not sure in which VN they fixed it. I assumed it from your comment on a previous issue that updating Xamarin.Android can resolve many issued related to Embeddination. I will explore at Xamarin.Android github's, I will report if I manage to find the it.


Update datadog.md

changes for authentication is updated in datadog.md file

Fixed merge issues I think


Windows Cuda Compatibility?

On windows unable to use candle with cuda enabled. I'm using Cuda SDK 12.4 since that is what is supported by my pytorch install which runs fine. 🤔 Is Windows supported? Is there a specific Cuda SDK version that should be used? Not sure why it doesn't compile. There are hundreds of errors. Log below, but could be longer probably.
  C:\Program Files\NVIDIA GPU Computing Toolkit\CUDA\v12.4\include\sm_32_intrinsics.hpp(117): error: asm operand type size(8) does not match type/size implied by constraint 'r'
    static __declspec(__device__) __inline short4 __ldg(const short4 *ptr) { short4 ret; asm volatile ("ld.global.nc.v4.s16 {%0,%1,%2,%3}, [%4];"  : "=h"(ret.x), "=h"(ret.y), "=h"(ret.z), "=h"(ret.w) : "r" (ptr)); return ret; }
                                                                                ^

  C:\Program Files\NVIDIA GPU Computing Toolkit\CUDA\v12.4\include\sm_32_intrinsics.hpp(118): error: asm operand type size(8) does not match type/size implied by constraint 'r'
    static __declspec(__device__) __inline int2 __ldg(const int2 *ptr) { int2 ret; asm volatile ("ld.global.nc.v2.s32 {%0,%1}, [%2];"  : "=r"(ret.x), "=r"(ret.y) : "r" (ptr)); return ret; }
                                                                                ^

  C:\Program Files\NVIDIA GPU Computing Toolkit\CUDA\v12.4\include\sm_32_intrinsics.hpp(119): error: asm operand type size(8) does not match type/size implied by constraint 'r'
    static __declspec(__device__) __inline int4 __ldg(const int4 *ptr) { int4 ret; asm volatile ("ld.global.nc.v4.s32 {%0,%1,%2,%3}, [%4];"  : "=r"(ret.x), "=r"(ret.y), "=r"(ret.z), "=r"(ret.w) : "r" (ptr)); return ret; }
                                                                                ^

  C:\Program Files\NVIDIA GPU Computing Toolkit\CUDA\v12.4\include\sm_32_intrinsics.hpp(120): error: asm operand type size(8) does not match type/size implied by constraint 'r'
    static __declspec(__device__) __inline longlong2 __ldg(const longlong2 *ptr) { longlong2 ret; asm volatile ("ld.global.nc.v2.s64 {%0,%1}, [%2];"  : "=l"(ret.x), "=l"(ret.y) : "r" (ptr)); return ret; }
                                                                                ^

  Error limit reached.
  100 errors detected in the compilation of "src/cast.cu".
  Compilation terminated.
  cast.cu

  --- stderr
  thread 'main' panicked at C:\Users\Chase\.cargo\registry\src\index.crates.io-6f17d22bba15001f\bindgen_cuda-0.1.5\src\lib.rs:391:13:
  nvcc error while compiling "src\\affine.cu":

  # CLI "nvcc" "--gpu-architecture=sm_89" "--ptx" "--default-stream" "per-thread" "--output-directory" "E:\\Dev\\WebNN\\target\\debug\\build\\candle-kernels-fc639e1b2bb65e3d\\out" "-Isrc" "-IC:\\Program Files\\NVIDIA GPU Computing Toolkit\\CUDA\\v12.4\\include" "src\\affine.cu"

  # stdout

  # stderr

  note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
warning: build failed, waiting for other jobs to finish...

compilation errors are sometimes because missing libraries in the path. do you have the proper coda paths set.


Added API and docker

Hello,
I have made some additions to the repository to enhance its functionality. This pull request adds Docker Compose support and an API implementation. Here is a brief overview of the changes:
Added Docker Compose file: The Docker Compose file (docker-compose.yml) is included to simplify the setup and deployment of the project. It defines the necessary services and their configurations, allowing for easy containerization and orchestration.
API Implementation: I have added an API to the project to provide a programmatic interface for interacting with the application. The API endpoints are implemented according to the project requirements and specifications.
I believe these additions will improve the overall usability and maintainability of the project. Please review the changes and provide any feedback or suggestions for improvement.
Thank you for your attention to this pull request. I look forward to your feedback and the opportunity to contribute to the project.
Best regards,
Mahdi

@mahdikiani
First off, I appreciate your effort and the contributions you're making to the project. It's clear you've put thought into how these changes might enhance the project's functionality.
That being said, I've been using the existing API across multiple Next.js applications, and from my perspective, it's performing perfectly. It's efficient, straightforward to use, and satisfies all the necessary requirements for my projects.
I'm slightly concerned about the introduction of Docker Compose and an additional API layer. While these might seem like improvements in some aspects, they could potentially complicate a project that is currently easy to work with.
This is just my viewpoint based on my usage scenarios. If you could shed more light on the advantages that these changes would bring to various use cases, it might help facilitate more understanding of your proposed enhancements.
Thanks again for your work and for considering my feedback. I look forward to discussing this further.


Dialog Component - Add TextListComponent to support list of text

Add a new Component to hold a list of strings with a minimal count
Implement a specific renderer hook
Load and dump the list from and to the answer file as a json
Add test cases for the dialog components
Add test cases for the answerstore

review please @oamg/developers

Hi, thank you for your contribution @ygalblum! :rocket:
Could you please update the description with some additional information about the reason behind the changes?

@ygalblum @PeterMocary  we need to discuss this yet as the use case behind this feature is aside of the original purpose of Dialogs. Please do not do other changes until we discuss that. We are now affected by vacations so waiting for a time most of the team is back to discuss it.
Below sharing the use case behind the PR:

An actor needs to get multiple URLs from a user. The number of URLs is not known in advance. So the point is to get all URLs that customer needs to be used for something and consumed by an actor.

The possible solution (that is nowadays more expected regarding the dialogs design) is to tell user to create a file under /etc/leapp/files/ that will address such a problem. In this case actors can simply read the content of such a configuration file and work with it as needed.

@pirat89 thanks for the detailed comment and yes, you are correct, this was the reason for this PR.
The reason I prefer this solution over your proposal is that I think that users will prefer to provide all the information in one place instead of multiple files. Don't think only about my actor. Once this becomes a known practice, what stops the next actor from getting answers in another file?
As for the way it was implemented, i.e. JSON inside an INI file, I looked around for how arrays are implemented in INI files and this was the most common and elegant way I found.
Having said that, because of your response (in the chat) and knowing that even if this change is merged, it will take time until it gets released, I decided to go with a hybrid solution. That is, the actor uses a JSON file to get the information it needs, but it does not force a location. Instead, it uses a Text dialog to get a path or URL to get the json from.
The advantage I saw in this solution is that it allows the user to maintain this JSON file next to where the files, to which the URLs are needed to being with, are located.

@ygalblum We have discussed this in the team and we agreed this is not way we want to go in future. We want to keep dialogs for purposes they were designed and do not want to extend them in this way. Also we do not want to mix multiple formats in one file - but this was just a minor fact for the decision as that part could be done differently. In this case we suggest to go the way I've proposed above - e.g.

ask for a path to the file in the dialog
just inhibit the upgrade and inform user to create the file on a specified path using the specified format

Closing, we are sorry about that but thanks for your time and the contribution.


Unhandled Exception: System.ArgumentException: Illegal characters in path.

Hi.
Thank you for Obfuscar 2.0 rc7.
Windows 7 x64 SP1
From within a command prompt, running:
Obfuscar.Console.exe /?
Causes the following output:
Obfuscar is available at http://obfuscar.codeplex.com.
(C) 2007-2014, Ryan Williams and other contributors.
Loading project...
Unhandled Exception: System.ArgumentException: Illegal characters in path.
at System.IO.Path.CheckInvalidPathChars(String path, Boolean checkAdditional)
at System.IO.FileStream.Init(String path, FileMode mode, FileAccess access, I
nt32 rights, Boolean useRights, FileShare share, Int32 bufferSize, FileOptions o
ptions, SECURITY_ATTRIBUTES secAttrs, String msgPath, Boolean bFromProxy, Boolea
n useLongPath, Boolean checkHost)
at System.IO.FileStream..ctor(String path, FileMode mode, FileAccess access,
FileShare share, Int32 bufferSize, FileOptions options, String msgPath, Boolean
bFromProxy)
at System.IO.FileStream..ctor(String path, FileMode mode, FileAccess access,
FileShare share)
at Obfuscar.Obfuscator..ctor(String projfile)
at Obfuscar.Program.Main(String[] args)
And produces the following dialog to appear:
[Window Title]
Obfuscar Console Utility
[Main Instruction]
Obfuscar Console Utility has stopped working
[Content]
A problem caused the program to stop working correctly. Windows will close the program and notify you if a solution is available.
[Debug] [Close program]
I expected to see "usage" information for the console application command line parameters/options instead of a crash.
Thank you.
Original Reported Date: 2014-05-07T17:56:39.697-07:00
Planned For Release:
Original CodePlex ID: 9

Copied from CodePlex without authors:
This is desired, as /? is not yet supported. Will consider it in vNext.
Thanks for the report.
Original Posted Date: 2014-05-31T19:12:56.11-07:00

Copied from CodePlex without authors:
Fixed in 2.0.2 beta.
Original Posted Date: 2015-04-17T23:41:05.687-07:00

Original Closed Date: 2015-04-17T23:41:12.63-07:00


How I can save media data when streamming as h.264*(mp4)?

Hi!
We currently do not provide an API for doing media recording, but you could modify our SDK to do so. You would need to take a look at the RespokeCall class and modify it to extract the video from the WebRTC classes directly. I have not done this myself, so I am not entirely sure what would be involved. A number of other people are working on similar things, and a good place to start would be to look through the Discuss-WebRTC forums and post a question:
https://groups.google.com/forum/#!forum/discuss-webrtc
Thanks,
Jason

Thanks .I did this :)


Is it safe to run this command?

I did the installation on my iMac.  I can run smckit perfectly, but my HDD fan shows Current: 0 RPM.
That is my problem I changed my HD because the original one failed. I put a Western Digital and the sensor cable did not match. I got one for the Western Digital but it seems is not reporting temp data back.  The HD fan is running high.  Can I use this command to set the speed of the HD fan:
sudo smckit -n 1 -s 1102? Is it safe to run this?

Hello @josejmoran!
Sorry about the late response!
I'm surprised the HD fan would drop 0 RPM even though its not getting a reading from temperature sensor.

Can I use this command to set the speed of the HD fan: sudo smckit -n 1 -s 1102?

Yes! That is the correct command assuming you want to set the second fan.

Is it safe to run this?

There are no guarantees of course, especially when tinkering with hardware, but it should be a safe command to run. It simply sets the minimum fan speed. OS X itself will interject if it needs to raise it. In addition, it doesn't persist after reboot. Finally, you can always reset your SMC incase of any issues.
You might also want to cross-check with the Apple Hardware Test/Diagnostics incase there is some other issue here.


[BUG] opensearch process exits during addition of documents with knn_vector field

Describe the bug
While adding ~1.3M documents with a knn_vector type field, the opensearch process exited with KNN related errors.  This happened after adding approximately 800K documents.  The document mapping and index info are below.   I was using the latest docker image (version data below), with a single mode.
Errors in log
(there were tens of these error pairs before the process exited)
[2021-07-30T18:36:47,770][WARN ][o.o.t.ThreadPool         ] [opensearch-node1] failed to run scheduled task [org.opensearch.indices.IndexingMemoryController$ShardsIndicesStatusChecker@4027ffab] on thread pool [same]
org.apache.lucene.store.AlreadyClosedException: this IndexWriter is closed
	at org.apache.lucene.index.IndexWriter.ensureOpen(IndexWriter.java:877) ~[lucene-core-8.8.2.jar:8.8.2 a92a05e195b775b30ca410bc0a26e8e79e7b3bfb - mdrob - 2021-04-06 16:33:27]
	at org.apache.lucene.index.IndexWriter.ensureOpen(IndexWriter.java:891) ~[lucene-core-8.8.2.jar:8.8.2 a92a05e195b775b30ca410bc0a26e8e79e7b3bfb - mdrob - 2021-04-06 16:33:27]
	at org.apache.lucene.index.IndexWriter.getFlushingBytes(IndexWriter.java:781) ~[lucene-core-8.8.2.jar:8.8.2 a92a05e195b775b30ca410bc0a26e8e79e7b3bfb - mdrob - 2021-04-06 16:33:27]
	at org.opensearch.index.engine.InternalEngine.getWritingBytes(InternalEngine.java:635) ~[opensearch-1.0.0.jar:1.0.0]
	at org.opensearch.index.shard.IndexShard.getWritingBytes(IndexShard.java:991) ~[opensearch-1.0.0.jar:1.0.0]
	at org.opensearch.indices.IndexingMemoryController.getShardWritingBytes(IndexingMemoryController.java:195) ~[opensearch-1.0.0.jar:1.0.0]
	at org.opensearch.indices.IndexingMemoryController$ShardsIndicesStatusChecker.runUnlocked(IndexingMemoryController.java:323) ~[opensearch-1.0.0.jar:1.0.0]
	at org.opensearch.indices.IndexingMemoryController$ShardsIndicesStatusChecker.run(IndexingMemoryController.java:303) ~[opensearch-1.0.0.jar:1.0.0]
	at org.opensearch.threadpool.Scheduler$ReschedulingRunnable.doRun(Scheduler.java:226) [opensearch-1.0.0.jar:1.0.0]
	at org.opensearch.common.util.concurrent.ThreadContext$ContextPreservingAbstractRunnable.doRun(ThreadContext.java:756) [opensearch-1.0.0.jar:1.0.0]
	at org.opensearch.common.util.concurrent.AbstractRunnable.run(AbstractRunnable.java:50) [opensearch-1.0.0.jar:1.0.0]
	at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:515) [?:?]
	at java.util.concurrent.FutureTask.run(FutureTask.java:264) [?:?]
	at java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.run(ScheduledThreadPoolExecutor.java:304) [?:?]
	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1130) [?:?]
	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:630) [?:?]
	at java.lang.Thread.run(Thread.java:832) [?:?]
Caused by: java.lang.RuntimeException: java.lang.Exception: Check failed: it's either a bug or inconsistent data!
	at org.opensearch.knn.index.codec.KNN80Codec.KNN80DocValuesConsumer.merge(KNN80DocValuesConsumer.java:165) ~[?:?]
	at org.apache.lucene.codecs.perfield.PerFieldDocValuesFormat$FieldsWriter.merge(PerFieldDocValuesFormat.java:155) ~[lucene-core-8.8.2.jar:8.8.2 a92a05e195b775b30ca410bc0a26e8e79e7b3bfb - mdrob - 2021-04-06 16:33:27]
	at org.apache.lucene.index.SegmentMerger.mergeDocValues(SegmentMerger.java:195) ~[lucene-core-8.8.2.jar:8.8.2 a92a05e195b775b30ca410bc0a26e8e79e7b3bfb - mdrob - 2021-04-06 16:33:27]
	at org.apache.lucene.index.SegmentMerger.merge(SegmentMerger.java:150) ~[lucene-core-8.8.2.jar:8.8.2 a92a05e195b775b30ca410bc0a26e8e79e7b3bfb - mdrob - 2021-04-06 16:33:27]
	at org.apache.lucene.index.IndexWriter.mergeMiddle(IndexWriter.java:4760) ~[lucene-core-8.8.2.jar:8.8.2 a92a05e195b775b30ca410bc0a26e8e79e7b3bfb - mdrob - 2021-04-06 16:33:27]
	at org.apache.lucene.index.IndexWriter.merge(IndexWriter.java:4364) ~[lucene-core-8.8.2.jar:8.8.2 a92a05e195b775b30ca410bc0a26e8e79e7b3bfb - mdrob - 2021-04-06 16:33:27]
	at org.apache.lucene.index.IndexWriter$IndexWriterMergeSource.merge(IndexWriter.java:5923) ~[lucene-core-8.8.2.jar:8.8.2 a92a05e195b775b30ca410bc0a26e8e79e7b3bfb - mdrob - 2021-04-06 16:33:27]
	at org.apache.lucene.index.ConcurrentMergeScheduler.doMerge(ConcurrentMergeScheduler.java:624) ~[lucene-core-8.8.2.jar:8.8.2 a92a05e195b775b30ca410bc0a26e8e79e7b3bfb - mdrob - 2021-04-06 16:33:27]
	at org.opensearch.index.engine.OpenSearchConcurrentMergeScheduler.doMerge(OpenSearchConcurrentMergeScheduler.java:113) ~[opensearch-1.0.0.jar:1.0.0]
	at org.apache.lucene.index.ConcurrentMergeScheduler$MergeThread.run(ConcurrentMergeScheduler.java:682) ~[lucene-core-8.8.2.jar:8.8.2 a92a05e195b775b30ca410bc0a26e8e79e7b3bfb - mdrob - 2021-04-06 16:33:27]
Caused by: java.lang.Exception: Check failed: it's either a bug or inconsistent data!
	at org.opensearch.knn.index.v2011.KNNIndex.saveIndex(Native Method) ~[?:?]
	at org.opensearch.knn.index.codec.KNN80Codec.KNN80DocValuesConsumer$1.run(KNN80DocValuesConsumer.java:117) ~[?:?]
	at org.opensearch.knn.index.codec.KNN80Codec.KNN80DocValuesConsumer$1.run(KNN80DocValuesConsumer.java:115) ~[?:?]
	at java.security.AccessController.doPrivileged(AccessController.java:312) ~[?:?]
	at org.opensearch.knn.index.codec.KNN80Codec.KNN80DocValuesConsumer.addKNNBinaryField(KNN80DocValuesConsumer.java:114) ~[?:?]
	at org.opensearch.knn.index.codec.KNN80Codec.KNN80DocValuesConsumer.merge(KNN80DocValuesConsumer.java:161) ~[?:?]
	at org.apache.lucene.codecs.perfield.PerFieldDocValuesFormat$FieldsWriter.merge(PerFieldDocValuesFormat.java:155) ~[lucene-core-8.8.2.jar:8.8.2 a92a05e195b775b30ca410bc0a26e8e79e7b3bfb - mdrob - 2021-04-06 16:33:27]
	at org.apache.lucene.index.SegmentMerger.mergeDocValues(SegmentMerger.java:195) ~[lucene-core-8.8.2.jar:8.8.2 a92a05e195b775b30ca410bc0a26e8e79e7b3bfb - mdrob - 2021-04-06 16:33:27]
	at org.apache.lucene.index.SegmentMerger.merge(SegmentMerger.java:150) ~[lucene-core-8.8.2.jar:8.8.2 a92a05e195b775b30ca410bc0a26e8e79e7b3bfb - mdrob - 2021-04-06 16:33:27]
	at org.apache.lucene.index.IndexWriter.mergeMiddle(IndexWriter.java:4760) ~[lucene-core-8.8.2.jar:8.8.2 a92a05e195b775b30ca410bc0a26e8e79e7b3bfb - mdrob - 2021-04-06 16:33:27]
	at org.apache.lucene.index.IndexWriter.merge(IndexWriter.java:4364) ~[lucene-core-8.8.2.jar:8.8.2 a92a05e195b775b30ca410bc0a26e8e79e7b3bfb - mdrob - 2021-04-06 16:33:27]
	at org.apache.lucene.index.IndexWriter$IndexWriterMergeSource.merge(IndexWriter.java:5923) ~[lucene-core-8.8.2.jar:8.8.2 a92a05e195b775b30ca410bc0a26e8e79e7b3bfb - mdrob - 2021-04-06 16:33:27]
	at org.apache.lucene.index.ConcurrentMergeScheduler.doMerge(ConcurrentMergeScheduler.java:624) ~[lucene-core-8.8.2.jar:8.8.2 a92a05e195b775b30ca410bc0a26e8e79e7b3bfb - mdrob - 2021-04-06 16:33:27]
	at org.opensearch.index.engine.OpenSearchConcurrentMergeScheduler.doMerge(OpenSearchConcurrentMergeScheduler.java:113) ~[opensearch-1.0.0.jar:1.0.0]
	at org.apache.lucene.index.ConcurrentMergeScheduler$MergeThread.run(ConcurrentMergeScheduler.java:682) ~[lucene-core-8.8.2.jar:8.8.2 a92a05e195b775b30ca410bc0a26e8e79e7b3bfb - mdrob - 2021-04-06 16:33:27]
Killing performance analyzer process 11
OpenSearch exited with code 137
Performance analyzer exited with code 143
./opensearch-docker-entrypoint.sh: line 109:    10 Killed                  $OPENSEARCH_HOME/bin/opensearch "${opensearch_opts[@]}

Mapping info
{
  "test_knn" : {
    "aliases" : { },
    "mappings" : {
      "dynamic" : "true",
      "properties" : {
        "end" : {
          "type" : "integer"
        },
        "id" : {
          "type" : "keyword"
        },
        "index" : {
          "type" : "keyword"
        },
        "start" : {
          "type" : "integer"
        },
        "text" : {
          "type" : "keyword"
        },
        "type" : {
          "type" : "keyword"
        },
        "vector" : {
          "type" : "knn_vector",
          "dimension" : 768,
          "method" : {
            "engine" : "nmslib",
            "space_type" : "innerproduct",
            "name" : "hnsw",
            "parameters" : {
              "ef_construction" : 512,
              "m" : 24
            }
          }
        }
      }
    },
    "settings" : {
      "index" : {
        "number_of_shards" : "1",
        "knn.algo_param" : {
          "ef_search" : "100"
        },
        "provided_name" : "test_knn",
        "knn" : "true",
        "creation_date" : "1627660076506",
        "number_of_replicas" : "0",
        "uuid" : "FZj47j44TUyarSBU3ZjMaw",
        "version" : {
          "created" : "135217827"
        }
      }
    }
  }
}

Version info
{
  "name" : "opensearch-node1",
  "cluster_name" : "opensearch-cluster",
  "cluster_uuid" : "IeOOVXJoRACXnucMfKNzqQ",
  "version" : {
    "distribution" : "opensearch",
    "number" : "1.0.0",
    "build_type" : "tar",
    "build_hash" : "34550c5b17124ddc59458ef774f6b43a086522e3",
    "build_date" : "2021-07-02T23:22:21.383695Z",
    "build_snapshot" : false,
    "lucene_version" : "8.8.2",
    "minimum_wire_compatibility_version" : "6.8.0",
    "minimum_index_compatibility_version" : "6.0.0-beta1"
  },
  "tagline" : "The OpenSearch Project: https://opensearch.org/"
}

To Reproduce
Unclear.  This happens sporadically during loading large number of documents. Documents are being added  via the Python bulk update API  (which appears to hit the server with 500 documents at a time).
Expected behavior
Opensearch process doesn't exit
Plugins
opensearch-node1 opensearch-alerting             <IP_ADDRESS>
opensearch-node1 opensearch-anomaly-detection    <IP_ADDRESS>
opensearch-node1 opensearch-asynchronous-search  <IP_ADDRESS>
opensearch-node1 opensearch-index-management     <IP_ADDRESS>
opensearch-node1 opensearch-job-scheduler        <IP_ADDRESS>
opensearch-node1 opensearch-knn                  <IP_ADDRESS>
opensearch-node1 opensearch-notebooks            <IP_ADDRESS>
opensearch-node1 opensearch-performance-analyzer <IP_ADDRESS>
opensearch-node1 opensearch-reports-scheduler    <IP_ADDRESS>
opensearch-node1 opensearch-security             <IP_ADDRESS>
opensearch-node1 opensearch-sql                  <IP_ADDRESS>
Screenshots
If applicable, add screenshots to help explain your problem.
Host/Environment (please complete the following information):

OS: latest docker image from here https://www.opensearch.org/downloads.htm
Host is Centos 7.9
Version 1.0.0
Docker-compose file edited for only one node
OPENSEARCH_JAVA_OPTS=-Xms8g -Xmx8g"
those were the only changes 'before docker-compose up'

Additional context
AWS m5.xlarge host machine

OK, I was doing the exact same thing again.  About thirty 500 document bulk updates had completed when opensearch aborted.   This time there was nothing in the log but this:
double free or corruption (out)
Killing performance analyzer process 11
OpenSearch exited with code 134
Performance analyzer exited with code 143
./opensearch-docker-entrypoint.sh: line 109:    10 Aborted                 (core dumped) $OPENSEARCH_HOME/bin/opensearch "${opensearch_opts[@]}"

It did leave a 23GB core file. If anyone is interested, I can upload it to somewhere.

While this may not be actually related to k-nn, I'll move it into that repo. If we can build a repro that doesn't use k-nn feel free to move it back.


patch -  change recoreding_rules_group to recoreding_rule_groups_set

@oded-dd I don't think it requires new minor version. WDYT?


MENTAL ISSUES

Hi. Im having mental issues waiting for update. Please help

start by trying to not shit up the issues section

hi. i think we have the same problem. hopefully they had a fix.


Can't send my bitcoins.

can someone help me plz

You need to send a little bit less as you aren't taking into account the miner's fee.

but, i'm trying to send half of the btcs
:/

You are trying to send a small amount of bitcoin: 0.000 05 BTC - about 2 cents.
Fees can typically be more than you have in your wallet currently
Have a look at: http://www.cointape.com for what typical fees are currently on the network.
Looking at a wallet I have open in MultiBit HD, the last two sends had fees 0f 0.0001 BTC, which is the TOTAL in your wallet.
You just need more bitcoin

Thank you bro :+1:

You are welcome.
Closing


Added team namespace name validation policy

Related Issue(s)

Description
Adds a policy that validates the name of namespaces/projects created by all authenticated users and only allows namespaces names that start with one of the user's groups followed by a hyphen and something else.
This policy is also great to see how you can work with different JMESpath filters.
I added tests for the policy but I'm not completely sure those make sense since this policy is kind of dynamic in terms of validation (different for each user depending on their groups).
Checklist

[x] I have read the policy contribution guidelines.
[x] I have added test manifests and resources covering both positive and negative tests that prove this policy works as intended.

@chipzoller looks like this policy is causing failures in kyverno CLI tests.

I have this fixed in a PR.


Faster transport for in-process communications

In certain application, sending requests from RaftApp to RaftCore happens. For example, if it's a distributed storage application, it may want to send new writes to rebuild a missing part of the data.
In the current lol 0.7.0, it is possible to do this by sending requests through gRPC as usual client but it is not efficient as it goes through network stack.
But ideally, transport can be a simple channel.
My idea here is to expose such server from RaftCore as well as tonic::Server.
My first choice is tarpc which is a non-official Google product but well-maintained. It seems to be transport-agnostic

Pluggable transport: any type impling Stream<Item = Request> + Sink can be used as a transport to connect the client and server.

and it supports both channel and tcp.

Tarpc doesn't support streaming (https://github.com/google/tarpc/issues/189) which means we can't entirely replace tonic with tarpc because we need streaming in log replication.

Tarpc doesn't support streaming (https://github.com/google/tarpc/issues/189) which means we can't entirely replace tonic with tarpc because we need streaming in log replication.


Add more configurations to the quick start example

Add additional configurations to the upsert quickstart example. These configs are useful for upsert table backfill and remote segment push.

Codecov Report

Merging #8051 (4f76c6b) into master (16a0881) will decrease coverage by 1.20%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #8051      +/-   ##
============================================
- Coverage     71.41%   70.21%   -1.21%
  Complexity     4261     4261
============================================
  Files          1599     1599
  Lines         83075    83075
  Branches      12402    12402
============================================
- Hits          59330    58332     -998
- Misses        19722    20737    +1015
+ Partials       4023     4006      -17

Flag
Coverage Δ

integration1
?

integration2
27.72% <ø> (-0.13%)
:arrow_down:

unittests1
68.15% <ø> (+<0.01%)
:arrow_up:

unittests2
14.33% <ø> (+0.04%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

...pinot/minion/exception/TaskCancelledException.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...nverttorawindex/ConvertToRawIndexTaskExecutor.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...e/pinot/common/minion/MergeRollupTaskMetadata.java
0.00% <0.00%> (-94.74%)
:arrow_down:

...plugin/segmentuploader/SegmentUploaderDefault.java
0.00% <0.00%> (-87.10%)
:arrow_down:

.../transform/function/MapValueTransformFunction.java
0.00% <0.00%> (-85.30%)
:arrow_down:

...ot/common/messages/RoutingTableRebuildMessage.java
0.00% <0.00%> (-81.82%)
:arrow_down:

...data/manager/realtime/DefaultSegmentCommitter.java
0.00% <0.00%> (-80.00%)
:arrow_down:

...verttorawindex/ConvertToRawIndexTaskGenerator.java
5.45% <0.00%> (-80.00%)
:arrow_down:

...ache/pinot/common/lineage/SegmentLineageUtils.java
22.22% <0.00%> (-77.78%)
:arrow_down:

...ore/startree/executor/StarTreeGroupByExecutor.java
0.00% <0.00%> (-77.78%)
:arrow_down:

... and 105 more

Continue to review full report at Codecov.

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


Missing destination configuration to be warning

Background
Currently, when deployed to Kyma without destination service binding an exception is thrown - https://github.com/eclipse/dirigible/blob/master/releng/sap-kyma-base/src/main/java/org/eclipse/dirigible/kyma/KymaModule.java#L102
Destination service is optional dependency and as such not finding the configuration should be a warning and not an exception.
Target
Change the logger message to warning and continue do not set destination configuration

Issue cloned to https://github.com/eclipse/dirigible/issues/1553

Fixed with https://github.com/eclipse/dirigible/commit/4d651ee9f293f213ce79d0865b851c77c1cd33b6, will be available with the next release of Eclipse Dirigible.


[BUILD] Try to fix ONNX CPU tests being disabled

@mbrookhart Added a line that should run the tests on CPU-- hopefully this fixes the problem

@electriclilies It looks like the CPU image doesn't have the necessary depencencies for ONNX. Could you make a PR to run install onnx in the CPU docker file?

what's the status on this one?

@areusch I think that @mbrookhart fixed this through other means and we just didn't close this PR. Thanks for following up


how to set sketch with coordinates

Hello all,
I'm getting the coordinates of the sketch with onStrokeChanged. How can I to set the same sketch using these coordinates?
Regards

which version of react native you are working?
I am getting error with RCTUIManager in latest react native..


Unwanted whitespaces

The build-in prettification of the resulting html output does produce the unwanted whitespaces between tags, see example below:
	return (

		*itemline1
		*itemline4

	);

Output:

The expected output:

Please add an option to turn off the prettification of the output if necessary!

fyi, it is already possible to disable the formatting, via the compact option


AccessData Docker Image (Update)

Status

[ ] In Progress
[x] Ready
[ ] In Hold - (Reason for hold)

Related Content Pull Request
Accessdata API Docker Image (New) #6686
Related Issues
New docker image does not allow import for accessdata module.
Description
Previous docker image is not correctly installing the package using combination of pip and git. Updated to simply use a release archive.

Hi Thomas,
Following our session, please

update the pipfile.lock to include the hashes of the zipfile
add dependabot (see repo README for instructions)

for reference: VT results for FTK-API-SDK-0.7.3.zip (clean)

@dorschw Just tested the image and is working correctly now. Please see the changes are set to use Git tags and now copies in the requirements.txt during image build.

Docker Image Ready - Production
Docker automatic build at CircleCI has deployed your docker image: demisto/accessdata:<IP_ADDRESS>771
It is available now on docker hub at: https://hub.docker.com/r/demisto/accessdata/tags
Get started by pulling the image:
docker pull demisto/accessdata:<IP_ADDRESS>771

Docker Metadata

Image Size: 131.20 MB

Image ID: sha256:ff3acccdec345acb9eb9b7702969b762716bd2e9cabe1edde6763d06e92d543c

Created: 2022-01-06T11:23:17.591822171Z

Arch: linux/amd64

Command: ["python3"]

Environment:

PATH=/usr/local/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin
LANG=C.UTF-8
GPG_KEY=E3FF2839C048B25C084DEBE9B26995E310250568
PYTHON_VERSION=3.9.8
PYTHON_PIP_VERSION=21.2.4
PYTHON_SETUPTOOLS_VERSION=57.5.0
PYTHON_GET_PIP_URL=https://github.com/pypa/get-pip/raw/3cb8888cc2869620f57d5d2da64da38f516078c7/public/get-pip.py
PYTHON_GET_PIP_SHA256=c518250e91a70d7b20cceb15272209a4ded2a0c263ae5776f129e0d9b5674309
DOCKER_IMAGE=demisto/accessdata:<IP_ADDRESS>771

Labels:

org.opencontainers.image.authors:Demisto<EMAIL_ADDRESS>org.opencontainers.image.revision:b656440b2187ab64ac998bad1af6ffa909fe08e0
org.opencontainers.image.version:<IP_ADDRESS>771


LibWeb: Fix a few crashes found by Domato

This PR fixes a few crashes found via DOM fuzzing with Domato.
See individual commits for details.

Oh I got ninja'd! 😆


make subquery prefix supports nil

Perfect, can you please add a test too? Thank you.

I am doing a new release, so I will go ahead, merge it and write a test :)

:heart: :green_heart: :blue_heart: :yellow_heart: :purple_heart:

Thanks!


Add startup flag to override XR mode settings

This is a bit of a prelude to enabling XR in the editor but as a few people had asked for a switch to turn XR on/off from the command line, just added this in.

Thanks!


Wrong glyphs rendered in Label with custom theme derived from editor theme

Godot version
v4.0.dev.custom_build [5489f4384]
confirmed in v4.0.dev.custom_build [ac7505e27]
System information
manjaro 21, AMD Radeon r7 200 series, Vulkan, 8GB Memory
Issue description
After opening a project with a addon, the text was all messed:

Steps to reproduce

Open Project
Open Addon Scene

Minimal reproduction project
this is an addon, so in order to install:

Create a new project
Create an addon folder
inside use the command bellow
git clone git@github.com:nonunknown/gdextension-integration.git

How do you build Godot? Do you disable the text_server_adv module and use the fallback one instead? In which case this could be what #53221 fixed and you're just on an older (9 days old) commit.

can confirm in the most recent commit:

I can reproduce the issue, here's a MRP with the above extension properly set up:
Test Project.zip
No idea what causes it but it seems to be specific to what this extension does. The scene has a tool script which might mess with stuff, but I don't see any obvious issue. CC @bruvzg @Paulb23

Editor fonts are loaded from memory instead of files and FontData copy saved to the custom theme seems to be missing source ttf file data (only have cached glyphs which were rendered at the time of theme import, and without it Font can't convert char codes to glyph indices).

Yeah makes sense, I've imported the default editor theme and modified it!


[diffeoDemon] solve physical space failure

Following this old issue: https://github.com/Inria-Asclepios/medInria-public/issues/60
This toolbox wasn't working because:

the cast filter wasn't done in case of datasets not in double or float,
we needed to resample datasets to have the same spacing, etc, before the registration method.

:m:

PR updated:

the cast is still done at the beginning of the process,
we do not resample the dataset,
a label is added in the toolbox to notify the user to drop data with same size/origin/spacing.

I removed the cast (as seen in https://github.com/Inria-Asclepios/medInria-public/pull/254 )

Maybe you could catch the exception send by ITK regarding these "not the same spacing, origin, .." pb and display a clear message to the user ?

Ok, there is too much PR about registration, and everthing is linked. I'm going to close this PR @LoicCadour and add the message on https://github.com/Inria-Asclepios/medInria-public/pull/254

This PR has been updated with the chances i made on https://github.com/Inria-Asclepios/medInria-public/pull/254


Upgrade InstallSSHKeyV0 to NodeJS 16

Task name: InstallSSHKeyV0
Description: Task upgraded to NodeJS 16
Documentation changes required: N
Added unit tests: N
Attached related issue: https://github.com/microsoft/build-task-team/issues/3531
Checklist:

[x] Task version was bumped - please check instruction how to do it
[x] Checked that applied changes work as expected

/azp run


Fix missing Hebrew translation for CZ entry

The Hebrew translation for Czech Republic was missing.

Please make this change in the translation corrections file instead of anything under cache.


Error or warning hints inside text

Hi,
the CustomTokenAttribute with wave lines are nice to show the user code errors. But unfortunately there is no way to show him which errors occurred.
Maybe it would possible to add some kind of tooltip or (even nicer) like its done in Xcode...

Well, usually errors are shown in separate error list because error messages can be quite long and contain number of rows. Xcode does also show the error in list (left panel) but it also show the error on line. I don't like the idea of duplicated error messages.
But I think this is already possible to implement. There are Marks which can be used to show the error image on left panel and paint the background of the line. OnAfterLinePaint can be used to paint the rest.

As suggested i tried the Marks list to get the line paint in light red. To get the rightaligned dark red box i used the OnAfterLinePaint
if (ALineNumber == 2)
{
	String errorMsg = "No visible interface for 'blabla' declares the selector 'blabla'";

	TRect errorRect;
	errorRect.Top = ARect.Top;
	errorRect.Right = ARect.Right;
	errorRect.Bottom = ARect.Bottom;
	errorRect.Left = errorRect.Right - ACanvas->TextWidth (errorMsg);

	ACanvas->Brush->Color = TColor (0x9FAEF6);
	ACanvas->FillRect (errorRect);
	ACanvas->Font->Name = "Arial";
	ACanvas->Font->Color = clBlack;
	ACanvas->TextOut (errorRect.Left, errorRect.Top, errorMsg);
}

At the very first time the text is written properly in Black Arial.

But the following to lines are also black (unwanted)
After clicking in the editor, the error message gets the current highlighter color. Is that a bug or am i doing anything wrong ... ?! :(

You are setting canvas properties without setting them back to originals. Hmm... I will investigate, if that can be avoided easily.

Of course! I now save them before and set them back at the end. Works! Thank you very much.

I moved editor FillRect to public. It is a bit faster way to fill rect.


Autogenerated changelog for plugins

Motivation
Feature
Alternatives
Additional context

Issues go stale after 90d of inactivity.
Mark the issue as fresh with /remove-lifecycle stale.
Stale issues rot after an additional 30d of inactivity and eventually close.
If this issue is safe to close now please do so with /close.
Provide feedback via https://github.com/falcosecurity/community.
/lifecycle stale

@jasondellaluce can we close this?

Yeah, forgot to link it. Closing this due to https://github.com/falcosecurity/plugins/pull/138.
/close

@jasondellaluce: Closing this issue.

In response to this:

Yeah, forgot to link it. Closing this due to https://github.com/falcosecurity/plugins/pull/138.
/close

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.


Update documentation to include Sphinx target (-DBUILD_SPHINX_DOC=ON)

Update Markdown documentation to include flag to generate Sphinx website.
NOTE: Sphinx documentation requires some Python packages, which can be installed with:
pip install --user doc/sphinx/requirements.txt
documentation will appear under
${CMAKE_BINARY_DIR}/sphinx/major.minor.rev/html/index.html
thank you

@chipb can you eyeball whether this will effect the build?


createClass & Proptypes warnings with React 15.5

In react 15.5 the browser console and test results get polluted with the following warnings. The call stack in the logs says it is the react-virtualised import/require call that is triggering the warnings.
Warning: Accessing PropTypes via the main React package is deprecated, and will be removed in  React v16.0. Use the latest available v15.* prop-types package from npm instead. For info on usage, compatibility, migration and more, see https://fb.me/prop-types-docs
Warning: Accessing createClass via the main React package is deprecated, and will be removed in React v16.0. Use a plain JavaScript class instead. If you're not yet ready to migrate, create-react-class v15.* is available on npm as a temporary, drop-in replacement. For more info see https://fb.me/react-create-class

<EMAIL_ADDRESS><EMAIL_ADDRESS>
Minor issue I know but hopefully an easy fix?

Yep. The fix is simple. Import stuff from this packages
https://www.npmjs.com/package/create-react-class
https://www.npmjs.com/package/prop-types

Do you have propTypes and createClass in rows/cells? Or do you use typescript?

My theory is it is the import * as React from 'react'; in a couple of react-virtualized files that is causing the console errors.
This code pen exhibits the behaviour we are seeing: https://codepen.io/anon/pen/zRVNaQ

To be clear we do not import propTypes or createClass from React in any of our project's files.

I can try to fix this issue by submitting a PR.

@hartror You probably need to update react.

Would you accept a fixing PR for this?

@hartror have you seen https://github.com/facebook/flow/issues/4673 - im not sure that a PR would do the trick here (but if you have something in mind, id be keen on seeing it).  We're making use of this gist to work around it for the time being. https://gist.github.com/turadg/9bcf08a7279e82a030a645250639fe6e
And as @TrySound said, you could also go for react 16

I thought we could just replace the import * as React from 'react'; with more specific imports?

Yeah, should be doable. See the note just above https://flow.org/en/docs/react/types/#toc-react-node for some more info.

Since react 15 causes only warnings and do not break your application I don't think we should change virtualized code and a lot of projects who use the same way to import react.

I'm disappointed but accept that it is your decision to make. Thanks for your time and a great package.

I agree with @hartror that we need to remove import * from React and use explicit imports so that other projects using react-virtualized can confidently upgrade to React 16.

@theTechie They can. There's just a noise in console in dev mode. It's not the end of the world. Or do you suggest to upgrade dozens of projects who uses the same way and still supports react 15?

I understand it's just a noise. Its just that I will not be very confident
upgrading my app to React 16 having 'react-virtualized' still making a get
to creatClass or propTypes. Even though we are not using any of these in
the codebase, there is no guarantee it will not be in the future. Isn't it ?
Let me ask this question in a different way; Is 'react-virtualized'
compatible with React 16 ? The answer might be Yes today. But, there is no
guarantee since we can still write code referring to createClass /
propTypes which just breaks on React 16.
Correct me if I am missing something here.
On Thu, 24 May 2018 at 5:07 PM, Bogdan Chadkin<EMAIL_ADDRESS>wrote:

@theTechie https://github.com/theTechie They can. There's just a noise
in console in dev mode. It's not the end of the world. Or do you suggest
to upgrade dozens of projects who uses the same way and still supports
react 15?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/bvaughn/react-virtualized/issues/1037#issuecomment-391682781,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA_31gyM6ONY12ApRNjalPx7yJxDo13uks5t1ptsgaJpZM4SbuGc
.

--
Regards,
Gagan

React virtualized is compatible with react 16. You should never use propTypes and createClass from react namespace. If one of your dependencies uses them then send issue or replace with maintaned dependency.


URI encode more problematic characters

Closes https://github.com/CartoDB/cartodb-management/issues/4756
Encodes more characters. URI::Encode has an unsafe parameter which takes a regex on which characters to encode. By default, it takes al URL safe characters, this changes it to all special characters including things like comma or " which have special meaning in headers (but not in URLs)
Acceptance
Rename a map with some special characters and try to export it as .carto. Test at least the following characters: quotes (" and '), space, comma/semicolon (, ;), star (*), spanish characters, chinese characters, emoji.
It's important to test with different browsers. At least Chrome (as it was the one giving issues), but it would be great to cover the major browsers as well.

The test I made in Chrome for OS X/Windows and Opera for Windows made struggles in the filename.

Created a new map and renamed it to Untitled "map'
Exported the map as a .carto
The file was downloaded succesfully but the name is a bit weird: Untitled _map' (on 2017-01-19 at 10.08.45).carto;filename-=utf-8''Untitled _map' (on 2017-01-19 at 10.08.45).carto

The rest of the browsers (tested in Chrome, Firefox, Safari, IE (only Windows, the rest in OS X) and Opera) where ok.

Good catch :). Some quotes were missing. I added them back and tested with Firefox + Chrome on Linux and it works fine now. Thanks 💋

Yay!

Now acceptance is fully passing 🇩🇰


GEODE-5348: Ignore duplicate detroy events on tombstone entries.

Adds unit test for retried destroy events on tombstones.
Thank you for submitting a contribution to Apache Geode.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[x] Is there a JIRA ticket associated with this PR? Is it referenced in the commit message?

[x] Has your PR been rebased against the latest commit within the target branch (typically develop)?

[x] Is your initial contribution a single, squashed commit?

[x] Does gradlew build run cleanly?

[x] Have you written or updated unit tests to verify your changes?

[x] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?

Note:
Please ensure that once the PR is submitted, you check travis-ci for build issues and
submit an update to your PR as soon as possible. If you need help, please send an
email to<EMAIL_ADDRESS>

This solution may ignore events that should not have been ignored when the first node goes down before returning an exception that the entry does not exist. The client then retries on another node and we ignore that the entry does not exist and pretend we destroyed it.


Fix for #3361: Redirected to the same page instead of Home Page

Fixes #3361

Closing as per comments in the related issues.


Match connectivity and offset types

(built on top of #4 so it has a few extra commits at the moment)
Apparently cumsum defaults to the platform int size instead of the int size of the array passed in. I don't know if this is a new thing, but Paraview did not appreciate it.

The diff is a bit messed up. Maybe recreate?

The diff is a bit messed up. Maybe recreate?

Yeah.. I forgot Github doesn't update branches properly :\


The Second vLLM SF Bay Area Meetup (Jan 31st 5-7:30pm)

Welcome to the second vLLM Meetup @ The AI Alliance!
We are absolutely thrilled to invite you to the second vLLM meetup. This event is for the growing community of vLLM users and developers to connect, share, and learn together. The vLLM team will share the recent updates on the project. We will also have vLLM collaborators from IBM coming up to the stage to discuss their insights on LLM optimizations.
Please register here and join us!
(Tentative) Agenda
5:00pm - 6:00pm: Doors open and check-in. Mingles and discussions. Food and refreshments will be available.
6:00pm - 6:40pm: Talks

Introduction: AI Alliance - Alexy Khrabrov, IBM Research
vLLM Project Update - Zhuohan Li, Woosuk Kwon, UC Berkeley
torch.compile and vLLM - Nick Hill, IBM Research

6:40pm - 7:30pm: Q&A and Social Hour - This is your chance to mingle, share your experiences, ask questions, and get to know the vLLM team on a personal level.
Note from the vLLM Team:
We, the vLLM team, are incredibly excited to meet each and every one of you. This meetup is not just about sharing updates but also about celebrating the community that makes vLLM what it is today. Let's come together, share our stories, and envision the future of vLLM.
Special Thanks:
A heartfelt thank you to IBM for generously providing us with the venue for this event. Their continuous support has been invaluable to the vLLM's open-source community.

Hi, are there any public slides from this event? I'd be great to take a look if possible.
Thanks!

Hi, are there any public slides from this event? I'd be great to take a look if possible.
Thanks!

Yes, please check it here!

Many thanks @LiuXiaoxuanPKU!


Swagger definition

OK, I don't think we should actually merge this into main yet (it's not working code!). I just want to try out the PR function in ZenHub and see how it works. It's been a while since I used it.
In theory, "closes #13" should link the PR to the issue in GitHub and maybe show up on the ZenHub board. In theory, when the PR is merged into the main branch, the issue is closed. See https://docs.github.com/en/issues/tracking-your-work-with-issues/linking-a-pull-request-to-an-issue
Of course, communism works "in theory" as well. ;-)

@ConradBoydElliottGustafson LOL at communism!  Way to go for community engagement.  ;)
I like retaining pretty much everything for reference, even if we remove it later.  Maybe pick a better directory than ./src and we merge?

Is it possible to automate the generation of the go code.  If so could it be the first use of @DerekRoberts pipeline.  Start with openapi spec, generate a docker image in the GHCR with the go code in it.  Deploy it to openshift?

@ConradBoydElliottGustafson Please don't maintain long running branches!  That sort of thing can be retained in a wiki or <shudder/barf> Constipance.
@franTarkenton If that's useful, sure!  (BTW there is already fair bit in the pipeline.)

@DerekRoberts +1 to not having long-running branches. I will kill the branch when it is no longer useful.
@franTarkenton +1 to having the pipeline do the generation -- although checking in the generated code might be easier for devs to clone and run something.

@ConradBoydElliottGustafson Commit whatever, we'll do a cleanup when there's an actual app.  Same request though - not in src!

Killing this PR and deleting the branch. Will do better next time!


Style Circle disappears on water layers on zoom

I am drawing 3 circles in openlayers map using the following style code. On zoom the circles on sea region seem to  get disappeared. Attached is the screen shot. Please let me know if any further details is needed.
Note: 1 in 5 times, the circle looks proper on zoom out but the other 4 times the circle disappear in sea region.

var width=[3,2,1];
var radius =[33,88,110];
for (var i = 0; i < radius.length; i++) {
var markerSet = [];
iconFeature = new ol.Feature({
geometry:	new ol.geom.Point(ol.proj.transform([Number(locations[0].longitude), Number(locations[0].latitude)], 'EPSG:4326', 'EPSG:3857')),
name: locations[0].city,
id: guid()
});
			iconStyle = new ol.style.Style({
				image: new ol.style.Circle({

					radius: radius[i],
					stroke: new ol.style.Stroke({
						color: [169, 19, 19, 1],
						width: width[i]
					}),
				})
			});

Looks like sea polygons have a higher z-index than your circle style. Try to configure a high zIndex for your iconStyle`.

Hello, Thanks for your reply. I tried zIndex but still no luck. :(
iconStyle = new ol.style.Style({
				zIndex: 1000,
				image: new ol.style.Circle({
					radius: radius[i],
					stroke: new ol.style.Stroke({
						color: [169, 19, 19, 1],
						width: width[i]
					}),
				})
			});

Instead of Style if i assign Zindex to ol.layer.Vector then the issue is fixed.


ImmutabilityKey.fallback

// TODO This fallback method seems to be wrong, as the result of one cell a depends
// on the values of other cells in cells while cell does not actually have any
// outgoing dependencies via when. That is why it has been past to this function
// in the first place.
// Maybe we need to check quiescentResolveDefault. Does this method pass all incomplete
// cells to Key.fallback?

fixed


Arduino/Genuino Zero Support

Are there any plans for Zero support?
I see there's MKRZERO support, but I'm unsure regarding how similar the pin assignments are on these boards. [EDIT]: I see Bobbin seems to feature crates for support of this board, but frankly I'd rather not stray there.

The answer is unfortunately that they are not especially similar, you can compare the two:

https://github.com/arduino/ArduinoCore-samd/blob/master/variants/mkrzero/variant.cpp
https://github.com/arduino/ArduinoCore-samd/blob/master/variants/arduino_zero/variant.cpp

However adding the mapping between board pins and chip pins shouldn't be too bad, it would look extremely similar to the mkrzero one but with the values from arduino_zero sheet above.

Nice, this doesn't look too difficult to implement. When I have a minute, I'll do it - cheers. I'll leave this open until then.

Hello, I was porting Nano 33 Iot and after one upload the USB communication stopped working. There is a space reserved so the bootloader won't be rewritten but somehow it stopped working. I will update you here just in case there is a same problem with Zero.


Should setProgress automatically log change?

Should job.setProgress() automatically add a log entry just like the new job.update() functionality?
Should all the job.set*****() methods automatically log the change?

@TomKaltz, you are good at finding nooks and crannies in the project.
Your question prompted me to review the methods. In hind sight, I should change the name of Job.setProgress to Job.updateProgress.
The other set methods are simply changing the properties of the Job, not touching the database. Job.setProgress is the only method that changes the database.
I have created a Project and am adding items that should be reviewed for v2.0.
https://github.com/grantcarthew/node-rethinkdb-job-queue/projects/1
Feel free to add to it. I'll close this for now due to the project.

@TomKaltz, Just found out you can't add to projects.
Raise new issues if you have any more ideas.
Thanks.

Hi @TomKaltz, I just published v1.0.2 which includes automatic logs for Job.setProgress however I will not rename until v2.


Enable FOR JSON feature in Azure SQL Data Warehouse

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

Hi,
FOR clause is not supported yet in Azure SQL Data Warehouse:
https://docs.microsoft.com/en-us/sql/t-sql/queries/select-for-clause-transact-sql?view=sql-server-2017
Please add support for this feature so that we can generate JSON and XML code through FOR JSON and FOR XML:
https://docs.microsoft.com/en-us/sql/relational-databases/json/format-query-results-as-json-with-for-json-sql-server?view=sql-server-2017
This is a must to integrate Azure SQL Data Warehouse with several NoSQL databases and other Big Data technologies,
Thank you.

@furia10 this is a product feedback instead of content issue. please go to https://feedback.azure.com/forums/34192--general-feedback


From docs "TF.NET simulate a 'with' syntax to manage the Graph lifecycle "

I am working through the documentation at https://tensorflownet.readthedocs.io/ and the code sample it gives
with<Graph>(tf.Graph().as_default(), graph => { 	var variable = tf.Variable(31, name: "tree"); 	tf.global_variables_initializer(); 	variable.assign(12); });
Below it it says that, "TF.NET simulate a with syntax to manage the Graph lifecycle which will be disposed when the graph instance is no long need." I can't find how tf.net simulates the 'with' keyword, but I was able to find .tf_with'. Replacing that inches me closer, I believe, but now I am getting hit with another error that states:
CS0311	The type 'Tensorflow.Graph' cannot be used as type parameter 'T' in the generic type or method 'Binding.tf_with(T, Action)'. There is no implicit reference conversion from 'Tensorflow.Graph' to 'Tensorflow.ITensorFlowObject'.
I am unsure how to move forward from here.
Thanks,
Sam

Please get started with this example project.


[Discussion] Rename repository

Can we rename this repository to something other than awesome coronavirus.
Let's be honest here, the repo is an awesome resource link, true, but this virus thing is not.
The term awesome does not necessarily needs to be in an awesome-list.
Here are a few examples of awesome lists without the term awesome to support the motion.

https://github.com/bolshchikov/js-must-watch
https://github.com/maxogden/maintenance-modules
https://github.com/lvwzhen/tools
https://github.com/edobashira/speech-language-processing

PS: @maintainers I could just be saltly quarantined for so long. Thus added that discussion tag.

@karx Please read this comment.


Add JSON Storage Capability for Orders, Quantity and Products

Could you fix the CI errors before merging? Thanks :)


Can't run debugger using C# in VS Code

Issue Description
I'm just starting to code in C#, using VS Code with all the apparently needed extensions, but when I hit F5, the following message pops up on the lower right of the editor window:
"Cannot set properties of undefined (setting 'PYTHONPATH')"
EVEN if the "dotnet run" command in the terminal works fine. Just the debugger that doesn't run at all.
Steps to Reproduce
The code is of a simple hello world console project.
Expected Behavior
Actual Behavior
Logs
OmniSharp log
There is not any log printed in any of the terminal windows
C# log
No output from trying to run the debugger, just the aforementioned message writen above
Environment information
VSCode version: 1.67.1
C# Extension: 1.24.4
Dotnet Information
SDK do .NET (refletindo qualquer global.json):
 Version:   6.0.300
 Commit:    8473146e7d
Ambiente de runtime:
OS Name:     Windows
OS Version:  10.0.19043
OS Platform: Windows
RID:         win10-x64
Base Path:   C:\Program Files\dotnet\sdk\6.0.300\
Host (useful for support):
Version: 6.0.5
Commit:  70ae3df4a6
.NET SDKs installed:
6.0.300 [C:\Program Files\dotnet\sdk]
.NET runtimes installed:
Microsoft.AspNetCore.App 6.0.5 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.NETCore.App 6.0.5 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.WindowsDesktop.App 6.0.5 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
To install additional .NET runtimes or SDKs:
https://aka.ms/dotnet-download

Visual Studio Code Extensions

Extension
Author
Version

autodocstring
njpwerner
0.6.1

code-runner
formulahendry
0.11.7

csharp
ms-dotnettools
1.24.4

csharpextensions
kreativ-software
1.7.0

dotnet
formulahendry
0.0.4

jinja
wholroyd
0.0.8

jupyter
ms-toolsai
2022.4.1011282140

jupyter-keymap
ms-toolsai
1.0.0

jupyter-renderers
ms-toolsai
1.0.6

material-theme
zhuangtongfa
3.13.26

python
ms-python
2022.6.2

python-environment-manager
donjayamanne
1.0.4

python-extension-pack
donjayamanne
1.7.0

python-path
mgesbert
0.0.11

pythonpath-setter
datasentics
0.0.1

remote-wsl
ms-vscode-remote
0.66.2

vsc-python-indent
KevinRose
1.15.0

vscode-database-client2
cweijan
5.2.7

vscode-django
batisteo
1.10.0

vscode-docker
ms-azuretools
1.22.0

vscode-env-selector
andricdev
0.3.0

vscode-ipython
ChenJun
0.6.2

vscode-ipython
pancho111203
0.5.0

vscode-nuget-package-manager
jmrog
1.1.6

vscode-pylance
ms-python
2022.5.1

vscode-yaml
redhat
1.7.0

vscodeintellicode
VisualStudioExptTeam
1.2.20

Note: I've been looking all over the net over several foruns and found absolutely no one discussing anything like this. Maybe I'm doing something wrong. Anyway, I've reinstalled VS Code and some of the extensions, but to no avail. Thank you for your time.

@GuilhermeFrSaito I have no idea why you would see an error like "Cannot set properties of undefined (setting 'PYTHONPATH')". What does your launch.json look like?

In case it is helpful, here are the instructions: https://github.com/OmniSharp/omnisharp-vscode/blob/master/debugger.md. Make sure you have done step 3 under 'Once for each project'.

Hello Gregg, sorry I didn't answer earlier.
My launch.json file is as follows:
{
"version": "0.2.0",
"configurations": [
{
// Use IntelliSense to find out which attributes exist for C#
debugging
// Use hover for the description of the existing attributes
// For further information visit
https://github.com/OmniSharp/omnisharp-vscode/blob/master/debugger-launchjson.md
"name": ".NET Core Launch (console)",
"type": "coreclr",
"request": "launch",
"preLaunchTask": "build",
// If you have changed target frameworks, make sure to update
the program path.
"program": "${workspaceFolder}/bin/Debug/net6.0/WS_Project1.dll"
,
"args": [],
"cwd": "${workspaceFolder}",
        // For more information about the 'console' field, see

https://aka.ms/VSCode-CS-LaunchJson-Console
"console": "internalConsole",
"stopAtEntry": false
},
{
"name": ".NET Core Attach",
"type": "coreclr",
"request": "attach"
}
]
}
It's the default file that is created when you set and choose the debugger
up/left.
Anyway, thank you for your reply and I really appreciate the instructions
you sent. I'll be sure to apply at my first chance and let you know what
happens.
Thanks again and have a great weekend!
On Fri, May 13, 2022 at 2:22 PM Gregg Miskelly @.***>
wrote:

In case it is helpful, here are the instructions:
https://github.com/OmniSharp/omnisharp-vscode/blob/master/debugger.md.
Make sure you have done step 3 under 'Once for each project'.
—
Reply to this email directly, view it on GitHub
https://github.com/OmniSharp/omnisharp-vscode/issues/5206#issuecomment-1126277111,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AZEVEVIQ6J6HVI4BWLE23J3VJ2FTTANCNFSM5V2AFSYQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

Thanks. That all looks totally normal. I suspect you are running into a bug from one of your other extensions. Looks like you have an extension called 'python-path', so that would be my guess, though I could totally be wrong.


Replace keystore with script to generate accounts

By the way, which password you used to generate accounts? I am new to Ethereum and trying to deploy smart contract from one of predefined addresses and I think I need password.

It seems empty string is used as password :)

well this is a problem since for example mist does not allow empty password to be used


Documentation: Improve discoverability of "htmlAttributes" entry in ViewDataDictionary

From @dougbu on October 6, 2014 17:5
@Html.Editor() and @Html.EditorFor() special-case an entry named "htmlAttributes" in the ViewDataDictionary (optionally augmented using additionalViewData parameters) when executing their default templates. MVC 5.2 issue #2142 is an indication this feature needs additional documentation. Users will otherwise search for ways to control generated HTML attributes when using these methods.
Couple of details:

"htmlAttributes" key is case-insensitive
entry is handled very similarly to the htmlAttributes object passed to input helpers such as @Html.HiddenFor() and @Html.TextBox()

Copied from original issue: aspnet/Mvc#1233

Where should this be documented? https://docs.microsoft.com/en-us/aspnet/core/mvc/views/overview seems like the closest place, currently.

@ardalis that page doesn't mention @Html.Editor(...) or @Html.EditorFor(...) and those are the only methods that support "htmlAttributes" keys in the ViewDataDictionary. I suggest https://docs.microsoft.com/en-us/aspnet/core/mvc/views/working-with-forms
Side notes (FYI):

Doesn't really matter what adds the key to ViewDataDictionary. It's most commonly done in the @Html.Editor[For]() call (using the additionalViewData parameter) because it's rare for the same HTML attributes to be correct for multiple calls in a view.
@Html.Display() and @Html.DisplayFor() also support additionalViewData parameters but don't check the merged ViewDataDictionary for the "htmlAttributes" key.

@ardalis can you do this under PO?


Plesk disables opcache_get_cache() in PHP

Hi Lars
Plesk changed its defaults to use
Plesk 17.8.11 - PHP value "disable_functions" changed to default value "opcache_get_status"
And Hosters tend do follow this approach.

/src/voku/cache/AdapterOpCache.php in Line 36 uses !empty(@\opcache_get_cache()) to check that op cache is enabled.
PHP 7.4 is @silenced, but PHP 8 errors in Userland. This disabled method could be replaced by
!empty(@\opcache_get_configuration())

I assume we better have something like
$opcache_get_configuration['directives']['opcache.enable'] === true

to ensure opcache is really enabled.
What do you think?

@voku  ping


Handle invalid class constant

Just make sure the exception is catched so it doesn't break the pipeline

Codecov Report

Merging #321 into master will decrease coverage by 0.35%.
The diff coverage is 67.03%.

@@             Coverage Diff              @@
##             master     #321      +/-   ##
============================================
- Coverage     78.32%   77.96%   -0.36%
- Complexity      566      572       +6
============================================
  Files            76       76
  Lines          1707     1743      +36
  Branches        264      267       +3
============================================
+ Hits           1337     1359      +22
- Misses          261      274      +13
- Partials        109      110       +1

Impacted Files
Coverage Δ
Complexity Δ

...ne/patterndetector/ccspan/CCSpanPatternDetector.kt
100% <ø> (ø)
3 <0> (ø)
:arrow_down:

...n/org/cafejojo/schaapi/miningpipeline/CsvWriter.kt
100% <ø> (ø)
8 <0> (ø)
:arrow_down:

...line/testgenerator/jimpleevosuite/TestGenerator.kt
100% <ø> (ø)
3 <0> (ø)
:arrow_down:

...hgenerator/jimple/filters/BranchStatementFilter.kt
82.35% <0%> (ø)
7 <0> (ø)
:arrow_down:

...fejojo/schaapi/GitHubMiningCommandLineInterface.kt
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

...g/cafejojo/schaapi/miningpipeline/PatternFilter.kt
100% <100%> (ø)
6 <5> (ø)
:arrow_down:

...ojo/schaapi/DirectoryMiningCommandLineInterface.kt
100% <100%> (ø)
2 <0> (ø)
:arrow_down:

...otlin/org/cafejojo/schaapi/CommandLineInterface.kt
62% <25%> (-3.22%)
7 <0> (ø)

...graphgenerator/jimple/ControlFlowGraphGenerator.kt
64.28% <25%> (+2.74%)
6 <0> (ø)
:arrow_down:

.../usagegraphgenerator/jimple/filters/ValueFilter.kt
76.92% <40%> (-2.11%)
5 <0> (ø)

... and 2 more

Continue to review full report at Codecov.

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


Re-installing the scripting addition adds new entries to Privacy -> Accessibility pane without cleaning up old ones

In Security and Privacy -> Privacy -> Accessibility, where it lists the approved apps for accessibility features, I've currently got 7 entries for yabai, only one of which is valid. It appears every time the scripting addition is added (which happens when yabai updates or when the os updates) I get a new entry. I was going to clear them out with tcutil but it looks like the bundle id is randomized and I can only clear them out by removing all accessibility approvals. That's pretty awkward.
Any possibility yabai can clean up on its own?

If they annoy you, just unlock the lock in the lower left, and then you can remove them using the minus sign.

The scripting-addition doesn't have anything to do with accessibility permission.
If you are using the pre-built binaries, you should not be getting duplicate entries. If you build from source you need to codesign the binary as described in the wiki.

Got it - sounds like a homebrew issue then. Thanks!
On Wed, Jul 28, 2021 at 10:57 AM Åsmund Vikane @.***>
wrote:

The scripting-addition doesn't have anything to do with accessibility
permission.
If you are using the pre-built binaries, you should not be getting
duplicate entries. If you build from source you need to codesign the binary
as described in the wiki.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/koekeishiya/yabai/issues/969#issuecomment-888507411,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAAAFHSWQHGKAKW76BKQ2LT2BAIVANCNFSM5A5TUYAQ
.

--
Kevin Clark


LFS-533: Allow upload of vocabulary files

Allows for vocabularies to be uploaded in RDF/XML format, thus allowing for the installation of vocabularies that are not available through BioPortal.
At the present moment, I am marking this PR as a draft as code cleanup should be done and there are several bugs such as:

The list of installed vocabularies doesn't refresh after a vocabulary is installed from a file and the BioPortal API key is not present.
If the vocabulary upload fails (for example, due to a parser error) no error is displayed to the user.

Nonetheless, your feedback is much appreciated :slightly_smiling_face:
To test:

Build the LFS-533_test branch
Download the Human Phenotype Ontology in RDF/XML format from https://bioportal.bioontology.org/ontologies/HP
Ensure that the BIOPORTAL_APIKEY environment variable is not set
Start LFS
Go to Administration ... Vocabularies and install the downloaded ontology file with the following parameters; Identifier = HP, Name = Human Phenotype Ontology, Version = 1
Once installation is finished, create a new Demographics form.
Type something like myopia in the co-morbidities field. Auto-complete should happen and answers should allow for selection.

tested by following the steps described above

I was able to install GO and HANCESTRO with the method described above. For some reason I could not get HANCESTRO work in the Race/Ethnicity reported field. It worked well in Co-morbidities field as you can see in the screenshots below.

I was not able to have more than one vocabulary linked to one field.

I was not able to have more than one vocabulary linked to one field.

That's expected, LFS-198 isn't merged yet.

For some reason I could not get HANCESTRO work in the Race/Ethnicity reported field.

This is out of scope, but requires further investigation. The only notable difference I see at first glance is that Race/Ethnicity has display mode list+input and has predefined options, while Co-morbidities  has display mode input.
@sashaandjic can you do a quick test and change the display mode for Race/ethnicity to see if the suggestions work then?

Changing display mode to input didn't work.
I the meantime I installed ORDO and it worked well in co-morbidities.

I cannot believe I missed this. Once I removed vocabularyFilter from Race/Ethnicity field it WORKED. Duh!

And out of scope for this PR, but the INSTALLED table should not depend so much on the bioportal service, it should be able to display everything it does so far, except the Upgrade button.

* The INSTALL button is always clickable, and clicking it will actually send a request, and a new row will be added to the INSTALLED table, even though nothing is installed

Fixed by commit b1b927772 :heavy_check_mark:

* After installing HP, it will show up in the INSTALLED table, but with an "Install" button instead of the "Uninstall" one

Fixed by commit 6a43fde3ef :heavy_check_mark:

* Why are the inputs doubled, once as a hidden input (with a proper `name` attribute) and once as a unnamed MUI input? This actually causes a JS console warning about inputs with a value but no onchange prop

Fixed by commit 2c643dca6a :heavy_check_mark:

* The indexing servlet returns a 200 status even if there's an error, but with an error payload. The response should be a 500, and the frontend should handle the error payload.

Fixed by commits 95e3fc82 and b1b927772 :heavy_check_mark:

* Why only `OWL` is supported? The same code can handle all other formats, the only thing that is preventing it now is line 130 where we hadcode the format. Is there a way to detect the format from the input filename?

Fixed by commits e3577abc, 4795ec76 where getSourceFormat() is moved from EbiRepositoryHandler to the utility class OntologyFormatDetection. This same function is used here to determine the format of the upload vocabulary file from its file extension. :heavy_check_mark:

I was able to install HP.obo.
I had to manually enter proper Identifier, Name and Version. Can this be pulled out of the obo file instead?
After uninstall, the screen did not refresh.

I had to manually enter proper Identifier, Name and Version. Can this be pulled out of the obo file instead?

There should be a data-version field in the obo. For HPO it seems to look like this: data-version: hp/releases/2020-08-11.
The closest thing I can find to an identifier is ontology: hp.obo (for Go the field value is ontology: go).
There doesn't seem to be explicit mention of the human readable name of the ontology inside of the obo file.
My suggestion is to try to retrieve the version from the obo, but continue prompt the user for the name. I'm on the fence regarding the id.

I am able to install the same vocabulary under different ID.

I am able to install the same vocabulary under different ID.

In that case, as an amendment to my previous comment, let's attempt extracting the ID from the obo's ontology field and see if that works for most vocabularies. We'll hardcode stripping  down the .obo suffix if it appears in the field's value. Without a centralized source like bioportal to keep track of the files it's probably the only easy way to ensure vocabularies are not duplicated.

Tested last two commits.

Tested. Looking good.

It may look better like this?

Would it look better like this?
(all in one row; more space between 3 main headers; narrower SELECT FILE button, wider Install button)

I agree with more space between the headers, but not with having the entire "Install from local file" section on one line. If we were to do any further UI improvements, I suggest:

increase the space between sections
left-align the local file selection form instead of centering it
maybe try variant="outlined" for the select button

All of the above are non-blocking and can be done as part of a separate task for polishing the vocabularies UI in general.

Found one small issue, that will be added to LFS-615. Once vocabulary is installed, the About button is not present on the list of the installed vocabularies as it is for the vocabularies installed via BioPortal. See in the attached screen shot where ORDO was installed via BioPortal unlike HP and HPO.

One small complaint: After successful install the button needs to go back to disabled state with proper text label "Install"


Add update command to update apex

Basically just see if there's a new release, grab the correct binary, and replace itself. apex update? apex upgrade?

Definitely agree on this TJ, how do you think this would be implemented best? What did you have in mind?