Can not run webpack-dev-server in  Node V12

This isn't a bug report.
create-react-app不能在nodeV12的环境下启动webpack-dev-server，我把node环境降级到node11.5就解决了这个问题。

I tried node v12 and it worked fine for me.  Please provide a full repo as well as error message.

Thx repo:
I used webpack-dev-server in the Create-React-App,  when CRA start a dev server,It build a package,and when it finish,error is happen,here error message you are:
Starting the development server...

Compiled successfully!

You can now view part2-react in the browser.

  Local:            http://localhost:12121/
  On Your Network:  http://<IP_ADDRESS>:12121/

Note that the development build is not optimized.
To create a production build, use npm run build.

#
# Fatal error in , line 0
# Check failed: U_SUCCESS(status).

#
#
#
#FailureMessage Object: 0x7ffeefbfb2d0

And when i downgrade to nodeV12, It begin work without error;
version in package.json:
    "webpack": "4.19.1",
    "webpack-dev-server": "^3.2.1",
    "webpack-manifest-plugin": "2.0.4",
    "workbox-webpack-plugin": "3.6.3"

It is also possible that the other  dependencies packages of CRA are not compatible with node12, Is it possible?

same error here, running the sample project from CRA produce the same error :
npx create-react-app my-app
cd my-app
npm start

Seems to work for the build, only failing in dev mode ( so with webpack-dev-server ) .
( same error on my custom build from an old cra )

I have issues as well after upgrading to nodejs v12.1.0 when it comes to use the dev mode.
Here are my error logs:

/Users/user/Documents/git/someproject/src/main/frontend/node_modules/chokidar/lib/fsevents-handler.js:28
  return (new fsevents(path)).on('fsevent', callback).start();
          ^

TypeError: fsevents is not a constructor
    at createFSEventsInstance (/Users/user/Documents/git/someproject/src/main/frontend/node_modules/chokidar/lib/fsevents-handler.js:28:11)
    at setFSEventsListener (/Users/user/Documents/git/someproject/src/main/frontend/node_modules/chokidar/lib/fsevents-handler.js:82:16)
    at FSWatcher.FsEventsHandler._watchWithFsEvents (/Users/user/Documents/git/someproject/src/main/frontend/node_modules/chokidar/lib/fsevents-handler.js:252:16)
    at FSWatcher.<anonymous> (/Users/user/Documents/git/someproject/src/main/frontend/node_modules/chokidar/lib/fsevents-handler.js:386:25)
    at LOOP (fs.js:1622:14)
    at processTicksAndRejections (internal/process/task_queues.js:82:9)
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

This is maybe related to this issue: fsevents/fsevents#274
If I add the following line to my package.json file:
 "fsevents": "1.2.8"

I have no more any problem to start the dev mode.
Without the line above, the default version of fsevents is 1.2.4, which seems to be not enough to work with nodejs v12 apparently.

@Nowheresly you're problem is different . Fsevents is only installed on macOs, and your error is really different . I think it's better for you to ask an update of chokidar

@junjie-lean Node V12.1.0 ( out 2019-04-29 ) resolve the problem . Same on your side ?

@thib3113  yes, it's works in Node V12.1.0 , And what is change in V12.1 about CRA?


Create Word Doc to Review Sections

Need to review text for each housing type
Definitions (short)
Definitions (long)
History
Layouts
Potential Layouts
#4 Boxes
#9 Deeper Context

Report History Text for Editing: https://mapc365.sharepoint.com/:w:/s/DataServicesSP/EUHkYHjMUDlEgooF6h3W_FcBpGLSesmPLCLG986MRb7irQ?e=yPqbN6

Text saved here:
\data-sync\Public\SG Land Use\SUB-REGIONS\SouthWest\Projects\LL Phase 2 - SWAP-TRIC\Digital toolkit\TEXT

Have word documents closing issue


Not getting path of save file in iOS

as in android, we are getting path at which file will save,  but in iOS we are not getting path of file but are getting ios verison as iOS 14.5

Ok, I'll look into it.

This is now fixed, use from git if you want it urgently or I'll update on pub in some days


Generated Qr Code is not scannable

plz reply and help

Try changing dot scale from 0.35 to 1. I have no problems scanning the QR Codes.


Steps with enum parameter marked as continueOnFailure is not working as expected

Expected Behavior
If a step is marked as continueOnFailure(COF) the next step should get executed even if there is a failure in the current step.
Actual Behavior
Even if marked as COF on failure, the next steps are not getting executed.
Steps to replicate

Run the specs in the project.
enumWithCOF.zip

Check the html report.
The next steps are not executed.

Version
Gauge version: 0.6.3.nightly-2016-11-06

Plugins
-------
csharp (0.9.2.nightly-2016-09-14)
html-report (2.2.0.nightly-2016-11-09)
java (0.5.1.nightly-2016-11-21)
js (1.3.0)
ruby (0.3.2.nightly-2016-08-22)
spectacle (0.0.1)
xml-report (0.1.4.nightly-2016-07-25)

@sguptatw Gauge should allow ContinueOnFailure if the failure is on the functional side of things. In this case, Gauge is not even able to execute the functionality. It's a parameter conversion failure. In other words, the failure is on the language / implementation side. So even if the step is marked as ContinueOnFailure, the execution should ideally step here and the next steps should not get executed.

@apoorvam : I think this has to be documented along with COF.

Documented via https://github.com/getgauge/documentation/commit/16864a5f015f9ba1ff56be5ae277122190b7b5bc


feature/parkingDialogChange -> dev

This change is

Reviewed 1 of 1 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

app/src/main/res/values/strings.xml, line 271 at r1 (raw file):

	<string name="delete_inactive_vehicle_successful">Vehicle successfully deleted</string>
	<string name="deactivate_vehicle_successful">Vehicle successfully deactivated</string>
	<string name="activate_vehicle_successful">Vehicle successfully activated. Be aware that it can take up to 10 minutes in some cases for privileges to reflect on this vehicle.</string>

15

app/src/main/res/values/strings.xml, line 277 at r1 (raw file):

	<string name="invalid_vehicle">Invalid Input</string>
	<string name="adding_vehicle_progress_message">Adding vehicle, please wait…</string>
	<string name="registration_successful">Vehicle successfully registered. Be aware that it can take up to 10 minutes in some cases for privileges to reflect on this vehicle.</string>

15

Comments from Reviewable

Review status: 0 of 1 files reviewed at latest revision, 2 unresolved discussions.

app/src/main/res/values/strings.xml, line 271 at r1 (raw file):
Previously, romrell4 (Eric Romrell) wrote…
15

Done.

app/src/main/res/values/strings.xml, line 277 at r1 (raw file):
Previously, romrell4 (Eric Romrell) wrote…
15

Done.

Comments from Reviewable


Add target for EUDCCValidator

Target for EUDCCValidator was missing

Hey @ygini
Thanks for your PR and great work 👍
The missing EUDCCValidator library declaration in the Package.swift has already been fixed with this commit https://github.com/SvenTiigi/EUDCCKit/commit/e57cc65905eae705864aa5335dfb3cecd85406f8
Refactoring the testName and testNameAndManufacturer to be an optional String is a great addition as those properties are truly not required.
Would it be possible for you to merge the latest changes from the upstream in to your fork so that the commit https://github.com/SvenTiigi/EUDCCKit/commit/e57cc65905eae705864aa5335dfb3cecd85406f8 is included in your PR?
Alternatively it would be no problem to merge those changes later on 👌
Therefore I have changed the target branch from main to develop for this PR.

Hi
Sorry I didn't see you had a develop branch.
I've fetched your changes and updated my repo. Now the PR contain only the updates for optional values.
Those optional values have been determined based on what was missing in French passes.
Do you have a file format reference somewhere? I could help to review what's required and what's optional in all data structure.
By the way, thank for your work!

@ygini you can find the JSON schema for the EU Digital COVID Certificate in the appendix of the certificate specification
https://ec.europa.eu/health/sites/default/files/ehealth/docs/digital-green-certificates_dt-specifications_en.pdf
As well as here:
https://ec.europa.eu/health/sites/default/files/ehealth/docs/covid-certificate_json_specification_en.pdf


Introduce DxFontInfo

Summary of the Pull Request
This PR Introduces DxFontInfo to simplify the logic in DxFontRenderData.
DxFontInfo aims to be the DWrite equivalent of FontInfo & FontInfoBase in GDI. It encapsulates the needed information to represent a displayable font face. It also provides the ability to resolve a font face based on the available fonts on the system.

References
This is a follow-up of #9096.
Initial Italic support is introduced by #8580.
The motivation behind this is to support bold & bold-italic text in Windows Terminal.

PR Checklist

[ ] Closes #xxx
[X] CLA signed. If not, go over here and sign the CLA
[ ] Tests added/passed
[ ] Documentation updated. If checked, please file a pull request on our docs repo and link it here: #xxx
[ ] Schema updated.
[ ] I've discussed this with core contributors already. If not checked, I'm ready to accept this work might be rejected in favor of a different grand plan. Issue number where discussion took place: #xxx

Detailed Description of the Pull Request / Additional comments

Validation Steps Performed

Huh...the trick [skip ci] does not seem to work with Azure Pipelines.
See https://github.com/Microsoft/azure-pipelines-agent/issues/1270#issuecomment-486782068 . I don't honestly know what she meant by saying "Azure DevOps honors the [skip ci] commands on CI builds as the change being built has already been merged"

OK I have no idea how & why there tests has anything to do with my code:
2021-06-11T16:59:23.2143730Z Summary of Non-passing Tests:
2021-06-11T16:59:23.2144447Z     ControlUnitTests::ControlCoreTests::TestInitialize [Failed]
2021-06-11T16:59:23.2145027Z     ControlUnitTests::ControlCoreTests::TestAdjustAcrylic [Failed]
2021-06-11T16:59:23.2145574Z     ControlUnitTests::ControlInteractivityTests::TestScrollWithMouse [Failed]
2021-06-11T16:59:23.2146125Z     ControlUnitTests::ControlInteractivityTests::CreateSubsequentSelectionWithDragging [Failed]
2021-06-11T16:59:23.2146693Z     ControlUnitTests::ControlInteractivityTests::ScrollWithSelection [Failed]
2021-06-11T16:59:23.2147256Z     ControlUnitTests::ControlInteractivityTests::TestScrollWithTrackpad [Failed]
2021-06-11T16:59:23.2147786Z     ControlUnitTests::ControlInteractivityTests::TestQuickDragOnSelect [Failed]

OK I have no idea how & why there tests has anything to do with my code:
2021-06-11T16:59:23.2143730Z Summary of Non-passing Tests:
2021-06-11T16:59:23.2144447Z     ControlUnitTests::ControlCoreTests::TestInitialize [Failed]
2021-06-11T16:59:23.2145027Z     ControlUnitTests::ControlCoreTests::TestAdjustAcrylic [Failed]
2021-06-11T16:59:23.2145574Z     ControlUnitTests::ControlInteractivityTests::TestScrollWithMouse [Failed]
2021-06-11T16:59:23.2146125Z     ControlUnitTests::ControlInteractivityTests::CreateSubsequentSelectionWithDragging [Failed]
2021-06-11T16:59:23.2146693Z     ControlUnitTests::ControlInteractivityTests::ScrollWithSelection [Failed]
2021-06-11T16:59:23.2147256Z     ControlUnitTests::ControlInteractivityTests::TestScrollWithTrackpad [Failed]
2021-06-11T16:59:23.2147786Z     ControlUnitTests::ControlInteractivityTests::TestQuickDragOnSelect [Failed]

The failed tests all look like
2021-06-15T11:27:11.3398459Z TAEF: A crash with exception code 0xC0000094 occurred in module "Control.Unit.Tests.dll" in process "te.processhost.exe" (pid:6860).

Where 0xC0000094 is a thrown divide by zero exception. So I'm guessing something about the setup part of those tests is initializing something that is now having a divide by zero with your change. :(

The common thing in the unit tests looks like a call to ControlCore::Initialize, which calls ControlCore::_updateFont and hence Renderer::TriggerFontChange, which leads to DxEngine::UpdateFont, and hence DxFontRenderData::UpdateFont. This calls DxFontRenderData::_BuildFontRenderData, which is changed in this patch.
DxFontRenderData::_BuildFontRenderData has lots of division calls in it as well.
So at a guess, there's some code-path where some initialisation never happens and hence some value is 0.f. It's being hit by the unit tests' very minimal initialisation, that you aren't seeing in actual testing when you run WT.

wow thanks @TBBle for the investigation. I'll get to this PR later and found out why. I promise.


Request support for DELTACO SH-P02E

Log Message

Logger: custom_components.tuya_local.config_flow
Source: custom_components/tuya_local/config_flow.py:45
Integration: Tuya Local (documentation, issues)
First occurred: March 11, 2023 at 22:56:56 (14 occurrences)
Last logged: 08:33:58

Device matches smartplugv2 with quality of 90%. DPS: {"updated_at":<PHONE_NUMBER>.8825524, "1": true, "9": 0, "18": 0, "19": 0, "20": 2321, "21": 1, "22": 1739, "23": 40880, "24": 125000, "25": 168}
Device matches smartplugv2 with quality of 90%. DPS: {"updated_at":<PHONE_NUMBER>.9884288, "1": true, "9": 0, "18": 0, "19": 0, "20": 2405, "21": 1, "22": 1744, "23": 40960, "24": 124000, "25": 168}
Device matches smartplugv2 with quality of 90%. DPS: {"updated_at":<PHONE_NUMBER>.9433415, "1": true, "9": 0, "18": 0, "19": 0, "20": 2321, "21": 1, "22": 1739, "23": 40880, "24": 125000, "25": 168}
Report this to https://github.com/make-all/tuya-local/issues/
Device matches compteur_energy_meter with quality of 71%. DPS: {"updated_at":<PHONE_NUMBER>.931585, "1": false, "2": false, "9": 0, "10": 0, "17": 1, "18": 0, "19": 0, "20": 2327, "21": 1, "22": 601, "23": 26710, "24": 15896, "25": 2640, "26": 0}

Information about DPS mappings

{
  "result": {
    "category": "pc",
    "functions": [
      {
        "code": "switch_1",
        "dp_id": 1,
        "type": "Boolean",
        "values": "{}"
      },
      {
        "code": "switch_2",
        "dp_id": 2,
        "type": "Boolean",
        "values": "{}"
      },
      {
        "code": "countdown_1",
        "dp_id": 9,
        "type": "Integer",
        "values": "{\"unit\":\"s\",\"min\":0,\"max\":86400,\"scale\":0,\"step\":1}"
      },
      {
        "code": "countdown_2",
        "dp_id": 10,
        "type": "Integer",
        "values": "{\"unit\":\"s\",\"min\":0,\"max\":86400,\"scale\":0,\"step\":1}"
      }
    ],
    "status": [
      {
        "code": "switch_1",
        "dp_id": 1,
        "type": "Boolean",
        "values": "{}"
      },
      {
        "code": "switch_2",
        "dp_id": 2,
        "type": "Boolean",
        "values": "{}"
      },
      {
        "code": "countdown_1",
        "dp_id": 9,
        "type": "Integer",
        "values": "{\"unit\":\"s\",\"min\":0,\"max\":86400,\"scale\":0,\"step\":1}"
      },
      {
        "code": "countdown_2",
        "dp_id": 10,
        "type": "Integer",
        "values": "{\"unit\":\"s\",\"min\":0,\"max\":86400,\"scale\":0,\"step\":1}"
      },
      {
        "code": "cur_current",
        "dp_id": 18,
        "type": "Integer",
        "values": "{\"unit\":\"mA\",\"min\":0,\"max\":30000,\"scale\":0,\"step\":1}"
      },
      {
        "code": "cur_power",
        "dp_id": 19,
        "type": "Integer",
        "values": "{\"unit\":\"W\",\"min\":0,\"max\":80000,\"scale\":1,\"step\":1}"
      },
      {
        "code": "cur_voltage",
        "dp_id": 20,
        "type": "Integer",
        "values": "{\"unit\":\"V\",\"min\":0,\"max\":5000,\"scale\":1,\"step\":1}"
      }
    ]
  },
  "success": true,
  "t":<PHONE_NUMBER>085,
  "tid": "6f205a21c0aa11eda9e9d694da825c81"
}

"Add Electricity"
Code: "17"
"Test Bit"
code: "21"
"Voltage coe"
code: "22"
"Current coe"
code: "23"
"Power coe"
code: "24"
"Electricity coe"
code: "25"
"Fault"
code: "26"
Product ID

"product_id": "3jnmzqoyuopg4vtf"
"product_name": "DELTACO Smart Plug SH-P02"
Information about how the device functions

https://www.deltaco.eu/en/products/deltaco/smart-home/power-strips/SH-P02E
I have testet:
Digoo_dgsp202
simple_dual_switch_timer_v2
Smartplugv2
Only the 2 switches is not working.
The switchs does not work when switch to on from HA but when activating the plug on the plug it self, HA sees it is on.
The current, power and voltage seems to work fine with on these.
Best regards

Hi
Tested to add the "dual_power_monitor_smartplugv2.yaml" and added the deltaco plug integration.
The messurement works but the switches does not work from HA but HA update the state when I press the button on the plug it self.

This behaviour has been seen before when the device id was entered incorrectly.  Try removing the device from HA and add it again, taking care to get the device id right.

Now I see what I did wrong.
Though it was "uuid" I should use but not, it was "id" from IoT tuya.
Thanks:)


Make debugger port runtime configurable

Convert debug server port from a compile-time constant to a context
variable. This enables each engine instance (either running in the
same process or in different processes) to listen for connections
at different ports, i.e., multiple engines can be debugged at the
same time on the same machine.
JerryScript-DCO-1.0-Signed-off-by: Akos Kiss<EMAIL_ADDRESS>

LGTM


Ability to emit multiple parse errors for a single token

In looking at what it would take to fix some of the html5lib tests, I noticed that in certain situations multiple errors can sometimes be expected when emitting a single token. For example:
<p></p attr/>

According to the spec in 12.2.5 Tokenization:

When an end tag token is emitted with attributes, that is an end-tag-with-attributes parse error.

When an end tag token is emitted with its self-closing flag set, that is an end-tag-with-trailing-solidus parse error.

This means that when emitting an EndTag where both of those are true, nextToken should error once with EndTagWithAttributes and then again with EndTagWithTrailingSolidus, but zhtml doesn't currently make this possible as far as I can tell, so it seems like some type of error queue will end up being necessary (i.e. in nextToken, if there is an error in the queue, then just pop it off and return it immediately).
(side note: html5lib-tests doesn't have this specific test where both errors are emitted for the same end tag, but I believe this is the behavior the spec expects)

Damn I was hoping to avoid that, but oh well. If it can't be helped it can't be helped.


add missing dev dependency: less compiler

Perhaps it should be a regular dependency (not dev) since we do not ship compiled assets?

It is a regular dependency. I moved all of the dev dependencies up to regular dependencies since they didn't install for -g installs of Strider, which fixed a whole deal of issues that people were having with installing/building Strider.

Ah, didn't see it. I must have forgot to npm install, thanks


NUC

Hi Kayman, I want to buy a NUC to learn how to use OpenCR, ROS and various Robotis platforms. I cannot find information about the model of the NUC used in the OP3. The specs only say Intel NUC i3, dual core, 8GB DDR4 RAM 2133MHz and 128 SSD.  Can you tell the model number of that NUC? If buying now, which NUC do you recommend? Don't know if any NUC and iGPU is compatible with your products hardware and software-wise.

Hi,
The model name of NUC used in OP3 is nuc 7i3BNK. It was changed to the above model in the second half of 2018, before that it was the NUC6i3SYK.
It may change in the future depending on the revision of NUC.
Other models are not compatible with the frame of the OP3, and even in the same model, power line work is required to put it in the OP3.
Regards,
Kayman

Thanks Kayman. What suggestion do you have in this case? Sounds like I need to get the NUC7i3BNK to avoid compatibility issue? I don't know if your ROS software libraries use some specific instruction sets such as AVX and may be Intel GPU is required. Can you please provide more information about the required power line work?

Hi,
Any NUC can be used on our platform, but certain platforms may require more computing power.
Two of our products are using NUC. One is THORMANG3 and the other is OP3. In the case of THORMANG, i5 is used because it requires more computing power.
OP3 libraries and framework may work regardless of CPU or GPU, but we can not gaurantee because we didn't test other SBCs.
About power line, refer to #52 please.
Regards,
Kayman

Hi Kayman,
Good morning. Thank you for your reply.
In #52, it was mentioned that your company would soon make a manual about the wiring and made it available.  I cannot find it. Where is it?

Hi again,
We tried to release the manual with the kit type OP3, but, unfortunately, the plan has changed and we do not provide a manual.
Regards,
Kayman

Hi Kayman,
Thank you.
In #59, you posted a diagram to show that one leg is connected directly to a RS485 port of the OpenCR. Then, another leg and two arms are each connected to a separate RS485 port of a hub. What are the reasons for doing that?

Hi,
because ... It can solve the problem of "Voltage Drop". That's why we do like that.
Regards,
Jin

Hi Jin,
Thank you. So, if I make a biped or hexapod later, it is better to split the limbs and connect each limb separately to an individual RS485 connector of a hub the way shown in #59 ?

Hi again.
Yes. that is better.
and if you have a plan for the hexapod, you have to do.
Regards,
Jin.

Thanks Jin.
In the case of the OP3, when only the LB-012 battery is used, how does the battery power the entire robot? Is it like LB-012 (11.1V, 1800mAh)-->Power barrel of the NUC--usb-->OpenCR->20 dynamixels?
I am amused that the LB-012 can power both the NUC, OpenCR and 20 dynamixels.
From the photo in #59, power source for the entire robot seems to be connected to the OpenCR barrel (the red line). There is no mention about how the SBC/NUC gets the power. I am a bit confused. Can you clarify please?

Hi,
You can check the issue from "#52"
Regards.
Jin.

Thanks Jin.
It seems to be:
LIPO--(T-connector on OpenCR)-->OpenCR & Dynamixels-- (SMW250-02 on OpenCR)-->Intel NUC--(USB port of NUC)-->U2D2--(RS485)-->OpenCR.
Am I correct? I am a bit confused about the connection between the U2D2 and other components. Given that we can connect the Dyamixels from the OpenCR, why we need a U2D2?

Hi,
Of course .. you make a another system if you want.
That is another way.
But OpenCR has some functions - external power, sensing, and so on.
we just want to separate those - Dynamixel control & ETC
That is why we use a U2D2.
Regards.
Jin

Thanks. I have been thinking about this but I am still a bit confused.
As far as I understand, OpenCR gets power from the LIPO battery. It then supplies power to the NUC and the 20 dynamixels. From the communication map diagram in #59, it seems that the right leg is connected directly to the OpenCR but the left leg, two arms and the head are connected to the PR28-EXT hub. Where is the hub connecting to? Another RS485 port on the OpenCR or the U2D2?
Can you clarify please?

Hi.
As you can see, You may find the 3 RS485 ports on the OpenCR.

but we need 5 points.
so we use a PR28-EXT hub. it can connect 4 points.
To conclude, PR28-EXT hub connect to RS485 port.
Regards.
Jin

Thanks Jin. So although the U2D2 is in the above diagram and those in #52, it actually does not do anything?

Hi,
SBC is using this device(U2D2).
if you are programming in ROS(in NUC),
you can send commands through this device
Regards.
Jin

Thanks Jin. I am very sorry but I am still a bit confused. So, if I use SBC such as Raspberry Pi, I can send commands through the U2D2 to the Dynamixel. In the case of the OP3 which uses ROS and NUC, the U2D2 is not used? Also, for the hub that merges the limbs and the neck to the OpenCR, is it a passive hub (i.e. the GND, VDD, D+ and D- of the limbs and the neck just physically connect to each other and no power source is connected to the hub)?

Hi,
OP3 uses U2D2 for controlling Dynamixel.
and power is connected to the OpenCR(Battery or Powersupply)
There's(-OpenCR-) both power and communication is connected.
Regards.
Jin.

Thanks Jin. Do you mean in addition to connecting different branches of limbs to the hub, I also need to supply additional power to the hub?

Hi,
and sorry for the late answer.
If you have already power on OpenCR, you don't need power anymore.
check below.

Thanks Jin.
It looks like in the OP3, the NUC's case is removed but for the Thormang, the two NUC were installed on the robot as is from retail boxes. When using only the NUC's motherboard to control a robot, what to do with the metal frame which came with the NUC? It seems to be used for some kind of shielding. Does removing it cause some kind of issues?

Hi,
sorry for late reply.
We don't have any problems with removing the case.
Regards.
Jin.

Thanks Jin. I successfully removed the case.


WIP: [BEAM-6268] Adjust Cassandra ports

Embedded Cassandra ports of hadoop-input-format should be adjusted to avoid conflicts with hadoop-format Cassandra tests

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

[x] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

It will help us expedite review of your Pull Request if you tag someone (e.g. @username) to look at it.
Post-Commit Tests Status (on master branch)

Lang
SDK
Apex
Dataflow
Flink
Gearpump
Samza
Spark

Go

---
---
---
---
---
---

Java

Python

---

---
---
---

R: @mxm
Please, take a look

Seems to be failing with
java.lang.IllegalStateException: Failed to bind port 9042 on <IP_ADDRESS>.
	at org.apache.cassandra.transport.Server.start(Server.java:164)
	at java.util.Collections$SingletonSet.forEach(Collections.java:4767)
	at org.apache.cassandra.service.NativeTransportService.start(NativeTransportService.java:128)
	at org.apache.cassandra.service.CassandraDaemon.startNativeTransport(CassandraDaemon.java:649)
	at org.apache.cassandra.service.CassandraDaemon.start(CassandraDaemon.java:511)
	at org.apache.cassandra.service.EmbeddedCassandraService.start(EmbeddedCassandraService.java:53)
	at org.apache.beam.sdk.io.hadoop.format.HadoopFormatIOCassandraTest.startCassandra(HadoopFormatIOCassandraTest.java:179)

Is this related to the 0 port change?

No, I think it's a different one. I'm trying to find a way how to fix it.

@aromanenko-dev What is the state here?

@mxm I'm going to fix this

Run Java_Examples_Dataflow PreCommit

Run Java PreCommit

Run Java_Examples_Dataflow PreCommit

Run Java PreCommit

Run Java PreCommit


Could you please provide an update on when the next release is planned?

Issue description
I'm waiting some fixes

see: https://github.com/micronaut-projects/micronaut-core/discussions/10527
https://micronaut.io/micronaut-roadmap/

We can probably release a minor version already, but you will need to reference it directly.

@dstepanov it'll be great thank you


The Demo app doesn't work

The demo app has numerous errors and I can't get it to run

Hey, could you try once again, please? I've pushed a quick fix. We will push a proper fix in a few days.

Thanks, its's working now


[FEAT] Option for automatic type-casting in datediff_level

Is your proposal related to a problem?
Sometimes I may be working with date columns which are formatted as strings, which may be useful if e.g. I want to have a mix of datediff and levenshtein levels (although for duckdb at least levenshtein works on dates), or I don't have ready control of my input columns/want to make work reproducible between parties with only settings (i.e. not requiring source data modification)
Describe the solution you'd like
Flag to datediff_level comparison level, something like cll.datediff_level("dob", 1, "year", cast_strings_to_date=True), and similar for the comparison datediff_at_thresholds
Describe alternatives you've considered
Can be worked around by using an explicit date column, or hand-writing the relevant SQL.
Additional context
Would need to consider the case that there is a chance that the data quality is not verified and may contain invalid dates (e.g. '1981-04-31) - probably would want this to act as automatically FALSE for date comparisons, but be able to levenshtein on it (could well be a typo for '1981-04-30'). Might require some use of TRY_CAST and checking for NULL values in the result.

I foresee various problems arising by not enforcing one type, or being too lenient with allowing strings.
Sidenote: What happens if we create a linker for two datasets with DOB columns of different types?
Potential edge cases to consider with a string column:

"2023-01-01" vs "not a date" - should be captured by null_level, not else_level
"not a date" vs "not a date" - should be captured by null_level, not exact_match_level
"2023-01-01" vs "1/1/23" - should be captured by exact_match_level, not else_level

If we don't want to enforce date type on columns provided to datediff comparisons, then I think we need to ensure both sides are formatted similarly, or formatted at all. Casting strings to date only allows for dates formatted as YYYY-MM-DD right?
Basically, you can easily turn dates to strings if you want to use string operations, but it's a pain to do the reverse.

Closed by #1162


Feature Request: Default sdist: false where not specified in scenario

For a simpler test environment for pip I am trying to avoid setting up multiple Python environment versions, packse requires Python 3.12+ so I am only running tests with a Python 3.12 environment.
Most scenarios in packse are set to Python 3.8, for most scenarios this is not relevant to the results of the test. However for a few scenarios it is actually important to resolve the scenario against the Python version specified.
To get pip to run as though it were Python 3.8 when it's Python environment is Python 3.12 I must use the --python-version flag, however this flag is incompatible with sdists.
I beleive, but correct me if I'm wrong, that packse will default to sdist: true when it's not explictly specified by the scenario. This is problematic, as it means I can not distinguish when it's required to process sdists or not, from the output of packse inspect.

Hm interesting, we don't have this problem because we will still attempt to perform source distribution builds with a mismatched Python version to get metadata.
I think you should be able to tell if the source distribution matters based on the ResolverOptions like no_binary

I'm a little hesitant to default it to false since by in the real-world generally both are available.


[Event Request] Codeunit 1720

Following Event desired:
OnAfterSetEndDateCalculateDaysPerPeriod(VAR DeferralHeader: Record 1701;VAR DeferralLine: Record 1702;DeferralTemplate: Record 1700;VAR EndDate: Date)
Call in Function CalculateDaysPerPeriod just before the line:
OnAfterSetEndDateCalculateDaysPerPeriod(DeferralHeader,DeferralLine,DeferralTemplate,EndDate)
NumberOfDaysInSchedule := (EndDate - DeferralHeader."Start Date");

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


Windows executable output overwritten by appended string

Version
Microsoft Windows [Version 10.0.19044.1466]
WSL Version

[X] WSL 2
[ ] WSL 1

Kernel Version
<IP_ADDRESS>
Distro Version
Ubuntu 20.04
Other Software
PowerShell 7.2.1 (for repro steps)
Repro Steps

Open WSL distro terminal
Execute echo $(pwsh.exe -c Write-Output 123)456

Expected Behavior
echo output to result in 123456
Actual Behavior
echo output results in 456
Diagnostic Logs
No response

This is not WSL issue, your experimenting syntax is wrong.
try this: echo "$(powershell.exe -c echo hi)"; echo "123";

This is not WSL issue, your experimenting syntax is wrong. try this: echo "$(powershell.exe -c echo hi)"; echo "123";

this would be two lines, was wanting one line as shown in expected behavior.
it did dawn on me that later this is because the Windows command output is including a carriage return (CR), causing the cursor to return to the beginning overwriting the first few characters - stripping that out via tr -d '\r' solved it


DefaultToken color missing from PSReadLineOptions with preview 6

I upgraded to the new preview version (6) and got welcomed by an error from my usual profile script that sets the colors to the Solarized Dark theme using Set-PSReadLineOption -Colors.
Even though it is listed as one of the possible colors when I run Get-PSReadLineOption
CommandColor                           : "`e[34m"
CommentColor                           : "`e[32m"
ContinuationPromptColor                : "`e[34m"
DefaultTokenColor                      : "`e[37m"
EmphasisColor                          : "`e[96m"
ErrorColor                             : "`e[91m"
KeywordColor                           : "`e[92m"
MemberColor                            : "`e[36m"
NumberColor                            : "`e[36m"
OperatorColor                          : "`e[32m"
ParameterColor                         : "`e[32m"
SelectionColor                         : "`e[30;47m"
StringColor                            : "`e[94m"
TypeColor                              : "`e[33m"
VariableColor                          : "`e[92m"

Steps to reproduce
Set-PSReadLineOption -Colors @{ "DefaultToken"       = [ConsoleColor]::DarkBlue }

Expected behavior
No error.

Actual behavior
Set-PSReadLineOption: 'DefaultToken' is not a valid color property

Environment data

Name                           Value
----                           -----
PSVersion                      7.0.0-preview.6
PSEdition                      Core
GitCommitId                    7.0.0-preview.6
OS                             Microsoft Windows 10.0.19025
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

Please move the report to https://github.com/PowerShell/PSreadline

@totkeks The key is Default:
Set-PSReadLineOption -Colors @{
    Default = [ConsoleColor]::DarkBlue
}

@totkeks The key is Default:
Set-PSReadLineOption -Colors @{
    Default = [ConsoleColor]::DarkBlue
}

Thanks @SeeminglyScience. It works now. I just don't understand if it is a bug with the new PSReadline version or it never worked like this and they added a check to the color hashmap supplied.


imageRatio is calculated incorrectly with FREE as cropMode

in getRatioX() and getRatioY(), FREE mode is not taken into account. So in scaleBitmapIfNeeded(Bitmap) the value of imageRatio will always be 1.0 in FREE mode, which results in a loss of aspect ratio in scaled bitmap.

Hi,
Sorry, I will fix it as soon as possible.

I have fixed the bug. Please use the version 1.1.2.
Thank you for telling me about my bug : )

Thank you for taking time fixing this bug. Will test ASAP.


'map' method on FluidContext

(2nd PR of the day), I also found myself needing this feature while processing an XML document. I thought it would be a nice addition.

Coverage increased (+0.002%) to 99.818% when pulling 09394f324a883e61d7d609db0b535e1a41a329db on alepeino:map-method into d5cadee48731c3d3f4f770572b2eca61d26a01a9 on servo-php:master.

Awesome PR. Thank you very much!


Some tests for patterns fail because the 'records' feature isn't enabled in them

The issue is discovered in the following tests.

language/patterns/shared_case_variable_error_test
language/patterns/shared_case_variable_test
language/patterns/switch_case_scope_test

The error was encountered during work on https://dart-review.googlesource.com/c/sdk/+/276524.
This issue is assigned label area-front-end tentatively, and it needs further investigation, and possibly reassignment of the tags.

/cc @eernstg

Just noticed that there may be a typo: I can't see any declaration of result here.

Currently block by assertion failure in the flow analysis.

The tests either appear to be fixed by now or fail for a different reason. I'm closing the issue.


[Doc-Bug] Teams app in M365 portal - SDK initialization timed out

Hi Team,
Support for apps running on the microsoft365.com domain was not added to teams-js until v2.5.0. So, we see SDK initialization time out error with using older versions. Documentation is only mentioned to use >=2.0.0. So, please add note for the same.

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

ID: 6fcea0c3-c249-a292-913b-dc0271e4e810
Version Independent ID: 6fcea0c3-c249-a292-913b-dc0271e4e810
Content: Extend Teams apps across Microsoft 365 - Teams
Content Source: msteams-platform/m365-apps/overview.md
Service: msteams
Sub-service: m365apps
GitHub Login: @erikadoyle
Microsoft Alias: mosdevdocs

@surbhigupta12 - Could you please help here to add the note in the document.


[SMALLFIX]remove the extra 'be'

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/10818/
Test PASSed.

LGTM


Streamlit app down

Error message states "Error running app". Unable to view any error logs.

Resolved


This seems to crash sadly when updating album art I think

Any ideas? Maybe the API signature has changed for the .get method in more recent versions of Node?
TypeError [ERR_INVALID_CALLBACK]: Callback must be a function
at maybeCallback (fs.js:133:9)
at Object.exists (fs.js:201:3)
at ClientRequest.http.get (/home/pi/serverorig/server.js:58:19)
at Object.onceWrapper (events.js:273:13)
at ClientRequest.emit (events.js:182:13)
at HTTPParser.parserOnIncomingClient [as onIncoming] (_http_client.js:546:21)
at HTTPParser.parserOnHeadersComplete (_http_common.js:109:17)
at Socket.socketOnData (_http_client.js:432:20)
at Socket.emit (events.js:182:13)
at addChunk (_stream_readable.js:283:12)
at readableAddChunk (_stream_readable.js:264:11)
at Socket.Readable.push (_stream_readable.js:219:10)
at TCP.onread (net.js:638:20)
Thanks.

Which version of node.js are you running?

Wow. Thanks for the quick reply. All these web technologies are a bit obscure to me coming from an iPhone iOS native background.
I think I’m running 10.6? It’s running on a Raspberry Pi Stretch install. It’s a complete clean install as of this morning following the instructions given. Is this even correct that I have both these binaries and they’re different versions?
pi@utility:~/serverorig $ node --version
v10.6.0
pi@utility:~/serverorig $ nodejs --version
v8.11.1
I’m not even sure I know the difference between nodejs and node at this point as I just jumped in.
Thanks for your help.

On 18 Jul 2018, at 12:57, Jimmy Shimizu<EMAIL_ADDRESS>wrote:
Which version of node.js are you running?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/jishi/node-sonos-web-controller/issues/64#issuecomment-405905488, or mute the thread https://github.com/notifications/unsubscribe-auth/AArNQjQvdu3PX4hn7ULbWL5nPfI6WNfpks5uHyKOgaJpZM4VUc28.

nodejs is the debian maintained packaging of node.js. Both of them should work with this, but I haven't ran this web-controller in a while. Looking at the code, I don't get that error, but also the line reference seems wrong (I'm assuming you checked out the files to /home/pi/serverorig?)
I would need to start it up at home to check it out.

Thanks for looking. I did check it out into that directory as I already had a different forked version in the original directory. Sadly the fork has the same problem too.

On 18 Jul 2018, at 13:15, Jimmy Shimizu<EMAIL_ADDRESS>wrote:
nodejs is the debian maintained packaging of node.js. Both of them should work with this, but I haven't ran this web-controller in a while. Looking at the code, I don't get that error, but also the line reference seems wrong (I'm assuming you checked out the files to /home/pi/serverorig?)
I would need to start it up at home to check it out.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/jishi/node-sonos-web-controller/issues/64#issuecomment-405910313, or mute the thread https://github.com/notifications/unsubscribe-auth/AArNQh6Y-L_Eu0DJaBmBeZ8AvhNhEpbuks5uHybbgaJpZM4VUc28.

I don't see why it wouldn't run in 10.x as well, but makes a good point to make sure it does, thanks for reporting it.

It seemed pretty conclusive. I installed 10.6 using instructions from from here:
http://thisdavej.com/beginners-guide-to-installing-node-js-on-a-raspberry-pi/#install-node http://thisdavej.com/beginners-guide-to-installing-node-js-on-a-raspberry-pi/#install-node
Ran the App and it gave me this.
pi@utility:~/sonos-web $ node server.js
no settings file found, will only use default settings
http server listening on port 8080
queue-changed Office
queue-changed Bedroom
queue-changed Lounge
fetching album art from undefined
fetching album art from undefined
fetching album art from undefined
fetching album art from undefined
fetching album art from undefined
fetching album art from undefined
200
fs.js:133
throw new ERR_INVALID_CALLBACK();
^
TypeError [ERR_INVALID_CALLBACK]: Callback must be a function
at maybeCallback (fs.js:133:9)
at Object.exists (fs.js:201:3)
at ClientRequest. (/home/pi/sonos-web/server.js:58:19)
at Object.onceWrapper (events.js:273:13)
at ClientRequest.emit (events.js:182:13)
at HTTPParser.parserOnIncomingClient [as onIncoming] (_http_client.js:546:21)
at HTTPParser.parserOnHeadersComplete (_http_common.js:109:17)
at Socket.socketOnData (_http_client.js:432:20)
at Socket.emit (events.js:182:13)
at addChunk (_stream_readable.js:283:12)
at readableAddChunk (_stream_readable.js:264:11)
at Socket.Readable.push (_stream_readable.js:219:10)
at TCP.onread (net.js:638:20)
Installed NVM and downgraded 8.11.3 (lts/carbon?)
Updated all the packages, and ran again
pi@utility:~/sonos-web $ node server.js
no settings file found, will only use default settings
http server listening on port 8080
queue-changed Office
queue-changed Bedroom
queue-changed Lounge
fetching album art from undefined
fetching album art from undefined
fetching album art from undefined
fetching album art from undefined
fetching album art from undefined
fetching album art from undefined
404
404 'linking' '/home/pi/sonos-web/cache/86b9d7c3ff5e309dff2237b4b20348e9'
404
404 'linking' '/home/pi/sonos-web/cache/e4d53b0fe4b8a9a58b6b70fc04a22f19'
serving /getaa?u=x-file-cifs%3a%2f%2fSynology%2fmusic%2fiTunes%2520New%2fiTunes%2520Media%2fMusic%2fCompilations%2fAt%2520The%2520Controls%2520%255bDisc%25201%255d%2f1-07%2520Michelle.m4a&v=71
serving /getaa?u=x-file-cifs%3a%2f%2fSynology%2fmusic%2fiTunes%2520New%2fiTunes%2520Media%2fMusic%2fCompilations%2fAt%2520The%2520Controls%2520%255bDisc%25201%255d%2f1-06%2520El%2520Encuentro.m4a&v=71
404
All fine.
One last quick question - Im looking at the mobile ’skin’. It seems to default to one on my Sonos Zones. Is there any way currently to switch which zone it controls?

On 18 Jul 2018, at 15:42, Jimmy Shimizu<EMAIL_ADDRESS>wrote:
I don't see why it wouldn't run in 10.x as well, but makes a good point to make sure it does, thanks for reporting it.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub https://github.com/jishi/node-sonos-web-controller/issues/64#issuecomment-405955920, or mute the thread https://github.com/notifications/unsubscribe-auth/AArNQiBC13wwmXXZG11fbwZk2innXIyuks5uH0lxgaJpZM4VUc28.


Icon not drawn for image of svgGraphicsSize(1,1)

Expected Behavior
All step icons should be drawn
Actual Behavior

Steps to Reproduce the Problem

Draw icons whose svgGraphicsSize is (1,1) like (ESRI Shapefile Reader, Cassandra Input, Cassandra Output)

Specifications
This is a regression of ea43b9526ba3e1050497f21e065e4bab215ea987

Resolved by 595231248aeb11bc13352b501bec15338407594e.


Missing example

It would be nice to have a real example pdf in the doc to see the resulting layout. (Or as part of the main documentation via \includegraphics).

Thanks for the feedback. I'll work on it.


[New Feature]: Adding a dynamic effect to the title "Physi-c Tech"

🎮 feature Request
I'd like to enhance the title on the landing page and make it look dynamic which will replace the plain text as it will make it look more enganging. Please assign this issue to me under GSSOC'24
Point down the features
CSS Effects
Select program in which you are contributing
No response
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

Please add the level label

Please add the level label


build: fix tests (PROOF-620)

Rationale for this change
In switching to clang, I missed an option in the build that is skipping all of the tests.
What changes are included in this PR?

Add missing alwayslink option so that tests can run
Fix a warning that clang gives.

Are these changes tested?
Yes.

:tada: This PR is included in version 0.2.4 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Initial Commit to implement Dora Adapter based on LoRA

Implementation of DoRA based on the paper.
I have tried to resuse the Lora components as much as possible. Haven't implemented merging and unmerging till now.

Closing in favor of an even better implementation the PR: https://github.com/huggingface/peft/pull/1474

Thanks for this PR, even if there is already the other PR. Please feel free to contribute to that one, e.g. I see that you worked on embedding layers, which are not supported by the other one yet.


Crash with newer version of neoforge

Simply put, the problem is the same as with https://github.com/Uraneptus/SnowPig-forge/issues/15.

Thanks! I hate this 😭
Will release a fix for this mod and DITR later today ^^


added codecov configuration

Description
added some configuration that might be required to fix the codecov uploading
Type of change

[X] Bugfix

/test

nice :)
/lgtm

/lgtm

closing this PR as all my squashing attempts failed please look at the fresh PR https://github.com/openshift/hac-dev/pull/20


Users can exclude custom categories from base weight

This is the first bit of my work on https://github.com/maplethorpej/packstack/issues/11.
I have worked off https://github.com/maplethorpej/packstack/pull/42 as I like @darinalleman's toggle feature 😄 so this PR will need to wait for that one.
For this feature to be complete, I believe it needs the following:

[x] When creating a new Category as part of an Inventory Item create, users should be able to toggle to exclude that category from the base weight.
[ ] When editing an Item (via the edit item modal) and moving it to a new custom category, users should be able to toggle to exclude that category from the base weight.
[ ] In the Inventory display table, users should be able to click the name of their custom category and see a modal giving them the option (toggle) to change the exclude_weight field of their category. (There should also be a warning about how changing this value will affect all Items in that category.)
[x] In the Pack weight breakdown, items in custom categories which are excluded from weight, should not be counted as part of the "Worn" or "Consumable" weights, but have a separate breakdown row. (If a pack has no custom categories with excluded weights, this row should not appear in the breakdown.)

I have done the easy bits (see demo below), but the others are trickier: I may actually need to learn some React rather than simply going by trial-and-error/guesswork on how things wire together 😅 . Any tips/help (especially with the 3rd task which is killing me) would be awesome 🙏 .
https://user-images.githubusercontent.com/8898786/107118827-3ab2e200-687b-11eb-8fd0-e9a4299ebd75.mov

Wow, this looks really clean!
For the 3rd task, are you looking more for UI suggestions or for more of the code side of things?

Code side mainly, I have a good idea of how I want things to look. I think what I need to do (as in would be the tidiest pattern, I'm sure there are lazier ways) is have a new component which will handle users updating that one option on their category. I have zero web dev/FE/react experience so I thought I would use the EditItem component as a template, which is helpful but also confusing as I can tell there is a lot of non-explicit things happening which I don't have background context for.

Oh I see what you mean. I feel you, I didn't have any react experience before working on Packstack either. I'm still definitely in the trial-and-error/guesswork phase too 😂 I'll try forking off your branch and see if I can figure something out.
Although, I'm kinda wondering if there is any other things that we'd want to allow users to edit with categories, that maybe we would also want to allow users to edit the name of the category too. I don't know if there is anything else we'd want them to edit but if it's just the name and toggling whether or not it's included in the baseweight, I think we could get away with it not being in a popup at all. I'm thinking like, it could be just inline. So maybe you click an "edit" button on the category header row and then the category name becomes a text box and the 'include in baseweight' toggle also appears. The edit button turns in to a 'save' button, and a cancel button appears, kinda like how inline editing items currently works.
A lot of word vomit there. We can probably chat on Slack or whatever else works for you if you wanna hash this out a bit more.

I don't know if there is anything else we'd want them to edit (...) So maybe you click an "edit" button on the category header row and then the category name becomes a text box and the 'include in baseweight' toggle also appears. The edit button turns in to a 'save' button, and a cancel button appears, kinda like how inline editing items currently works.

Yeh you are probably right that there wont be much to edit for categories. Also maybe this is simpler to implement and I was just making things difficult for myself 😂 . I will hopefully have time to get back to this this weekend, I will ping you on slack.


Can you provide with instruction generation model?

Hello, I'm curious about the setting that uses generated instruction, which adopts ViLBERT and is trained to generate captions for single BnB image-caption pairs. Can you provide the scripts and trained model? Thanks.

Hi, sorry I don't have the code anymore, but it is quite trivial to implement. Results were quite disappointing though. I think you could look at more recent papers such as Visual-Language Navigation Pretraining via Prompt-based Environmental Self-exploration (Liang et al.).


Typo fix

Fix's the type specified in Issue: #13

Just realised i pushed with the wrong Commit Name


Autocomplete directories with slash at the end.

For example, I want to execute vim edit command so I type :e /ho:

after hitting tab I get :e /home:

I wish I get :e /home/.
It reduces typing and is compatible with vim.

This seems like a duplicate of #754

I'll resolve this as a duplicate - if you could '+1' #754 , that would be helpful in our prioritization!


[Improvement] Use safe constructor with snake yaml

Purpose of the pull request

Brief change log

Verify this pull request

This pull request is code cleanup without any test coverage.
(or)
This pull request is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:

(or)
If your pull request contain incompatible change, you should also add it to docs/docs/en/guide/upgrede/incompatible.md

Codecov Report
Attention: Patch coverage is 30.00000% with 7 lines in your changes are missing coverage. Please review.

Project coverage is 39.07%.  Consider uploading reports for the commit d91cfbb to get more accurate results

Files
Patch %
Lines

...scheduler/common/utils/ClassFilterConstructor.java
0.00%
7 Missing :warning:

Additional details and impacted files
@@             Coverage Diff              @@
##                dev   #15758      +/-   ##
============================================
- Coverage     39.09%   39.07%   -0.02%
+ Complexity     4852     4851       -1
============================================
  Files          1316     1317       +1
  Lines         44965    44936      -29
  Branches       4809     4786      -23
============================================
- Hits          17578    17559      -19
+ Misses        25487    25477      -10
  Partials       1900     1900

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

This is a case where unit test is helpful and should be mandatory, to verify that the nested types can be parsed without error. Can you add some?

Sure

I am new to DS and have a question here.
Login users are allowed to exectue code on worker side by default, then do we still need to worring about security issues inside modules like org.apache.dolphinscheduler.plugin.task

I am new to DS and have a question here. Login users are allowed to exectue code on worker side by default, then do we still need to worring about security issues inside modules of org.apache.dolphinscheduler.plugin.task?

I think it's normal for the server to execute certain commands to the worker. However, security issues such as those in the org.apache.dolphinscheduler.plugin.task module affect the server, not the worker.

Hi @an5er ,do you mean that
https://github.com/apache/dolphinscheduler/blob/08ac1322864edf42903c7c03942fcad62c37da35/dolphinscheduler-task-plugin/dolphinscheduler-task-api/src/main/java/org/apache/dolphinscheduler/plugin/task/api/k8s/impl/K8sTaskExecutor.java#L143-L149
runs on the master machine?

No, it runs on the machine where the dolphinscheduler is deployed, not in the k8s cluster.

This is a case where unit test is helpful and should be mandatory, to verify that the nested types can be parsed without error. Can you add some?

Sure

@EricGao888 will you update this pr?

This is a case where unit test is helpful and should be mandatory, to verify that the nested types can be parsed without error. Can you add some?

Sure

@EricGao888 will you update this pr?

Yes, but later this week. Quite busy recently. 😢

This is a case where unit test is helpful and should be mandatory, to verify that the nested types can be parsed without error. Can you add some?

Tests added.


Efficiency of shortcutting

Follow-up to #508
Methods such as IO.map and IO.flatMap currently switch over the IO's tag.
There may be more efficient ways of deciding code paths.
@jdegoes mentioned the Visitor Pattern
Subclassing may also be an option.
We should create some benchmarks to compare the different solutions.

Sorry for butting in.
I think maybe using an explicit if-then-else might be another approach to consider.
Of course it isn't as nice but it might prove to be more performant, expecially if some code paths are taken far more often than others.
If a larger number of cases has to be considered using the numeric values of the tags and essentially doing binary search might be interesting. Something like:
    if (tag <= 4) {
      if (tag <= 2) {
        if (tag == 1) {
          ???
        } else {
          ???
        }
      } else {
        if (tag == 3) {
          ???
        } else {
          ???
        }
      }
    } else {
      if (tag <= 6) {
        if (tag == 5) {
          ???
        } else {
          ???
        }
      } else {
        if (tag == 7) {
          ???
        } else {
          ???
        }
      }
    }


RFC: the output format

The current output format uses nested objects to locate the file. While in this approach, the depth of the messaging content may changes. It's hard to tell which parts are file paths and which are actually messages without looking through the file system again.
I would recommend the following format
Current
{
  "ja": { // for `ja` locale`
    "App": { // src/App.vue
      "title": "アプリケーション",
      "lang": "言語切り替え"
    },
    "components": { // src/components
      "Modal": { // src/components/Modal.vue
        "ok": "OK",
        "cancel": "キャンセル"
      }
    },
    "pages": { // src/pages
      "Login": { // src/pages/Login.vue
        "id": "ユーザーID",
        "password": "パスワード",
        "confirm": "パスワードの確認入力",
        "button": "ログイン"
      }
    }
  },
  "en": { // for `en` locale
    "App": { // src/App.vue
      "title": "Application",
      "lang": "Change languages"
    },
    "components": { // src/components
      "Modal": { // src/components/Modal.vue
        "ok": "OK",
        "cancel": "Cancel"
      },
      "nest": { // src/components/nest
        "RankingTable": { // src/components/nest/RankingTable.vue
          "headers": {
            "rank": "Rank",
            "name": "Name",
            "score": "Score"
          }
        }
      }
    }
  }
}

Proposed
{
  "target": "/home/root/vue-i18n",
  // TODO: some other options passed to the tool, which may used for infusing

  "components": { // flat component paths
    "src/App.vue": [ // use array for mutliple i18n blocks
      {
        "lang": "json", // or "yaml", "json5"
        // TODO: some other block queries

        "messages": { // messages
          "ja": { // languages
            "title": "アプリケーション",
            "lang": "言語切り替え"
          },
          "en": {
            "title": "Application",
            "lang": "Change languages"
          }
        }
      }
    ],
    "src/components/Modal.vue": [
      {
        "lang": "yaml",
        "messages": {
          "ja": {
            "ok": "OK",
            "cancel": "キャンセル"
          }
        }
      }
    ]
  }
}

This propose may have following pros and cons:
Pros:

Bilateral transform-able
Components stay in a consistent location
Allowing for representing multiple i18n blocks, #12
Keeps more information can be utilized by 3rd party tools

Cons:

Can not directly used by vue-i18n
A little more complex

Further more, there is also a proposal solution for overcome the problem introduced by this format:

Thank you for your RFC!
I like this idea!
It make sense!  👍
This idea hard to handle JSON format directly at translation services, but it's awesome because it contains meta information that can be processed by third-party vendor tools.
I will further refine specifications the your RFC specifications that can be processed with the squeeze / infuse API.

Great work!
May I have some docs about this? Thanks!

OK!
Please wait for a little while longer. :pray:


Add ServiceTemplate.pre_creation_check for api use

A new API method is needed to validate whether a service template can be created with a given config_info. The return should give the caller very detailed validation message so that the caller can correct the problems before actually creating the service template.
This is intended to be used by the v2v project. The base implementation is not given this time. ServiceTemplateTransformationPlan will first implement this method. All subtypes can add their own implementation in the future as needed.

@miq-bot add_label transformation, enhancement
@miq-bot assign @gmcculloug
cc @eclarizio @abellotti @jntullo

replaced by https://github.com/ManageIQ/manageiq/pull/17177


Feature request: allow reading mongo config from file

I'd like to use this tap with some MongoDB clusters that use hashicorp vault to manage credentials. If I could read the mongo config setting from a file (JSON, YAML, even just a plain text file with a MongDB URI connection string) then I'd be able to easily use this tap in that scenario.

Thanks for contribution + issue sir!


Potential for enhancements to pdMS_TO_TICKS?

C++ has awesome support for time.
I want to get all the advantages of std::chrono (mbed has decent support).
By strongly typing time point and duration variables, we can get some pretty cool code.
Unfortunately:
vTaskDelayUntil(&xPreviousWakeTime, pdMS_TO_TICKS(std::chrono::duration_cast<std::chrono::milliseconds>(XMIT_PERIOD_ms).count()));
Is not cool code :(
TaskBase::DelayUntil(100ms); could be quite righteous.
How would a contributor manage obtaining the initial xPreviousWakeTime? Does this belong in a different task class instead? Should it somehow be opt-in?
#if INCLUDE_vTaskDelayUntil
void delay_until(const std::chrono::duration nextWake)
{
    vTaskDelayUntil(&xPreviousWakeTime, std::chrono::duration_cast<std::chrono::milliseconds>(XMIT_PERIOD_ms).count());
}
#endif

Usage:
...
delay_until(1.5s);
...

Of course I included a pull request!
#10

Of course I included a pull request!
#10

I'll need to check how widely  is supported in embedded C++, as that is adding support for it as a dependency to all users of the class. It also adds the need for 64-bit math libraries to the code.
Also, there isn't really a need for 'delay' to be a member function, it can be a freestanding function with no need to access the task context, and might well be wanted outside of a direct member function of the task class. I had some similar thoughts about this when I added take, but that at least dealt with task-specific data (so less likely for an arbitrary library to want to do this action). They also were made protected to point out that other code shouldn't 'take' the signal from a different task. I actually have a simple delay function as part of a general library package I use.
The 'hiding' of the need to initialize the xPreviousWakeTime is perhaps a significant issue too if there is a possible delay before getting into the delay_until loop, as well as forcing ALL tasks to save that value (even if not using it), seems an issue too.

I'll need to check how widely  is supported in embedded C++, as that is adding support for it as a dependency to all users of the class. It also adds the need for 64-bit math libraries to the code.
Also, there isn't really a need for 'delay' to be a member function, it can be a freestanding function with no need to access the task context, and might well be wanted outside of a direct member function of the task class. I had some similar thoughts about this when I added take, but that at least dealt with task-specific data (so less likely for an arbitrary library to want to do this action). They also were made protected to point out that other code shouldn't 'take' the signal from a different task. I actually have a simple delay function as part of a general library package I use.
The 'hiding' of the need to initialize the xPreviousWakeTime is perhaps a significant issue too if there is a possible delay before getting into the delay_until loop, as well as forcing ALL tasks to save that value (even if not using it), seems an issue too.

re: delay:
I'm not sure how this would look as a non-protected/private non-member. I don't see an overload for passing a task handle, so wouldn't it delay the currently executing task? (i.e. task2.delay(1s);) would delay the calling task? I can see it being static definitely.
As for the hiding:
Sure, a created but suspended task (I have one of these waiting on a semaphore to start) would likely have a "too-old" previous wake time.
...I could initialize xPreviousWakeTime to a certain value (say, 0) and check in the delay_until where xPreviousWakeTime is set correctly on first use. I believe that would result in exactly what users expect. WDYT?
With const/consexpr, my particular delay looks like (optimizations on):
loc_8001CC8             ; xTimeIncrement
MOVS    R1, #0x64 ; 'd'
MOV     R0, R7          ; pxPreviousWakeTime
BL      vTaskDelayUntil

With the lazy initialization, ~10 bytes are added:
A3 B1       CBZ     R3, loc_80021B8
...
03 F0 C8 FF BL      xTaskGetTickCount
B8 60       STR     R0, [this,#8]
E6 E7       B       loc_800218E

re: delay:
I'm not sure how this would look as a non-protected/private non-member. I don't see an overload for passing a task handle, so wouldn't it delay the currently executing task? (i.e. task2.delay(1s);) would delay the calling task? I can see it being static definitely.
As for the hiding:
Sure, a created but suspended task (I have one of these waiting on a semaphore to start) would likely have a "too-old" previous wake time.
...I could initialize xPreviousWakeTime to a certain value (say, 0) and check in the delay_until where xPreviousWakeTime is set correctly on first use. I believe that would result in exactly what users expect. WDYT?
With const/consexpr, my particular delay looks like (optimizations on):
loc_8001CC8             ; xTimeIncrement
MOVS    R1, #0x64 ; 'd'
MOV     R0, R7          ; pxPreviousWakeTime
BL      vTaskDelayUntil

With the lazy initialization, ~10 bytes are added:
A3 B1       CBZ     R3, loc_80021B8
...
03 F0 C8 FF BL      xTaskGetTickCount
B8 60       STR     R0, [this,#8]
E6 E7       B       loc_800218E

I think you may be misunderstanding my comment. The call to vTaskDelay always operates on the current task, so a member call like someothertask.delay(100ms) wouldn't make sense. Also, it is quite possible that some function, not also a member function of the task class, will want to use the delay function, but because it is a member function, it would need access to the task class object to make the call. We also have the fact that the implementation of delay makes NO reference to any member of the task class. This means that delay has no need to be a member function, and being one actually makes it harder to use. This also points out that one of your examples, the call to TaskBase::delay_until(100ms) is actually NOT a valid call, as delay_until NEEDS an object to call it on, so it needs to be mytask.delay_until(100ms) (yes, within a member function the 'this' object is assumed, but then you don't need the TaskBase:: prefix either).
This basically means that delay really doesn't belong as a member function of the Task class, at best, it might be a static member function for namespacing purposes, but in my mind is better as a 'free' function, as it has many usages in code that doesn't otherwise need to know about 'tasks'.
Then we get to delay_until, yes that now has reason to want the task object to hide the last time value, but that then comes at the cost of adding that variable to EVERY task in the system, even those that have no need for it. I will say that I find delay_until tasks to be fairly rare in my usages, so the expense is pretty high. Making your delay_until a free function taking a reference to the previousWakeTime variable makes the nice code not that much less nice and makes the variable explicit, and puts a clear point to initialize the time.

I think you may be misunderstanding my comment. The call to vTaskDelay always operates on the current task, so a member call like someothertask.delay(100ms) wouldn't make sense. Also, it is quite possible that some function, not also a member function of the task class, will want to use the delay function, but because it is a member function, it would need access to the task class object to make the call. We also have the fact that the implementation of delay makes NO reference to any member of the task class. This means that delay has no need to be a member function, and being one actually makes it harder to use. This also points out that one of your examples, the call to TaskBase::delay_until(100ms) is actually NOT a valid call, as delay_until NEEDS an object to call it on, so it needs to be mytask.delay_until(100ms) (yes, within a member function the 'this' object is assumed, but then you don't need the TaskBase:: prefix either).
This basically means that delay really doesn't belong as a member function of the Task class, at best, it might be a static member function for namespacing purposes, but in my mind is better as a 'free' function, as it has many usages in code that doesn't otherwise need to know about 'tasks'.
Then we get to delay_until, yes that now has reason to want the task object to hide the last time value, but that then comes at the cost of adding that variable to EVERY task in the system, even those that have no need for it. I will say that I find delay_until tasks to be fairly rare in my usages, so the expense is pretty high. Making your delay_until a free function taking a reference to the previousWakeTime variable makes the nice code not that much less nice and makes the variable explicit, and puts a clear point to initialize the time.

OK, I think I have a better understanding now.
I could probably use timers instead of tasks for my periodic requirements.
That said, I'll whip something up. I liked the hiding of previousWakeTime, but I like the std::chrono more.

As I said, making delay a free utility function seems to make more sense than making it a member function, and it looks like you could create an overload set, one that takes an integer, interpreted as ticks as currently, and another that takes a duration and does the conversion itself.

Great,
I've done that. I tried to use Task as a namespace, but that was taken by the dynamic task typedef. I settled on the "blah" TaskUtil
The PR has been updated

This basically means that delay really doesn't belong as a member function of the Task class, at best, it might be a static member function for namespacing purposes, but in my mind is better as a 'free' function, as it has many usages in code that doesn't otherwise need to know about 'tasks'.

In this fashion, how would you handle the function xTaskAbortDelay()?
https://freertos.org/xTaskAbortDelay.html
Would you include it in the class or you would keep it as freewheel function?

xTaskAbortDelay would make sense as a member function, and in fact, IS currently in the TaskBase class conditioned on INCLUDE_xTaskAbortDelay. Since this can be written as an inline non-virtual function, its presence is free.


Manage WebSockets Across Multiple Tabs

Right now a WebSocket connection is created on every tab.  This connection can remain open even if tabs are hidden.  This environment allows a single user to establish multiple connections with the server.  This can be an unnecessary burden on the server.  There are changes that would improve the situation:

For browsers that support it, a Shared Worker can be accessed from multiple tabs.  This worker will manage the WebSocket connection and proxy messages between the server and the tabs.
If the page is talking directly to the server and is not using a Shared Worker proxy for any reason, the Page LifeCycle API is used to detect when the page becomes inactive or active and close and reopen the server connection in response.

For older browsers, you can open/close WS connections using window.onfocus and window.onblur .


FileNotFoundError: [Errno 2] No such file or directory

Hi ,
There is an error of you are code , the file is there in the specified path but it ssowing there No file,

I tried both v1 and v1,1 same error in both cases.

This is the file we need to sent through the email

@epneo-banuprakash
You may have most likely not cloned your repo, or it may have been overridden.
This GitHub Action does not do the actions/checkout.
Here's a working example (notice line 12-13):
https://github.com/meysam81/debug-send-email/actions/runs/10281462038/workflow

HTML is already supported. Here's an example of a successful run: https://github.com/meysam81/debug-send-email/actions/runs/10295217273/workflow


Simplify the vsl connection to the CIAM slrcost component

@bryanparthum I think this should just simplify things without changing any results, will double check the built in testing suite before merging.

All tests passing locally.  Will wait 24 hours and then merge.


Building CUDA code in mickledore SDK fails due to "-mbranch-protection=standard" flag

I encountered an issue trying to build CUDA code in the SDK of our yocto image based on mickledore. The issue seems to be related to #1302.
When I try to configure a cmake project with includes CUDA code, I get the following error message:

...
CMake Error at /opt/sdk/sysroots/x86_64-pyritesdk-linux/usr/share/cmake-3.25/Modules/CMakeDetermineCompilerId.cmake:739 (message):
Compiling the CUDA compiler identification source file
"CMakeCUDACompilerId.cu" failed.
Compiler:
/opt/sdk/sysroots/x86_64-pyritesdk-linux/usr/local/cuda-11.4/bin/nvcc
Build flags:
--gpu-architecture=compute_87;--gpu-code=sm_87;--include-path;/opt/sdk/sysroots/armv8a-leica-linux/usr/local/cuda-11.4/include;--library-path;/opt/sdk/sysroots/armv8a-leica-linux/usr/local/cuda-11.4/lib;-Xcompiler;--sysroot=/opt/sdk/sysroots/armv8a-leica-linux;-Xcompiler;-march=armv8-a+crc,-mbranch-protection=standard,-fstack-protector-strong,-O2,-D_FORTIFY_SOURCE=2,-Wformat,-Wformat-security,-Werror=format-security
...
#$
"/opt/sdk/sysroots/x86_64-pyritesdk-linux/usr/bin/aarch64-leica-linux"/aarch64-leica-linux-g++-8.5.0
-D__CUDA_ARCH__=870 -E -x c++ -DCUDA_DOUBLE_MATH_FUNCTIONS -D__CUDACC__
-D__NVCC__ --sysroot=/opt/sdk/sysroots/armv8a-leica-linux
-march=armv8-a+crc -mbranch-protection=standard -fstack-protector-strong
-O2 -D_FORTIFY_SOURCE=2 -Wformat -Wformat-security -Werror=format-security
-I"/opt/sdk/sysroots/armv8a-leica-linux/usr/local/cuda-11.4/include"
"-I/opt/sdk/sysroots/x86_64-pyritesdk-linux/usr/local/cuda-11.4/bin/..//include"
-D__CUDACC_VER_MAJOR__=11 -D__CUDACC_VER_MINOR__=4
-D__CUDACC_VER_BUILD__=315 -D__CUDA_API_VER_MAJOR__=11
-D__CUDA_API_VER_MINOR__=4 -include "cuda_runtime.h"
"CMakeCUDACompilerId.cu" -o "tmp/CMakeCUDACompilerId.cpp1.ii"
aarch64-leica-linux-g++-8.5.0: error: unrecognized command line option
‘-mbranch-protection=standard’; did you mean
‘-fexcess-precision=standard’?
--error 0x1 --
...

The build flag "-mbranch-protection=standard" seems to cause this problem. This flag has recently been removed from gcc-configure-common.inc (see #1302).
In the SDK, the build flag is defined in $SDKTARGETSYSROOT/environment-setup.d/cuda_target.sh:

export CUDA_NVCC_ARCH_FLAGS="--gpu-architecture=compute_87 --gpu-code=sm_87"
export CUDA_ARCHITECTURES="87"
export CUDACXXARCHFLAGS="-Xcompiler -march=armv8-a+crc,-mbranch-protection=standard,-fstack-protector-strong,-O2,-D_FORTIFY_SOURCE=2,-Wformat,-Wformat-security,-Werror=format-security"
export CUDAHOSTCXX="aarch64-leica-linux-g++-8.5.0"

This file is generated by the cuda-target-environment_1.0.bb recipe:

...
inherit cuda-gcc
...
def arch_flags(d):
archflags = d.getVar('TARGET_CC_ARCH')
if archflags:
return "-Xcompiler " + ','.join(archflags.split())
return ""
do_compile() {
sed -e"s!@CUDA_NVCC_ARCH_FLAGS@!${CUDA_NVCC_ARCH_FLAGS}!"
-e"s!@ARCHFLAGS@!${@arch_flags(d)}!"
-e"s!@CUDA_ARCHITECTURES@!${CMAKE_CUDA_ARCHITECTURES}!"
-e"s!@COMPILER_CMD@!${COMPILER_CMD}!" ${S}/cuda_target.sh.in > ${B}/cuda_target.sh
}
...

The TARGET_CC_ARCH variable is set to TUNE_CCARGS in poky/meta/conf/bitbake.conf.
The fix implemented in #1306 removes the -mbranch-protection=standard flag from TUNE_CCARGS in the cuda.bbclass, but not in the cuda-gcc.bbclass which is inherited in cuda-target-environment_1.0.bb.
Therefore, the cuda_target.sh file is still generted with the -mbranch-protection=standard flag.
I would propose moving the fix implemented in #1306
TUNE_CCARGS:remove:cuda = "-mbranch-protection=standard"
from the cuda.bbclass class to the cuda-gcc.bbclass class, since the cuda class also inherits cuda-gcc.
Alternatively, the flag could also be removed in the cuda-target-environment_1.0.bb recipe directly.
Thanks for your help!
Cheers,
Patrick

Thanks a lot for the quick response!


Upgrade sidekick installer

Upgraded Sidekick Plugin Installer to ^0.2.1
Upgraded install file
Moved tool/build_ios_command.dart to template/build_ios_command.template.dart
Formatted and fixed analyzer

Works :)


Reproducible build pipeline

Related to #121
For the reproducible build pipeline we are:

Creating a temp folder under a gitignored .w3 folder
Creating a web3api.build.yaml file where we declare configuration parameters for the build
Copy all sources and a bash script with build commands into temp folder
Copy all temp folder contents into Docker environment
Build sources inside Docker environment by executing the bash script
Copy build artifacts to host computer
Delete temp folder

R&D notes:

Sources are built in the Docker image's build process, there is no container daemonizing and we can avoid needing one entirely.
Copying directories in Docker is a thing; conditional and iterative logic are not supported in Dockerfile natively, they are supported in bash commands. That's why a temp folder is needed with all sources, and then they can all be copied in a single COPY instruction in Dockerfile.
Copying files from an image to the host computer without using mounts or volumes require creating and running a temp container that does the copying and then gets removed; this seems to be the cleanest/easiest approach I've seen so far, given our use case.
At this stage we should be aiming to diff built sources only. Since there are factors which are very hard to mitigate (discussed in #121) that make the diffing of 2 image hashes/signatures difficult.
@dOrgJelli found interesting resources for exploring a docker image, layer contents, and discovering ways to shrink its size (https://github.com/wagoodman/dive). This is related to the standard: https://opencontainers.org/

Misc notes:

There is a need for a multi-manifest abstraction/tooling now that we have a build manifest.
The actual WASM build was tested in my fork. I copied the script used but won't work here until packages are published to NPM.
The custom Dockerfile declared in the build manifest can be commented/removed and the CLI will automatically use the default Node image. I had the project's custom image use a different node alpine version on purpose.

Steps to test:

Run yarn and yarn build:cli at root level
Navigate through command line to /demos/simple-storage/repro-build-protocol
Run yarn reproducible-build

@dOrgJelli some still missing TODOs based on our last conversation:

Have the copying of source files into the host be a step in the docker file: this is not something straightforward; it's mostly recommended to create a dummy container that runs the image and then copy from the container to the host (straightforward docker cp)

Verify that the WASM modules have the correct exports (_w3_invoke, _w3_init)

Make two seperate env variables for build folder & output folder. Don't do anything if there's no output directory specified.

Make sure we support multiple build profile, for example web3api.build.debug.yaml -> docker/debug-DockerFile: I think this was related to copying a Dockerfile by specifying the actual Dockerfile and not it's containing folder. This should be easy with fs-extra, which we are already using. Anyways I just stumbled upon this: https://stackoverflow.com/questions/26077543/how-to-name-dockerfiles

closes https://github.com/Web3-API/monorepo/issues/207
closes https://github.com/Web3-API/monorepo/issues/121


AdvisoryData().to_dict() throws error re AffectedPackage object

While trying to print the output of a simple test, where the output was a single AdvisoryData() object, I received this error:
FAILED vulnerabilities/tests/test_archlinux.py::test_parse_advisory - AttributeError: 'AffectedPackage' object has no attribute 'to_dict'

This is the current to_dict() method for AdvisoryData():
    def to_dict(self):
        return {
            "aliases": self.aliases,
            "summary": self.summary,
            "affected_packages": [pkg.to_dict() for pkg in self.affected_packages],
            "references": [ref.to_dict() for ref in self.references],
            "date_published": self.date_published.isoformat() if self.date_published else None,
        }

More context: I'm working on migrating archlinux.py.  Inside my newly-named parse_advisory() method, I create and return a list named advisories containing 1+ AdvisoryData() objects.  Just before I return, I run this simple print that is triggered by my test:
        for apple in advisories:
                pprint.pprint(apple.to_dict())

@johnmhoran the AffectedPackage https://github.com/nexB/vulnerablecode/blob/main/vulnerabilities/utils.py#L41 you are getting is from nearest_patched_package https://github.com/nexB/vulnerablecode/blob/main/vulnerabilities/utils.py#L160 , which is different from the AffectedPackage https://github.com/nexB/vulnerablecode/blob/main/vulnerabilities/importer.py#L121  in importer.py https://github.com/nexB/vulnerablecode/blob/main/vulnerabilities/importer.py, please look at AdvisoryData https://github.com/nexB/vulnerablecode/blob/main/vulnerabilities/importer.py#L224  class in importer.py to know which AffectedPackage class has been used.

@johnmhoran can we close this now?

@TG1999 I have no idea, have not been following this issue, have not encountered anything like this in the recent past.  Perhaps the best approach is to close this issue and if I encounter the same or a similar problem I can open a new issue.

This appears to have been resolved.  Closing now, will reopen/open new issue if need be.


XRP withdraw

XRP withdraw is not working it's given error "Signing is not supported by this server"

@antechdemo probably problem with your xrp node

I am using https://s.altnet.rippletest.net:51234 address to connect to the XRP Test Net.

It will not work. It requires personal node server only

You can try using bitgo

Ok.
Thanks @rxx

Hey ,Ripple address is not generate using "wallet_propose" methog, given error “Forbidden”,When we use personal node server..


implement multiple outputs in get_text_output wrapper

Context: https://github.com/lastmile-ai/aiconfig/pull/421#discussion_r1419581757

Dupe of #467. Closing for now.

@saqadri , this is not a quite a duplicate because run_and_get_output_text also hard-codes just returning the first result: https://github.com/lastmile-ai/aiconfig/blob/ebe5f0f84d48d0597873e2e598126ee4def9bf05/python/src/aiconfig/Config.py#L355


Step to create rexray secrets

Hi @BretFisher
Following the steps of deployment in README,
in Step 1, the REXRAY_DO_TOKEN env. variable is set and in Step 5, external secrets rexray_do_token is used during deployment of the rexray stack.
However this secret is not created at any step prior to Step 5.
A command such as  echo $REXRAY_DO_TOKEN | docker secret create rexray_do_token - could be included in Step 5, prior to the stack deployment command or a new step 5 can be added can be added and subsequent steps renumbered.
I could open a PR for the same, if you agree.

Sure, just add it to step one in a PR, thanks.


[BUG] Vision Transformers not working on Distributed Data Parallel

When trying to use Vision Transformers for feature extraction along side with fastai distrib_ctx(), the code bugs just when starts start training.
In general lines I use a vit_base_patch16_224 model to extract features and compose a multi-task learning model. Using other backbones as ResNet, the code runs smooth but when using transformers it always break the process.
This is the error message
RuntimeError: Expected to have finished reduction in the prior iteration before starting a new one. This error indicates that your module has parameters that were not used in producing loss. You can enable unused parameter detection by passing the keyword argument find_unused_parameters=True to torch.nn.parallel.DistributedDataParallel, and by
making sure all forward function outputs participate in calculating loss.
If you already have done the above, then the distributed data parallel module wasn't able to locate the output tensors in the return value of your module's forward function. Please include the loss function and the structure of the return value of forward of your module when reporting this issue (e.g. list, dict, iterable).
Parameters which did not receive grad for rank 1: encoder.head.bias, encoder.head.weight
Parameter indices which did not receive grad for rank 1: 150 151
The output of my loss is a single number but my model forward function returns a vectors which is disentangled at the loss function and included in a weighted sum to generate loss value.
This is the forward of my loss function:
def forward(self, preds, target):

    v1, v2, v3 = target

    mse = MSELossFlat()

    loss0 = mse(preds[0], v1)
    loss1 = mse(preds[1], v2)
    loss2 = mse(preds[2], v3)

    precision0 = torch.exp(-self.log_vars[0])
    loss0 = precision0*loss0 + self.log_vars[0]

    precision1 = torch.exp(-self.log_vars[1])
    loss1 = precision1*loss1 + self.log_vars[1]

    precision2 = torch.exp(-self.log_vars[2])
    loss2 = precision2*loss2 + self.log_vars[2]

    outloss = (10*loss0)+loss1+loss2

    return outloss

Using a single GPU (instead of distributed processing) it works perfectly.
Desktop (please complete the following information):

OS: Ubuntu 20.04
This repository version: fastai 0.4.12
PyTorch version w/ CUDA/cuDNN : Pytorch 1.10.2, Cuda 11.3.1, cuDNN 8.2.1

@tiagojc it's not a timm bug, in your customizations you've likely left modules in the model that aren't being used and it's telling you which those are, you need to remove the head... ie model.reset_classifier(0) or do it manually....

Thanks @rwightman , thats works perfectly!


Have to create a LOGO

We have to create an amazing LOGO for ProjectsHut Website.

I already kinda new it so I already had  a design created for ProjectsHut (it includes redesigning of heading and a little modification to sub-heading too) + logo
will create PR in few hours hopefully you will like the complete design !

I already kinda new it so I already had  a design created for ProjectsHut (it includes redesigning of heading and a little modification to sub-heading too) + logo
will create PR in few hours hopefully you will like the complete design !

Can you share the final logo?

have already created a PR for this

hey, @ujjawaltyagii I'm thinking that the logo should be changed. Such as add coding related logos.

ok !! will remove this for now

ok !! will remove this for now

One more suggestion for you please change the font style I think it would be great. If you need any help you can ask me on Twitter

Hey , Can you assign this issue to me

@ujjawaltyagii would you like to work on this issue?


Documentation Update request

[Enter feedback here]
Please update this document to say that a minimum of 2 users is required to use the bulk import feature. While it may seem obvious, when testing this feature with one user and getting a generic error "user name/object ID of an existing user in this group  and resubmit your request." It is not clear why.

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

ID: 6d9b26a1-e401-4d6d-f37e-cceed1c04549
Version Independent ID: 3db7c764-4815-ee19-a229-8bad3da4a381
Content: Bulk upload to add or create members of a group - Azure Active Directory
Content Source: articles/active-directory/users-groups-roles/groups-bulk-import-members.md
Service: active-directory
Sub-service: users-groups-roles
GitHub Login: @curtand
Microsoft Alias: curtand

@greendx Thank you for pointing that out . We will review this and have it updated.

@greendx Thank you for your feedback. I have updated the document.


Remove Google Analytics

We've had some users complain about the Ghostery extension flagging
these. As far as we know, they're unused, so remove them.

@redrun45 so how confident are you that this is fine to remove? I don't know much about Google Analytics, but I imagine there's a management console somewhere that this used to report into?

None of the current admins have access to the Google Analytics account at all, so there's no manual process we'd break.  Archive folks wouldn't have it either, and I don't see anything in our code that might use it programmatically...
I think we can be pretty sure about this one.


add lifetime argument for the update command

#139
Change is below.

Add a command-line argument lifetime. When set, the command will stop executing if its runtime exceeds the specified lifetime.
Add test code for the new command.

@Luliko-hub thanks for your contribution! However, this looks like a pretty intrusive solution, since it will break the process somewhere in the middle (who knows what this may lead to). Maybe, instead of using timeout, we can simply check time before starting every next judge and simply skip them if time is out?


fix: swap sol crash

📝 Description
Remove swap edit fee button from no editable families
Affected: LLD & LLM

❓ Context

JIRA or GitHub link: https://ledgerhq.atlassian.net/browse/LIVE-11609

✅ Checklist
Pull Requests must pass the CI and be code reviewed. Set as Draft if the PR is not ready.

[ ] npx changeset was attached.
[ ] Covered by automatic tests.
[ ] Impact of the changes:

...

🧐 Checklist for the PR Reviewers

[ ] The code aligns with the requirements described in the linked JIRA or GitHub issue.
[ ] The PR description clearly documents the changes made and explains any technical trade-offs or design decisions.
[ ] There are no undocumented trade-offs, technical debt, or maintainability issues.
[ ] The PR has been tested thoroughly, and any potential edge cases have been considered and handled.
[ ] Any new dependencies have been justified and documented.
[ ] Performance considerations have been taken into account. (changes have been profiled or benchmarked if necessary)

/generate-screenshots


Use of Loris IIIF server at Wellcome

Stacks post!

First draft is out for review, and I’m busy editing. Hopefully get it up tomorrow or Monday.


Added GravityData and removed gravity from ArmorStandData

API | Common
Adds GravityData and removes gravity from ArmorStandData. This is due to a change in Minecraft  1.10 that allows all entities, not just armor stands, to be affected by the NoGravity tag.
Resolves #1300.

Well this is just my personal opinion, but for compatibility reasons, you should just mark the methods and Key constant as deprecated.
In fact you could "bind" them to the same value.

@ST-DDT Since this is going into bleeding, and will be apart of API 5, I though it would make sense to just remove the methods.

I'm ready to squash the commits, if everything looks good.

@Flibio see my comments

@kashike I've resolved both in b3cad6e.

@kashike They were both resolved, is everything good now?

You changed the javadocs but didn't change the name.
Should be gravity(), not ignoreGravity()

@kashike  @ZephireNZ
My apologies, I thought the comments were both referencing the Javadocs. The method names have been changed.

@Aaron1011 What is wrong with it? I'm unsure what I must change.

@Flibio: The comment says "If the value is true, the * {@link Entity} will ignore gravity" - but if it's true, gravity is turned on.

@Aaron1011: I'll get that fixed right away.

All the comments have been resolved. I'm ready to squash the commits, if everything looks good.

Is there anything holding this PR back? I could use this for a plugin I am working on.

Since all Entity instances can have this flag usable, could you add a default getter on Entity and then override in the implementation (as exemplified before) for ease of use? After that's done, squash these commits.

@gabizou I've added the method hasGravity to Entity.

@gabizou I've got that changed.

@gabizou I've got that changed.

@ZephireNZ @gabizou I've updated the name of the method.

@ZephireNZ @gabizou @kashike I've fixed the return value.

@gabizou Is there anything else that should be changed?

If you could add a test, that would make testing and pulling easier:

Tests that the datamanipulator can be fetched from a relevant entity, modified with keys, and then given.
Tests that keys can be fetched from a relevant, and a new value can be passed.
Test should be easily observable in the game, and/or be printed (can just use System.out.println, no need to be fancy)

You don't have to bother with making it a unit test, just a simple plugin class with command and/or event listeners will do.

Oh, and don't add it to the pull commit, just copy/paste into the description.

@ZephireNZ I've added a small test.

That tests the key getter/setter, but not the data.

@ZephireNZ: Sorry, I missed that. I've updated the example.

Why do you just call it ENTITY_HAS_GRAVITY instead of GRAVITY?

The key this is replacing is called ARMOR_STAND_HAS_GRAVITY, so I
followed that style.
On Monday, July 25, 2016, liach<EMAIL_ADDRESS>wrote:

Why do you just call it ENTITY_HAS_GRAVITY instead of GRAVITY?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/SpongePowered/SpongeAPI/pull/1301#issuecomment-234954322,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ALsgNlWei3nbPUAFkCYADayASU0c6v3jks5qZLuQgaJpZM4JEni_
.

@liach: The key this is replacing is called ARMOR_STAND_HAS_GRAVITY, so I followed that style.

The data should be as general as possible. While the vanilla implementation of gravity may only work on entities, mods may add gravity for blocks, etc. Therefore you should avoid using the data's target in the name when possible. Or, as we've seen with this, newer versions may add or remove targets for data.
In hindsight the armour stand data could have just used NO_GRAVITY and avoided needing to change the key name with a change.
correct, @gabizou ?

If gravity on blocks ever becomes a thing, I think that it's a different enough concept to be named differently, block gravity only updates on physics updates for example. I also suspect that it would be implemented as a blockstate, unless a future tile entity has support to have gravity.

Keep it as GRAVITY.

@gabizou: I've update the key name, let me know if this is ready to be squashed and merged.

@gabizou: Does anything else need to be changed?

@Flibio: Can you rebase this against bleeding?

@Aaron1011: I've rebased the branch, should I squash the commits now?

@Flibio: Yes, please.

@Aaron1011: Everything is squashed and ready to go.

Is there anything else I must do or is this ready? This would be useful for a plugin I am working on.


Add Perf Test to Azure Pipeline

Branching off of #2125 to test perf-test integration in ADO pipeline.
Microsoft Reviewers: Open in CodeFlow

Warnings

:warning:
There are no updates provided to CHANGELOG. Ensure there are no publicly visible changes introduced by this PR.

  Generated by :no_entry_sign: dangerJS

Closing PR per comment on #2125


Prevent Nav from Trapping while tabbing

Bug/issue #, if applicable: rdar://94044029 ([AX] Localnav should not FocusTrap for documentation pages)
Summary
For accessibility considerations, when we tab to the end of the nav menu, we should tab to the browser menu bar next (instead of tabbing to the beginning of the nav menu again).
Dependencies
NA
Testing
Steps:

Medium or small viewport, tab open the nav menu bar
tab through the items in the menu, at the end, focus should be brought to the browser address bar. It should not take you to the first link within our nav menu bar.

Checklist
Make sure you check off the following items. If they cannot be completed, provide a reason.

[x] Added tests
[x] Ran npm test, and it succeeded
[ ] Updated documentation if necessary - NA

@swift-ci test

@swift-ci test

@swift-ci test

@swift-ci test

@swift-ci test


can't run python .py

Issue Type: Bug
Can't run a helloworld python file that is located inside a OneDrive folder. It gives error.
The file contains line
print("Hello World!")
Starting debug or run without debugging throws the same error
ImportError: can't find 'main' module in 'c:\Users\***\OneDrive'
The system cannot find the path specified.
VS Code version: Code 1.66.0 (e18005f0f1b33c29e81d732535d8c0e47cafb0b5, 2022-03-30T05:50:14.623Z)
OS version: Windows_NT x64 10.0.19044
Restricted Mode: No

System Info

Item
Value

CPUs
11th Gen Intel(R) Core(TM) i7-11850H @ 2.50GHz (16 x 2496)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: enabledopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_renderer: enabled_onvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
31.73GB (22.61GB free)

Process Argv
--crash-reporter-id 67b3a954-596f-4132-8b48-52b1eb583774

Screen Reader
no

VM
0%

Extensions (16)

Extension
Author (truncated)
Version

doxdocgen
csc
1.4.0

better-cpp-syntax
jef
1.15.13

python
ms-
2022.4.0

vscode-pylance
ms-
2022.3.4

jupyter
ms-
2022.3.1000901801

jupyter-keymap
ms-
1.0.0

jupyter-renderers
ms-
1.0.6

remote-containers
ms-
0.231.4

remote-ssh
ms-
0.78.0

remote-ssh-edit
ms-
0.80.0

remote-wsl
ms-
0.66.0

cmake-tools
ms-
1.10.5

cpptools
ms-
1.9.7

cpptools-extension-pack
ms-
1.1.0

LiveServer
rit
5.7.5

cmake
twx
0.0.17

(1 theme extensions excluded)

A/B Experiments
vsliv368cf:30146710
vsreu685:30147344
python383cf:30185419
vspor879:30202332
vspor708:30202333
vspor363:30204092
pythonvspyl392:30443607
pythontb:30283811
pythonvspyt551:30345470
pythonptprofiler:30281270
vshan820:30294714
vstes263cf:30335440
pythondataviewer:30285071
vscod805:30301674
pythonvspyt200:30340761
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
vsaa593:30376534
vsc1dst:30438360
pythonvs932:30410667
wslgetstarted:30449410
pythonvsnew555:30457759
vscscmwlcmt:30465135
pynewfile477:30463512

/extPython

It turns out that the folder name where the python code was located had an '&' symbol. This is a valid character for a folder in Windows 10 but VScode debugger or python throw this error.

Please report this valuable information in the Python extension repository.
VS Code cannot do anything with it...