FR - Spanner - ID Generation for UUIDs

In Spanner, support ID Generation, based on annotation.
I don't feel we can get into full JPA-type ID generation. However, if the type is a String or UUID, we can consider automatically generate a new UUID if annotated w/ a UUID strategy.

Would it make sense to just automatically generate the ID when one is not specified, without requiring an annotation requesting it?

@meltsufin what do you mean specifically?
When there are not any  properties labeled as ID?
When there is a prop labeled as ID but is null?
When there is a prob labeled as ID and is null but not labeled as auto-generate?
What about if there are multiple properties that comprise the PK? What if some of them are null but others aren't?

Since this is about UUIDs, I assume it implies non-composite IDs.
So from your list, if there is a (String?) property annotated with @ID but null.

In that case, I would assume that the parent foreign key is not something we can generate, but has to be specified. So we would only auto-generate the components that are null. Would that make sense?

I agree we definitely can't auto-generate the key components that are from the parent, but I have a different idea of the policy we should have.
My proposal is to have a boolean attribute in our key component annotation called AutoGenerateOnWrite ,so requiring the user to explicitly mark the key components for auto-generated UUID.
The pros are:

if the user marks a parent column for auto-gen, then he will get back an error from the client lib during a write attempt saying that parent ID doesn't exist.
also, it means we don't need to do schema/metadata checking for the user to tell him if a column is a parent key column. This also means a given child class doesn't need to be linked in Java with a parent class, because such relationships don't need to exist from one spanner instance to another.
the specific name "AutoGenerateOnWrite" makes it clear to the user exactly when the auto-gen happens. While we will certainly also explain this in the docs, it's a good reminder and reduces confusion.
the user also doesn't need to make assumptions or guesses about what types of checking we might be doing on his behalf. the behavior becomes more clear
we give him the option of crashing on null key values. he may not want us to auto-gen in every situation.

In JPA, there is "Generated" annotation that specifies the different strategy.  Would probably need a more detailed design doc and chat w/ Oliver.


Fix broken link in 06-parallelism.ipynb

It looks like the notebook 07-state-in-jax.ipynb was renamed to 07-state.ipynb, causing a link to it to break. This commit fixes the link by using the correct name.

Thanks! We'll need the equivalent change in the associated markdown file as well.

Thanks! We'll need the equivalent change in the associated markdown file as well.

Done!

Looks good, thanks! The last thing we need is to squash these changes into a single commit (See jax.readthedocs.io/en/latest/contributing.html#single-change-commits-and-pull-requests) Thanks!

Sure thing! I squashed the commits.


Translations Management

What I'm trying to achieve
Hi there, the README contains information regarding with the translation but I did not see this functionality.
In the near future there will be any changes regarding the translations management?

Hello @olythy. Thank you for pointing this out. Currently, that function is not available. I'll pass the issue to the team and we'll discuss it. If I have any news, I'll let you know.

@karolkielecki
Hey! I have a live shop running on saleor. Would like to know whether this is a feature that will be implemented in the near future or whether I can rely on #399 to accomplish it on my own.
Thanks!


Release 1.3.0

[x] bump version to 1.3.0
[x] update changelog (also for 1.2.0 because it was missing)

If you merge this, do not forget to add a tag to the release as I can not do that from the PR.
I also noticed there is an inconsistency in the naming of the release tags. The release tag for version 1.2.0 does not have a v but all other release tags do have it (like v1.1.0).

Thanks!


Should Unused Symbols be yellow with red strikethrough?

Questions

Do you like the look for occasional unused symbols in your code?
Do you like the look for many unused symbols in Library APIs?
Will the many instances of Yellow/Red unused symbols in Library APIs cause you to use @SuppressWarnings("unused") or add Entry Points?

Background
While developing this color scheme plugin, I was working with some test files and files I had from a few other projects.
The Yellow Foreground, Red Strikethrough looked good with occasionally unused resources; however, I have been seeing code (such as Library APIs) where every method is "unused" and the editor is littered with these Yellow / Red symbols.

I am looking at changing the coloring of unused symbols to be yellow underline with the normal foreground. Right now I think this coloring will stick for a while.
If you disagree, then comment below!

Fixed in v2.1


Broken link to Files and Directories

Link from our doc: https://dvc.org/doc/dvc-files-and-directories
But the page was moved to https://dvc.org/doc/user-guide/dvc-files-and-directories
Unfortunately, we don't know the exact source of the broken link.

@dmpetrov What do you mean by Unfortunately, we don't know the exact source of the broken link.?
I've fixed the link. Btw, it's pretty easy to fix it using the Edit on Github button on the top of each page.


Wrong parameter name in Rest API call documentation of Azure DevOps Server

The following page:
https://docs.microsoft.com/en-us/rest/api/azure/devops/security/access control entries/remove access control entries?view=azure-devops-rest-5.1&source=docs
Wrongly names the parameter for the SecurityTokens as "token", when it should be "tokens" (if you try it with "token" it will throw an error).

@mijacobs Do you know who owns these API docs?

Closing this bug because I cannot recreate it myself any more... not sure how I managed to get this error, but now I actually have to use the parameter "token" (not "tokens"), as is currently documented.


[ELE-304] Re-add source freshness support in dbt package

Describe the user problem
After the breaking change in dbt 1.3.0 we re-added the option to upload source freshness results using the CLI, but this is not relevant for dbt cloud users and might be problematic for others as well.
Describe the solution
Find a new way to support source freshness upload in the dbt package, probably by making a contribution to dbt.
Describe the tests of the solution
Test upload of source freshness against new and RC versions of dbt.
Describe the documentation
Back to how it was before dbt 1.3.0.
From SyncLinear.com | ELE-304

https://elementary-community.slack.com/archives/C02CTC89LAX/p1674494106833429

https://elementary-community.slack.com/archives/C02CTC89LAX/p1676556306013699

https://elementary-community.slack.com/archives/C02CTC89LAX/p1678305728608989
https://elementary-community.slack.com/archives/C02CTC89LAX/p1677715870599249

The fact that we run the upload of results in a separate command causes the metadata to be outdated:
https://elementary-community.slack.com/archives/C02CTC89LAX/p1695821762222709

https://elementary-community.slack.com/archives/C02CTC89LAX/p1695957431765649


Table overflow

Fix table overflow in mobile view.

fixed in https://github.com/branyang02/notie/commit/ddae7d56449e26b91e9b80b96bd31bdba32ef7c9.


Ignite as a cache for SQL Server database

Hi all,
is there a way to configure and make apache ignite as a caching layer for SQL Server database. something similar to Redis RDI 'Only available in the enterprise version'.

Yes, you can use SQL Server JDBC driver and CacheJdbcPojoStore - see RDBMS Integration docs.

Thanks @ptupitsyn is there any example 'docker-compose' or something similar that can kick start it? I'm not very familiar with Ignite. we are having this issue with some tables in SQL Server with millions of records with 1000s requests per minute making it very slow to only depend on SQL. that's why we want to create a cache where all the read query will be using it. and normal write to SQL that it will be 'somehow' synced to Ignite. I'm currently looking into some CDC solutions like Debezium and such to solve this issue.

There is a full example based on embedded H2 database:
https://github.com/apache/ignite/blob/master/examples/src/main/java/org/apache/ignite/examples/datagrid/store/jdbc/CacheJdbcStoreExample.java
Switching to SQL Server should be a matter of replacing the driver and connection string.


Flame fails when tool installed with pnpm

Expected Behavior
clinic flame -- node script.js # run test and ctrl+c should generate a flame graph

Current Behavior
ctrl+c returns error if `clinic was installed with pnpm:
[==  ] Analysing dataevents.js:183
      throw er; // Unhandled 'error' event
      ^

Error: ENOTDIR: not a directory, stat '/Users/jsumners/n/pnpm-global/1/node_modules/.registry.npmjs.org/clinic/3.0.0/node_modules/clinic/bin.js/node_modules/d3-dispatch'

Steps to Reproduce (for bugs)

npm i -g pnpm
pnpm i -g clinic
clinic flame -- node script.js
ctrl+c to finish test and attempt to generate graph

Environment

Clinic version: v3.0.0

Node version: v8.15.0

Browser name and version:

Operating system and version: macOS 10.14.3

Thanks for reporting this, I can confirm that I get the same error in Node 11 as you do in Node 8.
I suspect it's because d3-dispatch is an implicit dep of a package it's required into. I get this issue in Clinic Flame only, not Bubbleprof or Doctor, and only with d3-dispatch.
As a short term workaround, pnpm i -g d3-dispatch gets Clinic Flame working on pnpm like the other tools.
For a longer term fix, I've posted a PR on d3-fg to fix the implicit dependency: https://github.com/davidmarkclements/d3-fg/pull/31 - there doesn't seem to be anything else unusual about d3-dispatch.

@goto-bus-stop I see you've done some work on fixing Browserify stack paths for things like pnpm e.g. https://github.com/browserify/module-deps/pull/133 - do you think a required dep being implicit and not in package.json seems like an adequate explanation for why the browserify / module-deps stack might give an incorrect path on pnpm but not npm?

Yes, I am sure that is the problem.

Appears fixed by d3-fg 6.13.2 release


After auto-attaching a port, "Open in Browser" option do not respect protocol

Type: Bug

Open a remote project via SSH
From terminal, run a server that prints its URL when it runs, for instance "Server running on https://localhost:5000"
This should trigger port forwarding on 5000 automatically
This should also open a popup with an "Open in browser" option
The opened URL will be "http://localhost:5000" instead of expected HTTPS protocol "https://localhost:5000".

I guess the URL is rebuilt from the port value instead of being used as-is. For this open button, it would be better to respect the URL logged by the server, including the protocol.
 For this open button, it would be better to respect the URL logged by the server, including the protocol.

We actually default all forwarded ports to http when they're forwarded. The original protocol of the port is ignored. Keeping the original protocol is fair though. In the mean time, you can change the default to https if you want:
"remote.otherPortsAttributes": {
  "protocol": "https"
}

This feature request is now a candidate for our backlog. The community has 60 days to upvote the issue. If it receives 20 upvotes we will move it to our backlog. If not, we will close it. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

This feature request has not yet received the 20 community upvotes it takes to make to our backlog. 10 days to go. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

This feature request has not yet received the 20 community upvotes it takes to make to our backlog. 10 days to go. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

Technically this could be considered a bug more than a feature request, because VS code does try to open an URL that is incorrect

Took a quick look at implementing this to see if it would be quick. I'm sad to say that it won't be quick. Some notes for myself or whoever looks at this next:

For process based port discovery we don't actually know the protocol used at the moment. We'd need to add discovery of this into the findPorts logic in the extension where we do all the proc file system reading, if it's even possible to do so.
For output based port discovery this is easier as we have already found a URI that likely includes a protocol.

:slightly_frowning_face: In the last 60 days, this feature request has received less than 20 community upvotes and we closed it. Still a big Thank You to you for taking the time to create this issue! To learn more about how we handle feature requests, please see our documentation.
Happy Coding!


Add missing Coinbase currency pairs

Expected Behavior
Lean support all Coinbase currency pair.
Actual Behavior
Symbol can't be found in the Symbol Properties Database: BTCUSDC.
Update: SUSHIUSD is not in SPDB but QuantConnect has its data.
Potential Solution
Add missing pair (see #3341)
Reproducing the Problem
Algorithm with AddCrypto("BTCUSDC").
Checklist

[x] I have completely filled out this template
[x] I have confirmed that this issue exists on the current master branch
[x] I have confirmed that this is not a duplicate issue by searching issues
[x] I have provided detailed steps to reproduce the issue

BTCUSDC already present for GDAX, Binance & kraken 👍


Query bytea type result error

What happens?
I am developing quals push-down feature and have found that the query result is abnormal
pg_lakehouse=# select binary_col from t1   where  binary_col = '\x68656c6c6f';
 binary_col
------------
(0 rows)

pg_lakehouse=# select binary_col from t1   where  binary_col = E'\\x68656c6c6f';
 binary_col
------------
(0 rows)

pg_lakehouse=# select binary_col from t1;
    binary_col
------------------
 \x68656c6c6f
 \x
 \x70617271756574
(3 rows)

pg_lakehouse=# select binary_col from t1   where  binary_col = E'';
 binary_col
------------
 \x
(1 row)

pg_lakehouse=#

To Reproduce

export parquet file in the code.  test_arrow_types.parquet
create foreign table

CREATE FOREIGN DATA WRAPPER parquet_wrapper
HANDLER parquet_fdw_handler
VALIDATOR parquet_fdw_validator;

CREATE SERVER parquet_server
FOREIGN DATA WRAPPER parquet_wrapper;

CREATE FOREIGN TABLE t1 (
    boolean_col       boolean,
	int8_col          smallint,
	int16_col         smallint,
	int32_col         integer,
	int64_col         bigint,
	uint8_col         smallint,
	uint16_col        integer,
	uint32_col        bigint,
	uint64_col        numeric(20),
	float32_col       real,
	float64_col       double precision,
	date32_col        date,
	date64_col        date,
	binary_col        bytea,
	large_binary_col  bytea,
	utf8_col          text,
	large_utf8_col    text
)
SERVER parquet_server
OPTIONS (files '/home/kyss/labs/paradedb/test_arrow_types.parquet');

query

select binary_col from t1   where  binary_col = '\x68656c6c6f';
select binary_col from t1   where  binary_col = E'\\x68656c6c6f';
select binary_col from t1   where  binary_col = E'';
select binary_col from t1;

OS:
x86
ParadeDB Version:
0.8.6.
Are you using ParadeDB Docker, Helm, or the extension(s) standalone?
ParadeDB pg_lakehouse Extension
Full Name:
kyss
Affiliation:
NA
Did you include all relevant data sets for reproducing the issue?
Yes
Did you include the code required to reproduce the issue?

[X] Yes, I have

Did you include all relevant configurations (e.g., CPU architecture, PostgreSQL version, Linux distribution) to reproduce the issue?

[ ] Yes, I have

Hi, I think it's about the difference of duckdb and postgres hex representation. https://www.postgresql.org/docs/current/datatype-binary.html https://duckdb.org/docs/sql/data_types/blob

Yep seems like this is the actual problem. I checked the query execution but its working fine.
Will raise a PR as soon as I come up with a possible fix.

Hi @destrex271. Any update on this?

Nope not any as of now, I think it's better if someone else picks it up for now

Thank you!


Use git diff command to get diff of a GitHub pull request

[x] Updated Unreleased section in CHANGELOG or it's not notable changes.

Fix https://github.com/reviewdog/reviewdog/issues/1696
Based on https://github.com/reviewdog/reviewdog/issues/1696#issuecomment-2028410272, I use git diff command to get diff of a GitHub pull request instead of the GitHub API.

Hi @massongit ,
Thank you for investigating the fix for this issue. Are we on track to resolve the check failure (reviewdog / reviewdog (github-check) (pull_request)) soon? We're eager to know if this can be addressed soon, as the resolution is critical for our pending PRs for which reviewdog is failing.
Appreciate your efforts on this.

I think it will take several days to weeks to resolve this issue.

Therefore, I leave it as is.

@massongit Does that mean this PR will not be merged anytime soon?

I think that it will be merged soon if the review passes.

@shogo82148 @haya14busa Please review it.

CI reviewdog / reviewdog (github-check) (pull_request) seems to fail if a pull request is created from a forked repository.
Therefore, I leave it as is.

btw, it's actually unexpected. reviewdog should not fail from a forked repository too.
I guess there are some bug about exit code handling 🤔

https://github.com/reviewdog/reviewdog/pull/1697#issuecomment-2030085520
I submit https://github.com/reviewdog/reviewdog/pull/1700 that fixes that issue.
However, accoding to https://github.com/reviewdog/reviewdog/pull/1697#discussion_r1555827335, it doesn't seem to be a desirable solution...

https://github.com/reviewdog/reviewdog/pull/1697#issuecomment-2042785381

btw, it's actually unexpected. reviewdog should not fail from a forked repository too.
I guess there are some bug about exit code handling 🤔

I fixed it: https://github.com/reviewdog/reviewdog/pull/1707
Therefore, please also review it.

Hi @massongit ,
Thank you for the quick updates in response to the feedback!
Could the reviewers @haya14busa and @shogo82148 please take a look at the recent changes when you have a moment?
Thank you for your efforts on this!


Missing a GitHub issue form template for docs issues

What version of starlight are you using?
n/a
What version of astro are you using?
n/a
What package manager are you using?
n/a
What operating system are you using?
n/a
What browser are you using?
n/a
Describe the Bug
I'd like to suggest some additions to docs, but the only template I can use is 🐛 Bug Report. Could we have a template for https://starlight.astro.build docs suggestions, please?
Filing as a bug report issue as suggested on Discord: https://discord.com/channels/830184174198718474/1070481941863878697/1303808425943236608
Link to Minimal Reproducible Example
n/a
Participation

[X] I am willing to submit a pull request for this issue.

Could we have a template for starlight.astro.build docs suggestions, please?

Sure! Seems like a good idea to me.
Although I might suggest a distinction based on how we use the Starlight bug tracker:

An issue template for errors/problems with docs
A discussion template for new docs content suggestions

So I guess that’s two new things 😁
Basically, a while back we moved to using GitHub issues exclusively for bug reports, moving feature requests to discussions and I love having that distinction. A feature request might be a good idea but still take 2 years to get to sometimes and I found having those long-lived ideas gathering feedback alongside more urgent bug reports was difficult to manage. It lets us have a really clear “bugs” inbox of stuff to fix while still letting people share ideas that may take longer to get to (or require more community feedback).
So I think a similar distinction between “SOMETHING IS WRONG IN DOCS!” and “Hey, I would have loved to have a guide about X” would be good here too.
Are you interested in making a PR proposing these templates to kick things off?

I really like that distinction, yeah! +1
Is it ok if I take a step back and not send the PR? It seems to me like you have a pretty solid idea of what you want. My hunch is I'd have to get a few rounds of PR review to get there. 🙂

Templates added!

I'd like to suggest some additions to docs

Your time is now @JoshuaKGoldberg 😁
https://github.com/withastro/starlight/discussions/new?category=docs-suggestions


Really slow performance (list of 1000 items)

It takes more than 10 seconds to show a list of 1000 items (compared to Angular or React which takes less than 2 seconds).
To reproduce it, use the Visual Studio Blazor template and change the SampleDataController like this :
 [HttpGet("[action]")]
        public IEnumerable<WeatherForecast> WeatherForecasts()
        {
            var rng = new Random();
            return Enumerable.Range(1, 1000).Select(index => new WeatherForecast
            {
                Date = DateTime.Now.AddDays(index),
                TemperatureC = rng.Next(-20, 55),
                Summary = $"Summary {index}"
            });
        }

For me, it takes about 2 seconds in Firefox, about 4 in Edge, and yes about 10 in Chrome.
We've known for a while that Chrome's WebAssembly performance lags a long way behind the other browsers. It gets reported often. We spoke with some of the team leads from V8 (the underlying JS/WASM engine for Chrome) and they told us that they have some major work in progress that they expect will make Chrome at least as fast, or faster, than Firefox for WebAssembly. So this difference should go away long before Blazor is ready. I think their ETA was something like "a few months" but can't be more precise about that; you'd have to ask them.
More generally we are working with the Mono team, who supply the underlying .NET-on-WebAssembly runtime to hopefully get some major speedups across all browsers. The Mono folks have told us that their interpreted runtime hasn't yet had any optimisation work done on it, and when they do they expect some very major gains.

I understand.
For information, for me, it would be more :

Firefox (last version 47.0.2) : 31 seconds
Google Chrome (last version 67.0.3396.87) : 21 seconds
Edge : 4 seconds
=> As you see, Firefox is slower on my computer than Chrome, strange !!!

Hope for you than efforts will be done quickly on performance side else it could discourage people to invest time on your project which seems to be a really great way to see future of web.
Best regards,

@sylvainonweb the latest version of firefox is 60.0.2, i'm not sure v47 even supports webassembly

Probably not :) which explains the poor performance.
Now that I downloaded the "real" latest version (47.0.2 should probably be the latest x86 version), it takes 2 seconds as you.
Thanks.

Is the issue on the rendering side? And if so wouldn't virtualization of the visual tree be the answer the same way it is in Wpf / Uwp? I'd never put a thousand rendered items in a list box at the same time on any platform

Yes, I think it's on the rendering side. Yes, you're probably right that virtualization could be the answer but I'm a newbie in web and don't know a solution to virtualize data of a .
Paging could also be another answer but compared to angular or react Template projets, there is a huge difference (before Steve told me that my version of browser wasn't compatible with webassembly).
Now, for me, there isn't anymore matter. performance are really acceptable.
Note : In WPF, it doesn't take 20s to fill a grid that isn't virtualize (5s OK but not 20s).

Virtualization ain't as hard as it seems. I'm sure if TPTB don't build it into Blazor a clever component designer will give us a solution.
PS - It depends on the complexity, but 1000 items in a WPF Listbox could easily take in the double-digit seconds to render the visual tree without virtualization. Also, Angular craps out at around 100-200 items without virtualization with anything complex, I've found. So I don't think this is anything new.

Cool
In WPF, never see this. it takes time but not 10s.
When I do the same test with the Angular Template, it takes 2s so...

Again depends on the complexity... But I'm sure you're right that Blazor is slower overall.

But soon, it will be a matter anymore :)


Update supported k8s versions to 1.23,1.24, and 1.25

Signed-off-by: Waleed Malik<EMAIL_ADDRESS>What this PR does / why we need it:
This PR adds support for k8s 1.25 and removes k8s 1.22 as supported k8s version since it reached EOL on 28 Oct 2022.
Which issue(s) this PR fixes:

Fixes #
What type of PR is this?

/kind cleanup
/kind feature
Special notes for your reviewer:
Does this PR introduce a user-facing change? Then add your Release Note here:

Update supported k8s versions to 1.23,1.24, and 1.25.

Documentation:

NONE

/retest


Request: Ability to whitelist mobs surviving various conditions

Some mobs should logically be able to survive hostile environmental conditions, especially modded mobs.

I believe there's already a config for that. Idr what it is off the top of
my head.  I'll get the name when I get back to a computer

"entityAtmBypass" property.  Take unlocallized entity names


clarifying variable type

Although expressions are mentioned in the preamble to the example, it was not stated which parts of the example were expressions. This cost some time to figure out. This line identifies the variables id and email as expressions.

Somehow missed this PR. Thanks for your contribution! Do you feel that the existing sentence (which links to the expressions section of the documentation) was not helpful enough? I'm trying to figure out the balance of where and where not to over-explain.

Yes, there does need to be balance and I wrestled with the PR myself. But
yes, see - you do actually mention higher on the page about the
expression... but it's not pointed out which of the items in the examples
are expressions.
Once you know it, it's very easy. But for somebody brand new to your
project or working with this aspect of swift, it's easy to expect those
variables to be strings.
I've been making heavy use of this in the latest update to my shipping open
source app (also MIT license.) I'd love to give your project a shout out in
the app credits.
http://citypaper.net/phillybikedock-indego/
Peace,
Jack
On Wed, Jun 24, 2015 at 6:50 PM, Stephen Celis<EMAIL_ADDRESS>wrote:

Somehow missed this PR. Thanks for your contribution! Do you feel that the
existing sentence (which links to the expressions section of the
documentation) was not helpful enough? I'm trying to figure out the balance
of where and where not to over-explain.
—
Reply to this email directly or view it on GitHub
https://github.com/stephencelis/SQLite.swift/pull/143#issuecomment-115036482
.

Going to close this for now as I revisit the documentation for Swift 2. Thanks for the PR, though!

Alright, I guess.  good luck with your effort
On Mon, Aug 24, 2015 at 4:35 PM, Stephen Celis<EMAIL_ADDRESS>wrote:

Going to close this for now as I revisit the documentation for Swift 2.
Thanks for the PR, though!
—
Reply to this email directly or view it on GitHub
https://github.com/stephencelis/SQLite.swift/pull/143#issuecomment-134371503
.


Document how to use different settings for different pipelines of the same tap/target

Migrated from GitLab: https://gitlab.com/meltano/meltano/-/issues/2176
Originally created by @DouweM on 2020-07-16 20:54:21

See https://meltano.slack.com/archives/CFG3C3C66/p1594667270109000?thread_ts=1594434709.063300&cid=CFG3C3C66

View 1 previous comment from the original issue on GitLab


WindowShineTR v12

WindowShineTR has been updated to v12 for KSP 1.1.3, but it's marked as frozen in NetKAN.
http://forum.kerbalspaceprogram.com/index.php?/topic/110080-113-windowshine-v12-stock-and-mod-dynamic-part-reflections-1-jul-2016/

This mod has to be manually updated for each release, so many thanks for bringing it to our attention!

Awesome, thanks for acting so quickly!


Build error: use of undeclared identifier 'CDVUserAgentUtil'

Hi, I'm getting this error when I try to build for iOS:
error
:
use of undeclared identifier 'CDVUserAgentUtil'
   [CDVUserAgentUtil releaseLock:vc.userAgentLockToken];
    ^
/Users/vagrant/git/platforms/ios/[app]/Plugins/cordova-plugin-ionic-webview/CDVWKWebViewEngine.m
:
871
:
6
:
error
:
use of undeclared identifier 'CDVUserAgentUtil'
   [CDVUserAgentUtil releaseLock:vc.userAgentLockToken];
    ^
/Users/vagrant/git/platforms/ios/[app]/Plugins/cordova-plugin-ionic-webview/CDVWKWebViewEngine.m
:
878
:
91
:

It's the same thing happening here. Might it be related to this PR?
Env:

cordova: 9.x.x (latest)
cordova-ios: 6.0.0-dev (forked and updated the package.json to use latest cordova-node-xcode v3.0.0
ionic-webview version: 4.2.1

I tried building with v4.2.0 of this plugin, and instead got only this error:
error
:
no known instance method for selector 'userAgent'
    NSString* userAgent = [self.commandDelegate userAgent];
                                                ^~~~~~~~~
/Users/vagrant/git/platforms/ios/[app]/Plugins/cordova-plugin-file-transfer/CDVFileTransfer.m
:
390
:
52
:

So I went back to cordova-ios v5.1.1 and the error went away. This is with v4.2.0 of this plugin. Trying with v4.2.1.

Yes with cordova-ios v5.1.1 and latest version of this plugin (4.2.1) the problem goes away. Closing this issue.

Worked for me as well

This is still an issue for the latest cordova-ios (6.1.0).  This should be re-opened.

That issue still valid :(

so how to solve it with ios version 5.? Thanks

Just for the record: I got this error after upgrading the Cordova CLI to version 10 (npm i -g cordova) and adding plugins with Cordova 10. Cordova 10 suddenly added plugins to the devDependencies in package.json compared to Cordova 9 where it was in dependencies. Going back to version 9 (npm i -g cordova@9) made the error go away.

@retoheusser  perfect man!! I happen to install Cordova 10 by mistake. Didn't know 10 is latest now. Thanks a ton, downgrading to 9 fixed for me too

@retoheusser huge thanks! 🙏 solved my issue


🔩 Ticket #98998 - Create Azure Virtual Network from Backstage.

Creating Azure Virtual Network in stack bobitjkk9
This is an initial pull request to create an Azure Virtual Network with suffix 001 in stack bobitjkk9 and was created based on the Backstage template.
If you need to add more parameters, check the official documentation - https://github.com/claranet/terraform-azurerm-vnet
created by: Backstage Software Template 👷♂⚙👷♀

Terraform Format and Style 🖌success
Terraform Initialization ⚙success
Terraform Validation 🤖Success! The configuration is valid.
Terraform Plan 📖success
Show Plan

Terraform used the selected providers to generate the following execution
plan. Resource actions are indicated with the following symbols:
  + create

Terraform will perform the following actions:

  # module.azure_virtual_network_98998.azurerm_virtual_network.main_vnet will be created
  + resource "azurerm_virtual_network" "main_vnet" {
      + address_space       = [
          + "<IP_ADDRESS>/16",
        ]
      + dns_servers         = []
      + guid                = (known after apply)
      + id                  = (known after apply)
      + location            = "eastus"
      + name                = "vnet-bobitjkk9-client-use-prod-001"
      + resource_group_name = "rg-bobitjkk9-client-prod-001"
      + subnet              = (known after apply)
      + tags                = {
          + "CostCenter"  = "00055"
          + "Criticality" = "High"
          + "Entity"      = "CA"
          + "Owner"       = "Bob Tayara"
          + "env"         = "prod"
          + "stack"       = "bobitjkk9"
        }
    }

  # module.rg.azurerm_resource_group.main_rg will be created
  + resource "azurerm_resource_group" "main_rg" {
      + id       = (known after apply)
      + location = "eastus"
      + name     = "rg-bobitjkk9-client-prod-001"
      + tags     = {
          + "env"   = "prod"
          + "stack" = "bobitjkk9"
        }
    }

Plan: 2 to add, 0 to change, 0 to destroy.

Pusher: @ibrt2016, Action: pull_request, Workflow: Terraform Plan


MM-54323 - Calls: Incoming call from different server

Summary

Changed the banner for an incoming call from a different server.

Ticket Link

https://mattermost.atlassian.net/browse/MM-54323

Checklist

[ ] Added or updated unit tests (required for all new features)
[x] Has UI changes
[ ] Includes text changes and localization file updates
[x] Have tested against the 5 core themes to ensure consistency between them.

Device Information

Android: 13, Pixel 6
Android: 13, Galaxy Tab s7+
iOS: 16.5.1, iPhone 14

Screenshots

iOS other server DM
iOS other server GM
Android tablet other server GM

Release Note
Calls: Added the server name to the call notification banner for a better user experience.

Building app in separate branch.

More of a UX comment I suppose. I'd probably expect to always show the server name if the app has more than one configured to avoid the inconsistency and potential confusion. In other words, is the gain in banner height worth not being explicit?

More of a UX comment I suppose. I'd probably expect to always show the server name if the app has more than one configured to avoid the inconsistency and potential confusion. In other words, is the gain in banner height worth not being explicit?

I think that's a very fair point @streamer45. I can get behind that rationale. So this means we would always show the server name when your mobile app has multiple servers configured, but if you only have one server configured in the mobile app, you will never see the server name on incoming call banners.

What just happened? I rebased and force pushed, and Github thought I closed it?


fix(dark mode): Use css variables for main theme color to avoid dark mode flash (removes previous css hack)

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Release v1.4.0

Configuration

The RUM configuration is now stable.

Semantic Conventions
Breaking Changes

Remove namespace field from stack-lines and incorporate it to the function
field.
Remove line and column ranges in stack lines.

Bugfixes

Clarify that there are two metadata lines (thread metadata and thread state),
both of which can be left empty, containing only the newline.

Enhancements

Add a required telemetry.sdk.language attribute to ResourceLogs.resource.
Allow : characters in file names.
Clarify source.event.period unit is milliseconds.

Link checks are failing because release has not been made yet.


Improve connectivity issues

Task/Issue URL: https://app.asana.com/0/1199371158290272/1202061499094755/f
Description
To improve connectivity issues this PR will track network hand-offs and reconfigure the VPN DNS accordingly
Steps to test this PR
Test AppTP config defaults

[ ] install from this branch and enable AppTP
[ ] go to settings -> AppTP dev settings
[ ] verify only IPv6 support option is ON in the CONFIG section
[ ] verify apps work and AppTP blocks trackers both in WIFI and CELLULAR networks
[ ] go to settings => AppTP dev settings and enable Network Switch Handling
[ ] (VPN should automatically restart)
[ ] verify apps work and AppTP blocks trackers both in WIFI and CELLULAR networks
[ ] go to settings => AppTP dev settings and enable Set active network DNS
[ ] (VPN should automatically restart)
[ ] verify apps work and AppTP blocks trackers both in WIFI and CELLULAR networks

NOTE: The feature always set DNS is added to internal features but we don't plan to use it for now in production, see this task for more info about why

Current dependencies on/for this PR:

develop

PR #1843   (this PR)

This comment was auto-generated by Graphite.


SW-5066 Mock Server spike [DO NOT MERGE]

Codecov Report

Merging #76 (0ae1ab4) into main (4e4a0f1) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #76   +/-   ##
=======================================
  Coverage   51.57%   51.57%
=======================================
  Files          15       15
  Lines         539      539
=======================================
  Hits          278      278
  Misses        210      210
  Partials       51       51

Continue to review full report at Codecov.

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


I am missing about 10% in the _count of a prometheus.SummaryVec

thank you for prometheus, I love it!
I am very new to prometheus so probably I am doing something wrong but I am consistently loosing 10% counts in the SummaryVec I report with "github.com/prometheus/client_golang/prometheus".
a summary from my test (32.826 events over a duration of 180 sec):
01_01_teststep, 50.195683, 50.017329, 60.545467, 17774
02_01_teststep, 100.196233, 100.025423, 105.679511, 9029
03_01_teststep, 150.200282, 150.034356, 160.413986, 6023

Querying the prometheus graph tool I get the following _counts: 16411, 8335, 5560 (30306). This is about 10% loss in event counts. I have a logfile of the events which shows the 32.826 events. I am not exactly sure about how prometheus handles the metrics. So is a loss in count something to be expected?
As I said most likely I am doing something wrong. Any tips on how to investigate this further? Has the prometheus reporter a logger or something?
The relevant part of my code to register the SummaryVec:
	elapsed := prometheus.NewSummaryVec(prometheus.SummaryOpts{
		Name: "gogrinder_elapsed_ms",
		Help: "Current time elapsed of gogrinder teststep in ms.",
		Objectives: map[float64]float64{0.5: 0.05, 0.9: 0.01, 0.95: 0.005, 0.99: 0.001},
	}, []string{"teststep"})  // , "timestamp"
	regElapsed := prometheus.MustRegisterOrGet(elapsed).(*prometheus.SummaryVec)

I am using your docker container to run Prometheus - some build info:
branch	stable
buildDate	20160122-21:30:47
buildUser	@9f8f0f8d724a
goVersion	1.5.3
revision	287d9b2
version	0.16.2

We expect some loss due to application crashes and the like, but that shouldn't happen when everything is healthy. Can you confirm the /metrics page is showing the wrong values for the count?

As @brian-brazil said, please confirm that the /metrics endpoint on your target has the same numbers. If yes, this is a pure client-side issue (so we should then move the issue over to prometheus/client_golang).
The client-side metric objects are supposed to count losslessly. So this would either be a bug in the client library or in your code (like concurrent registration of multiple summaries ore something – the way you register would allow multiple summaries, it should in principle work with a simple MustRegister, perhaps try it that way and see if you get a registration error).

@beorn7 @brian-bazil thank you for your help
I did not really think through what it means to have a pull model. Problem was that I did not run the exporter long enough for prometheus to scrape all metrics. Looking at /metrics like you suggested helped me to see that.
sorry for the noise
Mark


Unexpected SA1131

I updated this repository to 1.2.0-beta.553 and got lots of unexpected SA1131.
Triggers for example on these cases:
x + 1 < y
a() < y
The common denominator seems to be that the left side or the operator is not just a variable, so this is probably a very common problem.

Investigating...

Likely caused by #3710

Indeed it is. Sorry.
I pushed a fix. If you feel the slightest bit unsure about this, you could revert dfd5ed2e454d71a563dac50ca7bed57a37d31e9f instead.

Were you planning to submit a PR here?

Done now.


Switch Sullen-Ural connector to v2 format

Simplified setup of player selector.
#tracks-async and .band-col-right selectors are parents of .album-tracks one, so it makes no sense to use them.

Much nicer!


[Bug]: UC 18 is missing

Is there an existing issue for this?

[X] I have searched the existing issues

Contact Name
Mikel Emmanuel
Description of the bug?
UC 18, All the capital cities in a continent organised by largest population to smallest., is missing.
Version
Develop Branch
Steps to Reproduce the Error
n/a

Code of Conduct

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

Acknowledged


Update artemis-jms-client-all to 2.37.0

About this PR
📦 Updates org.apache.activemq:artemis-jms-client-all from 2.19.1 to 2.37.0
📜 GitHub Release Notes - Version Diff
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.apache.activemq", artifactId = "artemis-jms-client-all" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "org.apache.activemq", artifactId = "artemis-jms-client-all" }
}]

labels: library-update, early-semver-minor, semver-spec-minor, commit-count:1

Superseded by #458.


Is there original code commited by neopenx at Mar 2, 2016

Anybody has this original code ?
my email address<EMAIL_ADDRESS>thanks!

Dragon 0.1 was removed due to enormous potential bugs and poor designing of architecture.
If you still want to refer it,  download from https://pan.baidu.com/s/1slRVTzn.

I got it, thanks!


Securing Sensitive Information in Transit and at Rest

Securing Sensitive Information in Transit and at Rest
In Transit
It is imperative that sensitive information while getting exchanged between Cloud Controller and EVE is not in plain text but in encrypted form. For increased level of security, object level encryption should be used not relying on the underlying transport security (which may involve a TLS level proxy, thus exposing the object in clear form in the middle).
Approach
Zedagent microservice is responsible fetching the configuration from Controller, periodically. Controller will encrypt configuration with device public key before sending it to zedagent. Zedagent will decrypt using device private key before processing the configuration. In future, a key refreshing mechanism can be brought in place. I.e. instead of using a fixed keypair, we can rotate the keypair on a periodic basis. This will be investigated in later phases.
At Rest
It is also imperative to store sensitive information on desk in encrypted format, instead of plain text. This is required in addition to disk/partition/directory level encryption, so that even in case of a hacker getting root access to the device, can access the file system, but the contents of the file can not be decoded without getting access to the decryption access. I.e.

File system level encryption ensures security in case the disk is physically stolen/compromised

Encrypting sensitive information before storing on the disk ensures that even if some hacker/malware gets root access on the device, the sensitive information stored in the file is not in plain text.

Approach:
Zedagent microservice is responsible for parsing the configuration from controller, and passing the configuration to other microservices using pubsub infrastructure. In the process, zedagent writes device config under /persist/config/checkpoint and /persist/config/GlobalConfig. While writing to these directories, sensitive information like VncPassword, S3 Storage credentials will be encrypted using public key of the device. Accordingly the readers/subscribers who are processing this config will also be modified to decrypt using device.key.pem(for devices without TPM) or with device key inside TPM.
Staging Plan for Implementation
Since this activity involves code changes in Controller as well as in EVE, the following transition plan is proposed:

First implement a POST api in controller for fetching device config. This is in addition to existing GET method

In the POST method, send device config after encrypting it using device’s public key

Change EVE code to do a POST instead of GET, and from reply to the POST message decrypt the device config sent from Controller using device’s private key

Devices running old code will still do a GET and Controller will reply to GET with device config in plain text - This is till all devices are upgraded to use POST instead of GET

Once all devices are upgraded to new method, stop processing GET in Controller

Hi Hari,
Is this document being sent for review? The REST API semantics are
completely off. A POST API is used for posting something to the REST API
endpoint. You seem to be using it for fetching the device configuration.
You might want to use GET verb but with different REST API endpoint, to
differentiate from the existing GET device config API, instead of POST.
Chirag Kantharia
Zededa India
On Mon, Jul 8, 2019 at 4:42 PM cshari-zededa<EMAIL_ADDRESS>wrote:

Securing Sensitive Information in Transit and at Rest In Transit
It is imperative that sensitive information while getting exchanged
between Cloud Controller and EVE is not in plain text but in encrypted
form. For increased level of security, object level encryption should be
used not relying on the underlying transport security (which may involve a
TLS level proxy, thus exposing the object in clear form in the middle).
Approach
Zedagent microservice is responsible fetching the configuration from
Controller, periodically. Controller will encrypt configuration with device
public key before sending it to zedagent. Zedagent will decrypt using
device private key before processing the configuration. In future, a key
refreshing mechanism can be brought in place. I.e. instead of using a fixed
keypair, we can rotate the keypair on a periodic basis. This will be
investigated in later phases.
At Rest
It is also imperative to store sensitive information on desk in encrypted
format, instead of plain text. This is required in addition to
disk/partition/directory level encryption, so that even in case of a hacker
getting root access to the device, can access the file system, but the
contents of the file can not be decoded without getting access to the
decryption access. I.e.

File system level encryption ensures security in case the disk is
physically stolen/compromised
2.
Encrypting sensitive information before storing on the disk ensures
that even if some hacker/malware gets root access on the device, the
sensitive information stored in the file is not in plain text.
Approach:
Zedagent microservice is responsible for parsing the configuration from
controller, and passing the configuration to other microservices using
pubsub infrastructure. In the process, zedagent writes device config under
/persist/config/checkpoint and /persist/config/GlobalConfig. While writing
to these directories, sensitive information like VncPassword, S3 Storage
credentials will be encrypted using public key of the device. Accordingly
the readers/subscribers who are processing this config will also be
modified to decrypt using device.key.pem(for devices without TPM) or with
device key inside TPM.
Staging Plan for Implementation
Since this activity involves code changes in Controller as well as in EVE,
the following transition plan is proposed:

First implement a POST api in controller for fetching device config.
This is in addition to existing GET method
2.
In the POST method, send device config after encrypting it using
device’s public key
3.
Change EVE code to do a POST instead of GET, and from reply to the
POST message decrypt the device config sent from Controller using device’s
private key
4.
Devices running old code will still do a GET and Controller will reply
to GET with device config in plain text - This is till all devices are
upgraded to use POST instead of GET
5.
Once all devices are upgraded to new method, stop processing GET in
Controller
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/lf-edge/eve/issues/127?email_source=notifications&email_token=AHI2LU6IARSST53K6EE6LP3P6MOKDA5CNFSM4H62SHRKYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4G52LBPQ,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AHI2LU344347BWSYHODD2ATP6MOKDANCNFSM4H62SHRA
.

Hi Chirag,
Thanks for reviewing. my bad, I wasn't explicit in explaining the intent behind the POST API.
With POST, we will actually be posting of a hash of last received configuration. Only if the configuration for the device has changed,  Controller will respond with the new configuration. This is to save sending the same bits unnecessarily over the network, when there is no change in the configuration. As part of this exercise(subject of the issue), we will also be encrypting the configuration with device key, whenever POST has to reply with the new configuration.

I question if a POST is the best approach here? You can use etags in a GET request to indicate what the latest version you have (define versioning how we want; hashing is fine). It has several nice advantages:

It is an RFC standard
It can be controller-optional. A controller that doesn't implement it can choose simply to ignore it
It is arbitrary. As long as the etag is sent with the original config, EVE can send it back with the next one without knowing or caring what it means, leaving it entirely up to the controller.

Practically, we will need to have some understanding when we start breaking down the config into its parts, but in principle, it is opaque to the client: receive with one GET, send with the next. Controller decides whether that warrants a full new send or just nothing to update.

I think there are 3-4 issues here, each of which should get its own. I am quite glad you got it all in here, @cshari-zededa ; they needed to be brought up and begun to be attacked in a public place.
May I suggest that we break these into separate issues and address each separately?

encryption at rest
encryption in transit
efficient caching of configuration

Each definitely deserves its own conversation.

Hey @cshari-zededa to add on top of what others have said let me first publish a set of project requirements #123 and then we can see how to split this proposal of yours.
Makes sense?


Feature/5582 event check in UI scan qr code/5850 save checkin to store

Description
-Save Checkin to the event store
-save the new default value for "add checkin to contact diary" switch "be default first time is true"
Link to Jira
https://jira-ibs.wbs.net.sap/browse/EXPOSUREAPP-5850
Screenshots
https://user-images.githubusercontent.com/15270737/112648097-7153b480-8e49-11eb-9efd-5dea81cd854c.MP4

Hey @30mar ,
thanks for implementing this! Is there any public documentation/information about the concept of this feature? I've only found this https://github.com/corona-warn-app/cwa-documentation/blob/master/event_registration.md#interoperability-with-other-contact-tracing-apps-draft - but it's only a draft...
thank you!


Cannot invoke CLI due to dependency conflict

Error:
<git:(master*)> npm install -g cdk8s-cli
npm WARN ERESOLVE overriding peer dependency
npm WARN While resolving<EMAIL_ADDRESS>npm WARN Found<EMAIL_ADDRESS>npm WARN node_modules/cdk8s-cli/node_modules/cdk8s
npm WARN<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm WARN   node_modules/cdk8s-cli
npm WARN     cdk8s-cli@"*" from the root project
npm WARN
npm WARN Could not resolve dependency:
npm WARN peer<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm WARN node_modules/cdk8s-cli/node_modules/cdk8s-plus-22
npm WARN<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm WARN   node_modules/cdk8s-cli
npm WARN
npm WARN Conflicting peer dependency<EMAIL_ADDRESS>npm WARN node_modules/cdk8s
npm WARN   peer<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm WARN   node_modules/cdk8s-cli/node_modules/cdk8s-plus-22
npm WARN<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm WARN     node_modules/cdk8s-cli

changed 180 packages, and audited 189 packages in 2s

47 packages are looking for funding
  run `npm fund` for details

found 0 vulnerabilities

A temporary workaround is to install a different version. This currently is working for me:
npm install -g<EMAIL_ADDRESS>

Two proposed changes:

Change cdk8s-plus-XX to model cdk8s and constructs as caret peer dependencies, with pinned dev dependencies. This does not fix the bug, but I think this more correctly models our dependency relationships (i.e. cdk8s-plus-XX should be compatible with newer minor/patch versions of cdk8s).

Change cdk8s-cli to require cdk8s as a caret peer dependency instead of a pinned peer dependency. This should resolve the dependency conflict since both cdk8s-cli and cdk8s-plus-22 will both allow the latest version of cdk8s as their dependency.

Fixed in cdk8s-cli v1.0.0-beta.54.


Deletion of Dockershim section from the Kubernetes website repost

/hold
This PR is dup with:

#30961
#31503

Related issue:

#30953

@jihoon-seo  May I resolve the conflicts and commit?


read unix @->/var/run/docker.sock: read: connection reset by peer

Version:           19.03.0-beta1
API version:       1.40
Go version:        go1.11.5
Git commit:        62240a9
Built:             Thu Apr  4 19:18:48 2019
OS/Arch:           linux/amd64
Experimental:      false
uname -a
Linux yuanyufeng-R730 4.15.0-47-generic #50~16.04.1-Ubuntu SMP Fri Mar 15 16:06:21 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux
is anyone has the same error?

@CLIsVeryOK Please let us know what are the resources you are using and recipe details so that we can validate.

With our observation we found that we confront this error if docker engine is not running. Please follow https://github.com/chef-cookbooks/docker#usage and use docker_service resource before any other resource.
Please let us know if there is anything
Thank you !!

Closing issue considering above comments.
Please let us know if there is anything.
Thanks!


Add webgpu IDL spec and generated code

This allows removing manual OffscreenRenderingContext and RenderingContext declarations.

Looks like this needs to be rebased?

I've rebased it, but there are still a few commits that don't matter in the end diff, and they will be squashed. Yes, the diff on Generated.swift is huge, because webgpu is apparently that big...


Add Lighthouse Audit

With #136 we got our accessibility score up to 100. While this is just a first step towards enhancing the accessibility of our website, it might be nice to automatically keep track of this score with updates, so we are aware of any potential areas of concern.
I noticed there's a GitHub Action for this: https://github.com/marketplace/actions/lighthouse-audit

the action will print the five scores (out of 100) to the output and upload HTML and JSON versions of the report as artifacts

It seems like this might be nice to incorporate.

good idea! microsoft is also working on accessibility insights: https://github.com/microsoft/accessibility-insights-action but it isnt production ready yet and i dont know the extra benefits that might be incorporated. for automated tests both rely on axe-core/Axe-coconut would be interesting which tests each turns on and off. but in any case a good way to take care that no basic no no's slip in in future commits by accident for covidtrends.

Thanks for the feedback @rpkoller!
To be honest I'm a bit out of my depth with this as I've new to using Github Actions. Ideally, it would be great if we could audit any PRs before they are merged, so that we can correct any issues before they go live. It's not clear to me how to set this up.
Also, there are also some other implementations of this listed here, in case any of the others are easier to set up for our purposes.

hmmm your workflows yaml file says:
on:
  push:
    branches: [ master ]
  pull_request:
    branches: [ master ]

when a commit is pushed into master the jobs listed afterwards are being triggered. if you push a commit into the branch test the jobs wont run. same goes for the pull requests. every pull request against master should trigger the jobs. if i create branch newfeature on my fork of covidtrends and place a pulll request to the branch newfeature it wont be checked either. If you leave out the branches part for pull request ( something i found at jeff geerlings repo in the documentary i havent)
on:
  push:
    branches: [ master ]
  pull_request:

the jobs run for pushes to master and for all pull_requests to the whole repo no matter which branch they are in. an useful addition might also be:
  push:
    branches:
      - master
  pull_request:
  schedule:
    - cron: '0 6 * * 0'

so the jobs are ran once per week anyway no matter a commit was placed or not. might be of interest especially for the performance aspect.
two nitpicks i picked up from jeff geerling in regards of yaml files in general. they should start with --- in the first line. then you normally have a key, a colon and a value so he puts on normally within quotes because it is a string ( https://youtu.be/dE7IksPs9EY?t=917 ). if you dont do that that yaml actually identifies it as a boolean value (on, yes, true).
---
name: CI
'on':
  push:
    branches:
      - master
  pull_request:
  schedule:
    - cron: '0 6 * * 0'

and about the performance tests. the first you choose looks fine. https://github.com/marketplace/actions/lighthouse-audit Paul Jarvis rings a bell. meaning it is a familiar name, savvy,  and the stars and number of contributors look decent. but taken a quick look at the others listed in your second link that one sticks out: https://github.com/marketplace/actions/lighthouse-ci-action paul irish and a few other googlers on the contributors list. and they are at version 2.x while the other is version .3.
but testing the performance you have to keep one thing in mind basically testing for "performance" makes only sense on the production server. otherwise you have environmental differences. and just testing for applied patterns is half baked ;)  but looks you are able to test several websites. so you might name the repo on github as well as https://aatishb.com/covidtrends/ ... then you could see if the production version is steady and behaving well while checking the impact on the github version for the changes before they a merged.


feat: Obtain file metadata

Depends on #52
Closes #27
Includes:

Endpoint to get metadata of a file
Tests for #52 and the new endpoint
Remove pool connection limit. As described in the commit message, it was limiting the API to 8 concurrent requests.

@shoriwe Why is this endpoint needed?. I thought it was to obtain the archiveUUID from a given fileUUID but the archiveUUID is not being used by the workers since the file is saved with the fileUUID.

@shoriwe Why is this endpoint needed?. I thought it was to obtain the archiveUUID from a given fileUUID but the archiveUUID is not being used by the workers since the file is saved with the fileUUID.

Just in case we need to query metadata in the frontend

Just in case we need to query metadata in the frontend

OK, then, let me add more metadata in the response. I´m only returning the archiveUUID and the volume now.


[DNM, WIP] Reorganization of SourceFileDepGraph building and mocking.

Streamlining and reorganizing and preparing for cross-type dependencies.

@swift-ci please smoke test


[SPARK-31322][SQL] rename QueryPlan.collectInPlanAndSubqueries to collectWithSubqueries

What changes were proposed in this pull request?

rename QueryPlan.collectInPlanAndSubqueries to collectWithSubqueries
Why are the changes needed?

The old name is too verbose. QueryPlan is internal but it's the core of catalyst and we'd better make the API name clearer before we release it.
Does this PR introduce any user-facing change?

no
How was this patch tested?

N/A

cc @hvanhovell @gatorsmile

Test build #120680 has started for PR 28092 at commit b3e3484.

Merged build finished. Test PASSed.

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

Test build #120680 has finished for PR 28092 at commit b3e3484.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

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

Merged to master/3.0.


Remove duplicate entry "sap.m.IconTabBar"

The list of controls supporting certain CSS classes contained module "sap.m.IconTabBar" twice. Since the list seems to be sorted alphabetically, the first entry should be the undesired one.

Hi @andre-schaerpf , thanks for your contribution. We appreciate it and will look into this.

Hi @andre-schaerpf, I have adapted the documentation . This will be available in 1.113. Thanks again for your contribution !

https://people.sap.com/andre.schaerpf


fix einsum

If you can, please also add a test to the CI so we won't have regressions on their model in future fixes.
It should have a similar format to this


Added initial, optional DNS resolution for cluster peers.

Signed-off-by: Bartek Plotka<EMAIL_ADDRESS>

👍


Implement snapit

Should fully function now, just need to make it merg properly

Test ran and found some issues/improvements to make
Pushed them in my dapal/snapit branch in my person repo


services.AddAuthentication().AddMicrosoftAccount is obsolete

@Rick-Anderson  edit: See UPDATING MICROSOFT ACCOUNT LOGINS IN ASP.NET CORE WITH OPENID CONNECT AND AZURE ACTIVE DIRECTORY by @damienbod
I am just finishing up an incident with Microsoft Tech support regarding the correct way to add Azure AD as an "external login provider". The method described in this document is obsolete:
services.AddAuthentication()
    .AddMicrosoftAccount` is obsolete.

Other Microsoft documentation describes using
services.AddAuthentication(AzureADDefaults.AuthenticationScheme)
    .AddAzureAD(options => Configuration.Bind("AzureAd", options));

This logs in with Azure AD, but the login happens automatically (due to the AzureADDefaults.AuthenticationScheme argument). Removing that argument, so that Azure AD might function alongside other login providers, does not work.
According to Microsoft Tech Support (who consulted with the developers), in order for Azure AD to function alongside local accounts and other providers such as Facebook, you must call
services.AddAuthentication()
    .AddOpenIdConnect(options => Configuration.Bind("AzureAd", options))

(where "AzureAd" identifies a set of configuration values, e.g. in appsettings.json) such as:
"AzureAd": {
	"Instance": "https://login.microsoftonline.com/",
	"Authority": "https://login.microsoftonline.com/{tenant ID}/v2.0/",
	"TenantId": "{tenant ID}",
	"ClientId": "{client ID",
	"CallbackPath": "/signin-oidc"
}

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

ID: 1f4b263b-e8f9-8b44-8c13-5bc0128fb1c9
Version Independent ID: 985774b6-a165-f22c-c354-6c12a6783063
Content: Migrate authentication and Identity to ASP.NET Core 2.0
Content Source: aspnetcore/migration/1x-to-2x/identity-2x.md
Product: aspnet-core
Technology: aspnetcore-migration
GitHub Login: @scottaddie
Microsoft Alias: scaddie

@sbsw AddMicrosoftAccount is used to Configure Microsoft Account Authentication, not Azure AD.
cc @CamSoper @scottaddie

Excuse me if I'm way off chipping into this conversation here.
Perhaps another MS team that confusing everything by bringing Microsoft accounts and OpenID into some kind of Microsoft "identity platform" and bringing the AD brand into the fold. I complained recently about all this.
https://github.com/MicrosoftDocs/azure-docs/issues/43056#issuecomment-555506355

here's the ASP.NET Core issue which covers this as well
https://github.com/aspnet/AspNetCore/issues/10037

Eu consigo utilizar isso para me conectar a um AD do Windows Server ??

Can I use this... to connect in Windows AD ??

OMG - I have been trying to get .Net Core 3.1 with Individual Accounts to work with single tenant AD authorization for over a week.
The 2 missing links:

Confirmation to use OpenIDConnect vs Microsoft Accounts
Clear AzureAD json options (including specification to put the tenant id both as an option by itself AND in the Authority url):
"AzureAd": {
"Instance": "https://login.microsoftonline.com/",
"Authority": "https://login.microsoftonline.com/{tenant ID}/v2.0/",
"TenantId": "{tenant ID}",
"ClientId": "{client ID",
"CallbackPath": "/signin-oidc"
}

CAUTION TO THOSE STARTING OUT:  The microsoft doc's and tutorials reference example source in Core 2.x which has completely different syntax and won't work out-of-the-box with individual accounts if you require a single tenant AD authentication.


[MPWG Legal Review] Participation Agreement

Provided on 9/16 by Sharon (EF- EMO) via MPWG mailing MicroProfile Working Group Participation Agreements
MicroProfile Working Group Participation Agreement.pdf
MPWG- MP public Folder > LEGAL

I carefully reviewed the agreement on 10/5, looks great.
Closing ticket as 3 partners have signed the MPWG Agreement without requesting adjustments.


[Question] Support for other versions other than xp/win7 ?

So we were trying to parse a Win 2012 CIM db and consistently get a MissingIndexFileError when passing win7 as an argument with the 2012 db.
Just wanted to make sure that this is expected if we try to parse a CIM belonging to a different version.
Thanks!

Hello,
Do you have the index.btr and mapingX.map in the same folder as
objects.data? You need to get the whole WBEM folder in order to parse the
WMI repo.
Thanks,
Claudiu.
On Friday, October 16, 2015, Priyank Nigam<EMAIL_ADDRESS>wrote:

So we were trying to parse a Win 2012 CIM db and consistently get a
MissingIndexFileError when passing win7 as an argument with the 2012 db.
Just wanted to make sure that this is expected if we try to parse a CIM
belonging to a different version.
Thanks!
—
Reply to this email directly or view it on GitHub
https://github.com/fireeye/flare-wmi/issues/3.

Hey,
Yes, I have Index.btr, Objects.data and Mapping1.map, Mapping2.map and Mapping3.map in a directory, which is supplied as an argument.
I guess this is what constitutes the whole WBEM folder. Correct me if my understanding is incorrect.
Thanks!

Yes. That's correct.
The command line:
WMIParser.exe -p "Path_of_the_folder_where_objects.data_is"
I will try to replicate the issue. Is it Win 2012 or Win 2012 R2?
Can you share your DB?
Thanks,
Claudiu

Well, I wasn't using WMIParser.exe (Was the compiled  binary included?)
I used this - python ui.py <xp|win7> /path/to/CIM/directory
The Gui fired up, but the data could not be read. (something wrong here?)
I am not sure about R1 or R2. let me get back to you on that in a while.

Hi,
Resurrecting an old thread.....is there going to be support for Windows Server 2008? I've used ui.py (with the Win7 option) to view a repo, but I'm not getting anything in the right-hand pane, so don't think it's parsing properly.
Thanks

Hey @BannersSecret,
I developed the python library on a Windows 10 system, so I'd expect the library to also support Server 2008. Haven't tested this, though.
Do you have a repo that you can share with us, to help triage?

Hi Willi,
Thanks a lot for coming back to me. Apologies, but unfortunately the repo is from a client's machine, so I'm unable to share. However if there's anything I can do to provide you further info, please let me know.
Thanks.

On 30 Jul 2016, at 01:01, Willi Ballenthin<EMAIL_ADDRESS>wrote:
Hey @BannersSecret,
I developed the python library on a Windows 10 system, so I'd expect the library to also support Server 2008. Haven't tested this, though.
Do you have a repo that you can share with us, to help triage?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

would you mind trying the list.py script and seeing if there is any output? this will help triage if the issue is within ui.py or the underlying cim parsing code. try using both the xp and win7 profiles.

Hi Willi,
Sorry for the delay in responding.
I ran list.py and it worked fine. I did some further digging and it turns out I was looking at the wrong data. On a previous engagement, using up.py I'd found the smoking gun in a certain location and was expecting to see it there again. My mistake.
Apologies for the confusion. The tool is working perfectly.
Thanks again,
A
Sent from my iPad

On 30 Jul 2016, at 18:15, Willi Ballenthin<EMAIL_ADDRESS>wrote:
would you mind trying the list.py script and seeing if there is any output? this will help triage if the issue is within ui.py or the underlying cim parsing code. try using both the xp and win7 profiles.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

@BannersSecret
I'm very excited to hear that you've had success inspecting the WMI repo using these tools. Its recently been radio silent in the WMI forensics world, but I had a feeling there were some quiet analysts doing a good and thorough job :-)
Let me know if there's anything I can help out with in the future.


Implement simple example of hooking kernel pages via EPT

Hi! I have already experimental version of this in my local branch, but since this project got lots of focus in the game-cheating scene, I decided to wait out a little bit with releasing and let those people figure some basic stuff for themselves. :)

ah i see :) i'll be sure to keep an eye out for it then, kernel hooks would be a very welcome extra layer of security in my application (not game related) but i unfortunately lack any type of knowledge of hypervisors to figure out what i need to do. thanks for your release, it has been a real joy reading through and trying to learn from this project.

I got this working for kernel, it's actually extremely easy, just read the hvvpctrl source how he allocates memory for the original function+hooked and how he aligns. You do the exact same thing for kernel functions, then can inline hook the kernel function, and hide it using this wonderful project. The only thing you need to change for a quick test inside the driver is
 ULONG_PTR Cr3;
        HvppAttachAddressSpace(&Cr3);
        Data->PageRead = MmGetPhysicalAddress(Context->RdxAsPointer);
        Data->PageExec = MmGetPhysicalAddress(Context->R8AsPointer);
        HvppDetachAddressSpace(Cr3);

change Context->RdxAsPointer to OriginalFunctionBackupAligned and r8 to OriginalFunctionAligned (these being your kernel allocated buffers, again exact same as usermod just for kernel functions).

Hey wbenny! After several years of crawling the hub, this project was the first one that actually inspired me to make an account. Not to beat a dead horse, but I was also wondering if you could maybe give a small tip as to getting this working with kernel pages. Simply allocating and page aligning in the kernel instead of usermode doesn't seem to work as I thought it would. Perhaps you'd be willing to give us an early Christmas present and share some advice on what we can do? I can't speak for others, but I would be thrilled to understand what I am doing wrong.

I got this working, but not sure if he wants me to share how, I can say though it's extremely simple.

As a side note, the following examples might help as well:
https://github.com/tandasat/DdiMon
https://github.com/Bareflank/extended_apis_example_hook
Cheers,

Rian

On Wed, Dec 5, 2018 at 2:09 PM DebugBuggin<EMAIL_ADDRESS>wrote:

I got this working, but not sure if he wants me to share how, I can say
though it's extremely simple.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/wbenny/hvpp/issues/7#issuecomment-444649015, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AFqD4xOncO59rrpl0-1MtHXgHefwS9s1ks5u2DX7gaJpZM4WUUyc
.

@assemblyw0t Hello! There are several ways to do it. The simplest approach is to use the exact same steps as for UM hooking - which kinda requires you to port Detours-like behavior to KM - patch the jump and create trampoline. Second approach is much easier and is used in DdiMon (pointed out by @rianquinn) - use int3 + redirect RIP in the handler and create "micro-trampoline" (first original instruction, jmp + address). Third approach might be using DR - which, if placed at right position, can give you easily all Nt* calls.
@DebugBuggin We're on github, I can't prevent anyone who wants to fork this repo :) Quite the opposite, I highly encourage it.

@DebugBuggin Maybe he wouldn't mind if it was in private? Would you mind helping a fellow out? My email is<EMAIL_ADDRESS>I would really appreciate it if I could get in contact with you and have a few minutes of your time.

I did the first method wbenny just listed and it works great.

That's what I tried to do. I allocate kernel buffers and pass them through the same way as you did in the usermode example, but it just ends up in a kernel security check BSOD. I didn't even actually hook anything yet.

Maybe I am just dumb. But I don't see what I am doing wrong.
`   pfnIofCallDriver IofCallDriveFn = (pfnIofCallDriver)GetSystemFunctionAddress(L"IofCallDriver");
PVOID OriginalFunction = (PVOID)IofCallDriveFn;
PVOID OriginalFunctionAligned = PAGE_ALIGN(OriginalFunction);
PVOID OriginalFunctionBackup = KernelAllocateMem(PAGE_SIZE * 2);
PVOID OriginalFunctionBackupAligned = PAGE_ALIGN((ULONG_PTR)OriginalFunctionBackup + PAGE_SIZE);
memcpy(OriginalFunctionBackupAligned, OriginalFunctionAligned, PAGE_SIZE);
`
Just as the usermode example does, and then I pass these addresses in the same way as usermode example does.

Maybe I am just dumb. But I don't see what I am doing wrong.
pfnIofCallDriver IofCallDriveFn = (pfnIofCallDriver)GetSystemFunctionAddress(L"IofCallDriver");
PVOID OriginalFunction = (PVOID)IofCallDriveFn;
PVOID OriginalFunctionAligned = PAGE_ALIGN(OriginalFunction);
PVOID OriginalFunctionBackup = KernelAllocateMem(PAGE_SIZE * 2);
PVOID OriginalFunctionBackupAligned = PAGE_ALIGN((ULONG_PTR)OriginalFunctionBackup + PAGE_SIZE);
memcpy(OriginalFunctionBackupAligned, OriginalFunctionAligned, PAGE_SIZE);
Just as the usermode example does, and then I pass these addresses in the same way as usermode example does.

looks fine assuming GetSystemFunctionAddress returning proper address. Make sure you don't do those calls in, HvppHandleExecuteVmcall callback, but do them beforehand. thne can look like
Data->PageRead = MmGetPhysicalAddress(OriginalFunctionBackupAligned);
Data->PageExec = MmGetPhysicalAddress(OriginalFunctionAligned);

Addresses seem fine. Debug output:
13:45:31.395 INF #3     4    32 System          OriginalFunctionBackupAligned 77e0a000
13:45:31.395 INF #3     4    32 System          OriginalFunctionAligned fc4c9000
But once I try to pass them through,
ia32_asm_vmx_vmcall(0xc1, (uint64_t)OriginalFunctionBackupAligned, (uint64_t)OriginalFunctionAligned, 0);
Big fat BSOD.

literally just replacing
 Data->PageRead = MmGetPhysicalAddress(Context->RdxAsPointer);
  Data->PageExec = MmGetPhysicalAddress(Context->R8AsPointer);

with
  Data->PageRead = MmGetPhysicalAddress(OriginalFunctionBackupAligned);
   Data->PageExec = MmGetPhysicalAddress(OriginalFunctionAligned);

then using the usermod tool, worked fine for me. The example btw only hides 1 page so after you get this working you have to modify the code to support multiple hidden pages, which i'm working on rn.

@wbenny Sorry to bother you again. I ran into a bit of a dilemma and was hoping to get some insight. I decided to go the HyperPlatform method route you recommended with the software breakpoint hooking, but I can't seem to catch the exception? I BSOD without being able to redirect execution to my detour function. I'm doing as follows:
void vmexit_custom_handler::handle_exception_or_nmi(vcpu_t& vp) noexcept
{

  auto interrupt = vp.exit_interrupt_info();

  switch (interrupt.type())
  {

  case vmx::interrupt_type::software_exception:
    switch (interrupt.vector())
    {
    case exception_vector::breakpoint:

      if (vp.guest_rip() == IofCallDriver_Address)
      {
        vp.guest_rip(reinterpret_cast<uintptr_t>(&Detoured_IofCallDriver));
      }
      break;

    default:
      break;
    }
  }

  vp.inject(interrupt);
  vp.suppress_rip_adjust();
}

Is there something obvious that I am missing here?

I see two possible issues:
First, ensure that that breakpoint is set in the exception bitmap:
  // Inside vmexit_handler::setup() method:
  auto exception_bitmap = vmx::exception_bitmap_t{};
  exception_bitmap.breakpoint = true;
  vp.exception_bitmap(exception_bitmap);

Second, you don't probably want to reinject the breakpoint exception, when you handle it your way:
  // vp.inject(interrupt);

I ruled out the problem being in the function I'm attempting to detour into. Seems that regardless of what I change the instruction pointer to I just immediately freeze permanently. Strange...

@ assemblyw0t
can you send your project as zip to me?i will rty to do it with IofCallDriver.

@ wbenny
I'm glad to see you updated again. I don't understand EPT hidden pages, but I've been learning other people's projects to do IOPXXXControlFile and other NTOs have not exported API Hook. At present, I finished her with inline Hook with De Tors and IDT Hook with other people's projects. Win 7 is ok. Win 10 has a memory error BSOD from time to time. I'm looking forward to your EPT hiding FAKE IOPXXXControlFile Page and then continuing to correct BSOD. I've been learning and never succeeded. In order to register some data recovery software, EPT is bound to CPU Info and Disk Info, some to network card MAC, and IRP has been used to solve it.

@wbenny Any plans to implement this feature or should we implement it locally? It's been a few months and just wondering if you'll be pushing out an official version or if I should just start implementing a local version.

I got this working, but not sure if he wants me to share how, I can say though it's extremely simple.

Hi, I'm work on kernel hook and I just got multi pages, can u help me with kernel hook stuff?


Start Craft test site

This is a start, but I don't really know what I'm doing. Directions would be much appreciated @rfay!! Specifically, should  we provide file and DB tarball URLs and if so then what should they contain? My guess would be any files (templates, images, etc.) that are required and a DB dump for testing?

So I believe the next step is for you to add the files and DB tarballs to the ddev repo and send URLs to them?

For now, you can put them anywhere, including right here in a release in this repo. That way you'll be in full control. Then we'll throw them into the junk repo that ddev uses just a little farther down the line. But for now, you'll be in complete control if you just throw them somewhere including a release on this repo.

I'm thinking if we just follow the directions we give people for spinning up a new site:
mkdir testpkgcraftcms
cd testpkgcraftcms
ddev config --project-type=craftcms
ddev start
ddev composer create craftcms/craft
ddev launch

...and use that our as working project we tarball up, that'll be a good basis for a project, no?

Yup, that would be perfect, as long as it ends up with a page you can hit and a static URL and a dynamic URL, which I'll bet it will.

Yeah easy. We can create a basic "hello, world" Twig template for the dynamic page, and have any image for the static URL.
@bencroker I suggest the legendary Happy Brad image for the image:
https://github.com/mattstauffer/happybrad/blob/master/happybrad-in-action.png

I humbly suggest the legendary Happy Brad image for the image.
It's gotta be!!

Ok, so uploading the files here:
craftcms-files.tar.gz
craftcms-db.sql.tar.gz
And added to the test site. Merging and handing over to @khalwat to test.

Updating the Craft CMS source to the latest version too.
craftcms-4.2.3.tar.gz


Update installation notes

Updates some installation notes. Removes the reference to it being hosted on the Sinon site. Has not been true for quite a while. Also adds some info on using it in a browser.

Looks good. Just that npm is officially written "npm", not "NPM" because they say it's not an acronym.

First edit, commit and squash on my phone. Not pleasant, but not too shabby either :-)


remove dependency on package.xml

This extension currently requires a package.xml to be present. Similar to how colcon-cmake and colcon-ros work, I suggest that it would make a lot of sense to make this optional.
There could be a package colcon-poetry that extracts all information solely from the pyproject.toml and a colcon-poetry-ros that merges the information from the pyproject.toml and package.xml. This is useful in order to build "pure" python packages without a package.xml that only require python packages available on PyPI, and to have the option to define additional non-python dependencies in a package.xml.

Thanks for bringing this up! I considered splitting the extension up into pure-Poetry and Poetry+ROS variants, but at the time I didn't understand when that would be useful in practice. When do people make pure Python packages? I could see that being useful if Colcon was a popular tool outside the ROS ecosystem, but I don't know if it is. Do people make pure Python packages even within the context ROS?

When do people make pure Python packages?

Most Python packages are "pure" packages, i.e. do not have a package.xml.

I could see that being useful if Colcon was a popular tool outside the ROS ecosystem, but I don't know if it is.

Colcon may not be popular among Python developers, but Python is certainly popular and colcon is "just" a workspace tool for heterogeneous package sets. You do not have to use ROS in order to use colcon.

Do people make pure Python packages even within the context ROS?

I would say that most people that use ROS in Python also use other Python packages. And those are not made to work with colcon via an additional metadata file. Also, you can just use colcon without ROS.
With all that said, the basic question remains why require a package.xml when all the dependencies can be resolved with a standard pyproject.toml? My understanding is that, as long as you only require pure Python dependencies, you should be able to resolve dependencies and build packages just using poetry. The package.xml would only be useful when you have to resolve additional non-python dependencies via rosdep.

Looking at the code again, it seems it would be pretty easy to accomplish this. I've started work on it.

Looking at the code again, it seems it would be pretty easy to accomplish this. I've started work on it.

Great! I am happy to test this. Let me know if/when I can give it a try.

:+1: for this.

@christian-rauch I have #32 up to add this feature. Would you mind testing it out with your setup to see if it works for your use-case? It can be installed with this Pip command:
pip3 install git+https://github.com/UrbanMachine/colcon-poetry-ros.git@0b2b4923b419624b9ce112c73137d5eacbd364e5

@stonier I think once #32 is merged, any old pyproject.toml should work. You won't need special sections like [tool.colcon-poetry-ros.data-files] for non-ROS Poetry packages, if that's what you're concerned about. Perhaps there are some ways that a pyproject.toml could be configured that would conflict with this extension, but I'm not sure what they would be yet.

Oh, indeed. I parsed the README a little too quickly, that custom section is only for ROS files. Excellent.


Typo "**App fps: **"→"App fps:"

https://docs.microsoft.com/en-us/windows/winui/api/microsoft.ui.xaml.debugsettings.enableframeratecounter?view=winui-3.0-preview

@hyoshioka0128 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

The formatting fix looks good; merging. Thanks @hyoshioka0128

@Karl-Bridge-Microsoft In the two tables, are the asterisks that are visible meant to be literal asterisk values or is it an attempt to make the enclosed text italic? If the latter, asterisks won't work because the table is created by using HTML. To get italic font there, you'd need to use <i></i>. I'd fix it but not sure about the intention. Thanks!

@ktoliver - thanks for pointing this out. I've updated the topic with revised formatting.

@ktoliver - thanks for pointing this out. I've updated the topic with revised formatting.


add more Samsung tests

Coverage increased (+0.5%) to 79.769% when pulling e48055bf5e3e427de95a2859de009ba20e0b308c on add-samsung-tests into 24ee3e01094ff70875c668cbfab5422051289136 on master.

Codecov Report

Merging #51 into master will increase coverage by 0.51%.
The diff coverage is 100%.

@@             Coverage Diff              @@
##             master      #51      +/-   ##
============================================
+ Coverage     79.65%   80.16%   +0.51%
- Complexity     5607     5613       +6
============================================
  Files           328      328
  Lines         11388    11400      +12
============================================
+ Hits           9071     9139      +68
+ Misses         2317     2261      -56

Impacted Files
Coverage Δ
Complexity Δ

src/Factory/Device/Mobile/SamsungFactory.php
100% <100%> (+3.82%)
738 <0> (+6)
:arrow_up:

Continue to review full report at Codecov.

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


update example/rethinkdb to v1beta3

update example/rethinkdb to v1beta3, add external load balancer to work on GCE, add a comment about nodeSelector in gen-pod.sh.
@nikhiljindal @antmanler

@nikhiljindal The UI works. But I think I need to change the run.sh as well, as pointed out in #7258. Please hold until I fix it. Thanks.

https://github.com/GoogleCloudPlatform/kubernetes/blob/master/examples/rethinkdb/image/run.sh#L26

I fix the run.sh to v1beta3. @antmanler, could you rebuild your docker image? or you can copy mine at caesarxuchao/rethinkdb:v2. Thank you.
@nikhiljindal  let's wait for greens

Will wait for @antmanler's LGTM before merging.

@nikhiljindal. Thank you for the picks. I was too hurry to go home : )

@caesarxuchao Thank you for you contribution, I'm so busy these days, and sorry for late reply.
I've copied and pushed your image to antmanler/rethinkdb, and it works well on my local cluster.
LGTM.

LGTM, thanks.
Shippable is green, merging.


luci-app-attendedsysupgrade: "Unsupported package(s): libubus20191227"

I'm on a archer-c7-v4 running LuCI openwrt-19.07 branch (git-21.044.30835-34e0d65) / OpenWrt 19.07.7 r11306-c4a6851c72, attendedsysupgrade-common	0.2-1 and luci-app-attendedsysupgrade	git-21.267.59215-1524972-1; when I try to run attended sysupgrade to 19.07.8 from luci using the defaults, I get the error below. I suspect it's related to https://github.com/openwrt/luci/issues/5219, but the error message explicitly says to report the issue, so here goes:
Error building the sysupgrade

Unsupported package(s): libubus20191227
Please report the error message and request
Request to server:
{
    "board_name": "tplink,archer-c7-v4",
    "target": "ath79/generic",
    "version": "19.07.8",
    "packages": [
        "ath10k-firmware-qca988x-ct",
        "base-files",
        "busybox",
        "cgi-io",
        "collectd-mod-conntrack",
        "dnsmasq",
        "dropbear",
        "firewall",
        "fstools",
        "fwtool",
        "getrandom",
        "hostapd-common",
        "ip6tables",
        "iptables",
        "iw",
        "iwinfo",
        "jshn",
        "jsonfilter",
        "kernel",
        "kmod-ath",
        "kmod-ath10k-ct",
        "kmod-ath9k",
        "kmod-ath9k-common",
        "kmod-cfg80211",
        "kmod-gpio-button-hotplug",
        "kmod-ip6tables",
        "kmod-ipt-conntrack",
        "kmod-ipt-core",
        "kmod-ipt-nat",
        "kmod-ipt-offload",
        "kmod-lib-crc-ccitt",
        "kmod-mac80211",
        "kmod-nf-conntrack",
        "kmod-nf-conntrack6",
        "kmod-nf-flow",
        "kmod-nf-ipt",
        "kmod-nf-ipt6",
        "kmod-nf-nat",
        "kmod-nf-reject",
        "kmod-nf-reject6",
        "kmod-ppp",
        "kmod-pppoe",
        "kmod-pppox",
        "kmod-slhc",
        "kmod-usb-core",
        "kmod-usb-ledtrig-usbport",
        "kmod-usb-storage",
        "kmod-usb2",
        "libblobmsg-json",
        "libip4tc2",
        "libip6tc2",
        "libiwinfo-lua",
        "libiwinfo20181126",
        "libjson-c2",
        "libjson-script",
        "liblua5.1.5",
        "liblucihttp-lua",
        "liblucihttp0",
        "libnl-tiny",
        "libubox20191228",
        "libubus-lua",
        "libubus20191227",
        "libuci20130104",
        "libuclient20160123",
        "libustream-mbedtls20150806",
        "libxtables12",
        "logd",
        "lua",
        "luci",
        "luci-app-adblock",
        "luci-app-attendedsysupgrade",
        "luci-app-firewall",
        "luci-app-opkg",
        "luci-app-statistics",
        "luci-app-upnp",
        "luci-base",
        "luci-i18n-adblock-en",
        "luci-i18n-upnp-en",
        "luci-lib-ip",
        "luci-lib-jsonc",
        "luci-lib-nixio",
        "luci-mod-admin-full",
        "luci-mod-network",
        "luci-mod-status",
        "luci-mod-system",
        "luci-proto-ipv6",
        "luci-proto-ppp",
        "luci-ssl",
        "luci-theme-bootstrap",
        "miniupnpd",
        "mtd",
        "netifd",
        "odhcp6c",
        "odhcpd-ipv6only",
        "openwrt-keyring",
        "opkg",
        "ppp",
        "ppp-mod-pppoe",
        "procd",
        "rpcd",
        "rpcd-mod-file",
        "rpcd-mod-iwinfo",
        "rpcd-mod-luci",
        "rpcd-mod-rrdns",
        "swconfig",
        "tcpdump-mini",
        "uboot-envtools",
        "ubox",
        "ubus",
        "ubusd",
        "uci",
        "uclient-fetch",
        "uhttpd",
        "urandom-seed",
        "urngd",
        "usign",
        "wireless-regdb",
        "wpad-basic"
    ],
    ".anonymous": false,
    ".name": "request",
    ".type": "request",
    ".index": 0
}

Apparently in 19.07.08 libubus has been updated to libubus20210603

So if I enable advanced mode and manually drop libubus20191227 from the list, it runs but then runs into this error (that again I am reporting as instructed):
Error building the sysupgrade

Please report the error message and request
Request to server:
{
    "board_name": "tplink,archer-c7-v4",
    "target": "ath79/generic",
    "version": "19.07.8",
    "packages": [
        "ath10k-firmware-qca988x-ct",
        "base-files",
        "busybox",
        "cgi-io",
        "collectd-mod-conntrack",
        "dnsmasq",
        "dropbear",
        "firewall",
        "fstools",
        "fwtool",
        "getrandom",
        "hostapd-common",
        "ip6tables",
        "iptables",
        "iw",
        "iwinfo",
        "jshn",
        "jsonfilter",
        "kernel",
        "kmod-ath",
        "kmod-ath10k-ct",
        "kmod-ath9k",
        "kmod-ath9k-common",
        "kmod-cfg80211",
        "kmod-gpio-button-hotplug",
        "kmod-ip6tables",
        "kmod-ipt-conntrack",
        "kmod-ipt-core",
        "kmod-ipt-nat",
        "kmod-ipt-offload",
        "kmod-lib-crc-ccitt",
        "kmod-mac80211",
        "kmod-nf-conntrack",
        "kmod-nf-conntrack6",
        "kmod-nf-flow",
        "kmod-nf-ipt",
        "kmod-nf-ipt6",
        "kmod-nf-nat",
        "kmod-nf-reject",
        "kmod-nf-reject6",
        "kmod-ppp",
        "kmod-pppoe",
        "kmod-pppox",
        "kmod-slhc",
        "kmod-usb-core",
        "kmod-usb-ledtrig-usbport",
        "kmod-usb-storage",
        "kmod-usb2",
        "libblobmsg-json",
        "libip4tc2",
        "libip6tc2",
        "libiwinfo-lua",
        "libiwinfo20181126",
        "libjson-c2",
        "libjson-script",
        "liblua5.1.5",
        "liblucihttp-lua",
        "liblucihttp0",
        "libnl-tiny",
        "libubox20191228",
        "libubus-lua",
        "libuci20130104",
        "libuclient20160123",
        "libustream-mbedtls20150806",
        "libxtables12",
        "logd",
        "lua",
        "luci",
        "luci-app-adblock",
        "luci-app-attendedsysupgrade",
        "luci-app-firewall",
        "luci-app-opkg",
        "luci-app-statistics",
        "luci-app-upnp",
        "luci-base",
        "luci-i18n-adblock-en",
        "luci-i18n-upnp-en",
        "luci-lib-ip",
        "luci-lib-jsonc",
        "luci-lib-nixio",
        "luci-mod-admin-full",
        "luci-mod-network",
        "luci-mod-status",
        "luci-mod-system",
        "luci-proto-ipv6",
        "luci-proto-ppp",
        "luci-ssl",
        "luci-theme-bootstrap",
        "miniupnpd",
        "mtd",
        "netifd",
        "odhcp6c",
        "odhcpd-ipv6only",
        "openwrt-keyring",
        "opkg",
        "ppp",
        "ppp-mod-pppoe",
        "procd",
        "rpcd",
        "rpcd-mod-file",
        "rpcd-mod-iwinfo",
        "rpcd-mod-luci",
        "rpcd-mod-rrdns",
        "swconfig",
        "tcpdump-mini",
        "uboot-envtools",
        "ubox",
        "ubus",
        "ubusd",
        "uci",
        "uclient-fetch",
        "uhttpd",
        "urandom-seed",
        "urngd",
        "usign",
        "wireless-regdb",
        "wpad-basic"
    ],
    ".anonymous": false,
    ".name": "request",
    ".type": "request",
    ".index": 0
}
STDERR:
Downloading http://downloads.cdn.openwrt.org/releases/19.07.8/targets/ath79/generic/packages/Packages.gz
Updated list of available packages in /home/debian/asu/worker2/cache/19.07.8/ath79/generic/build_dir/target-mips_24kc_musl/root-ath79/../../../../../../../../../../../home/debian/asu/worker2/cache/19.07.8/ath79/generic/dl/openwrt_core
Downloading http://downloads.cdn.openwrt.org/releases/19.07.8/packages/mips_24kc/base/Packages.gz
Updated list of available packages in /home/debian/asu/worker2/cache/19.07.8/ath79/generic/build_dir/target-mips_24kc_musl/root-ath79/../../../../../../../../../../../home/debian/asu/worker2/cache/19.07.8/ath79/generic/dl/openwrt_base
Downloading http://downloads.cdn.openwrt.org/releases/19.07.8/packages/mips_24kc/freifunk/Packages.gz
Updated list of available packages in /home/debian/asu/worker2/cache/19.07.8/ath79/generic/build_dir/target-mips_24kc_musl/root-ath79/../../../../../../../../../../../home/debian/asu/worker2/cache/19.07.8/ath79/generic/dl/openwrt_freifunk
Downloading http://downloads.cdn.openwrt.org/releases/19.07.8/packages/mips_24kc/luci/Packages.gz
Updated list of available packages in /home/debian/asu/worker2/cache/19.07.8/ath79/generic/build_dir/target-mips_24kc_musl/root-ath79/../../../../../../../../../../../home/debian/asu/worker2/cache/19.07.8/ath79/generic/dl/openwrt_luci
Downloading http://downloads.cdn.openwrt.org/releases/19.07.8/packages/mips_24kc/packages/Packages.gz
Updated list of available packages in /home/debian/asu/worker2/cache/19.07.8/ath79/generic/build_dir/target-mips_24kc_musl/root-ath79/../../../../../../../../../../../home/debian/asu/worker2/cache/19.07.8/ath79/generic/dl/openwrt_packages
Downloading http://downloads.cdn.openwrt.org/releases/19.07.8/packages/mips_24kc/routing/Packages.gz
Updated list of available packages in /home/debian/asu/worker2/cache/19.07.8/ath79/generic/build_dir/target-mips_24kc_musl/root-ath79/../../../../../../../../../../../home/debian/asu/worker2/cache/19.07.8/ath79/generic/dl/openwrt_routing
Downloading http://downloads.cdn.openwrt.org/releases/19.07.8/packages/mips_24kc/telephony/Packages.gz
Updated list of available packages in /home/debian/asu/worker2/cache/19.07.8/ath79/generic/build_dir/target-mips_24kc_musl/root-ath79/../../../../../../../../../../../home/debian/asu/worker2/cache/19.07.8/ath79/generic/dl/openwrt_telephony
Downloading file:packages/Packages
Updated list of available packages in /home/debian/asu/worker2/cache/19.07.8/ath79/generic/build_dir/target-mips_24kc_musl/root-ath79/../../../../../../../../../../../home/debian/asu/worker2/cache/19.07.8/ath79/generic/dl/imagebuilder
Downloading https://feed.libremesh.org/master/Packages.gz
Updated list of available packages in /home/debian/asu/worker2/cache/19.07.8/ath79/generic/build_dir/target-mips_24kc_musl/root-ath79/../../../../../../../../../../../home/debian/asu/worker2/cache/19.07.8/ath79/generic/dl/lime-packages
Downloading https://feed.libremesh.org/profiles/Packages.gz
Updated list of available packages in /home/debian/asu/worker2/cache/19.07.8/ath79/generic/build_dir/target-mips_24kc_musl/root-ath79/../../../../../../../../../../../home/debian/asu/worker2/cache/19.07.8/ath79/generic/dl/lime-profiles
Collected errors:
 * check_data_file_clashes: Package iw wants to install file /home/debian/asu/worker2/cache/19.07.8/ath79/generic/build_dir/target-mips_24kc_musl/root-ath79/usr/sbin/iw
	But that file is already provided by package  * iw-full
 * opkg_install_cmd: Cannot install package iw.
make[2]: *** [Makefile:158: package_install] Error 255
make[1]: *** [Makefile:128: _call_manifest] Error 2
make: *** [Makefile:218: manifest] Error 2

Finally, if I remove both libubus20191227 and iw I get a successful build:

@aparcar

This should be fixed via https://github.com/aparcar/asu/commit/bfecd216f362766cde3dfaa137fb0043e0c8bb79
I'm not sure how to handle the iw issue, I think this is a deeper issue with the opkg package manager.

@jow- if you have any ideas on that matter please let me know.

Just checking in to report that, after upgrading to git-21.271.68289-79af897-1, the libubus problem is gone.
The iw issue is still present.

@CAFxX the iw issue is an upstream problem fixed via this opkg commit in later releases


Unable to add API key

I attempted to add True RNG tonight on 0.8.6.
The place to insert the API key does not appear in the module settings in the configuration spot.
I see that there's a compatibility warning for 0.8.5, so if this is a non issue at the moment I understand. I'm only posting as I don't think the configuration settings have changed in the last few versions as I have a couple of lower version modules working without any issue.

I found the issue! I had the dice anti-cheat module installed by mistake. Entirely my fault for not going more in depth on troubleshooting beforehand.


Add German Email Providers

Add Domains for more random data.

Are you sure these ones are representative? Please provide a datasource as a justification for this data change.

Thanks!