[SPARK-29032][CORE] Add PrometheusServlet to monitor Master/Worker/Driver

What changes were proposed in this pull request?
This PR aims to simplify Prometheus support by adding PrometheusServlet. The main use cases are K8s and Spark Standalone cluster environments.
Why are the changes needed?
Prometheus.io is a CNCF project used widely with K8s.

https://github.com/prometheus/prometheus

For Master/Worker/Driver, Spark JMX Sink and Prometheus JMX Converter combination is used in many cases. One way to achieve that is having the followings.
JMX Sink (conf/metrics.properties)
*.sink.jmx.class=org.apache.spark.metrics.sink.JmxSink

JMX Converter(conf/spark-env.sh)

https://repo1.maven.org/maven2/io/prometheus/jmx/jmx_prometheus_javaagent/0.12.0/jmx_prometheus_javaagent-0.12.0.jar

export SPARK_DAEMON_JAVA_OPTS=
"-javaagent:${PWD}/jmx_prometheus_javaagent-${JMX_PROMETHEUS_VERSION}.jar=
${PORT_AGENT}:jmx_prometheus.yaml"

This agent approach requires PORT_AGENT additionally. Instead, this PR natively support Prometheus format exporting with reusing REST API port for the better UX.
Does this PR introduce any user-facing change?
Yes. New web interfaces are added along with the existing JSON API.

JSON End Point
Prometheus End Point

Master
/metrics/master/json/
/metrics/master/prometheus/

Master
/metrics/applications/json/
/metrics/applications/prometheus/

Worker
/metrics/json/
/metrics/prometheus/

Driver
/metrics/json/
/metrics/prometheus/

How was this patch tested?
Manually connect the new end-points with curl.
Setup (Master/Worker/Driver)
Add the followings at conf/metrics.properties (conf/metrics.properties.template has these examples)
*.sink.prometheusServlet.class=org.apache.spark.metrics.sink.PrometheusServlet
*.sink.prometheusServlet.path=/metrics/prometheus
master.sink.prometheusServlet.path=/metrics/master/prometheus
applications.sink.prometheusServlet.path=/metrics/applications/prometheus

$ sbin/start-master.sh
$ sbin/start-slave.sh spark://`hostname`:7077
$ bin/spark-shell --master spark://`hostname`:7077

$ curl -s http://localhost:8080/metrics/master/json/ | jq
{
  "version": "3.1.3",
  "gauges": {
    "master.aliveWorkers": {
      "value": 1
    },
    "master.apps": {
      "value": 1
    },
    "master.waitingApps": {
      "value": 0
    },
    "master.workers": {
      "value": 1
    }
  },
...
$ curl -s http://localhost:8080/metrics/master/prometheus/ | grep master
metrics_master_aliveWorkers_Value 1
metrics_master_apps_Value 1
metrics_master_waitingApps_Value 0
metrics_master_workers_Value 1

$ curl -s http://localhost:8080/metrics/applications/json/ | jq
{
  "version": "3.1.3",
  "gauges": {
    "application.Spark shell.1568261490667.cores": {
      "value": 16
    },
    "application.Spark shell.1568261490667.runtime_ms": {
      "value": 108966
    },
    "application.Spark shell.1568261490667.status": {
      "value": "RUNNING"
    }
  },
...
$ curl -s http://localhost:8080/metrics/applications/prometheus/ | grep application
metrics_application_Spark_shell_1568261490667_cores_Value 16
metrics_application_Spark_shell_1568261490667_runtime_ms_Value 143174

$ curl -s http://localhost:8081/metrics/json/ | jq
{
  "version": "3.1.3",
  "gauges": {
    "worker.coresFree": {
      "value": 0
    },
    "worker.coresUsed": {
      "value": 16
    },
    "worker.executors": {
      "value": 1
    },
    "worker.memFree_MB": {
      "value": 30720
    },
    "worker.memUsed_MB": {
      "value": 1024
    }
  },
...
$ curl -s http://localhost:8081/metrics/prometheus/ | grep worker
metrics_worker_coresFree_Value 0
metrics_worker_coresUsed_Value 16
metrics_worker_executors_Value 1
metrics_worker_memFree_MB_Value 30720
metrics_worker_memUsed_MB_Value 1024

$ curl -s http://localhost:4040/metrics/json/ | jq
{
  "version": "3.1.3",
  "gauges": {
    "app-20190911211130-0000.driver.BlockManager.disk.diskSpaceUsed_MB": {
      "value": 0
    },
    "app-20190911211130-0000.driver.BlockManager.memory.maxMem_MB": {
      "value": 732
    },
    "app-20190911211130-0000.driver.BlockManager.memory.maxOffHeapMem_MB": {
      "value": 0
    },
    "app-20190911211130-0000.driver.BlockManager.memory.maxOnHeapMem_MB": {
      "value": 732
    },
...
$ curl -s http://localhost:4040/metrics/prometheus/ | head -n5
metrics_app_20190911211130_0000_driver_BlockManager_disk_diskSpaceUsed_MB_Value 0
metrics_app_20190911211130_0000_driver_BlockManager_memory_maxMem_MB_Value 732
metrics_app_20190911211130_0000_driver_BlockManager_memory_maxOffHeapMem_MB_Value 0
metrics_app_20190911211130_0000_driver_BlockManager_memory_maxOnHeapMem_MB_Value 732
metrics_app_20190911211130_0000_driver_BlockManager_memory_memUsed_MB_Value 0

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/testing-k8s-prb-make-spark-distribution-unified/15474/
Test PASSed.

Test build #110499 has started for PR 25769 at commit 27a9f0c.

Test build #110499 has finished for PR 25769 at commit 27a9f0c.

This patch fails Spark unit 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/110499/
Test FAILed.

Retest this please

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/testing-k8s-prb-make-spark-distribution-unified/15478/
Test PASSed.

Test build #110504 has started for PR 25769 at commit 27a9f0c.

Gentle ping~

LGTM. Merged into master. Thanks.

Thank you, @dbtsai , @srowen , and @itsvikramagr!

@dongjoon-hyun nice to see this feature on Spark 3.0. :+1: .
I wanted to ask you, if you have any ides, how hard would it be to transform metric keys to Prometheus labels? Is it possible at this point? Or is it more feasible to use Prometheus relabel feature, something like this:
 - job_name: spark
    relabel_configs:
      - source_labels: ['__name__']
        regex: 'metrics_(\w+)_(\w+)_.*'
        target_label: 'app_id'
        replacement: '$1-$2'
      - source_labels: ['__name__']
        regex: 'metrics_(\w+)_(\w+)_(.*)'
        target_label: '__name__'
        replacement: 'spark_$3'

I saw you did some job on labeling executor metrics, but in this case we have only metric keys from metric registry, so we cannot really be sure how to label each part of metric key?

Thanks, @pdambrauskas . For the new features or improvement, please file a JIRA for 3.1.0. For now, branch-3.0 is in a feature freeze phase. For now, we are focusing on bug fixes for 3.0.0 release.

Is it possible at this point?

@dongjoon-hyun This seems to be pull based instead of push based as it was implemented in #19775, is that correct?
If so, why was pull chosen instead of push? It seems to me that push would be a better fit for most Spark applications because they are generally short running (batch) jobs instead of persistent/long running services.

@dongjoon-hyun Amazing work - I'm using your PrometheusServlet at Ripple for our real-time monitoring of our Spark platform. 😌
Do you have any plans to add Executor-Task metrics to the Prometheus Servlet? If so, I would love to contribute. I'm a third year computer science student studying at the University of Waterloo and it's been a dream of mine to contribute to Spark.


Create IOU participant selection step.

<If necessary, assign reviewers that know the area or changes well. Feel free to tag any additional reviewers you see fit.>
Details

[x] Added request page for requesting funds from users.
[x] Added split page for splitting bill amount.
[x] Managed steps for the communication.

Tests

[x] Performed UI tests on each of the platforms.
[x] The search works well on each.
[x] On selecting the options the logic for Next button to appear or the request amount page is well rendered.

Tested On

[x] Web
[x] Mobile Web
[x] Desktop
[x] iOS
[x] Android

Screenshots
Web

Mobile Web

Desktop

iOS

Android

recheck

Hey @barun1997, thanks for the submission! Unfortunately I completely missed this today, but look forward to reviewing first thing Monday.

Thank you @roryabraham for the comments. I will look into it and resolve the issues.

recheck

Hey @Julesssss, before the weekend, the file /libs/actions/Sidebar was in place, but looks like it was removed, should I implement the requirement on my own, or is there an alternative that we prefer over those functions?
That is the reason the tests are failing!

cc @marcaaron I think this might be related to the react-navigation switch?

@Julesssss I re-implemented the loading logic. Please kindly take a look into it, and let me know if you have any suggestions.

@Julesssss Sure, no worries! I will implement the change first thing tomorrow.

As a final change, would it be possible to pass the participants back to IOUModal every time the selection changes, rather than on the button press? Then pass them back down to IOUParticipantsRequest|IOUParticipantsSplit via a participants prop. This would allow us to remember the selection if the user navigates back to the amount page from the participants selection.

Hi @Julesssss, I implemented the change as you requested. But I am a bit not sure how the UI for IOUParticipantsRequest will look in this case. For instance, when the user selects a participant, the current implementation navigates them immediately to the confirmation screen. If they navigate back to IOUParticipantsRequest, should we show a tick mark beside the previously selected participant? Or allow him to select a new participant  (while holding the current participant state in IOUModal)?
If we don't have to show a tick mark beside the selected participant, in that case, perhaps it's not necessary to pass the participants prop onto IOUParticipantsRequest. We could only send a callback prop to add a participant to IOUModal when an option is selected.

If they navigate back to IOUParticipantsRequest, should we show a tick mark beside the previously selected participant? Or allow them to select a new participant (while holding the current participant state in IOUModal)?
If we don't have to show a tick mark beside the selected participant, in that case, perhaps it's not necessary to pass the participants prop onto IOUParticipantsRequest. We could only send a callback prop to add a participant to the state in IOUModal when an option is selected.

Good point, I agree that we should not pass the prop into IOUParticipantsRequest, but only for IOUParticipantsSplit.

@roryabraham there are some responses to your original comments, would you mind reviewing them?

Looks great! 👍


FIX: tof_indices

Hey. I was not aware there was an offset error in my tof indices. Looking at the diffs, you seem to suggest that this is due to the scan ordering (which I agree always felt somewhat funky and bug-susceptible, but I was under the impression that this is also the way Bruker stores their data). Do you have an example dataset where this happens?

This change should cause some difference in most files.
I think the algorithm does cumulative sums over tof_indices in partitions specified by scan_indices. So if last_scan is not 0, that will cause a difference.

Hi @guoci . Sorry for not following up on this any sooner. You are completely right about this bug and have the right fix. Since we are several versions ahead by now, I copy-pasted your fix to avoid any merge conflicts. It will be released in the next version of AlphaTims (currently available on the dev branch).


Reorg and refactor to be closer to current standards

No major functional changes in this one- other than to get the automated tests to pass.
I've followed the direction of the other helmsman repos in doing a minor restructuring. It's come to my attention that this impacts other projects that have this dependency.
To that effect- I think we should restructure, whether it happens today or in the future. But it looks like we're going to need to adopt a versioning strategy and some guidelines about expectations around this. I have a couple of thoughts on this one, but I actually think we should avoid becoming held up on process right now and see where this goes. That said, I completely agree, it can be a little chaotic.
I think one thing that we can do to help steer in the right direction is to consider what we might consider a "library" or a "package" and what we'd consider a "demo" or a "project" and to start organically evolving our process to reflect that. Ie: much stricter versioning on assets that could be attractive to use as building blocks for other projects.
Anyway, I pushed this forward before I saw @chrislovecnm's comment- and we've spoken. This is still highly WIP and I've somewhat redirected/paused on this today for other priorities.
CHANGELOG

No more .sh in /. Created scripts/

moved .tf and supporting modules to to terraform/

moved the elasticsearch stuff into one place

updated checks and linting and paths and everything and got Jenkins wired up with the invaluable assistance of @jramsdale

Side note: i'd like to move towards IAC repos that are of a broader scale than just terraform/.... we don't currently have a home for kubernetes manifests like the policy directory and as more moves to kubernetes, we need a cohesive strategy. i have many thoughts on this if you want to have a really   boring gvc about project structure, just let me know :)

Helmsman CI checks are not expected to pass until #9 goes in.

/lgtm


Lz2alerts

Overview/Summary
Addition of Landing Zone Alerts coving Traffic Manager, Frontdoor, s and App Service plan base alerts t o the LZ initiative
This PR fixes/adds/changes/removes

Adds 14 new alert files
Updates patterns\alz\policySetDefinitions\Deploy-LandingZone-Alerts.json with new alerts

Breaking Changes

None but adding more parameters to LZ initiative
Replace me

As part of this Pull Request I have

[ x] Read the Contribution Guide and ensured this PR is compliant with the guide
[x ] Checked for duplicate Pull Requests
[ x] Associated it with relevant GitHub Issues or ADO Work Items (Internal Only)
[ x] Ensured my code/branch is up-to-date with the latest changes in the main branch
[ ] Ensured PR tests are passing
[ ] Updated relevant and associated documentation (e.g. Contribution Guide, Docs etc.) (to be done in separate request)

@Alboroni everything's good, just need to resolve the merge conflict.


Feat(eos_designs): Support for L3 Inband ZTP

Change Summary
Adding L3 In band ZTP functionality
Component(s) name
arista.avd.eos_designs
Proposed changes

Enable L3 In Band ZTP by making the network device upstream of the ZTP a dhcp server that will allocate the appropriate IP address on the connected p2p interface.

How to test

Updated eos_designs_unit_tests for the INBAND_MGMT_TESTS group
Checklist
User Checklist

N/A

Repository Checklist

[x] My code has been rebased from devel before I start
[x] I have read the CONTRIBUTING document.
[x] My change requires a change to the documentation and documentation have been updated accordingly.
[x] I have updated molecule CI testing accordingly. (check the box if not applicable)

Please rebase and resolve conflicts


Googłe Play Games installation not found

Hi, unfortunately I'm still getting the error:
27-12 23:48:17.649|ERROR|GooglePlayGamesLibrary#GooglePlayGamesLibrary:Google Play Games installation not found.
And I don't know how to debug it. I verified that there are keys in the registry:
Počítač\HKEY_LOCAL_MACHINE\SOFTWARE\Google\Play Games
And the files
"C:\Program Files\Google\Play Games\Bootstrapper.exe"
"C:\Program Files\Google\Play Games\current\service\Service.exe"
Thank You.

Google changed their installer after my first installation and broke the updater for earlier installations.
Should be resolved with v0.1.2.


chore(main): release sandworm 1.0.0

:robot: I have created a release beep boop
1.0.0 (2022-09-15)
⚠ BREAKING CHANGES

issue affecting detected caller paths
anonymous event data collection

Features

access denied callback (45ea81a)
anonymous event data collection (4ad3988)

Bug Fixes

cache access performance issue (16ffc24)
issue affecting detected caller paths (d8e6199)
url detect performance issue (ae6b829)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/sandworm-hq/sandworm-js/releases/tag/sandworm-v1.0.0 :sunflower:


Add fail snippet type

Markdown snippets currently do not allow code causing exceptions. However, these are important for documenting how some software works in particular edge-cases. Thus, a fail snippet type (or similar one with a different name) should be implemented, to support faulty code.

Solved in #15


Process tree output looks not totally correct.

Using --ps-tree on a container checkpoint I get following output:
counter
└── [1]  bash
    ├── [7]  bash
    ├── [7]  counter.py
    ├── [8]  bash
    └── [8]  tee

That doesn't look like what I expected. I would expect something like:
counter
└── [1]  bash
    ├── [7]  counter.py
    └── [8]  tee

@snprajwal PTAL

Can you share the process you used for this? This is slightly weird

$ skopeo copy docker://quay.io/adrianreber/checkpoint-test:tag43 oci:/tmp/test1
$ checkpointctl show /tmp/test1/blobs/sha256/65de3f48dccf3b3adde6221a90eb81f322d4367745cb3585aa7da917140415f3  --ps-tree
Process tree

counter
└── [1]  bash
    ├── [8]  bash
    ├── [8]  tee
    ├── [7]  bash
    └── [7]  counter.py

@adrianreber Could you also show the output of crit show pstree.img?

{
    "magic": "PSTREE",
    "entries": [
        {
            "pid": 1,
            "ppid": 0,
            "pgid": 1,
            "sid": 1,
            "threads": [
                1
            ]
        },
        {
            "pid": 7,
            "ppid": 1,
            "pgid": 1,
            "sid": 1,
            "threads": [
                7
            ]
        },
        {
            "pid": 8,
            "ppid": 1,
            "pgid": 1,
            "sid": 1,
            "threads": [
                8
            ]
        }
    ]
}

It would be great if you could also show the output of crit x . ps, ideally with the CRIT binary from go-criu. Since this is what we're using the fetch the info, it will help identify if the issue exists in go-criu or in checkpointctl.

crit/bin/crit x /tmp/forensic/checkpoint/ ps
{
    "pId": 1,
    "pgId": 1,
    "sId": 1,
    "comm": "bash",
    "children": [
        {
            "pId": 7,
            "pgId": 1,
            "sId": 1,
            "comm": "counter.py"
        },
        {
            "pId": 8,
            "pgId": 1,
            "sId": 1,
            "comm": "tee"
        }
    ]
}


Active console while working with notebooks

When using Jupyter Notebook web version, very often I like to do some prototype lines before inserting it in the notebook. Instead of creating a cell to exclude it right away, I much prefer to start a qtconsole linked to my jupyter server and do the prototypes in the console.
Sure I could do the same with Jupyter in VSCode, but it would be great if we have a native console for jupyter in VS.
This is my feature request, an integrated console with jupyter to do code prototypes or even some prints without having to create any cell.

Dupe of #6484


[WIP] Faster Console Logger Mark 1

@davidfowl @glennc @halter73 @rynowak
~35% increase with an app hosted in azure.
Ignore test failures for now.

Added some changes for memory and went from 748,341 bytes for 1000 logs to 307,173 bytes
Will test RPS after lunch

Perf decreased ~2% after the memory changes

🆙 📅

:shipit:


ModLoaderStore: store the Profile Object instead of the profile name in current_user_profile

https://github.com/GodotModding/godot-mod-loader/blob/97eda11fe144dd6c7ccc3b48db6b784e0eaf3b0a/addons/mod_loader/mod_loader_store.gd#L78-L79

closed by #260


Ballerina doesn't give a proper error when there are duplicate variables as function arguments

I have the following code snippet. When compiling, it gives an internal server error.
Affected Version: 0.93
function generateResponseStruct (json response)(VerifyResponsePayload payload) {

    var response, castErr = <VerifyResponsePayload> response.TransferCFCInfoResponse.TransferCFCInfoResult;
    if (castErr != null) {
        logger:error("Error while casting : " + castErr.msg);
        // lets throw an error
    } else {
        logger:info("Successfully Casted the Json to Struct!!");
    }
   // payload = responseM;
    return;
}

Fixed in 0.93.1-SNAPSHOT


Redesign landing page and add more events

Description

Redesign to the new design.
Add past and future events.
Clean project.

Some questions

[x] I read the contributing guidelines
[x] I'm adding a new event/footer link
[x] I'm editing an existing event/footer link
[ ] I'm adding a new feature/fixing a bug

As you requested @casassg, I've commented all the old events (except HackNight Beta until we launch HackNight season). I wanted to create a new page with all these events but I'll do after HackUPC if there's no problem.


feat(userstudy): Added info study program

Description
closes #293
Changes: Added "Informasjonsbehandling" as a study program.
Pull request checklist
Please check if your PR fulfills the following requirements:

[x] The PR includes a closes #issueID
[ ] The PR includes a picture showing visual changes
[ ] Added Google Analytics tracking if relevant
[x] Pull request title follows conventional commits (type(scope): description)
[x] CHANGELOG.md has been updated. Guide

Vil veldig gjerne ha noen mer erfarne frontendere til å se over denne :)

Da har jeg gjort de endringene kommentert over, inkludert å fjerne Drift!


Update Quickstart

[x] use all td layouts
[x] use better mock data
[x] add logos & icons
[x] use stepper in an example form
[x] use expansions & other td components

merged into https://github.com/Teradata/covalent-quickstart


Can I use UVAtlas code in commercial application?

Thanks for this project, and big thanks for MIT license, but isn't UVAtlas usage constrained by patent US7224356? Doesn't that patent makes impossible production usage of UVAtlas?

The original MS-PL license addressed patents, but the preferred MIT license does not. I'll work with our legal team  to see if I can get a patents statement to attach to clarify any concerns here.

Thanks, I found MS-PL revision and that's great.
Maybe you have some rough ETA for clarification?

Sorry for impatience, but I think that it can help - Apache License 2.0 seems to address patents while seems to be similar to MIT. May be this is exactly what is needed?

I've contacted our legal department to get some official guideance on this issue... Thank you for your continued patience.

For reference there is also patent https://patents.google.com/patent/US7425954B2/en


Compatibility Issues with LangevinVelocityIntegrator

I have used the LangevinVelocityIntegrator on a one-dimensional potential, and it outputs its trajectory positions (and velocities, I think) as an array containing two values for each step. While I don't know the implementation reasons or which one of them is the actual position, I have found this leads to issues down the line:

Systems with the LangevinVelocityIntegrator as a sampler fail to work with oneD_simulation_analysis_plot()
The LangevinVelocityIntegrator does not work with replicaExchangeEnvelopingDistributionSampling at least during my testing due to the exchange criterion failing to calculate anything using the two-value array.
This one is more of an incovenience: I have not found a way to conveniently slice one of those positional values out of the trajectory (pandas.DataFrame containing these arrays as objects) without looping (but I am no python expert). The LangevinIntegrator in contrast just contains the positions as floats.
The documentation gives no hint as to what these pairs of values signify and I have not dug far enough into the code to understand, whether one of these values might be a half-step or just the position in the second dimension (which I should not have in my 1D case).

I am not sure whether this deserves its own issue, but since I like swapping out samplers I have found three different ways positions were stored for the three samplers I tried (all of this while doing 1D simulations, I cannot comment on higher dimensions):

LangevinIntegrator: Positions neatly stored as floats, very convenient for plotting and calculations.
LangevinVelocityIntegrator: Positions stored as value pairs in arrays as described above.
MetropolisMonteCarloInterator: Positions stored as single values, but each of them in an array containing only that one value. This makes for some surprises when switching from Langevin to MetropolisMonteCarlo.


How do I run these codes in Jupyter Notebook?

How do I run these codes in Jupyter Notebook?

I've manage to copy each py file into the same Jupyter Notebook ipykernel file. Instead of using the cliargs.py, I've created a new class "Args" as follow:
.
class Args:
places = (f"Penang Kopitiam Kafe Cafe")
query = ""
pages = 10
scrape_website = 1
verbose = 0
.
And I use the above Args as my scrape argument:
.
if name == "main":
args = Args()
scrape(args)
.
It works well. But after I run it, and I open the excel workbook, the workbook only shows the column title. No data elements were added. How shall I do it now?


is there any api can Refresh Extracts a datasource ?

is there any api can Refresh Extracts a datasource ?
I hope there is a python api that can Refresh Extracts a datasource and wait for the Refresh finish then return it's status

sure. I took a look at our sample (refresh.py) and realized that while it is "elegant" python, it might not be super easy to understand. Your code would look something like
tableau_auth = TSC.TableauAuth(args.username, password, args.site)
server = TSC.Server(args.server, use_server_version=True)
with server.auth.sign_in(tableau_auth):
// First you need to get the datasource you want to refresh. There are samples for this
job = server.datasources.refresh(ds)
// Then you can put in some type of loop getting an update of the job and checking its
// finish_code and completed_at values
job = server.jobs.get(job.id)

just want to correct the code, it should be:
resource = server.datasources.get_by_id(datasource_id)
job = server.datasources.refresh(resource)
server.jobs.get_by_id(job.id)

anyone know from where I can get datasource_id in tableau


Centre-align labels on the x-axis

This graph looks silly:

The labels are too far right!
I couldn't find a way to line up the centre of the labels with the point. Is there a way to do this? If not, how tricky would it be to implement? It would be a good feature to have.

I am having an issue with this also. I implemented the code referenced in the JSbin but it appears that the starting point is different for each of my labels, and therefore the "(data.x + data.space) / 2" is not working. Does anyone know how to correct this? It is a problem in both Safari and IE 10. Here is a fiddle with my code: http://jsfiddle.net/e6kjqw3j/1/  Any help is much appreciated!!!

This should no longer be an issue as you can use the fullWidth option on line charts.

My fiddle is not for a line chart, I do not think the width is the issue, but I have added it here to test: http://jsfiddle.net/e6kjqw3j/2/ With the fullWidth in place it only displays one bar chart on safari and chrome, and IE has a horizontal scrollbar with labels still very off place. Are you able to look at this in IE (version/s 9, 10, or 11)? That is where the issue is. It does look better in Safari 8, I think my work computer is Safari 6 (I cannot update it due to restrictions). If you could see it in IE I think you would better see the problem, if not I can post a screenshot.

I'll be able to look at this in older browsers but for a better understanding it would be nice anyway to have some screenshorts.

Of course, here are some screenshots
Without the "(data.x + data.space) / 2" code:
Safari 6

With the "(data.x + data.space) / 2" logic (http://jsfiddle.net/e6kjqw3j/1/):
Safari 6

IE 10

This issue seems to be specific to browsers using the "text" XML node versus the foreignObject/span nodes. While Safari 6 is outdated, IE 11 is the current standard and looks the same as IE 9 & 10. I am hoping this is an easy fix... I do think that correcting the logic for the "text" node XML will correct it on all of these browsers.

Hey Christina. This issue is fairly old and some things have changed also in the API. Unfortunately even IE11 does not support foreignObject while all other modern browsers do :cry: You can also see this from the Chartist browser compatibility list: http://gionkunz.github.io/chartist-js/#browser-compatibility
You should definitely update to the latest Chartist version 0.7.2 since you're still running with version 0.5 :scream_cat:
For modern browsers that do support the extensibility specification of SVG (actually the foreignObjects), you can simply use CSS to center your span labels as they have the full segment width. This will work just fine:
.ct-chart .ct-label.ct-horizontal {
  text-align: center;
}

You could then implement a fallback for non-supportive browsers and use the text anchor and position offset like this:
var chart = new Chartist.Bar('.ct-chart', {
  labels: ['Test', 'Long test', 'Very long test'],
  series: [[1, 5, 3]]
});

// Uncomment this line to test rendering without foreign objects
//chart.supportsForeignObject = false;

chart.on('draw', function(event) {
  // If the draw event is for labels on the x-axis
  if(event.type === 'label' && event.axis.units.pos === 'x') {
    // If foreign object is NOT supported, we need to fallback to text-anchor and event.width / 2 offset.
    if(!chart.supportsForeignObject) {
      event.element.attr({
        x: event.x + event.width / 2,
        'text-anchor': 'middle'
      });
    }
  }
});

Check this example on JSBin: http://jsbin.com/pujoxe/1/edit?css,js,output

Also a side note while looking at your code. For static offset like the one for your Y-Axis label you best use the axisY.labelOffset.x configuration property.

Gion, thank you so much for your quick reply. I see that your jsbin is working in IE, and I appreciate being able to see how to use the "text-anchor" correctly. You are right, I do need to update my chartist as your fallback draw snippet is not working with 0.5.0. When I updated it to 0.7.2, a few of my other charts got messed up and unfortunately I am in the midst of a job wildfire and can't trouble-shoot it this time around. I will try this approach with the next set of charts that I create. Also thank you for the tip about the axisY.labelOffset.x property, I am definitely still learning this library :) Thanks again for helping me with this!!

For reference, I was able to center labels in a line chart by using the event code gionkunz posted in the JSBin, but it took me forever to get the text to center in the  elements. I had to set text-align: center; display: inline-block; in the CSS. Not sure why it appears to work fine in that JSBin, but in my graph the text within the spans wouldn't center without the inline-block:

I'm working on 0.9.4 and was having trouble getting this to work. text-anchor: middle had no effect on my chart. I ended up using.
.ct-label.ct-horizontal.ct-end {
	transform: translate(-50%,0);
	display: block;
	text-align: center;
}

Make sure you properly browser prefix transform to work in iOS (-webkit-transform).

I've been trying to achieve the same thing as the OP, but the only thing that worked for me was the most recent suggestion by @unhinged , the problem with that is that the axis do not scale/move when the browser zoom function is used. I tried changing both the text-anchor properties for horizontal labels in the chartist scss and recompiling but nothing seemed to change. Is there any way to achieve this and retain zoom scaling? Thanks

The text-anchor property had no effect. Even when I edited the existing text-anchor property in inspector, it had no effect on the graph.
After fiddling around in inspector I figured out that a combination of display: block, text-align center did the trick as unhinged suggested.

.ct-label.ct-horizontal.ct-end {
//    transform: translate(-50%,0);
    display: block;
    text-align: center;
}


partial fc with triplet loss

can fast-reid support trainning with partial fc + triplet loss?
could you provide a yaml?

Sorry, it cannot support partial fc + triplet loss right now.


kprocesshacker cannot be loaded on Insider Builds

Describe the bug
When booting up a recent Insider Build of Windows 11, a warning message appears, saying the driver cannot load because it was detected as a vulnerable driver.
On top, when opening Process Hacker, the + is missing from the window's title, meaning the driver is not available.
To Reproduce
Steps to reproduce the behavior:

Join the Insider Dev Ring
Upgrade to build 22509.1011
Turn on Core Integrity from the Windows Defender settings.
Reboot
See error

Expected behavior
The driver should not be blocked by Windows.
Screenshots

Desktop (please complete the following information):

OS: Windows 11
Version 22509.1011

Additional context
This might require communication with Microsoft to figure what lead to it being blocked, resolve the issue, and then getting it unblocked from the OS.

Open some previous closed issues. You'll get the idea.

This is different. While the previous issue was an issue with their computer, this is the OS explicitly blocking the kernel mode driver.

Seems like a duplicate of #997 (I tried looking through the issue tracker before filling an issue, however my search didn't bring up this issue), but I ran Windows 10 and 11 release builds with core isolation active and the driver would not get blocked, so it's weird it only got blocked upon upgrading to Insider builds...


Switch to MiniTest or Upgrade Test::Unit

Bane is currently using the Test::Unit version included with Ruby 1.8.7.
Let's upgrade either to a more modern version of Test::Unit or MiniTest.

Upgraded today to support Ruby 2.x and used installed version of test-unit present in Ruby 2.7.
We may still choose to replace test-unit with Minitest, but for now, this is done.


This plugins breaks "Styler" plugin

This plugins resets highlight when switching between windows, and that breaks Styler plugin https://github.com/folke/styler.nvim

Styler allows setting a colorscheme per filetype and even per buffer. It does so by making use of vim.api.nvim_set_hl, but modes resets that...
Could you make this plugin work with Styler ?

Hmm I'm not sure about resetting, but styler mentioned to use vim.api.nvim_set_hl which the current main branch does not. You could try our canary branch which I believe uses that exclusively but there are other bugs with that branch. Could at least narrow down the problem, though.


add value padding for non text values

Text values can be padded by adding blanks before and after the string
but numerical values can not they are converted to float or int.
So this adds the ability to pad the value field the same way as the
label field can be padded.
Signed-off-by: Robert Marklund<EMAIL_ADDRESS>

@trollkarlen I have put out a new draft 1.7.0rc1 release.  This addresses the challenges your're seeing.  It has some updates to the width calculations, which was the main problem affecting the padding around labels and values.  It also introduces num_label_padding_chars and num_value_padding_chars arguments for the control that I believe you wanted.
I'm going to close this PR since it shouldn't be needed based on this new release.

Perfekt!
Ill try it out.
//T
On Sun, Mar 15, 2020, 18:05 Jon Grace-Cox<EMAIL_ADDRESS>wrote:

@trollkarlen https://github.com/trollkarlen I have put out a new draft 1.7.0rc1
release https://github.com/jongracecox/anybadge/releases/tag/v1.7.0rc1.
This addresses the challenges your're seeing. It has some updates to the
width calculations, which was the main problem affecting the padding around
labels and values. It also introduces num_label_padding_chars and
num_value_padding_chars arguments for the control that I believe you
wanted.
I'm going to close this PR since it shouldn't be needed based on this new
release.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/jongracecox/anybadge/pull/36#issuecomment-599237388,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAWIHCOO6ASG7OFN7RT5NLRHUDHNANCNFSM4KGCQZDA
.

I got some time to test fully, and I've released 1.7.0 so you should be able to try that out.
Here's a screenshot from the repl.it demo using the new version:


engine: deprecate gc-merge-rewrite option

Signed-off-by: tabokie<EMAIL_ADDRESS>What is changed and how it works?
Issue Number: Close #12797
What's Changed:
Deprecate gc-merge-rewrite option.

Related changes

PR to update pingcap/docs/pingcap/docs-cn:

Check List
Tests

No code

Release note
Deprecate gc-merge-rewrite option.

/merge


Support a large number of plug-ins

Is your feature request related to a problem? Please describe.

In order to make APM even more useful, we will need to support a lot of famous plugins.
Describe the solution you'd like

The work includes finding, validating, and entering data for the plugins. @hal-shu-sato Is it possible to maintain the list of plugins as a Github repository to do these things?
Describe alternatives you've considered

Additional context

I think so too.
Rather, I was thinking about what to do about it, and forgot about it.
I was kind of thinking of a wiki-style thing, but I was worried about it being vandalized.
Surely with GitHub, it can be controlled by PR.
I'll prepare the repository.

Thanks for the reply. It helps a lot.

I made it.
https://github.com/hal-shu-sato/apm-data

Thank you for creating the repository. I would like to start adding plugins.
Close this issue in favor of the solution.


Limit the input field length

What should be added?
Option to limit the input field length
Why should it be added?
TMI doesn't allow to send messages longer than 500 characters, so it'll be convenient to have not only characters count, but also an option to limit the input field length.

Sometimes I paste a message that's more than 500 chars, I see that and split it up to two messages instead. You wouldn't be able to do this if the max length of the input field was 500 chars.
Something like highlight characters over the max length of 500 chars would be better imo.

If you are talking about copypasting messages, i think it can be cut at 500 mark.

I'm not talking about copypasting messages. More about copying info from some website and posting it in chat.

I see. My idea was that you could paste info longer, but cut it to 500 characters. Would you think implementing both will be a good idea?

The issue is that if you aren't mod/vip/broadcaster/etc... you can't post message that are longer than 250 chars.
It could still work if you remove either 250 or 500 chars after you press enter.

If you change this issue or create a new one with splitting messages bigger than twitch accepts (or similar) I'll leave that open.

The issue is that if you aren't mod/vip/broadcaster/etc... you can't post message that are longer than 250 chars.

Oh, i didn't know that. I've tested, without some user-type i was able to get to 270 characters separating words with spaces (216 characters without spaces) and when i tried to send a wall of text without any spaces, i was limited to only 49 characters for some reason.

I wouldn't say that it's dumb, considering that you could drop in a larger text and it will be cut at a needed mark, and you wouldn't be able to write more characters, imo it's always better to not allow to do something (if it's restricted), than showing that you are not supposed to do something. But that's fine because implementing that would be a total shitshow because of how inconsistent twitch is.


Latest android update brought back drm protection

Not sure what happened but since I updated my android device, DRM is now active again on my streaming service, any chance to update what was broken to bring it back? specifically One UI 6.1.1 (Galaxy s23)

Android >= 12? #2129 #36

Yes. But it used to work now it doesnt, I tried using a phone with previous android version and still no go. Something changed.. it worked for weeks now..


Add Prefer not to answer as an option for dropdowns

Thanks for creating an issue! Please fill out this form so we can be sure to have all the information we need, and to minimize back and forth.

What are we trying to do?

'Pass' is an eminently reasonable answer to most data collection, but especially for abortion funds! Funds would like a way to denote that someone asked this question.

What feature or behavior is this required for?

Patient safety; funds not asking the same questions over and over

How could we solve this issue? (Not knowing is okay!)

Let's add a 'Prefer not to answer' to the following dropdowns in the patient record:

[x] Employment status
[x] Income
[x] Race/Ethnicity
[x] Patient insurance
[x] How they heard about DCAF
[ ] Adults in household
[ ] Children in household

A minor x-factor: Adults and children in household are stored in db as integers, so you'll need to write a migration to change them to strings.

Anything else?

(doing this because I want it done)


Do not call union_spans for empty tuple

Fix #8

Hi Lasse, thanks!
Though I think calling union_span with zero spans should return the Dummy range, so I'd rather fix that function directly.
(and that removes a call to an exception-yielding function, which always feels nice!)

just pushed a quick commit that patches the function directly. Closing this PR thus.


Fix: Do not add an undefined className to a DOM Element

Styled elements with className={undefined} do not add a class "undefined" to the DOM element anymore.

resolves #537

Merge it please, really need it! 😄

Thanks for the contribution 🙏
New features should ideally be accompanied by a test case, to prevent future regressions.
Would you mind adding one? Thanks!


suppresses thrown errors

Code that I'm testing throws an error.  This error does not appear in the test output, making it difficult to determine why the test failed.  It would be much more helpful if mocha always listed the stacktrace of any errors thrown during a test.
This test (along with others):
it ("creates the top-level object", function () {
var result = fillObjectTree.fillObjectTree (dataSet1, example);
result.id.should.equal (1001);
});
Produces:
✖ 17 of 58 tests failed:

fill_object_tree createMap fillObjectTree creates the top-level object:

fill_object_tree createMap fillObjectTree creates the measurableQuantity:

Note the lack of detail under the failed test.
If I wrap the test code in a try/catch, it outputs the error in the console output:
it ("creates the top-level object", function () {
	try {
	var result = fillObjectTree.fillObjectTree (dataSet1, example);
	} catch (e) {
	    log.error(e);
	}
	result.id.should.equal (1001);
    });

ERROR The provided object to fill (object) does not match the example object map's type (array) at path [.measurableQuantity.unitConversions], Error
at Object.error (/Users/jacobmiles/projects/sefaira-svn/trunk/node_app/lib/log.js:23:64)
at Test.fn (/Users/jacobmiles/projects/sefaira-svn/trunk/node_app/lib/fill_object_tree.spec.js:346:11)
at Test.run (/opt/local/lib/node/.npm/mocha/0.10.0/package/lib/runnable.js:153:3
And provides more output under that test result:

fill_object_tree createMap fillObjectTree creates the top-level object:
TypeError: Cannot read property 'id' of undefined
at Test.fn (/Users/jacobmiles/projects/sefaira-svn/trunk/node_app/lib/fill_object_tree.spec.js:348:9)
at Test.run (/opt/local/lib/node/.npm/mocha/0.10.0/package/lib/runnable.js:153:32)
at Runner.runTest (/opt/local/lib/node/.npm/mocha/0.10.0/package/lib/runner.js:271:10)
at /opt/local/lib/node/.npm/mocha/0.10.0/package/lib/runner.js:315:12
at next (/opt/local/lib/node/.npm/mocha/0.10.0/package/lib/runner.js:199:14)
at /opt/local/lib/node/.npm/mocha/0.10.0/package/lib/runner.js:208:7
at next (/opt/local/lib/node/.npm/mocha/0.10.0/package/lib/runner.js:157:23)
at Array. (/opt/local/lib/node/.npm/mocha/0.10.0/package/lib/runner.js:176:5)
at EventEmitter._tickCallback (node.js:126:26)

mocha should display exceptions thrown, not if you try/catch obviously but otherwise it will, im not sure I get what the issue is you're having, is there something I can reproduce?

Yeah, I'm unable to repro this - I've changed a number of things and something along the way fixed this problem.  Part of the problem may have been that in some cases I was throwing strings instead of Error objects.  If you're printing e.stack, that would of course produce no output for a string.


[Autocomplete] Input in Autocomplete has width=0

[x] The issue is present in the latest release.
[x] I have searched the issues of this repository and believe that this is not a duplicate.

Current Behavior 😯
When fullWidth of the input component to Autocomplete is turned off, the input component has width set to 0, resulting in a very short input field that is unreadable.
https://github.com/mui-org/material-ui/blame/4bfc2e48be1a33bd4994b4bf444aa4264f378ae6/packages/material-ui-lab/src/Autocomplete/Autocomplete.js#L47
Expected Behavior 🤔
When fullWidth of the input component to Autocomplete is turned off, the input component should use auto width.
Steps to Reproduce 🕹
The Combo Box demo on the Material UI website, without a fullWidth parameter.
https://codesandbox.io/s/material-demo-0ute2
Context 🔦
We have several components in our app that relies on the auto width sizing for an input field, instead of a predefined width. Currently, none of those fields are readable for the end user.
Your Environment 🌎

Tech
Version

Material-UI lab
v4.0.0-alpha.39

React
16.12.0

Browser
Chrome

@ldezhenkong Thanks for the feedback and the reproduction. It seems that the following diff is enough to solve the problem:
diff --git a/packages/material-ui-lab/src/Autocomplete/Autocomplete.js b/packages/material-ui-lab/src/Autocomplete/Autocomplete.js
index eebefc730..07af5b541 100644
--- a/packages/material-ui-lab/src/Autocomplete/Autocomplete.js
+++ b/packages/material-ui-lab/src/Autocomplete/Autocomplete.js
@@ -37,7 +37,7 @@ export const styles = theme => ({
     flexWrap: 'wrap',
     paddingRight: 62,
     '& $input': {
-      width: 0,
+      width: 'auto',
       minWidth: 30,
     },
     '&[class*="MuiInput-root"]': {

What do you think? Do you want to check that out and submit a pull request? Thanks

It seems that there is no good alternatives, you will need to set the fullWidth prop.


Fix bios boot python script due to 2.0.x changes

Change Description
Script was modified to:

Activate PREACTIVATE_FEATURE first
Deploy eosio.system version 1.8.0
Add all protocol features, including those introduced by 2.0.x
Deploy eosio.system version 1.9.0

Updated README.md file with steps to build the eosio.contracts older version as well.
Resolves #8261

[x] Documentation Updates

I assume this is no longer draft.

I assume this is no longer draft.

so vigilentè, me like it!

@b1bart Can you weigh in on the subject?


testing adding feature request

Contact Details
<EMAIL_ADDRESS>Describe the solution you'd like
testing adding another feature

Additional information
testing adding feature request

Testing answering


Add support for publishing to Cloud Pub/Sub

Goals:

Allow producing applications to publish messages to Google Cloud Pub/Sub
Producing applications can specify rabbitmq or google cloud pubsub adapters

Questions:
What if an applications wants to publish some messages to rabbit and some to cloud pub/sub, can the configuration be extended to support multiple adapters and not one or the other?  Then the message could specify which adapter?

Design goal for producing applications
Emque::Producing.configure do |c|
  c.app_name = ENV["MESSAGE_QUEUE_APP_NAME"]
  c.rabbitmq_options[:url] = ENV["RABBITMQ_URL"]
  c.google_cloud_pubsub_options[:project_id] = ENV["GOOGLE_CLOUD_PUBSUB_PROJECT"]
  c.google_cloud_pubsub_options[:credentials] = ENV["GOOGLE_CLOUD_PUBSUB_CREDENTIALS"]
end
MemberCreatedMessage.new(:user_id => 1).publish([:google_cloud_pubsub, :rabbitmq, :logger])

[1] pry(main)> MemberCreatedMessage.new(:user_id => 1).publish([:logger])
2020-02-07T02:11:10.980Z 1593 TID-gp8z0g7m9 INFO: MESSAGE LOG: publishing... {"metadata":{"host":"ccf5fc79fa97","app":"apiv3","topic":"members","created_at":"2020-02-07T02:11:10Z","uuid":"e65b8639-87e9-4a71-a728-9f64e90a04c8","type":"members.created"},"change_set":{"original":{},"updated":{},"delta":"_created"},"user_id":1,"team_id":null,"source":null}
2020-02-07T02:11:10.980Z 1593 TID-gp8z0g7m9 INFO: MESSAGE LOG: valid... {"metadata":{"host":"ccf5fc79fa97","app":"apiv3","topic":"members","created_at":"2020-02-07T02:11:10Z","uuid":"a9f066c5-8ee5-468a-a4c5-e6ffa7900f90","type":"members.created"},"change_set":{"original":{},"updated":{},"delta":"_created"},"user_id":1,"team_id":null,"source":null}
NoMethodError: undefined method `publish' for #<Logger:0x000055b44a112f70>
from /snapi_bundle/bundler/gems/emque-producing-5d424b6b29a5/lib/emque/producing/message/message.rb:140:in `block in publish'

I forget, is there a way to publish to logger only today?

I forget, is there a way to publish to logger only today?

Not logger only. It will log messages if you have the publish_messages config turned on. But if you turn it off, no log at all. Thinking about how to fix that here.


ci: fix and update dependencies

Support for Python versions 2.7 and 3.5 were dropped by the Github action. See https://github.com/actions/setup-python/issues/672 for more info.

@P403n1x87 we need to allow the docker python images in the repo settings:
docker://${{ matrix.python-image }} is not allowed to be used in DataDog/envier. Actions in this workflow must be: within a repository that belongs to your Enterprise account, created by GitHub, or matching the following: pypa/gh-action-pypi-publish@master.

The current ddtrace usage requirement is still 2.7, right?

Yes but we could pin the envier dependency on 1.x (I'm assuming the new features would only be needed starting with 2.x).

Can we get the existing CI passing before making the separate decision of dropping support for the other Python versions? Python 2 is currently supported by envier and the CI isn't working.

Done, but the workflow file seems to be invalid

Thanks @P403n1x87! I'll look into it, there likely will be a problem with riot since it can't run in the Python 2 image. Maybe we can reuse the ddtrace dev image with all the Python versions baked in.


Create a body channel implementation that knows when buf is written

It is very common to want a body type that allows for a push-based API, and knowledge of when those bytes have been written to the socket. We can provide a channel type that wraps the user's B: Buf in one that alerts the channel when Buf::advance has reached the end.

At work we run into this issue, where we wanted to get callback on body chunks and then on end of streaming.
We implemented custom Stream wrappers for both cases and wrapped them with Body::wrap_stream. I think it suffers from additional heap allocations, as hyper boxes the stream you pass.
Either way, I'd be more than happy to tackle this. One think I'd like to sort out first, is do we go with alerting via channel (if so, can you elaborate a bit more how you see it), or providing a callback the way we did.
Also, what should be sent by the channel/available in the callback. We did &Bytes so that we could call chunk.len().

I think there's 2 parts to figure out, the easier one is the Buf wrapper:
struct Alert<B: Buf> {
    inner: B,
    signal: SignalThingy,
}

impl<B: Buf> Buf for Alert {
    fn advance(&mut self, cnt: usize) {
        self.inner.advance(cnt);
        if !self.inner.has_remaining() {
            self.signal.thingy();
        }
    }
}

The harder part is what is the signal thingy? Is it like a tokio::sync::Notify, or similar kind of channel-looking thing?

Alright, there is a fundamental flaw in the bytes::Buf design that makes it impossible to implement this feature.
There is no method that will guarantee that the buffer was actually consumed. One is allowed to call Buf::chunk just for peaking, so the logic to trigger notification can't go there. We are not guaranteed that somebody will call buf.advance(0) after sending the whole buffer.
Relying on a call to Buf::remaining() that returns 0 might be possible, but still doesn't guarantee anything, as one could do:
let buf = Bytes::from_static(b"abc");
loop {
    let chunk_size = buf.chunk().len();
    send(buf.chunk());
    if chunk_size == buf.remaining() {
        break;
    }
}

Notice that buf.remaining() is not guaranteed to return 0 (actually, it will only do so for empty buffer).
I could write such snippets for each of Buf methods and prove that none of them give us a definite EOS.
We would require something like:
trait ConsumingBuf: Buf {
    fn next_chunk(&mut self) -> &[u8] {
        let chunk = self.chunk();
        self.advance(chunk.len());
        if self.remaining() == 0 {
            self.signal.thingy()
        }
    }
}

(where Buf::chunk is private) to guarantee we notify if-and-only-if buffer was consumed.

It's true there's no guarantee, but hyper does make sure to always advance to the end. The thing with readiness-based IO is that we need to peek, because there might not be enough room in the socket's write buffer to send it all.


sc-7101 GDS USER UI: implement multi-user ui

Scope of changes
Type of change

[ ] bug fix
[x] new feature
[ ] documentation
[ ] other (describe)

Acceptance criteria
https://app.shortcut.com/rotational/story/7101/gds-user-ui-turn-multi-user-figma-ui-to-react-components
You can test the component by opening storybook

Author checklist

[x] I have manually tested the change and/or added automation in the form of unit tests or integration tests
[ ]  I have updated the dependencies list
[ ]  I have recompiled and included new protocol buffers to reflect changes I made
[ ]  I have added new test fixtures as needed to support added tests
[x]   Check this box if a reviewer can merge this pull request after approval (leave it unchecked if you want to do it yourself)
[x]  I have moved the associated Shortcut story to "Ready for Review"

Reviewer(s) checklist

[ ] Any new user-facing content that has been added for this PR has been QA'ed to ensure correct grammar, spelling, and understandability.

Codecov Report
Base: 44.23% // Head: 47.12% // Increases project coverage by +2.89% :tada:

Coverage data is based on head (e8c00ac) compared to base (e16facc).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #796      +/-   ##
==========================================
+ Coverage   44.23%   47.12%   +2.89%
==========================================
  Files         605      263     -342
  Lines       22481    19317    -3164
  Branches     1373      619     -754
==========================================
- Hits         9945     9104     -841
+ Misses      11258     8947    -2311
+ Partials     1278     1266      -12

Flag
Coverage Δ

unittests
47.12% <ø> (+2.89%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

...icateRegistration/CertificateRegistrationTable.tsx

web/gds-user-ui/src/components/ui/Button.tsx

...ateManagement/X509IdentityCertificateInventory.tsx

...ui/src/components/Footer/LandingFooter.stories.tsx

...-user-ui/src/modules/dashboard/overview/service.ts

web/gds-user-ui/src/components/Section/Line.tsx

web/gds-user-ui/src/locales/zh/messages.js

...rc/components/Modal/ConfirmationResetFormModal.tsx

web/gds-user-ui/src/hooks/useCustomAuth0.ts

...gds-user-ui/src/components/ErrorFallback/index.tsx

... and 332 more

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

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Customizing Async Call Return Type

I have the following api endpoint:
@ApiOperation(value="Creates a model.")
@RequestMapping(value = "/clients/{clientId}/model", method = RequestMethod.POST)
@ResponseStatus(HttpStatus.CREATED)
public DeferredResult createModel(
@ApiParam(required=true, name="clientId", value="The id for the client accessing the api.")
@PathVariable("clientId") final long clientId,
@ApiParam(required=true, name="model", value="The model to create.")
@RequestBody @Valid final ModelDTO model) throws MyException {
final DeferredResult deferredResult = new DeferredResult();
saveModel(clientId, model, deferredResult);
return deferredResult;
}
Swagger-ui is showing the response model as
DeferredResult«ModelDTO» {
result (object, optional),
setOrExpired (boolean, optional)
}
Is there any way to override return types so that instead of DeferredResult I can have "ModelDTO", even though my endpoints are all asynchronous?

Gah... This is a duplicate of #346.  Thanks.


How to generate multiple pieces of data in one json file instead of one line of data per file

PR #52 actually adds this functionality but I haven't have time to merge it after I had asked for some changes.  Take a look at that and see if it does what you need.


sphinx-book-theme causes sphinx-build to crash

Describe the bug
sphinx-build -b html build/rst build/html
Running Sphinx v5.0.2
loading pickled environment... done
building [mo]: targets for 0 po files that are out of date
building [html]: targets for 2 source files that are out of date
updating environment: 0 added, 2 changed, 0 removed
reading sources... [100%] index
looking for now-outdated files... none found
pickling environment... done
checking consistency... done
preparing documents... done
/home/bradbell/.local/lib/python3.11/site-packages/pydata_sphinx_theme/bootstrap_html_translator.py:27: RemovedInSphinx60Warning: generate_targets_for_table() is deprecated
self.generate_targets_for_table(node)
Exception occurred:
File "/home/bradbell/.local/lib/python3.11/site-packages/pydata_sphinx_theme/bootstrap_html_translator.py", line 29, in visit_table
self._table_row_index = 0
^^^^^^^^^^^^^^^^^^^^^
AttributeError: property '_table_row_index' of 'BootstrapHTML5Translator' object has no setter
The full traceback has been saved in /tmp/sphinx-err-tiy3mvim.log, if you want to report the issue to the developers.
Please also report this if it was a user error, so that a better error message can be provided next time.
A bug report can be filed in the tracker at https://github.com/sphinx-doc/sphinx/issues. Thanks!
Reproduce the bug
Execute the following bash script (note that if you change sphinx_book_theme to sphinx_rtd_theme, the build works without error.
#! /usr/bin/env bash
set -e
dir='build/rst'
#
if [ -e $dir ]
then
   rm -r $dir
fi
mkdir -p $dir

#
# conf.py
cat << EOF > $dir/conf.py
html_theme = "sphinx_book_theme"
EOF
#
# example.rst
cat << EOF > $dir/example.rst
Examples
########

.. csv-table::
   :header:  "Name", "Title"
   :widths: auto

   name,title
EOF
#
# index.rst
cat << EOF > $dir/index.rst
xrst
####

.. toctree::

   example
EOF
#
echo "sphinx-build -b html $dir build/html"
sphinx-build -b html $dir build/html

List your environment
cppad.git>uname -a
Linux fedora 6.1.8-200.fc37.x86_64 #1 SMP PREEMPT_DYNAMIC Tue Jan 24 20:32:16 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

dnf info python3-sphinx
Last metadata expiration check: 4:30:00 ago on Sun 19 Feb 2023 01:26:09 AM MST.
Installed Packages
Name         : python3-sphinx
Epoch        : 1
Version      : 5.0.2
Release      : 2.fc37
Architecture : noarch
Size         : 11 M
Source       : python-sphinx-5.0.2-2.fc37.src.rpm

pip show sphinx-book-theme
Name: sphinx-book-theme
Version: 0.0.39
Summary: Jupyter Book: Create an online book with Jupyter Notebooks
Home-page: https://jupyterbook.org/
Author: Project Jupyter Contributors
Author-email<EMAIL_ADDRESS>License: BSD
Location: /home/bradbell/.local/lib/python3.11/site-packages
Requires: beautifulsoup4, click, docutils, pydata-sphinx-theme, pyyaml, sphinx
Required-by: xrst

This should be fixed in the latest pre-release, give it a shot here:

https://github.com/executablebooks/sphinx-book-theme/issues/686

I'll close but please re-post if it's still broken w/ the latest versoin

Works for me.


valset: Don't include SimulateActiveValidators by default

Closes #75

Is this the same as https://github.com/confio/poe-contracts/pull/77 ?

Is this the same as #77 ?

Yes, it is. Working on it, as I don't see a clear path forward for this yet.
Perhaps remove this entry point entirely, and revisit later?

Is this the same as #77 ?

Yes, it is.

Ah, okay! Happy to close this one then.


Unable to bring up grails application - Grails 3.3.0 RC1

Unable to start the grails application with spring-security-core plugin 3.2.0.M1 and spring-security-ui 3.0.2 .
build.gradle has the following:
compile 'org.grails.plugins:spring-security-ui:3.0.2'
compile 'org.grails.plugins:spring-security-core:3.2.0.M1'
Caused by: java.lang.ClassNotFoundException: org.springframework.boot.context.embedded.FilterRegistrationBean

Thanks for reporting an issue.
Version 3.1.1 is compatible with Grails 3.3.x
compile 'org.grails.plugins:spring-security-ui:3.1.1'
Please, reopen the issue if you are still experiencing issues.


⚠ Bofrost.fr has degraded performance

In b6237b5, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1721 ms

Resolved: Bofrost.fr performance has improved in a6c7a9e after 6 minutes.


local folder is only the internal memory

while playing around with tfiletransfer (on android) against tfiletransfer_desktop I discovered that the topmost level is the level of the internal memory (only), while e.g. a file browser shows an level above,which can select internal and external memory, then.
Because the external memory is often the bigger one, is it possible take tfiletransporter the ability (maybe switchable) to go one level up?

I wiil add this feature in next version.

v2.2.0 already support SD Card's files share.

It works! Like you said they both have to be v2.2.0!
Thank you!!!

😊

Tans5, do you think you could add external as home folder as well? I dont want to open new issue!


[Bug] : Replaced entity renderer doesnt provide the head pitch/yaw

As the title says the head pitch and yaw aren't provided/set in the EntityModelData for a replaced entity renderer even when they are already calculated.
As you can see in the normal renderer its added https://github.com/bernie-g/geckolib/blob/1.18/src/main/java/software/bernie/geckolib3/renderers/geo/GeoEntityRenderer.java#L131
But in the Replaced Entity Renderer it doesn't add them even when they are calculated: https://github.com/bernie-g/geckolib/blob/1.18/src/main/java/software/bernie/geckolib3/renderers/geo/GeoReplacedEntityRenderer.java#L140
In the replaced entity renderer f2 would be netHeadYaw and f6 would be headPitch

Fixed in newest version for 1.16.5 and 1.18.1 Forge/Fabric. 1.12.2 already had it


tsd does not update 'tsd.json' if it is executed via npm script

I defined the following 'shortcut' in the package.json:
"scripts": {
    "tsd": "tsd",
    "test": "gulp test"
  },

If I wanted to execute the following command:
npm run tsd install mocha --save
the typedef of mocha will be downloaded but the installed object in the tsd.json remains empty.
"installed": {}

If the tsd is installed globally and I execute the same command (without npm run of course) it works as expected. Typedefinition of the package is in the typings folder and the tsd.json is up-to-date.
tsd install mocha --save
"installed": {
    "mocha/mocha.d.ts": {
      "commit": "efd40e67ff323f7147651bdbef03c03ead7b1675"
    }
  }

I think it would be very useful if the tsd could be executed with the npm run

You need to use npm run tsd -- install mocha --save.
The NPM run-script command requires you to put a -- between the script name (and NPM args) and the script args.

Closing this as resolved. Please follow @janslow's comment to use arguments with an NPM script.


Reconstruction loss for Parametric UMAP

Thanks for the wonderful addition of Parametric UMAP to this repo.
I was trying to get this to work on my own dataset which has grayscale images using an autoencoder setup and noticed that the reconstruction loss was not changing. I was looking through the repo and the code to see if I could understand why and I noticed that the reconstruction loss is currently BCE.
https://github.com/lmcinnes/umap/blob/b1223505ca56ae104feb35e4196227277d1e8058/umap/parametric_umap.py#L299
Is there an option to make this MSE? Perhaps I am not understanding how the decoder works.
@timsainb

Hi ayl,
I agree, it would be reasonable to an option for MSE instead of BinaryCrossentropy for reconstruction. There should also be a weight to balance UMAP vs reconstruction loss (right now it's hardcoded at 1.0 loss_weights["reconstruction"] = 1.0 https://github.com/lmcinnes/umap/blob/246c4d010ee3526fa5a43926947c6ae0f64929ab/umap/parametric_umap.py#L302).
Do you want to make a PR?


Add advanced data validation function

Description
This pull request introduces an advanced data validation function to the Koheesio framework. The new functionality allows users to validate datasets by checking for required columns and ensuring the correct data types. It enhances data pipeline robustness by identifying and reporting missing or incorrectly typed columns.
Key Features
ValidationConfig: A configurable class using pydantic to define required columns and expected data types.
validate_data Function: Validates datasets to ensure they meet configuration requirements, raising detailed errors for issues.
Unit Tests: Comprehensive test cases added to verify validation logic, including scenarios with missing columns and incorrect data types.
Types of changes
Added validate.py under src/koheesio to implement the validation logic.
Created test_validate.py under tests to test the new functionality.
Updated README.md to document the usage of the validation function.

There has not been any response for the last 2 weeks. Please respond or address the aformentioned concerns before the end of the week.


Coverage report from all files

It looks like to coverage generated with this plugin comes only from files that were tested. Is there a way to have a coverage report from the whole project (include files that were not touched by storybook)?
I'm using https://storybook.js.org/addons/@storybook/test-runner to run tests and collect coverage

I don't think this issue belongs to @storybook/addon-coverage, but instead to storybookjs/test-runner. This plugin only instruments the code. It would be the task of the test-runner, though, to deliver a coverage report for all files. WDYT @yannbf

Great point. Moved!


AzureFunctionApp@1 deployment 503 & 404 Errors

Required Information
Type: Bug
Enter Task Name: AzureFunctionApp@1
Environment

Server - Azure Pipelines
Agent - Hosted VS 2017

Issue Description
When I try to create a Linux Function App (Consumption Plan) via an ARM template and then subsequently update it via the Azure Function (Preview) v1.* task, I get one of the following errors:
Initially, for a while (10 minutes) after the function app is created, I get a 503.
Task Logs
2019-03-21T14:22:07.4592940Z<EMAIL_ADDRESS>2019-03-21T14:22:07.9622530Z ##[debug]Could not parse response: {}
2019-03-21T14:22:07.9623790Z ##[debug]Response: undefined
2019-03-21T14:22:07.9624573Z ##[debug]updateDeployment. Data: {"statusCode":503,"statusMessage":"Site Unavailable","headers":{"content-type":"text/html","server":"Microsoft-IIS/10.0","date":"Thu, 21 Mar 2019 14:22:07 GMT","connection":"close","content-length":"27"},"body":"The service is unavailable."}
2019-03-21T14:22:07.9627088Z ##[warning]Error: Failed to update deployment history. Error: Site Unavailable (CODE: 503)

If I try the release after ~10 minutes, I get a 404:
Task Logs
2019-03-21T15:42:04.4219019Z<EMAIL_ADDRESS>2019-03-21T15:42:04.7494352Z ##[debug]updateDeployment. Data: {"statusCode":404,"statusMessage":"Not Found","headers":{"server":"Kestrel","date":"Thu, 21 Mar 2019 15:42:04 GMT","connection":"close","content-length":"0"}}

@damienwebdev , currently, the support for functions on Linux consumption plan is not available in Azure DevOps. Recently, we added the feature and it wil be available in ~4-5 weeks.

The support has been added. please feel free to try the feature and share your feedback.


Download folder is being stripped at the wrong period.

The auto download of the small model en-us (http://alphacephei.com/vosk/models/vosk-model-small-en-us-0.15.zip) is wrongly be unpacked and saved as;
/home/ovos/.local/share/vosk/vosk-model-small-en-us-0
It strips the last two digits and period as it being the file extension which it isn't
Similar as; https://github.com/OpenVoiceOS/ovos-stt-plugin-vosk/pull/10

fixed by #15


Update 20230317

Collection PR
Fix #160
Fix #284
Fix #274
Fix #275
Fix #282

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

GitHub release
v6.10.0

Your semantic-release bot :package::rocket:


Vue router adds trailing slash if base config provided

Version
4.1.6
Reproduction link
github.com
Steps to reproduce

Create vue3 app with router and custom base route:

console.log(window.location.href); // http://localhost:8080/en-us

const router = createRouter({
    history: createWebHistory('/en-us'),
    routes,
  });

console.log(window.location.href); // http://localhost:8080/en-us/

Open main page http://localhost:8080/en-us

What is expected?
Location remains as http://localhost:8080/en-us
What is actually happening?
Location changed to http://localhost:8080/en-us/

This is actually expected and consistent with how the root / is handled without a base. It also makes links in general consistent as they all start with /. It's worth noting the browser hides the root /, you can try yourself:
history.pushState({}, '', '/')
// undefined
location.href
// 'http://<IP_ADDRESS>:5173/'

I recommend you to setup a redirect in your server and to set /en-us/ as canonical with the canonical link.
Is there any technical reason you want the location to remain as /en-us instead of /en-us/?

It looks strange that developer can control trailing slash on specific route, but can't control it for home root with base provided.

It's not something that changes the behavior so I rather have something that just works by default than yet another thing to care about. The trailing slash can be controlled for routes other than the root and when defining a base, the root is consistently at /base/

Now we have problems with migrating to vue3 and vue-router 4.

What problems? I'm still looking for a non cosmetic reason to see if this should be added back or not

Our SEO team asks developers to generate consistent links for each routes (without trailing slashes). What was possible on vue-router3,  not possible on vue-router4 now.
I understand that this is expected behaviour, / for home route will consisently be converted to /en-us/.
But it would be great that it could be controlled, as it could lead to failing tests during migration

Upon further research, I found two things:

Vue Router 3 actually adds the trailing slash as well when navigating to the root creating an inconsistent behavior and leading to other SEO problems. Now I remember that I made sure to keep the resolving of the URL consistent when loading the page and navigating
Regarding SEO: having a trailing slash doesn't matter for SEO as long as you properly configure the canonical link (see link in the first comment) and consistently refer to only the trailing slash version of it on your website (which vue-router will do by default when resolving links).

In conclusion, for a good SEO with a defined base:

Setup a redirect 301 or 302 from /en-us to /en-us/
Make sure to only use links to /en-us/ in your code, or, use the named route: router.push({ name: 'home' })

Note that these two steps are necessary no matter if you want a trailing slash or not (tweaking the values of course). Therefore, adding an option to set the trailing slash on the root path when a base is defined (quite a specific case) would not help in any way


New version of Boolean Binder Provider

This should finally fix the issue for the binding of boolean values

Feel free to merge whatever version you prefer


Update parse date to leverage cuDF support for single digit components

Signed-off-by: Andy Grove<EMAIL_ADDRESS>This PR removes the code we had to convert single-digit date/time components to double-digit and now leverages cuDF support for parsing variable-width components.
The previously failing tests are reinstated.

build

build

build


Release/1.5.1

This is a maintenance release for 1.5.1, addressing a small number of specific issues.
Changes:

Retrieval of file metadata in endaq.plot.plots.table_plot_from_ide() made less brittle.
Reverted to the old name of scipy.integrate.cumulative_trapezoid() for broader SciPy version compatibility.

Codecov Report

Merging #208 (c0363dc) into main (0ae3c2a) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main     #208   +/-   ##
=======================================
  Coverage   70.59%   70.59%
=======================================
  Files          34       34
  Lines        3095     3095
=======================================
  Hits         2185     2185
  Misses        910      910

Impacted Files
Coverage Δ

endaq/calc/integrate.py
100.00% <ø> (ø)

endaq/plot/plots.py
67.54% <100.00%> (ø)

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more


[only test tmp]

Proposed changes
Issue Number: close #xxx

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run buildall


nixos/nix-daemon: support machine protocol

Make ssh-ng:// users happier.
@moduon MT-904
Description of changes

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

Why does this lack a release notes entry? In fact that's breaking configs at runtime when using e.g. hostName = "ssh-ng://builder";. Yes, you can use protocol now, but given that it's not even documented and the module doesn't warn you, you'll only find out about it after rolling out this change.

In fact that's breaking configs at runtime when using e.g. hostName = "ssh-ng://builder";

This a hack that just happened to work because of our string concatenation

but given that it's not even documented

It is https://search.nixos.org/options?channel=22.11&show=nix.buildMachines..protocol&from=0&size=50&sort=relevance&type=packages&query=nix.buildMachines..protocol or at which documentation part are you looking at?

This a hack that just happened to work because of our string concatenation

This is absolutely true and I'm not saying that this was a valid solution at any point (I was young and I needed ssh-ng ;-) ).
However: it was a way to configure distributed builds that used to work. It's perfectly fine to drop it on unstable (and thus on the next stable release), don't get me wrong on that! My only point is that it should be mentioned in the release notes because otherwise it will break silently after deploying 22.11 when you wonder why distributed builds are broken.

It is https://search.nixos.org/options?channel=22.11&show=nix.buildMachines..protocol&from=0&size=50&sort=relevance&type=packages&query=nix.buildMachines..protocol or at which documentation part are you looking at?

I'm missing a heads-up about this in the release-notes because it's technically a breaking change (though a small one). I don't think that we can expect administrators to scan through the entire manual (or options reference) on each release, upgrading after reading the release notes should be sufficient.

I'll create a PR. Maybe we can backport that.

#202918


Update CLI extensions available doc

Update CLI extensions available doc.
Triggered by Azure/azure-cli-extensions - TRAVIS_BUILD_ID=510659794
https://github.com/Azure/azure-cli-extensions/commit/5a819c9c253356e2a856fbe052cca319a8628d4a

OPS Build status updates of commit 91f95dd:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-ref-conceptual/azure-cli-extensions-list.md
:white_check_mark:Succeeded
View (azure-cli-latest) View (azure-cli-2017-03-09-profile) View (azure-cli-2018-03-01-hybrid)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


logo design

Hello @OmkarPathak
I contribute to open source software in my spare time. I noticed that you don't have a logo. If you want, I can design a logo for this warehouse. I will wait for feedback. Thanks...

@reallinfo thats great. Sure I would love to have a logo. Open a PR when your are done 😄

Yeah, I designed a logo. Also, I can apply them if there's a color you want. What do you think about the logo?

@reallinfo also, can you design a new logo for this website: https://omkarpathak.in. This is my personal portfolio website and it would be really nice if you can design a logo for that too 😄

I think you want this :) How is these? Also, also i will design a logo for your website.

Perfect! This is what I wanted :smile:
Amazingly done. Thank You :smile: :rocket:

I sent a pr and readme updated.
Btw, Can you give your website a repository link?

Thanks. Here it is: https://github.com/OmkarPathak/omkarpathak.github.io

@reallinfo any updates?

https://github.com/OmkarPathak/omkarpathak.github.io/issues/6
Please look here. :) I'm closing this problem, please let us continue from there.


Issue with selector with regex in return

I have seen a few examples like below:
selector: function(match) {
  regex = new RegExp(match, 'i')
  return {'profile.firstName': regex}
}

I am actually trying the code above. If I put a string in place of regex, it is passed and works correctly, but when I use the regex variable, the output sent to query looks like this 'profile.firstName': {} } { limit: 10, sort: { _id: 1 } }. Notice the regex portion is missing.
Anyway, I am not sure why this doesn't work yet. Any thoughts?

Were you able to workaround to this?

I don't think you can use a Javascript regex in a Mongo query that way; you should try the string literal version: https://docs.mongodb.com/manual/reference/operator/query/regex/ . Otherwise, use $regex.

@mizzao Yes. I did the same. Will share the code in some time to help others who face this issue.
Thanks for responding and making this great package!


Unable to locate package rocfft

What is the expected behavior
rocfft is installed.
What actually happens
E: Unable to locate package rocfft
How to reproduce
max@supernova:~$ cat /etc/apt/sources.list.d/rocm.list
deb [arch=amd64] http://repo.radeon.com/rocm/apt/debian/ xenial main
max@supernova:~$ sudo apt install rocfft
Reading package lists... Done
Building dependency tree
Reading state information... Done
E: Unable to locate package rocfft

On the other hand, I was able to install rocm-dkms and rocm-opencl-dev from rocm repo successfully:
max@supernova:~$ sudo apt-get install rocm-dkms rocm-opencl-dev
Reading package lists... Done
Building dependency tree
Reading state information... Done
rocm-dkms is already the newest version (1.7.60).
rocm-opencl-dev is already the newest version (1.2.0-2017121952).

Environment
max@supernova:~$ cat /etc/lsb-release
DISTRIB_ID=Ubuntu
DISTRIB_RELEASE=16.04
DISTRIB_CODENAME=xenial
DISTRIB_DESCRIPTION="Ubuntu 16.04.4 LTS"

rocfft was not included in the last release of rocm; it will be available in the next release

Why was it not included int the last release of rocm please?

You can build rocFFT from source always. Current ROCM release is 1.7.0, next release 1.7.1 will include rocFFT very soon.  You won't wait long

actually rocm 1.7.137 has been released, and rocfft is included in that

update your rocm-dkms to the most recent version and you should be able to install rocfft also


comment on new line after return adds unwanted semicolon

function bar() {
    return /* this is comment */ 5;
}

function foo() {
    return
        // this is comment
        5;
}

Results in:
function bar() {
    return 5;
}
function foo() {
    return;
    // this is comment
    5;
}

After the last return an semicolon is inserted, which is wrong. In the language specs I haven't found a restriction on the use of comments in expressions.
I'm using the master of today

This is actually technically correct.  The automatic semicolon insertion "feature" that is built-in to JavaScript actually puts that semicolon there regardless of whether you've typed it.  Calling foo() returns undefined.


Radial charts are not working in IE 11

Not sure this is a bug or issue at my end.
I went through the documentation mentioned for IE 11 in order to work, so i have include the 3 polyfills  in my index,html as below
As mentioned in the documentation and as per above @sarathcs comments, i have include those polyfills in my index.html file as below :
<script src="https://cdn.jsdelivr.net/npm/eligrey-classlist-js-polyfill"></script> <script<EMAIL_ADDRESS><script src="https://cdn.jsdelivr.net/npm/findindex_polyfill_mdn"></script>S <title>React App</title>
and in my index.js file i have included below polyfills
import 'react-app-polyfill/ie11';
import 'react-app-polyfill/stable';

But i still didnt see it is loading correctly in IE, is there anything i am missing or any thing to be corrected ?
Thank you

Are there any errors in the developer tools?

Hi @junedchhipa  Thank you for the prompt response
yes i am getting promise error as below in IE
Unhandled promise rejection TypeError: Unable to get property 'add' of undefined or null reference

Hi @junedchhipa will it be resolved in next release?

@SandeepKapalawai Hi Sandeep, I don't have IE11 right now to debug the issue.
Hence, I don't have an ETA for this fix.

I've run into this issue too. It doesn't seem to be specific to a particular kind of chart. The issue is that IE doesn't support the classList property on SVG elements. I was able to fix it for my project by installing and importing the svg-classlist-polyfill package.

IE11 support has been ended


GDCM ignores image geometry in secondary captures

Description
Secondary capture images can optionally have ImageOrientationPatient and PixelSpacing fields, but GDCM currently forces the orientation to identity and looks for NominalScannedPixelSpacing instead of PixelSpacing, so otherwise valid images are loaded incorrectly.
There is more information in the discussion at this PR:
https://github.com/Slicer/Slicer/pull/7089
Steps to Reproduce
The data available from the idc-open-data set on aws is the Visible Human Project male dataset converted to dicom by @dclunie.
You can get one file with this command:
aws s3 cp --no-sign-request s3://idc-open-data/4aaf9181-fb6a-4a4c-bf49-d1eb9ed4a385/fbc89e1c-29c4-48cf-89fd-53b7c1dc1488.dcm .
Or the whole dataset using this command (about 13 gigabytes).
s5cmd --no-sign-request --endpoint-url https://s3.amazonaws.com cp 's3://idc-open-data/4aaf9181-fb6a-4a4c-bf49-d1eb9ed4a385/*' .
Expected behavior
The index to physical transform should be correctly populated.
Actual behavior
The direction cosines are explicitly set of identity:
https://github.com/InsightSoftwareConsortium/ITK/blob/master/Modules/ThirdParty/GDCM/src/gdcm/Source/MediaStorageAndFileFormat/gdcmImageHelper.cxx#L731-L740
The PixelSpacing is ignored, and the NominalScannedPixelSpacing is looked for, but is typically absent.
https://github.com/InsightSoftwareConsortium/ITK/blob/master/Modules/ThirdParty/GDCM/src/gdcm/Source/MediaStorageAndFileFormat/gdcmImageHelper.cxx#L1265-L1273
Reproducibility
Always
Versions
Tested on current main branch,  but I suspect the code has been around for decades.
Environment
Any
Additional Information
I'll probably fix this, although it may not be a high priority if we fix https://github.com/InsightSoftwareConsortium/ITK/issues/4108 instead.

The upstream PR was rejected. Unfortunately, since there are many SC data sets with IPP/IOP, I have also seen such RGB fusion (CT/PT) data sets from Osirix and many other. I fact, IPP/IOP is not part of SC, but, AFAIK, GDCM has workarounds for many issues in data sets, these are "extended". PixelSpacing is perfectly legal, the CP-586 was merged into the standard over 15 years ago.

@issakomi I have asked @malaterre to reopen and implement the referenced PR, and uploaded to the PR a copy of a proposed DICOM CP specifying the behavior.

Thank you very much for your attention to the problem.

Since @malaterre has decided not to be helpful, we will need to work around his recalcitrance in ITK as he suggests.

Your long history of valuable contributions is not undermined by one deviation.

Closed via #4521


CORE-6150: Simulator now runs liquibase migration scripts

Also implemented missing page functionality in named and paged queries
Don't forget to close Simulator, fibers or persistence service if any persistence service is created

Closing as these changes will be merged from the DP2 version of this PR.


HasVortoValue does not work if string is empty

Just noticed I get an Macro Render Error if the Sting is empty and I call HasVortoValue

Could you please explain your setup and the code you are using so that I can replicate your circumstances?

Create a Voto-Datatype:
Data-type: Textstring
Lang-Source: Installed
XPath: empty
Display Native: not selected
Primary Lang: German
Mand. Field: Ignore
RTL: Ignore
Hide Label: not selected
I get a Render Macro Error, if I do not fill out the textstring-Field.
Code in my Macro:
if(Model.Content.HasVortoValue("linkZuTwitter", recursive: true))
{
...
}
Also happens, if I remove the recursive:true Property.

Where does the Macro part come into it? I don't get why you'd be getting a Macro error? Is that vorto code located directly in a template .cshtml file? or inside a macro? Is that property you've defined an the page doc type you are rendering? or are you using it along side nested content? or doc type grid editor? (you mentioned the grid on twitter) If so, how is this all linked?

The PartialView-Macro-Part is rendered in my Master-Template:
@Umbraco.RenderMacro("RenderBodyStartScript")
"linkZuTwitter" is just a vorto doctype Property in my MainPage, that's why I do the recursive call.
The vorto-code is located in a Partial View Macro File under the developer-section.

 Updated build can be found here https://ci.appveyor.com/project/mattbrailsford/umbraco-vorto/build/<IP_ADDRESS>/artifacts

Great! Seems to work perfect now.


add featured in table in main readme.md file

The task involves updating the main README.md file to include a table highlighting project participation in open source program.
something like this

@ratnesh-maurya  Assign this to me under gssoc 2024 ext and hacktoberfest i will work on this issue.

Currently not accepting readme file changes


fix: Allow benchmark to specify eval_splits

This PR allow for benchmarks to specify eval. splits. This allow us to fully specify a benchmark within the benchmark object.
To do this it add the following:

added eval_splits to the Abstask object, which default to metadata.eval_splits
use the task.eval_splits unless overwritten in mteb.MTEB.run
added eval_splits arg to mteb.get_tasks, which filter the tasks based on splits
updated documentation

renamed the "Advanced Usage" to "Usage Documentation" to make it more accessible

added tests where relevant

Checklist

[x] Run tests locally to make sure nothing is broken using make test.
[x] Run the formatter to format the code using make lint.

It seems like there's a relevant test failure


Client Allocation Request for: 爱建信托

Client Allocation Questions
Core Information

Name: 爱建信托
Website / Social Media: http://www.ajxt.com.cn/ajxt/
Region: Greater China
DataCap Requested: 10TiB
Addresses to be Notarized: f1ru6n2lvnaawezj3qc24v6bfwtvwud2u4grqviby
Notary Requested: ozhtdong

Hi. @zzshao11 Thanks for your application!
I want to know more basic information of you and your company. Please answer following question:
1.What's your name? What's your position?
2.What's your company's business? Do you have any other social media link?
3.Cloud you show me your company's business license?

@ozhtdong Sorry for the late reply. I am 张嵩, a technical manager of 爱建. Established in 1986, 爱建信托 is the first private financial institution in China. You can find more detailed information in our Baidu Baike:
https://baike.baidu.com/item/上海爱建信托投资有限公司/10861672?fr=aladdin
Our company's business license:

Thanks for your detailed information.
I have further more questions about data and storing.
1.What kind of data do you want to store and how much? Could you please send me some demos of your data?
2.Have you/your company ever used cloud? Which platform do you use? What's your usage volume and frequency?

Hello!
Sending a friendly ping to this issue since it hasnt had movement and closing out. Should you wish to continue please feel free to reopen.  Thanks and Best!


Revisit TSS signed blocks windows implementation

Description/Reasoning
#1198 implemented a fix for #1190 to have a shorter window for heartbeats. But this implementation does not handle the case where the slashing SignedBlocksWindow parameter is upgraded to become smaller than the Tss window. If this occurs, the missed blocks check will be incorrect. This is tricky to validate since we don't control Cosmos module's upgrade script. An alternative method is to re-implement the missed block tracking in TSS so we have complete control over it. But the downside of this approach is that we now have an additional commitment to port over any bug fixes made to this logic in Cosmos SDK.
Current Behaviour
Expected Behaviour
Steps to reproduce (for bugs)
Relevant Logs or Files

Should just revert https://github.com/axelarnetwork/axelar-core/pull/1198 once we have a proper coin distribution and less centralized network.


[SSR]  cause hydration to fail

Version
3.1.0-beta.6
Reproduction link
https://codesandbox.io/s/wizardly-kepler-y8n48
Steps to reproduce
Open sandbox link and check browser console.
What is expected?
Hydration should succeed.
What is actually happening?
It seems that some Teleport comments are causing hydration to fail.

This is expected because you need to explicitly handle SSRed Teleports.
They are exposed on the ssr context:
const ctx = {}
const html = await renderToString(app, ctx)

console..log(ctx.teleports)

You need to return this from render in entry-server.js and then inject them into the final HTML in server.js.

What's your mean is i should use some library like js-dom and inserts  ctx.teleports into the exactly dom node in server side? But if i use renderToNodeStream what should i do?

How does the injection in rendered HTML work? when i check the rendered html string, the container it should teleport to is empty, without any comment. So how do i know where i have to add the teleport code?

console..log(ctx.teleports)

@yyx990803 can you provide an example? Doesn't it make sense to mention this in The Vue documentation part about teleport?

This is expected because you need to explicitly handle SSRed Teleports.
They are exposed on the ssr context:
const ctx = {}
const html = await renderToString(app, ctx)

console..log(ctx.teleports)

You need to return this from render in entry-server.js and then inject them into the final HTML in server.js.

@yyx990803 can you provide an example? Doesn't it make sense to mention this in The Vue documentation part about teleport?

when i check the mentioned ctx object i get:
{                                                                                23:20:11
  modules: Set(24) {
    'src/App.vue',
    'src/components/Noscript.vue',
    'src/components/Browserhint.vue',
    'src/layouts/MasterLayout.vue',
    'src/components/TheHeader.vue',
    'src/components/menu/MainMenu.vue',
    'src/pages/ScenePage.vue',
    'src/components/TheFooter.vue',
    'src/components/TheScene.vue',
    'src/components/FormComponent/FormComponent.vue',
    'src/components/FormValidationGroup/FormValidationGroup.vue',
    'src/components/FormFieldset/FormFieldset.vue',
    'src/components/FormInput/FormInput.vue',
    'src/components/FormField/FormField.vue',
    'src/components/BaseInput/BaseInput.vue',
    'src/components/FormSelect/FormSelect.vue',
    'src/components/BaseSelect/BaseSelect.vue',
    'src/components/FormCheckbox/FormCheckbox.vue',
    'src/components/BaseCheckbox/BaseCheckbox.vue',
    'src/components/FormConditional/FormConditional.vue',
    'src/components/FormRadioGroup/FormRadioGroup.vue',
    'src/components/BaseRadio/BaseRadio.vue',
    'src/components/IntroComponent/IntroComponent.vue',
    'src/components/BaseButton/BaseButton.vue'
  },
  __teleportBuffers: {
    "[data-scope-buttons='111']": [
      [Array],
      [Array]
    ]
  }
}

so there is no teleports object, only a __teleportBuffers object containing arrays which i don't know how to handle. And the  containers it should teleport to do not contain any <!--teleport start--><!--teleport end--> comments, so i really don't know how to do this.
Using vue 3 with vite.

It seems my problem is related to this: https://github.com/vuejs/core/issues/4440
There is a PR provided since december to fix it which needs to be merged


SqlLiteDatabaseLockedException

I recieve the attached crashlog
crashlog.txt
. (It says candybar version 3.1.4 but has already been updated the library to 3.1.6)

Forgot to change version number to 3.1.6


RFE: ability to install dependency via remote ZIP file

Hello,
Is it possible to specify a zipball of a python setuptools project? (Basically the equivalent of the requirements.txt feature of supporting https://github.com/blabla.zip#egg=SomePyPkg )
Thanks and thanks for trying to make a sane package/dependency manger for python!

Installing from an archive is not currently supported.
However, if you want to install from github you can declare you dependency like so:
blabla = { git = "https://github.com/blabla/blabla.git", branch = "master" }
blabla = { git = "https://github.com/blabla/blabla.git", tag = "0.1.2" }
blabla = { git = "https://github.com/blabla/blabla.git", rev = "d12189e2076fbdb40ed4902b2b4d9579003c607c" }


fix(package.json):

updated package.jsons to include a link to the repo. this is to qualify for tea and allow users to find repo from package managers
GitHub Issue
NA
Changes
Updated the package.json files to include references to the monorepo.
Packages Added
NA
Checks
Before making your PR, please check the following:

[x] Critical lint errors are resolved
[x] App runs locally
[x] App builds locally (run the build command for any impacted package and check for any errors before the PR)

@skuhlmann I'm adding references to the repo in the package.json to allow us to register the project with Tea. We will probably need to redeploy the packages to npm for these changes to be reflected.


chore: release prep for v0.61.0

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 55.15%. Comparing base (4d30454) to head (7a9100c).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #488   +/-   ##
=======================================
  Coverage   55.15%   55.15%
=======================================
  Files          71       71
  Lines        5691     5691
=======================================
  Hits         3139     3139
  Misses       1935     1935
  Partials      617      617

Flag
Coverage Δ

2.1
33.38% <ø> (ø)

2.2
43.85% <ø> (ø)

2.3
44.42% <ø> (ø)

2.4
44.49% <ø> (ø)

2.5
44.49% <ø> (ø)

2.6
44.49% <ø> (ø)

2.7
45.96% <ø> (ø)

2.8
45.96% <ø> (ø)

3.0
49.51% <ø> (ø)

3.1
50.85% <ø> (ø)

3.2
50.85% <ø> (ø)

3.3
50.85% <ø> (ø)

3.4
52.87% <ø> (ø)

3.5
51.02% <ø> (ø)

3.6
51.02% <ø> (ø)

3.7
51.02% <ø> (ø)

3.8
52.62% <ø> (ø)

community
41.60% <ø> (ø)

enterprise
53.75% <ø> (ø)

integration
55.15% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

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


[GPT-90] Post: Retain toggle dropdown for H2 headings

Added the dropdowns with the animation.
Closes #349
Looks like this now

Could the arrows be changed to be solid right/down like on the current website:

Now looks like this after changes