Errors with dockerized MPI application trying to run on various nodes

Hi all,
I'm trying to run a simple mpi example on Batch with docker (but without using Batch Shipyard, as I didn't want to set up and configure the Shipyard tool).
I have a 4-nodes pool with a docker image from an Azure Container Registry, I followed a recipe in the shipyard repository to build a docker image with openssh and all the required stuff (I think)  to let docker container be able to communicate with each node
You can check the Dockerfile and the related files here:
When I created a Task with this command line using an AutoUserSpecification of Pool & NonAdmin credentials:
cd /app && mpirun --allow-run-as-root -np 64 --host $AZ_BATCH_HOST_LIST mpi_prime
I got these errors:

Could not create directory '/home/_azbatch/.ssh'.
Could not create directory '/home/_azbatch/.ssh'.
Host key verification failed.

So I figured out that I should create a volume on the container that point out to that dir: /hom/_azbatch/.ssh , so I modified the containersettings of the task to include this param:
-v /home/_azbatch/.ssh:/home/_azbatch/.ssh
After that, I tried to execute another task with the same command line, and I got this error:

bash: orted: command not found

After reading some forums I thought that my problem was that orted wasn't able to be found on the slave compute nodes, so I added a new param to my command line:

--prefix /usr/lib64/openmpi

so the final command line was:
cd /app && mpirun --allow-run-as-root -np 64 --prefix /usr/lib64/openmpi --host $AZ_BATCH_HOST_LIST mpi_prime
(that's where are my openmpi libraries, inside the container, of course)
...and I got this error when executing the task:

There are not enough slots available in the system to satisfy the 64 slots
that were requested by the application:
mpi_prime
Either request fewer slots for your application, or make more slots available
for use.

...that's not true, because I have 4 nodes of 16 cores each, so I have 64 slots available (and I tried to execute the mpirun with --np 1 and I had the same error message, Not enough slots available to satisfy the 1 slots ...
So I started to read again some forums and I got to one that said that including this parameter in the mpirun sentence will solve the problem: --oversubscribe , so I threw a new CloudTask with this command line:
cd /app && mpirun --allow-run-as-root -np 64 --prefix /usr/lib64/openmpi --oversubscribe --host $AZ_BATCH_HOST_LIST mpi_prime
...and I got this error:

mpirun was unable to find the specified executable file, and therefore
did not launch the job.  This error was first reported for process
rank 16; it may have occurred for other processes as well.
NOTE: A common cause for this error is misspelling a mpirun command
line parameter option (remember that mpirun interprets the first
unrecognized command line token as the executable).
Node:       <IP_ADDRESS>
Executable: /app/mpi_prime
48 total processes failed to start

After reading this, I noticed that the 48 processes failed to start matches 3 of the four nodes of the pool (3 nodes * 16 cores = 48 processes), so it seems that the master node can locate the executable /app/mpi_prime but not the slave nodes.
¿Can anybody help me?
Thanks in advance.
Regards.

There was a mistake with the way I was configuring ssh and using mpirun command, I'll update my repo with all the changes I made to make it run properly.


Disable pool_connection () for stopped bootstrap or bootstrap_client

Fixes #1496

@SergiySW Can you find out when this issue first showed up? And also if anything we have done recently has made this worse?  Trying to evaluate where to include this fix.

Ptobably it's related to lazy_bootstrap implementation (https://github.com/nanocurrency/raiblocks/pull/1332/) especially https://github.com/nanocurrency/raiblocks/blob/master/rai/node/bootstrap.cpp#L594
But it also happening with previous versions or with disabled lazy bootstrap, just less often.

Looking at including this in V18 unless there is an urgency to push earlier.


Portfolio home defaults to About Me content

User Story
AS AN employer looking for candidates with experience building single-page applications,
I WANT to view a potential employee's deployed React portfolio of work samples,
SO THAT I can assess whether they're a good candidate for an open position.

Acceptance Criteria
GIVEN a single-page application portfolio for a web developer,
WHEN I load the portfolio the first time,
THEN the About Me title and section are selected by default.

Complete. See PR https://github.com/nestibry/portfolio/pull/25


[AutoRelease] t2-eventgrid-2022-08-18-97191(Do not merge)

https://github.com/Azure/sdk-release-request/issues/3095
Live test success
https://dev.azure.com/azure-sdk/internal/_build?definitionId=993
BuildTargetingString
azure-mgmt-eventgrid
Skip.CreateApiReview
true

issue link:https://github.com/Azure/sdk-release-request/issues/3095


[Feature] Render Dataview queries before encryption

That would be pretty cool!
For inspiration, here’s how Mara does it for her GitHub publisher:
https://github.com/ObsidianPublisher/obsidian-github-publisher/blob/824ac410ef5c0cc349fe7bfd61353ba0c0c2dcc4/plugin/contents_conversion/convertText.ts#L21-L53

That would work at least for code blocks. Although I think only for DQL and not Dvjs.
For dvjs:
https://github.com/blacksmithgu/obsidian-dataview/blob/03d0c5de51e992a650de0c1c769093bedb1c7817/src/api/plugin-api.ts#L436
Normal Inline queries probably need a combo of parse and renderValue.
https://github.com/blacksmithgu/obsidian-dataview/blob/03d0c5de51e992a650de0c1c769093bedb1c7817/src/api/plugin-api.ts#L221
https://github.com/blacksmithgu/obsidian-dataview/blob/master/src/api/plugin-api.ts#L519
For inline js queries the eval functions combined with renderValue.
https://github.com/blacksmithgu/obsidian-dataview/blob/03d0c5de51e992a650de0c1c769093bedb1c7817/src/api/inline-api.ts#L422

Contained example for rendering inline queries:
https://github.com/blacksmithgu/obsidian-dataview/blob/03d0c5de51e992a650de0c1c769093bedb1c7817/src/ui/lp-render.ts#L178-L245


fix warnings in python 3.10 for tests, event_loop

fixes following deprecating warnings on running tests make test

tests/core/views/test_api.py::test_core_views_generic
/home/dni/repos/lnbits/FinalAdminUI/lnbits/extensions/subdomains/__init__.py:24: DeprecationWarning: There is no current event loop
loop = asyncio.get_event_loop()

tests/core/views/test_api.py::test_core_views_generic
/home/dni/repos/lnbits/FinalAdminUI/lnbits/extensions/splitpayments/__init__.py:34: DeprecationWarning: There is no current event loop
loop = asyncio.get_event_loop()

tests/core/views/test_api.py::test_core_views_generic
/home/dni/repos/lnbits/FinalAdminUI/lnbits/extensions/lnaddress/__init__.py:25: DeprecationWarning: There is no current event loop
loop = asyncio.get_event_loop()

tests/core/views/test_api.py::test_core_views_generic
/home/dni/repos/lnbits/FinalAdminUI/lnbits/extensions/jukebox/__init__.py:33: DeprecationWarning: There is no current event loop
loop = asyncio.get_event_loop()

Codecov Report

Merging #1078 (9036cf9) into main (09871bb) will decrease coverage by 2.94%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #1078      +/-   ##
==========================================
- Coverage   51.52%   48.57%   -2.95%
==========================================
  Files         238      238
  Lines       13396    13396
==========================================
- Hits         6902     6507     -395
- Misses       6494     6889     +395

Impacted Files
Coverage Δ

lnbits/extensions/boltz/mempool.py
32.75% <0.00%> (-50.00%)
:arrow_down:

lnbits/extensions/boltz/crud.py
29.23% <0.00%> (-49.24%)
:arrow_down:

lnbits/extensions/boltz/utils.py
23.52% <0.00%> (-41.18%)
:arrow_down:

lnbits/extensions/boltz/boltz.py
17.91% <0.00%> (-39.31%)
:arrow_down:

lnbits/wallets/cln.py
19.00% <0.00%> (-29.76%)
:arrow_down:

lnbits/wallets/lndrest.py
17.09% <0.00%> (-29.06%)
:arrow_down:

lnbits/wallets/lndgrpc.py
22.72% <0.00%> (-27.93%)
:arrow_down:

lnbits/extensions/boltz/views_api.py
31.40% <0.00%> (-18.19%)
:arrow_down:

...nbits/wallets/lnd_grpc_files/lightning_pb2_grpc.py
38.22% <0.00%> (-12.25%)
:arrow_down:

lnbits/wallets/lnd_grpc_files/router_pb2_grpc.py
40.54% <0.00%> (-11.49%)
:arrow_down:

... and 5 more

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


Make process name check case insensitive

In some environments, this process name registers itself as FFXIV_DX11
instead of ffxiv_dx11, such as when the game is launched remotely from a
network share. Make the check case insensitive to accommodate for this
case.

Heads up: Visual Studio refuses to run for me since my free trial expired, so I can't test to see if it compiles. But it should work, in theory.

Visual Studio Community Edition is completely free, you shouldn't run out of a trial.
Anyway, thanks for the PR.

┐('～`；)┌

Did you click to check for a new license?
On Fri, Apr 5, 2019 at 1:06 PM Niklas Haas<EMAIL_ADDRESS>wrote:

┐('～`；)┌
https://camo.githubusercontent.com/c81f308eed6228158c92ad6e90afe4ef9ff7cfb5/68747470733a2f2f3078302e73742f7a5a586d2e706e67
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/imchillin/SSTool/pull/1#issuecomment-480350956, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAmwpnnSWvrUiOVkpFgFkbwS7QdBmr0Jks5vd4KngaJpZM4cdk0R
.


Run workflow: publish bundle to charmhub

@orfeas-k If this is dummy/debug PR please indicate it in the title, so we do not consider it for merging.


[TS Migration] NewDatePicker

Details
Migration of DatePicker to Typescript. This PR also contains the fix for the dates comparison in the getDayValidationErrorKey function. Currently on the main branch in the getDayValidationErrorKey dates are compared based on getDayOfYear function from date-fns.  This function checks if the input date is in the future compared to the reference date. But this function doesn't work correctly when the input date has an earlier day of the year than the reference date, but has a later year. It can be noticed on the main branch when we try to set the date according to the mentioned conditions in StatusClearAfterPage:
https://github.com/Expensify/App/assets/47774969/071d52d4-cd17-4be3-a947-6f68a65f4850
Fixed Issues

$ https://github.com/Expensify/App/issues/25148
PROPOSAL: N/A
Tests

[ ] Verify that no errors appear in the JS console
[ ] Open any screen where the DatePicker is available, for example: StatusClearAfterPage.
[ ] Check whether all elements of the DatePicker behave correctly: YearPickerModal, switching months by arrows etc.
[ ] Verify that the dates comparison now works properly

[ ] Open StatusClearAfterPage
[ ] Try to set the date with a later year and an earlier day than today's date
[ ] Check if any error occurred

Offline tests

N/A
QA Steps

Same as tests
PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(theme.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If the PR modifies the form input styles:

[x] I verified that all the inputs inside a form are aligned with each other.
[x] I added Design label so the design team can review the changes.

[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.

Screenshots/Videos

Android: Native

https://github.com/Expensify/App/assets/47774969/c6f4e251-c285-494a-9c8f-ce336611f151

Android: mWeb Chrome

https://github.com/Expensify/App/assets/47774969/07120c05-9d25-40be-857f-a6076d088203

iOS: Native

https://github.com/Expensify/App/assets/47774969/649923ce-f6d8-4c88-9baa-a71ede7f208c

iOS: mWeb Safari

https://github.com/Expensify/App/assets/47774969/9d35ef1a-6b9d-49dd-b159-61788bd0c372

MacOS: Chrome / Safari

https://github.com/Expensify/App/assets/47774969/9341b5af-7069-4ec7-8849-0f55ddbf45dd

MacOS: Desktop

https://github.com/Expensify/App/assets/47774969/49914c51-3122-42ea-a7c7-317a42417316

@WojtekBoman Are there test steps for validating the getDayValidationErrorKey fix?

Reviewer Checklist

[ ] I have verified the author checklist is complete (all boxes are checked off).
[ ] I verified the correct issue is linked in the ### Fixed Issues section above
[ ] I verified testing steps are clear and they cover the changes made in this PR

[ ] I verified the steps for local testing are in the Tests section
[ ] I verified the steps for Staging and/or Production testing are in the QA steps section
[ ] I verified the steps cover any possible failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[ ] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)

[ ] I checked that screenshots or videos are included for tests on all platforms
[ ] I included screenshots or videos for tests on all platforms
[ ] I verified tests pass on all platforms & I tested again on:

[ ] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[ ] MacOS: Chrome / Safari
[ ] MacOS: Desktop

[ ] If there are any errors in the console that are unrelated to this PR, I either fixed them (preferred) or linked to where I reported them in Slack
[ ] I verified proper code patterns were followed (see Reviewing the code)

[ ] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick).
[ ] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[ ] I verified that comments were added to code that is not self explanatory
[ ] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[ ] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method
[ ] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[ ] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[ ] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[ ] I verified the JSDocs style guidelines (in STYLE.md) were followed

[ ] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[ ] I verified that this PR follows the guidelines as stated in the Review Guidelines
[ ] I verified other components that can be impacted by these changes have been tested, and I retested again (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar have been tested & I retested again)
[ ] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[ ] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[ ] If a new component is created I verified that:

[ ] A similar component doesn't exist in the codebase
[ ] All props are defined accurately and each prop has a /** comment above it */
[ ] The file is named correctly
[ ] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[ ] The only data being stored in the state is data necessary for rendering and nothing else
[ ] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[ ] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[ ] All JSX used for rendering exists in the render method
[ ] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[ ] If any new file was added I verified that:

[ ] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[ ] If a new CSS style is added I verified that:

[ ] A similar style doesn't already exist
[ ] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(theme.componentBG)

[ ] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[ ] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[ ] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[ ] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[ ] If the PR modifies the form input styles:

[ ] I verified that all the inputs inside a form are aligned with each other.
[ ] I added Design label so the design team can review the changes.

[ ] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[ ] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[ ] I have checked off every checkbox in the PR reviewer checklist, including those that don't apply to this PR.

Screenshots/Videos

Android: Native

Android: mWeb Chrome

iOS: Native

iOS: mWeb Safari

MacOS: Chrome / Safari

MacOS: Desktop

@WojtekBoman I get the following error trying to access the Date of Birth screen from Personal Details.
Uncaught TypeError: Cannot read properties of undefined (reading 'length')
    at generateMonthMatrix (generateMonthMatrix.ts:54:1)
    at CalendarPicker (index.tsx:141:47)
    at renderWithHooks (react-dom.development.js:16175:18)
    at mountIndeterminateComponent (react-dom.development.js:20908:13)
    at beginWork (react-dom.development.js:22411:16)
    at HTMLUnknownElement.callCallback (react-dom.development.js:4161:14)
    at Object.invokeGuardedCallbackDev (react-dom.development.js:4210:16)
    at invokeGuardedCallback (react-dom.development.js:4274:31)
    at beginWork$1 (react-dom.development.js:27400:7)
    at performUnitOfWork (react-dom.development.js:26511:12)

@WojtekBoman I get the following error trying to access the Date of Birth screen from Personal Details.
Uncaught TypeError: Cannot read properties of undefined (reading 'length')
    at generateMonthMatrix (generateMonthMatrix.ts:54:1)
    at CalendarPicker (index.tsx:141:47)
    at renderWithHooks (react-dom.development.js:16175:18)
    at mountIndeterminateComponent (react-dom.development.js:20908:13)
    at beginWork (react-dom.development.js:22411:16)
    at HTMLUnknownElement.callCallback (react-dom.development.js:4161:14)
    at Object.invokeGuardedCallbackDev (react-dom.development.js:4210:16)
    at invokeGuardedCallback (react-dom.development.js:4274:31)
    at beginWork$1 (react-dom.development.js:27400:7)
    at performUnitOfWork (react-dom.development.js:26511:12)

This issue has been solved :)

@WojtekBoman Are there test steps for validating the getDayValidationErrorKey fix?

I've updated Details and Tests sections :)

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to staging by https://github.com/puneetlath in version: 1.4.42-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/chiragsalian in version: 1.4.42-5 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


Reverse Engineered?

Hi there.
I was just wondering, if you have reversed engineered the existing iOS app in order to extract the structure of UDP packets, the FaceBlendShape enums, etc. I couldn't find anything explaining these values in the UE4 documentation.
Thank you.

At first, I have capturee the udp packages with wireshark and reversed engineered the bits and bytes, looking which are changing, which are not (fortunatelly the IPhone also sends packages when no faces are in the image, so it was easy to identify which bits are the actual blendshapes). You can also show the actual number of the blendshape arraving in the Unreal Engine.
Then I found this, which helped me with the enum:
https://docs.unrealengine.com/4.26/en-US/API/Runtime/AugmentedReality/EARFaceBlendShape/
Then I found this which gave me everything I needed: https://github.com/EpicGames/UnrealEngine/blob/4.26.2-release/Engine/Plugins/Runtime/AR/AppleAR/AppleARKitFaceSupport/Source/AppleARKitFaceSupport/Private/AppleARKitLiveLinkSource.cpp

Thanks a bunch!


Embedded canvases are empty (400 from /runJS)

Description
Embedded demonstrations in section 3.5 "Composing transformations" in web/help/GuideUnit1.md show up empty. E.g. "Translation and rotation." zoomy opens with just the text and no canvas. Javascipt console seems to indicate that the server is returning a 400 for /runJS URLs.


How to define that a field can be null

Describe the usage question you have. Please include as many useful details as  possible.
When defining a model using pa.schema, how do you define a field if the entire column may be empty?
For example, I define a name field, which may or may not be empty in the entire column
The definition is as follows:
pa.field('name', pa.string())
When the name column is empty, an error is reported, how do I define field so that it is compatible with the case where the whole column is empty
Component(s)
Parquet, Python

Seems default of nullable is true ( https://arrow.apache.org/docs/python/generated/pyarrow.field.html#pyarrow.field ). What error does it report? And can you provide how to reproduce the error?

import random

import pandas as pd
import pyarrow as pa
import pyarrow.dataset as ds

schema = pa.schema([
    pa.field('id', pa.string(), metadata={b"table_field": b"ID"}),
    pa.field('name', pa.string(), metadata={b"table_field": b"NAME"}),
    pa.field('age', pa.int8(), metadata={b"table_field": b"AGE"})

])
data = {
    "id": [i for i in range(3)],
    "name": [None] * 3,
    "age": [random.randint(10, 30) for i in range(3)]
}
df = pd.DataFrame(data)
print(df)

table = pa.Table.from_pandas(df)
file_path = 'test'
ds.write_dataset(table, file_path, format='parquet', schema=schema)

pyarrow has nullable type
Maybe you should set type using pa.Table.from_pandas(df, schema=schema)

@phpsxg

pyarrow has nullable type
Maybe you should set type using pa.Table.from_pandas(df, schema=schema)

@phpsxg

table = pa.Table.from_pandas(df, schema=schema)

ds.write_dataset(table, file_path, format='parquet', existing_data_behavior='delete_matching')

Is there any difference in schema parameters?

I'm not expert on pyarrow, but seems that pyarrow has its internal schema. So, when casting to pyarrow, if not explicit set a schema, [None] * 3 might be reduce as "null" type. And write null type to parquet might failed.
With explicit set a schema, it would be set to "nullable string", which is a valid schema here.

I'm not expert on pyarrow, but seems that pyarrow has its internal schema. So, when casting to pyarrow, if not explicit set a schema, [None] * 3 might be reduce as "null" type. And write null type to parquet might failed.
With explicit set a schema, it would be set to "nullable string", which is a valid schema here.

I found pa. Table.from_pandas and ds.write_dataset do not add the schema parameter and will not report an error.
table = pa.Table.from_pandas(df)
ds.write_dataset(table, file_path, format='parquet', existing_data_behavior='delete_matching')

Is there no need to add this schema parameter?

@mapleFU
I also have a question, how to automatically type conversion, for example, id is int type, want to save is the string type, thank you
import random

import pandas as pd
import pyarrow as pa
import pyarrow.dataset as ds

schema = pa.schema([
    pa.field('id', pa.string(), metadata={b"table_field": b"ID"}),
    pa.field('age', pa.int64(), metadata={b"table_field": b"AGE"})

])
data = {
    "id": [i for i in range(3)],
    "age": [random.randint(10, 30) for i in range(3)]
}
df = pd.DataFrame(data)

table = pa.Table.from_pandas(df,schema=schema)

file_path = 'test'
ds.write_dataset(table, file_path, format='parquet', existing_data_behavior='delete_matching',schema=schema)

I'm not a contributor of pyarrow, so I suggest you to ask them :)
Personally, I guess explicit type cast helps https://arrow.apache.org/docs/python/generated/pyarrow.Table.html#pyarrow.Table.cast .

Is there no need to add this schema parameter?

Please read the document carefully https://arrow.apache.org/docs/python/generated/pyarrow.Table.html#pyarrow.Table.from_pandas :

The column types in the resulting Arrow Table are inferred from the dtypes of the pandas.Series in the DataFrame. In the case of non-object Series, the NumPy dtype is translated to its Arrow equivalent. In the case of object, we need to guess the datatype by looking at the Python objects in this Series.

Be aware that Series of the object dtype don’t carry enough information to always lead to a meaningful Arrow type. In the case that we cannot infer a type, e.g. because the DataFrame is of length 0 or the Series only contains None/nan objects, the type is set to null. This behavior can be avoided by constructing an explicit schema and passing it to this function.

You can print the type to check can it reduce the right type. Personally I think it's proper to provide a explicit type.

how to automatically type conversion

I guess you need to cast the type https://arrow.apache.org/docs/python/generated/pyarrow.Table.html#pyarrow.Table.cast

@phpsxg I think it is a deliberate choice of the PyArrow maintainers to require explicit casting in cases like this, for safety

I'm running into the same issue. When I create a table that has only non-null values, it still defines the schema as nullable:
import pyarrow as pa

df = pa.Table.from_pylist(
    [
        {"city": "Amsterdam", "lat": 52.371807, "long": 4.896029},
        {"city": "San Francisco", "lat": 37.773972, "long": -122.431297},
        {"city": "Drachten", "lat": 53.11254, "long": 6.0989},
        {"city": "Paris", "lat": 48.864716, "long": 2.349014}
    ],
)

Here you can see that not null is not there.
city: string
lat: double
long: double


Backup Support is now 32TiB for Azure Backup.

I think the 4 TiB limit is for ASR.  Backup is larger now.   Here is the paragraph you may want to correct:
Azure Backup support
To protect against regional disasters, Azure Backup can be used to create a backup job with time-based backups and backup retention policies. This allows you to perform easy VM restorations at will. Currently Azure Backup supports disk sizes up to four tebibyte (TiB) disks. Azure Backup supports backup and restore of managed disks. Learn more about Azure VM backup support.

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

ID: bfaec879-f3e0-afba-fbe8-85e6e28937df
Version Independent ID: b21d00cd-0323-26d0-34b4-f0a76ee8aaea
Content: Azure Disk Storage overview - Azure Windows Virtual Machines
Content Source: articles/virtual-machines/windows/managed-disks-overview.md
Service: virtual-machines
Sub-service: disks
GitHub Login: @roygara
Microsoft Alias: rogarana

@stdistef
Thanks for the feedback! I have assigned the issue to content author to check and update the document as appropriate.

@roygara
Can you please check and add your comments on this doc update request as applicable.

@stdistef Thanks for reaching out! I've gone ahead and confirmed that it is up to 32TiB at this point. I'll update the doc accordingly, it'll be live later today.
We appreciate you helping improve our docs, thank you! :D
#please-close


Clarify how setting a frontend priority works

What does this PR do?
Clarify how setting a frontend priority works.
See #1962 for discussion, and https://github.com/containous/traefik/blob/c0563f1a395d53edbbf1030b2387435077eeb871/server/server.go#L1338-L1347 for the now documented code.
Motivation
I set the priority on a rule, and it wasn't effective the way I understood it from the documentation. This gave us trouble in routing in our test environment.
Reading #1962, I realized from the code snippet that setting the priority works more like an offset than an absolute ordering of frontends.
More
Purely a documentation change.

[ ] Added/updated tests
[x] Added/updated documentation

Additional Notes
I would be amazing to have a way to get the previously documented behavior, i.e. comparing priorities first, only using rule lengths to break ties.

Thanks for the fast processing! I'd like to reiterate:
I would be amazing to have a way to get the previously documented behavior, i.e. comparing priorities first, only using rule lengths to break ties.
Would someone like to pick up on that?


Status of project

The readme is a bit sparse. How far along is the project? What needs to be done? https://www.ruma.io/ doesn't have any more detail.
Answers to these question would be a nice addition to the README.

Thanks for your interest and apologies for the lack of information in the README. ruma-client is currently a work in progress. Right now the main blockers on finishing it are:

Converting ruma-client-api, which contains types for each API endpoint's request and response, to use the procedural macro from ruma-api-macros. This is roughly 1/3 done and I'm making progress on it.
A final release of tokio-based hyper, which the community has been told is coming "soon" for like the last 6 months, so who knows.

Update from Jimmy, yesterday in #ruma:matrix.org:

ruma-client is mostly as far done as it needs to be (might be missing a few APIs, I haven't looked at it in a long time) and is waiting on stabilizations of its dependencies […]. the other big thing […] is places where synapse doesn't actually follow the spec and so ruma-client won't work against a live synapse server as-is. the goal is to fix those things properly in synapse rather than have ruma-client go off spec.

Source: https://matrix.to/#/!wzHrsErnsyaqbpFiRQ:matrix.org/$15246419371107088KVHHl:matrix.org

Assigning myself as I intend to update the README accordingly (hopefully soon).


python311Packages.timezonefinder: 6.2.0 -> 6.4.0

Diff: https://github.com/jannikmi/timezonefinder/compare/refs/tags/6.2.0...6.4.0
Changelog: https://github.com/jannikmi/timezonefinder/blob/6.4.0/CHANGELOG.rst
Description of changes

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 286078 run on x86_64-linux 1

2 packages built:

python311Packages.timezonefinder
python311Packages.timezonefinder.dist


C++ build needs Java on OS X

C++ build needs Java on OS X even this is not mentioned in the list of prerequisites for c++.
Not an issue on my Mac. But I would like to build on an embedded arm64 system where there may be no java available...

The SBE compiler is written in Java and this is needed to generate the codecs. You can build the codec without Java on another platform. Where is the list of C++ prerequisites you are referring to?

Thanks. Sorry what is the SBE-compiler ??
Prerequisites....
It is on the project root-dir in the readme-file...

snip

C++ Build
You require the following to build the C++ API for Aeron:
3.6.1 or higher of CMake
C++11 supported compiler for the supported platform
C11 supported compiler for the supported platform
Requirements to build HdrHistogram_c.
Note: Aeron support is available for 64-bit Linux, OSX, and Windows.
For convenience, the cppbuild script does a full clean, build, and test of all targets as a Release build.

SBE: https://github.com/real-logic/simple-binary-encoding.
I've updated the README.md to reflect the build requirement.

I did this
sudo ln -sfn /opt/homebrew/opt/openjdk/libexec/openjdk.jdk /Library/Java/JavaVirtualMachines/openjdk.jdk
java -version


Dead link in azure-init

https://github.com/cloudfoundry/docs-bosh/blob/master/content/init-azure.md?plain=1#L9 link seems to be dead an leaves to a 404 page

@schmidtsv do you want to provide a pr for this? It looks like the place of the templates changed to https://github.com/Azure/azure-quickstart-templates/tree/master/application-workloads/bosh/bosh-setup.

Hi I created a fix for this issue: https://github.com/cloudfoundry/docs-bosh/pull/772


Devicelab agent mac18 has been non-pingable for days

For some reason, mac18 is not pingable for a while. A physical investigation is needed.
\cc @godofredoc

Machine was powered off. I just turned it on again and it should be back online.
@keyonghan can we power cycle it to ensure it comes back after a power cycle?

Confirmed it comes back after reboot.


Unable to sign PSBT with Ledger Devices...

Not sure if i'm missing something but I've imported my Unchained Vault into Sparrow, as part of experimenting I sent some Sats to a new address in Sparrow rather than the Unchained Address for the Vault. As a result I have a new UTXO in the Vault that Unchained can't see but Sparrow can.
My 2 of 3 keys are ledger (rightly or wrongly for this vault) so when I create a new PSBT to withdraw the UTXO to the Unchained recognised address I can't for the life of me work out how to sign it with the 2 ledgers.
I couldn't work it out before I updated the Bitcoin app on ledger to 2.0 and the firmware and also after the update.
Are there instructions for signing PSBT's generated by Sparrow with ledgers?

I'm unclear from your post on what the setup of your Sparrow wallet is (and how it differs from the Unchained one). But in answer to your question:
It should be as simple as clicking the "Sign" button on the Transaction screen. The guide for Coldcard does show the process, but the Sign button is greyed out there as the signing is done airgapped. For your wallet, it should be enabled. You should then see a dialog with the Ledger (whichever is connected) and you can sign from there.

I create the transaction, plug in the first of my ledgers and this is all I see:

If you go to the wallet Settings, what Type are the keystores? They should be Connected Hardware Wallet.

"Watch only wallet" I didn't change anything after importing the wallet configs. Do I need to re-import each of the keys and can I assume that doesn't do anything detrimental to the multisig vault?

You need to create a new wallet, replicating the current Sparrow wallet but importing the Ledger(s) as Connected Hardware Wallets (I'm still not clear if your Sparrow wallet is single or multisig). There's no risk to funds in doing this.

I did an File > Import Wallet > Caravan Multisig
and then imported the json file. It's then set up the keystores as watch only wallets.

That's strange. Can you compare your caravan json file with the sample one at https://github.com/sparrowwallet/sparrow/blob/master/src/test/resources/com/sparrowwallet/sparrow/io/caravan-multisig-export-1.json?
Sparrow looks at the field "method" in extendedPublicKeys to determine the keystore type. e.g.
"method": "ledger"

Does yours differ?

There's no method field in the json

Ok that explains it, although I'm unclear why they weren't included. It may be easiest to add the fields to the json file and import the file again.

Yep will try that in the morning, thank you so much! Will let you know. Have let unchained know too.

That worked a treat once I remembered to add an additional , after the prior label. All working perfectly now, thanks for the assistance. I've suggested that unchained add the label to the output when they generate the JSON too, can't believe i'm the first to bump up against this but least we worked it out.


Atom editor does not start up for me.

.ova image
Shows 'loading icon' for a bit but eventually does not start.

@martenson which image did you used? Do you have a link?

I believe @davebx is looking at it. I used the latest from images.galaxyproject.org

the link is https://images.galaxyproject.org/planemo/latest.ova

This might be fixed in #75

This is not resolved, behaves the same. :(

I have found some more details, when I try to execute it from the shell
$ /opt/atom/atom %F
A JavaScript error occurred in the main process
Uncaught Exception:
Error: Failed to set path
    at Error (native)
    at Object.<anonymous> (/opt/atom/resources/atom.asar/browser/lib/init.js:142:5)
    at Module._compile (module.js:425:26)
    at Object.Module._extensions..js (module.js:432:10)
    at Module.load (module.js:356:32)
    at Function.Module._load (module.js:313:12)
    at Function.Module.runMain (module.js:457:10)
    at startup (node.js:151:18)
    at node.js:1007:3

seems fixed in http://images.galaxyproject.org/planemo/20160623.ova

Okay - I have updated latest.ova to point to this new image - a big thanks to whoever built and uploaded it!


secret stores support HuaweiCloud CSMS

Description
Add HuaweiCloud Cloud Secret Management Service (CSMS) as a secret component.
https://support.huaweicloud.com/intl/en-us/usermanual-dew/dew_01_9994.html
Issue reference
We strive to have all PR being opened based on an issue, where the problem or feature have been discussed prior to implementation.
Please reference the issue this PR will close: #[1704]
Checklist
Please make sure you've completed the relevant tasks for this PR, out of the following list:

[x] Code compiles correctly
[x] Created/updated tests
[x] Extended the documentation / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[2418]

Please fix linter failures goimports, gofumpt etc

@berndverst Let me check it and I will back.

Close this PR and recreate a new one to solve the signoff issue.


Added tdlib new php extension for tdlib

Added tdlib new php extension for tdlib which integrates via tdjson JSON interface

Thanks, @aurimasniekis!


[building] Recommendation to add inline scripts in the index.html

My use case is add the Google Analytics script.
It's something like:
<!-- Global site tag (gtag.js) - Google Analytics -->
<script async src="https://www.googletagmanager.com/gtag/js?id=UA-XXXXX"></script>
<script>
  window.dataLayer = window.dataLayer || [];
  function gtag(){dataLayer.push(arguments);}
  gtag('js', new Date());

  gtag('config', 'UA-XXXXX');
</script>

I think the best idea could be have some option to add this script to the index.html in the build step. Because we don't needed during the development process, just in production/staging.

I agree that we need to do something here, I've had some different ideas but I haven't had time to work it out yet.

Not stale, working on this.

Any news here? Thanks!

I have almost finished a refactor of the html plugin, this allow you more customizations like this.

Great, thanks Lars! 👏

@LarsDenBakker this is fixed with the last version?

Yes you can add it in the html plugin

Thanks!
Reference to docs: https://github.com/open-wc/open-wc/tree/master/packages/rollup-plugin-html#transform-output-html


Webview breaks Angular bindings within template

Hey there,
I hope there is a simple solution to this problem
Problem
I have a webview where I do a few actions [payment gateway],
when the action is complete (I check this by looking at the onLoadFinishedEvent and its url which matches a predefined url I set), is set a component boolean field to TRUE.
This boolean field is binded to a Label for debugging purposed, its default value is false, but when the function is reached where it changes it to true, the label value doesnt change.
Tried
I put a button in the component which simply alerts the boolean status, and believe it or not, the instant I press the button, the label changes to true and the alert pops up and says true.
Possible Reason
I feel as if the "focus" of the component is within the webview and the rest of the component "freezes" while in that webview focus so variable changes don't work etc.

Was on wrong account, sigh. This is my main account

And I have to mention that if I do not alert the boolean field it doesnt work. So I have to access the field for it to update

Used ChangeDetectorRef detectChanges() method and this worked


feat: Generalize quadratic isometries to quadratic maps

Also .prod and .pi, since these are heavily entangled with Isometry and IsometryEquiv.
Follows on from #7569.

maintainer merge

bors merge


Enhancement: add ability to unpatch evaled templates

el-patch-unpatch does not work for them.

Really? They should just expand to a standard el-patch form, which means they should work with the normal registration and deregistration mechanism. Is there a bug?


CM-801: Remove unused fields from public-advisories/items

Jira Ticket:
CM-801
Description:
Removed fields from this API call that are never used on the website.  The lists grow very large when a public advisory is associated with many parks.

I had an error when I run gatsby build:
ERROR #95313

Building static HTML failed for path "/intro/"

See our docs page for more info on this error:
https://gatsby.dev/debug-html

  19 |
  20 |  // Split the array in 2 parts
> 21 |  var left = components.slice(0, split);
     | ^
  22 |  var right = components.slice(split);
  23 |
  24 |  return Array.prototype.concat.call([], decodeComponents(left),
decodeComponents(right));

I think it'll be fixed if you have this commit: https://github.com/bcgov/bcparks.ca/pull/818


[Feature] Add a New file upload form

Description
I want to add a new component (a file upload form) in the forms section under components. Please assign me this issue under IWOC
Screenshots
No response
Checklist

[X] I have checked the existing issues.
[X] I follow Contributing Guidelines of this project.
[X] I am a IWOC'24 contributor.
[ ] I am a JWOC'24 contributor.
[ ] I am a SWOC'24 contributor.

Code of Conduct

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

Proceed @SoumyadevSaha


assignment in if statments in bus.js

Running the code through jslint and found assignement in if statements.
Just want to check to see if that's what you intended.
https://github.com/sidorares/node-dbus/blob/master/lib/bus.js#L123
https://github.com/sidorares/node-dbus/blob/master/lib/bus.js#L128

I think it's intended but I'd rewrite it to be more clean


Download Links fail from Baidu

Hi,
Thanks for the dataset.
I am downloading the dataset, and the subset can be downloaded from Google. but the all data cannot be downloaded from Baidu.
Can you check it?
Thanks.

Hi,
Thanks for your attention.
Could you describe your operation on Baidu？
By the way, do you use the code "v6mt" to access the data?
Thanks.

Hi,
Thanks for your feedback.
I find out root course due to taiwan IP issue...
Current can be downloaded from Baidu.
Thanks.


Add negative test for IN parsing

While reviewing https://github.com/sqlparser-rs/sqlparser-rs/pull/463 I wrote this test for expected errors errors when trying to parse <expr> IN <expr>

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

6 of 6 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.007%) to 90.369%

Totals

Change from base Build<PHONE_NUMBER>:
0.007%

Covered Lines:
8004

Relevant Lines:
8857

💛  - Coveralls


Run cloud-init -execute after all config files have been written by console start script...

Run cloud-init -execute after all config files have been written by console start script...
so they can be overridden by cloud-init -execute

E.g. to make sshd configurable: http://forums.rancher.com/t/change-the-default-sshd-port/859


Avoid checking the same conditions multiple times to improve performance

Also enabled strict mode and removed a global variable.

This was a great PR @theodorejb. Thanks! That switch statement.. why on earth did I not think of that myself!?


Backport rawinput fix for do_debug

Might be worth backporting / applying: https://github.com/python/cpython/pull/2947

Done and released.


[staging] Add recent changes to new branch

Changes:
#231
#232
#233

Pull Request Test Coverage Report for Build 1000

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-49.9%) to 0.0%

Totals

Change from base Build 993:
-49.9%

Covered Lines:
0

Relevant Lines:
0

💛  - Coveralls


Refs #24997 -- Allowed bulk_create on a proxy model

This is a review-only pull request for forward-porting the fix from #1 to Django 1.9 on the master branch.

Renaming the branch, so closing this PR


Remote sessions prompted for Windows password twice

Windows Version 10.0.16299.64   RDPWrap
For a few weeks now, when remote sessions log in to Windows, the login screen goes away and then it comes back as if the password was incorrect, but it was entered correctly and there are no error messages.
With malware, this is an indication of something trying to grab the user's password by presenting a fake logon screen.
Norton doesn't detect any issues, so I'm suspecting it is related to the new version of Windows and rdpwrap?  It never happens on the console, only remote sessions.
Has anyone else experienced this issue?

Jup. I see exactly the same on a win10 pro install.
I just reinstalled the machine and the "problem" persists. Cant tell for sure  when it has begun, it is new for sure, but I don't use windows that often, only for testing software. (why I didn't hesitate running the reinstall either :)
I'm not perfectly sure which version I'm running. It might be 1.6.1, but rdpconf says <IP_ADDRESS> and 10.0.16299.15. Probably not the very latest, haven't bothered upgrading for a while.

@KevinRoo & @MortenGuldager
Can you confirm that you don't use Network Level Authorization?

I've been having the same issue, seemed to start with last fall's windows update, and it seems to be worse (more accounts, at least) since the spring update. It seems to happen in situations where the credentials aren't cached. It's mostly an annoyance, but I'd be happy to troubleshoot if there isn't a known solution.

Hi, I'm not sure it's the same issue, but I'm using a win7 Pro x64 "as a remote server".  Credentials are saved in .rdp file of users but when connecting to the server it shows the screen with all opened session and doesn't resume existing session (shadow session?), so user has to clic on his login and enter password again.  Any idea how to fix this ?  (we are not  using Network Level Authorization on the server). thanks

It stopped for me after changing to NLA. A bit slower for logon, but otherwise works fine.

Closed as solved. If not - reopen it.


Drop non-compliant Promises/A+  progress/notify on $q

Makes it really hard to replace $q when using a compliant library, since angular uses progress/notify handlers on core itself.

Would it be possible to know what library are you trying to use? The extra
parameters should be ignored, so more info is needed

I managed to replace $q with bluebird, and set the scheduler to $rootScope.$applyAsync. I was trying to use $interval but uses notify. Another module included in my project also uses notify from $http.
I implemented the defer method manually, along with when.
        Bluebird.defer = function() {
            var dfd: any = {};

            dfd.promise = new Bluebird(function(_resolve, _reject) {
                dfd.resolve = _resolve;
                dfd.reject = _reject;
                dfd.notify = angular.noop;
            });

            return dfd;
        };

        var bb: any = Bluebird;
        bb.when = Bluebird.resolve.bind(Bluebird);

        var originalAll = Bluebird.all;

        Bluebird.all = function(promises: any|any[]) {

            if (angular.isObject(promises) && !angular.isArray(promises)) {
                return Bluebird.props(promises);
            } else {
                return originalAll.call(Bluebird, promises);
            }

        };

        Bluebird.onPossiblyUnhandledRejection(angular.noop);

        function BluebirdConstructor(cb: any): any {
            if (!(this instanceof Bluebird)) {
                return new Bluebird(cb);
            }
            return Bluebird;
        }

        angular.extend(BluebirdConstructor, Bluebird);

        $provide.decorator('$q', function() {
            return BluebirdConstructor;
        });

I was hit by this problem as well. @pocesar have you been able to fix $interval. It seems to me I would need to replace the whole service to make it work.

would it be possible to have a plunker that shows the entire problem?

http://plnkr.co/edit/ITbUfnXOogSdO2r1w52J?p=preview
If you remove 'promise' module from dependencies you can see interval ticking as expected
The problem seems to be $interval depending on promise progress notification that is no longer part of promises

@Agamnentzar thanks for the plunker. I think there should be an easy fix.

btw. a dumb/simplified workaround for $interval needing notify is simply swapping it with a custom one:
  app.config(function($provide) {
    $provide.decorator('$interval', function($delegate, $rootScope) {
      var $newInterval = function(fn, timeout) {
        if(arguments.length !== 2 || typeof fn !== 'function' || typeof timeout !== 'number') {
          return console.error(new Error('custom $interval supports only 2 arguments, Function and timeout'));
        }

        var interval = setInterval(function() {
          $rootScope.$apply(fn);
        }, timeout);

        return interval;
      };

      $newInterval.cancel = function(handle) {
        clearInterval(handle);
      };

      return $newInterval;
    });
  });

Given that dropping the API is a breaking change to $q, it probably is safe to close this issue unless it is desired to remove it completely from the codebase. To fully remove notify usage from $interval would require a breaking change as well (under the assumption someone is hooking into the notifications from the interval promise).
Not sure how it is felt, but IMO it probably isn't worth removing since this is a more uncommon case.

@lgalfaso, WDYT (since this is assigned to you) ?

Given that the issue with $interval is fixed, I would close this issue.
On Friday, June 3, 2016, Georgios Kalpakas<EMAIL_ADDRESS>wrote:

@lgalfaso https://github.com/lgalfaso, WDYT (since this is assigned to
you) ?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/angular/angular.js/issues/12533#issuecomment-223588760,
or mute the thread
https://github.com/notifications/unsubscribe/AAX44urCgJAgQv8yD7cOULKn6IAoNEBfks5qIDWogaJpZM4FoUuu
.

Thx @lgalfaso!
So, closing this (since we are not going to remove the notify method), but since Angular itself does not rely on notify for any internal fnctionality any more, you should be able to substitute $q for other promise libraries (e.g.  as described in https://github.com/angular/angular.js/issues/12533#issuecomment-129262925).


Turn data ingest into OOP

Improve the organisation of the code

Consider whether the zip file needs to be written out before extracting. It might be that it can be unzipped in memory. Or maybe request a single file from the zip.
Class attributes:

these are the variables or state
CRS is fixed, so will be in __init__

Methods:

main data object would be a geopandas df, so all methods that read or write would do so from the gpd_df
check file exists (for csv, feather, zip etc). If filepath is passed as a parameter this might be a static method, decorator @static, otherwise if path is stored in class, then it's a standard method)
csv to pandas dataframe
json to pandas df (possibly other formats as well)
read shp file --> geopandas df
pandas to geopandas df
writes to csv (for persistence)
writes to feather (for persistence)
retrieve data from zip file

I have referred to this issue in #120 .
It might be a good idea to solve this ticket before that.
Not only would the any_to_pd function be better as a class, but also it also needs to:

have an method/function to handle Excel files
the Excel handler needs to accept parameters for extracting particular sheets in a workbook
possibly needs to be able to be able to extract particular rows/columns in any particular sheet too, depending on the structure of the population estimates we are downloading.

Make use of ABC's abstractmethod
See the Core Programming section in the BPI. There is a section on Classes and using the abstractmethod decorator.


Incorrect error for batching

I ran some batch with 30000 entries and it works correctly. With 40000 it does not work. I am fine with that however, the errors appears incorrect. The error that occurs is
thread 'main' panicked at 'called `Result::unwrap()` on an `Err` value: InvalidMessage("Frame error: Could not deserialize frame: Integer conversion out of range")', src/main.rs:58:53

The code that is triggering it is
use bcs::serialize_into;
use scylla::query::Query;
use scylla::{IntoTypedRows, Session, SessionBuilder};

fn get_upper_bound_option(key_prefix: &[u8]) -> Option<Vec<u8>> {
    let len = key_prefix.len();
    for i in (0..len).rev() {
        let val = key_prefix[i];
        if val < u8::MAX {
            let mut upper_bound = key_prefix[0..i + 1].to_vec();
            upper_bound[i] += 1;
            return Some(upper_bound);
        }
    }
    None
}

async fn find_keys_by_prefix(session: &Session, key_prefix: Vec<u8>) -> Vec<Vec<u8>> {
    let len = key_prefix.len();
    let rows = match get_upper_bound_option(&key_prefix) {
        None => {
            let values = (key_prefix,);
            let query = "SELECT k FROM kv.pairs WHERE dummy = 0 AND k >= ? ALLOW FILTERING";
            session.query(query, values).await.unwrap()
        }
        Some(upper_bound) => {
            let values = (key_prefix, upper_bound);
            let query =
                "SELECT k FROM kv.pairs WHERE dummy = 0 AND k >= ? AND k < ? ALLOW FILTERING";
            session.query(query, values).await.unwrap()
        }
    };
    let mut keys = Vec::new();
    if let Some(rows) = rows.rows {
        for row in rows.into_typed::<(Vec<u8>,)>() {
            let key = row.unwrap();
            let short_key = key.0[len..].to_vec();
            keys.push(short_key);
        }
    }
    keys
}

async fn write_batch(session: &Session, n: usize) {
    let mut batch_query =
        scylla::statement::batch::Batch::new(scylla::frame::request::batch::BatchType::Logged);
    let mut batch_values = Vec::new();
    for i in 0..n {
        let mut key = vec![0];
        serialize_into(&mut key, &(i as usize)).unwrap();
        let value = key.clone();
        let query = "INSERT INTO kv.pairs (dummy, k, v) VALUES (0, ?, ?)";
        let values = vec![key, value];
        batch_values.push(values);
        let query = Query::new(query);
        batch_query.append_statement(query);
    }
    session.batch(&batch_query, batch_values).await.unwrap();
}

async fn create_test_session() -> Session {
    let session_builder = SessionBuilder::new().known_node("localhost:9042");
    let session = session_builder.build().await.unwrap();
    session
        .query(
            "CREATE KEYSPACE IF NOT EXISTS kv WITH REPLICATION = { 'class' : 'SimpleStrategy', 'replication_factor' : 1 }",
            &[],
        )
        .await.unwrap();
    session
        .query("DROP TABLE IF EXISTS kv.pairs;", &[])
        .await
        .unwrap();
    session
        .query(
            "CREATE TABLE IF NOT EXISTS kv.pairs (dummy int, k blob, v blob, primary key (dummy, k))",
            &[],
        )
        .await.unwrap();
    session
}

#[tokio::main]
async fn main() {
    let session = create_test_session().await;

    let n = 40000;
    write_batch(&session, n).await;

    let key_prefix = vec![0];
    let keys = find_keys_by_prefix(&session, key_prefix.clone()).await;
    println!("key_prefix={:?} |keys|={}", key_prefix.clone(), keys.len());
}

@piodul @MathieuDutSik . Did a little digging around and found the root cause, it's here when trying to parse the length of the statement into an i16, specifically the try_into?
https://github.com/scylladb/scylla-rust-driver/blob/main/scylla-cql/src/frame/request/batch.rs#L84
Tried your example with i16:: MAX  and that worked, but i16::MAX + 1 failed with the same error
It may be worthwhile to add this limitation to the documentation or enforce the limitation at the method call boundary 🤷
Another option could be to just change it to an i32, but I don't know what the repercussions of that will be when serializing / deserializing, Happy to whip up a PR with some guidance as I'm a first-time contributor 😄

From the java docs, it seems a batch statement can take 65536 statements

Thank you for the report! It looks like the i16 is a bug.
The CQL  protocol spec says that the number of statements in a batch should be serialized as a 2-byte unsigned integer ([short]):
https://github.com/apache/cassandra/blob/865d7c30e4755e74c4e4d26205a7aed4cfb55710/doc/native_protocol_v4.spec#L451
https://github.com/apache/cassandra/blob/865d7c30e4755e74c4e4d26205a7aed4cfb55710/doc/native_protocol_v4.spec#L221C3-L221C3
We are serializing the length as an i16 which cuts the available range in half. I think it would be best to change the i16 to a u16. It's also possible that this bug is present in other places that (de)serialize [short] values, it would be good to go and take a look at all of them.

Happy to whip up a PR with some guidance as I'm a first-time contributor 😄

That would be great, thank you!
It's a standard Rust project, there are some additional tips in CONTRIBUTING.md.

Ok, will take a stab at it

PR here: https://github.com/scylladb/scylla-rust-driver/pull/824


clarification on annotation entries from alpaca_eval

I wanted to clarify the notation in the alpaca_eval command. Let's say I passed in --model_outputs my_model_A.json and
--reference_outputs my_model_B.json. The resulting annotations.json will contain keys output_1, output_2, and rankings under raw_completions:
    "raw_completion":{
      "concise_explanation":"....",
      "ordered_models":[
        {
          "model":"M",
          "rank":1
        },
        {
          "model":"m",
          "rank":2
        }
      ]
    }

According to this template, m corresponds to output_1 and M corresponds to output_2, correct? Then would this mean the model_outputs corresponds to m as well? I'm not sure which one takes precedence.
Thanks!

Do you see a field referenced_models alongside? It should look something like this:
 "referenced_models": {     "M": "output_1",     "m": "output_2"   }
which should be easier to understand / hopefully resolves the confusion

Done @xwinxu , to add referenced_models you need to (1) update alpaca_eval, and (2) rerun the parsing by using the flag --is_reapply_parsing True. Note that this won't rerun the actual OAI annotations, which are cached.

Thanks Yann!


cargo format error on build for test job

I'm tracking the master branch remotely and building with circleci, I seem to be getting format errors like so
#!/bin/bash -eo pipefail
cargo fmt -- --write-mode=diff
Diff in /root/project/src/server.rs at line 226:
     let uuid_display = uuid_path.display();⏎
 ⏎
     let mut uuid_file = match File::create(&uuid_path) {⏎
-        Err(why) => panic!("couldn't create {}: {}",⏎
-                           uuid_display,⏎
-                           why.description()),⏎
+        Err(why) => panic!("couldn't create {}: {}", uuid_display, why.description()),⏎
         Ok(uuid_file) => uuid_file,⏎
     };⏎
 ⏎
Diff in /root/project/src/server.rs at line 233:
     match uuid_file.write_all(uuid.as_bytes()) {⏎
-        Err(why) => {⏎
-            panic!("couldn't write to {}: {}", uuid_display,⏎
-                                               why.description())⏎
-        },⏎
+        Err(why) => panic!("couldn't write to {}: {}", uuid_display, why.description()),⏎
         Ok(_) => println!("successfully wrote to {}", uuid_display),⏎
     }⏎
 ⏎
Diff in /root/project/src/server.rs at line 240:
     let vpncreds_display = vpncreds_path.display();⏎
 ⏎
     let mut vpncreds_file = match File::create(&vpncreds_path) {⏎
-        Err(why) => panic!("couldn't create {}: {}",⏎
-                           vpncreds_display,⏎
-                           why.description()),⏎
+        Err(why) => panic!(⏎
+            "couldn't create {}: {}",⏎
+            vpncreds_display,⏎
+            why.description()⏎
+        ),⏎
         Ok(vpncreds_file) => vpncreds_file,⏎
     };⏎
 ⏎
Diff in /root/project/src/server.rs at line 251:
     match vpncreds_file.write_all(vpncreds.as_bytes()) {⏎
-        Err(why) => {⏎
-            panic!("couldn't write to {}: {}", vpncreds_display,⏎
-                                               why.description())⏎
-        },⏎
+        Err(why) => panic!(⏎
+            "couldn't write to {}: {}",⏎
+            vpncreds_display,⏎
+            why.description()⏎
+        ),⏎
         Ok(_) => println!("successfully wrote to {}", vpncreds_display),⏎
     }⏎
 ⏎
Exited with code 4

I'm unsure what to make of these and so far have gotten by by using
- run: cargo fmt -- --write-mode=overwrite
- run: cargo clippy

This feels kinda hacky and I'm not sure if there is a more correct way to address what the build error is trying to tell me.
What's most perplexing is that i'm confident these are files I have not changed and should have parity with your master branch.
Thoughts?

On the test job in the CircleCI configuration there is a check whether cargo fmt (rustfmt) was executed before commits. Here is the output for the current 4.0.6 version: https://circleci.com/gh/resin-io/resin-wifi-connect/862.
On your side you are most probably running a bit older version of rustfmt, which leads to the warnings. You are using probably our older build toolchains. One way is to upgrade to the newer toolchains. Or you may just remove the cargo fmt -- --write-mode=overwrite check from your CircleCI configuration, since this is just a formatting check.
Please let me know if my assumptions are incorrect.

Suppressing clippy warnings seems to have worked.
Comparing our runs side by side I think they are the same, least I can't see where the version of rustfmt is different.
I'm using the same nightly container and it's near identical as far as I can tell.

@dhoney oh, now looking at the exact diff again, the difference is in the modifications on your side. I don't know where I was looking at... :) All makes sense now, since that part was not processed with the same rustfmt version.
I will reopen the ticket and change its name, since it is a problem for people doing forks when they try to use the same build environment. Probably I need to provide some git pre-commit hooks for other people wanting to maintain forks that will not allow pushing code that does not pass the clippy or rustfmt checks on the local machine.
BTW I spend some time today working on a new cross-compilation workflow based on crosstool-NG, since our current images are tricky to upgrade as some of them are based on the Emdebian toolchains which are getting outdated. The Rust compiler itself is cross-compiled for the different platforms with the help of toolchains generated with crosstool-NG. The machine code itself is generated by the Rust compiler, but the cross-compilation toolchains are needed for linking and stripping the final binaries. In the case with WiFi Connect, those are used for linking to libdbus-1 more specifically. Anyway, just mentioning it, since somewhere in the near future we need to have a more reliable way for cross-compiling Rust binaries.

Something like that as a title..


[MENFORCER-372] Drop PlexusTestCase and junit:junit dependency

Following this checklist to help us incorporate your
contribution quickly and easily:

[x] Make sure there is a JIRA issue filed
for the change (usually before you start working on it).  Trivial changes like typos do not
require a JIRA issue.  Your pull request should address just this issue, without
pulling in other changes.
[x] Each commit in the pull request should have a meaningful subject line and body.
[x] Format the pull request title like [MENFORCER-XXX] - Fixes bug in ApproximateQuantiles,
where you replace MENFORCER-XXX with the appropriate JIRA issue. Best practice
is to use the JIRA issue title in the pull request title and in the first line of the
commit message.
[x] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[x] Run mvn clean verify to make sure basic checks pass. A more thorough check will
be performed on your pull request automatically.
[x] You have run the integration tests successfully (mvn -Prun-its clean verify).

If your pull request is about ~20 lines of code you don't need to sign an
Individual Contributor License Agreement if you are unsure
please ask on the developers list.
To make clear that you license your contribution under
the Apache License Version 2.0, January 2004
you have to acknowledge this by using the following check-box.

[x] I hereby declare this contribution to be licenced under the Apache License Version 2.0, January 2004

[x] In any other case, please file an Apache Individual Contributor License Agreement.

Turns out it was way easier to remove PlexusTestCase than I had originally thought. :sweat_smile:

Hmm the additional test failures are somewhat puzzling; needs a closer look, which in my case is limited to evenings/weekends. :disappointed:

Turns out we did need the junit-jupiter-engine after all; Time for another quick review @slawekjaranowski ?

hint add to pluginManagment in root pom
        <plugin>
          <groupId>org.apache.maven.plugins</groupId>
          <artifactId>maven-surefire-plugin</artifactId>
          <version>3.0.0-M5</version>
        </plugin>

Done! That indeed helps, and seems a nice addition to not have to declare the engine anymore.
Would you want to restore the mockito-core dependency + management? Keep in mind that would trigger Dependabot twice, or require a version variable.

For many components from the same project please use property as version and will be one update by dependabot.
Please also check if maven-plugin-testing-harness, maven-compact is still needed.

Went through the mvn dependency:analyze output once more and removed all unused declared dependencies.


Using code.cmd --installextension results in a [createInstance] warning

VSCode Version: 1.27.2
OS Version: - X64 Windows

Steps to Reproduce:

Use the command line for vscode "code.cmd --install-extension
VSCode logs to the console (through console.warn)
"[createInstance] extensionManagementService depends on downloadService which is NOT registered." (this happens in VSCode

Does this issue occur when all extensions are disabled?: Yes
This is an issue for us beause we have scripts and eventually a larger wrapped installer that are trying to detect whether either an install or upgrade of an VSCode extension has failed.
Currently the exit code of the process is always zero, so that cannot be used as an indicator, errors do come through "Standard Error", but due to the warning above, without doing some type of string parsing, we cannot reliably tell if the extension successfully installed.

Fixed. Please try latest insiders or wait for new stable release (in couple of days)


v0.9.2: Vision: Add transform for resize.

Currently, transforms only handle flatten and unflatten.
Need to handle transform for resizing.
Ex. images.resize = (h, w)

done in v0.9.3


Update configuring-a-service.md - update --data "url=" to "host="

Review

@reviewer
Summary

Reason

Testing

@awi93 Hi Andy and thanks for your PR! Can you explain the need for it? Did you encounter issues with the host attribute? The url attribute is supported, and as the docs state, it's a "[s]horthand attribute to set protocol, host, port and path at once. This attribute is write-only (the Admin API never returns the URL)."
If you encountered issues with it, however, we'd like to know about it. You can file an issue, or reply to this comment. Thanks again!

@awi93 it's been two weeks since we asked for any underlying issues that might have produced the need for this PR, and we haven't heard from you. Closing to keep the queue clean -- if you still want to respond, feel free to reopen with an explanation. Thanks!


Lazy-loading chemistry dependencies

I was able to eliminate dependency on baybe.utils.chemistry and dynamically load it when needed.
In optional.py I wrote a function for lazy loading modules or a specific attribute from a module.

Hi @rjavadi, thanks for the PR. I'll have a detailed look as soon as I find time for it, but just the sneak peak from the checks tells me that you seem not to have installed our pre-commit hooks, which would have prevented committing code that fails linting. In the contributing guidelines, you'll find a suggested workflow, including a step to set up the linters. Once activated, they will help you isolate the code problems and you can easily fix the errors you currently see in the pipeline 🙃

I couldn't reproduce mypy's type checking error on my local machine, I'm working on it.

@AdrianSosic
One of the serious downsides of lazy loading modules is that you are going to lose the ability to look into the loaded module. Since modules are loaded at runtime, errors might not be caught until the code execution reaches the point of import. This can make debugging more challenging. Also, If a module has dependencies that aren't installed, the error won't be revealed until the import attempt, potentially leading to unexpected crashes.
You win some performance, and lose code maintenance.

@AdrianSosic @Scienfitz
I removed the validator method, and imported baybe.utils.chemistry the way @AdrianSosic mentioned.
I kept optional.py for possible future uses for now. If we don't end up using it, I'll remove it.

Hi @rjavadi, thanks for pushing the new changes, I‘ll have a look tomorrow. Meanwhile, could you do me a favor and rebase your branch as preparation for the upcoming merge?

@AdrianSosic
I believe this branch is now a mess after so many tries to make commits linear and resolve conflicts. I can transfer changes to a new branch if you wish.

Hey @rjavadi, no need to create a new branch, which would also require to set up a new PR and so on ... I've taken the liberty of rebasing your branch quickly so that we have a clean linear history on top of the current main. Apart from the dozens of commits that resulted from your previous merges, the latest merge was unfortunately not done cleanly and reverted some of the latest changes from our main, breaking the existing code. You can easily see this by comparing the top of your branch (which I've backup up under "backup/import-optimization") against the main branch. You'll see that the changes also affect other files (e.g. interval.py, targets.md, ...) which should have nothing to do with your PR. Anyway, this is fixed now. I'll have look at your latest version now and see if everything's OK or if there are further changes needed 👍🏼

Hi @AVHopp, @rjavadi, @Scienfitz,
I've rebased Roya's commits and also made the remaining changes I considered missing. From my end, the PR is now ready for merge 🚀
@rjavadi: Could you please carefully go through my additional commits and 1) check if you agree with the additional changes  and 2) if yes, keep them in mind for future refactorings. What I did mostly is taking care of the missing refactoring of the boolean helper variables that were still around at other places in the code.
@AVHopp, @Scienfitz: please have another look and let me know if you see problems. Otherwise, approve 😃

Thank you @AdrianSosic for fixing this.
I checked the new commits. It looks good to me.

Hi @rjavadi, agree with Martin, feel free to add your name with the appropriate affiliation to the contributors list, afterwards I’ll merge 👌🏼


Documentation Issue. Specific instance URLs no longer recommended.

Correct me if I'm wrong. But, as I recall, we're supposed to be using https://login.salesforce.com for all API connections to prod.
The documentation here ( https://jsforce.github.io/document/#query ) may be slightly misleading by having a suggestion to use "na1.salesforce.com"
Thanks

closing this, na1.salesforce.com is mentioned as an example of an actual instance URL.
We recommend always passing the My Domain URL.


Extrem short prompt wav can not be generated

Checks

[X] This template is only for usage issues encountered.
[X] I have thoroughly reviewed the project documentation but couldn't find information to solve my problem.
[X] I have searched for existing issues, including closed ones, and couldn't find a solution.
[X] I confirm that I am using English to submit this report in order to facilitate communication.

Environment Details
I found it maybe a bug or might my processing was not right.
Here is the problem:

Extrem short prompt wav, less than 1s, can not be generate, even thought it has more than 2 words;
vgfbv

Steps to Reproduce
fev
✔ Expected Behavior
20241206_154831_115.7_116.2.wav.webm
th
20241206_154831_115.7_116.2_output.wav.webm
e
the promt wav have prompt text: 走一个
and the target text set to go one
but this siutation actully can not produce anything. Other situation is normal.
What should do it?
❌ Actual Behavior
fregtfeg

Set the speed for the generation much lower when generating short sentences - like 0.3


Fleet merge confuses the "go to" pointer

How I saw this happen: on my primary planet I created a colony ship and in the turn when it appeared, a cargo ship arrived on-planet. I merged them to create a bigger colony fleet.
When I got to the message that the cargo ship's mission had completed and clicked "go to," it took me to a different ship on a different planet.
The funny thing: I believe this may have been a bug in the original DOS Stars! Go figure.

The goto is based on the fleet number, which goes away when merging.
There is probably a solution. I could update message targets for a player on merge.

OG Stars! actually grays out the button. Good fix!


Fixes to Android GN instructions

Fix MarkDown errors that weren't noticed in #79

Don't you just love that the Sublime preview and the GitHub code review previews are much more forgiving to MarkDown errors than our real web site?
@louquillio is there a better way to preview so we can catch errors like this?

@hkjellander Oh, man, there are so many Markdown plugins for Sublime it'll make your head spin. Essentially, you want one backed by kramdown, cuz Jekyll3/gh-pages now uses it exclusively. Not obvious to me (looking quickly) that there is such a beast for Sublime, but it does seem there's at lease one for Atom: https://atom.io/packages/markdown-preview-kramdown


Do not mutate source Image in Image::save_jpg and use encoder return value

In the patch, p_img comes from casting a this pointer a few stack frames above:
https://github.com/godotengine/godot/blob/4c96e9676b66d0cc9a25022b019b78f4c20ddc60/core/io/image.cpp#L2529
Meaning that calling Image::save_jpg may eventually call Image::convert which mutates the source Image data into a different format if needed to overcome an encoder bug. See https://github.com/godotengine/godot/pull/66929#issue-1397725680.
I don't think the source Image data should be mutated just to save it. Additionally the encoder return value was not previously used.
This is a follow up to https://github.com/godotengine/godot/pull/84459#discussion_r1383346570.

It's highly likely the other image importers have this pattern to be checked.

It's highly likely the other image importers have this pattern to be checked.

I checked PNG in image_to_png, it seems to operate on a duplicate already. Could be worth checking others.

Thanks!


Where to get the MS1Mv2 train list？

Hi, you need to generate the training list by yourself (format: "path_to_img, 0, id, 0").
Here is an example:

THANKS FOR YOUR REPLY！！！


Rebase while uploading release

If trying to upload a dev release to the director, is there any way to emulate the rebase flag's functionality?

If you pass the resource the release tarball via the releases configuration the resource will ensure the version in the deployment manifest matches that of the tarball. Does that sufficiently cover the use case for you?

@drich10 I suppose if I name the release with the sha or something. The problem arises when I don't name it and it collides with a previous dev release.

Got it. I'm going to do some investigation and hopefully add the functionality if it is within reason. I will report back shortly!

@apoydence had a few conversations with bosh folks. The recommendation I received was to create the release with the --timestamp-version flag. This should give you increasing versions across runs of CI. Is that a reasonable solution for you?

@drich10 Would this occur for only dev releases?

at the end of the pipeline, once dev passed end to end tests for example, you can use "bosh finalize-release dev.tgz" and commit final metadata. if you dont specify --version on finalize-release it will pick next final version automatically. (https://bosh.io/docs/cli-v2.html#release-creation)
Sent from my iPhone

On Jul 11, 2017, at 7:26 PM, Andrew Poydence<EMAIL_ADDRESS>wrote:
@drich10 Would this occur for only dev releases?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

We will update our pipeline to do this. Closing.


Trigger an action on load

I can't find a simple way to have livewire trigger an action on page load/ready. This is often very handy to ensure quick pageload times: display all "fast" content on first request, then display "slow" content via livewire after it is e.g. fetched from some third-party API.
Describe the solution you'd like
<div wire:load="fetchSlowContent">
{{ $mySlowContent }}
</div>

Is there any technical limitation, which prohibits doing this?
Describe alternatives you've considered
Have some button, that fetches/refreshes the slow content and trigger it automatically on pageload.
function click() {
   document.getElementById("fetcher").click()
}
document.addEventListener("DOMContentLoaded", click);

However, this is clearly not as elegant

Good call, I've thought about this too. Added wire:init on latest release (v0.1.6)

Awesome!


possible for a name change?

This library is great and i intend to use it forever.   However, i'm afraid that with a name that looks to english-speaking people as "piss words", this will not really get off the ground.   Many great python libraries don't have "py" in their names.   Its not a requirement, or even a convention.

Hello @colindickson
Yes, I've been thinking about a name change. English is not my first language so I didn't realise it might sound bad in other languages, for the moment it is complicated because of the structure of a pyssword database. But I am trying some ideas, if it works right I will change the name.
Thank you for your contribution

Done, package now is called passpie


Repology unlisted WinGet; no longer providing package statuses

The Repology section in the README.md does not work as WinGet has been temporarily delisted from Repology. https://github.com/repology/repology-updater/commit/e676156f23fe6e28d8d3098695edfd1c57cba082
https://github.com/microsoft/winget-pkgs/issues/156468#issuecomment-2146635690

https://github.com/microsoft/winget-pkgs/blob/20070f1f67a38f10522bb1c4bf86fd28b206381a/README.md?plain=1#L24-L27
Going to https://repology.org/repository/winget, you get:

Sorry, but Repology no longer tracks repository winget.
Last time Repology has processed this repository was 2024-05-31 12:20 (3 days ago).
Some of the reasons which may have caused this:

The repository has reached it's end of life, is no longer updated or supported, should no longer be used, and has probably disappeared from the mirrors. There may be a newer version of this repository available though, check the list.
Repository has stopped publishing information about it's packages in format usable to Repology, so it's can't be processed any more. You may try contacting repository owners and ask them to fix this.

https://github.com/repology/repology-updater/commit/e676156f23fe6e28d8d3098695edfd1c57cba082

Still not fixed after 1 month.

Still not fixed after 1 month.

Work is being done to address this. Ex:

https://github.com/microsoft/winget-cli/issues/1725
https://github.com/microsoft/winget-cli/pull/4588


docs: fix basepath

Current dependencies on/for this PR:

main

PR #24   👈

This comment was auto-generated by Graphite.


Revert "Bump actions/upload-pages-artifact from 2 to 3"

Reverts athackst/jekyll-theme-profile#188

Reverting this one as well until issue can be diagnosed.


!cancel and !pause command

Is your feature request related to a problem? Please describe.
IT can be very useful, if the bot has a couple commands, which will may be simplify the work with the bot for the user and admin.
Describe the feature you'd like
Two new commands:
!cancel (for everyone) - The cancels actual bot's trade offer. Works for user and admin.
!pause (for admins only) - That command will pause the bot works, putting it into the mode "Away" ir "Busy", and leave the non-game app. It can be useful for transferring items to someone (e.g. bptf donate)
Describe alternatives you've considered
None alternatives

I’ll work on cancel offer today, should be easy enough

I don’t think the pause command would be very useful though

Instead of !pause you maybe could add !donate.
It would look like this:
!donate name=Max's Severed Head&amount=1
And that would automatically send a trade offer to the backpack.tf donation bot with the correct item and amount.

Instead of !pause you maybe could add !donate.
It would look like this:
!donate name=Max's Severed Head&amount=1
And that would automatically send a trade offer to the backpack.tf donation bot with the correct item and amount.

Yes, but I think that there is already an issue for that

https://github.com/Nicklason/tf2-automatic/issues/132

Ah ok.

https://github.com/Nicklason/tf2-automatic/commit/4d4e28a2c55faab86d12982f2202f5eb3e8d6101
https://github.com/Nicklason/tf2-automatic/commit/58e3d2ee1052076450a635452a75a5688d147587


spin out new threads to reduce clutter when X replies to Y on a trading post

oh that's fixed haha


Added system property refine.max_form_content_size

To allow configuring the maximum allowed form content size in Jetty. We recieved
java.lang.IllegalStateException: Form too large

when uploading over 1MB of operation data to apply-operations. We wanted to be able to increase the hardcoded limit.
Being able to configure the allowed form content size can be useful in several cases, see https://github.com/OpenRefine/OpenRefine/issues/638 and https://github.com/OpenRefine/OpenRefine/issues/863.

How can i add the setting if i want to run directly the openrefine.exe file ?

I was able to get this to work by adding REFINE_MAX_FORM_CONTENT_SIZE=4194304 in the refine.ini file


Empty state 'Add sources' button does not navigate to sources page

Describe the bug
Empty state 'Add sources' button does not navigate to sources page
To Reproduce
Steps to reproduce the behaviour:

Go to 'Overview' on user without providers
Click on 'Add sources'
Add source modal appears

Expected behaviour
Should redirect to sources page
Screenshots

Desktop (please complete the following information):

OS: Fedora
Browser Firefox
Version 66

Additional context
Testing against PROD

It appears that you're testing with old code?
The "Add Sources" button was intended to launch the providers modal. It was never meant to navigate to the sources page.
That said, this has been changed to show a "View all sources" link which does navigate to the sources page.
See https://github.com/project-koku/koku-ui/issues/801


Static Tableviews don't hide section footers when hiding (the last?) rows

Hi there,
Thanks for sharing this!
When I hide the last row in a table it'll hide the section header, but not the section footer on a static UITableView. Is this an easy fix?

@peterpaulis thanks for the great lib! Could you please integrate these two methods in the codebase? I use StaticDataTableViewController as a pod and therefore I'd rather not patch the code manually.

Fixed in PR #28


Upgrade to PureScript 0.14.3 and add / update docs

Thanks for starting on the upgrade from 0.14.x! I never got around to it and am short on time at the moment. As you can see there is still some work to be done to get the exercise solutions updated as well. Hopefully I'll have some time in a couple of weeks.

@emiel Yeah, sorry for a pull request sent too early - I misread the Configlet CI action and thought the actual build / test CI action completed fine...
Anyway, this should be good now - please check and let me know if there's something you'd like me to correct before merging.


Manually provide documents for UI

Hi, I'm trying to use the package to render UI for my service, but I'm having troubles with code-first document generation. Some of the issues I'm facing are:

not possible to annotate single method with both PublishOperation and SubscribeOperation (this makes sense for example for WS service, where you use same websocket connection for messaging in both directions).
NPEs in generated pages for Message descriptions (probably because of weird message structure that we use).
etc

But I really like the UI that Neuroglia.AsyncApi provides!
So I'd like to be able to provide the documents for UI manually, e.g. using Building an AsyncAPI Document API.
Is it possible? Does it make sense at all?

Hey @mikhail-barg!

not possible to annotate single method with both PublishOperation and SubscribeOperation (this makes sense for example for WS service, where you use same websocket connection for messaging in both directions).

I see, it would indeed make sense in some case to mark an operation with both attributes. I'll make sure to enable that asap.
I just opened feature request #11  to do so.

NPEs in generated pages for Message descriptions (probably because of weird message structure that we use).

What are the NPEs you are facing? Can you elaborate a bit on this and provide some examples? The lib must be able to work with any and all (valid) messages, so I'd be happy to fix whatever's the problem(s) in your case.

So I'd like to be able to provide the documents for UI manually, e.g. using Building an AsyncAPI Document API.
Is it possible? Does it make sense at all?

Well, yes, it does make sense. However, I would personally avoid generating documentation manually, because it's both tedious, and is subject to changes (possibly on a daily basis during dev). I added the fluent builders specifically for supporting exotic use cases and definitions, but you should rather rely IMO, whenever possible, on the code-first generator, as you would with OpenAPI, for instance.

@mikhail-barg Hey, sorry for the late update!
Please take a look at https://github.com/neuroglia-io/asyncapi/releases/tag/v2.6.0!
It now features new extensions to manually add an AsyncAPI document to the UI, or to add one built programatically.
Usage
Configuration:
services.AddAsyncApiGeneration(builder =>
    builder.WithMarkupType<StreetLightsService>());

Retrieve generated documents:
var documentList = serviceProvider.GetRequiredService<IAsyncApiDocumentProvider>().ToList();

Add a document manually:
services.AddAsyncApiDocument(myDocument);

OR:
services.AddAsyncApiDocument(builder => builder
  .WithAsyncApiVersion(...)
  ...);


Feat/UUID

This is a minor addition for uuid::Uuid which just map it to a string.
It's gated behind a uuid-impl feature, similar to chrono-impl.
I also added it to the example :)

thanks!


Rename UIPageControl_SpecSpec to UIPageControlSpec+Spec to be inline with other Spec files.

Just caught this myself.
Addendum to https://github.com/pivotal/PivotalCoreKit/pull/197

Thanks!


Implement service: and image: fields

image: field is implemented in 7.8.0. Rename this issue

image: field is implemented in 7.8.0. Rename this issue


Proposal to Implement Versioning Using GitHub Releases

Hi maintainers!
I wanted to raise this issue regarding to the version management in this great project. I've noticed that it does not use GitHub's releases or tags to manage different versions of the package and the upstream project. While this approach can work for quick iterations, it might pose challenges as the project scales and the user base grows.
If there's interest in this approach, I'd be delighted to contribute to the setup of a versioning and release process based on GitHub's features. This could include updating upstream project dependency, tagging specific commits as releases, drafting initial release notes, and setting up a cycle for future releases.
I am also open and willing to assist in any other areas where you might need support, whether it’s further development, testing, or documentation.

We are more than interested to use GitHub releases and reduce human intervention. Contributions are welcome :-)

Thanks. The library is currently in pre-release and it does not have, to my knowledge, any user. Testing is minimal and the current design has a performance issue: it can be 10x slower than the Go standard library.
Please run:
go test -bench Benchmark -run -

The fairly long delay is not due to the ada library itself, but to the wrapper which is currently highly inefficient.
The Go standard library is not standard compliant so using goada could be warranted, I'd very much like to solve the performance issue as it seems unacceptable to me.
The problem derives from runtime.SetFinalizer. I could not find a way to avoid this anti-pattern.

Hello again!
Thanks for your replies!
Regarding to @anonrig

We are more than interested to use GitHub releases and reduce human intervention. Contributions are welcome :-)

I think I could contrib to automate the updates of Ada upstream and regarding releases.
Concerning to @lemire

The fairly long delay is not due to the ada library itself, but to the wrapper which is currently highly inefficient.
The Go standard library is not standard compliant so using goada could be warranted, I'd very much like to solve the > performance issue as it seems unacceptable to me.
The problem derives from runtime.SetFinalizer. I could not find a way to avoid this anti-pattern.

I was looking for alternatives, but I had no luck with that. Apparently it will be a built-in drawback for the package that will be the price to pay for using CGO.