Ci/acceptance

The objective is to propose a solution to execute acceptance tests in dev and CI.

Codecov Report

Merging #1 (fa09930) into main (0726b9d) will decrease coverage by 3.20%.
The diff coverage is 55.82%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##             main       #1      +/-   ##
==========================================
- Coverage   82.19%   78.99%   -3.20%
==========================================
  Files          11       12       +1
  Lines         483      538      +55
==========================================
+ Hits          397      425      +28
- Misses         82      104      +22
- Partials        4        9       +5

Impacted Files
Coverage Δ

condition.go
0.00% <0.00%> (ø)

error.go
0.00% <ø> (ø)

schema/schema.go
100.00% <ø> (ø)

connector/drivers/field.go
70.58% <16.66%> (-29.42%)
:arrow_down:

builder.go
53.01% <53.01%> (ø)

payload.go
75.00% <72.72%> (-1.00%)
:arrow_down:

connector/drivers/mysql.go
96.77% <90.00%> (-3.23%)
:arrow_down:

connector/drivers/where.go
100.00% <100.00%> (ø)

schema/field.go
100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Dedupe local types from ATA and reuse old programs correctly

Ports #20464

√ 57813 passing (11m25s)


resolution choice / arguments ?

First of all thank you for this.
Is there a way to choose dl resolution (at least for youtube 360p, 720p or 1080p) ?
Regards

My pleasure hopefully you find it useful.
Currently it will just download the highest quality audio and video available, I will probably end up adding functionality whereby you can pass custom flags and then your functionality should be possible.

Thank you very much for your answer :)
Your tool is great, even for a very code noob as i am.
Thank you

Awesome to hear. Are you on Windows?
Also if you go here:
https://github.com/mayeaux/videodownloader/blob/master/renderer.js#L48
You can pass different flags to youtube-dl and customize the arguments passed. Read up on how youtube-dl works and then you can use that to download custom resolutions

Yes i'm on windows
i found the arguments but don't know how to insert it:
-f "best[height=720]
(Where idi i insret it ? did i remove the // in front of line ? )
anyway not a real problem, downloaded in max resolution and then re-convert it ;)
Thank you again for this tool and for your answers (and sorry for my bad english)

 // select download as audio or video
  if(downloadAsAudio){
    arguments.push('-f');

    // arguments.push('bestaudio');

    arguments.push('bestaudio[ext!=webm]');

    /** conversion taking too long atm **/
    // arguments.push('--extract-audio');
    //
    // arguments.push('--audio-format');
    //
    // arguments.push('mp3');

    // can add something here later
  }  else {

     arguments.push('-f');
     arguments.push('best[height=720]')
  }

No problem, add the part I just added in the else {} block and it should work! And your English is fine, happy to help and happy to hear it's working well on Windows :)

Thank you for your so quick answer :)
But it don't work. i certainly did something wrong.
i insert the code in the else {} block of the renderer.js part but then i have an error:
`renderer.js:165 stderr: ERROR: requested format not available
renderer.js:165 stderr: Traceback (most recent call last):
File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmphnjfkuku\build\youtube_dl\YoutubeDL.py", line 807, in extract_info
File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmphnjfkuku\build\youtube_dl\YoutubeDL.py", line 861, in process_ie_result
File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmphnjfkuku\build\youtube_dl\YoutubeDL.py", line 1634, in process_video_result
youtube_dl.utils.ExtractorError: requested format not available
renderer.js:182 child process exited with code 1`
The video is playable in 720 via youtube.
Anyway really not a problem i convert it to the size i needed and this is perfect.
Again thank you very much for your coding skills and you kindness.

Try to run youtube-dl -F yoururl.com to get the available formats. If you get youtube-dl working from the command line you should be able to test things manually before you plug it into videodownloader

will try, thank you


baseImages: Update base images docs

Add new stack: dotnet
Update latest node and golang version

Change-type: patch
Signed-off-by: Trong Nghia Nguyen<EMAIL_ADDRESS>

Should the updated node and golang versions also be added to the "Multiple language stacks" section?


feat: support CodeBlock

FX-4184
Description
The result of merging all separate PRs
How to test

Temploy
play with the storybook

Screenshots

Development checks

[x] Add changeset according to guidelines (if needed)
[x] Read CONTRIBUTING.md and Component API principles
[x] Make sure that additions and changes on the design follow Toptal's BASE design, and it's been already discussed with designers at #-base-core
[x] Annotate all props in component with documentation
[x] Create examples for component
[x] Ensure that deployed demo has expected results and good examples
[x] Ensure the changed/created components have not caused accessibility issues. How to use accessibility plugin in storybook.
[x] Self reviewed
[x] Covered with tests (visual tests included)

Breaking change

codemod is created and showcased in the changeset
test alpha package of Picasso in StaffPortal

All development checks should be done and set checked to pass the
GitHub Bot: TODOLess action

PR commands

List of available commands:

@toptal-bot run package:alpha-release - Release alpha version
@toptal-anvil ping reviewers - Ping FX team for review

PR Review Guidelines

When to approve? ✅
You are OK with merging this PR and

You have no extra requests.
You have optional requests.

Add nit: to your comment. (ex. nit: I'd rename this variable from makeCircle to getCircle)

When to request changes? ❌
You are not OK with merging this PR because

Something is broken after the changes.
Acceptance criteria is not reached.
Code is dirty.

When to comment (neither ✅ nor ❌)
You want your comments to be addressed before merging this PR in cases like:

There are leftovers like unnecessary logs, comments, etc.
You have an opinionated comment regarding the code that requires a discussion.
You have questions.

How to handle the comments?

An owner of a comment is the only one who can resolve it.
An owner of a comment must resolve it when it's addressed.
A PR owner must reply with ✅ when a comment is addressed.

@toptal-anvil ping reviewers

@toptal-anvil ping reviewers

Do you think the output could be cleaned up a bit so it does not have those extra <span/>?

Tested locally.
When only the empty lines are present in the editor, it lacks the margin that is there otherwise.
https://github.com/toptal/picasso/assets/18409292/eb953c45-bcb1-4ab6-870c-b4bd2bea0cc8
Also there's one thing that I'm not sure we should fix, but you can't really select existing part of the text and turn it into code block, it creates a block up until the existing node / paragraph boundary
https://github.com/toptal/picasso/assets/18409292/d5676ba9-6f00-4403-bc79-7d64a4bcecfd

@mkrl The second issue is the same as we had with headings, it is still one paragraph as there is just br as space between.
I was actually trying even to convert just selected text, but it is pretty complex with a lot of edge cases. There is no specific requirement for this behaviour and I realized that there is not much benefit and it would be inconsistent with headings

@toptal-anvil ping reviewers

thank you guys for finding these bugs 🙇, I addressed them all


Under Multisig account co-signers are displayed as "(Multisig)" in "Multisig accounts" filter

Steps to reproduce:

Create a Multisig account
Go to Dashboard or Accounts/restrictions page, Mosaics, Namespaces
Click on the "Multisig accounts" filter

Expected result: Co-signers are indicated as "(Co-signer)" or similar
Actual result: Co-signers are displayed as "(Multisig)" in the filter
Screenshot

@Eugenity please verify.

Verified


Everytime i open the Picker or Explorer, the galaxyline disappears

Man, it is just not usable this way.
I tried using the plugin again, after many months (when i installed the last time it was like this), and the same behavior happens:
https://user-images.githubusercontent.com/17055027/144287716-3d4f24a5-31e6-4abe-ae5f-bc01172bdaf4.mp4

This plugin does not touch the status line, nor am I familiar with it's inner workings. I'm afraid you'll have to open an issue on galaxyline issue tracker instead.
Like I said last time, the issue is not present for me on feline so there must be something galaxyline can do about it.

It's best if you provide a minimal reproducible config if you want plugin developer to look into your issue...

I installed a random galaxyline config and couldn't reproduce the issue... There's no way for me to look into this if you don't share your config.

I was able to reproduce the issue with the config found in your dotfiles. :setl statusline yields
statusline=%=

in the situation from your gif. Again, this plugin does not touch the statusline so I suggest you open an issue with galaxyline instead.

The issue is that galaxyline relies on autocmd events to refresh the statusline. Specifically the WinEnter or BufEnter events should fire when closing the nnn window but for some reason that doesn't happen when closing nnn with q.
I could add a workaround but this seems to be a bug in neovim to me. Perhaps something to do with autocmds not firing during an on_exit callback.


Change appearance of Today's date

Hi
Is there any option to replace today date's background image (filled circle default).
Can we add dash line instead of image?
Can we make today's date bold?

Hi @ummerakbar, you go through the DIYCalendarCell.swift in issue #1281 to find solution for your question. Even I added the file in my project and modified the code as per my requirement. I guess, you can do the same thing.


Automatic MONAI install

Instead of asking the user to install MONAI manually, we should display a confirmation popup: OK to install MONAI or cancel to install it manually.

This could be done similarly to https://github.com/fepegar/SlicerTorchIO/blob/c3fe328b3a940e2e2f11ea7735f5030153be7a4b/TorchIOModule/TorchIOModule.py#L64-L94

It can wait, as the user can install using the Python console, but it would be a significant usability improvement.
Probably some time is needed for internal testing and small fixes and improvements so everything could come together in the next few weeks for a public announcement.

I am looking into this. Can someone assign the issue to me?


HOM: Annotations on Beans and Bean Fields for CfDef and ColumnDef Respectively

Having annotation such as @Index on fields and @RowChache on beans would give the user a lot of flexibility in tuning cassandra to their application without too much code on their part. Keeping these annotations optional will reduce or eliminate impact on user's who wouldn't use these features.
From my perspective though, the biggest win is being able to make indexed queries on beans persisted by HOM. I think this use case is common enough that it is worth including in HOM.

as you can see, i'm just now getting around to these issues, sorry.
did you invision using @index and @RowCache with regard to a keyspace/schema creation tool?  not sure how they would help during runtime, but i could see that if a schema creation tool existed, these annotations would guide it to produce an optimal keyspace definition.

Yes that's right. The user annotates schema creation hints along side their structures and fields and can use HOM to create the schema. We obviously want to have an on and off switch for this though.
The idea is that the developer does not have to code to anything other than HOM, yet still have a lot of power over Cassandra itself.
HOM can take advantage of knowing the schema, like creating queries based on secondary indices.

care to take a stab at it and offer the code :)
On Fri, Oct 21, 2011 at 11:29 AM, Javier Sotelo
<EMAIL_ADDRESS>wrote:

Yes that's right. The user annotates schema creation hints along side their structures and fields and can use HOM to create the schema. We obviously want to have an on and off switch for this though.
The idea is that the developer does not have to code to anything other than HOM, yet still have a lot of power over Cassandra itself.
HOM can take advantage of knowing the schema, like creating queries based on secondary indices.
--
Reply to this email directly or view it on GitHub:
https://github.com/rantav/hector/issues/194#issuecomment-2484628

Sure. Would you like me to take a stab at #227 and #310 also?

take a look at 310, i've already been playing around with 227
On Fri, Oct 21, 2011 at 1:03 PM, Javier Sotelo
<EMAIL_ADDRESS>wrote:

Sure. Would you like me to take a stab at #227 and #310 also?
--
Reply to this email directly or view it on GitHub:
https://github.com/rantav/hector/issues/194#issuecomment-2485610


multiple tickets

Keep an eye on GoNPC line that you've added; the game lets you use dialogs if you're targetting someone else


capture cli output ready for docs consumption

Capture output and asserted lines to be used in doc systems for highlight.

Discussed this with @MykolaGolubyev , here's the list of things to capture that we came up with:
• actual command ran
• stdout
• stderr
• exit code
• asserted lines

closed by #360


(Fix) #511 - QR scan button

Closes #480
Closes #511
Description
Adds QR code scan button on the following pages:

Add Owner
Replace Owner
Load Safe
Create new address book entry

Also fixs the add recipient by qr

Travis automatic deployment:
https://pr873--safereact.review.gnosisdev.com/app

@Agupane
Issue.
The QR code in the "Load Safe" form is not working.
URL: https://pr873--safereact.review.gnosisdev.com/app/#/load
The console is not showing any errors.
I've tried the QR code in other places like "Send funds" and it works there, so the QR code itself is not the issue
I leave you the QR code i used to test it, this is the address it should display:
0x9913B9180C20C6b0F21B6480c84422F6ebc4B808

In the load-safe flow, it's not working.
It works well in the SendFunds modal. But doesn't work in the Send Collectible, nor Custom Tx (this last one, not sure if it's that important as it's completely refactored in #741.

They are fixed now, even Custom Tx, I wasn't a big problem to fix it so I did it also
@francovenica you will be able to test again

Travis automatic deployment:
https://pr873--safereact.review.gnosisdev.com/app

The issue with the load safe form was fixed
Also re checked the other QR's: Send funds, Send collectibles, add owner, replace owner, address book.
Looks good to me


Rakugo, set_variable, get_variable, can handle character's variable

By exemple
Rakugo.get_variable("sy.name")

Make things easier for Parser, instead of check character's variable, and next variable, we can do this at same time.
And it shorter func name like you want in https://github.com/rakugoteam/Rakugo/pull/83#issuecomment-1135503905 :)

Make things easier for Parser, instead of check character's variable, and next variable, we can do this at same time.
And it shorter func name like you want in https://github.com/rakugoteam/Rakugo/pull/83#issuecomment-1135503905 :)

Great to hear that :smiley:


Match cadence to template using cadence AST

Match cadence to template using cadence AST
Cadence shouldn't have to be exactly match the cadence in a template for the template to be returned. By comparing ASTs, matching should become more reliable.

Implemented, closing


After 82f5a23 the make procedure fails

Hello,
After the latest commit, the make procedure fails.
Best Regards,
Can
Summary: 24 packages finished [12.3s]
  1 package failed: micro_ros_msgs
  4 packages aborted: builtin_interfaces lifecycle_msgs std_srvs unique_identifier_msgs
  8 packages had stderr output: builtin_interfaces lifecycle_msgs micro_ros_msgs microxrcedds_client rcutils rmw std_srvs unique_identifier_msgs
  33 packages not processed
arm-none-eabi-gcc -mthumb -mcpu=cortex-m4 -mfpu=fpv4-sp-d16 -mfloat-abi=hard -ffunction-sections -fdata-sections -MD -std=c99 -Wall -pedantic -DPART_TM4C123GH6PM -c -g -D DEBUG -O0 -DTARGET_IS_TM4C123_RB1 -DUART_BUFFERED -I~/ti/tivaware_2_2_0_295/examples/boards/ek-tm4c123gxl/ -I~/ti/tivaware_2_2_0_295 -Imicroros/install/include -Isrc -Dgcc -o gcc/microros.o src/microros.c
In file included from src/microros.c:2:
src/./microros_usbcdc_transport.h:8:10: fatal error: rmw_microros/custom_transport.h: No such file or directory
    8 | #include <rmw_microros/custom_transport.h>
      |          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
compilation terminated.
make: *** [/home/can/ti/tivaware_2_2_0_295/makedefs:175: gcc/microros.o] Error 1
can@hurricane:~/ros2_microros/src/micro_ros_tivac_launchpad_app$ make
arm-none-eabi-gcc -mthumb -mcpu=cortex-m4 -mfpu=fpv4-sp-d16 -mfloat-abi=hard -ffunction-sections -fdata-sections -MD -std=c99 -Wall -pedantic -DPART_TM4C123GH6PM -c -g -D DEBUG -O0 -DTARGET_IS_TM4C123_RB1 -DUART_BUFFERED -I~/ti/tivaware_2_2_0_295/examples/boards/ek-tm4c123gxl/ -I~/ti/tivaware_2_2_0_295 -Imicroros/install/include -Isrc -Dgcc -o gcc/microros.o src/microros.c
In file included from src/microros.c:2:
src/./microros_usbcdc_transport.h:8:10: fatal error: rmw_microros/custom_transport.h: No such file or directory
    8 | #include <rmw_microros/custom_transport.h>
      |          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
compilation terminated.
make: *** [/home/can/ti/tivaware_2_2_0_295/makedefs:175: gcc/microros.o] Error 1

Hello @altineller,
Are you working in main branch or in rosrider branch?
In any case, remove the whole microros/install folder and make make clean && make again.

it is at the 'main' branch.
I have tried both removing the folder and repeating instructions at readme.md, but i still get a similar error:
tarting >>> std_srvs
--- stderr: unique_identifier_msgs
CMake Error at /home/can/ros2_microros/src/micro_ros_tivac_launchpad_app/microros/install/micro_ros_src/install/share/rosidl_typesupport_c/cmake/get_used_typesupports.cmake:35 (message):
No 'rosidl_typesupport_cpp' found
Call Stack (most recent call first):
/opt/ros/foxy/share/rosidl_typesupport_cpp/cmake/rosidl_typesupport_cpp-extras.cmake:8 (get_used_typesupports)
/opt/ros/foxy/share/rosidl_typesupport_cpp/cmake/rosidl_typesupport_cppConfig.cmake:41 (include)
/home/can/ros2_microros/src/micro_ros_tivac_launchpad_app/microros/install/micro_ros_src/install/share/rosidl_default_generators/cmake/rosidl_default_generators-extras.cmake:21 (find_package)
/home/can/ros2_microros/src/micro_ros_tivac_launchpad_app/microros/install/micro_ros_src/install/share/rosidl_default_generators/cmake/rosidl_default_generatorsConfig.cmake:41 (include)
CMakeLists.txt:15 (find_package)

Failed   <<< unique_identifier_msgs [0.75s, exited with code 1]
Aborted  <<< builtin_interfaces [0.75s]
Aborted  <<< lifecycle_msgs [0.73s]
Aborted  <<< micro_ros_msgs [0.74s]
Aborted  <<< std_srvs [0.73s]
Summary: 24 packages finished [9.70s]
1 package failed: unique_identifier_msgs
4 packages aborted: builtin_interfaces lifecycle_msgs micro_ros_msgs std_srvs
8 packages had stderr output: builtin_interfaces lifecycle_msgs micro_ros_msgs microxrcedds_client rcutils rmw std_srvs unique_identifier_msgs
33 packages not processed
arm-none-eabi-gcc -mthumb -mcpu=cortex-m4 -mfpu=fpv4-sp-d16 -mfloat-abi=hard -ffunction-sections -fdata-sections -MD -std=c99 -Wall -pedantic -DPART_TM4C123GH6PM -c -g -D DEBUG -O0 -DTARGET_IS_TM4C123_RB1 -DUART_BUFFERED -Itivaware_c_series/examples/boards/ek-tm4c123gxl/ -Itivaware_c_series -Imicroros/install/include -Isrc -Dgcc -o gcc/microros.o src/microros.c
In file included from src/microros.c:2:
src/./microros_usbcdc_transport.h:8:10: fatal error: rmw_microros/custom_transport.h: No such file or directory
8 | #include <rmw_microros/custom_transport.h>
|          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
compilation terminated.
make: *** [tivaware_c_series/makedefs:175: gcc/microros.o] Error 1

Let me take a look

@altineller do you have sourced your ROS 2 installation? In order to build micro-ROS is better to not have it sourced.
Make sure that you are not doing source /opt/ros/foxy/setup.bash and this command is not in your .bashrc

I have tried foxy both sourced and not sourced, and same results. I fired a foxy lxd image and repeated procedure, still fail.
I checked out the rosrider branch and it builds.


Command line app to run an instance of the perun node.

Motivation / Context

Implement a command line app that can be used to run an instance of the perun node.
Description

The app should provide the following commands and corresponding options:

run: Run an instance of the node. It should have following options:
a. --configFile: path to the node config file.
b. --logLevel:  log level for the node and all derived loggers. Default value: node.log.
c.  --logFile: filepath to write logs. Empty string represents stdout. Default value: info.
d. --chainURL: Address of the default blockchain node. Default value: ws://<IP_ADDRESS>:8545.
e. --adjudicator: Address of the default Adjudicator contract.
f. --asset: Address of the default Asset Holder contract.

If config file is provide, it is optional to specify the other parameters. The value of other parameters (if specified), will override the values specified in the config file.
If '--configFileis not specified, it is mandatory to specify at --adjudicatorand--asset` and default values will be used for other parameters.

generate: Generate the demo artifacts for trying out the perun-node.
a. --type: Type of the artifact to be generated. It can have take three values: node, alice and bob. These options are mutual exclusive and more than one of them can be provided as a comma separated list.

node will generate a node configuration file, that be passed to the --configFile option of the run command.
alice and bob will generate the demo artifacts for alice role and bob role respectively. The artifacts include a session config file, keystore and contacts file. These artifacts can be used in a client api instance to connect with a perun node and open sessions for alice and bob respectively.

version: Print the version of perun-node and go-perun sdk used.
help: Print a help message.

Relates to

Testing

A client should be able to connect to the server, once it is started using run command.
The generate command should produce artifacts that resemble the ones used in integration tests.

Thread Safety

Not required, only one of the commands will be invoked by the user at a given point of time.

I wonder whether the --configFile option is specific to the run command or more of a global app option?
Are the options of the generate command mutually exclusive? What about the following interface then: generate --config=type where type is node, alice, or bob?

@choeppler The --configFile option is specific to the run command.
The options of the generate command are mutually exclusive. Thanks for the suggestion, I have updated the description with a more simplified interface for generate command.

@manoranjith, regarding the updated description of the generate command:

if options are "mutually exclusive", I would expect that they cannot be used at the same time...
generate --type=alice and generate --type=bob create three files? Would they be stored in the current working directory using some fixed names?
does generate --type=node write the configfile to the standard output stream or to a file using a default path? I guess the former would be more flexible, also in terms of testability. But in terms of consistency of the interface, it should behave as the generate commands for alice and bob, I think.

Since the files will be created distinct fixed names, it should be safe to make the values for the --type option independent, so that they can be used simultaneously.
Regarding writing to standard output stream or a file, I prefer the latter for consistency of interface (as you have also mentioned).
I have updated the above info and the details of what artifacts will be generated in the issue description.

@choeppler when implementing this, i see a constraint that both alice and bob config should be generated in one go. This is because, when creating the config, we query for two ports that are free and assign to alice and bob.
Since we need to add each other's details to the contacts file, it is need to know the config of one role when creating artifacts for the other role. Also, each of this config might not be useful by themselves, so it is both these can be created with a single option "session".
This will change --type=node,alice,bob to --type=node,session. But such an interface offers of possibility for the user to mispel the values and hence additional condition checks will be required. I think flags --node, --session could be a better interface. If they're misspelled, option not found error will be returned.

Okay, I see. Actually, I now wonder what the benefit of those type-options is. How about always generating all of the required configs if none of them exists, yet, and returning an error message if they are already there? Then there would only be the need for one option, I guess: --force | -f to overwrite configs with the generated defaults.

Actually, if in the CLI grammar there is no object detailing what to generate but only flags, I'd go with generate everything by default. So, how about
perunnode generate                # generates sensible default configs for node and session
perunnode generate --only-session # generates only session configs for alice and bob
perunnode generate --only-node    # generates only node config

I'd probably refrain from overwriting pre-existing configs but error out and either have the user delete them as you suggest, or offer a --force option.

Yeah, i think it can be done this way as well. But one diff, I see is that:

In previous approach, perunnode generate would print the usage message. So by default, the command will never fail. It will only if he specifies --node when there is node.yaml, it will error. This behavior is explicit.
But in this approach, if perunnode generate  fails, it could that any one of the config is there. I find this to be not very explicit. But it has the benefit that, in the first run, all config are generated without any option.

What are your thoughts on the explicitness of the behavior ?

Well, for me it would be surprising if a command just returns a usage message. If the command fails, I'd expect both a non-zero exit code and an error message.
As for usage message, I'd expect that if invoked with the '-h' option or via an explicit help command. I guess you'd get that kind of behavior anyway, if you use cobra to build the CLI.

Okay, in the case we can do the way you suggested in your previous comment, without a force flag (so it is upto the user to delete it). Yes, I am using cobra and it does have the help or -h flag enabled by default.

I am doing this task in three PRs, each comprising one commit with the following changes:

[ ] Move the cmd/perunnode package (that implements node API related functionalities) to node, so that the former package path can be used for the executable package implementing perunnode app.
[ ] Implement perunnode run, vesion and help command.
[ ] Implement perunnode generate command.


Question: Using a local version of Polkadot with xcm-emulator

Hello!!!
i would like to use a local version of Polkadot so i can log custom debug messages in the executor and builder. I tried redirecting all the polkadot pallets to my local repository but when i compile tons of errors and unmet trait bounds appear specially with the xcm-emulator. Is this possible?

Yes, it is possible. The compile error is probably because your local repo doesn't have the same version of upstream Substrate dependencies with the xcm-emulator you used.

Yes, it is possible. The compile error is probably because your local repo doesn't have the same version of upstream Substrate dependencies with the xcm-emulator you used.

Thanks for @shaunxw ! actually i'm using release-v0.9.42 and polkadot-v0.9.42 for all the imports, should i specify the commits as well using rev?

It should work with the same polkadot release branch. I would suggest to double check your cargo lock file to see if there are multiple substrate versions used.


Kubernetes > WeaveNet > use MAC address

When I use Docker Compose I can specify the MAC address of a container. This is needed because some containers require a specific MAC address for e.g. license purposes.
This is currently not possible to do via a Kubernetes Deployment yaml file. There is a discussion thread on this topic at https://github.com/kubernetes/kubernetes/issues/2289. The thread mentions on the end It is not something we support directly. It's possible that some network drivers support this via annotations, but I have not heard about it.
Is this possible for WeaveNet?

No. See #21.

if / when would this be supported?

Not anytime soon unless demand increases; you are the first person to ask for this, ever. That said... PRs welcome :)

@bergtwvd do you need a specific MAC address, or just a consistent one?  I think #2691 would give you the latter, if you specified an IP address (which Weave already lets you do).

In need to have a specific MAC address. In my situation the software license is checked-out against a given MAC address.

Same situation here.
With Docker mac-address option it is possible but k8s or weave its not.

I am also facing the same issue. Software licence is dependent on MAC and that's why looking for static MAC

I need this for a project as well.  Using docker-compose for now

The solution pointed by @colek42 works  but only with one replica and in bare metal environment and weave-net addon
In EKS dosen't works.
Looking for a solution :(


design database

Because PouchDB is document-based database, let me show you the high-level view of our database as an object:
{
  user: {
    id: String<uuid>,
    name?: String
  },
  subscriptions: [
    {
      id: String<uuid>,
      title: String,
      cost: Number,
      created_at: Number<epoch>,
      updated_at: Number<epoch>,
      period?: Number<Period>,
      color?: String<hex>
    }
  ]
}

enum Period {
  1 = "monthly"
  2 = "annual"
}

example:
user
userDB.put({
  _id: generateUuid(),
  name: null
})

subscription
db.put({
  _id: generateUuid(),
  title: 'Spotify',
  cost: 50000,
  created_at: Date.now(),
  updated_at: null, // no type-checking in js!
  period: 0,
  color: '#666' // taken from `data.json`
})

detail
db.get('915f293d-a824-4f6d-801e-22a56a5c354d').then(doc => console.log(doc))

generate uuid
(((a,b)=>{for(b=a='';a++<36;b+=a*51&52?(a^15?8^Math.random()*(a^20?16:4):4).toString(16):'-');return b})())

@evilfactorylabs/nadya how do you think guys? Are there any missing field in current schema? Or are there any mistake here for naming a field?

@evilfactorylabs/nadya what do you think guys? Are there any missing field in current schema? Or are there any mistake here for naming a field?

@evilfactorylabs/nadya what do you think guys? Are there any missing field in current schema? Or are there any mistake here for naming a field?


Marlin FP8

Support fp16 x fp8 compute

I rebased your branch and added a few commits to fix some compilation issues. I also added a unit test, but the results are not so good so I might have done something wrong or we are falling in numerical ranges where float8 conversions are too lossy.
see my branch here: https://github.com/huggingface/optimum-quanto/tree/marlin-fp8.


Switch NuGetPackageVersion to be based on SemVer2 when the nuget gallery is ready

When NuGet/NuGetGallery#3547 is fixed, flip the NuGetPackageVersion property to be set to SemVer2 instead of SemVer1.
Either that or have some easy way to update it using an MSBuild prop. Goal is to make it easy to switch once NuGet supports it.
As suggested by @onovotny.

Given that support for this will be opt-in and require NuGet 4.3, some packages may wish to choose to use SemVer1 versions. The PR makes this it an option to use SemVer1 or SemVer2 as the package version string.


Overlay issue on IE 8 & 9

If wmode opaque can be added to youtube to prevent iframe player overlay on top of fixed elements on IE8 and 9 that would be great,

Can someone verify that this is still an issue.


perf(stats): query optimization

WIP.
Changes:

blockscout migration detection
conditional statement choice (optimized or old) based on applied migrations
test both choices
(WIP) update mock blockscout db to keep up with migrations

blocked by https://github.com/blockscout/blockscout-rs/pull/1041

I guess we'll have to kostyl something here..


MJPEG Camera stream giving 502 bad gateway in browser console and no stream is visible

Home Assistant release with the issue:

0.85.1
Last working Home Assistant release (if known):
N/A
Operating environment (Hass.io/Docker/Windows/etc.):

Raspberry Pi with VirtualEnv
Component/platform:

https://www.home-assistant.io/components/camera.mjpeg/
Description of problem:
When configuring the camera I am able to get a still image (which refreshes every x seconds) in lovelace, however when clicking it I get no image and the console gives a 502 bad gateway error. Going to the link provided by the camera proxy I also get bad gateway.
I already created a reddit post (https://www.reddit.com/r/homeassistant/comments/agvrph/cant_get_mjpeg_ip_camera_working_correctly/ ) since I tought it was an issue on my end; but no luck fixing it there, so I'm guessing there is more going wrong.
My camera is a TP-Link nc210 and I'm able to view video via a range of urls via the browser.
Note that the still image is working and if I don't provide a still image and only the mjpeg url it is also able to fetch an image. So it seems it is just the stream that is not working.
Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):
  - platform: mjpeg
    name: Livingroom Camera
    mjpeg_url: http://<IP_ADDRESS>:8080/stream/video/mjpeg
    still_image_url: http://<IP_ADDRESS>:8080/stream/snapshot.jpg
    username: admin
    password: !secret encoded_camera
    verify_ssl: false
    authentication: basic
  - platform: mjpeg
    name: temp Camera
    mjpeg_url: http://<IP_ADDRESS>:8080/stream/video/mjpeg?resolution=VGA&&Username=admin&&Password=XXXXXXXXXXX

Traceback (if applicable):
GET http://<IP_ADDRESS>:8123/api/camera_proxy_stream/camera.livingroom_camera?token=f41fb799294a40f5a0165aa31444638ea06306504f197bc625007927a495f4b4 502 (Bad Gateway)

Additional information:

Why is this closed? The solution above is just a workaround, not a real solution. Mine stopped working since yesterday. I think it's related to the recent Surveillance Station package update in DSM as I updated it yesterday


Issue: Fullscreen not functioning properly on Linux

Description:
Currently, when the screen size passed into the rendering engine is the same size as the monitor it is displayed on, there are different results on different platforms. On Windows, the window full screens as expected, but on Linux, the screen instead appears windowed and extends past the edges of the display, causing the user to be unable to see parts of the window.
Steps to Reproduce:

Launch the rendering engine on a Linux system.
Create a rendering engine using the monitor's dimensions (can be queried with pg.display.Info()).
Observe that the window never becomes full screen as intended. Shortcuts such as F11 do not work either.

Expected Behavior:
Full screen mode should function consistently across different operating systems. Users on Linux should be able to enter fullscreen mode without encountering any issues, similar to the behavior observed on Windows.
Proposed Solution:
To address this issue, it is suggested to introduce a flag option in the engine class's initializer that allows users to specify their preferred fullscreen behavior. This flag would provide users with the flexibility to enable or disable fullscreen functionality based on their specific requirements.
Additional Information:
Operating System: Ubuntu 23.10 and Fedora Workstation 39
Reproducibility: Consistently reproducible

Hey @krakengamedev thank you for the detailed feedback! I noticed the same behavior too on my Linux machine.
I implemented your solution on a new branch named screen_flags and it seems to work for me. It would be very useful if you could verify that it works for you too. You can follow these steps:

Uninstall pygame-render:

pip3 uninstall pygame-render

Clone the branch:

git clone -b screen_flags https://github.com/MarkelZ/pygame-render.git

Change the directory and install the package locally:

cd pygame-render/
pip3 install -e .

Run example7.py, which implements full screen.

cd examples/
python3 example7.py

You may need to install a package called screeninfo before running example7.py. I use this package to get the screen size to be able to fullscreen properly.
You can also try running the project that you're working on with the fullscreen flag:
engine = RenderEngine(screen_width, screen_height, fullscreen=1, vsync=1)

I noticed that the fullscreened window would flicker unless vsync is enabled, so that's why the additional flag.
If it works for you too, could you also check if it still works on Windows? Also, if you have a Mac, I would appreciate it if you could check that too. I only have Linux so I can't do it myself.
Thanks again :)

@MarkelZ Thank you so much for the solution!
I just ran your example program in Windows and it works as intended. I don't have my Ubuntu machine with me at the moment, but I will let you know when I get a chance to test it.

@krakengamedev Sounds good! I'll wait

I'm getting an unusual error during installation with pip install -e . on Ubuntu 23.10.
ERROR: Project file: path-to-cwd/pygame-render has a 'pyproject.toml' and its build backend is missing the 'build_editable' hook. Since it does not have a 'setup.py' nor a 'setup.cfg', it cannot be installed in editable mode. Consider using a build backend that supports PEP 660.
I didn't get this error on windows, but I'm not sure how to resolve this.

I'm pretty sure this is not related to the last changes I made.
To fix it, you can try to install the package in non-editable mode, by not passing the -e flag:
pip install .

If this doesn't work either, you can add an empty setup.cfg file and try again, since the error you're getting is complaining about a missing setup file.
Let me know how it goes :)

Hey @krakengamedev I tried the changes on different systems and it seems to work. I will soon merge it with the main branch and publish it as version 1.3.0. Thanks again for your feedback!


proposal: maps: add a SafeInsert function for the common case of checking a map for existence before inserting

Proposal Details
When inserting into a map m that may be nil the following kind of snippet is needed:
if m == nil {
    m := make(map[string]string)
}
m[key] = value

It would be nice if the maps package included a generic function to simplify this to one line.
I'd imagine there is a better name for this sort of function, but here is what the implementation could be:
// SafeInsert first creates the map if it is nil, then inserts 'v' into the map at key 'k'.
func SafeInsert[Map ~map[K]V, K comparable, V any](m *Map, k K, v V) {
    if *m == nil {
        *m = Map{}
    }
    (*m)[k] = v
}

This could then be used like:
maps.SafeInsert(&m, key, value)

https://go.dev/play/p/sMRAojtakEu

Related Issues

proposal: Better way of handling nil maps #68805 (closed)

(Emoji vote if this was helpful or unhelpful; more detailed feedback welcome in this discussion.)

Not even discussing if this is a problem that needs a fix.
I don't even think this signature allows to do what you say you want it to do. It "solves" a nil deref panic by adding an other one.
If anything copying append's signature makes more sense to me:
// SafeInsert first creates the map if it is nil, then inserts 'v' into the map at key 'k'.
func SafeInsert[Map ~map[K]V, K comparable, V any](m Map, k K, v V) Map

No, the original suggestion would work.
var m map[string]int // &m will be a non-nil pointer to a nil map.
maps.SafeInsert(&m, "example", 3)

I do think that the append()-style one is better, though.

I get why this would be bundled with insert but I think that would look odd when there are multiple inserts as only the first would be the Safe variety. Either version could just ensure a non-nil map:
func X[Map ~map[K]V, K comparable, V any](m *Map) {
    if *m == nil {
        *m = Map{}
    }
}
// or
func X[Map ~map[K]V, K comparable, V any](m Map) Map {
    if m == nil {
        return Map{}
    }
    return m
}

though with #12854 that line of code is just
if m == nil {
	m = {}
}

and if we also get universal zero-compariability and cmp.Or is extended to any type it could be m = cmp.Or(m, {})

No, the original suggestion would work.

I agree but then you have to make sure you give it a non-nil *Map pointer otherwise it panics which seems weird to me because the goal is to solve this bug.

No, the original suggestion would work.

I agree but then you have to make sure you give it a non-nil *Map pointer otherwise it panics which seems weird to me because the goal is to solve this bug.

I figured that map pointer variables are pretty rare and unusual so callers are unlikely to make the mistake of passing in a nil pointer and will instead use '&myMap' which will never be nil. "rare and unusual" is not "always" though :smile:
The append style does seem better to me since it returns a valid value rather than requiring a non-nil pointer. I'll close this. Thanks for the discussion.


feat: Add highlighting to commit based on signature flag

Modifies the log view buffer to highlight the commit based on the status of the commit signature.
I tried mapping the highlight colors from magit but feel free to adjust them.
Still need to add the signature output on the commit details buffer but that one is a bit more messy as I need to record if --show-signature was passed in the parent BufferWindow and pass it down to the CommitDetailView to be used in git show. I'll work on that in another PR 🤔 .
Solves half of #784

Great stuff! Small change to the highlight groups, but otherwise looks good to me.

@CKolkey Modified to use the graph colors. Thanks for letting me know!

Thanks for the contribution!


[Suggestion] Forget pokemon not to capture

Please check configuration at http://jsonlint.com/ before posting an issue.
This is not exactly an issue but an improvement rather, just want to ask whether this can be implemented.
Expected Behavior
The bot to forget the pokemon that is filtered as not to capture
Actual Behavior
The bot keeps seeing the same pokemon that is filtered as not to capture, seems like time is wasted on this pokemon
Steps to Reproduce
Other Information
OS:
Git Commit: (run 'git log -n 1 --pretty=format:"%H"' and paste it here)
Python Version: (run 'python -V' and paste it here)

Would you mind putting something like [SUGGESTION] in the title so people can quickly sort through recent issues? Thanks 😃

Have you tried adding said pokemon to the catch filter and marking it as "always_catch": false? I believe this is what you are looking for.

@kyleboe Thanks for pointing this out mate, done!

@nelsong by setting always_catch = false, does it mean that I will never catch that particular pokemon please? no matter its cp or lv?
because I actually want to filter it by cp or lv

I need this feature, so I changed the script. pokemon_catch_worker_改.py. Compare my changes and decide if you want to use this code or not.
(file path is PokemonGo-Bot/pokemongo_bot/cell_workers/pokemon_catch_worker.py)
Example:
[ignored]A wild Rattata appeared! [CP 330] [Potential 0.73] [S/A/D 8/15/10]

No duplicated info will be produced.

@g2384
Thanks for sharing your code! I see how you do it, basically you are preventing the message to display again and again.
Do you think it is possible to 'completely' ignore this pokemon? e.g. not calling catchpokemonworker at all?
I understand it may be a bit difficult, as the logic to evalulate whether to catch or not is inside catchpokemonworker, so that means we may have to return a value to its upper level to 'delete' or 'remove' this particular pokemon from the api response, and this is where I am stuck.

@luckyvictor Yes, it's possible to not call catch_pokemon_worker at all (check the should_capture_pokemon at upper level), and the number of API requests can be reduced.
Steps:

check visible pokemon
should capture? Yes -> call catch_pokemon_worker. No -> ignore
if ignore it, remember it's pokemon['latitude'], pokemon['longitude']

@g2384
The should_capture_pokemon funciton is however inside the catch_pokemon_worker, surely we can copy it, but to evaluate the CP and IV of this particular pokemon, it is done in the work in catch_pokemon_worker, that means we have to copy that part of code out and call it before we decide to call catch_pokemon_worker.
Sounds like it is more like merging or basically embed half of the catch_pokemon_worker into other upper level modules, e.g catch_lured_pokemon.py and catch_visible_pokemon.py
Does it sound right to you?

I have slightly changed the position of my code, so that it should save several API requests. checkout the update.
pokemon_catch_worker_改.py (saved lots of API requests, I suppose)
catch_visible_pokemon_改.py
(need to examine other code in order to save more API requests)

@g2384
That is very interesting, and i think u nail it. Using global variable so that two modules can communicate.
I am not so good at coding, just in my early learning stage. May I ask what/why this code here for please? what is this magic number 20 please?
            if len(ignored_poke_arr) > 20:
                ignored_poke_arr.pop(0)

Just in case if the length of ignored_poke_arr becomes larger and larger. 20 is a rough estimation. It assumes that after ignoring 20 unique pokemon, the first pokemon has already disappeared/expired, so remove that one from the list. I didn't use timestamp here because for the lured pokemon, no timestamp is available.
Two modules cannot communicate, the global variables are separate, although they have the same name.
I'm really glad that you are interested in this question and discuss this issue with me. I'm disappointed by the contributors (lack of communication). So I won't post anything here anymore.

Thank you!
So why using pop but not remove please?
and do you think that it could 'save' some time by ignoring the unwanted pokemon and hence increase the efficiency of catching wanted pokemon?

I checked the differences and tested the speed.

pop : Takes Index & returns Value
remove : Takes value, removes first occurrence and returns nothing
delete : Takes index, removes value at that index and returns nothing

Surprisingly, remove is faster than pop. However, I think remove is not the best function, because we don't know what the first element is. It's easier to delete by index. If the code can be redesigned, remove should be considered.
Reducing API calls will increase the efficiency, according to the discussion in another issue (I won't make a reference). The minimum interval between API calls is 5-10 sec if I understand it correctly.

I am glad to have some discuss as well, and you answer questions very promptly.
In fact, I forgot to ask, you say two modules cannot communicate, the global variables are separate, although they have the same name.
I think u mean the global variable are the same variable, right? I mean it has to be the same variable so it can be called in different modules and act as a flag to decide whether to ignore or not.

I suppose Python files are separate, that means global vairable var_A in file1.py and global vairable var_A in file2.py are two variables stored in two cache addresses. file1.py does not know file2.py also has a variable called var_A. Just like the reddit.com user TheMostUniqueName doesn't know if there's a user called TheMostUniqueName in Pokemon Go. But, of course, they can be the same variable (& same person). You need to write the code in another way. Please let me know if I'm wrong. You can test this by yourself.
Using global variable is the laziest way and worst way. It is not recommended in most cases.

@g2384
I hope that you are interested on discussing the code with me, as I just struggle to understand the following:
In the original catch_visible_pokemon.py, I have insert a print statement to help me debug
class CatchVisiblePokemon(BaseTask):
SUPPORTED_TASK_API_VERSION = 1
global visited_ID

def work(self):
    if 'catchable_pokemons' in self.bot.cell and len(self.bot.cell['catchable_pokemons']) > 0:
        # Sort all by distance from current pos- eventually this should
        # build graph & A* it
        **print('Victor: Catchable pokemon')**
        self.bot.cell['catchable_pokemons'].sort(
            key=
            lambda x: distance(self.bot.position[0], self.bot.position[1], x['latitude'], x['longitude'])
        )

and the result on the console is
2016-08-11 08:26:18,991 [MoveToFort] [INFO] [moving_to_fort] Moving towards pokestop One Tree Hill - 0.07km
Victor: Catchable pokemon
2016-08-11 08:26:30,860 [PokemonCatchWorker] [INFO] [pokemon_appeared] A wild Drowzee appeared! [CP 374] [Potential 0.62] [S/A/D 14/2/12]
2016-08-11 08:26:33,012 [MoveToFort] [INFO] [moving_to_fort] Moving towards pokestop One Tree Hill - 0.06km
Victor: Catchable pokemon
2016-08-11 08:26:44,112 [MoveToFort] [INFO] [moving_to_fort] Moving towards pokestop One Tree Hill - 0.05km
Victor: Catchable pokemon
2016-08-11 08:26:54,912 [MoveToFort] [INFO] [moving_to_fort] Moving towards pokestop One Tree Hill - 0.04km
Victor: Catchable pokemon
2016-08-11 08:27:07,453 [PokemonCatchWorker] [INFO] [pokemon_appeared] A wild Drowzee appeared! [CP 374] [Potential 0.62] [S/A/D 14/2/12]
so the question I have is, why this print statement would be called every time after moving_to_fort? I just don't see how the catch_visible_pokemon.work is called.

It is not after moving_to_fort, it is before PokemonCatchWorker.
I don't really understand the code either. But it starts with:

pokecli.py -> main() -> while True: bot.tick()
tick() is defined in pokemongo_bot/__init__.py
in tick(), self.cell = self.get_meta_cell() adds new tasks (catch pokemon and spin forts)
in get_meta_cell(), cells = self.find_close_cells(*location) gets all objects from the map, and those objects are from API.
the first task is initialised by config.json and created in pokecli.py, this line tree = TreeConfigBuilder(bot, config.raw_tasks).build() (and this is why in some issues, developers suggested to move some tasks up in the config.json file.)

For the global variable part, I didn't test it so I don't know the exact answer. Please let me know if you discovered something.

i agree with you, it is called before PokemonCatchWorker, but three times? and only after the last time, it shows 'A Wild Drowzee appeared'
This is where I get lost....

 That's why you see that print() frequently.

but then, from where the catch_visible_pokemon is called?? CatchVisiblePokemon is set in my config, but i just don't see where it calls this function, apparently it is not from Move_to_Map_Pokemon, as I have added a statement there before pokemoncatchworker is called, but this statement never print...

@luckyvictor fixed?

my account is now permanently banned (I guess, because I have been seeing server is busy or offline for 3 days), so I can't do it anymore unless I create a new account

Going to close, happy to reopen if any raises that it still needs to be done


Added Upcoming Rocket Launch Schedule | Web Scrapper

Description
This script scrapes the upcoming Rocket Launch Schedule and saves it into a JSON and CSV File
Fixes #181
Type of change
Choosing one or more options from the following as per the nature of your Pull request.

NOTE: These boxes can be checked using [X]

[ ] Bug fix (non-breaking change which fixes an issue)

[x] New feature (non-breaking change which adds functionality)

[ ] Documentation Update

Checklist:
Please tick all the boxes that are fulfilled by your Pull Request.

[x] I have named my files and folder, according to this project's guidelines.
[x] My code follows the style guidelines of this project.
[x] My Pull Request has a descriptive title. (not a vague title like Update index.md)
[x] I have commented on my code, particularly in hard-to-understand areas.
[x] I have created a helpful and easy to understand README.md, according to the given README_TEMPLATE.
[x] I have included a requirements.txt file (if external libraries are required.)
[x] My changes do not produce any warnings.
[x] I have added a working sample/screenshot of the script.

@chavarera the required changes have been done

@all-contributors please add @Rajdeep-Ray for code


update setuptools version

Description
Fix nightly test issue with setuptools version.
This is the issue that setuptools version >= 71.X doesn't work: https://github.com/pypa/setuptools/issues/4483
Tests
https://60bb71acc6784780b3bbdbae4ffa636f-dot-us-central1.composer.googleusercontent.com/dags/pytorchxla-torchbench-release/grid?dag_run_id=manual__2024-07-23T17%3A01%3A51.682195%2B00%3A00&task_id=torchbench_all-v4-8.provision.setup&tab=logs
Checklist
Before submitting this PR, please make sure (put X in square brackets):

[x] I have performed a self-review of my code.
[x] I have necessary comments in my code, particularly in hard-to-understand areas.
[x] I have run one-shot tests and provided workload links above if applicable.
[x] I have made or will make corresponding changes to the doc if needed.

Could you summarize the reason why we need to select the setuptools version ">=70.0.0,<71.0.0" in the PR description?
Also, do you need to add a TODO to remove the version constraint later?

Could you summarize the reason why we need to select the setuptools version ">=70.0.0,<71.0.0" in the PR description?
Also, do you need to add a TODO to remove the version constraint later?

Added in the description. Personally I have no plan to remove the version constraint in the near term. This requires other packages to update their setup.py accordingly, which may take time.


Dynamic blogs discovery

[x] latest blog list in homepage
[x] blogs page dynamic featured discovery
[x] dynamic blog list view
[x] on-demand detail view og blog

[] add user-auth to narrow viewport as well

[ ]  add user-auth to narrow viewport as well

postponed till refactor and completion of first milestone


Imaging Format Compatibility

Currently, the Topology Radiomics packages only supports nifty (.nii) images via nibabel.
Sometimes, the labs use the .mha or .mhd imaging formats. Could you please add SimpleITK and/or NumPy support in the compute_morphology_features function to expand imaging format compatibility?
SimpleITK example:
volume = sitk.ReadImage("/path/to/volume.mha") # vol type is SimpleITK.SimpleITK.Image
features = rad.compute_morphology_features(volume, config)

NumPy Array from SimpleITK example:
volume = sitk.ReadImage("/path/to/volume.mha")
volume_np = sitk.GetArrayFromImage(volume) # vol_np type is numpy.ndarray
features = rad.compute_morphology_features(volume_np, config)

To pass a NumPy array from SimpleITK into the compute_morphology_features function, I made the following change:
def compute_morphology_features(mri_mask_voxels: BinaryVoxelMask,
                                config: MorphologyConfig = MorphologyConfig()) -> MorphologyFeatures:
    #mask = mri_mask_voxels.mri_voxel_mask # commented this line out because NumPy array does not have this object
    mask = mri_mask_voxels

Tagging @neshdev and @robtoth to incorporate.

Tagging @neshdev and @robtoth to incorporate.

Working this on branch image-formats

@tgd15 - I just looked over the code and realized that we wanted to make the package bare bones. We planned to only support numpy arrays. Loading of other mri images would be done by some external functions and packages. For example, the code you have above can convert to numpy arrays from SimpleITK formats.
However, looking at the code above, it looks like you are using the function incorrectly
sitk_volume = sitk.ReadImage("/path/to/volume.mha")
volume = sitk.GetArrayFromImage(volume)
sanitized_voxels = rad.convert_volume_into_mask(
volume, merge_labels=[])
features_data = rad.compute_morphology_features(sanitized_voxels)

Hello,
i am trying to use this for extracting topology features from a CT scan. but i am getting an error "AttributeError: 'numpy.ndarray' object has no attribute 'mri_voxel_mask'" when i pass numpy array, so i wanted to ask if this is only for MRI scans or can i use this for CT Scans too.


Swernli/merge/feature/decomp

This merges in the payload from feature/decomp, which includes the accumulated work for the targeting feature described in #249. These changes have been reviewed and tested e2e from the feature branch.
Several major changes included here:

Microsoft.Quantum.Qsharp.Core.dll has some functionality split out into Microsoft.Quantum.Qsharp.Foundation.dll, which has it's own package that forms the basis for any target package.
Microsoft.Quantum.Type1.Core.dll and Microsoft.Quantum.Type2.Core.dll are introduced as alternative target packages that make use of the Qsharp.Foundation but utilize a different set of decompositions and intrinsic gates.
Each target package has it's own simulator, where currently the Type1 and Type2 simulators are not directly used anywhere other than in tests.
The implementation of gate decompositions are broken out on a file-by-file basis inside of src/Simulation/TargetDefinitions to allow the rapid addition or recombination of these pieces into new target packages.

The goal is to introduce this new code hierarchy and design now so it can be extended and incorporated into the QIR work, while not changing any requirements or dependencies in a way that would cause breaking changes for developers building on top of the QDK and the existing full state simulator.

To capture minimally what we had discussed today morning:
1.) With this PR the default behavior changes for giving any target implementation precedence to what is defined in Q# to a strictly hierarchical structure where only what is declared as intrinsic in Q# will be provided by the target. We need to have the option to override this default behavior in order to support the current functionalities until we have fully built the alternative. The override needs to be possible both when using a driver to construct a target and also for standalone executables (a project configuration or a command line argument would both be fine, I think). This needs to go into this PR.
2.) Properly splitting out the connecting piece for each instruction set from the target implementation of these instruction should allow to a) autogenerate the connecting piece, b) package the connecting piece with the instruction set and thus eliminate the InternalsVisibleTo used to expose internals to the simulator implementation (might make sense to do with this PR, tbd), and c) ultimately have a single simulator implementation that supports all three target packages (definitively not part of this PR).

1.) With this PR the default behavior changes for giving any target implementation precedence to what is defined in Q# to a strictly hierarchical structure where only what is declared as intrinsic in Q# will be provided by the target. We need to have the option to override this default behavior in order to support the current functionalities until we have fully built the alternative. The override needs to be possible both when using a driver to construct a target and also for standalone executables (a project configuration or a command line argument would both be fine, I think). This needs to go into this PR.

Regarding this, I've made the changes in this commit: https://github.com/microsoft/qsharp-runtime/pull/421/commits/53d204f5495c14ed923ad4601255689eb2f57e1c. As part of that, the old behavior of C# always overriding the Q# (even if there was a Q# implementation) is now the default, and C# only overriding Q# that is body intrinsic is opt-in. QuantumSimulator opts into the new behavior, since I've rearchitected it to depend on this behavior (old behavior will fail tests), but any existing experiments that inherit from either SimulatorBase or QuantumProcessorDispatcher will have no change in behavior. Hopefully that ensures no one is broken by these updates!

3.) We need to decide when we want to require distinct qubits. The current behavior is closer to only require them for operations that are implemented by the target machine. For this PR we should limit it to that, and we can consider how much value there is to being more strict. With that restrictions, it means one of the tests needs to be updated, since it indeed doesn't/shouldn't produce an error. Once the test is updated, it should be possible to remove CheckQubitUniqueness from the Q# API (we haven't actually gotten around to chatting about RotationAngleValidation - I think I need the reminder what it is used for first).

For this one, I went ahead and removed both CheckQubitUniqueness and RotationAngleValidation and updated the tests accordingly. I'm using a preprocessor constant to skip certain tests for the Type1 and Type2 simulators where we know the validation won't work given the decomposition structure. I had to restructure the tests a little big to get to them to run in a timely fashion, as the previous set up would continue using qubits after an exception in a way that could cause the wave function to get rather large at the cost of slow processing and huge amounts of memory. See https://github.com/microsoft/qsharp-runtime/pull/421/commits/53d204f5495c14ed923ad4601255689eb2f57e1c for the details of this update.

/azp run

/azp run

2.) Properly splitting out the connecting piece for each instruction set from the target implementation of these instruction should allow to a) autogenerate the connecting piece, b) package the connecting piece with the instruction set and thus eliminate the InternalsVisibleTo used to expose internals to the simulator implementation (might make sense to do with this PR, tbd), and c) ultimately have a single simulator implementation that supports all three target packages (definitively not part of this PR).

We've talked more about this approach, and have an idea to define this connecting piece as a common interface package that both the target and the simulator packages refer to, providing the indirection that allows the simulator to stop compiling directly against the target package. This is now captured by #432, and looks like it will be more work than will fit into this PR. We can merge this PR without this work, and then continue that refactor directly in main!

Closing this in favor of #442 that puts the contents back into the feature branch. I'm going to collect some more work there first and then reopen a PR to main.


Add Spake2+ Implementation

Problem
CHIP requires Spake2+ as a pairing mechanism.
Summary of Changes
I've added a base class Spake2p which contains the necessary functions to run the Spake2+ protocol. This base class also acts as a PAL to some of the underlying cryptographic functions necessary for Spake2+ (point arithmetic, field arithmetic, etc). This base class, as implemented in CHIPCryptoPAL.cpp, is therefore independent of the underlying cryptographic library.
In addition, I've added Spake2p_P256_SHA256_HKDF_HMAC which represents the Spake2+ configuration to be used in CHIP 1.0. An implementation of the underlying cryptographic functions using OpenSSL is provided.
Finally, tests for the entire protocol as well as the underlying cryptographic functions are provided.
fixes #258

Awesome, thanks @balducci-apple & @pan-apple !

Can we handle the LGTM issues in this PR?

@jelderton @saurabhst @robszewczyk ?

@jelderton @hawk248 @robszewczyk ?

@emargolis, @dougsteed, @roydsouza

Shoot, I hit merge on the wrong thing. @pan-apple can we address @andy31415 ’s comments?

Approving however not submitting yet, please let me know if you would like to do some updates in this PR or iterate over subsequent PRs on my review points.

@andy31415 , I've taken note of your review comments, and we'll address these in a subsequent PR, as we still need to make changes WRT mbedTLS implementation of SPAKE2P.


Starter Quests: Barbarian

This is a draft for a quest for new players using the barbarian class.
Player:

Greetings, barman. I am new in this town and have not yet met many of its inhabitants.
A fresh beer for me barman.

Barman: And greetings to you, barbarian. I am Roux le Maboul, ancient warrior of the clan of the Red Terror. It is a pleasure to meet another of our kind.
Player:

The Red Terror? I have heard of them before but i can't remember where...
The Red terror? I know them! They were the strongest clan in the ancient times.
I got no time for this, goodbye.

Barman:
Ah, we are a fierce and proud clan. We were known for our prowess in battle and our skill in battle. But I digress. You look like warrior in need of job, and I have a task for you.
Player:

What task is that?
No thanks.

Barman:
Recently, a band of thieves have been plaguing the city. They have taken something very dear to me and I need a brave warrior to retrieve it. Will you take on this quest?
Player:

Count on me.
I can't help you at the time.
What is it that these thieves have taken from you?

Barman:

They have stolen my ancient battle axe, passed down to me from my ancestors. It is not just a weapon, but a symbol of my clan and my heritage. I cannot let it fall into the hands of thieves.

Player : I understand. I will retrieve your axe and bring these thieves to justice.
Quest accepted:
Barman: Good. The thieves are known to reside in the outskirts of the city. Be careful, they are cunning and dangerous. And remember, the axe is a sacred symbol of the Red Terror, it must be brought back to me. May the gods be with you on your quest.
Quest denied:
May the god be with you adventurer!

This should be in discussions or in the forum, not on issues. There is no quest system planned for now.


No test files found even though one specified with --file

Prerequisites

[x] Checked that your issue hasn't already been filed by cross-referencing issues with the faq label
[x] Checked next-gen ES issues and syntax problems by using the same environment and/or transpiler configuration without Mocha to ensure it isn't just a feature that actually isn't supported in the environment in question or a bug in your code.
[x] 'Smoke tested' the code to be tested by running it outside the real test suite to get a better sense of whether the problem is in the code under test, your usage of Mocha, or Mocha itself
[x] Ensured that there is no discrepancy between the locally and globally installed versions of Mocha. You can find them with: node node_modules/.bin/mocha --version(Local) and mocha --version(Global). We recommend that you not install Mocha globally.

Description
If mocha doesn't find any test files, it returns a No test files found error. This appears to happen even if --file is specified.
Steps to Reproduce
mocha --file foo.js --exclude '**/*.js'
Per the docs, "Files specified using --file are not affected" by --exclude, so this should work.
Expected behavior: foo.js is loaded
Actual behavior: No test files found error

Reproduces how often: 100%
Versions

The output of mocha --version and node node_modules/.bin/mocha --version: 6.1.4
The output of node --version: v8.15.1
Your operating system

name and version: macOS
architecture (32 or 64-bit): 64bit

Your shell (e.g., bash, zsh, PowerShell, cmd): zsh
Your browser and version (if running browser tests): n/a
Any third-party Mocha-related modules (and their versions): none
Any code transpiler (e.g., TypeScript, CoffeeScript, Babel) being used (and its version): none

Additional Information
Use case for this is to run only a given set of files and override any "default" files loaded via positional arguments. For example, mocha --file test/foo.js --exclude '**/*.js' 'test/**/*.js' would only run test/foo.js`.

I think this is the source of the issue:
https://github.com/mochajs/mocha/blob/f3ed729b6908f3893e6f87722c6e996cca5f7733/lib/cli/run-helpers.js#L142-L168
Note how files from --file is only added after the error is thrown.

I didn't understand the use case. Why should we load --file even if there are no target test files. --file is not for test files. It is for pre-loaded files before testing.

The previous comment that I deleted is caused by my misunderstanding. Sorry.


How to detect redirect

What versions are you running?
0.4.0
What did you do? Include clear steps.
I am trying to add a redirect listener, and when the page redirects to certain URL, I would like to perform certain tasks. But how should I do that?

See how https://godoc.org/github.com/chromedp/chromedp#WaitNewTarget is implemented. The redirect will show up as an event.


Circles are not big enough, own blob runs into circles

The issue issue_one is that drawn circles for cells that are over twice as big are so small that my own cell (call it John) runs into the split zone for these cells. The screenshot screenshot_one demonstrates this.

The other issue that contributes to issue_one is the destination/path_to_destination runs over the larger cells. This should not happen and it should re-locate the destination if it does. The screenshot screenshot_two demonstrates this.

I would love to work (collaborate) with you on these issues, so if you tell me how to fix these, I'll happily contribute to this open-source project.
-jjc12

If I understand issue your one correctly, that is the intended behavior. It's usually safe to assume that really big people will not try to split on your if the size difference is really big.
Issue two is a two part problem. In your screenshot, you can notice that you are outside of the danger zone of the big green cell. This means that to avoid the current danger zones right above, you can get closer to the green cell first before having to handle it too. When you do get closer to the green threat, the bot will start orbiting around it.
I'm not sure if those are problems that need solving.

"It's usually safe to assume that really big people will not try to split on your if the size difference is really big."
After running your bot for 50 games with no modifications, the top score was 66, due to eating one cell and a lot of food.
After modifying the JS code so that all cells were avoided, even the small ones, I was able to run 5 games with a top score of 320.
Consider the long-term implications of your bot. If I somehow miraculously got to a score of 800, maybe after 1,500 or so games, if ever - then got eaten by a cell with a size of 3,000, no one is going to be happy about it.
As for issue_two, you didn't answer my question. Although I am outside of the danger zone, the cell is still heading toward the much bigger cell.

I see what you mean for #1 now. I'll write a better response tomorrow. Never getting into bad situations isn't always possible, but I had a bunch of stuff planned for handing every bad situation. It's just not coded in yet.
For #2, I did answer, it won't actually end up running into the cell. Sure the white line could be made shorter so that it doesn't go past any enemies, but it isn't required as when you get inside the danger zone, the enemy will be handled correctly by the obstacle code.
In previous version, the white line was much shorter, the problem was that in some cases, the bot would end up getting stuck between it's own cells when it was split. Making the line longer was a fine fix at the time.

After looking over the relevant parts of your code, this can be fixed in 30 minutes. Tell me what I have to do in regards to these issues. How do I make the circles larger for these cells? How do I fix the path so it doesn't go over large cells? Any more?

Search for where the function "canSplit" and "compareSize" is used. That's where I make the distinction for sizes.

Thanks, Apos!

When I'm back, I really need to get around to documenting everything. I need to make it easier to make unique bots.


Add in projects to publication listing

Need to mockup some ideas about how to do this. @htadeusiak, this is a more design-oriented task if you want to give it a try...

@jonfroehlich do you still want me to tackle this. I feel like this could be a good first project for Ketaki.

Sure. I’d want to see mocks first.
Sent from my iPhone

On Mar 2, 2019, at 3:12 PM, Hank Tadeusiak<EMAIL_ADDRESS>wrote:
@jonfroehlich do you still want me to tackle this. I feel like this could be a good first project for Ketaki.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

This is now done and unified throughout the website through shared code


Broken in 11.3 (beta)

Looks like Apple caught on and is no longer using sysctl. AssetCacheManagerUtil might need patching at the source.

Is there another workaround available?

Not at this stage. I tried a number of things at the kernel level back in April but nothing that worked.
I think a Lilu plugin to patch the AssetCache binaries directly is the logical next step, but I don't have the time at the moment to examine them and learn Lilu myself unfortunately.


tfsec workflow, tfsec rule updates + ignores for lab requirements

add workflow for https://aquasecurity.github.io/tfsec/v1.28.5/
rules ignored for lab to pass based on https://medium.com/@nanditasahu031/how-to-use-different-tools-to-do-terraform-checks-in-github-actions-b16e9fa73c42
workflow added based on https://aquasecurity.github.io/tfsec/v1.28.5/guides/github-actions/github-action/

ignore test commits, unrelated


Determine a "better solution" for avoiding workers in the Rails console

PR#99 merged a feature to prevent workers from running in the Rails console, but that was then disabled in later commits 0902676 and d191fb3 without much detail.
This bug is to track getting some details on what a "better solution" might look like - aka what was the issue encountered that required the feature being disabled again, and then the fix.
I'm happy to do some more work on this, but need details on what you'd like to see as the behaviour here.

Hi @mattbnz
I have faced multiple issues in the benchmarks and tests that I had to retract it, I am weighing out some options and would like your input:

Make it a global config option, e.g. auto_run: false and require explicit running (e.g. in an initializer)
A specific config option for this use case auto_run_without_a_server: false
Drop the auto running completely and always require an explicit start (either in an initializer or anywhere in the code)

3 seems like the most reasonable to me, given that an initializer can be created with the current generator and the experience remains largely the same without running in console surprises
Thoughts? other options?

Option 3 definitely seems the most attractive of those options, and is in-line with my experiences with other job queues which usually have to be explicitly started.
Sorry for not thinking about the test/benchmark use-cases up front, seems blatantly obvious in hindsight :)
Do you have a plan for how to do option 3 already, or would you welcome a PR with some thoughts?
Thanks.

Happy to accept a PR

should have a proper solution for this in a coming release, but I figured you can use Litejobqueue.jobqueue.stop to stop the background workers in IRB/Rails console until the new solution is released

Thanks, sorry I haven't got to this - it pops up in my inbox every weekend for my TODO list and I've been punting it out another week every time as other things take priority... I look forward to seeing how you've solved it.


How to get the logged in user in the application

How can we help?
I am currently using cognito@edge and attached it to the CDN. How can I retrieve the userId who has logged into the application?

all retrieved information are saved in the cookie. one of them has a value of userId.

Thanks!
From: Junghwa Theodore Park @.>
Date: Saturday, April 15, 2023 at 2:01 PM
To: awslabs/cognito-at-edge @.>
Cc: nagendrakumar02 @.>, Author @.>
Subject: Re: [awslabs/cognito-at-edge] How to get the logged in user in the application (Issue #62)
all retrieved information are saved in the cookie. one of them has a value of userId.
—
Reply to this email directly, view it on GitHubhttps://www.google.com/url?q=https://github.com/awslabs/cognito-at-edge/issues/62%23issuecomment-1509916115&source=gmail-imap&ust=1682186468000000&usg=AOvVaw0PMOS6CSPA0C9Tf1XNdCxZ, or unsubscribehttps://www.google.com/url?q=https://github.com/notifications/unsubscribe-auth/AY4N7ZORVHEQR6FU7T4LNOLXBLO6HANCNFSM6AAAAAAWUUBCNE&source=gmail-imap&ust=1682186468000000&usg=AOvVaw39B_wxa-84rWgUq1We1jRV.
You are receiving this because you authored the thread.Message ID: @.***>

At a high-level, this is the flow of the code in the application at-present.
this._cookieBase = `CognitoIdentityServiceProvider.${params.userPoolAppId}`;

_buildCookieAccessTokenKey = (username: string) => `${this._cookieBase}.${username}.accessToken`;
_buildCookieIDTokenKey = (username: string) => `${this._cookieBase}.${username}.idToken`;
_buildCookieRefreshTokenKey = (username: string) => `${this._cookieBase}.${username}.refreshToken`;
_buildCookieScopeKey = (username: string) => `${this._cookieBase}.${username}.tokenScopesString`;
_buildCookieUsernameKey = () => `${this._cookieBase}.LastAuthUser`;

// Assumed that `cookieHeaders[string] => { key: string, value: string, };`
const cookieUserName = cookieHeaders[this._buildCookieUsernameKey()];
const cookieAccessToken = cookieHeaders[this._buildCookieAccessTokenKey(cookieUserName?.value)];
const cookieIDToken = cookieHeaders[this._buildCookieIDTokenKey(cookieUserName?.value)];
const cookieRefreshToken = cookieHeaders[this._buildCookieRefreshTokenKey(cookieUserName?.value)];

const tokens: Tokens = {
  accessToken: cookieAccessToken.value,
  idToken: cookieIDToken.value,
  refreshToken: cookieRefreshToken.value,
};


setup a webservice that resolves content location

Use case: I'd like know what is at a specific content-based location (e.g., cut:hash://sha256/abc123!/b123-234 ), but I don't want to install preston and get the data.
Steps:

curl https://deeplinker.bio/cut:hash://sha256/abc123!/b123-234
resolver receives request
looks up the associated content
returns only the content at specific location
curl returns with 200 OK and content "my first pony" (or 404 not found, or 400 bad request)

Also consider the content itself.
$ curl "https://deeplinker.bio/locate/http://arctos.database.museum/guid/MSB:Mamm:232346"
returns giant list of:

id
location

http://arctos.database.museum/guid/MSB:Mamm:232346
cut:hash://sha256/abc123!/b123-234

http://arctos.database.museum/guid/MSB:Mamm:232346
cut:hash://sha256/abc456!/b33-55

...
...

and the next step would be, get the eml -
$ curl "zip:hash://sha256/abc123!/eml.xml"
$ curl "tar:gz:hash://sha256/abc123!/eml.xml"

implemented. thanks @mielliott !


Update go to 1.20

Magic mirror failed to merge https://github.com/open-cluster-management-io/multicloud-operators-channel/pull/67 due to different github actions.

/lgtm


Integrate grid placement system plugin

The Grid Placement System from https://assetstore.unity.com/packages/tools/game-toolkits/grid-placement-system-218314 should be integrated for the architecture layout

Will not (yet) be implemented.


Is there a missing .run in the code?

At https://github.com/metaplex-foundation/js-examples/blob/main/getting-started-expressjs/index.cjs#L38 0 shouldn't there be a run()?

Hi 👋
Yes, there should! I've not had the time to update these examples with the latest SDK version yet I'm afraid.
Thanks for raising this.

@lorisleiva Hi! I think this issue has been resolved. run() is there


Correct ServiceAccount name

Use the correct value for the service account name.

@MXfive could you please do a Chart version bump?

Done @pierluigilenoci

@desaintmartin could you please take a look also at this?


KNX throws exception before state has been received

Make sure you are running the latest version of Home Assistant before reporting an issue.
You should only file an issue if you found a bug. Feature and enhancement requests should go in the Feature Requests section of our community forum:
Home Assistant release (hass --version):
0.49.1
Python release (python3 --version):
Python 3.5.3
Component/platform:
knx
Description of problem:
Hass throws exception before any value have been received via the knx bus
Expected:
Default value returned until first value
Traceback (if applicable):
ERROR:homeassistant.helpers.entity:Update for light.spegelbelysning_badrum fails
Jul 24 15:44:29 hass hass[26476]: Traceback (most recent call last):
Jul 24 15:44:29 hass hass[26476]:   File "/home/homeassistant/env/lib/python3.5/site-packages/homeassistant/helpers/entity.py", line 225, in async_update_ha_state
Jul 24 15:44:29 hass hass[26476]:     yield from self.hass.async_add_job(self.update)
Jul 24 15:44:29 hass hass[26476]:   File "/usr/lib/python3.5/asyncio/futures.py", line 380, in __iter__
Jul 24 15:44:29 hass hass[26476]:     yield self  # This tells Task to wait for completion.
Jul 24 15:44:29 hass hass[26476]:   File "/usr/lib/python3.5/asyncio/tasks.py", line 304, in _wakeup
Jul 24 15:44:29 hass hass[26476]:     future.result()
Jul 24 15:44:29 hass hass[26476]:   File "/usr/lib/python3.5/asyncio/futures.py", line 293, in result
Jul 24 15:44:29 hass hass[26476]:     raise self._exception
Jul 24 15:44:29 hass hass[26476]:   File "/usr/lib/python3.5/concurrent/futures/thread.py", line 55, in run
Jul 24 15:44:29 hass hass[26476]:     result = self.fn(*self.args, **self.kwargs)
Jul 24 15:44:29 hass hass[26476]:   File "/home/homeassistant/env/lib/python3.5/site-packages/homeassistant/components/light/knx.py", line 111, in update
Jul 24 15:44:29 hass hass[26476]:     self._state = self.value("state")[0]
Jul 24 15:44:29 hass hass[26476]: TypeError: 'NoneType' object is not subscriptable

Additional info:

I can confirm that I observe similar behaviour.

May you help testing #8725 ?

I will certainly give it a go. Sadly it would require a lot of changes to the config files as I have a lot of switches and sensors. One thing I really wish is the support for setpoint shift and modes for thermostat (dew point/comfort/away/night)

I think this issue may be closed.

Yes
On Sep 22, 2017 11:18, "Julius Mittenzwei"<EMAIL_ADDRESS>wrote:

I think this issue may be closed.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/home-assistant/home-assistant/issues/8642#issuecomment-331396104,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAdeoLxwDI4SxKfJ6aWbe5cqJOpWP-bmks5sk3tGgaJpZM4Oi-k9
.


fix: update UCC UI to v1.12.4

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

v5.14.2
GitHub release

Your semantic-release bot :package::rocket:


New maintainer?

This is a fantastic core library, which is broadly useful for me.  However, there are a few long-standing issues that have not been resolved in the core code (NPM publishing, unbind messing with call order, and others), because @jeromeetienne is too busy to be able to do this.  Is there a process in place for changing maintainers?  Is there a different library that is basically the same that has a creator that is more available for maintenance?

There are simple alternatives, all without external dependencies:

https://github.com/developit/mitt (last update: 2018)
https://github.com/stephenpoole/observer-pattern (last update: 2017)
https://github.com/moll/js-concert (last update: 2017)
https://github.com/mkuklis/asEvented (last update: 2017)

There is also

https://github.com/Olical/EventEmitter
https://github.com/kawanet/event-lite


Add java google default creds to cloud-to-prod interop scenarios

Remaining TODO is to add cloud to prod auth scenarios

****************************************************************

libgrpc.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

****************************************************************

libgrpc++.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

[trickle] No significant performance differences

Objective-C binary sizes
*****************STATIC******************
  New size                      Old size
 1,965,918      Total (=)      1,965,918

 No significant differences in binary sizes

***************FRAMEWORKS****************
  New size                      Old size
10,891,800      Total (>)     10,891,793

 No significant differences in binary sizes

[microbenchmarks] No significant performance differences

****************************************************************

libgrpc.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

****************************************************************

libgrpc++.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

[trickle] No significant performance differences

Objective-C binary sizes
*****************STATIC******************
  New size                      Old size
 1,965,822      Total (=)      1,965,822

 No significant differences in binary sizes

***************FRAMEWORKS****************
  New size                      Old size
10,909,005      Total (>)     10,908,994

 No significant differences in binary sizes

[microbenchmarks] No significant performance differences

****************************************************************

libgrpc.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

****************************************************************

libgrpc++.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

[trickle] No significant performance differences

Objective-C binary sizes
*****************STATIC******************
  New size                      Old size
 1,965,822      Total (=)      1,965,822

 No significant differences in binary sizes

***************FRAMEWORKS****************
  New size                      Old size
10,909,001      Total (<)     10,909,006

 No significant differences in binary sizes

[microbenchmarks] No significant performance differences

known test failures:
macos dbg: https://github.com/grpc/grpc/issues/16846, https://github.com/grpc/grpc/issues/15766
macos opt: https://github.com/grpc/grpc/issues/16846
multi-lang linux: https://github.com/grpc/grpc/issues/16846


chore: disable ref auth e2e test

Problem
Temporarily disable test if needed

Issue number, if available:
Changes

Corresponding docs PR, if applicable:
Validation

Checklist

[ ] If this PR includes a functional change to the runtime behavior of the code, I have added or updated automated test coverage for this change.
[ ] If this PR requires a change to the Project Architecture README, I have included that update in this PR.
[ ] If this PR requires a docs update, I have linked to that docs PR above.
[ ] If this PR modifies E2E tests, makes changes to resource provisioning, or makes SDK calls, I have run the PR checks with the run-e2e label set.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

won't be needing this anymore


feat(arukereso): Add GDS entry for arukereso

Feel free to squash if merging

Upstream PR in https://github.com/googledatastudio/ds-data-registry/pull/116


servicegraph responses 404

Hi, I tried this tutorial, it worked very well except servicegraph which return 404
open http://servicegraph-istio-system.<IP_ADDRESS>.nip.io/ in browser
404 page not found
here is the service log:
2018/02/23 08:01:46 Starting servicegraph service at :8088

Try http://servicegraph-istio-system.<IP_ADDRESS>.nip.io/dotviz


add hvv train lines

Adding HVV S, U and A train lines.
Colours are not constant across sources, so i settled for the wikipedia line icon colours.
I filled shortOperatorName, hafasOperatorCode and hafasLineId to the best of my ability, but the API wouldn't just tell me all of it.
Please do cross check and amend before merging!

Should be fine now, feel free to squash my oopsies. 😅

Github did some weird shit, sorry for that

Absolutely no worries! I'll check it later


Documentation: Merge vehicle modeling and add a vehicle

Hi all,
I would like to suggest to merge 2 pages.
https://carla.readthedocs.io/en/latest/tuto_A_vehicle_modelling/
and
https://carla.readthedocs.io/en/latest/tuto_A_add_vehicle/
Everything in "tuto_A_add_vehicle" can be put under Rigging in "tuto_A_vehicle_modelling".
The whole rigging part can be replaced as the same information is there too.
Maybe split off 2 wheels vehicles?
Kind regards,
Rob

Hi @BAROBA-3d Thank you for your input on this. I have rewritten the tutorial and integrated the modelling into the import tutorial and it's being reviewed. I still need to review the two wheeled vehicle section.


Adjust env conf

What type of PR is this?

What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:

/lgtm

/lgtm


updated scripts for Oracle Database 23.5/aarch64

This pull request introduces a few changes required for Oracle Database Free 23.5/aarch64. The REGULAR and SLIM builds have completed successfully on MacOS Sonoma/arm64 using Rancher but more testing, especially with regards to older (23.{2,3,4} builds and native Linux/aarch64 needs to be performed.
A few changes have been made to the code, mainly to the name of the database RPM which now reflects another build arg, ARCH. The preinstall RPM is now installed as well with a necessary change to the oracle user's memlock setting that prevented /bin/su from working. The required FORTRAN library has been bumped to libgfortran.so.5 to prevent dbca from crashing with [FATAL] ORA-01034: The Oracle instance is not available for use. Start the instance. The install.23.sh script now features a few more extra echo lines to stop the script from executing when it tries to remove files/directories that do not exist.
Finally the Readme has been adjusted.

Thanks a lot for your contribution, @martin-bach-oracle.
Unfortunately, there are more changes required to accommodate the ARM release, and these changes now conflict with yours. Hence, I will close this PR.


Invalid provider for the NgModule 'AppModule' - only instances of Provider and Type are allowed

I'm getting this issue while importing in app.module.ts file

Issue was solved when i put /ngx at the end of the import


Feature request: ability to change decimals from it's default value (18)

It would make it easier to represent values in certain contexts. E.g., fp("1", 8) for the following example.
In this feature, 18 would be the default number of decimals (e.g., fp("1") is the same as fp("1", 18)).

Thanks! But I'll close this since it's a duplicate of #2.