Shell decoration: show the last return code status on current prompt

The decoration on current prompt, shows the status of current command execution.
The problem is that if you execute a command that produces a long output, in order to see the status of the just executed command and/or copy its output, the user has to scroll all the way up to the previous prompt in order to see its decoration.
I wish the decoration on the active prompt/line would actually show the status code of the previous command that was run.
What this means is that if you execute some command which produces a long output (taller than the terminal height), once it returns and you get your prompt back, the decoration you will see would be the status of the previous command: you don't have to scroll all the way up to see what was the status of your command and/or copy its output.

That's how you would do it with a traditional prompt but it would be pretty confusing if it was like that here. You should use cmd+up/down to quickly see what the exit code if you need it, or keep an indicator in your regular prompt


iDEAL confirmIdealSetup missing API

Hello, dear, Stripe contributors,
while integrating stripe for our client's platform, we've stumbled upon a piece of documentation which describes how one would be able to setup recurring iDEAL payments, but at the same time @stripe/react-stripe-js is missing API described here.
implementation code is pretty straightforward
function IDealPayment() {
  const { stripe } = useStripe();

  function submitHandler() {
    // ...
   stripe._**confirmIdealSetup**_(...); // missing
    // ...
  }

  return <Component... onSubmit={submitHandler} />
}

Am I missing something or is this feature not released yet documentation is already there?
Many thanks 🙏

Hi @mvirbicianskas, thanks for reporting. I've just published v1.10.0, which includes the missing stripe.confirmIdealSetup type.

hey, @christopher-stripe, very much appreciated 🙏  will def be trying it out 👋


configStore.get(that.name) returns undefined causes uncaught exception

[31merror 2015-03-10T18:05:24.806Z: 5c00c7dd3ce2 pid:560 uncaughtExceptionHandler (requirements/uncaughtExceptionHandler.js:15) Aborting on uncaught top-level exception:  TypeError: Cannot read property 'firstChunk' of undefined
    at DestroyableTransform._transform (/opt/bb/experiments/build/node_modules/gcloud/lib/storage/file.js:866:58)
    at DestroyableTransform.Transform._read (/opt/bb/experiments/build/node_modules/gcloud/node_modules/through2/node_modules/readable-stream/lib/_stream_transform.js:184:10)
    at DestroyableTransform.Transform._write (/opt/bb/experiments/build/node_modules/gcloud/node_modules/through2/node_modules/readable-stream/lib/_stream_transform.js:172:12)
    at doWrite (/opt/bb/experiments/build/node_modules/gcloud/node_modules/through2/node_modules/readable-stream/lib/_stream_writable.js:237:10)
    at writeOrBuffer (/opt/bb/experiments/build/node_modules/gcloud/node_modules/through2/node_modules/readable-stream/lib/_stream_writable.js:227:5)
    at DestroyableTransform.Writable.write (/opt/bb/experiments/build/node_modules/gcloud/node_modules/through2/node_modules/readable-stream/lib/_stream_writable.js:194:11)
    at write (/opt/bb/experiments/build/node_modules/gcloud/node_modules/through2/node_modules/readable-stream/lib/_stream_readable.js:623:24)
    at flow (/opt/bb/experiments/build/node_modules/gcloud/node_modules/through2/node_modules/readable-stream/lib/_stream_readable.js:632:7)
    at /opt/bb/experiments/build/node_modules/gcloud/node_modules/through2/node_modules/readable-stream/lib/_stream_readable.js:600:7
    at process._tickCallback (node.js:419:13)
```

One possible cause is that the startupUpload()->makeAuthorizedRequest()'s error handler is handleError(), which can call setTimeout(resumeUpload) if the err.code is between > 499 and < 600. And skip the configStore.set(that.name).
One fix is: if handleError() is triggered by the startupUpload(), it should never trigger resumeUpload().

I think I know the cause. The file.js use configure store ('gcloud-node') to keep track of the object upload process. If there are multiple node processes running at the same time, they may overwrite others' config file and cause problems. This effectively prohibit multiple node process running (under one user) to access the GCS at the same time. This is a serious limitation and is not documented anywhere.
I don't understand this design. Why do you need to use persist configure store to keep track every object's upload process? Why cannot you use a simple in-memory-only object to do that? Resumable upload is nice to have, but do you really need to think resumable-upload cross node process restart (at a such high expense)?

I make some changes like below and my stress test that uses multiple node processes is now happy.
diff --git a/node_modules/gcloud/lib/storage/file.js b/node_modules/gcloud/lib/storage/file.js
index 7c04437..adfdf35 100644
--- a/node_modules/gcloud/lib/storage/file.js
+++ b/node_modules/gcloud/lib/storage/file.js
@@ -21,7 +21,14 @@
 'use strict';

 var bufferEqual = require('buffer-equal');
-var ConfigStore = require('configstore');
+//var ConfigStore = require('configstore');
+var all = {};
+var configStore = {
+   all: {},
+   get: function(key) { return all[key]; },
+   del: function(key) { delete all[key]; },
+   set: function(key, value) { all[key] = value; },
+};
 var crc = require('fast-crc32c');
 var crypto = require('crypto');
 var duplexify = require('duplexify');
@@ -766,7 +773,7 @@ File.prototype.startResumableUpload_ = function(stream, metadata) {
   metadata = metadata || {};

   var that = this;
-  var configStore = new ConfigStore('gcloud-node');
+  //var configStore = new ConfigStore('gcloud-node');
   var config = configStore.get(that.name);
   var makeAuthorizedRequest = that.bucket.storage.makeAuthorizedRequest_;

I suppose the real important question here is: Do we really need to support resumable across node process restarts? I'm going to assume yes, so we just likely need better error checking to ensure that the value exists at the time we need to use it, otherwise resumable will not be used.


New tutorials

This PR includes two new tutorials, promised as part of my Processing Foundation fellowship (and delivered very late 😞 ), on the topic of using Processing.py at the command line and reasoning about Python/Jython/Java integration. Also includes #100 and #97 which add author names to tutorials (among other fixes). The text of the tutorials may require an additional copy-editing pass—if anyone has a second to look them over for typos or errors, that would be great!

This is fantastic. My impulse is simply to accept the patch and worry about nits and copyediting later, because I'm so time-constrained. Does that suit you?

that works for me!


Question: Configuring two nodes on the same server

I have a server with multiple IP addresses and enough processing and memory power to run two mater nodes.
I am wondering if I can configure the two nodes on the server by having each one use a different IP address.
Also wondering if the networkid parameter when running gman, has anything to do with it:
gman --datadir chaindata  --networkid 1 --outputinfo 1 -- syncmode full
Thanks in advance.

u can use docker for it


ui: fix add Vmware cluster

Description
When adding a cluster from existing host-datacenter UI shoudl list and autofill host, DC name while making addCluster API call.

Types of changes

[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] New feature (non-breaking change which adds functionality)
[x] Bug fix (non-breaking change which fixes an issue)
[ ] Enhancement (improves an existing feature and functionality)
[ ] Cleanup (Code refactoring and cleanup, that may add test cases)

Feature/Enhancement Scale or Bug Severity
Feature/Enhancement Scale

[ ] Major
[ ] Minor

Bug Severity

[ ] BLOCKER
[ ] Critical
[ ] Major
[x] Minor
[ ] Trivial

Screenshots (if appropriate):
Before fix:

After the fix, Center cluster successfully added
How Has This Been Tested?
UI

@blueorangutan ui

@shwstppr a Jenkins job has been kicked to build UI QA env. I'll keep you posted as I make progress.

UI build: :heavy_check_mark:
Live QA URL: http://qa.cloudstack.cloud:8080/client/pr/4663 (JID-3832)


Wrong version number info outputted by latest release

Adam,
In the latest release (v0.2.0) the executable is outputting "go-lockss dev build" for the version number instead of the project name's actual version number info (i.e. v0.2.0).
Example output from running hayoh executable on lockss-adpn2.  The executable was installed using the go-lockss-v0.2.0.x86_64.rpm package.
[zrbdmas@lockss-adpn2 ~]$ hayoh --version

go-lockss dev build
https://github.com/atc0005/go-lockss

The output is the same when running the uncompressed binary hayoh-linux-amd64.
[zrbdmas@lockss-adpn2 ~]$ ./hayoh-linux-amd64 --version

go-lockss dev build
https://github.com/atc0005/go-lockss

@aucompbiker Thanks for reporting this. Will get this fixed in the next release.

@aucompbiker
This should be resolved in the v0.2.1 release:

https://github.com/atc0005/go-lockss/releases/tag/v0.2.1

Thanks again for reporting this.


Physics; nape components; Rotation argument for BoxCollider

Especially useful when the body is static and cannot be rotated after
being added to scene.

Good idea!
Can you tweak it, and add import luxe.Log.*; and use def( instead?

Sure, it's done :)

Thanks!


RFC: Custom type-safe directives

Summary

Hi!
I'm building a GraphQL client leveraging the power of gql.tada (it's really mighty!), and I would like to implement a @paginate directive to enable dynamic pagination following relay specification.
query {
  me {
    friends(first: 10) @paginate {
      pageInfo { hasNextPage endCursor }
      edges { node { id } }
    }
  }
}

I'd like this directive to be typed, i.e. its effects are visible in the query output type.
(The underlying implementation of the directive is for me to build)
Proposed Solution

I'd like to declare the @paginate directive in initGraphQLTada so that the result type of the query is augmented with the directive:
const graphql = initGraphQLTada<{
  // ...
  directives: {
    paginate: <Output extends { pageInfo: {}; edges: Array<{}> }>(output: Output) => (
      Output & { paginated: Reactive<Output> }
    )
  }
}>()

// The request output type would be:
type Q = {
  me: {
    friends: {
      pageInfo: { hasNextPage: boolean; endCursor: string };
      edges: Array<{ node: { id: string } }>;
      paginated: Reactive<{
        pageInfo: { hasNextPage: boolean; endCursor: string };
        edges: Array<{ node: { id: string } }>
      }>
    }
  }
}

I've been trying to build a prototype of this, but I can't find a way to properly instantiate the generic parameter of directives['paginate']. Maybe this feature requires advanced black magic, I'll keep digging.
Requirements

None?

Just to add this as a side-note for arguments on directives themselves to be typed or rather for GraphQLSP to recognise them, the schema that's passed to GraphQLSP/gql.tada has to be modified.
That said, regarding result types per directives, I strongly disagree with supporting this, sorry 😅 By spec and by convention, directives should never alter the return types of fields they're being put on.
We've reached a similar conclusion for @urql/exchange-graphcache, where client-side directives are supported (and automatically filtered by them having a _ prefix). The result type, while it can be modified, should never be modified.
This is even more important because, if we did alter the return type, the selection and the schema wouldn't be accurate any longer. A GraphQL schema/resolve/API can by following the spec never actually go against this either.
While it may seem for Reactive to be an exception, imho, it makes a lot more sense to have a wrapper for this, or something on the actual usage site, that either:

enforces that a certain field is "reactive" (depending on what that means in the context of your app/project)
enforces that an entire result type is altered so the Reactive type doesn't have to only be applied to a field

Thanks for your quick reply!
Having GraphQL-LSP to recognize custom client-side directives is as simple as that:
# .graphqlrc.yml
schema: [schema.gql, client-directives.gql]

# client-directives.gql
directive @paginate(asc: Boolean = True) on FIELD

By spec and by convention, directives should never alter the return types of fields they're being put on.

Depending on your exact definitions, you can consider than most directives actually alter the return types of the fields they're being put on:
# schema.gql
type Query {
  resolver: String!
  slowResolver: String!
}

# query.gql
query {
  resolver @skip(if: true)
  ... @defer {
    slowResolver
  }
}

For this query, without directives, I'd expect the output type to be
type Output = {
  resolver: string;
  slowResolver: string;
}

Yet gql.tada rightfully produces the following type:
type Output = {
  resolver?: string | undefined;
  slowResolver?: string | undefined;
}

if we did alter the return type, the selection and the schema wouldn't be accurate any longer

100% agree on this and it leads me to a slightly better idea. So far, for all the use cases I have, it turns out I don't need gql.tada to output my actual runtime types, I just need it to annotate that a certain directive was put at a certain place.
For instance, back with @paginate:
query {
  me {
    friends(first: 10) @paginate {
      pageInfo { hasNextPage endCursor }
      edges { node { id } }
    }
  }
}

Would it be possible to have gql.tada output the following type?
type Output = {
  me {
    friends: {
      pageInfo: { hasNextPage: boolean; endCursor: string | null };
      edges: Array<{ node: { id: string } }>;
      '@paginate': true;
      // or `[DirectivesSymbol]: ['@paginate']` with `const DirectivesSymbol = Symbol()`
      // or something else that cannot clash with actual runtime types
    }
  }
}

Then, to use it with a frontend framework, I'd write the following code
import { paginate } from 'somewhere';

export let data: Output;

const friends = paginate(data.me.friends);

// friends now has additional runtime features:
// - friends.loadMore() to load more pages
// - friends.isLoading() to know if a page is loading
// ... without compromising type-safety! The type of `data.me.friends` reflects
// what the server produces, but under-the-hood, the GraphQL client attached
// additional hidden data to it; it would look roughly like this:
data.me.friends === {
  pageInfo: { hasNextPage: false, endCursor: null },
  edges: [{ node: { id: 'hello' } }],
  [PaginationSymbol]: {
    replayQuery: document,
    // ...
  }
}

Thanks for your time

In your specific example, what stops you from matching the type on { pageInfo: PageInfoLike, edges: readonly EdgeLike[] }, for example?
Since the directive needs to annotate a field and the data needs to match a certain shape, you could also match on the shape and do an additional runtime check that the data is known, which is likely where the runtime value check is coming in?
That said, my concern here is mostly type inference performance. It's already not cheap to parse GraphQL documents in TypeScript's type system (hence, turbo output exists to make sure that cost doesn't balloon)
What I think is a bit more interesting is fragment-level annotations, as those are potentially much cheaper. So, I could imagine this:
const FriendPagination = graphql(`
  fragment FriendsPagination on FriendsConnection @paginate {
    pageInfo { hasNextPage endCursor }
    edges { node { id } }
  }
`)

since that potentially reduces the cost.
That also means that you could leverage the fragment itself at the call site to enforce unwrapping and that the corresponding fragment is known, e.g. paginate(FriendPagination, data)
This solves the cost problem, since we don't have to:

get the directives per field
pass them on when the child is a selection
intersect the directives and map them to an object-like on each selection / clean up empty directives / filter directives for known values

The benefit I see here is that we're never left with a redundant symbol field per selection check, or a type that reflects this, when directives are unknown (i.e. not part of the supported/known set of directives)
The only thing I'm unsure of is whether we should enforce underscore-syntax, which is something we've been pushing for in @urql/exchange-graphcache, where underscore-prefixed directives are automatically assumed to be "private" i.e. not sent to the API.

In your specific example, what stops you from matching the type on { pageInfo: PageInfoLike, edges: readonly EdgeLike[] }, for example?

A bit of ergonomics, the @paginate directive could be in charge of adding pageInfo { hasNextPage endCursor } to the query, so that the developer only has to query the data they use, and let the framework handle the boring stuff. Other than that, absolutely nothing, I thought about it too, and this is a nice and light way to handle this.

What I think is a bit more interesting is fragment-level annotations

I really like your proposal, let me make some experiments and get back to you in a few days. Thank you very much for your tips

The only thing I'm unsure of is whether we should enforce underscore-syntax

This seems like a nice and sensible convention, +1


issue with docs images

As described in the mkdocs documentation, while absolute paths might work in local deployment they are not supported and didn't work in the deployed docs.
This PR fixes the issue

Codecov Report

Merging #178 (06e0586) into main (8d4a9a2) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #178   +/-   ##
=======================================
  Coverage   90.74%   90.74%
=======================================
  Files          67       67
  Lines        1459     1459
=======================================
  Hits         1324     1324
  Misses        135      135


[bug] You can not change loadPrevNext and loadPrevNextAmount params

var photosSlider = new Swiper($('.swiper-container', photos), {
lazy: true,
loadPrevNext: true,
loadPrevNextAmount: 2
});
Swiper will never enter "if (params.loadPrevNext) {" (line 6104 in swiper.js).
Swiper 4.1.6

check docs about syntax:
...
lazy: {
  enabled: true,
  loadPrevNext: true,
  loadPrevNextAmount: 2
}
...

Sorry, still trying to use Swiper in the old way.


LCAM-935| added crime-commons library

What
Link to story
added crime-commons library
Checklist
Before you ask people to review this PR:

[x] Tests should be passing: ./gradlew test
[x] Github should not be reporting conflicts; you should have recently run git rebase main.
[x] Avoid mixing whitespace changes with code changes in the same commit. These make diffs harder to read and conflicts more likely.
[x] You should have looked at the diff against main and ensured that nothing unexpected is included in your changes.
[x] You should have checked that the commit messages say why the change was made.

Could you also add the service configuration required for the web client call ?

Since at this point we are not clear on all the api calls, I'll add the hardship and contribution endpoints that we know of.


Consider: a more modular data mechanism

Having a single data file is a bit annoying. Especially as more data gets added.

Resolved by 009c5673a50326b0d3754ad9b4c6a595cf3a0b1e


Compatibility issue with cx_Freeze with Python 2.7

Currently, jsonschema breaks any attempt to deploy a Python script using cx_Freeze, a common tool for deploying self-consistent Python applications, (not only, but for example) on Windows platforms.
The source of the problem is in jsonschema/compat.py:
try:
    from collections import MutableMapping, Sequence  # noqa
except ImportError:
    from collections.abc import MutableMapping, Sequence  # noqa

When commenting those lines, cx_Freeze works as expected.
The fact is jsonschema is part of the huge dependency list of IPython. IPython is very invasive when installed: even if not needed by your application, deploying with cx_Freeze will take all IPython related modules with it (and jsonschema is among them).
In other words, building the most insignificant Python script will fail only because IPython is installed (which happens a lot) and it depends on jsonschema.
See PR #264

For anyone finding this: see the discussion on #264, which was a proposed fix (and a probably working one I assume :), for what seems like an upstream issue, but it'd need a test to be mergeable.

It seems like the upstream issue was fixed, so I'm going to close this as "unsupported, but willing to support if the above is addressed."


Support source dependencies that follow upstream branches

Source dependencies can use explicit versions (which match on tags), version ranges (which match on tags) or latest.integration (which is interpreted as "tip of default branch").
If you had multiple repositories that all had a develop branch, you might want all of your source dependencies to build against the latest develop and gradually switch dependencies to a stable branch or pin the versions exactly with tags (or commits).
Implementation

[x] error reporting when branch not found.
[ ] can combine version + branch criteria.
[ ] can use with dependency constraint.

An initial implementation is here: https://github.com/gradle/gradle/pull/4260
This change introduces a placeholder API (basically option 3 above) pending some further design work.

Some more changes: https://github.com/gradle/gradle/pull/4271

@adammurdoch Is anything else needed for this?

I want to add a few more tests

This is done enough. Raised a few more issues out of this.


HBASE-27598 Upgrade mockito to 4.x

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 41s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

_ master Compile Tests _

+0 :ok:
mvndep
0m 19s
Maven dependency ordering for branch

+1 :green_heart:
mvninstall
3m 23s
master passed

+1 :green_heart:
compile
4m 32s
master passed

+1 :green_heart:
checkstyle
1m  2s
master passed

+1 :green_heart:
spotless
0m 39s
branch has no errors when running spotless:check.

+1 :green_heart:
spotbugs
9m 19s
master passed

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 14s
Maven dependency ordering for patch

+1 :green_heart:
mvninstall
3m 16s
the patch passed

+1 :green_heart:
compile
4m 31s
the patch passed

+1 :green_heart:
javac
4m 31s
the patch passed

-0 :warning:
checkstyle
1m  8s
root: The patch generated 1 new + 40 unchanged - 0 fixed = 41 total (was 40)

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
xml
0m  1s
The patch has no ill-formed XML file.

+1 :green_heart:
hadoopcheck
12m 48s
Patch does not cause any errors with Hadoop 3.2.4 3.3.4.

+1 :green_heart:
spotless
0m 37s
patch has no errors when running spotless:check.

+1 :green_heart:
spotbugs
9m 56s
the patch passed

_ Other Tests _

+1 :green_heart:
asflicense
0m 45s
The patch does not generate ASF License warnings.

61m 44s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4998/1/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/4998

Optional Tests
dupname asflicense javac spotbugs hadoopcheck hbaseanti spotless checkstyle compile xml

uname
Linux 873d6ccc2c14 5.4.0-1088-aws #96~18.04.1-Ubuntu SMP Mon Oct 17 02:57:48 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / 235864056c

Default Java
Eclipse Adoptium-11.0.17+8

checkstyle
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4998/1/artifact/yetus-general-check/output/diff-checkstyle-root.txt

Max. process+thread count
179 (vs. ulimit of 30000)

modules
C: hbase-http hbase-server hbase-mapreduce hbase-rest hbase-hbtop . U: .

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4998/1/console

versions
git=2.34.1 maven=3.8.6 spotbugs=4.7.3

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
1m  3s
Docker mode activated.

-0 :warning:
yetus
0m  3s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ master Compile Tests _

+0 :ok:
mvndep
0m 17s
Maven dependency ordering for branch

+1 :green_heart:
mvninstall
3m 25s
master passed

+1 :green_heart:
compile
1m 52s
master passed

+1 :green_heart:
shadedjars
4m 24s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
3m  7s
master passed

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 16s
Maven dependency ordering for patch

+1 :green_heart:
mvninstall
3m 13s
the patch passed

+1 :green_heart:
compile
1m 50s
the patch passed

+1 :green_heart:
javac
1m 50s
the patch passed

+1 :green_heart:
shadedjars
4m 24s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
3m  6s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
266m  1s
root in the patch passed.

299m 44s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4998/1/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/4998

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux 1edfe52bb437 5.4.0-135-generic #152-Ubuntu SMP Wed Nov 23 20:19:22 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / 235864056c

Default Java
Eclipse Adoptium-11.0.17+8

Test Results
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4998/1/testReport/

Max. process+thread count
4507 (vs. ulimit of 30000)

modules
C: hbase-http hbase-server hbase-mapreduce hbase-rest hbase-hbtop . U: .

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4998/1/console

versions
git=2.34.1 maven=3.8.6

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 55s
Docker mode activated.

-0 :warning:
yetus
0m  3s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ master Compile Tests _

+0 :ok:
mvndep
0m 19s
Maven dependency ordering for branch

+1 :green_heart:
mvninstall
2m 47s
master passed

+1 :green_heart:
compile
1m 32s
master passed

+1 :green_heart:
shadedjars
4m 16s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
2m 27s
master passed

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 16s
Maven dependency ordering for patch

+1 :green_heart:
mvninstall
2m 40s
the patch passed

+1 :green_heart:
compile
1m 33s
the patch passed

+1 :green_heart:
javac
1m 33s
the patch passed

+1 :green_heart:
shadedjars
4m 13s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
2m 34s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
371m 50s
root in the patch passed.

402m 24s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4998/1/artifact/yetus-jdk8-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/4998

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux 6ee3c7fede3d 5.4.0-135-generic #152-Ubuntu SMP Wed Nov 23 20:19:22 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / 235864056c

Default Java
Temurin-1.8.0_352-b08

Test Results
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4998/1/testReport/

Max. process+thread count
4734 (vs. ulimit of 30000)

modules
C: hbase-http hbase-server hbase-mapreduce hbase-rest hbase-hbtop . U: .

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4998/1/console

versions
git=2.34.1 maven=3.8.6

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.


[explore] nvd3 sort values in rich tooltip

Coverage remained the same at 69.052% when pulling f43834e4292d1ec6db5423d862497748a9d3ec72 on mistercrunch:sort_tooltip into e4fba0ffb75f8210fc0a894d355851bcbfb0d285 on apache:master.

Coverage remained the same at 69.052% when pulling f43834e4292d1ec6db5423d862497748a9d3ec72 on mistercrunch:sort_tooltip into e4fba0ffb75f8210fc0a894d355851bcbfb0d285 on apache:master.

Coverage remained the same at 69.052% when pulling f43834e4292d1ec6db5423d862497748a9d3ec72 on mistercrunch:sort_tooltip into e4fba0ffb75f8210fc0a894d355851bcbfb0d285 on apache:master.

Coverage remained the same at 69.37% when pulling dbd191cf5ba34c55206e3cee356bf06657db9432 on mistercrunch:sort_tooltip into 1e325d964524dc04e1e48aa86dbe463d1a10c81a on apache:master.


Fix 316

Fixes #316 (or, at the very least, hopefully makes it better...).

Codecov Report

Merging #318 into master will increase coverage by 0.14%.
The diff coverage is 62.5%.

I consider this PR ready for review and merging at this point.


PV-RCNN training and testing- Custom dataset-USED METRICS

HELLO folks,
I am training PV-RCNN on a custom dataset that i created myself and i customized as required.
I am trying to locate the METRICS used for testing in order to test the trained model using this command "python test.py --cfg_file ${CONFIG_FILE} --batch_size ${BATCH_SIZE} --eval_all"
Any help will be apperciated.
Thank you

I am confused on what you mentioned METRICS here.

@jihanyang By metrics i meant evaluation criteria such as IoU(intersection over union)
Recall//Precision
RMSE(Root mean square erro ) etc...


Chart configuration horizontal true and shared tooltip false not showing the correct values for all the bars

Description
Chart configuration like horizontal=true and shared tooltip=false and chart type=bar is not showing the correct values in the tooltip. It only show the only first value where in shared=true it shows 3 value. Expected is it should show correct x axis value in tooltip.

Steps to Reproduce

Create a bar chart using apex chart
Set horizontal=true and shared tooltip=false and chart type=bar
Check tooltip it show only first tooltip from X axis values

Expected Behavior
It should show single series but corrected one same like when horizontal is set to false
Actual Behavior
In tooltip it shows only first value in when its single and non stacked bar chart and horizontal is true.
Expected same behavior in horizontal is false as well
Screenshots
See here My cursor is at purple line but tooltip show always blue value.

Reproduction Link

The required reproduction link was not provided


The new beta when clicking on a link it doesn't open in my default browser

Bug description:

Instead, it only opens within the app
Steps to reproduce:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior

Screenshots

Environment:

Operating System:

Ferdi Version:

Server:

Debug information:

At first I thought: nice, it does not involve another browser. But then: in my browser I have Scriptblocker, AdBlocker and such plugins... I would prefer either the old way (default browser) or add an option where to open links. Of course, if at any time there could be plugin support within ferdi, the internal way may be superior.

You can right-click on the link, and choose "Open in default browser" - which should be the behavior that you describe @VOOM108 .
@idonthaveanametoday - please use the same technique (context menu) to see what options work for you as well.
IF the issue can be closed - could I request either one of you to please affirm the suggested options, and then close the bug please?


Performance is better in 1.6.1 release compared to 1.7.4 release in many models

System Info
Optimum-habana - 1.7.4
Synapse AI - 1.12.0
Docker - 1.12.0-463
Gaudi2 (HLS 225) - 1x and 8x.

Information

[X] The official example scripts
[ ] My own modified scripts

Tasks

[X] An officially supported task in the examples folder (such as GLUE/SQuAD, ...)
[ ] My own task or dataset (give details below)

Reproduction
Steps to Reproduce
Writing down the steps to reproduce to run SwinT in 1x

Download and install optimum-habana
git clone https://github.com/huggingface/optimum-habana.git
cd optimum-habana
git chekout v1.6-release
pip install -r examples/image-classification/requirements.txt
pip install optimum-habana==1.6.1
python3 /root//optimum-habana/examples/image-classification/run_image_classification.py  --model_name_or_path microsoft/swin-base-patch4-window7-224 --dataset_name cifar10 --output_dir /tmp/swint_hf/results/ --remove_unused_columns False --do_train --learning_rate 2e-05 --per_device_train_batch_size 64 --evaluation_strategy no --save_strategy no --load_best_model_at_end True --save_total_limit 3 --seed 1337 --use_habana --use_lazy_mode --gaudi_config_name Habana/swin --throughput_warmup_steps 3 --ignore_mismatched_sizes --bf16 --num_train_epochs 1 --logging_steps 20 --dataloader_num_workers 8

Expected behavior
The expected behaviour is that
1.12.0-463 having similar perf with optimum-habana 1.6.1 and optimum-habana 1.7.4
But what is observed is that perf is better in optimum-habana 1.6.1 and comparitively lesser in 1.7.4
This is applicable for SwinT, ViT, Bert-Large in 8x and 1x.
Eg: values in SwinT is given below
OH - 1.7.4 values
362.524
362.566
360.719
358.089
OH - 1.6.1 values
389.045
390.971
389.587
Almost 7.5% drop

I'm going to look into it

@regisss  - Did you get some time to check this behaviour

@regisss - Did you get some time to check this behaviour

Not yet. I don't think I'll have time to do it this week and before releasing Optimum Habana v1.8. I'll investigate this by next week and will release a patch if needed.

@vineethanandh can you close this issue. It has been idle for almost a year!


Add Codecov and Julia 0.6

Codecov seems to be gaining popularity, so I figured it might be a good idea to add it to the example package. I've included a configuration YAML that turns off PR comments. If you want Codecov to work on this repo, a JuliaLang owner (I think) will have to enable it.
I've also added the Julia 0.6 badge to the README.

Coverage remained the same at 100.0% when pulling f202e1975433d5d1a40c87de5a382b08d6ad5010 on ararslan:aa/codecov into e55241c7f73667c8e1c3c252f9883fb10cae2d17 on JuliaLang:master.

Coverage remained the same at 100.0% when pulling f202e1975433d5d1a40c87de5a382b08d6ad5010 on ararslan:aa/codecov into e55241c7f73667c8e1c3c252f9883fb10cae2d17 on JuliaLang:master.

Coverage remained the same at 100.0% when pulling f202e1975433d5d1a40c87de5a382b08d6ad5010 on ararslan:aa/codecov into e55241c7f73667c8e1c3c252f9883fb10cae2d17 on JuliaLang:master.


for Knuu

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 44.77%. Comparing base (2469e7a) to head (d07533b).
Report is 72 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3410      +/-   ##
==========================================
- Coverage   44.83%   44.77%   -0.07%
==========================================
  Files         265      272       +7
  Lines       14620    15278     +658
==========================================
+ Hits         6555     6840     +285
- Misses       7313     7650     +337
- Partials      752      788      +36

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

Can be closed as not needed anymore


Importing CSS and compress not working

I'm having a little trouble getting stylus to import CSS and minify it at the same time. Compress works correctly when the import is a stylus file, but not CSS
Here is my task.
    gulp.src( '/path/to/files.styl' )
        .pipe( sourcemaps.init() )
        .pipe( stylus({
            compress: true,
            use: nib(),
            "include css": true
         }) )
        .pipe( sourcemaps.write() )
        .pipe( gulp.dest( 'output/path/to/files' ) );

Same here :'(

If it helps at all, I'd suggest using a gulp css minifier rather than trying to compress within stylus. This is a much more solid strategy in the long run. Not entirely sure on this bug though!

@jenius true, that's what I actually moved to after finding this. Better solution by far

I will close this now as it seems a solutions was found

You mean a work around has been found. :) The documentation should at least mention this exception. I just wasted 10min before I noticed the bottom of the file was compressed. The top wasn't because it was a normalize.css import. So it looked like the file was not compressed.
It should be noted in the docs.


[main] diaguids fix

~Backport #210 & #214~ forward-port #199
~Closes #221~
This PR was pointed to the wrong branch (main instead of 15.x), but that's not too bad because it means we separate the diaguids fix from the version bump

This PR was pointed to the wrong branch (main instead of 15.x), but that's not too bad because it means we separate the diaguids fix from the version bump

Argh, because of a lack of build number increase, we're not getting the artefacts from this merge. 😑
Anyway, moving on.


[Bug]: Missing sandbox_high setting

Discord Username
sgamez
What happened?
The issue is that after level 20 the ground zero map (sandbox_high) revert the timer to default and all my setting are adjusted while this issues is not happening anywhere else 3.9.8.
config-copy.txt
Made it txt purely to post it on git.
SPT Version
3.9.1
Server Log
There was an error creating your issue: body is too long, body is too long (maximum is 65536 characters).

server-2024-08-28.log

Resolved in v1.7.3


All products available?

Are all products available on azure stack? I feel like I might be missing something super obvious.
Cheers,
Nick

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

ID: e7365a8d-a66f-b338-117a-abbb49bb4db0
Version Independent ID: 0e32791c-3fee-636f-27f6-37b08dfcd5b4
Content: What is Azure Stack?
Content Source: azure-stack/operator/azure-stack-overview.md
Service: azure-stack
GitHub Login: @mattbriggs
Microsoft Alias: mabrigg

Never mind. Sorry

Yea, the answers are elsewhere. Thank you!

@niheinba Thanks for asking your question. And no ... only a subset of Azure resources are available on Azure Stack. Glad you found your answer! #please-close

@niheinba We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.

Hi guys, a customer sent me a list of resource types he is using in Azure. He wants to know if all this resource types are also available in Azure Stack. Is there a public list, comparisson, whatever...?
This is his list, thanks in advance! @traviscragg-msft
Analysis Services
API Connection
App Service
App Service (Slot)
App Service plan
Application Insights
Automation Account
Container registry
Data factory (V2)
Data Lake Analytics
Data Lake Storage Gen1
Disk
Event Grid Topic
Event Hubs Namespace
IoT Hub
Key vault
Log Analytics workspace
Logic app
Network interface
Network security group
Notification Hub
Notification Hub Namespace
On-premises Data Gateway
Public IP address
Recovery Services vault
Runbook
Scheduler Job Collection
Service Bus Namespace
SignalR
Solution
SQL database
SQL server
Storage account
Stream Analytics job
Virtual machine
Virtual network

@juajac @TravisCragg-MSFT This is on my list of things to provide in a clear way in our docs (which doesn't help your customer right this second!)
One way thing to point out is that some of Azure stack services are 'add ons' and are built on top of the Azure's IaaS infrastructure, such as Kubernetes and App Service.
At the moment I think it is a bit cryptic even though much of the information is there. For example, you can review the API Profiles and list of available RPs:
https://docs.microsoft.com/en-us/azure-stack/user/azure-stack-profiles-azure-resource-manager-versions?view=azs-1910
Or, you can use the an ARM template validation tool. There are two topics that address this:
https://docs.microsoft.com/en-us/azure-stack/user/azure-stack-validate-templates?view=azs-1910.
https://docs.microsoft.com/en-us/azure-stack/user/azure-stack-resource-manager-deploy-template-vscode?view=azs-1910
Finally, you can review the RPs and services. Many of them have a considerations  page that explains who the Azure Stack Hub version varies from the global Azure. Here is one of thes topoic for 'Compute' (which provides VMS): https://docs.microsoft.com/en-us/azure-stack/user/azure-stack-vm-considerations?view=azs-1910


Suggest widget shows up randomly

Cursor up or down a line, landing at the end of a word
The suggest widget appears

This reproes totally consistently for me, but I was editing for a while today without noticing it, so I don't know whether I did something else to make it happen

Likely caused by being able to arrow key left/right when completions show.

Steps:

have the two lines below
trigger suggest at end of line 1
repeatedly press arrow right and see how widget reappears at word ends

console.log
console.log

Thanks @roblourens 👏


Require maintainer verification before running CI

Author the user story for this feature
As a Flank developer, I want to prevent spam PRs consuming CI resources so I can focus on adding features for users..
Describe the solution
Add a GitHub check that requires a user with write access to approve a PR. By default, PRs submitted by 3rd party accounts should not trigger any CI workflows unless they've been manually approved and the user whitelisted.
Describe alternatives considered
Allowing all users to submit PRs that build is no longer an acceptable risk.

https://github.com/Flank/flank/pull/1830/files

https://github.blog/2021-04-22-github-actions-update-helping-maintainers-combat-bad-actors/ may resolve this.

Second, pull requests from first-time contributors will require manual approval from a repository collaborator with write access before any Actions workflows run. When a first-time contributor opens a pull request, they’ll see a message that a maintainer must approve their Actions workflow before it will run.


Add automatic dependency and security updates

in order to keep software update to date and the repo continuously active (to run CI jobs), automated dependency updates should be enabled through either dependabot or renovatebot. There must be a build step to ensure the app still builds before merging.

Dependabot has been enabled and there are now security scans and some audits running

One of the audits ran and unsurprising not verifying TLS is considered a high risk.
https://github.com/cncf-infra/verify-conformance/issues/64

this is completed


CURL does not return complete list of image tags

Hello,
I am not sure if this issue has been raised previously or if it has been resolved already. When I try to curl for the tags in a private repo, I don't get the complete list back. ie
curl -s -k -u : registry.hub.docker.com/v1/repositories/ibmdashdb/common/
does not return the complete list of tags that are available in the repo.
I've read that it's a limitation that was fixed with the v2 repository. However, we are trying to integrate docker with UrbanCode and that does not support v2 yet. Is there a workaround available that we can use?
Thanks!

@awaissindhu Unfortunately, we no longer support the v1 protocol. For security reasons, I would recommend staying away from registries that continue to support the v1 protocol.


Feature/connections

This cannot automatically merge

There were additions of ckeditor to QuestForm.vue, NodeForm.vue, and BookmarkForm.vue


Docker image uploaded?

I've been looking for something like this! I'm pretty excited to try it out. I see that the README.md file says there is a TODO to upload the Docker image to dockerhub. I've been searching around there and haven't found anything that suggests the image exists yet. I'm wondering when it might be available. Any word?
In the meantime, I will try building the image myself or running it through Node.

There is currently an issue with the docker build as there is no pin on the alpine container.
Will upload to dockerhub when the build is successful

Pinned the container to the latest node LTS, I think there is still some work to get the nextjs version up to something more recent.
I have push the (working) image to dockerhub here: https://hub.docker.com/r/stephenmorgan/owasp-sammwise


fix: dataset update permission out of sync

SUMMARY

The perm and schema_perm attributes for dataset model is out of sync due to unreliably using sqlalchemy get_history method where the history state depends on whether the session has flushed or not.
Previously we have logic in before_update and after_update and permission sync logic in after_update will not be deterministic due to session flush.
This PR moves permission sync logic into before_update and directly using connection.execute to check current db state instead of relying on session state to make sure we can detect actual db/table/schema changes to trigger permission update.
It also clean up shadow_dataset_write logic out as we no longer support it but the database attribute is still not lazily loaded when dataset.database attribute is accessed. Thus I leave the load_database method there still.
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TESTING INSTRUCTIONS

ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Required feature flags:
[ ] Changes UI
[ ] Includes DB Migration (follow approval process in SIP-59)

[ ] Migration is atomic, supports rollback & is backwards-compatible
[ ] Confirm DB migration upgrade and downgrade tested
[ ] Runtime estimates and downtime expectations provided

[ ] Introduces new feature or API
[ ] Removes existing feature or API

LGTM! I still think it would be best to eventually move this logic into the DAO layer, but this is a great way to get things working properly consistently with sqlalchemy events.
To @craig-rueda's point, there are several existing tests prefixed test_after_update_dataset that test this functionality. I'm not sure any new tests are needed since this PR is pretty much just a refactor/improvement of this existing functionality, but it would probably be good to rename them to test_before_update_dataset to be consistent with naming

agreed. Consolidating to domain's command layer is likely the better way of handling side effect like permission update, as there's existing patterns here.
But I would defer the problem to a larger refactoring until we all agree upon ideal state/target state of how permissions system should work in Superset.
This PR is purely continuing with existing patterns but more on a fix side of thing.


systray-x: init at 0.9.2

Description of changes
Closes https://github.com/NixOS/nixpkgs/issues/212815
The only catch and last mile I didn't figure out is that thunderbird seems to ignore the files in /run/current-system/sw/lib/mozilla. So after installation, I manually did the following commands and could confirm it works properly:
cp /run/current-system/sw/lib/mozilla/native-messaging-hosts/SysTray_X.json ~/.mozilla/native-messaging-hosts
cp<EMAIL_ADDRESS>~/.thunderbird/${myprofile}/extensions/

Any ideas @edolstra @lovesegfault @nbp @vcunat ?
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Are you sure this works?

Are you sure this works?

As I mention in the PR,

The only catch and last mile I didn't figure out is that thunderbird seems to ignore the files in /run/current-system/sw/lib/mozilla. So after installation, I manually did the following commands and could confirm it works properly:
cp /run/current-system/sw/lib/mozilla/native-messaging-hosts/SysTray_X.json ~/.mozilla/native-messaging-hosts
cp<EMAIL_ADDRESS>~/.thunderbird/${myprofile}/extensions/

But why a separate package Thunderbird extension is? No similar practice in nixpkgs.

But why a separate package Thunderbird extension is? No similar practice in nixpkgs.

The addon uses the WebExtension API's to control an external system dependent system tray application.
Needs both the addon AND the companion app installed to work.

I think it should remove the extensions and just package the binary.

I think it should remove the extensions and just package the binary.

I am fine with removing the extension, which is<EMAIL_ADDRESS>However, we still need the /run/current-system/sw/lib/mozilla/native-messaging-hosts/SysTray_X.json file to tell thunderbird to load the binary. When I look in that folder, I also have a file called org.kde.plasma.browser_integration.json, I think it works with Firefox. So we have a package doing this type of installation already, but not for thunderbird so far.

It seems support was added to firefox in https://github.com/NixOS/nixpkgs/pull/31572 and latest modified in https://github.com/NixOS/nixpkgs/commit/ec8a9cddc814ea3ee60ae3984ccd17d70bf26d0a
I can try to replicate this for thunderbird, a very similar patch should work. Can I do this in this PR or should it be a seperate PR?

Hello, I'm curious about the status of this. Are we waiting for something with regards to this, does it conflict too much with the norms of Nix packages, or was it just forgotten over the months? As someone who's trying to switch from a distro with systray-x, I've grown very used to having it, lol. No pressure on anyone, though - I know how busy life can get

I personnaly shifted to wayland and betterbird, which now includes a systray option. I am fine with closing this PR.

I didn't know about Betterbird before, but between that and the "Minimize on close" add-on, I think I'm pretty much covered and won't need to use Systray with Thunderbird. Thank you @Zahrun !


Add/remove cluster role commands

Adds a new role subcommand for lxc cluster to add and remove cluster roles without using lxc cluster role edit <member>.
Closes #10116.

As part of this please can you update the test_clustering_events tests to use these commands rather than this style of edit:
printf "roles: [\"database\", \"event-hub\"]\nfailure_domain: \"default\"\ngroups: [\"default\"]"  | LXD_DIR="${LXD_ONE_DIR}" lxc cluster edit node1

Thanks!

@markylaing sorry I shouldn't have merged this as you've pulled in the until package from lxd into lxc which is preventing it building on non linux systems.
Please can you move this function into shared package
https://github.com/lxc/lxd/pull/10172/commits/632000dfb91fa3c5203172b77f4a65b97d143c63#diff-ca7f9d974e4ad0268e74a074fd6dc0906bda49b4fd72bbd09b994629e5be4dcaR81

@markylaing sorry I shouldn't have merged this as you've pulled in the util package from lxd into lxc which is preventing it building on non linux systems.
Please can you move this function into shared package

@tomponline no worries I wasn't sure what was causing the build failure but had to head out. I can see you've fixed this in #10174 now though.


Does xtForm support radio buttons ?

xtform has been rewritten for 2.0 please try this branch https://github.com/refactorthis/xtform/tree/development or 2.0.0-beta.1 on bower.


Update stellar/go and fix testnet integration tests

Update stellar/go and update integration tests so that they pass on testnet.

Seems to be blocked on stellar/go#4434.


How To Use Nested Resources?

By default my SAM temaplate from CodeStar creates /Stage and /Prod resources in api gateway. I want to add another resource on the end of the url so that it looks something like this:

Then when I deploy this I have a working url that looks something like this:
https:/XXXXXXX.us-east-2.amazonaws.com/Stage/
which works fine.
But, when I try to call the url of the nested resource:
https://XXXXXXX.us-east-2.amazonaws.com/Stage/send-score
I always get this response:

Is it aws-serverless-express that's causing my issues here, or is there something else I'm doing wrong?
thanks!

It looks like your API Gateway configuration is correct because you are getting Cannot GET /send-score which is an Express error. So the problem is likely with your Express route. What does the Lambda event.path look like and does that match your route in Express?

Is this still an issue?

Hi Brett, I'm pretty the problem was with my code as I'm not having the issue anymore.
But I did was to ask you- I tried checking event.path (where event is the parameter in index.handler), but it seems to always be undefined.


Fixing Kubernetes UI Graph Component's Legend Height Issue

As I found Kubernetes UI graph component has set a fixed height for the legend section. As a result when more than three elements were present it was not fully displayed. Please see the below image:

I fixed this by setting the legend height to auto, in the diff the change is not clear because the css is having all the content in one line:
svg.legend { height: auto; }
Fixed preview:

I have now signed the CLA. Thanks!

This fixes #9347 so marking v1

lgtm

Thanks for accepting the PR! It's nice to contribute to Kubernetes.

Thanks for the fix @imesh!

Really appreciate the fix @imesh keep up the awesome work?

@bcbroussard, @preillyme It's a pleasure!


index missing in SortableElement

Problem can be seen in https://jsfiddle.net/XeniaSiskaki/kajt284L/1/
I would like for each SortableElement to have an x icon so that the user can click on it and delete the element from the list. However, I cannot access the index in the SortableElement. Is that a bug or desired behavior?

Duplicate of https://github.com/clauderic/react-sortable-hoc/issues/128
This is an unfortunate side-effect of omitting react-sortable-hoc props from being passed down. I'll try to come up with a more suitable workaround, but in the meantime, you can do something like this:
https://jsfiddle.net/kajt284L/2/

Thanks!


Can't build python with a specific tag

In python builder however the build method takes tag as an argument, it's never used so it will always build python3.8 which is not compatible with other parts of jumpscale like PlatformTypes.py, but even if it's compatible we should still support choosing python version in the builder

you can git pull and try again, fix in: https://github.com/threefoldtech/jumpscaleX/commit/5e5840b394c15567314b4a5f61cbc36dfee02c94
Note: do with caution because not all versions are compatible with JS

when trying to use this tag v3.7.2  in python builder i got this error
 j.builder.runtimes.python.build(reset=True, tag="v3.7.2")

you can pull and try again
fix: https://github.com/threefoldtech/jumpscaleX/commit/f8b64f2979687c079579dd97408ea6f9dab7b7c5
reset=True wasn't operating well

it's working fine now , we can close this issue.


[EXCEPTION][UNITY] AOT Exception on Dictionary<string, bool>

Bug:
AOT exception with  Dictionary<string, bool> values { get; set; } = new Dictionary<string, bool>(); on device when build is done with IL2CPP.
Description:
I want to have an external solution with model, build dll, copy to unity project, use it on device.
Actual result: Exception appears
Expected result: Serialization (deserialization) of Dictionary<string, bool> works fine
Attached both projects:

DllSolution - external solution
UnityProject

Steps to reproduce:
Flow Is split into 2 parts: .net standard 2.1 library and unity project.

Generate external DLL

Create new project class library with .net standard 2.1

Add MemoryPack/1.9.10 to the project

Create model Foo.cs

    [Serializable]
    [MemoryPackable()]
    public partial class Foo
     {
         public Dictionary<string, bool> values { get; set; } = new Dictionary<string, bool>();
    }

Build DLL

Generate unity project

Create empty project

Copy DLL

Create script to serialize / de this model

Make build for Android

Run on device or emulator

Exception appears

Message
Attempting to call method 'MemoryPack.Formatters.UnmanagedFormatter`1<System.Boolean>::Serialize<MemoryPack.Internal.ReusableLinkedArrayBufferWriter>' for which no ahead of time (AOT) code was generated.  Consider increasing the --generic-virtual-method-iterations=1 argument,

Stacktrace
at MemoryPack.Formatters.KeyValuePairFormatter.Serialize[TKey,TValue,TBufferWriter] (MemoryPack.IMemoryPackFormatter`1[T] keyFormatter, MemoryPack.IMemoryPackFormatter`1[T] valueFormatter, MemoryPack.MemoryPackWriter`1[TBufferWriter]& writer, System.Collections.Generic.KeyValuePair`2[TKey,TValue] value) [0x00000] in <00000000000000000000000000000000>:0
 at MemoryPack.Formatters.DictionaryFormatter`2[TKey,TValue].Serialize[TBufferWriter] (MemoryPack.MemoryPackWriter`1[TBufferWriter]& writer, System.Collections.Generic.Dictionary`2[TKey,TValue]& value) [0x00000] in <00000000000000000000000000000000>:0
 at Models.Foo.Serialize[TBufferWriter] (MemoryPack.MemoryPackWriter`1[TBufferWriter]& writer, Models.Foo& value) [0x00000] in <00000000000000000000000000000000>:0
 at MemoryPack.MemoryPackSerializer.Serialize[T,TBufferWriter] (MemoryPack.MemoryPackWriter`1[TBufferWriter]& writer, T& value) [0x00000] in <00000000000000000000000000000000>:0
 at MemoryPack.MemoryPackSerializer.Serialize[T] (T& value, MemoryPack.MemoryPackSerializerOptions options) [0x00000] in <00000000000000000000000000000000>:0
 at Test.Start () [0x00000] in <00000000000000000000000000000000>:0

MemoryPackBug.zip

You can't use NuGet's MemoryPack to Unity.
Unity version's MemoryPack is slightly different from NuGet's DLL because it includes IL2CPP support.

@neuecc Thank you for the answer! My goal is to have solutions with models, backend, Unity. I want to have the same serializer for all of this projects. I assume that I can create DLL from sources from unity and use it for all projects and somehow run generator manually. It would be amazing to hear you advise.

share code instead of dll.
For example, we provide this sample guideline.
https://github.com/Cysharp/MagicOnion/tree/main/samples/ChatApp

<ItemGroup>
  <Compile Include="..\ChatApp.Unity\Assets\Scripts\ServerShared\**\*.cs" />
</ItemGroup>


WIP: fix open/session encoding keyword argument

This PR implements layer opening and creation options and adds a test for overriding of shapefile encoding.
Resolves #510.

Coverage increased (+0.02%) to 76.506% when pulling ae990a509ed0c7f3e1b496136900a078ebdeec8a on issue512 into 0123e3911a7d1082d6e6411c0a2c8cb2598a2dfc on master.

Coverage increased (+0.02%) to 76.506% when pulling 0ea6e7a89b214940fa4030d9e8ff0fbc9a3a6d17 on issue512 into 0123e3911a7d1082d6e6411c0a2c8cb2598a2dfc on master.

@sgillies I won't be able to get to this until next week at the earliest.

LGTM.

BTW, complaints about AttributeError: 'Collection' object has no attribute 'session' come from Collection instances in tests that are being garbage collected after the class has already been torn down. If you stick to  with fiona.open(...)  in your code, you will never see this error.

@geowurster No problem! I'm going to be checking here only sporadically until 1/2.


Auto-focus window renamer textbox on open

Does what it says on the tin. This is maximal BODGE.
TeachingTip doesn't provide an Opened event.
(https://github.com/microsoft/microsoft-ui-xaml/issues/1607). But we
want to focus the renamer text box when it's opened. We can't do that
immediately, the TextBox technically isn't in the visual tree yet. We
have to wait for it to get added some time after we call IsOpen. How do
we do that reliably? Usually, for this kind of thing, we'd just use a
one-off LayoutUpdated event, as a notification that the TextBox was
added to the tree. HOWEVER:

The first time this is fired, when the box is first opened,
yeeting focus doesn't work on the first LayoutUpdated. It does work on
the second LayoutUpdated. Okay, so we'll wait for two LayoutUpdated
events, and focus on the second.
On subsequent opens: We only ever get a single LayoutUpdated. Period.
But, you can successfully focus it on that LayoutUpdated.

So, we'll keep track of how many LayoutUpdated's we've ever gotten. If
we've had at least 2, then we can focus the text box.
We're also not using a ContentDialog for this, because in Xaml Islands a
text box in a ContentDialog won't receive any keypresses. Fun!
References

microsoft/microsoft-ui-xaml#1607
microsoft/microsoft-ui-xaml#6910
microsoft/microsoft-ui-xaml#3257
microsoft/terminal#9662

PR Checklist

[x] Will close out #12021, but that's an a11y bug that needs secondary
validation
[x] Closes #11322
[x] I work here
[x] Tests added/passed
[n/a] Requires documentation to be updated

Validation Steps Performed
Tested manually

@msftbot merge this in 1 minute

Should we file a follow-up

Well, so, I filed the PR to add Opened to MUX, so I REALLY doubt I'll forget to include that one 😜


placemarkFromCoordinates() not working

🐛 Bug Report
i try use placemarkFromCoordinates() but i get bellow error:
[ERROR:flutter/lib/ui/ui_dart_state.cc(199)] Unhandled Exception: PlatformException(IO_ERROR, A network error occurred trying to lookup the supplied coordinates (latitude: 25.697771, longitude: 78.654394)., null, null)
E/flutter ( 5364): #0      StandardMethodCodec.decodeEnvelope
package:flutter/…/services/message_codecs.dart:597
E/flutter ( 5364): #1      MethodChannel._invokeMethod
package:flutter/…/services/platform_channel.dart:158
E/flutter ( 5364): <asynchronous suspension>
E/flutter ( 5364): #2      MethodChannelGeocoding.placemarkFromCoordinates
package:geocoding_platform_interface/…/implementations/method_channel_geocoding.dart:56
E/flutter ( 5364): <asynchronous suspension>
E/flutter ( 5364): #3      _SingleAddressDetailWidgetState.build.<anonymous closure>.<anonymous closure>
package:jeanswest/…/userAddresses/single-address-detail-widget.dart:390
E/flutter ( 5364): <asynchronous suspension>
E/flutter ( 5364):

I use geocoding: ^2.0.0 plugin and check other older versions like 1.0.5
my code:
388    LatLng newLatLng = LatLng(25.697771, 78.654394);
390    List<Placemark> dbAddress =
391       await placemarkFromCoordinates(
392          newLatLng.latitude,
393          newLatLng.longitude,
394        );

Version: 2.0.0
Platform:

[ ] :iphone: iOS
[x] :robot: Android

I encounter the same problem since yesterday when testing on my real device. Before it was working fine.
PlatformException(NOT_FOUND, No address information found for supplied coordinates (latitude: 51,53, longitude: 7,26)., null, null)

(Lat & Lng shortened for privacy reasons)
However, on an android emulator it works as expected.

@C4s4r I test on the android emulator and my real device android phone! but both showing this error 😕.
please inform me if u get a reason or solution, tancks

@mshamsi502 It also seems like your error message is different, but with the same result.
This problem worries me since I have a released app that is build on this API call, which now doesn't work due to this address resolving fail.
Looks like an API problem to me... Hopefully this can be solved as fast as possible.

After the confirmation of a friend that it works fine on his device, I restarted my phone it works again.
I have no Idea why, but I guess it was just my phone because it also worked on my emulator.
Still don't know why I encountered this behaviour on my phone...

I get the same issue, and it goes away after restarting my phone (OnePlus 7 pro), it seems to be random. Any suggested more reliable alternative way o get placeMark from lat/long?

I am experiencing the same error as well, only on Android, using geocoding: 2.0.0. Weird thing is it happens both on emulator and real device, but sometimes it works randomly, likewise said before as well.
I/flutter ( 4502): PlatformException(IO_ERROR, A network error occurred trying to lookup the supplied coordinates (latitude: -23.583747, longitude: -46.678073)., null, null)

Has someone with an device that has these problems already checked, if this can be reproduced with the native Google geoparsing service that his package uses?
Because than it would not be related to the flutter-geocoding package.

I also had the same issue and I fixed it by rebooting my phone. my phone also (OnePlus 7 pro)

I am also experiencing the same error as well, geocoding versions 2.0.0.

I get the same issue, and it goes away after restarting my phone (OnePlus 7 pro), it seems to be random. Any suggested more reliable alternative way o get placeMark from lat/long?

Rebooting worked on my (OnePlus 7 pro) as well but after the first reload the exceptions are back.

This issue is  caused by the rate limitations related to the free services provided by Android (see README and also issues #9, #12, #23, #28, 35# and #49).
I will be closing this issue now as there is nothing I can do about this error. You are simply hitting the limitations of the free services provided by Google and Apple.

Still having same issue on my Emulators 2 years from original post


lsdsng-tools change KIT P/L/V from FAST to STEP

Using lsdsng-export and lsdsng-import on this .sav changes KIT instrument 26 from FAST to STEP.
https://defensemech.com/REVISION.sav

tested and it seems lsdsng-export is the culprit. also showing weird garbled instrument names for some instruments

Here's what I think (and feel free to delete this bug and create an enhancement) - lsdsng-export and -import should be entirely version-agnostic. They should only deal with compression/decompression and reading bytes for song titles/version numbers. A new tool should be created to convert songs between versions.

also, it's naming all of the instruments after the last instrument or something. I dunno, something really strange is going on lol

Gonna keep this issue, because the bug is still a bug even if we split this off in different tools. I'll create another issue for that.

The names don't seem to be garbled in v1.0. It also seems like the volume
col in tables is getting weird data. I can provide another example lsdsng
if that will help
On Fri, Jul 27, 2018, 7:54 AM Stijn Frishert<EMAIL_ADDRESS>wrote:

Gonna keep this issue, because the bug is still a bug even if we split
this off in different tools. I'll create another issue for that.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/stijnfrishert/liblsdj/issues/24#issuecomment-408396984,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFbBWTz1zG3MPjZ__KpEd_v46V0KAMisks5uKv9sgaJpZM4VVCCW
.

Fixed PLV, but turns out loop2 was also written incorrectly. Both are fixed


Resolve thread-safety issues in distributed-ucxx

Instead of creating a per-UCXX context lock that may result in race conditions during the creation of the locks, use a module-level lock that is guaranteed to be thread-safe as it occurs at import time.

/merge


Datagrid fails to load. "SelectionType is not exported."

Select one ...  (check one with "x")
[x] bug
[ ] feature request
[ ] enhancement

Expected behavior
Datagrid should load.
Actual behavior
Datagrid fails to load. Error: "export 'SelectionType' was not found in './providers/selection'.
Reproduction of behavior
Run ng build --prod --build-optimizer.
Environment details

Angular version: 4.4.0-RC.0

Angular CLI version: 1.4.0-rc.2

Clarity version: 0.10.3

OS and version: macOS 10.13 Beta

Browser: [Chrome 61]

@karmasakshi: Do you see the error without the --build-optimizer flag?

No @adityarb88. Only when the CLI's optimizer is turned on. Issue persists on 1.4.0 which was released yesterday.

related:

angular/angular-cli#2034
webpack/webpack#2977

This issue does not exist in the latest builds of Clarity (0.10.6) and Angular CLI (1.4.3). Closing.


[CLI] Dumping of pubkeys is broken since adding base-dirs for ed25519, rsa3072

Bricked since: 14c65e2488490f2604c47e24fa6f268b8497fbc9
This can be fixed while finalizing #1292

False positive, read the code wrong. When I actually tested it, both worked.


feat: http client & max error depth configs

Hey,
I would like to add options to configure httpClient and maxErrorDepth of underlying sentry-go library.
Cheers

Codecov Report

Merging #11 (57dfb95) into master (1fa3486) will decrease coverage by 2.36%.
The diff coverage is 20.00%.

@@            Coverage Diff             @@
##           master      #11      +/-   ##
==========================================
- Coverage   62.57%   60.22%   -2.36%
==========================================
  Files           1        1
  Lines         171      181      +10
==========================================
+ Hits          107      109       +2
- Misses         58       66       +8
  Partials        6        6

Files
Coverage Δ

writer.go
60.22% <20.00%> (-2.36%)
:arrow_down:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

Thanks, merging
v1.6.0


Fix #2039 - ignore of I18n files

Fixes #2039

Sorry to be a newb but what is the process for getting this into a release?  It seems like it was in all the development versions of 4.3.0 but didn't make the cut for the final release and every point release since then.  Has it just been forgotten?  It would be really useful for us to have this.

4.3 became 5.0 which isn't out yet. I can backport stable releases.

Published as v4.3.4

Fantastic, thank you!!


Mino Feat: Change the Manifest To have a new type

[x] Fix the build (dependencies:634 map for option)
[x] Test

[x] Start a docker sleep infinity
[x] Stop/start LR
[x] Stop/start Reg
[x] More

Proofs/ Checks

[x]  When stopped, docker still running in background?

Image shows the image is stopped, docker is running in sleep infinity and the curl shows no running ui, therefore not running service in background
[x] Started, we can reach the service inside like normal

Image shows the image is running, docker is running in sleep infinity and the curl shows running ui, therefore running service in background

[x] Start/ Stop/ Restart

https://user-images.githubusercontent.com/2364004/185174042-bea137c8-dea3-4c2b-bd4e-e455509e6178.mp4

[x] Ensure we can Backup and restore

TODO

[ ] Uninstall


helm-values idea plugin deadlock on IntelliJ IDEA 2022.3.2

Hi!
Intellij IDEA constantly hangs (i suspect during index scan) after some time of usage of a large project. Disabling the plugin solves the issue.
It seems like it was not reproducible with previous IDEA 2021.
threadDump-20230221-115555.txt
Intellij IDEA Build #IU-223.8617.56, built on January 26, 2023
Runtime version: 17.0.5+1-b653.25 aarch64
VM: OpenJDK 64-Bit Server VM by JetBrains s.r.o.
Apple M1 Max Mac OS Ventura 13.1

Perhaps this happens due to symlink ref.
Based on info from Mostafa Nagib from IDEA plugin review page.

Hello @gsaukov
thanks for reporting the issue, I'll look into it.
Do you have an example of large project (preferably open source) where the issue can be reproduced ?
Could you also provide more info on usage of symlink refs in helm charts?
Feedback provided in plugin review page was unclear.
Thx in advance !

@gsaukov ,
I've made some performance improvements on JsonSchema providers with release 0.6.1.
Please let me know if latest version of the plugin has improved the situation on your side.
If there is any information that you can share about your project that could help me track the issue, please let me know.

@fstaudt
Hi, it is still 0.6.0 in Idea's market place.

@gsaukov
release 0.6.1 was done yesterday in github.
It takes a few days to be available for update.

Hi @fstaudt
Same thing 0.6.1, it crashes almost immediately on idea start.
The project that crashes is SAP commerce, it is very large JAVA monolith.
You may find IDEA log and thread dump in the attachment. However it is possible that my Idea configuration or some other conditions are very unique. But when I disable helm-values plugin, it does not crash at all.
IDEALOG_AND_THREAD_DUMP.txt

Hello @gsaukov ,
does your project contain any Helm chart ?
Performance improvements in 0.6.1 were focused on background tasks performed by the plugin when IDE is already running.
I had the understanding that issues on your project only happened after some time of usage.
A crash at startup can only be caused by JSON schema provider factory in helm-values plugin.
The factory searches the complete repository to find directories containing a file named Chart.yaml.
For large projects containing a lot of files and subdirectories, I imagine it can take a lot of time.
Inspection at startup could be limited to the first level of directories to improve performance and possibly avoid a crash.
I can create a SNAPSHOT version of the plugin with a fix that you can test on your project.
Would you be OK to test a SNAPSHOT version of the plugin on your project ?


/opt/android should be owned by dev container user instead of root to be writeable

I just wanted to let you know that I've (only very briefly, without doing any further debugging) tried this feature out, for https://github.com/google/android-fhir/issues/2481, and it seems to me that there is a file permission issue?
In https://github.com/google/android-fhir/pull/2482 (here) I originally tried:
"features": {
    "ghcr.io/akhildevelops/devcontainer-features/android-cli:0": {}

This installed just fine, but when a Gradle Script tried to write to $ANDROID_HOME, which with this is set to /opt/android it failed, because that directory is owned by root but the UID when running a GitHub Codespace Dev Container is  vscode.
./gradlew dokkaHtml

(...)

> Configure project :workflow:benchmark
Checking the license for package Android SDK Build-Tools 33.0.1 in /opt/android/licenses
License for package Android SDK Build-Tools 33.0.1 accepted.
Preparing "Install Android SDK Build-Tools 33.0.1 v.33.0.1".
Warning: Failed to read or create install properties file.

FAILURE: Build failed with an exception.

* Where:
Build file '/workspaces/android-fhir/workflow/benchmark/build.gradle.kts' line: 3

* What went wrong:
An exception occurred applying plugin request [id: 'androidx.benchmark']
> Failed to apply plugin 'androidx.benchmark'.
   > Could not create task ':lockClocks'.
      > Failed to install the following SDK components:
            build-tools;33.0.1 Android SDK Build-Tools 33.0.1
        The SDK directory is not writable (/opt/android)

* Try:
> Run with --stacktrace option to get the stack trace.
> Run with --info or --debug option to get more log output.

* Get more help at https://help.gradle.org

BUILD FAILED in 3m 11s
5 actionable tasks: 5 up-to-date

I was able to work-around this with sudo chown -R vscode:vscode /opt/android/, but doing that probably shouldn't be required.

Thanks.... Can you please provide a PR for this.


Enforce column type when removing column

🚨 NOTE: this is based on my PR to the original fork -- https://github.com/LendingHome/zero_downtime_migrations/pull/32 -- but that gem appears to not be actively maintained. Figured we can add Envoy-specific migration styles to this fork in the meantime. 🚨
Adds a new validation that warns when a remove_column migration is run without providing a column type.
Bad (rails raises this error when attempting a rollback):
class RemovePublishedFromPosts < ActiveRecord::Migration
  def change
    remove_column :posts, :published
  end
end

Good:
class RemovePublishedFromPosts < ActiveRecord::Migration
  def change
    remove_column :posts, :published, :boolean
  end
end

Note that this branch only checks for database-agnostic types listed in the Rails API docs. I'm open to also including types specific to Postgres, MySQL, and SQLite, so that users of the most common DBs don't get extraneous errors from this gem. The guides have an official page for Postgres types -- I don't see a corresponding one for MySQL or SQLite, but can try to hunt down an official types list for each of those if you think that's a reasonable next step.

Closing in favor of using a different gem


How to specify  The graphics card number during  multi-gps  training？

Hi @liwuhen,
To change the number of GPUs you can simply change the -np argument in the torchpack command. E.g.:
torchpack dist-run -np 4 ...

means that you want to use 4 GPUs.
Best,
Haotian

thank you!
But, for example,  I want to use some graphics   eg: 1, 4, 7 graphics card
What are the startup commands ？

You're welcome. For your use case, please just try out
CUDA_VISIBLE_DEVICES=1,4,7 torchpack dist-run -np 3 ...

Closed due to inactivity.


Cluster API enhancements

Background:  While designing device-level API, a few small gaps in the cluster-level API were causing friction with implementation.  This is an attempt to address.
@Apollon77 - I think I covered all your asks here, see summary before you look at the code.  Definitely fixed the initial naive implementation once I actually ran it.  I didn't squash so you can see the differences.
Might be helpful to scan ClusterServerFactoryTest.ts and CommandServerTest.ts before looking at implementation.
Quick summary of scope:

Problem: It would be good to give the user a convenient mechanism for creating any cluster along the lines of createDefaultScenesCluster().  Currently convenience is limited to clusters with hand-crafted server implementations.  This also made it difficult to automatically implement the clusters on the device.
Proposal: Implement generalized ClusterServerFactory.create that creates a ClusterServer for any cluster, regardless of whether we have better default handlers available.
Side benefits: The cluster server files can be a bit smaller as ClusterServerFactory handles more automatically.  Imports are smaller since there's only one factory method and it already knows about default handlers.  The default attribute values only use the schema as source-of-truth in this version.

Problem: Cluster definitions weren't available centrally.  AllClustersMap contained most clusters but it was keyed by ID so couldn't contain extended clusters which share an ID with their parent.  Previously I was scanning exports.  Worked but a bit icky.
Proposal: Create a central registry for clusters and extended clusters.  Populate the registry automatically when the cluster is defined.
Side benefits: Devs don't need to remember to update AllClustersMap.  Previously some handler exports accidentally didn't follow conventions; won't be an issue now.

Problem: It would be good to offer a mechanism for API users to invoke matter.js's "smart" handlers from within their own handlers.  There was a bit of an inversion of control issue here as InteractionHandler looks for the presence of a handler to understand if it should respond with StatusCode.UnsupportedCommand.
Proposal: Create an alternate path for handlers to convey status information to CommandServer.  In addition to the current approach (return value) create a specialized exception class that conveys a status code.
Side benefits: This pattern may make implementation a bit simpler because a handler can bail at any time with an exception.  Some of the current handlers threw exceptions that were effectively internal errors, now they'll just result in StatusCode.UnsupportedCommand.  Handlers could also use this to send other status codes though I'm not sure if that's desirable.  Finally, CommandServer has unit tests now. :)

@Apollon77 FYI, I haven't forgotten about this guy...  Waiting for some variant of #162 and #165 to land, will finish modifications to cluster registration then do the 😱 merge.

I agree. Lets do the current cluster topic first. Then Generation of feature clusters and this. In my eyes this used generated ones and Turnschuhs ones as "all in" clusters.

Closing this as the generated clusters make the factory largely irrelevant. Will resurrect other bits as necessary as part of device generation.


How to generate different patient conditions

I am looking to generate patient data for a data science project, I want to change the conditions of the population with let's say higher rates of heart attacks or cardiovascular problems in general, to match closer to the reality of the state I currently live in.
How do I easily change properties to do that?

You have to modify the disease modules. You can do this with a text editor (see the documentation on the format and edit the files in src/main/resources/modules) or graphically with a point-and-click interface by using the Module Builder. Using the module builder still requires that you save your modifications locally, and run the data locally.

You have to modify the disease modules. You can do this with a text editor (see the documentation on the format and edit the files in src/main/resources/modules) or graphically with a point-and-click interface by using the Module Builder. Using the module builder still requires that you save your modifications locally, and run the data locally.

Thank you very much I will close it now

Thank you very much I will close it now


Update API.md

Thank you for the PR @benrogmans! This got already fixed in https://github.com/orbitdb/orbit-db/pull/219 so I'm gonna go ahead a close this one.


Html msg ++ Html msg

It's often useful to define functions like
foobar :: List (Html Msg)
foobar = ...

But it becomes a pain to append with Html Msg. Workaround is wrapping the element, but that often breaks layout due to css rules like ui > li where there shouldn't be an element in between.
@roberth proposed
type alias Htmls msg = List (Html msg)

and rebuild the whole api on top of that.
I'd prefer this was solved inside the language, so that really basic primitives compose well.

Discussion here, possibly a bit different angle.
https://discourse.elm-lang.org/t/why-is-html-a-single-dom-node/3966


equivalent to .eq in jQuery?

something like assert.dom('.title').eq(1).hasText(...) // second title has text

while I have had need for something like this in the past it's easy enough to do outside of qunit-dom and just pass in the element directly. I'd be a bit worried about this getting out of hand when we start implementing all sorts of selectors like this, so I think I'd prefer to not include this functionality in qunit-dom itself.


Add clippy with yellow hat

Yellow is the only color clippy doesn't support
@marcduiker is this tricky to do?

No prob, I'll add it 😊


Inconsistency between target riscvOVPsim.ic and riscv_core_setting.sv

@udinator @taoliug
Having reviewed a few of the recent commits (which we have commented), I think there is an inconsistency between what you are defining in terms of supported privilege modes, in the following directory (and I am sure others)
riscv-dv/tree/master/target/rv32i
You have defined that the DUT supports MACHINE mode only with this statement

// Supported Privileged mode
privileged_mode_t supported_privileged_mode[] = {MACHINE_MODE};

This maps to RVB32I as the --variant option in riscvOVPsim.ic, but the actual variant you have assigned is RV32I - this includes USER and SUPERVISOR modes
So these 2 configurations are now in conflict with each other
Is the intention to have
MACHINE + USER + SUPERVISOR, or
MACHINE
Thx
Lee

I tried to change it to RVB32I, but got error like below. Do I need a newer ovpsim version?
Error (OP_IU) Setting enumerated parameter 'riscvOVPsim/cpu/variant' to the illegal value 'RVB32I'. Legal values are:
RV32I
RV32IM
RV32IMC
RV32IMAC
RV32G
RV32GC
RV32GCN
RV32GCV
RV32E
RV32EC
RV64I
RV64IM
RV64IMC
RV64IMAC
RV64G
RV64GC
RV64GCN
RV64GCV

RV32I includes Machine, Supervisor and User
RVB32I, and RVB32E and RV64I were committed to riscv-ovpsim  earlier this year - from June 2019
so yes, please use a later release.
Simon

This issue is currently blocked by #283

I tried the latest ovpsim version, but still get the same error as the error. RVB32I is not accepted.

Hi Tao
Can you post the command line you are using and the full output log from the simulator
Thx
Lee


CGBitmapContextCreate() regression when using latest HEAD of master branch.

We use some code like the following to resize a UIImage using CoreGraphics.
When using a build of winobjc from the HEAD of master (ffe05fa6e5c7819a6cb2c9105ed9694245656842) this code no longer works and the created newImage is nil.
Picking through it, it seems CGBitmapContextCreate() returns a nil CGContextRef.
This code works fine using the 0.2 170201 1st Feb release.
UIImage *image = [UIImage imageNamed:@"someImage"];
CGSize size = CGSizeMake(64.0, 64.0);
CGFloat scaleFactor = [UIScreen mainScreen].scale;
CGRect newRect = CGRectIntegral(CGRectMake(0, 0, size.width, size.height));
CGImageRef imageRef = image.CGImage;

CGContextRef bitmap = CGBitmapContextCreate(NULL,
                                                newRect.size.width * scaleFactor,
                                                newRect.size.height * scaleFactor,
						CGImageGetBitsPerComponent(imageRef),
						0,
						CGImageGetColorSpace(imageRef),
						CGImageGetBitmapInfo(imageRef));

CGContextScaleCTM(bitmap, scaleFactor, scaleFactor);

CGContextDrawImage(bitmap, newRect, imageRef);

CGImageRef newImageRef = CGBitmapContextCreateImage(bitmap);
UIImage *newImage = [UIImage imageWithCGImage:newImageRef scale:scaleFactor orientation:UIImageOrientationUp];

CGContextRelease(bitmap);
CGImageRelease(newImageRef);

@rupertdaniel can you share the debugging output from context create time? We should be printing out the failing context parameters.
When you don't pass data, CGBitmapContext should accept almost any format. It's very strange that it's not!

@rupertdaniel @DHowett-MSFT  I suspect it's due to bytesPerRow being 0 and automatic calculations aren't happening. So we try to find a bpp of 0 in the map.

Good catch, @msft-Jeyaram.

@DHowett-MSFT if you still need it here is some input/output from the call to CGBitmapContextCreate()
bitmapRef: 00000000 bitsPerComponent: 8 colorSpace: 09A32760 bitmapInfo: 8196


Add Deprecation Notice

Per explanation from Bret Logan
Signed-off-by: davidliu<EMAIL_ADDRESS>

Thank you!


Codeimprovement/npm audit fix

Update dependencies. See also https://github.com/openstad/openstad-management-panel/issues/56

Yes. Thanks for the explanation.


Fix link on README

What kind of change does this PR introduce?
Fixes link to point to Netlify's repo rather than the forked version.
What is the current behavior?
Links to existing repo.
What is the new behavior?
Links to the original Netlify gotrue repo.

Thanks for the contribution!


[v1.x] Backport #19244

Description
Backport #19244 for website version artifacts pipeline to v1.x.

@mseth10 @ys2843  can you please review? Thanks


convert a binary tree to full tree by removing node that have only on…

…e child

merge binary tree changes


OpenPGP support missing

OpenPGP support missing
The information seems to be more or less available on the forum and in some update blog posts.
But it'd be really nice to have a place to subscribe to the progress or just being informed the moment support for use with gpg is released.
Would be nice if the issue could be confirmed and updated or at least closed when support for OpenPGP / gpg arrives.

Hi!
OpenPGP component for use in the Nitrokey 3 firmware was released publicly lately:

https://github.com/Nitrokey/opcard-rs/

It is currently in the alpha tests.
Here is the latest blog post about it:

https://www.nitrokey.com/blog/2022/openpgp-card-alpha-nitrokey-3

OpenPGP component for use in the Nitrokey 3 firmware was released publicly lately:

can you provide build instructions or something to make it clear how the two repositories are linked together?

Would be nice to have this ticket updated once the support for openpgp is at least public beta and available for all nitrokey3.

@chri2
I am sure this will be published on blog, as the main communication channel. Can't guarantee the same for this ticket. Let's close it for now to not increase the confusion further.
You can always subscribe to releases RSS, or just add yourself to "Watchers" to follow the development.
@runcom
As far as I know the OpenPGP support is integrated into Nitrokey 3 Mini already, but I do not know where is the source code for it. The build guide should be the same as the regular one, but simply with the openpgp component added.

The build guide should be the same as the regular one, but simply with the openpgp component added.

sure, that's what I'm asking, I want to be able to have docs around how to flash a firmware with opengpg support and test it myself

@daringer Can you share, where the firmware's source code for the beta tests of the OpenPGP component is located? Is it merged to master?

I'm pretty sure it's here https://github.com/Nitrokey/opcard-rs/ - although the "hard part" is a guide/tutorial/docs on how to build a firmware and flash it to the nk

https://github.com/Nitrokey/opcard-rs is the application, https://github.com/Nitrokey/nitrokey-3-firmware/releases/tag/v1.2.2-alpha.opcard.0.1.0 is the firmware containing the application.  As all sold Nitrokey 3 devices are locked and require signed firmware updates, it is not possible for you to compile and flash your own firmware.  If don’t know if it possible to get unlocked devices at the moment.  Let me know if you would be interested, then I can follow up on this.
Regarding the firmware build instructions:  We currently target two chips, lpc55 and nrf52.  For lpc55, there is a guide: https://github.com/Nitrokey/nitrokey-3-firmware/blob/v1.2.2-alpha.opcard.0.1.0/docs/lpc55-quickstart.md  For nrf52, there is no guide yet but a Makefile that takes care of all required steps: https://github.com/Nitrokey/nitrokey-3-firmware/tree/v1.2.2-alpha.opcard.0.1.0/utils/nrf-builder

I’ve created #100 for tracking the opcard integration into the firmware.

I don’t know if it possible to get unlocked devices at the moment. Let me know if you would be interested, then I can follow up on this.

I'd love to have the possibility to get something for development, since the firmware is open source I believe it would be just inline with the whole philosophy as it allows me to contribute, test and learn


Listed documentation to setup Teams Room does not work

Thank you for writing a nice, condensed article that covers Microsoft Teams rooms! We have been fighting an issue with sign-in that support does not know how to correct. I have followed your documentation exactly and have ensured we meet all requirements listed. Can you please give me some guidance on how we can move forward?
Our Environment

Exchange 2013 On-Premise w/ SP1 and CU22
On-premise Skype for Business with autodiscover DNS entries pointed on-prem
Office 365 E3 with all necessary licensing added per article
Azure AD Sync

Things that may be missing from the article

Required DNS entries - There is no direction in the article for DNS entries(we have tried changing the UPN of the Teams Room to @onmicrosoft.com)
Domain field (bottom of Teams Room provisioning page - There is no direction in the article regarding what you should use for sign-in information.

Things that are working

We can sign-in to https://teams.microsoft.com using the on-prem synced room account
We can get the calendar showing by using the Exchange address of @onpremdomain.com and using the domain\username field.
If we keep the Skype sign-in and Exchange sign-in fields exactly the same using @onmicrosoft.com address, the room account is able to sign-in successfully, but then obviously it breaks the calendar fetching.

Things that are not working

We cannot get BOTH the calendar fetching and Teams Room sign-in working at the same time. We can only get one or the other. I'm making an educated guess here that it doesn't like the DNS entries we have for autodiscover for our on-premise S4b. It could very well be that we need to fill in the "Configure Domain" with something, but there's no clear direction on what that should be if we have on-premise S4b possibly conflicting with the Teams autodiscover. That's why I thought we might be able to get away with @onmicrosoft.com but that doesn't appear to be working at this time. I do have a test domain where we could configure proper autodiscover entries that point to O365 for Skype and Exchange on-prem, which would rule out any sort of bug in the code that doesn't want to let us use a mix of domains.

Thank you for any assistance you can provide!

@jwinte  - thank you for submitting feedback.
I think the best way forward is if you open a service ticket in your tenant so this can get resolved ASAP. Based on the outcome let me know if it is something that can be called out in the docs.
Please follow this link to contact support for business products:  https://docs.microsoft.com/office365/admin/contact-support-for-business-products
Please keep us posted here on the resolution so we can feed whatever information you discover into the content.

@officedocsbot close


Adds minimum supported versions for databases

This is related to #693. I retrieved the minimum required versions from the docker-compose file and from the GitHub actions.

This is likely best consumed in a table

Like this maybe?

Database
Version

SQLite
>= 3

PostgreSQL
>= 15

MySQL
>= 5.7

MariaDB
>= 10.11

Please add the relevant DCO metadata.

Sorry for the mess with the DCO 😁

Thanks!

Awesome, this was my first PR on an open-source!


Added AryanBabber (myself) in contributors.json

this is meant to contain only the Active Members, for now we are not gonna merge this

closing this as this will be the list of the ACtive members on discord


More generic approach

It is possible to define metadata not only for the node settings, but also globally for the whole page. In this case the extension does not work as og:description in the node settings is empty. As I have a page where I am likely to not provide a description for every page (e.g. as it is not possible to do so on generated pages from some extensions), the extension only works on some pages. So maybe it would be possible to use the already composed metadata value from $meta instead of copying it from the node? Then also the global configuration should be considered.
Edit The event which initially adds the metadata can be found here: https://github.com/pagekit/pagekit/blob/c76ae5741bafc8b66bc07d102338c7ffe2484638/app/system/modules/site/index.php#L179

I will consider this for the next release.

Well, I'll change it this way:
<?php

return [
	'name' => 'metadesc',

	'type' => 'extension',

	'main' => function( $app ) {

	},

	'events' => [

		'site' => function( $event, $app ) {
			$app->on(
				'view.meta',
				function( $event, $meta ) use ( $app ) {
				var_dump($meta->get( 'og:description' ));
					$meta(
						[
							'description' => $meta->get( 'description' )
						]
					);
				},
				40
			);
		}
	]
];

This loads the default description if the node does not provide a custom description.
I will release this after a quick test on different Pagekit-based websites.

General approach looks good. But I see two things which might need to be changed:

var_dump($meta->get( 'og:description' )); should be removed
'description' => $meta->get( 'description' ) shouldn't it be 'description' => $meta->get( 'og:description' )?

Sorry, I posted the content from my history-clipboard.
In my project I already got the "cleaned" version.
But there is still a problem occuring: If the Site-settings (general) in Pagekit are providing a global description, $meta->get('og:description') always returns the global description, even if there is a custom description.
Just checking it out right now.

Closed with https://github.com/SPQRInc/pagekit-metadesc/commit/f5e4f81e86ff9e3336447b50a2eb863b4be1575b .
Update is available on marketplace.

Tahks a lot. Works great :)


Scale TTS volume with in-game volume

Would be nice to have an option that scales the spoken volume to your in-game master or voice volume

I just had this idea as well when I decided to install the StanleyParableXiv plugin and saw that it had the configuration option to tie the narrator volume to the game volume.
I'll link it here as inspiration in case Karashiiro or maybe I feel like tackling this feature in the future.
https://github.com/rekyuu/StanleyParableXiv/blob/e14e7cd19e966a9f594abdb4d1ce2557eb2b4de2/StanleyParableXiv/Configuration.cs#L24

Volume can easily be read by this:
`public static unsafe float GetVoiceVolume(IGameConfig gameConfig)
{
var voiceVolume = .5f;
var masterVolume = .5f;
var instance = Framework.Instance();
if (instance != null && instance->SoundManager != null)
{
var soundManager = instance->SoundManager;
masterVolume = soundManager->MasterVolume;
voiceVolume = soundManager->GetEffectiveVolume(SoundManager.SoundChannel.Voice);
var isMasterMuted = false;
var isVoiceMuted = false;
    gameConfig.System.TryGetBool("IsSndMaster", out isMasterMuted);
    gameConfig.System.TryGetBool("IsSndVoice", out isVoiceMuted);

    if (isMasterMuted || isVoiceMuted)
        return 0f;
}

var volumeFloat = masterVolume * voiceVolume;
return volumeFloat;

}`

Thank you! I may try that sometime and see how it goes, but I'll want to run it by Karashiiro before I merge anything into the main branch.