[Bug] cannot execute statement locally after code change in PR #2214

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Search before asking

[X] I have searched in the issues and found no similar issues.

Describe the bug
 }.orElse {
      // 3. get the main resource from dev environment
      val cwd = getClass.getProtectionDomain.getCodeSource.getLocation.getPath
        .split("kyuubi-server")
      assert(cwd.length > 1)
      Option(Paths.get(cwd.head, "externals", module, "target", jarName))
        .map(_.toAbsolutePath.toFile.getCanonicalPath)
    }

the cwd is not getting the correct path for local build.
It should be <my_local_path>/incubator-kyuubi/, but it is getting: <my_local_path>/incubator-kyuubi/kyuubi-assembly/target/scala-2.12/jars/.
How to reproduce:
build/mvn clean package -Pmirror-cdn -DskipTests

bin/kyuubi start

$SPARK_HOME/bin/beeline -u "jdbc:hive2://xxxx:10009"  (or using bin/beeline)

then execute statement, for example, select timestamp '2018-11-17';

When executing statement, error is thrown:
22/04/11 18:29:55 WARN DependencyUtils: Local jar /Users/tiliao/Documents/Code/incubator-kyuubi/kyuubi-assembly/target/scala-2.12/jars/externals/kyuubi-spark-sql-engine/target/kyuubi-spark-sql-engine_2.12-1.6.0-SNAPSHOT.jar does not exist, skipping.

Affects Version(s)
1.6.0
Kyuubi Server Log Output
18:29:52.248 INFO org.apache.kyuubi.engine.EngineRef: Launching engine:
/Users/tiliao/Documents/Software/spark-3.2.1-bin-hadoop2.7/bin/spark-submit \
        --class org.apache.kyuubi.engine.spark.SparkSQLEngine \
        --conf spark.kyuubi.session.engine.idle.timeout=PT10M \
        --conf spark.hive.server2.thrift.resultset.default.fetch.size=1000 \
        --conf spark.kyuubi.ha.zookeeper.quorum=<IP_ADDRESS>:2181 \
        --conf spark.kyuubi.client.ip=<IP_ADDRESS> \
        --conf spark.kyuubi.engine.submit.time=1649672992214 \
        --conf spark.app.name=kyuubi_USER_SPARK_SQL_anonymous_default_98374ac9-17d3-4bc5-a77a-56d4c223df3f \
        --conf spark.kyuubi.ha.engine.ref.id=98374ac9-17d3-4bc5-a77a-56d4c223df3f \
        --conf spark.kyuubi.ha.zookeeper.auth.type=NONE \
        --conf spark.kyuubi.session.engine.spark.max.lifetime=PT2M \
        --conf spark.yarn.tags=KYUUBI \
        --conf spark.kyuubi.ha.zookeeper.namespace=/kyuubi_1.6.0-SNAPSHOT_USER_SPARK_SQL/anonymous/default \
        --proxy-user anonymous /Users/tiliao/Documents/Code/incubator-kyuubi/kyuubi-assembly/target/scala-2.12/jars/externals/kyuubi-spark-sql-engine/target/kyuubi-spark-sql-engine_2.12-1.6.0-SNAPSHOT.jar

...

18:29:59.216 ERROR org.apache.kyuubi.server.KyuubiTBinaryFrontendService: Error executing statement:
java.util.concurrent.ExecutionException: org.apache.kyuubi.KyuubiSQLException: org.apache.kyuubi.KyuubiSQLException: Exception in thread "main" java.lang.reflect.UndeclaredThrowableException
 See more: /Users/tiliao/Documents/Code/incubator-kyuubi/work/anonymous/kyuubi-spark-sql-engine.log.3
        at org.apache.kyuubi.KyuubiSQLException$.apply(KyuubiSQLException.scala:69)
        at org.apache.kyuubi.engine.ProcBuilder.$anonfun$start$1(ProcBuilder.scala:165)
        at java.lang.Thread.run(Thread.java:748)
.
FYI: The last 10 line(s) of log are:
Caused by: org.apache.spark.SparkUserAppException: User application exited with 101
at org.apache.spark.deploy.SparkSubmit.org$apache$spark$deploy$SparkSubmit$$runMain(SparkSubmit.scala:928)
at org.apache.spark.deploy.SparkSubmit$$anon$1.run(SparkSubmit.scala:165)
at org.apache.spark.deploy.SparkSubmit$$anon$1.run(SparkSubmit.scala:163)
at java.security.AccessController.doPrivileged(Native Method)
at javax.security.auth.Subject.doAs(Subject.java:422)
at org.apache.hadoop.security.UserGroupInformation.doAs(UserGroupInformation.java:1746)
... 6 more

Kyuubi Engine Log Output
22/04/11 18:29:55 WARN DependencyUtils: Local jar /Users/tiliao/Documents/Code/incubator-kyuubi/kyuubi-assembly/target/scala-2.12/jars/externals/kyuubi-spark-sql-engine/target/kyuubi-spark-sql-engine_2.12-1.6.0-SNAPSHOT.jar does not exist, skipping.
Error: Failed to load class org.apache.kyuubi.engine.spark.SparkSQLEngine.
Exception in thread "main" java.lang.reflect.UndeclaredThrowableException
        at org.apache.hadoop.security.UserGroupInformation.doAs(UserGroupInformation.java:1761)
        at org.apache.spark.deploy.SparkSubmit.doRunMain$1(SparkSubmit.scala:163)
        at org.apache.spark.deploy.SparkSubmit.submit(SparkSubmit.scala:203)
        at org.apache.spark.deploy.SparkSubmit.doSubmit(SparkSubmit.scala:90)
        at org.apache.spark.deploy.SparkSubmit$$anon$2.doSubmit(SparkSubmit.scala:1043)
        at org.apache.spark.deploy.SparkSubmit$.main(SparkSubmit.scala:1052)
        at org.apache.spark.deploy.SparkSubmit.main(SparkSubmit.scala)
Caused by: org.apache.spark.SparkUserAppException: User application exited with 101
        at org.apache.spark.deploy.SparkSubmit.org$apache$spark$deploy$SparkSubmit$$runMain(SparkSubmit.scala:928)
        at org.apache.spark.deploy.SparkSubmit$$anon$1.run(SparkSubmit.scala:165)
        at org.apache.spark.deploy.SparkSubmit$$anon$1.run(SparkSubmit.scala:163)
        at java.security.AccessController.doPrivileged(Native Method)
        at javax.security.auth.Subject.doAs(Subject.java:422)
        at org.apache.hadoop.security.UserGroupInformation.doAs(UserGroupInformation.java:1746)
        ... 6 more

Kyuubi Server Configurations
default

Kyuubi Engine Configurations
default

Additional context
No response
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Do you run bin/kyuubi start in the kyuubi source dir?

yes, I am running in the source dir. which is /Users/tiliao/Documents/Code/incubator-kyuubi in my local.

yes, I am running in the source dir. which is /Users/tiliao/Documents/Code/incubator-kyuubi in my local.

This doesn't look like a good way, you can start kyuubi in kyuubi dist dir.

I am running in the source dir.

We should support such dev mode

@wForget looks good if works

@wForget looks good if works. Can you send a pr ?

sure, I will send a pr later.


make error

noisegen.cpp:7:23: fatal error: fir_coeff.h

Yes I forgot at add a file, thanks!  Unfortunately the fix will have to wait for a few days.

I added the missing include and the python code used to generate the coefficients.


fix generated columns in mutable columns for PostgreSQL

Fix https://github.com/go-jet/jet/issues/209

Need data from https://github.com/go-jet/jet-test-data/pull/3

LGTM. I've merged test data PR. To fix this PR build you'll need to push one more commit, with updated sub-module master head. You can just call make checkout-testdata and then push changes.

CircleCI didn't take the new link to jet-test-data. Did I misse something?

I cannot restart the CI myself.

Ahh, sorry I missed this. postgres tests are run on two databases postgres and cockrouchdb. pg tests are successfully, but cockrochdb are failing. To fix the tests we will have to add a the same table to cockroach test init data.

https://github.com/go-jet/jet-test-data/pull/4

@go-jet Thank you 🥳


Typescript: How to add a new compilation hook on a readonly hook?

How do I go about adding a new hook to webpack compiler? In webpack 4 I could do something like
interface CustomHooks extends CompilerHooks {
  tsCompiled: SyncHook<string[], string[]>;
}

interface CustomCompiler extends Compiler {
  hooks: CustomHooks;
}

const compiler = webpack(config) as CustomCompiler;
compiler.hooks.tsCompiled = new SyncHook(['warnings', 'errors']);

The guide here says you can add a custom hook by doing compiler.hooks.custom = ... but looking at the code, hooks is frozen https://github.com/webpack/webpack/blob/master/lib/Compiler.js#L120. Indeed, if I try to add extend this anyway by using // @ts-ignore, I get

Failed to create a webpack compiler: Cannot add property tsCompiled, object is not extensible

So how do you go about extending hooks?

This is intentional: https://webpack.js.org/blog/2020-10-10-webpack-5-release/#hook-object-frozen
To offer a custom hook you need to do the following:
Create a module-scope weak map for compilation hooks:
const compilationHooks = new WeakMap<Compilation, MyHooks>();

interface MyHooks {
  custom: SyncHook<[number, string]>
}

Create a static method on your plugin:
static getCompilationHooks(compilation: Compilation) : MyHooks {
  let hooks = compilationHooks.get(compilation);
  if(hooks === undefined) {
    compilationHooks.set(compilation, hooks = {
      custom: new SyncHook()
    });
  }
  return hooks;
}

Call hooks like this in your plugin:
const hooks = MyPlugin.getCompilationHooks(compilation);

hooks.custom.call(1, "hello");

Other plugins could access your custom hooks too:
import MyPlugin from "my-plugin";

const hooks = MyPlugin.getCompilationHooks(compilation);

hooks.custom.tap("OtherPlugin", (n, s) => {
  // magic
});

see https://github.com/jantimon/html-webpack-plugin#events for an example

Answer above, feel free to feedback

It might be a good idea to change the documentation to reflect this: https://webpack.js.org/api/plugins/#custom-hooks

@EECOLOR Yep, can you open an issue?


Programming languages with alphabet "S"

Many projects need to show a list of programming languages as checkboxes in their application. Hence the idea. I have finished till K. Can someone help me finish the alphabet S in checkboxes.html?
Here is the list from which I have referred to the list of programming languages
https://en.wikipedia.org/wiki/List_of_programming_languages

Can i work on this?

Sure @shivam1317. Thanks again


adapt #532 to windows stacker.cmd

Because it can't be easy on Windows, this PR incorporates the script update in #532 to stacker.cmd

Sorry, totally missed that we have a windows version. We really need to run the functional tests on windows in CI somehow.
Anyways, this lgtm. Thanks!

Actually, @troyready, can you re-open this against stable-1.2? I can release this with 1.2.1.

Just went ahead and cherry-picked this into stable-1.2 and master. Thanks @troyready!


Feature Request Make the Terminal accessible outside of the Windows 10 Store

Summary of the new feature/enhancement

App is accessible outside of the Windows Store to Download as an .exe File
Because of company policies i can never use this app will working. An Environment where i will definetly need to have this app.
Proposed technical implementation details (optional)

The app can be installed as an .exe setup which auto updates.

This is a dupe of #1386.
For the record, we do have releases available on our releases page.


message is incomplete

The sever called the client with a complex object,and the browser's console log the data received correct. However, an error throws after the log:uncaught error:message is incomplete.

Ive already found out the issue which is the Chinese en code triggered

@sweetwxh are you sure you're using this SignalR and not github.com/SignalR/SignalR?

@davidfowl yes,I'm using asp.net core 2.0 preview with this signalr. The library was installed with nuget（https://dotnet.myget.org/F/aspnetcore-ci-dev/api/v3/index.json"), I'm trying encode Chinese characters for transporting now.

@sweetwxh - can you provide a repro (ideally as a github repo)?

@moozzyk
On the server side, just a simply call with an object that one of it's field has Chinese characters:
var message  = new Message{
   UserName="Fred",IsPrivate=true,Msg="测试消息"
};
await realtime.Clients.All.InvokeAsync("SendNotify",message);

The client throws an exception:

I can reproduce this. It's an encoding problem both ways. The length is being encoded in javascript incorrectly and we're assuming utf8 on the server side. On the client side it's the same problem, we send back the utf8 length of the string payload and the client expects utf16.


Raw output

I notice there is a flag that has been merged from a PR to allow the client to output raw content, I am wondering how to do this using the package programatically.?
What is the mechanism to get the output as "raw" or is this something that is expected to just do manually (by stripping quotes for instance?)

You can implement the behavior by checking the type of result values, and just print it if it is a string.


nit: firecrawl fixes

not sure why page_content was set instead of the text property

page_content is langchain heh


fix(38388) Add support for this parameters in iterationTypes and fix caching issues

Fixes #38388
Fixes #39693
This PR:

Adds support for this parameters in iterationTypes!

const obj = {
	x: 1,
	y: 2,
	z: 3,

	*[Symbol.iterator]<P>(this: P) {
		for (const k in this) yield k;
	},
};

for (const value of obj) {
    console.log(value); // value is `"x" | "y" | "z"`
}

Fixes the iterator caching logic. Looking at the code, I noticed a number of problems that could arise from the caching logic. Here is a reproduction of two of the issues I found. Basically, getIterationTypesOfIterable had 3 problems:

Cached iterationTypes don't error like they should (see the first bug in the playground).

To fix this, we don't cache iterationTypes that produced errors, which makes getIterationTypesOfIterableSlow get called every time and produce the proper errors.
We can also take advantage of this design by detecting in getIterationTypesOfIterable whether something was cached to determine whether or not to cache or reportTypeNotIterableError (which we don't want to do if there was already an error from resolveCall, for example).
Keep in mind that caches should only be read from or written to when implicitThisType === type.

When use permits either AsyncIterables or (Sync)Iterables converted via getAsyncFromSyncIterationTypes it would call these functions in this order: getIterationTypesOfIterableCached(type, asyncResolver) || getIterationTypesOfIterableFast(type, asyncResolver) || getIterationTypesOfIterableCached(type, syncResolver) || getIterationTypesOfIterableFast(type, syncResolver) || getIterationTypesOfIterableSlow(type, asyncResolver) || getIterationTypesOfIterableSlow(type, syncResolver). This is incorrect because an object could have an Symbol.asyncIterator that is neither cached nor an instance of a simple global type, and that logic could then fall back on the syncResolver section. Now it runs cached || fast || slow in order for both Symbol.asyncIterator and Symbol.iterator when applicable. This led to the 2nd and 3rd bugs demonstrated in the playground link above.
iterationTypesOfAsyncIterable was shared between Symbol.iterator and Symbol.asyncIterator. Now iterationTypesOfAsyncedSyncIterable is used to store Symbol.iterator calls that got converted via getAsyncFromSyncIterationTypes.

To make implicit Symbol.iterator calls like actual Symbol.iterator calls, I hooked them up to the resolveCall function. This function and a few of the ones it calls had to be modified so that arbitrary Nodes could be passed into it without analyzing the Node as whatever it is. The implicit call case is differentiated by the implicitThisType being passed around. I.e. when implicitThisType is present, then the node should be treated solely as an errorNode, and the only NodeKind-specific behavior should be in getting the boundaries for error messages. typeArguments and parametric args should always be empty.
Implicit iterator calls do have different behavior from regular calls though. Intersected iterator functions will produce intersected results instead of just grabbing the first one. See #39693. Unioned iterator functions will evaluate each unioned constituent separately (passing the constituent, not the top-level type, as the implicitThisType) and the results will be unioned together. See here for examples. This means there will be inconsistency between how for (const x of obj) vs for (const x of obj[Symbol.iterator]()) is evaluated, although there was already inconsistency before and I think it is for the better.

I noticed on my machine that the errors in errorsInGenericTypeReference changed from Could not find symbol V to Cannot find name 'V'. Is that actually the product of one of my changes...? I don't think it is, but I could update the files if wanted.

There's a lot going on here and it will take me some time to review. I'm not certain I agree with some of the changes here, as there are a large number of places where you've switched to use a less precise type so that you can treat node as only an "error node". A separate errorNode argument argument would likely be preferable in those cases.
It might be worth breaking out the changes to fix the caching issues you mentioned into a separate PR, as those changes seem less controversial to me.

Thanks for getting back to me! I really wasn't sure whether to prioritize making fewer changes and making it work with what's already there or to make changes like you suggested. I tried to think of how to solve the problem while making the smallest number of changes, then take one step towards what I would consider ideal. I wasn't sure what kind of philosophy there is about these kinds of things so I just guessed. Since my guess was wrong, I'll go back and make the changes I think would be ideal instead of trying to guess where my hands should be tied. I did kinda deviate from that anyway and got caught up a bit when I noticed a bunch of other issues while looking at the code. I will work on this more this week. I'll make this better.

There's a lot going on here and it will take me some time to review. I'm not certain I agree with some of the changes here, as there are a large number of places where you've switched to use a less precise type so that you can treat node as only an "error node". A separate errorNode argument argument would likely be preferable in those cases.
It might be worth breaking out the changes to fix the caching issues you mentioned into a separate PR, as those changes seem less controversial to me.

Thanks for getting back to me! I really wasn't sure whether to prioritize making fewer changes and making it work with what's already there or to make changes like you suggested. I tried to think of how to solve the problem while making the smallest number of changes, then take one step towards what I would consider ideal. I wasn't sure what kind of philosophy there is about these kinds of things so I just guessed. Since my guess was wrong, I'll go back and make the changes I think would be ideal instead of trying to guess where my hands should be tied. I did kinda deviate from that anyway and got caught up a bit when I noticed a bunch of other issues while looking at the code. I will work on this more this week. I'll make this better.

🧹 Closing due to staleness. Please open a new PR if you'd like to continue. Thanks!


Type error on streamingMode

Using the boilerplate code from the bluetooth streaming tutorial streaming mode returns a type error for any input (including the values that are meant to be typed: 'bluetooth-with-wifi-fallback' etc)
import { Neurosity, WebBluetoothTransport } from "@neurosity/sdk";
...
const neurosity = new Neurosity({
        autoSelectDevice: true,
        bluetoothTransport: new WebBluetoothTransport(),
        streamingMode: 'wifi-only'
      });

Hey @oliverrees,
Thanks for reporting this issue.
I've published<EMAIL_ADDRESS>with an enum you can import when using TypeScript:
import { Neurosity, WebBluetoothTransport, STREAMING_MODE } from "@neurosity/sdk";

const neurosity = new Neurosity({
    autoSelectDevice: true,
    bluetoothTransport: new WebBluetoothTransport(),
    streamingMode: STREAMING_MODE.WIFI_ONLY
});


mkdir error

Hi.
On ubuntu, with locators like '.header-menu.header-menu__left > ul > li:nth-child(1) > a' i has error, when gimin trying to create directory for screenshot
EPROTO: protocol error, mkdir '.header-menu.header-menu__left > ul > li:nth-child(1) > a'
Can you make names for directories with base64, or somethign similar?
Thank you.

so your suite has name '.header-menu.header-menu__left > ul > li:nth-child(1) > a'?

@pandoraizida to my mind it's strange desire.
Why you don't want use a more understandable name?
And I don't understand if suite name will be in base64 how you understand which a suite has failed?

@sipayRT @DudaGod
its not a suite name. its locator.
As i understand, gemini make directory structure like this:
suite\locator\chrome.png
problem with locator directory.
base64 make LmhlYWRlci1tZW51LmhlYWRlci1tZW51X19sZWZ0ID4gdWwgPiBsaTpudGgtY2hpbGQoMSkgPiBh from '.header-menu.header-menu__left > ul > li:nth-child(1) > a'
this more correct for directory name than locator.
and we always can decode locator from base64.

@kkochetov no, you're wrong. structure will be suite-name/state-name/browser.png. It makes no sense to generate directories with base64 because all suites are unique.


Chef 13 error: uninitialized constant Chef::Resource::CloudcliAwsS3File

* cloudcli_aws_s3_file[/tmp/testfile] action get

           ================================================================================
           Error executing action `get` on resource 'cloudcli_aws_s3_file[/tmp/testfile]'
           ================================================================================

           NameError
           ---------
           uninitialized constant Chef::Resource::CloudcliAwsS3File

           Cookbook Trace:
           ---------------
           /tmp/kitchen/cache/cookbooks/cloudcli/providers/aws_s3_file.rb:32:in `load_current_resource'

           Resource Declaration:
           ---------------------
           # In /tmp/kitchen/cache/cookbooks/ws/recipes/default.rb

            13: cloudcli_aws_s3_file file_path do
            14:   aws_access_key_id node['aws_key']
            15:   aws_secret_access_key node['aws_token']
            16:   region 'us-east-1'
            17:   bucket node['s3bucket']
            18:   key fname
            19: end
            20:

Closing


bug:

Describe the bug
Receiving a panic: not a string error message without further indication of where the underlying issue could be located.
To Reproduce
Steps to reproduce the behavior:

Run tfsec . from. the module's directory

Expected behavior
tfsec's error log should pinpoint which terraform resource or variable (line number?) was not correctly understood by its parser.
Output of your tfsec command with --debug flag

07:29.736937000 system.info                      APP       tfsec
07:29.737062000 system.info                      VERSION   v1.28.1
07:29.737063000 system.info                      OS        darwin
07:29.737064000 system.info                      ARCH      arm64
07:29.737065000 system.info                      KERNEL
07:29.737066000 system.info                      TERM      xterm-256color
07:29.737068000 system.info                      SHELL     /bin/zsh
07:29.737068000 system.info                      GOVERSION go1.19.3
07:29.737071000 system.info                      GOROOT    /opt/homebrew/Cellar/go/1.19.3/libexec
07:29.737072000 system.info                      CGO       false
07:29.737074000 system.info                      CPUCOUNT  10
07:29.737075000 system.info                      MAXPROCS  10
07:29.737076000 system.info                      WORKDIR   /Users/myusername/Library/CloudStorage/OneDrive-mycompanynamedirectory/Git/mydirectoryname/tf-mod-az-keyvault
07:29.737078000 system.info                      UID       501
07:29.737119000 system.info                      EUID      501
07:29.737121000 system.info                      DOCKER    false
07:29.737122000 system.info                      CI        false
07:29.737123000 system.info                      HOSTNAME  MACHOSTNAME
07:29.737124000 system.info                      TEMP      /var/folders/zt/ww2w2gpx6x5cn0lmsj4tyj5h0000gn/T/
07:29.737125000 system.info                      PATHSEP   /
07:29.737127000 system.info                      CMD       tfsec . --debug
07:29.737132000 cmd                              Command args=[]string{"."}
07:29.737766000 cmd                              Determined path dir=/Users/myusername/Library/CloudStorage/OneDrive-mycompanynamedirectory/Git/mydirectoryname/tf-mod-az-keyvault
07:29.737862000 cmd                              Determined path root=/
07:29.737865000 cmd                              Determined path rel=Users/myusername/Library/CloudStorage/OneDrive-mycompanynamedirectory/Git/mydirectoryname/tf-mod-az-keyvault
07:29.737946000 terraform.scanner                Scanning [&{/ /}] at 'Users/myusername/Library/CloudStorage/OneDrive-mycompanynamedirectory/Git/mydirectoryname/tf-mod-az-keyvault'...
07:29.740641000 terraform.scanner.rego           Loaded 3 embedded libraries.
07:29.757557000 terraform.scanner.rego           Loaded 119 embedded policies.
07:29.855858000 terraform.scanner                Scanning root module 'Users/myusername/Library/CloudStorage/OneDrive-mycompanynamedirectory/Git/mydirectoryname/tf-mod-az-keyvault'...
07:29.855882000 terraform.parser.          Setting project/module root to 'Users/myusername/Library/CloudStorage/OneDrive-mycompanynamedirectory/Git/mydirectoryname/tf-mod-az-keyvault'
07:29.855884000 terraform.parser.          Parsing FS from 'Users/myusername/Library/CloudStorage/OneDrive-mycompanynamedirectory/Git/mydirectoryname/tf-mod-az-keyvault'
07:29.856064000 terraform.parser.          Parsing 'Users/myusername/Library/CloudStorage/OneDrive-mycompanynamedirectory/Git/mydirectoryname/tf-mod-az-keyvault/locals.tf'...
07:29.858170000 terraform.parser.          Added file Users/myusername/Library/CloudStorage/OneDrive-mycompanynamedirectory/Git/mydirectoryname/tf-mod-az-keyvault/locals.tf.
07:29.858188000 terraform.parser.          Parsing 'Users/myusername/Library/CloudStorage/OneDrive-mycompanynamedirectory/Git/mydirectoryname/tf-mod-az-keyvault/module.tf'...
07:29.859035000 terraform.parser.          Added file Users/myusername/Library/CloudStorage/OneDrive-mycompanynamedirectory/Git/mydirectoryname/tf-mod-az-keyvault/module.tf.
07:29.859046000 terraform.parser.          Parsing 'Users/myusername/Library/CloudStorage/OneDrive-mycompanynamedirectory/Git/mydirectoryname/tf-mod-az-keyvault/outputs.tf'...
07:29.859252000 terraform.parser.          Added file Users/myusername/Library/CloudStorage/OneDrive-mycompanynamedirectory/Git/mydirectoryname/tf-mod-az-keyvault/outputs.tf.
07:29.859260000 terraform.parser.          Parsing 'Users/myusername/Library/CloudStorage/OneDrive-mycompanynamedirectory/Git/mydirectoryname/tf-mod-az-keyvault/providers.tf'...
07:29.859423000 terraform.parser.          Added file Users/myusername/Library/CloudStorage/OneDrive-mycompanynamedirectory/Git/mydirectoryname/tf-mod-az-keyvault/providers.tf.
07:29.859430000 terraform.parser.          Parsing 'Users/myusername/Library/CloudStorage/OneDrive-mycompanynamedirectory/Git/mydirectoryname/tf-mod-az-keyvault/variables.tf'...
07:29.860197000 terraform.parser.          Added file Users/myusername/Library/CloudStorage/OneDrive-mycompanynamedirectory/Git/mydirectoryname/tf-mod-az-keyvault/variables.tf.
07:29.860206000 terraform.parser.          Evaluating module...
07:29.861207000 terraform.parser.          Read 38 block(s) and 0 ignore(s) for module 'root' (5 file[s])...
07:29.861213000 terraform.parser.          Added 0 variables from tfvars.
07:29.861223000 terraform.parser.          Error loading module metadata: open //Users/myusername/Library/CloudStorage/OneDrive-mycompanynamedirectory/Git/mydirectoryname/tf-mod-az-keyvault/.terraform/modules/modules.json: no such file or directory.
07:29.861236000 terraform.parser.          Working directory for module evaluation is '/Users/myusername/Library/CloudStorage/OneDrive-mycompanynamedirectory/Git/mydirectoryname/tf-mod-az-keyvault'
07:29.861631000 terraform.parser..evaluator Filesystem key is 'f9088d3ed2db9899500f703a07bb505300c2b5cbc122ac4365ca04af35422e64'
07:29.861635000 terraform.parser..evaluator Starting module evaluation...
07:29.863055000 terraform.parser..evaluator Starting submodule evaluation...
07:29.863270000 terraform.parser..evaluator Finished processing 0 submodule(s).
07:29.863277000 terraform.parser..evaluator Starting post-submodule evaluation...
07:29.865051000 terraform.parser..evaluator Module evaluation complete.
07:29.865070000 terraform.parser.          Finished parsing module 'root'.
07:29.865087000 terraform.parser..evaluator Added module output key_vault_id=cty.StringVal("68a86542-0a2a-428f-8ee8-5bc1bdbc8f7f").
07:29.865091000 terraform.parser..evaluator Added module output key_vault_name=cty.NilVal.
07:29.865094000 terraform.executor               Adapting modules...
panic: not a string
goroutine 1 [running]:
github.com/zclconf/go-cty/cty.Value.AsString({{{0x0?, 0x0?}}, {0x0?, 0x0?}})
/Users/brew/Library/Caches/Homebrew/go_mod_cache/pkg/mod/github.com/zclconf/go-cty@v1.10.0/cty/value_ops.go:1256 +0x130
github.com/aquasecurity/defsec/internal/adapters/terraform/azure/keyvault.adaptKey()
/Users/brew/Library/Caches/Homebrew/go_mod_cache/pkg/mod/github.com/aquasecurity/defsec@v0.82.2/internal/adapters/terraform/azure/keyvault/adapt.go:154 +0x15c
github.com/aquasecurity/defsec/internal/adapters/terraform/azure/keyvault.(*adapter).adaptVault(, _, )
/Users/brew/Library/Caches/Homebrew/go_mod_cache/pkg/mod/github.com/aquasecurity/defsec@v0.82.2/internal/adapters/terraform/azure/keyvault/adapt.go:97 +0x360
github.com/aquasecurity/defsec/internal/adapters/terraform/azure/keyvault.(*adapter).adaptVaults(0x14001dfb288, {0x140013d21c8, 0x1, 0x14001dfb298?})
/Users/brew/Library/Caches/Homebrew/go_mod_cache/pkg/mod/github.com/aquasecurity/defsec@v0.82.2/internal/adapters/terraform/azure/keyvault/adapt.go:34 +0x16d8
github.com/aquasecurity/defsec/internal/adapters/terraform/azure/keyvault.Adapt({0x140013d21c8, 0x1, 0x1})
/Users/brew/Library/Caches/Homebrew/go_mod_cache/pkg/mod/github.com/aquasecurity/defsec@v0.82.2/internal/adapters/terraform/azure/keyvault/adapt.go:20 +0xa4
github.com/aquasecurity/defsec/internal/adapters/terraform/azure.Adapt({, _, _})
/Users/brew/Library/Caches/Homebrew/go_mod_cache/pkg/mod/github.com/aquasecurity/defsec@v0.82.2/internal/adapters/terraform/azure/adapt.go:30 +0x1a8
github.com/aquasecurity/defsec/internal/adapters/terraform.Adapt({0x140013d21c8, 0x1, 0x1})
/Users/brew/Library/Caches/Homebrew/go_mod_cache/pkg/mod/github.com/aquasecurity/defsec@v0.82.2/internal/adapters/terraform/adapt.go:20 +0x6c
github.com/aquasecurity/defsec/pkg/scanners/terraform/executor.(*Executor).Execute(0x14000b51a00, {0x140013d21c8?, 0x1, 0x1})
/Users/brew/Library/Caches/Homebrew/go_mod_cache/pkg/mod/github.com/aquasecurity/defsec@v0.82.2/pkg/scanners/terraform/executor/executor.go:96 +0xd4
github.com/aquasecurity/defsec/pkg/scanners/terraform.(*Scanner).ScanFSWithMetrics(0x140002b44b0, {0x102385580, 0x140001a6030}, {0x12af15b30?, 0x14000e0bf00?}, {0x140006315c1, 0x5c})
/Users/brew/Library/Caches/Homebrew/go_mod_cache/pkg/mod/github.com/aquasecurity/defsec@v0.82.2/pkg/scanners/terraform/scanner.go:216 +0x604
github.com/aquasecurity/tfsec/internal/app/tfsec/cmd.Root.func1(0x14000c4ec00, {0x14000e0be20, 0x1, 0x2})
/private/tmp/tfsec-20221102-4479-e7dyae/tfsec-1.28.1/internal/app/tfsec/cmd/root.go:83 +0x53c
github.com/spf13/cobra.(*Command).execute(0x14000c4ec00, {0x14000192190, 0x2, 0x2})
/Users/brew/Library/Caches/Homebrew/go_mod_cache/pkg/mod/github.com/spf13/cobra@v1.6.0/command.go:916 +0x5c8
github.com/spf13/cobra.(*Command).ExecuteC(0x14000c4ec00)
/Users/brew/Library/Caches/Homebrew/go_mod_cache/pkg/mod/github.com/spf13/cobra@v1.6.0/command.go:1040 +0x354
github.com/spf13/cobra.(*Command).Execute(...)
/Users/brew/Library/Caches/Homebrew/go_mod_cache/pkg/mod/github.com/spf13/cobra@v1.6.0/command.go:968
main.main()
/private/tmp/tfsec-20221102-4479-e7dyae/tfsec-1.28.1/cmd/tfsec/main.go:12 +0x20

System Info

tfsec version: v1.28.1
terraform version: v1.5.3
OS: macOS Ventura 13.5

Example Code
Since the underlying issue is not clear, I'd need to post the entire module's code here, which I'm prevented by my company's policies from doing. It is an Azure KeyVault base module.
Additional context
The question here is: how to locate the resource or variable that might be incorrectly declared in the module that is causing this error?

Hi @RoFz !
Could you provide an example of the configuration that you have a problem with?

Hi @RoFz ! Could you provide an example of the configuration that you have a problem with?

Example Code
Since the underlying issue is not clear, I'd need to post the entire module's code here, which I'm prevented by my company's policies from doing. It is an Azure KeyVault base module.

Additionally, if I could post a code snippet, that would mean I have managed to track which part of the module has the problem, which is exactly the issue here: tfsec does not provide any indication as to where the underlying problem is.

@RoFz It seems this problem has already been solved. You can use Trivy to scan configurations. https://github.com/aquasecurity/tfsec/discussions/1994

@nikpivkin thanks! I can confirm that it is the same root cause. Here's the code snippet.
resource "azurerm_key_vault_key" "kv_stkvdiagkey" {
  name            = format("key-%s", azurecaf_name.caf_stkvdiag.result)
  key_vault_id    = azurerm_key_vault.main.id
  key_type        = "RSA-HSM"
  key_size        = 2048
  expiration_date = time_rotating.kv_stkvdiagkey.rotation_rfc3339
  key_opts = [
    "decrypt",
    "encrypt",
    "sign",
    "unwrapKey",
    "verify",
    "wrapKey"
  ]
  depends_on = [
    azurerm_key_vault_access_policy.tf_user,
    azurerm_key_vault_access_policy.stkvdiag
  ]
}

So, from #1994, tfsec will not be maintained anymore as an individual tool and I should not expect this same fix to be applied to its homebrew version, would that be correct?

@RoFz Right, the efforts of the developers are aimed at Trivy.

@RoFz That's right, the developers' efforts are aimed at Trivy.


Integration tests to cargo - part 1

Initial steps needed to move integration tests to cargo. This PR mainly focuses on replacing println! with writeln! to be able to test the outputs. The current example tests are moved to confirm this works correctly.

Nice work @chesedo. LGTM :)


Got an unexpected result

I downloaded the ml-100k(MovieLens 100K Dataset) and did the experiments with default args ( 3 epochs), however, the results I got are much higher than those reported in the paper.
Test AUC 0.9506 |  ACC 0.8792 | Test F1 0.8764
I am wondering whether it is caused by my data processing scripts or other reason.

Thanks for your question. The reason is that the results here are reported in our previous KDD paper. In this repository, we actually show the implementation of our TOIS paper, which is the extended version. You can refer to the paper (https://arxiv.org/pdf/2011.12683.pdf) here.


Update documentation to bring it consistent with Fusion 5.2.0

The on-line documentation is showing some age and needs to be brought up to date.
Changes to make:

4412421.html has a link to nationalatlas.gov which no longer exists (replaced by nationalmap.gov)
4412447.html and a few other places: the images for Imagery Project is out of date because it doesn't show the "Legend" section.
The tutorial references an SFBayAreaMercator project, but it never has the user create the project.  This could be fixed by adding instructions to create the referenced project or just using the SFBayArea project.
The tutorial does not tell the user to set up default paths for Imagery, Terrain, and Vector resources, so a warning dialog box comes up every time you look up a resource.
4412421.html, terrain step 9 has an unnecessary carriage return
4412447.html says the default percentage under representative subset per tile is 75%, but it’s actually 50%
4412449.html: gevectorimport-tool doesn’t exist and gevectorimport doesn’t seem to have the functionality described
4412684.html: there’s a missing step between 7 and 8 where you save the first field and create a new one
4412455.html: in step 6 it says to click the name field, but you have to double click it
Some of the references to CA_POIs include a space in the name, which is not a valid resource name.
4412446.html: link to Google Earth Administration Guide would be useful.
4412446.html: link to the Google Earth Client would be useful.
4412690.html: validate if the list of tutorial files is current and include the .jp2 files
4412447.html: fusion reference guide link is missing
In '4412421.html', under "Build a Vector Project", last Step: The tutorial says to click the 'X' in the upper, right-hand corner. Under ubuntu 16.04, it is in the upper, left-hand corner.  Make this more generic so it just says to click the close button.
In 4412443.html, the link at the top to "Define a Vector Resource" doesn't do anything

Closed via PR #457, PR #462, and PR #464.


Improvement: pressing left and right on keyboard can change quote

Pressing ⬅  and ➡  on keyboard can change quote.

Feature added last week ;)


Update dependencies to cennnznet 1.0.0

Integrated with<EMAIL_ADDRESS>tested in the following check-points:

[x] Sending tx between Alice and Bob;
[x] Creating new account;
[x] Sending rpc call in toolbox (tested on getBlockhash method);

I think the dependency of @cennznet/api needs to be introduced in this PR? and we need to remove @cennznet/types?

I think the dependency of @cennznet/api needs to be introduced in this PR? and we need to remove @cennznet/types?
Which means the rpc call that we need to check is the newly introduced calls in @cennznet/api for get is cennzx buyPrice/sellPrice?
@holygits ? need ur help with my understanding on this

Now that we have metadata updated, we could get those rpc.cennz calls from metadata directly?

I think the dependency of @cennznet/api needs to be introduced in this PR? and we need to remove @cennznet/types?
Which means the rpc call that we need to check is the newly introduced calls in @cennznet/api for get is cennzx buyPrice/sellPrice?
@holygits ? need ur help with my understanding on this

Now that we have metadata updated, we could get those rpc.cennz calls from metadata directly?

No, I guess we can't get rpc calls in metadata.. and thats the reason we decorated it in our api...

If you go to toolbox and select rpc and methods(), the methods are listed there, I am not sure if they can be used, or I misunderstood it. @KarishmaBothara

If you go to toolbox and select rpc and methods(), the methods are listed there, I am not sure if they can be used, or I misunderstood it. @KarishmaBothara

Yes, this only shows all the available methods in the rpc.. but it doesn't support, meaning you can't make a call for it...

If you go to toolbox and select rpc and methods(), the methods are listed there, I am not sure if they can be used, or I misunderstood it. @KarishmaBothara

Yes, this only shows all the available methods in the rpc.. but it doesn't support, meaning you can't make a call for it...

Ok, I will add api now and give a try. May I wonder how can I test it?

If you go to toolbox and select rpc and methods(), the methods are listed there, I am not sure if they can be used, or I misunderstood it. @KarishmaBothara

Yes, this only shows all the available methods in the rpc.. but it doesn't support, meaning you can't make a call for it...

Ok, I will add api now and give a try. May I wonder how can I test it?

In the toolbox, I guess it should be available under RPC calls,  'Call the selected endpoint' should show cennzx and next to it, the methods..

Can make a follow up PR to re-introduce @cennznet/api this is good for now 👍

Can make a follow up PR to re-introduce @cennznet/api this is good for now 👍

Cool..lets update the end point, and get this merged..


Roll Engine from 1a76ac24c509 to 037d03855eb5 (18 revisions)

https://github.com/flutter/engine/compare/1a76ac24c509...037d03855eb5
2021-12-03<EMAIL_ADDRESS>Roll Fuchsia Linux SDK from NJK-w4N99... to RyloAtJNT... (flutter/engine#30105)
2021-12-03<EMAIL_ADDRESS>Fix platform env variable not registering with sdk_manager (flutter/engine#30107)
2021-12-03<EMAIL_ADDRESS>Fail loudly with verbose logs when failing to download dart sdk. (flutter/engine#30093)
2021-12-03<EMAIL_ADDRESS>Revert "[ci.yaml] Remove default properties (#29694)" (flutter/engine#30100)
2021-12-03<EMAIL_ADDRESS>Accessibility: emit FOCUS_CHANGED events (flutter/engine#30091)
2021-12-03<EMAIL_ADDRESS>Close image reader when the resource is unreachable (flutter/engine#30089)
2021-12-02<EMAIL_ADDRESS>[Windows keyboard] Remove redundant parameter FlutterWindowsView (flutter/engine#30086)
2021-12-02<EMAIL_ADDRESS>Roll Skia from 2756b0ee0250 to f278a8058eaa (2 revisions) (flutter/engine#30084)
2021-12-02<EMAIL_ADDRESS>[Proposal] Improve Canvas Documentation (flutter/engine#30069)
2021-12-02<EMAIL_ADDRESS>Migrate sk_cf_obj to sk_cfp (flutter/engine#30029)
2021-12-02<EMAIL_ADDRESS>Make fragment leftover from an attach/detach race slightly safer (flutter/engine#29908)
2021-12-02<EMAIL_ADDRESS>Roll Dart SDK from 9f61c2487bbd to ba5e72e4b2d7 (16 revisions) (flutter/engine#30075)
2021-12-02<EMAIL_ADDRESS>[ci.yaml] Remove default properties (flutter/engine#29694)
2021-12-02<EMAIL_ADDRESS>Move swiftshader forward a couple of weeks (Nov 20 2019 -> Dec 5 2019) to add missing implementations in SubzeroReactor.cpp. (flutter/engine#30068)
2021-12-02<EMAIL_ADDRESS>Remove todos (flutter/engine#30002)
2021-12-02<EMAIL_ADDRESS>Roll Fuchsia Mac SDK from XankF6weU... to MaLRScmfU... (flutter/engine#30076)
2021-12-02<EMAIL_ADDRESS>fuchsia: Add a SoftwareSurfaceProducer for debug (flutter/engine#29657)
2021-12-02<EMAIL_ADDRESS>Roll Skia from 9d74c28e823a to 2756b0ee0250 (5 revisions) (flutter/engine#30078)
Also rolling transitive DEPS:
fuchsia/sdk/core/linux-amd64 from NJK-w4N99BKi to RyloAtJNTAv0
fuchsia/sdk/core/mac-amd64 from XankF6weUGAJ to MaLRScmfUUsq
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


App Config Fix Push Refresh URI Issue

Description
Fixes https://github.com/Azure/azure-sdk-for-java/issues/29162
A period was left on the URI after domain is removed

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


support: Create process_support_view_request BillingSession method.

Creates a process_support_view_request method for BillingSession abstract class to process the various support requests that relate to the billing system.
Notes:

Moves approve_realm_sponsorship, update_realm_sponsorship_status, and attach_discount_to_realm to this new BillingSession method.

Approving sponsorship is implemented in the child classes.
The other two (attaching a discount and/or updating the sponsorship status) are implemented in the abstract class since these are fields on the Customer (default_discount and sponsorship_pending).

Adds a new abstract property, billing_session_entity_name, to BillingSession to have a string value to use for support messages sent when these requests are processed.

This could potentially have a better name.

Self-review checklist

[ ] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[ ] Explains differences from previous plans (e.g., issue description).
[ ] Highlights technical choices and bugs encountered.
[ ] Calls out remaining decisions and concerns.
[ ] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[ ] Each commit is a coherent idea.
[ ] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[ ] Visual appearance of the changes.
[ ] Responsiveness and internationalization.
[ ] Strings and tooltips.
[ ] End-to-end functionality of buttons, interactions and flows.
[ ] Corner cases, error conditions, and easily imagined bugs.

Replied on the naming question above; otherwise this looks like a great direction to me.

Updates:

Adds basic discount and sponsorship support actions for the remote server support view.

Notes:

Because the update plan type for a remote server is not yet implemented. If the remote server is fully sponsored, then the options for updating the customer default_discount and sponsorship_pending fields are not shown. Instead, text indicating the remote server's sponsored status is shown (see final commit).

Screenshots:

Initial support view for a remote server

Updating sponsorship pending and approving sponsorship

Updating discount

The test failure is a clearly unrelated test flake in the drafts code path.


Unlike ServiceCredentials, ClientCredentials doesn't inherit from SecurityCredentialsManager class

public class ClientCredentials : System.ServiceModel.Description.IEndpointBehavior

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

ID: ecf81204-9954-e09a-5557-ad595b137ef9
Version Independent ID: 93ced467-d94c-3f78-1adb-0348be9a4279
Content: Walkthrough: Creating Custom Client and Service Credentials
Content Source: docs/framework/wcf/extending/walkthrough-creating-custom-client-and-service-credentials.md
Product: dotnet-framework
Technology: dotnet-wcf
GitHub Login: @HongGit
Microsoft Alias: wcfsrvt

Thank you for opening this issue. I'll add it to the backlog. You can always submit the fix yourself by editing the original article. Click on the Content Source link at the bottom of your original comment. To learn how to edit, see the Editing files in a repository article from GitHub.
I think the paragraph where this is mentioned just needs to be reworded to not mention a base class...

Both the ClientCredentials and the ServiceCredentials classes inherit from the abstract SecurityCredentialsManager class that defines the contract for returning the SecurityTokenManager.

I think that paragraph can just be deleted.
Thanks again!


updated SR with correct information

Codecov Report

Merging #18 (364d8e0) into dev (37faa7c) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##              dev      #18   +/-   ##
=======================================
  Coverage   85.71%   85.71%
=======================================
  Files           1        1
  Lines           7        7
=======================================
  Hits            6        6
  Misses          1        1

Continue to review full report at Codecov.

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


Add folder and neatify

its true

I don't see why this isn't already organized.
What kind of "neatify" could you be referring to?

h


fix: IconSetItem properties name type

Fixes #17 where v2.3.x introduced a faulty type declaration for IconSetItem.properties.name.

A sad typo. Thanks for noticing and fixing it. Thank you @rohrlaf 🔥


Do not output default error messages when custom message is given.

Working on encoding tests, I discovered that testharness.js outputs both the default error message and the custom message when the later is given as an argument. It looks like this:
Fail    HTML/Web Encoding 'windows-874' of byte value: DB   assert_equals: expected "U+FFFD" but got "" (U+F8C1) expected "�" but got ""
The first part (up to "(U+F8C1)") is the custom message, where I have added the character number. I don't need the second part anymore, but I don't know how to get rid of it. Tobie said "Yes, we need to fix the library. ;-)" and asked me to post this issue here.
/cc @tobie

This issue has been moved to https://github.com/w3c/web-platform-tests/issues/7151; please continue all discussion there.


fix(deps): update terraform terraform-ibm-modules/observability-agents/ibm to v1.23.0

This PR contains the following updates:

Package
Type
Update
Change

terraform-ibm-modules/observability-agents/ibm (source)
module
minor
1.22.3 -> 1.23.0

Release Notes

terraform-ibm-modules/terraform-ibm-observability-agents (terraform-ibm-modules/observability-agents/ibm)
v1.23.0
Compare Source
Features

Add Kubernetes/IKS support (#304) (21dc4c4)

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

/run pipeline

/run pipeline

:tada: This PR is included in version 1.1.9 :tada:
The release is available on:

GitHub release
v1.1.9

Your semantic-release bot :package::rocket:


Fix default for CONTACT_REPLY_BCC_EMAIL

There was a typo in the variable name for this default value.

Hi @Natureshadow
Thank you for your contribution, on this PR and the others  💯 👍🏾
The fix is corrected and will be merged.


DCOS-56234 - Do not collect logs from successful jobs.

High-level description
Do not collect logs from successful jobs.
We clean up these lines by just checking for the all_tests_passed marker file. I am not sure what the purpose of the removed lines was, but they made the logic more complicated and brittle.
Corresponding DC/OS tickets (required)

DCOS-56234 Do not collect logs from successful jobs.

@mesosphere-mergebot label Ready For Review

@mesosphere-mergebot changelog-not-required test only changes

@mesosphere-mergebot request-review @cprovencher @orsenthil @pcrews

Running my changes: https://teamcity.mesosphere.io/viewQueued.html?itemId=2054361&buildTypeId=DcOs_Open_Test_IntegrationTest_AwsOnpremWStaticBackendGroup1TerraformSpotBlock&tab=buildLog

Why do we do this? To save money? We rarely use these logs, but when we use them they are very valuable to us.

Why do we do this? To save money? We rarely use these logs, but when we use them they are very valuable to us (usually in a comparative scenario, good case vs. bad case).
I'd love to be pointed to some argument here.
I trust that you've already discussed and properly decided, and I approve. But still, this makes me sad.

This is something we've been doing forever and the main reason for it is to save on disk usage. I just broke it during the Terraform migration.
I think if we'd like to add collection of logs on successful jobs, we should open a ticket for it and make a case for why it's worth the disk usage.

Thanks for the feedback Justin.

This is something we've been doing forever and the main reason for it is to save on disk usage.

Oh. Well, not the "forever" from my perspective, but probably for a long time w/o me personally noticing.

I think if we'd like to add collection of logs on successful jobs, we should open a ticket for it and make a case for why it's worth the disk usage.

@makkes once advocated for it and I think this is a bigger topic in his team.


@dnd-kit list sorting not working properly when parent div has transform:scale() css

In one of my project there is one feature for zoom In/Out. And That I am managing with transform:scale(). But now in the child component I am using @dnd-kit sortable library to sort my blocks. but because of that transform:scale() css sorting is not working properly like cursor and block are not moving aligned.
here is my demo link: https://codesandbox.io/p/sandbox/dndkit-sorting-4gxnz7.
Anyone has any idea to resolve above issue please share your thoughts.
Thanks in advanced!

Have the same issue, and reported here: https://github.com/clauderic/dnd-kit/issues/1225.

Maybe this will help: https://github.com/clauderic/dnd-kit/issues/1225#issuecomment-1899570081


Allow for an empty instance name to be used

I would like the ability to replace the distribution specific configuration to get an installation that looks a bit more default. If that makes sense.
This Change allows for instance to be set to '', which then allows the use of standard locations for configuration files and init scripts.
Not sure though if this is something that is in line with what the Cookbook originally wants to do.
Thanks!

Hm, it looks good!
I think, that solution will help to migrate from previous major versions of mysql cookbook.

Glad to see this change, but there appears to be an issue with it not setting the initial root password when the instance name is empty. Running this on trusty ..
mysql_service '' do
port '3306'
version '5.5'
initial_root_password 'changeme'
action [:create, :start]
end
will result in the root password not being set.

I'd welcome this: having the "/etc/init.d/mysql" and other copies of init scripts or working directories lying arouund gets confusing, and the enforced use of /var/lib/mysql-[instance] instead of /var/lib/mysql/ for the mysql.sock breaks default perl scripts.

@jordant: Leaving the instance name empty doesn't work properly indeed. I wouldn't expect it to.
You'll have to define the service as:
  mysql_service 'default' do
    port '3306'
    version '5.5'
    instance ''
    initial_root_password 'changeme'
    action [:create, :start]
  end

Hi. I am an automated pull request bot named Curry. There are commits in this pull request whose authors are not yet authorized to contribute to Chef Software, Inc. projects or are using a non-GitHub verified email address. To become authorized to contribute, you will need to sign the Contributor License Agreement (CLA) as an individual or on behalf of your company. You can read more on Chef's blog.
Non-GitHub Verified Committers
There are 1 commit author(s) whose commits are authored by a non-GitHub verified email address. Chef will have to manually verify that they are authorized to contribute.
GitHub Users Who Are Not Authorized To Contribute
The following GitHub users do not appear to have signed a CLA:

@CBeerta

Please sign the CLA here.

+1 for allowing to manage the 'standard' instance.
All I need for the current project is one db service, with many databases which can all be created inside it (quite a common scenario for web apps imho).
Forcing a "non-standard" service name had the following drawbacks:

teach devs not to use the 'usual' init service to start/stop the db
set a global env var MYSQL_HOME, or all connections to 'localhost' would not work, because when you use 'localhost' mysql tries to connect via pipe, and the pipe name is different
set the same env var in cronjobs (cronjobs by default get no env)
set the name of the different pipe in 3 settings in php.ini, in 2 separate files => 6 settings!)

Hi. I am an automated pull request bot named Curry. There are commits in this pull request whose authors are not yet authorized to contribute to Chef Software, Inc. projects or are using a non-GitHub verified email address. To become authorized to contribute, you will need to sign the Contributor License Agreement (CLA) as an individual or on behalf of your company. You can read more on Chef's blog.
Non-GitHub Verified Committers
There are 1 commit author(s) whose commits are authored by a non-GitHub verified email address. Chef will have to manually verify that they are authorized to contribute.
Please sign the CLA here.

Hi. Your friendly Curry bot here. Just letting you know that all commit authors have become authorized to contribute. I have added the "Signed CLA" label to this issue so it can easily be found in the future.

So eh... does this mean that you reject this feature? With what reasons?


Fix the problem that shadowdom cannot be supported and cannot be packaged

修复无法支持shadowdom以及无法打包的问题
Fix the problem that shadowdom cannot be supported and cannot be packaged

Dear @yangdan8,
my friend and creator of this repo passed away.
If you want please submit this to https://github.com/SergioCrisostomo/v-click-outside-x, I can help maintain this project he started.


Sprint board management automation

Can't get to this right away but added a TODO I'll take a look asap, tyvm for your help @pranavgaikwad


Unexpected behavior when evaluating similar strings with .containDeep()

Replication steps
// fails
['code-for-days', 'code'].should.containDeep(['code', 'code-for-days'])

// passes
['code-for-days', 'code-fast'].should.containDeep(['code-fast', 'code-for-days'])

// also passes (ordered)
['code', 'code-for-days'].should.containDeep(['code', 'code-for-days'])

Incorrect assertion is made when similar strings are tested using this method.
It appears to me that .containDeep() is traversing strings as arrays (e.g. 'foo'[0] === 'f') and asserting as if these are nested arrays.

Yes looks like a bug. I will take a look at Friday I am on vacation now and do not have PC near me.

I found the reason, because .containDeep search in strings with .indexOf and exclude first found strings.
So in first case it take 'code-for-days'.indexOf('code') >= 0 so it will take next 'code'.indexOf('code-for-days') which will not be found. So i think simply drop .indexOf to be .equal (===). As a best matched search will be overcompicated for this task.
Thoughts?


Lower Google permissions

The app asks for permission to "View and manage the files in your Google Drive." Other apps will ask to manage the files opened with the application instead.
I'm unwilling to grant permission to all my files. I'd suggest going with the lower permissions instead.

You can change that in Settings>Advanced>Permissions.


Building causes errors

build causes errors due to not install colorama

this is due to build creating an isolated environment and importing your module without dependencies

I found a turnaround at #2

i haven't been able to get it to work (stackoverflow), so thank you for trying to help.
i merged your changes, and it still fails (https://github.com/silvncr/osutool/actions/runs/9333697215/job/25690982940).
idk how to fix it, even though i've published modules the same way before.

Ow, realy sorry

The turnaround I found was working localy since setuptools was already installed, I will please you to merge this second pull

i'm willing to try something else if you want me to

View the latest pull please

i haven't been able to get it to work (stackoverflow), so thank you for trying to help.
i merged your changes, and it still fails (https://github.com/silvncr/osutool/actions/runs/9333697215/job/25690982940).
idk how to fix it, even though i've published modules the same way before.

well... saw your other packages as qrs as jarguments, but I do not yet sees the difference

yeah, i'm using the exact same setup for those and they work fine. that's why i'm so confused about this one.

Please try merging this pull #4

Ow I see the latest workflow run, did not happen to me but, could you try adding the pytest_runner to the pip install in the workflow?

https://github.com/silvencr/osutool/edit/main/.github/workflows/python-publish.yml

https://github.com/silvncr/osutool/commit/ec49e56a11bdaf428fe98f62cefd214dc635322f
trying now

https://github.com/silvncr/osutool/actions/runs/9334155760/job/25692009749

I see, going further!, adding wheel now..

Also you could temporarily change the workflows on to push as:
on: push

so you do not have to create a release each time.

https://github.com/silvncr/osutool/actions/runs/9334229545/job/25692193983
it worked! thank you so much for your help!

Wow, my first pull request is a success, or... half a success but I am still happy.

i'm going to credit you on my stackoverflow post, too

Ow to good! maybe will I have more than one reputation now

Ow much better I had 25pts and more that 5 badges and privileges, really thanks!


Spyder 6 on Windows cannot load Spyder's icon theme --> no start

Issue Report Checklist

[ x] Searched the issues page for similar reports

[ x] Read the relevant sections of the Spyder Troubleshooting Guide and followed its advice

[ ] Reproduced the issue after updating with conda update spyder (or pip, if not using Anaconda)
no conda installed, python anstead
but restart reproduces it, with or without adminrights

[ ] Could not reproduce inside jupyter qtconsole (if console-related)

[ ] Tried basic troubleshooting (if a bug/error)

[ x] Restarted Spyder
[ ] Reset preferences with spyder --reset
[ ] Reinstalled the latest version of Anaconda
[ ] Tried the other applicable steps from the Troubleshooting Guide

[ ] Completed the Problem Description, Steps to Reproduce and Version sections below

Problem Description
installed spyder on Win10 (as domain PC with SRP).
to avoid conflicts installation in c:\prg-win, not on c:\programdata [where software shouldn#T be installed anyway]
after start there appears the spyger 6 logo, and thereafter teh error meassage:
"It was not possible to load Spyder's icon theme, so Spyder cannot start on your system.
The most probable causes for this are either that you are using a Windows version earlier than Windows 10 1803/Windows Server 2019, which is no longer supported by Spyder or Microsoft, or your system administrator has disabled font installation for non-admin users. Please upgrade Windows or ask your system administrator for help to allow Spyder to start."
Click on OK stops the program, no choices to do enything on it.
What steps reproduce the problem?

uninstall - re-install; download a fresh version, all the same

What is the expected output? What do you see instead?
expected: software would start
I see the error mesage (see above)
Paste Traceback/Error Below (if applicable)

PASTE TRACEBACK HERE

Versions

Spyder version: 6
Python version: 3.1.2
Qt version: ?
PyQt version: ?
Operating System name/version:
Windows 10 21H2

Dependencies

PASTE DEPENDENCIES HERE

Any ideas what could cause this issue, and how to get that solved ?
Thanks and Best
Albrecht

correction due to typo:
python version is 3.12.6

Hey @ITanatFR, thanks for reporting. As the message you copy/pasted here clearly says, this is a problem you need to solve on your side.
In this case it seems fonts installation for users is disabled or blocked by your IT department. So, you need to talk to them and ask them to enable it.
Let us know if that's helpful.

I have the same issue with Spyder 6. Can you tell me which font(s) need(s) to be installed? I was able to install spyder.ttf, but this does not appear to solve the issue.
I know that previous versions of Spyder depended on Font Awesome, etc. Have these dependencies been removed?

@nsgates, the fonts required by Spyder should be installed automatically, unless you're using an old Windows version or your system doesn't allow to install fonts for users (as the message says).
Please check those conditions don't apply for you and try again.

@ccordoba12, I appreciate your help. Can you tell me which fonts are required to load Spyder 6's icon theme? I can install fonts manually, but the automatic install is being blocked. Apparently, Spyder 6 needs more than just spyder.ttf, which I believe I was able to install:

Spyder 5 would fall back to the Spyder 2 icon theme when it was unable to load the Spyder 3 icon them (e.g., when Font Awesome was blocked because it was not marked trusted by Windows, https://github.com/spyder-ide/qtawesome/issues/167), which was very helpful. I am not on an old Windows version, and I would certainly enable the automatic font installs if I could.

My attempts to trace the error above triggered in spyder/app/mainwindow.py have not been successful in revealing which specific fonts are missing. How can I find out which ones these are?

In this case it isn't about fonts being missing. It seems that Spyder 6 tries to copy fonts to a local directory to read (which qtawesome does so through qtpy.QtGui.QFontDatabase.addApplicationFontFromData. If your PC has the policy enabled to block untrusted fonts then QFontDatabase will fail to add it (returning -1). One solution for this would be for qtawesome to add a check in icon_font.py load_font function that if -1 is returned, to look for the font in the system font directory - perhaps with the FontError indicating which fonts cannot be found (noting Spyder would need to be updated to display which font/s are missing). This would allow the user to install fonts with admin privileges.
Locally, the fix could look like the following in iconic_font.py "# Load font" section:
if QApplication.instance() is not None:
    with open(os.path.join(directory, ttf_filename), 'rb') as font_data:
        data = font_data.read()
        id_ = QFontDatabase.addApplicationFontFromData(data)
    font_data.close()
    # added two lines here to look for the font in the windows Font directory if using windows and font could not be loaded from binary data.
    if (id_ == -1) & (sys.platform.startswith('win') or sys.platform.startswith('cygwin')):
        id_ = QFontDatabase.addApplicationFont(os.path.join(os.environ["WINDIR"], "Fonts", ttf_filename))
    loadedFontFamilies = QFontDatabase.applicationFontFamilies(id_)
    if loadedFontFamilies:
        self.fontids[prefix] = id_
        self.fontname[prefix] = loadedFontFamilies[0]
        self.fontdata[prefix] = data
    else:
        raise FontError(u"Font at '{0}' appears to be empty. "
                        "If you are on Windows 10, please read "
                        "https://support.microsoft.com/"
                        "en-us/kb/3053676 "
                        "to know how to prevent Windows from blocking "
                        "the fonts that come with QtAwesome.".format(
                                os.path.join(directory, ttf_filename)))

    with open(os.path.join(directory, charmap_filename), 'r') as codes:
        self.charmap[prefix] = json.load(codes, object_hook=hook)

I'm not a a dev or anything, so I'm sure there is probably a better way to implement this or handle it.

@russellfrost, thanks for chiming in. You said:

In this case it isn't about fonts being missing.

Sorry but this is an incorrect assessment of the situation. The case is about missing fonts, the fonts that Spyder requires to display most of its icons.

It seems that Spyder 6 tries to copy fonts to a local directory to read (which qtawesome does so through qtpy.QtGui.QFontDatabase.addApplicationFontFromData.

It's not Spyder which tries to copy the fonts it needs but QtAwesome (just clarifying that misunderstanding because first you say Spyder and then QtAwesome)

If your PC has the policy enabled to block untrusted fonts then QFontDatabase will fail to add it (returning -1).

Which is pretty much what our error message says (see the second screenshot in https://github.com/spyder-ide/spyder/issues/22499#issuecomment-2365009893):

or your system administrator has disabled font installation for non-admin users.

One solution for this would be for qtawesome to add a check in icon_font.py load_font function that if -1 is returned, to look for the font in the system font directory

The only way those fonts will be available in the system fonts directory is if users install them manually. And that's not so simple because they come bundled with the QtAwesome package (i.e. they are not even part of Spyder).

perhaps with the FontError indicating which fonts cannot be found (noting Spyder would need to be updated to display which font/s are missing). This would allow the user to install fonts with admin privileges.

As I said, the "fonts that cannot be found" are several fonts that come with QtAwesome (not fonts that you'll find easily on the web). So, I don't see this as a good solution either.

I think the best solution would be to show a dialog to users asking them if they accept Spyder to install those fonts automatically with admin privileges, in case they can't be installed for the current user. And then change QtAwesome to check if those fonts are available in the system fonts directory, as you said. But that's too much work for something that feels like a fringe use case (because you can simply allow font installation for regular users).

You are absolutely right about the ideal solution - and that its a fringe issue. The point I was trying to make at the start about it not being about fonts missing was directly responding to nsgates who was trying to find what font was missing, more than trying to address the whole issue. My point was that manually installing the fonts (even if it is buried in site-packages) would not fix the issue at all since qtawesome is always looking to read the font data.
But I am happy to take my answer down since it isn't really addressing a fix.

I want to mention that this is a major problem for me as well. I can't use Spyder anymore because our IT policy blocks "untrusted fonts". I realize this is technically an issue with QTFontAwesome, but it's a real bummer because I found Spyder so incredibly useful. I previously asked IT to make an exception to remove the trusted font policy from Windows, but they refused to do that. Frankly, asking IT to make an exception for anything is virtually impossible at most companies or government organizations.
Can we at least publish some instructions on how to work around the problem by hacking QTFontAwesome or something else?  If we could just substitute fonts for trusted fonts, that would work for us.
The devs can claim "not my problem", but I am not aware of any other software that I use that seems to suffer from this issue. From a practical perspective, that makes Spyder an outlier in terms of not being able to run on Windows in a corporate or government environment.

I had the same issue on Windows 11 on my work computer where my Spyder 6.0.2 installed via Anaconda could not started (The splash screen would initialize and quickly disappear) when launched from Anaconda Prompt, a traceback displays that it could not find find fontawesome4-webfont-4.7.ttf from my fonts folder.
Someone else suggested it online, and it turns out it was indeed the local security policies that blocked the install of the font, so I disabled it using admin rights in the registry (Since I couldn't easily adjust the Local Security Policies via the GUI) based on this article: https://learn.microsoft.com/en-us/windows/security/operating-system-security/device-management/block-untrusted-fonts-in-enterprise#turn-on-and-use-the-blocking-untrusted-fonts-feature
After making the registry change, I:

Restarted computer
conda uninstall spyder
conda install spyder
launched spyder via Anaconda prompt

And viola - Spyder starts up properly (It works through the App icon in the Start menu also)
Hope this helps someone

@joncox123, sorry for the inconvenience. We'll try to find a workaround so that Spyder can keep working in such restricted environments.
You said:

I can't use Spyder anymore because our IT policy blocks "untrusted fonts".

So, are you able to install fonts for your own user? Or is that also blocked?

@joncox123, sorry for the inconvenience. We'll try to find a workaround so that Spyder can keep working in such restricted environments.
You said:

I can't use Spyder anymore because our IT policy blocks "untrusted fonts".

So, are you able to manually install fonts for your own user? Or is that also blocked?

Our IT policy blocks the installation of any "untrusted" fonts. Even with my pseudo-administrator account, I still can't install an untrusted font. I believe it claims to install, but it still doesn't become trusted. It seems the only way around this issue is either to make the fonts trusted, which probably requires paying $$$ to Micro$soft, or to substitute for "trusted" fonts. I think the easiest work around here would be to add some condition that would allow QT/Spyder to use standard, pre-installed Microsoft fonts in place of the open source QTFontAwesome fonts on Windows, if configured to do so in the INI (or if the failure occurs it could fall back automatically).
Is it possible to implement a condition in the Spyder INI file whereby, if it is set, Spyder instructs QT to use a set of standard, pre-installed, trusted Microsoft fonts when running on Windows, instead of the QTFontAwesome fonts? Maybe this is an option in QT.

It seems the only way around this issue is either to make the fonts trusted, which probably requires paying $$$ to Micro$soft, or to substitute for "trusted" fonts.

Do you know how to make fonts trusted? We could pay for that if necessary.

This is what ChatGPT says:

Ok, then download one of the fonts from here and put it in a directory inside C:\Users\<your user>. Then run the last script you posted, but with path/to/your/font.ttf adjusted to where you placed the font.
My guess is that that will fail because the font is not properly installed and not trusted.

@ccordoba12 I can look into the process for getting fonts signed with a certificated from a CA and submitted to MS to be trusted. I'm willing to donate a few dollars to the cause, hopefully others are as well. I'll let you know if I get anywhere.

I had the same issue on Windows 11 on my work computer where my Spyder 6.0.2 installed via Anaconda could not started (The splash screen would initialize and quickly disappear) when launched from Anaconda Prompt, a traceback displays that it could not find fontawesome4-webfont-4.7.ttf from my Windows fonts folder.
Someone else suggested it online, and it turns out it was indeed the local security policies that blocked the install of the font, so I disabled it using admin rights in the registry (Since I couldn't easily adjust the Local Security Policies via the GUI) based on this article: https://learn.microsoft.com/en-us/windows/security/operating-system-security/device-management/block-untrusted-fonts-in-enterprise#turn-on-and-use-the-blocking-untrusted-fonts-feature
After making the registry change, I:

Restarted computer
conda uninstall spyder
conda install spyder
launched spyder via Anaconda prompt

And viola - Spyder starts up properly (It works through the App icon in the Start menu also) Hope this helps someone

@deltaex1, you are amazing. I was able to follow your approach and I now have Spyder 6 working! In addition, Spyder 5 now displays all of the icons instead of red X's.
Using the registry with admin rights to modify the local security policy and stop blocking untrusted fonts was the key. I was able to access this setting using the Group Policy GUI, but for some reason it showed that untrusted font blocking was disabled. However, when I viewed the event log I saw many instances of untrusted fonts being blocked, and the registry edit did the trick. One note I'll add is that I couldn't find a way to set the MitigationOptions key base to Hexadecimal (as the Microsoft support link instructs), so I had to convert from binary to hex and vice versa to implement the fix.
@russellfrost, thank you for your helpful comment that manually installing the fonts would not fix the issue. I definitely experienced this for myself, and thought I had missed something there for a while.
@joncox123, thank you for returning attention to this issue. I echo your comments about the IT limitations of corporate and government environments. In my case, I was able to solve the issue using temporary admin rights. I hope you too are able to find a way to resolve it. Does your pseudo-admin account allow you to do registry edits?
Lastly, thank you, @ccordoba12, for adding this issue to a future milestone, and more importantly for your dedication to developing Spyder these many years. All of these comments are testaments to the value and usefulness of this tool.

@nsgates Glad I was able to help! For posterity, I simply changed the 1 to a 2 in Binary (Figured it was equivalent) instead of converting binary -> hex to change the setting to disable to blocking untrusted fonts:

Thanks @joncox123 @ccordoba12 @russellfrost for all of your efforts in troubleshooting!

Thanks for your work on this.  After some troubleshooting with my IT department, I'm not able to use Spyder 6 as the registry change suggested by deltaex1 is overwritten on reboot and the fix is only temporary in my environment.  Thanks again for you work for those of us 'edge cases' with limited permissions.


Change props options to json url

How to load json url in options props?

Hi, @atokge, you can't use raw JSON. You need to parse JSON to the array. (JSON.parse method can help you)


fix install step

; was missing in deps.ts, causing the install step to fail for me

Thanks for making a PR!
So  deno install --allow-read --allow-write -fn deno-init<EMAIL_ADDRESS>is failing for you? I cannot reproduce this error. Can you share your output of deno --version?

tl;dr - my bad! it is because my deno version was way out of date (1.9.2). I assumed brew upgrade deno would switch to 1.20 like I expected, but deno upgrade did for me instead
fwiw: here's my deno -version output when running 1.9.2
deno 1.9.2 (release, x86_64-apple-darwin)
v8 9.1.269.5
typescript 4.2.2

and the commandline output when copy&pasting the install command with 1.9.2
benmedcalf@Bens-MBP rpdbot % deno install --allow-read --allow-write -fn deno-init<EMAIL_ADDRESS>error: Expected ;, got assert at https://deno.land/x/init@v2.4.4/deps.ts:4:74

Feel free to close this if you'd like

Thought it might be something along those lines :). The code uses a JSON import in deps.ts which was first supported in Deno 1.17 I think. I will update the readme to be more clear about this version requirement. Thanks for pointing this out!


Migrate config.yaml into test tree

and provide another config.yml or even a set of examples in the git repo instead.

[x] moved config.yaml into an examples directory with multiple files
[x] enhanced the config parser to load multiple config paths (files or directories of *.yaml)


Can potree converter extract 360 images from a point cloud file?

Is possible get the 360 images from e57 file?

Hello, not possible with potreeConvert, but their are othe tool which you can use to do that. am aslo in a process of developer a tool to that and prepare coordinate.txt files need

What tool?

If your pointcloud are in e57 format you can use pye57 to extract the images stored, in my case blk360 scans gives 6 cube faces which i have to convert to equirectangle.


Feature/toast-fix

Made some quick fixes to the layout styling for toasters.
Add Variant to ToastOptions
Add Variant Helpers

Everything should now appear correctly.

Hi @aceofwings
I tried to use it in an app, but this doesn't work:
import BToastPlugin from 'bootstrap-vue-3/plugins/BToast'

Would you please update the usage doc with correct syntax for 3rd party apps?

Toast Components can be used standalone or part of the toast plugin. What's the difference? What is standalone?

Sorry about that, Due to how boostrap-vue-3 can't be directly imported into the docs I opted for  a different way for fetching the plugin instance. The correct way is now `import {useToast} from './components/BToast/plugin'
You can currently use the BToast component and provide a true/false value to the modal to hide and show.
If no title is set.... it still show a title. This can easily be updated to behave that way.

Apologies for some of the missing stuff.

Looks like the way the library is done. It should resolve to 'boostrap-vue-3/components/ToastPlugin'. Ill test it out.
Ill look at refactoring for plugins so that it can be included as `bootstrap-vue-3/ToastPlugin'

How would the usage be in each case and which one do you recommend?

I can default a global property in prime vue

You mean 'like prime vue' ? Now I use prime vue like this:
main.ts
import Toast from "primevue/toast"
import ToastService from "primevue/toastservice"
...
    app.use(ToastService)
    app.use(router)
    app.component("Toast", Toast)

app.vue
<template>
    <div>
        <router-view />
        <Toast position="bottom-center" />
    </div>
</template>

<script setup>
import { useToast } from 'primevue/usetoast'
const toast = useToast()

toast.add({ severity: 'info', summary: 'Message 1', detail: 'Message 1 Content', life: 3000 })

toast.add() is called only in app.vue by an event listener. All other modules just emit the event to show toast with some properties.


fixed reassignment of stdout in client mode

removed pointlessly doubled stdout reassignment that used non-portable way of assinment as already solved in #168
fixed writing pcapng to stdout in client mode

fixes #171

Thanks, merged.


Throttle metadata CPU usage

Set cpuShares for the metadata container to less than the default 1024.
More details to come
Rancher versions:
rancher/server:
rancher/agent:
Infrastructure Stack versions:
healthcheck:
ipsec:
network-services:
scheduler:
kubernetes (if applicable):
Docker version: (docker version,docker info preferred)
Operating system and kernel: (cat /etc/os-release, uname -r preferred)
Type/provider of hosts: (VirtualBox/Bare-metal/AWS/GCE/DO)
Setup details: (single node rancher vs. HA rancher, internal DB vs. external DB)
Environment Template: (Cattle/Kubernetes/Swarm/Mesos)
Steps to Reproduce:
Results:

Work is being tracked here: https://github.com/rancher/rancher/issues/9957
Going to close this out for housekeeping.


Run GPM behind a reverse proxy using a subpath

I'd like to run GPM behind my traefik reverse proxy using a subpath like https://mydomain.com/gpm. Unfortunately this does not seem to be supported right now, as the main page tries to load resources from the root-path of the domain.
I'm not very familiar with web development, but I think this can be fixed by simply using
Relative path: <script defer="defer" src="./static/js/main.9d2388a4.js"></script> (note the dot in src)
instead of
Absolute path: <script defer="defer" src="/static/js/main.9d2388a4.js"></script> (current code)
when referencing resources.

Hi @monsdar
Just to confirm that yes, GPM doesn't work correctly under a subpath. There are a few places that need to be tweaked for this to work. We'll keep it in the radar for the next releases.


Update dockerfile

Switch to new stable version of Debian - buster
Skip updating local package database since no new packages are installed or upgraded. It only increases image size.

Thank you! 😊


Always respect ZephyrBuildConfiguration_ROOT

The default mechanism for finding the ZephyrBuildConfig.cmake is this:
find_package(ZephyrBuildConfiguration
  QUIET NO_POLICY_SCOPE
  NAMES ZephyrBuild
  PATHS ${ZEPHYR_BASE}/../*
  NO_CMAKE_PATH
  NO_CMAKE_ENVIRONMENT_PATH
  NO_SYSTEM_ENVIRONMENT_PATH
  NO_CMAKE_PACKAGE_REGISTRY
  NO_CMAKE_SYSTEM_PATH
  NO_CMAKE_SYSTEM_PACKAGE_REGISTRY
)

This can be very annoying, since the ${ZEPHYR_BASE}/../* is hard-coded and, depending on the project structure, it will just find some random ZephyrBuildConfig.cmake file, since there is no way to override the hard-coded path.
I would propose something like this:
if (NOT ZephyrBuildConfiguration_ROOT)
  set(ZephyrBuildConfiguration_ROOT "${ZEPHYR_BASE}/../*")
endif()

find_package(ZephyrBuildConfiguration
  QUIET NO_POLICY_SCOPE
  NAMES ZephyrBuild
  NO_CMAKE_PATH
  NO_CMAKE_ENVIRONMENT_PATH
  NO_SYSTEM_ENVIRONMENT_PATH
  NO_CMAKE_PACKAGE_REGISTRY
  NO_CMAKE_SYSTEM_PATH
  NO_CMAKE_SYSTEM_PACKAGE_REGISTRY
)

That way it will always respect the given root directory, even if it finds no file at that root directory. In my opinion, this a better default, because when you specify a root directory, you probably don't expect it to also search the default path.
And if you insist on the old mechanism; just set ZephyrBuildConfiguration_ROOT to <mycustomfirstguess>;${ZEPHYR_BASE}/../*.

@tejlmand Please take a look at my PR. It's only 5 changed lines :P If you would like some additional documentation about this change I can add it as well.

This can be very annoying, since the ${ZEPHYR_BASE}/../* is hard-coded and, depending on the project structure, it will just find some random ZephyrBuildConfig.cmake file,

not some random ZephyrBuildConfig.cmake, but a ZephyrBuildConfig.cmake within your Zephyr workspace, ref: https://docs.zephyrproject.org/latest/develop/west/workspaces.html
And the reason for this lookup is to allow users or high-level projects to provide a configuration, similar in nature to a  like how the .zephyrrc is provided, ref: https://docs.zephyrproject.org/latest/build/zephyr_cmake_package.html#zephyr-build-configuration-cmake-package
Zephyr itself doesn't provide a default ZephyrBuildConfig.cmake file, so either you have created such a file yourself or the project on which you are working on is providing such a file.
If the latter, and the file in question doesn't meet your needs or expectations, then you have the possibility of providing your own file, as you seem to be aware.
If you want to able to completely disable the provided config file, even though it is provided by the downstream project, then you should probably request the project in question to support an opt-out option.

since there is no way to override the hard-coded path.

As you already noticed, there is the possibility to point ZephyrBuildConfiguration_ROOT to a different location which will be searched before the default paths.
CMake v3.27 even allows you to set <package>_ROOT in your environment, meaning you won't need to apply it on command-line on each build.

you probably don't expect it to also search the default path.

trying the default / hard-coded guess if <package>_ROOT fails is the default CMake behavior, and so far I have seen no compelling arguments that we should deviate from what CMake does, ref: https://cmake.org/cmake/help/latest/command/find_package.html#config-mode-search-procedure.

And if you insist on the old mechanism; just set ZephyrBuildConfiguration_ROOT to <mycustomfirstguess>;${ZEPHYR_BASE}/../*.

and that is the case today, the ZephyrBuildConfiguration_ROOT is searched before any hard-coded guesses.

If there is a need for an enhancement to the current functionality then please provide a good use-case.
Also note that the ZephyrBuildConfig.cmake is intended for settings handling, as the name implies.
A project can in principle do anything it likes in such file, like target creation, but that would be a misuse of the intention of the feature.
As the example shows: https://docs.zephyrproject.org/latest/build/zephyr_cmake_package.html#zephyr-build-configuration-cmake-package
such use could be setting of default compiler if none is already defined.
Or adding extra BOARD_ROOT for projects (although doing so in a Zephyr module is a better approach).
If a project is doing something foolish, then it's probably better to make a request towards the project itself for an improved build config file.

@tejlmand Something "foolish" is using the recommended T3: Forest Topology. Suppose you have a something like this (basically a copy from the T3 doc):
t3_workspace
├── app1
│   ├── CMakeLists.txt
│   └── ZephyrBuildConfig.cmake
├── app2
│   └── CMakeLists.txt
│
└── zephyr

app1 is happy, since it doesn't even need to set a ZephyrBuildConfiguration_ROOT. But app2 now has to actively deter CMake from finding ZephyrBuildConfig.cmake and being applied to app2. It will always find it, except I implement measures to prevent that from happening like:

Create own (dummy) ZephyrBuildConfig.cmake and set ZephyrBuildConfiguration_ROOT (this is the most secure way currently!)
Use CMAKE_DISABLE_FIND_PACKAGE_ZephyrBuildConfiguration until I need to use such a file myself and then option above applies.
Tell app1 to move its ZephyrBuildConfig.cmake to an unintuitive folder which is not part of CMake's find paths and set ZephyrBuildConfiguration_ROOT on its own.
Various even worse hacks... (or I forgot an easy elegant solution; please tell me).

My initial idea was to add this to every application as a way to "namespace" the search:
get_filename_component(ZephyrBuildConfiguration_ROOT ${CMAKE_CURRENT_LIST_DIR} ABSOLUTE)
find_package(Zephyr REQUIRED HINTS $ENV{ZEPHYR_BASE})

But then I realized this is not helping since the default path still applies...
So I made a PR with the idea to keep the current, but in my opinion flawed, default path intact, while still allowing this kind of namespacing, to not make breaking changes.
But maybe this was the wrong proposition... Maybe we should just delete the default path altogether or replace it with a more sensible default, like instead of a *-search, just search in the application root or just the manifest repository to allow global configuration. I don't see a reason a user/an app should have to fight against the inclusion of a custom build configuration, which should be reserved for advanced users, as you yourself implied above.

@nylnx thanks for your comment, it helps a lot in understanding what your intentions are.

t3_workspace
├── app1
│   ├── CMakeLists.txt
│   └── ZephyrBuildConfig.cmake

This structure helps a lot in understanding what you try to achieve.
The ZephyrBuildConfig.cmake was not intended to be on a per application build configuration, but more high level settings / environment kind of settings.
See it as a possibility to set "environment" settings without users requiring to do a source zephyr-env.sh as described here: https://docs.zephyrproject.org/latest/develop/env_vars.html#option-3-using-zephyrrc-files
The zephyr-env.sh would be identical across multiple applications, just like the ZephyrBuildConfig is intended to be used across applications.
But I do see that the name "BuildConfiguration" can give the impression that it can be used for individual applications.
The Zephyr build config is a very open-ended feature and allows users a lot of freedom.
Meaning that you are of course free to structure your apps as you which and use the feature as it suits you.
Let me give you a proposal on how to achieve what you want.
Create your structure as this:
t3_workspace
├── app1
│   ├── CMakeLists.txt
│   ├── ZephyrBuildConfig.cmake
│   └── ZephyrBuildConfigVersion.cmake

and inside ZephyrBuildConfigVersion.cmake you write:
# Only source Build configuration if building app1
if(NOT "${CMAKE_CURRENT_LIST_DIR}" STREQUAL "${CMAKE_SOURCE_DIR}")
  set(PACKAGE_VERSION_UNSUITABLE TRUE)
endif()

That will ensure that the corresponding ZephyrBuildConfig.cmake is sourced only when building for app1.
And app2 will be happy to, as it don't need to care about app1/ZephyrBuildConfig.cmake cause it will not be sourced when building app2.

and as bonus, no sample needs to set ZephyrBuildConfiguration_ROOT 😉

@tejlmand Thank you for your answer and for the solution you gave!
But is this really the best we can do? A separate dummy version file for every ZephyrBuildConfig.cmake there is...
I still feel there might be a place for a ZephyrAppConfig.cmake which is application-spaced. I do see now that ZephyrBuildConfig.cmake is (workspace-)global and should apply to all/most applications. But I think there are valid use-cases for both of those and a clear separation between those two options would make it clearer and easier to use in general.
How do you see this? Do you, as a collaborator/maintainer, think adding ZephyrAppConfig.cmake would make sense or just add unnecessary/additional complexity? Is the problem even a valid one?
I would be willing to help with the PR, but would certainly be a bit more involved than the current PR.

I still feel there might be a place for a ZephyrAppConfig.cmake which is application-spaced. I do see now that ZephyrBuildConfig.cmake is (workspace-)global and should apply to all/most applications. But I think there are valid use-cases for both of those and a clear separation between those two options would make it clearer and easier to use in general.

yes, I believe a dedicated app config can be a valid feature, and I also believe it will be good to keep the two features independent. It gives a clear separation between the high level config, and the app specific.
A dedicated app config also allows us to improve an example like this:
https://github.com/zephyrproject-rtos/zephyr/blob/001795b24c980df3a3dfc0e7b385f93a9392d4f9/samples/application_development/out_of_tree_board/CMakeLists.txt#L4-L6
Having such setting in a app config will make config cleaner as well as solve a problem with sysbuild not knowing about app specific board roots. Sysbuild would be able to source such app config independent of the app's CMakeLists.txt file.
Just as it would allow Zephyr package helper to also be able to source such file and be aware of any extra settings.

I would be willing to help with the PR, but would certainly be a bit more involved than the current PR.

that would be great, and I think such a solution will mostly be related to updating the docs, as the actual sourcing of such config will be similar to existing ZephyrBuildConfig code.
FYI @57300 @nordicjm

@jhedberg It's not a bug as @tejlmand stated above. At this point it's basically an RFC. But I think I will close the current linked PR, open a new one and change the title of this issue to reflect that somewhat. I just haven't had the time recently to create a fitting PR as discussion basis.

@jhedberg It's not a bug as @tejlmand stated above. At this point it's basically an RFC. But I think I will close the current linked PR, open a new one and change the title of this issue to reflect that somewhat. I just haven't had the time recently to create a fitting PR as discussion basis.

All issues need a category. This was processed in yesterday's release engineering meeting where issues are looked at only quite superficially, which then resulted in the bug categorisation. "RFC" is also a valid category, so I've now added that label and removed the bug label.


ListView Blank Top

Is this a bug report?
Yes
Have you read the Contributing Guidelines?
Yes
Steps to Reproduce

Launch the app.
Scroll down
Set data ListView

tôi có 1 data khoảng 8 dòng trên listview. khi tôi kéo thanh scroll xuống cuối và gán lại data thành 1 dòng (lọc dữ liệu), sau đó gán lại như cũ thì thanh scroll bị 1 khoản trống, nhưng khi chạm và kéo nhẹ thanh scroll thì nó bình thường trở lại.

By Google translate
I have 1 data about 8 lines on the listview. When I drag the scroll bar to the end and reassign the data to a line (filtered data), then reassign the same scroll bar is empty, but when touching and scrolling light scroll bar it is normal again.

Please see at: http://recordit.co/NLWbIAOvrl
Environment
react-native-cli: 1.0.0
react-native: 0.47.1 and 0.48.0-rc.0
node: v7.4.0
npm: 4.1.1

Target Platform: android

Development Operating System: window 7

Build tools: 25.0.1
Thanks for helping!

Thanks for opening the issue! It does not appear like a community member will be able to reliably reproduce this issue. This may be for several reasons; perhaps it affects a particular app but a minimal repro has not been provided, or the issue may be sporadic. As it happens, we need a concrete set of steps that can demonstrably reproduce the issue as this will allow your fellow community members to validate a fix. We'll close the issue for now, but feel free to submit a new issue once you're able to reliably reproduce the issue locally. Thanks for your understanding!


With subscription

Hey dude, Jed and I were playing with this on Friday and we found that the 'function as children' pattern lead to us doing lots of logic in the render method, which was messy. It also felt messy having to pass a state container all over the place if we wanted to break the rendering down into separate functions.
This HOC allows you to map containers to props which means that state is accessible anywhere in your component, not just in the render.

I'll add types, tests etc. later if this is something you'll accept. Also happy to bikeshed the name.

Exactly!
https://twitter.com/thadeuluz/status/966787300779806725?s=21

Looks good to me besides some minor nitpicks, I really like that this allows easy access to containers in other methods than render 👍

I don't see what this improves, if you need the containers outside of the render method you can re-wrap your component:
class Stopwatch extends React.Component {
  componentDidMount() {
    this.stopwatch.start();
  }

  componentWillUnmount() {
    this.stopwatch.stop();
  }

  render() {
    // ...
  }
}

export default () => {
  return (
    <Subscribe to={[StopwatchContainer]}>
      {(stopwatch) => <Stopwatch stopwatch={stopwatch}/>}
    </Subscribe>
  );
};

I don't see that as something that needs to get extracted away so that we have 2 apis instead of one

IMHO id rather have only 'connect' than 'Subscribe'.

That only adds more weight, and you can easily create your own higher-order component if that's what you choose. It's more work to go the other way around.


pull request for e2e test at 01/27/2017 00:15:12

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Min capacity constraint not working

Language: Python
Python Package version : ortools==8.1.8487
Which solver are you using (e.g. CP-SAT, Routing Solver, GLOP, BOP, Gurobi)
Routing Solver
What did you do?
Steps to reproduce the behavior:

Go to '...'

Hi,
The minimum capacity constraint doesn't seem to be implemented by the solver, In this particular example I have used SetCumulVarSoftLowerBound to add a minimum capacity soft constraint to each of my vehicle due to which if any vehicle doesn't visit certain number of nodes then a penalty cost should be added to the objective cost. But in my case I have used 5 vehicles and the fifth vehicle doesn't visit any node but still no penalty cost is added to the objective cost. I have attached a google colab link to a mini problem i created, kindly go through it.
Google Colab Link: https://colab.research.google.com/drive/1zrBibaeAZErSuihjwSvlXeDNSHOZ1PwT?usp=sharing
Kindly let me know if this is a bug and if there is any alternative to achieve this functionality.
Edit: I apologize, the colab link wasn't public. You can visit it now. @Mizux
Thank you in Advance
Anything else we should know about your project / environment

By default empty vehicles are not considered, in your case simply add:
    for vehicle_id in range(data['num_vehicles']):
        routing.ConsiderEmptyRouteCostsForVehicle(True, vehicle_id)

note: can also be added to your existing loop
potential output
Route for vehicle 0:
 0 Load(0) ->  8 Load(8) ->  0 Load(8)
Distance of the route: 616m
Load of the route: 8

Route for vehicle 1:
 0 Load(0) ->  13 Load(4) ->  15 Load(12) ->  11 Load(13) ->  12 Load(15) ->  0 Load(15)
Distance of the route: 1552m
Load of the route: 15

Route for vehicle 2:
 0 Load(0) ->  10 Load(2) ->  16 Load(10) ->  14 Load(14) ->  9 Load(15) ->  0 Load(15)
Distance of the route: 1552m
Load of the route: 15

Route for vehicle 3:
 0 Load(0) ->  7 Load(8) ->  0 Load(8)
Distance of the route: 388m
Load of the route: 8

Route for vehicle 4:
 0 Load(0) ->  4 Load(4) ->  3 Load(6) ->  1 Load(7) ->  2 Load(8) ->  6 Load(12) ->  5 Load(14) ->  0 Load(14)
Distance of the route: 2464m
Load of the route: 14

Total distance of all routes: 6572m
Total load of all routes: 60
The objective cost value is: 6572


[K8s] API Servers Restart Script

Script to restart all API servers based on K8s in staging

Can one of the admins verify this patch?