mysql connector with GTID mode error " Duplicate entry '1' for key 'PRIMARY' "

hi, @ebyhr I found you have fixed issue #2251 in version 327. well done!
when testing it again, I get another error:
 io.prestosql.spi.PrestoException: Duplicate entry '1' for key 'PRIMARY'
	at io.prestosql.plugin.jdbc.BaseJdbcClient.finishInsertTable(BaseJdbcClient.java:570)
	at io.prestosql.plugin.jdbc.jmx.StatisticsAwareJdbcClient.lambda$finishInsertTable$19(StatisticsAwareJdbcClient.java:192)
	at io.prestosql.plugin.jdbc.jmx.JdbcApiStats.wrap(JdbcApiStats.java:47)
	at io.prestosql.plugin.jdbc.jmx.StatisticsAwareJdbcClient.finishInsertTable(StatisticsAwareJdbcClient.java:192)
	at io.prestosql.plugin.jdbc.ForwardingJdbcClient.finishInsertTable(ForwardingJdbcClient.java:144)
	at io.prestosql.plugin.jdbc.ForwardingJdbcClient.finishInsertTable(ForwardingJdbcClient.java:144)
	at io.prestosql.plugin.jdbc.JdbcMetadata.finishInsert(JdbcMetadata.java:258)
	at io.prestosql.metadata.MetadataManager.finishInsert(MetadataManager.java:790)
	at io.prestosql.sql.planner.LocalExecutionPlanner.lambda$createTableFinisher$3(LocalExecutionPlanner.java:2850)
	at io.prestosql.operator.TableFinishOperator.getOutput(TableFinishOperator.java:297)
	at io.prestosql.operator.Driver.processInternal(Driver.java:379)
	at io.prestosql.operator.Driver.lambda$processFor$8(Driver.java:283)
	at io.prestosql.operator.Driver.tryWithLock(Driver.java:675)
	at io.prestosql.operator.Driver.processFor(Driver.java:276)
	at io.prestosql.execution.SqlTaskExecution$DriverSplitRunner.processFor(SqlTaskExecution.java:1075)
	at io.prestosql.execution.executor.PrioritizedSplitRunner.process(PrioritizedSplitRunner.java:163)
	at io.prestosql.execution.executor.TaskExecutor$TaskRunner.run(TaskExecutor.java:484)
	at io.prestosql.$gen.Presto_327____20191221_104034_2.run(Unknown Source)
	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149)
	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624)
	at java.lang.Thread.run(Thread.java:748)
Caused by: com.mysql.jdbc.exceptions.jdbc4.MySQLIntegrityConstraintViolationException: Duplicate entry '1' for key 'PRIMARY'
	at sun.reflect.NativeConstructorAccessorImpl.newInstance0(Native Method)
	at sun.reflect.NativeConstructorAccessorImpl.newInstance(NativeConstructorAccessorImpl.java:62)
	at sun.reflect.DelegatingConstructorAccessorImpl.newInstance(DelegatingConstructorAccessorImpl.java:45)
	at java.lang.reflect.Constructor.newInstance(Constructor.java:423)
	at com.mysql.jdbc.Util.handleNewInstance(Util.java:425)
	at com.mysql.jdbc.Util.getInstance(Util.java:408)
	at com.mysql.jdbc.SQLError.createSQLException(SQLError.java:936)
	at com.mysql.jdbc.MysqlIO.checkErrorPacket(MysqlIO.java:3933)
	at com.mysql.jdbc.MysqlIO.checkErrorPacket(MysqlIO.java:3869)
	at com.mysql.jdbc.MysqlIO.sendCommand(MysqlIO.java:2524)
	at com.mysql.jdbc.MysqlIO.sqlQueryDirect(MysqlIO.java:2675)
	at com.mysql.jdbc.ConnectionImpl.execSQL(ConnectionImpl.java:2465)
	at com.mysql.jdbc.ConnectionImpl.execSQL(ConnectionImpl.java:2439)
	at com.mysql.jdbc.StatementImpl.executeInternal(StatementImpl.java:829)
	at com.mysql.jdbc.StatementImpl.execute(StatementImpl.java:729)
	at io.prestosql.plugin.jdbc.BaseJdbcClient.execute(BaseJdbcClient.java:810)
	at io.prestosql.plugin.jdbc.BaseJdbcClient.finishInsertTable(BaseJdbcClient.java:567)
	... 20 more

It looks like when executing " insert into target_table select ...",  the temp_table's id(auto_increment) always begins at  ‘1’ but expect to generate max(target_table_id)+1 which can append insert new rows to target table successfully.

Thanks for testing it and sorry, I should have confirmed the case: the target table was already populated.

@getGitxxx Could you share the steps to reproduce (DDL & DML)? I can reproduce it easily, but I would like to make sure the situation just in case.

@ebyhr Thanks reply quickly.
step1: create table SQL
CREATE TABLE `user_tmp` (
  `id` int(11) NOT NULL AUTO_INCREMENT,
  `name` varchar(32) NOT NULL,
  PRIMARY KEY (`id`)
) ENGINE=InnoDB DEFAULT CHARSET=utf8;

step2: execute insert query first time, it works fine.
INSERT INTO test.user_tmp(name)
SELECT
cast(name as varchar(32))
FROM test.user_temp4 LIMIT 10;
---------
INSERT: 10 row

Query 20191223_070031_00003_br3ma, FINISHED, 1 node
Splits: 35 total, 35 done (100.00%)
0:01 [1 rows, 0B] [1 rows/s, 0B/s]

step3: repeat step2 ,will goes fail.
Query 20191223_070221_00005_br3ma failed: Duplicate entry '1' for key 'PRIMARY'

@getGitxxx Thanks! I think we should omit the primary key from INSERT statement to the final table (≠temp table) in this case.

This is likely covered by https://github.com/prestosql/presto/issues/1185

@getGitxxx Thanks! I think we should omit the primary key from INSERT statement to the final table (≠temp table) in this case.

hi, @ebyhr any plan to fix this bug？

@ebyhr I have tried the latest version 329，The issue still exists.

@getGitxxx Can you check with the master branch. It is in under the milestore 330. 330 will be release in a few days I guess.

@Praveen2112 will try it.

@getGitxxx I have tested with the master branch and the above issue is fixed by #2265. Feel free to re-open this issue if the above patch didn't work

Hi @Praveen2112 ,
Sorry for confirming the patch late, It's worked for me. Thanks a lot!


It's stuck here

Installing flarum/flarum (v0.1.0-beta.11)
As there is no 'unzip' command installed zip files are being unpacked using the PHP zip extension.
This may cause invalid reports of corrupted archives. Installing 'unzip' may remediate them.

Installing flarum/flarum (v0.1.0-beta.11): Downloading (100%)
Created project in ./pockethold/download
Loading composer repositories with package information
Updating dependencies (including require-dev)

我放弃安装flarum了 我太笨了

安装fileinfo拓展


custom strings

Greetings.
For devise views I have used devise-i18n gem and I added this section to the view:

<%= t('registration_tips')%>

<%= t('registrations_guide_html')%>

and in the YML file, I did this:
registrations:
 new:
  registration_tips: "some text"
  registration_guide_html: "some text"

But it doesn't work. How can I actually get it to work and display my texts correctly?
Thanks!

Looks like you asked and received an answer on Reddit.


We need a better CFHTLS LRG catalog for position matching

We use CFHTLS red galaxies to decide where to plop down our simulated lenses on sky, by matching nearest neighbors (in redshift and velocity dispersion--although the latter is not for long) between catalogs.
Apparent cuts on the CFHTLS LRG magnitudes in our current reference catalog ($OM10_DIR/data/CFHTLS_LRGs.txt) limit our matching ability on an OM10-PS1 sample to 73%
@drphilmarshall , can you investigate loosening these cuts to expand our CFHTLS reference catalog? You can use the following plot as a guide:

In the future, we might also consider drawing from PS1 itself, or other surveys. See #2

OK, I made an initial foray, trying to select sensible galaxies from CFHTLS (basically the brightest galaxies in each of 6 broadish redshift bins - I checked that we cannot query by color directly). Here's my notebook, in a CFHTLS branch (that I pulled your notes into as well, Mike).
From this I think I conclude that its best to first paint the OM10 lenses with reasonable colors, extrapolating from SDSS (like we do with the quasars), and then go looking for a suitably faint and correctly colored object to position the system. I worry that we won't have enough red objects unless we go looking for them specifically - I might need to download a lot of CFHTLS galaxies at some point soon.

Hey Phil,
The painting script now assigns the colors from the SDSS reference, rather than the magnitudes...sorry for messing up something so simple!
This means you're now good to go to paint a PS1-like sample of lenses using db.paint(), and then find CFHTLS galaxies that match those colors. The painting method now operates on db.sample, so feel free to paint before or after down-sampling!

Sweet! I'll give it a try :-)
On Mon, Jul 20, 2015 at 12:02 PM, Mike Baumer<EMAIL_ADDRESS>wrote:

Hey Phil,
The painting script now assigns the colors from the SDSS reference,
rather than the magnitudes...sorry for messing up something so simple!
This means you're now good to go to paint a PS1-like sample of lenses
using db.paint(), and then find CFHTLS galaxies that match those colors.
The painting method now operates on db.sample, so feel free to paint before
or after down-sampling!
—
Reply to this email directly or view it on GitHub
https://github.com/drphilmarshall/OM10/issues/22#issuecomment-122989849.

Downloading CFHTLS galaxies is getting painful. Let's switch to using SDSS LRGs for the sky positioning - and start using the painted-on lens colors to find a good LRG to use when simulating images.

Try matching on one color; check that others are sensible?

Hmm. I downloaded the SDSS LRGs and checked them in. They're very bright! Mostly brighter than i=19 or so. This is too bright for PS1QLS. So, back to CFHTLS. I think the next thing to do is to use the SDSS-painted OM10 lens colors to define the CFHTLS wget statement, in a download_CFHTLS_LRGs() function. I can work on this while you upgrade the nearest neighbor matching, @mbaumer.

That’s unfortunate, but I can work on the matching algorithm in parallel while you sort out the CFHTLS selection. Thanks for doing all that wget legwork!

On Jul 27, 2015, at 8:33 PM, Phil Marshall<EMAIL_ADDRESS>wrote:
Hmm. I downloaded the SDSS LRGs and checked them in. They're very bright! Mostly brighter than i=19 or so. This is too bright for PS1QLS. So, back to CFHTLS. I think the next thing to do is to use the SDSS-painted OM10 lens colors to define the CFHTLS wget statement, in a download_CFHTLS_LRGs() function. I can work on this while you upgrade the nearest neighbor matching, @mbaumer https://github.com/mbaumer.
—
Reply to this email directly or view it on GitHub https://github.com/drphilmarshall/OM10/issues/22#issuecomment-125319699.

OK, I upgraded the download script and it is running. Can you try it out as well, @mbaumer, and see whether the new CFHTLS_LRG.v2.txt catalog works better in the NN position match? We might still need to go to downloading the data in redshift bins as well as magnitude bins... Thanks!

BTW You'll see from the issue22 notebook that download errors are common - it still might be worth looking for a better way to get all the CFHTLS galaxies we need...


[Bug]: Reset widget doesn't set value to current default value instead sets it to initial default value

Is there an existing issue for this?

[X] I have searched the existing issues

Description
When we reset a widget, currently evaluated values and meta values are not in sync.
For example:-
For the Input widget, If default text = "ABC" and then it was changed to default text = "ABCDEF".
On reset

Input.text = "ABC"
text visible in widget = "ABC"

We expect the widget to reset to its current default values but it resets to initial default values.
Steps To Reproduce

Add an Input widget, Text widget, and Button widget to the canvas.
Bind Text widget = {{Input1.text}}
Button widget onClick = {{resetWidget("Input1")}}
Change default value of input widget and click on button to reset.
Notice, that the input widget doesn’t reset to its default value.

Public Sample App
https://app.appsmith.com/app/resetwidget/page1-62614f3b00cba01cd02f03f5
Steps To Reproduce in Sample App

Type 1 in Input1
Click on the Reset Input2 button
Again Type 1 in Input1
Click on the Reset Input2 button

Repeating this multiple times will showcase cases where

Input2 resets to the current default value visible in Input1 or
Input2 resets to the initial default value which is "ABC"

Ideally, we should always reset the widget to its current default value IMO.
Recording -

Version
1.6.22

@Rishabh-Rathod this seems to be working for me. Can you provide a public example?

@Nikhil-Nandagopal I have added a public sample add, steps to reproduce, and a recording showing the issue.

Stats

Stat
Values

Reach
104

Effort (months)
0.5

@Rishabh-Rathod able to reproduce it now. We really need to fix framework breaks like this

https://github.com/appsmithorg/appsmith/pull/13084 PR fixed this issue also.
This issue is not reproducible on release. I have done verification for it. Closing it.
@ramsaptami could you also verify it.

@Rishabh-Rathod awesome!


--source is broken

The --source option is broken. Output looks like:
src/pydocstyle/violations.py:87 in public method `__lt__`:
	    D105: Missing docstring in magic method

876:     def __lt__(self, other):
	87:     def __lt__(self, other):
886:         return (self.filename, self.line) < (other.filename, other.line)
	88:         return (self.filename, self.line) < (other.filename, other.line)

When it (presumably) should look like:
src/pydocstyle/violations.py:87 in public method `__lt__`:
	    D105: Missing docstring in magic method

87:     def __lt__(self, other):
88:         return (self.filename, self.line) < (other.filename, other.line)

The issue originates in Errors.lines, found here: https://github.com/PyCQA/pydocstyle/blob/master/src/pydocstyle/violations.py#L49
It looks like some code may have been committed accidentally before it was complete.

Fixed by #260 .


Update risk level accordingly with ASC log values

When there is no user level risk, it displays value None on ASC logs

#sign-off


Complete ContractWrapper for Exchange

Expected Behavior
Complete ContractWrapper for Exchange

@m-chrzan can you describe what solidity methods should we port.. and or any particular interface you see fit

Definitely want:
exchange
getBuyTokenAmount
getSellTokenAmount
getBuyAndSellBuckets

One thing to consider is whether we want to expose these as they are in the contract, with a sellGold flag passed in, or each in two versions, like sellGold() and sellDollar() instead of exchange().
Whether we want to expose onlyOwner functions is an open question - it could potentially be useful for people playing around with a local network? But other than that, they pollute the namespace.


Links to organization site are not working

Descriptions
Something went wrong when I use organization and organizations in your service links.
Some examples use different ways and I've checked it out:
organization
Link is wrong
https://opencollective.com/aiogram/organization/0/website
Image is OK (but have a bad size #2668)
https://opencollective.com/aiogram/organization/0/avatar
organizations
Link is OK
https://opencollective.com/aiogram/organizations/0/website
Image is OK (but have a bad size #2668)
https://opencollective.com/aiogram/organizations/0/avatar

Looks like the link is correct now. Maybe some temporary cache issue?

Not it works fine


add: Galgotias College of Engineering and Technology

@areeburrub Thank you for your patience while waiting for a review and processing your request. Pull request merged. Please start applying for JetBrains educational subscription at https://www.jetbrains.com/shop/eform/students in an hour, to allow the changes to propagate through JetBrains systems.


There is no step "Choose organization" when changing deployment source to "GitHub"

OS: All
Build Version: 20230629.1
Regression: Previous Release
Repro Steps:

Right click a function app -> Click "Configure Deployment Source..." -> Click "GitHub".
Check whether the next step is "Choose organization".

Expect:
The next step is "Choose organization".

Actual:
Pop up a notification of successful update to "GitHub", but doesn't allow to choose organization.
More Info:

The "GitHub" label doesn't appear next to "Deployments" node after changing deployment source to "GitHub".
The "Current source" is "None" when executing "Configure Deployment Source..." command again.

Related to https://github.com/microsoft/vscode-azuretools/pull/1450


fontbakery-build-fontmetadata.py fails if it can't find a pre-existing images directory

We should check if the image folder exists and, if not, create it.
felipe@guarana:~/devel/github_felipesanches/fontbakery$ python test_args.py
Traceback (most recent call last):
  File "test_args.py", line 29, in <module>
    target = __import__(module_name)
  File "/home/felipe/devel/github_felipesanches/fontbakery/fontbakery-build-fontmetadata.py", line 143, in <module>
    generate_italic_angle_images()
  File "/home/felipe/devel/github_felipesanches/fontbakery/fontbakery-build-fontmetadata.py", line 141, in generate_italic_angle_images
    im.save(filepath, "PNG")
  File "/usr/lib/python2.7/dist-packages/PIL/Image.py", line 1462, in save
    fp = builtins.open(fp, "wb")
IOError: [Errno 2] No such file or directory: '/home/felipe/devel/github_felipesanches/fontbakery/fontmetadata_tool/images/angle_1.png'

https://github.com/googlefonts/fontbakery/issues/922 depends on fixing this.

fontbakery-build-fontmetadata.py is no longer actively needed; don't delete
it but don't work on it :)

well... I just fixed it. I'll merge the fix and leave it alono then :-)


Use AKV extension instead of the --secrets option

I understand that the recommended way to download certificates to virtual machines is using the Azure Key Vault extension. IMHO it should be the method used in this article, or at least referred to for users reading this.
FWIW, I have written a blog article on how to deploy nginx with a certificate downloaded via the AKV extension here, happy to enhance this article with the AKV extension if you agree to that.

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

ID: 996eb4d1-198e-a69a-b9c5-d247db46d025
Version Independent ID: 5bfe27b3-22e9-efa8-6a1c-3637aa39a049
Content: Tutorial: Secure a web server with TLS/SSL certificates - Azure Virtual Machines
Content Source: articles/virtual-machines/linux/tutorial-secure-web-server.md
Service: virtual-machines
GitHub Login: @mattmcinnes
Microsoft Alias: mattmcinnes

@erjosito
Thanks for your feedback! We will investigate and update as appropriate.

@erjosito
Thanks for your feedback! I've assigned this issue to the author who will investigate and update as appropriate.

Working to resolve this. Sent messages to relevant PMs last week and meeting with them to discuss.

Thanks for your contribution to our documentation
We are moving to a new user feedback system. As part of that effort, we are creating a content request in our backlog to address this gap and are closing this GitHub issue. The updated content will be published shortly.

Please continue to provide feedback about the documentation. We appreciate your contributions to our community.
#please-close


refactor: Remove unnecessary use of anyhow

Single line change. No need for custom error type if we just return a string anyway.

Whoops, just noticed I "assigned" you instead of "requesting review", @Alystrasz
Sorry about that ^^"


Closed schools show close date during search

When a school changes some of its attributes (name, status, etc) it is closed and then re-opened with the new attributes in the Get information about schools data.
We want to show this close date on the claim school and current school search pages so a user can make an informed desicion about which school to select in cases where there are duplicates or the when the school is actually closed.
This is an iterim solution whilst we work on the right way to approach these 'closed' schools.
Handing this off to @titlescreen to look at the display of the close date in the UI.

 How do we deal with closed/renamed schools?

 How do we deal with closed/renamed schools?


Update Guide to use oh-my-posh v3

oh-my-posh v3 has been out for some time now and is cross platform. This means a single installation can provided powerline glyphs in Windows & WSL at the same time.
Can we update the tutorial docs with this new version?
The main difference would be that it would have to be installed via winget/scoop to get the oh-my-posh-wsl executable installed in wsl.
[Enter feedback here]

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

ID: d3cbeb9a-9e7c-f8f4-5c03-5d6262d2b440
Version Independent ID: eb575c79-5b2a-45f6-646f-9d41d5d223e8
Content: Windows Terminal Powerline Setup
Content Source: TerminalDocs/tutorials/powerline-setup.md
Product: dev-environment
Technology: windows-terminal
GitHub Login: @cinnamon-msft
Microsoft Alias: cinnamon

Closing this as we did some major updates to the command prompt customizing tutorial, including the update to Oh My Posh 3: https://docs.microsoft.com/windows/terminal/tutorials/custom-prompt-setup
Please reopen or let me know if you see any issues and thanks for the contribution!


Handle thunks

This forces all the thunks (created with Html.Lazy) so that later elm-html-in-elm can parse the forced node values out of the stringified JSON. This follows the pattern used in the real rendering code: https://github.com/elm-lang/virtual-dom/blob/master/src/Native/VirtualDom.js#L293-L359

This, along with eeue56/elm-html-in-elm#2, is a possible solution to #4.
This adds tests to @avh4's PR (commit by av is included)
💭 I wonder if we should call .thunk with it's arguments. It works without that change though.
                    vNode.node = vNode.thunk.apply(vNode.thunk, vNode.args);
This replaces #5

@eeue56 let me know if there is anything that I need to change.

I also refactored forceThunks, I hope that looks good.

@stoeffel one comment, will merge after that

cool. I can just revert me last commit, it's a lot more code to maintain
though. Meaning more surface for bugs to sneak in. your call, I happy
either way.
On Sat, Feb 25, 2017, 09:27 Noah<EMAIL_ADDRESS>wrote:

@stoeffel https://github.com/stoeffel one comment, will merge after that
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/eeue56/elm-html-test/pull/7#issuecomment-282498547,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABKUka_Kj8tLpGItdXC-NNO-sy9EziHfks5rgGSWgaJpZM4MKszM
.

Can't it just be applied in a seperate pass from stringify? I'd rather this version to the old, but I'm worried stringify will not behave properly

you are right second pass should work. will do it tomorrow or on the flight
back to Switzerland.
On Sat, Feb 25, 2017, 16:55 Noah<EMAIL_ADDRESS>wrote:

Can't it just be applied in a seperate pass from stringify? I'd rather
this version to the old, but I'm worried stringify will not behave properly
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/eeue56/elm-html-test/pull/7#issuecomment-282524130,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABKUkV2bsZ2cVfKBIP3TIAOSdOos181Hks5rgM18gaJpZM4MKszM
.

will do it tomorrow or on the flight
back to Switzerland.

actually already did it 😆 . SFO has wifi 🎉 .
@eeue56 What do you think? Better?

Better 👍

@stoeffel I added you as a collaborator, do you want to merge + publish?

thanks! will do

published in 1.0.1


Output path of current Prisma in -v

Problem
This is a situation that can happen:
--- yarn prisma -v ---
warning package.json: No license field
prisma               : 2.26.0-dev.7
@prisma/client       : 2.26.0-dev.7
Current platform     : debian-openssl-1.1.x
Query Engine         : query-engine c838e79f39885bc8e1611849b1eb28b5bb5bc922 (at<EMAIL_ADDRESS>Migration Engine     : migration-engine-cli c838e79f39885bc8e1611849b1eb28b5bb5bc922 (at<EMAIL_ADDRESS>Introspection Engine : introspection-core c838e79f39885bc8e1611849b1eb28b5bb5bc922 (at<EMAIL_ADDRESS>Format Binary        : prisma-fmt c838e79f39885bc8e1611849b1eb28b5bb5bc922 (at<EMAIL_ADDRESS>Default Engines Hash : c838e79f39885bc8e1611849b1eb28b5bb5bc922
Studio               : 0.402.0
--- ls node_modules/@prisma/engines/ ---
ls: cannot access 'node_modules/@prisma/engines/': No such file or directory

As you can see yarn prisma -v tells us the engines comes from node_modules/@prisma/engines, but when you try to list that directory, it is actually not present.
What is happening here is that yarn prisma is actually from a few folders above our project. This is kinda visible when you look to not run yarn -s but just yarn:
codespace ➜ /workspaces/e2e-tests/platforms-serverless/firebase-functions (janpio-patch-5 ✗) $ yarn prisma -v
yarn run v1.22.10
warning package.json: No license field
$ /workspaces/e2e-tests/node_modules/.bin/prisma -v
prisma               : 2.25.0-dev.36
@prisma/client       : 2.25.0-dev.36
...

Suggested solution
Output the directory of where the executed prisma is from, either similar to Yarn the absolute path to the binary in node_modules, or the folder where the package.json that is responsible for the installation of Prisma comes from.
Alternatives
Rely on yarn itself to give that information. Will not work with npx unfortunately for example.

/attempt #7771

Is this issue still open? /attempt #7771


ASP.NET Core RTM 1.0.0

I see the sample for ASP.NET 5.0 which was in RC at the time. Are there any plans to update this to the official ASP.NET Core 1.0.0 RTM release at all?

Duh - my bad - just spotted https://github.com/IdentityServer/IdentityServer4


Focus on global text search

Tested versions
Godot 4.2.2 Stable
System information
Nobara Linux 40
Issue description
When i launch the global search (CTRL + SHIFT + F) to search for text within the project, a popup window is opened. It lloks like the popup window gets the focus but the focus doesn't go to the search textfield. This is annoying as the only way to search (as far as I know) is to manually set the focus by clicking with the mouse on the search textfield.
I find myself often starting typing and realising I'm not writing.
Nothing major but it is a bit annoying.
Many thanks in avance for your help!
Fed
Steps to reproduce
Presse CTRL + SHIFT + F
Start typing
=> you are not typing any text
Minimal reproduction project (MRP)
Any project with at least a script in it

This seems like it could be a good first issue. I would like to try fixing this as a first issue if that would be okay?

Thank you for reporting, consolidating in:

https://github.com/godotengine/godot/issues/88847

See there for more details, if you think something was missed about this and it's not the same issue, please comment here and it can be reopened
This has already been fixed, please check 4.3.rc1 to confirm

The ticket looks good. Thank you for the very prompt reply.


Initial implementation of DaphneSerializer

This PR is about implementing Daphne's serializer.

Added DaphneSerializer.
DaphneSerializer provides 3 static functions, save, load and length, used for serializing Daphne objects into byte streams (and vice versa).
Current support for DenseMatrix and CSRMatrix (Frame not implemented).
Updated ReadDaphne and WriteDaphne to use DaphneSerializer.
Updated distributed kernels and distributed gRPC worker to serialize objects with DaphneSerializer and use protobufs to transfer bytes only, instead of complex protobuf messages.
Removed ProtoDataConverter.

Issue for discussion:

DaphneSerializer is a templated class, with static methods. Distributed kernels need to downcast from Structure* to Dense/CSR<int/double> matrices before serializing an object, leading to an if-else chain using dynamic_cast. I believe that this could be avoided with virtual functions, i.e. instead of serializing an object in a seperate (static) templated class, we could provide virtual functions for Dense/CSR/Frames that serialize an object. Feel free to comment and provide alternative ideas/suggestions.

Related issues #465.

Thanks for this contribution @aristotelis96. I will look into it and will provide feedback today or tomorrow.

FYI: This PR contributes to #103. We should mention it in the commit message once we merge this PR.

Thanks @pdamme for your review and feedback.
Regarding the problem of templates:
Distributed runtime kernels (distribute, broadcast, etc.) are called from DistributedWrapper, which is initialized and called from DistributedPipeline.h kernel. Distributed kernels are templated, however DistributedWrapper only uses templates for the pipeline output, whereas each pipeline input is treated as a Structure pointer. Therefore the template parameter passed to the distributed kernel (and by extension the serializer) would be Structure (instead of DenseMatrix/CSRMatrix). For now it is not clear to me how we could provide the template parameter for the pipeline inputs, especially since we want to use Structure pointers in case there are different data types of pipeline inputs.
An option would be to create a specialization of DaphneSerializer for Structure where we could hide the dynamic_cast chain there, but this shifts/hides the issue instead of actually solving it.
Let me know what you think.

I've updated DaphneSerializer and included a few additional implementations used for (de)serializing in chunks. One implementation for in order serialization using iterators and one for out of order.

DaphneDe/SerializerChunks : Used for (de)serializing in order using an iterator. Example use:

auto ser = DaphneSerializerChunks<DT>(mat, chunkSize);
size_t idx = 0;
for (auto it = ser.begin(); it != ser.end(); ++it){
std::copy(it->second.begin(), it->second.begin() + it->first, tempBuff.begin() + idx);
    idx += it->first;
}
DT* newMat = nullptr;
idx = 0;
auto deser = DaphneDeserializerChunks<DT>(&newMat, chunkSize);
for (auto it = deser.begin(); it != deser.end(); ++it){
    size_t chnck = std::min(size_t(chunkSize), tempBuff.capacity() - idx);
    std::copy(tempBuff.begin() + idx, tempBuff.begin() + idx + chnck, it->second.begin());
    it->first = chnck;
    idx += chnck;
}

DaphneDe/SerializerOutOfOrderChunks. Used for (de)serialiazing in chunks, out of order. The chunks produced are identical to the previous implementation but contain an additional header used to identify the index of the chunk. Example use:

std::vector<std::vector<char>> message;

DaphneSerializerOutOfOrderChunks<DT> serializer(mat, chunkSize);
while (serializer.HasNextChunk())
{
  std::vector<char> bufferTmp(200);
  serializer.SerializeNextChunk(bufferTmp);
  // Push to vector of messages
  message.push_back(bufferTmp);
}

// Shuffle vector of messages
std::shuffle(std::begin(message), std::end(message), rng);

// Deserialize
DaphneDeserializerOutOfOrderChunks<DT> deserializer;
size_t i = 0;
DT *res = nullptr;
// We want to iterate using HasNextChunk() method
while (deserializer.HasNextChunk())
{
  // Deserialize message
  res = deserializer.DeserializeNextChunk(message[i++]);
}

Updated ReadDaphne and WriteDaphne kernels to serialiaze an object and read/write in chunks, instead of using one big buffer.
TODO: In both Read/WriteDaphne kernels specializations for DenseMatrix and CSRMatrix are the same. Could we provide a single implementation for both of them? Instead of deleteing the default implementation we could implement that, however I am not sure if we want to allow calling Read/WriteDaphne with other objects (e.g. Matrix, Structure).

Yes, that sounds good to me. Thank you @pdamme
for reviewing this PR!


Comparator support for the Smelter

Suggestion
It would be useful if Smelters would output a value indicating how many items are in the input slot
Current Behavior [Optional]
Currently, Smelters output a value of 0 always
Justification
It would be nice to be able to control an indicator (lamp or noteblock) for when a Smelter is done smelting stuff, but there is a possible balance concern (depending on how the heat system is supposed to be worked) of being able to automate when heat will be provided to a smelter, which makes heat systems much more efficient.

I'm going to expand on this issue: A lot of machines should have comparator/circuit support.

Smelters for items in input
Millstone for items in input
Blast furnace for items in input
Stamp mill for either items in input, or accumulated progress
[Clockwork] Beam stabilizer for stored beam power
Tesla coil base for stored FE
Ore cleanser for items in input
Steamer for items in input
Ice/firebox for either burntime or items in input
Voltus generator for stored voltus


updated from 0.3.1 to 0.5.1 lots of issues

is there any way that i can get my hands on the plugin 0.3.1 ? since I've upgraded it to the latest issue, the quality lowered drastically, and I mean by a mile, It's havind difficulties printing since it seems that the plugin manages way more the printer than it used to. It also looks like there was way too much pressure applied while printing QR codes or big characters. see attached picutre: on top is the 0.5.1 ones, below them, its the ones generated in 0.3.1

Hi,
You are free to download and install 0.3.0 from releases.
The major difference in functionality between 0.3.0 and 0.5.1 is the new initial sequence in the printer settings that allows you tu
setup the printer. There is a default setting that fits fine for my printer. It might be different for your printer. So please have a look there.
If the print gets too dark you can reduce the darkness value in the settings. Hope that helps.
Michael

is there any way to have 0.3.1 back man?

the dot per mm optuion does not seem usable.

ok, i figured it all out. it's all working good now. I got another question tho, is there any way to get multiple printers to work with the plugin?

Kaum macht mans richtig... :-)
Nice to hear that the plugin works fine for you now. Multiple printers cannot be used withthis plugin. The
inventree interface is not really made for this. The new machine interface available in 0.14.0 is made for this.
In #6  is already a request for this plugin to use the new interface but I did not start this so far.

Is there any way to tweak it so that i can install the plugin twice as if
the environment would think this is a different plugin?
On Mon, Mar 4, 2024 at 1:12 PM Michael @.***> wrote:

Kaum macht mans richtig... :-)
Nice to hear that the plugin works fine for you now. Multiple printers
cannot be used withthis plugin. The
inventree interface is not really made for this. The new machine interface
available in 0.14.0 is made for this.
In #6 https://github.com/SergeoLacruz/inventree-zebra-plugin/issues/6
is already a request for this plugin to use the new interface but I did not
start this so far.
—
Reply to this email directly, view it on GitHub
https://github.com/SergeoLacruz/inventree-zebra-plugin/issues/7#issuecomment-1977179885,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOXZLLTNZSLBWALKLF3DSWLYWS2QLAVCNFSM6AAAAABDVOMPAOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNZXGE3TSOBYGU
.
You are receiving this because you authored the thread.Message ID:
@.***>

I think we can close this.


Adding event on disabled items

i think adding event on disabled items developers may need it (i need it 😸), for exemple if item is disable and user click on it may i need to show a toast message like "This feature is coming soon" or "This feature are availble only for 18+ age"... what do you think ?

Cool idea !
I'll put it into working


Write csr verification

@CharlyCst I solved at the same time a csrRead (which was only adding the LCOFIE filter in one of the csr register)


Freeze on infiniband with slurm

System Info
`Accelerate` version: 0.19.0
- Platform: Linux-4.18.0-425.13.1.el8_7.x86_64-x86_64-with-glibc2.28
- Python version: 3.10.4
- Numpy version: 1.22.3
- PyTorch version (GPU?): 1.12.0 (False)
- System RAM: 503.49 GB
- `Accelerate` default config:
	Not found

Using the command-line parameters:

srun bash -c 'accelerate launch \
    --main_process_ip $MASTER_ADDR \
    --main_process_port $MASTER_PORT \
    --multi_gpu \
    --mixed_precision=no \
    --num_processes=$(($NNODES * 4)) \
    --dynamo_backend=no \
    --num_machines=$NNODES  \
    --machine_rank=$SLURM_NODEID \
    --rdzv_conf "rdzv_endpoint=$MASTER_ADDR:$MASTER_PORT,rdzv_backend=c10d" \
    distrib.py'

Information

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

Tasks

[ ] One of the scripts in the examples/ folder of Accelerate or an officially supported no_trainer script in the examples folder of the transformers repo (such as run_no_trainer_glue.py)
[ ] My own task or dataset (give details below)

Reproduction
Fails with multi-node.
This machine has the infiniband interfaces suffixed with i, so a compute node responds to hostname with something like juwels07 but the right interface is juwels07i. There's some script magic for that on the script.
Slurm launch is:
#!/bin/bash -x
#SBATCH --account=training2306
#SBATCH --nodes=2
#SBATCH --job-name=ai-multi-gpu
#SBATCH --ntasks-per-node=1
#SBATCH --cpus-per-task=48
#SBATCH --output=out-distrib.%j
#SBATCH --error=err-distrib.%j
#SBATCH --time=00:20:00
#SBATCH --partition=booster
#SBATCH --gres=gpu:4

# srun doesnot inherit cpus-per-task from sbatch
export SRUN_CPUS_PER_TASK=${SLURM_CPUS_PER_TASK}
# so processes know who to talk to
MASTER_ADDR="$(scontrol show hostnames "$SLURM_JOB_NODELIST" | head -n 1)"
# Allow communication over InfiniBand cells.
MASTER_ADDR="${MASTER_ADDR}i"
# Get IP for hostname.
export MASTER_ADDR="$(nslookup "$MASTER_ADDR" | grep -oP '(?<=Address: ).*')"

export MASTER_PORT=7010
export GPUS_PER_NODE=4
export NNODES=$SLURM_JOB_NUM_NODES
# do not remove or the training will hang and nodes will be lost w/o this workaround
export CUDA_LAUNCH_BLOCKING=1

# hide duplicated errors using this hack - will be properly fixed in pt-1.12
export TORCHELASTIC_ERROR_FILE=/tmp/torch-elastic-error.json

# force crashing on nccl issues like hanging broadcast
export NCCL_ASYNC_ERROR_HANDLING=1

# handle timeouts
export NCCL_IB_TIMEOUT=20

# Make sure we are on the right directory
cd $HOME/2023-may-intro-to-supercompting-jsc/src

# This loads modules and python packages
source sc_venv_template/activate.sh

export LOGLEVEL=INFO
# Run the demo
time srun bash -c 'accelerate launch \
    --main_process_ip $MASTER_ADDR \
    --main_process_port $MASTER_PORT \
    --multi_gpu \
    --mixed_precision=no \
    --num_processes=$(($NNODES * 4)) \
    --dynamo_backend=no \
    --num_machines=$NNODES  \
    --machine_rank=$SLURM_PROCID \
    --rdzv_conf "rdzv_endpoint=$MASTER_ADDR:$MASTER_PORT rdzv_backend=c10d" \
    distrib.py'

This is the error output, and it stays like this until the job times out (the identical job but with only one node works):
+
INFO:torch.distributed.launcher.api:Starting elastic_operator with launch configs:
  entrypoint       : distrib.py
  min_nodes        : 2
  max_nodes        : 2
  nproc_per_node   : 4
  run_id           : none
  rdzv_backend     : static
  rdzv_endpoint    : <IP_ADDRESS>:7010
  rdzv_configs     : {'rdzv_endpoint': '<IP_ADDRESS>:7010', 'rdzv_backend': 'c10d', 'rank': 1, 'timeout': 900}
  max_restarts     : 0
  monitor_interval : 5
  log_dir          : None
  metrics_cfg      : {}

INFO:torch.distributed.launcher.api:Starting elastic_operator with launch configs:
  entrypoint       : distrib.py
  min_nodes        : 2
  max_nodes        : 2
  nproc_per_node   : 4
  run_id           : none
  rdzv_backend     : static
  rdzv_endpoint    : <IP_ADDRESS>:7010
  rdzv_configs     : {'rdzv_endpoint': '<IP_ADDRESS>:7010', 'rdzv_backend': 'c10d', 'rank': 0, 'timeout': 900}
  max_restarts     : 0
  monitor_interval : 5
  log_dir          : None
  metrics_cfg      : {}

INFO:torch.distributed.elastic.agent.server.local_elastic_agent:log directory set to: /tmp/torchelastic__7nlgv08/none_jscf2i4f
INFO:torch.distributed.elastic.agent.server.api:[default] starting workers for entrypoint: python
INFO:torch.distributed.elastic.agent.server.api:[default] Rendezvous'ing worker group
[W socket.cpp:558] [c10d] The client socket cannot be initialized to connect to [jwb0092i.juwels]:7010 (errno: 97 - Address family not supported by protocol).
[W socket.cpp:558] [c10d] The client socket cannot be initialized to connect to [jwb0092i.juwels]:7010 (errno: 97 - Address family not supported by protocol).
INFO:torch.distributed.elastic.agent.server.api:[default] Rendezvous complete for workers. Result:
  restart_count=0
  master_addr=<IP_ADDRESS>
  master_port=7010
  group_rank=1
  group_world_size=2
  local_ranks=[0, 1, 2, 3]
  role_ranks=[4, 5, 6, 7]
  global_ranks=[4, 5, 6, 7]
  role_world_sizes=[8, 8, 8, 8]
  global_world_sizes=[8, 8, 8, 8]

INFO:torch.distributed.elastic.agent.server.api:[default] Starting worker group
INFO:torch.distributed.elastic.multiprocessing:Setting worker0 reply file to: /tmp/torchelastic__7nlgv08/none_jscf2i4f/attempt_0/0/error.json
INFO:torch.distributed.elastic.multiprocessing:Setting worker1 reply file to: /tmp/torchelastic__7nlgv08/none_jscf2i4f/attempt_0/1/error.json
INFO:torch.distributed.elastic.multiprocessing:Setting worker2 reply file to: /tmp/torchelastic__7nlgv08/none_jscf2i4f/attempt_0/2/error.json
INFO:torch.distributed.elastic.multiprocessing:Setting worker3 reply file to: /tmp/torchelastic__7nlgv08/none_jscf2i4f/attempt_0/3/error.json
INFO:torch.distributed.elastic.agent.server.local_elastic_agent:log directory set to: /tmp/torchelastic_j88hm8vm/none_tgket3up
INFO:torch.distributed.elastic.agent.server.api:[default] starting workers for entrypoint: python
INFO:torch.distributed.elastic.agent.server.api:[default] Rendezvous'ing worker group
[W socket.cpp:401] [c10d] The server socket cannot be initialized on [::]:7010 (errno: 97 - Address family not supported by protocol).
[W socket.cpp:558] [c10d] The client socket cannot be initialized to connect to [jwb0092i.juwels]:7010 (errno: 97 - Address family not supported by protocol).
[W socket.cpp:558] [c10d] The client socket cannot be initialized to connect to [jwb0092i.juwels]:7010 (errno: 97 - Address family not supported by protocol).
INFO:torch.distributed.elastic.agent.server.api:[default] Rendezvous complete for workers. Result:
  restart_count=0
  master_addr=<IP_ADDRESS>
  master_port=7010
  group_rank=0
  group_world_size=2
  local_ranks=[0, 1, 2, 3]
  role_ranks=[0, 1, 2, 3]
  global_ranks=[0, 1, 2, 3]
  role_world_sizes=[8, 8, 8, 8]
  global_world_sizes=[8, 8, 8, 8]

INFO:torch.distributed.elastic.agent.server.api:[default] Starting worker group
INFO:torch.distributed.elastic.multiprocessing:Setting worker0 reply file to: /tmp/torchelastic_j88hm8vm/none_tgket3up/attempt_0/0/error.json
INFO:torch.distributed.elastic.multiprocessing:Setting worker1 reply file to: /tmp/torchelastic_j88hm8vm/none_tgket3up/attempt_0/1/error.json
INFO:torch.distributed.elastic.multiprocessing:Setting worker2 reply file to: /tmp/torchelastic_j88hm8vm/none_tgket3up/attempt_0/2/error.json
INFO:torch.distributed.elastic.multiprocessing:Setting worker3 reply file to: /tmp/torchelastic_j88hm8vm/none_tgket3up/attempt_0/3/error.json
[W socket.cpp:558] [c10d] The client socket cannot be initialized to connect to [jwb0092i.juwels]:7010 (errno: 97 - Address family not supported by protocol).
[W socket.cpp:558] [c10d] The client socket cannot be initialized to connect to [jwb0092i.juwels]:7010 (errno: 97 - Address family not supported by protocol).
[W socket.cpp:558] [c10d] The client socket cannot be initialized to connect to [jwb0092i.juwels]:7010 (errno: 97 - Address family not supported by protocol).
[W socket.cpp:558] [c10d] The client socket cannot be initialized to connect to [jwb0092i.juwels]:7010 (errno: 97 - Address family not supported by protocol).
[W socket.cpp:558] [c10d] The client socket cannot be initialized to connect to [jwb0092i.juwels]:7010 (errno: 97 - Address family not supported by protocol).
[W socket.cpp:558] [c10d] The client socket cannot be initialized to connect to [jwb0092i.juwels]:7010 (errno: 97 - Address family not supported by protocol).
[W socket.cpp:558] [c10d] The client socket cannot be initialized to connect to [jwb0092i.juwels]:7010 (errno: 97 - Address family not supported by protocol).
[W socket.cpp:558] [c10d] The client socket cannot be initialized to connect to [jwb0092i.juwels]:7010 (errno: 97 - Address family not supported by protocol).
[W socket.cpp:558] [c10d] The client socket cannot be initialized to connect to [jwb0092i.juwels]:7010 (errno: 97 - Address family not supported by protocol).
[W socket.cpp:558] [c10d] The client socket cannot be initialized to connect to [jwb0092i.juwels]:7010 (errno: 97 - Address family not supported by protocol).
[W socket.cpp:558] [c10d] The client socket cannot be initialized to connect to [jwb0092i.juwels]:7010 (errno: 97 - Address family not supported by protocol).
[W socket.cpp:558] [c10d] The client socket cannot be initialized to connect to [jwb0092i.juwels]:7010 (errno: 97 - Address family not supported by protocol).
[W socket.cpp:558] [c10d] The client socket cannot be initialized to connect to [jwb0092i.juwels]:7010 (errno: 97 - Address family not supported by protocol).
[W socket.cpp:558] [c10d] The client socket cannot be initialized to connect to [jwb0092i.juwels]:7010 (errno: 97 - Address family not supported by protocol).
[W socket.cpp:558] [c10d] The client socket cannot be initialized to connect to [jwb0092i.juwels]:7010 (errno: 97 - Address family not supported by protocol).
[W socket.cpp:558] [c10d] The client socket cannot be initialized to connect to [jwb0092i.juwels]:7010 (errno: 97 - Address family not supported by protocol).

This is the tail of the NCCL_DEBUG output:
jwb0093:11954:12012 [2] NCCL INFO threadThresholds 8/8/64 | 64/8/64 | 512 | 512
jwb0093:11954:12012 [2] NCCL INFO 8 coll channels, 8 p2p channels, 2 p2p channels per peer
jwb0093:11953:12015 [1] NCCL INFO Connected all trees
jwb0093:11953:12015 [1] NCCL INFO threadThresholds 8/8/64 | 64/8/64 | 512 | 512
jwb0093:11953:12015 [1] NCCL INFO 8 coll channels, 8 p2p channels, 2 p2p channels per peer
jwb0092:26778:26839 [2] NCCL INFO comm 0x3792dff0 rank 2 nranks 8 cudaDev 2 busId 84000 - Init COMPLETE
jwb0092:26776:26834 [0] NCCL INFO comm 0x37fb73a0 rank 0 nranks 8 cudaDev 0 busId 3000 - Init COMPLETE
jwb0092:26777:26840 [1] NCCL INFO comm 0x37e42e20 rank 1 nranks 8 cudaDev 1 busId 44000 - Init COMPLETE
jwb0093:11955:12013 [3] NCCL INFO comm 0x36e7c930 rank 7 nranks 8 cudaDev 3 busId c4000 - Init COMPLETE
jwb0092:26779:26841 [3] NCCL INFO comm 0x38ee4b30 rank 3 nranks 8 cudaDev 3 busId c4000 - Init COMPLETE
jwb0093:11953:12015 [1] NCCL INFO comm 0x379b7bc0 rank 5 nranks 8 cudaDev 1 busId 44000 - Init COMPLETE
jwb0093:11952:12014 [0] NCCL INFO comm 0x36f49380 rank 4 nranks 8 cudaDev 0 busId 3000 - Init COMPLETE
jwb0093:11954:12012 [2] NCCL INFO comm 0x36be8130 rank 6 nranks 8 cudaDev 2 busId 84000 - Init COMPLETE

The full error and output are here https://gist.github.com/surak/5f3f236616e5db48f19d31df457b4350
Expected behavior
A similar script works in an ethernet cluster. I would like to see what is actually frozen, but there's no output other than that.

Update:
Turns out that there's something wrong with processing of the command line.
If I create a YAML file with the same parameters per node, I get it to run just fine:
So, this works:
#!/bin/bash
#SBATCH --account=training2306
#SBATCH --nodes=2
#SBATCH --job-name=ai-multi-gpu
#SBATCH --ntasks-per-node=1
#SBATCH --cpus-per-task=48
#SBATCH --output=out-distrib.%j
#SBATCH --error=err-distrib.%j
#SBATCH --time=00:20:00
#SBATCH --partition=booster
#SBATCH --gres=gpu:4

# Without this, srun does not inherit cpus-per-task from sbatch.
export SRUN_CPUS_PER_TASK="$SLURM_CPUS_PER_TASK"

# so processes know who to talk to
MASTER_ADDR="$(scontrol show hostnames "$SLURM_JOB_NODELIST" | head -n 1)"
# Allow communication over InfiniBand cells.
MASTER_ADDR="${MASTER_ADDR}i"
# Get IP for hostname.
export MASTER_ADDR="$(nslookup "$MASTER_ADDR" | grep -oP '(?<=Address: ).*')"

export MASTER_PORT=7010

export GPUS_PER_NODE=4

# Make sure we are on the right directory
cd $HOME/2023-may-intro-to-supercompting-jsc/src

# This loads modules and python packages
source sc_venv_template/activate.sh

# Set up accelerate config.
export ACCELERATE_CONFIG_YAML=accelerate_config_"$SLURM_JOB_ID".yaml
srun bash -c "((\$SLURM_PROCID)) || cat <<EOT > \"\$ACCELERATE_CONFIG_YAML\"
compute_environment: LOCAL_MACHINE
distributed_type: MULTI_GPU
downcast_bf16: 'no'
gpu_ids: all
machine_rank: \$SLURM_NODEID
main_process_ip: '\$MASTER_ADDR'
main_process_port: \$MASTER_PORT
main_training_function: main
mixed_precision: 'no'
num_machines: \$SLURM_JOB_NUM_NODES
num_processes: \$((SLURM_JOB_NUM_NODES * GPUS_PER_NODE))
rdzv_backend: c10d
same_network: false
tpu_env: []
tpu_use_cluster: false
tpu_use_sudo: false
use_cpu: false
EOT"

# Run the demo
time srun bash -c 'accelerate launch \
    --config_file=$ACCELERATE_CONFIG_YAML \
    distrib.py'

cc @muellerzr for the CLI

The issue is that the command-line settings don't set properly the rdvz_backend (which defaults to static), while the yaml file does.

Thanks @muellerzr for the quick response, this was really lacking from accelerate. :)
There were other issues regarding --rdzv_backend that were closed (e.g. https://github.com/huggingface/accelerate/pull/1337), but it makes total sense to have this parameter.

Np @janEbert! It was on my list to get to, with how our CLI works as long as our argparser knows the param to use, that is usually all it takes to get it up and going (you can see literally I added a single line in the PR here: https://github.com/huggingface/accelerate/pull/1490)

I tried with
branch 'rdzv-endpoint' set up to track 'origin/rdzv-endpoint'.

This is the log when using the generated yaml file described above, here. This worked:
  entrypoint       : distrib.py
  min_nodes        : 2
  max_nodes        : 2
  nproc_per_node   : 4
  run_id           : none
  rdzv_backend     : static
  rdzv_endpoint    : <IP_ADDRESS>:7010
  rdzv_configs     : {'rank': 0, 'timeout': 900}
  max_restarts     : 0
  monitor_interval : 5
  log_dir          : None
  metrics_cfg      : {}

And this is with the command-line parameters above. This DOES NOT work:
  entrypoint       : false
  min_nodes        : 2
  max_nodes        : 2
  nproc_per_node   : 4
  run_id           : none
  rdzv_backend     : c10d
  rdzv_endpoint    :
  rdzv_configs     : {'timeout': 900}
  max_restarts     : 0
  monitor_interval : 5
  log_dir          : None
  metrics_cfg      : {}

Two things:

the virtual environment with the new branch broke functionality with the working yaml script, for whatever reason.
shouldn't the rdzv_endpoint, config and entry point settings be filled properly by the command-line settings as it is by the YAML file?

What's really happening here I believe is the new version fixed a broken issue, aka c10d is your issue for your timeout, your working version has always been using static because Accelerate wasn't actually using it.
To test our theory, run your code with raw torchrun please 😄 :
srun bash -c 'torchrun \
    --master_addr $MASTER_ADDR \
    --master_port $MASTER_PORT \
    --nproc_per_node=4 \
    --nnodes=$NNODES  \
    --rdzv_backend c10d \
    distrib.py'

This should fail, and only work when you use static as the backend.
srun bash -c 'torchrun \
    --master_addr $MASTER_ADDR \
    --master_port $MASTER_PORT \
    --nproc_per_node=4 \
    --nnodes=$NNODES  \
    --rdzv_backend static \
    distrib.py'

--same_backend is an argument with action=store_true. If it is specified, rdzv_endpoint will not be set.

@janEbert however master_addr and master_port will be instead, which (last I checked at least) is what's recommended.

The issue I linked to is patched with the PyTorch module we supply on the machine, but it still requires usage of --rdzv_backend c10d and --rdzv_endpoint [...], IIRC. That's why --rdzv_backend static can't work on our machine. It's complex and nasty, but that's how we manage to work around the issue. :p

Got it, I'm with you now.

In summary, the command @surak posted should be the following (just removing the --same_network line):
time srun bash -c 'accelerate launch \
    --main_process_ip $MASTER_ADDR \
    --main_process_port $MASTER_PORT \
    --multi_gpu \
    --mixed_precision no \
    --num_processes=$(($SLURM_JOB_NUM_NODES * 4)) \
    --dynamo_backend=no \
    --num_machines=$SLURM_JOB_NUM_NODES \
    --machine_rank=$SLURM_NODEID \
    --rdzv_backend c10d \
    distrib.py'

If we can verify that works, then I'll go ahead and merge the PR :)

Works for me indeed!

Does that grant a new point release? :-)

@surak we'll have a new release soon on our usual release cycle


Design: overview page for page templates

[x] Determine location in site information architecture
[x] Identify main audiences and outcomes users on the page
[x] Page content structure
[x] Review – design and research
[x] Dev hand off
[x] Translation
[x] Bilingual review
[x] Hand off for publishing

Replace Styles with Templates.

Draft overview page

Page templates overview page to link from:

Card on the homepage
Menu/top nav
Components overview page

https://docs.google.com/document/d/1EyzKiHkdkb2IJ4ZNLXLuu8T83qtqSqQ6XbywDFTQxHI/edit#heading=h.z7sh5e38f282

https://www.figma.com/design/nfpEteHyEJ9dhJ2YvuvLeo/Documentation-site%2FComponent-Pages?node-id=15046-4281

@EliseKa Page template is ready for translation: https://www.figma.com/design/nfpEteHyEJ9dhJ2YvuvLeo/Documentation-site%2FComponent-Pages?node-id=15046-4281

@SmartMouthWords Added the French version in the same Figma page. I left a comment about a heading, and provided an alternative (shorter) heading in French in case it's needed.

@MatildePerrusclet for final bilingual review.

Content Good to publish !
Moving to publishing card and the documentation card with the owner

Here is the card for publishing page templates on the documentation website - https://app.zenhub.com/workspaces/design-system-6100624a19f4cf000e46e458/issues/gh/cds-snc/design-gc-conception/1149


1inch Plugin v5 Upgrade

Smart Contracts
Smart contracts covered by this upgrade:

Network
Version
Smart Contract

Ethereum
V5
0x1111111254eeb25477b68fb85ed929f73a960582

Polygon
V5
0x1111111254eeb25477b68fb85ed929f73a960582

Functions implemented:

Function
Selector
Displayed Parameters

Swap (v5)
0x12aa3caf
   address desc.srcToken address desc.dstToken address desc.dstReceiver uint256 desc.amount uint256 desc.minReturnAmount uint256 desc.flags

Unoswap (v5)
0x0502b1c5
  address srcToken uint256 amount uint256 minReturn

unoswapToWithPermit (v5)
0x3c15fd91
  address recipient address srcToken uint256 amount uint256 minReturn

clipperSwap (v5)
0x84bd6d29
   address srcToken address dstToken uint256 inputAmount uint256 outputAmount

clipperSwapToWithPermit (v5)
0xc805a666
  address recipient address srcToken address dstToken uint256 inputAmount uint256 outputAmount

fillOrderRFQ (v5)
0x3eca9c0a
  address order.makerAsset address order.takerAsset address order.maker uint256 order.makingAmount uint256 order.takingAmount

fillOrderRFQToWithPermit (v5)
0x70ccbd31
  address order.makerAsset address order.takerAsset address order.maker uint256 order.makingAmount uint256 order.takingAmount

@GuilaneDen few comments from the Donjon

context->valid does not seem necessary to use (remove all its usage and definition)  https://github.com/LedgerHQ/app-plugin-1inch/blob/349666e3ccb9c82240c2ad85c587fcf56775e533/src/handle_init_contract.c#L19
Code duplication should be avoided sent_network_token/received_network_token is defined twice.
Should SWAP_V5 also be included in https://github.com/LedgerHQ/app-plugin-1inch/blob/349666e3ccb9c82240c2ad85c587fcf56775e533/src/handle_finalize.c#L22 ?

@adrienlacombe-ledger Thank you for the review the SWAP_V5 was indeed missing, hence a screen was not displayed. The review is implemented on the new fix.

thank you @GuilaneDen !

@GuilaneDen @adrienlacombe-ledger I can't see a version bump in the Makefile, you should probably do it before I merge it.

@agrojean-ledger good point, I'll update the version asap.

:heavy_check_mark: CI is ok.
:heavy_check_mark: Version bump.
:heavy_check_mark: Merge to develop for P4 deployment.
Ok for merge.


location count layer query

There should be a default template to count the number of locations passing the layer.filter.

@AlexanderGeere Added a query that performs a count on the locations that pass the filter of a layer.


Dashboard app for gtm (PR feedback)

Hi guys,
I found gtm an extremely valuable tool. It would be even better if it has a nice dashboard where you can see your stats. So in the last weeks I started a Dashboard for gtm. Here is the link to the repo
https://github.com/acuarica/gtm-web
and you can find the demo here
https://acuarica.github.io/gtm-web/
This is probably linked to issues #15, #32 and #41, so I include them here.
I have in mind two options of integrating it to gtm.

Using a local web server embedded in gtm
Using an Electron app that runs gtm on demand

For both options gtm would need to be upgraded, I hope that it can be done.
I'm looking forward to hear your feedback.
Best,
Luis.

I've decided to go for 2. An Electron app that reads gtm data and lets you visualize your time data.
You can find the pre-release here: https://github.com/acuarica/gtm-web/releases
Sure, there is a lot of room for improvement. Glad to hear your feedback.
Best,
Luis.


[TBD]Support apache elastic-job

Please answer these questions before submitting pull request

Why submit this pull request?

[ ] Bug fix

[x] New feature provided

[ ] Improve performance

Related issues
#5152

New feature or improvement

I will finish at elastic-job 3.0.0-alpha publish

What is the status of this? Any block?

What is the status of this? Any block?

Just wait apache elastic-job publish

Just wait apache elastic-job publish

FYI @tristaZero, when you have a release, please let us know. This depends on that.

@zhaoyuguang @wu-sheng Glad to see this PR. The latest version is expected to be released this week. I will give the update once it releases. Besides, welcome @zhaoyuguang to be my backup in case I forget. :)

@zhaoyuguang @wu-sheng Glad to see this PR. The latest version is expected to be released this week. I will give the update once it releases. Besides, welcome @zhaoyuguang to be my backup in case I forget. :)

Glad, And hope some day can become SS committer by contribution 👨💻

@zhaoyuguang @wu-sheng Glad to see this PR. The latest version is expected to be released this week. I will give the update once it releases. Besides, welcome @zhaoyuguang to be my backup in case I forget. :)

Glad, And hope some day can become SS committer by contribution 👨💻

Aha, looking forward to that as well!

@tristaZero Friendly Ping, PR is ready.

@tristaZero Friendly Ping, PR is ready.

Hi  @zhaoyuguang  Thanks for your reminder.
Apache ShardingSphere ElasticJob-3.0.0-alpha has been released successfully [1].
@wu-sheng It is the time to considering merging this PR, thanks.
[1] https://mvnrepository.com/artifact/org.apache.shardingsphere.elasticjob/elasticjob-lite

Hi @wu-sheng It seems these tags come from ShardingContexts, which is a necessary job context for users. I am not familiar with what content skyWalking needs for its plugin, but if it wants to get core job information, I think it is enough. :)

We don't need anything. I just want to be sure, these are enough information for the ElasticJob users.

@wu-sheng Got it. It is pleasing to see this plugin merged!
@zhaoyuguang Nice job 😉


Add UK regions

Adds a column containing UK regions:

East Midlands
East of England
London
North East England
North West England
South East England
South West England
West Midlands
Yorkshire and the Humber
Scotland
Wales
Northern Ireland
Crown Dependencies

Sourced from:

https://en.wikipedia.org/wiki/Subdivisions_of_England
https://en.wikipedia.org/wiki/Category:Postcode_areas_of_the_United_Kingdom
https://en.wikipedia.org/wiki/List_of_counties_of_Scotland_by_area_in_1951

Thanks for the PR


[SCD] Simulated Stars

estimate 2

[ ] How can stars be represented in simulations
[ ] How is their evolution implemented
[ ] How is their formation implemented

Merged with #13


gpio: gd32: initial support

This PR introduces a GPIO driver for GigaDevice SoCs, including AF (e.g. F4XX) and AFIO (e.g. F403) variants.
Tested on:

GD32F450I-EVAL

samples/basic/blinky
samples/basic/button
tests/drivers/gpio/gpio_basic_api
tests/drivers/gpio/gpio_api_1pin

GD32VF103V-EVAL (not mainlined yet, https://github.com/teslabs/zephyr/tree/gpio-gd32-vf103)

samples/basic/blinky
samples/basic/button
tests/drivers/gpio/gpio_basic_api
tests/drivers/gpio/gpio_api_1pin

GD32F403Z-EVAL (@nandojve)

samples/basic/blinky
samples/basic/button

@cameled FYI

Hi @gmarull, can we add pin default value setting for output mode? like: https://github.com/zephyrproject-rtos/zephyr/blob/38c42f69d28b8ce7cefb2b1a265400e83dd654aa/drivers/gpio/gpio_nrfx.c#L214
spi cs_control need this feature to disable the slave device by default.

Codecov Report

Merging #40676 (38c42f6) into main (f8f2936) will increase coverage by 0.08%.
The diff coverage is n/a.

:exclamation: Current head 38c42f6 differs from pull request most recent head 4fd78c3. Consider uploading reports for the commit 4fd78c3 to get more accurate results

@@            Coverage Diff             @@
##             main   #40676      +/-   ##
==========================================
+ Coverage   51.12%   51.20%   +0.08%
==========================================
  Files         604      604
  Lines       70771    70815      +44
  Branches    16298    16313      +15
==========================================
+ Hits        36179    36262      +83
+ Misses      28642    28577      -65
- Partials     5950     5976      +26

Impacted Files
Coverage Δ

subsys/logging/log_backend_uart.c
61.70% <0.00%> (-34.97%)
:arrow_down:

drivers/timer/native_posix_timer.c
76.47% <0.00%> (-10.20%)
:arrow_down:

subsys/testsuite/ztest/src/ztest_error_hook.c
74.54% <0.00%> (-9.10%)
:arrow_down:

include/spinlock.h
75.00% <0.00%> (-8.34%)
:arrow_down:

subsys/pm/device.c
40.33% <0.00%> (-6.73%)
:arrow_down:

subsys/pm/pm.c
68.26% <0.00%> (-2.21%)
:arrow_down:

kernel/idle.c
91.66% <0.00%> (-2.09%)
:arrow_down:

drivers/timer/hpet.c
85.56% <0.00%> (-1.04%)
:arrow_down:

kernel/thread.c
82.14% <0.00%> (-0.60%)
:arrow_down:

subsys/fs/littlefs_fs.c
70.12% <0.00%> (-0.56%)
:arrow_down:

... and 34 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 f8f2936...4fd78c3. Read the comment docs.

Hi @gmarull, can we add pin default value setting for output mode? like:
https://github.com/zephyrproject-rtos/zephyr/blob/38c42f69d28b8ce7cefb2b1a265400e83dd654aa/drivers/gpio/gpio_nrfx.c#L214
spi cs_control need this feature to disable the slave device by default.

done

@nandojve would appreciate testing on F403, since it uses the AFIO model (implementation not validated on my side)

Hi @gmarull , Looks good!
I tested with success samples/basic/[led/button]. Just few comment and I was wondering if it is possible add gpio_basic_api sample to F403 to test both AF/AFIO.

Added overlay for F403Z-EVAL. You should be able to test it by bridging the 2 DAC pins with a jumper (see overlay details).

Hi @gmarull ,

Added overlay for F403Z-EVAL. You should be able to test it by bridging the 2 DAC pins with a jumper (see overlay details).

Not sure why, GPIO at DAC port fail.
My config is: JP12 1-2 and JP7 1-2
I switch to port D 0-1 P2 + P3 which result is:
Running test suite gpio_basic_test
===================================================================
START - test_gpio_port
Validate device GPIOD
Check GPIOD output 0 connected to input 1

    Assertion failed at WEST_TOPDIR/zephyr/tests/drivers/gpio/gpio_basic_api/src/test_gpio_port.c:99: setup: (v1 & BIT(PIN_IN) not equal to 0)
out low does not read low
 FAIL - test_gpio_port in 0.21 seconds
===================================================================
START - test_gpio_callback_add_remove
callback_2 triggered: 1
callback_1 triggered: 1
callback_2 triggered: 1
callback_2 triggered: 2
FAIL - test_callback_add_remove@113. not trigger callback correctly

    Assertion failed at WEST_TOPDIR/zephyr/tests/drivers/gpio/gpio_basic_api/src/test_callback_manage.c:235: test_gpio_callback_add_remove: test_callback_add_remove() not equal to TC_PASS

 FAIL - test_gpio_callback_add_remove in 2.432 seconds
===================================================================
START - test_gpio_callback_self_remove
callback_remove_self triggered: 1
callback_1 triggered: 1
callback_1 triggered: 1
callback_1 triggered: 2
FAIL - test_callback_self_remove@174. not trigger remaining callback correctly

    Assertion failed at WEST_TOPDIR/zephyr/tests/drivers/gpio/gpio_basic_api/src/test_callback_manage.c:241: test_gpio_callback_self_remove: test_callback_self_remove() not equal to TC_PASS

 FAIL - test_gpio_callback_self_remove in 2.534 seconds
===================================================================
START - test_gpio_callback_enable_disable
callback_2 triggered: 1
callback_1 triggered: 1
callback_2 triggered: 1
callback_1 triggered: 1
 PASS - test_gpio_callback_enable_disable in 3.610 seconds
===================================================================
START - test_gpio_callback_variants
callback triggered: 1
OUT init a01, IN cfg 34000, cnt 1
callback triggered: 1
OUT init 600, IN cfg 54000, cnt 1
callback triggered: 1
OUT init 600, IN cfg 5c000, cnt 1
callback triggered: 1
OUT init a01, IN cfg 3c000, cnt 1
Mode 44000 not supported
Mode 24000 not supported
Mode 4c000 not supported
Mode 2c000 not supported
callback triggered: 1
callback triggered: 2
OUT init a01, IN cfg 74000, cnt 2
 PASS - test_gpio_callback_variants in 5.537 seconds
===================================================================
Test suite gpio_basic_test failed.
===================================================================
PROJECT EXECUTION FAILED

Hi @gmarull ,

Added overlay for F403Z-EVAL. You should be able to test it by bridging the 2 DAC pins with a jumper (see overlay details).

Not sure why, GPIO at DAC port fail. My config is: JP12 1-2 and JP7 1-2
I switch to port D 0-1 P2 + P3 which result is:

Are P2/P3 connected to 1-2 (EXMC signals) and display removed from the board?

Are P2/P3 connected to 1-2 (EXMC signals) and display removed from the board (to bridge EXMC_D2/EXMC_D3) ?

PD1-PD0 are connected by P2-2 + P3-2 using a jumper. No need to remove LCD.

Hi @gmarull ,

Added overlay for F403Z-EVAL. You should be able to test it by bridging the 2 DAC pins with a jumper (see overlay details).

Not sure why, GPIO at DAC port fail. My config is: JP12 1-2 and JP7 1-2
I switch to port D 0-1 P2 + P3 which result is:
Running test suite gpio_basic_test
===================================================================
START - test_gpio_port
Validate device GPIOD
Check GPIOD output 0 connected to input 1

    Assertion failed at WEST_TOPDIR/zephyr/tests/drivers/gpio/gpio_basic_api/src/test_gpio_port.c:99: setup: (v1 & BIT(PIN_IN) not equal to 0)
out low does not read low
 FAIL - test_gpio_port in 0.21 seconds
===================================================================
START - test_gpio_callback_add_remove
callback_2 triggered: 1
callback_1 triggered: 1
callback_2 triggered: 1
callback_2 triggered: 2
FAIL - test_callback_add_remove@113. not trigger callback correctly

    Assertion failed at WEST_TOPDIR/zephyr/tests/drivers/gpio/gpio_basic_api/src/test_callback_manage.c:235: test_gpio_callback_add_remove: test_callback_add_remove() not equal to TC_PASS

 FAIL - test_gpio_callback_add_remove in 2.432 seconds
===================================================================
START - test_gpio_callback_self_remove
callback_remove_self triggered: 1
callback_1 triggered: 1
callback_1 triggered: 1
callback_1 triggered: 2
FAIL - test_callback_self_remove@174. not trigger remaining callback correctly

    Assertion failed at WEST_TOPDIR/zephyr/tests/drivers/gpio/gpio_basic_api/src/test_callback_manage.c:241: test_gpio_callback_self_remove: test_callback_self_remove() not equal to TC_PASS

 FAIL - test_gpio_callback_self_remove in 2.534 seconds
===================================================================
START - test_gpio_callback_enable_disable
callback_2 triggered: 1
callback_1 triggered: 1
callback_2 triggered: 1
callback_1 triggered: 1
 PASS - test_gpio_callback_enable_disable in 3.610 seconds
===================================================================
START - test_gpio_callback_variants
callback triggered: 1
OUT init a01, IN cfg 34000, cnt 1
callback triggered: 1
OUT init 600, IN cfg 54000, cnt 1
callback triggered: 1
OUT init 600, IN cfg 5c000, cnt 1
callback triggered: 1
OUT init a01, IN cfg 3c000, cnt 1
Mode 44000 not supported
Mode 24000 not supported
Mode 4c000 not supported
Mode 2c000 not supported
callback triggered: 1
callback triggered: 2
OUT init a01, IN cfg 74000, cnt 2
 PASS - test_gpio_callback_variants in 5.537 seconds
===================================================================
Test suite gpio_basic_test failed.
===================================================================
PROJECT EXECUTION FAILED

Should be fixed now (tested on GD32VF103V-EVAL which has the same GPIO IP).

Comment to "drivers: gpio: gd32: initial version" mentions "TODO: Add support for interrupts.". However support for interrupts is already implemented. Could be corrected.

removed TODO, thanks for spotting

Comment to "drivers: gpio: gd32: initial version" mentions "TODO: Add support for interrupts.". However support for interrupts is already implemented. Could be corrected.

removed TODO, thanks for spotting

@gmarull - can you please rebase and fix merge conflicts?

@gmarull - can you please rebase and fix merge conflicts?

done


Octave compatibility: 'feature' does not exist in Octave

@jsherfey how do you think it is better to deal with this?
error: 'feature' undefined near line 306 column 29
error: called from
    dsSimulate at line 306 column 1
    dsDemos at line 136 column 5

We could use matlab specific features only in matlab by using this function:
function ui = reportUI
% function that reports ui, that is whether Octave or matlab is in use
LIC = license('inuse');
ui = LIC.feature;
% matlab toolboxes and octave packages are added to inuse, unfortunately 'matlab'' appears last (after tootlboxes) and octave first (before packages), what follows is a workaround to always report the proper UI:
if ~strcmp(ui,'octave')
  ui = 'matlab';
end

fixed in commit 833c5fc

adding some more info for octave in commit 9bd7a2f


Install protoc in project directory to avoid permissions issue

Our current install-protobuf.sh downloads protoc into /usr/local/bin/ and /usr/local/include/. This changes the user's system for no good reason. We should install these files in the project directory and reference in these files when running them, later.
Problematic installation:
https://github.com/spacemeshos/go-spacemesh/blob/6b0785df47291fdcd994da8d11bbbfbc5eeab3b9/scripts/install-protobuf.sh#L22-L28
Usage:
https://github.com/spacemeshos/go-spacemesh/blob/6b0785df47291fdcd994da8d11bbbfbc5eeab3b9/scripts/genproto.sh#L8-L14
This will fix #1678
TODO:

[ ] Update readme.md to also use the local installation.

@avive okay - open source readiness is currently a "project" but there is no associated milestone yet. Let me know if you want to change this.

Note that this will become obviated by #2078 - will leave this open until that's dealt with and all the protobuf tooling has been removed from go-spacemesh (hopefully soon).

duplicate of #2000, fixed in #2001. readme.md looks okay too.


Broken test on master - has been for a while

https://github.com/jonataslaw/get_storage/runs/956543996?check_suite_focus=true
00:43 +0 -1: loading /Users/runner/work/get_storage/get_storage/test/getstorage_test.dart [E]
  Failed to load "/Users/runner/work/get_storage/get_storage/test/getstorage_test.dart": MissingPluginException(No implementation found for method getApplicationDocumentsDirectory on channel plugins.flutter.io/path_provider)
  package:get_storage/src/storage_impl.dart 48:7   GetStorage._init
  ===== asynchronous gap ===========================
  package:get_storage/src/storage_impl.dart 28:13  new GetStorage._internal.<fn>

00:43 +0 -1: Some tests failed.
##[error]Process completed with exit code 1.

You can do the tests locally, the flutter has a problem in path_provider that is not being recognized in tests. All storage libraries have been broken for at least 2 months.

That's what I'm doing...
MacBook:get_storage rhisiart$ flutter test test/getstorage_test.dart
00:02 +0 -1: loading /Users/rhisiart/Cufflink.io/git/get_storage/test/getstorage_test.dart [E]
  Failed to load "/Users/rhisiart/Cufflink.io/git/get_storage/test/getstorage_test.dart": MissingPluginException(No implementation found for method getApplicationDocumentsDirectory on channel plugins.flutter.io/path_provider)
  package:get_storage/src/storage_impl.dart 48:7   GetStorage._init
  ===== asynchronous gap ===========================
  package:get_storage/src/storage_impl.dart 28:13  new GetStorage._internal.<fn>

Try adding and then immediately removing the pubspec path_provider plugin, it is an alternative work.
And that doesn't depend on us, but on the Flutter team that maintains the path_provider

Fixed in #18


Fix unicode decode error

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

in_transaction parameter is good idea. But do we need to upgrade python version from 3.8 to 3.10 @vmarkovtsev

We tested everything on 3.10. Unfortunately, I cannot test the changes on 3.8. It would be nice to make the version configurable ofc.
That being said, I switched jobs and am no longer involved in pgbouncer. How about cherry-picking individual commits in separate PRs?


Problem compiling OSVR-Core from current master on mac

There is a difference in .h and .cpp return type, which looks like it might be tied to OSVR_UTIL_EXPORT.  This was found in commit cb295fe44f5caa81094572de0976499f53e28314 after compiling libfunctionality and doing git submodule update --init on OSVR-Core

Here is the start of the set of error messages:
/Users/taylorr/sensics/src/OSVR-Core/src/osvr/Util/Logger.cpp:153:9: error: no type named
'StreamProxy' in 'osvr::util::log::Logger'; did you mean
'detail::LineLogger::StreamProxy'?
Logger::StreamProxy Logger::trace(const char *msg) {
^~~~~~~~~~~~~~~~~~~
detail::LineLogger::StreamProxy
/usr/local/include/osvr/Util/LineLogger.h:92:23: note: 'detail::LineLogger::StreamProxy'
declared here
class StreamProxy {
^
/Users/taylorr/sensics/src/OSVR-Core/src/osvr/Util/Logger.cpp:153:37: error: return type of
out-of-line definition of 'osvr::util::log::Logger::trace' differs from that in the
declaration
Logger::StreamProxy Logger::trace(const char *msg) {
~~~~~~~~~~~         ^
/usr/local/include/osvr/Util/Logger.h:126:49: note: previous declaration is here
OSVR_UTIL_EXPORT detail::LineLogger trace(const char *msg);
~~~~~~~~~~~~~~~~~~ ^
/Users/taylorr/sensics/src/OSVR-Core/src/osvr/Util/Logger.cpp:154:20: error: no matching
constructor for initialization of 'detail::LineLogger::StreamProxy'
return { *this, LogLevel::trace, msg };
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
/usr/local/include/osvr/Util/LineLogger.h:94:21: note: candidate constructor not viable:
requires single argument 'lineLogger', but 3 arguments were provided
StreamProxy(LineLogger &lineLogger)
^
/usr/local/include/osvr/Util/LineLogger.h:107:21: note: candidate constructor not viable:
requires single argument 'other', but 3 arguments were provided
StreamProxy(StreamProxy &&other)

It's pulling in this old header /usr/local/include/osvr/Util/LineLogger.h. It no longer exists in the latest master branch.
Also, for building on OS X with the latest Boost 1.62.0, you'll need to use the boost-1.62.0 branch.

Deleting the old header files from /usr/local and then deleting and rebuilding worked.


Rename http-receiver extension

Feature Request
The http-receiver is not really self-explainatory, it could be renamed to data-plane-transfer-sync-http-edr or something like this. Maybe it's too verbose, but I was wondering if we need that data-plane prefix on the data-plane-transfer-* extensions.
Which Areas Would Be Affected?
extensions
Why Is the Feature Desired?
comprehensiveness
Solution Proposal

maybe dpf-transfer instead of data-plane-transfer?
Would shorten a bit...

to be honest, I don't think data-plane should be mentioned, it's misleading, building a control-plane but defining data-plane (or dpf ) dependencies on it... at the end we're talking about providing transfer capabilities to the control plane.
I'd propose:

data-plane-transfer-client -> transfer-client
data-plane-transfer-sync -> transfer-sync
http-receiver -> transfer-sync-http-edr-receiver

+1. transfer-sync-http-edr-receiver is also verbose. How about transfer-sync-receiver?


Code scanning PR experience

Adds instructions to the README on introducing vulnerabilities in a PR. This new section is called "Introduce a Security Vulnerability in a PR" and walks the user through merging the "new-feature" branch which introduces vulnerabilities.

Looks great! Thanks @Eldrick19


Twister has a hard dependency on west.log

A recent change to Twister (#51045) began importing a Python module (domains.py) that in turn imports west.log. This adds a hard dependency on West from Twister, which previously has not been the case. Before, it was only an optional dependency only needed if using Twister to run on-device tests.
Based on comments in the code and my own previous issue (#45355), my understanding is that Twister is intended to be usable without West. Can anybody confirm whether or not this is still the case?
If Twister is indeed meant to be standalone and not dependent on West, I'd like to propose:

Removing the from west import log import from domains.py and simply use Python's standard logging module for the messages that module needs to print (this is what Twister uses for logging, anyways) in order to take out the dependency.
Clarifying the documentation and maybe even add some kind of stricter dependency checks to the CI.

If not, could we update comments / requirements / documentation to explicitly make West a required dependency?

Copy from https://github.com/zephyrproject-rtos/zephyr/issues/50389:
"I think I am convinced with Anas's argumentation (presented in various other places, e.g. https://github.com/zephyrproject-rtos/zephyr/pull/36729#pullrequestreview-752684699) that the core build system scripting shouldn't be part of twister, since twister is a test runner and users should be able to do general building/flashing without it. IMO the fact that twister is doing it's own scripting for building is against this axiom but I think it is a legacy issue, since as Anas pointed out, twister existed before west. In twisterV2 we do building by directly calling west, but will yet to see if this can be the general solution for all configurations."
However, If such hard dependency can be easily removed it might be useful to do this. I don't see a reason why a user should be blocked from picking its own building tool as a backend. It is not possible in the current architecture but should be rather easy in V2. Not that I have any alternative to west in mind, but by following dependency inversion principle we won't need such tight bounding between modules.

IMO the fact that twister is doing it's own scripting for building is against this axiom

In what way? Twister needs to build stuff in someway, it uses the build system. What is wrong with that? West is not the build system, cmake is, and twister just uses that.

It is not possible in the current architecture but should be rather easy in V2.

Not possible how? Eveything is possible, the question is why do we need to do anything differently? Twister is supposed to work for everyone the same way and supporting different build systems was never a goal or make sense. We only have one build system.

I've opened #51867 as a fix for this issue. One concern I have with this approach is that domains.py does not use the same logger instance as twister does. I'd prefer to use twister's logging instance, but since domains.py is imported before the "twister" logging instance gets configured, there isn't a straightforward way for the domains.py module to identify it is running using twister, and use the "twister" logging module.

Thanks for the fix!

One concern I have with this approach is...

Could you instead have Twister pass a Logger object when initializing either of those classes? That should also spare you from needing to set up the verbosity and format since it'll just assume the log settings from twister (or whatever is using this script) automatically.

Could you instead have Twister pass a Logger object when initializing either of those classes? That should also spare you from needing to set up the verbosity and format since it'll just assume the log settings from twister (or whatever is using this script) automatically.

We could, but I'd rather not add that parameter. The idea of the logging module is that if you call logging.getLogger('name'), the same instance should be returned, even if you are making the call from another module. So ideally, we would simply call logging.getLogger('twister') and be done. However, since west doesn't configure a logger with this name, we'd be left with a very basic logger. The core problem is how to detect that twister is being run versus west, and only reconfigure the logger when using west.


Implement IAsyncLifetime in integ test

Description
Implement IAsyncLifetime instead of IDisposable to simplify the test fixture as pointed out in https://github.com/aws/aws-dotnet-extensions-configuration/issues/16.
Testing
Integ test passes.
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist

[x] My code follows the code style of this project
[ ] My change requires a change to the documentation
[ ] I have updated the documentation accordingly
[x] I have read the README document
[x] I have added tests to cover my changes
[x] All new and existing tests passed

License

[x] I confirm that this pull request can be released under the Apache 2 license

Hey guys, I found this repo through Amazon since it looks like they adopted it for as their .NET Core configuration provider (according to their blog). However, it's running strict analysis on everything in my projects. Some of them include CA1721, CA1801, CA1822, CA2225 and CS0649. They don't have much to do with the nuget package and are very inconvenient to sift through. Is there a reason the Microsoft.CodeQuality.Analyzers and Microsoft.NetCore.Analyzers are dependencies of the main nuget package? Sorry, I'm not too familiar with this project.

@jingrah1 You are correct those dependencies should not be there. I created the following issue to track the issue. https://github.com/aws/aws-dotnet-extensions-configuration/issues/26

@normj , thanks for the responsiveness!


Improved tests and fixed bugs

Changes Unknown when pulling ea4c516b253b1d4e437dc653f878377e13468424 on biz-model-fix into ** on staging**.


Create Keptn Shipyard file for Flux Demo

For our Keptn Flux Integration, we will need to create a shipyard file to use so that we can use to send the event to.
Reach out to Adam Gardner for his Hello world example, he also has a blog on this.
https://agardner.net/keptn-hello-world/
Acceptance Criteria:
Get this working on your local computer and showcase it working

@bradmccoydev can we take a help of this: keptn.sh/docs/0.15.x/manage/shipyard/?

yes you can use that to help you, just do a hello world one and get it working on your computer doing the tutorials will help

@bradmccoydev  created a shipyard.yaml:
apiVersion: spec.keptn.sh/0.2.3
kind: "Shipyard"
metadata:
  name: "flux"
spec:
  stages:
    - name: "test"
      sequences:
      - name: "delivery"
        tasks:
        - name: "test"
        - name: "promote"
    - name: "uat"
      sequences:
      - name: "delivery"
        triggeredOn:
          - event: "test.delivery.finished"
        tasks:
        - name: "test"
        - name: "promote"
    - name: "prod"
      sequences:
      - name: "delivery"
        triggeredOn:
          - event: "uat.delivery.finished"
            selector:
              match:
              result: "pass"
        tasks:
        - name: "test"

And I got it working in the Keptn Bridge:

apiVersion: spec.keptn.sh/0.2.0 kind: Shipyard metadata:   name: flux-demo spec:   stages:     - name: dev       sequences:         - name: flux-deploy           tasks:             - name: checkout-git-repo               properties:                 project: my-flux-demo                 service: frontend                 gitUser: <username>                 gitToken: <token>               image: keptn/git-checkout-task             - name: flux-apply               properties:                 namespace: flux-system               image: docker.io/fluxcd/flux:1.24.2               env:                 - name: GITHUB_TOKEN                   value: <github_token>                 - name: GITHUB_USER                   value: <github_username>                 - name: GITHUB_EMAIL                   value: <github_email>               timeout: 5m
Have a look at it

Created a yml file have a look , :
apiVersion: spec.keptn.sh/0.2.0 kind: Shipyard metadata:   name: flux-demo spec:   stages:     - name: dev       sequences:         - name: flux-deploy           tasks:             - name: checkout-git-repo               properties:                 project: my-flux-demo                 service: frontend                 gitUser: <username>                 gitToken: <token>               image: keptn/git-checkout-task             - name: flux-apply               properties:                 namespace: flux-system               image: docker.io/fluxcd/flux:1.24.2               env:                 - name: GITHUB_TOKEN                   value: <github_token>                 - name: GITHUB_USER                   value: <github_username>                 - name: GITHUB_EMAIL                   value: <github_email>               timeout: 5m


Port version bumps from 1.4.6

This PR brings the version and changelog changes over from release-1.4.6

bors merge


fixed@4642:sometime, pasteing image no ext from clipbroad,default giv…

…e .png

Issue and pull request IDs are referenced via the # character.
@ is for usernames (and I do mean usernames, not people's real names).
Also, don't write your whole description in the title.
Closing ... #4643 has the details why.


Setup Server Media

******** Server-Media-Service **********

Media cloudinary
Code Upload image and remove cloudinary

Setup server email
Start 28/12/2022
End 29/12/2022
Time 2 day -> 16 hour

Done task Setup Server Media


GIDSignIn signInSilently not work

SOLUTION
SOLUTION 1
Need to call signInSilently later, in didFinishLaunchingWithOptions does not work (NOT TRUE)
SOLUTION 2
Need to configure the GIDSignIn (clientID, ...) like the first time we use [[GIDSignIn sharedInstance] signIn];
GIDSignIn *signIn = [GIDSignIn sharedInstance];
    signIn.shouldFetchBasicProfile = YES;
    signIn.clientID = kDCGooglePlusClientID();
    signIn.scopes = @[@"https://www.googleapis.com/auth/plus.login"
                      ];
    [signIn signInSilently];

SOLUTION 3
Login using allowsSignInWithWebView set to YES

This solutions didn't work for me. Im using the latest Google SignIn library, and running in iOS 9. [GIDSignIn sharedInstance] hasAuthInKeychain] always returns NO. This happens only when I kill the app (after successfully login In) and then open the app again. [[GIDSignIn sharedInstance] signInSilently]; will return error -4. It seems the normal SignIn doesn't save anything in the Keychain.. or the Keychain gets cleared when I the app relaunches or something like that.

@omarojo were you able to figure this out? I'm having this exact issue.

Nope, I just decided not to use the library.  I dont know how other famouse apps are doing it with this library.
=====================
Omar Juarez Ortiz
Software Developer
www.omarojo.com
On Tue, Mar 15, 2016 at 10:11 AM -0700, "Noah Labhart"<EMAIL_ADDRESS>wrote:
@omarojo were you able to figure this out? I'm having this exact issue.
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub


fix: big query run id message

I started out on this PR by trying to add additional unit test scenarios around running row level validations where I'm writing results to BQ.
In particular these scenarios...

log level = INFO + some results returned with filter_status = fail -> Does not write results to console + we get the run id message
log level = INFO + no results returned with filter_status = fail -> Does not write results to console + we get the "No results" message
log level = DEBUG + some results returned with filter_status = fail -> Does write results to console + we get the run id message
log level = DEBUG + no results returned with filter_status = fail -> Does not write results to console (because there are none) + we get the "No results" message

I care about this stuff because I don't want to write sensitive info to the console and I want to keep track of the run IDs (or lack of one where there is no row validation failure)
I found that I was getting an exception when I was setting filter_status = fail.
Once I looked into it, I found that result_df["run_id"][0] was looking for a key = 0, which did not exist in my test case.
result_df.iloc[0]["run_id"] returns the run id for the first item in the DF so should get around this bug.
Related PR -> https://github.com/GoogleCloudPlatform/professional-services-data-validator/pull/1240

@nj1973 @nehanene15 @helensilva14 - Happy to review?

/gcbrun

/gcbrun

/gcbrun

/gcbrun

Thanks all 🙇


Create a Linux VM from the ground up using the Azure CLI: Error in the script

Hi, new here. I think there is an error in the https://github.com/Azure/azure-content/blob/master/articles/virtual-machines/virtual-machines-linux-create-cli-complete.md article: instead of
# Associate the Public IP to the NIC
azure network nic set --public-ip-name test pip

it shoud be something like:
azure network nic set --public-ip-name testpip --resource-group TestRG testnic

Additionaly I'm unable to connect to the vm using ssh: the port 22 is not accesible.

Hi @ciberado, I'll walk through it again, for sure. As to the port, did you examine your inbound rule? It would look like:
azure network nsg rule set --destination-port-range 22 --source-port-range "*" --protocol tcp --access allow --priority 1000 testrg testnsg testnsgrule
One of the tricks is to make sure that you specify the source-port-range of * to allow connections initiated by any port. If you're thinking about classic deployments, the source port is NOT what you think it is. :-|

Hmmmm. @ciberado did we fix it in the meanwhile? I see this: https://github.com/Azure/azure-content/blob/master/articles/virtual-machines/virtual-machines-linux-create-cli-complete.md#associate-the-public-ip-and-the-network-security-group-to-the-nic
Associate the public IP and the network security group to the NIC
azure network nic set --public-ip-name testpip testrg testnic

Bind the NSG to the NIC:
azure network nic set --network-security-group-name testnsg testrg testnic

Wow, thank you for answering so fast. And yes! Looks like it's already corrected :+1:  We can close this thread now.


how to do drop down 4 level deep sub menu items' navbar?

possible to add navbar up to 4 levels deep?

Sorry, I'm not sure if I understand your question. Can you be a little bit more specific?

@ChristopherBilg like the multilevel dropdown
https://bootstrap-menu.com/demos/multilevel.html#

Closing for now. Feel free to re-open with an associated PR.


Data: Item Purchase / Sale History

We'd like to include a chronological item purchase / sale history, so that item builds can be analyzed for efficiency.

/CC: @tmcarmona16

Can we also please add on which Shop was used to buy/sell the items. Thank you

Hey everyone - This is happening in the Telemetry data.  Its pending integration, but the answer to this is a resounding YES YES YES.  I'm closing the ticket since we've converted it to a ticket on the dev side of life.


B #4793: Fix VM disks monitoring for LVM

Can you change all variables to the upper-case?
Also LVM_SIZE_CMD seems unused (although it was there before)

So the preferred style in OpenNebula is to use upper-case?
I saw recommendations to use lower case https://stackoverflow.com/questions/32144204/what-is-a-convention-for-naming-a-constant-in-bash
In other OpenNebula scripts the style is mixed https://github.com/OpenNebula/one/blob/master/src/mad/sh/scripts_common.sh#L330
In this script I used the same convention as nearby code

I think upper case is mainly used, mixing the case within the short file seems ugly to me.


Can Not Download

I'm using your amazing application on MyFidelio.
I thank you so much.
But, I can not download Siegfried (1992) only.
This is probably because the VIDEO and AUDIO values are both -30.
If it is possible to rewrite the database, please do so.
Sorry for my poor English.

I have fixed the bug, now uploading the fixed version. Just to give you the instructions on how to implement it - are you using Windows or Linux? Did you use an installer?

It is not the right file. Here's a link to the correct one:
https://github.com/rchenwh1te/StreamDownloader/blob/master/media/database.db


adds parse_random_state and associated types

this would be good to get in

test failure is relevant. I think the max seed is 2**32-1


findAll is not a function

I'm trying to do a basic store request on the application route of my Ember Engine add-on, but parent app is tripping up on the below.
Error while processing route: products.index this.get(...).findAll is not a function TypeError: this.get(...).findAll is not a function
import Ember from 'ember';

const { Route } = Ember;

export default Route.extend({
  model() {
    return this.get('store').findAll('product');
  }
});

What is strange is that I can see the GET request return the expected data in the network tab.

ember-cli 2.8.0
ember 2.8.0
ember-engines 0.3.0

The engine is a separate add-on I set up following the repo guides and functions as expected if I remove the findAll. The store dependency is defined in the engine and parent app.

For me, it helped to manually inject the store service in the engine:
export default Route.extend({
  store: inject.service(),
  model() { /* ... */ }
});

After declaring it as a dependency on the engine.

@mydea Thank you! Perhaps we should update the documentation to include this?

If you have to manually inject the store then that means your Engine isn't running the Initializers from Ember Data (such as this one). Closing as this isn't particularly an issue with Engines, just a setup problem.

Do you know what setup problem this indicates?

As noted above, the problem is that the initializers aren't running. This could happen for a myriad of reasons. Two most common would be that you're either not properly using loadInitializers in your engine.js file or you have don't have the initializers included in your application's namespace.

Interesting. My engine setup was working and then I started having this same problem myself. I'll revisited my config to see what might be wrong.

@trentmwillis Thanks, I'll take a look the app setup.

As @mydea mentions, I fixed this by explicitly injecting the store in the routes and doing this.get('store') instead of this.store.
It turns out that all routes have a store property on them with a find method, I have no idea why.

Basically: legacy, we should remove...

I am unclear about what the current resolution is here-- do we need to continue manually injecting the store into our engine routes for the time being? I'm using Ember/Data/Cli 2.11 and Engines 0.4.

Encountering the same issue, although I've included store as a dependency for my engine both in app.js and in the engine's engine.js, I still have to manually inject the store into routes for the engine, or else it cannot find models in the application store.
ember v2.8.3
ember-engines v0.3.4


The preview is very dark

The preview is very dark

I resolved this preview is dark issues by doing these steps.
//set brightness
mPreviewBuilder.set(CaptureRequest.CONTROL_AE_TARGET_FPS_RANGE, getRange());
and add this method
/**
 * An {@link CameraCharacteristics} method for camera preview not dark
 */

private Range<Integer> getRange() {
    CameraCharacteristics chars = null;
    CameraManager manager = (CameraManager) getSystemService(Context.CAMERA_SERVICE);
    try {
        String cameraId = manager.getCameraIdList()[1];
        chars = manager.getCameraCharacteristics(cameraId);
        Range<Integer>[] ranges = chars.get(CameraCharacteristics.CONTROL_AE_AVAILABLE_TARGET_FPS_RANGES);
        Range<Integer> result = null;
        for (Range<Integer> range : ranges) {
            int upper = range.getUpper();
            // 10 - min range upper for my needs
            if (upper >= 10) {
                if (result == null || upper < result.getUpper().intValue()) {
                    result = range;
                }
            }
            Log.e("Avaliable frame fps :", "" + range);
        }
        if (result == null) {
            result = ranges[0];
        }

        Log.e("frame fps :", "" + result);
        return result;
    } catch (CameraAccessException e) {
        e.printStackTrace();
        return null;
    }
}

this will resolve by issues.
Thanks

This sample has been migrated to a new location so we can allow Pull Requests (check README for more information).
As recommended by GitHub, we are closing all issues and pull requests now that this older repo will be archived.
If you still see this issue in the updated repo, please reopen the issue/PR there. Thank you!


[middleware/requestlog] Check that handlers are only called once.

Add a check that HTTP handlers are only called once in middleware/requestlog. Currently solutions that call the handler twice can pass, e.g. https://gitlab.manytask.org/go/students-2024-spring/mayorov.m.a/-/blob/c36d621385113cd9e07278fa1edff68ef320c0fe/middleware/requestlog/requestlog.go#L51-52
We could also

use synchronization/atomics
use oneShotHandler for each request path in this test
use it for the other two middleware tasks

+Fix a typo in middleware/httpgauge.

merged


Fixed #208 - Cleaned up Javadoc

Javadocs build on Java 1.8 without lint warnings
Some Javadoc comments and tags cleaned up
Javadocs for deprecated generated classes excluded from generation

This PR replaces #226 with a single commit.


Make sure we enable dev/shm when required

Fixes #545
See also https://github.com/DiamondLightSource/dodal/pull/819
To test:

Confirm test fails on main and passes here

As discussed, naming is confusing, we should removeuse_gpu for now and rename use_cpu_and_gpu_zocalo to compare_cpu_and_gpu_results then only turn dev/shm on when compare_cpu_and_gpu_results is True