ViewExecution.viewHive fails to overwrite partitions

The following issues were faced while overwriting partitions using viewHive (with append mode as false). eg. viewHive(conf.transformationTable, pipe, false). These issues are related to the fix for CommBank/maestro#382. etl-util version used is 1.14.0-20150625002151-e89a7bc.

If the table location doesn't exist, viewHive will fail with a ResultException as shown below. This occurs as Line:101 in HiveTable.scala tries to list all the files inside a directory that does not exist. But unit test is not getting failed in this case (which is not expected), the failure is seen only while running the jar. But while trying to explicitly list the same directory (outside viewHive), test fails as expected (Line:123 in CarBaseExecution.scala). The code is pushed into stash repo etl.commsee-car (branch: car_phase2).

MSCK REPAIR TABLE (Line:122 in HiveTable.scala) is trying to repair the table in default database instead of actual database. This causes NoSuchObjectException (exception shown below) while running the jar, but the job finishes with status 0.

As per the current approach, it seems MSCK REPAIR TABLE is not needed, as no partitions are added or deleted while overwriting. Only the files within a partition are removed and added, which does not require MSCK REPAIR TABLE query. In the above case 2, the partitions were modified as expected even though this query has failed.

The current approach could fail if there is no data to overwrite in an existing partition. In that case, no parquet files will be created, but the existing files in the partition needs to be removed, which may not happen as per the current approach.

The following exception occurs if the table location doesn't exist.
15/07/15 11:23:10 ERROR car.CseCcaVerifiedItemsExecution$: error running execution
au.com.cba.omnia.maestro.scalding.ResultException: HDFS operation failed: '<table_location>' must be a directory!
	at au.com.cba.omnia.maestro.scalding.RichExecutionObject.fromHdfs(RichExecution.scala:53)
	at au.com.cba.omnia.maestro.hive.PartitionedHiveTable$$anonfun$writeExecution$2.apply(HiveTable.scala:101)
	at au.com.cba.omnia.maestro.hive.PartitionedHiveTable$$anonfun$writeExecution$2.apply(HiveTable.scala:98)
	at com.twitter.scalding.Execution$FlatMapped$$anonfun$runStats$2$$anonfun$3.apply(Execution.scala:268)
	at com.twitter.scalding.Execution$FlatMapped$$anonfun$runStats$2$$anonfun$3.apply(Execution.scala:267)
	at scala.util.Success$$anonfun$map$1.apply(Try.scala:236)
	at scala.util.Try$.apply(Try.scala:191)
	at scala.util.Success.map(Try.scala:236)
	at scala.concurrent.Future$$anonfun$map$1.apply(Future.scala:235)
	at scala.concurrent.Future$$anonfun$map$1.apply(Future.scala:235)
	at scala.concurrent.impl.CallbackRunnable.run(Promise.scala:32)
	at scala.concurrent.impl.ExecutionContextImpl$AdaptedForkJoinTask.exec(ExecutionContextImpl.scala:121)
	at scala.concurrent.forkjoin.ForkJoinTask.doExec(ForkJoinTask.java:260)
	at scala.concurrent.forkjoin.ForkJoinPool$WorkQueue.pollAndExecAll(ForkJoinPool.java:1253)
	at scala.concurrent.forkjoin.ForkJoinPool$WorkQueue.runTask(ForkJoinPool.java:1346)
	at scala.concurrent.forkjoin.ForkJoinPool.runWorker(ForkJoinPool.java:1979)
	at scala.concurrent.forkjoin.ForkJoinWorkerThread.run(ForkJoinWorkerThread.java:107)```

The following error occurs while running MSCK REPAIR TABLE query.
15/07/13 11:38:54 ERROR metadata.Hive: NoSuchObjectException(message:default.CSE_CCA_APP_transformation table not found)
	at org.apache.hadoop.hive.metastore.api.ThriftHiveMetastore$get_table_result$get_table_resultStandardScheme.read(ThriftHiveMetastore.java:29338)
	at org.apache.hadoop.hive.metastore.api.ThriftHiveMetastore$get_table_result$get_table_resultStandardScheme.read(ThriftHiveMetastore.java:29306)
	at org.apache.hadoop.hive.metastore.api.ThriftHiveMetastore$get_table_result.read(ThriftHiveMetastore.java:29237)
	at org.apache.thrift.TServiceClient.receiveBase(TServiceClient.java:78)
	at org.apache.hadoop.hive.metastore.api.ThriftHiveMetastore$Client.recv_get_table(ThriftHiveMetastore.java:1036)
	at org.apache.hadoop.hive.metastore.api.ThriftHiveMetastore$Client.get_table(ThriftHiveMetastore.java:1022)
	at org.apache.hadoop.hive.metastore.HiveMetaStoreClient.getTable(HiveMetaStoreClient.java:1008)
	at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:57)
	at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.lang.reflect.Method.invoke(Method.java:606)
	at org.apache.hadoop.hive.metastore.RetryingMetaStoreClient.invoke(RetryingMetaStoreClient.java:90)
	at com.sun.proxy.$Proxy19.getTable(Unknown Source)
	at org.apache.hadoop.hive.ql.metadata.Hive.getTable(Hive.java:999)
	at org.apache.hadoop.hive.ql.metadata.Hive.getTable(Hive.java:973)
	at org.apache.hadoop.hive.ql.metadata.HiveMetaStoreChecker.checkTable(HiveMetaStoreChecker.java:187)
	at org.apache.hadoop.hive.ql.metadata.HiveMetaStoreChecker.checkMetastore(HiveMetaStoreChecker.java:97)
	at org.apache.hadoop.hive.ql.exec.DDLTask.msck(DDLTask.java:1968)
	at org.apache.hadoop.hive.ql.exec.DDLTask.execute(DDLTask.java:349)
	at org.apache.hadoop.hive.ql.exec.Task.executeTask(Task.java:153)
	at org.apache.hadoop.hive.ql.exec.TaskRunner.runSequential(TaskRunner.java:85)
	at org.apache.hadoop.hive.ql.Driver.launchTask(Driver.java:1516)
	at org.apache.hadoop.hive.ql.Driver.execute(Driver.java:1283)
	at org.apache.hadoop.hive.ql.Driver.runInternal(Driver.java:1101)
	at org.apache.hadoop.hive.ql.Driver.run(Driver.java:924)
	at org.apache.hadoop.hive.ql.Driver.run(Driver.java:914)
        at au.com.cba.omnia.ebenezer.scrooge.hive.Hive$$anonfun$query$1.apply(Hive.scala:256)
	at au.com.cba.omnia.ebenezer.scrooge.hive.Hive$$anonfun$query$1.apply(Hive.scala:249)
	at au.com.cba.omnia.ebenezer.scrooge.hive.Hive.run(Hive.scala:61)
	at au.com.cba.omnia.maestro.scalding.RichExecutionObject$$anonfun$fromHive$1$$anonfun$apply$3.apply(RichExecution.scala:82)
	at au.com.cba.omnia.maestro.scalding.RichExecutionObject$$anonfun$fromHive$1$$anonfun$apply$3.apply(RichExecution.scala:81)
	at com.twitter.scalding.Execution$FutureConst$$anonfun$runStats$1$$anonfun$1.apply(Execution.scala:252)
	at com.twitter.scalding.Execution$FutureConst$$anonfun$runStats$1$$anonfun$1.apply(Execution.scala:252)
	at scala.util.Try$.apply(Try.scala:191)
	at com.twitter.scalding.Execution$FutureConst$$anonfun$runStats$1.apply(Execution.scala:252)
	at com.twitter.scalding.Execution$FutureConst$$anonfun$runStats$1.apply(Execution.scala:251)
	at com.twitter.scalding.Execution$MapEvalCache.getOrElseInsert(Execution.scala:225)
	at com.twitter.scalding.Execution$FutureConst.runStats(Execution.scala:251)
	at com.twitter.scalding.Execution$FlatMapped$$anonfun$runStats$2$$anonfun$3.apply(Execution.scala:269)
	at com.twitter.scalding.Execution$FlatMapped$$anonfun$runStats$2$$anonfun$3.apply(Execution.scala:267)
	at scala.util.Success$$anonfun$map$1.apply(Try.scala:236)
	at scala.util.Try$.apply(Try.scala:191)
	at scala.util.Success.map(Try.scala:236)
	at scala.concurrent.Future$$anonfun$map$1.apply(Future.scala:235)
	at scala.concurrent.Future$$anonfun$map$1.apply(Future.scala:235)
	at scala.concurrent.impl.CallbackRunnable.run(Promise.scala:32)
	at scala.concurrent.impl.ExecutionContextImpl$AdaptedForkJoinTask.exec(ExecutionContextImpl.scala:121)
	at scala.concurrent.forkjoin.ForkJoinTask.doExec(ForkJoinTask.java:260)
	at scala.concurrent.forkjoin.ForkJoinPool$WorkQueue.runTask(ForkJoinPool.java:1339)
	at scala.concurrent.forkjoin.ForkJoinPool.runWorker(ForkJoinPool.java:1979)
	at scala.concurrent.forkjoin.ForkJoinWorkerThread.run(ForkJoinWorkerThread.java:107)

15/07/13 11:38:54 INFO log.PerfLogger: </PERFLOG method=runTasks start=1436751534740 end=1436751534899 duration=159 from=org.apache.hadoop.hive.ql.Driver>
15/07/13 11:38:54 INFO log.PerfLogger: </PERFLOG method=Driver.execute start=1436751534729 end=1436751534899 duration=170 from=org.apache.hadoop.hive.ql.Driver>
15/07/13 11:38:54 INFO ql.Driver: OK
15/07/13 11:38:54 INFO log.PerfLogger: <PERFLOG method=releaseLocks from=org.apache.hadoop.hive.ql.Driver>
15/07/13 11:38:54 INFO log.PerfLogger: </PERFLOG method=releaseLocks start=1436751534900 end=1436751534900 duration=0 from=org.apache.hadoop.hive.ql.Driver>
15/07/13 11:38:54 INFO log.PerfLogger: </PERFLOG method=Driver.run start=1436751533753 end=1436751534900 duration=1147 from=org.apache.hadoop.hive.ql.Driver>
15/07/13 11:38:54 INFO log.PerfLogger: <PERFLOG method=releaseLocks from=org.apache.hadoop.hive.ql.Driver>
15/07/13 11:38:54 INFO log.PerfLogger: </PERFLOG method=releaseLocks start=1436751534907 end=1436751534907 duration=0 from=org.apache.hadoop.hive.ql.Driver>
15/07/13 11:38:54 INFO zookeeper.ZooKeeper: Session: 0x34e3507875aaa5e closed
15/07/13 11:38:54 INFO zookeeper.ClientCnxn: EventThread shut down
15/07/13 11:38:54 INFO Configuration.deprecation: io.bytes.per.checksum is deprecated. Instead, use dfs.bytes-per-checksum
Exited job au.com.cba.omnia.etl.commsee.car.CseCcaAppExecution (0.2.0-20150713004736-e43d88f-SNAPSHOT) at Mon Jul 13 11:39:00 AEST 2015 with status 0```

Are you sure a repair table is not needed? The files that make up a partition have changed.
With regards to 4, if no data is written how do we know which partition has changed? This is to overwrite specific partitions in an existing table not the whole table. If you don't write any data how do we know which partition to overwrite?
If you want to get rid of all the previous data you should delete the table beforehand?

@SajeevAr I also got error mentioned in case 1, when a table already existed and corresponding HDFS location was not there. But I am able to create a new table and load data in overwrite mode successfully. Could you please confirm  whether you tried to  create a new hive table

@SajeevAr do you know why the underlying directory has been deleted?

@stephanh In my understanding, MSCK REPAIR TABLE will add metadata about partitions to Hive metastore, for partitions for which such metadata doesn't already exist. So if new partitions are not added and only if the files within the partitions have changed, I think MSCK REPAIR TABLE is not needed while using Hive. I was referring to 'Recover Partitions (MSCK REPAIR TABLE)' section in https://cwiki.apache.org/confluence/display/Hive/LanguageManual+DDL
But I'm not quite sure how this behaves with ViewExecution.viewHive. This may require further analysis.
I agree with you on Case 4. Though I don't have a specific use case to explain, I just thought about a situation where we have to overwrite some existing partitions (not the whole table), when the input pipe can be empty. I was wondering if such scenarios could be handled.

@jithinjohn Yes, this issue happens only while loading data into a table in overwrite mode, whose external location does not exist.
@stephanh As part of testing, the HDFS directories may be cleared without dropping the table and may reload the table with new data. The issue was noted because the same scenario was working fine in the previous implementation.

No we can't handle 4. Without information how do we know which partitions you want to override?

@SajeevAr I think I fixed the bugs. I ended up removing MSCK REPAIR TABLE if the hive documentation is correct then you are right and we don't need it. Can you please the latest version and let me know if it fixes your problems.

Thanks @stephanh . I shall test with the latest maestro version and shall provide an update.

@stephanh Published util.etl in local with the maestro version 2.12.1-20150716022418-87386a0 and tested. The issues mentioned in case 1, 2 and 3 are fixed in this version.

Thank you @SajeevAr! And thank you for raising such a detailed bug report!


Expose in builtins' members internal type & size (on top of offset) in extension_api.json

fix #64189
(replaces #64365 that was incorrectly based on a branch own by godotengine/godot instead of touilleMan/godot)

@akien-mga rebased

I don't entirely understand the need for this, can't this information be inferred from the existing json?

@reduz see #64189
tl;dr:
currently float in gdextension_api.json means different things (e.g. in float_32 config, Vector2 type contains float fields that are 4bytes, where the float builtin type is 8bytes long).
This makes sens from GDScript point of view (given foo = 0.1 and foo = Vector2(0.1, 0).x both return the same real variant.
However it is misleading in GDExtension where providing a parameter with the wrong size means memory corruption 😄
Currently to know that Vector2 is composed of 4bytes long floats, one has to do arithmetic with builtin total size per-field offset information...
On top of that it's easy to jump to the conclusion that "float builtin is 4bytes long, builtin fields are 4bytes long on config float_32/64 and 8 bytes on double_32/64"... then missing the fact Color's float fields are always 4bytes long !
So this PR provide a meta field for the builtins attribute that works exactly like the meta field already existing for classes methods' parameters.

rebased ;-)

@groud Updated to remove the size field ;-)

Thanks!


Keeping the expanded state of the tree!

Hello,
I want to keep the current state, of expansion of the tree even when some action is done. I tried with the following code, but it turns out that expansion needs to be done twice.
if "current_expanded_folder" not in st.session_state:
    st.session_state.current_expanded_folder = []

checked_value = tree_select(
                    nodes,
                    expanded= st.session_state.current_expanded_folder,
                )
st.session_state.current_expanded_folder = checked_value["expanded"]

Any clues would be appreciated! Thank you!

Got it's solution for now, a simple hack worked as the state was not being updated, just posting in any case if someone requires it. The idea is to wrap the last statement with while loop.
while state.expanded_nodes != tree_value["expanded"]:
         state.expanded_nodes = tree_value["expanded"]


Review Dockerfile and RPM files for building odh-modelmesh-serving-controller in onflux

Resolves #https://issues.redhat.com/browse/RHOAIENG-14146

Hi, it is still a draft, is this PR ready for review?

Hi, it is still a draft, is this PR ready for review?

Hello, Please review the PR

Hi, it is still a draft, is this PR ready for review?

Hello, Please review the PR


Web component: Dotcom shell - Locale modal- After giving the search input and activation of clear search input button the dialog is getting closed

Detailed description
On JAWS FF
Issue:
After giving the search input and activation of clear search input button
the dialog is getting closed, and the focus comes back to to the Select geographic button in the parent page.
Note: in react staging it is working fine
Expected behavior :  is after activation of clear search input button, the focus should set back to the search edit field.
Additional information

PR was opened in carbon-web-components repository.


Upstream v5.72.0

Upgrades upstream to v5.72.0
New resources:

iam/rolePolicyAttachmentsExclusive.RolePolicyAttachmentsExclusive

In this release upstream has introduced deprecations for AWS IAM Role inline_policy and managed_policy_arns arguments. These deprecations are not yet propagated into the Pulumi provider (https://github.com/pulumi/pulumi-aws/issues/4654 tracks the context).
Fixes https://github.com/pulumi/pulumi-aws/issues/4642

      /**
  +     * Set of exclusive IAM managed policy ARNs to attach to the IAM role. If this attribute is not configured, Pulumi will ignore policy attachments to this resource. When configured, Pulumi will align the role's managed policy attachments with this set by attaching or detaching managed policies. Configuring an empty set (i.e., `managedPolicyArns = []`) will cause Pulumi to remove _all_ managed policy attachments.
  +     *
        * @deprecated The managedPolicyArns argument is deprecated. Use the aws.iam.RolePolicyAttachmentsExclusive resource instead.
        */

There's new deprecations rolling out, need to double-check this.

My hope is that it should go in after https://github.com/pulumi/pulumi-aws/pull/4673 as the failing test started failing independently. This should not release though as subsequent upstream release fixes some immediate bugs.


Github pages changes broke the media handling?

The current deployment on pages doesn't work:
https://carlosperate.github.io/jekyll-theme-rtd/
I get a similar but slightly different problem when I try:
https://skyl.github.io/bonsai/
I have a setup on a slightly older version of Github Enterprise that seemed to work fine yesterday .. hrm ..

Note: I fixed mine with a change to baseurl: "bonsai"


Draw schema of infrastructure

Draw schema of infra using something like https://draw.io
Put screenshot of schema in README

Use plantuml instead.


for..in loop crashes on months object array

Context
Where is the bug?
Here.
Description:
for...in iterates over all enumerable properties, not just the array's elements (Ref).
In our use case, We have a function move attached to all the Array objects. for..in iterates not just on the indices of months array but also on this function and crashes.
Expected Behavior
The loop iterating on the months array shouldn't crash.
Actual Behavior

Possible Fix
Replace the for..in loop here with a simple for loop.
for (i = 0; i < months.length; i++) {
    numberOfRows += Math.ceil(months[i].days.length / 7);
}

Steps to Reproduce

set
 Array.prototype.voice = "James Earl Jones";
Try to load more than a month.

 lengthOfTime: {
     months: 2,
     interval: 2
    }

Observe the crash.

Your Environment

Version used:  1.4.6
Browser Name and version: Chrome (56.0.2924.87)
Operating System and version (desktop or mobile): Desktop and Mobile

@manojkalaivanan I like this and I appreciate you filling out the bug form. My last ask is if you could submit this fix as a PR?

@mikegioia Done. Here is the PR https://github.com/kylestetz/CLNDR/pull/302.
Thanks!

Merged into master


[HDInsight] Add initial support for HDInsight

I'm adding a new command module for HDInsight. This change contains support only for a subset of features, but we will expand support at a later point. Basic tests have also been added, and we will also expand those as soon as possible.
This change allows users to create, resize, and delete HDInsight clusters with limited customization.
We will likely be making breaking changes to the user interface, so we would like to release this under preview if possible. Or alternatively, we could get a review for the current changes while we hammer out the details and release a command module where breaking changes are less likely.
Any feedback would be appreciated- I tried to borrow patterns from other command modules, but if something seems off or missing, please let me know.

This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] The PR has modified HISTORY.rst describing any customer-facing, functional changes. Note that this does not include changes only to help content. (see Modifying change log).

[ ] I adhere to the Command Guidelines.

Thanks for the review. I've updated the PR for most of the comments.

Please make sure the CI passes. Code itself looks fine.

Closes #3571.


Don't log inside the signal handler

This is not supported by python: https://bugs.python.org/issue24283

This change is

Reviewed 2 of 2 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved, some commit checks failed.

Comments from Reviewable


How to translate  ?

I'm submitting a ...  (check one with "x")
[ ] bug report => check the FAQ and search github for a similar issue or PR before submitting
[x] support request => check the FAQ and search github for a similar issue before submitting
[ ] feature request

Current behavior
I can't find the way to translate the label of a select option group (<select><optgroup label>).
Expected/desired behavior
I'd have expected something like :
<select name="mySelect" [(ngModel)]="myModel">
	<optgroup *ngFor="let value of list" label="my.translation.id.{{value}}" translate>
		<option></option>
		...
	</optgroup>
</select>

Please tell us about your environment:

ngx-translate version: 10.x.x

Angular version: 6.x.x

Browser: all

label= is not working.
Please take a look at https://github.com/ngx-translate/core#4-use-the-service-the-pipe-or-the-directive how to use the translate pipe or directive.

You can use the translate pipe:
<optgroup label="{{ 'demo.text' | translate}}">

Yeah ! It works !!!
Many many thanks !

Hi
I am trying to acheive same but it does not reflect changes can you help ??
<select placeholder="State">      <option *ngFor="let state of states" [value]="state.code">           {{ 'STATEDROPDOWN.'+state.name | translate}}      </option> </select>
StackBlitz for referrence : https://stackblitz.com/edit/trying-ngx-tranlate?file=src%2Fapp%2Fapp.component.html

Hi
I got it done in same way.
Thank You
<select placeholder="State"> <option *ngFor="let state of states" [value]="state.code"> {{ 'STATEDROPDOWN.'+state.name | translate}} </option> </select>
StackBlitz for referrence : https://stackblitz.com/edit/trying-ngx-tranlate?file=src%2Fapp%2Fapp.component.html

This is not working for me in Angular 8. What I did is as follows:-

<select class="ems_create_budget_elements" name="ems_create_budget_financial_month">
<option [ngValue]="null" selected translate>Select Financial Month</option>
<option [ngValue]="1">{{'JAN' | translate}}</option>
<option [ngValue]="2">{{'FEB' | translate}}</option>
<option [ngValue]="3">{{'MAR' | translate}}</option>
<option [ngValue]="4">{{'APR' | translate}}</option>
<option [ngValue]="5">{{'MAY' | translate}}</option>
<option [ngValue]="6">{{'JUN' | translate}}</option>
<option [ngValue]="7">{{'JUL' | translate}}</option>
<option [ngValue]="8">{{'AUG' | translate}}</option>
<option [ngValue]="9">{{'SEP' | translate}}</option>
<option [ngValue]="10">{{'OCT' | translate}}></option>
<option [ngValue]="11">{{'NOV' | translate}}</option>
<option [ngValue]="12">{{'DEC' | translate}}</option>
</select>

Where am I going wrong?


Do not return null objects in getAll of the InMemoryRequestJournalStore

Under heavy load it's possible that serveEvents.get() is called with an UUID that doesn't exist anymore in the Map. The reason for this is that the spliatator (required for stream()) of the ConcurrentLinkedDeque is weakly consistent. This means that it's not guaranteed that an element that is currently processed in a Stream still exists in the collection. So under heavy load it may happen, that one thread reads an element of the ConcurrentLinkedDeque. Then this thread gets paused and another thread gets activated which than completely runs the remove method of the InMemoryRequestJournalStore. Now the first thread gets activated again and calls the get method of the serveEvents with an UUID that doesn't exist anymore in the Map which results in a null value. As the remaining code doesn't expect null values in the Stream they should be filtered out. Another solution would be to synchronize the code of the InMemoryRequestJournalStore but this would slow down the whole thing dramatically.
I haven't added any tests for this as it's hard to reproduce a concurrency issue.
References

#2447

Submitter checklist

[ ] Recommended: Join WireMock Slack to get any help in #help-contributing or a project-specific channel like #wiremock-java
[ ] Recommended: If you participate in Hacktoberfest 2023, make sure you're signed up there and in the WireMock form
[x] The PR request is well described and justified, including the body and the references
[x] The PR title represents the desired changelog entry
[x] The repository's code style is followed (see the contributing guide)
[ ] Test coverage that demonstrates that the change works as expected
[ ] For new features, there's necessary documentation in this pull request or in a subsequent PR to wiremock.org

Despite the lack of tests (which I agree are hard to reliably do) I think this looks safe to merge.


Display all properties in non-interactive mode

In non-interactive mode, only the build generator is displayed.
Now, when there is a second property, the toolchain, it must be displayed too.

Iterate over all properties and show value.

Fixed on 2021-05-25.


Packaging request: Python package btrfsutil

Project description
Python 3 bindings of the libbtrfsutil, library for managing BTRFS file systems.
This binding is part of the btrfs-progs project.
Metadata

homepage URL: https://github.com/kdave/btrfs-progs/tree/master/libbtrfsutil
source URL: https://github.com/kdave/btrfs-progs/tree/master/libbtrfsutil/python
license: lgpl2Only
platforms: linux (I'm not sure if it works on BSD or Mac)

See #173436


indexHtml is not working

indexHtml is not working....
i use indexHrml: './index.html'
index.html is in the root directory,and it includes:
<!DOCTYPE html>

    <meta charset="UTF-8">
    <meta name="description" content="{{ description }}">
    <meta name="viewport" content="width=device-width, initial-scale=1.0">
    <meta http-equiv="X-UA-Compatible" content="ie=edge">
    <title>{{ title }}</title>
    {{ head }}
    <link rel="stylesheet" href="./index.css">

    {{ footer }}

indexHtml is not working for me.

sorry , i misspelled it.  But i link .css in ./index.html, i get 404.
index.html:
<!DOCTYPE html>

    <meta charset="UTF-8">
    <meta name="description" content="{{ description }}">
    <meta name="viewport" content="width=device-width, initial-scale=1.0">
    <meta http-equiv="X-UA-Compatible" content="ie=edge">
    <title>{{ title }}</title>
    {{ head }}
    <link rel="stylesheet" href="{{ publicUrl }}/index.css">

    {{ footer }}

Elements:
<link rel="stylesheet" href="/index.css">
<link rel="stylesheet" href="{{ publicUrl }}/index.css">  ==>  <link rel="stylesheet" href="/index.css">
i get 404.
directory:

Since v0.12.10 if you want to add static assets in a public folder, you need to put it inside the /public, or you can set a custom public directory on your doczrc. Please, see the release to know more about that.

yes，I saw it, but it work in yarn docz build , not work in yarn docz dev


Remove missing file from demo server Dockerfile

logging.conf is no longer present, preventing demo image container from being built.

How can I resolve the following error when using Restler for testing functionality?
2024-01-24 16:39:01.091: Connection error: 'Exception: [Errno 104] Connection reset by peer'


Remove terminating RPs from container edge list

We somewhat recently added the concept of "terminated" RoadPoints, which are meant to represent RoadPoints which are not connected on one or both sides, but are not meant to be considered as open edges from the standpoint of the parent RoadContainer (and thus, not available for connections across RoadContainers).
This change actually ensures the edge list doesn't include these RoadPoints, to avoid downstream checks for "terminated == true", and adds a set-getter to ensure the RoadContainers edge list is always up to date in the editor.

Extended one of the existing tests:
24 passed 0 failed.  Tests finished in 1.8s


Copy Face stable swagger from cognitive service to ai

Data Plane API - Pull Request
Copy existing swagger
API Info: The Basics
Most of the information about your service should be captured in the issue that serves as your API Spec engagement record.

Link to API Spec engagement record issue:

Is this review for (select one):

[ ] a private preview
[ ] a public preview
[ ] GA release

Change Scope
This section will help us focus on the specific parts of your API that are new or have been modified. Please share a link to the design document for the new APIs, a link to the previous API Spec document (if applicable), and the root paths that have been updated.

Design Document:
Previous API Spec Doc:
Updated paths:

Viewing API changes
For convenient view of the API changes made by this PR, refer to the URLs provided in the table
in the Generated ApiView comment added to this PR. You can use ApiView to show API versions diff.
Suppressing failures
If one or multiple validation error/warning suppression(s) is detected in your PR, please follow the
Swagger-Suppression-Process
to get approval.
❔Got questions? Need additional info?? We are here to help!

 Contact us!
The Azure API Review Board is dedicated to helping you create amazing APIs. You can read about our mission and learn more about our process on our wiki.

💬 Teams Channel
💌 email

Click here for links to tools, specs, guidelines & other good stuff
Tooling

Open API validation tools were run on this PR. Go here to see how to fix errors
Spectral Linting
Open API Hub

Guidelines & Specifications

Azure REST API Guidelines
OpenAPI Style Guidelines
Azure Breaking Change Policy

Helpful Links

Schedule a data plane REST API spec review

Checks stuck in `queued` state?
If the PR CI checks appear to be stuck in `queued` state, please add a comment with contents `/azp run`.
This should result in a new comment denoting a `PR validation pipeline` has started and the checks should be updated after few minutes.

@AzureRestAPISpecReview Could you help suppress Swagger LintDiff? This PR is copying existing swagger. We are not going to modify the content.
We also need APIStewardshipBoard-SignedOff tag in order to merge.
Thanks

I added the ApiStewardshipBoard-signedOff label.
For help with suppressing LintDiff errors, post to the Service API Toolset Teams channelhttps://aka.ms/azsdk/support/spectools-channel
Mike
From: Han Chiang @.>
Date: Monday, March 25, 2024 at 5:41 PM
To: Azure/azure-rest-api-specs @.>
Cc: Mike Kistler @.>, Comment @.>
Subject: Re: [Azure/azure-rest-api-specs] Copy Face stable swagger from cognitive service to ai (PR #28424)
@AzureRestAPISpecReviewhttps://github.com/AzureRestAPISpecReview Could you help suppress Swagger LintDiff? This PR is copying existing swagger. We are not going to modify the content.
We also need APIStewardshipBoard-SignedOff tag in order to merge.
Thanks
—
Reply to this email directly, view it on GitHubhttps://github.com/Azure/azure-rest-api-specs/pull/28424#issuecomment-2019183552, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AUNNB3Z7IOLB3P2BFPPRNN3Y2C73XAVCNFSM6AAAAABFGDPWGCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAMJZGE4DGNJVGI.
You are receiving this because you commented.Message ID: @.***>

If you believe the errors in SwaggerLintDiff should be suppressed rather than fixed, you should add the suppressions to your PR:
https://eng.ms/docs/products/azure-developer-experience/design/specs-pr-guides/pr-suppressions#suppress-autorest-extension


dotnet watch not working

Problem encountered on https://dotnet.microsoft.com/en-us/learn/aspnet/hello-world-tutorial/run
Operating System: windows
Provide details about the problem you're experiencing. Include your operating system version, exact error message, code sample, and anything else that is relevant.

Can you share what's the output of dotnet --info?

Can you share what's the output of dotnet --info?

C:\Users\james\MyWebApp>dotnet --info
.NET SDK (reflecting any global.json):
Version:   5.0.403
Commit:    bf9d168ac2
Runtime Environment:
OS Name:     Windows
OS Version:  10.0.19043
OS Platform: Windows
RID:         win10-x64
Base Path:   C:\Program Files\dotnet\sdk\5.0.403\
Host (useful for support):
Version: 5.0.12
Commit:  7211aa01b3
.NET SDKs installed:
5.0.402 [C:\Program Files\dotnet\sdk]
5.0.403 [C:\Program Files\dotnet\sdk]
.NET runtimes installed:
Microsoft.AspNetCore.App 3.1.21 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 5.0.12 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.NETCore.App 3.1.21 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 5.0.12 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.WindowsDesktop.App 3.1.21 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
Microsoft.WindowsDesktop.App 5.0.12 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
To install additional .NET runtimes or SDKs:
https://aka.ms/dotnet-download
C:\Users\james\MyWebApp>

Thanks for confirming @jamessebastian. I'll change the tutorial a bit to indicate you need the latest SDK version (.NET 6) for the tutorial to work as-is.

Tutorial was changed to indicate versions.


bug tx-generator: empty data in log entries

make ci-test-plutus-autonix
generator logs have:

{"app":[],"at":"2022-06-01T02:22:58.72Z","data":{},"env":"<unknown>:00000","host":"","loc":null,"msg":"","ns":["cardano.cli.generate-txs.submitN2N"],"pid":"1870553","sev":"Debug","thread":"2495"}

..which have empty data field

This normal behavior for the tracer that uses minimal verbosity.
https://github.com/input-output-hk/cardano-node/pull/3952/commits/1291cfb613ec5de47ce612a3f8faa3016f0dee54
switches the tracer to maximal verbosity.
https://github.com/input-output-hk/cardano-node/blob/9a177fdd6ff48649c38f88dad060a14d437abbbd/bench/tx-generator/src/Cardano/Benchmarking/Tracer.hs#L116-L120
The llSubmit and submitN2N tracers had been set to minimal verbosity for at least a year.
But looking at it I see that llSubmit is not even used anywhere in the code.


Add a visitor for IR  and related tests

@markww replied to your comments PTAL.  There is one thing about auto I will address as soon as we resolve the other comments.


404 at: /engine/reference/commandline/dockerd/

URL: /engine/reference/commandline/dockerd/

This problem seems to have been fixed: https://docs.docker.com/engine/reference/commandline/dockerd/


Watch on cert-manager api causes a timed out waiting for cache to be synced error

Bug Report
Everthing with the operator works as expected until I add a watcher for any resource of cert-manager api
	options.Watches(&source.Kind{Type: &certmanagerApiV1.ClusterIssuer{}}, &watcher)
Then I got an error:

2022-05-23T08:41:20.618+0200    INFO    controller-runtime.metrics      Metrics server is starting to listen    {"addr": ":8080"}
2022-05-23T08:41:20.619+0200    INFO    setup   starting manager
2022-05-23T08:41:20.619+0200    INFO    Starting server {"kind": "health probe", "addr": "[::]:8081"}
2022-05-23T08:41:20.619+0200    INFO    Starting server {"path": "/metrics", "kind": "metrics", "addr": "[::]:8080"}
2022-05-23T08:41:20.619+0200    INFO    controller.corestack    Starting EventSource    {"reconciler group": "core.cloud.local", "reconciler kind": "CoreStack", "source": "kind source: *v1alpha1.CoreStack"}
2022-05-23T08:41:20.619+0200    INFO    controller.corestack    Starting EventSource    {"reconciler group": "core.cloud.local", "reconciler kind": "CoreStack", "source": "kind source: *v1.OperatorGroup"}
2022-05-23T08:41:20.619+0200    INFO    controller.corestack    Starting EventSource    {"reconciler group": "core.cloud.local", "reconciler kind": "CoreStack", "source": "kind source: *v1alpha1.InstallPlan"}
2022-05-23T08:41:20.619+0200    INFO    controller.corestack    Starting EventSource    {"reconciler group": "core.cloud.local", "reconciler kind": "CoreStack", "source": "kind source: *v1.ClusterIssuer"}
2022-05-23T08:41:20.619+0200    INFO    controller.corestack    Starting Controller     {"reconciler group": "core.cloud.local", "reconciler kind": "CoreStack"}
2022-05-23T08:41:30.721+0200    ERROR   controller.corestack    Could not wait for Cache to sync        {"reconciler group": "core.cloud.local", "reconciler kind": "CoreStack", "error": "failed to wait for corestack caches to sync: timed out waiting for cache to be synced"}
sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).Start.func2
/home/dev/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.11.1/pkg/internal/controller/controller.go:208
sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).Start
/home/dev/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.11.1/pkg/internal/controller/controller.go:234
sigs.k8s.io/controller-runtime/pkg/manager.(*runnableGroup).reconcile.func1
/home/dev/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.11.1/pkg/manager/runnable_group.go:218
2022-05-23T08:41:30.721+0200    INFO    Stopping and waiting for non leader election runnables
2022-05-23T08:41:30.721+0200    INFO    Stopping and waiting for leader election runnables
2022-05-23T08:41:30.721+0200    INFO    Stopping and waiting for caches
2022-05-23T08:41:30.721+0200    INFO    Stopping and waiting for webhooks
2022-05-23T08:41:30.721+0200    INFO    Wait completed, proceeding to shutdown the manager
2022-05-23T08:41:30.721+0200    ERROR   setup   problem running manager {"error": "failed to wait for corestack caches to sync: timed out waiting for cache to be synced"}
main.main
/mnt/c/IDE/Projects_Git/playground/moveup/stacks/core/main.go:129
runtime.main
/home/dev/sdk/go1.17/src/runtime/proc.go:255
exit status 1

Environment
Operator type:
go
Kubernetes cluster type:
k3s version  1.23.4+k3s1

version cert-manager 1.8.0
$ operator-sdk version operator-sdk version: "v1.20.0", commit: "deb3531ae20a5805b7ee30b71f13792b80bd49b1", kubernetes version: "1.23", go version: "go1.17.9", GOOS: "linux", GOARCH: "amd64"

Issue is reolved.
I forgot
	utilruntime.Must(certmanagerApiV1.AddToScheme(scheme))
But the logging is confusing. I think there was an appropriate logging in the past


i dont like clojure

sand was not meant to think. replace with minimum page labor of some teenager

Too bad


Add code to determine if client authentication is required for OAuth2…

An attempt to improve OAuth2 endpoint "revocation.py", and to fix django-oauth-toolkit issues https://github.com/evonove/django-oauth-toolkit/issues/244 and https://github.com/evonove/django-oauth-toolkit/issues/242.
The "client authentication" is always required in the current method validate_revocation_request. There are cases that the authentication can be ignored though, like when grand type is "Resource Owner Password Credentials" and the "Client type" is Public.
The patch makes this possible so you can just pass a client_id (without secret) to revoke a token.
@ib-lundgren could you please take a look at this one :)

LGTM

LGTM too

Awesome! Just ran across :)


vote interface - adding discord, discussions and new proposal CTA

as we would like to make it visible for people to follow discord and discourse conversations on the voting topics, we should include the relevant buttons that allow easy navigation.
discord link: https://discord.gg/EHK8wHbgau
please update to this URL on the menu as well.

diiscussion link: gov.bancor.network
+: open new proposal popup
https://www.figma.com/file/3yED7o7TgoLwoEjV2EkRLL/Bancor-V2?node-id=548%3A10009
https://www.figma.com/file/3yED7o7TgoLwoEjV2EkRLL/Bancor-V2?node-id=548%3A11347

on staging now.

there is a mix of colors.
icon is gray while. text is black.
can it all be gray?

I have updated the icons to be all gray.
changing the text of the button to be gray is possible. Should that be only for those buttons on the vote page or for all buttons?
Or leave it as it is now @ashachaf? It is on staging now.

Closing due to inactivity. Please reopen if further changes required.


Reused regular expression for jQuery parameter, creates ability to use jPlayer for SOME-attacks

Hi,
I want to explain an issue still present in jPlayer due to the regex pattern you're using today in the jQuery-functionality/prevention checks (the function checkFlashVars together with illegalChar). This issue opens up for an attack called SOME – Same Origin Method Execution (the concept was initially posted public by Ben Hayak. More info here: http://securityaffairs.co/wordpress/29338/hacking/same-origin-method-execution-attack.html)
The issue is that the same regex – being used to validate that the FlashVars-parameters are valid – is also used to check the jQueryIllegal-function on the jQuery-parameter sent into the SWF-file. The character which makes the issue possible, and should not be in the jQuery-parameter is the dash-character: "-".
Using this character in the jQuery-parameter you can get any JS-function to be called, by inserting jQuery as a bogus property of a string and by adding the dash inbetween:
Jplayer.swf?jQuery=String.jQuery-alert
Now an alert won't do any harm, but utilizing the SOME-attack you're able to do bad things on whatever is hosted on the same domain.
Attack Scenario
For example. Imaging you have a "install application" page on the same domain as jPlayer is hosted. This page requires the user to click "Install" for it to be installed:

Installation of Application

<form action="http://example.com" method="post" id="x">
	<input type="hidden" name="csrf-token" value="XXXYYYZZZ" />
	<button type="submit">INSTALL APP X</button>
</form>

This is just an example, that's why I've used example.com as the target of the POST-request. Also note that this installation page is patched against CSRF.
Now, to do this attack we can now use the jPlayer with the issue mentioned above. The jPlayer will submit the form, with the CSRF-token from another window.
There are two ways to execute this attack. It all depends on if the server - that is hosting the jPlayer.swf - is sending any CSP or X-Frame-Options headers. If not, we only need the victim to click a link posted anywhere. The response of this link would then look like this:
<iframe name="y" src="http://example.com/install.html" onload="x.innerHTML=''">

This will create two iframes, first the installation page and then jPlayer.
If the server is using CSP or X-Frame-Options, we need to host the link on a place where we can trigger a location-change when the user clicks the link, so this one needs two interactions, first clicking the link from the attacker, and then click the link on the attacker's page for the popup to be allowed (due to popup-blockers):
HELLO!
As you see here, the same thing happens in both examples:

Two frames are opened, in iframes or in new windows/same window.
Since the jPlayer.swf and the "installation of application"-example are on the same domain, these two windows can communicate with each other. Using window.opener or window.parent.frames depending on the method we're using.
The jPlayer is launched after the installation page (also since SWF has some delay to load).
The jPlayer then calls the function we need, and since both submit() and click() doesn't really care about the parameter - which in jPlayer's case will be "#"+id - they will both trigger.
The form is now submitted and the application is added.

Here's also a movie showing the complete scenario in both examples:
http://detectify-labs.s3.amazonaws.com/jPlayer-some/jPlayer-SOME.mov
The first one needs interaction and the second one will trigger automatically (since my test page is not using any CSP or X-Frame-Options).
There are several other ways to execute this attack, this is just one example.
Here's the vulnerable code:
for each (_local2 in _arg1) {
	if (this.illegalChar(_local2)){
		this.securityIssue = true;
	};
};
if (!this.securityIssue){
	if (this.jQueryIllegal(_arg1.jQuery)){
		this.securityIssue = true;
	};
};

and:
private function illegalChar(_arg1:string):boolean {
	var _local2:RegExp = /^[-A-Za-z0-9_.]+$/;
	return (!(_local2.test(_arg1)));
}
private function jQueryIllegal(_arg1:string):boolean {
	var _local2:RegExp = /(jQuery)/;
	return (!(_local2.test(_arg1)));
}

How to fix:
It's easy, and will most certainly not break anyone's implementation of jPlayer: You should not allow "-" to be a part of the jQuery parameter, and since you're not really able to make function names that contain dash at all (not in this case at least, since only -a-zA-Z0-9._ is allowed here) there's no reason for it to be a part of the jQuery-parameter in particular.
Regards,
Frans Rosén (https://twitter.com/fransrosen) and Mathias Karlsson (https://twitter.com/avlidienbrunn)

Hi Frans,
i using this  tag ""  is inject but markdown Github for use image using this format :
https://guides.github.com/features/mastering-markdown/
Images:

Format: ! [Alt Text] ( path_url)
but i using this image TAG " "
is display in github command, why? this is a bug ?

also i using iframe tag nothing :),
Thanks,
Ramin


Ask about hash expansion

During the hash expansion, the data in the hash bucket on the disk will be moved to the new bucket. Is the performance affected by the disk IO?

I'm not sure I understand the question. NuDB only works with SSD drives, and the performance is near the theroetical maximum for what is possible with a single write queue. Are you asking me if the overall performance of the library is affected by the performance of the SSD drive? Obviously yes, so I am maybe not understanding the question fully.

I'm not sure I understand the question. NuDB only works with SSD drives, and the performance is near the theroetical maximum for what is possible with a single write queue. Are you asking me if the overall performance of the library is affected by the performance of the SSD drive? Obviously yes, so I am maybe not understanding the question fully.

I know that you use linear hash to solve hash table expansion. Do you directly use the native linear hash expansion algorithm? In my understanding, the expansion of the native linear hash table will cause the data in some slots to be moved to the new slot. I think this process will involve random io. Have you optimized this process?

Do you directly use the native linear hash expansion algorithm?

No. The principle is the same, but since NuDB is block/bucket based, every expansion of the hash table adds an entire block of slots and not just one. NuDB is very fast. You should run your own benchmarks to confirm that the performance meets your requirements.

Do you directly use the native linear hash expansion algorithm?

No. The principle is the same, but since NuDB is block/bucket based, every expansion of the hash table adds an entire block of slots and not just one. NuDB is very fast. You should run your own benchmarks to confirm that the performance meets your requirements.
There's a benchmark program that compares NuDB to other key/value stores: https://github.com/CPPAlliance/NuDB/tree/master/bench
Thank you for your reply, I would also like to ask, is your key a fixed length? How many bits do you map the key to, and what is the hash algorithm for the key? What is the approximate number of keys you support? What is the approximate conflict rate?

Do you directly use the native linear hash expansion algorithm?

No. The principle is the same, but since NuDB is block/bucket based, every expansion of the hash table adds an entire block of slots and not just one. NuDB is very fast. You should run your own benchmarks to confirm that the performance meets your requirements.
There's a benchmark program that compares NuDB to other key/value stores: https://github.com/CPPAlliance/NuDB/tree/master/bench

Thank you for your reply, I would also like to ask, is your key a fixed length? How many bits do you map the key to, and what is the hash algorithm for the key? What is the approximate number of keys you support? What is the approximate conflict rate?

is your key a fixed length?

Yes. You set the key size when the database is created.

How many bits do you map the key to

64 bits

what is the hash algorithm for the key

The default algorithm uses xxhasher which is fine for general purpose use. But this can be changed, if you can take advantage of how your keys are distributed.

What is the approximate number of keys you support

Because of the birthday problem, you can insert 2^32 (over 4 billion) keys before performance will begin to degrade. But since a database can be trivially split up (by dividing the key space by a power of two) the number of keys is essentially unlimited. NuDB is in production use with databases that are over 13 terabytes in size.

What is the approximate conflict rate?

The default settings aim for 50% occupancy in buckets. Remember that each bucket is designed to hold many keys not just one. And the cost for I/O is much higher than the cost for in-memory lookup.

https://en.wikipedia.org/wiki/Birthday_problem

Thank you very much for your patient answer. At present, I want to design a kv database, the key is a string, and the value is a list, which is used to store some integer values. Currently, I need to support adding kv operations and adding numbers to a list of a key. And want to get the list of a key as quickly as possible, do you mind? At present, my idea is to use linear hash to solve the expansion of the hash table when adding keys, but adding an integer value to a key to the list may cause the value of a key to be distributed in multiple disk blocks, which requires multiple disk io to obtain the complete value, so the read performance is poor.

Thank you very much for your patient answer. At present, I want to design a kv database, the key is a string, and the value is a list, which is used to store some integer values. Currently, I need to support adding kv operations and adding numbers to a list of a key. And want to get the list of a key as quickly as possible, do you mind? At present, my idea is to use linear hash to solve the expansion of the hash table when adding keys, but adding an integer value to a key to the list may cause the value of a key to be distributed in multiple disk blocks, which requires multiple disk io to obtain the complete value, so the read performance is poor.

NuDB is not going to work well for you, because you intend to change the value of already-inserted entries. Modification and deletion of existing values is not possible in NuDB.


[WIP] 576 Add methods to check SSH connections/commands with retry

Early PR for feedback on approach for addressing #576
@rhoboat, I'm working on the helper methods you suggested in 576 and I want to make sure this is an approach you are comfortable with.
I wrote CheckSshConnectionWithRetry and CheckSshConnectionWithRetryE to accept an optional function. They default to wrapping CheckSshConnectionE, but the option to inject means we can swap it for a mock function and unit test much easier. My worry is that the interface might not be as clear as it could be to users. What do you think?
If this looks good, I'll move on to CheckSshCommandWithRetry and CheckSshCommandWithRetryE. Thanks!

Just a heads up that @rhoboat is currently OOO, but they will take a look when they are back!

hey @rhoboat! Bumping this for visibility. I can keep going, make alterations or close the PR. Thanks!

Bump.

@rhoboat Could you take a look?
@ztbrown Could you pull in the latest from master?

rebased against master. This is still what I would consider a WIP or early PR, looking for feedback. Do we like this approach? Is it too clever by half? If we don't like it, how would you like to proceed?
Thanks!

Bump. If you all would like me to drop it, I will close. LMK!

Thank you but it's totally understandable. :)

OK @rhoboat, looking for a final review. I've added the retry logic to CheckSshCommandWithRetry and CheckSshCommandWithRetryE. Added some tests for when retries get exceeded on CheckSshCommandWithRetryE and CheckSshConnectionWithRetryE. Let me know what you think. Thanks!

It's odd that the checks for this PR are not running, i.e., tests. I'd like to see them succeed on CI.

I just kicked off the tests.

Looks like an unrelated failure. Do you typically re-kick the tests in this situation?

lmk if you want me to rebase against master, squash down to one commit or just press the "resolve conflicts" button above.

@ztbrown Would be best if you can rebase. Since it's a go.sum conflict, this is typically better to resolve by wiping out the file and running go mod tidy to ensure all the dependencies resolve correctly.

awesome. I'm on it.


Define zoom shortcut before using it

Fixes https://github.com/godotengine/godot/issues/61775
The error doesn't show up after this.

Thanks!


Version of apps are mismatched

Here is an example of lastpass

i did not use brew cu to update as i was prompted to from auto update. is this what causing the error

Hello @Parthvsquare , please read the not about auto-updates here: https://github.com/buo/homebrew-cask-upgrade#apps-with-auto-update


Add conditional retry

Current coverage is 97.40%

Merging #2 into master will increase coverage by +0.30% as of 8eff1d9

@@            master     #2   diff @@
=====================================
  Files            1      1
  Stmts           69     77     +8
  Branches         0      0
  Methods          0      0
=====================================
+ Hit             67     75     +8
  Partial          0      0
  Missed           2      2

Review entire Coverage Diff as of 8eff1d9

Powered by Codecov. Updated on successful CI builds.


Istio Config Details Service Mesh tab iFrame loads only config page with validations

RFE https://github.com/kiali/openshift-servicemesh-plugin/issues/52
Istio config objects which have validations in Kiali side now have "Service Mesh" tab in Console.

I think the view is much more integrated now. Some comments that I can think about:

The Envoy filter or the HttpRoute doesn't have a Mesh Tab. I guess this is because they don't have validations?

As the first impression the view is a bit confusing because you can see the same information in both tabs. I was wondering if indicating something more explicit like "Validations" can be more useful:

Now in "Service Mesh" tab, the "YAML" subtab is renamed to "Validations", to avoid the confusion with the original "YAML" tab of console above.

Hi @andrew-ronaldson , wdyt from UX perspective about the "Service Mesh" tab?
It shows the Kiali Istio Config page with validations but in read-only mode.

Reviewed this in the UX meeting sync. The proposal is, if we are going to have only one tab, to remove the "Validations" subtab, and update the name in the "Service Mesh"

Approached comments, thanks for review:


Release releasetool 0.1.0

This pull request was generated using releasetool.

Release is at https://github.com/GoogleCloudPlatform/releasetool/releases/tag/releasetool-0.1.0


link or button to the next page or next section

User feedback from<EMAIL_ADDRESS>As I understand this request:
We could add one (or more) link at the bottom of every page to continue the reading.
It could be a 'Next' button to the next page or next section.
Current state: some pages have links at the bottom but not all pages

@jmarlena 's additions look good to me.

I agree the provided links are good.
Yet... :)
Do we keep the issue open, for links to be added in other guides (Detailed editing, Coordination...) or to make them programatically as suggested by https://github.com/hotosm/learnosm/issues/270#issuecomment-67222443 ...?
Clever links to other sections and other guides are a very nice touch for criss-cross reading but I must admit a simple link to the 'next page' still appeals to me. People unfamiliar with the subject may be more comfortable with linear reading as it ensures that nothing is missed.

Working on the next page button! I have the script for the next page but just trying to figure out where it goes.

Well after much trial and error I've added previous and next buttons that link to the other language versions of the post circled in red in the example below:

After playing with the file structure I checked out the Jekyll documentation and it basically says it's not possible to use the pagination "next" and "previous" for different categories!

http://jekyllrb.com/docs/pagination/
However, I came across a blog post describing a plugin built for Jekyll to make a "category aware" next button so I will try exploring that as an option.
http://stravid.com/en/category-aware-previous-and-next-for-jekyll-posts/

Closing this now - although we don't have a next or previous button, we now have a button at the bottom of each page that takes you back to the top, so you can easily access the menu to chose where you wish to go next.


make Windows Terminal available to v1809

Summary of the new feature/enhancement
There are a significant quantity of corporate machines not able to run 1903 today for a variety of reasons (policy, compatibility, etc) and hence are locked out of terminal.  I wasn't able to find much discussion about the reasons that exist for 1903 to be a requisite.

In my guessing, there is probably a feature within the windows source code that is used in the new terminal which is why 1903 is required.

I guess it's because maintaining compatibility is a trouble. You CAN try to run on 1809 but without any support.

Sorry. Right now, there's two big blockers to adoption on 1809.

XAML islands was a technology preview and didn't support high-DPI, DPI changes, or accessibility in 1809. We rely on them heavily.
1903 added support for side-by-side WinRT component activation, something deep in the COM stack that lets us find our DLLs when they're right next to our EXE.

We just can't go back to 1809.

We just can't go back to 1809.

TY for the detail

We just can't go back to 1809.

Please Microsoft, fix the Surface Book 2 then! 😭

It looks like WInUI 3.0 will backport full XAML Islands support to most versions of Windows 10 and is releasing this year - could you use WinUI 3.0 to add support for Windows Terminal on earlier versions of Windows 10?

That's certainly a possibility we'd have to investigate. If XAML Islands was detached from the platform, then I'm not sure how many of our own features are actually dependent upon 1903.
I guess Dustin mentioned the SxS winrt activation above - I'm not positive that is still needed, but if it is, then that might be exceptionally hard for us to overcome. That would be one of the things to investigate when WinUI 3.0 is available.

WinUI 3.0

It seems to me that there is now a preview available for it: https://docs.microsoft.com/en-us/windows/apps/winui/winui3/

XAML Islands is not supported in this release

That's the single most important part of the WinUI stack we need, so unfortunately, that's a non-starter for us. Don't worry, we work pretty closely with the WinUI team (in the same hallway in fact), so when WinUI 3 is ready for us to move to it, we'll work on it 😉

WinUI 3.0 Preview 2 has been released some days ago. Anything new on LTSC support?
https://docs.microsoft.com/en-us/windows/apps/winui/winui3/

I've gone ahead and marked up their list of limitations with ones that stop us from being able to use Preview 2:

As I've said before, we won't be able to start experimenting with WinUI 3.0 until there's support for XAML Islands.

Can we migrate the whole app to WinUI on Desktop instead of a Win32 Window with XAML islands?

After another three months having passed:
Is there any update on the WinUI3 transition?

No, and it is neither in our roadmap nor committed or planned to ever happen. :smile:

If I'm not mistaken, this means a Windows Server 2019 machine can't run Windows Terminal at all — the current release for that is still 1809. (And you can't do an in-place upgrade to a newer release, because those are on a different channel.)

You are not mistaken.


Allow EntityFields to use dotted-name class strings

Implement importing logic when evaluating the type of an EntityField
This was done to allow circular references in entities that may
refer to one another.
Rename TestEntity in a test module to address a pytest warning.
Tests passing: 12 passed in 0.08 seconds

There seems to be something missing here, hold off on this for now.
My test must not be complete - using my branch in another project is causing failures related to the change in this PR.

My last push fixed my earlier problem. I added more tests as well.


feat: remove compute engine category

Description
Do not use task category in the compute engine
How has this been tested?
Local E2E and CI
Checklist

[ ] changelog was updated with notable changes
[ ] documentation was updated

/e2e


Handle HTTP Errors

Issue
If there is an HTTP Error due to restricted access or missing file, the program will crash. Only retrieve the file if there is no HTTP Error.

Currently looking at urllib.error


Remove rapids_twine_new

This tool is now completely redundant, rapids_twine can always be used instead.

Since we don't follow the same branching strategy in gha-tools as the rest of RAPIDS, it's probably best to wait to merge this until after we've completed 23.08 release builds so that this doesn't accidentally get into images used to build 23.08 packages that still use workflows that use this tool.


Fix/swift2.3 bug

Thanks for your interest. Main branch was already updated for Swift 2.3 including 0.6 release. Please check that out.


ArgumentError: only virtual fields can have type :any

Hi!
I curiously couldn't find a bug report or a post on elixirforum.com regarding this. Guess I'm the first to encounter the issue.
Environment

Elixir version: 1.11.0
Database and version: PostgreSQL 12
Ecto version: 3.5.4
Database adapter and version (mix deps): 0.15.7
Operating system: Docker image hexpm/elixir:1.11.0-erlang-23.1.1-alpine-3.12.0

Current behavior
This used to work with ecto 3.4.6:
defmodule MyApp.Thing do
  use Ecto.Schema

  schema "things" do
    # jpath is a :jsonb column
    field :jpath, {:array, :any}
  end
end

But now it raises an ArgumentError:
** (ArgumentError) only virtual fields can have type :any, invalid type for field :jpath
    (ecto 3.5.4) lib/ecto/schema.ex:2079: Ecto.Schema.check_field_type!/4
    (ecto 3.5.4) lib/ecto/schema.ex:2084: Ecto.Schema.check_field_type!/4
    (ecto 3.5.4) lib/ecto/schema.ex:1796: Ecto.Schema.__field__/4
    lib/myapp/things.ex:6: (module)

I think it's because of the composite?(type, name) condition where the type is unpacked and check_field_type!/4 is called recursively.
Expected behavior
It should compile and be able to cast the value to an array of JSON values.
If this error should remain even if it's a composite then I'm not sure how to accomplish this easily. I'd have to write a parametrized type maybe.

We should address this. Could you please submit a PR?

Sure thing. My idea atm would be the following:
      type == :any and !opts[:virtual] and !opts[:composite?] ->
        raise ArgumentError, "only virtual fields can have type :any, " <>
                             "invalid type for field #{inspect name}"

      composite?(type, name) ->
        {outer_type, inner_type} = type
        opts = put_in(opts[:composite?], true)
        {outer_type, check_field_type!(mod, name, inner_type, opts)}

I'm wondering something. Why is :any not allowed as a type for non-virtual fields? How would you define a field that can have any value if it's a :jsonb column?

@azizk you would need a custom type for that, which is fine, it is not very common.
Regarding the fix, it is probably simpler to move the type == :any and !opts[:virtual] check to the __field__ function.

I believe :any is allowed in virtual only because they may be not be able to be encoded. For instance, %MyStruct{} is accepted in :any, but if you try to encode it to store on a jsonb, you'll get a runtime error unless it implements an encoding protocol.

If you want something like that, I'd go for a parameterized type for `serializable'.

@v0idpwn That sounds reasonable, however consider that you will still be able to do that once we fix this bug. So with my example schema you would be able to assign a non-serializable value in an array to jpath and the Ecto changeset would be valid. It would only fail when it's time to encode I think.


Add class ID to API report output

This is a companion to this portal-reports PR: https://github.com/concord-consortium/portal-report/pull/90
We want to use the class ID instead of the class name when recording events with Google Analytics. (Class name's could potentially contain personally identifiable information.)

👍


Typo in Penetrator Rounds description

In the English language, there is a typo in the description for the Hard Target's penetrator rounds; the word "Incredibly" is misspelt as "Increadibly". This may also affect other weapons that can be given Penetrator Rounds.

Details
546CC - HardTarget WEAP
128C4C, 128C6C, 1298AE, 12A0B0, 12A784, 12D03D, 133A6B - Penetrator Rounds OMOD
Game Version
<IP_ADDRESS>
New Game
Yes
Reproduction Steps

Head to any weapon workbench, such as one in Laredo Arms by using the console command "coc CityAkilaCityLaredoFirearms"
Obtain a Hard Target, you can do so with the command "player.additem 546CC 1"
Use the nearby weapon workbench and select the Hard Target
Click on "Magazine and Battery" and hilight the Penetrator Rounds option

This typo applies to:

co_gun_mod_HardTarget_Mag_Penetrator[000454B2]
co_gun_mod_Lawgiver_Mag_Penetrator[00057561]
co_gun_mod_MagSniper_Mag_Penetrator[00045BAF]
co_gun_mod_Razorback_Mag_Penetrator[00044C30]
co_gun_mod_Regulator_Mag_Penetrator[00057579]
co_gun_mod_Tombstone_Mag_Penetrator[00044C4D]
co_gun_mod_UrbanEagle_Mag_Penetrator[0007C747]

https://github.com/Starfield-Community-Patch/Starfield-Community-Patch/commit/a51cf6dbb83968aa19b2ab8b3bb16811260bd46d


Correct defective pixels with mean of neighborhood

This PR modifies the way defective pixels are handled in NSI preprocess function NSI_load_scans_and_params().
The values of defective pixels will be replaced by the mean of its neighborhood pixels in the scan images. The sino weight of corresponding pixels are NOT set to 0.0 anymore.

Close this PR for further modification.


sdl2: build fails with dash fix

Should print a warning for #872
Existing package

[x] I am the maintainer of this package.

Fails in dash but works in busybox sh?

Yep. I build with busybox sh with no problems.

I don't like this solution as it affects users with dash installed (but with something else as /bin/sh). Does CONFIG_SHELL=/bin/sh ./configure \ work at all?

CONFIG_SHELL= ./configure \ does work.
When busybox is /bin/sh I can make it fail. And when dash is /bin/sh I can make it succed.
We could modify the warning to only print when dash is /bin/sh but I have no nice idea how to query that.

I have found a way to query dash and put that in line with other kiss-tools.
#!/bin/sh -e

log() {
    printf '\033[31;1m->\033[m %s.\n' "$@"
}

die() {
    log "$@"
    exit 1
}

kiss l dash &&
kiss a | grep "dash /usr/bin/sh" ||
die "ERROR: sdl2 can not be build with dash. Please change /usr/bin/sh to something appropiate"

./configure \
    --prefix=/usr

make
make DESTDIR="$1" install

This should be fixed now. Simply update dash.


docs: Revise descriptions

New account (inactive until today) that started opening PRs with unmeaningful changes after the Starknet Airdrop.
This is airdrop farming, with a high chance of botting/automatisation, that make the work harder for real contributors and repos reviewers.
I would strongly recommand to close that type of PRs for the time being to contain the post-airdrop spam there's going to be due to some Twitter engagement bait post, to avoid proliferation, and doing the changes yourself if needed.
By rejecting those unmeaningful farming PRs, that newly found "github farm" strategy will be considerated ineffective and the spam will soon end, for the benefit of reviewers, devs, users and tech.
If you're doing that with no malicious intent, please consider doing more meaningful contributions.
He also opened a similar PR for Taiko, another anticipated project, with the same unmeaningful changes : https://github.com/taikoxyz/taiko-mono/pull/15795


fix(region): sync servers

这个 PR 实现什么功能/修复什么问题:

[x] 功能、bugfix描述
[ ] 冒烟测试
[ ] 单元测试编写

是否需要 backport 到之前的 release 分支:

release/3.4

/area region
/cc @zexi

Codecov Report

Merging #9360 (ae060f2) into master (2b2ea48) will decrease coverage by 1.84%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master   #9360       +/-   ##
==========================================
- Coverage    5.85%   4.00%    -1.85%
==========================================
  Files         838     170      -668
  Lines      130234   25242   -104992
==========================================
- Hits         7619    1012     -6607
+ Misses     121926   24116    -97810
+ Partials      689     114      -575

Flag
Coverage Δ

aFlag
4.00% <ø> (-1.85%)
:arrow_down:

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

Impacted Files
Coverage Δ

pkg/util/iproute2/link.go

.../monitor/alerting/conditions/suggestrulereducer.go

pkg/compute/models/vcenters.go

pkg/hostman/system_service/telegraf.go

pkg/keystone/models/serviceresource.go

pkg/compute/models/routetables.go

pkg/compute/models/quotas.go

pkg/hostman/monitor/monitor.go

pkg/util/conditionparser/template.go

pkg/multicloud/aws/loadbalancerlistener.go

... and 655 more

Continue to review full report at Codecov.

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

/lgtm
/approve


Problems outputting to directory using tile-join

I'm trying to output to a directory using this command:
tile-join --output-to-directory="./tiles" OS-Open-Zoomstack.mbtiles -z14 --no-tile-size-limit --no-tile-compression
However I'm getting this error:
OS-Open-Zoomstack.mbtiles: select failed: no such table: tiles

I can't reproduce this. Is OS-Open-Zoomstack.mbtiles in fact an mbtiles file? To be one, it has to be a sqlite database with a tiles table, hence the error message.

It was a dud download I think. Got it working now thanks.


Use initials to identify different calendar entries

I'm looking in to use this module as a information bord on our company.
This module is perfect for showing me and my colleges appointments in "one schedule".
But instead of the "Font Awesome" symbol in front of each entry I would like to have different initials to each calendar instead. Is that possible to easily add to this module?

I'll look into adding this to the module, thank you!


Inherit labels

We already have a workload with app: fluentd running. The logging-operator interferes with it. The selector for the service is only using app, but should be using release as well.

Absolutely true. I'll look into it asap ;)


Upgrading a Dank causes crash to desktop.

Was upgrading a Dank 2 (redstone) to Dank 3 (gold). When attempting to pull the upgraded Dank from the crafting grid, the game crashed to the desktop.
Crash Report: https://gist.github.com/Zarepheth/34728cf7e258bc39ef4b8cf6ebcff430#file-crash-report-2021-oct-14

Upon returning to the world, I suffer another crash to desktop just as the client starts responding after connecting to the server (single player game).
Crash Report #2: https://gist.github.com/Zarepheth/e8e43d27cca80e9972042641764e9592#file-crash-report-2-2021-oct-14

Weird

Ok, this one isn't possible anymore.


ERROR: Unable to extract sts

Hi everyone. I got bug and because it says I should post it here, I'm posting it.
(Ignore that video itself, pls)
I've reinstalled youtube-dl using "sudo aptitude reinstall youtube-dl", but it remained the same
b7@debian:~$ youtube-dl --verbose https://www.youtube.com/watch?v=2svVkkNuSq0
[debug] System config: []
[debug] User config: []
[debug] Command-line args: ['--verbose', 'https://www.youtube.com/watch?v=2svVkkNuSq0']
[debug] Encodings: locale UTF-8, fs UTF-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2014.08.05
[debug] Python version 2.7.9 - Linux-3.16.0-4-amd64-x86_64-with-debian-8.2
[debug] Proxy map: {}
[youtube] Setting language
[youtube] 2svVkkNuSq0: Downloading webpage
[youtube] 2svVkkNuSq0: Downloading video info webpage
[youtube] Confirming age
ERROR: Unable to extract sts; please report this issue on https://yt-dl.org/bug . Be sure to call youtube-dl with the --verbose flag and include its complete output. Make sure you are using the latest version; type  youtube-dl -U  to update.
Traceback (most recent call last):
File "/usr/lib/python2.7/dist-packages/youtube_dl/YoutubeDL.py", line 516, in extract_info
ie_result = ie.extract(url)
File "/usr/lib/python2.7/dist-packages/youtube_dl/extractor/common.py", line 170, in extract
return self._real_extract(url)
File "/usr/lib/python2.7/dist-packages/youtube_dl/extractor/youtube.py", line 623, in _real_extract
r'"sts"\s*:\s*(\d+)', video_webpage, 'sts'),
File "/usr/lib/python2.7/dist-packages/youtube_dl/extractor/common.py", line 391, in _search_regex
raise RegexNotFoundError(u'Unable to extract %s' % _name)
RegexNotFoundError: Unable to extract sts; please report this issue on https://yt-dl.org/bug . Be sure to call youtube-dl with the --verbose flag and include its complete output. Make sure you are using the latest version; type  youtube-dl -U  to update.
youtube-dl -U says:
"It looks like you installed youtube-dl with a package manager, pip, setup.py or a tarball. Please use that to update."

You are using an outdated version of youtube-dl. Read our FAQ if you have troubles updating.

I feel dumb now. Thanks.


Redirect /space/collaborators and /login/refresh to Auth

Redirect POST/DELETE /space/:spaceID/collaborators to Auth. Tested in chrome and 307 redirects work fine with POST/DELETE.
Redirect POST /login/refresh to Auth (instead of delegation)
Drop space resources table

It's part of the cleanup phase of https://github.com/fabric8-services/fabric8-auth/issues/79

@alexeykazakov snapshot fabric8-wit image is available for testing.  docker pull docker.io/fabric8/fabric8-wit:SNAPSHOT-PR-1660-1

[test]

[test]

[test]

@alexeykazakov snapshot fabric8-wit image is available for testing.  docker pull docker.io/fabric8/fabric8-wit:SNAPSHOT-PR-1660-2

@alexeykazakov snapshot fabric8-wit image is available for testing.  docker pull docker.io/fabric8/fabric8-wit:SNAPSHOT-PR-1660-5

@alexeykazakov snapshot fabric8-wit image is available for testing.  docker pull docker.io/fabric8/fabric8-wit:SNAPSHOT-PR-1660-6

@alexeykazakov snapshot fabric8-wit image is available for testing.  docker pull docker.io/fabric8/fabric8-wit:SNAPSHOT-PR-1660-7

Replaced by https://github.com/fabric8-services/fabric8-wit/pull/1667

@alexeykazakov snapshot fabric8-wit image is available for testing.  docker pull docker.io/fabric8/fabric8-wit:SNAPSHOT-PR-1660-8


Add Rank to User Profile

Description
This pull request introduces the concept of "rank" to the user profiles on the HappiHub platform. The rank allows users to have a hierarchical status within the platform, enhancing the user experience and enabling new features based on user rank.
Changes Made

Added a rank field to the userSchema in userModels.js.
Updated the userController.js to handle the rank in profile updates.
Modified the userRoutes.js to accommodate the rank during user updates.
Updated the GUIDE.md to document the new rank feature and how it is used within the platform.

How to Test

Update User Profile with Rank:

Use Postman or a similar tool to send a PUT request to /api/users/profile with a valid rank field in the request body.
Ensure the user profile is updated correctly and the rank is saved in the database.
Verify that the response returns the updated profile including the new rank.

Breaking Changes

The user profile schema now includes a rank field. This field must be provided in update requests to ensure data consistency.

Resources

Updated GUIDE.md for details on the rank feature.
Relevant files: userModels.js, userController.js, userRoutes.js.

Checklist

[x] Code follows the style guidelines of this project
[x] Changes have been documented in the GUIDE.md
[x] Added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] Any dependent changes have been merged and published in downstream modules

Please review the changes and provide feedback. Thank you!

ok pour le merge


Possible for custom classes to merge?

Would it be possible for some of the custom classes from homebrew and such to get merging? Such as Dread Knight with Demon?

It might be if they are reusing spellbooks. But if not then it would require additional work to optionally integrate with other mods. I have stopped playing wrath and don't have time for additional features now. But since this is open source I can accept any contributions/PRs.


Add foam particles to player and for enemy explosions.

Add EnemySpawner and move logic from Game to it.
Add ResourceMap that stores all transient resources in a map.
Add AssetLoader that generates async load tasks to populate resources.
Add foam bubble texture, some Rgba styles.
https://github.com/cpp-gamedev/spaced/assets/16272243/c9d76282-924e-4b08-a82f-0a6271872c9e

Merging in 12h to unlock dependent work


Cldr43

Checklist

[ ] At least one test case is included for this feature or bug fix.
[ ] ReleaseNotes has added or is not needed.
[ ] This PR has passed all of the test cases on Nodejs and Chrome Browser.
[ ] This PR has passed all of the test cases on QT/QML.
[ ] This is an API breaking change.
[ ] Requires a major version change.

Issue Resolved / Feature Added
Resolution
Links

Abandon in favor of Goun's PRs instead


[17.06] backport "Do not add duplicate platform information to service spec"

Backport fix:

moby/moby/pull/33867 Do not add duplicate platform information to service spec

With cherry-pick of moby/moby@da85b62:
$ git cherry-pick -s -x -Xsubtree=components/engine da85b62
$ git cherry-pick -s -x -Xsubtree=components/cli/vendor/github.com/docker/docker da85b62

LGTM

The code duplication between the client library itself and the vendored copy in the same repo feels very strange to me. I do worry it will lead to mistakes in the future. But I understand that it's necessary with the current docker-ce concept.
So LGTM, I guess.

In this case, isn't it true that only the code under /cli/vendor/github.com/docker/docker will be used?


[Improvement][Task] Use default sql segment separator for the SQL task

Search before asking

[X] I had searched in the issues and found no similar feature requirement.

Description
Most usage scenarios, use ; as sql separator,  there is no need to enter a delimiter.
Associated issue: #9177
Are you willing to submit a PR?

[X] Yes I am willing to submit a PR!

Code of Conduct

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

This issue has been automatically marked as stale because it has not had recent activity for 30 days. It will be closed in next 7 days if no further activity occurs.

no stale


Pass variables through playbook

Hi there, I've created playbook which tries to pass some mandatory vars:
- hosts: all
  become: true
  vars:
    grafana_security.admin_password: adm_pwd
  roles:
    - role: cloudalchemy.grafana

..but it doesn't work:
ERROR! Invalid variable name in vars specified for Play: 'grafana_security.admin_password' is not a valid variable name

Please advice how to do it correctly?

You need to specify full grafana_security dictionary. Ansible doesn't allow changing only parts of dicts defined in defaults/main.yml
So in your case you need to specify:
- hosts: all
  become: true
  vars:
    grafana_security:
      admin_password: adm_pwd
      admin_user: adm
  roles:
    - role: cloudalchemy.grafana

@paulfantom that did help, many thanks for fast reply!


Layout/BlockAlignment does not recognize "start of line" containing endless method

Based on the rule description for Layout/BlockAlignment with start_of_line

the end shall be aligned with the start of the line where the expression started

I would expect this
def foo = bar(123,
              456) do
  # ...
end

to be allowed, but it isn't. It seems that "start of the line" actually means the start of the method body, and not the actual line.

Expected behavior
No offense
Actual behavior
woot.rb:4:1: C: [Correctable] Layout/BlockAlignment: end at 4, 0 is not aligned with bar(123, at 1, 10 or 456) do at 2, 14.
end
^^^

Steps to reproduce the problem

Enable Layout/BlockAlignment with EnforcedStyleAlignWith set to start_of_line or either
Lint the code block above

RuboCop version
1.62.1 (using Parser <IP_ADDRESS>, rubocop-ast 1.31.2, running on ruby 3.2.2) [arm64-darwin21]
  - rubocop-rails 2.22.1

ElseAlignment also has this issue, but not EndAlignment or BeginEndAlignment, interestingly.


Write a TypeScript program to convert centimeters to feet

Description
Write a TypeScript program to convert centimeters to feet
1 foot = 30.48 centimeters
Input  : 152.4
Output : 5

How to contribute

Save the solution in program/ConvertCentimetersToFeet.ts file
Add ConvertCentimetersToFeet.ts file in convert-centimeters-to-feet folder

!assign


OAK-9564: Make the renewLease mechanism a bit more tolerant against unexpected updates

Added a new RuntimeID field to ensure the update is coming from the same ClusterNode.
Added a new condition to ensure not older updates are executed.

If an update comes from the same RuntimeID and it's newer, we will still accept it even if it was not expected at that moment.

Introducing the runtimeId makes previousLeaseEndTime obsolete. The latter can now be removed from ClusterNodeInfo.
https://github.com/apache/jackrabbit-oak/pull/359/files#diff-f9b067ac268deaf9345ecba3c52f7028bb596a533dd47d902dc9c0d8e29c2646R311

There are test failures with the current changes in the PR: https://ci-builds.apache.org/job/Jackrabbit/job/oak-trunk-pr/job/PR-359/3/testReport/junit/org.apache.jackrabbit.oak.plugins.document/?cloudbees-analytics-link=scm-reporting%2Ftests%2Ffailed
I can also reproduce them locally. Can you please analyze why they fail and whether they need to be updated because some assumptions no longer hold?

There are test failures with the current changes in the PR: https://ci-builds.apache.org/job/Jackrabbit/job/oak-trunk-pr/job/PR-359/3/testReport/junit/org.apache.jackrabbit.oak.plugins.document/?cloudbees-analytics-link=scm-reporting%2Ftests%2Ffailed
I can also reproduce them locally. Can you please analyze why they fail and whether they need to be updated because some assumptions no longer hold?

Finally! Could you please review again? @mreutegg


"News" is not new at all

The News box seems to contain not very new information, with the most recent being from 11 months ago

I can remove it if we want to do that. Also, maybe we should just automatically not display "news" that are more than a certain age.

My alpha branch implements infinite scrolling news; it'll always show the ~5 latest news on top, and you can infinitely load more items as you scroll down.
https://github.com/icssc/AntAlmanac/blob/aponia-monorepo/apps/client/src/components/Header/News.tsx
However, this doesn't address the concern of filtering for recent dates. Any thoughts? @MinhxNguyen7

@johnlorenzini is working to replace this with the patch notes modal I think. Having infinite scroll on patch notes could make sense.

This feature was inspired by the notification bell on YouTube; clicking on the bell opens a popup that can scroll infinitely (I don't have any notifications so it's empty).

I don't think it makes much sense to implement this for the patch notes modal mainly because modals shouldn't be the primary way of displaying lot of info. Also, the infinite scroll exists as a way to browse previous news if the user is interested in viewing them, but the patch notes modal is currently intended to be hidden unless the user has never seen the modal before, defeating the purpose of being able to view the news at any time

I'm not that familiar with what the vision with the news/patch notes is. You'll just have to ask @johnlorenzini


Update to Kafka 1.1.0

It seems that org.apache.kafka.test.ProcessorTopologyTestDriver has disappeared from Kafka 1.1.0, but this library relies on it to create a Fake topology for tests. Did you verify this?

It looks like it was removed on this commit, so ProcessorTopologyTestDriver no longer exists in branch trunk, but it still existed at the time 1.1.0 was released, so it should be ok to update to 1.1.0.
What do you think?


generating random secret key if it does not exist

On a more general note, do we need to generate a random file? This possibly create issues later someone uses this file to encrypt their password - then (accidentally) deletes the file - and we generate a random file which is then used to decrypt the passwords - and that fails (due to the key being wrong) thus rendering all generated passwords now useless.
Can we just throw some warnings/errors that spell out what the problem is and then let the execution continue?

@santhosh-warrior please approve if you want this merged


[BIOMAGE-1563] Add a horizontal scroll to the Data Management tile in Data Exploration module

Background
Link to issue
https://biomage.atlassian.net/jira/software/c/projects/BIOMAGE/boards/1?modal=detail&selectedIssue=BIOMAGE-1563
Link to staging deployment URL
Links to any Pull Requests related to this
Anything else the reviewers should know about the changes here
Changes
Code changes
Definition of DONE
Your changes will be ready for merging after each of the steps below have been completed:
Code updates
Have best practices and ongoing refactors being observed in this PR

[ ] Migrated any selector / reducer used to the new format
[ ] Validated that current unit tests work as expected and are enough

Testing

[ ] Unit tests written
[ ] Tested locally with Inframock (with latest production data downloaded with biomage experiment pull)
[ ] Deployed to staging
[ ] Passed integration tests

To set up easy local testing with inframock, follow the instructions here: https://github.com/biomage-ltd/inframock
To deploy to the staging environment, follow the instructions here: https://github.com/biomage-ltd/biomage-utils
Documentation updates
Is all relevant documentation updated to reflect the proposed changes in this PR?

[ ] Relevant Github READMEs updated
[ ] Relevant wiki pages created/updated

Approvers

[ ] Approved by a member of the core engineering team
[ ] (UX changes) Approved by vickymorrison (this is her username, tag her if you need approval)

Just before merging:

[ ] After the PR is approved, the unstage script in here: https://github.com/biomage-ltd/biomage-utils is executed. This script cleans up your deployment to staging

Optional

[ ] Photo of a cute animal attached to this PR

I have to scroll down to the bottom to be able to access the horizontal scroll bar (see video). Can you make it so that the horizontal scroll bar appears without me having to scroll down to the bottom? That would make it easier to access the controls.
https://user-images.githubusercontent.com/2862362/143431609-0620a425-6f21-4c89-9a0b-8380192508d3.mov


ContextMenu API Annotations

Contains
Adds API annotations to the required ContextMenu Classes in order for the context menu of NUI to be used with modules.
How to test
Create a module that uses the contextMenu from NUI and test it out.
Current version of Scenario module uses it with PR https://github.com/Terasology/Scenario/pull/13

Hooray Jenkins reported success with all tests good!


[NUMBERS-149] port tests in commons-lang for Fraction.

Can you rebase this on master to bring in the new pow() function and then we can see what is still not compatible between lang and numbers.

Can you rebase this on master to bring in the new pow() function and then we can see what is still not compatible between lang and numbers.

@aherbert done.

Coverage remained the same at 99.634% when pulling 2f3b93860518ff94f295a55d32eff8491bdd5c50 on XenoAmess:port_commons_lang_Fraction into c3d173ae9cf25329644b87b8e064f56a2c31ea53 on apache:master.

@aherbert
sqruashed.
please wait for travis-ci & recheck.


feat(python): add ftdetect & ftplugin support for requirements.txt

Not a fan of this. Closing...
Thank you anyway!


About experiments

When i run the wire_SISR，the result i got is lower than the paper。
I got this：

While the paper is 27...., 0.90....
Are there parameters need to be changed？

@XLR-man in our experiments, we resized image to 1/2 (instead of 1/4), and use omega=8 and sigma=8. Could you try those settings instead?

Well，your parameters is resized image to 1/3，and use omega=8，sigma=6，so got the results above。
And I try your advice，I got the results below：

the ssim seems lower，can’t achieve 0.90


PP1739 Tidy up repository as part of Open Source

WHAT
Adding a brief contribution guide and moving the API spec from the readme.

There were no errors, go have a cup of coffee...

There were no errors, go have a cup of coffee...


Possible bug rendering sprites with "Red" in the file name

Bevy version
0.10.1
What you did
I'm just rendering different sprites which are effectively the same but in a different colour.
What went wrong
Sprites with Blue in the filename render correctly.
Sprites with Red in the filename do NOT render correctly.
The filenames look like this:

Archer_Blue2.png
Archer_Red2.png

Anyone would think this is an issue with the Red PNG. That's what I thought too. But then I tried the following:
🟢 Rename the red sprite to Archer_Blue2.png - it renders the sprite correctly
🔴 Rename the blue sprite to Archer_Red2.png - it does NOT render the sprite
🟢 Rename the red sprite to Archer_Redi2.png (basically add an i character so it's not just "Red") - it renders the sprite correctly
At this point, I think I'm either going insane or Bevy is doing something weird. I even thought that the sprite sheets were messed up and somehow being "fixed" when I copied them to give them a new name. But the Blue ones that work correctly stop working when I just rename it to have Red in the name.
I know there shouldn't be a reason for Bevy misbehaving, but I've proved that the Blue sprite that renders correctly stops working if I rename it to Red, and viceversa. This is a long shot, but any ideas?
Additional information
This issue happens on both Linux and Windows. I tried on both to make sure Linux wasn't doing anything doggy.
Source code that is having the issue https://github.com/srodrigo/strategy-game-rs/blob/ch02-wip/02-units/src/main.rs#L331
Assets are in assets folder as expected.

Hilarious! I can reproduce. I'm investigating

@nicopap  haha no worries. Hilarious indeed... I thought I was loosing my mind, but good that you can reproduce it too.

It gets even more strange...
Similar issue with Archer_Green3 and Archer_Green2. Green3 works, Green2 doesn't.
I've tried the same rename trick and renamed Blue2 to Green3, and it works. But if I rename Blue2 to Green2, it doesn't!
At least there is some consistency in the weirdness... but not it seems numbers are involved too?
😵

I found where the bug comes from. The archer is spawned regardless of the file name. What actually happens is that which sprite is rendered in front of another is more or less undefined. The issue is that the translation.z of both the archer and the terrain is the same. Which means sometimes the terrain renders on top, sometimes the archer renders on top. It probably depends on which order the images are loaded (which the file name might influence)
I found that other file names would trigger the behavior. For example, just naming the file  d.png would cause the archer to not appear.

OMG interesting... So the filename might be playing its role.
I didn't thought of looking into the z index, as usually sprites rendered later appear on top of existing sprites, at least on most engines. But I'm happy with the z index, it's a good practice anyway (that I forgot about...).
Thanks a lot for helping out 🙏
If there's already a discussion going on, I'm happy for this issue to be closed. Up to you, whatever is more useful.

This can be safely closed. But feel free to keep it open. It is a real issue, and the culprit is not you, but bevy: For not telling you what was happening, and not giving you the tools to figure it out.
I would recommend you consider using something like https://github.com/jakobhellermann/bevy-inspector-egui, it gives much more visibility on what is going on in the game. I think with it you would have immediately understood what was the issue and how to fix it.

Thanks for the tip! I'll give it a try :)

Definitely want better tools to detect and warn about z-fighting! See #1275 :D Going to close this out though as I think that this issue isn't super clear for people trying to tackle it :)


feat(site-builder):SEINT-275 Add http headers default cache-control to public, max-age=3600

After discussion with @Tzal3x we decided to leave this default cache-control policy for the future. Lets leave this PR open-pending and we decide depending on the type of sites being published.


feat(TCOMP-332) : Add SSL/TLS common properties

Please check if the PR fulfills these requirements

[x] The commit message follows Talend standard
[x] Tests for the changes have been added (for bug fixes / features)

What is the current behavior? (You can also link to an open issue here)
SslProeprties is a copy from tSetKeystore component, which used for DI and Spark component.
https://jira.talendforge.org/browse/TCOMP-332
What is the new behavior?
Does this PR introduce a breaking change?

[ ] Yes
[x] No

If this PR contains a breaking change, please describe the impact and migration path for existing applications: ...
Other information:

Pull request test result: failure :x:

Hello, I just found I'd better to rename all the https to ssl, I will push one commit soon.

Pull request test result: failure :x:

LGTM but there is a build failure to be fixed.

thx @sgandon, but I don't know how to fix the error for snowflake unit test, not sure it's relate to this PR

indeed the snowflakes errors are not related to this issue, please merge it.


PNG Signature !

Error: Command failed: java -jar /usr/local/lib/node_modules/apk-mitm/jar/apktool.jar build /tmp/2c68de93fb75c4923037827b0682719f/decode --output /tmp/2c68de93fb75c4923037827b0682719f/unsigned.apk --use-aapt2
W: /tmp/2c68de93fb75c4923037827b0682719f/decode/res/drawable-mdpi/src_assets_img_background.png: error: failed to read PNG signature: file does not start with PNG signature.
brut.androlib.AndrolibException: brut.common.BrutException: could not exec (exit code = 1): [/tmp/brut_util_Jar_9729835945442411667.tmp, compile, --dir, /tmp/2c68de93fb75c4923037827b0682719f/decode/res, --legacy, -o, /tmp/2c68de93fb75c4923037827b0682719f/decode/build/resources.zip]
I: Using Apktool 2.4.0
I: Checking whether sources has changed...
I: Smaling smali folder into classes.dex...
I: Checking whether sources has changed...
I: Smaling smali_classes2 folder into classes2.dex...
I: Checking whether resources has changed...
I: Building resources...

I don't think Apktool can do anything here. If it was given a bad PNG, it error`d out. I would expect that. What else can we do in that situation? Fill a fake resource? I'd much rather error out.
Closing as non-issue.

@iBotPeaches How about adding option to ignore error? I'm sure it would still work since the original APK works fine.

good suggestion, unfortunately this error is still ongoing


Merge pull request #1 from jdpguerra/sn_instances/dev278408

Initial state of application at link to source control

va


Metadata example is giving an error

I loaded the Metadata example on try.haxe.org and it compiles with this error:
TypeError: haxe_rtti_Meta.getFields(...).value is undefined,[https://try.haxe.org/program/eaACf364/run?r=0.9682007150246583](https://try.haxe.org/program/eaACf364/run?r=0.9682007150246583#23),line:23

seems like it needs DCE No/Std or analyzer off for Haxe versions beyond 3.2.1


sample of training about Webvid_2M

I randomly selected 500 videos in webvid_2m data for training, and set max_train_steps to 10000. Below, I used the last obtained motion checkpoints sampling results. I did not change other configurations, but the results were very bad.

Can someone tell me why? Thank you very much.
train:
torchrun --nnodes=1 --nproc_per_node=3 --master_port 20456 train.py --config configs/training/v1/training.yaml
training.yaml.txt
inference:
python -m scripts.animate --config /nvme-ssd/yanyi/code/AnimateDiff-main/configs/prompts/my_config.yaml
my_config.yaml.txt

hello, is the unet_checkpoint_path: "/nvme-ssd/yanyi/code/AnimateDiff-main/output_dir/image_finetune/image_finetune-2024-04-07T10-18-11/checkpoints/checkpoint.ckpt" in the training.yaml is trained on webvid_2m? However, in fact, the image finetune stage in the paper should be to train a domain adapter, but the setting in the config is to train a complete unet. Have you found this problem?

Thanks for answering my question, the unet_checkpoint_path is trained on webvid_2m_val, the profile I trained this checkpint with is image_finetune .yaml, the reason why the profile here is training.yaml, I want to train video actions on fine-tuned checkpoints. I used the same dataset and training steps in another animateDiff-based project to train and implement image animation, seems to be a problem with xformers == 0.0.16, but pip install xformers > 0.0.16 need to install torch > 2.0, I used another anaconda virtual environment, using xformers == 0.0.22.

Hello, I met the similar problem. I use the vailation of webvid-2M, about 5k videos, to finetune the motion lora. For 50 steps, the results are similar like you(different imgs change quickly). For 2.5k steps, the results tend to be stable, and no difference between frames. I think we use small dataset, or there are some configurations I ignored. I also hope a solution for this problem.

您好，我遇到了类似的问题。我使用 webvid-2M（大约 5k 视频）的 vailation 来微调运动 lora。对于 50 个步骤，结果与您相似（不同的 imgs 变化很快）。对于 2.5k 步长，结果趋于稳定，帧之间没有差异。我认为我们使用小数据集，或者我忽略了一些配置。我也希望这个问题能得到解决。
Thank you for your reply. Increasing the number of training steps can indeed improve the results.


Custom tables for statblocks?

I'm working on a gear/creature browsing site for an unrelated scifi game, Eclipse Phase… I've been making pretty extensive use of rpg-cards in my own sessions for equipment and loot, but the statblock format used by EP is understandably different than D&D.
I hacked up a local version of RPG-Cards that supports an 'epstats' table specifically for its stat system. It works fairly well, but I'm curious if you think it might be a generic feature worth moving to the main project?

Hello eaton and Crobi, I am in a similar situation:
Instead of "Eclipse Phase" I am using "Savage Worlds". And also hacked up a local version of RPG-Cards.
Maybe all those customs statblocks are worth moving to the main project in a modularized way? And perhaps we could add those systems/statblocks to the documentation?