feat(db): add description to entities and plugin fields

XR: https://github.com/Kong/kong/pull/10150
This PR contains added descriptions for all OSS plugins and core entities.

Nice work @Guaris
there is a syntax issue here
ERROR: error loading module 'kong.db.schema.typedefs' from file './kong/db/schema/typedefs.lua':
	./kong/db/schema/typedefs.lua:681: '}' expected (to close '{' at line 678) near 'required'

and there seems to be a general error with
error loading module 'kong.db.schema.entities.consumers' from file './kong/db/schema/entities/consumers.lua':
	./kong/db/schema/entities/consumers.lua:12: '<name>' expected near '"The unique username of the Consumer. You must send either this field or custom_id with the request."'
	```

@jschmid1 I think I fixed them.

Aand another one:
ERROR: ./kong/db/schema/typedefs.lua:562: attempt to perform arithmetic on a string value

closed in favor of #10765


XAML Namespace Resolution Fails

Portable.Xaml.XamlParseException: Cannot resolve runtime namespace from XML namespace 'clr-namespace:Editopia.Editor'
I get above Error Message, though only when working with the Designer, and only if I try to declare a namespace like:
xmlns:local="clr-namespace:Editopia.Editor"
The Namespace definitely exists, and the project builds, and runs, just fine. It's only the Editor not working.
I am using the following Nuget Packages:
Avalonia (0.6.1)
Avalonia.Desktop (0.6.1)
I am using The following Visual Studio Version:
Microsoft Visual Studio Community 2017 Version 15.7.5
Here is the full Stack Trace the Designer Extension gives me:
Portable.Xaml.XamlParseException: Cannot resolve runtime namespace from XML namespace 'clr-namespace:Editopia.Editor'
   at Avalonia.Markup.Xaml.PortableXaml.AvaloniaXamlSchemaContext.GetXamlType(String xamlNamespace, String name, XamlType[] typeArguments)
   at Portable.Xaml.XamlXmlParser.ReadObjectElement(XamlType parentType, XamlMember currentMember)+MoveNext()
   at Portable.Xaml.XamlXmlParser.ReadMember(XamlType parentType, XamlMember xm)+MoveNext()
   at Portable.Xaml.XamlXmlParser.ReadMemberElement(XamlType parentType, XamlType xt)+MoveNext()
   at Portable.Xaml.XamlXmlParser.ReadMembers(XamlType parentType, XamlType xt)+MoveNext()
   at Portable.Xaml.XamlXmlParser.ReadObjectElement(XamlType parentType, XamlMember currentMember)+MoveNext()
   at Portable.Xaml.XamlXmlParser.Parse()+MoveNext()
   at Portable.Xaml.XamlXmlReader.Read()
   at Portable.Xaml.XamlServices.Transform(XamlReader xamlReader, XamlWriter xamlWriter, Boolean closeWriter)
   at Avalonia.Markup.Xaml.AvaloniaXamlLoaderPortableXaml.LoadFromReader(XamlReader reader, AvaloniaXamlContext context)
   at Avalonia.Markup.Xaml.AvaloniaXamlLoaderPortableXaml.Load(Stream stream, Object rootInstance, Uri uri)
   at Avalonia.DesignerSupport.DesignWindowLoader.LoadDesignerWindow(String xaml, String assemblyPath)
   at Avalonia.DesignerSupport.Remote.RemoteDesignerEntryPoint.<>c__DisplayClass17_0.<OnTransportMessage>b__0()

can you try adding the assembly name together with the clr-namespace? something like this clr-namespace:Editopia.Editor;assembly=assemblynamehere

Can do, I'll edit the results when I get around testing that.
Just want to note, that I tried building this on master (60ebf67b84c7b7d4016aea84135cef5e5e7bd698) and it does not happen there, while it does happen on the latest nuget package.

Ah yeah, there has been a lot of changes since the last release. If you can stick with nightlies/master then the better :)


Add statistics on projects to /criteria.  Fixes #314

Signed-off-by: David A. Wheeler<EMAIL_ADDRESS>

A day may come[0] when we want to do page caching.
[0] Sons of Gondor! Of Rohan! My brothers! I see in your eyes the same fear that would take the heart of me! A day may come when the courage of men fails, when we forsake our friends and break all bonds of fellowship. But it is not this day. An hour of wolves and shattered shields when the age of Men comes crashing down! But it is not this day! This day we fight!  Men of the West!

But it is not this day.  Page caching has a complication: Currently our pages vary depending on whether or not you're logged in, and as who.  You only see "edit" if you can edit, for example.  The easy solution would be to make the pages invariant; then we could cache nearly everything.  Even then, we could make this page an exception.
Okay to add for now?

Sure, looks good to add.


tensorflow c++ api session->run() consumes too much time.

Here are my codes:
void tensorflow_class_wrapper::load_model(string model_path)
{
	// set gpu options
	SessionOptions opts;
	GraphDef graph_def;
	ConfigProto* config = &opts.config;
	opts.config.mutable_gpu_options()->set_per_process_gpu_memory_fraction(0.3);
	opts.config.mutable_gpu_options()->set_allow_growth(true);
	assert(_session == NULL);
	TF_CHECK_OK(NewSession(opts, &_session));
	printf("Create session done.\n");

	//read the pb file into the grapdef member
	TF_CHECK_OK(ReadBinaryProto(Env::Default(), model_path, &graph_def));
	printf("Load model done.\n");

	//Add the graph to the session
	TF_CHECK_OK(_session->Create(graph_def));
	printf("Create graph done.\n");
	return;
}

void tensorflow_class_wrapper::predict()
{
	// convert data
	convert_data();
	printf("Convert data done.\n");

	clock_t t1, t2;
	t1 = clock();
	//The session will initialize the outputs
	vector<Tensor> outputs;
	TF_CHECK_OK( _session->Run(_input_dict,  _output_tensor_names , {}, &outputs));
	t2 = clock();
	cout << "predict consume time: " << float(t2 - t1) / CLOCKS_PER_SEC << endl;

	for (auto _o : outputs)
		cout << _o.DebugString() << endl;
	vector<vector<float>> predict_res(outputs.size());
	for (int nr = 0; nr < outputs.size(); nr++)
		predict_res[nr].assign(outputs[nr].flat<float>().data(), outputs[nr].flat<float>().data() + outputs[nr].dims());

}

My input img size is 512x512x1. PB model is just single convolution and the output shape is 512x512x1. It's confused that session->Run() consumes over 2 seconds even more. By the way, i compile tensorflow1.8 release codes to get tensorflow lib/dll. Is there anything wrong when i compiled the source codes? Or I did not set some options in the right way?

Thank you for your post. We noticed you have not filled out the following field in the issue template. Could you update them if they are relevant in your case, or leave them as N/A? Thanks.
Have I written custom code
OS Platform and Distribution
TensorFlow installed from
TensorFlow version
Bazel version
CUDA/cuDNN version
GPU model and memory
Exact command to reproduce

It seems that the model's warm up consumes too much time when run is called for the first time.
you can refer to https://github.com/tensorflow/serving/issues/385.

@weberxie Thanks for your code and comments. I also met a similar problem. Do you know why the model's warm-up consumes that much time? Is there any way to warm up the model in advance before _session->Run?

@feihugis You can make a dummy call for session->Run after the model was loaded.


Dump orebag into ore processor

About the PR
Orebag can now be used on ore processor to dump content into it. It's use modified code of DumpableSystem and have same cons:

LeftClick to ore processor to dump content into it.
Use verb from context menu as alternative

TryInsertMaterialEntity was little modified to be able turn off popup message to remove message spam from every single insert of ore.
P.S. You can add popup for DisposalUnit later if you willing to do so.
Screenshots

Changelog

:cl:

add: you can now dump collected ore into ore processor with orebag

Yes, this also allows to dump something into every entity that have MaterialStorageComponent but only if used object have DumpableComponent

You can try to dump trash to ore process and action bar will appear with followed popup message after completion, but whitelist don't allow to insert trash into ore processor. So is it really matter?

For some reason Ore Bag don't have TagComponent even when it's use tags in whitelist while Ore Processor have TagComponent. So, I code simple check without it.

For some reason Ore Bag don't have TagComponent even when it's use tags in whitelist while Ore Processor have TagComponent. So, I code simple check without it.

The bag doesnt have any tags. The tags for the ore processor are for the types of materials that can be inserted.

shoveling coal into the ore processor manually is a rite of passage

@Jackrost are you coming back to this.

@Jackrost are you coming back to this.

Yes, but I need some help with whitelist

Re-open when ready


[Docs]Responsiveness issues

I have been spending my time reading about meshery and have encountered a few issues  . The more I'll dive deep about project , the more issues shall be listed here😆
Current State

[ ]  The issue is on Android devices (probably for width<800px)
[x]  Responsive issue of announced vulnerabilities
[x]  Responsive issue in supported platforms
[ ] Incomplete navigation components
[x] The calender in the footer doesn't turns to blue on hover as expected

The current sidebar components headings have incomplete subheadings (for example in concepts) [I dont know its a bug or feature]

Shown Subheadings
Complete subheadings

#last_issue
Desired State
The GraphQl API reference and
REST API reference should be visible.

The headings should have complete subheadings

Contributor Guides and Handbook

📚 Instructions for contributing to documentation

Meshery documentation site and source

🛠 Meshery Build & Release Strategy
🎨 Wireframes and designs for Meshery UI in Figma
🙋🏾🙋🏼 Questions: Discussion Forum and Community Slack

I would like to work on these issues 📃

Thanks @Ghat0tkach.
There you go :rocket:

Please sound off with you progress so far @Ghat0tkach

Please sound off with you progress so far @Ghat0tkach

I have already checked the boxes which I have done , just need to make sure for the navigation part . I have already asked on slack  , if the issue is valid . I'll raise my PR.
Thank you  for the concern vishal


Count playlists with description

Add Yt::PlaylistAudit::Description class
Bump version to 0.2.1
use audit_item instead of video_audit in the usage

:tada: add another commit to bump version, update README, CHANGELOG :joy:

I squashed the commit. is it okay?

:yellow_heart:

Thank you


kscript doesn't appear to work with Clikt and Kotlin 1.4.10

I get this cryptic NoSuchMethodError with Kotlin 1.4.10...works fine if I back down to 1.3.72
Exception in thread "main" java.lang.reflect.InvocationTargetException
	at sun.reflect.NativeConstructorAccessorImpl.newInstance0(Native Method)
	at sun.reflect.NativeConstructorAccessorImpl.newInstance(NativeConstructorAccessorImpl.java:62)
	at sun.reflect.DelegatingConstructorAccessorImpl.newInstance(DelegatingConstructorAccessorImpl.java:45)
	at java.lang.reflect.Constructor.newInstance(Constructor.java:423)
	at Main_Scriptlet_a9f5a3bb1e4ba312$Companion.main(Main_Scriptlet_a9f5a3bb1e4ba312.kt:6)
	at Main_Scriptlet_a9f5a3bb1e4ba312.main(Main_Scriptlet_a9f5a3bb1e4ba312.kt)
	at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
	at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.lang.reflect.Method.invoke(Method.java:498)
	at org.jetbrains.kotlin.runner.AbstractRunner.run(runners.kt:64)
	at org.jetbrains.kotlin.runner.Main.run(Main.kt:149)
	at org.jetbrains.kotlin.runner.Main.main(Main.kt:159)
Caused by: java.lang.NoSuchMethodError: kotlin.jvm.internal.PropertyReference1Impl.<init>(Ljava/lang/Class;Ljava/lang/String;Ljava/lang/String;I)V
	at Scriptlet_a9f5a3bb1e4ba312$SonarqubeAnalyzer.<clinit>(scriptlet.a9f5a3bb1e4ba312.kts)

Could you provide a short example?

kscript14bug.zip
Run that with Docker after unzipping it into a directory:
docker build . -t kscriptbug
You can change the Kotlin version to 1.3.72 and it works fine.  I set it up as a Docker build so you know it's a fully clean environment to repro this in.

I had the same issue. I wonder if we can somehow tell kscript to use certain version of kotlin. I'm not sure how that can be configured either in kscript or in my environment as a whole.
What I found so far:

It happened inconsistently – it worked for me until I tried reinstalling kscript to help a coworker and I had it again.
If I remove the by option(...) stuff it will work. It seems that the code needs a PropertyReference1Impl constructor with only two arguments and that's not present in Kotlin 1.4 at least (it always receives 3 args).

Also tried using argparser:
https://github.com/xenomachina/kotlin-argparser
and same issue...looks like they both use the same technique...

public PropertyReference1Impl(Class owner, String name, String signature, int flags) is only available since Kotlin 1.4:
https://github.com/JetBrains/kotlin/blob/master/libraries/stdlib/jvm/runtime/kotlin/jvm/internal/PropertyReference1Impl.java#L22
Which would hint at the wrong stdlib being in the classpath?

Indeed, something is wrong in the classpath. IMHO its clearly clikt to blame here. See https://repo1.maven.org/maven2/com/github/ajalt/clikt/2.7.1/clikt-2.7.1.pom
Since it is relying on kotlin 1.3.72 (see dependencies section) it brings a wrong outdated stdlib into the classpath.
When removing 1.3.72 manually from the classpath in the example when running it via kscript, the example works fine for me.
I guess clikt would need to be published with kotlin-stdlib as compile-only and not runtime dependency.
FYI @ajalt
@kenyee It's quite a coincidence that kotlin-argparser seems to suffer from the same problem.

Since it is relying on kotlin 1.3.72 (see dependencies section) it brings a wrong outdated stdlib into the classpath.

Clikt having 1.3.72 as a compile dependency shouldn't prevent other consumers like kscript here to substitute 1.4.10 at runtime. From my understanding, this is the default Gradle behaviour: If no ranges are involved, then the highest version that is not rejected will be selected.
@kenyee could you workaround the issue by specifying stdlib explicitely?
@file:@DependsOn("org.jetbrains.kotlin:kotlin-stdlib:1.4.10")

If that works, it would be nice to tweak kscript dependency resolutions to allow "upgrading" the versions in this case

I'd rather think that clikt should add the stdlib as compileOnly. See https://docs.gradle.org/current/userguide/java_library_plugin.html
This may be just problematic if a library is intended to be used with java directly without kotlin being involved. If so, the library developer would be correct in using a compile/api declaration on the stdlib. Not sure what the recommended best practice is.
kscript will always run scripts with kotlin executable (and thus including the corresponding version of the stdlib). It must be (and is) unopinionated about dependencies and will resolve them as defined in the artifact descriptors (pom-files).

kscript will always run scripts with kotlin executable (and thus including the corresponding version of the stdlib).

Then I would expect kotlin-stdlin:1.4.10 in the classpath there?

It must be (and is) unopinionated about dependencies and will resolve them as defined in the artifact descriptors (pom-files).

But ultimately both the compiled script and the dependencies share the same classpath, right? If kscript compiles main.kts against 1.4.70 and some depencency declares 1.3.72 in the compile classpath, which should win?

I think when running kotlin it adds the corresponding version of stdlib by default. The docs are very clear in the case of kotlinc, see https://kotlinlang.org/docs/reference/compiler-reference.html#-no-stdlib. For kotlin I could not find a direct reference, except for
kotlin: run Kotlin programs, scripts or REPL.

Usage: kotlin <options> <command> <arguments>
where command may be one of:
  foo.Bar                    Runs the 'main' function from the class with the given qualified name
  app.jar                    Runs the given JAR file as 'java -jar' would do
                             (-classpath argument is ignored and no Kotlin runtime is added to the classpath)
and possible options include:
  -classpath (-cp) <path>    Paths where to find user class files
  -Dname=value               Set a system JVM property

I'd understand that such as a Kotlin runtime is added to the classpath when a class name is provided as  argument.
So I'd guess that having 2 conflicting stdlibs in the classpath is causing the problem.
I rather think that kscript is already compiling against the classpath with both stdlibs.
I do not know which jar has precedence, it's up to the used classloader to decide I think. And kotlin is using not the default JVM classloader afaik.
@ligee Can you help to shed some light (a) on library development stdlib ex/inclusion best practice, and (b) kotlin internal conflict management when having 2 conflicting stdlibs in the classpath for compilation & runtime?

What about adding the kotlin stdlib from the installed kotlin distribution to the aether resolution algorithm explicitely? If aether uses the same principles as Gradle, I think that should "upgrade" the stdlib dependency for everyone?

@kenyee could you workaround the issue by specifying stdlib explicitely?
@file:@DependsOn("org.jetbrains.kotlin:kotlin-stdlib:1.4.10")

FYI, tried this:
sdkman use kotlin 1.4.10
kscript --clear-cache
./sonarqube_status.kts --help
and same error, so this doesn't work:
Caused by: java.lang.NoSuchMethodError: kotlin.jvm.internal.PropertyReference1Impl.(Ljava/lang/Class;Ljava/lang/String;Ljava/lang/String;I)V
This is indeed a weird one...
Also, I thought 1.4.10 doesn't require the stdlib in the classpath any more?  It automagically adds it in app gradle files.

Also, I thought 1.4.10 doesn't require the stdlib in the classpath any more? It automagically adds it in app gradle files.

The stdlib is definitely still needed in the JVM runtime classpath. What changed is that the Kotlin plugin adds it to the Gradle dependencies automatically by default: https://github.com/JetBrains/kotlin/blob/07aee8831eb45690053028f39f3312e1c8e90f69/libraries/tools/kotlin-gradle-plugin/src/main/kotlin/org/jetbrains/kotlin/gradle/internal/KotlinDependenciesManagement.kt#L60

What about adding the kotlin stdlib from the installed kotlin distribution to the aether resolution algorithm explicitly?
I think there is no need, it's added by kotlin executable by default.

Kscript is not using gradle (unless --idea is being used to create a temporary script project), it simply resolves declared dependencies with aether (and more recently also the kotlin resolver API).
And since there is not even an option to not include the stdlib when calling kotlin in https://github.com/holgerbrandl/kscript/blob/master/src/main/kotlin/kscript/app/Kscript.kt#L284 the kotlin core team does not seem to consider it a sensible option to exclude stdlib when running applications.
I'd deployed a simple test project with v1.4.10 of the kotlin gradle plugin, and this is what I find

no dependency being declared --> installed pom does not list stdlib but gradle still links stdlib for compiling
compileOnly dependency on stdlib --> installed pom does not list stdlib
compile dependency on stdlib --> install pom lists stdlib

<?xml version="1.0" encoding="UTF-8"?>
<project xmlns="http://maven.apache.org/POM/4.0.0" xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xsd/maven-4.0.0.xsd" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance">
  <modelVersion>4.0.0</modelVersion>
  <groupId>com.github.holgerbrandl</groupId>
  <artifactId>kscript-annotations</artifactId>
  <version>1.7-SNAPSHOT</version>
  <dependencies>
    <dependency>
      <groupId>org.jetbrains.kotlin</groupId>
      <artifactId>kotlin-stdlib</artifactId>
      <version>1.4.10</version>
      <scope>compile</scope>
    </dependency>
  </dependencies>
</project>

To me this this seems like a correct behavior. tldr It's the obligation of a library vendor to correctly use no dependency on stdlib or compileOnly. If so, the library can be used with kotlin or kscript without any stdlib version conflicts.

@holgerbrandl in contrast to gradle, maven, and other advanced build systems, the kotlin/kotlinc tools do not offer any classpath conflict management. Basically, the first jar in the classpath wins, or so.
The kotlinc has the -no-stdlib and -no-reflect options, which allow specifying the stdlib/reflect explicitly in the classpath, but that's about it.
The kotlin runner, unfortunately, has even less - only -no-reflect. Maybe -no-stdlib should be added there too, but seems we weren't aware of any demand for that until now.
So, at the moment, the simplest solution I can think of is to use kotlinc instead of kotlin in the kscript and use -no-stdlib to handle stdlib conflicts.

Using kotlinc in kscript seems like a reasonable workaround 😃
In the meantime, if folks hit this, you can use the Apache Commons CLI arg parser.
It's a bit more crude and verbose than Clikt (namely no Env var support) but works fine otherwise.  Use this artifact:
@file:DependsOn("commons-cli:commons-cli:1.4")

@ligee

I think kotlinc can't be used here because it is meant to compile code and not to run it.  E.g. there is afaik no way to define the application entrypoint.
Indeed, adding -no-stdlib to kotlin would allow a version agnostic usage, where dependencies declare what version of the stdlib to use, which is - as I have learnt in this thread - the new default enforced by the gradle kotlin plugin. So to me  -no-stdlib for kotlin would be the natural followup to support this new default policy.

@kenyee Nobody wants argparser anylonger :-) With kscript v3.0 clikt works at least when matching a configuration (version 1.4.10 of kotlin being installed and clikt compiled against 1.4.10) is being used. See examples/clikt-example.kts
brandl@ubuntu2:kscript$ kscript --version
kscript - Enhanced scripting support for Kotlin on *nix-based systems.

Usage:
 kscript [options] <script> [<script_args>]...
 kscript --clear-cache

The <script> can be a  script file (*kts), a script URL, - for stdin, a *.kt source file with a main method, or some kotlin code.

Use '--clear-cache' to wipe cached script jars and urls

Options:
 -i --interactive        Create interactive shell with dependencies as declared in script
 -t --text               Enable stdin support API for more streamlined text processing
 --idea                  Open script in temporary Intellij session
 -s --silent             Suppress status logging to stderr
 --package               Package script and dependencies into self-dependent binary
 --add-bootstrap-header  Prepend bash header that installs kscript if necessary

Copyright : 2020 Holger Brandl
License   : MIT
Version   : v3.0.2
Website   : https://github.com/holgerbrandl/kscript
Kotlin    : 1.4.10-release-411
Java      : JRE 11.0.9+11
brandl@ubuntu2:kscript$ kscript examples/clikt-example.kts
Your name:

To me, this solution solves the described issue sufficiently well, so I'm closing this ticket for now. Feel welcome to reopen if needed.

@holgerbrandl I think there are quite easy workarounds that would allow using kotlinc, the entry point is detected automatically if you have a single main function, and if it doesn't work for some reasons, the runner code is not really sophisticated. But of course, adding -no-stdlib to kotlin runner is a good idea. I created an issue about it - https://youtrack.jetbrains.com/issue/KT-43294.

confirmed this fixes it...thanks :-)
Starred the youtrack issue too :-)

apparently, it doesn't fix it...I got into a bad state again by switching kotlin from 1.4.10 to 1.3.72 and back.
Same error:
Exception in thread "main" java.lang.reflect.InvocationTargetException
	at sun.reflect.NativeConstructorAccessorImpl.newInstance0(Native Method)
	at sun.reflect.NativeConstructorAccessorImpl.newInstance(NativeConstructorAccessorImpl.java:62)
	at sun.reflect.DelegatingConstructorAccessorImpl.newInstance(DelegatingConstructorAccessorImpl.java:45)
	at java.lang.reflect.Constructor.newInstance(Constructor.java:423)
	at Main_Scriptlet_dfc6de29a95db418$Companion.main(Main_Scriptlet_dfc6de29a95db418.kt:6)
	at Main_Scriptlet_dfc6de29a95db418.main(Main_Scriptlet_dfc6de29a95db418.kt)
	at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
	at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.lang.reflect.Method.invoke(Method.java:498)
	at org.jetbrains.kotlin.runner.AbstractRunner.run(runners.kt:64)
	at org.jetbrains.kotlin.runner.Main.run(Main.kt:149)
	at org.jetbrains.kotlin.runner.Main.main(Main.kt:159)
Caused by: java.lang.NoSuchMethodError: kotlin.jvm.internal.PropertyReference1Impl.<init>(Ljava/lang/Class;Ljava/lang/String;Ljava/lang/String;I)V
	at Scriptlet_dfc6de29a95db418$AppDirCli.<clinit>(scriptlet.dfc6de29a95db418.kts)
	at Scriptlet_dfc6de29a95db418$Startup$Companion.main(scriptlet.dfc6de29a95db418.kts:134)
	at Scriptlet_dfc6de29a95db418.<init>(scriptlet.dfc6de29a95db418.kts:165)
	... 13 more

Version info from kscript:
Copyright : 2020 Holger Brandl
License   : MIT
Version   : v3.0.2
Website   : https://github.com/holgerbrandl/kscript
Kotlin    : 1.4.10-release-411
Java      : JRE 1.8.0_242
Basically the same issue I think... it's somehow using the kotlin 1.3 stdlib but it's Kotlin 1.4.
Doing sdk uninstall kotlin 1.3.72 does not help :-(
The difference now is that doing
sdk use kotlin 1.3.72 doesn't make this work...

Hmm, I've tried to switch back and further the used version of kotlin with sdk use but I can not reproduce the problem yet. It just seems to work fine.
Could you clear the cache --clear-cache once and maybe check with --version that indeed the correction version of kotlin is being used by kscript?

I did that before I printed the version info above.
I'm a bit puzzled how I got it into this state as well....tried using sdk man to switch kotlin 1.3.72 and 1.4.10 back and forth w/ the same problem.  Did kscript --version and kotlin -version to confirm.
I suspect once we switch to -no-stdlib in the kotlin runner, this will no longer happen.

I'm not sure how to proceed here. In order to do so, we'd need to find a way for me to reproduce the problem.

Can leave this closed...I'll see if I can figure out how to repro this in a docker container so the test environment is consistent.


Populate registry storage account

Which issue this PR addresses:
Old cluster will not have this and egress will fail.
What this PR does / why we need it:
Egress adoption for old clusters should fail
Test plan for issue:
N/A
Is there any documentation that needs to be updated for this PR?
No

merging because this is now Green on E2E


Can't extract string literals from .uasset/.uexp files

I try to extract strings from an asset files but only some show in the resulting dumps while many of them don't. For example string with Key 419719443B6B978C3A4D6B35AC51928 in the attached dump is nowhere to be seen. Is it a parsing bug or am I doing it wrong?
Here's the code and the sample files I used:
from UE4Parse.Assets.PackageReader import LegacyPackageReader
from UE4Parse.Provider import DefaultFileProvider
from UE4Parse.Versions import EUEVersion, VersionContainer
from UE4Parse.BinaryReader import BinaryStream

import os, json
path0 = r"FilePath.uasset"
path1 = r"FilePath.uexp"

provider = DefaultFileProvider(os.path.dirname(path0), VersionContainer(EUEVersion.LATEST))
provider.initialize()
pack = LegacyPackageReader(BinaryStream(path0), BinaryStream(path1), provider=provider)
if pack is not None:
    package_dict = pack.get_dict()
    with open(path0 + ".txt", "w", encoding="utf-8") as f:
        f.write(json.dumps(package_dict, ensure_ascii=False, indent=4))

how do you know that string is there in the asset?

@MinshuG: by using binary search of it's UTF-16 bytes. The Key field is also there.

offset?

0x4C1C in .uexp


Numeric extension doesn't seem to support min/max values

I've tried a number or ways to set up an integer mask with min/max values, but I'm not getting anywhere.
I've done:
<input type="text" data-inputmask="'alias': 'integer', 'min': '0', 'max': '255'" />

I've also tried:
$input.inputmask('integer', { min: 0, max: 255 });

It limits the field to integers, but it doesn't seem to limit the integer range. Does something need to be added for this to work?

@matthew-dean ,
I will fix this for integer.


vagrant up --provider=virtualbox ERROR

Hi,
I am completely new to vagrant and bosh-cli...
I just gone through the documents to learn about bosh.
when i followed the doc, the initial step is to install vagrant in our local machine.so i started and followed all the steps.
when I run the command
vagrant up --provider=virtualbox
it given me error as There was an error while executing VBoxManage, a CLI used by Vagrant for controlling virtualbox
the complete logs are..
**Bringing machine 'default' up with 'virtualbox' provider...
==> default: Checking if box 'cloudfoundry/bosh-lite' is up to date...
==> default: Clearing any previously set forwarded ports...
==> default: Clearing any previously set network interfaces...
==> default: Preparing network interfaces based on configuration...
default: Adapter 1: nat
default: Adapter 2: hostonly
==> default: Forwarding ports...
default: 22 (guest) => 2222 (host) (adapter 1)
==> default: Running 'pre-boot' VM customizations...
==> default: Booting VM...
There was an error while executing VBoxManage, a CLI used by Vagrant
for controlling VirtualBox. The command and stderr is shown below.
Command: ["startvm", "ddf3ae79-8885-4cac-a5fe-eb83660b295a", "--type", "headless"]
Stderr: VBoxManage: error: Failed to load R0 module /usr/lib/virtualbox/VMMR0.r0: World writable: '/usr' (VERR_SUPLIB_WORLD_WRITABLE).
VBoxManage: error: Failed to load VMMR0.r0 (VERR_SUPLIB_WORLD_WRITABLE)
VBoxManage: error: Details: code NS_ERROR_FAILURE (0x80004005), component ConsoleWrap, interface IConsole**
I dont understand anything from these logs.
Please suggest me for the same.
thanks in advance.
Mohan.V

I haven't personally seen this before, but a search for the error turns up this forum topic which suggests double checking your /Applications and /Applications/Virtualbox.app directory permissions. Might want to try that?


TF & Sonnet versions easy to specify.

Codecov Report

Merging #56 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #56   +/-   ##
=======================================
  Coverage   74.19%   74.19%
=======================================
  Files          57       57
  Lines        2593     2593
=======================================
  Hits         1924     1924
  Misses        669      669

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 5f3489e...d3a61cc. Read the comment docs.


link input plugin

As a Content Creator I want a input plugin so that users may provide Generic links.
Feature: Link Input Plugin
  As a content creator
  I want an input plugin for generic links
  So that user can add generic links

  Scenario: Links - URL
    Given a Link would require a URL
    When I enter content about a Links
    Then I would need to enter the URL
      And data type would be url
      And this type is required to save

  Scenario: Links - Title
    Given a Link would require a title
    When I enter content about a Link
    Then I would need to enter the title
      And data type would be text

link input plugin

[ ] URL
[ ] Title

need multi-input plugin js first: https://github.com/punchcard-cms/shared-tests/issues/11


running against IE 11

trying to break the internet by running stuff against IE

👍

thanks man. it s still WIP :)


Uninstalling doesn't actually uninstall lefthook

We had lefthook installed on a project, but do to some issues, we removed it.
However, the lefthook.yml file keeps getting created and added to the repo. I don't know where the file that keeps adding it is and feel that npm uninstall should remove all traces of the package from the project.
At the very least there should be some documentation in the README about uninstalling.

I'm not sure, but maybe there is a leftover hook that is actually reinstalling lefthook. I think they use the prepare-commit-message hook for this, take a look at .git/hooks/prepare-commit-message.

@vaporyhumo Yes that might be the case. I have this in .git/hooks/prepare-commit-message
#!/bin/sh

if [ "$LEFTHOOK" = "0" ]; then
  exit 0
fi

if [ -t 1 ] ; then
  exec < /dev/tty ; # <- enables interactive shell
fi

dir="$(git rev-parse --show-toplevel)"

call_lefthook()
{
  if lefthook -h >/dev/null 2>&1
  then
    eval lefthook $1
  elif test -f "$dir/node_modules/@arkweid/lefthook/bin/lefthook"
  then
    eval "$dir/node_modules/@arkweid/lefthook/bin/lefthook $1"
  elif bundle exec lefthook -h >/dev/null 2>&1
  then
    bundle exec lefthook $1
  elif npx lefthook -h >/dev/null 2>&1
  then
    npx lefthook $1
  elif yarn lefthook -h >/dev/null 2>&1
  then
    yarn lefthook $1
  else
    echo "Can't find lefthook in PATH"
  fi
}

# lefthook_version: a25d332481a8bb43fe436eb390e6e4c1

call_lefthook "install"

call_lefthook "run prepare-commit-msg $@"

Seems like this would be safe to remove, right?

Yes, it is. By default repos only have samples of what hooks should look like, If by any chance you want to restore them, just copy them from a new repo.

lefthook uninstall uninstalls the hooks, not the lefthook executable. If you have it as a NPM dependency, you should uninstall it from the package.json. I'll add this to the docs


hardware address issue

I am using your BLE library for ESP32 in arduino ide.
I am able to print the address of all the devices but not of my iBeacon tag.
Please give some solution for the above issue asap.
My iBeacon is CC2541 based HM10 module.
Thanks and Regards
Kamleshwar

Kamleshwar,
Does your beacon show up in the nefConnect app ?

Yes, it shows all the information

Try to run scan example code and watch logs. You will see if this ibeacon will be discovered and logs will be printed with or without address.

where are the log files created in it& where are it stored!

Use putty to connect esp32 and print logs. Just remember to change log level to verbose or at least to debug.

we are using Arduino IDE not ESP-IDF so please go through the issue again and carefully.

Thats fine, you can use arduino-ide. In Tools menu you have Core Debug Level, set it to debug or verbose.
Now you can use putty or serial monitor to see logs, like i mention before.

You have nice instructions here:
https://github.com/nkolban/esp32-snippets/blob/master/cpp_utils/ArduinoBLE.md#switching-on-debugging

thank you !! i had seen the logs ,
though could you help me to get the adddress ! & the it is ignorred to!

As you can see everything works, beacon is discovered and MAC is printed in logs. Now here you can filter devices by name or by UUID (this is just example how to filter with UUID):
https://github.com/nkolban/ESP32_BLE_Arduino/blob/af865a916795289c8e7e09b091ff2140c33fc3fe/examples/BLE_client/BLE_client.ino#L78
and then get address:
https://github.com/nkolban/ESP32_BLE_Arduino/blob/af865a916795289c8e7e09b091ff2140c33fc3fe/examples/BLE_client/BLE_client.ino#L84

What if we don't know the UUID?
Can we get the MAC without it? because i was able to do so in esp-idf

i tried the client.ino file as you mentioned though it is unable to get the address

How do you know? How did you try to print address to console log when you get it with pServerAddress = new BLEAddress(advertisedDevice.getAddress());

yes i tried it & hereby is the screenshots

Try this one:
advertisedDevice.getAddress().toString().c_str()
as above in your code, instead of simple getAddress()

https://github.com/nkolban/esp32-snippets/blob/master/cpp_utils/BLEAdvertisedDevice.cpp#L490

nope nothing !

Serial.println("Add %s", advertisedDevice.getAddress().toString().c_str());

nothing again

bhai tujhe kuch nahi ata
though thanks !

thank you friend
4 !

then:
Serial.printf("Add %s", advertisedDevice.getAddress().toString().c_str());
or
`Serial.println( advertisedDevice.getAddress().toString().c_str());'

thank you got it!

@kapil97 this issue seems solved, could you please close it to keep the issue list clean?
cc @chegewara


Null-valued headers will never be considered equal

While I won't defend null-valued headers, some libraries set them and curl will happily ignore them. They should either not stored or should compare as equal.

Fixed in 394737a317f79abe1c7bdbe15621fe45700e24b7


ERROR Error: NG0909: Expected to be in Angular Zone, but it is not!

Demonstration
This issue happens when upgrading the system to Angular is older than 17.04

Expected Behavior
Actual Behavior
localhost-1713262609482.zip

#511 It's might be similar problem.
For my application => 17.0.8 breaks Angular Zone.


Tiny-imagenet ECE of cross entropy

Thanks for your great paper and code.
I have some questions about reproducing paper results.
I ran this code without any changes except the dataset path on tiny-imagenet. However, the ResNet50 cross entropy (baseline) ECE is too high than paper.
In my experiment, the ECE of cross entropy is 18.387 without temperature scaling, while the ECE of the paper is 3.73.
I also used your split and the script in README.
I appreciate your attention to this issue.
Thank you for your time and support.

I have attached the config and log
dist_train_rank0.log
config.txt


feat: add blas/base/zswap

If applied, this commit will add the package blas/base/zswap
Description

What is the purpose of this pull request?
Add the package blas/base/zswap

This pull request:

includes the support for C and Fortran and obtains Fortran reference from this page

Related Issues

Does this pull request have any related issues?

This pull request:

resolves #2047

Questions

Any questions for reviewers of this pull request?

No.
Other

Any other information relevant to this pull request? This may include screenshots, references, and/or implementation notes.

No.
Checklist

Please ensure the following tasks are completed before submitting this pull request.

[x] Read, understood, and followed the contributing guidelines.

@stdlib-js/reviewers

Refactoring the code to use real instead of realf as reviewed in blas/base/zcopy, will reopen once done.

@performant23 Looks like you are missing the Makefile for the /src directory.

/stdlib update-copyright-years

Ah, wait. You need to add the missing Makefile before we can merge.

Ah, wait. You need to add the missing Makefile before we can merge.

Added, thank you so much for your review!

@performant23 As you are on Windows, you were testing against the C implementation which did not have any issues. However, the Fortran implementation did have bugs, and the Fortran implementation was what was being tested against in CI. The issues should now be addressed.


nix-index: Wrap an unwrapped derivation

Essentially allow overriding the nix package used in the wrapping,
without rebuilding the rust package.

Motivation for this change
Fix https://github.com/NixOS/nixpkgs/issues/106515 - cc @poscat0x04 .
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

The main concern with wrapping in this way is that it makes it much harder to override the Rust "unwrapped" derivation.
Would it be more helpful to use "--suffix" instead of "--prefix" so the user can use their own Nix version?

Would it be more helpful to use "--suffix" instead of "--prefix" so the user can use their own Nix version?

I think it depends on what the maintainer meant when they wrapped it. If it was meant to ensure compatibility, it would defeat the purpose to use --suffix, because a user might have a nix in their path (which would then be used instead of what in the suffix of the PATH in the wrapper) which might not be compatible.

The main concern with wrapping in this way is that it makes it much harder to override the Rust "unwrapped" derivation.

I modified the changes so now it is possible to override the unwrapped derivation as well, with examples in the comments of wrapper.nix.

Does the nix version used by nix-index even matter?

Does the nix version used by nix-index even matter?

It probably does not, as nix-index only relies on nix-env. A better option might be to remove the wrapper altogether.

It probably does not, as nix-index only relies on nix-env. A better option might be to remove the wrapper altogether.


Pass PathString.Value to prevent encoding

Fixes https://github.com/microsoft/reverse-proxy/issues/345

Thanks


copyresources task is broken. needs a slash between projectDir & resourceDir

Running the copyresources task with --debug shows the following:
//
12:08:57.634 [DEBUG] [org.gradle.api.internal.tasks.execution.ExecuteActionsTaskExecuter] Executing actions for task ':copyresources'.
12:08:57.643 [INFO] [org.gradle.api.Task] from /Projects/zero/zerojs-ace-demosrc/main/resources to /Projects/zero/zerojs-ace-demo/build
//
Notice that there needs to be a slash between the projectDir and the resourceDir.
The workaround is to manually set the resourceDirs manually with a slash at the front:
resourceDirs = ['/src/main/resources'];

Thanks for the bug report.
Are you using a Windows, Linux or Mac device?

Fixed it in the 'develop' branch. Will be in the next release. Thanks again for the bug report.

Awesome! Thanks for creating GradleFx. I'm very happy to have an alternative to FlexMojos.

I just released GradleFx 0.4.1 which contains the fix for this issue.


I can't find a way to change backgroudColor of  "text only" Hud.

how to change "text only" hud background Color?

Use following property to change text only hud background color.
NOTE: It will change color but hidden behind blur view.
MBProgressHUD *hud = [MBProgressHUD showHUDAddedTo:self.view animated:YES];
hud.bezelView.backgroundColor = [UIColor redColor];


[NETBEANS-96] New PAC Script evaluation environment

Implements a completely new PAC script environment to rectify the issues described in NETBEANS-96.
Also, Architecture documentation has been created for the module. (didn't exist previously).  There are a lot more test cases now. Previously there was virtually no testing of the PAC execution.
For now, the module requires Friend for everything. So even though it has been documented how one can plug in own PAC script evaluation environment, it is at this stage deliberately kept in Friend mode.
There are still many more things I have in the pipeline both for ProxySelector and for Authenticator. But I thought I would do it in incremental steps. Otherwise it gets too big for anyone to fully understand and thus review. Therefore this PR is solely about replacing the PAC script evaluation environment.
The new code requires Java 8. I get a few warnings on build, mainly related to my use of @NonNull annotation, but it doesn't seem to create problems. Happy to change.

Matthias, your comments are a pleasure to work with. !!
Amazing. Thanks. I'm on it.

Unit tests fixed.

The timeRange() test failed because it did assertTrue(timeRange(0, 23)), and since you did your test at 23:17 it would fail the test. The test case was obviously wrong to begin with and his been fixed.

The dnsResolveEx() test failed because it made assumptions about IPv6 being available on the test host and was in any case incorrect to begin with. All tests that use name lookup have now been changed to use the FakeDns class, so that tests become independent of the environment of test host.

Thx.

I gave it another look I have one further comment on the code itself - the spec version of lib.jna.platform needs to be incremented, as the change in the friend-packages indicates that the package needs an update. Some of the files are also missing license headers. I made these here:

based on a rebased+squashed changeset:
https://github.com/matthiasblaesing/incubator-netbeans/tree/pr161-rebase
https://github.com/matthiasblaesing/incubator-netbeans/commit/6e21bcdd1c5d9f3eab4868b5ed1245c94b9c02be

based on your branch:
https://github.com/matthiasblaesing/incubator-netbeans/tree/pr161-review
https://github.com/matthiasblaesing/incubator-netbeans/commit/5826080b5301846ff2e38cb2f004e0f470526cc4

One structural question: Is there a reason for the friend-exports? I don't see usage of the org.netbeans.core.network packages in org.netbeans.core. Is there a concrete use-case/usage you have planed? Is it really sensible to have a different implementation of the JS evaluation environment for the PAC, that is not better done in the core implementation?
From my perspective that would be the last two things - I noticed this while I prepared the merge.

I've done the requested changes. (license headers, bumped libs.jna.platform version)
Wrt frind-packages.
It is correct that org.netbeans.core doesn't need to be friend on org.netbeans.core.network, but there needs to be at least one friend in <friend-packages>  for  the XML to be valid. If I leave out <friend-packages> completely then I loose two things:

I do not get any Javadoc produced, thus any adopter will not know what is possible to do.
I have no way way to "document" which packages I intend (in a controller manner for now, hence friend restriction) to be exposed, and which I intend not be exposed.

In other words: The friend declaration is a trick. If there's another way to accomplish the same, then by all means.
The <friend-packages> section will become even more important in the next iterations, but even in next iteration I do not really have a specific friend that I can add, so org.netbeans.core kinda acts like a dummy in this respect.

Thank you very much for the work you put in here and the time you took to answer my questions/comments.
I rebased the work onto master and squashed the changeset together to improve readability (from experience I know, that there is a need to read comments from commits in the past and a compact commit helps there).
A final thought about a public SPI (pluggable PAC evaluation):

a separate SPI module should be created on which the
core.network would depend on

PacJsEntryFunction should be removed from
org.netbeans.core.network.proxy.pac package, as it is not
used in the SPI

PacScriptEvaluatorNoProxy and PacUtils should be removed from
org.netbeans.core.network.proxy.pac package, as they are
not part of the SPI, but utility classes

Only the org.netbeans.core.network.proxy.pac package
should be exposed

Usage of interface/class should be unified:
org.netbeans.core.network.proxy.pac.PacHelperMethods
org.netbeans.core.network.proxy.pac.PacScriptEvaluator
org.netbeans.core.network.proxy.pac.PacHelperMethodsMicrosoft
org.netbeans.core.network.proxy.pac.PacHelperMethodsNetscape
org.netbeans.core.network.proxy.pac.PacScriptEvaluatorFactory
should either be interfaces or abstract classes.
With default methods interface can be evolved similar to  abstract classes.

@lbruun Does this patch completely replace the old NetBeansProxy2 project?
(I've been including NetBeansProxy2 in my NetBeans Platform application for some years now, and was wondering if I can now drop it on the newer platform versions.)


registry pod restarted many times after a long run

NAME                                                              READY   STATUS      RESTARTS   AGE
harborcluster-sample-harbor-harbor-chartmuseum-7469cccf75-9pdfx   1/1     Running     9          6d20h
harborcluster-sample-harbor-harbor-core-994547f45-9n5bl           1/1     Running     0          6d20h
harborcluster-sample-harbor-harbor-exporter-db875ccd5-4fk69       1/1     Running     0          6d20h
harborcluster-sample-harbor-harbor-jobservice-7999c9646d-58fcv    1/1     Running     10         6d20h
harborcluster-sample-harbor-harbor-notaryserver-5fd56f54df2drmh   1/1     Running     0          6d20h
harborcluster-sample-harbor-harbor-notarysigner-8559bd7694q8znk   1/1     Running     0          6d20h
harborcluster-sample-harbor-harbor-portal-7598c7c4b9-mjzq5        1/1     Running     5          6d20h
harborcluster-sample-harbor-harbor-registry-667bc8cc95-gqchb      1/1     Running     1796       6d20h
harborcluster-sample-harbor-harbor-registryctl-5748699f8-c8fxf    1/1     Running     12         6d20h
harborcluster-sample-harbor-harbor-trivy-76b6855fd9-tsgkd         1/1     Running     1          6d20h
minio-harborcluster-sample-jxmd6                                  0/1     Completed   0          6d20h
minio-harborcluster-sample-zone-harbor-0                          1/1     Running     0          6d20h
minio-harborcluster-sample-zone-harbor-1                          1/1     Running     0          6d20h
postgresql-cluster-sample-ns-harborcluster-sample-0               1/1     Running     0          6d20h
rfr-harborcluster-sample-redis-0                                  1/1     Running     0          6d20h
rfs-harborcluster-sample-redis-6b7f4c4756-22sqw                   1/1     Running     2          6d20h

k8s cluster is kind 0.10

After following the instructions for v1.1.1, I am getting the same issue.  It appears the registry container is failing health or readiness checks.
This is frustrated because the "Check the overall status of the deployed Harbor cluster" step reports it is healthy.
GET / HTTP/2.0" 503 125 "" "kube-probe/1.21 is repeated many times in the logs for the registry container.

Is there any fix for this problem?  I'm getting the same issue after following the instruction for v1.2.0.
The cluster is the the unhealthy state most likely due to the fact that the registry container is crashlooping.
time="2022-03-30T07:11:11.292020425Z" level=debug msg="s3aws.Stat("/")" go.version=go1.17.2 instance.id=b3a63ccc-0715-4011-bdd8-dc53f367b12c trace.duration=332.251904ms trace.file="/go/src/github.com/docker/distribution/registry/storage/driver/base/base.go" trace.func="github.com/docker/distribution/registry/storage/driver/base.(*Base).Stat" trace.id=463aa209-2158-4b82-8380-78c54ea2c6f1 trace.line=155 version=v2.7.1.m
<IP_ADDRESS> - - [30/Mar/2022:07:11:15 +0000] "GET / HTTP/2.0" 503 125 "" "kube-probe/1.21"
<IP_ADDRESS> - - [30/Mar/2022:07:11:15 +0000] "GET / HTTP/2.0" 503 125 "" "kube-probe/1.21"

In my case the registry container wasn't working because DNS wasn't working. Now that I can resolve core, notary and minio hostnames I was also able successfully deploy Harbor.

In my test cluster, I resolved this issue by configuring pod HostAliases to add Pod-level hostname resolution.

but why not use service


Change product snapshot version

This PR changes the product snapshot version from 4.1.0-SNAPSHOT to 4.2.0-SNAPSHOT.

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


[BUG] Wombat HP Value

Upon entering the game, the first value sent is 'nil'.
Upon wombat dying, health is nil instead of expected "0".
Initial health should be set to 100 at the beginning of the game. When viewing game after it has closed, health should be the last value of the player's hp.

@oconn @dehli
I'd like to check the implementation when we start sending back game status with sockets. Things I'm concerned about
When the game status is closed and you're viewing the game, the last HP value isn't persisted and sends back nil. This will cause the bots to be grayed out since I adjust a nil hp to 0 if the game isnt pending-open or pending-closed. If this isn't the desired state on :closed then we shuold create another ticket.

Another thing to be conscious about is the round intermission HP values.


Pre-work Checklist: DevOps-Security-Member: Joss Graf (endlesslupita)

Prerequisite
We are looking forward to having you on our team. Please ensure you have prior experience with the HfLA website team before contributing to our repository.
Overview
As a new member on the HfLA devops-security team, fill in the following fields as you complete each onboarding item.
Special Notes

Keep this issue open until you've completed all steps, including learning to provide updates for longer tasks.
Normally, handle one issue at a time, but this one is an exception as it teaches how to manage multiple tasks.
Work on action items sequentially, but proceed if possible. For example, set up your dev environment without waiting for the weekly meeting.

Action Items

[x] Before starting to work on the below instructions, make sure to join the #ops Slack Channel. And are a member of devops-security repository.

[x] Self-assign this issue (gear in right side panel).

[x] Add this issue to the Project Board under the Projects section (gear in right side panel).

[x] Attend weekly team meeting, Wednesdays 6-8pm PST.

[x] Note: There are no meetings on the 1st-7th of every month.

[x] Complete the steps in Creating a personal AWS account and Login as root user & setup MFA.

[x] Read and follow the instructions in Setting up IAM and AWS CLI for:

[x] Creating an IAM User
[x] Creating an IAM Group
[x] Attaching IAM user to IAM Group
[x] Providing AdministratorAccess policy to IAM group
[x] Log in as the newly created user instead of continuing to log in as the root user (it is not recommended to login with root access).
[x] Generating user access keys

[x] Complete the instructions in AWS Documentation and choose your operating system to install AWS CLI.

[x] Complete the instruction in AWS Documentation to setup the AWS CLI.

[x] Read follow the instructions in Creating a backend state.

[x] Install Terraform locally by following the instructions of the installation guide mentioned in Installing Terraform

[x] Install Terraform Docs locally by following the instructions of the installation guide mentioned in Installing Terraform docs

[x] Complete the instructions in Clone the repository

[x] Submit a new request to create new AWS user account and then self-assign this issue.

[ ] Create a new branch from main by executing the command
git checkout -b issue-number-add-new-iam-user

[ ] Navigate to the aws-user.tf file and add your user information and follow the below template.

module "iam_user_testiamuser" {
source = "./modules/aws-users"

user_name = "testiamuser"
user_tags = {
  "Project"      = "devops-security"
  "Access Level" = "1"
}
user_groups = ["read-only-group"]
}

[ ] In your code editor navigate to terraform directory. cd terraform

[ ] Execute the command terraform init to initialize terraform in the directory. Address any failures that arise (if any).

[ ] Execute the command terraform plan this will output a plan replicating the same IAM resources as the devops security account. Address any failures that arise (if any).

[ ] Then execute the command terraform apply this will create all of the resources that are currently managed by Devops Security. All of the resources created here incur zero cost except for the Dynamo DB installation, which should remain in the free tier.
- [ ] ** If you have cost concerns, Run a Terraform Destroy to take down all of the resources you created (don't worry, you can recreate them just as quickly). **

[ ] Once you have tested your changes, stage them in git with

[ ] git status command.
[ ] then git add path/to/file (you can copy from above output for the file path).

[ ] Commit the changes by executing git commit -m "briefly describing the changes".

[ ] Push the changes with git push --set-upstream origin name-of-branch

(Apologies in advance that this suggestion is not in standard format. I haven't learned how to propose documentation edits yet.)
Some questions I considered while drafting these changes: 1. How much should we expect the new member of the team to already know about GitHub or version control in general? vs. spelling everything out in detail 2. How quick is it to troubleshoot if the step is not complete? i.e. which links should we add, and what is the chance that each link will become outdated? 3. Is the pre-work only to help onboard to contribute (infrastructure level) or also to have the new members familiarize themselves with the team's GitHub content and features (knowledge level)?
Suggested edit to Checklist: Split into two list items and revise text.
Original:
Before starting to work on the below instructions, make sure to join the #ops Slack Channel. And are a member of devops-security repository.
New:
[1] Join #ops Slack channel. If you are not a member of the Hack for LA Slack group, request membership via this link: [LINK].
NOTE: Or is the Slack access given during Hack for LA orientation and not being in the Slack group yet means the new member should go back and complete orientation?
[2] Verify that you have permissions in GitHub needed to write to this issue and participate in the DevOps team.
Link from Verify above:
A. Verify that you are a member of the hackforla organization on GitHub. You are a member if the readme.md file you see on the hackforla Overview page says "This is a private page only visible to Hack for LA members." You would have been added during Hack for LA orientation.
B. Verify that you are a member of these teams in the hackforla organization: ops-security, ops-security-write, and ops-write. To view the teams you are in: 1) Click on People in the right column of the hackforla Overview page. 2) Search for your Github user name. 3) On the right of the box with your user info, you should see something like "Public Member [number of] teams." Click on "teams" and a list of teams should appear.
C. Verify that you are a [member?] of the devops-security repository. This is not the same as appearing in the list of contributors (which is only a subset of members).
NOTE: I am not sure that "member" is a role or permission level for a repository. I can't find it in the GitHub documentation. Instead, I see "read, write, maintain, or admin permission levels." They seem to be based on "membership" at the team level. Do we want to say "Verify that you have write permission in the devops-security repository"? Would the easiest test be whether it is possible to check a box in the checklist of the issue and/or whether it is possible to assign oneself to the issue? "Self-assign" is the next task in the checklist.

Hi Joss, what is your role?  The ones we have to choose from are:

[ ] Security Engineer
[ ] Site Reliability Engineer


Missing config causes exception in Appsignal::IPC.forked!, which kills all resque jobs

Hi,
setup is rails 4.2 with resque integration.  Hence, for appsignal there wasn't any configuration.
But the consequence turned out to be quite bad.
Appsignal::IPC.forked! tries to call stop_thread on Appsignal.agent, which is nil as it is only set when a config is present. That in turn stops the (all!) jobs from working/respawning at all with an exception.
I'd expected that we - naturally - wouldn't receive any signals, but that the job framework is killed is a rather harsh consequence :).
Cheers
SH

Hi there,
We try to make sure AppSignal doesn't bring down the app when something is wrong with the gem itself, but as you unfortunately experienced there are some edge cases.
I'll make sure we'll fix this in the next version, and will keep you updated.
Thanks for reporting!


Ex range command improvements

Known Issues Fixed:

#718 (named marks in ex range)
#700 (ex copy feature request)

Improvements:

named marks supported in ex ranges
swapping backwards ex range
change . to use insertion point instead of selection begin
ex yank implemented
ex delete implemented
ex copy implemented
ex move implemented
ex shift left and right implemented
updated Feature List with Ex range and range command information

@JugglerShu , just checking on this pull request.  Do I need to rebase this against the develop branch in order to get it considered for inclusion?  Or is there something else amiss with this?
Thanks

Thank you for the PR. Yes, please PR to develop branch.
And also, please understand that current develop branch is buggy and I currently only accept the bug fixes for the branch. So I cannot merge your PR immediately. (Maybe after some work on the branch, I'll merge yours.)

Ok, PR #735 created against develop.  Closing this PR.

I will leave my original master based feature branch open until my develop based feature branch has been merged into develop, and develop stabilizes.


add to window, not this

Fixes an issue pointed out by http://elementaryos.stackexchange.com/questions/2848/getting-started-with-gtk-grid-with-error

Looks good. It makes more sense now as the docs are only talking about putting everything in the main function.


Fix CI: Make cargo doc warnings critical

Recent changes to GH page deployment made doc warnings critical, but this was not caught in CI. This PR adds the missing CI warning and fixes some newly introduced doc warnings.

Confirming: the first commit that fixed CI actually does it:

I will fix the doc warnings now.


Application example does not remove app

Executing the applications-example a second time it gives me:
Failed, error: conflicting resource
I will commit a fix soon

replace this
	if _, err := client.Application(applicationName); err != nil {
		deployID, err := client.DeleteApplication(applicationName)
		assert(err)
		waitOnDeployment(client, deployID)
	}

with this
    if deployID, err := client.DeleteApplication(applicationName); err == nil {
        waitOnDeployment(client, deployID)
    }

close issue because of PR


utils/ruby.sh: various tweaks

Disable SC2039 check ("In POSIX sh, ___ is undefined.")
find_ruby:

set HOMEBREW_RUBY_PATH instead of echo-ing its location to stdout. To avoid using a subshell.
Make ruby_exec and IFS variables local.

setup-ruby-path:

upgrade_fail:

Unify the message for macOS and Linux.
Remove Linux-specific comment as it doesn't apply to the 'upgrade' step
Add a clarifying note if HOMEBREW_FORCE_VENDOR_RUBY is set

On Linux: move advice note to install_fail
Remove conditional block that (I believe) is never executed. setup-ruby-path is called for non-Bash commands (see brew.sh). When Homebrew calls vendor-install, it doesn't source utils/ruby.sh nor it calls setup-ruby-path.# removed block
if [[ "$HOMEBREW_COMMAND" == "vendor-install" ]]
then
  return 0
fi

Check if system Ruby can be used when Homebrew fails to upgrade portable Ruby.

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.

[x] Have you successfully run brew typecheck with your changes locally?
[x] Have you successfully run brew tests with your changes locally?
[x] Have you successfully run brew man locally and committed any changes?

Review period will end on 2020-12-15 at 22:20:15 UTC.

Review period ended.

I think before any more refactoring is done on this file some basic tests should be added. Previous refactors have broken the functionality here and I don't have the ability on our confidence to catch these regressions with review alone.

Makes sense. I bundled various tweaks together to see how overwhelming it's going to look. I'll break it down into smaller pieces. Let's also discuss what kind of tests are needed.


Multi-Monitor Window Cycling Issue

With the current release of chunkwm, it seems that properly cycling focus between windows on different monitors and sending windows to specific monitors by index depends on the monitor arrangement in System Preferences.
My workstation setup is two external monitors and a MacBook Pro with a Retina display. More details can be given if pertinent.
The following screenshot shows the only System Preferences configuration I've found so far that allows chunkwm to function as expected. My MacBook is the center display.

The above arrangement isn't ideal for me because my desk mandates having my MacBook beneath my two monitors. The below System Preferences arrangement is preferred but causes chunkwm to not cycle window focus correctly or send windows to specified desktops as expected. On occasion, attempting to send a window to a specific desktop causes the window in question to disappear (seems to happen more often when sending to desktop 3).

Here are links to my current chunkwm configuration and my khd configuration with the relevant khd lines being 25-29 and 81-97.

Probably a bug, but do not have 3 monitors to investigate with.

So I have investigated this behaviour using 3monitors, with your display arrangement as mentioned (the 2 above, 1 below) and I can reproduce the behaviour 100% of the time with mpv.
The funny thing is that there is no issue in the code what so ever for this window.
Checking the source, destination and transformed coordinates, they are all calculated properly as expected, however when we try to move the window through the accessibility API it simply does not move even close to the position that we tell it to; same goes for its size.
This is not really a bug in chunkwm, but with how these applications interact with the AX API.
I'm not sure what we can do about this.. if anything.

@koekeishiya Awesome! I'm away from my workstation this week, but I'll test soon and report any issues I still have. You rock!


First feature (Full Width Tabs) - Tab row

[ ] Create tab row with one clickable tab (mobile size)

[ ] add container for holding all individual tab elements
[ ] create a ul and an individual li for the first tab
[ ] include icon centered within each tab
[ ] tab should have a lightgray border and receive a hover psuedo element that changes the color to a darker gray color
[ ] include a click event listener on the tab

#20

https://github.com/sjoyal/TIY-Assignments/pull/24


Update QRD FPGA readme

Existing Sample Changes
Description
Restructured to match the new template, updated some branding, restructured multiple sections, added information on running in DevCloud, fixed a lot of formatting issues, rewrote some text for clarity, and fixed some grammar/spelling issues.
Fixes Issue#
External Dependencies
N/A
Type of change
Please delete options that are not relevant. Add a 'X' to the one that is applicable.

[x] Documentation

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

[x] Command Line
[x] VSCode

Adding @jimmytwei for notices.


Add support for cluster apps without spec.version

Towards https://github.com/giantswarm/roadmap/issues/3466
In the new releases implementation cluster-<provider> do not have version set. Instead they have release version set in app's Helm value .global.release.version.
This changes adds a cluster app mutation with the following behaviour:

Check if the provider is using the new Releases
Check if the App is a cluster app
Read release version from config .global.release.version.
Get Release CR
Get cluster app version from the Release CR
Set missing cluster app version in App .spec.version, or update already set version (needed for cluster upgrades).

Checklist

[ ] Update changelog in CHANGELOG.md.

Rebased PR and re-testing the change.


add array of plugins to icinga2.conf

squashed commits for feature "array of plugins to icinga2.conf"

@scopenco thank you fixing the commits, really appreciate it :+1: . i was able to add this commit last night after PR was closed, sorry.
thanks again though.
for kitchen test integration, you may update https://dev.icinga.org/issues/8420.

is it possible to release last updates to supermarket?


errors occur in retrain.py if there are less than 20 images in directory so delete the directory

Thanks @eclennett


Sort indices of dictionary string values

Which issue does this PR close?

Closes #2697.
Rationale for this change

What changes are included in this PR?

Are there any user-facing changes?

dict string 2^12        time:   [76.482 µs 76.916 µs 77.548 µs]
                        change: [-39.917% -39.407% -38.813%] (p = 0.00 < 0.05)
                        Performance has improved.

Thanks.


Add files via upload

This is the doc for assignment 3: describes the tech stack we're using and how we chose it. Please check accuracy. (a review is required to merge to main branch)