Ensure that the perf tool binaries are always present

What this PR does / why we need it:
bazel run targets require at this stage explicit build steps for a
dependency to be available locally if the remote fetch strategy does not
fetch all artifacts by default.
Fixes issues like
18:24:55: cp: cannot stat 'bazel-out/k8-fastbuild/bin/tools/perfscale-audit/perfscale-audit_/perfscale-audit': No such file or directory

Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Release note:

NONE

Turns out there is another tool there too now:
07:46:33: cp: cannot stat 'bazel-out/k8-fastbuild/bin/tools/perfscale-load-generator/perfscale-load-generator_/perfscale-load-generator': No such file or directory

Added to the PR.

/retest

/lgtm

/retest
/approve

Everything is green. Merging this to help other PRs not running into this issue which happens quite often now.


Vite will add new instance of dat.gui if hot reloaded

[x] Read the docs.
[x] Use Vite >=2.0. (1.x is no longer supported)
[NA] If the issue is related to 1.x -> 2.0 upgrade, read the Migration Guide first.

Describe the bug
Vite's hot reload will add a new instance of dat.gui each time to the DOM. (I am actually not  100% sure if this is a bug or it is expected behavior.)
Reproduction
https://github.com/fnick851/vite-shader-water
System Info

vite version: 2.0.0-beta.45
Operating System: macOS Big Sur x86 11.1
Node version: 14.15.4
Package manager (npm/yarn/pnpm) and version: npm 6.14.11

Logs (Optional if provided reproduction)

Run vite or vite build with the --debug flag.
Provide the error log here.

This is "kind of" expected behavior because:

You are importing and using renderWaterScene in a Vue component
The Vue component is a HMR boundary, so when it hot updates, it re-imports its changed depdencies
The file that exports renderWaterScene creates dat-gui as a side-effect-on-import

What you should do is to avoid on-import side-effects in files that export code being used in Vue components - i.e. separate dat-gui setup logic from functions like renderWaterScene.

This is "kind of" expected behavior because:

You are importing and using renderWaterScene in a Vue component
The Vue component is a HMR boundary, so when it hot updates, it re-imports its changed depdencies
The file that exports renderWaterScene creates dat-gui as a side-effect-on-import

What you should do is to avoid on-import side-effects in files that export code being used in Vue components - i.e. separate dat-gui setup logic from functions like renderWaterScene.


ExtensionValidator.getExtension is incorrect if filename includes extra period characters.

Windows, Linux and Mac all allow filenames like  foo.bar.txt
The getExtension method will return 'bar' instead of 'txt'.

I've submitted a pull request with a fix along with additional tests.

@rick-boardontrack Thanks for detecting this issue. Let me take a look.


pvoutput deprecated addbatchoutput

I found recently that pvoutput has deprecated the addbatchoutput API, batching is now part of addoutput, but batching is only available for donations mode, see https://github.com/jwillemsen/saj_collector/issues/34. It looks this project needs to be updated because it uses addbatchoutput for example at https://github.com/wildmountainfarms/solarthing/blob/29e4c4f8058e6ac2bca05c3ee995b4a102a9c803/serviceapi/src/main/java/me/retrodaredevil/solarthing/pvoutput/service/PVOutputService.java

Thanks for letting me know. I like the change they made to make addoutput multipurpose, but really dislike non-donators cannot do batch uploads now.
I guess I'll have to add an option to upload data one day at a time.


Support specifying non-default browser for login

Describe the solution you'd like
In short...
aio auth:login --browser=Firefox

Describe alternatives you've considered
#61 defines an alternate approach for the same problem (and they're not necessarily mutually exclusive), but given that cli-ux supports specifying a different browser, more literally implementing support for an alternate browser has better IMO ergonomics.

closing this in favor of the --no-open flag which will let the user copy/paste the url and open in whatever browser they wish.
Otherwise we would sort of be responsible for making sure all browsers work, and CAN post back which is a lot to verify and potentially a lot of work


check that the channels in the config file exist in the raw data before attempting to load it

related to #299

Implemented


Transit gateway support

Closes: #40, #39

optionally build TGW subnets
must supply transit_gateway_id
optionally create route_table_association to subnets, must supply transit_gateway_route_table_id

updates have been requested:

route from public/private to tgw
allow tgw to route to nat

  subnets = {
    public = {
      netmask = 24
      route_to_transit_gateway = ["<IP_ADDRESS>/0"]
    }

    private = {
      netmask = 24
      # if route_to_nat = true
      # route_to_transit_gateway != <IP_ADDRESS>/0
      route_to_nat = true
      route_to_transit_gateway = ["<IP_ADDRESS>/0"]
    }

    transit_gateway = {
      netmask            = 24
      transit_gateway_id = aws_ec2_transit_gateway.example.id
      route_to_nat       = true # default false
      transit_gateway_default_route_table_association = true
      transit_gateway_default_route_table_propagation = true
    }
  }


Clean up of perf results accesses and storage

Description
Removes some boilerplate from the tests around the printing and storage of various perf results after the tests have executed. Now, perf results are all handled within the integration tests base struct, and the test simply has to call WritePerfResults in TearDownSuite
This also fixes an issue where container stats weren't initialized properly, resulting in invalid perf reporting.
Checklist

[ ] Investigated and inspected CI test results
[ ] Updated documentation accordingly

Automated testing

[ ] Added unit tests
[ ] Added integration tests
[ ] Added regression tests

If any of these don't apply, please comment below.
Testing Performed
Tested locally against RHEL 8 and RHEL 9

I just noticed that the perf comment in the PR is empty. This is not related, right ?

No, I think this bug has existed for a little while, though it might be affected by the change in names in the perf result (benchmark -> TestBenchmarkCollector) I'll investigate before merging


[WIP] Bundle Helm charts for Redis, MariaDB and PostgreSQL

This PR adds basic support for deploying Redis, MariaDB or PostgreSQL as part of an Authelia deployment.
Typically the dependency subcharts would be integrated under the top level mariadb: or postgresql: keys, where the config can be inherited by the subcharts, and the main app (i.e. Authelia) would also draw its config from those keys. However the database config in the chart is under storage.mysql and storage.postgres and changing this would be a breaking change.
So I have added booleans storage.redis.deploy, storage.mysql.deploy and storage.postgres.deploy which enable the user to deploy the Bitnami subcharts. This does not affect any existing Authelia deployments. At the moment the user has to copy a few settings (e.g. credentials) from the mariadb section to the storage.mysql section.
In future we could consider a tighter integration, but it would be a more major refactoring, so let's go with this for now.
I'll keep the [WIP] tag on until I've fully tested this on my own cluster Happy to discuss or tweak this :slightly_smiling_face:

Fixes #172
Fixes #89
Supersedes #223
Supersedes #225
Supersedes #232

Superseded by #244


Simulator disconnection on remote socket termination

When the the connection is terminated on remote, simulator stays as if the connection is still open. It should stop running logging that the connection ended.

Good point murat :+1: This will be fixed in next versions.

Congratulations! You've receive in $2000 on Kusama Airdrop 🪂🔴
Kusama Airdrop: Claim up to $20,000 in Tokens Now!
→ Earn rewards on connect you wallet
→ Claim Your $KSM Today
Connect your wallet, follow the instructions, and claim this exclusive airdrop from Kusama. For GitHub users only—share with friends to make it eligible. Redeem your reward first.

🚀 Claim Your Bonus Now! Check Eligibility
Access the Kusama platform and connect your wallet securely.


AWS DynamoDB

Hi,
Do you have any plan to support for DynamoDB?

Hi, I do not plan it, but it should be quite easy to implement. Pull requests are welcomed.

Just as reference for others not watching the PR list like me: #108 and #112 are adding such provider once merged.

Released as 2.2.0


GA: In a new chat window, Cody provides an explanation of of visible code if none is selected

Current functionality is that Cody explains the open file if no code is selected (not just what's "visible"). I think this is fine, and if we get feedback to the contrary, we can revisit.


_vm.socket is undefined

Hi  !
I'm trying to set up simple webpage with nodeEditor using Vue ( to which I'm complete beginner). I have set-up some app using examples from rete.js.org in which nodeEditor is created inside Vue component. In principle editor works and I can create Num nodes, but the sockets are missing. Console shows an error:
[Vue warn]: Error in render: "TypeError: _vm.socket is undefined"

found in

---> <Socket>
       <Anonymous>
         <Root>

I know it is not much info to replicate the problem, but maybe someone more experienced will exactly know what happens - I looked through all resources and found no clues :( especially that when I start app from rete.js.org repo editors are working fine.
Best,

I know it is not much info to replicate the problem

Exactly, you need to share the code you changed and call stack.

Hello,
I'm attaching the callstack and source files - it was scaffolded using vue-cli. When you open webpage functioning editor is on purple part of the window.
my-project.zip
callstack.txt
Best regards.

I've just realized call stack from previous post had some folded entries - belowe there is a full one:
callstack.txt

Try to use Vue CLI. Perhaps the problem in webpack configs

I got error Error in render: "TypeError: Cannot read property 'name' of undefined"

There s a wrong arguments
let out = new Rete.Output('Number', numSocket)

Correct:
let out = new Rete.Output('num', 'Number', numSocket)

I feel very stupid :( I do not know how it happened. Anyway thank you very much for your help and maybe in some time I can contribute to the project :)


Can the initial image be a data URL?

Doesn't seem to be working for me (shows up as null).

I think I have figure it out after studying the source code. You can pass Image object to initialImage property.
here is what i did:
0. define an image property on your data
export default {
  data() {
    return {
      myCroppa: {},
      image: null,

do not render <croppa> till the Image is loaded (v-if is used)

   <div  v-if="image">
            <croppa v-model="myCroppa" :initial-image="image"></croppa>
        </div>

load Data URL image

         var img = new Image();
          img.onload = e => {
            this.image = img;
          };
          img.src = e.target.result;

viola! as soon as you load dataURL into Image object the Croppa will initialise with your image.


Artemis sasl delegation

Ports are probably still wrong - need to test connecting from outside openshift

@lulf Agree on tests - which I'll work on today; we can merge if not having the proper auth is holding up any of the other testing that we'd want to do against the brokered address space otherwise we can wait for the tests to be done :)


Added exclude support for Pyboard upload script

PyboardDeviceProvider now recognizes two settings from project structure:

Excluded folders (marked by red)
Exclude files (it will simply append those filenames to the project basedir)
It allows to effectively exclude:
.git folder
.idea folder
any other files unwanted for deployment (README.md for example)
Tested with
PyCharm 2018.1.2 (Professional Edition)
Build #PY-181.4668.75, built on April 25, 2018
JRE: 1.8.0_152-release-1136-b29 amd64
JVM: OpenJDK 64-Bit Server VM by JetBrains s.r.o
Linux 4.15.0-20-generic

Providers for other devices like ESP8266 could also benefit from this update. But I'm not that comfortable with Intellij Plugin SDK to dare to interfere with what's already working :)

@max-kviatkouski Thanks for your pull request!
Please submit a contributor license agreement online. You can follow the steps described at http://www.jetbrains.com/agreements/cla/ to sign it.

I'm closing this pull request for now.


Instructions for redis-cli.exe, and lack of TLS support

Article states: "The redis-cli.exe command-line tool doesn't support TLS."
This is incorrect - Documentation states this can be enabled with --tls -- https://redis.io/docs/ui/cli/

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

ID: 17de8ad0-ac40-26a5-7718-1738f63f0700
Version Independent ID: 797d8d3f-132d-d71f-2df4-caade2b1240b
Content: Use redis-cli with Azure Cache for Redis
Content Source: articles/azure-cache-for-redis/cache-how-to-redis-cli-tool.md
Service: cache
GitHub Login: @flang-msft
Microsoft Alias: franlanglois

@sth2258
Thanks for your feedback! We will investigate and update as appropriate.

Hi @flang-msft,
Could you please look into this document request, thanks.

Thanks for the doc suggestion! I've noted it for a future update of this article. According to our usual policy, I'll now proceed to close this issue. If you have further information or would like to continue the discussion, please tag me and or reopen the issue. Thank you!
#please-close


Columar and CSV upload don't work in superset / preset.io

When you try to upload a columar or CSV file through the Superset SQL Lab, it doesn't work.  No error message.

enabled file upload and DML
Please check your query for syntax errors near "TEXT' at line 1". Then, try running your query again.


Add access control to ndt7:80 and ignore TxController errors

This change adds the access controller to ndt7 clear text handler and generalizes the access controller setup to allow chaining multiple controllers in the future. This change also allows TxController to be disabled with a warning.

This change is

Pull Request Test Coverage Report for Build 1193

11 of 13 (84.62%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.2%) to 80.125%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

ndt-server.go
11
13
84.62%

Totals

Change from base Build 1190:
0.2%

Covered Lines:
1661

Relevant Lines:
2073

💛  - Coveralls


Configures build rollup options to remove hashes from file names

Scope of changes
Configures Vite build rollup options to remove hashes from file names. After a new build, the hashes added to a filename usually change, so if a user is using an older version of Beacon they will see an error message if they click to view a page where the filename has changed because the hash is different.
Removing the hashes should ensure the filenames remain consistent after each build and should reduce or prevent the following error:
Failed to load module script: Expected a JavaScript module script but the server responded with a MIME type of "text/html". Strict MIME type checking is enforced for module scripts per HTML spec.
Fixes SC-22525
Type of change

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

Acceptance criteria
[Copy acceptance criteria checklist from story for reviewer, or add a brief acceptance criteria here]
[Front-End: add screenshots/videos of changes made]
Definition of Done

[x] I have manually tested the change running it locally (having rebuilt all containers) or via unit tests
[ ] I have added unit and/or integration tests that cover my changes
[ ] I have added new test fixtures as needed to support added tests
[ ] I have updated the dependencies list if necessary (including updating yarn.lock and/or go.sum)
[ ] I have recompiled and included new protocol buffers to reflect changes I made if necessary
[x] Check this box if a reviewer can merge this pull request after approval (leave it unchecked if you want to do it yourself)
[ ] I have notified the reviewer via Shortcut or Slack that this is ready for review
[ ] Front-end: Checked sm, md, lg screen resolutions for effective responsiveness
[ ] Backend-end: Documented service configuration changes or created related devops stories

Reviewer(s) checklist

[ ] Front-end: I've reviewed the Figma design and confirmed that changes match the spec.
[ ] Any new user-facing content that has been added for this PR has been QA'ed to ensure correct grammar, spelling, and understandability.
[ ] Are there any TODOs in this PR that should be turned into stories?

@pdeziel Yes, the hashes help check if the code is outdated. Since we added an ErrorBoundary this is why an error page appeared while attempting to access pages that changed when we tried to test out the version check. The version check should work, if the hashes are removed. However, now I wonder if it's better to keep the hashes.

Ok, we can merge this in for now and see how the version check works.


Aqmon hits ordering fix

What does the code in this PR do / what does it improve?
aqmon_hits are now ordered by time, as expected. Also, we check if the raw_records_aqmon are assigned wrong times so that we get overlapping records.
Can you briefly describe how it works?
We thought we'd order aqmon_hits by time, already. As the strax.sort_by_time method returns the result instead of an in place change, so far we didn't. This is fixed.
As done with raw_records already, we also check here that there are no overlapping records, which can lead to time-ordering issues again. (as done in the other case, this check can be turned off via a strax-option)
Comment
Time ordering would currently only lead to issues if, within one channel, time-ordering is not given. However, if records from two different channels are swapped, this doesn't do anything.
As we always expect our data to be ordered by time in strax and to prevent funny things from happening, it doesn't hurt to make sure everything is correctly ordered.

Thanks Robin for spotting this issue!
Your PR seems to be from an out of date branch? Could you fix the merge conflict or open a new PR starting from current master?
Three small additional questions:

could you increment the version number?
Could you use an URLConfig (such as here https://github.com/XENONnT/straxen/blob/e341819beb9b44eba0b734397c02777e8131f9b4/straxen/plugins/acqmon_processing.py#L68) it's a new - fancier version of the strax.Option
Perhaps we should move this statement https://github.com/XENONnT/straxen/blob/e341819beb9b44eba0b734397c02777e8131f9b4/straxen/plugins/acqmon_processing.py#L112  to here https://github.com/XENONnT/straxen/blob/e341819beb9b44eba0b734397c02777e8131f9b4/straxen/plugins/acqmon_processing.py#L91 such that we are certain the hits are also always sorted by time.

Thanks again!

I thought I did git pull before creating the new branch... well, turns out I didn't. Sorry for that.
I created a new branch from the latest straxen version with the changes you pointed out above, @JoranAngevaare. PR follows.


Using CUE to send HTTP POST requests

Hi there,
I am trying to utilize the CUE scripting layer to send a HTTP POST request to a webserver.
So far my script looks like this, I have encoded the payload in bytes but however, the request is not accepted.
command: req: {
  query: http.Do & {
    method: "POST"
    url:    "https://api.meraki.com/api/v1"
    request: {
      body: "7b226e616d653a22666f6f227d"
      header: {"Content-Type": "application/json"}
    }
  }
  print: exec.Run & {
    cmd: ["echo", query.response.body]
  }
}

When sending GET and DELETE requests, everything works fine. I guess it is something wrong with the encoded data.
The server is responding the following error message back: { "error": "body contains badly-formed JSON (at character 2)" }
As a second question: Is it also possible to directly encode CUE-generated values to bytes and send it within the HTTP request body?
Something like this:
album: {
  foo: "Led Zeppelin"
  bar: "BBC Sessions"
}

Encoded and then added to the HTTP request body like so:
// The same code from above
request: {
  body: album
}

Maybe some background information, what I am trying to achieve: I want to generate switch configuration for Meraki switches using
CUE, validate the planned configuration and then send the complete payload directly as HTTP POST request to the Meraki controllers.
Thanks.

Try using single quotes for bytes, you have a string for body
To use a CUE value, try out the encoding/json.Marshal function

Yeah, thanks. That was my problem.
Using
data: {
  settings: {
    name: "foo"
  }
}

along with
json.Marshal(data)

solved my problem.


Incremental indexing for vector search (with Arroy)

Motivation

Develop incremental indexing in arroy to drastically improve vector search performances
Usage

Nothing to do for the user, immediately available when using the experimental vectorStore feature
TODO

[x] Merge https://github.com/meilisearch/arroy/pull/41
[x] Bump arroy version in Meilisearch

Will be fixed by https://github.com/meilisearch/meilisearch/pull/4332, upgrading the Arroy dependency

#4332 has been merged.


Error setting up entry PyLoxone for loxone

Hey,
When i restart HA i get this error, i already tried to reinstall the plugin but no success.
Also tried to make a new account to be sure but same error
Error setting up entry PyLoxone for loxone
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/config_entries.py", line 236, in async_setup
result = await component.async_setup_entry(hass, self)  # type: ignore
File "/config/custom_components/loxone/init.py", line 122, in async_setup_entry
res = await lox.async_init()
File "/config/custom_components/loxone/api.py", line 320, in async_init
_ = await self._ws.recv()
File "/usr/local/lib/python3.8/site-packages/websockets/protocol.py", line 509, in recv
await self.ensure_open()
File "/usr/local/lib/python3.8/site-packages/websockets/protocol.py", line 812, in ensure_open
raise self.connection_closed_exc()
websockets.exceptions.ConnectionClosedOK: code = 1000 (OK), reason = normally closed

Please give more info's! Did the error occur after a update? Or did it never connect? Can you connect via Loxone App?
Which port do you use?

It never connected, yes i can connect with the Loxone app, port 80

Try a different port then 80 maybe 8080.

i just did that and i get a different error, but it looks like he is  still trying to connect to port 80
Error setting up entry PyLoxone for loxone
Traceback (most recent call last):
File "/usr/local/lib/python3.8/site-packages/requests_async/adapters.py", line 48, in send
response = await self.pool.request(
File "/usr/local/lib/python3.8/site-packages/http3/interfaces.py", line 49, in request
return await self.send(request, verify=verify, cert=cert, timeout=timeout)
File "/usr/local/lib/python3.8/site-packages/http3/dispatch/connection_pool.py", line 130, in send
raise exc
File "/usr/local/lib/python3.8/site-packages/http3/dispatch/connection_pool.py", line 120, in send
response = await connection.send(
File "/usr/local/lib/python3.8/site-packages/http3/dispatch/connection.py", line 53, in send
await self.connect(verify=verify, cert=cert, timeout=timeout)
File "/usr/local/lib/python3.8/site-packages/http3/dispatch/connection.py", line 81, in connect
reader, writer, protocol = await self.backend.connect(
File "/usr/local/lib/python3.8/site-packages/http3/concurrency.py", line 204, in connect
stream_reader, stream_writer = await asyncio.wait_for(  # type: ignore
File "/usr/local/lib/python3.8/asyncio/tasks.py", line 491, in wait_for
return fut.result()
File "/usr/local/lib/python3.8/asyncio/streams.py", line 52, in open_connection
transport, _ = await loop.create_connection(
File "/usr/local/lib/python3.8/asyncio/base_events.py", line 1025, in create_connection
raise exceptions[0]
File "/usr/local/lib/python3.8/asyncio/base_events.py", line 1010, in create_connection
sock = await self._connect_sock(
File "/usr/local/lib/python3.8/asyncio/base_events.py", line 924, in _connect_sock
await self.sock_connect(sock, address)
File "/usr/local/lib/python3.8/asyncio/selector_events.py", line 496, in sock_connect
return await fut
File "/usr/local/lib/python3.8/asyncio/selector_events.py", line 528, in _sock_connect_cb
raise OSError(err, f'Connect call failed {address}')
ConnectionRefusedError: [Errno 111] Connect call failed ('<IP_ADDRESS>', 80)
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/config_entries.py", line 236, in async_setup
result = await component.async_setup_entry(hass, self)  # type: ignore
File "/config/custom_components/loxone/init.py", line 110, in async_setup_entry
request_code = await lox_config.getJson()
File "/config/custom_components/loxone/api.py", line 54, in getJson
version_resp = await requests.get(url_version,
File "/usr/local/lib/python3.8/site-packages/requests_async/api.py", line 11, in get
return await request("get", url, params=params, **kwargs)
File "/usr/local/lib/python3.8/site-packages/requests_async/api.py", line 6, in request
return await session.request(method=method, url=url, **kwargs)
File "/usr/local/lib/python3.8/site-packages/requests_async/sessions.py", line 79, in request
resp = await self.send(prep, **send_kwargs)
File "/usr/local/lib/python3.8/site-packages/requests_async/sessions.py", line 136, in send
r = await adapter.send(request, **kwargs)
File "/usr/local/lib/python3.8/site-packages/requests_async/adapters.py", line 58, in send
raise ConnectionError(err, request=request)
requests.exceptions.ConnectionError: [Errno 111] Connect call failed ('<IP_ADDRESS>', 80)

oh not sure that it matters but i have the new version off mini server with the build in Tree module

Have restart HomeAssistant after changing the port?

Yes i even tested with a new install of HA.
Did you ever had feedback off a user connecting the new MiniServer?

No I think the new Version is not supported because it communicates with TLS.

is there something we can do about it?
I am more then willing to test some stuff if you want

I just disabled the SSL certificate and now get a new error btw
2020-12-29 14:32:23 ERROR (MainThread) [homeassistant.config_entries] Error setting up entry PyLoxone for loxone
Traceback (most recent call last):
File "/usr/local/lib/python3.8/site-packages/h11/_state.py", line 249, in _fire_event_triggered_transitions
new_state = EVENT_TRIGGERED_TRANSITIONS[role][state][event_type]
KeyError: <class 'h11._events.ConnectionClosed'>
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/config_entries.py", line 236, in async_setup
result = await component.async_setup_entry(hass, self) # type: ignore
File "/config/custom_components/loxone/init.py", line 110, in async_setup_entry
request_code = await lox_config.getJson()
File "/config/custom_components/loxone/api.py", line 54, in getJson
version_resp = await requests.get(url_version,
File "/usr/local/lib/python3.8/site-packages/requests_async/api.py", line 11, in get
return await request("get", url, params=params, **kwargs)
File "/usr/local/lib/python3.8/site-packages/requests_async/api.py", line 6, in request
return await session.request(method=method, url=url, **kwargs)
File "/usr/local/lib/python3.8/site-packages/requests_async/sessions.py", line 79, in request
resp = await self.send(prep, **send_kwargs)
File "/usr/local/lib/python3.8/site-packages/requests_async/sessions.py", line 136, in send
r = await adapter.send(request, **kwargs)
File "/usr/local/lib/python3.8/site-packages/requests_async/adapters.py", line 48, in send
response = await self.pool.request(
File "/usr/local/lib/python3.8/site-packages/http3/interfaces.py", line 49, in request
return await self.send(request, verify=verify, cert=cert, timeout=timeout)
File "/usr/local/lib/python3.8/site-packages/http3/dispatch/connection_pool.py", line 130, in send
raise exc
File "/usr/local/lib/python3.8/site-packages/http3/dispatch/connection_pool.py", line 120, in send
response = await connection.send(
File "/usr/local/lib/python3.8/site-packages/http3/dispatch/connection.py", line 59, in send
response = await self.h11_connection.send(request, timeout=timeout)
File "/usr/local/lib/python3.8/site-packages/http3/dispatch/http11.py", line 58, in send
http_version, status_code, headers = await self._receive_response(timeout)
File "/usr/local/lib/python3.8/site-packages/http3/dispatch/http11.py", line 130, in _receive_response
event = await self._receive_event(timeout)
File "/usr/local/lib/python3.8/site-packages/http3/dispatch/http11.py", line 161, in _receive_event
event = self.h11_state.next_event()
File "/usr/local/lib/python3.8/site-packages/h11/_connection.py", line 444, in next_event
exc._reraise_as_remote_protocol_error()
File "/usr/local/lib/python3.8/site-packages/h11/_util.py", line 80, in _reraise_as_remote_protocol_error
raise self
File "/usr/local/lib/python3.8/site-packages/h11/_connection.py", line 427, in next_event
self._process_event(self.their_role, event)
File "/usr/local/lib/python3.8/site-packages/h11/_connection.py", line 242, in _process_event
self._cstate.process_event(role, type(event), server_switch_event)
File "/usr/local/lib/python3.8/site-packages/h11/_state.py", line 238, in process_event
self._fire_event_triggered_transitions(role, event_type)
File "/usr/local/lib/python3.8/site-packages/h11/_state.py", line 251, in _fire_event_triggered_transitions
raise LocalProtocolError(
h11._util.RemoteProtocolError: can't handle event type ConnectionClosed when role=SERVER and state=SEND_RESPONSE

Do you get the json config?
Try this in your browser:
http://<IP_ADDRESS>:8080/data/LoxAPP3.json
change the ip adress and port! Do you get the json?

Yes i do

I think I can not help you with this problem. I can not implement it because I have no Loxone Minserver v2 available. Can you implement it by yourself?

No i have no coding expierence, if you want i can make you a virtual machine with access to my miniserver if that would help?

But I need access to you miniserver and some developer tools installed (Ide, python, homeassistant).

i can setup all that, do you have discord or something to chat?

I have discord but I used it 2 or 3 times :-) not much experience what do you need?

add me as a friend
shilex#5389
If you cant find how to add me you can always join this server :p
https://discord.gg/NaYQXN9F

I think we can close this issue. The new gen2 is now supported.

I think we can close this issue. The new gen2 is now supported.

I encountered the exact same issue with Home Assistant 2021.3.3/4 and a 1g Miniserver. It had been working well for over a year, and after installing another integration this problem started to occur. When I disabled/deleted the integration, things went back to normal. I just wanted to share this.

I encountered the exact same issue with Home Assistant 2021.3.3/4 and a 1g Miniserver. It had been working well for over a year, and after installing another integration this problem started to occur. When I disabled/deleted the integration, things went back to normal. I just wanted to share this.

OK, I still get the exact error message as in the first post with version 0.3.4. and current Home Assistant version. No difference if I connect through dns.loxonecloud (Port 80) or directly on public ip (and public port).
Which other integration did you install and disabled to get it working?


Initial stab at the StaffingBot.

Coverage decreased (-0.5%) to 92.114% when pulling d07109ec122151befac7551a2e1aadcbf19d0f90 on staffing_bot into c660b2ad5d178df49c96ad8b750916e020e9ad46 on master.

Coverage decreased (-0.6%) to 92.222% when pulling 6e27e7019b38dd09c0fb80a60c1db8b74aa6f623 on staffing_bot into 816bfbdfc20513a0762f9fdba69d868e78d95d7e on master.

Coverage decreased (-0.6%) to 92.222% when pulling 6e27e7019b38dd09c0fb80a60c1db8b74aa6f623 on staffing_bot into 816bfbdfc20513a0762f9fdba69d868e78d95d7e on master.

lgtm after rebase

Coverage decreased (-0.8%) to 92.074% when pulling 6e27e7019b38dd09c0fb80a60c1db8b74aa6f623 on staffing_bot into 816bfbdfc20513a0762f9fdba69d868e78d95d7e on master.

Coverage decreased (-0.5%) to 92.346% when pulling 21d3d424cf9f7a35a9ee8152ffd3947c9819fa38 on staffing_bot into 816bfbdfc20513a0762f9fdba69d868e78d95d7e on master.


Submitter of Proxy Submission is being asked to agree to a blank deposit agreement for PMC

Step to replicate
the above screenshot was obtained by the following step:

prepare a proxy submission that will be sent to J10P AND PMC
log in as submitter, click submit --> prompted to agree to j10p agreement, which is correct, but there's an additional step to agree to a blank popup, this is not correct.

Suspected cause:

Ember is probably looping through the UI and displaying every repository's agreement text and asking for agreement for all of them without checking to see if there is actually any text in that field.  Such check should exist to avoid the above popup.

Closed by #804


Render anchor tags for hyperlinks

This gem's been working great for a project. I've recently needed to start rendering hyperlinks contained in docx files as anchor tags in html output, so made this change on a forked version but thought I'd put in a pull request in case it's useful for others.
The hyperlink targets get stored in a separate rels file, so the first step after unzipping the document is to extract those targets from the correct file and store them ready to be used later. Then if a text run is a hyperlink element, the .to_html method inserts the stored target into the href attribute of an anchor tag.

Closing because I've made a PR into the docx gem instead


Fix few typos in the docs

optimization are optional to optimizations are optional
bottlenck to bottleneck
accesible to accessible

Thank you for your contribution!


The payload decoded from the real QR is larger than example

I found you use QR code example from https://nzcp.covid19.health.nz/#examples, it succeed.
and I run with real QR code, an assert failed.

@zinwalin you're testing a live pass using example issuer and pub key. Please set IS_LIVE to true and then test your pass. Cheers.

This has been resolved.


Refactor publish workflow and pom

Refactor pom to include the nexus-staging-maven-plugin and to move the maven-gpg-plugin into the build tags.
This also refactors the publish workflow by removing the build settings action and to update the deploy command to pass in additional arguments.

:shipit:


certificate has expired

Description:
Since today i get the error certificate has expired when running the setup-java action in my github project. It runs on ubuntu-latest.
I have changed nothing in the setup for a longer time and it did not have any problems yesterday. I was running the step with java 8, then i tried version 17 (thought, maybe the version is the problem). Still the same.

Task version:
v2
Platform:

[x] Ubuntu
[ ] macOS
[ ] Windows

Runner type:

[x] Hosted
[ ] Self-hosted

Repro steps:
Just ran the setup task with an error response.
Expected behavior:
It should just install java.
Actual behavior:
I get the error response "certificate has expired".

Same happens here with:
- name: Setup Java and Dependency Caching
  uses: actions/setup-java@v2
  with:
    distribution: 'temurin'
    java-version: '11'
    cache: 'gradle'

I believe the problem is api.adoptium.net. Its certificate expired today at 12:29:15 CET.

.

Hello everyone, could you please check if it works for you now ?

Hello everyone, could you please check if it works for you now ?

Works for me now:

@dmitry-shibanov

Yes, works. Sorry for all the fuzz. Should i close the bug?

Thank you all for confirmation.
Do not worry @dready. Thank you for the fast report of the issue. I think the issue can be closed.


update msquic to 1.7

Why do we have 2 1.7 prerelease branches in this repo? https://github.com/dotnet/msquic/branches :confused:

Why do we have two 1.7 prerelease branches in this repo? https://github.com/dotnet/msquic/branches

One we polluted with additional changes. Both are gone now.


Prettify README

You can see how it looks with the changes by going here:
https://github.com/int19h/RTVS/tree/readme

:+1:


ETH holdback warning OR ETH-withdrawal-purchase

Hi
In attempting to withdraw USDT I realized I had insufficient ETH. Urgh. This got me thinking.

ETH holdback warning
You could warn someone, when they are about to trade out of their last bit of ETH that they will have insufficient ETH left for any withdrawal. I seem to think I saw some message like that when I traded but it's not ideal.

ETH-withdrawal-purchase
My preferred option is a cleaner one because it will cover us whenever we have more than one coin we may want to withdraw. A button at the top of the Account Page that becomes available ONLY WHEN YOU HAVE LESS THAN 0.002 ETH available in your ETH wallet. That button must only appear to allow you to cover THE BALANCE of the required withdrawal cost.

In my example below, having 0.0001 ETH available... The button would say "Purchase 0.0019 Ether to cover withdrawal fee".

The reason for the hard-line Purchase calculated-limit is because some users may use this opportunity to purchase less than the minimum 0.005 ETH.
On the other hand, you don't want users to have to go back to the trading page and force them to buy 0.005 ETH just so they can cover 0.002 ETH.
How you allow the transaction is up to you. How to decide which pairing to trade on needs further discussion. This brings me to my final option..

UNLESS YOU HAVE A BUTTON NEXT TO EACH PAIRING WITH SUFFICIENT FUNDS. That seems even cleaner. Having said that, once the purchase is done through the chosen PAIR, all buttons next to all Assets disappear because the ETH-withdrawal-purchase is no longer needed.

I hope this is clear.
Thank you,
Pierre

My understanding of a withdrawal is incorrect, but this got me thinking. You charge 0.002 as withdrawal cost, but the cost of a recent transaction for me was 0.0048 (in addition to the 0.002). That defeats the purpose. Surely. We are paying the network twice the amount of money than you receive. See transaction below. https://etherscan.io/tx/0x1b3d0ef2a34045db8e460dcdfde808504df8bdbae4e95d74453ca4ea9d02b9a7
I don't know if this is a DEX requirement, but surely you could simply get an approval from my wallet - as you do for login - then take the 0.002 from the loopring balance.
I would rather you take 0.005 than 0.002 and only pay the network when doing the actual withdrawal.

Please ask this question in https://discord.com/invite/KkYccYp
Thanks


Python models Dataproc Serverless setup with packages

Add description on how to setup python models with Dataproc Serverless using a custom image in order to use third-party packages.
What are you changing in this pull request and why?
In the context of running Python models in Spark using Dataproc, the documentation (python-models.md) says:

Installing packages: If you are using a Dataproc Cluster (as opposed to Dataproc Serverless), you can add third-party packages while creating the cluster.

I dug and found it is possible to run python models using third-party packages in dataproc serverless. It requires to use a custom docker image. This is very well documented on GCP's end. We currently run this in prod without any issue. I added this in the documentation. Let me know if you need more details on how to set this up.
Checklist

[x] Review the Content style guide so my content adheres to these guidelines.
[x] For docs versioning, review how to version a whole page and version a block of content.
[x] Add a checklist item for anything that needs to happen before this PR is merged, such as "needs technical review" or "change base branch."

Adding or removing pages (delete if not applicable):
N/A

hey @matthewshaver , wanted to check in on this -- are ok to review adn merge this pr?


Remove xfail for spark 3.1.0 test_broadcast_join_mixed FullOuter

Test is passing now so it was fixed somewhere along the line. Just remove the xfail.
fixes https://github.com/NVIDIA/spark-rapids/issues/575
Signed-off-by: Thomas Graves<EMAIL_ADDRESS>

build


Allow a custom SocketModeReceiver to be used with Socket Mode

Summary
This PR allows a custom SocketModeReceiver to be used with Socket Mode when initializing an app.
Reviewers
These changes can be verified with the following snippets:
Socket Mode receiver
This should now work fine! The custom route at http://localhost:3000/health and the message response should both respond with something expected.
/** Socket Mode receiver */
const receiver = new SocketModeReceiver({
    appToken: process.env.SLACK_APP_TOKEN,
    customRoutes: [
    {
      path: '/health',
      method: ['GET'],
      handler: (req, res) => {
        res.writeHead(200);
        res.end(`Things are going just fine at ${req.headers.host}!`);
      },
    },
  ]
});

/** App initialization */
const app = new App({
    token: process.env.SLACK_BOT_TOKEN,
    socketMode: true,
    appToken: process.env.SLACK_APP_TOKEN,
    logLevel: LogLevel.DEBUG,
    receiver
});

app.message('hello', async ({ say }) => {
  await say('hi!');
});

(async () => {
    await app.start(process.env.PORT || 3000);
    console.log("⚡ Bolt app is running!");
})();

Express receiver
This will still error with an AppInitializationError trying to run the app.
/** Express receiver */
const receiver = new ExpressReceiver({
    signingSecret: process.env.SLACK_SIGNING_SECRET,
});

receiver.router.use((req, res, next) => {
    console.log(`Request time: ${Date.now()}`);
    next();
})

receiver.router.get("/health", (req, res) => {
    res.writeHead(200);
    res.end(`Things are going just fine at ${req.headers.host}!`);
});

/** App initialization */
const app = new App({
    token: process.env.SLACK_BOT_TOKEN,
    socketMode: true,
    appToken: process.env.SLACK_APP_TOKEN,
    logLevel: LogLevel.DEBUG,
    receiver
});

app.message('hello', async ({ say }) => {
  await say('hi!');
});

(async () => {
    await app.start(process.env.PORT || 3000);
    console.log("⚡ Bolt app is running!");
})();

Requirements

[x] I've read and understood the Contributing Guidelines and have done my best effort to follow them.
[x] I've read and agree to the Code of Conduct.

@seratch @filmaj Thank y'all for the reviews! This was found while exploring for #1971 though doesn't directly address the issue.


Proportional text doubt

I am using a gauge chart and I see how to set the text size on the detail:
    detail: {
      valueAnimation: true,
      width: '90%',
      lineHeight: 30,
      borderRadius: 8,
      offsetCenter: [0, '-15%'],
      fontSize: 50,
      fontWeight: 'bolder',
      formatter: '{value}',
      color: 'auto'
    },

The problem with that, is that if the final user resizes the widget, the text do not scale with the content, and it can scramble everything. Is there a way to know the size of the widget, so fontSize can be calculated?

@otger That's a great question. You can get them from the echartsInstance. I will update documentation.
      width: echartsInstance.getWidth(),
      height: echartsInstance.getHeight(),

Let me know if it helps.

Thank you very much!

@otger Thank you for confirming. README is updated in #85.
Let us know if there is anything else and check out our YouTube tutorials: https://www.youtube.com/playlist?list=PLPow72ygztmQHGWFqksEf3LebUfhqBfFu


Cyprus Beta version final changes 28/07/21

@kazlauskis

Will you take the revised English text from the Species-data.xls sheet? Some of Ioanna’s requests for changes are already showing in the English version of the “insect-guide-photos” sheet, but she has asked for slight changes to others.
Are you happy with the final list of habitat types for Cyprus? I hope this corresponds with what is listed as TRUE in the Species-data.xls sheet, as I have not edited this further.
I have changed the requested beetle photo in your Google drive but still need to edit one of the hoverfly photos once hearing back from Ioanna.

@CCarvell

Yes, I'll pull in any latest sheet changes to the app once you're done.
Yes, there was an issue in the app which showed UK AND Cyprus habitats when Cyprus was selected. This was fixed now. Otherwise, the habitat types in the sheet is more for Ioanna to comment on.
Thanks, let me know when the photos are ready and I will add these in.

@kazlauskis

I have made the requested changes to English text in the Species-data sheet. Relevant cells are coloured green to show where these edits occurred, more for my benefit to see that I have completed them all.
Habitats seem sorted thanks
New/ replacement photos for the insect guide are saved in your Google Drive:
hoverflies_4
beetles_3

Will you be able to send out a confirmation message and link once the Beta version is live?
many thanks


There is a full stop in 2.000, it should be 2,000 or 2000

Issue in ./help/using/features/traits/manage-folder-traits.md
PFA screenshot. It looks like its 2 folders, not 2000.

Thank you for raising this. We've updated the documentation with the correct formatting.


Fix go vet errors

Fix https://github.com/docker/docker/issues/14993

Please sign your commits following these rules:
https://github.com/docker/docker/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "master" git@github.com:chrisseto/docker.git somewhere
$ cd somewhere
$ git commit --amend -s --no-edit
$ git push -f

Ammending updates the existing PR. You DO NOT need to open a new one.

LGTM, thanks @chrisseto
windows janky won't turn green tho :(

@runcom Any chance its probabilistic?

Could be a networking error? But those addresses look like test fixtures.
Thoughts? ¯_(ツ)_/¯

Yes don't worry about it :) there are some issues with the ci servers today

:+1: Cool, so rerun the tests tomorrow or something?

No need ;-) LGTM thank!


Windows 1.3.10 fails to establish connections to CIDR and DNS resources

Looks like the connection intents never make it through? Possibly related to new control protocol or TCP DNS.
connlib.2024-11-04-16-43-10.zip

Likely duplicate of #7200.

Hmm I'm not so sure. This issue is preventing connections to CIDR and DNS resources. The Internet Resource works fine for me.
It seems that the connection intents never make it to the Gateway(s)?

Connection to the portal is fine.

Only in the beginning, later down the line we see "Connecting to portal" and then nothing there after. This is also what we have been seeing in #7200.

Doesn't reproduce on my VM :(

I don't think this is fixed, I just ran into this on my Windows VM.

firezone_logs_2024_11_07-00-39.zip


Как добавить размер бумаги в pmp файл?

How add custom page size in pmp file?

How do I add custom page size?

@Doublefishc3d  @wzfxue
There is no direct way to add an custom paper size in the PMP File. But it's not impossible. I've done it through customizing the PiaNode Class. First try understanding the Media Object by adding few custom papers manually in the PMP file and debugging it. In the PiaNode class the properties like ChildNodes, Values, Parent, Name are all protected. We need to make it public to access it through the PlotterConfiguration Object
Hope You Understand. If you have any questions, comment below.
Thank You

While debugging you'll find a ChildNode called 'udm' which stands for UserDefinedMedia. This is where you should add your Custom Paper Sizes


VAE Experiment uses old run_experiments function

The README.md also uses the old one.

Hi @albertwujj , thank you for reporting this issue ! Now it should be fixed.

Great! Easy issue now: README says

Oh, I see you removed the last parameter from the README. But the next two are required params.

Hi @albertwujj , sorry for the very late reply. If I got it correctly, do you mean that in the documentation, we should add logdir as the fourth argument to the run function as it is ?


\IDA_scripts\IDA_flowchart\IDA_flowchart.py: module 'idc' has no attribute 'Exit'

when running the IDA FlowChart plugin over the IDBs of the Dataset-Vulnerability, an error occured:
.../binary_function_similarity-main\IDA_scripts\IDA_flowchart\IDA_flowchart.py: module 'idc' has no attribute 'Exit' Traceback (most recent call last):   File "...\IDA_Pro_v7.5\python\3\ida_idaapi.py", line 616, in IDAPython_ExecScript     exec(code, g)   File ".../binary_function_similarity-main/binary_function_similarity-main/IDA_scripts/IDA_flowchart/IDA_flowchart.py", line 197, in <module>     idc.Exit(0) AttributeError: module 'idc' has no attribute 'Exit'
How to solve this problem?

Hi @XX-Official,
thank you for opening this issue.
The IDAPro plugins that we have released have been developed for IDA Pro 7.3 Linux x86_64 and Python 2 (more info here). From the exception logs, I see that you're using IDAPro 7.5.
Looking at the IDAPro guide (https://hex-rays.com/products/ida/support/ida74_idapython_no_bc695_porting_guide.shtml), idc.Exit should be changed to ida_pro.qexit.
This is also related to: https://github.com/Cisco-Talos/binary_function_similarity/pull/9

The problem has been resolved. Thank you very much！

The problem has been resolved. Thank you very much！


Configure plugin versions (and perhaps other options) in one place

We have some pipeline that use the same plugin several times.  We lock each to a version, as recommended, but this leads to a lot of duplication and unnecessary noise in the yml file.
Would it be possible to introduce a mechanism for specifying the versions (and perhaps other configuration parameters) of plugins in a single location in the yml file?  I suppose I'm thinking of something along the line of a Maven pluginManagement sections.
Many thanks,
2m Steve.

@twometresteve super cool suggestion! Sorry for the delay getting back to you about this 😔I'll bring it up with the team and see what we can come up with 🤓I'll leave this issue open until we figure something out.

I'm closing this for now, will document changes to the plugin system as they happen.


MH-13611 duplicate events fix

As I posted in Jira, This patch modifies the Duplicate workflow to add the parameters that were set by the user in the original event when this was created.
Thanks to @staubesv for the suggestion.

Travis having a problem with:
0.01s$ (! grep -rn ' $' etc)
etc/workflows/duplicate-event.xml:54:        <configuration key="property-namespaces">org.opencastproject.assetmanager.security,org.opencastproject.workflow.configuration</configuration>
The command "(! grep -rn ' $' etc)" exited with 1.

Maven doesn't give the error. Line too long for checkstyle?

This command checks if there is a space at the end of a line. GitHub highlights this as well.

Looks good to me and I can quash-merge this anytime.
As an alternative, @mliradelc, do you want to rebase this or want us to do that together?
This is a good example for cleaning up a pull request for submission if you are interested.
We could do that in a 10min video call together.

Yes we can do the video call :)
El El mar, 18 jun 2019 a las 11:03, Lars Kiesow<EMAIL_ADDRESS>escribió:

Looks good to me and I can quash-merge this anytime.
As an alternative, @mliradelc https://github.com/mliradelc, do you want
to rebase this or want us to do that together?
This is a good example for cleaning up a pull request for submission if
you are interested.
We could do that in a 10min video call together.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/opencast/opencast/pull/955?email_source=notifications&email_token=AB5LBNAPNVBAPMB7XINQBIDP3CQFDA5CNFSM4HYGKMZ2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODX5WTBY#issuecomment-503015815,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AB5LBNAGD2HSAV2HBBBNGCLP3CQFDANCNFSM4HYGKMZQ
.

--
Maximiliano Lira Del Canto
Telematikingenieur
Handynummer: +49 163 863 00 86
Köln, Deutschland.


Uncaught Error: Syntax error, unrecognized expression: :nth-child(…)

When I initialize the slider, and I specify the starting slide, the script does not check the input value to the type of data
    carousel.itemslide({
        one_item: true,
        start: start_pos
    });

So if I, for example, pass by localStorage I should convert the value yourself.
        parseInt(start: start_pos)
It is not obvious and can cause many problems to the user.

#70


Fix build failed due to missing AndroidNdkModules

AndroidNdkModules.cmake has been deleted by NDK

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.tao.pei seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

@jypeitao which version of NDK had this file removed?

Change user and further confirm the problem.

C++ build system [configure] failed while executing:
/usr/bin/cmake
-H/home/peter/wk2/os/OpenXR-Tutorials/Chapter5
-DCMAKE_SYSTEM_NAME=Android
-DCMAKE_EXPORT_COMPILE_COMMANDS=ON
-DCMAKE_SYSTEM_VERSION=27
-DANDROID_PLATFORM=android-27
-DANDROID_ABI=x86
-DCMAKE_ANDROID_ARCH_ABI=x86
-DANDROID_NDK=/home/peter/Android/Sdk/ndk/23.1.7779620
-DCMAKE_ANDROID_NDK=/home/peter/Android/Sdk/ndk/23.1.7779620
-DCMAKE_TOOLCHAIN_FILE=/home/peter/Android/Sdk/ndk/23.1.7779620/build/cmake/android.toolchain.cmake
-DCMAKE_MAKE_PROGRAM=/usr/bin/ninja
-DCMAKE_LIBRARY_OUTPUT_DIRECTORY=/home/peter/wk2/os/OpenXR-Tutorials/Chapter5/app/build/intermediates/cxx/Debug/3v245142/obj/x86
-DCMAKE_RUNTIME_OUTPUT_DIRECTORY=/home/peter/wk2/os/OpenXR-Tutorials/Chapter5/app/build/intermediates/cxx/Debug/3v245142/obj/x86
-DCMAKE_BUILD_TYPE=Debug
-DCMAKE_FIND_ROOT_PATH=/home/peter/wk2/os/OpenXR-Tutorials/Chapter5/app/.cxx/Debug/3v245142/prefab/x86/prefab
-B/home/peter/wk2/os/OpenXR-Tutorials/Chapter5/app/.cxx/Debug/3v245142/x86
-GNinja
from /home/peter/wk2/os/OpenXR-Tutorials/Chapter5/app
CMake Error at CMakeLists.txt:97 (include):
include could not find requested file:
AndroidNdkModules

Found cmake in another path instead of Android sdk.
Add cmake.dir=/home/peter/Android/Sdk/cmake/3.22.1 will be OK. But it needs to be added manually.
I think even if it is cmake under another path, gradle can also add the cmake Modules path under android sdk.
Anyway, close this pr first.

I'm running into this issues as well. I'm not very familiar with gradle. My fix was to make CMake aware of it in the CMakeLists.txt:
    # native_app_glue
    list(APPEND CMAKE_MODULE_PATH "$ENV{HOME}/Android/Sdk/cmake/3.22.1/share/cmake-3.22/Modules")
    include(AndroidNdkModules)
    android_ndk_import_module_native_app_glue()


legacy customer account example

WHY are these changes introduced?
Introduce legacy customer account flow example. We aim to keep this example for a quarter / till all the functionalities of customer account experience had been ported over to Customer Account API
special shout out to @frandiox for making example so much simpler!

@juanpprieto The test for upgrade is failing in CI but passing locally, if you know why?


Transfer config validator tests from pytests to unittest framework

We are moving away from pytest into supporting the unittest library.
This issue covers the migration from pytest to unittest for the config validator tests.

Already done


Add perf improvements to 3.0 release notes

@shirhatti you da real MVP

Random: Do we have any mention of the new distributed tracing support?

Random: Do we have any mention of the new distributed tracing support?

Sadly, no😭

Add it! First class support for the distributed tracing using the new w3c trace context standard

@shirhatti Except for "call stack" and "stack trace," we don't use "stack" very much. Ok to use "framework" instead? If not, I'll convert it back to "stack."

s/throughout/throughput


window.NotifyOthers / on_notify_data

Previously, this supported any kind of object like a handle, image, etc but that no longer works...
Panel 1
var m = fb.GetFocusItem();
window.NotifyOthers("test", m);

Panel 2
function on_notify_data(name, info) {
    if (name == "test") console.log(info.Path); // undefined
}

Plain strings/numbers/booleans seem to work OK.
I can workaround it as-is but the docs would need updating and it would need marking as a breaking change.

Regretfully, I don't think I can fix that...
Full story: each panel is a separate JavaScript context, so transferring data via NotifyOthers means transferring data between JS contexts. The problem is that all modern JS engines are rather strict about making such transfers, i.e. you can't just pass the object between contexts. I 'solved' it with a JSON wrapper, which only works for simple parsable objects.
I'll try delving again into SpiderMonkey docs to see how this can be achieved, though I'm afraid that such transfers might incur huge performance penalty.

Don't worry about it - I only use it in my own personal scripts. It's not required for anything I share. Because the only handles I ever pass are part of my library, I can pass a plain text query from one panel specifying the path and subsong index and the other side, I can use fb.GetQueryItems  to get the handle I need.

Haha, actually I've just discovered, that it's achievable and not really that hard to do =)
One limitation though: transfered objects should not be stored inside the callee, since it might lead to memory leaks (i.e. soft leaks): they might prevent some GC inside caller context until the object is freed in the callee context); and hard to trace behaviour: the object is shared, so if modified from one panel, it will be modified in all other panels as well

Note to self: use JS_WrapValue +  js::NukeCrossCompartmentWrappers

I would actually need to port my heavily modified version of Br3tt's JSplaylist to SMP to even use this so really do put this as the lowest priority. It's unlikely I'll be doing that anytime soon because I'd rather have a play trying to learn the new ES6+ stuff this component supports.


Finish the flint_base submodule, depreciate old roots and factor out more utils

Just a few additional things I wanted to do before finishing what I started in #61, this PR in part addresses Issue #62 by depreciating roots in the general case and asks the user to first convert to acb_poly()

Looks good. Thanks!


Upcoming panel shows Module tasks & lessons instead of all tasks & lessons

When switching from a module tab back to upcoming tab, the upcoming tab will only show the tasks and lessons of that particular module and not all modules.


could not get decompression stream: fork/exec /usr/bin/unpigz: exec format error when pulling an image

Describe the bug
I started to get this error when pulling images:
$ docker pull     us.gcr.io/xxx/yyy:branch-main
branch-main: Pulling from xxx/yyy
2f7952cfdde9: Extracting  10.14MB/10.14MB
53dc04ad3c83: Download complete
2931308e7452: Download complete
could not get decompression stream: fork/exec /usr/bin/unpigz: exec format error

The same happened for some other, public images.
To Reproduce
I don't know how to reproduce it. :(
Expected behavior
It should pull the image without an issue
Diagnostic report (REQUIRED)
OrbStack info:
Version: 1.6.3
Commit: 10d8fa4cb507142066b51b2febbd367720a5732a (v1.6.3)
System info:
macOS: 14.5 (23F79)
CPU: arm64, 10 cores
CPU model: Apple M1 Max
Model: MacBookPro18,2
Memory: 32 GiB
Full report: https://orbstack.dev/_admin/diag/orbstack-diagreport_2024-07-03T11-08-21.839677Z.zip

Screenshots and additional context (optional)
It seems that it's a variant of https://github.com/docker/for-linux/issues/302.
I had pigz installed from homebrew:
$ which pigz
/opt/homebrew/bin/pigz

...but without a binary in the expected (?) location:
$ /usr/bin/unpigz
-bash: /usr/bin/unpigz: No such file or directory

I created a symlink but it did not help.
What did help was a restart of OrbStack.

This is because you ran an image like tonistiigi/binfmt, which messes up the binfmt settings. These images are made for x86 hosts, not arm64, and they install emulators to run programs built for other architectures—including arm64, because the native architecture is x86 as far as they're concerned. Emulating arm64 on arm64 isn't going to work: that makes it impossible to actually run anything, because the emulator also needs to run!
OrbStack already emulates all common architectures used in Docker images, so there's no need for images like this.


Prepare V1

Description

This is a prep for the V1 release.
Core changes:

Reverse the abstraction hierarchy -> AbstractClient/Provider -> Harmonizer -> Provider
Decouple abstract provider for CMS client implementation
Generalize documentation
Decouple types from code
Refactor Contentful implementation
Strengthen type contracts

Related Issue

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[x] Chore and documentation

Checklist:

[x] My code follows the code style of this project.
[x] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[x] I have added tests to cover my changes.
[x] All new and existing tests passed.

fyi @inigomarquinez
This is a followup integrating #39 and docs + tests
In this PR I have implemented a way to add any valid CF config (including pagination properties)


Move rule 920300 from PL2 to PL3

As discussed in issue #2001 we want to move the rule 920300 from PL2 to PL3.

Thank you for implementing this. Merging now.


Azure Files does not support RA-GRS and RA-GZRS

There is a note in the Read access to data in the secondary region section that conflicts with a table in the Supported Azure Storage services
The note states:

Azure Files does not support read-access geo-redundant storage (RA-GRS) and read-access geo-zone-redundant storage (RA-GZRS).

The table still lists Azure Files in the GZRS/RA-GZRS and GZRS/RA-GZRS sections

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

ID: f618f40d-ae53-5c3b-d8bf-0e80614f66d3
Version Independent ID: 93083407-8f1e-b3e2-6a62-e30be5e1fbcb
Content: Data redundancy - Azure Storage
Content Source: articles/storage/common/storage-redundancy.md
Service: storage
Sub-service: common
GitHub Login: @tamram
Microsoft Alias: tamram

Thanks for reporting this. I have assigned the issue to the content author to investigate further and update the document as appropriate.

@artemislee88 - Thanks for your feedback. This discrepancy has been addressed.
#please-close


App is crashing when we do backup on device.

When we click on Level 3> Share Recovery Key 1> Backup on Device, the app crashes.
It is also happening on level 2> Share recovery key 2.

Verified this issue on upgraded app v2.0.80(483)


Update Composer dependencies (2021-12-25-00-11)

Visual regression test passed!
View the visual regression test report


Update download-and-copy-artifacts-using-the-command-step.md

formatting error from migration.
Harness Developer Pull Request
Thanks for helping us make the Developer Hub better. The PR will be looked at
by the maintainers.
What Type of PR is This?

[ ] Issue
[ ] Feature
[x ] Maintenance/Chore

If tied to an Issue, list the Issue(s) here:

formatting

House Keeping
Some items to keep track of. Screen shots of changes are optional but would help the maintainers review quicker.

[ ] Tested Locally
[ ] Optional Screen Shoot.

Preview environment: https://ci.pr.harness.io/pr-453


How many rtmp connections a single Origin server can take?

I have been testing it with many features, and its a wonderful product.
Is there some where a hard limit or soft limit where a single origin server can take max limit of rtmp connections?
I have a cloud vm 2 cores - 4 GB RAM with 500 Mbps bandwidth. After 42 connections with bitrate of 5000k+ its not accepting any more connections. The publishing stream times out.
Please reply.
Thanks

Did a benchmark. Seems good.

@xoraingroup there is a max_connections config parameter. default is 1000.


Enforce explicit Some around raw values

Since a ron::value::RawValue can contain anything, implicit Some must be made explicit around it.
* [ ] I've included my change in CHANGELOG.md

Codecov Report
Patch coverage: 100.00% and no project coverage change.

Comparison is base (ba66c8e) 96.11% compared to head (bcdcfb9) 96.12%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the GitHub App Integration for your organization. Read more.
Additional details and impacted files
@@           Coverage Diff           @@
##           master     #494   +/-   ##
=======================================
  Coverage   96.11%   96.12%
=======================================
  Files          77       77
  Lines        9867     9890   +23
=======================================
+ Hits         9484     9507   +23
  Misses        383      383

Files Changed
Coverage Δ

src/ser/mod.rs
96.77% <100.00%> (+0.03%)
:arrow_up:

tests/407_raw_value.rs
100.00% <100.00%> (ø)

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


Eclipse plugin installation hangs

Expected behavior
As a IDEasy user, I want to automate my IDE installation with Eclipse including all required plugins so that I can develop with ease.
Actual behavior
Eclipse plugin installation stalls and the process does not seem to end.
The last thing I see in the logs when running ide -t eclipse is this:
Running command 'D:\projects\_ide\software\default\eclipse\eclipse\2024-09\eclipsec.exe' with arguments '-data' 'D:\projects\demo9\workspaces\main' '-keyring' 'C:\Users\hohwille\.eclipse\.keyring' '-configuration' 'D:\projects\demo9\plugins\eclipse\configuration' '-consoleLog' '-nosplash' '-application' 'org.eclipse.equinox.p2.director' '-repository' 'https://www.genuitec.com/updates/devstyle/ci/' '-installIU' 'com.genuitec.eclipse.theming.core.feature.feature.group' ...

BTW: standard windows taskmanager does not show any process named eclipsec.exe but I can find it in ProcessExplorer.
I killed the process and then IDEasy continued with the next plugin.
For the other plugins it did not hang but all plugin installations failed.
Steps to reproduce (bug) / Use Case of feature request (enhancement)

ide eclipse

Related/Dependent Issues
#652 - I assume I broke Eclipse commandlet.
Comments/Hints:
Affected version:

2024.09.002-beta-SNAPSHOT

From my trace logs:
HTTP Server 'Gateway Timeout': https://www.genuitec.com/updates/devstyle/ci/content.xml

So I think that we need to disable devstyle plugin as well. It also stopped working with Eclipse 2024-09 and prevents startup of Eclipse if the plugin installed. So we need to disable it.
Still need to analyze why the other plugins could not be installed.


Error: control transfer failed: LIBUSB_ERROR_PIPE

I have compiled and installed the library (and followed all required configuration steps) but I'm not able to run the tests. This is the output I see when running seek_test:
Error: control transfer failed: LIBUSB_ERROR_PIPE
Error: control transfer failed: LIBUSB_ERROR_PIPE
Error: control transfer failed: LIBUSB_ERROR_PIPE
Error: control transfer failed: LIBUSB_ERROR_PIPE
Error: control transfer failed: LIBUSB_ERROR_PIPE
Error: init_cam failed
Error accessing camera
Error: control transfer failed: LIBUSB_ERROR_PIPE
Error: control transfer failed: LIBUSB_ERROR_PIPE
Error: control transfer failed: LIBUSB_ERROR_PIPE

Any ideas what could be the issue? I had a look at the code and checked the type of error, it seems that the init commands sent to the camera are not correct. I'm using a SeekThermal CompactXR and running on Ubuntu 18.04

Same issue for me, running on Ubuntu 16.04

Did you find a workaround for this issue..? I'm facing a similar issue with Seek Compact Pro and Ubuntu 18.04.
Camera works well if started within a few seconds of plugging in, but results in this error if started later after plugging the camera to the USB (10 secs or later).

i had the same issue on raspberrypi3.
so i connected another USB port as a workaround.

My colleague helped me to find a workaround. Apparently the camera went to auto suspend mode and could not be started. Disabling USB auto suspend on Ubuntu helped to get the camera running successfully in all the cases. Guide: https://askubuntu.com/questions/1140925/how-can-i-disable-usb-autosuspend-on-ubuntu-18-04


Make the connection invitation in the issuer a URI

Per @sukalpomitra the invitation from the issuer is not a URI - it's just the encoded invitation. That seems to work fine with the Streetcred agent, but is not correct.  Please verify and if that is correct, adjust and verify it still works with Streetcred and with OSMA.
Example URI: http://iiwbook.vonx.io?c_i=gsgsgsgsg
What the issuer sends: gsgsgsgsg
Where gsgsgsgsg is the encoded invitation JSON.

Link to relevant Aries RFC section: https://github.com/hyperledger/aries-rfcs/tree/master/features/0160-connection-protocol#standard-invitation-encoding

Fixed in #127

I could successfully go through the issuance process using StreetCred.
With OSMA I am now able to read the invitation and successfully establish a connection, however the flow stops when the issuer sends the credential offer as it appears that it is not received/processed by OSMA.

@sukalpomitra the above scenario happens when running in localhost using ngrok.
I also tried the identity-kit deployed to OpenShift:

Connection remains in Negotiating state
A credential offer is received while the connection is still in Negotiating
Accepting the credential offer updates the credential exchange status to Requested
The flow seems to get stuck and breaks here

@esune Everything worked for me


Fix local sessions and add test to ensure session preservation

Completes the session setup by ensuring the rendering context is including session context
Refactor some HTML validation into a friendly HTMLValidator interface (This is the bulk of the change!)
Refactor test_search_blueprint.py to use the new HTMLValidator
Add test_user_stays_logged_in to ensure session preservation

Again, this only covers file system caching. This should now work in a deployed instance, as long as there is only one instance in the environment (as is the case for dev ).

I would suggest you read and understand the HTMLValidator and that question will be answered. If you find it confusing, please ask questions about the code that I can clear up for you.

Ok, great. I just wanted to make sure you understood the framework there since you're likely to use and/or expand it in the future, so didn't want to give any shortcuts through that one 😄.


Fix to retry for HttpStatusBasedRetryStrategy when it gets exceptions…

… while processing response
Motivation:
Currently, HttpStatusBasedRetryStrategy ignores exceptions in the result.
It is common to have exceptions like ClosedSessionException, which should be retried
since they can be random socket closures, so it seems like a bad default to ignore
the exceptions.
Modifications:

Change to retry on any Exception
Change not to look up the response contents to retry by default

Result:

Retry on any exception

To-do:

Provide knobs for the RetryStrategy

I'm wonder if it's a good idea to replace Function<HttpStatus, Optional<Backoff>> to Function<? super Exception, Optional<Backoff>> and convert an HttpStatus to an HttpStatusException. This way, a user doesn't need to specify two functions, i.e. easier to compose.

I think it is a really good idea to have just one Function. But I have a doubt about converting the HttpStatus to an HttpStatusException. Because I think the HttpStatusException has to represent an Exception but it may not.
How about taking a BiFunction which is BiFunction<HttpStatus, ? super Throwable, Optional<Backoff>> or just Function<Object, Optional<Backoff>> ??

I think Function<? super Exception, Optional<Backoff>> is fine:
Function<? super Exception, Optional<Backoff>> fn = cause -> {
    if (cause instanceof HttpStatusException) {
        ...
    } else if ... {
    }
    return Optional.empty();
};

However, it would be pretty painful to write this from user's standpoint, so I think BiFunction approach may be better: BiFunction<HttpStatus, Throwable, Optional<Backoff>> and yeah, it's probably OK to omit ? super for Throwable.

Another question is: should we return Optional<Backoff> or just nullable Backoff?

I forgot why I brought the Optional here instead of nullable Backoff. Nullable Backoff looks just fine here and I actually prefer it.
But It is a breaking change, so I'm not sure if it's worth it. What do you think??

Using BiFunction already guarantees a breaking change, so I see no issue with breaking more. ;-)

Alright then I will change it~! Thank you~!

Thanks, @minwoox !


Travis and buildbot integration

Discussions started in #459 and #456 to be continued here

Continuing discussion:

What's the deal with buildbot ? Once configured it starts automatically when the host is online or do I >have to start it manually ?

Usually it's configured to start on boot.

If so, within the embedded devices that I've got, the Rpi 2 is the one I turn on most frequently.
As for PCs, Mint is almost always online. The others need to be started "on demand" (even though I find >myself playing pretty frequently on Windows lately... :confounded: )

It's best to only add hosts that are on 24x7, otherwise they have a tendency to get behind on builds, and then you have to remember to go flush the queue.

@nomis52 then unfortunately I don't have 24/7 devices online. If you want though I can offer my fun park machines ( :smile: ) as "medium rate" contributing slaves.
I've seen in your waterfall that there are already some not-always-online slaves. Am I correct ?
@peternewman I think I managed to silence the false-positive results of Travis with this 410c3c26800c3f69da430ab6e057b48723dcbe58 and this 3b2ed24d6e232dc0ea0b7e6bc05e193a87857313
Basically the timer test is not performed when building with Travis
I've seen that the qlcplus buildbot has been setup ! Many thanks for that !
Where can I monitor the builds it produces ?

Alright: https://qlc-buildbot.openlighting.org/ is up and running. We'll need to setup the slaves now.
What do you mean by "medium rate" machines?

What do you mean by "medium rate" machines?

I mean machines not 24/7 online but upon request, maybe 6-7 hours a week ?
If you let me know on which of the devices I mentioned you are interested in, I'll make sure to setup buildbot on them and get them online a decent number of times per week

@nomis52 can we move it to https://qlcplus-buildbot.openlighting.org/ just in case... and to match the config/repo.

Done.
@mcallegari it really depends on:
i) how many changes you have
ii) the time it takes for each build
Lets add some of your slaves and see how they go. Choose the two that would be the most useful and follow the instructions on https://wiki.openlighting.org/index.php/OLA_Buildbot
For QLC+ use buildmaster.openlighting.org:9990

@nomis52 if we add the is_slow option to @mcallegari 's bots, they'll have fewer builds to do and may manage to keep up hopefully.
@mcallegari I think I have a slightly nicer workaround for Travis; expect a PR shortly.
@mcallegari I've now got our Ubuntu slave working fine, and the Mac one is almost there, apart from it's failing one test:QWARN  : InputOutputMap_Test::profileDirectories() QCoreApplication::applicationDirPath: Please instantiate the QApplication object first
FAIL!  : InputOutputMap_Test::profileDirectories() 'dir.exists() == true' returned FALSE. ()
Loc: [inputoutputmap_test.cpp(491)]

Digging further @mcallegari that test seems to be failing because Library/Application Support/QLC+ doesn't exist, but it's seemingly not trying to create it, which seems to me to be a slightly flawed test. Although I appreciate running the test without running the app is a bit odd, I could see people doing make && make check && make install before running it, but at that point the tests will fail because they've not run it.

@nomis52 Ok, I'll set up my raspberry 2 and my Mint 17 as soon as I have some time
@peternewman Thanks for spotting the test. I think it's simply because when you develop QLC+ you surely did make install and ran the app before running the test units :smile:
On OSX the create-dmg script doesn't actually run the test units. They are checked only during the Linux build, which is normally the first that I pack.
I would say we can add this case to the "IS_TRAVIS" workaround && "are we on OSX" or eventually don't even run the tests on OSX

@mcallegari I'd say the reverse actually, there's no point installing something that's broken. So if I'm going to bother running the tests at all, I'd make check, then make install.
Also regarding your dmg script, aside from/before Buildbot, how did you know there weren't Mac specific bugs/test failures if the unit tests are only run on Linux? Personally I'd say it should be run on all platforms where possible.
Not running the tests on OSX via Buildbot completely misses the point of unit tests and Buildbot. All that means you're doing then with Buildbot is checking there are no syntax/complilation errors against the OSX libraries. I'm guessing I can fix it by just creating that folder or similar, but I'd still be inclined to fix the test somehow (work in a scratch folder, create the real folder or something similar).

@peternewman you are absolutely right, but please consider that I didn't have a real Mac until the other day.
Before I was using a crappy iAtkos image, without the proper hardware support and many other limitations.
So I didn't even attempt to run test units on such an unreliable system. I've always relied on users feedbacks.
It's a fact though that I've done 28 releases in this way and I can't say OSX users are unhappy :wink:
Now a few things have changed: I've got a Mac Mini, there are Travis and Buildbot. Please bare with me in these settlement days :smiley:
I'll sit down on my Mac and review the test units on OSX. I agree that some tests must be refined.

@peternewman so, I've started my adventure into buildbot slavery :smile:
I'm a bit confused by test units though:
1- I don't understand why Travis build is OK while this slave fails:
https://qlcplus-buildbot.openlighting.org/builders/buildcheck-qlcplus-ubuntu-tahr-i686-noopenslp/builds/33/steps/unit tests/logs/stdio
2- I've setup a slave myself and it fails at the first test requiring a running X server. Are you aware of any solution/workaround to solve this ?
https://qlcplus-buildbot.openlighting.org/builders/buildcheck-qlcplus-mint-rebecca-x86_64-1/builds/1/steps/unit tests/logs/stdio
3- I've had a look at the test failing on OSX. Actually the underneath code is supposed to create the requested directory. See here:
https://github.com/mcallegari/qlcplus/blob/master/engine/src/qlcfile.cpp#L246
I'll try it manually on my Mac to see if something clearly wrong is going on.

Yeah ! After 5702ac6af7dd006b2b6d8232bedee855ef04fc2b my Linux slave reaches the end successfully.
I am now setting up my RPi 2 slave as well. Will be online soon.
Next step is fixing the unit test(s) failing on OSX.
@nomis52 I noticed that your ubuntu-tahr slave is failing the timer unit test. Are you running it on a virtual machine ? If so, how can we fix that in this case ? Is it even possible to detect that we're running on a VM ?

It's not on a VM but it is a fairly old machine:
Athlon(tm) XP 2200+
MemTotal:  474748 kB


[TC-411] Traffic Router Log Aggregation

Aggregate Traffic Router logs into one minute intervals
Breaking down bytes, requests,  time to serve (ttms), and users into various buckets allows for rapid charting for analytics, reporting, and monitoring
Sum of requests, count chi (user IPs)
by Server
by Service
by CDN
by Type (HTTP Redirect or DNS Nameserver)
by Service (CDN service)
by Host (Traffic Router)
by Destination (Cache Group)
by Request Type (A or AAAA)
Other?
Author: Ryan Durfey
JIRA Link: https://issues.apache.org/jira/browse/TC-411

@ryandurfey
@ashish-timilsina

this is outside of the scope of TC. closing.

this is outside of the scope of TC. closing.


[MB-12493] frontend for quality assurance reports page

Jira ticket for this change
Summary
I added a new Evaluation reports tab visible to office users that takes you to a page that lists evaluation reports.
It shows any reports that have been submitted for the current move, as well as any draft reports created by the current user.
All of the buttons and links on this page (Create report, view report, download) are placeholders that do nothing.
So far the markup is more or less done, but there is no styling.

Setup to Run Your Code

💻 You will need to use three separate terminals to test this locally.
Terminal 1
Start the Storybook locally.
make storybook

Terminal 2
Start the UI locally.
make client_run

Terminal 3
Start the Go server locally.
make server_run

Additional steps
make db_dev_e2e_populate

log in as a QAE/CSR user
search for the move EVLRPT
go to the Evaluation reports tab for this move

Verification Steps for Author
These are to be checked by the author.

[ ] Tested in the Experimental environment (for changes to containers, app startup, or connection to data stores)
[ ] Request review from a member of a different team.
[ ] Have the Jira acceptance criteria been met for this change?

Verification Steps for Reviewers
These are to be checked by a reviewer.

Frontend

[ ] User facing changes have been reviewed by design.
[ ] There are no aXe warnings for UI.
[ ] This works in Supported Browsers and their phone views (Chrome, Firefox, IE, Edge).
[ ] There are no new console errors in the browser devtools
[ ] There are no new console errors in the test output
[ ] If this PR adds a new component to Storybook, it ensures the component is fully responsive, OR if it is intentionally not, a wrapping div using the officeApp class or custom min-width styling is used to hide any states the would not be visible to the user.

Warnings

:warning:

Files located in legacy directories (src/shared or src/scenes) have
been edited. Are you sure you don’t want to also relocate them to the new file structure?
View the frontend file org ADR for more information

Messages

:book:
:link: MB-12493

  Generated by :no_entry_sign: dangerJS against f604460c1a07ce17351bd6ca51a7d84ee1d0a0c7

I have been working off of this branch for a couple days now and am confident it is a good building block PR for the QA reports work. There is some styling and wording/copy tweaks needed (as was noted by Marty when creating the PR) but overall it has the functionality/pages/routing that other work we are doing will build off of.  In my opinion we should get this PR merged in as is (with fixed client tests) and address the remaining work on it and Leia's good feedback in follow on PRs.

Client tests are fixed now.
I think merging this and hitting the rest of the work in future PRs makes sense.
Still tinkering with the backend branch that this work is based on though, which we'll have to merge before this.

Really struggling to merge this due to a flaky test. I'm going to try using click({force: true}) in the cypress test, because it's failing to find an element despite it being clearly visible in the video.


Scores and Nightly History to Improve Release Comparisons

Currently, for every release some tables are produced to compare with the previous release.  These are based on single runs and do not factor operational variability into the equation.

Look at how to match up nightly run data with release data to improve variability calculations
Add ZScore's to releases like is done for nightlies

This task is somewhat obsoleted by recent improvements in adhoc testing.  There is a task (https://github.com/deephaven/benchmark/issues/176) for handling benchmarks that have higher variability that will be used for nightly runs that can also be used for release runs.


Convert JavaCorePipeline to use Packman

FYI @ethanbao @mvashishtha @shinfan @michaelbausor - I will start work on this.

This was done awhile back... and now we actually want to do the opposite! Closing.


itk::QuadEdgeMeshPolygonCell::PointIdsBegin should not be creating new point ids if no point is set

Description
The itk::QuadEdgeMeshPolygonCell::PointIdsBegin iterator method creates point ids, even if the point container is empty:
https://github.com/InsightSoftwareConsortium/ITK/blob/89d3c3c0158806ab44a33d3c76b436537864398e/Modules/Core/QuadEdgeMesh/include/itkQuadEdgeMeshPolygonCell.h#L202
Not sure about that behavior. Also, the Ids created seem to be random. Both the begin and end iterators should be investigated when this case happens.
Steps to Reproduce
1 Run the itkQuadEdgeMeshPolygonCellTest.cxx test with ctest -V -R itkQuadEdgeMeshPolygonCellTest -C Debug and check that the test passes: the iterator returns a value to some point id despite no point value has been set.
2. Debug and investigate if this behavior is correct; if it is, document it in the method declaration (header file); it is isn't provide a fix, and change the test accordingly.
Expected behavior
A begin iterator method should probably not create new point ids if there is no data in the point container.
Actual behavior
The begin iterator method creates new point data even if there is no data in the point container.
Reproducibility
%100.
Versions
Current HEAD: 89d3c3c0158806ab44a33d3c76b436537864398e.
Environment
Any.
Additional Information
Reported in https://github.com/InsightSoftwareConsortium/ITK/pull/4156#discussion_r1292905860.
Maybe the issue falls into the Design Impact Report category or maybe the behavior is correct but needs to be documented; have not investigated enough, but is seems counter-intuitive to create a point id if no point exists.

Reported in https://github.com/InsightSoftwareConsortium/ITK/pull/4156#discussion_r1292905860.

This new part of the test looks suspicious. I am not yet sure how the class should work (no points are set?). But that new without later TakeOwnership or delete triggers Valgrind defect
  // Test point container iterators
  auto * quadEdgeMeshPolygonCell = new PolygonCellType();

  PolygonCellType::PointIdIterator pointId = quadEdgeMeshPolygonCell->PointIdsBegin();
  PolygonCellType::PointIdIterator endId = quadEdgeMeshPolygonCell->PointIdsEnd();
  ITK_TEST_EXPECT_TRUE(pointId);
  ITK_TEST_EXPECT_TRUE(endId);

  PolygonCellType::PointIdConstIterator constPointId = quadEdgeMeshPolygonCell->PointIdsBegin();
  PolygonCellType::PointIdConstIterator constEndId = quadEdgeMeshPolygonCell->PointIdsEnd();
  ITK_TEST_EXPECT_TRUE(pointId);
  ITK_TEST_EXPECT_TRUE(endId);

Maybe this part should be temporarily commented out until the matter clears up?

Thanks for catching this @issakomi 🦅. PR #4167. I prefer not to comment the statements in the tests; the suspicious behavior is tracked here, and whenever somebody in the community takes the time to investigate they will need to change the test so that it passes.


General refactoring

It's probably easier to go through this pull request commit-by-commit.
@elliotcm and I did some general refactoring on what we deemed as low hanging fruit. Some of these may be controversial. Where possible, we tried to use idiomatic Ruby/Rails features and follow standards.

Apart from one comment on naming, :+1:

Error name changed, checked with @heathd.


jormungandr: 0.3.2 -> 0.3.3

Motivation for this change
https://github.com/input-output-hk/jormungandr/releases/tag/v0.3.3
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

@GrahamcOfBorg test jormungandr

@GrahamcOfBorg test jormungandr

@GrahamcOfBorg build jormungandr


Issues with schedules

Hi,
If I understand correctly the /config/appdaemon/apps/apps.yaml  can be edited to add user defined entries ?
    # daily
    - schedule: "daily, 07:30:00"
      reminder: "Good morning. Today is {day}, {date}, and it's {time}."
    - schedule: "daily, 12:00:00"
      reminder: "Good afternoon. Today is {day}, {date}, and it's {time}."
    - schedule: "daily, 17:00:00"
      reminder: "Good evening. It's {time}."
    - schedule: "daily, 21:30:00"
      reminder: "It's {time}. Good night. And sweet dreams."

I seem to be having issues as that in edited schedules do not announce.
For example if I add a schedule say for 15:00 it will not announce
    - schedule: "daily, 15:00:00"
      reminder: "Good afternoon. Today is {day}, {date}, and it's {time}."

I restart the appdaemon after every change to the config, I have also tried restart of HA, still the same.
Any thoughts?
AppDaemon 4
Current version: 0.4.3
alexatalkingclock 3.0.2

hi, are you still having issues running the app? I think you have some misconfigurations that we can figure out from your appdaemon logs. let me know if you want to debug remotely and I'll send you my contact info.


update Dockerfile(s)

remove old Dockerfile based on outdated Ubuntu 16.04
add Dockerfiles for recent version of Ubuntu, Debian as well as Alpine

May want to use Ubuntu 22.04 as the Ubuntu base, since it's now released and LTS.

May want to use Ubuntu 22.04 as the Ubuntu base, since it's now released and LTS.

It released a few days ago! The .deb packages for Ubuntu 22.04 are not available yet. I will make Dockerfile for Ubuntu 22.04 as soon as we have the packages ready.
Other than that, Ubuntu 20.04 is an LTS release too ;)


systemctl start osbuild-composer.service failed at centos8

-- Subject: Unit failed
-- Defined-By: systemd
-- Support: https://access.redhat.com/support
-- The unit<EMAIL_ADDRESS>has entered the 'failed' state with result 'exit-code'.
4月 30 09:17:10 arm-kojid systemd[1]: Failed to start OSBuild Composer Worker (1).
-- Subject<EMAIL_ADDRESS>单元已失败
-- Defined-By: systemd
-- Support: https://access.redhat.com/support
--<EMAIL_ADDRESS>单元已失败

Hello, can you write more details about this issue? When did it happen? What were you doing? Is the distribution CentOS Stream 8 or CentOS Linux 8?

After I yum install, It happened. And CentOS Stream 8 and CentOS Linux 8 all for this.
And I run "dnf update", and it work.
So I think composer to centos8 rpms version sometthing wrong


Validator tip selection

For validator tip selection, we will use a larger number of maximum parents.
This PR:

renames those parameters as they were confusing names before.
adds this simple validator tip selection to the block issuer in the test suite.

closes #406


[Question]: How do I recover my stats?

What is your question?
I clicked one of the buttons on the overlay to bind it to my top-right/left corners and the overlay disappeared. I tried to toggle hide/show overlay to get it to show again, but it didn't work. I tried to use Ctrl+Shift+D to reset the overlay to default so that it might show again, but this just zeroed out all my stats in the app.
I hope I didn't just erase everything, but is there a way to get the app to re-sync my stats again?

UPDATE: found out the key combo just filtered to today's stats and that swapping the filter back fixed it; my apologies!


PWSH 7.3.6 on Linux/Mac cannot find $pshome\System.Windows.Forms.dll

Prerequisites

[X] Write a descriptive title.
[X] Make sure you are able to repro it on the latest released version
[X] Search the existing issues.
[X] Refer to the FAQ.
[X] Refer to Differences between Windows PowerShell 5.1 and PowerShell.

Steps to reproduce
I have installed PWSH on Ubuntu 20.04 and Mac OSX (forget the exact version) for work to develop a powershell script with a portable gui (assigned task by manager) and i have been unable to find the System.Windows.Forms.dll outside of Windows, which drives the form for the gui.
i have searched via google and duckduckgo both this site specifically and the web in general, i have seen a few issues pointing to .net needing to be ported, but they all say that starting with 7.x a ported version is embedded in PowerShell.
i am sorry if this should be in .Net instead, but is there a reason i have missed on why System.Windows.Forms.dll is not ported?
Expected behavior
System.Windows.Forms.dll is in $PSHome in a fresh install

Actual behavior
System.Windows.Forms.dll is not present in $PSHome

Error details
No response
Environment data
not available (work computers are locked down. i had to search once i got home to find a solution)

Visuals
No response

WinForms, i.e. the APIs in the System.Windows.Forms namespace, are a Windows-only technology, and therefore by definition not available on non-Windows platforms.
It is legacy technology with deep roots in Windows and therefore won't be ported to other platforms (similarly, WPF will remain Windows-only).
You'll need to use cross-platform UI frameworks, such as Avalonia (there's also MAUI, but it lacks Linux support).

Thank you. that explains what i was unable to find.


Remix polyfill breaks certain streams

The installGlobals() polyfill is causing problems with newer remix version and certain http requests. Especially related to streams. Remix suggests that it's not necessary for node versions >=20. I wonder what the best strategy is to also support older remix versions with SST.
Currently, I patched the polyfill to installGlobals({ nativeFetch: true }) to make it work. Maybe that's the way forward until it can be removed completely?

This is for a deployed Remix site?

Yeah! So, here's my understanding of installGlobals:

installGlobals() installs a custom fetch polyfill the remix team maintains (https://github.com/remix-run/web-std-io). It has some slight differences to the real deal though.
in remix v2.9 they introduced installGlobals({ nativeFetch: true }) which uses the same fetch implementation node uses. This is especially needed for node versions < 20. As of node 20 node's fetch implementation is pretty solid and you might node need polyfills at all anymore.

See remix's changelog: https://github.com/remix-run/remix/blob/main/CHANGELOG.md#undici
My suggestion is to switch to installGlobals({ nativeFetch: true }) and document it somewhere. And when Lambda gets node 22 in the future it should be safe to drop the polyfill altogether.
Or maybe even add a flag to let users control the polyfill behavior?!

@klaemo just to clarify, does changing installGlobals() to installGlobals({ nativeFetch: true }) break for remix versions prior to v2.9?
If installGlobals({ nativeFetch: true }) works for everyone (node 18/20, remix prior/after v2.9), we probably don't need to expose the polyfill behavior to user?

just to clarify, does changing installGlobals() to installGlobals({ nativeFetch: true }) break for remix versions prior to v2.9?

It shouldn't, since that flag doesn't exist at all in those versions.

If installGlobals({ nativeFetch: true }) works for everyone (node 18/20, remix prior/after v2.9), we probably don't need to expose the polyfill behavior to user?

I agree that it's probably unnecessary to expose this to the user.


Convert between the API implictily absolute and DO standard record names

Previously, this DNS provider incorrectly assumed that DNS records
passed to it followed the standard notation, used by the DigitalOcean
API, that absolute DNS record names end with a dot, and relative DNS
record names don't.
This is incorrect because libdns uses de-facto implicitly absolute DNS
record names with a dot suffix, which causes any DNS record added with
eg certmagic to incorrectly be treated as relative, resulting in records
like: _acme-challenge.example.org.example.org
This fixes this issue by converting between the standard dot-suffix DNS
record name notation used by the DigitalOcean API and the implicitly
absolute record names used by libdns.
See: https://github.com/libdns/libdns/issues/12

Thanks for the PR! We will look at this closer as soon as we decide on the best path going forward in the linked issue.

I'm here from an issue raised in the wrong place due to record-name parsing bug! Glad to see the PR. Will this be fixed soon hopefully? :) Eagerly awaiting for this, thank you!

Awesome, +1

Ran into this today as well. I had to remove the digitalocean resolver reference from my caddyfile to get a cert to generate, which is leaving me without a wildcard at the moment. The txt hostnames present in the dns records are akin to: _acme-challenge.domain.tld.domain.tld -- which is obviously incorrect. Is there anything that is preventing this from going in that I'm missing from the discussion thread above or other context? @mholt

Ran into this today as well. I had to remove the digitalocean resolver reference from my caddyfile to get a cert to generate, which is leaving me without a wildcard at the moment. The txt hostnames present in the dns records are akin to: _acme-challenge.domain.tld.domain.tld -- which is obviously incorrect. Is there anything that is preventing this from going in that I'm missing from the discussion thread above or other context? @mholt

The discussion in the linked thread has not been completed yet. I am awaiting feedback from a sufficient number of users before committing to a decision. We need to make sure it serves the vast majority of DNS providers first.

The discussion in the linked thread has not been completed yet. I am awaiting feedback from a sufficient number of users before committing to a decision. We need to make sure it serves the vast majority of DNS providers first.

seems like there is a lot of people facing this issue
https://github.com/caddy-dns/digitalocean/issues/8

The upstream issue has been resolved; record names passed in are now relative to zone. Feel free to patch this library accordingly! :) These new helper functions might be useful: https://pkg.go.dev/github.com/libdns/libdns#AbsoluteName

given https://github.com/libdns/libdns/pull/28 , very LGTM

Thanks Sven!

Thank you @SvenDowideit ! 🙏