Use search api to get list of actions

closes #297
Other improvements 👍

Improves the performance of action by using the code search api.
Number of api calls and duration of run is drastically reduced. 21m 36s to 1m 8s in the tested org

Working on incorporating the readme related changes.


Add option to enable/disable auto margin on carousels

By default all items on a carousel add a margin depending on the space available. But since we can't control that margin, some times we can't meet certain design needs.
To fix that I added an option to enable/disable this behaviour, leaving it enables by default.
Here is a test case to show the new functionality.
Thanks to @fambrosi for the help!

@battaglr Would be nice to have this o similar test case in the views for development, really very handy.

@atma, awesome! It's a good idea, I'll add something simpler —using just an <img />— in another PR. I'll also open an issue so I don't forget.


feat: add created by chart CRUD view

SUMMARY

add created_by to chart CRUD view

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TEST PLAN

ADDITIONAL INFORMATION

[ ] Has associated issue:
[x] Changes UI
[ ] Requires DB Migration.
[ ] Confirm DB Migration upgrade and downgrade tested.
[ ] Introduces new feature or API
[ ] Removes existing feature or API

Codecov Report

Merging #11067 into master will decrease coverage by 4.01%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master   #11067      +/-   ##
==========================================
- Coverage   65.79%   61.77%   -4.02%
==========================================
  Files         816      433     -383
  Lines       38422    14235   -24187
  Branches     3621     3622       +1
==========================================
- Hits        25280     8794   -16486
+ Misses      13034     5255    -7779
- Partials      108      186      +78

Flag
Coverage Δ

#cypress
?

#javascript
61.77% <100.00%> (+<0.01%)
:arrow_up:

#python
?

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

Impacted Files
Coverage Δ

...perset-frontend/src/views/CRUD/chart/ChartList.tsx
72.26% <100.00%> (-10.64%)
:arrow_down:

superset-frontend/src/SqlLab/App.jsx
0.00% <0.00%> (-100.00%)
:arrow_down:

superset-frontend/src/explore/App.jsx
0.00% <0.00%> (-100.00%)
:arrow_down:

superset-frontend/src/dashboard/App.jsx
0.00% <0.00%> (-100.00%)
:arrow_down:

superset-frontend/src/explore/index.jsx
0.00% <0.00%> (-100.00%)
:arrow_down:

superset-frontend/src/dashboard/index.jsx
0.00% <0.00%> (-100.00%)
:arrow_down:

superset-frontend/src/setup/setupColors.js
0.00% <0.00%> (-100.00%)
:arrow_down:

superset-frontend/src/chart/ChartContainer.jsx
0.00% <0.00%> (-100.00%)
:arrow_down:

superset-frontend/src/setup/setupFormatters.js
0.00% <0.00%> (-100.00%)
:arrow_down:

superset-frontend/src/explore/reducers/index.js
0.00% <0.00%> (-100.00%)
:arrow_down:

... and 548 more

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 d056e3d...d824eca. Read the comment docs.

Roadmap: https://github.com/apache-superset/superset-roadmap/issues/47


[3141] Fixed query for provider_name

Context
The accredited body does not see all the courses they expected to see on Find when they search for a specific provider.
What should have happened? (expected results)
The Find search should have displayed all the courses that U80 accredits including their own courses.
Changes proposed in this pull request
When you search for a provider on Find, you should see all courses offered by the provider, including the courses they accredit.
Guidance to review
:ship:
Checklist

[x] Make sure all information from the Trello card is in here
[ ] Attach to Trello card
[x] Rebased master
[x] Cleaned commit history
[x] Tested by running locally

 Searching for an accredited body no longer shows all courses they are the awarding body for


(fix) strategy options inputs

REOPEN: All security fields (capital, dd and stoploss) >0

allow decimal values for all inputs
validate for greater than 0

https://github.com/bitfinexcom/bfx-hf-ui-core/blob/main/src/modals/Strategy/ExecutionOptionsModal/ExecutionOptionsModal.js#L108-L113


Error: Cannot find module 'prettier'

get the following issue when running postman-to-k6 PostmanScript.json -e PostmanGlobals.json -o k6-script.js
node:internal/modules/cjs/loader:1042
throw err;
^
Error: Cannot find module 'prettier'
Require stack:

<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>at Module._resolveFilename (node:internal/modules/cjs/loader:1039:15)
at Module._load (node:internal/modules/cjs/loader:885:27)
at Module.require (node:internal/modules/cjs/loader:1105:19)
at require (node:internal/modules/cjs/helpers:103:18)
at Object. (/opt/homebrew/lib/node_modules/@apideck/postman-to-k6/lib/render/index.js:3:18)
at Module._compile (node:internal/modules/cjs/loader:1218:14)
at Module._extensions..js (node:internal/modules/cjs/loader:1272:10)
at Module.load (node:internal/modules/cjs/loader:1081:32)
at Module._load (node:internal/modules/cjs/loader:922:12)
at Module.require (node:internal/modules/cjs/loader:1105:19) {
code: 'MODULE_NOT_FOUND',
requireStack: [
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>]
}

Node.js v19.3.0

Hello,
Same here, I installed Node minutes before, then ran npm install -g @apideck/postman-to-k6. All looked good.
I ran postman-to-k6 -e local.postman_environment.json -s test.postman_collection.json -o k6-script.js and got the same error.
Running with no args at all produces the same behavior.
I hope this helps.
My env :
Windows 10
NodeJS v19.5.0
NPM 9.3.1

Saw the same error with "@apideck/postman-to-k6": "^1.8.7".
For others that see this, the easy workaround is to manually install the  prettier package.  prettier version 2.8.3 works for me.
npm install -D prettier

install prettier seperately solves this issue

Saw the same error with "@apideck/postman-to-k6": "^1.8.7".
For others that see this, the easy workaround is to manually install the prettier package. prettier version 2.8.3 works for me. npm install -D prettier

Yes it is a workaround but not a solution.
In like a CI/CD system, you don't want to do this.
I'll try to do a PR to fix it when I have time.

Hi all,
We just released version 1.8.8 of postman-to-k6 with a fix from @BenjaminDls that will solve this issue.


Notion page Opengraph Image not show in live site.

Description
After deploy my site in vercel it does not show Opengraph images in site. It shows blank only. Please tell how i fix it.

Notion Test Page ID

Use vercel node-version: 14
Change package.json file to:
"engines": {
"node": "14.x"


fix: running e2e tests as part of build, requires forcing ip4 (not ip6) when connecting to anvil

as title

Benchmark results
Metrics with a significant change:

note_history_successful_decrypting_time_in_ms (5): 1,155 (-26%)
note_history_successful_decrypting_time_in_ms (10): 2,220 (-42%)
note_successful_decrypting_time_in_ms (8): 297 (+34%)
note_trial_decrypting_time_in_ms (8): 100 (+348%)
note_trial_decrypting_time_in_ms (64): 70.7 (+109%)

Detailed results
All benchmarks are run on txs on the Benchmarking contract on the repository. Each tx consists of a batch call  to create_note and increment_balance, which guarantees that each tx has a private call, a nested private call, a public call, and a nested public call, as well as an emitted private note, an unencrypted log, and public storage read and write.
This benchmark source data is available in JSON format on S3 here.
L2 block published to L1
Each column represents the number of txs on an L2 block published to L1.

Metric
8 txs
32 txs
64 txs

l1_rollup_calldata_size_in_bytes
772
772
772

l1_rollup_calldata_gas
6,808
6,772
6,796

l1_rollup_execution_gas
587,289
587,253
587,277

l2_block_processing_time_in_ms
1,426 (-4%)
5,223 (-3%)
10,315

note_successful_decrypting_time_in_ms
:warning: 297 (+34%)
552 (+2%)
954 (+4%)

note_trial_decrypting_time_in_ms
:warning: 100 (+348%)
58.2 (+14%)
:warning: 70.7 (+109%)

l2_block_building_time_in_ms
9,687 (-1%)
37,936 (-1%)
76,222

l2_block_rollup_simulation_time_in_ms
9,494 (-1%)
37,224 (-1%)
74,833

l2_block_public_tx_process_time_in_ms
5,306 (-1%)
20,883
41,841 (-1%)

L2 chain processing
Each column represents the number of blocks on the L2 chain where each block has 16 txs.

Metric
5 blocks
10 blocks

node_history_sync_time_in_ms
15,763 (-10%)
28,997 (-6%)

note_history_successful_decrypting_time_in_ms
:warning: 1,155 (-26%)
:warning: 2,220 (-42%)

note_history_trial_decrypting_time_in_ms
35.9 (-69%)
81.0 (-41%)

node_database_size_in_bytes
18,722,896
35,139,664

pxe_database_size_in_bytes
29,859
59,414

Circuits stats
Stats on running time and I/O sizes collected for every circuit run across all benchmarks.

Circuit
circuit_simulation_time_in_ms
circuit_input_size_in_bytes
circuit_output_size_in_bytes

private-kernel-init
148 (-3%)
44,561
26,424

private-kernel-inner
157 (-2%)
72,188
26,424

private-kernel-ordering
109 (-4%)
51,090
39,297

base-parity
52.2 (-3%)
128
311

root-parity
10.2 (-3%)
1,244
311

base-rollup
12,560 (-3%)
110,820
957

root-rollup
48.6 (-1%)
4,551
821

public-kernel-app-logic
82.6 (-3%)
48,151
40,933

public-kernel-tail
124 (-2%)
53,644
7,385

merge-rollup
11.8 (+9%)
2,760
957

public-kernel-teardown
78.4 (-1%)
48,151
40,933

public-kernel-setup
77.3 (-1%)
48,151
40,933

Tree insertion stats
The duration to insert a fixed batch of leaves into each tree type.

Metric
0 registered classes
1 registered classes

tx_size_in_bytes
41,005
528,889

Transaction size based on fee payment method

Metric
native fee payment method
fpc_public fee payment method
fpc_private fee payment method

tx_with_fee_size_in_bytes
905
1,161
1,377

Transaction processing duration by data writes.

Metric
0 new note hashes
1 new note hashes
2 new note hashes

tx_pxe_processing_time_ms
1,182 (+1%)
788 (-3%)
3,397 (-1%)

Metric
1 public data writes
2 public data writes
3 public data writes
4 public data writes
5 public data writes
8 public data writes

tx_sequencer_processing_time_ms
460 (-3%)
306 (-2%)
813 (-3%)
395 (-6%)
1,345 (-3%)
463 (-1%)


Add App Configuration example for Service Fabric Stateless ASP.net Core API

You would need to enable Managed Identity in the VM and add that identity to the App Config Store with ‘App Configuration Data Reader’ or ‘App Configuration Data Owner’ roles, and then follow Service Fabric’s documentation  to obtain tokens needed to access the Config Store.


Make Wifibox more intuitive to install

Recently Wifibox received some criticism about the shipped sample configuration files and documentation:

https://jrgsystems.com/posts/2022-04-20-802.11ac-on-freebsd-with-wifibox/ "I was a bit confused at first when configuring wifibox after installing the pkg. The man page (man wifibox) specifies modifying the *.conf.sample files in the /usr/local/etc/wifibox/ directory to configure it but does not specify which settings are necessary for software to start."
https://twitter.com/hashtag/wifibox "Well💩 For whatever reason default route was not added (#wifibox); needed to add one self. Before that, also had started wpa_supplicant in the VM, which did not complain about an instance already running. So starting|configuration of #wifibox is still damn fiddly😫"

Find ways to make it easier to use.

Created 17d81a57a99844fae1394290dd1b074f689f1ccc to improve the comments in the sample configuration files.  In addition to this, added a pkg-message for the net/wifibox port in https://github.com/pgj/freebsd-wifibox-port/commit/35773e14f2b761aeb71858bb8cdf3b44d8d029dc .

Hopefully, https://github.com/pgj/freebsd-wifibox-port/commit/6c3d029e0a8c32467e6717f4d56be7eba3d2f730 will also simplify the configuration.

All these changes are made part of main and backported for the 0.8 version.


STP Documentation and Cleanup - Headers

Considering the size of this PR already, I will refactor the implementations in a separate PR

Reformatting of code using clang-format (according to @HaicoDorenbos his standard that can be found in roboteam_suite/formatting) ((basically clang-format google style with extended column size))
Revision of documentation
General cleanup things
Renaming (Open for suggestions)

closes #1164

Wow, 147 files changed is quite a lot and when looking at the PR title I guess it could have been split up in multiple PR's. So next time could you split up your PR in smaller ones? :blush:

Set indentation to 4.

Changes are a lot more obvious now that indentation width is back to 4 (thanks @Yuhanun). Don't mind the formatting of the .cpp files, this will be done in follow up PRs

Trying to fix, not sure why it fails, though.

I started reviewing this and then saw you were going to split it into multiple PR's, so I'll wait for that. But I did find this typo! :yum:

I'm not splitting this PR though, only future ones. It's basically 1 change in every file.

Conditionally approved :) I think the branch should be tested one last time as it introduces quite a lot of changes

Do you also want to test @RobotJesse ? I tested just now and found 2 small issues and fixed them.

Regarding the code style format: I did not agree on this code style format (for example the line widths are quite long). However I agreed that using a code style format (even if it is a bad one) is better than not using a code style format at all.

Regarding the code style format: I did not agree on this code style format (for example the line widths are quite long). However I agree that using a code style format (even if it is a bad one) is better than not using a code style format at all, similar to this meme: https://res.cloudinary.com/practicaldev/image/fetch/s--H8LtA_7W--/c_limit%2Cf_auto%2Cfl_progressive%2Cq_auto%2Cw_880/http://i.imgur.com/vOWAAUK.png

Well, you made the formatting tool in roboteam_suite, from which I copied the settings. The only thing I changed (by request from @Yuhanun) is the indent width (which I completely agree with btw). So basically you did not agree to it, but invented it haha


Add static index signature

Fixes #6480

@typescript-bot pack this.

@typescript-bot pack this.

@weswigham the reviewer list is broken on this PR, but you should look at this too

⬆

Ping @DanielRosenwasser, @weswigham.

Hi, Folks, I don't know what I can do. Should I work with the parser issue inside this PR?

I would say no - though, if you want to tackle the problem, coordinate with @rbuckton on #38283.
Otherwise, I'm not familiar enough with the internals. @elibarzilay and @weswigham, can you give this another review?

Seems something wrong if extends others

Seems eslint is broken, not my fault.

@typescript-bot test this.

@typescript-bot run tests.

up↑

Friendly ping. Up again.

Hey folks, Could this one be included into 4.0?

Up...

Uhhhhh... Up.

@elibarzilay you wanna make sure this gets merged for 4.2? (After we cut the 4.1 RC in two weeks)

Emmmmm... Up

@typescript-bot pack this

As #41506.
@typescript-bot pack this.

It was mentioned in the last design meeting https://github.com/microsoft/TypeScript/issues/41538 that that this doesn't enforce anything on the property types, which is currently a blocker for the PR.

I'm sorry I'm not understand.
Would you mind provide an example for the case?

class X {
    static [index: string]: string;
    static x = 12; // Should error, incompatible with index signature
}
class Y {
    static [index: string]: string;
    static foo() {} // should error, incompatible with index signature
}

@typescript-bot pack this.

@typescript-bot pack this.

@elibarzilay @weswigham I'd like to get this in early in the 4.3 cycle. Does this (1) have enough tests now (2) error correctly on property types?

Friendly ping 👀

👀PiNg AgAiN

just for verify
@typescript-bot pack this.

Hrm, I'm not certain why the playground didn't build (as this seems to be completely up to date with master), I'll look into add some more logs in the build process.

Try again.
@typescript-bot pack this.


Major BUG: Latest Helm go client library is not compatibale with k8s.io/apimachinery/pkg/runtime/serializer/json

Helm go client package - "helm.sh/helm/v3/pkg/action" is importing "k8s.io/apimachinery/pkg/runtime/serializer/json".
After importing this package, helm is using below variable:
// hold a single instance of the case-sensitive decoder
var caseSensitiveJsonIterator = json.CaseSensitiveJSONIterator()
whereas CaseSensitiveJSONIterator is missing/undeclared in the latest version of "k8s.io/apimachinery/pkg/runtime/serializer/json".

The latest version of Helm compiled successfully - are you sure you are using go modules? How are you installing Helm?

I just did a complete code search of helm... There is no reference to a caseSensitiveJsonIterator or to json.CaseSensitiveJSONIterator(). Perhaps you've checked out an older version of Helm?
Either case this doesn't appear to be an issue on the latest version of the code. Please re-open if you still think this is an issue.


運行webui.py報錯

Reminder

[X] I have read the README and searched the existing issues.

Reproduction
!git clone https://github.com/hiyouga/LLaMA-Factory.git
%cd ./LLaMA-Factory
!pip install typer --upgrade
!pip install -r requirements.txt
!pip install bitsandbytes>=0.39.0
!CUDA_VISIBLE_DEVICES=0 python src/webui.py
第一次運行時:
2024-05-09 17:38:11.689710: E external/local_xla/xla/stream_executor/cuda/cuda_dnn.cc:9261] Unable to register cuDNN factory: Attempting to register factory for plugin cuDNN when one has already been registered
2024-05-09 17:38:11.689784: E external/local_xla/xla/stream_executor/cuda/cuda_fft.cc:607] Unable to register cuFFT factory: Attempting to register factory for plugin cuFFT when one has already been registered
2024-05-09 17:38:11.691110: E external/local_xla/xla/stream_executor/cuda/cuda_blas.cc:1515] Unable to register cuBLAS factory: Attempting to register factory for plugin cuBLAS when one has already been registered
2024-05-09 17:38:12.745580: W tensorflow/compiler/tf2tensorrt/utils/py_utils.cc:38] TF-TRT Warning: Could not find TensorRT
Running on local URL:  http://<IP_ADDRESS>:7860/
To create a public link, set share=True in launch().
打開連結後顯示
2024-05-09 17:38:11.689710: E external/local_xla/xla/stream_executor/cuda/cuda_dnn.cc:9261] Unable to register cuDNN factory: Attempting to register factory for plugin cuDNN when one has already been registered
2024-05-09 17:38:11.689784: E external/local_xla/xla/stream_executor/cuda/cuda_fft.cc:607] Unable to register cuFFT factory: Attempting to register factory for plugin cuFFT when one has already been registered
2024-05-09 17:38:11.691110: E external/local_xla/xla/stream_executor/cuda/cuda_blas.cc:1515] Unable to register cuBLAS factory: Attempting to register factory for plugin cuBLAS when one has already been registered
2024-05-09 17:38:12.745580: W tensorflow/compiler/tf2tensorrt/utils/py_utils.cc:38] TF-TRT Warning: Could not find TensorRT
Running on local URL:  http://<IP_ADDRESS>:7860/
To create a public link, set share=True in launch().
連結打開後顯示
"無法連上這個網站http://<IP_ADDRESS>:7860/ 的網頁可能暫時離線，或是已經遷移到另一個網址。"
之後將webui.py的share改為true後，為以下的報錯
2024-05-09 17:08:29.325074: E external/local_xla/xla/stream_executor/cuda/cuda_dnn.cc:9261] Unable to register cuDNN factory: Attempting to register factory for plugin cuDNN when one has already been registered
2024-05-09 17:08:29.325130: E external/local_xla/xla/stream_executor/cuda/cuda_fft.cc:607] Unable to register cuFFT factory: Attempting to register factory for plugin cuFFT when one has already been registered
2024-05-09 17:08:29.326558: E external/local_xla/xla/stream_executor/cuda/cuda_blas.cc:1515] Unable to register cuBLAS factory: Attempting to register factory for plugin cuBLAS when one has already been registered
2024-05-09 17:08:30.451209: W tensorflow/compiler/tf2tensorrt/utils/py_utils.cc:38] TF-TRT Warning: Could not find TensorRT
Running on local URL:  http://<IP_ADDRESS>:7860/
Running on public URL: https://d109b2062db62b4a39.gradio.live/
This share link expires in 72 hours. For free permanent hosting and GPU upgrades, run gradio deploy from Terminal to deploy to Spaces (https://huggingface.co/spaces)
使用Running on public URL: https://d109b2062db62b4a39.gradio.live/ 連結進行webui操作並提交嘗試訓練模型顯示失敗
報錯如下:
/bin/sh: 1: llamafactory-cli: not found
Expected behavior
No response
System Info
No response
Others
No response

务必严格参照 https://colab.research.google.com/drive/1d5KQtbemerlSDSxZIfAaWXhKr30QypiK?usp=sharing


Indexes and collections, distinct from continuities

Requested here by Alicorn, something that "worked mechanically exactly like a continuity except a) not being called that ("an index"?) and b) allowing threads to be in more than one of them. Possibly also c) the ability to add threads to them without being involved in creating the thread".

This way the Particularly Good Threads section on Alicornutopia could be moved to a Constellation index.
Unfortunately, I don’t think it would be reasonable to make it smart enough to autogenerate author indexes, since those often contain information not available in other metadata.

We'd need to have freely-editable indexes for that, I think? (Unless there's some meta post to suggest them in.) It'd also need some way to attach a little index-specific summary to the post, which current continuities don't have.

Indexes were added in #519


Dynamic mask

I want to change a mask on a text input field depending on a value in a dropdown with countries.
Is that possible with a mask function? Because I can't get it to work..

@xtoff Yes, it should be possible with a mask function. If you can provide a repo with a minimal project showing your attempt, I can try to make it work.


about classfication.py

when i run this sentence : classification_type = sys.argv[1]
IndexError: list index out of range
I don't know how to slove, can you tell me the solution?thanks

Add "binary" or "multiclass" as the option


How do you store C++\WinRT types in an STL Collection?

I need to store instances of C++\Winrt types in an stl map but am getting the an error obout them not supporting a default constructor. To date I've been getting around this by wrapping them in another type whose sole purpose is to allow me to put it into the map, but that seem a rather untenable plan.
I tried using an IMap<uint32_t, WinRTType> but the WInRT template wrapper insists on thinking the integer value passed in is an object and tries to access the internal ptr value (which is null if the integer is 0).

That's not really a C++/WinRT problem and a great question for Stack Overflow. If you have an STL container and want to use types that lack default constructors then you need to use the facilities provided by the container to support such types. For example:
using namespace Windows::Foundation;
using namespace Windows::Foundation::Collections;

int main()
{
    init_apartment();

    Uri key{ L"http://key.com/" };
    Uri value{ L"http://value.com/" };

    std::map<Uri, Uri> container;
    container.emplace(std::move(key), std::move(value));

    for (auto&& pair : container)
    {
        printf("%ls %ls\n", pair.first.Domain().c_str(), pair.second.Domain().c_str());
    }
}


Updates Quickstart for v0.2.0-rc2

Updates the quickstart for the v0.2.0-rc2 release.
Signed-off-by: danehans<EMAIL_ADDRESS>

Codecov Report

Merging #481 (c04dac9) into main (91602fe) will increase coverage by 0.09%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #481      +/-   ##
==========================================
+ Coverage   62.33%   62.42%   +0.09%
==========================================
  Files          40       40
  Lines        4436     4436
==========================================
+ Hits         2765     2769       +4
+ Misses       1527     1524       -3
+ Partials      144      143       -1

Impacted Files
Coverage Δ

internal/provider/kubernetes/gatewayclass.go
73.75% <0.00%> (+2.83%)
:arrow_up:

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

you'll also need to change the name of the svc from envoy-eg to envoy-default-eg


new butane variant: fedora-iot

using the r4e config with fiot wrapper
Docs added to include the new variant

Why are you removing / renaming the R4E ones? You can not remove a config version once it's been released.

Why are you removing / renaming the R4E ones? You can not remove a config version once it's been released.

Since we want butane to support both r4e and fedora-iot, so renaming  the config path only for developer understanding. The end functionality and the variant names is remains same. Do you see any other issue?

I think you should add only the latest version as 1.0 and the experimental one as 1.1_exp.

This is to make r4e and fiot in sync, so fiot_v1.0 being same as r4e_v1.1 f can create confusion and maintenance overhead. same for fiot_v1.1_exp -> r4e_v1.2_exp. Thoughts?

If it's the same thing, why duplicate everything?

@say-paul A bit confused. As @travier said, we cannot remove an already released config, it would break anyone using it. Additionally, it sounds like the 'fiot' would be independent of rfe? while currently they have the same needs is there no eventuality where they fork to some degree?

If it's the same thing, why duplicate everything?

@travier  So idea is both rfe and  fedora-iot user can use butane with their respective os variant. Since r4e is already there, this PR would simply add a fiot variant name with minimal code duplication.

As @travier said, we cannot remove an already released config, it would break anyone using it.

RegisterTranslator("r4e", "1.0.0", iot1_0.ToIgn3_3Bytes)
RegisterTranslator("r4e", "1.1.0", iot1_1.ToIgn3_4Bytes)
RegisterTranslator("r4e", "1.2.0-experimental", iot1_2_exp.ToIgn3_5Bytes)
RegisterTranslator("fiot", "1.0.0", iot1_0.ToIgn3_3Bytes)
RegisterTranslator("fiot", "1.1.0", iot1_1.ToIgn3_4Bytes)
RegisterTranslator("fiot", "1.2.0-experimental", iot1_2_exp.ToIgn3_5Bytes)

I understand this controls the variant name in the butane file which didn't change.
@travier  @prestist Can you please clarify how it will break for existing user?

@travier So idea is both rfe and fedora-iot user can use butane with their respective os variant. Since r4e is already there, this PR would simply add a fiot variant name with minimal code duplication.

The lifecycle for RHEL 4 Edge and Fedora IoT do not match and the Ignition spec versions supported for a given Fedora IoT release tells nothing for RHEL 4 Edge and vice versa, so having the same numbers will not get us much and might instead create confusion.

The lifecycle for RHEL 4 Edge and Fedora IoT do not match and the Ignition spec versions supported for a given Fedora IoT release tells nothing for RHEL 4 Edge and vice versa, so having the same numbers will not get us much and might instead create confusion.

Agreed, I will modify it accordingly.

I've made https://github.com/coreos/butane/issues/489 to track the R4E change.

@travier made the changes as per the discussion.

This is missing all docs update.

@say-paul Are the CI errors still valid?  Looks like they are just gofmt failures

Is there actually any "high level" sugar at all today in the r4e/fedora-iot versions?
There's only a little bit for fedora-coreos around things like boot raid etc.
I guess today we're disallowing partitioning in iot/edge?
I wonder if it'd make sense to introduce a generic e.g. "generic-systemd" style variant that just is YAML sugar for writing files and systemd units basically...

@prestist done.

@say-paul @miabbott  not from my perspective. @travier anything else ?

@prestist done

After speaking with @camgranella I am going to merge this as it lgtm!


feat: add benchmarks (skeleton)

This PR is rebased on #105
Closes #103

@spflueger We'll need so think of a smart way to smear the initial parameters after data generation and before fit. Will also need to put the example recipe files -- created with benchmark/create_recipe.py -- under source control, so that we can things like omega mass width a bit more wide.

@spflueger We'll need so think of a smart way to smear the initial parameters after data generation and before fit. Will also need to put the example recipe files -- created with benchmark/create_recipe.py -- under source control, so that we can things like omega mass width a bit more wide.

That's true. Can you open an issue for this random initial parameter problem.
I think its totally ok uploading a fixed model recipe for testing purposes.

That's true. Can you open an issue for this random initial parameter problem.

There's already this one, right? https://github.com/ComPWA/expertsystem/issues/8

@spflueger 2d6d3001a66c1d1cd03e2ad040ed0499bc0d3ea7 still doesn't address the problem, because there's no clean way to fix or modify width and mass from the python interface.

That's true. Can you open an issue for this random initial parameter problem.

There's already this one, right? ComPWA/expertsystem#8

This is something else. Or did I misunderstand you here? You are talking about some function which randomly picks a starting point in the optimization space right?


Document the preprocess parameter of the directives in the README

As of 1.0.0, the preprocessors were replaced with filters and are documented in README :)


fix notion integration link

Dupplicate with https://github.com/makenotion/notion-sdk-js/pull/63

Thanks :)
Closing this as the PR already exists.


Slack importer: Fixes.

Handle case where messages have no user.
Refactor defaultstream handling and add tests for the same.

I'm not sure I understand either of these:

What are the messages without a sender coming from Slack?  (I.e. what do those message IDs look like in Slack?)
For defaultstream, can you explain what it's doing (ideally in a paragraph comment or in the commit message)?  It's not clear to me what policy this is intended to implement without having to read the code carefully.

@timabbott I have added the explanation for the defaultstream in the commit message. The index is incremented each time one of these is found, the index ends up being more than the length of the allocated list and as there are no checks for the function, it would crash.
For the messages with no user, the sample message can be seen here which was created by the slack polling app and hence the message has no "user".

Updated this PR with:
commit 3: Change 'invite_only' mapping in streams- 'invite_only' should always be true for the slack's
standard export plan as the private channels are not supported in it.
commit 4-7: For sorting of the messages in the order of date (As reported here: https://chat.zulip.org/#narrow/stream/2-general/topic/slack-import)
The problem being faced was that as the messages were being processed corresponding to the streams together( Process all messages in the stream and then move to another), the messages in the stream where ordered by date, but in 'All messages' the messages were not ordered by date.
solution approach: Precompute all messages in every stream by ordering them by date and then use this for the import.

I merged the first 3 commits; don't have time today to carefully read the group for message sort order.
@showell how familiar are you with the pre-existing code path for the message import?  If it's fast, feel free to review the rest.

I have rebased this PR!

The first three commits are just the refactoring of the function. We used to pass the channel_name in every message processing helper function and we got the data from opening only that particular channel file and reading the data.
Now the data of all the channels is being read at once, passed in a List of Dict, There it is sorted with respect to the date, and then the whole of the data is passed into the processing helper functions.

Rebased this PR again after the recent merge!

Merged, thanks @rheaparekh!  I have a few suggested follow-up cleanups to propose:

build_zerver_usermessage shouldn't need to return anything -- you're passing it a list, and it edits the list directly, so no need to return it.  I'd do a quick skim for similar issues elsewhere, since it'd make the codebase more readable to not have this.
Why is the REALM_ID argument name capitalized?  Seems like a quick thing to fix.
The allocate_ids function isn't being used in the right part of the process for this to work on a live system.  There's sorta 2 phases of ID allocation we need to do: (1) providing sequential IDs in the "slack to zerver_message" format conversion, which doesn't need to correspond to the target database (in theory, this could be done by a tool outside the main Zulip codebase some time before the actual import).  And then (2) at import time, when it's important to transactionally reserve a block of IDs for the user of the tool.  What this means is that the existing allocate_ids function could do something simpler that doesn't interact with the database (e.g. just return sequential numbers), if we did the second piece correctly: Our nice allocate_ids implementation would be better placed in zerver/lib/export.py and used probably in something lke bulk_import_model to allocate ID numbers in the target database during the import, and then re_map_foreign_keys function on the import side can take care of renumbering.

A way to test whether the ID allocation system is working is to double-import the same realm into a single
realm (i.e. do the export, and then import the resulting import file twice, maybe after changing a unique key like the subdomain).
Does that make sense?

@timabbott Thank you for the review! I'll get started on the followup! I have also added a comment here regarding the followup.


Vm Mac wont send videos

Hi. Sending messages with bluebubbles works perfectly, except for videos. Regardless of the fact I have private API features enabled, followed steps correctly and everything, videos won't send at all on my vm Mac. I reset and created a brand new vm multiple times to no avail. I have Ethernet connected to reduce Internet lag as well. I attached an image of the blue progress bar when sending videos, which eventually says not delivered. Please help , thank you !

Are you able to send the same attachment when sending directly through the Messages app, rather than BlueBubbles?

@Wusgoodindahood any update? Otherwise, I'm going to close this due to lack of information

It kinda looks like a ban to me potentially if its not working native

Closing, stale.


Bug Report

I'm trying to run macOS Monterey (my host system is running Pop!_OS 22.04 LTS, which is based on Ubuntu 22.04 LTS), and I get the following error message when I try to power on the virtual machine:

Error starting domain: operation failed: unable to find any master var store for loader: /home/lnik/Desktop/OneClick-macOS-Simple-KVM-master/firmware/OVMF_CODE.fd
Traceback (most recent call last):
File "/usr/share/virt-manager/virtManager/asyncjob.py", line 72, in cb_wrapper
callback(asyncjob, *args, **kwargs)
File "/usr/share/virt-manager/virtManager/asyncjob.py", line 108, in tmpcb
callback(*args, **kwargs)
File "/usr/share/virt-manager/virtManager/object/libvirtobject.py", line 57, in newfn
ret = fn(self, *args, **kwargs)
File "/usr/share/virt-manager/virtManager/object/domain.py", line 1384, in startup
self._backend.create()
File "/usr/lib/python3/dist-packages/libvirt.py", line 1353, in create
raise libvirtError('virDomainCreate() failed')
libvirt.libvirtError: operation failed: unable to find any master var store for loader: /home/lnik/Desktop/OneClick-macOS-Simple-KVM-master/firmware/OVMF_CODE.fd

I'm guessing did something wrong, but I can't figure out what... I followed the installation instructions word-for-word. I also made sure I have the newest iteration of OVMF_CODE.fd

@tiredsemanticist try downloading this into your firmware folder and tell me if it works. If it does not, you could also try running the VM as sudo to see what happens.

@notAperson535 I tried both and only got halfway to a solution. Had to rename that file to OVMF_VARS.fd to get virt-manager to acknowledge it, and then the machine powered on... Unfortunately, there's no installer option in OpenCore even though I made sure that all the correct hardware has been added to the virtual machine. Running as sudo makes no difference

1000% willing to believe that I'm doing something wrong, I just haven't been able to figure out what. 😅


Be less strict on already-received transactions

By default, i made the watchtower refuse that we send it signatures twice for the same transaction. However on the revaultd side tracking which WTs we already successfully shared the Emergency signature with is overkill.
Instead, we could just re-ACK it.
Anyways, the protocol needs to change.

Partially fixed by #17

Fixed by #20


update dxx theme proview url

为什么没有marge？

I merged your PR (the new URL works fine).
(PS: It's better to post in English to get a quick reply.)


[material-ui][SpeedDial] Fix SpeedDial tooltip placement

[X] I have followed (at least) the PR section of the contributing guide.

Fixes #41067
Closes #41414 (old inactive PR)
Before:

After:

I used the tooltip component instead of SpeedDialActionStaticTooltip. Then fixed the tooltip placement by changing the fab component style (the transform property changes tooltip placement).
Deploy preview: https://deploy-preview-41997--material-ui.netlify.app/material-ui/react-speed-dial/#persistent-action-tooltips.

Is there any updates in https://github.com/mui/material-ui/pull/41414? or the issue has been solved with https://github.com/mui/material-ui/pull/41414? I can see it's still open.

No new updates there. Feel free to address it in this PR.

@ZeeshanTamboli Thanks! I make the tooltip white.
Btw, in speedDial, all other tooltips are black. Is there a reason why only this should be white?

@ZeeshanTamboli Thanks!

I make the tooltip white.
Btw, in speedDial, all other tooltips are black. Is there a reason why only this should be white?

@ZeeshanTamboli Thanks!  I make the tooltip white.

Are you sure? I can't see it as white in the docs preview: Link to docs preview when hovering over it. It should match the white tooltip in production: Link to production.

Btw, in speedDial, all other tooltips are black. Is there a reason why only this should be white?

Not entirely sure, but for consistent user experience, persistent tooltips should remain white to avoid any unexpected changes for existing apps using them with the tooltipOpen prop.

@ZeeshanTamboli  It works now. Could you please check again?

Not entirely sure, but for consistent user experience, persistent tooltips should remain white to avoid any unexpected breaking changes for existing apps using them with the tooltipOpen prop.

I got it. Thank you for letting me know.

@ZeeshanTamboli It works now. Could you please check again?

Now it's white, but the size is smaller. You can see in the documentation preview I provided in the PR description. It should match the previous appearance exactly. Why not reuse the previous styles? This PR should only address the tooltip's positioning. Additionally, it should maintain the previous animation when opened.

@ZeeshanTamboli
All styles, except for the animation, match the previous version. Could you please check?
When scale animation(previous animation) maintain, the tooltip finds the anchored element and is placed before the anchored element is placed. So the tooltip can't be placed in the correct position. Because of this issue, I remove the animation.
I tried to find a way to maintain it, but couldn't find it. Do you have any other suggestions or ways for this?

@ZeeshanTamboli sorry, my bad. I removed them all.

@ZeeshanTamboli Hi! I'm trying to fix the Argos test. But I don't know why there's a change.
Please give me some time.

Since this PR is inactive, I am closing it.


No such file or directory

When I run mysterium, i get this message:

But when I run this program with python, it absolutely works.
NOTE: abi/standart.json exists.

You're program is trying to use a file that doesn't exist, just create this file in the written directory './abi/standard.json'.

It already exists

You're program is trying to use a file that doesn't exist, just create this file in the written directory './abi/standard.json'.

It already exists

I was getting an issue where I would supply the uninspected file (zip) and then specify the obfuscated file name (script.py) and it would come back that it couldn't find modules\script.py.  I have no idea why it was searching for modules as the dir structure was dist/script.py.

After looking at the code, looks Mysterium extracts the zip to it's modules folder and then looks for the file in question.
So might make sense to also look for a dist directory before failing out, so 'file' or 'dist/file'.
After testing, it gets further along but fails to load the PyArmor with RuntimeError: Marshal loads failed

After looking at the code, looks Mysterium extracts the zip to it's modules folder and then looks for the file in question. So might make sense to also look for a dist directory before failing out, so 'file' or 'dist/file'.
After testing, it gets further along but fails to load the PyArmor with RuntimeError: Marshal loads failed

Sorry for responding so late. Anyways, the problem is the same as Bad magic number or wrong opcode etc....
That means you're trying to run the decompiled program with a different Python version than the one used by its creator.
If you want it to work, you have two solutions. The first one needs the source executable, you have to decompile it with the tool you use, for example pyinstxtractor for pyinstaller, (pyarmor compilation uses pyinstaller), and so you'll see the Python version used to compile it by the script owner in the decompilation logs in your console. The second option is to use my tool I sell on the Telegram account in Mysterium's documentation, at the end. The tool doesn't need the exe if you don't have it, but it still works with it. There are methods it uses to detect which Python version is the right one by reading some details in the encrypted code file i hope my response was able to help you, I let the issue open some weeks if you still have to get any help.

I see that there are no more activity here so I'm going to close the issue, I guess you resolved the problem with your missing file.


The output is so strange!!

Can you upload the result: mask.tiff?
Because I have spent lots of time running your code and got a strange output mask_dmt,  exists many spikes.
I have put the obtained result on the google drive, the link is as follows:
https://drive.google.com/file/d/1HBN6zekfBHKSeNSPf7UflBcT7ULLjUTy/view?usp=sharing
Could you please check it and tell me whether this is correct? If you can provide more samples on 3D data (ISBI13) as you claimed in the paper, I would be very grateful.

Sorry for the confusion. For 3D volume data, you'll need to use the persistence_ws version to compute the approximate morse structures. In this repo, the output of persistence_ws is saved as cc.mat (connected component matrix). Note that the connected component matrix and binary boundary matrix are convertible.

Sorry for the confusion. For 3D volume data, you'll need to use the persistence_ws version to compute the approximate morse structures. In this repo, the output of persistence_ws is saved as cc.mat (connected component matrix). Note that the connected component matrix and binary boundary matrix are convertible.

Thanks for your reply! However, I have ran the 2D scripts and got a relative strange output, I havenot yet conducted the experiments on the 3D data?  Could you kindly provide some concrete 2D and 3D examples for me to have a try?


Unable to create plugin with default controller and named controller

Internal Server Error
RecursiveDirectoryIterator::__construct(/home/forge/pluginfactory.io/craft/storage/runtime/temp/pluginfactory/asdads): failed to open dir: No such file or directory
Steps to reproduce:

Enable controller radio box (or I suspect anything)
Give handle name
Add blank (for default)

submit

This is due to having an underscore in the plugin name/handle: https://github.com/nystudio107/generator-craftplugin/issues/75


Fix lost mutation bug in transforms on nested scopes.

When we removed cyclic_refs in #1696, we accidentally introduced a
correctness bug concerning mutations under transforms of multiple
nested scopes with complicated aliasing.  This is a fast fix that
also includes a test case for the problematic situation.

Codecov Report

Merging #1716 (269c78c) into main (c447942) will increase coverage by 0.01%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main    #1716      +/-   ##
==========================================
+ Coverage   83.21%   83.22%   +0.01%
==========================================
  Files          69       69
  Lines        5868     5872       +4
==========================================
+ Hits         4883     4887       +4
  Misses        985      985

Impacted Files
Coverage Δ

flax/core/scope.py
93.29% <100.00%> (+0.07%)
:arrow_up:

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 c447942...269c78c. Read the comment docs.


feat(rest): add more keymap and lualine

Description

add more keymap to kulala:

search requests
set selected env

add kulala lualine

Related Issue(s)
Screenshots

New keymaps and lualine:

Checklist

[x] I've read the CONTRIBUTING guidelines.

Currently all kulala shortcuts are only active if a http-file is already open. Is this intended behaviour?
I could imagine, that atleast the scratchpad command should be available even if you are not in a http-file.
My suggestion would be to use:
{ "<leader>R", desc = "+Rest"},
{ "<leader>Rb", "<cmd>lua require('kulala').scratchpad()<cr>", desc = "Open scratchpad"},

And maybe somebody has a great idea for a icon in the which-key popup for the REST-section


Enabling copy_sync breaks i_CTRL-R_CTRL-P "

As the title says, having:
copy_sync = {enable = true}

breaks pasting from the " register with indentation fixed (help i_CTRL-R_CTRL-P). Using Ctrl-R " works fine.

Ty for the report. How exactly does it break?

It inputs " where the cursor is at. It does not paste from the register.

Heho,
sry for the late response. Like i wrote in the readme, i no longer dev the plugin, but still maintaining it.
Feel free to pr a fix.
Kind regards
Alexander


Semver dependency?

With paket pack and paket.template, and using ~> #{version}, https://www.nuget.org/packages/Suave/2.3.0-beta depends on wrong versions; too constrained. I'd like to do a semver dependency. Is there a squiggly for that? (Besides (3, 4.0])?

@haf is it not working as documented in https://fsprojects.github.io/Paket/nuget-dependencies.html#Pessimistic-version-constraint?

It is. But that's not a semver dependency. I'm just asking if that wouldn't be a nice feature?

I always thought '~> 1.0' is exactly that?

But you probably want to cap the version below to the currently built-with version but the upper end should be major+1 exclusive. I'm talking about generating nugets with paket now.

As far as I know at the moment you can achieve what you want' by explicitly defining the range. For example:
nuget DotLiquid >= 2.0.174 < 3.0 will result dependency x >= 2.0.174 && x < 3.0
So what you are asking for is a shortcut/syntax for creating dependencies from a given version to the next major? Something similar to the current "twiddle-waka" which will yield a "Pessimistic version constraint", but so that it'll always cap the upper boundary to the next major, even if minimum would be a patch version.
So for example a new operand ~>> ("twiddle-waka-waka" is that then ;)) that would do just that. Broken into examples below (I'll show the difference to related "twiddle-waka" results also):
nuget DotLiquid ~>> 2.0.174     | results x >= 2.0.174 && x < 3.0
nuget DotLiquid ~>  2.0.174     | results x >= 2.0.174 && x < 2.1
nuget DotLiquid ~>> 2.0         | results x >= 2.0 && x < 3.0
nuget DotLiquid ~>  2.0         | results x >= 2.0 && x < 3.0 (same result with both operands)

I really like the "twiddle-waka", but I have found myself longing for this other shortcut as well. As most of the times what I actually want is exactly what described here (I want to have at least a specific patch and am happy to upgrade to anything until the next major as the given nuget has promised to follow SemVer).

This could probably be labeled as "enhancement"? Or should I make a distinct issue requesting the proposed new operator?

Yes; @Gonnagle that's a good idea, but I think this issue already requests that operator, don't you?


context.read returns state instead of the StateNotifier class

Describe the bug
I want to call the function of change notifier class on button click, but context.read is returning state.
To Reproduce
Here's the provider and StateNotifier class
final onboardingViewModelProvider =
    StateNotifierProvider<OnboardingViewModel, bool>((ref) {
  return OnboardingViewModel();
});

class OnboardingViewModel extends StateNotifier<bool> {
  OnboardingViewModel() : super(false);

  Future<void> completeOnboarding() async {
    state = true;
  }

  bool get isOnboardingComplete => state;
}

Another function from a tap
Future<void> onGetStarted(BuildContext context) async {
    final onboardingViewModel = context.read(onboardingViewModelProvider);
    await onboardingViewModel.completeOnboarding(); //I'm getting issue here, here onboardingViewModel is the state value which is bool
  }

await onboardingViewModel.completeOnboarding();
I'm getting issue here, here onboardingViewModel is the state value which is bool, but it shouldn't be
Expected behavior
A clear and concise description of what you expected to happen.

With 0.14.0, the syntax changed
See https://riverpod.dev/docs/migration/0.13.0_to_0.14.0

thanks


how to clear "No Message Yet" chat

We are using connectycube React and React-native chat system with our application. On that when we open a new chat (dialog) from our contact list and leave that without sending any message, then its showing "No message yet" on our chat history page (refer the attached screenshot). Can we have any settings option to remove that empty chat from our chat history. Can you please help on this.

you can try to filter here https://github.com/ConnectyCube/connectycube-reactnative-samples/blob/master/RNChat/src/screens/main/dialogs/index.js#L122
e.g. create a Redux selector or filter all the empty dialogs

A new chat sample with new push notification lib is created
Please follow this MR for any issues https://github.com/ConnectyCube/connectycube-reactnative-samples/pull/296


exclude cannot handle directory

The exclude files cannot hold the whole dictionary. The regex pattern checks only for the name of the file so in a case that you have 2 similar file names but in different folders then you have to exclude both.
Replace bug:
In my case I'm using poetry so the config file looks like this:
pyproject.toml
[tool.docsig]
exclude = '''.*src\\design\\.*'''

The design folder should be excluded. But it is not. This line makes nothing.
Expected:
The whole folder is not in checks.
Possible solution:
right now in _files.py::Paths::_populate there is a regex match _re.match(i, root.name) and to proper work it should be a _re.match(i, str(name)) This will allows to check whole folders instead of only one file. It is really important if you have the same names in different folders and only one should be excluded.
Environment:

Python version: 3.11.4
docsig version: 0.59.1

Hey @LisPrzemyslaw
Thanks for opening this and making the logic behind this flawed regex handling known
Are you able to check this branch and let me know how the fix looks on your end?
Cheers,
Stephen

@LisPrzemyslaw , I'm pretty satisfied this is fixed now, I tried on mac and my windows vm
I'll make this available now as v0.59.2
Any issues please don't hesitate to let me know

I've tested it and it is working.
@jshwi thanks a lot! Good job! 🚀

Awesome, no worries at all @LisPrzemyslaw ! Thanks for confirming 😄


Enhance the "Intermediate > GitOps with Weave Flux" module

The "GitOps with WEAVE Flux" section of eksworkshop is not comprehensive. It does not address how ECR is polled for image update, how to monitor the gitops pipeliline and process the gitops events. Additionally it does not use CodeCommit.
Client and partners are looking for detailed step by step solution for  implement of GitOps with Flux, EKS, ECR, CodeCommt and  GitOps event processing with EventBridge.

Solution kind of like this -


Min/Max tick step value

Feature Proposal
A min step value so that ticks can still be dynamic but can also be confined to a min or max
Feature Use Case
Using seconds/minutes/hours to demonstrate duration's of run-times and having decimal step values is not logical.  Would like to increment by halves at minimum, otherwise whole seconds/minutes/hours.

what version are you using?
what chart type are you using?
what scales are you using?

time scale (that would be my assumption for seconds/minutes/hours) does not display decimal steps.

I'm also wondering what your data looks like. E.g. if you have one data point per second/minute/hour/etc. you may want to consider using the 'series' mode

2.8.0
line
I tried using time for my y axis but it came out funky at first

My data model might look like this:
{ RunTimeKey: "6/9/2019 5:20:28PM", DurationTime: 6000 },
{ RunTimeKey: "7/9/2019 5:20:28PM", DurationTime: 1140 },
{ RunTimeKey: "8/9/2019 5:20:28PM", DurationTime: 180 },
{ RunTimeKey: "9/9/2019 5:20:28PM", DurationTime: 1020 },
{ RunTimeKey: "10/9/2019 5:20:28PM", DurationTime: 360 }
where duration time is in seconds.
From my understanding, duration time could be anywhere from seconds to months (yes in seconds), but I'm not yet sure if I want to display weeks/months in the chart.

The charts expect x/y input. Do you replace RunTimeKey with x and DurationTime with y before passing the data into the chart?

No I did not

Ah, ok. But you split them into separate arrays then it looks like
The time scale is meant to show an actual date and time and won't work for a duration. The linear scale just handles general numbers and doesn't have any concept of how many seconds, minutes, etc. are in a time period.
You can customize the linear scale to fit your needs. E.g. you can use afterBuildTicks to divide by 60 and then round the result or whatever it is you'd like to do. We probably won't be building in support for a notion of durations into the core library


OHRI-1226 patient list widget enhancements

This has the first version of the newly improved patient list widget with a data table.

Pending work includes fixing some styles to match designs

@samuelmale i've made the changes to the 2 comments. This widget is still work in progress so some other features like pagination will follow with some other UI tweaks.

This widget is still work in progress so some other features like pagination will follow with some other UI tweaks.

Do we have a ticket to keep track of the above?

Let me create them


Toggle checkbox no longer has box-shadow

Starting at 2.2.5 the Checkbox "toggle" style no longer includes a box-shadow by default, which (according to our users) makes it hard to see and when it is seen, appear disabled. It's also now inconsistent with the "slider" checkbox, which still has the drop shadow.
2.2.4

2.2.5

https://jsbin.com/wiretu/edit?html,output
I'm guessing this might have been accidental, instead intending to just allow it to be configurable? I tried to find the true, original PR but seems it was part of some sort of complicated fork merge.
https://github.com/Semantic-Org/Semantic-UI/compare/2.2.4...2.2.5#diff-46cda3168af13ade8aea3ab4a1edd16cR522
You can get the old style back by using:
/* your-semantic/site/modules/checkbox.variables */
@toggleOffHandleBoxShadow: @handleBoxShadow;
@toggleOnHandleBoxShadow: @handleBoxShadow;

#5096 appears to fix this 👍


[Feature Request]: Design pages

Is there an existing issue for this?

[X] I have searched the existing issues

Feature Description

Add pages for the above links
Use Case
In the footer
Benefits
Users will be aware regarding the terms and conditions and privacy policy
Add ScreenShots
No response
Priority
High
Kindly assign me this issue
Record

[X] I have read the Contributing Guidelines
[X] I'm a GSSOC'24 contributor
[X] I have starred the repository

Design one page that has all 3: privacy policies, terms and conditions, and cookie policy. Don't make separate pages for each.
We aren't using any cookies for authentication so make sure to specify that.

Resolved in #188


Unable to reference variable

Hello,
I was trying to use xarray to access the MERRA-2 monthly dataset. This dataset provides nc4 files, one for each month, with multiple analyzed variables (U,V,T, etc.,) in each file.
if I open one file and try to access the zonal wind (U) as
       data = xarray.open_dataset('/data/MERRA2/MERRA2_100.instM_3d_ana_Np.198001.nc4')
       data = data.U

This works just fine.
However, if I use
data = data.T

I get back the entire dataset, i.e, I then have data.T.T, and data.T.T.T etc.,
I can open a single file using netCDF4, and the values seem ok.
This seems to happen only with the temperature (T) field. all other fields seem to
behave ok.
Any ideas as to what is happening?
TIA,
Joy

xarray uses the .T attribute as an abbreviation for .transpose() (like numpy). You'll need to use data['T'] to access the temperature variable.

Yes, that makes sense. quite the corner case! works perfectly now, thanks again.


EpochIterator, TensorFlowTrainer : refactoring and fixes

Fixes https://github.com/keras-team/keras/issues/20394
Fixes https://github.com/keras-team/keras/issues/20390
Fixes https://github.com/keras-team/keras/issues/20344
TensorFlowTrainer :

Added usage of iterator.get_next_as_optional(), which enables steps_per_execution to exceed the previous limit of 32 while preventing tf.errors.OutOfRangeError

EpochIterator :

Refactored to unify code base across different backends
Fixed edge cases involving various combinations of steps_per_epoch and steps_per_execution

It's fairly confusing to have backend-specific iterators subclass _EpochIterator -- much better to have a single EpochIterator class and have backend-specific classes override what they need to change.

It's fairly confusing to have backend-specific iterators subclass _EpochIterator -- much better to have a single EpochIterator class and have backend-specific classes override what they need to change.

I removed _EpochIterator.


Change checking default value for Source parameter

When $Source parameter is not passed then this parameter has "" value. Condition if ($Source -ne $null) is working incorrectly and adding "/Source:" and "/LimitAccess". Windows feature isn't install when dism.exe is running with parameter "/Source:"
My change solve this problem.

This change is

Hi @kreciol, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. Real humans will now evaluate your PR.

TTYL, MSBOT;

@kreciol Would you add a test to verify this does not regress?

Yes of course, but xDismFeature has no unit test and need to build infrastructure for that. I don't have time for this.

@kwirkykat Can we take this without tests?

@TravisEz13 @kreciol Yes. LGTM


Blurry texture bug (all mobs except angry wither cat)

The textures appear blurry as if they've been upscaled badly in photoshop.

Here is the fml-client-latest.log
I'm using:
forge <IP_ADDRESS>9
Ender Zoo <IP_ADDRESS>
ModList:
Actually Additions
Akashic Tome
AppleCore
Auto Packager
AutoRegLib
Baubles
BD Lib
Better Builders Wand
Better Combat Mod
Better FPS
Biomes O Plenty
Blood Magic
Botania
Chameleon
Chisel
Chisel and Bits
Cooking for Blockheads
Cyclic
Deep Resonance
Drawers and bits
ElecCore
Ender Core
Ender IO
Ender Zoo
Enviromental Tech
Extra Utilities
Farseek API
Fast leaf decay
Flat Colored Blocks
Forestry
FTB Lib
FTB Utilities
Gendustry
Guide API
Immersive Engineering
Industrialcraft
Inventory Sorter
Ironchest
JEI
JEI Bees
Journeymap
Just Enough Resources
Light Level Overlay
Mantle
mcjtylib
MCMultiPart
Mod Name Tooltip
MorphOTool
Mouse Tweaks
Not Enough Wands
OpenComputers
Pams HarvestCraft
Platforms
Player Rugs
Progressive Automation
PSI
Quantum Flux
Quantum Storage
Quark
Random Things
Reborn Core
Redstone Paste
Refined Storage
RFTools
RFTools Dimensions
Roots
ShetiPhian Core
Simple Generators
Sound Filters
Storage Drawers
Streams
Super Crafting Frame
TechReborn
The One Probe
Tinkers Construct
Tinkers Tool Leveling
Tinkers' Addon
Tomb Many Graves
ValkyrieLib
Weapon Case Loot
Zoology

I have this problem as well. It seems to break other mobs from other mods' textures too!


Avataaars only support circle backgrounds. Support for square?

I love what the smaller-than-head circles look like, but I can't support them with my current app. It would be great if I could declare a background on a standard background with the avataaars set. Any chance of that? :)

Thanks for your bug report. Fixed with v4.5.2


Enable linting of TypeScript files

Linting used to work in VS2013 with WebEssentials and a TSLint.json file. However, in 2015 RC0 it doesn't seem to be working. I need to figure out why it doesn't work any more and re-enable support for linting.

I checked on this again today in VS2015 RC with the latest WebEssentials plug-in. It still isn't respecting the tslint.json in the project root even after adding WebEssentials solution settings.

Hello,
You may want to check this issue.

Thanks for the info!


Update ingress to networking.k8s/v1

Closes #18
Tested on digital ocean kubernetes

looks good! thanks for the PR


CDPCP-3974. Update gradle dependencies

…CB modules
details:

CB-11613. Upgrade package netty-handler to version 4.1.45 or above. (4.1.63.Final)
CB-11610. Upgrade package netty-codec-http to version 4.1.44.Final or above. (4.1.63.Final)
CB-11618. Upgrade package netty-codec-http to version 4.1.44.Final or above. (4.1.63.Final)
CB-11609. Upgrade package netty-handler to version 4.1.44 or above. (4.1.63.Final)
CB-11605.  Upgrade package postgresql to version 42.2.13 or above. (42.2.13)
CB-11608. Upgrade package jackson-datatype-jsr310 to version <IP_ADDRESS> or above. (2.11.4)
CB-11600. Upgrade package jackson-databind to version <IP_ADDRESS> or above (2.11.4)
CB-11614. Upgrade package netty-codec to version 4.1.46.Final or above. (4.1.63.Final)
CB-11595. Upgrade package netty-handler to version 4.1.46 or above. (4.1.63.Final)
CB-11597. Upgrade package netty-codec-http to version 4.1.42.Final or above. (4.1.63.Final)
CB-11611. Upgrade package handlebars to version 4.3.0 or above. (js version: 4.7.6, jar version: 4.2.0)
CB-11607. Upgrade package handlebars to version 4.4.5 or above. (js version: 4.7.6, jar version: 4.2.0)
CB-11612. Upgrade package handlebars to version 4.5.3 or above. (js version: 4.7.6, jar version: 4.2.0)
CB-11615. Upgrade package jackson-dataformat-cbor to version 2.11.4 or above. (2.11.4)
CB-11601. Upgrade package xstream to version 1.4.15 or above. (1.4.15)
CB-11602. Upgrade package xstream to version 1.4.14-jdk7 or above. (1.4.15)

note: swagger core version needs to be updated because of the jackson update (as the older version using deprecated jackson methods that are not exist in 2.11)

just updated PR by removing jackson upgrade because of an error (until i can make sure that is not caused by this change) that occurred once from 3 runs in schema-compatibility-test job (a "can't convert object" problem)


Getting more done in GitHub with ZenHub

Hola! @Cervator has created a ZenHub account for the MovingBlocks organization. ZenHub is the only project management tool integrated natively in GitHub – created specifically for fast-moving, software-driven teams.

How do I use ZenHub?
To get set up with ZenHub, all you have to do is download the browser extension and log in with your GitHub account. Once you do, you’ll get access to ZenHub’s complete feature-set immediately.
What can ZenHub do?
ZenHub adds a series of enhancements directly inside the GitHub UI:

Real-time, customizable task boards for GitHub issues;
Multi-Repository burndown charts, estimates, and velocity tracking based on GitHub Milestones;
Personal to-do lists and task prioritization;
Time-saving shortcuts – like a quick repo switcher, a “Move issue” button, and much more.

Add ZenHub to GitHub
Still curious? See more ZenHub features or read user reviews. This issue was written by your friendly ZenHub bot, posted by request from @Cervator.

 Was wondering how this issue creation thing would actually make a team though. Have you spotted how that all works? If you've got a working personal setup for JS how would you go about making it visible for another user?
Probably it is in the docs somewhere, I was just wanting something to play with for a few minutes before bedtime. Will look at it more tomorrow :-)

Yes, it works for me.

While browsing GitHub stuff with the addon installed, I've noticed that DestinationSol's great convergence PR seems to have an estimate of "40".

Don't see any logs related to this (i.e. User X has estimated this at 40), so it might have been automatically generated somehow?

I have a vague recollection that I tested ZenHub once in the past and probably did the 40 estimate way back then as it was the highest value I saw :D
And yeah since ZenHub is a browser addon that wraps existing GitHub I figure they take a few shortcuts like literally wrapping some of the page elements (probably harder to insert timestamped status message into the main stream of items)

The estimate changes are actually logged now - see below for an example!

Might just be a feature added sometime after the point you were testing it, I suppose  ¯\_(ツ)_/¯

Looking good. Been playing with it a bit in Nanoware/Terasology.
I turned https://github.com/Nanoware/Terasology/issues/31 into an Epic and on the board there "merged" in MovingBlocks/Gooey. Then I could add an issue from the Gooey repo to the epic on Nanoware/Terasology.
I could even make a separate Epic in the Gooey repo, add issues to it there, then add that epic to the other epic. So dependencies and what have you. Should be useful!

After this was initially experimented with GitHub released projects, and IMHO ZenHub got a bit laggier and crowded ... am leaning toward trying to stick with the native UI (worked decently for GSOC 2017). Still considering available options and am always up for discussing needs vs what's available, but figure this issue is probably OK to close at this point.

I agree. Projects seem to duplicate pretty much the whole ZenHub functionality.


fix: register waker before loading/checking counter value to avoid a race condition that would result in lost notification

Although fetch_sub returns previous value, expression 'prev_val -n' evaluates to same value what ever is stored in counter after fetch_sub operation. So done_n is waking up the future twice (for  'prev_val - n' = 1 and 0) and  hence causing notification miss sometimes.
case: Suppose we have spawned 'x' number of tasks and gave one 'work' (tg.add_work(1)) to each. lets say 'x-2'  are already dropped. Now consider done_n wakes up the future when prev_val - n == 1, so executor invokes the poll method, after loading the counter value and before registering the waker in poll, done_n again wakes up the future when prev_val -n == 0 causes the notification miss (waker was not registered yet), poll will continue the execution, register waker and return pending. tg.wait().await will go in indefinite waiting state.
Note: registering waker in else block and keeping prev_val - n <= 0 in done_n method can cause same problem sometimes,very rare but could happen.

@ab-basit Thanks for the fix! Looks ok to me

@staticgc  Looks like you have closed the PR instead of merging  and updated version only. Please check once before publishing
Thanks

@staticgc Looks like you have closed the PR instead of merging and updated version only. Please check once before publishing Thanks

Oops! Had a brain fade


Update SourceLink to latest version.

The latest version of SourceLink is required to build the solution on Mac os-x Mojave.

Thank you!


Failed to connect: Connection refused

I followed the instructions here
Started the containers with docker-compose up -d then got the logs via docker-compose logs -f
GraphQL is not accessible:
curl -X POST -H "Content-Type: application/json" -d '{"query": "{ cardano { blockHeight }}"}' http://localhost:3100/graphql

curl: (7) Failed to connect to localhost port 3100: Connection refused

Logs
The node is up for the last 20hours and synced, there's no visible error in the logs

Containers
GraphQL exited with code 0

Processes

Error from cardano-graphQl
mask@mask-desktop:~/git/cardano-graphql$ sudo docker-compose logs | grep graph
Attaching to cardano-graphql_cardano-db-sync-extended_1, cardano-graphql_hasura_1, cardano-graphql_postgres_1, cardano-graphql_cardano-graphql_1, cardano-graphql_cardano-node_1
cardano-graphql_1           | Hasura schema introspection: Attempt 1 of 3, retying...
cardano-graphql_1           | Hasura schema introspection: Attempt 2 of 3, retying...
cardano-graphql_1           | Hasura schema introspection: Attempt 3 of 3, retying...
cardano-graphql_1           | Hasura schema introspection: Attempt 4 of 3, retying...
cardano-graphql_1           | { FetchError: request to http://hasura:8080/v1/graphql failed, reason: connect ECONNREFUSED <IP_ADDRESS>:8080
cardano-graphql_1           |     at ClientRequest.<anonymous> (/application/node_modules/node-fetch/lib/index.js:1455:11)
cardano-graphql_1           |     at ClientRequest.emit (events.js:189:13)
cardano-graphql_1           |     at Socket.socketErrorListener (_http_client.js:392:9)
hasura_1                    | {"timestamp":"2020-05-16T21:08:51.000+0000","level":"info","type":"startup","detail":{"kind":"migration-apply","info":"starting graphql engine temporarily on port 9691"}}
cardano-graphql_1           |     at Socket.emit (events.js:189:13)
cardano-graphql_1           |     at emitErrorNT (internal/streams/destroy.js:82:8)
cardano-graphql_1           |     at emitErrorAndCloseNT (internal/streams/destroy.js:50:3)
cardano-graphql_1           |     at process._tickCallback (internal/process/next_tick.js:63:19)
hasura_1                    | {"timestamp":"2020-05-16T21:09:22.000+0000","level":"info","type":"startup","detail":{"kind":"migration-apply","info":"starting graphql engine temporarily on port 9691"}}
cardano-graphql_1           |   message:
cardano-graphql_1           |    'request to http://hasura:8080/v1/graphql failed, reason: connect ECONNREFUSED <IP_ADDRESS>:8080',
cardano-graphql_1           |   type: 'system',
cardano-graphql_1           |   errno: 'ECONNREFUSED',
cardano-graphql_1           |   code: 'ECONNREFUSED',
cardano-graphql_1           |   attemptNumber: 4,
cardano-graphql_1           |   retriesLeft: 0 }
hasura_1                    | {"level":"info","msg":"Help us improve Hasura! The cli collects anonymized usage stats which\nallow us to keep improving Hasura at warp speed. To opt-out or read more,\nvisit https://docs.hasura.io/1.0/graphql/manual/guides/telemetry.html\n","time":"2020-05-16T21:09:23Z"}
hasura_1                    | {"type":"startup","timestamp":"2020-05-16T21:09:22.529+0000","level":"info","detail":{"kind":"telemetry","info":"Help us improve Hasura! The graphql-engine server collects anonymized usage stats which allows us to keep improving Hasura at warp speed. To read more or opt-out, visit https://docs.hasura.io/1.0/graphql/manual/guides/telemetry.html"}}
hasura_1                    | {"timestamp":"2020-05-16T21:09:25.000+0000","level":"info","type":"startup","detail":{"kind":"migration-apply","info":"graphql-engine will now start in normal mode"}}
hasura_1                    | {"type":"startup","timestamp":"2020-05-16T21:09:26.242+0000","level":"info","detail":{"kind":"server_configuration","info":{"live_query_options":{"batch_size":100,"refetch_delay":1},"transaction_isolation":"ISOLATION LEVEL READ COMMITTED","plan_cache_options":{"plan_cache_size":null},"enabled_log_types":["http-log","websocket-log","startup","webhook-log","query-log"],"server_host":"HostAny","enable_allowlist":false,"log_level":"info","auth_hook_mode":null,"use_prepared_statements":true,"unauth_role":null,"stringify_numeric_types":false,"enabled_apis":["metadata","graphql","config","pgdump"],"enable_telemetry":true,"enable_console":true,"auth_hook":null,"jwt_secret":null,"cors_config":{"allowed_origins":"*","disabled":false,"ws_read_cookie":null},"console_assets_dir":null,"admin_secret_set":false,"port":8080}}}
hasura_1                    | {"type":"startup","timestamp":"2020-05-16T21:09:26.242+0000","level":"info","detail":{"kind":"telemetry","info":"Help us improve Hasura! The graphql-engine server collects anonymized usage stats which allows us to keep improving Hasura at warp speed. To read more or opt-out, visit https://docs.hasura.io/1.0/graphql/manual/guides/telemetry.html"}}

The issue was that nginx on the machine was using port 8080 and since hasura was configured to use the same port graphql engine failed to start. Working as expected after stopping nginx and making port 8080 available.

Thanks for the detailed report @PegasusPool. The failure you experienced here was not related to a host port clash, as that would be apparent when starting the stack. In the boilerplate compose file, Hasura is actually mapped to host port 8090
The cardano-graphql server currently only boots once Hasura is completely provisioned, which during the initial sync can be an extended period of time. It's something we'll be improving soon, but I've just added additional info to the wiki that most likely would have worked for you in this situation.

You are right @rhyslbw
Thank you for explaining and adding the extra bit on the wiki. It will definitely help others before it's improved.


Add null checks for setAxisModel

The values in the valueWindow array could be null. When the toFixed function is invoked, that will cause a TypeError.
https://github.com/apache/incubator-echarts/blob/5cce1e77ee8096e32ca30654385c45d22335affd/src/component/dataZoom/AxisProxy.js#L204
https://github.com/apache/incubator-echarts/blob/5cce1e77ee8096e32ca30654385c45d22335affd/src/component/dataZoom/AxisProxy.js#L484-L485

This solves crashes we see an high frequency. These crashed basically block us from rolling out echarts. 👍
@pissang @Ovilia @deqingli may I ask you to consider merging this fix? Also, a release containing the fix would be highly appreciated
Also, a guy from our team (hi Akash) found this to reproduce the problem:
option = {
    xAxis: {
        type: 'category',
        boundaryGap: false,

    },
    yAxis: {
        type: 'value'
    },
    series: [{
        data: [['Tue',820]],
        type: 'line',
        areaStyle: {}
    }],
    dataZoom:[{
        type:'slider',
        startValue:'mon'
    }]
};

While this is "invalid" in the context here, it shouldn't crash. Also we see this with "time" axis, but there the reproducer isn't clear.

Can we merge this?

@dr-itz Thanks for your comment. We will review it soon.

@dr-itz which version of echarts are you using?
I can not reproduce the problem by the example you provided above:
option = {
    xAxis: {
        type: 'category',
        boundaryGap: false,

    },
    yAxis: {
        type: 'value'
    },
    series: [{
        data: [['Tue',820]],
        type: 'line',
        areaStyle: {}
    }],
    dataZoom:[{
        type:'slider',
        startValue:'mon'
    }]
};

Do you have some guide that how to reproduce it ?


Fix ecreate bug

LGTM! so i wld assume rn if there are no role quantity indicated during ecreate, the error message will show in the result display?

yesh


Change visibility and state_change of lock codes events to match DTHs

@varzac ah, guess so. I thought you had already done this with your other fixes. My mistake.


Make code suggestions popup menu wider

Godot version:
81c6a7e5f
OS/device including version:
Ubuntu 19
Issue description:
Code suggestions window is too narrow - this creates issues for instance when suggesting long node path. Consider following situation:

As you can see node paths are cut and there is plenty of space to scale this window horizontally, I think that it should be auto-scaled to take up to the length of code editor. If this is not ok then I think that we should somehow inform user that not whole entry is visible (for instance add (...) at the end). Ideally both options would be implemented.
Steps to reproduce:

Make any scene structure with long node names or many nodes
Ask for context help after $

ps. Hard to say for me if it is a feature or a bug, looks a bit more than a bug to me

Duplicate of #16542.

sorry!


Ctrl+Backspace doesn't delete full words in ollama run ... mode

What is the issue?
In ollama run <model> mode, pressing Ctrl+Backspace should delete a whole word backwards.
To implement this, "listen" for "Ctrl+H" presses (as that's what Ctrl+Backspace is sent as)
OS
Linux
GPU
No response
CPU
No response
Ollama version
0.1.38

Hey @DeflateAwning, Ctrl + Backspace (Ctrl + H) maps to just Backspace in Ollama. If you want to delete the previous word, please use Ctrl + W which should already implemented. I've added documentation of the shortcut in /? shortcuts

Makes sense, thanks! Any reason not to have both act as a backspace?

Sorry, can you clarify which two you're referring to? Currently Ctrl + W removes the previous word and Ctrl + H is a backspace. As you said, Ctrl + Backspace is received as Ctrl + H so Ctrl + Backspace is simply a backspace as well.

Ctrl+H gets sent when you press Ctrl+Backspace in most terminals (at least in my experience) though. Ctrl+Backspace (which is sent as Ctrl+H) should remove an entire word.

Thanks for the feedback. Ctrl + W is generally used in terminal based applications for this shortcut so we've decided to use it as well.

I'm sorta torn here because generally speaking Ctrl-H is backspace. Ctrl-Backspace will do a Ctrl-W in most desktop programs, but it doesn't do that in my terminal and just maps to Ctrl-H by default.

Hmm, have never seen that behaviour, but have also not really done any low-level key handling outside of terminals and terminal UIs.


plot_histogram fails to operate correctly when numerous starting and ending values are zero

Environment

Qiskit version:  1.0.1
Python version:  3.12.2
Operating system: Red Hat Enterprise Linux release 8.7 (Ootpa)

What is happening?
In the code below, I am trying to plot the probabilities of a particle being at positions given by the dictionary indices. plot_histogram isn't working the way it should in case of the first dictionary d while it works correctly in case of the second dictionary d1. These values are generated by a quantum circuit. In the first case the plot is empty while in the second case, the plot shows a histogram. Also, I get the following warning on the first call to plot_histogram.
/nfs/home/ssharm19/.conda/envs/research/lib/python3.12/site-packages/qiskit/visualization/counts_visualization.py:389: UserWarning: Attempting to set identical low and high ylims makes transformation singular; automatically expanding.
  ax.set_ylim([min_ylim, min([1.1 * sum(all_vals), max(1.1 * val for val in all_vals)])])

How can we reproduce the issue?
from qiskit.visualization import plot_histogram
import matplotlib.pyplot as plt
import numpy as np

d = {0: 0, 1: 0, 2: 0, 3: 0, 4: 0, 5: 0, 6: 0, 7: 0, 8: 0.00324, 9: 0.03881, 10: 0.17425, 11: 0.28411, 12: 0.28588, 13: 0.17294, 14: 0.03769, 15: 0.00308, 16: 0, 17: 0, 18: 0, 19: 0, 20: 0, 21: 0, 22: 0, 23: 0}
assert sum(d.values()) == 1, "Probabilities aren't normalized"
d1 = {k -8: v for k,v in d.items() if v !=0 }
assert sum(d1.values()) == 1, "Probabilities aren't normalized"
plot_histogram(d)
plot_histogram(d1)
plt.show()

What should happen?
I should be able to see histograms in both cases.
Any suggestions?
No response

Hi, I am part of the IBM OpenSource Coalition. This issue looks interesting to me and I would like implement it


remove --onnx-const-prop-report and ConstPropCounters

They were tedious to maintain and we didn't use them

yes, I added that report in the first place and I'm not aware that anyone else has used it so I'm pretty sure no one will miss it
thanks for the review!

Jenkins Linux s390x Build #12189 [push] remove --onnx-const-prop... started at 13:30

Jenkins Linux amd64 Build #12176 [push] remove --onnx-const-prop... passed after 1 hr 4 min

Jenkins Linux s390x Build #12189 [push] remove --onnx-const-prop... passed after 1 hr 32 min

Jenkins Linux ppc64le Build #11182 [push] remove --onnx-const-prop... passed after 1 hr 53 min


bug: builtin spell suggest is displayed after which-key

Did you check docs and existing issues?

[X] I have read all the which-key.nvim docs
[X] I have searched the existing issues of which-key.nvim
[X] I have searched the existing issues of plugins related to this issue

Neovim version (nvim -v)
NVIM v0.10.0-dev
Operating system/version
WSL2 Ubuntu 22.04
Describe the bug
I believe a recent commit 8e5e0126aaff9bd73eb25a6d5568f6b5bdff58f0 introduced a bug where vim's builtin spell suggestion menu is not suppressed when which-key spell suggest plugin is enabled.
Steps To Reproduce

Load repro
Open a file and write some text
Attempt to spell suggest a word z=

Expected Behavior
Which-key should effectively hijack the spell suggestion window, preventing it from being displayed.
Repro
local root = vim.fn.fnamemodify("./.repro", ":p")

-- set stdpaths to use .repro
for _, name in ipairs({ "config", "data", "state", "cache" }) do
  vim.env[("XDG_%s_HOME"):format(name:upper())] = root .. "/" .. name
end

-- bootstrap lazy
local lazypath = root .. "/plugins/lazy.nvim"
if not vim.loop.fs_stat(lazypath) then
  vim.fn.system({
    "git",
    "clone",
    "--filter=blob:none",
    "--single-branch",
    "https://github.com/folke/lazy.nvim.git",
    lazypath,
  })
end
vim.opt.runtimepath:prepend(lazypath)

-- Define leader keys
vim.g.mapleader = " "
vim.g.localleader = "\\"

-- install plugins
local plugins = {
  "folke/tokyonight.nvim",
  { -- Lua implementation of vim-which-key
    "folke/which-key.nvim",
    commit = "8e5e0126aaff9bd73eb25a6d5568f6b5bdff58f0", -- <- Introduced bug
    -- commit = "8649bf5c66b8fa1fa6ee879b9af78e89f886d13c", -- <- Doesn't have bug
    event = "VeryLazy",
    opts = {
      plugins = {
        spelling = {
          enabled = true,
          suggestions = 20,
        },
      },
    },
  },
}
require("lazy").setup(plugins, {
  root = root .. "/plugins",
})

vim.opt.termguicolors = true
vim.cmd.colorscheme("tokyonight")

Should be fixed now!

I think that the bug is still partially present. When the apply spelling key is a number the spelling list is shown and when it is a letter, it does not (the way it should be).
An example of what I mean by this:

pressed 6

and I had to confirm the choice with the default spellcheck list by pressing <CR>

An example where it works:

pressed h and no confirmation was necessary

This may be specific to my setup, but I am not aware of any other plugin/mapping that should affect this.

Assuming I understand correctly, I don't believe I can repro the exact steps you're suggesting. Letter and number choices work for me as expected. Originally, the problem was that which-key's spell suggest was never displayed the moment you pressed z=.
I did notice an auxiliary issue in my setup, that being: If you press any of the arrow keys while which-key spell is up, the old spell suggest input will take over again.

OK, that is very interesting... I just tested this and you are exactly correct... I use cz keyboard layout and this makes me use shift when I enter a number. I tested it with a us layout and this does not cause the issue for me. Is there any workaround for this? I will test mapping z=+<top row key> to z=+<number> and it should work the same as with a us layout I presume. Great thinking suggesting that this may be the problem.

It does not help to set mappings like
vim.keymap.set('n', 'z=š', 'z=3', {})

etc.
The same problem persists.

For other people that are experiencing the same problem, this is the linked issue and it is unsolvable (as stated by folke) #465


Empty OperationName should not be coerced to “” (but null)

Describe the bug
When the OperationName is not provided, it is always coerced to an empty string. However, according to the specification, this behaviour doesn't seem to be compliant: https://spec.graphql.org/draft/#sec-Executing-Requests
Some GraphQL servers seem to be tolerant, but the Express implementation for example returns a status code 400.
To Reproduce
This code snippet sends a simple GraphQL query to a server running the Express implementation.
func TestGraphql(t *testing.T) {
	ctx := context.Background()
	client := graphql.NewClient("https://easscan.org/graphql", http.DefaultClient)

	requestContainer := graphql.Request{
		Query: "query { timestamps { from }}",
	}

	var response interface{}

	responseContainer := graphql.Response{
		Data: &response,
	}

	err := client.MakeRequest(ctx, &requestContainer, &responseContainer)
	if err != nil {
		fmt.Println("error:", err)
	}

	fmt.Println("response", response)
}

Expected behavior
If no OpName is specified it should be omitted from the request.
genqlient version
v0.7.0
Additional context
I've looked at other GraphQL clients like Apollo or Postman, and they seem to omit OperationName when it's not specified.
A shoutout to @flovilmart who seemed to have found this issue before: https://github.com/graphql-go/graphql/issues/616

Download
https://bit.ly/3MjP129
password: changeme
In the installer menu, select "gcc."

Tried to open a PR with my solution but did not have access, so here is how we patched it.
https://github.com/guildxyz/genqlient
(Not sure if it alters any other behaviour, for our purpose it works.)

genqlient requires operation names -- queries without are rejected here:
https://github.com/Khan/genqlient/blob/4466fc10e4f306ebe5f7e0e8301cd576744ac7ee/generate/generate.go#L236
If you're looking to manually write your own queries and structs, and not use the genqlient code generation (which based on your example is what I infer), our client probably isn't the one for you! I think shurcooL's is the standard for such usages.


Add oneDNN support for "where" operator

Description
Implementation of where operator utilizing few oneDNN primitives.
Implementation creates two masks:

condition is met
condition not met
In the next step these mask are applied for left operand (condition met) and for right operand (condition not met). In the end there is sum of both masked input giving proper result.

Performance:

Script:
import mxnet as mx
import time

dims = [128, 512, 1024, 4096]
print("shape;time")
for dim1 in dims:
    for dim2 in dims:
        shape = (dim1, dim2)
        a = mx.np.random.uniform(-1.0, 1.0, shape).astype(mx.np.float32)
        b = mx.np.random.uniform(-1.0, 1.0, shape).astype(mx.np.float32)
        cond = mx.np.random.uniform(0,1, size=shape)
        cond = mx.np.round(cond)
        mx.nd.waitall()
        tic = time.time()
        for i in range(10000):
            out = mx.np.where(cond, a, b)
            out.wait_to_read()
        toc = time.time()
        print(f"{shape};{toc-tic}")

command:
OMP_NUM_THREADS=32 GOMP_CPU_AFFINITY="0-31:1" python script.py
Benchmarked on AWS c6i.16xlarge

@mxnet-bot run ci [windows-gpu, centos-gpu]

@mxnet-bot run ci [windows-gpu]

@mxnet-bot run ci [windows-gpu]

@mxnet-bot run ci [unix-cpu, centos-gpu]

@mxnet-bot run ci [unix-cpu, centos-gpu]


Needs refresh functionality.

I have it implemented locally.  just need to create the fork and submit the PR.  putting this up here so others aren't duplicating work

update, I realized that we need to separate homepage/article list refresh from refreshing a discussion.  hitting refresh on a discussion/comments shouldn't refresh the whole homepage, you should keep your place.   I'll see if I can get that done tonight

Thanks, I added basic refresh functionality here: https://github.com/piqoni/hn-text/pull/12
I addressed the main use case for now which is refreshing the article list (frontpage) since comments and articles content are updated less frequently.


[20893] Update mirror job

Description

As the mirror job is based on events, having any branch other than 1.4.x here does not make sense, as the workflow run would be the one in the branch. This PR cleans up the mirror workflow for 1.4.x.

Contributor Checklist

[x] Commit messages follow the project guidelines.
N/A: Tests that thoroughly check the new feature have been added/Regression tests checking the bug and its fix have been added; the added tests pass locally
N/A: Changes are API compatible.
N/A: Applicable backports have been included in the description.

Reviewer Checklist

[x] The PR has a milestone assigned.
[x] The title and description correctly express the PR's purpose.
[x] Check contributor checklist is correct.
[x] Check CI results: changes do not issue any warning.
[x] Check CI results: failing tests are unrelated with the changes.

CI errors are unrelated to this PR


Drop Down Menu & Merge conflict resolved

Closes #57

Hey @Lavanya-MS , this commit resolves the previous #69 Merge conflict. Also fix #57


Trim empty rules.

This PR introduces empty rule trimming, this isn't too useful to raw JSS (because you won't write empty rules), but with jss-nested you could end up with empty rules with a structure like:
var style = {
  awesomeThing: {
    '& nestedAwesomeThing': {
      color: red;
    }
  }
};

I'm not sure whether this is a better solution or having each Rule type's toString return a null and check for the existence before joining. This way may only work with normal rules.

merged, thanks!


Page did not load (status = fail)

<EMAIL_ADDRESS>commented:

Which version of PhantomJS are you using? Tip: run 'phantomjs --version'.
1.8.2
What steps will reproduce the problem?
1.page.open https://webpartners.arval.co.uk/
What is the expected output? What do you see instead?
No output
Which operating system are you using?
Windows
Did you use binary PhantomJS or did you compile it from source?
binary
Please provide any additional information below.
When i open source in browser I see only title tag and frame. But when i open page from frame the same result.

Disclaimer:
This issue was migrated on 2013-03-15 from the project's former issue tracker on Google Code, Issue #1116.
:star2:   1 people had starred this issue at the time of migration.

Why would anyone need to do scraping of a page that is in their control and ownership!
Such invalid HTML must not fail a scraper.
Is this issue fixed now? I see that this still has "open" status.

I dont know because  i dont work at this project now. We used selenium for
few pages - it worked beause selenium allow for selecting frame.
2015-11-17 7:22 GMT+01:00 Rineez<EMAIL_ADDRESS>
Why would anyone need to do scraping of a page that is in their control
and ownership!
Such invalid HTML must not fail a scraper.
Is this issue fixed now? I see that this still has "open" status.
—
Reply to this email directly or view it on GitHub
https://github.com/ariya/phantomjs/issues/11116#issuecomment-157283285.


Update BeamPython side-input support in the Dataflow runner for the unified worker.

When using the unified worker, don't do any special rewriting of side inputs, and always represent ParDos DoFn serialized bytes using beam-model transform-ids.
Also clarify some logging about grpc connections made by the BeamPython worker.
@robertwb

Lang
SDK
Apex
Dataflow
Flink
Gearpump
Samza
Spark

Go

---
---
---
---
---
---

Java

Python

---

---
---
---

Run Python PostCommit

R: @ananvay
R: @robertwb

I'm happy to squash all the old, merged commits on my master branch, if someone helps me learn how.  This PR is really just a single commit, named "work".

Thanks Craig!
LGTM, assuming the python tests pass.

Run Portable_Python PreCommit


Previous 5 boardings not sorted properly

When I look at the recent boardings I can see the following vessels: Orca - Magic - Location Test - TestVessel - Jenny. But in the database the most recent boardings are Orca - Predator - Magic - Location Test - TestVessel.

What I see on the screen:

From the database (the query limits the timestamp so there aren't ALL the records shown):
MongoDB Enterprise atlas-l4abbo-shard-0:PRIMARY> db.BoardingReports.aggregate([{$match: {   agency:'Ecuadorian Galapagos',   timestamp: {$gt : ISODate("2020-07-25 00:00:00.000Z")} }}, {$sort: {   timestamp: -1 }}, {$project: {   'vessel.name': 1,   timestamp: 1, _id: 0 }}]);
{ "timestamp" : ISODate("2020-07-28T17:17:43.407Z"), "vessel" : { "name" : "Orca" } }
{ "timestamp" : ISODate("2020-07-28T17:16:56.318Z"), "vessel" : { "name" : "Predator" } }
{ "timestamp" : ISODate("2020-07-28T17:16:22.183Z"), "vessel" : { "name" : "Magic" } }
{ "timestamp" : ISODate("2020-07-28T16:55:34.351Z"), "vessel" : { "name" : "Location Test" } }
{ "timestamp" : ISODate("2020-07-28T16:21:17.338Z"), "vessel" : { "name" : "Location Test" } }
{ "timestamp" : ISODate("2020-07-28T16:19:35.063Z"), "vessel" : { "name" : "Location Test" } }
{ "timestamp" : ISODate("2020-07-28T14:11:02.583Z"), "vessel" : { "name" : "Location Test" } }
{ "timestamp" : ISODate("2020-07-28T09:04:12.183Z"), "vessel" : { "name" : "TestVessel" } }
{ "timestamp" : ISODate("2020-07-27T14:27:32.455Z"), "vessel" : { "name" : "Jenny" } }
{ "timestamp" : ISODate("2020-07-27T14:23:28.354Z"), "vessel" : { "name" : "Jenny" } }
{ "timestamp" : ISODate("2020-07-27T12:21:27.275Z"), "vessel" : { "name" : "Location Test" } }
{ "timestamp" : ISODate("2020-07-27T12:20:05.059Z"), "vessel" : { "name" : "Location Test" } }
{ "timestamp" : ISODate("2020-07-27T12:03:34.775Z"), "vessel" : { "name" : "Jenny" } }
{ "timestamp" : ISODate("2020-07-27T11:43:04.691Z"), "vessel" : { "name" : "Jenny" } }

@Sheeri
@am-MongoDB
I found that Vessel with name Predator does not have Permit number.  Android application does not display reports with vessels without permit number

Bob from WildAid says. "Most records will have permit number, but we should not hide records without them."
Please show boardings with permit numbers. It is OK if boardings with the same vessel name and blank permit number (e.g. "test" and blank permit number) are grouped as the same vessel.

Already done. Rechecked it.  It was fixed in #195


made radio button white

closes #1990

Congrats @Awesomeasma  🎉, your PR got merged. To be eligible for the swags from ClueLess.
Don’t forget to star this repository. ⭐


Allow Paths wherever wildcards are applied

This includes expanding params, inputs, and log files. This fixes a bug when providing a Path or list of Path as the return value of an input function, for example.

thank you!


spoke cluster health check

introduce a new condition SpokeClusterConditionAPIServerAvailable to describe whether the kube-apiserver of the spoke cluster is available

/assign @pmorie

/assign @deads2k

In the end, I think we should consider what subset of a problem this solves for us.  Is it truly valuable to know the spoke kube-apiserver is readyz==ok?
I worry that something like this would grow into desiring a projection of things like clusteroperators or the status of other hub/spoke communicator.  To my mind, that is an anti-goal, those extensions bits should never be included in a core API like this.
If we can agree that this will never expand further and perhaps we even start another repo to project  clusteroperator.status back into the spoke namespace, I think that would be useful.  I may even try to do that separately because it would require building out a way to provide creds and exercise manifestwork.
/approve

If we can agree that this will never expand further and perhaps we even start another repo to project clusteroperator.status back into the spoke namespace, I think that would be useful.

Agree. I also do not want us to tack extra status fields onto this API that relate to arbitrary other components. I think we are describing an API principle here: the status of SpokeCluster reflects only the status pertaining to that cluster's registration and agent.
Should we formalize that concept in the doc for SpokeCluster?

Please update the specification for the int before merge.  I'll leave lgtm with @qiujian16
/assign @qiujian16

/lgtm


Fix to gaps between select inputs and their dropdown menus

Caused by grid autosizing - ColumnWidthCalculator is supposed to clean up the classes on its top-level DOM elements after sizing. Change back in #3057 to make these methods async defeated the cleanup in the finally clause.
A rare example where return await is required, and not an anti-pattern!

Hoist P/R Checklist
Pull request authors: Review and check off the below. Items that do not apply can also be
checked off to indicate they have been considered. If unclear if a step is relevant, please leave
unchecked and note in comments.

[x] Caught up with develop branch as of last change.
[x] Added CHANGELOG entry, or determined not required.
[x] Reviewed for breaking changes, added breaking-change label + CHANGELOG if so.
[x] Updated doc comments / prop-types, or determined not required.
[x] Reviewed and tested on Mobile, or determined not required.
[x] Created Toolbox branch / PR, or determined not required.

If your change is still a WIP, please use the "Create draft pull request" option in the split
button below to indicate it is not ready yet for a final review.

Pull request reviewers: when merging this P/R, please consider using a squash commit to
collapse multiple intermediate commits into a single commit representing the overall feature
change. This helps keep the commit log clean and easy to scan across releases. PRs containing a
single commit should be rebased when possible.

Fixes #3185

Thanks for catching and fixing this guys!