Model resource requirements for onboarding teams to Managed Cloud Services

Describe the Issue:
The Early Adopter Program will onboard a few teams to our emerging Managed Cloud Service - we'll learn what works, what doesn't, what help teams need, who's responsible for what. We need an effective mechanism to capture lessons learned and shape the Service Blueprint - this ticket focuses on the resource requirements aspect.
DOD:

Alpha model of resourcing required to enable each part of the Customer Journey in our Service Blueprint (ready to adjust as we learn more through our Early Adopters Program).
Peer reviews - must include Tech Stream PO, as well as Business Stream.
Gen to approve.

Drafted by Paul, input from Julian, Andy, Stu. Mapped resources proposed to Managed Cloud Services capabilities model (endorsed by ES executive in 2019) and adjust to fit.

Reviewed with Andy and Julian (2020-09-22), and made some minor tune-ups to the Resource Plan to ensure that the roles assigned in the plan map well to the capabilities required for operating a Managed Cloud Service (per capability analysis previously endorsed by Enterprise Services SMT, combined with learnings from the Cloud Pathfinder work so far).


Building debian lstore-release package fails.

Looks like:
https://github.com/accre/lstore-release/blob/master/source/release/deb-release/CMakeLists.txt#L12
doesn't jive with:
https://github.com/accre/lstore-release/blob/master/scripts/generate-docker-base.sh#L79
I'm not sure what happened, this appears like it would never work... @hellermf ?
+ build_lstore_package release /tmp/source/source/release 0.0.0-undefined-tag ubuntu-vivid
+ set -e
+ TO_BUILD=release
+ SOURCE_PATH=/tmp/source/source/release
+ TAG_NAME=0.0.0-undefined-tag
+ DISTRO_NAME=ubuntu-vivid
+ case $DISTRO_NAME in
+ CPACK_ARG='-G DEB'
+ CMAKE_ARG='-DCPACK_GENERATOR=DEB -DCPACK_SOURCE_GENERATOR=DEB'
+ NATIVE_PKG='cp -ra /tmp/source/source/release/ ./ ; pushd release ; dpkg-buildpackage -uc -us ; popd'
+ PKG_TYPE=deb
+ case $TO_BUILD in
+ case $PKG_TYPE in
+ cmake -DCPACK_GENERATOR=DEB -DCPACK_SOURCE_GENERATOR=DEB --debug --verbose -DCMAKE_INSTALL_PREFIX=/ /tmp/source/source/release/deb-release
-- The C compiler identification is GNU 4.9.2
-- The CXX compiler identification is GNU 4.9.2
-- Check for working C compiler: /usr/bin/cc
-- Check for working C compiler: /usr/bin/cc -- works
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Check for working CXX compiler: /usr/bin/c++
-- Check for working CXX compiler: /usr/bin/c++ -- works
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Configuring done
-- Generating done
-- Build files have been written to: /tmp/lstore-release/build/release
+ make package
Run CPack packaging tool...
CPack: Create package using DEB
CPack: Install projects
CPack: - Run preinstall target for: Project
CPack: - Install project: Project
CPack: Create package
CPack: - package: /tmp/lstore-release/build/release/lstore-release-1.0.0-1_all.deb generated.
+ mkdir -p /tmp/source/package/ubuntu-vivid/release/0.0.0-undefined-tag
+ rm '*.source.rpm'
rm: cannot remove '*.source.rpm': No such file or directory
+ true
+ umask 000
+ cp lstore-release-1.0.0-1_all.deb /tmp/source/package/ubuntu-vivid/release/0.0.0-undefined-tag
+ chmod 666 /tmp/source/package/ubuntu-vivid/release/0.0.0-undefined-tag/lstore-release-1.0.0-1_all.deb
++ shopt -s nullglob
++ set +u
++ echo
+ test -n ''
+ set +x
Selecting previously unselected package lstore-release.
dpkg: regarding .../lstore-release-1.0.0-1_all.deb containing lstore-release, pre-dependency problem:
 lstore-release pre-depends on lsb-release
  lsb-release is not installed.

dpkg: error processing archive /tmp/source/package/ubuntu-vivid/release/0.0.0-undefined-tag/lstore-release-1.0.0-1_all.deb (--install):
 pre-dependency problem - not installing lstore-release

fixed with #28


Issue playing Top Tracks

Running version 1.9
Go to Top Tracks
Play a song
I get a “ Something went wrong”
Does not happen with the default skin or ipeng

Not fixed in the latest update
Sent from ProtonMail Mobile
On Wed, Sep 23, 2020 at 10:01 AM, CraigD<EMAIL_ADDRESS>wrote:

Closed #344 via 1e718c5.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

Then you need to provide debug, as it is for me. i have just now confirmed, and I can play a track from 'Top Tracks'

Wierd tried again and it worked :)
Thanks heaps
On Sun, Sep 27, 2020 at 11:05 PM, CraigD<EMAIL_ADDRESS>wrote:

Then you need to provide debug, as it is for me. i have just now confirmed, and I can play a track from 'Top Tracks'
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

The search in Qobuz is not working after upgrading to Material Skin 2.0
On Sun, Sep 27, 2020 at 11:05 PM, CraigD<EMAIL_ADDRESS>wrote:

Then you need to provide debug, as it is for me. i have just now confirmed, and I can play a track from 'Top Tracks'
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

@cellhunt search in Qobuz is not releated to "top tracks" Please dont update closed bugs with irrelevant information - as I amy miss it. Anyhow, search should now be fixed in master - and will be in a future 2.0.1 release.


execute with current input if no available matches

[X] I have read through the manual page (man fzf)
[X] I have the latest version of fzf
[X] I have searched through the existing issues

Info

OS

[X] Linux
[ ] Mac OS X
[ ] Windows
[ ] Etc.

Shell

[X] bash
[ ] zsh
[ ] fish

Problem / Steps to reproduce

printf 'foo\nbar' | fzf --bind 'enter:execute(echo {})'

typing test

press enter

expect test being echoed
actual the execute action isn't executed at all.
Can we have an execute variant that runs no matter if there is a selected candidate?

If the command template contains {q}, the action will trigger even when {} is null.
printf 'foo\nbar' | fzf --bind 'enter:execute(echo {}/{q}; sleep 1)'

Sorry for the necro, but is there a way to automatically exit with 0 status in this situation? I've tried the -0 flag, and also
$ printf 'foo\nbar' | fzf --bind 'enter:execute(echo {}/{q}; exit 0)'

You can use --print-query and process the output using shell script.


fix(compare_map_segmentation): vertical check adjust for low height object detection

Description

This is a temporal change for compare map filter to retain the obstacle pointcloud of low height objects.

Releated Link
TIER IV INTERNAL LINK: ticket 1 and ticket 2
Tests performed

Not applicable.
Effects on system behavior

Not applicable.
Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.

[ ] I've confirmed the contribution guidelines.
[ ] The PR follows the pull request guidelines.

In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.

[ ] The PR follows the pull request guidelines.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[ ] There are no open discussions or they are tracked via tickets.

After all checkboxes are checked, anyone who has write access can merge the PR.

@YoshiRi 先日直接にワイドボードで話したCompare_map_filterの件はこのPRを反映しますのでApproveしていただけますか？


Add option to force disable heartbeats

This adds a DisableHeartbeat bool option in Config that overrides the pick behavior to set the heartbeat interval to 0, even if the server proposes one.
Justification: Our client is running in an AWS Lambda, which can have its execution context "frozen" when the Lambda is idle. This means that if the Lambda is invoked intermittently, there's no guarantee that heartbeats will fire in time. We're also using CloudAMQP, which doesn't provide any control over its server-proposed heartbeat intervals.
As a side note, CloudAMQP recommends disabling heartbeats when using their service, since they implement TCP keep-alive: https://www.cloudamqp.com/docs/celery.html (See the note under "broker_heartbeat")

Thank you for the contribution.
All other clients use the same negotiation logic that's found in pick and CloudAMQP have been OK with it for years. You can set heartbeat timeout to a really high value (say, 30 minutes) which effectively disables them.


EVLink fails to connect, and "serial too long" error

As stated in this thread, I also get several errors: https://github.com/lbbrhzn/ocpp/issues/213
Sometimes, for a brief moment, the connections works, but then the connection to the web socket seems to be closed?
When rebooting both HA and the charger, I get the serial too long error.
Supervision URL on the EVlink: "ws://<IP_ADDRESS>:9000/" (version 1.6, and "OCPP Clock Aligned Data Interval" set to 0)
Home Assistant 2022.12.9
Supervisor 2022.12.1
Operating System 9.4
Frontend-versie: 20221213.1 - latest
Any help is welcome!

For all that are having the same error in the future: you can change the configuration on your charger.
Please go to Maintenance > Report en scroll down until you see "Configuration files".
In that block, you search for "[OCPP]" and probably you'll see this setting:
truncateBootNotificationsSerialNumbers=    false
The serial for the Smart Wallbox consists of 2 parts ("Serial Number Part 1" and "Serial Number Part 2"). When the setting above is "false", the charger will send both numbers combined, resulting in the "serial too long" error.
Now, go to "Configuration". And export "Charging station configuration".
In that textfile you can change the value for "truncateBootNotificationsSerialNumbers" to true. Then save your file and import it again.
Restart your charger and restart Home Assistant.


Add PixelCNN++ improvements to GatedPixelCNN

Some things we could add:

[ ] [easy] Use down/right shifted 3x3 conv instead of a right shifted 1x3 conv in the horizontal stack.
[ ] [medium] Use a U-Net architecture with skip connections (this has shown improvement in some early experiments)
[ ] [medium] [optional] Use gated ResNet layers in both the vertical and horizontal stacks instead of the adhoc GatedPixelCNNLayer.
[ ] [hard] [optional] Implement the discretized logistic mixture for non-binary outputs

Unlikely to ever implement this.


Preparation for release of System.ClientModel 1.1.0

API change check
API changes are not detected in this pull request.


Vulnerability roundup 103: nss-3.53.1: 1 advisory [9.1]

search, files

[ ] CVE-2020-12403 CVSSv3=9.1 (nixos-20.09, nixos-21.05, nixos-unstable)

Scanned versions: nixos-20.09: 86d3781c390; nixos-21.05: 0b8b127125e; nixos-unstable: 1905f5f2e55.

At a quick glance, this seems to be fixed in 3.53 branch upstream but there's no tag/release :-(  Main packages using 3.53 are the current firefox-esr and thunderbird (both synced on nixos-unstable and nixos-21.05).

But I suppose both of these 78-based versions will get abandoned by us soon in favor of new 91-based versions (FF ESR released within two weeks, and thunderbird probably as well at some point).

thunderbird-78 is still around, so let's wait

Now the only remaining part is xmlsec in NixPkgs 21.05:

nss_3_53 is used instead of the latest due to a number of issues:

21.05 got out of support at the end of 2021.


Carousel touch movement is jerk ridden

@akshayaurora Can you confirm is this is still a problem?
Is there some minimal test code we can run to see the problem?


V5

Hi @nahankid, sorry for the late reply.
Javascript sources must be changed in https://github.com/zurb/foundation-sites, not this repo. Also, the change seems to be out-of-date with the latest Foundation versions.


can't run file without prefix

If I want to execute a file in the current directory it won't work without prefix (ie $ opal-node test.rb) will result in something like
  LoadError: cannot load such file -- test.rb

while it will run fine if I set a relative path prefix (ie $ opal-node ./test.rb)

Node support is now backed right into opal: opal -rnodejs -e 'puts "hello node!"'


Open web socket to continue sending message, Connecting App running on simulator to local server running on macbook

when ever i am trying to connect to my local running server using simulator i am not able to connect.
for connection i am using "ws://MyLocalIP:4000:/websocket"  and also "ws://<IP_ADDRESS>:4000:/websocket" url. can we connect simulator to local running server? please give me some solution.
when i am using direct server url it is working but there is one issue is that some times it connecting and some times it getting crash by saying this reason: 'Attempting to send message without an open connection'
how to handle this crash?  because meteor-ios framework always checking for web socket is open or not to send message and if it found that web socket is not open then it is getting crash while sending message in this method of this line.
inside - (void)sendMessage:(NSDictionary *)message { getting crash at this line NSAssert(self.open, @"Attempting to send message without an open connection")
How to handle this crash ?
Please give some solution as soon as possible. waiting for your valuable response.
Thanks

I'm not sure if this is a type in this message or your code, but ws://MyLocalIP:4000:/websocket is not a valid URL. You'll have to remove the colon after the port number: ws://MyLocalIP:4000/websocket.

Here  MyLocalIP is  my current IP ws://MyLocalIP:4000:/websocket
and my local server is running and message showing in terminal is App running at: http://localhost:4000/
but i am not able to connect to local server from my simulator. can we connect simulator to local server.
**And also major problem is **
when it is connecting to remote server there is one issue is that some times it connecting and some times it getting crash by saying this reason: 'Attempting to send message without an open connection'
how to handle this crash? because meteor-ios framework always checking for web socket is open or not to send message and if it found that web socket is not open then it is getting crash while sending message .
inside - (void)sendMessage:(NSDictionary *)message { getting crash at this line NSAssert(self.open, @"Attempting to send message without an open connection")
How to handle this crash ?
Thank you for your response :+1:  can you please give me some solution  How to handle this crash ?

I understand MyLocalIP is your current IP, but you've again included an extra colon in the URL. Is that what you're also doing in code? Because that would explain your issue.
The web socket disconnecting may have to do with an issue with compression that was fixed in 0.2.1. Are you using the latest version?

Sorry it is typing mistake during posting you- url is  "ws://MyLocalIP:4000/websocket" and it's not working
just now i update my pod it is showing  Using Meteor (0.2)  but as you told latest version is 0.2.1 why?

Please find the screen shot and message of crash while sending message to web socket.
i want to conform one thing that can we connect local server running on my mac to app running in simulator. If yes then why i am not able to connect plz suggest me.

*** Terminating app due to uncaught exception 'NSInternalInconsistencyException', reason: 'Attempting to send message without an open connection'

Thanks ...By adding new version " 0.2.1",  I am able to connect my simulator with local server which is running on my macbook .:+1:
But Still some times i am getting crash by  saying this message
Assertion failure in -[METDDPConnection sendMessage:]
*** Terminating app due to uncaught exception 'NSInternalInconsistencyException', reason: 'Attempting to send message without an open connection'
you can find image in above comment. can you please suggest how can i handle this crash.
This crash is happening even if app is running and connection is established but while sending message to web socket if socket is unfortunately not open or got close then app is getting crash.
please provide some solution to handle this crash.


How to get point clouds from all lidars for a frame in v2 devkit

Hi,
I followed the v2 tutorial and was able to get point clouds for individual lidar sensors as follows:
`def generate_trainval_dataset(path_dataset_root, frame_step, max_frames_save):
"""
Function generates BEV frames from base Waymo dataset
"""
components = {"lidar": "lidar",
"lidar_calibs": "lidar_calibration",
"lidar_labels": "lidar_box",
"lidar_pose": "lidar_pose",
"lidar_seg": "lidar_segmentation",
"ego_pose": "vehicle_pose",
}
path_pcl = path_dataset_root + "/" + components["lidar"]
path_labels = path_dataset_root + "/" + components["lidar_labels"]
path_sem = path_dataset_root + "/" + components["lidar_seg"]
path_calibs = path_dataset_root + "/" + components["lidar_labels"]
path_ego_pose = path_dataset_root + "/" + components["ego_pose"]
frames_pcl = os.listdir(path_pcl)
n_frames_pcl = len(frames_pcl) - 1  # there's a metadata file, don't know yet if this can be deleted
frames_labels = os.listdir(path_labels)
n_frames_labels = len(frames_labels)
frames_sem = os.listdir(path_sem)
n_frames_sem = len(frames_sem)
for index in range(0, n_frames_pcl, frame_step):
context_name = frames_pcl[index]
pcl_path = path_pcl + "/" + frames_pcl[index]
labels_path = path_labels + "/" + frames_labels[index]

lidar_df = read(dataset_train_dir_root, components["lidar"], context_name)
lidar_labels_df = read(dataset_train_dir_root, components["lidar_labels"], context_name)
lidar_calibs_df = read(dataset_train_dir_root, components["lidar_calibs"], context_name)

df_laser_calibs = v2.merge(lidar_df, lidar_calibs_df)
df = v2.merge(df_laser_calibs, lidar_labels_df)

for index, data_row in iter(df.iterrows()):
#   print(data_row)

  lidar = v2.LiDARComponent.from_dict(data_row)
  print(lidar.key.laser_name)
  lidar_calib = v2.LiDARCalibrationComponent.from_dict(data_row)

  pcl = v2.convert_range_image_to_point_cloud(range_image=lidar.range_image_return1, calibration=lidar_calib)
  #viz_utils.draw_lidar(pcl)`

However, the above only returns all TOP lidars at first, then FRONT lidar and so on.

I wish to create a single point cloud out of all 5 lidars for each frame (and collect objects for the frame).
What is the most efficient way to do that?
I have alsso tried the tfrecords format and I can do this there easily. However, tfrecord loading is extremely slow and impractical.
Please help

Hi @SM1991CODES ,
For each run segment / context, you may execute dask.dataframe.DataFrame.compute to actually read the data of your dataframe first (or a sub-dataframe by key matching if you just want a part of the context, e.g. one timestamp). The key.laser_name and key.frame_timestamp_micro columns can help you select the rows you want to concatenate together.
Be careful that the laser names of your lidar and lidar calibration must match before calling convert_range_image_to_point_cloud. See https://github.com/waymo-research/waymo-open-dataset/issues/656


Run deploy only after successull check

Description
This PR updates GH Actions workflow so it runs CfloudFlare Page deploy only after CI check compelted successfully.

Will this work, isnt cloudflare deploy relying on some push event details like ref?

@mpetrunic Ideally it would be nice to disable CloudFlare autodeploy on the ClpudFlare side completly (you can check how Gaming Docs are working). Bc right now we are sora have two deploys. One initiated by watchers of the repo on CloudFlare side and another one initiated by GitHub Actions.
Is we are proactively doing deploy from here you do not need git ref or anything. The buld is actually happning inside GH Action.
The downside - there is no nice comment with deployerd URL inside PR itslef. The plus side - you still can see this data in the comment on GitHub workflow page like this:
https://github.com/ChainSafe/game-docs-v2/actions/runs/4234551979

@mpetrunic Ideally it would be nice to disable CloudFlare autodeploy on the ClpudFlare side completly (you can check how Gaming Docs are working). Bc right now we are sora have two deploys. One initiated by watchers of the repo on CloudFlare side and another one initiated by GitHub Actions.
If we are proactively doing deploy from here you do not need git ref or anything. The buld is actually happening inside GH Action.
The downside - there is no nice comment with deployerd URL inside PR itslef. The plus side - you still can see this data in the comment on GitHub workflow page like this: https://github.com/ChainSafe/game-docs-v2/actions/runs/4234551979

Another downside is that Cloudflare keeps the URL the same if you push from the same ref. We could figure out a way to hack but maybe it's easier to add to the ci script as the last task?

But also cloudflare deploy didn't even run here or created a deployment here

Yep, that is becasue workflow_run is not triggering until it's on the deafult branch. https://docs.github.com/en/actions/using-workflows/events-that-trigger-workflows#workflow_run
There second Note about it. But after the merge it works as expected. The Gaming Docs repo working in this exact way.

The only thing that I generally wanted to achieve here is to make sure build is happening on the GH Actions side and we will not have the same problem as with Render when sometimes their watchers was broken or sometimes did not even report the problem back.
But I'm totally open for any other solutions to this.

Oh, yes, agree. Let me add docs also to this PR so it is more or less a single package. And if we decide to roll it back we will remove the related docs too.

Ah, I just realised that this one is closed. Will create another one.

Ah, I just realised that this one is closed. Will create another one.

I already opened one https://github.com/ChainSafe/engineering-handbook/pull/92 But feel free to add reasoning regarding workflow trigger there!

@kalambet Maybe take a look at https://github.com/ChainSafe/engineering-handbook/pull/93


net: zperf: Fix net_socket_service blocked traffic

If run the TCP RX traffic for both WiFi STA interface from external AP's backend PC, and uAP interface from external STA at the same time, then stop the uAP network, the TCP traffic will decrease to zero for STA interface, and some net packets are not freed in socket layer. The root cause is the net_socket_service thread has two places to suspend itself, one is socket_service_thread-> zsock_poll, another one is tcp_svc_handler -> tcp_recv_data-> zsock_recv, and when the uAP network stopped, this thread will wait forever in zsock_recv as the uAP ctx->recv_q is empty, which will also block the STA traffic and cache the net packets. For tcp_recv_data or udp_recv_data, when call the zsock_recvfrom, should use flag ZSOCK_MSG_DONTWAIT to avoid thread pending here.

@MaochenWang1 what kind of device is this, do you have two network interfaces in the system?

Hi @jukkar , it's a Wi-Fi device, have two network interfaces, including station mode, and micro-AP (uAP) mode.

Ok, so basically it is not easily reproducible without this wifi setup.

Hi @rlubos
This case has two network interfaces running TCP RX, one is wifi station interface, another one is wifi micro-AP interface. If we disable the micro-AP, the traffic of micro-AP is stopped, but the socket is not closed. As the station interface is alive, so socket_service_thread->zsock_poll->z_impl_zsock_poll->zsock_poll_internal->k_poll still can get event, and it will call z_fdtable_call_ioctl(vtable, ctx, ZFD_IOCTL_POLL_UPDATE, pfd, &pev) and update pfd->revents as ZSOCK_POLLIN for both station and micro-AP, although the micro-AP's ctx->recv_q is empty.

Hi @jukkar
yes, I am not sure if other device has two interfaces, but station mode and uap mode are common for wifi device.

Hi @jukkar @rlubos please see my reply and help review the commit, thanks

As @rlubos said, it looks like your change might hide a bug in network stack. I think we need to figure if that is the case before accepting this PR.

Hi @jukkar @rlubos the socket_service_thread (handles zperf server) has two places to pend itself but with same reason (ctx->recv_q is empty), I think it's not reasonable, right? so this commit remove one redundant place to pend, and only modify the APP layer

@MaochenWang1 the poll in socket_service thread should verify that there is data to read from the socket, so the fix in this PR should not be needed. Is there a way to replicate this issue (without the hardware you are using)?

Hi @jukkar, yes, the poll in socket_service thread verifies that there is data to read from the socket, but if there are two network interfaces, either one of the two interfaces has data, then the check will pass and can continue to run (you can check k_poll->z_impl_k_poll->register_events->is_condition_met), so even if one network interface is up, and another is down, the k_poll will not be blocked.  Howerver, in tcp_recv_data->zsock_recv, either one of the two interfaces don't have data, then it wll be blocked here, i.e., if one network interface is up, and another is down, tcp_recv_data->zsock_recv will be blocked. can you get my point?

@jukkar , to reproduce the issue, need device support two network interface with different ip address, then run the Zperf TCP RX for both interface at same time.

I'm sorry but I'm totally confused now. The socket layer has no notion of the network interfaces, it doesn't matter which interface reports the data, in the end the packet ends up on a struct k_fifo recv_q struct associated with the socket, which is monitored by both zsock_poll() and zsock_recv() functions.
Now the question is, if the zsock_poll() reported that recv_q associated with a socket has some packet (i. e. is not empty), as configured here, where would that packet disappear when zsock_recv() is called? Again, unless there is some bug in the socket layer, I could only imagine that some other thread called zsock_recv() on the very same socket that zperf uses.

@MaochenWang1 you seem to have a working setup and an easy way to replicate the issue. Would you be able to investigate this more deeply, your original fix proposal seems to solve your issue but we believe it is hiding the actual issue. The zperf in theory supports multiple sessions at the same time, but it is not tested much so the issue could be related to that.

Hi @rlubos @jukkar , as I explained above, packet ends up on a struct k_fifo recv_q struct associated with the socket, which is monitored by both zsock_poll() and zsock_recv() functions, but zsock_poll() checks one of the recv_q is not empty then it will not block, zsock_recv() checks one of the recv_q is empty then it will block, can you get the difference? Let me show the zsock_poll()->zsock_poll_internal()->k_poll()->z_impl_k_poll() here in picture:

that's right, but zsock_poll() checks one of the recv_q is not empty then it will not block, zsock_recv() checks one of the recv_q is empty then it will block, can you get the difference?

I think I understand what are you referring to here, yes that's true that zsock_poll() monitors multiple sockets and can unblock on a different one, while zsock_recv() only monitors a single fifo associated with the socket.
However, it still doesn't explain the behavior you're seeing, not that socket service layer does not call the registered callback for each registered socket, it wil only call the respective callback for the socket if there is an active event on the socket:
https://github.com/zephyrproject-rtos/zephyr/blob/main/subsys/net/lib/sockets/sockets_service.c#L265
So it still doesn't explain why would the zsock_recv() in tcp_svc_handler() block. That's why we're pushing to investigate this further, as there might be some other underlying issue we're not seeing here.


Error when using setup.py

After cloning the repo and installing cmake in windows 11. When I run setup.py I am faced with this error printed in the terminal
An error occurred while running 'make': Command '['cd ./build && cmake .. && make']' returned non-zero exit status 1.
Output:

I edited the file to not use try: and was faced with this error
Command '['cd ./build && cmake .. && make']' returned non-zero exit status 1.
  File "C:\Users\SumYin\Desktop\fastLLaMa\setup.py", line 23, in run_make
    result = subprocess.run(["cd ./build && cmake .. && make"], shell=True, check=True, capture_output=True, text=True)
  File "C:\Users\SumYin\Desktop\fastLLaMa\setup.py", line 153, in main
    run_make()
  File "C:\Users\SumYin\Desktop\fastLLaMa\setup.py", line 156, in <module>
    main()
subprocess.CalledProcessError: Command '['cd ./build && cmake .. && make']' returned non-zero exit status 1.

I am not sure why. I have tried reinstalling Cmake and nothing worked

Hi, can you cd inside "build" and run
"cmake .." and then "make"
Please share the logs and we can help debug!

Hey! I recloned the repo and there is no build folder. I ran setup.py which made the build folder. Here is everything that happened in the terminal
PS C:\Users\SumYin\Desktop\llama\fastLLaMa> cd build
cd : Cannot find path 'C:\Users\SumYin\Desktop\llama\fastLLaMa\build' because it does not exist.
At line:1 char:1
+ cd build
+ ~~~~~~~~
    + CategoryInfo          : ObjectNotFound: (C:\Users\SumYin...fastLLaMa\build:String) [Set-Location], ItemNotFoundException
    + FullyQualifiedErrorId : PathNotFound,Microsoft.PowerShell.Commands.SetLocationCommand

PS C:\Users\SumYin\Desktop\llama\fastLLaMa> python setup.py
An error occurred while running 'make': Command '['cd ./build && cmake .. && make']' returned non-zero exit status 1.
Output:
PS C:\Users\SumYin\Desktop\llama\fastLLaMa> cd build
PS C:\Users\SumYin\Desktop\llama\fastLLaMa\build> cmake ..
-- Building for: NMake Makefiles
CMake Error at CMakeLists.txt:2 (project):
  Running

   'nmake' '-?'

  failed with:

   The system cannot find the file specified

CMake Error: CMAKE_C_COMPILER not set, after EnableLanguage
CMake Error: CMAKE_CXX_COMPILER not set, after EnableLanguage
-- Configuring incomplete, errors occurred!
See also "C:/Users/SumYin/Desktop/llama/fastLLaMa/build/CMakeFiles/CMakeOutput.log".
PS C:\Users\SumYin\Desktop\llama\fastLLaMa\build>

I am not sure what you mean by logs, are they stored some where?

Check if you have Visual Studio with "Desktop development with C++" workload if not you can download https://visualstudio.microsoft.com/downloads
Can you try using Developer Command Prompt.
Check if windows environment variables for C/C++ is set properly.

I am yet to test this out on windows. If you get it working else please share the errors

I'm having the same error. I'm sure 1 and 3 are not issues as I have alpaca and built that with cmake, tried 2 no dice.
When I load it in VisualStudio and try and built with cmake through there I get:
Severity	Code	Description	Project	File	Line	Suppression State
Warning	D9002	ignoring unknown option '-O3'	C:\AI\fastLLaMa\out\build\x64-Debug\fastLLaMa	C:\AI\fastLLaMa\out\build\x64-Debug\cl	1
Error	C7555	use of designated initializers requires at least '/std:c++20'	C:\AI\fastLLaMa\out\build\x64-Debug\fastLLaMa	C:\AI\fastLLaMa\bridge.cpp	237
Error	C7555	use of designated initializers requires at least '/std:c++20'	C:\AI\fastLLaMa\out\build\x64-Debug\fastLLaMa	C:\AI\fastLLaMa\bridge.cpp	574
Error	C7555	use of designated initializers requires at least '/std:c++20'	C:\AI\fastLLaMa\out\build\x64-Debug\fastLLaMa	C:\AI\fastLLaMa\bridge.cpp	579
Error	C2039	'accumulate': is not a member of 'std'	C:\AI\fastLLaMa\out\build\x64-Debug\fastLLaMa	C:\AI\fastLLaMa\bridge.cpp	967
Error	C3861	'accumulate': identifier not found	C:\AI\fastLLaMa\out\build\x64-Debug\fastLLaMa	C:\AI\fastLLaMa\bridge.cpp	967
Error	C2737	'stop_token_len': const object must be initialized	C:\AI\fastLLaMa\out\build\x64-Debug\fastLLaMa	C:\AI\fastLLaMa\bridge.cpp	967
Error	C3536	'stop_token_len': cannot be used before it is initialized	C:\AI\fastLLaMa\out\build\x64-Debug\fastLLaMa	C:\AI\fastLLaMa\bridge.cpp	971

Can you build using the refactor branch? The errors you're getting indeed c++ errors not msvc.


[SPARK-25003][PYSPARK][BRANCH-2.3] Use SessionExtensions in Pyspark

(Branch-2.3)
What changes were proposed in this pull request?
Previously Pyspark used the private constructor for SparkSession when
building that object. This resulted in a SparkSession without checking
the sql.extensions parameter for additional session extensions. To fix
this we instead use the Session.builder() path as SparkR uses, this
loads the extensions and allows their use in PySpark.
How was this patch tested?
This was manually tested by passing a class to spark.sql.extensions and making sure it's included strategies appeared in the spark._jsparkSession.sessionState.planner.strategies list. We could add a automatic test but i'm not very familiar with the Pyspark Testing framework. But I would be glad to implement that if requested.

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Test build #94146 has started for PR 21989 at commit c82bdb8.

Test build #94146 has finished for PR 21989 at commit c82bdb8.

This patch fails Python style tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/94146/
Test FAILed.

Test build #94152 has started for PR 21989 at commit 0951637.

Test build #94152 has finished for PR 21989 at commit 0951637.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

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/SparkPullRequestBuilder/94152/
Test PASSed.

Is this same as #21990? Would it be possible to close this?

@kiszk sure, it all depends which branch the merge target should be I wasn't sure which one was being used for changes of this nature. Technically it's a bug fix I believe.


Switching rolled up parents resulting in duplicates on prior parent

I've successfully duplicated this issue with a couple of different object scenarios, but not completely thoroughly.
I have a rollup from child to parent, all managed via flows calling CMDT, when a child is reparented, the correct value for the prior-parent is doubled - which points to regression, but I cannot see the triggering flow firing twice in debug. Flow setup:

with a simple CMDT record:

Manually recalculating the prior parent resolve the miscalculation
apex-07LTu000002YI3pMAG.log
apex-07LTu000002YJK3MAO.log
apex-07LTu000002YLNUMA4.log
apex-07LTu000002YLNVMA4.log
apex-07LTu000002YNNdMAO.log
apex-07LTu000002YNNeMAO.log
apex-07LTu000002YNPFMA4.log
apex-07LTu000002YNQrMAO.log
apex-07LTu000002YNSTMA4.log

@thomasadchapman I will take a look - thanks for submitting! I'll let you know if I have any questions

@thomasadchapman I'm noticing that your rollup version is several dozen behind the latest: you're running #460, and (notably), I can tell you that I added a test for this specific issue in #493 (ten versions past where you're at, but still far behind the present version). I'd recommend upgrading - it should fix your issue!

@jamessimone Well that's embarrassing! Problem resolved...

Hey now, no worries! Glad to hear we're good - thanks again for submitting!


Gradle issue

There is an issue while including the gradle into project.It giving the error as not reolved

Sorry, I don't quite understand what the problem is.
Do you have a stacktrace?

I assume you found a solution to your problem?
If not, feel free to reopen this ticket and add additional information.


confirm CSVR import makes sense

content to structure value ratios in HEC-FDA Version 1.4.3 are not actually ratios, that would make too much sense! they are in terms of percent. we need to make sure that the import of 1.4.3 occupancy types correctly converts to the 2.0 occupancy types and that we have documented clearly for the user that the tab delimited import file should reflect a 1.4.3 occupancy type.

we're using percent instead of ratios too


update links after the transition from fangohr to `oscovida

To remove the leftovers from the previous repo I just run a one-liner:
grep "fangohr/coronavirus" ./* -rl | xargs sed -i 's|fangohr/coronavirus-2020|oscovida/oscovida|g'

And because that was an automated action, I might accidentally have substitute something that I shouldn't.

Looks good to me.


Added Python

Dark Python.org

Oops, I will have a look at script error.

I've fixed the error message https://travis-ci.org/alexanderby/darkreader/builds/353711370?utm_source=github_status&utm_medium=notification
It says it cannot parse your JSON.


feat: Will dart_frog support graphql?

Hi, Dart Frog wont support GraphQL out of the box itself but any GraphQL implementation that is made for Shelf should work with Dart Frog as it uses Shelf internally.
But if we look at it from the context of Dart Frog it wont be supported. If you have any other questions, feel free to open another issue!

any GraphQL implementation that is made for Shelf should work

are there any projects you know of and can recommend?


Quick fix for: Initial discovery of Gateways works fine, but when starting the integration with all gateways configured it can't find any gateway

This closes #26
Hi @Iminet72, this is the "quick fix" to make at least the configuration of the discovered gateway on startup using the same source address like the initial discovery.
It works for me, but the fix here is not working well if a user has multiple interfaces enabled in HASS network configuration. Basically for every enabled network interface (enabled in HASS config), the code needs to send a separate discovery or update request to the corresponding broadcast address of the network interface.
For this both code parts need to be a loop:

discovery needs to collect all gateways (e.g., possible those in the guest/DMZ LAN and others in the private network. Of course they should not look into WAN interfaces. All found gateways must be added to the final list
when configuring a specific interface, the broadcast must be sent to all interfaces and the method must return true, if at least one gateway with the given ID responds about the reconfiguration or has returned its config.

I will check tomorrow and possibly open another PR to fix the discovery "correctly" - like it is done in HASS core code (for discovery of several device types like shelly components, yeelight). They use always the same pattern, e.g. here for shelly devices:
https://github.com/home-assistant/core/blob/94534f714cf4f2e5cecc6c1d569609325a1d08da/homeassistant/components/shelly/utils.py#L225-L241
So basically the code must look like this - for discovery and also for configuring a single gateway (as we do not know where is it connected at and communication to gateway only works via broadcast).

if I have time in the evening, I will add it to mine.
And I will try it


Unable to generate Pull request : 403 error

Changed the progress bar status but unable to generate the pull request.
@kroy96

Could u please attach a Screenshot of the same.

It's fixed now. I have made the pull request.
Done


Add timeout parameter

Add a timeout parameter that specifies the number of seconds that a gadget will run
Add some logic to handle it in filetop

1. We should really add a guide under `/docs` which would describe the `CommonFlags` (it can be done later, for example once #500 is merged).

I agree!

2. Can you please add this:

--timeout int            Number of seconds that the gadget will run for

to output of kubectl gadget --help in README.md?

I'm not sure where do you want to add it. We don't have any flag in README.md.

3. We should also add the `filetop` particularly to `tcptop` and `biotop` (can be done later).

What do you mean? Add the help output of filetop to a readme?

1. We should really add a guide under `/docs` which would describe the `CommonFlags` (it can be done later, for example once #500 is merged).

I agree!

2. Can you please add this:

--timeout int            Number of seconds that the gadget will run for

to output of kubectl gadget --help in README.md?

I'm not sure where do you want to add it. We don't have any flag in README.md.

Oh sorry, I thought we have a description of them in the README, then nevermind.

3. We should also add the `filetop` particularly to `tcptop` and `biotop` (can be done later).

What do you mean? Add the help output of filetop to a readme?

No, to port the work you did on filetop to tcptop and biotop.

No, to port the work you did on filetop to tcptop and biotop.

Right. I updated this PR to handle it in tcptop. Since https://github.com/kinvolk/inspektor-gadget/pull/551 is not merged yet, I didn't touch biotop at all.

Right. I updated this PR to handle it in tcptop. Since https://github.com/kinvolk/inspektor-gadget/pull/551 is not merged yet, I didn't touch biotop at all.

Thank you for tcptop but I did not mean you had to do it (I could have handle it in a separate PR).
I tested it and it works fine.
Regarding biotop, I will either address this in another PR, or rebase once yours is merged.


[9.x] Fix parsing config('database.connections.pgsql.search_path')

Fixes https://github.com/laravel/framework/issues/41062
e.g., Config value 'test-db' was being parsed as set search_path to "test", "db" instead of set search_path to "test-db"
9.x renamed config('database.connections.pgsql.schema') to config('database.connections.pgsql.search_path') and introduced special handling if the config value is a string containing many segments. The originating PR has some examples. However the given PostgresConnector regex doesn't consider the full range of characters allowed in a schema or variable name - specifically '-' and accented characters.
Replace the 'search_path' regex alphanumeric-only allowlist of characters with a blocklist of  delimiters when  config('database.connections.pgsql.search_path') is a string value. Technically Postgres does allow our config delimiter characters (spaces, comma, quotes) in symbols so an array configuration can instead be used for such schema names. (However non-wrapping single/double quote characters in such array configs still aren't supported.)
return [
    'default' => env('DB_CONNECTION', 'mysql'),
    'connections' => [
        'pgsql' => [
            'search_path' => [
                "'public'",
                '"$user"',
                'containing commas, and spaces'
                // 'can\'t name schema with "inner quotes"'
            ],

set search_path to "public", "$user", "containing commas, and spaces"

Roll methods testPostgresSearchPathCommaSeparatedValueSupported() & testPostgresSearchPathVariablesSupported() into testPostgresSearchPathIsSet() with the fuller coverage provideSearchPaths() data set.
testPostgresSearchPathArraySupported() is repurposed to show config('database.connections.pgsql.schema') from versions < 9.x is used when the 'search_path' config key is absent.
Fix PostgresConnector::quoteSearchPath() parameter docblock since passing in a string value would throw an exception for being un-Countable. Its only  use is being passed parseSearchPath()'s return value which is an array.

Thanks!


Consent styles are served in by stylesheet

Related Issues

closes https://transcend.height.app/T-17364

Security Implications

consent-manager-ui stylesheets are now loaded in as a standard .css file appending instead of using injected styles. This should make the consent manager default UI work with strict CSP rules, while also allowing for full customization.

@eligrey wonder how much this shrinks the build!


[Application] Setup meson file for libyolov2_loss_layer.so

Setup meson file for making libyolov2_loss_layer.so in yolo v2 application.
Self evaluation:

Build test:	 [X]Passed [ ]Failed [ ]Skipped
Run test:	 [X]Passed [ ]Failed [ ]Skipped

:memo: TAOS-CI Version: 1.5.20200925. Thank you for submitting PR #2310. Please a submit 1commit/1PR (one commit per one PR) policy to get comments quickly from reviewers. Your PR must pass all verificiation processes of cibot before starting a review process from reviewers. If you are new member to join this project, please read manuals in documentation folder and wiki page. In order to monitor a progress status of your PR in more detail, visit http://ci.nnstreamer.ai/.

:octocat: cibot: @baek2sm, A builder checker could not be completed because one of the checkers is not completed. In order to find out a reason, please go to http://ci.nnstreamer.ai/nntrainer/ci/repo-workers/pr-checker/2310-202309131527310.43626689910889-9c2783a45846894370127e0420310fd6fd4a7f1e/.

:octocat: cibot: @baek2sm, A builder checker could not be completed because one of the checkers is not completed. In order to find out a reason, please go to http://ci.nnstreamer.ai/nntrainer/ci/repo-workers/pr-checker/2310-202309131847580.50294494628906-9c2783a45846894370127e0420310fd6fd4a7f1e/.

:octocat: cibot: @baek2sm, A builder checker could not be completed because one of the checkers is not completed. In order to find out a reason, please go to http://ci.nnstreamer.ai/nntrainer/ci/repo-workers/pr-checker/2310-202309141112480.77514290809631-50c1fdf272010b7b505a614d24fb9632289fb27d/.

Thank you, have a nice day!


[IMPROVEMENT] Change UniGetUI package id in WinGet

Please confirm these before moving forward.

[X] I have searched for my feature proposal and have not found a work-in-progress/duplicate/resolved/discarded issue.
[X] This improvement refers to an existing feature. If you want to suggest a new feature, please use this template.
[X] This improvement is not a bug. If you want to report a bug, please use this template.

Describe the improvement
Currently UniGetUI has the old name in the Winget package listing:

Not only that, UniGetUI is published under a totally strange id, with no relation to the author or the programming language used (which seems to be C#):
SomePythonThings.WingetUIStore
The package id should be changed to some options like:
MartiCliment.UniGetUI
MartiCliment.UniGetUIStore

Anyone searching for WingetUI would still find UniGetUI because the package name mentions the old name "(formerly WingetUI)".
Describe how this improvement could help users
It helps improve the classification, identification and location of the package, as the id is correctly assigned to the author and directory name on github and the official website:

https://github.com/marticliment/UniGetUI/
https://www.marticliment.com/unigetui/

Yes, but can an Id be changed in WinGet?

@marticliment I think it is indeed possible
https://learn.microsoft.com/en-us/windows/package-manager/package/
And I see a recent commit made here too - https://github.com/microsoft/winget-pkgs/commit/1d077a47171e02b6b468bd74af2b7f3c05a85b54

Yes yes this is a thing that is being worked on. The migration should be done in the following days.
More details here: https://github.com/microsoft/winget-pkgs/issues/174102


Getting rid of global jquery

We need an indirection to be able to use different jquery implementations (jquery, cheerio).
Without any browserify configuration, cheerio will be bundled for the web, even if not used.
Edit: this is now without indirection.

@michael let's discuss this

this works. i rebased controller-refactor to it.


[Feature][Install] add upgrade 2.0 ddl

#6597

Codecov Report

Merging #6648 (6dd0d5b) into dev (00eea95) will decrease coverage by 0.00%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##                dev    #6648      +/-   ##
============================================
- Coverage     38.51%   38.51%   -0.01%
  Complexity     3223     3223
============================================
  Files           646      646
  Lines         25847    25852       +5
  Branches       2800     2801       +1
============================================
  Hits           9956     9956
- Misses        14978    14983       +5
  Partials        913      913

Impacted Files
Coverage Δ

...scheduler/dao/upgrade/DolphinSchedulerManager.java
0.00% <0.00%> (ø)

...pache/dolphinscheduler/dao/upgrade/UpgradeDao.java
4.21% <0.00%> (-0.05%)
:arrow_down:

Continue to review full report at Codecov.

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


[Imserso_ToTheLimit] First version of this new sheet

Changes / Comments
This is the sheet for the Spanish System Imserso: To The Limit, that recently has funded its second version in verkami.
Roll20 Requests
Comments are very helpful for reviewing the code changes. Please answer the relevant questions below in your comment.

[x] Does the pull request title have the sheet name(s)? Include each sheet name.
[ ] Is this a bug fix?
[ ] Does this add functional enhancements (new features or extending existing features) ?
[ ] Does this add or change functional aesthetics (such as layout or color scheme) ?
[ ] If changing or removing attributes, what steps have you taken, if any, to preserve player data ?
[x] If this is a new sheet, did you follow Building Character Sheets standards ?

If you do not know English. Please leave a comment in your native language.

Character Sheet Info Roll20 Internal Use only.

Few things:

Don't worry about the failed check, that appears with all PRs for new character sheet.
It looks great!
I see you have commented out code in the HTML in a few places, is that intentional?
You have the CSS code minified, is that because you used some pre-processors or frameworks to generate it? If so, committing the source files to the repo would be a good idea, making it possible for other to also work/see on the original source files. Saving the CSS prettified would also make it easier to deal with

Hello, thanks for the clarification and the positive feedback.
The comments were left behind by mistake, thanks for pointing to it.
The css minification is part of the framework i was using yes, i've disabled it now to make easier the collaboration of other members of the community. The reason not to share the original source files is that some are part of a framework from another developer that hosts it in his own repository, and I'm not confortable sharing them, that's why I decided to share only the compiled files (besides, it involves the use of grunt, less, etc. and some developers may not feel confortable using those)


Add Resources Allocation doc

@Puppy95
For experiment branch, you just need to hand in your doc files.
For code, we will talk with you about how to merge your work in class.
Thanks.


Missing link

This link on the page is giving 404 error: Data Collection Rule

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

ID: 213a7430-9a39-70c1-36c0-78c57bea0f16
Version Independent ID: d3cecb6e-eee3-1152-31d8-4e9b6defc24a
Content: Deploy the Azure Monitor Agent with auto provisioning
Content Source: articles/defender-for-cloud/auto-deploy-azure-monitoring-agent.md
Service: defender-for-cloud
GitHub Login: @bmansheim
Microsoft Alias: benmansheim

@bmansheim Could you please help to check and update the document with the correct link for this section?

I have a PR in to resolve the link. Will be fixed later today.

#please-close


my side of story

Add some lines to the story

Hi! Thanks for your submission.
Unfortunately it seems that there were changes in the main that conflict with yours.
Could you please perform a merge as described in Tutorial 3 slides 90-103 and integrate those changes?
Start with synchronizing your local main with my upstream main as described in slide 98.
We do not want to lose any existing content, so you should keep what others have written and add yours.
You're free to reorganize the content blocks so that it still makes a story, but please do minimal changes
to the content of what others have written to integrate them with yours.
The merges will be approved according to arrival time starting from October 23th, 22:00,
so expect that you need to merge several times after that time.
If I do not approve the changes you will be asked to update the content.

Hi @sh2111sa , I am not sure about what you're doing but there is still some garbage in your version. I will open a review.

It appears that your problems where generated when doing this merge commit: 7c913da
I suggest you restart from the commit before that this way:
# Retrieve the latest version of the upstream main
git checkout main
git pull upstream main
# Switch to the mystory branch
git checkout mystory
# restore the commit before the merge. NOTE: you will lose the commits done after that commit.
git reset<PHONE_NUMBER>79f43950669775a421d8465407b995
# retrieve the unchanged version of fairytale.md before the merge
git checkout fairytale.md
# perform a merge again
git merge main
# resolve conflicts
# After resolving, force push to your mystory branch. NOTE: this will delete the problematic commits on both your local copy and github, that is, will overwrite the changes history.
git push -f origin mystory

Does the above make sense for you? If you are unsure you can visit my office and I can explain in more detail.
Another way to go is to do the merge from the current upstream main and delete all that is contained in your version, keep all of the upstream main version and adding your contribution to that. It's in principle easy if you understand what you're doing, but from your mistakes I think you did not understand what happens in slides 97-102. If so we should definitely meet.

Closing as student never replied back to teacher.


Removed superfluous UMD wrapper/AMD check that causes issues with Requir...

...eJS

@bcbweb your PR is not correct as you have modified the dist file and not the src file.
I would add that if you touch on the AMD loader you should not modify the whitespace or create a commit to change all the whitespaces.
And last why would you want to remove the UMD wrapper ?

@stephanebachelier Apologies, the request was an accident (I meant to merge it into the master on my fork for testing purposes)


Determine role of each team member

Vuetify - Caity and Kendra
Google Firebase - Dianne and Preston
Open to any - Brian


FAPI-CIBA Backchannel Authentication Request : support id_token_hint

The design document for FAPI-CIBA states that it requires to support id_token_hint.
However, the current CIBA prototype implementation does not support it.
The goal of this issue is to implement on the CIBA prototype implementation id_token_hint by following the design document for FAPI-CIBA.

Please assign this to me also.

Thank you @HryhoriiHevorkian

Hello @HryhoriiHevorkian ,
FYI, the User Resolver described in page 50 of the CIBA Prototype Implementation Practical Guide might be helpful.

merged. https://github.com/tnorimat/keycloak/pull/1


Regarding installation error

Hi Team
I am trying to install package using given syntax  i.e.
pip install prompt2model
After installation, when I am trying to import
from prompt2model.utils import api_tools., I am getting following error.
ImportError: cannot import name 'api_tools' from 'prompt2model.utils' (/anaconda/envs/chintan/lib/python3.10/site-packages/prompt2model/utils/init.py)
I am using conda with python=3.10 version, please help.
Thanks

There's a bug - will fix it asap.

In the meanwhile, you can just skip this code block to use gpt-3.5-turbo by default.

Hi @viswavi
Thanks for the response. But I am unable to run any import

Thanks

Hi @chintanshrinath, the problem was that the latest PyPi release was out-of-sync with the latest notebook. We've updated our package on Pip, which should resolve all of these issues.
Can you do pip install --upgrade prompt2model, try the notebook again, and report back if this doesn't work? Sorry for the inconvenience.

Hi @viswavi
yes it is working now.
Thank you for your prompt response and support


Refacto logger's code

What does this PR do?
This PR remove duplicated code and align lock logic to every function, moreover function that work on same part of the logger are group together to make the research inside this file easier

Motivation

Due to last changes in the logger to avoid data race, the code was a bit messy, this PR is here to make the code easier to read and understand.
Additional Notes

Possible Drawbacks / Trade-offs

Due to the add of sub-function, according to the number of call on this function the execution may be slower
Describe how to test/QA your changes

See if the number of lock is more or less the same as before
See if the logs are the same (ie. it don't goes in a if condition that it shouldn't)

/trigger-ci --variable RUN_ALL_BUILDS=true --variable RUN_KITCHEN_TESTS=true --variable RUN_E2E_TESTS=on --variable RUN_UNIT_TESTS=on

/trigger-ci --variable RUN_ALL_BUILDS=true --variable RUN_KITCHEN_TESTS=true --variable RUN_E2E_TESTS=on --variable RUN_UNIT_TESTS=on

PS: thanks for the proper commit splitting, made reviewing much easier !

/merge


Difference from go-avahi-cname

Hi!
I'm really glad you mentioned me 😄
I thought about doing the same functionality for Trafik (I use it too) inside go-avahi-cname, but realized that it's not necessary since *.<anyname>.local ("all subdomain") is supported.
You don't have to announce all the entries once every 600 seconds (unlike the subdomain option), which isn't very pretty to be honest. Because it clogs traffic and if a new device appears on the network, it can get an update only after 600 seconds in the worst case.
Can you describe your case, what you needed to use it for. I'm interested + maybe I can help :)

I was playing around with Traefik, when I came to a point where I asked my self how to register DNS names in my "Homelab" there is not much I can do in my broadband router, an alternative would be to use an official DNS server and provide it with internal IPS. So I could achieve something like intranet.mydomain.tld but this may confuses family members not familiar with concepts like this and they will complain why the service is not available outside the house. It also requires a manual process.
So I came across this article from Andrew Dupont, I thought this maybe possible with the approach traefik does by scanning the the Docker Labels. So I searched in this direction and came across https://www.hardill.me.uk which already is what I was looking for. But it uses nodejs since I am starting to learn Go I thought this is a nice project to develop in Go. I was searching for Avahi and go + CNAME what brought me to your project. Which mostly fulfills my needs except the fact I was looking for the abritray-name.local instead of the subdomain approach you are doing. But yeah it is also a smart way to just announce the service which is asked for.
My use case with the names like traefik.local and homeserver.local is to be a independent of the machine the service is running at. But you are right it is also achievable with your subdomain service like this starting the subdomain service with go-avahi-cname subdomain --fqdn homeserver.local and the register the services subsequently traefik.homeserver.local or grafana.homeserver.local  But this may can cause issues when splitting services across machines which honestly I should take care of when needed  😄 (YAGNI)
In the end it is some fun project where I already have learned a lot of stuff, I took away a lot from your project and I could use almost 100% of your GitHub Workflows without adapting a lot :-) like the usage of FROM scratch as a bare minimum image.

Oh, got it.
I gave up on the use of .local, and just switched to using a public domain, which reply with a private IP.
https://who.is/dns/lab.grishy.dev
Different OS have a different mDNS support. I spent a few days and even patched wireguard, but gave up at the user device stage.  And mDNS like an additional now for domain.
Have fun 😄

Yeah, the VPN fact is another downer I do agree. This is due to the fact .local is not available due to the TTL set to -1 or something like this which prevents the multicast messages being transferred across network. I guess in the end I will end up with a similar approach you choose. 😄

Write if you have any questions or ideas, I'd be happy to help.
Nice to communicate with you 😄
p.s.
The problem is how mDNS is handled, usually VPNs just don't work with mDNS (multicast protocols in general) and that on iOS the settings don't change for mDNS.
The system has like if (domant.end_with(".local")) -> do specific logic.


What version of Node are we targeting? (OutgoingMessage & emit close)

If I remember correctly, OutgoingMessage does not always emit close in all versions of Node.  Something we depend on.
Should we use eos or on-finished to make sure we are as compatible as possible?

Totally ok using on-finished.
As far as versions of node targeted: I think the test matrix currently goes from v6 to v10 but I'm mostly just trying to keep ~ last two LTS's. I figure a few years back is fine. Certainly not supporting v0.10 anymore!


Some 9.0.0 icons are missing the 45 degree inside corner

Detailed description

Describe in detail the issue you're having.

It would appear a few of the icons missed the 45 degree inside corners

Is this a feature request (new component, new icon), a bug, or a general issue?

bug

What version of the Carbon Design System are you using?

9.0.0
Additional information

cc @tay-aitken just to confirm if this is in the Sketch file, or not. Could easily be a mistake in our build step.
Also, just a heads up @jackiwakin make sure not to make any public references to that one project 👍

Hi @jackiwakin! Thanks for bringing this to our attention. These have now been fixed. They will be updated in the next kit release in the design kit/library (which currently does not have an expected/planned release date) . @joshblack it it a breaking change to update these icons in the repo?

Just a note: will not be updating the forum icon to have those corners as the library we inherited from does not follow that styling on that particular icon.

Going to close this issue as the icons with the error have been fixed :)

This was addressed in PR #76


model comparison in netron

Hi Lutz,
Thank you for the Netron, we use it on a regular basis.  Have you ever considered to add possibility to compare networks, like you would compare source code in meld. We are working on optimizations of neural nets and just use two Netrons side by side to visualize changes.   Appreciate your thoughts.
Thanks,
-Mike

Duplicate of #705


Can't open image sequence frame01.png - frame80.png

Opening image sequence frame01.png to frame80.png fails. It incorrectly detects it as frame1.png - frame80.png and fails to open frame1.png. It was working fine in DJV 2.0.6.
Version: DJV 2.0.8
OS: Linux

Hi, I have not been able to reproduce this; how are you opening the files, drag and drop, file browser, or the command line? Also could you attach the log file as described here:
https://darbyjohnston.github.io/DJV/contributing.html#Bugs

I am opening using File > Open. I have Enable File Sequences checked under Settings - Miscellaneous in the File Browser settings (it was unchecked by default for some reason).
djv.log
It is strange. If I copy the directory containing the files to another directory, it works.
Maybe something to do with the order of the file entries when listing a directory during the sequence detection?

Try this.
Create frame80.png to frame01.png in reverse order in new folder
cd ~
mkdir seq
cd seq
for i in `seq 80 | tac`; do convert -size 320x240 canvas:black $(printf frame%02d.png $i); done

Open the folder in DJV using File > Open and navigate to it.

That did it; I checked in a fix and will include it with the next release.


[CARBONDATA-1411][CARBONDATA-1414] Show Segment command gives Null Pointer Exception after updation

Fix issues while showing Segment command gives Null Pointer Exception after updation

SDV Build Success with Spark 2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/343/

retest this please

retest this please

retest this please.

SDV Build Success with Spark 2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/358/

retest this please.

test again

retest this please

retest this please

SDV Build Success , Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/548/

LGTM


Make the renewal margin configurable, in days before certificate expiry

Currently certificates are renewed if they have less than a week before they expire. This is reasonable, but hard coded, and some people may wish to adjust that. In my case I'd like to match other ACME tool configurations on other system that are set for 10 days before expiry.
I propose adding an --renewal-margin-in-days argument for this setting. It will have a default value of 7. Values less that 1 will be treated as 1. Values greater that 89 will log a warning that this might be longer than the certificate lifetime; just a warning because Let's Encrypt or another ACME provider could ofer a different period or change their policy.
https://github.com/whereisaaron/kube-cert-manager/blob/master/processor.go#L337-L342
// If certificate expires in more than a week, don't renew
if parsedCert.NotAfter.After(time.Now().Add(time.Hour * 24 * 7)) {
	return false, nil
}

log.Printf("[%v] Expiry for cert is in less than a week (%v), attempting renewal", cert.Spec.Domain, parsedCert.NotAfter.String())

@whereisaaron it seems it is at master already
https://github.com/PalmStoneGames/kube-cert-manager/blob/master/processor.go#L428
https://github.com/PalmStoneGames/kube-cert-manager/blob/6cf132dd3b2adadc607c9ca0cfca6d344495d1ed/main.go#L83

Yep @sfrique ! Kindly added by @stigsb a couple months ago in #62.


Q: Just using it for one specific site?

Is there a way to only allow the proxy to visit one specific site and to not allow them to use it to go to any other sites?

Sure, check the BlockListPlugin:
https://github.com/Athlon1600/php-proxy/blob/master/src/Plugin/BlockListPlugin.php
Putting something like this inside config.php would work:

$config['blocklist.custom'] = function($data){

     if(strpos($data['url'], "my_site.com") !== false){
          return true;
     }

     return false;
};

Appears to have been solved.


part brand

Hi again,
I was surprised not to find a brand among the part details.
Can this be added?

There are many such fields which it could be argued would be useful.
I would suggest adding "brand" as an "parameter", or even adding it to the notes field


UX improvements for open-choice widget

We should update this widget with Shelagh's really nice UX improvements. Eg, consider single-line for single-select "Other", and remove the "Other" checkbox altogether for multi-select (just have a button to add additional options).

From Shelagh:
Thanks for elaborating. I hear you on that it seems more clear to use the remove '-' button than to uncheck the other box if you want to unselect/remove the other choice. The 'add other' button could also help with people typing in the text, rather than just selecting other without adding the descriptive text.  I mocked up what it could look like with an 'add other' button and without "other" listed in the list of options and I think the pattern works. see below
Is there a scenario where we want to enable selecting other, but not adding edit text?

Originally posted by @shelaghm in https://github.com/google/android-fhir/issues/801#issuecomment-933666790

Adding a screenshot of the current build of this component in the SDCCatalog app. It's great to see we have the functionality to add free text and remove the choice.

I'd love to see the open choice widget not be contained within the modal. I think it would be easier for folks to understand the add open choice without the visual distraction of it being displayed inside a modal.
@kevinmost is there a timeframe for when you'll have time to work on this?

@juanchazo does this relate to any of the work you did on modals?

@joiskash Could you leave your feedback here on improving the open choice widget to reduce the number of clicks? Thank you!

The feedback that we received from the field is that if a selection is made from the given choices, then pressing save should not be required. This will reduce the number of clicks.
Save needs to be pressed only if the user types in the edit text and adds a new answer option.

discussed with @shelaghm, the priority in this ticket is to make open choice not a modal dialog. @joiskash i think this would solve the issue with # of clicks too.

We heard from healthcare workers how important this feature is. Multiple choice options are often too limiting and the ability to add custom response is very valuable.
Usability testing design recommendations:

Open choice should look the same as multiple choice (not in a modal)
Replace minus (-) icon with another icon and use text to describe action
Reduce number of steps in open choice flow (auto display the text box instead of showing add another option button.
Add an on-screen button to close the keyboard (research is required to understand if this is possible to do)

I have detailed designs for this in figma that I can share. Please reach out when you start working on this.


rename PDHG lizard

They've moved to the directory 'notebooks' (lowercase). I'll rename them and add them to one pull request.


[core] OSGi packaging

Currently OSGi packaging is straight forward: each Maven bundle becomes an OSGi bundle by adding the Manifest at build time.
When using XO in the OSGi environment  the user only works with the API bundle (+ store provider in some cases) The user doesnt know anything about impl and spi bundle. But both bundles are necessary at runtime: must be part of the runtime. Additionally the impl bundle must be "started" to get the Bundle Listener work properly. Well, one could extend documentation, but I propose to change the OSGi bundle build in a way to make it easier for developers:
a) build a separate bundle that depend on api + spi + impl and contains the BundleListener
b) attach spi + impl to api bundle

There is a "hidden" feature in Eclipse: Bundles with "lazy" activation policy will be started by the framework on startup. I will prepare a PR to make use of this feature and close this issue afterwards.

should be solved by #169


Instructions assume your are on a public network profile

These instuctions assume your network profile is set to public. If you make the changes using powershell the network rules only allow the new port over a public network. So if you are on a private network and don't realize this, once you restart your system you won't be able to connect to it.

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

ID: 13054e70-b770-f803-4596-6d0ab3fe865c
Version Independent ID: 3e474a79-1129-fdf6-bbd4-a12e0e7ebe27
Content: Change the listening port in Remote Desktop
Content Source: WindowsServerDocs/remote/remote-desktop-services/clients/change-listening-port.md
Product: windows-server
Technology: remote-desktop-services
GitHub Login: @lizap
Microsoft Alias: elizapo

We hugely appreciate your contribution, feedback, and support to our technical documentation.
We’ve phased out GitHub Issues on this repository as the feedback mechanism for content and have replaced it with a new feedback system. For more information see, Content User Feedback. As part of this rollout, we are migrating your issue where our team will use your comments to improve the documentation experience. Unfortunately, this does mean you won't be able to access this issue after the migration is complete.
All issues will be migrated by Thursday 29th August 2024.
Once again, thank you for your valuable feedback, we hope you’ll continue to contribute using our new feedback system.


refactor: add more clippy rule

Summary
related to #7589

Checklist

[ ] Tests updated (or not required).
[ ] Documentation updated (or not required).

!bench

Would you please fix the conflict and let's get it merged first?


Turn on --enable-shared by default for all supported MRI Rubies

Some gems need the host ruby to have shared library enabled. Currently, we're not aware of any potential downsides to having this enabled by default.
Fixes https://github.com/rbenv/ruby-build/issues/35, ref. https://github.com/rbenv/ruby-build/pull/921
Should we support a flag to disable shared, i.e. to override the default?

As mentioned in https://github.com/rbenv/ruby-build/issues/35#issuecomment-504406843, one potential concern is that shared library might be a bit slower. I do remember observing that a long time ago on macOS.
I'm measuring on Linux with OptCarrot, ran from the TruffleRuby repository to use a simple harness.

It's not a very large difference (2-6% in those measurements), but it seems to not be in favor of --enable-shared consistently.
For the record, this is with two builds of MRI 2.6.5 installed with ruby-install.

Here is a run with a bit less noise:
https://gist.github.com/eregon/2e41061391979f8ab19010179d7dc173
All 3 runs are 46 FPS on --enable-shared and all 3 runs are 47 FPS without.

@eregon Thank you for checking!
Unless performance with --enable-shared is abysmal and would noticeably affect people's experience when developing with Ruby, I vote that we enable this. If someone comes to us and says that this change to ruby-build led to their production environment reporting a % of slowdown in their Ruby processes, we can always make a case that ruby-build is not responsible for performance-tuning your production Rubies— one should maintain custom definitions for their production environments to enable fine-tuning.

@hsbt Let's leave this open for a day or two and merge if there is no evidence of significant regression. I thought that we might want to introduce a flag to disable shared, but we can also instruct people to copy the build definition to a separate file and remove enable_shared from that definition. It's a bit of a hassle, but hopefully people won't have to do it too often

@mislav I think it would be safer to introduce a flag or document a way to build without the shared library (maybe -- --disable-shared works?) as an easy opt-out, should any problem arise.
One more thing on this subject is RubyGems puts C extensions files for enable-shared Rubies in a different directory (which is good), see https://github.com/postmodern/chruby/pull/410#issuecomment-512993069
That might lead to warnings like Ignoring msgpack-1.3.0 because its extensions are not built. Try: gem pristine msgpack --version 1.3.0 if somehow mixing already-installed non-shared-library gems and a --enable-shared Ruby.
Given that rbenv doesn't set GEM_HOME, this should only happen for gems installed with gem install --user or to a specific path (e.g., bundler --install path) but not for gems installed in the default gem directory.

@eregon Thank you for that bit of info!
Merging this as-is. Let's see if it causes trouble. 🔥🔥 #famouslastwords

Thanks @mislav, @hsbt, and @eregon!

As mentioned in #35 (comment), one potential concern is that shared library might be a bit slower.

Agreed. We recently disabled that flag and got a ~1.5% improvement on our response time.
Could this PR be reconsidered? Or at least could we make it easier to opt out? Because right now passing RUBY_CONFIGURE_OPTS=" --disable-shared " is ineffective, we actually have to patch all the definitions to opt-out.

Could this PR be reconsidered?

I think we should avoid changing the default back, as that would be an incompatible change for users relying on having the shared library (some gems need it).

Or at least could we make it easier to opt out?

That would be good. Could you make a PR to add that?

I think we should avoid changing the default back

Agreed.

Could you make a PR to add that?

I'll look into it.


[DO NOT MERGE] Api drops some ids

Closes #2525

Coverage increased (+0.5%) to 71.66% when pulling b7a8a6386f4a89e9f3c9f7f87711b73672fe43bb on eatonphil:api-drops-some-ids into d7ecd35034a329e44c373f5ff21ce9d853cedcf9 on linode:master.


Pooling: fix documentation

It seems the output size of the pooling operation was incorrectly documented.

Thank you!


[terra-section-header] Migrate Docs to terra-core-docs

Summary

Migrate Docs to terra-core-docs

Closes #
Deployment Link

https://terra-core-deployed-pr-#.herokuapp.com/
Testing

Additional Details

Thank you for contributing to Terra.
@cerner/terra

can we remove these dependency https://github.com/cerner/terra-core/blob/section-header-docs/packages/terra-section-header/package.json#L32 ?

can we remove these dependency https://github.com/cerner/terra-core/blob/section-header-docs/packages/terra-section-header/package.json#L32 ?

No terra-arrange is used here: https://github.com/cerner/terra-core/blob/1f6eecf589defdb00f694ae94aefc97fd2f91a4a/packages/terra-section-header/src/SectionHeader.jsx#L7


Switch to triangle

Swaps out meshpy for triangle as discussed in #24, #22 and #87.

Nice!  This will certainly help Windows users since triangle publishes wheels to PyPI and thus they won't have to compile it.
We may want to test or explore any potential issues that using the locale may have, as discussed in #143.

Nice! This will certainly help Windows users since triangle publishes wheels to PyPI and thus they won't have to compile it.
We may want to test or explore any potential issues that using the locale may have, as discussed in #143.

I did not see any changes to locale set in the conf.py from this branch and it seems that this branch passes all tests and the docs build without special modification. This looks like a winner to me!

Yes, the locale changes where in meshpy, so it makes sense that we don't have that issue now.  But, if someone writes a program and uses some other locale, when they could, potential, get different results.  Though it would depend on how triangle works, which I haven't gone investigating.


CORS error in browser when utilizing Azure Blob Storage or Azure CDN via the W3 Total Cache (W3TC) plugin

When utilizing the latest WordPress on App Service marketplace deployment and the Azure Blog Storage (check box during deployment) or the Azure CDN (check box during deployment. In either situation, the fonts do not load properly. This is due to a CORS error (shown below on the storage account) when leveraging the W3 Total Cache (W3TC plugin). The only difference for the Azure CDN would be to change the URL endpoint in the W3TC plugin under CDN advanced settings but the same error exists as shown below.

When the Azure CDN endpoint URL is used (the default when Azure Blob Storage and Azure CDN are both checked during deployment. Overwriting the CORS via Azure CDN rules (as documented here) does resolve the issue.
When just the Azure Blob Storage is checked during deployment. The only thing I found that worked so far is this process a user submitted here but it seems to be open as far as the many methods exposed this way.

Obviously, if you add a custom domain you may have to do some additional configuration but the Azure generated DNS records I feel could be added in by default. Is there a change that can be made to the deployment process that adds in the proper CORS settings on the Azure storage account or Azure CDN to successfully utilize the W3TC plugin with how it is deployed today. Or at least some documentation that can be created to show what the proper CORS settings on the storage account that are needed to get this working (as the linked content provided above)?
Access to font at 'https://redactedNameHere.blob.core.windows.net/redactedNameHere/wp-content/themes/redactedThemeNameHere/assets/fonts/redactedFontNameHere.woff' from origin 'https://redactedawesomewordpressdomainhere.com' has been blocked by CORS policy: No 'Access-Control-Allow-Origin' header is present on the requested resource.
GET https://redactedNameHere.blob.core.windows.net/redactedNameHere/wp-content/themes/redactedThemeNameHere/assets/fonts/redactedFontNameHere.woff net::ERR_FAILED 200 (OK)

Hello @ShawnPOWER, thank you very much for your contributions and bringing this to our notice. I have pushed a fixed that will be available for new deployments, in next weeks.
Furthermore, I have added a documentation about this: CORS Issue With Azure CDN / FrontDoor or Azure Blob Storage
Thanks again! :)


[SPARK-20986] [SQL] Reset table's statistics after PruneFileSourcePartitions rule.

What changes were proposed in this pull request?
After PruneFileSourcePartitions rule, It needs reset table's statistics because PruneFileSourcePartitions can filter some unnecessary partitions. So the statistics need to be changed.
How was this patch tested?
add unit test.

Test build #77744 has started for PR 18205 at commit 20a6043.

Test build #77744 has finished for PR 18205 at commit 20a6043.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

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/SparkPullRequestBuilder/77744/
Test PASSed.

Test build #77763 has started for PR 18205 at commit c53a0c7.

Test build #77763 has finished for PR 18205 at commit c53a0c7.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/77763/
Test PASSed.

Merged build finished. Test PASSed.

Test build #77842 has started for PR 18205 at commit c53a0c7.

Test build #77842 has finished for PR 18205 at commit c53a0c7.

This patch fails from timeout after a configured wait of `250m`.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/77842/
Test FAILed.

Can you please explain why reset?
To my understanding, we don't have any statistics in CatalogStatistics initially, and we can get the right size in computeStats through HadoopFsRelation.sizeInbytes, because in the original code we already replace it with prunedFsRelation.

OK. I get your point. But the test case is not clearly show the problem. We can first analyze the table to fill stats in CatalogStatistics, then show difference after partition pruning.

Test build #77892 has started for PR 18205 at commit 120662e.

Test build #77892 has finished for PR 18205 at commit 120662e.

This patch fails from timeout after a configured wait of `250m`.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/77892/
Test FAILed.

Test build #77934 has started for PR 18205 at commit f7c3dfc.

@wzhfy I have addressed your comments. Thanks.

Test build #77934 has finished for PR 18205 at commit f7c3dfc.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

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/SparkPullRequestBuilder/77934/
Test PASSed.

LGTM except one comment.

Test build #77958 has started for PR 18205 at commit d1513a8.

Test build #77958 has finished for PR 18205 at commit d1513a8.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

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/SparkPullRequestBuilder/77958/
Test PASSed.

LGTM, ping @cloud-fan

@cloud-fan I have addressed your comments. Thanks.

Test build #77989 has started for PR 18205 at commit 16a3f7e.

Test build #77989 has finished for PR 18205 at commit 16a3f7e.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

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/SparkPullRequestBuilder/77989/
Test PASSed.

We have another related PR: https://github.com/apache/spark/pull/14655/

thanks, merging to master/2.2!

@cloud-fan  Thanks.


When formatting JSX single quotes option is ignored.

I am evaluating the tool on an existing code base. By coincidence, prettier matches the existing style except that it prefers single quotes and uses 4 spaces. So I changed those options in the editor plugin.
With settings Single Quotes enabled and Tab Width set to 4, the following code is changed from:
function SomeComponent() {
    return
Hello World

}

Into:
function SomeComponent() {
    return
Hello World

}

 function SomeComponent() {
-    return
Hello World

+    return
Hello World
;
 }

Turns out this is by design https://github.com/prettier/prettier/pull/126


keyserver client: retry updates after process failures

If we want to avoid the need for manual recovery in case a profile update makes it to the keyserver but the reply does not make it back, the client should retry the update until it gets a definitive response, EVEN IF the client has not been running continuously since it first issued the update. Not doing this can cause the affected client to sign messages using the previous PGP key while its contacts see the results of an update whose reply got list (and thus they reject the signatures).

your description doesn't sound there's a need to retry updates (re-upload the key).
does it mean more like every time before a signature is made, make a fresh lookup for the verifiable public key, and uses only the corresponding private one for signature?


upgrade chart versions

change jenkins-x/jenkins-x-platform to version 2.0.80

/test all

/test ng

/test ng


How to use sql NOW() as default value for the column when db.Create() using struct

How to use sql NOW() as default value for the column when db.Create() using struct

This works for me:
TimeCreated   time.Time gorm:"default:CURRENT_TIMESTAMP"

@franklingu you can use BeforeSave hook for that https://gorm.io/docs/hooks.html


Expose JS defined in

It would be great if variables, functions, etc, that are defined in a .webc component's

It would also be great if we could reference global Eleventy data here, something like:

It could also be really interesting if the functions, variables, etc, defined in the


US 03.03.01

As an owner, I want to select items from the list of items and apply one or more tags to the selected items.
Requirements/Prerequisites:

US 03.01.01: There must already be some collection of tags in the database.
US 02.03.01: Multiselect functionality for items must already be in place.
When one or more items are selected through multiselect there should be a dropdown type list where multiple tags may be selected and applied to all selected items.
If any of the currently selected items do not have a tag then selecting a tag should apply it to all of them (or leave already tagged items unchanged)
If all of the currently selected items have a tag then selecting that tag should remove it from all of them.

Current Progress: Selecting items by tags is done, deselection requires clarification, UI mockup and storyboard needs to be updated and testing not finished.
Needs further clarification on last point:
If all of the currently selected items have a tag then selecting that tag should remove it from all of them.
Situation:
Suppose there is one item in the list with two tags, and we unselect one of them, should we deselect it or leave it selected:

If left selected when reopening the tag multi-selection dialog it will show all of the tags being selected
If unselected when reopening the tag multi-selection dialog it will show no tags selected


ConditionTimeoutException thrown when using ignoreExceptions could expose last thrown exception

When using ignoreExceptions() we get a ConditionTimeoutException
await().atMost(ONE_MINUTE)
           .ignoreExceptions()
           .pollDelay(ZERO)
           .pollInterval(ONE_SECOND).until(() ->.......

However, it doesn't include any information about the exceptions thrown. So all you know is that the operation failed.
It'd be handy for the last exception thrown to be added to the exception hierarchy.

Yes that would be nice. Maybe you could have a look at it?


deprecated method SearchResult getSourceAsObjectList(Class t)

What is recommended alternative to using getSourceAsObjectList(Class t)?

I posted a related question: http://stackoverflow.com/questions/41582150/how-to-cast-jest-searchresult-hits-to-domain-object .