MM-45981_Desktop: Add Server Screen: Improve Onboarding screens

Summary
When the user has no servers configured in the desktop app, a friendlier ‘add server’ screen will be displayed.
Ticket Link
https://mattermost.atlassian.net/browse/MM-45981
UX Design
https://www.figma.com/file/7BM8S6BL597zBE83U6H8CK/MM-29159-Webapp-Login-Update?node-id=1023%3A92011
Checklist

[x] Added or updated unit tests (required for all new features)
[x] Has UI changes
[x] read and understood our Contributing Guidelines
[x] completed Mattermost Contributor Agreement
[x] executed npm run lint:js for proper code formatting

Device Information
This PR was tested on: macOS 11.6.8
Screenshots
Light mode

Dark mode

Mobile view (< 1200px)

Release Note
Adding Configure Server Screen when starting the app without servers created.

Building app in separate branch.

Successfully building:
https://circleci.com/gh/mattermost/desktop/22448
https://circleci.com/gh/mattermost/desktop/22447
https://circleci.com/gh/mattermost/desktop/22446

Successfully building:
https://circleci.com/gh/mattermost/desktop/22448
https://circleci.com/gh/mattermost/desktop/22447
https://circleci.com/gh/mattermost/desktop/22446

build-linux-pr:
tmp/artifacts/mattermost-desktop-5.2.0-develop.1-linux-x64.tar.gz
build-windows-pr:
tmp/artifacts/mattermost-desktop-5.2.0-develop.1-win-ia32.zip
tmp/artifacts/mattermost-desktop-5.2.0-develop.1-win-x64.zip
build-mac-pr:
tmp/artifacts/mattermost-desktop-5.2.0-develop.1-mac-m1.dmg
tmp/artifacts/mattermost-desktop-5.2.0-develop.1-mac-x64.dmg

Building app in separate branch.

Successfully building:
https://circleci.com/gh/mattermost/desktop/22457
https://circleci.com/gh/mattermost/desktop/22456
https://circleci.com/gh/mattermost/desktop/22455

Successfully building:
https://circleci.com/gh/mattermost/desktop/22457
https://circleci.com/gh/mattermost/desktop/22456
https://circleci.com/gh/mattermost/desktop/22455

build-linux-pr:
tmp/artifacts/mattermost-desktop-5.2.0-develop.1-linux-x64.tar.gz
build-windows-pr:
tmp/artifacts/mattermost-desktop-5.2.0-develop.1-win-ia32.zip
tmp/artifacts/mattermost-desktop-5.2.0-develop.1-win-x64.zip
build-mac-pr:
tmp/artifacts/mattermost-desktop-5.2.0-develop.1-mac-m1.dmg
tmp/artifacts/mattermost-desktop-5.2.0-develop.1-mac-x64.dmg

Building app in separate branch.

Successfully building:
https://circleci.com/gh/mattermost/desktop/22528
https://circleci.com/gh/mattermost/desktop/22527
https://circleci.com/gh/mattermost/desktop/22526

Successfully building:
https://circleci.com/gh/mattermost/desktop/22528
https://circleci.com/gh/mattermost/desktop/22527
https://circleci.com/gh/mattermost/desktop/22526

build-windows-pr:
tmp/artifacts/mattermost-desktop-5.2.0-develop.1-win-ia32.zip
tmp/artifacts/mattermost-desktop-5.2.0-develop.1-win-x64.zip
build-linux-pr:
tmp/artifacts/mattermost-desktop-5.2.0-develop.1-linux-x64.tar.gz
build-mac-pr:
tmp/artifacts/mattermost-desktop-5.2.0-develop.1-mac-m1.dmg
tmp/artifacts/mattermost-desktop-5.2.0-develop.1-mac-x64.dmg

2. Also for @matthewbirtch, we cannot have a link to our Cloud hosting page in our app on the Mac App Store. It violates Apple's restrictions on advertising purchases outside of the application, and they will reject us. We can leave it there if we really want for the standalone app, but it must be disabled for the MAS build.

@devinbinnie @julmondragon I guess we have to remove this then. I don't think we have another page to send people to on mattermost.com that is more informational about creating a cloud server.

Building app in separate branch.

Successfully building:
https://circleci.com/gh/mattermost/desktop/22660
https://circleci.com/gh/mattermost/desktop/22659
https://circleci.com/gh/mattermost/desktop/22658

Successfully building:
https://circleci.com/gh/mattermost/desktop/22660
https://circleci.com/gh/mattermost/desktop/22659
https://circleci.com/gh/mattermost/desktop/22658

build-linux-pr:
tmp/artifacts/mattermost-desktop-5.2.0-develop.1-linux-x64.tar.gz
build-windows-pr:
tmp/artifacts/mattermost-desktop-5.2.0-develop.1-win-ia32.zip
tmp/artifacts/mattermost-desktop-5.2.0-develop.1-win-x64.zip
build-mac-pr:
tmp/artifacts/mattermost-desktop-5.2.0-develop.1-mac-m1.dmg
tmp/artifacts/mattermost-desktop-5.2.0-develop.1-mac-x64.dmg

@matthewbirtch :

Removed the white flash between welcome and configure screens.
Updated the color for Connect button disabled
Removed the icons for Input's info messages. This was just replicating the design from webapp component.
The alternate link was causing the content to look a little bit pushed to the bottom, now that it was removed all is centered.

Building app in separate branch.

Successfully building:
https://circleci.com/gh/mattermost/desktop/22692
https://circleci.com/gh/mattermost/desktop/22691
https://circleci.com/gh/mattermost/desktop/22690

Successfully building:
https://circleci.com/gh/mattermost/desktop/22692
https://circleci.com/gh/mattermost/desktop/22691
https://circleci.com/gh/mattermost/desktop/22690

build-linux-pr:
tmp/artifacts/mattermost-desktop-5.2.0-develop.1-linux-x64.tar.gz
build-windows-pr:
tmp/artifacts/mattermost-desktop-5.2.0-develop.1-win-ia32.zip
tmp/artifacts/mattermost-desktop-5.2.0-develop.1-win-x64.zip
build-mac-pr:
tmp/artifacts/mattermost-desktop-5.2.0-develop.1-mac-m1.dmg
tmp/artifacts/mattermost-desktop-5.2.0-develop.1-mac-x64.dmg

@matthewbirtch @devinbinnie All remaining feedback is addressed now. Thank you!

There is no immediate validation on the server URL box to check if the URL is even valid to begin with, we can do that using urlUtils.isValidUrl. Check the NewServerModal.tsx code to see how we do it there.

@julmondragon This bit is still missing, you can still type in whatever you want and hit Connect before validation occurs.

I also noticed that if you remove all your servers, even after you've seen the Welcome screen you'll see it a second time.
We should only be showing the Welcome screen once, but we can show the new modal.

There is no immediate validation on the server URL box to check if the URL is even valid to begin with, we can do that using urlUtils.isValidUrl. Check the NewServerModal.tsx code to see how we do it there.

@julmondragon This bit is still missing, you can still type in whatever you want and hit Connect before validation occurs.

This follows the pattern used in the webapp login/signup pages, where fields' validations occurs when executing the action. The only live validation is for empty fields to activate the action button.
cc @matthewbirtch

I also noticed that if you remove all your servers, even after you've seen the Welcome screen you'll see it a second time. We should only be showing the Welcome screen once, but we can show the new modal.

This was a requirement from UX implemented as part of the first stage of the onboarding work.
https://github.com/mattermost/desktop/pull/2220#issuecomment-1209475069
cc @matthewbirtch

I also noticed that if you remove all your servers, even after you've seen the Welcome screen you'll see it a second time. We should only be showing the Welcome screen once, but we can show the new modal.

This was a requirement from UX implemented as part of the first stage of the onboarding work. #2220 (comment)
cc @matthewbirtch

Yep, this is as designed and intended

@matthewbirtch I was under the impression that we should only show the Welcome modal once, since it's just an onboarding screen and if a user removes their server they don't really need to see it again. I think this is something we should consider since it doesn't seem like a good experience to show it multiple times, to me at least.

We discussed it as a team during the walkthrough and agreed to bring back the welcome screen when you've removed all your servers because it acts as a good 'reset'.
I don't see the vast majority of users removing all of their servers multiple times - if ever. If they do happen to do this, I don't see a big problem bringing them back to the welcome screen to orient them again with the product.
Is this a blocker for you @devinbinnie?

@matthewbirtch I was under the impression that we should only show the Welcome modal once, since it's just an onboarding screen and if a user removes their server they don't really need to see it again. I think this is something we should consider since it doesn't seem like a good experience to show it multiple times, to me at least.

We discussed it as a team during the walkthrough and agreed to bring back the welcome screen when you've removed all your servers because it acts as a good 'reset'.
I don't see the vast majority of users removing all of their servers multiple times - if ever. If they do happen to do this, I don't see a big problem bringing them back to the welcome screen to orient them again with the product.
Is this a blocker for you @devinbinnie?

I definitely see the value in coming back to the new Add Server modal added in this PR, since it's very welcoming and it shows the user that they will need to put in another server in order to use the app. Showing the Welcome screen a second time just adds extra clicks and shows them information that they've already seen.
It's not a blocker for me, but I think we should revisit this in a follow-up ticket if not right now. 2/5 on doing this.

I definitely see the value in coming back to the new Add Server modal added in this PR, since it's very welcoming and it shows the user that they will need to put in another server in order to use the app. Showing the Welcome screen a second time just adds extra clicks and shows them information that they've already seen.
It's not a blocker for me, but I think we should revisit this in a follow-up ticket if not right now. 2/5 on doing this.

Appreciate your thoughts here @devinbinnie. Quite honestly, I don't feel that strongly either, but as mentioned I do think that the welcome screens are a good reset and an opportunity to show some visual polish and interest in an empty state. It's also an opportunity to re-introduce users to the platform. I don't see that as a bad thing. If this was a state that I thought people would find themselves in very often, I might think differently, but I don't think that will be the case.
Again, I don't feel that strongly, so I'm open to change, but I think it's ready to merge as is and if we get feedback from users regularly being annoyed by this, we can address it. I'm perfectly fine with being proven wrong if users provide this feedback!

As for the URL validation, we can do that in a follow-up ticket as well if need be. Let me know how you'd like to proceed.

I agree that the live URL validation is a nice-to-have and could be looked at later given that we don't seem to have it in the current app anyway so there isn't a regression introduced here.


feat: support set storage class for clusters

Support set storage class in kblib.componentStorages
Adapt the value schema for all the addons if they use this template

Fix: #457

/cherry-pick release-0.9

/cherry-pick release-0.8


ldap ext with external oidc provider

hi, i´m trying to setup Azure AD as external provider in a deployment i have with LdapExtension, but the callback allways ends up calling the Search method and obviusly the user(Azure AD) does not exist in any ldap and the login fail, can we set up the ldap extension and also external oidc ??

I second this, the LdapExtension interferes with other external provider logins. Any workaround? Solution?

No, sorry. I don't have any workaround. I haven't played much with this project for the last year. It was at its beginning a PoC to prove that it was not so hard to add any LDAP to Identity Server. Since it was not documented anywhere, I did it so that it can help others.
However, if you wish to fix this issue, please do a PR.


Extend Slide CMS functionalities to Static Pages CMS

This should allow to speed up the production of the static pages

add research page and other stuff that's missing


Usage of new query field in NodeTo{Client,Node}VersionData

Hello!
This commit introduces a new flag query on the NodeTo{Client,Node}VersionData object.
What's its semantic? When should it be set to True (or False).
Thanks.

if query is enabled, when doing handshake the nodes will reply with a list of supported versions. You should set it to either True or False depending if you want that. Porbably you want it set to False.
It is useful for cardano-ping to have a --query flag


Pre-transform error: Failed to resolve import; when running project

Get Error Pre-transform error: Failed to resolve import when running example project in https://github.com/primefaces/primevue-examples/tree/main/nuxt-unstyled-tailwind
Reproduce Step:
pnpm i
pnpm run dev

same error, working in Windows + VSCode.

I got the same one and it's probably Windows related.
You can fix it by updating the importPT to:
importPT: { as: 'Lara', from: '/presets/lara' }

I was found an dirty solution:
const presetpath = path.resolve(__dirname, "presets", "lara");
const presetpathDoubleSlashes = presetpath.replace(/\/g, "//");
The error is only valid in Windows environment develop

Know this is closed, but the true fix is to let path.resolve do its thing:  pass it a string array.
     importPT: { from: path.resolve(__dirname, 'presets', 'lara') }
It will figure out to use '/'  or '' based on the OS - Window vs Linux.


How to insert a blank line before the key-values?

I'm using a slightly modified version of the 'Helium' theme, which is very similar to the default with no theme file. Therefore I have the ascii logo on the left and the key-values on the right. I just want to shift the lines on the left down by one line but I cannot figure out how to do that.

I'm not sure I understand the question...

Do you want to add an extra newline between the prompt and macchina's output?

I think padding is what you want here.

Do you want just the ASCII art to be one line lower?

This is not possible, AFAIK.

@grtcdr Thanks for replying. I don't want an extra line before all output or on the ascii art, but I want an extra line on the text to the right of it. I can't make the ascii logo any higher, so I want to lower the text, as it doesn't look well balanced as it is. It's not important, but I get a bit OCD about stuff like this.

So, you want every readout to be separated by a newline?
Something like this?
Kernel  ~  ...

Distribution ~ ...

00000     Host     -  foo@bar
000000000   Machine  -
000000000   Kernel   -
000000000   Local IP -
00000
00000
000000000   Host     -  foo@bar
000000000   Machine  -
000000000   Kernel   -
00000     Local IP -

This is not possible at the moment, as the padding is applied across the whole buffer and not at the component level. I'll turn this into a feature request, but I unfortunately won't be able to work on it right away.

Okay thanks.

No need to close it, this hurts visibility; perhaps a visitor or previous contributor may be able to work on it.
I have a question though considering the UI aspect of this request.
What is it exactly that you dislike about the current placement?
Both your proposal and the current placement are asymmetrical to the ASCII art, which leads me to believe that the only way to resolve this issue, is to offer three different alignment options for the readout component: "top", "bottom", and "center".

It's not important. I just wanted to lower the text by one line, but if I cannot it doesn't matter. I have been using linux_logo to remind me what I have logged into for 13 years or so and I could easily adapt it but it has no Android logo. I think it will be easier to add an Android ascii logo to it.

I was just thinking of a more long-term and adaptable solution.

I think it will be easier to add an Android ascii logo to it.

Sure, you could use ours if you want.

Closing due to inactivity.


Fix header append

Addresses some of the inconsistencies pointed out by https://github.com/http-rs/tide/pull/520. Thanks!

Am I right that it's pretty much a matter of string encoding and should be very unlikely for anyone to run into?

Yes, accurate. That's the only case. We'll never panic however if a valid HeaderValue / HeaderName has been provided already, which provides a way to construct values with the errors handled.


@Transform reference to "this"

Hello,
Sometimes, when de/serializing properties of an object, we need to apply specific rules. For example, let's say I have the following class:
class Test {
  a: string;
  b: boolean;
}

When using classToPlain(), I want a to be serialized only if b is true.
Possible solutions

Give the @Transform callback the parent object as second argument: @Transform((val, self) => self.b === true ? val : undefined)
Create a @SerializeIf and @DeserializeIf that can be used as follows: @SerializeIf((val, self) => self.b === true)

I like the first option as it does not add a new decorator.
All the best;

@pleerock Could you review this PR, and if okay with you, release a new minor version ?


Add builders to elements.

Next step: move validation to happen on every addFoo method call for more local problem detection.

Sorry this is long... mostly mechanical. Very JavaPoet-esque API now. This will make changes in the future easier (e.g., adding oneof would have been really nice after this change).

Huge win. More legible, more predictable, and you don't need to pass in nulls and empty strings when you don't care.
LGTM.


ARROW-6406: [C++] Fix jemalloc URL for offline build in thirdparty/versions.txt

The archive URL is not the released tarball and did not work with jemalloc_ep

Codecov Report

Merging #5246 into master will increase coverage by 1.55%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #5246      +/-   ##
==========================================
+ Coverage   87.63%   89.18%   +1.55%
==========================================
  Files        1033      750     -283
  Lines      148630   107752   -40878
  Branches     1437        0    -1437
==========================================
- Hits       130249    96103   -34146
+ Misses      18019    11649    -6370
+ Partials      362        0     -362

Impacted Files
Coverage Δ

cpp/src/arrow/filesystem/s3_internal.h
95.12% <0%> (-4.88%)
:arrow_down:

cpp/src/plasma/thirdparty/ae/ae.c
70.75% <0%> (-0.95%)
:arrow_down:

js/src/util/fn.ts

go/arrow/memory/memory_avx2_amd64.go

rust/datafusion/src/execution/filter.rs

rust/arrow/src/csv/writer.rs

rust/datafusion/src/bin/main.rs

go/arrow/ipc/file_reader.go

js/src/builder/index.ts

rust/parquet/src/arrow/converter.rs

... and 276 more

Continue to review full report at Codecov.

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

Codecov Report

Merging #5246 into master will increase coverage by 1.55%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #5246      +/-   ##
==========================================
+ Coverage   87.63%   89.18%   +1.55%
==========================================
  Files        1033      750     -283
  Lines      148630   107752   -40878
  Branches     1437        0    -1437
==========================================
- Hits       130249    96103   -34146
+ Misses      18019    11649    -6370
+ Partials      362        0     -362

Impacted Files
Coverage Δ

cpp/src/arrow/filesystem/s3_internal.h
95.12% <0%> (-4.88%)
:arrow_down:

cpp/src/plasma/thirdparty/ae/ae.c
70.75% <0%> (-0.95%)
:arrow_down:

js/src/util/fn.ts

go/arrow/memory/memory_avx2_amd64.go

rust/datafusion/src/execution/filter.rs

rust/arrow/src/csv/writer.rs

rust/datafusion/src/bin/main.rs

go/arrow/ipc/file_reader.go

js/src/builder/index.ts

rust/parquet/src/arrow/converter.rs

... and 276 more

Continue to review full report at Codecov.

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


Windows

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

Project coverage is 91.77%. Comparing base (06114f8) to head (a1f4e37).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##           master      #40   +/-   ##
=======================================
  Coverage   91.77%   91.77%
=======================================
  Files          35       35
  Lines        1179     1179
  Branches      182      182
=======================================
  Hits         1082     1082
  Misses         97       97

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

It seems the output name of the python binding conflicts with the helper library on windows. To avoid this, I disabled building the helper library and directly added registration_helper.cpp to small_gicp_python.


style(frontend): remove focus state if button is clicked

Motivation
Opinionated PR: when a button is clicked, it should not show the focus state colors. The focus state is visible only when the button is selection interaction.
Before
https://github.com/user-attachments/assets/c08c79a8-ee4e-4ea7-bf88-2e3cbbcb2d97
After
https://github.com/user-attachments/assets/290770dd-ace8-4b25-9886-f1f6431ad212

Correct! The last part of the video is when i use the keyboard (I know, it is not much obvious eheh)


using livereload when starting a web app with Node and Express

I tried to implement what was in the article on using npm as a build tool but when starting a web app with node and express.
I posted my question on S.O. here but will paste it below in case someone can resolve my issues:
Following this great article on how to use npm as a build tool, I would like to implement it when building a nodejs express web app.
My node app is created on port 8080, this is a simplified version of my server.js file:
var env = process.env.NODE_ENV

var path = require('path');
var express = require('express');
var logger = require('morgan');

var routes = require('./routes');

var app = express();

app.set('port', process.env.PORT || 8080);
app.set('views', __dirname + '/views');
app.set('view engine', 'jade');
app.engine('jade', require('jade').__express)

var Oneday = 86400000;
app.use(express.static(__dirname + '/www', {
	maxAge: env == 'development' ? 0 : Oneday
}));

app.use(logger('dev'));

app.use(express.static(path.join(__dirname, '/public'), {
	maxAge: env == 'development' ? 0 : Oneday
}))

if (env == 'development') {
	// var liveReloadPort = 9091;
	app.use(require('connect-livereload')({
		port: 8080
		// src: "js/"
			// port: liveReloadPort
	}));
}

routes.blog(app);
routes.frontend(app, passport);

app.use(function(err, req, res, next) {
	console.log(err.stack);
	res.status(500).send({
		message: err.message
	})
});

app.listen(app.get('port'));
console.log('Server starting on port: ' + app.get('port'));

The file that I'm watching before needing to reload is in www/js.
I am using npm as a build tool and before launching server.js with npm I launch a separate process that does watchify source/js/app.js -v -o wwww/js/bundle.js
I did checked that watchify works correctly, updating as I save my files. But there is no livereload once a file is changed.
The error I get in the console is:
Uncaught SyntaxError: Unexpected token <
and I can see that connect-livereload inserted this script in the html:
<script>
//<![CDATA[
document.write('<script src="//' + (location.hostname || 'localhost') + ':8080/livereload.js?snipver=1"><\/script>')
//]]>
</script>
<script src="//localhost:8080/livereload.js?snipver=1"> </script>

I tried also to use live-reload as mentionned in the original article but without success and I am not sure it's the right plugin to use as live-reload launches a server, when I already start one with express.
Any ideas?

This has been resolved in S.O.
Keeping it open for those it might help.


Create /var/run/pushpin directory

I installed pushpin using apt-get, and got this when I tried to run it:
Traceback (most recent call last):
  File "/usr/bin/pushpin", line 56, in <module>
    runner.run(exedir, config_file, verbose)
  File "/usr/lib/pushpin/runner/runner.py", line 58, in run
    m2sqlpath = services.write_mongrel2_config(configdir, os.path.join(configdir, "mongrel2.conf.template"), rundir, logdir, http_port, https_ports, m2sh_bin)
  File "/usr/lib/pushpin/runner/services.py", line 34, in write_mongrel2_config
    compile_template(configpath, genconfigpath, vars)
  File "/usr/lib/pushpin/runner/services.py", line 11, in compile_template
    f = open(outfilename, "w")
IOError: [Errno 2] No such file or directory: '/var/run/pushpin/mongrel2.conf'

Workaround: Create /var/run/pushpin manually.

On a normal Debian (and probably Ubuntu, as well - please correct me if I'm wrong), /var/run is a ramdisk (tmpfs). Therefore, one can't simply create /var/run/pushpin at installation time: One needs to recreate it after every reboot. This is solved by the init script by testing if /var/run/pushpin exists, and creating it if necessary.
I guess you are not using the init script. That's fine, but in that case, you need to create /var/run/pushpin yourself, or change the configuration to point to another, existing directory.

Pushpin now attempts to create the needed directories on startup. So, if you run it as root after installation, it should be able to make the directories. Also it now defaults to /var/run instead of /var/local/run.
Of course creating the dirs in the init script has the advantage that pushpin can then be run as non-root.

Thanks!
To clarify, I was trying to run it in a docker container.


KAFKA-3514: Part III, Refactor StreamThread main loop

Refactor the StreamThread main loop, in the following:

Fetch from consumer and enqueue data to tasks.
Check if any tasks should be enforced process.
Loop over processable tasks and process them for N iterations, and then check for 1) commit, 2) punctuate, 3) need to call consumer.poll
Even if there is not data to process in this iteration, still need to check if commit / punctuate is needed
Finally, try update standby tasks.

Add an optimization to only commit when it is needed (i.e. at least some process() or punctuate() was triggered since last commit).

Found and fixed a ProducerFencedException scenario: producer.send() call would never throw a ProducerFencedException directly, but it may throw a KafkaException whose "cause" is a ProducerFencedException.

Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

retest this please

retest this please

@vvcephei @mjsax @bbejeck for reviews.
Note the logic of processing enforcement is open for discussion, and currently I'm thinking to change it to time based than iteration based. I will include a KIP for this change for open discussions.

System test passed at https://jenkins.confluent.io/job/system-test-kafka-branch-builder/1898/parameters/
Need to wait for KIP-345 to be discussed and voted.

retest this please

retest this please

I've found out the actual perf difference is that for joins, we are using leftJoin actually. And we are populating the data before starting the run the tests, hence it is actually a "catching up" mode where the first fetch request may just return one partition of the data. With no synchronization (ms = 0), it means that there are less records being joined and output to sink topics.
I've changing leftJoin to join and the results are following:
trunk:
http://confluent-kafka-branch-builder-system-test-results.s3-us-west-2.amazonaws.com/2018-08-15--001.1534370019--guozhangwang--trunk-join--46ac266/report.html
max.idle = 0ms:
http://confluent-kafka-branch-builder-system-test-results.s3-us-west-2.amazonaws.com/2018-08-15--001.1534370062--guozhangwang--K3514-main-thread-iteration-join--551f2ae/report.html
max.idle = 500ms:
http://confluent-kafka-branch-builder-system-test-results.s3-us-west-2.amazonaws.com/2018-08-15--001.1534371559--guozhangwang--K3514-main-thread-iteration-500ms--f6860ce/report.html
max.idle = 5000ms:
http://confluent-kafka-branch-builder-system-test-results.s3-us-west-2.amazonaws.com/2018-08-15--001.1534369924--guozhangwang--K3514-main-thread-iteration-5000ms--fb51d7a/report.html
Some observations:

From the stream-stream join results we can see that the old manner (max.waits = 5 iterations) is actually very close to max.idle = 0ms (I guess it was maybe a few millis idle time).

From the table-table join results, the new code actually processes more data than the old code.

@guozhangwang Can you rebase?

The benchmark results looks good to me:
http://confluent-kafka-branch-builder-system-test-results.s3-us-west-2.amazonaws.com/2018-08-21--001.1534818582--guozhangwang--K3514-main-thread-iteration--e454e81/report.html
http://confluent-kafka-system-test-results.s3-us-west-2.amazonaws.com/2018-08-21--001.1534820334--apache--trunk--504824b/report.html

But with regard to my comments, I am wondering if we should add more tests to check the behavior we want in a tighter fashion?

The tests I added in unit suite checks that 1) idle time is respected, and timer is reset correctly once enforce has happened, 2) optimization on skipping commit indeed happens.
I think I can add more on 3) punctuate / commit happens as expected, 4) break loop and calls poll as expected. Anything else comes to your mind? @mjsax

retest this please

The latest perf numbers can be found here:
http://confluent-kafka-branch-builder-system-test-results.s3-us-west-2.amazonaws.com/2018-08-24--001.1535136054--guozhangwang--K3514-main-thread-iteration--d28ee44/report.html

@vvcephei @mjsax comments replied / addressed, please take another look.

LGTM! Thanks, @guozhangwang

failures unrelated
retest this please

Could you point out which part of the doc indicates that ProcessorContext.commit() is more of a suggestion? Maybe we do need to update it.

@guozhangwang I can't seem to find it, I must be mistaken so ignore my previous comment.

retest this please

Test failures seem to be related.
java.lang.AssertionError:
Expected: <2>
     but: was <1>
	at org.hamcrest.MatcherAssert.assertThat(MatcherAssert.java:20)
	at org.junit.Assert.assertThat(Assert.java:956)
	at org.junit.Assert.assertThat(Assert.java:923)
	at org.apache.kafka.streams.processor.internals.StreamThreadTest.shouldRespectNumIterationsInMainLoop(StreamThreadTest.java:360)

@mjsax could you give it another look?

Nice! Super happy we got this finally in! Great work @guozhangwang!


Does it meet my requirements?

Hi!
I have an app developed in meteor where I store some data related to customers through MongoDB.
I have a website running separately where users can create new data related to customers in a MySQL database.
My goal is to read that MySQL database and load it in the MeteorJS app DB.
Is this package suitable for me?

I would think it is suitable.


fix technologies download

Hi Ross,
If you like you can have a look at my take at this - I tested it briefly and it seems to work again. Might need some more QA.
Thanks,
Andreas

Thanks, I will probably take at least parts of this with some changes.  I'll put a credit to you in the readme and source if you like.  Do you have a preference on how I list your name?

Thanks Ross - no need for the credit, just a few lines of code :)
You did the hard work before...

On Sep 10, 2021, at 1:24 PM, Ross Geerlings @.***> wrote:
Thanks, I will probably take at least parts of this with some changes. I'll put a credit to you in the readme and source if you like. Do you have a preference on how I list your name?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/RossGeerlings/webstor/pull/3#issuecomment-917189313, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABHFZK5OV7KCYF57OY4W5YTUBJSO7ANCNFSM5DZ6IGCQ.
Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Hi Ross,
I added another commit, with several smaller changes. One of them saves the technologies directly as json into the MySQL DB, so you can drop all the complex single quote replacements. With this change I can run a search against all technologies without getting an error. Feel free to use.
Thanks - Andreas


Connect to SASL_PLAINTEXT SCRAM-SHA-256 Cluster (hashtag1094)

How I can connect to SASL_PLAINTEXT SCRAM-SHA-256 cluster without SSL ?

it can be configured in clusterapi configuration. The below property already exists in the file, which should be updated.
kafkasasl.saslmechanism.plain=SCRAM-SHA-256
Currently it cannot be done through UI, but try configuring the above. that should work.
We would add a ticket to our backlog to get it on UI as well.

Thanks it work.


Binary_crossentropy producing negative loss values

Hi all,
I have a built a classifier to learn various features of data. I am now using the functional API as i would also like to create a predictor of a class in my data at the same time. so as oppose to having 30 classes to classify (as in the normal sequential model) i would also like to use the same input to see if it can predict one class in the data. The code is as follows:

le = LabelEncoder()
metadataencode= metadata.apply(le.fit_transform)
metadataencode

##one-hot encode data
cat1=to_categorical(metadataencode['cat1'],num_classes=12)
cat2=to_categorical(metadataencode['cat2'],num_classes=7)
cat3= to_categorical(metadataencode['cat3'], num_classes=2)
cat4=to_categorical(metadataencode['cat4'], num_classes=5)
cat5=to_categorical(metadataencode['cat5'], num_classes=2)
cat6=to_categorical(metadataencode['cat6'],num_classes=3)
cat7=to_categorical(metadataencode['cat7'],num_classes=6)
cat8=to_categorical(metadataencode['cat8'],num_classes=2)

##remove unknown class categories and make them '0'--

cat5=np.delete(cat5, 4 ,axis=1)
cat6=np.delete(cat6, 0 ,axis=1)
cat7=np.delete(cat7, 0 ,axis=1)

#trainingtarget for classification
trainingtarget = np.concatenate((cat1,cat2,cat3,cat4,cat5,cat6,cat7, cat8),
axis=1)
trainingtarget ##multi-class multi label array for classification

####create encoding for classifying ONE class based on all features in the data

encoder = LabelEncoder()
encoder.fit(metadata['cat5'])
encoded_Y = encoder.transform(metadata['cat5'])## there are 5 classes to chose from

Then build and compile model. one input with two outputs:: one for the classification, and one to predict based on the data if it can learn which class it belongs based on all the data (output for this is 1)

X_train, X_test, y_train, y_test, cat5train, cat5test = train_test_split(normCountsscale1,
trainingtarget, encoded_Y, train_size = 0.8)

##input

mainInput= tf.keras.Input([20501,], name= 'main_input')
mainInputlayer= tf.keras.layers.Dense(units=400, activation='relu', name='main_inputL1')(mainInput)
mainInputlayer= tf.keras.layers.Dense(units=250, activation= 'relu', name='main_inputL2')(mainInputlayer)
mainInputlayer= tf.keras.layers.Dense(units=500, activation= 'relu', name='main_inputL3')(mainInputlayer)

## branching in to different multi-task output --classification and regression

outputClass=tf.keras.layers.Dense(units=36, activation='sigmoid', name='outputClass')(mainInputlayer)
outputsingle=tf.keras.layers.Dense(units=1, activation='sigmoid', name='outputsingle')(mainInputlayer)

finalModel=tf.keras.Model(inputs=mainInput, outputs= [outputClass, outputsingle])

## model summary
finalModel.summary()

##plot model
tf.keras.utils.plot_model(finalModel, show_shapes=True)

#endregion

#region complile and fit model

finalModel.compile(optimizer='adam',
              loss='binary_crossentropy', metrics=['accuracy'])

finalModel.fit({'main_input': X_train},
              {'outputsingle': cat5train, 'outputClass': y_train},
              epochs=20, batch_size=32,
              verbose=1,
              validation_split=0.2,

i achieve good accuracy for both outputs, however, i am reaching negative loss values for the outputsingle class.. an example of the last line of epoch 20 shows:
outputClass_loss: 0.2093 - outputRegress_loss: -9.1634 - outputClass_accuracy: 0.9094 - outputRegress_accuracy: 0.659318/318 [==============================] - 1s 2ms/sample - loss: -9.2815 - outputClass_loss: 0.2075 - outputRegress_loss: -9.5086 - outputClass_accuracy: 0.9102 - outputRegress_accuracy: 0.6667 - val_loss: 9.5608 - val_outputClass_loss: 0.3463 - val_outputRegress_loss: 10.6915 - val_outputClass_accuracy: 0.8740 - val_outputRegress_accuracy: 0.6125
the loss values are all very odd. any help here would be appreciated! thanks!

@amjass12 Is this still an issue for you? Are you trying to use same loss function for both the outputs?
Please check functional API guide on keras website. Thanks!

Hi @jvishnuvardhan ,
apologies - yes ths is all fine now!


View registration info

Short bio's are requested when people register- can these be displayed as part of the person node search results? The rationale for asking for this was to allow people to make connections within the community, but I don't believe they are currently viewable anywhere.

This is in paleobot/pbot-client@1758832d16ade7300e649c76fe118520fea47ae1


add memory filter lable select

What type of PR is this?
/kind feature

What this PR does / why we need it:
Which issue(s) this PR fixes:
Fixes # https://github.com/clusterpedia-io/clusterpedia/issues/380
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

add memory filter for lables select.

/cc @xyz2277

@lengrongfu, thanks for your contribution. It's ok for list, can you add this feature to the watch too?

Can I submit a new pr to add watch ？

ok, don't forget it~

/lgtm

@lengrongfu Thanks for your contribution


Support for response and schema examples

Hi, I got the same question describing in #2003
And I tried the @Examples annotation. It did not work.
Could you please describe more for how to use this annotation?
Thanks!

@chaojungc which @Examples annotation exactly did you use? I'm having trouble finding it.


Importer ignores redstone control

Issue description:
Importer ignores redstone control
What happens:
No matter what setting or redstone combination  the importer will import.
What you expected to happen:
Stop importing when redstone control statement is true.
Steps to reproduce:
1.Setup importer
2.Configure to work with redstone signal or work without
3.Do not apply signal or apply signal and it will still import
...
Version (Make sure you are on the latest version before reporting):

Minecraft: 1.10.2
Forge: <IP_ADDRESS>5
Refined Storage: refinedstorage-1.2.17

Does this issue occur on a server? [yes/no]
Not on a server
If a (crash)log is relevant for this issue, link it here:
[pastebin/gist/etc link here]

I'm a derp.  Found another importer on the same marked enderchest right after I posted that was hidden in a wall.


Generators

I'm curious as to whether it's possible, and what it would take to support ES6 (async) generators.
I imagine we could support them through the stream APIs

This would depend on how you expect the generators to work. Currently, we expect a single return value at the end of execution. If we wanted to actually use generators to the full extent this will need to have its own API and handle things a lot differently and there's a wide range of scenarios to consider as well.


host_required not working with username:password@

Initial Checks

[X] I confirm that I'm using Pydantic V2

Description
If using UrlConstraints with host_required=False it fails validation if username and password is provided as part of the DSN. This is problematic if wanting to use an S3 DSN, for example, without having to specify host.
This is the error that the example code results in:
pydantic_core._pydantic_core.ValidationError: 1 validation error for Model
v
  Input should be a valid URL, empty host [type=url_parsing, input_value='foo://foo:bar@/foo/bar', input_type=str]
    For further information visit https://errors.pydantic.dev/2.3/v/url_parsing

I reproduced the same error with a test in this commit: https://github.com/bolmstedt/pydantic/commit/6a16f92823fd41ab467a0368da25ccbae0446c72
Example Code
from typing import Annotated

import pydantic

class Model(pydantic.BaseModel):
    v: Annotated[pydantic.AnyUrl, pydantic.UrlConstraints(host_required=False)]

Model(v='foo://foo:bar@/foo/bar')

Python, Pydantic & OS Version
pydantic version: 2.3.0
        pydantic-core version: 2.6.3
          pydantic-core build: profile=release pgo=false
                 install path: /Users/bolmis/.cache/pants/named_caches/pex_root/installed_wheels/45b5e446c6dfaad9444819a293b921a40e1db1aa61ea08aede0522529ce90e81/pydantic-2.3.0-py3-none-any.whl/pydantic
               python version: 3.11.4 (main, Aug 17 2023, 11:44:21) [Clang 14.0.3 (clang-14<IP_ADDRESS>.1)]
                     platform: macOS-13.5-arm64-arm-64bit
     optional deps. installed: ['email-validator', 'typing-extensions']

Selected Assignee: @davidhewitt

It looks like this is happening in pydantic-core, and url::Url::parse("foo://foo:bar@/foo/bar") is returning an EmptyHost error.
I see that it correctly handles the obvious tweaks, and it looks like it should be able to represent this URL as well:
println!("{:?}", Url::parse("foo:///foo/bar"));
// Ok(Url { scheme: "foo", cannot_be_a_base: false, username: "", password: None, host: None, port: None, path: "/foo/bar", query: None, fragment: None })
println!("{:?}", Url::parse("foo://foo:bar@a/foo/bar"));
// Ok(Url { scheme: "foo", cannot_be_a_base: false, username: "foo", password: Some("bar"), host: Some(Domain("a")), port: None, path: "/foo/bar", query: None, fragment: None })
println!("{:?}", Url::parse("foo://foo:bar@/foo/bar"));
// Err(EmptyHost)

Ideally you'd just get
Ok(Url { scheme: "foo", cannot_be_a_base: false, username: "foo", password: Some("bar"), host: None, port: None, path: "/foo/bar", query: None, fragment: None })

However, I did a bit of digging, and the rust URL crate is based on https://url.spec.whatwg.org/. Under section 4.2, it explicitly says:

A URL cannot have a username/password/port if its host is null or the empty string, or its scheme is "file".

I'm not sure what we should do about this, but given this I think it will not be easy to change this behavior. @davidhewitt @samuelcolvin any ideas?

If the specification says that host is required for a username/password to be provided it's hard to argue that this is a bug.
I've also just now realized that the S3 DSN we are using is a format made up by us, so it's not official. I just assumed that it was somewhat valid because urlparse from urllib.parse works with it.
I think we just to create our own custom type for this, unless you feel this edge case should also work. Maybe add a test that asserts that this case fails if it is deemed to be the expected behavior.

One related thing that is interesting is that the same example fails even if I add default_host to the UrlConstraints.


Remove New Relic client side settings

Given that Chrome blocks these requests now given new browser security settings we might as well remove it.

Since we are not really using it server side, should we remove it as well?

Since we are not really using it server side, should we remove it as well?

Just to reference what we talked about during out Dev standup and close the loop in this conversation.  Keeping New Relic for the Server Side metrics could be beneficial in the long run when we want to start refactoring after all major features are done.

I think it's worth a note in the README that we're using server-side New Relic but nothing client-side.
https://github.com/cds-snc/covid-alert-portal/blob/72a6484d95d2854433fc78017425bb74b612d1ba/README.md#new-relic-configuration


[Actor inference] Extend global actor inference

Extend global actor inference in a few ways:

All value members of a type that has a global actor will infer a global actor, including initializers and static variables
A type can infer a global actor from conformance to a protocol that is marked with a global actor

Addresses rdar://75992299

@swift-ci please smoke test

@swift-ci please smoke test Linux


[AutoPR] mediaservices/resource-manager

Created to accumulate context: mediaservices/resource-manager

(message created by the CI based on PR content)

Codecov Report

Merging #3872 into master will decrease coverage by <.01%.
The diff coverage is 50%.

@@            Coverage Diff             @@
##           master    #3872      +/-   ##
==========================================
- Coverage   54.16%   54.16%   -0.01%
==========================================
  Files        9830     9826       -4
  Lines      207475   207466       -9
==========================================
- Hits       112373   112368       -5
+ Misses      95102    95098       -4

Impacted Files
Coverage Δ

...gmt-media/azure/mgmt/media/models/job_error_py3.py
45.45% <ø> (ø)
:arrow_up:

...re-mgmt-media/azure/mgmt/media/models/aac_audio.py
62.5% <ø> (ø)
:arrow_up:

...odels/content_key_policy_play_ready_license_py3.py
31.25% <ø> (ø)
:arrow_up:

...mt/media/models/filter_track_property_condition.py
55.55% <ø> (ø)
:arrow_up:

...re-mgmt-media/azure/mgmt/media/models/image_py3.py
54.54% <ø> (ø)
:arrow_up:

...t-media/azure/mgmt/media/models/deinterlace_py3.py
57.14% <ø> (ø)
:arrow_up:

...e-mgmt-media/azure/mgmt/media/models/h264_video.py
50% <ø> (ø)
:arrow_up:

...gmt-media/azure/mgmt/media/models/png_image_py3.py
62.5% <ø> (ø)
:arrow_up:

...re-mgmt-media/azure/mgmt/media/models/jpg_image.py
62.5% <ø> (ø)
:arrow_up:

...t-media/azure/mgmt/media/models/storage_account.py
62.5% <ø> (ø)
:arrow_up:

... and 64 more

Continue to review full report at Codecov.

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


refactor(starknet_sequencer_node): refactor component config

Stack:

#2542
#2541
#2540
#2539 ⬅
#2520

⚠ Part of a stack created by spr. Do not merge manually using the UI - doing so may have unexpected results.

This change is


Separate scripts for video demos

He mate,
i love the script.
but how i add to a Html, Youtube, Vimeo video on a button click????
i trying and final result showing repeated - Error: The requested image could not be loaded.
why this message showing when i trying to get video on popup window???
i guess i miss something but didn't get the actual issue....
Please help with a little but separate code for lightbox video player?? any feedback would be great..

I'd be happy to help you figure it out. Can you post a snippet of your code, or make an example on codepen or codesandbox?

I'd be happy to help you figure it out. Can you post a snippet of your code, or make an example on codepen or codesandbox?

Here is link of code - https://codesandbox.io/s/bigpicturejs-demos-utx4u?file=/index.html
I tried this one with youtube-video
this running well at first view, but after i add svg icon to the button, after refresh it didn't work again and showing the same issue - Error: The requested image could not be loaded.
Actually my current project have an image gallery - (I get it work! Fine!!!)
Youtube video - Getting error
Vimeo video - Not tested
Local html video - Getting error
It would be great if you have separate demos with separate init javascript code
it would be great if it like

Single image demo
demo html and js code

Image gallery demo
demo html and js code of gallery

local video demo
local video html and js code belongs to only local video

and Same for rest elements like vimeo and youtube html with separate js code..

currently only gallery and single image are separate but other iframe and videos types videos are wrapping in function like hard to understand for average Javascript user like me
Thanks for your time.
and Thanks too for awesome & cool lightbox plugin.. Keep it up..
I also love your another script side-panel like thing. really cool and lightweight, will use it also in near future..
Keep it on brother. but make it little easier to get done with demos for average users like me

No problem! You can make it work by changing e.target to e.currentTarget like this:
BigPicture({
  el: e.currentTarget,
  ytSrc: e.currentTarget.getAttribute("ytSrc")
});

After you put the svg inside the button, e.target becomes the svg when it's clicked (it's the element you clicked to trigger the event). e.currentTarget is always the button even when an element inside is clicked (because it's what you assigned the event listener to). It was failing because the svg does not have the ytSrc attribute.
Here's the working code: https://codesandbox.io/s/bigpicturejs-demos-forked-mwvtq?file=/index.html:3391-3525
I'm going to close this issue, but feel free to follow up with any question or comment.


1.9 items-to-deliver Dosen't Work

My quest have this stage
        '1':
          items-to-deliver:
            - name-cobblestone:amount-10
          npc-delivery-ids:
            - 2

But when i loads the mission the mission is not there. If i remove the stage and put some other stage in it works.

Please create a new, updated ticket if this is still an issue. Thanks!


Add Prettier

@jeremyphilemon do we need this? if not required close this.

Right now the codebase is using eslint + google's javascript style guideline. Would appreciate an issue raised first so we could discuss on style guidelines! Closing this for now.


1.19.4 crash when using command block

Minecraft 1.19.4
Mod verison 0.4.0
Fabric verison 0.14.19
If you interact with a command block and attempt to type in the text box, the game will crash. Crash log here.

Thanks for the report, i'll put out a fix as soon as i can

Pushed to CF and Modrinth, the former will probably take a while to approve


Fix type hints for pytype

We have to be careful for return types to specify that we return
Tuple[T, ...] instead of Sequence[T], at least in those places
where the caller assumes that the result is a tuple.

I am merging this now, to get the copybara going. Happy to continue the discussion. I do think there is going to be a general issue as to how we deal with lists/tuples. I propose we use tuples, as much as possible, and we write Tuple[T, ...] for the type hint.


Fixed #1085 : Why are those strings non translatable ?

Description
The strings are now translatable
Related issues and discussion
#1085

Merged, thanks @PrajwalM2212


Allow product attributes to be customised

Re https://groups.google.com/d/msgid/django-oscar/687fcc5c-6b62-4a50-9089-9bc597983c3d%40googlegroups.com
We should investigate making it easier to customise product attributes. Eg adding new field types - that kind of thing.

1
For example now I want to make some attributes hidden etc..

closing all issues that are feature requests that don't have PR's attached.


frost-icon flickers when re-rendering the containing component

Let’s say you have a simple component or view, just a record with an icon and some text.  User updates a value, causing a this.set to re-render that view.  That icon will blink, even though it hasn’t been changed.

Would make sense to take a second look at the #use mechanics - potentially steal some ideas from ember-svg-jar

https://github.com/ciena-frost/ember-frost-sort/pull/59/files


Success popup is shown twice

I couldn't reproduce it yet, but every once in a while the success popup after submitting an order is shown twice. Overlapping, meaning that I have to hit close twice to get back to the order / positions overview.

Is it critical? I would remove that from the MVP list as it doesn't struck me as anything we should focus on right now (compared to all other bugs).

Does not seem to have happened again.

still happening.. seen it on the latest version.

And it happened again on the latest version 😕

And it happened again on the latest version 😕

It's happening on your phone right?

And it happened again on the latest version 😕

It's happening on your phone right?

Yes

Is this still happening or can I close it again and you reopen it? 😬

Well I think it's still happening, but I am happy to close it and open it once I see it again 🙃

How about a middle ground.... we can agree that it's a shortcoming, but we don't fix it as it has 0 impact on stability and usefulness of the app? 🤓

Possibly related: I fixed a different behaviour where the payment success/fail popup would not always be shown in a prior commit: (https://github.com/get10101/10101/commit/6852939404abc53b54709e3fd390e4c4dde0c4b4). Possible that we need to add a similar check here (whether or not it's already been displayed).

I think the issue is related to how we are using addPostFrameCallback. I guess whats happening is that we are rendering the trade screen due to a change notification and thus re-registering the addPostFrameCallback. This is happening randomly because it depends on when we receive the update (maybe its the price update notification).
I suggest we fix that issue by not relying on the addPostFrameCallback altogether and rather publish an MatchEvent upon which we are showing the Dialog.

I managed to have it 3 times today 😅

that's another bug then.. 🤪


Back End: - MongoDB

[ ] DB config

Model - Users: Username, email, password, current projects*
Model - Projects: Title, description, tech-selection [DB, server, front-end], comments, date stamp.
Model - Technologies: Comments from users, owner, posts.

Completed


Add codecov for pkg

/assign mattmoor

/hold
We should pull in the actions first.

/lgtm
/unhold
Looks like we may want to disable a couple more directories, but nice start 🤩


Is there support for private packages from npm?

Many organizations have private packages hosted on the public registry. I think npm-registry-fetch has a way to handle that. But I'm not sure if this package does so. It would be an easy addition with options, but if there is already a way what would it be?

Yes, see the npmRegistry option at https://github.com/single-spa/self-hosted-shared-dependencies#config-file

No, I meant to use a way to use token, username:password to access private packages from the public registry. Not every company goes about implementing a different registry.

Sorry I misread the issue. Reopening.
We use npm-registry-fetch to load packages from the registry. It supports passing in username and password when calling the npm registry, so we should just add similar options to this repo and then pass them through to npm-registry-fetch.

This was released in https://github.com/single-spa/self-hosted-shared-dependencies/releases/tag/v1.3.0. See the new registryFetchOptions option for documentation - https://github.com/single-spa/self-hosted-shared-dependencies#config-file.


[Feature] New EMR on EKS Terraform module

Create a new EMR on EKS Terraform module under https://github.com/aws-ia/terraform-aws-emr-containers
Update all the EMR on EKS Patterns to use external EMR Containers module

Community Note

What is the outcome that you are trying to reach?

Describe the solution you would like

Describe alternatives you have considered

Additional context

I got a simile request from melody yang. Will help her for a workshop first and then contribute on this one. @vara-bonthu

This is now sub module on each blueprint. In future we will keep all the reusable add-ons and modules under https://github.com/awslabs/data-on-eks/tree/main/workshop/modules


AdMob Layout Issue

Hey there!
The Ad works fine, but the Behaivior in the ScrollView is not like Expected. So therefore I set:
<NativeViewHost RenderToTexture="true" > And now it behaves great.
But the Problem is, that if I go to the Page where the ad is (I have to scroll a bit down), then the Ad loads at the top of the Page (not expected) and also in the NativeViewHost. But the first visible Ad should not be there and is above another elements.
Edit: If i set <NativeViewHost RenderToTexture="false" > the Problem is, that the Ad is (when I scroll) always on Top. I have a Top Menu for the Subpages and when I Scroll the Ad is above that...
Does anyone know why?
(I hope you understand my Problem x) )
Thanks!

+1

Hello,
Can you post a minimal example code so I can take a look?
Thanks

I am going to upload an example Project 👌🏼 thanks for help! :)


"schematics-utilities": "^1.1.1" is outdated for angular 8

I'm submitting a...

[ ] Regression (a behavior that used to work and stopped working in a new release)
[ ] Bug report
[ ] Performance issue
[ ] Feature request
[ ] Documentation issue or request
[ ] Support request
[X] Other... Please describe:

Current behavior
NPM resolves @angular-devkit/core, @angular-devkit/schematics versions as 7.3.9 which leads to unexpected behaviour, unless explicitly set versions in package.json
Expected behavior
Don't have a mess with not really stablilized schematics modules.

I'm not sure what you mean. We explicitly using v7.0.4:
"@angular-devkit/core": "7.0.4",
"@angular-devkit/schematics": "7.0.4",

schematics-utilities is the package that breaks proper dependency resolution. This issue can be reproduced by following those steps:

ng new projec1 --skip-install (with 8.1.2 cli)
npm i @datorama/akita
npm i

After that you'll get @angular-devkit/core and @angular/devkit/schematics of version 7.3.9.
package-lock.json investigation shows that the only package that requires ^7.x.x version is schematics-utilities which, in turn, is required by akita.

We can't move to v8 as schematics-utilities still requires v7.


bump versions for dependabot

Signed-off-by: Steve Taylor<EMAIL_ADDRESS>

This PR will fix the other dependabot PRs in one go and automatically close them as well


Create Representations “e168434e-b812-48b0-b33b-54c3e0826103/wayyyyy-too-many-mice-in-the-air”

Automatically generated by Static CMS

👷 Deploy Preview for endearing-maamoul-70d6fa processing.

Name
Link

🔨 Latest commit
30cf0bad7eafa46ba5e39bb9068f8513178f9a3a

🔍 Latest deploy log
https://app.netlify.com/sites/endearing-maamoul-70d6fa/deploys/65f9f55b25ec8200081a2175


sample: fix socks5 server

in case of bind/listen failure, uv_close() isn't called appropriately

Maybe we should just eliminate this code? (https://github.com/libuv/libuv/issues/1604)

That's fine with me :)

but no sample at all... that's really weird in some way.

Now that we merged the guide I wouldn't mind deleting this one. Unless someone wants to polish it and put in the guide, I guess.


Could not resolve SDK "Microsoft.NET.Runtime.MonoAOTCompiler.Task"

Android framework version
net8.0-android
Affected platform version
.net 8.0.100
Description
Building android application with AOT enabled causes build issues with latest maui-android workload.
[ERR] ---: /Users/runner/.dotnet/packs/Microsoft.Android.Sdk.Darwin/34.0.113/targets/Microsoft.Android.Sdk.Aot.targets(25,5): error : Could not resolve SDK "Microsoft.NET.Runtime.MonoAOTCompiler.Task". Exactly one of the probing messages below indicates why we could not resolve the SDK. Investigate and resolve that message to correctly specify the SDK.
[ERR] ---: /Users/runner/.dotnet/packs/Microsoft.Android.Sdk.Darwin/34.0.113/targets/Microsoft.Android.Sdk.Aot.targets(25,5): error :   SDK resolver "Microsoft.DotNet.MSBuildWorkloadSdkResolver" returned null.
[ERR] ---: /Users/runner/.dotnet/packs/Microsoft.Android.Sdk.Darwin/34.0.113/targets/Microsoft.Android.Sdk.Aot.targets(25,5): error :   The NuGetSdkResolver did not resolve this SDK because there was no version specified in the project or global.json.

Steps to Reproduce
Building existing application was working fine.
Workload update on Azure pipelines caused build to fail.
When using a custom rollback-file workload.json file gives a different error:
[ERR] ---: /Users/runner/.dotnet/packs/Microsoft.Android.Sdk.Darwin/34.0.79/targets/Microsoft.Android.Sdk.Aot.targets(109,5): error MSB4044: The "MonoAOTCompiler" task was not given a value for the required parameter "CompilerBinaryPath". [/Users/runner/work/1/s/src/-----] [ERR] ---: /Users/runner/.dotnet/packs/Microsoft.Android.Sdk.Darwin/34.0.79/targets/Microsoft.Android.Sdk.Aot.targets(109,5): error MSB4044: The "MonoAOTCompiler" task was not given a value for the required parameter "CompilerBinaryPath". [/Users/runner/work/1/s/src/-----]
(removed project info)
Did you find any workaround?
Workload rollback does not fix the issue, but creates different error.
Relevant log output
No response

workload.json file content:
{     "microsoft.net.sdk.android": "34.0.113/8.0.100",     "microsoft.net.workload.mono.toolchain.current": "8.0.6/8.0.100",     "microsoft.net.workload.emscripten.current": "8.0.6/8.0.100",     "microsoft.net.workload.emscripten.net6": "8.0.6/8.0.100",     "microsoft.net.workload.emscripten.net7": "8.0.6/8.0.100",     "microsoft.net.workload.mono.toolchain.net6": "8.0.6/8.0.100",     "microsoft.net.workload.mono.toolchain.net7": "8.0.6/8.0.100" }

Can you attach a .binlog of the error? https://aka.ms/binlog
If you don't use a rollback file (workload.json) does the problem still occur? Microsoft.NET.Runtime.MonoAOTCompiler.Task is from dotnet/runtime, so the microsoft.net.workload.mono.toolchain.current entry would be control what version is used.

Can you attach a .binlog of the error? https://aka.ms/binlog
If you don't use a rollback file (workload.json) does the problem still occur? Microsoft.NET.Runtime.MonoAOTCompiler.Task is from dotnet/runtime, so the microsoft.net.workload.mono.toolchain.current entry would control what version is used.

This is with the latest workloads.
I have created a binlog file, hope this helps.
https://1drv.ms/u/s!AkcCWNpMUZmag91e1Gdgm83XIafawQ?e=CYNpc0

This looks like the installation of dotnet 8.0.300 is corrupt some how?
Warnings: null
Errors: MSB4276: The default SDK resolver failed to resolve SDK "Microsoft.NET.SDK.WorkloadAutoImportPropsLocator" because directory "C:\Program Files\dotnet\sdk\8.0.303\Sdks\Microsoft.NET.SDK.WorkloadAutoImportPropsLocator\Sdk" did not exist

Do you have a global.json in play here?

No, we do not have a global.json file.
This also an issue on our build pipelines on Azure devops.

In your csproj you have the following
<PackageReference Include="Microsoft.NET.Runtime.MonoAOTCompiler.Task" Version="8.0.7" />
I don't think you should be referencing that directly.

That should not be there.
I think it was added when testing the build issue.
Removed, here is a new binlog file.
https://1drv.ms/u/s!AkcCWNpMUZmag91fg78obAoqYgmCpg?e=VQGscR

Out of curiosity does this happen on a vanilla dotnet new android project?

No issue with the vanilla project.
I will compare and step by step try to break the vanilla project.

It can be that it is project related, but we can exclude code change on our side.
We noticed the issue first on a nightly build, so the same code was previously successful.
We have a workload install build step, and there we can see a difference.
The failing build has extra workload manifest installed
Successful build:

14:17:51 [INF] > /usr/local/bin/dotnet workload install maui-android
14:17:51 [DBG]
14:17:53 [DBG] Skipping NuGet package signature verification.
14:17:53 [DBG] Installing workload manifest microsoft.net.sdk.android version 34.0.113...
14:18:02 [DBG] Writing workload pack installation record for Microsoft.Maui.Controls.Core version 8.0.40...
14:18:02 [DBG] Installing pack Microsoft.Maui.Controls.Xaml version 8.0.40...
14:18:03 [DBG] Writing workload pack installation record for Microsoft.Maui.Controls.Xaml version 8.0.40...
14:18:03 [DBG] Installing pack Microsoft.Maui.Controls.Compatibility version 8.0.40...
14:18:03 [DBG] Writing workload pack installation record for Microsoft.Maui.Controls.Compatibility version 8.0.40...
14:18:03 [DBG] Installing pack Microsoft.Maui.Essentials version 8.0.40...
14:18:03 [DBG] Writing workload pack installation record for Microsoft.Maui.Essentials version 8.0.40...
14:18:03 [DBG] Installing pack Microsoft.Android.Sdk.Darwin version 34.0.113...
14:18:14 [DBG] Writing workload pack installation record for Microsoft.Android.Sdk.Darwin version 34.0.113...
14:18:14 [DBG] Installing pack Microsoft.Android.Sdk.Darwin version 33.0.95...
14:18:23 [DBG] Writing workload pack installation record for Microsoft.Android.Sdk.Darwin version 33.0.95...
14:18:23 [DBG] Installing pack Microsoft.Android.Ref.34 version 34.0.113...
14:18:24 [DBG] Writing workload pack installation record for Microsoft.Android.Ref.34 version 34.0.113...
14:18:24 [DBG] Installing pack Microsoft.Android.Runtime.34.android-arm version 34.0.113...
14:18:25 [DBG] Writing workload pack installation record for Microsoft.Android.Runtime.34.android-arm version 34.0.113...
14:18:25 [DBG] Installing pack Microsoft.Android.Runtime.34.android-arm64 version 34.0.113...
14:18:27 [DBG] Writing workload pack installation record for Microsoft.Android.Runtime.34.android-arm64 version 34.0.113...
14:18:27 [DBG] Installing pack Microsoft.Android.Runtime.34.android-x86 version 34.0.113...
14:18:28 [DBG] Writing workload pack installation record for Microsoft.Android.Runtime.34.android-x86 version 34.0.113...
14:18:28 [DBG] Installing pack Microsoft.Android.Runtime.34.android-x64 version 34.0.113...
14:18:30 [DBG] Writing workload pack installation record for Microsoft.Android.Runtime.34.android-x64 version 34.0.113...
14:18:30 [DBG] Installing pack Microsoft.Android.Templates version 34.0.113...
14:18:30 [DBG] Writing workload pack installation record for Microsoft.Android.Templates version 34.0.113...
14:18:30 [DBG] Installing pack Microsoft.NETCore.App.Runtime.Mono.android-arm version 7.0.20...
14:18:32 [DBG] Writing workload pack installation record for Microsoft.NETCore.App.Runtime.Mono.android-arm version 7.0.20...
14:18:32 [DBG] Installing pack Microsoft.NETCore.App.Runtime.Mono.android-arm64 version 7.0.20...
14:18:33 [DBG] Writing workload pack installation record for Microsoft.NETCore.App.Runtime.Mono.android-arm64 version 7.0.20...
14:18:33 [DBG] Installing pack Microsoft.NETCore.App.Runtime.Mono.android-x64 version 7.0.20...
14:18:38 [DBG] Writing workload pack installation record for Microsoft.NETCore.App.Runtime.Mono.android-x64 version 7.0.20...
14:18:38 [DBG] Installing pack Microsoft.NETCore.App.Runtime.Mono.android-x86 version 7.0.20...
14:18:40 [DBG] Writing workload pack installation record for Microsoft.NETCore.App.Runtime.Mono.android-x86 version 7.0.20...
14:18:40 [DBG] Installing pack Microsoft.NET.Runtime.MonoAOTCompiler.Task version 7.0.20...
14:18:41 [DBG] Writing workload pack installation record for Microsoft.NET.Runtime.MonoAOTCompiler.Task version 7.0.20...
14:18:41 [DBG] Installing pack Microsoft.NET.Runtime.MonoTargets.Sdk version 7.0.20...
14:18:41 [DBG] Writing workload pack installation record for Microsoft.NET.Runtime.MonoTargets.Sdk version 7.0.20...
14:18:41 [DBG] Installing pack Microsoft.NETCore.App.Runtime.AOT.osx-x64.Cross.android-x86 version 7.0.20...
14:18:43 [DBG] Writing workload pack installation record for Microsoft.NETCore.App.Runtime.AOT.osx-x64.Cross.android-x86 version 7.0.20...
14:18:43 [DBG] Installing pack Microsoft.NETCore.App.Runtime.AOT.osx-x64.Cross.android-x64 version 7.0.20...
14:18:45 [DBG] Writing workload pack installation record for Microsoft.NETCore.App.Runtime.AOT.osx-x64.Cross.android-x64 version 7.0.20...
14:18:45 [DBG] Installing pack Microsoft.NETCore.App.Runtime.AOT.osx-x64.Cross.android-arm version 7.0.20...
14:18:47 [DBG] Writing workload pack installation record for Microsoft.NETCore.App.Runtime.AOT.osx-x64.Cross.android-arm version 7.0.20...
14:18:47 [DBG] Installing pack Microsoft.NETCore.App.Runtime.AOT.osx-x64.Cross.android-arm64 version 7.0.20...
14:18:48 [DBG] Writing workload pack installation record for Microsoft.NETCore.App.Runtime.AOT.osx-x64.Cross.android-arm64 version 7.0.20...
14:18:48 [DBG] Installing pack Microsoft.NETCore.App.Runtime.Mono.android-arm version 8.0.6...
14:18:49 [DBG] Writing workload pack installation record for Microsoft.NETCore.App.Runtime.Mono.android-arm version 8.0.6...
14:18:49 [DBG] Installing pack Microsoft.NETCore.App.Runtime.Mono.android-arm64 version 8.0.6...
14:18:51 [DBG] Writing workload pack installation record for Microsoft.NETCore.App.Runtime.Mono.android-arm64 version 8.0.6...
14:18:51 [DBG] Installing pack Microsoft.NETCore.App.Runtime.Mono.android-x64 version 8.0.6...
14:18:53 [DBG] Writing workload pack installation record for Microsoft.NETCore.App.Runtime.Mono.android-x64 version 8.0.6...
14:18:53 [DBG] Installing pack Microsoft.NETCore.App.Runtime.Mono.android-x86 version 8.0.6...
14:18:54 [DBG] Writing workload pack installation record for Microsoft.NETCore.App.Runtime.Mono.android-x86 version 8.0.6...
14:18:54 [DBG] Installing pack Microsoft.NET.Runtime.MonoAOTCompiler.Task version 8.0.6...
14:18:54 [DBG] Writing workload pack installation record for Microsoft.NET.Runtime.MonoAOTCompiler.Task version 8.0.6...
14:18:54 [DBG] Installing pack Microsoft.NET.Runtime.MonoTargets.Sdk version 8.0.6...
14:18:55 [DBG] Writing workload pack installation record for Microsoft.NET.Runtime.MonoTargets.Sdk version 8.0.6...
14:18:55 [DBG] Installing pack Microsoft.NETCore.App.Runtime.AOT.osx-x64.Cross.android-x86 version 8.0.6...
14:18:57 [DBG] Writing workload pack installation record for Microsoft.NETCore.App.Runtime.AOT.osx-x64.Cross.android-x86 version 8.0.6...
14:18:57 [DBG] Installing pack Microsoft.NETCore.App.Runtime.AOT.osx-x64.Cross.android-x64 version 8.0.6...
14:18:59 [DBG] Writing workload pack installation record for Microsoft.NETCore.App.Runtime.AOT.osx-x64.Cross.android-x64 version 8.0.6...
14:18:59 [DBG] Installing pack Microsoft.NETCore.App.Runtime.AOT.osx-x64.Cross.android-arm version 8.0.6...
14:19:03 [DBG] Writing workload pack installation record for Microsoft.NETCore.App.Runtime.AOT.osx-x64.Cross.android-arm version 8.0.6...
14:19:03 [DBG] Installing pack Microsoft.NETCore.App.Runtime.AOT.osx-x64.Cross.android-arm64 version 8.0.6...
14:19:05 [DBG] Writing workload pack installation record for Microsoft.NETCore.App.Runtime.AOT.osx-x64.Cross.android-arm64 version 8.0.6...
14:19:05 [DBG] Garbage collecting for SDK feature band(s) 6.0.400 7.0.100 7.0.200 7.0.300 7.0.400 8.0.100 8.0.200 8.0.300...
14:19:05 [DBG]
14:19:05 [DBG] Successfully installed workload(s) maui-android.
14:19:05 [DBG]

Next the failing build:

06:30:32 [INF] > /usr/local/bin/dotnet workload install maui-android
06:30:33 [DBG]
06:30:34 [DBG] Skipping NuGet package signature verification.
06:30:35 [DBG] Installing workload manifest microsoft.net.sdk.android version 34.0.113...
06:31:00 [DBG] Writing workload pack installation record for Microsoft.Android.Sdk.Darwin version 33.0.95...
06:31:00 [DBG] Installing pack Microsoft.Android.Ref.34 version 34.0.113...
06:31:01 [DBG] Writing workload pack installation record for Microsoft.Android.Ref.34 version 34.0.113...
06:31:01 [DBG] Installing pack Microsoft.Android.Runtime.34.android-arm version 34.0.113...
06:31:02 [DBG] Writing workload pack installation record for Microsoft.Android.Runtime.34.android-arm version 34.0.113...
06:31:02 [DBG] Installing pack Microsoft.Android.Runtime.34.android-arm64 version 34.0.113...
06:31:04 [DBG] Writing workload pack installation record for Microsoft.Android.Runtime.34.android-arm64 version 34.0.113...
06:31:04 [DBG] Installing pack Microsoft.Android.Runtime.34.android-x86 version 34.0.113...
06:31:06 [DBG] Writing workload pack installation record for Microsoft.Android.Runtime.34.android-x86 version 34.0.113...
06:31:06 [DBG] Installing pack Microsoft.Android.Runtime.34.android-x64 version 34.0.113...
06:31:07 [DBG] Writing workload pack installation record for Microsoft.Android.Runtime.34.android-x64 version 34.0.113...
06:31:07 [DBG] Installing pack Microsoft.Android.Templates version 34.0.113...
06:31:07 [DBG] Writing workload pack installation record for Microsoft.Android.Templates version 34.0.113...
06:31:07 [DBG] Installing pack Microsoft.NETCore.App.Runtime.Mono.android-arm version 7.0.20...
06:31:09 [DBG] Writing workload pack installation record for Microsoft.NETCore.App.Runtime.Mono.android-arm version 7.0.20...
06:31:09 [DBG] Installing pack Microsoft.NETCore.App.Runtime.Mono.android-arm64 version 7.0.20...
06:31:10 [DBG] Writing workload pack installation record for Microsoft.NETCore.App.Runtime.Mono.android-arm64 version 7.0.20...
06:31:10 [DBG] Installing pack Microsoft.NETCore.App.Runtime.Mono.android-x64 version 7.0.20...
06:31:12 [DBG] Writing workload pack installation record for Microsoft.NETCore.App.Runtime.Mono.android-x64 version 7.0.20...
06:31:12 [DBG] Installing pack Microsoft.NETCore.App.Runtime.Mono.android-x86 version 7.0.20...
06:31:14 [DBG] Writing workload pack installation record for Microsoft.NETCore.App.Runtime.Mono.android-x86 version 7.0.20...
06:31:14 [DBG] Installing pack Microsoft.NET.Runtime.MonoAOTCompiler.Task version 7.0.20...
06:31:14 [DBG] Writing workload pack installation record for Microsoft.NET.Runtime.MonoAOTCompiler.Task version 7.0.20...
06:31:14 [DBG] Installing pack Microsoft.NET.Runtime.MonoTargets.Sdk version 7.0.20...
06:31:14 [DBG] Writing workload pack installation record for Microsoft.NET.Runtime.MonoTargets.Sdk version 7.0.20...
06:31:14 [DBG] Installing pack Microsoft.NETCore.App.Runtime.AOT.osx-x64.Cross.android-x86 version 7.0.20...
06:31:16 [DBG] Writing workload pack installation record for Microsoft.NETCore.App.Runtime.AOT.osx-x64.Cross.android-x86 version 7.0.20...
06:31:16 [DBG] Installing pack Microsoft.NETCore.App.Runtime.AOT.osx-x64.Cross.android-x64 version 7.0.20...
06:31:17 [DBG] Writing workload pack installation record for Microsoft.NETCore.App.Runtime.AOT.osx-x64.Cross.android-x64 version 7.0.20...
06:31:17 [DBG] Installing pack Microsoft.NETCore.App.Runtime.AOT.osx-x64.Cross.android-arm version 7.0.20...
06:31:19 [DBG] Writing workload pack installation record for Microsoft.NETCore.App.Runtime.AOT.osx-x64.Cross.android-arm version 7.0.20...
06:31:19 [DBG] Installing pack Microsoft.NETCore.App.Runtime.AOT.osx-x64.Cross.android-arm64 version 7.0.20...
06:31:21 [DBG] Writing workload pack installation record for Microsoft.NETCore.App.Runtime.AOT.osx-x64.Cross.android-arm64 version 7.0.20...
06:31:21 [DBG] Installing pack Microsoft.NETCore.App.Runtime.Mono.android-arm version 8.0.7...
06:31:23 [DBG] Writing workload pack installation record for Microsoft.NETCore.App.Runtime.Mono.android-arm version 8.0.7...
06:31:23 [DBG] Installing pack Microsoft.NETCore.App.Runtime.Mono.android-arm64 version 8.0.7...
06:31:26 [DBG] Writing workload pack installation record for Microsoft.NETCore.App.Runtime.Mono.android-arm64 version 8.0.7...
06:31:26 [DBG] Installing pack Microsoft.NETCore.App.Runtime.Mono.android-x64 version 8.0.7...
06:31:28 [DBG] Writing workload pack installation record for Microsoft.NETCore.App.Runtime.Mono.android-x64 version 8.0.7...
06:31:28 [DBG] Installing pack Microsoft.NETCore.App.Runtime.Mono.android-x86 version 8.0.7...
06:31:29 [DBG] Writing workload pack installation record for Microsoft.NETCore.App.Runtime.Mono.android-x86 version 8.0.7...
06:31:29 [DBG] Installing pack Microsoft.NET.Runtime.MonoAOTCompiler.Task version 8.0.7...
06:31:29 [DBG] Writing workload pack installation record for Microsoft.NET.Runtime.MonoAOTCompiler.Task version 8.0.7...
06:31:29 [DBG] Installing pack Microsoft.NET.Runtime.MonoTargets.Sdk version 8.0.7...
06:31:30 [DBG] Writing workload pack installation record for Microsoft.NET.Runtime.MonoTargets.Sdk version 8.0.7...
06:31:30 [DBG] Installing pack Microsoft.NETCore.App.Runtime.AOT.osx-x64.Cross.android-x86 version 8.0.7...
06:31:32 [DBG] Writing workload pack installation record for Microsoft.NETCore.App.Runtime.AOT.osx-x64.Cross.android-x86 version 8.0.7...
06:31:32 [DBG] Installing pack Microsoft.NETCore.App.Runtime.AOT.osx-x64.Cross.android-x64 version 8.0.7...
06:31:35 [DBG] Writing workload pack installation record for Microsoft.NETCore.App.Runtime.AOT.osx-x64.Cross.android-x64 version 8.0.7...
06:31:35 [DBG] Installing pack Microsoft.NETCore.App.Runtime.AOT.osx-x64.Cross.android-arm version 8.0.7...
06:31:40 [DBG] Writing workload pack installation record for Microsoft.NETCore.App.Runtime.AOT.osx-x64.Cross.android-arm version 8.0.7...
06:31:40 [DBG] Installing pack Microsoft.NETCore.App.Runtime.AOT.osx-x64.Cross.android-arm64 version 8.0.7...
06:31:44 [DBG] Writing workload pack installation record for Microsoft.NETCore.App.Runtime.AOT.osx-x64.Cross.android-arm64 version 8.0.7...
06:31:44 [DBG] Garbage collecting for SDK feature band(s) 6.0.400 7.0.100 7.0.200 7.0.300 7.0.400 8.0.100 8.0.200 8.0.300...
06:31:44 [DBG]
06:31:44 [DBG] Successfully installed workload(s) maui-android.
06:31:44 [DBG]

Adding this to the vanilla project causes it to fail:
true

@jonathanpeppers should people be using AndroidEnableProfiledAot  instead of just using standard Aot?

I checked the docs.
https://learn.microsoft.com/en-us/dotnet/maui/migration/android-projects?view=net-maui-8.0#ahead-of-time-compilation
I noticed that in our project, the RunAOTCompilation flag was missing.
So the combination
<RunAOTCompilation>true</RunAOTCompilation> <AndroidEnableProfiledAot>true</AndroidEnableProfiledAot>
Fixes the issue.
Not sure why this suddenly was an issue, but it was missing on our side.

<RunAOTCompilation>true</RunAOTCompilation> <AndroidEnableProfiledAot>true</AndroidEnableProfiledAot>

These are the default values for Release mode:
https://github.com/dotnet/android/blob/83626c9cf8a8af3b3e2f365de147662cbb38a2d6/src/Xamarin.Android.Build.Tasks/Microsoft.Android.Sdk/targets/Microsoft.Android.Sdk.DefaultProperties.targets#L96-L103
Why do you need to set these at all? Is your $(Configuration) named something other than Release?

Yes, we have multiple configurations for different environments.
We do not have a default release config.

I would not recommend using multiple $(Configuration) names like this, see:

https://github.com/dotnet/sdk/issues/31918

Since there are N potentially unknown properties you would need to set, no one know the exhaustive list of properties to set to get it right. I bet you don't have $(Optimize), set for example, which is fairly important for performance.
If you need to set something for different environments, I would simply use a different property other than $(Configuration) like:
-p:MySetting=Production

And then use Condition against your $(MySetting) property instead of $(Configuration).


The Experience Data Model (XDM) link in the README is broken

What are the schemas that are affected by the issue
What are examples of products that are impacted by the issue

I've added this link in anticipation of bringing the new XDM page online, which should happen by the end of the week. Thanks for reporting.


Warnings and a strict mode

With Lume there are currently many opportunities to make small mistakes that will not be reported. For example:

Mistyping the source path given to Site.copy (if the file doesn't exist, nothing will happen).
Using Site.copy multiple times for the same source path with different destinations (only the last call will take effect).
Making a typo in a variable in a nunjuck template (i.e. referencing undefined data).
Making a typo when specifying a tag name (either in the metadata or when calling e.g. search.pages).

I think it would be nice if such likely usage errors would be detected and reported. The CLI could even have a --strict mode that could fail if any such problem was to be detected for usage within CIs. Detecting the mistyped tags would require the ability to optionally declare them with e.g. lume({tags: ["post"]}, ...).
What do you think about this matter?

There are some interesting points here. Not sure if a strict mode is a good idea because it would be hard to implement some of the checkings.
For example, there are many ways to select and copy a file:

Using a extension: site.copy([".jpg"]
Using the file name: site.copy("image/one.jpg")
Using a folder name: site.copy("image")

The same file can match to multiple conditions, and this is okay. For example, you may want to copy all jpg files to a specific directory, except a couple of files that have a different destination. The only thing we can do is if the same condition was already used (for example, calling site.copy("image") twice), we can warn it.

Making a typo in a variable in a nunjuck template (i.e. referencing undefined data).

Nunjucks has already an option for that (throwOnUndefined) that by default is false.

Making a typo when specifying a tag name (either in the metadata or when calling e.g. search.pages).

I don't understand this. It's okay to filter by non existing fields. For example, you may want to get all pages with the category "lume", so you run search.pages("category=lume"). There may be pages without the category defined (because they are different types of pages that don't need it). What do you think it should happen here?

I think the best way to address the pitfall of the current Site.copy API is to enable the copying of one file to multiple destinations and perform the deprecations suggested in https://github.com/lumeland/lume/issues/428#issuecomment-1585699760.

I don't understand this.

I agree that it's okay to filter on non-existing fields. I was talking of some way of validating the metadata defined in frontmatters. I guess this could be achieved with an event handler.
If you were to define the schema for your data with e.g. JSON schema we could even validate queries. E.g. if you have defined that there only are the tags foo, bar and baz then filtering for a tag orange could be detected as an error. But yeah validating data is of course more important than validating queries.

Schema validation is a nice idea for a plugin. It could be a preprocessor that validate all pages before rendering and show warnings when a page has a different scheme.


add options.spacing

as mentioned, we have to be careful because we can quickly distort the visualization so its not proportionally representing the data. i ran into this last time i was playing with spacing/padding.. perhaps that's why i commented it out.
ideally this is something we could test.. get the visual pixel area of boxes and compare against the input data.
but i'm def not blocking your dev on that.


Demo third-party links with real external web site

You can test this at https://dewv.net/ftm/test.html, where I have set up a demo of a mechanism for embedded third party links.
(Note second magnifying glass icon is temporarily used for the third party link.)
I think this approach is simple and reasonably abuse-proof.
For any and only sites that we put in the partner list, their embed requests will be served with links in the embed location list. Those site(s) will just need to handle /ftm?id=X requests.

Interesting. This looks pretty good!
So basically each specific referrer would need to submit a PR to have themselves added with their own icon.
And then the callback URL format is also similarly encoded (nothing dynamic via parameters).
I like it!

Tagging @ngottlieb since we were chatting.  This approach seems to meet the security requirements and be even a little bit simpler (ie no URL parameter.)

Closing this PR without merge so I can create another to get a redeploy.


Cloudflare Workers support (by moving off of axios)

Enhancement description
This SDK is currently unusable in Cloudflare Workers because it uses axios, which strangely doesn't support using fetch instead of the super-old XHR api (and it doesn't seem they plan to address it anytime soon https://github.com/axios/axios/issues/1219). Moving from axios to something like isomorphic-fetch would solve the issue.
Bonus: doing this would make Deno work for free, I think, because Deno also supports fetch. (#19)
Alternative, easier solution: expose an options object for the TodoistApi constructor, with an adapter option that lets me pass a custom axios adapter to use.
The problem it solves
I can't use this Todoist SDK in Cloudflare workers currently. I get this error:
TypeError: adapter is not a function

This is caused by these lines in axios - it only supports node and environments with XHR. Cloudflare Workers is neither of those, unfortunately.
https://github.com/axios/axios/blob/65977f995c7b92aeca26aa1ba50e1d6678389002/lib/defaults/index.js#L22-L32
Alternatives
I haven't found a good workaround for this. One solution I've thought of is forking axios, gutting it and replacing the internals with fetch, and then using the resolutions/overrides field to make this package use my axios-but-actually-fetch package instead of the real axios. But that would be a lot of work :sweat_smile:

Great proposal @drmercer, we'd love to make this at least configurable 👍
Most of the code resides in the restClient.ts, there you can see we are also using axios-retry to get automatic retries on network failures. The retry mechanism is something we could ditch for users supplying their own "network client". Would you be willing to contribute a PR for this? You could get inspired by similar work where we made the base API URL configurable: https://github.com/Doist/todoist-api-typescript/pull/130/

Also it is not working on Browser extensions with Manifest v3 Service workers.

As a workaround, I got this working by using patch-package with this patch:
diff --git a/node_modules/axios/lib/defaults/index.js b/node_modules/axios/lib/defaults/index.js
index 9199818..1cfccc6 100644
--- a/node_modules/axios/lib/defaults/index.js
+++ b/node_modules/axios/lib/defaults/index.js
@@ -17,15 +17,7 @@ function setContentTypeIfUnset(headers, value) {
 }

 function getDefaultAdapter() {
-  var adapter;
-  if (typeof XMLHttpRequest !== 'undefined') {
-    // For browsers use XHR adapter
-    adapter = require('../adapters/xhr');
-  } else if (typeof process !== 'undefined' && Object.prototype.toString.call(process) === '[object process]') {
-    // For node use HTTP adapter
-    adapter = require('../adapters/http');
-  }
-  return adapter;
+  return require('@vespaiach/axios-fetch-adapter').default
 }

 function stringifySafely(rawValue, parser, encoder) {

@henningmu I'm not sure if I'll have time to contribute something, but I might give it a shot. :slightly_smiling_face: What would the ideal interface look like for that configuration? Would the user pass in a custom function to replace the request function from restClient.ts?

Awesome to hear @drmercer. Yes, we could make the whole request function configurable, that's probably the cleanest cut (as opposed to just the axiosClient).
Another alternative could be to rip out axios completely and replace it with isomorphic-fetch. We would accept a PR for that as well.
I'm off for this and next week, but if time allows I'll try my hands at the isomorphic-fetch solution after 👍

Hmmm, I'm actually not sure if isomorphic-fetch or cross-fetch will solve the problem, because it doesn't seem either of them use the native fetch if available (they always reimplement it, for consistency). The ideal would be to use node-fetch in Node, and the native fetch in any other environment... Would it be too much to ask the user to load node-fetch themselves if they're running in Node?

It looks like it was eventually addressed. Is this issue still applicable?

Not sure what happened but for my own project, it wasn't working before and it was working now.

It was failing due to some requestinfodict support thing being missing but it looks like the last of the roadblocks in the CF workers environment might have been lifted.

Thank you for the update @nelsonjchen. I'll go ahead and close this issue 👍


.NET 8: Create and documented a support model for distro maintainers

From https://github.com/dotnet/source-build/issues/2926
This issue focuses on creating and documenting a support model for distro maintainers, including a method for dealing with early disclosure of CVEs to distro maintainers. This would be an extension of the existing model we have some specific distros, with a different disclosure timeframe.

[Triage] @richlander, is this something you can help author?


HADOOP-18802. Collision of config key name fs.viewfs.mounttable.default.name.key to other keys that specify the entry point to mount tables

Description of PR
https://issues.apache.org/jira/browse/HADOOP-18802
This PR changes the name of the previous fs.viewfs.mounttable.default.name.key to fs.viewfs.mounttable.default-name-key to avoid collision.
I also grepped for the parameter name and changed it in all places.
How was this patch tested?
(1) set fs.viewfs.mounttable.default-name-key to default
(2) run org.apache.hadoop.fs.viewfs.TestFcMainOperationsLocalFs#testGlobStatusWithMultipleWildCardMatches
the test passes.
For code changes:

[x] Does the title or this PR starts with the corresponding JIRA issue id (e.g. 'HADOOP-17799. Your PR title ...')?
[ ] Object storage: have the integration tests been executed and the endpoint declared according to the connector-specific documentation?
[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[ ] If applicable, have you updated the LICENSE, LICENSE-binary, NOTICE-binary files?

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 31s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  0s

codespell was not available.

+0 :ok:
detsecrets
0m  0s

detect-secrets was not available.

+0 :ok:
markdownlint
0m  0s

markdownlint was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

-1 :x:
test4tests
0m  0s

The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ trunk Compile Tests _

+0 :ok:
mvndep
13m 33s

Maven dependency ordering for branch

+1 :green_heart:
mvninstall
20m  6s

trunk passed

+1 :green_heart:
compile
10m 45s

trunk passed with JDK Ubuntu-11.0.20+8-post-Ubuntu-1ubuntu120.04

+1 :green_heart:
compile
9m 47s

trunk passed with JDK Private Build-1.8.0_382-8u382-ga-1~20.04.1-b05

+1 :green_heart:
checkstyle
2m 24s

trunk passed

+1 :green_heart:
mvnsite
2m 19s

trunk passed

+1 :green_heart:
javadoc
1m 56s

trunk passed with JDK Ubuntu-11.0.20+8-post-Ubuntu-1ubuntu120.04

+1 :green_heart:
javadoc
2m  7s

trunk passed with JDK Private Build-1.8.0_382-8u382-ga-1~20.04.1-b05

+1 :green_heart:
spotbugs
3m 46s

trunk passed

+1 :green_heart:
shadedclient
23m  3s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 26s

Maven dependency ordering for patch

+1 :green_heart:
mvninstall
1m 21s

the patch passed

+1 :green_heart:
compile
9m 43s

the patch passed with JDK Ubuntu-11.0.20+8-post-Ubuntu-1ubuntu120.04

+1 :green_heart:
javac
9m 43s

the patch passed

+1 :green_heart:
compile
9m 33s

the patch passed with JDK Private Build-1.8.0_382-8u382-ga-1~20.04.1-b05

+1 :green_heart:
javac
9m 33s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

+1 :green_heart:
checkstyle
2m 22s

the patch passed

+1 :green_heart:
mvnsite
2m 18s

the patch passed

+1 :green_heart:
javadoc
1m 52s

the patch passed with JDK Ubuntu-11.0.20+8-post-Ubuntu-1ubuntu120.04

+1 :green_heart:
javadoc
2m  6s

the patch passed with JDK Private Build-1.8.0_382-8u382-ga-1~20.04.1-b05

+1 :green_heart:
spotbugs
3m 58s

the patch passed

+1 :green_heart:
shadedclient
22m 57s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

+1 :green_heart:
unit
16m 31s

hadoop-common in the patch passed.

+1 :green_heart:
unit
195m  8s

hadoop-hdfs in the patch passed.

+1 :green_heart:
asflicense
0m 59s

The patch does not generate ASF License warnings.

362m 45s

Subsystem
Report/Notes

Docker
ClientAPI=1.43 ServerAPI=1.43 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-6040/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/6040

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell detsecrets markdownlint

uname
Linux 3bdccc0a32d5 4.15.0-213-generic #224-Ubuntu SMP Mon Jun 19 13:30:12 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / 55444f8f2f27eaf1b09e5331b98630133b525eba

Default Java
Private Build-1.8.0_382-8u382-ga-1~20.04.1-b05

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.20+8-post-Ubuntu-1ubuntu120.04 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_382-8u382-ga-1~20.04.1-b05

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-6040/1/testReport/

Max. process+thread count
3145 (vs. ulimit of 5500)

modules
C: hadoop-common-project/hadoop-common hadoop-hdfs-project/hadoop-hdfs U: .

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-6040/1/console

versions
git=2.25.1 maven=3.6.3 spotbugs=4.2.2

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

This message was automatically generated.


Run upgrade tests after beta cut

Is there an existing issue for this?

[X] I have searched the existing issues

Problem Statement
We have breaking changes in 2.0, user would have broken controller after upgrade.
Proposed Solution
We need ensure there is no regression when user upgrade from 1.3 to 2.0
Run upgrade tests against beta
Run upgrade tests against Kong ee
Additional information
No response
Acceptance Criteria
No response

duplicate. close as it is.

duplicate. close as it is.

im confused by this comment: which issue does this duplicate?

Oh, its https://github.com/Kong/kubernetes-ingress-controller/issues/1656 :+1:


State Between Components in the Specified Configuration

Is there a way to share state between components in the config? Other than abusing dom tricks? Or is the state of each component isolated?
From my understanding, it would likely be the latter, and that makes more sense for what Puck is trying to achieve, but for my use case I would be hoping that there might be a way to share state. Global states would work as well.

Hey @AFK-Python!
You should be able to share state without DOM tricks by using React context, defining a provider using the root configuration. Something like:
import { createContext, useContext, useState } from "react";

const context = createContext({
  counter: 0,
  setCounter: (value: number) => {},
});

const { Provider } = context;

const config = {
  root: {
    render: ({ children }) => {
      const [counter, setCounter] = useState(0);

      return <Provider value={{ counter, setCounter }}>{children}</Provider>;
    },
  },
  components: {
    MyComponent: {
      render: () => {
        const { counter, setCounter } = useContext(context);

        return (
          <>

Clicked {counter} times

            <button onClick={() => setCounter(counter + 1)}>Click me</button>
          </>
        );
      },
    },
  },
};

Let me know if that works! Going to close the issue.


Use only cartovl account for examples

Fixes #1035
Some examples were still using personal accounts (mamataakella & rochoa)

thanks @VictorVelarde! looks good to me :)


getUniqueIdSync is not a function

Your ReactNativePlugin calls getUniqueIdSync(); and getModelSync()... both of which no longer exist in the react-native-device-info package.

Downgrading react-native-device-info-package to v4.0.1-beta.4 circumvents this issue. For anyone else who comes across this.

I downgraded to 3.1.4 to resolve this.
At the very least the README.md should modify the Keep react-native-device-info up-to-date clause as this tripped me up.

Sorry for the delay on this. I've republished the react-native plugin (2.0.1) to use the latest breaking change version of rndi. It should use constants where available, else it will use the sync APIs. Please let me know how it goes!


Docfx v3 migration

Migration PR to turn docfx build engine to v3

The PR is not being built because the target branch live is filtered in .openpublishing.publish.config.json on live branch.


[WIP] Coveralls in CI

Changes Unknown when pulling 72344750d1be7d6b3d7113055ee98749180e094d on features/codecov into ** on master**.


Generate code reports

Added report from goreportcard

Codecov Report

Merging #10 (b46e93e) into main (a217b91) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #10   +/-   ##
=======================================
  Coverage   94.11%   94.11%
=======================================
  Files           4        4
  Lines         170      170
=======================================
  Hits          160      160
  Misses          7        7
  Partials        3        3

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 a217b91...b46e93e. Read the comment docs.


Bump Docker nodejs version

I think we can bump up nodejs docker version here: https://github.com/Mainflux/mainflux/blob/master/Dockerfile#L6. I can see here: https://hub.docker.com/_/node/ that latest is 4.1.2

Agree! I will set the new version.

Please do, send PR and do not forger to reference this issue ID in the PR comment.

Fixed by https://github.com/Mainflux/mainflux/commit/ef00dac107a4fedaeb3424190c87eca22ad5d16f


Add oracle system variables

oracle's system variables via set command has been added.

merge conflicts has been checked - all is ok!


Pin to top

Description of the new feature / enhancement
I want to have something that can pin a particular window on top always something like Deskpins.
Scenario when this would be used?
If I'm working on a project and I want an image as a reference I just pin it on top so it will always stay on top no matter what am I using.
Supporting information
if you can convert this C++ tool into C# it would be cool.

Issue is a duplicate
Thank you for your issue! We've linked your report against another issue #13
Thanks for helping us to make PowerToys a better piece of software.

Duplicate of #13


CLI Should use eslint-plugin-hydrogen when user chooses eslint

3 quick things to fix this

add the eslint-plugin-hydrogen package
change the eslint config

module.exports = {
  extends: ['plugin:hydrogen/recommended'],
};

fix any eslint errors

Closing - feel free to transfer to new location.


fix: removing Gist dependency

How does this PR make you feel?

What does this PR do?
It removes the dependency of Gist file from our centralized repo
Tested the changes in the pipeline https://dashboard.heroku.com/apps/ds-test-repo-5-sfdocs/settings

Manual Deploy is successful
Auto deploy is successful.

Hope it's been tested.

@LokeshNSF Yeah Lokesh, I have also attached the pipeline settings in the PR.