[SPARK-33789][SQL][TESTS] Refactor unified V1 and V2 datasource tests

What changes were proposed in this pull request?

Move common utility functions such as test(), withNsTable() and checkPartitions() to DDLCommandTestUtils.
Place common settings such as version, catalog, defaultUsing, sparkConf to CommandSuiteBase.

Why are the changes needed?
To improve code maintenance of the unified tests.
Does this PR introduce any user-facing change?
No
How was this patch tested?
By running the affected test suites:
$ build/sbt -Phive-2.3 -Phive-thriftserver "test:testOnly *ShowPartitionsSuite"
$ build/sbt -Phive-2.3 -Phive-thriftserver "test:testOnly *ShowTablesSuite"
$ build/sbt -Phive-2.3 -Phive-thriftserver "test:testOnly *AlterTableAddPartitionSuite"
$ build/sbt -Phive-2.3 -Phive-thriftserver "test:testOnly *AlterTableDropPartitionSuite"

Test build #132826 has started for PR 30779 at commit 89e3898.

Test build #132826 has finished for PR 30779 at commit 89e3898.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

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

Test build #132828 has started for PR 30779 at commit 5836e54.

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

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/37430/

Test build #132834 has started for PR 30779 at commit 04fd90b.

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/37430/

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

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/37436/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/37436/

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

Test build #132828 has finished for PR 30779 at commit 5836e54.

This patch fails PySpark unit tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
trait AlterTableDropPartitionSuiteBase extends QueryTest with DDLCommandTestUtils

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

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

@MaxGekk can you fix the conflicts?

Test build #132860 has started for PR 30779 at commit 35dc058.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/37462/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/37462/

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

Test build #132860 has finished for PR 30779 at commit 35dc058.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
case class SubqueryExec(name: String, child: SparkPlan, maxNumRows: Option[Int] = None)

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

thanks, merging to master!


We are slowly moving to zstd:chunked format for container images but osbuild is not handling it.

ostree container image deploy --imgref=ostree-unverified-image:dir:/tmp/tmpxu_tjilg/image --stateroot=default --target-imgref=ostree-unverified-registry:quay.io/rhatdan/lamp:latest --karg=rw --karg=console=tty0 --karg=console=ttyS0 --karg=root=LABEL=root --sysroot=/run/osbuild/tree
error: Performing deployment: Importing: Unhandled layer type: application/vnd.oci.image.layer.v1.tar+zstd
This happens when trying to convert "quay.io/rhatdan/lamp:latest".
Which was pushed using zstd:chunked format.

@giuseppe FYI

@cgwalters FYI?  Is this an issue with ostree not supporting other formats, or does it just need to be updated with the lastes containers-image.

It's unrelated to bib, you'd see the same error from bootc switch for example.

Moving this to https://github.com/ostreedev/ostree-rs-ext/issues/608


App crashes on iPad

When trying to share a contact from an iPad, the app crashes:
*** Terminating app due to uncaught exception 'NSGenericException', reason: 'UIPopoverPresentationController (<_UISharingViewPresentationController: 0x100636430>) should have a non-nil sourceView or barButtonItem set before the presentation occurs.'
terminating with uncaught exception of type NSException

Hi @mmklug, which version of Xcode and iOS are you using? These samples have been tested with Xcode 12.5 (and 13.0 Beta 1 for the swift-concurrency branch). The changes in #2 should fix your issue if you'd like to try out that branch, and please let us know if it continues to be an issue.

Hi @sjrmanning, unfortunately adding .navigationViewStyle(StackNavigationViewStyle()) did not solve the issue.
I am testing the code an iPad Air 2 on iOS 14.6 and the issue is present both using Xcode 12.5 and 13 beta 1.
Please note: when pressing on the share button, the "Add People" sheet appears as expected. But pressing on one of the offered options (e.g. "Copy Link") will cause the app to crash.
No issue on iPhone.

Got it, thanks for your response. I see where the crash is occurring and I've been able to reproduce it locally. The latest commits on that branch fix it by using an explicit presentationStyle on the wrapped controller, presenting as a form sheet rather than a popover presentation (the default in this case).
If you wanted to use a popover presentation for this view, you'd need a UIKit view to reference as the sourceView.

We've merged those changes in to main, but please feel free to re-open this if you still run into issues. Thanks!

Hi @sjrmanning,
this is working as expected now.
Thank you!


Wrong Login message

Issue:  The login message should come with a pop up and Notify that"User Name / Password is Invalid. "

Forgot password is not yet implemented because we don't have any email system working. It's only in page design.... As far as re captcha is concerned, It will be integrated in next version.

It is not solved yet. S it should be kept open as bug until its fixed.

Fixed.


[support] Upgrade paas-admin

What
This version includes[1] and fixes[2].
[1] https://github.com/alphagov/paas-admin/pull/109
[2] https://github.com/alphagov/paas-admin/issues/110
How to review
Check the version number is correctly tagged in the repository.
Who can review
@keymon

LGTM


feature: lazy extra for Laravel blade and php

Did you check the docs?

[X] I have read all the LazyVim docs

Is your feature request related to a problem? Please describe.
yeah it is , basically i've had a hard time configuring lazyvim for laravel Blade and php files, after doing so many things i could only get to the point to just make php run good , but cant really do any blade in it because it has multiple things going on, someone created a pr some time ago but then that guy deleted the repo so now there's nothing for that present, i have some sort of things setup for laravel projects only upto these things:
Treesitter Highlight for blade (custom parser)
Lsp can recognize the namespaces etc when i call some method or function from those, lsps also do auto imports and give code actions so there is nothing else to do other than fixing blade files i copied someone's setup for laravel so that's why i've got these many things working on , without that there isnt even a ts highlighting for blade files officially, this is my current configuration for laravel:
https://github.com/muneebusmani/nvim
Describe the solution you'd like
it also has scripting in its directives
it has php code integrated like in angular
it has html and css
Describe alternatives you've considered
there is no alternative for that i have to stick to vscode whenever i do a laravel project
Additional context
No response

All the Extras are contributions from people who dabble in the specific language domain. If you want an Extra for this, feel free to submit a PR. Otherwise, you'll have to wait and hope for someone else to do it.

i have 90% of the stuff an extra would provide but still cant figure out a way to make it blade files work, any ideas?

No idea. I have 0 experience with PHP and blade.

i just figured out even vscode doesnt have accurate intellisense blade files because there is no official lsp for that, which means that the snippets i am getting in neovim are the same as vscode, i would like to open a pr to the extra for php + laravel because i have 100% of the things an extra would provide


[BUG]Atom.jl can not update to the latest version

Please search existing issues to avoid duplicates.
Details

Atom version: 1.41.0
Julia version: 1.3.1
OS: Windows 10 pro x64
Package versions:

Atom.jl: 0.8.8
julia-client: 0.12.2
ink:0.12.2

Steps to reproduce
the version of Atom.jl on my PC is fixed at version 0.8.8, it cannot upgrade to the latest verion with ']up Atom', which makes Juno not work.
']up Atom' only indicates that there is no change, and there is no error
I don't know if there is something dependencies wrong with the packages I installed.

Try pkg> add<EMAIL_ADDRESS>That should tell you which package is stopping the upgrade.

Many thanks, I have learned a trick.
It is JuliaInterpreter 0.7.8 that block the upgrade of Atom. The problem is solved by upgrade JuliaInterpreter 0.7.8 to the latest with pkg> add<EMAIL_ADDRESS>It's strange that I used to be able to update all the packages to the latest with ]up, but this time it failed.
(v1.3) pkg> add<EMAIL_ADDRESS>Resolving package versions...
ERROR: Unsatisfiable requirements detected for package Atom [c52e3926]:
Atom [c52e3926] log:
├─possible versions are: [0.8.0-0.8.8, 0.9.0-0.9.1, 0.10.0-0.10.2, 0.11.0-0.11.3, 0.12.0-0.12.7] or uninstalled
├─restricted to versions 0.12.7 by an explicit requirement, leaving only versions 0.12.7
└─restricted by compatibility requirements with JuliaInterpreter [aa1ae85d] to versions: [0.8.6-0.8.8, 0.9.0-0.9.1, 0.10.0-0.10.2, 0.11.0-0.11.3] or uninstalled — no versions left
└─JuliaInterpreter [aa1ae85d] log:
├─possible versions are: [0.1.1, 0.2.0-0.2.1, 0.3.0-0.3.2, 0.4.0-0.4.1, 0.5.0-0.5.2, 0.6.0-0.6.1, 0.7.0-0.7.12] or uninstalled
└─restricted to versions 0.7.8 by an explicit requirement, leaving only versions 0.7.8

I'd recommend pkg> free JuliaInterpreter so that it gets updated automatically in the future.


bug

An error occurred while running random_agent.py ,as

Traceback (most recent call last):
File "E:/powergym-master/random_agent.py", line 225, in
run_random_agent(args, worker_idx=None, use_plot=args.use_plot, print_step=False)
File "E:/powergym-master/random_agent.py", line 53, in run_random_agent
env = make_env(args.env_name, worker_idx=worker_idx)
File "E:\powergym-master\powergym\env_register.py", line 287, in make_env
return Env(folder_path, base_info, dss_act)
File "E:\powergym-master\powergym\env.py", line 240, in init
self.load_profile = LoadProfile(
File "E:\powergym-master\powergym\loadprofile.py", line 24, in init
for f in os.listdir(self.loadshape_path):
FileNotFoundError: [WinError 3] 系统找不到指定的路径。: 'E:\powergym-master\systems\13Bus\loadshape'
How to solve this problem?

@bruinWang it'd be good to add some basic tests here so that these reports can be reproduced.


Refactor tellstick code

Description:
This started as an attempt to solve issue #4403. However, the tellstick code was very convoluted and hard to follow. For instance, "telldus device" was used to mean either an object derived from the HA Entity, or an object from the tellcore python library, with no clear separation. Also, the separation of concern between the common base class TelldusDevice and the swich/light subclasses was vague, with the actual workflow going up and down the hierarchy for no apparent reason.
So I decided to clean up the code as a first step, to be able to solve #4403 properly.
This is supposed to be a pure refactoring, with no changes in how the code actually behaves. I have tested that it works on my tellstick setup, but as far as I can tell, tox does not really exercise the tellstick code. :-(
Checklist:
If user exposed functionality or configuration variables are added/changed:

[ ] Documentation added/updated in home-assistant.github.io

If the code communicates with devices, web services, or third-party tools:

[x] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[ ] New dependencies have been added to the REQUIREMENTS variable (example).
[ ] New dependencies are only imported inside functions that use them (example).
[ ] New dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
[ ] New files were added to .coveragerc.

If the code does not interact with devices:

[x] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[ ] Tests have been added to verify that the new code works.

It looks good. I don't think that we have anyone really taking care of this code so you're more than welcome to take control.

I'll add some comments with potential improvements.


README.md: Fix link to PDF

Thanks, I was missing that one!

You bet! Awesome project, btw :slightly_smiling_face:

Happy to hear you enjoyed it! Thanks! 😄


ScalaMeter Framework does not exist.

If I run sbt project/test projectJs/test on empty project I get the following error:
org.scalajs.testcommon.RPCCore$RPCException: scala.scalajs.js.JavaScriptException: TypeError: Access to host class org.scalameter.ScalaMeterFramework is not allowed or does not exist.

My build.sbt looks like this:
lazy val all = (project in file("."))
  .settings(version := "0.1")
  .aggregate(
    projectjvm,
    project.js
  )

lazy val project = crossProject(JVMPlatform, JSPlatform)
  .withoutSuffixFor(JVMPlatform)
  .crossType(CrossType.Pure)
  .in(file("project"))
  .jvmSettings(
    libraryDependencies += "com.storm-enroute" %% "scalameter" % "0.17" % "bench",
    testFrameworks := List(
      new TestFramework("org.scalameter.ScalaMeterFramework")
    ),
    bench := (test in Benchmark).tag(Exclusive).value,
  )
  .jsSettings(
    scalaJSUseMainModuleInitializer := true,
    testFrameworks := Nil
  )
  .enablePlugins(ScalaJSPlugin)

In other words, although ScalaMeterFramework should be only JVM specific, it still causes an error in the JS version of project.

You need to remove the .enablePlugins(ScalaJSPlugin) line. crossProject does this automatically. Like this both projects are Scala.js projects.

Duplicate of #3673 / #3861.

Also @Josef-Vonasek please take a while longer before concluding something is a bug, and ask on StackOverflow or on Gitter before submitting GitHub issues. The GitHub Issues in this repo are for bug reports and feature requests only.
Thank you for your understanding.

It seemed like a bug at first... And I gave up on asking specific questions on gitter, because usually nobody answers (when the questions is about any non-widely used library).
I highly appreciate your timely response!


fix(default-values): fix default value set for boolean field with fal…

…se value even if the field is defined

Perfect 👌 !


Do not overwrite bindShapeMatrix

with the identity matrix when adding defaults.
@lasalvavida can you please make this same fix in gltf-pipeline: https://github.com/AnalyticalGraphicsInc/cesium/pull/4792

Looking at https://github.com/AnalyticalGraphicsInc/gltf-pipeline/blob/13b5b65d32d625f3293cf6b1acab357c85eed42b/lib/addDefaults.js#L418 , it seems that this bug did not exist here, as it was fixed (based on some unit test) in https://github.com/AnalyticalGraphicsInc/gltf-pipeline/commit/0fd42083b3fe29c489e8e59bd4111829c948203a

Ah yes, shame on me for not also fixing Cesium then.


[core-util] GA library and adopt in core-rest-pipeline

Per recent discussion, I believe the time has come to establish core-util as the common place for stateless helper functions and utility types in our repository.
This PR simply updates the core-util version so we can avoid pinning and migrates a few helpers out of core-rest-pipeline that were not previously exposed.
Feedback welcome!

API change check for @azure/mixed-reality-remote-rendering
API changes are not detected in this pull request for @azure/mixed-reality-remote-rendering

API change check for @azure-rest/agrifood-farming
API changes are not detected in this pull request for @azure-rest/agrifood-farming

API change check for @azure/service-bus
API changes are not detected in this pull request for @azure/service-bus

API change check for @azure/keyvault-admin
API changes are not detected in this pull request for @azure/keyvault-admin

API change check for @azure/communication-chat
API changes are not detected in this pull request for @azure/communication-chat

API change check for @azure/communication-identity
API changes are not detected in this pull request for @azure/communication-identity

API change check for @azure/identity
API changes are not detected in this pull request for @azure/identity

API change check for @azure-rest/core-client
API changes are not detected in this pull request for @azure-rest/core-client

API change check for @azure/core-rest-pipeline
API changes are not detected in this pull request for @azure/core-rest-pipeline

API change check for @azure/core-util
API changes have been detected in @azure/core-util. You can review API changes here
API changes
+ export declare function getErrorMessage(e: unknown): string;
+ export declare function getRandomIntegerInclusive(min: number, max: number): number;
+ export declare function isError(e: unknown): e is Error;
+ export declare function isObject(input: unknown): input is UnknownObject;
+ export declare type UnknownObject = {
+     [s: string]: unknown;
+ };

/check-enforcer override


how to change fltk Scheme?

In Linux, the default is base, and the interface looks very old.

Hey @ystyle,
you can call FLTK's fltk.SetStyle() function after [ui.Init()] in your code:
import (
	"github.com/pwiecz/go-fltk"
	"github.com/roblillack/spot"
	"github.com/roblillack/spot/ui"
)

func main() {
	ui.Init()
	fltk.SetScheme("plastic")

	spot.MountFn(…)

	ui.Run()
}

Hope that helps!

thanks


Map arrow keys to move path point

Feature request: For a selected path point, allow the arrow keys to translate the point up-down-left-right

I'd like to keep keyboard traversal enabled. So, instead of using the arrow keys to translate a selected waypoint, this will be implemented as using the arrow keys to increment the value in a text field. This also has the benefit of working for things besides position. Such as heading, rotations, etc.


Autogenerate _all.js file

Create script for Autogenerate _all.js files for some folders.
It's needed for auto registration of all components/containers/reactLess.... files in theyr own _all.js file.

Think about more flexible script

Done


build error  while  build the example code.

Hello.
I was compile your   example code. Then  get the below  error message..
Could you help  why below error is occurred?
During the  step  --> cmake ..
*** Error message****
kangdaehern@kangdaehern-Samsung-DeskTop-System:~/c++_project/tensorflow_samples/tensorflow_cc/example/build$ cmake ..
-- The C compiler identification is GNU 5.4.0
-- The CXX compiler identification is GNU 5.4.0
-- Check for working C compiler: /usr/bin/cc
-- Check for working C compiler: /usr/bin/cc -- works
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Detecting C compile features
-- Detecting C compile features - done
-- Check for working CXX compiler: /usr/bin/c++
-- Check for working CXX compiler: /usr/bin/c++ -- works
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Detecting CXX compile features
-- Detecting CXX compile features - done
CMake Warning at CMakeLists.txt:5 (find_package):
By not providing "FindTensorflowCC.cmake" in CMAKE_MODULE_PATH this project
has asked CMake to find a package configuration file provided by
"TensorflowCC", but CMake did not find one.
Could not find a package configuration file provided by "TensorflowCC" with
any of the following names:
TensorflowCCConfig.cmake
tensorflowcc-config.cmake

Add the installation prefix of "TensorflowCC" to CMAKE_PREFIX_PATH or set
"TensorflowCC_DIR" to a directory containing one of the above files.  If
"TensorflowCC" provides a separate development package or SDK, be sure it
has been installed.
CMake Error at CMakeLists.txt:10 (find_package):
By not providing "FindTensorflowCC.cmake" in CMAKE_MODULE_PATH this project
has asked CMake to find a package configuration file provided by
"TensorflowCC", but CMake did not find one.
Could not find a package configuration file provided by "TensorflowCC" with
any of the following names:
TensorflowCCConfig.cmake
tensorflowcc-config.cmake

Add the installation prefix of "TensorflowCC" to CMAKE_PREFIX_PATH or set
"TensorflowCC_DIR" to a directory containing one of the above files.  If
"TensorflowCC" provides a separate development package or SDK, be sure it
has been installed.
-- Configuring incomplete, errors occurred!
See also "/home/kangdaehern/c++_project/tensorflow_samples/tensorflow_cc/example/build/CMakeFiles/CMakeOutput.log".

Sorry.  I found the reason for above  error messages.
I forgot the   below  step  in your tutorial document.
After  doing below step, issue is fixed.
cd tensorflow_cc
mkdir build && cd build
for static library only:
cmake ..
for shared library only (requires Bazel):
cmake -DTENSORFLOW_STATIC=OFF -DTENSORFLOW_SHARED=ON ..
make && sudo make install

Great.

@FloopCZ @KangDaehern
I'm getting the same error while compiling for static library. I ran,
cd tensorflow_cc
mkdir build && cd build
cmake ..
Then inside example folder,
mkdir build && cd build
cmake ..
It didn't worked. so I tried,
cd tensorflow_cc
mkdir build && cd build
cmake ..
and then,
make
It compiled several things for sometime.
Again, I ran,
inside example folder,
mkdir build && cd build
cmake

Please closely follow the README.md. You need to run make && sudo make install also after the first cmake .. while still in the tensorflow_cc folder (see step 3 in the README).

Please closely follow the README.md. You need to run make && sudo make install also after the first cmake .. while still in the tensorflow_cc folder (see step 3 in the README).

Thank you very much. was confused by this line,
# for shared library only (requires Bazel):
By the way, could you shed light on how is this method to build different from the method in official repo?

The build process is very similar to the one described in the official repo. See build_tensorflow.sh.in. What is novel in this repo is the installation procedure - make install installs the required headers and libraries to the operating system and creates the corresponding CMake targets for easier use (and without Bazel).

The build process is very similar to the one described in the official repo. See build_tensorflow.sh.in. What is novel in this repo is the installation procedure - make install installs the required headers and libraries to the operating system and creates the corresponding CMake targets for easier use (and without Bazel).

Thank you.
By following the appropriate procedures in main repo, I could generate,
tensorflow/contrib/makefile/gen/lib/libtensorflow-core.a
How could I make use of this? any idea?
Would you look at this?

You can compile your code by manually providing all the necessary tensorflow headers to your compiler and linking against the static library. But this tensorflow_cc project is here exactly for this purpose - to make this process easier so that you don't need to search for all the necessary headers manually by yourself.

Great work! Kudos!
Should I build this tensorflow_cc project for every platform(Linux, Windows, Mac) again?
Actually, I need to deploy(only model inference) using C++ into production.
How can I use this repo for production scenario?

This project officially only supports Linux, so on Win and Mac, you are on your own. And there is no difference between development and production, so just install and build as usual.

Okay.
To know,
Where(Under which directory) should I place my .cpp, BUILD files in the main repo after building it?
Should I create a new folder for my project in main repository tree?

No, you can create your own project completely independently from the main repo or this repo. Try it by putting the example folder from this repo into any folder you want (e.g., /tmp/example) and running the build commands from there and it should work as expected.

As mentioned, I copied this repo's example folder into main repo's root.
Hope I've to run tensorflow/contrib/makefile/build_all_linux.sh as mentioned in main repo to build.
Cmake stuff!!
right?
I did ran.
This does not make use of the already generated tensorflow/contrib/makefile/gen/lib/libtensorflow-core.a
It seems to build all the way form scratch.
Am I doing right?

No, the main TF repo has nothing to do with this, if you want to use this project or one of the examples, just delete the main TF repo, you don't need it and you don't need to run any scripts from it. Just build the example using CMake.


Enable code analysis rules S138, S2931

Description
This will enable the following analysis rules without any code changes:
New rules enabled

S138
S2931

Closes

#922
#932

Quality checklist

[x] Your code complies with our Coding Style.
[x] There's an open issue for the PR that you are making. If you'd like to propose a new feature or change, please open an issue to discuss the change or find an existing issue.

Can you please update the PR description with "closes"/"fixes" so these two issues get auto-resolved on merge?

I meant to add this at the bottom of the description:
Fixes #922
Fixes #932

which would update:

See https://docs.github.com/en/issues/tracking-your-work-with-issues/linking-a-pull-request-to-an-issue.

Thanks for the link @bart-vmware changes made as requested.


[integration-tests]Enable messaging tests to integration tests.


ERROR: We can’t query your org. Make sure that you’re connected to this org and have permissions to view the object and fields.

ERROR: We can’t query your org. Make sure that you’re connected to this org and have permissions to view the object and fields.

I am also facing this same issue. Do we have any resolution for this?

I am also facing this same issue. Do we have any resolution for this?

@ayushiagrawal3110 , Update the Extensions/VS code & try to create new folder & authenticate
Thanks
Nagarjuna

ERROR: We can’t query your org. Make sure that you’re connected to this org and have permissions to view the object and fields.
I am getting same error

ERROR: We can’t query your org. Make sure that you’re connected to this org and have permissions to view the object and fields.
how to resolve this?


Add jaq support

Useful snippets and references:
jaq -L . -n 'include "jqjq"; _builtins_src | lex'
jaq -n -L . -rRs 'include "jqjq"; . | jqjq(["", "--run-tests"]; {})' < jqjq.test
jaq -n -L . 'include "jqjq"; eval("1+2")'

Disable TCO https://github.com/01mf02/jaq/pull/223#discussion_r1829094617

Current progress:
$ jaq -n -L . -rRs 'include "jqjq"; . | jqjq(["", "--run-tests"]; {})' < jqjq.test | grep -v OK:
ERROR: line 14: null | 123, 1.23, 0.123, .123, .123e1, 1.23e1, 2E3, 1.23e+1, 1.23E-2 -> 123,1.23,0.123,0.123,1.23,12.3,2000,12.3,0.0123
  cannot parse .123 as JSON: value expected
ERROR: line 40: "b" | "aaa \("a" + . + "c") ccc" -> "aaa abc ccc"
  cannot use null as iterable (array or object)
ERROR: line 52: null | "aaa \(1,2,3) bbb" -> "aaa 1 bbb","aaa 2 bbb","aaa 3 bbb"
  cannot use null as iterable (array or object)
ERROR: line 95: "abcäሴ💩" | @uri "aaa \(.) bbb \(.)" -> "aaa abc%C3%A4%E1%88%B4%F0%9F%92%A9 bbb abc%C3%A4%E1%88%B4%F0%9F%92%A9"
  cannot calculate 195.0 % 16
ERROR: line 131: {"a":1} | {a} -> {"a":1}
  cannot use null as iterable (array or object)
ERROR: line 135: {"a":1} | {"a"} -> {"a":1}
  cannot use null as iterable (array or object)
ERROR: line 139: null | 1 as $a | {$a} -> {"a":1}
  cannot use null as iterable (array or object)
ERROR: line 147: null | {a: 1, b: 2} | 3 as $c | {a, "b", $c, ("d"): 4} -> {"a":1,"b":2,"c":3,"d":4}
  cannot use null as iterable (array or object)
ERROR: line 489: {"a":1,"b":2} | del(.a) -> {"b":2}
  undefined function delpaths/1
ERROR: line 493: [1,2,3] | del(.[1]) -> [1,3]
  undefined function delpaths/1
ERROR: line 497: [1,2,3] | del(.[1],.[1]) -> [1,3]
  undefined function delpaths/1
ERROR: line 527: null | .a = 123 -> {"a":123}
  cannot use null as iterable (array or object)
ERROR: line 531: null | (.a,.b) = 123 -> {"a":123,"b":123}
  cannot use null as iterable (array or object)
ERROR: line 536: null | .a = 123 | .a |= . + 1 -> {"a":124}
  cannot use null as iterable (array or object)
ERROR: line 540: null | .a = 123 | (.a,.b) |= . + 3 -> {"a":126,"b":3}
  cannot use null as iterable (array or object)
ERROR: line 546: null | .a = 123 | .a += 3 -> {"a":126}
  cannot use null as iterable (array or object)
ERROR: line 550: null | .a = 123 | (.a,.b) += 3 -> {"a":126,"b":3}
  cannot use null as iterable (array or object)
ERROR: line 555: null | .a = 3 | .a += .a + 3 -> {"a":9}
  cannot use null as iterable (array or object)
ERROR: line 561: null | .a = 123 | .a -= 3 -> {"a":120}
  cannot use null as iterable (array or object)
ERROR: line 565: null | .a = 123 | .b = 0 | (.a,.b) -= 3 -> {"a":120,"b":-3}
  cannot use null as iterable (array or object)
ERROR: line 571: null | .a = 123 | .a *= 3 -> {"a":369}
  cannot use null as iterable (array or object)
ERROR: line 575: null | .a = 123 | .b = 1 | (.a,.b) *= 3 -> {"a":369,"b":3}
  cannot use null as iterable (array or object)
ERROR: line 581: null | .a = 120 | .a /= 3 -> {"a":40}
  cannot use null as iterable (array or object)
ERROR: line 585: null | .a = 120 | .b = 3 | (.a,.b) /= 3 -> {"a":40,"b":1}
  cannot use null as iterable (array or object)
ERROR: line 591: null | .a = 123 | .a %= 4 -> {"a":3}
  cannot use null as iterable (array or object)
ERROR: line 595: null | .a = 123 | .b = 4 | (.a,.b) %= 4 -> {"a":3,"b":0}
  cannot use null as iterable (array or object)
DIFF: line 697: null | try (1+"a") catch . -> "number (1) and string (\"a\") cannot be added"
  Expected: "number (1) and string (\"a\") cannot be added"
    Actual: "cannot calculate 1 + \"a\""
DIFF: line 707: null | error(123)? // 456 -> 456
  Expected: 456
    Actual:
ERROR: line 811: [[],[{"a":1}],[{"a":1},{"a":2},{"a":1}]] | map(group_by(.a)) -> [[],[[{"a":1}]],[[{"a":1},{"a":1}],[{"a":2}]]]
  undefined function setpath/2
ERROR: line 815: [{"a":1,"b":2},{"a":2},{"a":1,"b":2}] | group_by(.a, .b) -> [[{"a":1,"b":2},{"a":1,"b":2}],[{"a":2}]]
  undefined function setpath/2
ERROR: line 819: [[],[{"a":1}],[{"a":1},{"a":2},{"a":1}]] | map(group_by(.a)) -> [[],[[{"a":1}]],[[{"a":1},{"a":1}],[{"a":2}]]]
  undefined function setpath/2
ERROR: line 823: [{"a":1,"b":2},{"a":2},{"a":1,"b":2}] | unique_by(.a, .b) -> [{"a":1,"b":2},{"a":2}]
  undefined function setpath/2
ERROR: line 827: [[],[1],[3,2,2],[3,1,1,2]] | map(unique) -> [[],[1],[2,3],[1,2,3]]
  undefined function setpath/2
ERROR: line 863: [{"key":"a","value":1},{"Key":"b","Value":2},{"Name":"c","value":false}] | from_entries -> {"a":1,"b":2,"c":false}
  undefined function setpath/2
ERROR: line 867: {"a":1,"b":2} | with_entries(.key |= "_"+. | .value += 100) -> {"_a":101,"_b":102}
  undefined function setpath/2
ERROR: line 968: {"a":{"c":1},"b":{"c":2}} | .[].c = 123 -> {"a":{"c":123},"b":{"c":123}}
  undefined function setpath/2
ERROR: line 983: [[1,2],[3,4]] | .[][] += 1 -> [[2,3],[4,5]]
  undefined function setpath/2
DIFF: line 998: null | empty // empty // 123 -> 123
  Expected: 123
    Actual:
DIFF: line 1002: null | . // . // 123 -> 123
  Expected: 123
    Actual:
DIFF: line 1006: null | (1,null,2,3) // 123 -> 1,2,3
  Expected: 1,2,3
    Actual: 1
DIFF: line 1012: null | null // (1,null,2,3) -> 1,null,2,3
  Expected: 1,null,2,3
    Actual:
DIFF: line 1019: null | null // 1,2 // 123 -> 1,2
  Expected: 1,2
    Actual: 2
DIFF: line 1029: null | try (empty // 1,error("a"),2) catch . -> 1,"a"
  Expected: 1,"a"
    Actual: "a"
ERROR: line 1035: {"foo":0} | (.foo,.foo,.foo) |= .+1 -> {"foo":3}
  undefined function setpath/2
ERROR: line 1078: [1,2,3] | bsearch(0,2,4) -> -1,1,-4
  undefined function setpath/2
ERROR: line 1159: {} | [path(.a, .b.c)] -> [["a"],["b","c"]]
  cannot use null as iterable (array or object)
ERROR: line 1183: [[],[1],[3,2,2],[3,1,1,2]] | map(group) -> [[],[[1]],[[2,2],[3]],[[1,1],[2],[3]]]
  undefined function setpath/2
ERROR: line 1238: null | (.a | eval(".a, .b.c")) = 123 -> {"a":{"a":123,"b":{"c":123}}}
  cannot use null as iterable (array or object)
205 of 253 tests passed
null


[Deserializer] Error will not be thrown if top level "data" attribute is present but null.

Fix for issue: https://github.com/wvteijlingen/Spine/issues/77
Per JSONAPI specifications, one of data, errors, or meta MUST be present, but they can be null.  This change prevents an error from being returned if the value is present but null.

Looks good to me, thanks a lot!


Can't compile with error: 'Dependency not allowed'

Thanks for your work on this module.
I'm trying to compile this using make and this is the error I'm getting after adding the -vv flag:
make[1]: Entering directory '/Users/me/myapp/deps/erlang_v8/deps/jsx'
make[1]: Leaving directory '/Users/me/myapp/deps/erlang_v8/deps/jsx'
make -C /Users/me/myapp/deps/erlang_v8/c_src
make[1]: Entering directory '/Users/me/myapp/deps/erlang_v8/c_src'
################################################################################
/usr/bin/python -u tools/mb/mb.py gen -f infra/mb/mb_config.pyl -m developer_default -b x64.release out.gn/x64.release

  Writing """\
  is_debug = false
  target_cpu = "x64"
  """ to /Users/me/myapp/deps/erlang_v8/build/v8/out.gn/x64.release/args.gn.

  /Users/me/myapp/deps/erlang_v8/build/v8/buildtools/mac/gn gen out.gn/x64.release --check
    -> returned 1
  ERROR at //gni/v8.gni:116:3: Dependency not allowed.
    executable(target_name) {
    ^------------------------
  The item //:mkpeephole
  can not depend on //build/config/sanitizers:deps
  because it is not in //build/config/sanitizers:deps's visibility list: [
    //build/config:exe_and_shlib_deps
  ]

  GN gen failed: 1
Traceback (most recent call last):
  File "./tools/dev/v8gen.py", line 304, in <module>
    sys.exit(gen.main())
  File "./tools/dev/v8gen.py", line 298, in main
    return self._options.func()
  File "./tools/dev/v8gen.py", line 166, in cmd_gen
    gn_outdir,
  File "./tools/dev/v8gen.py", line 208, in _call_cmd
    stderr=subprocess.STDOUT,
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/subprocess.py", line 573, in check_output
    raise CalledProcessError(retcode, cmd, output=output)
subprocess.CalledProcessError: Command '['/usr/bin/python', '-u', 'tools/mb/mb.py', 'gen', '-f', 'infra/mb/mb_config.pyl', '-m', 'developer_default', '-b', 'x64.release', 'out.gn/x64.release']' returned non-zero exit status 1
make[1]: *** [Makefile:66: /Users/me/myapp/deps/erlang_v8/c_src/../priv/libv8_libbase.so] Error 1
make[1]: Leaving directory '/Users/me/myapp/deps/erlang_v8/c_src'
make: *** [erlang.mk:5841: app-c_src] Error 2

Any suggestions? Thanks again.

Hi!
Please give me some more detail about your system (hardware, os etc).
Thanks!

Gustaf

I'm not on that machine right now, but it's a Macbook Pro running OS X 10.11 El Capitan

If there's a specific command you want me to run to get more detailed information, please let me know. 👍


Fix same link in different file error

Currently, if there are same links but in different files (come under same file glob passed to mdox) like here and here,  mdox throws the error below,
remote link https://slack.cncf.io/: URL already visited

Even though the link is valid.
This PR fixes this and adds a test case for the same.

Linter failure seems to be due to goreportcard.com being down.

You should be able to retry @saswatamcode since you are maintainer now!

Hm, do you know why this error was created in the first place? (:

I think this error was created to improve performance and avoid duplicate requests in colly. But here it seems to cause this issue...is there a better way to handle this other than using AllowURLRevisit? (:
Also, goreportcard seems to still be down so check would fail if I rerun...


Create releases

Maybe after #12 is done it makes sense to generate releases for all of our supported platforms and put them here.
This makes me think about

[ ] Jenkins job to do nighty releases
[ ] Do we want to generate a CHANGELOG
[ ] automatically?
[ ] manually written?
[ ] based on closed issues since last release?

Please comment.

do we want actual nightly releases or just a "latest" build + tagged releases at end of sprint ?

I would say the latest build would do. As part of the 'continuous experience', we should be building and pushing to our test/stage whatever OpenShift instance as soon as master change. That job would create the 'latest build' that is later included in the Docker image and pushed to OpenShift. Essentially a  'current passed master' build.

As a proposal for naming versions I suggest to use the latest tag and an offset to it. I can tell from my previous job that the output of git describe works quite good for this. See this answer: http://programmers.stackexchange.com/a/141986.
This is what you get:
v2.5-0-deadbeef
^    ^ ^
|    | |
|    | SHA of HEAD
|    |
|    number of commits since last tag
|
last tag

The good thing is we get a numbered versioning without doing anything but tagging every now and then.

@kwk thats quite great and simple :) git describe ftw!

@maxandersen @aslakknutsen I think we can close this as out of date?

Closing this issue! Please @maxandersen and @aslakknutsen, feel free to re-open it.


gtest package version number is erroneous

Describe the bug
Per https://github.com/microsoft/vcpkg/issues/8452 gtest has been updated to 1.10.0.
For some reason the version number in the vcpkg metadata at https://github.com/microsoft/vcpkg/blob/master/ports/gtest/CONTROL contains the calendar date six days after the date on which version 1.10.0 was released, instead of the version number.
To be clear, 2019-10-09 is not a valid version number for gtest. This has caused problems for others, too, e.g. https://gitlab.kitware.com/cmake/cmake/issues/19837 .
gtest 1.10.0 release tag at https://github.com/google/googletest/releases/tag/release-1.10.0 .
To Reproduce
vcpkg search gtest
returns
gtest                2019-10-09     GoogleTest and GoogleMock testing frameworks.
Expected behavior
The version number "1.10.0" should be reported:
gtest                1.10.0     GoogleTest and GoogleMock testing frameworks.

Close this issue via #10963 merged.


hw03: Ocean Artwork

This is my completed artwork and copy of questions that I've answered upto section 4.

Code looks great! No fixes there. We do need to move some files around:

Rename copy_hw03_questions.md to sethutsa_hw03_questions.md. Move it into the Answers folder.
Upload your Google Doc into the PDFs folder.

Done with file organization and changes.


dcos-builder: bump Go to version 1.8.3

https://github.com/golang/go/issues/20040 / https://cve.mitre.org/cgi-bin/cvename.cgi?name=CVE-2017-8932 has raised a bit of discussion recently. Let's use the latest 1.8.x Go distribution to include the corresponding fix.
This is tracked via DCOS_OSS-1290.

The Marathon download was corrupt or the file was changed:
[12:48:24][Building package marathon variant <default>] Downloading source tarball https://downloads.mesosphere.io/marathon/snapshots/marathon-1.5.0-SNAPSHOT-582-g1793076.tgz
[12:48:31][Building package marathon variant <default>] Traceback (most recent call last):
[12:48:31][Building package marathon variant <default>]   File "/teamcity/work/19ad0dff41587ee6/build/env/lib/python3.5/site-packages/pkgpanda/build/__init__.py", line 1136, in _build
[12:48:31][Building package marathon variant <default>]     fetcher.checkout_to(root)
[12:48:31][Building package marathon variant <default>]   File "/teamcity/work/19ad0dff41587ee6/build/env/lib/python3.5/site-packages/pkgpanda/build/src_fetchers.py", line 317, in checkout_to
[12:48:31][Building package marathon variant <default>]     corrupt_filename, self.sha, file_sha, self.url))
[12:48:31][Building package marathon variant <default>] Failure building package(s): Validation error when fetching sources for package: Provided sha1 didn't match sha1 of downloaded file, corrupt download saved as /teamcity/work/19ad0dff41587ee6/packages/cache/packages/marathon/marathon/marathon-1.5.0-SNAPSHOT-582-g1793076.tgz.corrupt. Provided: 8bc00666df1825daf984ee945ec4fef1d72e3c5b, Download file's sha1: 5d8e93bb8197f4459a52ba900083123d85b7abdf, Url: https://downloads.mesosphere.io/marathon/snapshots/marathon-1.5.0-SNAPSHOT-582-g1793076.tgz
[12:48:31][Building package marathon variant <default>] pkgpanda.exceptions.ValidationError: Provided sha1 didn't match sha1 of downloaded file, corrupt download saved as /teamcity/work/19ad0dff41587ee6/packages/cache/packages/marathon/marathon/marathon-1.5.0-SNAPSHOT-582-g1793076.tgz.corrupt. Provided: 8bc00666df1825daf984ee945ec4fef1d72e3c5b, Download file's sha1: 5d8e93bb8197f4459a52ba900083123d85b7abdf, Url: https://downloads.mesosphere.io/marathon/snapshots/marathon-1.5.0-SNAPSHOT-582-g1793076.tgz
[12:48:31][Building package marathon variant <default>]

(https://teamcity.mesosphere.io/viewLog.html?buildId=686506)
Restarting.

@jgehrcke JIRA is back up now (although we may have a brief maintenance period before you wake up); please file a corresponding JIRA and link it.

Build failure is as of a broken master state: https://jira.mesosphere.com/browse/DCOS_OSS-1289

I have rebased on current master now that DCOS_OSS-1289 is taken care of.

@spahl and @gpaul please review! :)

@mesosphere-mergebot bump-ee

This is a good start but there are a couple of packages that specify specific golang versions:
Take for example https://github.com/dcos/dcos/blob/master/packages/mesos-dns/buildinfo.json#L2
[gustav@gpaul-dell dcos]$ grep -R '"docker"' packages/*/*info.json
packages/adminrouter/buildinfo.json:  "docker": "dcos/dcos-builder:adminrouter_dockerdir-latest",
packages/cni/buildinfo.json:    "docker": "golang:1.6",
packages/dcos-installer-ui/buildinfo.json:  "docker": "node:4.2.6",
packages/dcos-ui/buildinfo.json:  "docker": "node:4.4.4",
packages/docker-gc/buildinfo.json:  "group": "docker",
packages/dvdcli/buildinfo.json:  "docker": "golang:1.7",
packages/mesos-dns/buildinfo.json:  "docker": "golang:1.7",
packages/octarine/buildinfo.json:  "docker": "golang:1.7",
packages/rexray/buildinfo.json:  "docker": "golang:1.7",

This PR is good on its own, but perhaps a follow-up PR can remove the golang docker pins from the afore-mentioned packages? The same should be done for Enterprise.

The task of removing the custom golang docker keys is captured in https://jira.mesosphere.com/browse/DCOS_OSS-1322


[Bug]: The tests parent directory (".") is added to the PYTHONPATH when using py_pytest_main

What happened?
When using py_pytest_main the target's parent directory is added to the PYTHONPATH causing the same issues as in https://github.com/aspect-build/rules_py/issues/368.
Version
Development (host) and target OS/architectures:
Output of bazel --version: bazel 8.0.0
Version of the Aspect rules, or other relevant rules from your
WORKSPACE or MODULE.bazel file:
bazel_dep(name = "rules_python", version = "1.0.0")
bazel_dep(name = "aspect_rules_py", version = "1.0.0", dev_dependency = True)

python_version = "3.10"
python = use_extension("@rules_python//python/extensions:python.bzl", "python")
python.toolchain(
    configure_coverage_tool = True,
    is_default = True,
    python_version = python_version,
)
use_repo(python, "python_3_10", "python_versions", "pythons_hub")

Language(s) and/or frameworks involved: python
How to reproduce
py_pytest_main(
    name = "__test__",
    deps = ["@pip//pytest"],
)

py_test(
    name = "my_test",
    srcs = ["my_test.py", "//:__test__"],
    main = "__test__.py",
    deps = [
        "//:__test__",
        "@pip//pytest",
    ],
)

If my_test.py has an import for e.g. concurrent.futures and there is a concurrent.py in the same directory it will cause a clash.

### Any other information?

_No response_

This seems to be because pytest adds the test directory to sys.path through the mechanism described here: https://docs.pytest.org/en/latest/explanation/pythonpath.html#pythonpath
Setting --import-mode=append fixes this.

Cool, thanks for tracking down the culprit and a workaround. Do you think rules_py should include that flag for all users of py_pytest_main? Document its existence?

Cool, thanks for tracking down the culprit and a workaround. Do you think rules_py should include that flag for all users of py_pytest_main? Document its existence?

I think append is a less surprising default, but I haven't considered all of the use cases. I think at least documenting it would be nice so no one has to go on a wild goose chase like I had to.


✨ (charge): Robot helps to charge itself using IMU

Display something on screen, text or form, to help user to correctly place the robot. These instructions can be upload via BLE too.
Use the data from IMU, and more precisely pitch and roll to get correct current position of the robot. The robot base can be supposed on flat surface.

#1336
#1344
#1343
#1345


feat: improve block downloads

Closes #2875
This includes a few improvements:

downloads all missing parent blocks by walking backwards
try make chain canonical if the downloaded block connects to the canonical chain

This will ensure all commits are done by the tree.
However this introduces a new issue where the gap (sync target to local head) is too large to fit in the tree, for example after the loss of connection and node is offline for a while.
In which case we need to trigger the pipeline, perhaps the cutoff should be 1 epoch?

Codecov Report

Merging #2941 (3a3215a) into main (5dd49e4) will decrease coverage by 55.67%.
The diff coverage is 0.00%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@             Coverage Diff             @@
##             main    #2941       +/-   ##
===========================================
- Coverage   71.05%   15.38%   -55.67%
===========================================
  Files         517      517
  Lines       67056    67094       +38
===========================================
- Hits        47645    10322    -37323
- Misses      19411    56772    +37361

Flag
Coverage Δ

integration-tests
15.38% <0.00%> (-1.92%)
:arrow_down:

unit-tests
?

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

Impacted Files
Coverage Δ

crates/consensus/beacon/src/engine/forkchoice.rs
0.00% <0.00%> (-77.28%)
:arrow_down:

crates/consensus/beacon/src/engine/message.rs
0.00% <0.00%> (-34.43%)
:arrow_down:

crates/consensus/beacon/src/engine/mod.rs
2.85% <0.00%> (-82.53%)
:arrow_down:

crates/consensus/beacon/src/engine/sync.rs
0.00% <0.00%> (-75.74%)
:arrow_down:

crates/interfaces/src/blockchain_tree/mod.rs
0.00% <ø> (-50.00%)
:arrow_down:

... and 381 files with indirect coverage changes


How can get selected index ?

Hey guys, how can I get selected index for ZKCarousel? Thanks in advance.

Is it possible to receive a callback when the scroll has been executed (manual and automatic)?
Thanks!

@ZacharyKhan I want same feature that @Spettacolo83 comment.
When I slide it, get a index or other logic is executed.
Could you make it?

@ZacharyKhan I want same feature that @Spettacolo83 comment.
When I slide it, get a index or other logic is executed.
Could you make it?

@magichim I will take a look this evening and see if I can implement this for you.


TrimUI-"Salt and Sanctuary" initialization fails.

My machine is Trim Smart pro, running on Crossmix OS 1.21. When I try to run Salt and Sanctuary, the postmaster prompts a repack failure. Here's the record from repack.log. I'm using Salt and Sanctuary version v0.2.1.
"""
$$
'gfx/1x1.xnb' [3%] -> w: 1, h: 1
'gfx/alchemist.xnb' [3%] -> w: 1024, h: 684
'gfx/angler.xnb' [3%] -> w: 1024, h: 600
......
'gfx/zombow.xnb' [57%] -> w: 1024, h: 400
astcenc_config_init(): blk 4x4, quality: 10.000000
"""
I have tried many times, but it always fails at 57%. What do I need to modify?

That doesn't seem like a failure. Please delete the port, reinstall and post the entire log file.

the repack.log:
'''
$$
Mono Ahead of Time compiler - compiling assembly /mnt/SDCARD/Data/ports/saltandsanctuary/gamedata/Salt.exe
AOTID 5B092692-FD0B-0939-717C-DCFE39B7664F
Compiled: 3219/3219
Executing the native assembler: "as"   -o /tmp/mono_aot_c14P1V.o /tmp/mono_aot_c14P1V
Executing the native linker: "ld" -shared -o /mnt/SDCARD/Data/ports/saltandsanctuary/gamedata/Salt.exe.so.tmp  /tmp/mono_aot_c14P1V.o
JIT time: 5490 ms, Generation time: 946 ms, Assembly+Link time: 2570 ms.
'''

I found the reason。trimUI cannot recognize line 81 of SaltandSanctuary.sh。
“$TASKSET mono --ffast-math -O=all "${gameassembly}" |& tee "$gamedir/log.txt" /dev/tty0”
i delete " |& tee "$gamedir/log.txt" /dev/tty0",now  can enter successfully。
Thank you very much！


Question: Why keep the password and password confirmation on the Authorization model?

I was just going through the code and had a question about this line:
https://github.com/hassox/phoenix_guardian/blob/2a502ac49d696aac73857a092c92345328e899de/web/auth/user_from_auth.ex#L128
Could you explain why it's beneficial to set the password and password_confirmation on the Authorization model at all?  AFAICT from the code, it seems like the generated token would be enough for authorization purposes, and this just leaves the password hanging around in memory, which could potentially make it vulnerable to buffer overflow attacks.
You've clearly added it here for a specific purpose though so I'm assuming I'm missing something - I just can't seem to find a reference to these fields being used on an authorization struct anywhere.
Thanks!

Or, second thought: is it added to allow the password / password_confirmation form fields to be re-populated on signup / signin forms in case of an error?

I think this is just old code that I missed. It used to be that the authorization model converted the PW to the encrypted version but I moved it over.

i'm wondering what you guys think about storing the encrypted password in the token field.  In my past projects the password was on user table and the authorizations table was strictly oauth.


Upgrade to quarkus 3.8.0 in QuickStarts

Referenced pull requests
Related to https://github.com/apache/incubator-kie-optaplanner/pull/3070
See https://github.com/apache/incubator-kie-issues/issues/958

How to retest this PR or trigger a specific build:

for pull request checks
Please add comment: Jenkins retest this

for a specific pull request check
please add comment: Jenkins (re)run [optaplanner-quickstarts] tests

for a full downstream build
please add the label run_fdb

for quarkus branch checks
Run checks against Quarkus current used branch
Please add comment: Jenkins run quarkus-branch

for a quarkus branch specific check
Run checks against Quarkus current used branch
Please add comment: Jenkins (re)run [optaplanner-quickstarts] quarkus-branch

for quarkus main checks
Run checks against Quarkus main branch
Please add comment: Jenkins run quarkus-main

for a specific quarkus main check
Run checks against Quarkus main branch
Please add comment: Jenkins (re)run [optaplanner-quickstarts] quarkus-branch

for quarkus lts checks
Run checks against Quarkus lts branch
Please add comment: Jenkins run quarkus-lts

for a specific quarkus lts check
Run checks against Quarkus lts branch
Please add comment: Jenkins (re)run [optaplanner-quickstarts] quarkus-lts

for native checks
Run native checks
Please add comment: Jenkins run native

for a specific native check
Run native checks
Please add comment: Jenkins (re)run [optaplanner-quickstarts] native

for mandrel checks
Run native checks against Mandrel image
Please add comment: Jenkins run mandrel

for a specific mandrel check
Run native checks against Mandrel image
Please add comment: Jenkins (re)run [optaplanner-quickstarts] mandrel

for mandrel lts checks
Run native checks against Mandrel image and quarkus lts branch
Please add comment: Jenkins run mandrel-lts

for a specific mandrel lts check
Run native checks against Mandrel image and quarkus lts branch
Please add comment: Jenkins (re)run [optaplanner-quickstarts] mandrel-lts

How to backport a pull request to a different branch?

In order to automatically create a backporting pull request please add one or more labels having the following format backport-<branch-name>, where <branch-name> is the name of the branch where the pull request must be backported to (e.g., backport-7.67.x to backport the original PR to the 7.67.x branch).

NOTE: backporting is an action aiming to move a change (usually a commit) from a branch (usually the main one) to another one, which is generally referring to a still maintained release branch. Keeping it simple: it is about to move a specific change or a set of them from one branch to another.

Once the original pull request is successfully merged, the automated action will create one backporting pull request per each label (with the previous format) that has been added.
If something goes wrong, the author will be notified and at this point a manual backporting is needed.

NOTE: this automated backporting is triggered whenever a pull request on main branch is labeled or closed, but both conditions must be satisfied to get the new PR created.

i guess this is somehow forgotten and outdated as there is already used 3.8.4 version in https://github.com/apache/incubator-kie-kogito-runtimes/blob/e53d706385c627339c8661526e46f5440e59512d/kogito-build/kogito-dependencies-bom/pom.xml#L38  and https://github.com/apache/incubator-kie-optaplanner/blob/main/build/optaplanner-build-parent/pom.xml#L49?


[Offline] if order details are set don't set it again

This is read only repo, I cannot merge it here


Explicitly state Map types in RunMutation

The new RunMutation type declaration inferencing will default to <dynamic, dynamic> without explicitly declaring the expectation. Because of this, the Mutation widgets fail to build with:
type '(Map<String, dynamic>) => void' is not a subtype of type '(Map<dynamic, dynamic>) => void'
The stacktrace shows the error at
MutationState.build (package:graphql_flutter/src/mutation.dart:63:7)
I believe the input should always be a String indexed map for the variables, so changing the type on the typedef from Map to Map<String, dynamic> should be fine.

Thanks for the bug report.
I have already fixed it in the last update that comes with offline caching bashed on hashmap that I am going  to push in a few hours.


increase pytorch/CUDA space from 8GB to 16GB

Old Ubuntu 18.04.2023031501 version has issue with 8GB disk space installation. It has been solved in 18.04.2023063001. To be safe, increase to 16GB.

@LiliDeng LGTM

Old Ubuntu 18.04.2023031501 version has issue with 8GB disk space installation. It has been solved in 18.04.2023063001. To be safe, increase to 16GB.

After debug, the root cause is pytorch and CUDA requires more than 9GB.
Old 18.04.2023031501 version image used less space in /home folder, which made /home folder in sda1 has 9GB (> 8GB) free space. This would make pytorch and cuda installed on the /home folder but failed due to out of space.
The new 18.04.2023063001 version image used more space and left only 5.6GB (< 8GB) in /home folder, which made pytorch and cuda installed on the /mnt folder on sdb1 and passed.


Login with (cryptographic) passwords

Describe Your Changes

adjusts login to use passwords (see #130)

Non-Obvious Technical Information
Checklist Before Requesting a Review

[x] The code runs successfully.

2024-02-21 23:19:54 INFO:     <IP_ADDRESS>:43500 - "GET /admin/login HTTP/1.1" 200 OK
...
2024-02-21 23:20:06 DEBUG:      detected 'bcrypt' backend, version '4.0.1'
2024-02-21 23:20:06 DEBUG:      'bcrypt' backend lacks $2$ support, enabling workaround
2024-02-21 23:20:06 INFO:     <IP_ADDRESS>:44072 - "POST /admin/login HTTP/1.1" 302 Found
2024-02-21 23:20:06 INFO:     <IP_ADDRESS>:44072 - "GET /admin/dashboard HTTP/1.1" 200 OK

closes #130


Support float16 images in StableDiffusionXLWatermarker

What does this PR do?
StableDiffusionXLPipelineMixin https://github.com/huggingface/optimum/blob/7376c6ad7f0a852cb81dfbb9e9539eb6ed14fd36/optimum/pipelines/diffusers/pipeline_stable_diffusion_xl.py#L490 and StableDiffusionXLImg2ImgPipelineMixin https://github.com/huggingface/optimum/blob/7376c6ad7f0a852cb81dfbb9e9539eb6ed14fd36/optimum/pipelines/diffusers/pipeline_stable_diffusion_xl_img2img.py#L497
apply watermark to the output of vae_decoder.
However, the default watermarker StableDiffusionXLWatermarker does not support float16 images because cv2.cvtColor doesn't support it.
Running the following code
from optimum.pipelines.diffusers.watermark import StableDiffusionXLWatermarker
import numpy as np

watermarker = StableDiffusionXLWatermarker()
fp16_image = np.random.rand(1, 3, 1024, 1024).astype(np.float16)
watermarked_image = watermarker.apply_watermark(fp16_image)

gives the following error
error: OpenCV(4.8.1) ...
> Unsupported depth of input image:
>     'VDepth::contains(depth)'
> where
>     'depth' is 7 (CV_16F)

In this PR, we check whether images in StableDiffusionXLWatermarker.apply_watermark is float16 and cast it to float32 if so.
This enables the use of fp16 vae decoder in a pipeline loaded using from_pretrained (there is no option to disable the default watermark).
Before submitting

[ ] This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
[ ] Did you make sure to update the documentation with your changes?
[ ] Did you write any new necessary tests?

@echarlaix FYI


Fix runTask() / release build

Let's see if CI likes the first commit.

This times out on ValidatorCleanRestart.

Look at the integration test logs. First commit doesn't seem to do good there.

Ah so it was there because of vibe.d.. then I'll have to nuke the commit.

Rebased.

Rebased.

CI disagrees


Fix local variable

https://github.com/MrPaschenko/op-coursework/blob/ee1ca12bcd548f69d549f9b18d07abb1121c6310/index.js#L31
New Wallet instance is created locally, but we need to use it globally, not to call once in the function.

Fixed in b9f1d85212554f13810f10449234eb7024c4b3a0


Enduro

fixes the recursive step in pyx

Thank you for discovering this. It has saved me lots of time.


Typo in the guide of creating azure storage account

There's a typo at the Create a storage account section, at the Portal tab:

On the Storage accounts page, select New.

should be

On the Storage accounts page, select Create.

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

ID: 6ee9ec76-a91f-095a-5ba3-b9c413366751
Version Independent ID: 29c17263-ed56-d42e-bc3f-091ef2b4b9ff
Content: Create a storage account - Azure Storage
Content Source: articles/storage/common/storage-account-create.md
Service: storage
Sub-service: common
GitHub Login: @tamram
Microsoft Alias: tamram

@yah01 Thanks for the question! We are investigating and will update you shortly.

@SumanthMarigowda
I have created a pr to fix this typo:
https://github.com/MicrosoftDocs/azure-docs/pull/77668

@yah01  PR has been initiated necessary changes will take very soon! We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


Adding an example

As per the README.md, I'm opening an issue to see if there is interest in including https://github.com/ewohltman/ephemeral-roles in the list of curated discordgo examples.
Thanks!

Yeah that sounds like a good idea :) I'll add it to the list.


How to execute Java calls to GraphQL in a Spring Boot + GraphQL Java Tools' context?

Just posted a question on StackOverflow about the ability to call GraphQL from Java in a Spring Boot + GraphQL Java Tools' context.
All details are explained: https://stackoverflow.com/questions/58188938/how-to-execute-java-calls-to-graphql-in-a-spring-boot-graphql-java-tools-cont
Thanks if someone has an answer.

I finally found how to do this, browsing through the code of different classes in the graphql-spring-boot-autoconfigure but I wonder if this could be documented...?
https://stackoverflow.com/a/58299150/666414

Actually the better cleaner way to implement this would be as follows. This rest api will automatically route all requests to the graphql endpoint and the response will be received by your client without any intervention as is.
What you are doing in that StackOverflow answer is not needed.
/**
 * This controller can be enabled by setting property com.yourcompany.mock.enabled.
 */
@RestController
@RequestMapping("${com.yourcompany.mock.endpoint}")
@ConditionalOnProperty(value = "com.yourcompany.mock.endpoint.enabled", havingValue = "true", matchIfMissing = true)
public class GraphQLEndpointController {

    // if property is not set the default mapping will be /graphql
    @Value("${graphql.servlet.mapping:/graphql}")
    private String graphqlServletMapping;

    @RequestMapping(method = RequestMethod.POST)
    public void processRequest(HttpServletRequest httpServletRequest, HttpServletResponse httpServletResponse) throws ServletException, IOException {
        httpServletRequest.getRequestDispatcher(graphqlServletMapping).forward(httpServletRequest, httpServletResponse);
    }
}

Edit: Nvm, it seems like you just want to create Rest API specific request/response and not use graphql api specific/request response. Then that's the only way to go. I was under the assumption than you just require rest endpoints, but will still still send graphql requests and expect graphql response.

You got it! ;)


Add new domain name

Hi,
First, thanks for your project, it saved me time!
It seems<EMAIL_ADDRESS>is a temporary email domain from https://temp-mail.org/ that does not exist in your settings, could you add it ?
Complete list is on the https://temp-mail.org/en/option/change page
Thanks,

Could you add @mvrht.com from https://10minutemail.com :)

Also mvrht.net


Not updating to 9.15.1 on iOS

cordova plugin rm heyzap-cordova
cordova plugin add heyzap-cordova

The HeyzapAds.h in the KeyzapAds.framework/Headers shows is still: #define SDK_VERSION @"9.14.1"

Updated this morning on npm

I've been referring to the updated version. Tested today.

https://github.com/Heyzap/heyzap-cordova/blob/master/src/ios/sdk/Heyzap/HeyzapAds.framework/Versions/A/Headers/HeyzapAds.h#L64

Sorry, it must be the time difference between our locations.


Adds FT300 Support

I believe, this PR adds FT300 support to the jade-devel branch. We may have to go through and see what's  happening in the .dae file as I'm quite unsure. It's old code for me - perhaps I could get some help if I missed anything.
Thanks dudes!

I believe the FT300 is already supported (although I don't have the hardware).  It looks like this PR just adds a ftdi_id to the method calls.

Hi @TheClearpathDash or @TheDash, I was just wondering if you have tested tihs on the FT300 hardware as I'd like to use this.

Closing this since it is outdated and has been superseded by other PRs.

Since I wasn't able to find the URDF models, I add them on pr #138 (Check last comments)


[improve] support z-order range sort action

This pull request introduce a way to sort append-only table with z-order(mainly).
This topo will be like:

We use rangePartitioner to make global sort by the z-order of the given columns. Now assume that all the columns are
8 bytes, which mean, if you index a string by z-order, the useful bytes are only the front eight.

We use 512 M for sort memory on heap.

A usage example is :

flink run ./paimon-flink-action-0.5-SNAPSHOT.jar order-rewrite
      --warehouse /Users/yejunhao/paimontest/GlobalBatchReadWrite
      --database my_db
      --table Orders1
      --select "SELECT * FROM my_db.Orders1 WHERE f0 = 0"
      --order-by "zorder(f1,f2)"

CLASS FUNCTIONALITY
ZorderSorter is used to sort table of input, this meanly call ZorderSorterUtils.sortStreamByZorder to accomplish its job.
RangeShuffleUtil is used to make the input stream range shuffle by the key of given
ZorderSorterUtils do the main job:

add a z-index column to internal row
shuffle by z-index by calling RangeShuffleUtil
do the sort of every reduce task
remove z-index
back to origin row schema

NOTE
These class are copied from iceberg:
paimon-core/src/main/java/org/apache/paimon/utils/ByteBuffers.java
paimon-core/src/main/java/org/apache/paimon/utils/ZOrderByteUtils.java
paimon-core/src/test/java/org/apache/paimon/utils/TestZOrderByteUtil.java
TEST
OrderRewriteActionITCase tests for the action and check the result.
ZIndexerTest tests for z-index generation

Sort Data Test:
data num: 10000000
columns num: 16
sort time: 27min
scan and sink parallelism: 5
bucket: -1
target-file-size: 5 M
partition num: 1
other default


Code contributing

Hi @Ruin0x11
Is there any particular way to help with your project? I worked before with open source projects, e.g. I helped to port cherrytree from python to C++ last year.
So far, I read the wiki part (it's really nicely written) and did some tinkering. I'd love to do something but it is unclear from where to take the original code (from elonafoobar or decompiled elona sources)?

Hello. First of all, thank you for offering to help out.
OpenNefia's code is primarily sourced from the public domain source code of Elona 1.22 released by Noa in 2010. If you send him an email, I believe he will be happy to send you a copy. Elona_foobar was another project that took the approach of directly transpiling that source code to C++ (it was later archived in favor of OpenNefia). In contrast, I decided to do a direct port with the hopes of ending up with an architecture better suited for modding.
Right now I'm still trying to figure out what combination of extensibility and stability would work best for this kind of project. That includes how to involve other people, which isn't one of my strongest points. One of the issues that I have found in the past few years is that, due to the fact that I've mostly been working on OpenNefia alone for a significant period of time, when someone else wants to help contribute to the project it becomes difficult for me to put into words exactly how they might be able to contribute. I would very much like OpenNefia to be a project that people can help out with, but at the same time I don't want to give well-meaning people like yourself things to do when all that work will ultimately be in vain if I happen to change my mind later.
Also, unrelated to that specific problem, as I've continued developing OpenNefia I've run into some issues related to stability that have only cropped up now that the project has become significantly larger than I once imagined it would be. Those issues stem primarily from the choice of language, which wasn't something I put much thought into in the beginning. (OpenNefia is essentially a weekend project that overstayed its weekend.) This is an issue whose resolution I'm deciding entirely on my own foresight - since nobody uses OpenNefia yet, there are no deep attachments to the architecture or worrying technical debt that needs to be worked around. But when I try to play the game normally and run into game-breaking issues within an hour, some involving save data, I start to believe that a different approach is needed. OpenNefia hasn't actually seen an official release yet, partially stemming from these issues. I don't know when I'd feel comfortable releasing it seeing as it will need to reach stability at some point, and at the moment I don't quite understand what the roadmap for that will be.
But all that doesn't mean I will stop working on the project for good - far from it, actually. I'm only trying to figure out what would work best if OpenNefia is going to be taken seriously by people that just want to play Elona, not put up with the programmer-centric things that are most tantalizing to my psyche. To that end, I'm trying to see if OpenNefia's API would be better off implemented in a language like C#, for the sake of future-proofing all the hard work people will want to put in. (The repository for that effort is currently here). Additionally, this effort would allow me to try a different strategy that favors releasing the engine early and often even if it's vastly incomplete, in contrast to working on the code entirely by myself for years and releasing it all at once, hoping that things work out.
Apologies for the tangent. In response to your specific question, even if there isn't anything I can think of that would need more programming help, one thing that you can always do is attempt to write mods and see what breaks/needs design work. What I don't have a good idea of is what exactly people want to do with OpenNefia, besides use it as a modern update of Elona. If I had a better idea of that, then I would be able to design the engine around those issues and support more use cases. Unfortunately, since my time is fairly divided right now, I'm mostly trying to see how much I can improve OpenNefia by working on its C# implementation, so this repository in specific with the Lua API probably won't see many updates in the near future.
But thanks for your interest, and I hope I don't come off as uninterested in your help. I do hope to have OpenNefia in some kind of state in the future where contributing is way easier than it is right now, whether it be in Lua or C#. To me, OpenNefia is mostly a huge programming puzzle that I enjoy tinkering with. I work on it out of my own interest, and as of late I've decided that I need to balance that with the need to include other people who are interested in helping. So thank you for reaching out.

Hi,
For a weekend project, it is a hell of a lot of work. I can be only envious of your tenacity.
To be honest, I am more interested in how the game works inside instead of the game itself. Just being practical, I wanted to make something for the project as well while learning a couple of things from it. But I totally understand your points, so have no hard feelings.
I wish you all the best.


Consider supporting more KCL config options

For example, I need to use a custom RetrievalConfig in certain cases, but that can't be done with the mainline ConsumerConfig API.

@ewestern
That is true, the ConsumerConfig only supports common configuration options at the moment. The issue with enumerating all the config options in ConsumerConfig is it becomes more difficult to maintain as AWS makes changes. Do you have any suggested way of supporting different config options via the ConsumerConfig? Some sort of API pattern you would find useful?

Sorry for the delay, @ukayani
I guess one option would be just to leave your simplified ConsumerConfig as-is, but then provide a separate factory method that creates a Source and takes only a KinesisClientLibConfiguration as argument. That way, you wouldn't have to worry about changes on the aws side (at least within a given major version).

+1 on supporting an override of the RetreivalConfig
I took a shot at adding this in the pull request below.
https://github.com/500px/kinesis-stream/pull/6

0.1.7 released on maven central containing the code changes.


Getting type issues in typescript

Tried the code from docs and i was getting type error in typescript.
import { SolidPlyr } from 'solid-plyr';

const SOURCE = {
  type: 'video',
  sources: [
    {
      src: 'https://cdn.plyr.io/static/demo/View_From_A_Blue_Moon_Trailer-720p.mp4',
      type: 'video/mp4',
      size: 720,
    },
    {
      src: 'https://cdn.plyr.io/static/demo/View_From_A_Blue_Moon_Trailer-1080p.mp4',
      type: 'video/mp4',
      size: 1080,
    }
  ]
}

const OPTIONS = {
  autoplay: true,
  muted: true,
}

export default function Player() {
  return (
    <SolidPlyr source={SOURCE} options={OPTIONS} />
  );
}

Types of property 'type' are incompatible.
    Type 'string' is not assignable to type 'MediaType'.
    ```

You could try using the type SourceInfo from Plyr for the SOURCE variable.
import { SourceInfo } from 'plyr';

const SOURCE: SourceInfo = {...}

For additional context, the type MediaType is type MediaType = 'audio' | 'video';.

that worked!


The other player can't see the bullet

why The other player can't see the bullet self can

Hi! I'm also running into this issue. I'd like to try to fix it. Any ideas what may be causing this? @halftheopposite  @LukeWood

Really not sure honestly.  maybe a colyseus API changed with a major version?

Ah I found it. Issue with the bullet re-use on server side. I’ll PR a fix when I can :)

On Mar 9, 2022, at 1:26 AM, Luke Wood @.***> wrote:

Really not sure honestly. maybe a colyseus API changed with a major version?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.
You are receiving this because you commented.

@tabakd That'd be great if you can fix it and submit a PR 👍

@tabakd Could you post a quick fix here in the meantime? I'm able to get it working if I comment out the bullet recycling code on the server-side so it only pushes new bullets, and also if I just push the recycled bullet onto the existing bullets array. But if I delete the bullet from the array and re-push, no go. My guess is Colyseus isn't detecting the array change because it thinks it is the same item, and could probably fix it by adding a new ID like this monsters array does for new entries

So ?

@tabakd Could you post a quick fix here in the meantime? I'm able to get it working if I comment out the bullet recycling code on the server-side so it only pushes new bullets, and also if I just push the recycled bullet onto the existing bullets array. But if I delete the bullet from the array and re-push, no go. My guess is Colyseus isn't detecting the array change because it thinks it is the same item, and could probably fix it by adding a new ID like this monsters array does for new entries

That sounds like a plausible fix. I simply disabled the bullet re-use logic for my purposes but can look into fixing it.

@tabakd I think this is roughly what you had in mind.

It is actually happening since Bullet.reset method in server can trigger neither onAdd, onRemove nor onChange event from Colyseus server. I don't know why Bullet instance has to be remain inactive without destroying them. But #62 fixes it without too much hassle.


mgmt: mcumgr: Fix fs_mgmt log output on file write

If a file write was attempted with a file that does not exist, over the shell/UART when logging was enabled, it would output a fs_unlink error, this fixes the issue by checking if the file exists and needs truncating before performing that action. It also imrproves flash endurance slightly by performing a truncate operation instead of a delete, but will fall back to a delete if the truncation operation fails.
Fixes #36882
Note that this only fixes the specific issue mentioned, there might be other potential errors emitted if filesystem errors are encountered, however the mcumgr client is actually to blame for this as it should be searching through the received data for the mcumgr start header, 0x06 0x09, which it does not do.

No it is not. The problem is that the client is in the middle of receiving frame that has been marked with start of header tag, but got additional characters inside. Because this is UART and CBOR a client basically sees stream of characters, following the start of header tag, and tries to process it as it is; unfortunately it gets something that is not part of packet in the middle so it fails.

From their log in the linked bug:
time="2021-07-27 10:56:02.502" level=debug msg="Rx serial:
00000000  1b 5b 31 3b 33 32 6d 75  61 72 74 3a 7e 24 20 1b  |.[1;32muart:~$ .|
00000010  5b 6d 1b 5b 38 44 1b 5b  4a 1b 5b 31 3b 33 32 6d  |[m.[8D.[J.[1;32m|
00000020  75 61 72 74 3a 7e 24 20  1b 5b 6d 06 09 41 42 55  |uart:~$ .[m..ABU|
00000030  44 41 41 41 4e 41 41 68  54 41 4c 39 69 63 6d 4d  |DAAANAAhTAL9icmM|
00000040  41 59 32 39 6d 5a 68 6b  48 67 76 39 6d 36 41 3d  |AY29mZhkHgv9m6A=|
00000050  3d                                                |=|

The 0x06 0x09 start header can be seen, and the full packet is there, I'm not sure where there isn't a 0x0a end character but other than that, the packet is not split up. The issue where UART data does interleave mcumgr commands from what I've seen is due to usage of printk - noted in https://github.com/zephyrproject-rtos/zephyr/issues/38947 the solution to that is to process printk messages in the logging system - PR at https://github.com/zephyrproject-rtos/zephyr/pull/50594 however this has had the side effect of causing many tests to fail to output any data, but in my testing, once that is enabled, shell/log/printk output does not interleave mcumgr packets

So they did have specific case, but if you look at uart_mcumgr_send_raw you will notice that there are no exclusive locks on UART  and bytes are send one by one, additionally most of transitions on UART, by mcumgr, are done in chunks up to 4 bytes, by mcumgr_serial_tx_small, so there is plenty of time to jump in and make some mess.

Don't get me wrong, this still fixes 99% of cases, probably, and I would accept it I am just asking whether we should mark it as workaround, with explanation, before closing the issue.
Disabling logging also does workaround for the problem, except in case when something else starts to write to uart.

I remember having a look at trying to get locking working, but it was a free for all, shell has a mutex lock but logging doesn't, would probably make sense to implement a lock, although I think with some configurations it would end in a deadlock e.g. minimal footprint logging and mcumgr output, if a filesystem error is output during a UART mcumgr command runs. Alternatively, and this might not be a great idea, the smp thread priority could be changed to be negative/co-operative

@nordic-krch could you weigh in on this, what would be the best path forward if there's multiple systems using the UART that need to lock it for periods of time so that they can output their data without interruption from other threads, and ideally where SMP operation would not cause any issues?

I remember having a look at trying to get locking working, but it was a free for all, shell has a mutex lock but logging doesn't, would probably make sense to implement a lock, although I think with some configurations it would end in a deadlock e.g. minimal footprint logging and mcumgr output, if a filesystem error is output during a UART mcumgr command runs. Alternatively, and this might not be a great idea, the smp thread priority could be changed to be negative/co-operative. Or perhaps use a UART output function that outputs the full buffer instead of a few bytes at a time, though this would have a RAM impact.

For now this is good as is, we just need to notify in commit that this is workaround, and there is alternative like disabling logging in FS.

@de-nordic please rebase

@carlescufi Please review.


Unittest not found

I am following instructions in the README.md file, which states:
## running your tests locally
you can run all your tests in the assignment folder with `unittest`

cd assignment0
python -m unittest
You provide very detailed instructions, but the commands above don't work because unittest is not installed. In fact, it is not a part of the requirements file. Perhaps I am missing some instructions elsewhere? Of course, I know I can install `unittest`, but the idea is for me to follow your instructions since you want the system to work within Gradescope. I hope you are still listening. Thanks.

    Gordon

I solved my problem by running requirements.txt (not sure I understand why unittest became available. )

unittest is a python library that needs to be installed before it can be used. Since it was one of the libraries specified in requirements.txt, you installed it when you ran pip install -r requirements.txt.

Got it. This now works.


Add Ctrl+Backspace support

Summary of the Pull Request
Changes the Ctrl+Backspace input sequence and how it is processed by InputStateMachineEngine. Now Ctrl+Backspace deletes a whole word at a time (tested on WSL, CMD, and PS).

References

PR Checklist

[x] Closes #755
[x] CLA signed. If not, go over here and sign the CLA
[ ] Tests added/passed -> made minor edits to tests
[ ] Requires documentation to be updated
[x] I've discussed this with core contributors already. If not checked, I'm ready to accept this work might be rejected in favor of a different grand plan. Issue number where discussion took place: #755

Detailed Description of the Pull Request / Additional comments
Changed the input sequence for Ctrl+Backspace to \x1b\x8 so the sequence would pass through _DoControlCharacter. Changed _DoControlCharacter to process \b in a way which forms the correct INPUT_RECORDs to delete whole words.

Validation Steps Performed
Ctrl+Backspace works 🎉

Thanks for the review @zadjii-msft. That's a lot of good info. I'll work on making InputStateMachine process Ctrl+has Ctrl+Bksp.

With some new changes making InputStateMachine process ^H as Ctrl+Bksp, CMD and PS will delete whole words. In WSL, ^H does not follow the same path through  InputStateMachine as in CMD or PS (apparently). Once I figure out how to send WSL the correct INPUT_RECORD, we should be in business.

@zadjii-msft WSL standalone doesn't delete whole words when Ctrl+Bksp is pressed, which explains why I was having my problems above. After testing the behavior of ^H in showkey -a and conechokey, I believe this solves the problem without messing with the VT sequences like my last version. Thanks for pointing out those command line tools btw, very helpful.

@msftbot merge this in 168 hours

@msftbot forget everything I just told you

Glad to have this one merged! Thanks

I'm on 0.10.781.0 but ctrl-backspace simply seems to send ctrl-h in WSL2 :(

@wmertens You'll want to read #4397

:tada: Once again, thanks for the contribution!
This pull request was included in a set of conhost changes that was just
released with Windows Insider Build 19603.


update GUIDE.md according to new constructor of IPFS object

I updated the examples in the GUIDE.md.
I modified the constructor of the IPFS object. I checked all the examples, they
seem to be working.
This comes from this issue https://github.com/orbitdb/orbit-db/issues/846,
where I was told that the documentation needed to be updated.
I hope it helps
Edit from @aphelionz: fixes #847 and fixes #846

Re-running CI now

OK, great thanks!

The CI fixes aren't related so we can merge. Thanks again!

OK, great! Thank you


Determine the default precision and quantization in chat and generate

If you finetune a model with a certain quantization and precision setting, you still need to specify that in the chat and generate commands today:
litgpt chat \
    --checkpoint_dir out/qlora-codellama-13b/final \
    --precision bf16-true \
    --quantize bnb.nf4-dq

Otherwise you may get an OOM or different results that you were getting during training. Since we store the hyperparameters in a yaml file, we could select the two settings automatically if they are not specified:
# uses precision=bf16-true and quantize=bnb.nf4-dq from checkpoint folder
litgpt chat --checkpoint_dir out/qlora-codellama-13b/final

We already do this in other parts of LitGPT, so we could just reuse the utility function to read the two settings from the checkpoint.

I don't see how we can tie this decision. The training and inference dtypes can be entirely different.
If it trains on 16-mixed, what would you say that it needs to use during inference?
And if it trains on 16-true, inference already picks this by default.
For quantization it makes sense. Although I'm not sure that we should enable it silently


Increase decimal size to 5

Increase decimal size to 5 places

I have found that the default 2 decimal places is not accurate enough for latitude and longitude values


Bad Async performance under heavy load when appender is configured to not drop events

The Async appenders show very bad performance when configured to not drop events when the ring buffer is full (appendTimeout=-1).
The async appender first try publish the event in the ring buffer then sleeps for appendRetryFrequency milliseconds  before retrying. This relatively long sleep affects performance dramatically.
An alternative is to use an exponential backoff strategy capped by the appendRetryFrequency. This should give better performance while still limiting the CPU consumption.

Another problem arise in CPU constraint environments when more threads than the available cores are retrying at the same time. The first retries happen at high rate (a few nanoseconds pauses) then slow down up to the configured appendRetryTimeout. This behaviour increases throughput at the expense of additional CPU cycles. We should limit the retries to a single thread at once to avoid overwhelming the system when only a few cores are available.