Content-Length may be missed in response header

Since ACCEPT_ENCODING: 'gzip, deflate' is the default in aiohttp for requests, the apache webdav server may add Transfer-Encoding: chunked to the HTTP header instead of Content-Length. In this case, there will be an error when executing the download_to method. This bug can be reproduced by downloading a 500 megabyte file

Thanks for such a quick fix!
Another improvement could be getting the total value from self.info?
Now its value can be None,
and if the file is compressed on transfer, then the content_length(and total) will be less than the size of the resulting local file.

Hi, thank to you for detect the error :).

Now its value can be None, > and if the file is compressed on transfer, then the content_length(and total) will be less than the size of the resulting local file.

I understand, but if I make a call to info, not only would it be one more call that you don't expect to happen, but also when reporting the progress, in case it is a compressed file, you would never have 100% on the progress, because the total is contrasted with the amount of bytes received. E.g. a 100B file compressed to 80B the last call to progress would be at most: progress(current=80, total=100).

As far as I understand, the current value is calculated on an already uncompressed buffer, isn't it?

Well, technically yes, you are right, I made the pertinent changes to the https://github.com/jorgeajimenezl/aiodav/tree/issue10 branch. I can't test it right now because I'll be busy for a few days, if you could test it to see if it works correctly it would be very helpful 😅

I will be on vacation next week, but after that I will definitely test your changes, thanks

I tested your changes - it works.
But I have a question - why so?

        try:
            total = int(info["size"])
        except:
            total = response.content_length

size can be omitted in server response for files?

Yes, that's right, I re-read the corresponding RFC and this is so, thank you very much for the clarification again, :)

Ok, thanks. Are you planning a release soon?

I released it today, the changes should be available in version 0.1.12

Ok, thanks. This one can probably be closed...


Add unit test for encrypted EBS volume-backed instance

This commit contains a unit test that ensures that the amazonec2 driver contains a flag for encrypting an EBS volume.
PR for the change this unit test is testing: https://github.com/rancher/machine/pull/63
Related Issue: https://github.com/rancher/rancher/issues/22691

On an upgraded rancher server from v2.3.5 to v2.3-head, Encrypt EBS Volume is not working. Created a bug https://github.com/rancher/rancher/issues/25546


NNotepad: UI update

Updated UI with following changes:

Add navigation bar for logo, WebNN sample links, show generated code, show documentation and device type selector
Add code editor to replace the  which can show code lines
More style tuning

@inexorabletash @huningxin @Honry Please take a look and let me know if it's acceptable or not.

@huningxin There was the typo in the svg logo, changed the text from "NNodepad" to "NNotepad" now, please refer to the updated new screen shot as attached.

The UI update looks good to me. I'll defer to @inexorabletash for decision. Thanks!

@inexorabletash @huningxin @anssiko
Thanks for the review and feedback, removed monaco-editor and vite-plugin-monaco-editor npm packages, and fixed issues captured by @inexorabletash. PTAL.

Thanks - fixes look good. I appreciate the effort to make the simple HTTP server work! It's unfortunate to have to have a copy of the monaco editor in the source tree - is there any way to pull that in via npm to ease maintenance?
I'm on vacation this week so I'll give a quick LGTM to unblock.

@inexorabletash Wishing you a fantastic break!
The new commit removed monaco editor local codes from source tree, use the js files from CDN<EMAIL_ADDRESS>instead. @huningxin @anssiko PTAL
Verify it worked via python3 -m http.server 8888
http://localhost:8888/nnotepad/index.html
Verify it worked via npm run start
https://<IP_ADDRESS>:5173/nnotepad/index.html

@inexorabletash Thank you, added the /* global monaco */ instead of ignoring the whole file for eslint. PTAL.


Use kafka client consistently

Agent v6 only has one instance per instances, so we don't need to have a
separate cache... just cache the client on the instance.
This cleanup will make life simpler when the check is refactored as part
of #3957.

This is ready for review and tests are passing


alptis/net-gearman no more available

Tried to change to appropriate library, guess it's brianlmoon/net_gearman, but without success

Why did you close this?
There still is a missing package / typo...

Dont know, it's was long long ago
@Khez If you need working copy I can provide


ovn-kube-node crash-loops in update to 4.13.0-0.okd-2023-05-22-052007

Describe the bug
The ovnkube-node ds does not rollout.
ovnkube-node-5xcpc     3/5     CrashLoopBackOff    8 (13s ago)    16m18s

The describe says that container ovn-kube-node has
    State:       Waiting
      Reason:    CrashLoopBackOff
    Last State:  Terminated
      Reason:    Error
      Message:   sbin/iptables -t mangle -C OVN-KUBE-ITP -p TCP -d <nil> --dport 110 -j MARK --set-xmark 0x1745ec --wait]: e
xit status 2: iptables v1.8.7 (nf_tables): host/network `<nil>' not found

and the ovn-controller has
    State:          Waiting
      Reason:       CrashLoopBackOff
    Last State:     Terminated
      Reason:       Completed
      Exit Code:    0

Version
4.12.0-0.okd-2023-04-01-051724 to 4.13.0-0.okd-2023-05-22-052007 on bare metal UPI.
How reproducible
always.  After deleting the pod the newly created pod has the same symptoms.
Log bundle
https://send.vis.ee/download/a215b5bf7b6eaab7/#lLphvJFSmv0NZXyptZJwZQ

Anything I can do to debug?

Investigating further, this was due to a service of type ClusterIP, that had internalTrafficPolicy: Local, which arguably does not make sense.

Good catch! That still looks like a bug worth filing to https://issues.redhat.com - OVN should not crash when a bad service is created

I filed https://issues.redhat.com/browse/OCPBUGS-21925.  I do think that the best fix would be to do both: not crash and don't allow these nonsensical services.


New Tutorial - moveit_py motion planning python api tutorial

Description
Work in Progress.

UPDATE: From further digging, It seems like the imports are wrong, because the script imports things a little differently. Also, the RViz config for the Jupyter example seems like it doesn't work:

The library name changed to moveit which is causing the import issue, I will update this.
I am aware of the rviz config issue, it relates to how the config files was copied over, I will fix this and address your comments.

Initial comments:

Is this redundant with #564?  If so, can you close out the other PR?
For others to test more easily, it might be useful if you left basic instructions in the PR with the launch files to execute.
The Python API script worked fine, but I didn't have enough instructions to figure out how to run the notebook. I managed to run the launch file and went over to the folder with the notebook, but then failed to import moveit_py. So I'm doing something wrong.

This may be related to the fact that even after doing source install/setup.bash on the workspace, I also can't just go into Python and import moveit_py ... yet the script launch file works?
UPDATE: From further digging, It seems like the imports are wrong, because the script imports things a little differently. Also, the RViz config for the Jupyter example seems like it doesn't work:

I can swap to another RViz config and this makes things show up fine. This is as far as I got before I gave up:

Hi @sea-bass,
These issues should now be resolved. I have separated examples into the jupyter_notebook_prototyping example and the motion_planning_python_api_tutorial, each example has a separate pr this one is for motion_planning_python_api_tutorial.
To run the tutorial you simply need to build moveit2_tutorials with this branch, source  install/setup.bash and then execute:
ros2 launch moveit2_tutorials motion_planning_python_api_tutorial.launch.py

This will run the following script.
It would be nice to add support for use of buttons like next for executing motion plans as seen in the moveit_cpp tutorials however I haven't had the chance to implement this yet.
Also apologies for the broken imports the last time, this was due to a breaking change on the pr I had open. I was asked to rename the library from moveit_py to moveit.

@henningkayser would it be possible to activate CI for this pr? It would be useful for checking if I need to amend anything, I have already installed and run pre-commit locally so I think the changes should pass CI.

@sea-bass html_proofer passes but there appears to be another CI failure.
Also thank you for your patience with this, I have been jumping from one pr to the next and have missed some obvious mishaps in previous reviews.


di-ruby-lvm 0.2.0 compatibility

We need to ensure we can upgrade to di-ruby-lvm 0.2.0, which currently breaks the cookbook.  See #80 for details

The gem di-ruby-lvm was updated lately to be able to process multiple file-systems better. A minor refactoring of the code was done and then released. Unfortunately the version of di-ruby-lvm (0.2.0) that was released had a method outside of a class within physical_volumes.rb. A new release was made and pushed to ruby gems (0.2.1) which fixes this issue.

I think this can be closed now that 1.5.0 is out.


"simd-data-layout" issue

I find the “simd-data-layout” option in the help  “onnx-mlir” command，it says “Enable SIMD optimization for convolution (default=false). Set to 'true' if you want to enable SIMD optimizations.”  But when I execute the following command to compile mobilenetv2: “onnx-mlir mobilenetv2.onnx --simd-data-layout”, an error is printed：“loc("Conv_4"): error: failed to legalize operation 'onnx.Conv' that was explicitly marked illegal”. Does anyone know what the reason is?

Same question here, wondering if that is due to the lack of support for vector extension of x86 cpu

Sorry, we have not implemented the SIMD code generation for CONV in onnx-mlir, in part because our accelerator handles it very well.
If you are interested in contributing a CPU SIMD implementation, I started doing a layout optimization to change a NCHW to a layout that split C into an outer C and an inner C (NxC1xHxWxC2) where C2 is sized to be SIMD friendly. I paper by Jiyuan Zhang et al entitled "High Performance Zero-Memory Overhead Direct Convolutions" is a refinement where H and W are also tiled.
I can help with the layout transformation if you are willing to write the accelerated CONV also.

Hi Thanks for replying! I was planning on codegen/C API calls for further lowering from krnl to llvm level. I will be checking more on this. Thanks!


ES index written must match reader side

Change this line https://github.com/lyft/amundsendatabuilder/blob/v1.0.11/example/scripts/sample_data_loader.py#L177 to match the current hardcoded value in the search service at
https://github.com/lyft/amundsensearchlibrary/blob/v1.0.0_4/search_service/config.py#L23. This means it’s backwards compatible with existing search service Docker containers etc.
Further details originally shared on the #troubleshoot Slack channel, now here as well:
————
... but, I think it's better to start figuring out how/why your ElasticSearch is failing, you want that working anyway, and debugging it will benefit everyone by hopefully eradicating a bug. What I did to make sure hostnames, ports, networking, docker containers? etc. was set up right was this:
curl amundsensearch:5000/search?query_term=test

to mimick the frontend querying the search service - (with host/port adjusted to your settings) - it should return a JSON response. If that fails then go back and query ElasticSearch directly e.g.:
curl localhost:9200/_search?q=test

Again, yes - you guessed it, adjust host/port to your settings. If you're not getting anything useful containing table_key entries etc. try curl 'localhost:9200/_cat/indices?format=json&pretty' to see the current name of the index (it get's replaced for each example databuilder run). E.g mine right now is tables40f17543-0435-4fcd-9b54-93482f80313a ...with that I can check my alias is setup to point to the current index and what it's all called:
curl 'localhost:9200/_cat/aliases?format=json&pretty'
[
  {
    "alias" : "tables_alias",
    "index" : "tables40f17543-0435-4fcd-9b54-93482f80313a",
    "filter" : "-",
    "routing.index" : "-",
    "routing.search" : "-"
  }
]

I setup my metadata service ELASTICSEARCH_INDEX = 'tables_alias' ... I beleve the default settings and naming might be simplified as hinted at in the original issue thread from here on: https://github.com/lyft/amundsendatabuilder/issues/21#issuecomment-484012219 ... but, alas, that's the current state of the codebase. Hope this helps ... somewhat! (Maybe not the clearest of explanations I've ever written .. but it's Friday well EOD at work here :D)

thanks @jornh for the detail debugging info. LGTM

Yes, it sometimes takes a frustrating amount of debugging for a one line change 😦 oh well ... software ... or problem solving in general ...
I was actually wondering how to have less of Elasticsearch internals bleeding out to other components and even in an asymmetric way (if you compare update from Databuilder vs search API)? Should the search service abstract away ES internals and have ingest API endpoints as well? Should there at least be an API endpoint to ask for a stable current alias name?
Alternatively, to what extent should it be hidden or abstracted away in Databuilder Loader/Publisher?

@jornh , maybe you could share a bit more on exposing ES  in two places concern that we are not aware of :) ? Are you talking about including an ingestion endpoint in search service for databuilder to use?
So databuilder is designed to be an Apache Gobblin like data ingestion library which user could write any source extractor, or any sink publisher. And the library could be imported in an Airflow DAG or even a custom cron-like env. And the search service is a proxy to serve search request which uses ES as default engine, but could be back by Apache Solr later. Hence making databuilder not aware of the search service could help to grow the library into a more generic way. Hopefully that helps to answer your concern :) cc @jinhyukchang

Hence making databuilder not aware of the search service could help to grow the library into a more generic way.

Yes, that framing certainly puts a different perspective on databuilder. I guess what fooled me into that train of thinking was it being included under the Amundsen naming umbrella etc. instead of it being a lower level thing “closer to the metal” more in its own right. Perhaps longer term worth explaining more — somehow...
Thanks! 💯 Let’s leave the discussion there ...


Add to host request

Hey! We at Civilized Hosting LLC use this egg! We wanted to request being added to the page.
Civilized Hosting LLC - https://civilizedhosting.net

Added!


Missing indentition when using pattern matching via anonymous functions in a pipeline

When using function in a pipeline the actual patterns are not indented to the function keyword but instead are kept inline with it.
I looked into FormattingConventions.md, but have not anything directly related to this issue. The best i could find was the rule

Pattern matching of anonymous functions, starting by function, are indented with respect to the function keyword:
List.map (function
          | Abs(x, body) -> 1 + sizeLambda 0 body
          | App(lam1, lam2) -> sizeLambda (sizeLambda 0 lam1) lam2
          | Var v -> 1) lambdaList

Example
let expected =
    b
    |> function
       | Some c -> c
       | None -> 0
    |> id

let actual =
    b
    |> function
    | Some c -> c
    | None -> 0
    |> id

In this case are you talking about the indent that's suggested when you press  after the function keyword, or the formatting that occurs when you format the entire document?

after formatting the entire document

Can you raise an issue on the fsprojects/Fantomas repo please? We use that library to do the formatting so any fixed need to happen there first.
In the meantime, if the formatting is causing you problems it can be disabled by adding the editor.formatOnSave: false setting to the [fsharp] section of your vscode settings.json.

will do


Added Reddit

Fix for #15

Have to edit on shellphish.sh too.

Can you be more specific on what to edit? Because, I have tested it and it works. So I doubt on what to edit.


register_devices: Fastlane rejects new device id for iPhone XS

New Issue Checklist

[X] Updated fastlane to the latest version
[X] I read the Contribution Guidelines
[X] I read docs.fastlane.tools
[X] I searched for existing GitHub issues

Issue Description
Attempting to add a device for iPhone XS with the following uuid format:
0000XXXX-000834C81152002E                       Seiken's Iphone X
Based on a couple apple support articles:
https://forums.developer.apple.com/thread/108871
https://forums.developer.apple.com/thread/108848
let me know if you need anything else.. Thanks!

 [xxx-xxx-build] $ /bin/sh -xe /var/folders/j1/01gnpxr51mq07j1m3cf_y3b00000z8/T/****4018451021139750437.sh
+ bin/fastlane write_****_project_props
[13:57:47]: WARNING: fastlane requires your locale to be set to UTF-8. To learn more go to https://docs.fastlane.tools/getting-started/ios/setup/#set-up-environment-variables
[13:57:49]: ----------------------------------------
[13:57:49]: --- Step: Verifying fastlane version ---
[13:57:49]: ----------------------------------------
[13:57:49]: Your fastlane version 2.117.1 matches the minimum requirement of 2.117.1  ✅
[13:57:49]: ------------------------------
[13:57:49]: --- Step: default_platform ---
[13:57:49]: ------------------------------
[13:57:49]: Driving the lane 'write_****_project_props' 🚀
+------+----------------------------+-------------+
|                fastlane summary                 |
+------+----------------------------+-------------+
| Step | Action                     | Time (in s) |
+------+----------------------------+-------------+
| 1    | Verifying fastlane version | 0           |
| 2    | default_platform           | 0           |
+------+----------------------------+-------------+
[13:57:49]: fastlane.tools finished successfully 🎉
[13:57:49]: WARNING: fastlane requires your locale to be set to UTF-8. To learn more go to https://docs.fastlane.tools/getting-started/ios/setup/#set-up-environment-variables
[EnvInject] - Injecting environment variables from a build step.
[EnvInject] - Injecting as environment variables the properties file path '****.props'
[EnvInject] - Variables injected successfully.
[description-setter] Description set: full 1.0.0+2414.developmentdevelop e43b74dd
[xxx-xxx-build] $ /bin/sh -xe /var/folders/j1/01gnpxr51mq07j1m3cf_y3b00000z8/T/****3567267503844752331.sh

'[' -e set_env.sh ']'
./set_env.sh development full
Linking .env configuration -> .env.development
Setting build_props.productFlavor = "full"
[hcs-x01-build] $ /bin/sh -xe /var/folders/j1/01gnpxr51mq07j1m3cf_y3b00000z8/T/****6381959432240629416.sh
security unlock-keychain -p **** /Users/****/Library/Keychains/login.keychain
bin/fastlane ios ****_distrib flavor:full
[13:57:51]: WARNING: fastlane requires your locale to be set to UTF-8. To learn more go to https://docs.fastlane.tools/getting-started/ios/setup/#set-up-environment-variables
[13:57:51]: ----------------------------------------
[13:57:51]: --- Step: Verifying fastlane version ---
[13:57:51]: ----------------------------------------
[13:57:51]: Your fastlane version 2.117.1 matches the minimum requirement of 2.117.1  ✅
[13:57:51]: ------------------------------
[13:57:51]: --- Step: default_platform ---
[13:57:51]: ------------------------------
[13:57:51]: Driving the lane 'ios ****_distrib' 🚀
[13:57:51]: -----------------------
[13:57:51]: --- Step: xcversion ---
[13:57:51]: -----------------------
[13:57:53]: Setting Xcode version to /Applications/Xcode10.1.app for all build steps
[13:57:53]: ------------------------------------------------------
[13:57:53]: --- Step: Switch to ios verify_version_number lane ---
[13:57:53]: ------------------------------------------------------
[13:57:53]: Cruising over to lane 'ios verify_version_number' 🚖
[13:57:53]: -------------------------------------------------------------
[13:57:53]: --- Step: Switch to ios verify_version_number_common lane ---
[13:57:53]: -------------------------------------------------------------
[13:57:53]: Cruising over to lane 'ios verify_version_number_common' 🚖
[13:57:53]: Cruising back to lane 'ios verify_version_number' 🚘
[13:57:53]: --------------------------------
[13:57:53]: --- Step: get_version_number ---
[13:57:53]: --------------------------------
[13:57:53]: Project version numbers are consistent: 1.0.0
[13:57:53]: Cruising back to lane 'ios ****_distrib' 🚘
[13:57:53]: ------------------------------------------------------
[13:57:53]: --- Step: Switch to ios register_test_devices lane ---
[13:57:53]: ------------------------------------------------------
[13:57:53]: Cruising over to lane 'ios register_test_devices' 🚖
[13:57:53]: ------------------------------
[13:57:53]: --- Step: register_devices ---
[13:57:53]: ------------------------------
Available session is not valid any more. Continuing with normal login.
[13:57:55]: Fetching list of currently registered devices...
+------------------+---------------------+
|              Lane Context              |
+------------------+---------------------+
| DEFAULT_PLATFORM | ios                 |
| PLATFORM_NAME    | ios                 |
| LANE_NAME        | ios ****_distrib |
| VERSION_NUMBER   | 1.0.0               |
+------------------+---------------------+
[13:57:55]: Invalid device line, please provide a file according to the Apple Sample UDID file (http://devimages.apple.com/downloads/devices/Multiple-Upload-Samples.zip)

+------+-------------------------------------------------+-------------+
|                           fastlane summary                           |
+------+-------------------------------------------------+-------------+
| Step | Action                                          | Time (in s) |
+------+-------------------------------------------------+-------------+
| 1    | Verifying fastlane version                      | 0           |
| 2    | default_platform                                | 0           |
| 3    | xcversion                                       | 1           |
| 4    | Switch to ios verify_version_number lane        | 0           |
| 5    | Switch to ios verify_version_number_common lane | 0           |
| 6    | get_version_number                              | 0           |
| 7    | Switch to ios register_test_devices lane        | 0           |
| 💥   | register_devices                                | 1           |
+------+-------------------------------------------------+-------------+
[13:57:55]: fastlane finished with errors
[!] Invalid device line, please provide a file according to the Apple Sample UDID file (http://devimages.apple.com/downloads/devices/Multiple-Upload-Samples.zip)
[13:57:55]: WARNING: fastlane requires your locale to be set to UTF-8. To learn more go to https://docs.fastlane.tools/getting-started/ios/setup/#set-up-environment-variables
Build step 'Execute shell' marked build as failure

This is actually fixed with latest version.  My mistake was adding two tabs in the csv file


fix: Support analyze debug flag

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


netkvm: fix memory leak in TX overfragmenting flow

https://issues.redhat.com/browse/RHEL-24992
Memory leak of CNB::m_ExtraNBStorage happens if the packet os overfragmented but we do not succeed to
allocate extra pages.

@ybendito @YanVugenfirer Please ignore Static Tools Logo Test, this is a CI issue.

We have an ack from QE

Static logo tools failure: ignored (@kostyanf14 )
NetKVM-Win11_22H2: failed 2c_mini6rsssendrecv. Can't install protocol. As far as I remember this failure should be filtered out. @kostyanf14 Please check.
NetKVM-Win2019x64: Failed 1c_OffloadRsc: without any reason, NdisTest finished ok. Will look at it and update.
NetKVM-Win2022: Failed FlitchFreeDevice: will look at it and update,


175 enhance or add to code documentation classes functions etc

@Ben-Rey same here, you should bring all the commit you did in parallel branches. Just focus on what is to be done in this issue and leave the rest for the other branches.

@Ben-Rey i reviewed two functions to let you have a better understanding of what I'm expecting from this king of issue.
Can you do the same things for the others ?

@gregLibert can we close this one ?

@gregLibert can we close this one ?

Did you extract everything produce here to other issues/PR, if yes  then you can cloise it


Are resolution_x and resolution_y needed in database?

So far, the resolution is displayed only in the topography details
but not used in any other template or calculation.
Since the PyCo topography is evaluated in the topography details because
of the image plot, the resolution could be gathered quite easy at that point.
On the other hand, it is a bit extra effort to save the resolution
during the upload. Maybe we can remove it from there?!

I think we should keep it in the database. Some of this information needs to be cached to avoid opening the file to obtain it.


How can I scan specific target in single build?

New Issue Checklist

[ ] Updated fastlane to the latest version
[ ] I have read the Contribution Guidelines

Issue Description
Since we develop app with different test target, we need to test single target to ensure the target healthy.However, I didn't find a way to use scan to test single target.Is there any way to solve it?
Complete output when running fastlane, including the stack trace and command used

You can use: --capture_output as the last commandline argument to get that collected for you

[INSERT OUTPUT HERE]
Environment
Please run fastlane env and copy the output below. This will help us help you :+1:
If you used --capture_output option please remove this block - as it is already included there.
[INSERT OUTPUT HERE]

You can only pass a scheme. You could make a scheme for each target, right?


fix(thirdweb): handle Polygon Amoy Gas Station fee data

Problem solved
Short description of the bug fixed or feature added
Changes made

[ ] Public API changes: list the public API changes made if any
[ ] Internal API changes: explain the internal logic changes

How to test

[ ] Automated tests: link to unit test file
[ ] Manual tests: step by step instructions on how to test

Contributor NFT
Paste in your wallet address below and we will airdrop you a special NFT when your pull request is merged.
Address:

PR-Codex overview
The focus of this PR is to handle gas fees for the Polygon Amoy Testnet and update gas station URLs and gas prices accordingly.
Detailed summary

Updated gas fee handling for Polygon Amoy Testnet
Updated gas station URLs and gas prices for Polygon Amoy Testnet
Refactored functions related to gas fee data and gas priority fees

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

#3796  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @joaquim-verges and the rest of your teammates on  Graphite


Add support for disk and network_bandwidth resources to run in Mesos at Criteo

This PR is made of 2 patches:

Support for disk resource that is required to run Spark on Mesos at Criteo since we enabled the XFS isolator that requires that any container should have disk reserved.
Support for network bandwidth, otherwise Mesos will allocate a default amount based on the number of CPU. The user should be able to customize in order for the executor not to be killed by Mesos if the limit is exceeded and packets are dropped too abruptly.

Fix 1. is a backport in Spark 2.3 from a PR in 3.0 (https://github.com/apache/spark/pull/23758)
Fix 2. is completely Criteo specific since it relies on a resource that we introduced ourselves.

I ran the Mesos related tests on my machine to confirm my fix is ok since the CI build is failing for other reasons.


Improve type inference for DynamicGatherOp

Request description
When using the Python bindings for stablehlo.DynamicGatherOp the result has all dimensions unknown. This is likely because the DynamicGatherOp is defined to support type inference. Instead, there should be a way to pass in the result type, as for examples, for stablehlo.DynamicReshapeOp so that a front-end can set more precise return shapes.
For now my workaround is to generate a stablehlo.BitcastConvertOp following the DynamicGatherOp to set the more precise shape.

Long-term fix will likely be done in #621, but for the short-term fix we might be able to come up with something different.

"but for the short-term fix we might be able to come up with something different". I've taken the liberty to reformulate the ticket to improve type inference for DynamicGatherOp, because I think that a fix to this line will address the underlying issue: https://github.com/openxla/stablehlo/blob/e2aa7fe97cd09f44d864079c4e8be98064e5b425/stablehlo/dialect/TypeInference.cpp#L1677.

I'm realizing that a more precise shape function for DynamicGatherOp won't satisfy the original request, because sometimes StableHLO producers have more shape information than shape functions can possibly infer.
As mentioned above, because there is a shape function defined for DynamicGatherOp, its autogenerated Python bindings no longer include the result type in the signature of the DynamicGatherOp.__init__ function. This is not a problem for precise shape functions, but not so much for imprecise shape functions like this one. If the producer knows a more precise type than what the shape function can infer, then DynamicGatherOp.__init__ won't work for them, and they will need to use the much less convenient DynamicGatherOp.build_generic. Here's the workaround that was introduced in JAX (click here for full code):
  # TODO(burmako): Fix overly conservative type inference of DynamicGatherOp.
  # For now use the build_generic so that we can specify the result type.
  # return hlo.DynamicGatherOp(
  #     operand, indices, mlir.shape_tensor(slice_sizes),
  #     dnums, indices_are_sorted=ir.BoolAttr.get(indices_are_sorted)).results
  results = [mlir.aval_to_ir_type(aval_out)]
  operands = [operand, indices, mlir.shape_tensor(slice_sizes)]
  attributes = {
      "dimension_numbers": dnums,
      "indices_are_sorted": ir.BoolAttr.get(indices_are_sorted)
  }
  return hlo.DynamicGatherOp.build_generic(
      results=results, operands=operands, attributes=attributes).results

I imagine this API problem will be resolved by #32, but that will take some time. In the meanwhile, I'm tempted to simply remove shape inference support for DynamicGatherOp, so that we can remove the workaround in JAX.


Minimap doesn't resize properly after splitting

After Right Click on Editor > Split * the minimap is only resized for one pane:

(Atom v0.165.0; Windows 8.1, Ubuntu 14.04 Unity)

I thought most resize issues was fixed in latest version by listening to panes creation/destruction and resize all minimap in that situation. It seems there's some race conditions that will be hard to track efficiently (event triggered before the DOM was actually modified).
I'm working on v4 that will use the same DOM polling approach that atom text editors to resize themselves, so far I haven't got much issues when splitting panes. I'll plan to publish 3.5.0 with a setting to enable a preview of v4 to allow people to test if these kind of bug still occurs.

I just published v3.5.0 if you to give it a try, I still have some issues with the width of the minimap in flex box as webkit has some weird way to handle percentage in that layout but the problem of stretching/measurements should be fixed when V4 preview setting is enabled.

I'll give it a try for sure! :smiley:

Fixed with v4.


game crashes on world enter

I'm trying to use continuity to get connected textures to work in fabric 1.18.2, so I got Indium. Won't work at all, and crashes when I enter a world. Here's the log: https://pastebin.com/22KpfGLe
Any help is appreciated

I just took off every mod besides fabric api, sodium, indium, and continuity, and I'm still having that issue.

Update iris to 1.2.4.

Thanks, this fixes it


Redirect bootstrap-vcpkg.sh in Msys2 bash to bootstrap.bat

Redirect bootstrap-vcpkg.sh in Msys2 bash to bootstrap.bat
In Msys2 bash, uname -s returns MSYS_NT-6.1-7601 in Win7.

@JackBoosY close? I think it already does this. At least in git bash I can execute either

@JackBoosY close? I think it already does this. At least in git bash I can execute either

git bash can, but msys2 bash can't.

\vcpkg\bootstrap-vcpkg.bat
.vcpkgbootstrap-vcpkg.bat: command not found
do I need to chmod this?

do I need to chmod this?

No. In a sh shell, you need forward slashes:
./vcpkg/bootstrap-vcpkg.bat


[POS] Rewrite Staking loop to be fully async

Currently it is limited to 1 thread, and if you have more than 700 UTXOs, it starts to fail to accomplish the task of one round within the timeframe of 16 seconds on one CPU. If we made it fully async, it could use more processors.

Done by #576


Segmentation fault (core dumped) after "import mxnet as mx "

in mxnet folder,
make -j$(nproc)
I have the following problem:
cpchung:python$ python
Python 2.7.5 (default, Nov  6 2016, 00:28:07)
[GCC 4.8.5 20150623 (Red Hat 4.8.5-11)] on linux2
Type "help", "copyright", "credits" or "license" for more information.
>>> import mxnet as mx
Segmentation fault (core dumped)
cpchung:python$

My running environment:
Linux cam-linx-100 3.10.0-327.10.1.el7.x86_64 #1 SMP Tue Feb 16 17:03:50 UTC 2016 x86_64 x86_64 x86_64 GNU/Linux
g++ (GCC) 4.8.5 20150623 (Red Hat 4.8.5-11)
Copyright (C) 2015 Free Software Foundation, Inc.
This is free software; see the source for copying conditions.  There is NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.
gcc (GCC) 4.8.5 20150623 (Red Hat 4.8.5-11)
Copyright (C) 2015 Free Software Foundation, Inc.
This is free software; see the source for copying conditions.  There is NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.
/usr/local/cuda/bin/nvcc
nvcc: NVIDIA (R) Cuda compiler driver
Copyright (c) 2005-2016 NVIDIA Corporation
Built on Tue_Jan_10_13:22:03_CST_2017
Cuda compilation tools, release 8.0, V8.0.61
3.2.0
MXNet version:\n
750af1c191a8f42640c21ef0fa5739c137d83104
Python 2.7.5
Wed Feb 15 10:46:36 2017
+-----------------------------------------------------------------------------+
| NVIDIA-SMI 375.26                 Driver Version: 375.26                    |
|-------------------------------+----------------------+----------------------+
| GPU  Name        Persistence-M| Bus-Id        Disp.A | Volatile Uncorr. ECC |
| Fan  Temp  Perf  Pwr:Usage/Cap|         Memory-Usage | GPU-Util  Compute M. |
|===============================+======================+======================|
|   0  Quadro 600          Off  | 0000:05:00.0      On |                  N/A |
| 30%   67C    P0    N/A /  N/A |    384MiB /   962MiB |      0%      Default |
+-------------------------------+----------------------+----------------------+
+-----------------------------------------------------------------------------+
| Processes:                                                       GPU Memory |
|  GPU       PID  Type  Process name                               Usage      |
|=============================================================================|
|  No running processes found                                                 |
+-----------------------------------------------------------------------------+
Architecture:          x86_64
CPU op-mode(s):        32-bit, 64-bit
Byte Order:            Little Endian
CPU(s):                8
On-line CPU(s) list:   0-7
Thread(s) per core:    2
Core(s) per socket:    4
Socket(s):             1
NUMA node(s):          1
Vendor ID:             GenuineIntel
CPU family:            6
Model:                 45
Model name:            Intel(R) Xeon(R) CPU E5-1620 0 @ 3.60GHz
Stepping:              7
CPU MHz:               1316.390
BogoMIPS:              7182.16
Virtualization:        VT-x
L1d cache:             32K
L1i cache:             32K
L2 cache:              256K
L3 cache:              10240K
NUMA node0 CPU(s):     0-7

I had the same problem with segfaults during import. For me, it seems to have been caused by an incompatible version of the opencv-python library. I use anaconda and had installed opencv using conda install opencv, which installs the following version: opencv: 3.1.0-np111py27_1. I uninstalled that with conda uninstall opencv and installed an older version with conda install -c menpo opencv=2.4.11 and that seems to have resolved the problem for now.

Hi Michael,
Good catch! I built the opencv from source(3.2 as shown above). I remember first I was using 2.4 but did not work so I changed to 3.2 later. Anyway, will try again.
I will try your suggestion. I see it in mxnet installtion on centos:
wget https://codeload.github.com/opencv/opencv/zip/2.4.13
http://mxnet.io/get_started/setup.html#prerequisites
BTW, I am not using anaconda for python. Can I just install anaconda and then export the anaconda python in ~/.bashrc? Any standard tutorial I can follow?

Hm, I can't really help with the pdb/gdb stuff but I can give a brief tutorial on Anaconda.
Anaconda does not seem to be the "recommended" way of installing mxnet, but I prefer it anyway as it does not require altering the system python environment. I'm not at my development machine now, but these should roughly be the steps for installing mxnet with anaconda and GPU support:

Install cuda 8 and cudnn 5.1
Clone mxnet, set the gpu compilation flags in config.mk
Install dependencies with sudo apt-get update; sudo apt-get install -y build-essential libatlas-base-dev libopencv-dev graphviz
Build mxnet with make -j$(nproc)
Install Anaconda. The installer with instructions can be found here. The installer adds the anaconda location to your path. After running the installer and starting a new terminal (or sourcing .bashrc), which python should point to the anaconda executable. (At any point, you can uninstall anaconda by removing the anaconda directory and removing the line that adds anaconda to your path from your ~/.bash_profile or ~/.bash_rc-file.)
Now you can choose to install mxnet in a fresh anaconda environment or your root anaconda environment.
If you want a fresh environment, run something like conda create --name mxnet numpy to create a new mxnet-environment with numpy and activate the environment with source activate mxnet
Now, if mxnet was successfully built, you should be able to run python setup.py install (without sudo) in mxnet's python directory. This should install mxnet for either the root anaconda environment  or the fresh mxnet environment, depending on whether you completed step 7.
Finally, install opencv with conda install -c menpo opencv=2.4.11, and any other packages you may want. The anaconda root environment comes with quite a few scientific python packages. For a more compact option, there is also miniconda.

Hi, have you solved this problem now? I also encountered this problem.
I build mxnet with warp-ctc plugin. And I get segmentation fault when run 'import mxnet'

This issue is closed due to lack of activity in the last 90 days. Feel free to ping me to reopen if this is still an active issue. Thanks!
Also, do please check out our forum (and Chinese version) for general "how-to" questions.


chore: cleanup images, officially retire OpaqueImageView

This PR resolves PHIRE-1401
Description

Now that we have been using palette image under the hood for some time it should be safe to update all our components to use palette-image directly and get rid of the old OpaqueImageView code.

Component
Checked?

ArticleCard
Yes

ArtistShow

ArtistAutosuggestRow
Yes

ArtworkListImage
Yes

SavedItemRow

ShowItemRow

SaleArtworkTileRailCard

ToastComponent

ArtistSeriesHeader
Yes

ArtistSeriesListItem
Yes

ImageZoomView
Yes

DeepZoomTile
Yes

TabFairItemRow

CollectionHeader

FairArticles

FairEditorial
Yes

FeatureFeaturedLink

FeatureHeader

PartnerListItem
Yes

ArtistCard
Yes

ItemArtwork

ItemShow

Lot

MyCollectionImageView
Yes

ArtistItem

MedianAuctionPriceListItem

SaleArtworkListItem

SaleHeader
Yes

SaleListItem

SearchResultImage
Yes

TrendingArtistCard
Yes

ShowContextCard
Yes

ShowInstallShots

ViewingRoomArtworks
Yes

To do

[x] Remove the palette image flag and default to using
[x] Remove OpaqueImageView2 that is just a wrapper component, use single OpaqueImageView everywhere
[x] Replace simple usages of OpaqueImageView with direct use of Image
[x] Replace more complex usages of OpaqueImageView
[x] Remove OpaqueImageView component completely + associated code
[x] Test, Test, Test

Follow-ups

[ ] Retire the flag in echo

PR Checklist

[ ] I have tested my changes on iOS and Android.
[x] I hid my changes behind a feature flag, or they don't need one.
[x] I have included screenshots or videos, or I have not changed the UI.
[x] I have added tests, or my changes don't require any.
[x] I added an app state migration, or my changes do not require one.
[x] I have documented any follow-up work that this PR will require, or it does not require any.
[x] I have added a changelog entry below, or my changes do not require one.

To the reviewers 👀

[ ] I would like at least one of the reviewers to run this PR on the simulator or device.

Changelog updates
Changelog updates

Cross-platform user-facing changes

iOS user-facing changes

Android user-facing changes

Dev changes

remove OpaqueImageView and use palette image everywhere - brian

Need help with something? Have a look at our docs, or get in touch with us.

This PR contains the following changes:

Dev changes (remove OpaqueImageView and use palette image everywhere - brian - brainbicycle)

  Generated by :no_entry_sign: dangerJS against 1f04457a09da16fe51d4530215d037183918da0a

i spot checked a bunch of places but not all but didn't see anything at all broken, but please let me know if anything looks amiss 🙏


Apply detekt designsystem

Issue

close #565

Overview (Required)

add the plugin droidkaigi.primitive.detekt and fix the lint errors by ./gradlew composeLint.

Links

The errors before fix.
detekt report.pdf

Can you review this PR ?
I was not sure if commonMain and androidMain can be delete such way.. (ea1e3a6987577baceb7e59512cde238e31002af1)

I think they are not needed!


Visual Updates for FunnelChart

A couple of things would be good to add to the funnel chart / small changes to make. See current version

Reduce the height of the chart - our standard charts are mostly 200px height I think, which means users can fit more content per screen. Would be good to get closer to that (may involve removing some of the whitespace under the title)
Add an option for a stepped funnel, and make this default - this format can be easier to compare visually eg

Add value labels
Stop series labels being cut off - possibly make use of a legend to achieve this ?

@archiewood Hi! I'm happy to send a PR for this if its up for grabs, would you please assign me to this issue? Thanks

All yours, go for it!

Hi @archiewood , I've made most of the changes mentioned in the issue. But I had some doubts regarding the stepped funnel. At the moment, with the ECharts library it is not possible to customise shape of the step, there are some open issues on their Github for allowing more style changes to funnel charts.
Did you have something in mind regarding the stepped funnel?
Apart from that, the funnel chart now looks like this
Without Subtitle

With Subtitle

I will open a PR shortly with the changes I've done so far.

Hi @nidhi-kala,
Thanks for this, this looks like it's going to be a real step on.
I hadn't looked into what eCharts supported here. I guess there is an aspiration here to have a more stepped kind of funnel.
But for now, I think it probably makes sense to stick with what we can get out of echarts, and leave changes to the funnel until they ship something!
Looking forwards to reviewing your PR!

It might be possible to play around with the minSize and maxSize values to get something that's visually a bit less "squashed" when in a top to bottom configuration.
(Though this might just be a personal preference!)


Add a lot of docs

Alright, so I did a very poor job at documentation any of my changes so I decided to start just documenting the whole crate.
Of course this is quite difficult with a lot of the original drm api not being well documentated and me having not a good idea of how everything works, but it is a start.
I will maybe continue to work on this PR and also would welcome anyone to contribute, but I am also happy with a merge in this state, I can always follow up with more.
Also feedback is very welcome.
Some API changes creeped into this, if I should move these to a separated PR @Slabity just tell me, they are really small:

set_cursor and set_cursor2 now take a Buffer reference instead of a separate Buffer::Id and dimensions.
crtc::page_flip does not take a version anymore as this is not necessary at all.

Hey @Drakulix, it looks like the vast majority of the changes were made by rustfmt, correct? While a great idea, I'd prefer them to be in a separate from any documentation or API changes just to make it easier to find potential bugs.
Slightly related, what do you think about a linting program like clippy?

@Slabity Sorry that must have been my IDE (trying out RLS). I will try to revert all rustfmt related changes.
clippy is very good, but I usually only use it to lint my code periodically and fix a bunch of stuff, because it breaks so frequently with new rust-nightly builds, that I find it hard to automate clippy testing.

@Slabity Fixed and rebased on current master.
I also was able to separate one api change into another commit, the other two were not so easy, so I did not bother, but of course I can do those as well, if you want.
But all rustfmt changes should not be gone.

Bad typo. I meant those rustfmt changes should now(!) be gone.

Looks good to me!


Feature Request: Unread message count

Hi,
The previous Cordova SDK featured means to get the unread messages count for a user. Would it be possible to implement the mentioned feature here too please? Thanks.

Would also love this. Any plans?

hello there 👋
can you link to the existing behavior so I can take a look?
Thank you


chore(state): use strict_subtract_mappings for compiled classes

This change is

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

Project coverage is 77.91%. Comparing base (63c8b7e) to head (b58c267).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1909      +/-   ##
==========================================
+ Coverage   77.80%   77.91%   +0.11%
==========================================
  Files          61       61
  Lines        8749     8739      -10
  Branches     8749     8739      -10
==========================================
+ Hits         6807     6809       +2
+ Misses       1502     1491      -11
+ Partials      440      439       -1

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

crates/blockifier/src/state/cached_state_test.rs line 310 at r2 (raw file):

    state.set_storage_at(contract_address, key, storage_val).unwrap();
    state.increment_nonce(contract_address2).unwrap();

Why don't we add these too?
Suggestion:
    state.get_storage_at(contract_address, key).unwrap();
    state.set_storage_at(contract_address, key, storage_val).unwrap();
    state.get_nonce_at(contract_address2).unwrap();
    state.increment_nonce(contract_address2).unwrap();


rustfmt is not tested to be idempotent

If I understand correctly, rustfmt is expected to be idempotent, meaning that it should not change the content after one run. (Please let me know if that's not the case.) But looks like it's not tested to maintain this property.
And, besides maintaining idempotency, such tests can also surface bugs like this during dev/ci: https://github.com/rust-lang-nursery/rustfmt/issues/1674

We do test for idempotency by running the 'target' of every test through rustfmt as a 'source'. Obviously, this only catches bugs if there is a relevant test. We also test rustfmt on its own source code, so this should also catch any non-idempotency bugs as a side-effect. I'll close this issue for now because I don't know of any way to improve this testing, but if you think of something I can re-open.


modify slug_generator to include .

Previous implementation will not preserve dots in usernames. For example, "john.doe" would be converted to "john-doe".
To modify the function to accept usernames with dots, we need to update the NON_ALPHANUM_PATTERN and possibly adjust the OBJECT_PATTERN.
key changes made:

Added dot (".") to LOWER_PLUS_HYPHEN array.
Updated OBJECT_PATTERN and LABEL_PATTERN to include dot.
Modified NON_ALPHANUM_PATTERN to allow dots: /[^a-z0-9\.]+/

didn't need to modify the extract_safe_name method itself because the changes to NON_ALPHANUM_PATTERN are sufficient to preserve dots. ( I think)

Updated test-suite for

Dots are preserved in valid usernames.
Multiple dots are handled correctly.
Leading and trailing dots are removed.
Mixtures of dots and hyphens are preserved.
Long names with dots are truncated and hashed appropriately.
Slugs don't end with hyphens or dots


TypeError: undefined is not a function (evaluating 'LocalCollection._idParse(msg.id)')

Fast-render (used by flow-router-ssr) does not seems to work anymore for me after upgrading to Meteor 1.2 RC7.
TypeError: undefined is not a function (evaluating 'LocalCollection._idParse(msg.id)')
_livedata_datameteorhacks_fast-render.js:304
onMessagelivedata_connection.js:252
(anonymous function)stream_client_sockjs.js:172
forEach
forEachunderscore.js:105
onmessagestream_client_sockjs.js:171
dispatchEventsockjs-0.3.4.js:87
_dispatchMessagesockjs-0.3.4.js:1072
_didMessagesockjs-0.3.4.js:1130
onmessagesockjs-0.3.4.js:1277
collection.js:164

Do this add the updated fast-render to your app: meteor add meteorhacks:fast-render.
I'll do a release for SSR anyway now.

Wow, that was fast :) Yes, it works now again! Thx for all the great work with flow-router!

Published updated SSR version to atmosphere as well.


Compatibility with Ruby VMs that don't support RubyVM

I'm okay to merge this PR but let me confirm that it doesn't work -e option and LSP support (rstfilter-lsp).
Can I merge it?

But basically, yes it's ready to merge.

Would be good to add truffleruby in CI if the tests pass, to ensure it doesn't accidentally regress.


Add array support for Meteor

I know that this is listed in the todos. However can you point to a quick an dirty way of handling sorting within an array.
I am passing to my template 1 object which has an array, this one is then rendered into a list.
I tried to swap elements onSort, then update my Object:
            var tmplist = doc.list;
            var tmp = tmplist[event.oldIndex -1];
            tmplist[event.oldIndex -1] = tmplist[event.newIndex -1];
            tmplist[event.newIndex -1] = tmp;
            Lists.update({_id:Session.get('object_id')}, {$set: {list: tmplist}});

But then something very strange happens, the displayed list is immediately reset to original order. The collection however is updated, only when I refresh the page the order is correct.
I tried to set the option store:false, It didn't help.
Any advices on how to achieve that ?

This would indeed be helpful – for everyone reading this until then, the vanilla Sortable works perfectly fine (just wrap the items in {{#each}} ... {{/each}} instead of {{#sortable}} ... {{/sortable}} and call Sortable.create(element, options), including an onEnd callback, from within Template.YourTemplate.onRendered).

@dedcode did you find a work around for this in the end?

@phinmpofu Check out #287, which is a duplicate. There's a workaround there that works for me.

@skosch thanks mate. That'll be useful

+1 Array support

+1

Just wanted to let you guys know, in case you are using React, I've had good success implementing very pretty-looking drag-and-drop with react-motion. Might be a viable alternative; I've dropped sortable completely for now.

@skosch wow, thanks for showing off amazing react animation library!
I would love to migrate to this alternative for sortable, but it does not support IE9... just keep this in mind.

+1
Can please fix this? :)

would also love to see array support!

Dear all! Meteor is moved to the separate repository. If this issue is still actual, please create it there once again.
For your info: this project needs a maintainer.


Add IPv6 address support

I'm writing a bot for our network, which is only available by IPv6 address. The address should look like "[::1]:6667".
Not sure if everything here is good, but the bot works and quick tests pass.

Hi, not a maintainer of this project, but you might look in to net.SplitHostPort, it will gracefully handle grabbing a server and port from both Ipv4 and ipv6, and it's in the stdlib
https://golang.org/pkg/net/#SplitHostPort

Updated to use net.SplitHostPort

Perfect :)


Take ownership of spring-cloud-multi-auth

As really the only major users of spring-cloud-multi-auth, and as the release mechansim of the parent openbanking4-dev repository has broken, now seems to be the right time to take ownership of this repo.

[x] Break the fork and become independent of the parent project
[ ] Fix the poms to be com.forgerock.spring.security groupId
[ ] Change package names to be com.forgerock
[ ] Set up CI/CD

Support issue raised to Detach the fork and turn it into a standalone repo;
https://support.github.com/ticket/personal/0/1140325
Github have now confirmed this has been done.


Does NPM support HTTP PROPFIND method?

using jc21/nginx-proxy-manager:latest
This may not be a bug, just limited functionality report or feature request, please read...
I have Directory "/" not found error in owncloud, and I believe this is related to NPM

The 404 is likely thrown by Nginx because as I remember back in a days when wanted to run WebDav from Nginx I needed extended module for Nginx to include PROPFIND method, so I'm making a simple assumption here, that NPM is not compiled with extended Nginx functionality, thus can't provide 100% reverse proxy.
Can you confirm if you include ngx_http_dav_module ?
https://nginx.org/en/docs/http/ngx_http_dav_module.html
Actually probably unofficial
https://github.com/arut/nginx-dav-ext-module
https://forum.nginx.org/read.php?2,227415,227415
If not, would you be able to include it, or should I build my own container based on yours to include it?

I think the problem got solved when I disabled "Cache Assets", so maybe would just need to adjust something to let PROPFIND work without cache, and the rest to cache?


Provide Demos in the docs

Hello, I've been trying to make this module send the templates unsuccessfully.
My setup is Nuxt 3 (SSR) and want to use with nodemailer (but could work with any other really). After reading articles, documentation back and forth and attempting tons of tries, everything seems misleading.
There is nothing that would help most than a minimal demo page like they do with other modules.
See https://tailwindcss.nuxtjs.org/examples/basic for reference.
The basic stuff should cover form inputs that add info to props, that are included in the email template, that is sent with the desired provider (nodemailer or else).
Do you think this is possible? Would help a lot. Thanks!

This would be very helpful. I'm almost at the point of giving up on this and finding an alternative to use as vue-email seems to be abandoned.

we just merged a new project rewrite, please do check the docs and the updated logic https://vuemail.net/


Changing default settings per Javascript-Code

I would like Biodalliance genome browser to be started with some pre-configured default settings.
In particular, the option "Highlight mismatches and strands" in the bam-track shall be selected.
How can I modify the settings from javascript code? Is there something like an API?

This can be done via the stylesheet system.  An easy way to see how is to configure a track via the UI then click the Export button and select "Dalliance config".  To get the effect you're looking for, you'll want a config along the lines of:
{
    "name": "BAM test",
    "bamURI": "/path/to/subset22-sorted.bam",
    "style": [
      {
        "type": "density",
        "zoom": "low",
        "style": {
          "glyph": "HISTOGRAM",
          "COLOR1": "black",
          "COLOR2": "red",
          "HEIGHT": 30
        }
      },
      {
        "type": "density",
        "zoom": "medium",
        "style": {
          "glyph": "HISTOGRAM",
          "COLOR1": "black",
          "COLOR2": "red",
          "HEIGHT": 30
        }
      },
      {
        "type": "bam",
        "zoom": "high",
        "style": {
          "glyph": "__SEQUENCE",
          "FGCOLOR": "black",
          "BGCOLOR": "blue",
          "HEIGHT": 8,
          "BUMP": true,
          "LABEL": false,
          "ZINDEX": 20,
          "__SEQCOLOR": "mismatch"
        }
      }
    ]
  }

The key option here is the:
   "__SEQCOLOR": "mismatch"

Thanks for the answer!
I did what you suggested. But, it does not work.
Reads are still displayed in multi-color according to bases, not in a uniform color with highlighted mismatches.
Also the checkbox "Highlight mismatches and strands" is not selected.
What to do now?

Sorry to hear you're having trouble.
Could you send me the exact config you've been using (just for this track).  Also, what version of Biodalliance?

I'm currently seeing a rendering glitch under certain circumstances when "colour by mismatch" is selected in the git-master version of Biodalliance.  I'll fix this tonight.  But shouldn't affect the ability to configure this option (and 0.13.x should work fine).

I would love to see more examples and documentation of the JSON style sheet format, I've tried to "translate" the XML samples I have found into JSON, and I found it difficult and was unsuccessful.
Perhaps the fact that I'm XMLo phobic is part of the explanation, but I don't know that there is a cure from XML hatred affliction ! ;-)

Figured it out. I did something wrong. It works now.
Thank you very much!!


Wrong post _type typedoc

In get_posts

type_ isnt anymore just those there is now modview

https://join-lemmy.org/api/types/ListingType.html


Refactor the grafana dashboard template

Why are the changes needed?

The original dashboard dashboard template does not work for my use case:
In this PR:

I add the baseFilter variable, default value is job=~"kyuubi".
apply the baseFilter for all expressions
add baseLegend for the legendFormat
Fix hardcode typos
remove the sum from the expressions for JVM

How was this patch tested?

Was this patch authored or co-authored using generative AI tooling?

No.

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

Project coverage is 0.00%. Comparing base (aa33521) to head (9fc3b5c).

Additional details and impacted files
@@          Coverage Diff           @@
##           master   #6867   +/-   ##
======================================
  Coverage    0.00%   0.00%
======================================
  Files         687     687
  Lines       42504   42504
  Branches     5798    5798
======================================
  Misses      42504   42504

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

verified locally, lgtm.
nit, can we put $baseFilter at first?
- {instance=~\"$instance\",$baseFilter}
+ {$baseFilter,instance=~\"$instance\"}

verified locally, lgtm.
nit, can we put $baseFilter at first?
- {instance=~\"$instance\",$baseFilter}
+ {$baseFilter,instance=~\"$instance\"}

Updated.

thanks, merged to master


add volume and volumemount to deployment

the values file has volume and volumemount definitions but they aren't used in the deployment template
# Additional volumes on the output Deployment definition.
volumes: []
# - name: foo
#   secret:
#     secretName: mysecret
#     optional: false

# Additional volumeMounts on the output Deployment definition.
volumeMounts: []
# - name: foo
#   mountPath: "/etc/foo"
#   readOnly: true

hey @hofq could you take a look at this?

Thx for the PR @pnancarrow :)


Please add main to bower.json

please add to .bower.json
"main": "src/datetimeinput.js",
this will help wiredep grab the js file and inject it into our index.html.
thanks from all of us at generator-angular-fullstack :)  great components!

Sorry, bower is not supported.


Tests for resource compiler

Primarily addresses #162 but also addresses a few bullet points of #161

The checks are passing and the changes are up for another batch of review. Appreciated :)

Thanks for making the changes @ksubramz


no examples ?!

examples whould be very helpfull !!!
[Enter feedback here]

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

ID: d98186ad-af7e-d25d-2135-f0c65ffd5288
Version Independent ID: 55c29643-935f-7eab-6a3a-8814f96e8669
Content: mount
Content Source: WindowsServerDocs/administration/windows-commands/mount.md
Product: windows-server
Technology: windows-commands
GitHub Login: @JasonGerend
Microsoft Alias: jgerend

Hey JossBoulement,
Thanks again for contacting Microsoft. We value your feedback.
A documentation change has been made adding examples for the mount command and will be live after the next business day. We will now be closing this thread.
Thank you and have a most pleasant day.
#please-close


chore: added the brand logos

This  PR adds the logos of various brands that the kec computer club colabrated with in the images section.

Done!!!


Required tag doesn't work with nested object property

I define this struct and execute swag,
type Owner struct {
	Id        uint64   `json:"id" validate:"required"`
	Company   *Company `json:"company" validate:"required"`
}

but swag doesn't generate correctly.
  model.Owner:
    properties:
      company:
        $ref: '#/definitions/model.Company'
        type: object
      id:
        type: integer
    required:
    - id
    type: object

Expected behavior
expect:
  model.Owner:
    properties:
      company:
        $ref: '#/definitions/model.Company'
        type: object
      id:
        type: integer
    required:
    - company
    - id
    type: object

Your swag version
v1.6.2
Your go version
1.12.5

Can this be closed now @myoshi1012 ?

fixed by #541 . Please use the latest version against the master. Reopen it if you still have any problems.


HADOOP-19063. [thirdparty] Fix the docker image setuptools-scm && lazy-object-proxy.

Jira
JIRA: HADOOP-19063. [thirdparty] Fix the docker image setuptools-scm && lazy-object-proxy.
Describe
When compiling the docker-image, I found that the Dockerfile failed to execute pip install setuptools-scm && pip install lazy-object-proxy. This is because setuptools-scm && lazy-object-proxy has been upgraded many times and currently no longer supports python2.7.

setuptools-scm
https://pypi.org/project/setuptools-scm/#history

lazy-object-proxy
https://pypi.org/project/lazy-object-proxy/#history

I have consulted the release history lists of setuptools-scm and lazy-object-proxy, and I have found two versions that can be successfully installed. I have tested these two versions and confirmed that they can be installed properly and function correctly.
setuptools-scm==5.0.2
lazy-object-proxy==1.5.0

@Hexiaoqiao @steveloughran Thanks for reviewing the code! merged into trunk.


Update Demo Video in Readme

Replace the App demo video displayed in Readme with the latest Version.

can i do this

here is video that i replace

closed by #79


Fixing compiler errors for ESP Arduino 2.4.0

Long term testing has not been performed yet. Please join the effort :)

@njbuch Thank you for the Pull Request. I will do some sanity and do merge it.


Feat/lock mode top

Codecov Report

Merging #26 (fbc9ca1) into master (f586fd7) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##            master       #26   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            3         4    +1
  Lines          470       466    -4
=========================================
- Hits           470       466    -4

Impacted Files
Coverage Δ

tests/pools/test_null_pool.py
100.00% <ø> (ø)

tests/test_dbver.py
100.00% <ø> (ø)

tests/pools/test_begin.py
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


Speed up CIs with mamba?

I recently tried mamba as a drop-in replacement for conda. As advertised, it seems to be much faster, so this might be an easy way to speed up our CIs (not sure how much time is spent on bootstrapping envs and installing packages, but I guess every bit matters).
There's even micromamba, which enables the creation of new envs without having to install a base miniconda env first - I haven't tried that myself, but this tweet says it's really fast.

Seems worth trying, a bit of time is spent in the conda stages

As far as I can see we currently use conda only in one Windows job (disregarding the notebook job) on Azure Pipelines. Currently, micromamba doesn't work on Windows though, so maybe something for another time. I didn't know that Linux jobs use pip.

I didn't know that Linux jobs use pip.

Only one, the others use ci-helpers which uses miniconda

Thanks for pointing that out - I didn't know these scripts. It seems like they already support mamba (but not micromamba), so maybe the best thing would be to try to get micromamba support upstream.

Sure, and in the meantime I'll get a run going with MAMBA=true to see if it helps

Looks like they're already on it: https://github.com/astropy/ci-helpers/issues/458

I think we tried it and it didn't help but we can keep this in mind and try it again in the future. I'm not sure it will, though -- based on what I've seen, most of the time is actually spent downloading and installing packages rather than getting conda running at all. And now that we use GitHub actions and Azure conda seems to be better supported out of the box anyway


ROX-13947: Remove Central ownerReference on central-db-password

Description
the operator creates a garbage-collected central-db-password secret bound to the lifetime of the Central. Though, we don't delete the PVC using garbage collection. When users would reinstall Central, the password would be generated anew, and Central wouldn't be able to connect to the previous database because the password is lost.
I didn't use the SecretReconciliator because this logic is inherently different from the other generated secrets, who rely on the presence of an owner reference.
Checklist

[ ] Investigated and inspected CI test results
[x] Unit test and regression tests added
[ ] Evaluated and added CHANGELOG entry if required
[ ] Determined and documented upgrade steps
[ ] Documented user facing changes (create PR based on openshift/openshift-docs and merge into rhacs-docs)

Testing Performed

install the operator 4.0.2 & apply a Central CR
wait for the stack to be healthy. Verify that the central-db-password has an OwnerReference set to the created Central.
delete the Central CR. Wait until everything is uninstalled.
reinstall the Central with the same settings
Observe that it cannot connect to the database because the password is wrong
Delete the PVC, and the central-db Pod, and observe that after a while Central can connect again (restarting the operator makes this quicker)
delete the running operator deployment, and start the operator from this branch
wait for the reconciliation loop
observe that the central-db-password does not have an ownerReference anymore
verify that central can connect to the database
delete the Central CR
wait for the resources to be deleted
create the Central CR
observe that Central can connect to the central-db

/retest

/retest

/retest

/retest

/retest

/retest

/retest

/retest

/retest

/retest


[feature] Proposal: Specify HTTP Headers for CLI commands to enable Auth proxies

Many GUAC cli commands (guacone collect files, guacone query vuln) use an HTTP client to connect to the GUAC GraphQL server. The GUAC server does not provide any sort of AuthN/AuthZ support.
In the short term, I propose not adding any Auth support to GUAC and suggesting the GUAC services be run behind any kind of proxy that will do AuthN/AuthZ (AuthZ would only be allow/don't allow without any introspection to the queries). To enable this, the client commands will allow specifying raw http headers. This could support any type of Auth, such as basic / api key / or an Oauth flow ran outside the tool.
Every cli command that opens an HTTP connection will use the option "header-file". This will be expected to point to a CSV file on disk that looks like this:
name, value
Authorization, Basic abc123

A helper function will be added under pkg/cli that will read the file and parse using https://pkg.go.dev/encoding/csv ReadAll(). When a client is created it will use a custom transport that implements the RoundTripper interface. This transport will simply wrap http.DefaultTransport and add headers to the request using r.Header.Add().
Describe alternatives you've considered
Instead of a file, the cli could expect an array of strings and parse them. This would be cumbersome to the user to make sure it is correct for each call. It is easier to setup a file. Also a cli options show up on ps on a system, while a file can use filesystem permissions.
The file could be yaml/json instead of csv. However, I don't see the need for the flexability of those formats. We don't need nested objects, just simple key-value pairs.
Instead of accepting raw headers, the cli could either accept a basic auth password or build in an oauth flow with a provider like GitHub. At this point, choosing a number of providers is not clear. Which ones to choose, each one needs to be implemented, and some will not be supported. This may be a step later, but just allowing raw headers can be a solution for now that doesn't make us commit to certain providers.

Sounds good to me!

would it be better to use something like rfc822 encoding and mimic the cURL -H @file parameter ?
On the negative side, I imagine that'd require a third dependency such as this. On the upside, I can see somebody copying a cURL file that "works". I also suspect that testing proxy configuration using cURL first may make people's life easier

Agree on using a rfc822 file. I tested out this code and it works well:
package main

import (
	"log"
	"net/http"
	"os"

	"github.com/ProtonMail/gluon/rfc822"
)

func main() {
	// Read in header file to byte[]
	bts, err := os.ReadFile("headers.txt")
	if err != nil {
		log.Fatal(err)
	}

	// Create a header object from github.com/ProtonMail/gluon/rfc822
	// library. Store this object
	rfcHeaders, err := rfc822.NewHeader(bts)
	if err != nil {
		log.Fatal(err)
	}

	httpHeaders := http.Header{}

	// In the custom transport middleware, add headers to any request
	// (r.Header.Add)
	rfcHeaders.Entries(func(k, v string) {
		httpHeaders.Add(k, v)
	})

	// This log shows that this is working properly.
	if err := httpHeaders.Write(os.Stdout); err != nil {
		log.Fatal(err)
	}
}

Looks like I missed some things, like https://github.com/guacsec/guac/blob/main/pkg/ingestor/ingestor.go#L156-L157 . I am going to revisit my fix. I will probably make the flag a top-level flag for each command that supports it (as opposed to putting it at the sub-command level).


All examples merged to one project

Waiting for DevRel to merge all the READMES into one. And improve that 1 README in general.

which README's are you referring to @volovyk-s ?

@thisisjoshford Each separate examples project had a README file. Now all these READMEs are in this project. I think we need to merge them into one well-structured README file and make it shorter and more universal.

ahhh I see. Wasn't sure which READMEs you were referring to. So consolidate all README files from each example as well as the main README for this SDK, correct?

@thisisjoshford I think the SDK should have its own README. I'm talking about the consolidation of all examples READMEs into one.

@thisisjoshford I think the SDK should have its own README. I'm talking about the consolidation of all examples READMEs into one.

Yes! I agree. Was just clarifying that there are two READMEs that need enhancement from DevRel (the main SDK README as well as the consolidated examples README) Would you or someone on the tooling team be able to take the first stab at the example consolidation one?

I have added a simple consolidated README. Margin this now.


Remove or Replace base box image in storage pool.

There is no option to remove or replace image placed as volume in libvirt storage pool. It can be only uploaded in current version.
Also if uploading was interrupted, image need to be removed manually.

I suppose the issue here is primarily the fact that if a vagrant box is removed, the image in the storage pool is not deleted; the same likely goes for replacing a vagrant box with a newer version or even upgrading box versions.
This needs some testing to better define the problem before the feature can be added; in all likelihood this means full support for vagrant box versions.

Closing this as duplicate of #85 where there's a much better discussion around this issue.


unity client: migrate internal components to structs to improve performance

Tackled at https://github.com/decentraland/unity-renderer/pull/5376


fix fix case with same catalog name and check getCurrentCatalog()

Proposed changes

use different catalog name in test cases
check the return value of ConnectContext.getCurrentCatalog(), and throw runtime exception if null.
This is because the catalog may be dropped at any time, so getCurrentCatalog() may return null.
Simply throw a runtime exception to abort current process.

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

run buildall

run buildall

run buildall

run buildall


API Schemas are not updated

In many of the existing resources the plugin has warnings that suggests that the API Version is not support.
in fact most chances are that the plugin metadata information is out of sync with the latest API's
e.g : for Log Analytics workspaces it says apiVersion: "22-10-01" is not valid, while in fact that is the most up-to-date version

from Azure Docs :
https://learn.microsoft.com/en-us/azure/templates/microsoft.operationalinsights/workspaces?pivots=deployment-language-arm-template

@anthony-c-martin The latest version I see in https://schema.management.azure.com/schemas/common/autogeneratedResources.json is this:
                "$ref": "https://schema.management.azure.com/schemas/2021-12-01-preview/Microsoft.OperationalInsights.json#/resourceDefinitions/workspaces"

Bicep is fine with 2022-10-01, though.  Any "insights"?  :-)  Thx.

The schemas appear to have not been updated in this addon in quite a while.
I can see that the schemas exist, and have been generated in the repo here: https://github.com/Azure/azure-resource-manager-schemas/tree/main/schemas
However, they've not been imported into the Plugin for VSCode.
Example:

@S-Tasker Yes I'm working on that (https://github.com/microsoft/vscode-azurearmtools/pull/1558).  In the particular case @tshaiman mentions, though, the schema update doesn't seem to fix it because the one version doesn't appear to be propagated properly to the schema root.

Updated schemas will ship in next release.
Working:
Microsoft.Storage/storageAccounts/fileServices
Still not working (due to schema issues mentioned above):
Microsoft.OperationalInsights/workspaces

@StephenWeatherford I've kicked off a run of https://github.com/Azure/azure-resource-manager-schemas/actions/runs/4432345621 - should hopefully shed some light on what's going on with operationalinsights.

@jorgecotillo any context on this? https://github.com/Azure/azure-resource-manager-schemas/commit/f085aebf4b3f6487224a1f396a511f8e1fc60eeb

@anthony-c-martin there is an enum mismatch and hasn't been updated:
https://github.com/Azure/azure-rest-api-specs/blob/ee157d6bbf6e4c4fdc8a47b380e00ddd56a0e852/specification/operationalinsights/resource-manager/Microsoft.OperationalInsights/stable/2022-10-01/Tables.json#L761 -> enum vs values.
I'll create a PR in the rest-api-specs repo.

when trying to update the VS Code Plugin v0.15.9 you get an error :
2023-03-23 03:31:53.644 [error] Extension 'github.copilot' wants API proposal 'interactive' but that proposal DOES NOT EXIST. Likely, the proposal has been finalized (check 'vscode.d.ts') or was abandoned.
2023-03-23 03:31:56.154 [info] [perf] Render performance baseline is 16ms
2023-03-23 03:32:14.432 [error] RequestService#request (browser) - error https://marketplace.visualstudio.com/_apis/public/gallery/extensionquery {"name":"Canceled"}
2023-03-23 03:32:22.643 [error] Signature: PackageIntegrityCheckFailed
    at yt.download (vscode-file://vscode-app/c:/Users/tshaiman/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/code/node/sharedProcess/sharedProcessMain.js:91:468)
    at async X.x (vscode-file://vscode-app/c:/Users/tshaiman/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/code/node/sharedProcess/sharedProcessMain.js:91:20760)
    at async X.h (vscode-file://vscode-app/c:/Users/tshaiman/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/code/node/sharedProcess/sharedProcessMain.js:91:18897)
2023-03-23 03:32:22.663 [error] PackageIntegrityCheckFailed: Signature: PackageIntegrityCheckFailed
    at yt.download (vscode-file://vscode-app/c:/Users/tshaiman/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/code/node/sharedProcess/sharedProcessMain.js:91:468)
    at async X.x (vscode-file://vscode-app/c:/Users/tshaiman/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/code/node/sharedProcess/sharedProcessMain.js:91:20760)
    at async X.h (vscode-file://vscode-app/c:/Users/tshaiman/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/code/node/sharedProcess/sharedProcessMain.js:91:18897)

I saw that, too, but upon investigating it didn't seem to be related to this extension.  Can you try F1->Disable All Installed Extensions, then reenable ARM Tools and try again?

same same

@tshaiman Continuing this thread here: #1572

API Schemas are not updated with last version in v0.15.10 :

@Philippe85 Fixed in next release, thanks.

Microsoft.OperationalInsights/workspaces tracked in https://github.com/microsoft/vscode-azurearmtools/issues/1628, everything else should be working in next release.


Add URL Handler to pass winget commands from a browser

[x] Have you signed the Contributor License Agreement?
[ ] Are you working against an Issue?

This change would allow publishers to provide direct links to install with winget in the form winget:install Package.ID
This also allows users to run any winget commands from their browser using winget:<command>
I'm almost certain I've done quite a few things wrong here, but just trying to use it as a learning opportunity for myself. If this isn't something which the team wants to support/implement yet, let me know and I'll close the PR.

Microsoft Reviewers: Open in CodeFlow

I like the idea of a URI scheme, the MS Store does something similar and so does the App Installer. But we need to think this very thoroughly. We probably need to consider the security implications of allowing people to create links that install stuff in the computers of whoever clicks them, we'd at least need to add a confirmation prompt.
I think having a link in the web open an app is something that would be more at home in an app with a GUI than in a CLI because anybody could open it and I imagine most people would not know what is happening if a command prompt shows up.
The scheme should support only specific commands, not any arbitrary one. For example, I don't see the point of winget:source list.
When we introduce new URI schemes, we also have to do an internal review. The scheme should be prefixed with ms-, like ms-appinstaller or ms-edge. We probably would have to use a different syntax than with the CLI. For example, winget:install My.App --locale es-ES doesn't really look like an URI, and we'd have to make up a new syntax, something like winget:install/My.App?locale=es-ES.
I think this could be done with a separate GUI that used the COM API, instead of adding it to the CLI.

All great points which I hadnt considered; I think that right now, this is something which needs more refinement


compatresource transitive dependency failing

Hello, i am trying to use the cookbook, but unfortunately if fails at the compilation phase because one of the transitive dependencies (compat_resource), is failing. The compat_resource is a dependency of the rsyslog cookbook. The output is the following:
Recipe Compile Error in /tmp/vagrant-chef/beece1e7a10e090f42116de5cfb11692/cookbooks/compat_resource/libraries/autoload.rb
==> ncrcoe-prometheus-vm-centos7: ================================================================================
==> ncrcoe-prometheus-vm-centos7:
==> ncrcoe-prometheus-vm-centos7: NoMethodError
==> ncrcoe-prometheus-vm-centos7: -------------
==> ncrcoe-prometheus-vm-centos7: undefined method full_gem_path=' for #<Gem::Specification:0x227fc0c -> ==> ncrcoe-prometheus-vm-centos7:  ==> ncrcoe-prometheus-vm-centos7: Cookbook Trace: ==> ncrcoe-prometheus-vm-centos7: --------------- ==> ncrcoe-prometheus-vm-centos7:   /tmp/vagrant-chef/beece1e7a10e090f42116de5cfb11692/cookbooks/compat_resource/files/lib/compat_resource/gemspec.rb:10:in block in module:CompatResource'
==> ncrcoe-prometheus-vm-centos7:   /tmp/vagrant-chef/beece1e7a10e090f42116de5cfb11692/cookbooks/compat_resource/files/lib/compat_resource/gemspec.rb:8:in new' ==> ncrcoe-prometheus-vm-centos7:   /tmp/vagrant-chef/beece1e7a10e090f42116de5cfb11692/cookbooks/compat_resource/files/lib/compat_resource/gemspec.rb:8:in module:CompatResource'
==> ncrcoe-prometheus-vm-centos7:   /tmp/vagrant-chef/beece1e7a10e090f42116de5cfb11692/cookbooks/compat_resource/files/lib/compat_resource/gemspec.rb:7:in <top (required)>' ==> ncrcoe-prometheus-vm-centos7:   /tmp/vagrant-chef/beece1e7a10e090f42116de5cfb11692/cookbooks/compat_resource/libraries/autoload.rb:15:in require_relative'
==> ncrcoe-prometheus-vm-centos7:   /tmp/vagrant-chef/beece1e7a10e090f42116de5cfb11692/cookbooks/compat_resource/libraries/autoload.rb:15:in `<top (required)>'
==> ncrcoe-prometheus-vm-centos7:
==> ncrcoe-prometheus-vm-centos7: Relevant File Content:
==> ncrcoe-prometheus-vm-centos7: ----------------------
==> ncrcoe-prometheus-vm-centos7: /tmp/vagrant-chef/beece1e7a10e090f42116de5cfb11692/cookbooks/compat_resource/files/lib/compat_resource/gemspec.rb:
==> ncrcoe-prometheus-vm-centos7:
==> ncrcoe-prometheus-vm-centos7:   3:  end
==> ncrcoe-prometheus-vm-centos7:   4:
==> ncrcoe-prometheus-vm-centos7:   5:  require_relative 'version'
==> ncrcoe-prometheus-vm-centos7:   6:
==> ncrcoe-prometheus-vm-centos7:   7:  module CompatResource
==> ncrcoe-prometheus-vm-centos7:   8:    GEMSPEC = Gem::Specification.new do |s|
==> ncrcoe-prometheus-vm-centos7:   9:      # Gem path is cookbook root
==> ncrcoe-prometheus-vm-centos7:  10>>     s.full_gem_path = File.expand_path('../../../..', FILE)
==> ncrcoe-prometheus-vm-centos7:  11:      s.name = "compat_resource"
==> ncrcoe-prometheus-vm-centos7:  12:      s.version = CompatResource::VERSION
==> ncrcoe-prometheus-vm-centos7:  13:      s.platform = Gem::Platform::RUBY
==> ncrcoe-prometheus-vm-centos7:  14:      s.summary = 'Bring some new features of Chef 12.5 to previous 12.X releases'
==> ncrcoe-prometheus-vm-centos7:  15:      s.description = s.summary
==> ncrcoe-prometheus-vm-centos7:  16:      s.author = 'John Keiser'
==> ncrcoe-prometheus-vm-centos7:  17:      s.email =<EMAIL_ADDRESS>==> ncrcoe-prometheus-vm-centos7:  18:      s.homepage = 'http://chef.io'
==> ncrcoe-prometheus-vm-centos7:  19:      s.license = 'Apache 2.0'

technically this would have been a bug in compat_resource, but based on the error it looks like you're using a vagrant box with a /tmp/vagrant-chef install and not omnibus-chef which is probably chef 11.x or worse, in which case rsyslog and other community cookbooks have abandoned chef 11 support.
this cookbook should probably document that it is chef >= 12.1 only.

Hello there, apologies for the late delay. In our deployment we have combination of Chef 11 and Chef 12 systems. The problem is actually not of the prometheus cookbook but of a transitive dependency. I managed to solve this by pinning the version of compat_resources to the last compatible version with early versions of chef 12.
The chef version used in the particular box mentioned mentioned above is running an early version of chef 12. I could change it easily to the latest chef 12, but still the point is that we have old systems that are running chef 11 and can not be upgraded because of other cookbooks that are incompatible with chef 12.
I have managed to deploy prometheus using the workaround mentioned, but please document this issue in order for people to know what they should expect if they try to use it with old versions of chef.
Thank you for your time


fix(a11y): completed-step focus accessibility

Changed completed-step outline color to secondary color on focus
Checklist:

[x] I have read and followed the contribution guidelines.
[x] I have read and followed the how to open a pull request guide.
[x] My pull request targets the main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine, or GitPod.

Closes #50023

The focus outline looks good in both light/dark mode.
From the issue:

Also, while we are fixing this, I would recommend we increase the font weight from 700 to 900 and the font size from 1.3rem to 1.5rem on the completed steps to make the difference in those numbers stand out a little more since we can't rely on the background color for the completed steps to convey their state since they don't have enough color contrast with the background.

Any objection to doing this? I feel like the current difference in the display of the completed step numbers is barely enough to be detectable and thus we are just barely fulfilling the requirement for WCAG 1.4.1.

Had some doubts in that, you only want this behaviour to occur when the elements are keyboard focused only or on mouse hover as well. I think it makes sense to do it for both.

Hi @bbsmooth , any update on this? Are we taking the font size changes on as other features. Also, can you help me in understanding what changing of the title means?

Hi @bbsmooth , any update on this? Are we taking the font size changes on as other features. Also, can you help me in understanding what changing of the title means?

@harmeetsingh01, adding hover is outside the scope of the issue and what Bruce worried about.
We can discuss it after the original issue is closed, because we can improve the design as well if needed.

Understood @Sboonny , will add the discussed font changes on focus for now.

@harmeetsingh01 Sorry, I've been gone for the past couple of weeks and forgot to get back to you on the font size and weight. It's not my decision, but it sounds like @Sboonny is OK with increasing both to my suggestions for completed steps.
Please note, this has nothing to do with either focus or hover. The font changes should be present at all times for the completed step. Right now the font size on a completed step is 1.3rem and the font-weight is 700. But that change is hardly noticeable. So I am suggesting we up it to 1.5rem and 900 to make it stand out more.

Oh, Thanks for clearing it up, I was a bit confused and thought we want the increase in font size on focus or hover. Understood, will push the necessary change. Apologise for the misunderstanding.

Thanks to you @Sboonny and @bbsmooth as well for helping me. Looking forward to more such collaborations in the future.


UseAuthorization must come before UseCors

See https://github.com/dotnet/aspnetcore/issues/20709
cc @serpent5

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

ID: 59bca5db-1196-2a8b-a0d8-4e89b2382288
Version Independent ID: d3f332c4-4c60-039e-0e1e-bc6b11831f87
Content: Enable Cross-Origin Requests (CORS) in ASP.NET Core
Content Source: aspnetcore/security/cors.md
Product: aspnet-core
Technology: aspnetcore-security
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

I believe this is the wrong way round, UseCors() must come before UseAuthorization
Working code example (From other issue):
    public void Configure(IApplicationBuilder app)
    {
        app.UseDeveloperExceptionPage();

        app.UseRouting();

        app.UseCors();

        app.UseAuthentication();
        app.UseAuthorization();

        app.UseEndpoints(endpoints =>
        {
            endpoints.MapControllers();
        });
    }


Fix UB names in MHA

PR #499 introduces ub_name parameter, which is not optional when using UB, but did not update MHA to actually pass it. This PR fixes that.

/te-ci pytorch


[WFLY-15188]: Upgrade artemis-wildfly-integration to 1.0.5 in WildFly Preview.

Upgrade version in preview only as artemis upgrade is required for
global update
Use the new jakarta native package

Jira: https://issues.redhat.com/browse/WFLY-15188

WF Preview tests:
https://ci.wildfly.org/viewLog.html?buildId=270176&tab=buildResultsDiv&buildTypeId=WF_Ee9BootableJarLinuxJdk8
https://ci.wildfly.org/viewLog.html?buildId=270174&tab=buildResultsDiv&buildTypeId=WF_WildFlyPreviewLinuxJdk11


[mlr] fix multicast group re-register after bbr restarting.

After BBR restarting, in order to receive the packets from multicast group, the devices, which have registered multicast group addresses to that BBR, needs to re-register these groups.
In this PR, a member variable mRecordedSequenceNumber is added to record the BBR's sequence number, if the BBR's sequence number has changed,  all registered groups would be re-registered.

I think we have already implemented this feature in MlrManager::Reregister, which can be triggered when BBR sequence number changes.

Reregister

I think we have already implemented this feature in MlrManager::Reregister, which can be triggered when BBR sequence number changes.

Hi @simonlingoogle , In the original logic, when BBR restart, other devices should wait for  MLR Timeout (300s), then re-registered. In this PR,  Once detected BBR changed or restarted, other devices handle re-registeration immediately.

I think we have already implemented this feature in MlrManager::Reregister, which can be triggered when BBR sequence number changes.

Hi @simonlingoogle , In the original logic, when BBR restart, other devices should wait for MLR Timeout (300s), then re-registered. In this PR, Once detected BBR changed or restarted, other devices handle re-registeration immediately.

When the BBR sequence number change, the re-registration delay would be Random::NonCrypto::GetUint16InRange(1, config.mReregistrationDelay), which is inline with the Thread spec.
See https://github.com/openthread/openthread/blob/0a41196540044d074bc8d68c666d3efb6c66e0fc/src/core/thread/mlr_manager.cpp#L651-L656

I think we have already implemented this feature in MlrManager::Reregister, which can be triggered when BBR sequence number changes.

Hi @simonlingoogle , In the original logic, when BBR restart, other devices should wait for MLR Timeout (300s), then re-registered. In this PR, Once detected BBR changed or restarted, other devices handle re-registeration immediately.

When the BBR sequence number change, the re-registration delay would be Random::NonCrypto::GetUint16InRange(1, config.mReregistrationDelay), which is inline with the Thread spec.
See
https://github.com/openthread/openthread/blob/0a41196540044d074bc8d68c666d3efb6c66e0fc/src/core/thread/mlr_manager.cpp#L651-L656

I think the reregDelay in mlr_manager.cpp#L651-L656 is used for maintaining registered groups timing out, which is defined in spec <IP_ADDRESS>.2. For the scenario related in this PR, when BBR restarted, some definitions with BBR sequence number updating is related in <IP_ADDRESS>, the spec just defined Any change of the BBR Sequence Number should trigger Thread Devices to perform reregistration. I think in this scenario, Thread Devices should re-register as soon as possible. If followed the current logic, in the worst case, Thread Devices might wait almost 1 hour to re-register.

For the scenario related in this PR, when BBR restarted, some definitions with BBR sequence number updating is related in <IP_ADDRESS>, the spec just defined Any change of the BBR Sequence Number should trigger Thread Devices to perform reregistration. I think in this scenario, Thread Devices should re-register as soon as possible. If followed the current logic, in the worst case, Thread Devices might delay almost 1 hour to re-register.

I think the Reregistration Delay is for this very purpose (see <IP_ADDRESS>)
Thoughts? @jwhui

Reregistration Delay should be respected.

Okay, I know what you mean, should we decrease the default reregisteration Delay(which default value is 1200, it seems to large)?

Okay, I know what you mean, should we decrease the default reregisteration Delay(which default value is 1200, it seems to large)?

Agree that 1200s is too large for typical networks.
Changing the default value from a Thread Spec's perspective might be difficult, but I think we can make it configurable in OpenThread. Or you can simply set BBR dataset in production code using otBackboneRouterSetConfig.

Okay, I know what you mean, should we decrease the default reregisteration Delay(which default value is 1200, it seems to large)?

Agree that 1200s is too large for typical networks.
Changing the default value from a Thread Spec's perspective might be difficult, but I think we can make it configurable in OpenThread. Or you can simply set BBR dataset in production code using otBackboneRouterSetConfig.

The default 1200s seems not defined by spec. Per spec <IP_ADDRESS>, the Reregistration Delay value is vendor-specific, the range is (1 - 0.5 * MLR_Timeout) in seconds.
How about reducing the default Reregistration Delay valude in openthread to some small value (like 5)? Which should be accepted by most production users.

The default 1200s seems not defined by spec. Per spec <IP_ADDRESS>, the Reregistration Delay value is vendor-specific, the range is (1 - 0.5 * MLR_Timeout) in seconds.

Thanks for the info.

How about reducing the default Reregistration Delay value in openthread to some smaller value (like 5)? Which should be accepted by most production users.

+1 for reducing default reregistration delay.


What you can do when you get Max decoder steps reached

Whenever you get the max decoder steps reached in inference, your audio text pairs have errors. You have to train from scratch again with good pairs.

might be related to my issue #502 since I get the same error iirc- i still don't know the answer either though, sorry.

https://github.com/NVIDIA/tacotron2/blob/185cd24e046cc1304b4f8e564734d2498c6e2e6f/hparams.py#L59
You can change the max number of steps, however the model can have issues above 1000.
Normally this statement means you have input too much text to the model for inference or your model is not trained well.

your model is not trained well.

That's what I am referring to.

Ensure you have a large amount of data (30+ minutes)
Use a pretrained model as a base to --warm_start from
Ensure all transcripts in your dataset match the audio perfectly
Remove any files with background noise
Perform inference in batches and filter out files with poor alignments or use beam search/greedy search style inference with chunks of frames.

If things still fail.

Decrease hop_length if your speaker speaks abnormally fast and retrain the vocoder + tacotron from scratch on a large dataset before transfer-learning OR
Use a different model that uses duration based alignment

@Fennecai
https://github.com/NVIDIA/tacotron2#training-using-a-pre-trained-model
Example usage is on the README.

5: can you describe this again on a complete-idiot's level?
Not really. This is for the people who can write the inference code more than people who are just trying to use it. But it's a method of filtering out poor results + instability that works quite well.

that works quite well.

I have to agree. It's good to know that if there's an error, it's not in the original model so the area where problems arise is very limited and under your control.
I learnt it the hard way that a fastidiously fostered dataset is the be-all and end-all.

@CookiePPP hi, can you tell me what is the maximum duration of a file .wav in dataset. (because some file in my dataset > 11.5s and all file < 12s)
Thanks

@toanil315
There is no limit to the maximum duration of your training/validation audio files. The max_decoder_steps is used when generating new audio using Tacotron2 + WaveGlow using new text.
In fact, training on longer audio files will increase the max_decoder_steps that you can use safely with the model.

@CookiePPP thanks for reply, have good day

@CookiePPP when i train from scratch, should i change learning rate with recipe 0.001 * (batch_size/256)**0.5
because with my hardware i'm using, i'll use with batch_size = 16

@CookiePPP hi, this is my alignment after 38k iterations

i'm traning with batch_size = 8 and learning_rate = 1.8e-4 (with recipe: 0.001 * (batch_size/256)**0.5)
this seem model can't learn
should i wait for some iterations or retrain it?
this is my dataset https://drive.google.com/file/d/1QbvrLPjSQX6vrU2Zor8zOXIqg4xM9m4h/view?usp=sharing
thanks


[pod-identity] Scale for large number of identities

https://github.com/Azure/aad-pod-identity/issues/145


[v1.10] .github: add conformance AWS test

This conformance test is available in the master branch but hasn't been
backported for v1.10 branch. This commit is simply copying it into this
branch.
Signed-off-by: André Martins<EMAIL_ADDRESS>Depends on https://github.com/cilium/cilium/pull/16908

ci-awscni

The aws-cni tests have passed as seen in https://github.com/cilium/cilium/pull/16913/checks?check_run_id=3106710500

Closing in favor of #16950