ghdl simulation failure

I tried running the script ghdl_sim.sh on Ubuntu (WSL 18.04) and it gave and error about --max-sstacck-alloc=0' so I tried running it with that option and it output this message after running starting the simulation that at GHDL Bug occurred.
I was using GHDL 0.35.  I got 4 output files 2 empty output files and one for the uart.out with a single statement. Is this the correct way this simulation should run?
-rw-rw-rw- 1 ssheikh ssheikh 8992 Jan 11 11:44 neorv32-obj93.cf
-rwxrwxrwx 1 ssheikh ssheikh    0 Jan 11 11:45 neorv32.uart.sim_mode.text.out
-rwxrwxrwx 1 ssheikh ssheikh    0 Jan 11 11:45 neorv32.uart.sim_mode.data.out
-rwxrwxrwx 1 ssheikh ssheikh   26 Jan 11 12:01 neorv32.testbench_uart.out
ssheikh@GSLWL2020070597:~/neorv32/sim/ghdl$ more neorv32.testbench_uart.out
Blinking LED demo program
ghdl -r --work=neorv32 neorv32_tb --ieee-asserts=disable --assert-level=error $SIM_CONFIG
/home/ssheikh/neorv32/sim/ghdl/../../rtl/core/neorv32_cpu.vhd:170:3:@0ms:(assertion warning): NEORV32 CPU CONFIG WARNING! Atomic operations extension (A) only supports >lr.w< and >sc.w< instructions yet.
/home/ssheikh/neorv32/sim/ghdl/../../rtl/core/neorv32_cpu.vhd:179:3:@0ms:(assertion note): NEORV32 CPU CONFIG NOTE: Implementing physical memory protection (PMP) with 2 regions and 65536 bytes minimal region size (granulartiy).
/home/ssheikh/neorv32/sim/ghdl/../../rtl/core/neorv32_wishbone.vhd:141:3:@0ms:(assertion note): NEORV32 PROCESSOR CONFIG NOTE: Implementing external memory interface using STANDARD Wishbone protocol.
/home/ssheikh/neorv32/sim/ghdl/../../rtl/core/neorv32_wishbone.vhd:145:3:@0ms:(assertion note): NEORV32 PROCESSOR CONFIG NOTE: Using BIG-ENDIAN byte order for external memory interface.
/home/ssheikh/neorv32/sim/ghdl/../../sim/neorv32_tb.vhd:297:15:@1342745ns:(report note): NEORV32_TB_UART.TX: B
/home/ssheikh/neorv32/sim/ghdl/../../sim/neorv32_tb.vhd:297:15:@1915955ns:(report note): NEORV32_TB_UART.TX: l
/home/ssheikh/neorv32/sim/ghdl/../../sim/neorv32_tb.vhd:297:15:@2489165ns:(report note): NEORV32_TB_UART.TX: i
/home/ssheikh/neorv32/sim/ghdl/../../sim/neorv32_tb.vhd:297:15:@3062375ns:(report note): NEORV32_TB_UART.TX: n
/home/ssheikh/neorv32/sim/ghdl/../../sim/neorv32_tb.vhd:297:15:@3635585ns:(report note): NEORV32_TB_UART.TX: k
/home/ssheikh/neorv32/sim/ghdl/../../sim/neorv32_tb.vhd:297:15:@4208795ns:(report note): NEORV32_TB_UART.TX: i
/home/ssheikh/neorv32/sim/ghdl/../../sim/neorv32_tb.vhd:297:15:@4782005ns:(report note): NEORV32_TB_UART.TX: n
/home/ssheikh/neorv32/sim/ghdl/../../sim/neorv32_tb.vhd:297:15:@5355215ns:(report note): NEORV32_TB_UART.TX: g
/home/ssheikh/neorv32/sim/ghdl/../../sim/neorv32_tb.vhd:297:15:@5928425ns:(report note): NEORV32_TB_UART.TX:
/home/ssheikh/neorv32/sim/ghdl/../../sim/neorv32_tb.vhd:297:15:@6501635ns:(report note): NEORV32_TB_UART.TX: L
/home/ssheikh/neorv32/sim/ghdl/../../sim/neorv32_tb.vhd:297:15:@7074845ns:(report note): NEORV32_TB_UART.TX: E
/home/ssheikh/neorv32/sim/ghdl/../../sim/neorv32_tb.vhd:297:15:@7648055ns:(report note): NEORV32_TB_UART.TX: D
/home/ssheikh/neorv32/sim/ghdl/../../sim/neorv32_tb.vhd:297:15:@8221265ns:(report note): NEORV32_TB_UART.TX:
/home/ssheikh/neorv32/sim/ghdl/../../sim/neorv32_tb.vhd:297:15:@8794475ns:(report note): NEORV32_TB_UART.TX: d
/home/ssheikh/neorv32/sim/ghdl/../../sim/neorv32_tb.vhd:297:15:@9367685ns:(report note): NEORV32_TB_UART.TX: e
/home/ssheikh/neorv32/sim/ghdl/../../sim/neorv32_tb.vhd:297:15:@9940895ns:(report note): NEORV32_TB_UART.TX: m
/home/ssheikh/neorv32/sim/ghdl/../../sim/neorv32_tb.vhd:297:15:@10514105ns:(report note): NEORV32_TB_UART.TX: o
/home/ssheikh/neorv32/sim/ghdl/../../sim/neorv32_tb.vhd:297:15:@11087315ns:(report note): NEORV32_TB_UART.TX:
/home/ssheikh/neorv32/sim/ghdl/../../sim/neorv32_tb.vhd:297:15:@11660525ns:(report note): NEORV32_TB_UART.TX: p
/home/ssheikh/neorv32/sim/ghdl/../../sim/neorv32_tb.vhd:297:15:@12233735ns:(report note): NEORV32_TB_UART.TX: r
/home/ssheikh/neorv32/sim/ghdl/../../sim/neorv32_tb.vhd:297:15:@12806945ns:(report note): NEORV32_TB_UART.TX: o
/home/ssheikh/neorv32/sim/ghdl/../../sim/neorv32_tb.vhd:297:15:@13380155ns:(report note): NEORV32_TB_UART.TX: g
/home/ssheikh/neorv32/sim/ghdl/../../sim/neorv32_tb.vhd:297:15:@13953365ns:(report note): NEORV32_TB_UART.TX: r
/home/ssheikh/neorv32/sim/ghdl/../../sim/neorv32_tb.vhd:297:15:@14526575ns:(report note): NEORV32_TB_UART.TX: a
/home/ssheikh/neorv32/sim/ghdl/../../sim/neorv32_tb.vhd:297:15:@15099785ns:(report note): NEORV32_TB_UART.TX: m
/home/ssheikh/neorv32/sim/ghdl/../../sim/neorv32_tb.vhd:295:15:@15672965ns:(report note): NEORV32_TB_UART.TX: (13)
/home/ssheikh/neorv32/sim/ghdl/../../sim/neorv32_tb.vhd:295:15:@16246165ns:(report note): NEORV32_TB_UART.TX: (10)
******************** GHDL Bug occured ****************************
Please report this bug on https://github.com/tgingold/ghdl/issues
GHDL release: 0.35 (Ubuntu 0.35+dfsg-1~build3) [Dunoon edition]
Compiled with GNAT Version: 7.3.0
Target: x86_64-linux-gnu
In directory: /home/ssheikh/neorv32/sim/ghdl/
Command line:
/usr/bin/ghdl-mcode -r --work=neorv32 neorv32_tb --ieee-asserts=disable --assert-level=error
Exception CONSTRAINT_ERROR raised
Exception information:
raised CONSTRAINT_ERROR : grt-processes.adb:770 overflow check failed
Call stack traceback locations:
0x7f51e547c42e 0x7f51e547cbd4 0x7f51e547ce4a 0x7f51e547d02c 0x7f51e5412a4a 0x7f51e54bc4bd 0x7f51e56f2fea 0x7f51e56df51f 0x7f51e56d0dcd 0x7f51e56fcb9e 0x7f51e541266f 0x7f51e3631b95 0x7f51e5411498 0xfffffffffffffffe

A stack overflow error in GHDL can occur if you are running out of memory due to too many simulated signals. GHDL uses several bytes for each signal and you might get problems when simulating large memories. There are simulation-optimized versions of the IMEM and DMEM components, that use constants/variables instead of the 'signals' for the memory array: sim/rtl_modules
The default testbench features a simulated UART receiver. Eachprintable char that is received from the processor's UART will be printed to the console and also written to neorv32.testbench_uart.out.
If the UART is in simulation mode (setting bit 12 in the UART control register), the actual UART transmitter logic will be disabled. All data written to the UART's TX data register will be "printed" using VHDL text.io (which is way faster than using the testbench simulated UART receiver):

The printable ASCII char will be printed to the console and also to neorv32.testbench_uart.out.
The full 32-bit word written to the UART'S TX data register will be printed as hex word to neorv32.uart.sim_mode.data.out.

You can automatically activate the simulation mode by using USER_FLAGS+=-DUART_SIM_MODE when compiling your applications.

A stack overflow error in GHDL can occur if you are running out of memory due to too many simulated signals. GHDL uses several bytes for each signal and you might get problems when simulating large memories. There are simulation-optimized versions of the IMEM and DMEM components, that use constants/variables instead of the 'signals' for the memory array: sim/rtl_modules
The default testbench features a simulated UART receiver. Eachprintable char that is received from the processor's UART will be printed to the console and also written to neorv32.testbench_uart.out.
If the UART is in simulation mode (setting bit 12 in the UART control register), the actual UART transmitter logic will be disabled. All data written to the UART's TX data register will be "printed" using VHDL text.io (which is way faster than using the testbench simulated UART receiver):

The printable ASCII char will be printed to the console and also to neorv32.testbench_uart.out.
The full 32-bit word written to the UART'S TX data register will be printed as hex word to neorv32.uart.sim_mode.data.out.

You can automatically activate the simulation mode by using USER_FLAGS+=-DUART_SIM_MODE when compiling your applications.

Is your simulation now working as expected? 🤔

Is your simulation now working as expected? 🤔

You can close this. I haven't been able to run it yet. I was thinking more
of getting the Xilin Arty or Terasic DE0 board from work to put the SOC on
them and take it for a whirl.
Salman
On Thu, Jan 21, 2021 at 12:00 PM Stephan<EMAIL_ADDRESS>wrote:

Is your simulation now working as expected? 🤔
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/stnolting/neorv32/issues/5#issuecomment-764793650,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAZFYMHDREAMO3L73L2NQT3S3BMURANCNFSM4V53YDBA
.

--
Even a Smile is charity :)

Prophet Muhammad

You can close this. I haven't been able to run it yet. I was thinking more
of getting the Xilin Arty or Terasic DE0 board from work to put the SOC on
them and take it for a whirl.
Salman
On Thu, Jan 21, 2021 at 12:00 PM Stephan<EMAIL_ADDRESS>wrote:

Is your simulation now working as expected? 🤔
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/stnolting/neorv32/issues/5#issuecomment-764793650,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAZFYMHDREAMO3L73L2NQT3S3BMURANCNFSM4V53YDBA
.

--
Even a Smile is charity :)

Prophet Muhammad

I'm also working with Arty and E0-nano. So if you need any help, feel free to get in contact again. :wink:

I'm also working with Arty and E0-nano. So if you need any help, feel free to get in contact again. :wink:


switch to SimplePersistentEnvExtension in EquationsCommand

Our current magmaLawExt performs poorly, for two reasons:

it redoes the work of mkNatMagmaLaw on every import
it uses Array instead of List. (See this comment on Zulip).

We can avoid both of these problems by using SimplPersistentEnvExtension.
This allows us to uncomment the code that was commented out in #465, without a significant performance penalty.

This PR was made obsolete by #478.


Mutably dereferencing a Mut<ParticleEffect> causes an assertion failure

Crate versions
bevy version: 0.11
bevy_hanabi version: 0.7
Describe the bug
When a ParticleEffect component is mutably accessed without changing its effect handle, bevy_hanabi panics with an assertion failure:
thread 'Compute Task Pool (7)' panicked at 'assertion failed: self.asset != weak_handle', /home/sludge/code/bevy_hanabi/src/lib.rs:1061:9

The assertion is this:
debug_assert!(self.asset != weak_handle);

It looks like the assumption is made that any mutable dereference of the ParticleEffect will change its effect handle, but that isn't necessarily the case.
Expected behavior
No panic should happen. The effect should either be recompiled, or left as-is, if neither the handle nor any other data used by the compilation changed, and bevy_hanabi wants to do more fine-grained change detection here.
To Reproduce
This system causes an assertion failure in bevy_hanabi when the A key is pressed and any ParticleEffects exist:
fn boop_effects(mut q: Query<&mut ParticleEffect>, inp: Res<Input<KeyCode>>) {
    if inp.just_pressed(KeyCode::A) {
        for mut effect in &mut q {
            effect.deref_mut();
        }
    }
}

This check should have prevented update() from being called. This is why there's an assert in place, because update() shouldn't be called if the asset handle is not changed. So the bug is not the assert, it's update() being called in the first place. Not sure why the check doesn't work.
https://github.com/djeedai/bevy_hanabi/blob/5832bb99dd6881d6041bb2aff7b8a0696c19ab71/src/lib.rs#L1341-L1343

I just removed the assert, let it do a full rebuild. That should work now.


How to generate an answer without pre-defined length of tokens?

I notice there is a parameter --num-tokens in lora.py. If I want to generate an answer, the length of the answer is always depend on the question rather than a fixed token length. How can I generate an answer without pre-defined length of tokens?

Right so the parameter is actually named --max-tokens now. This is just an upper limit on the maximum number of tokens. Usually the model stops when it encounters an end of sentence token before it hits the upper limit.

Thanks a lot.


Unstable performance

Hello, jihan,
It's a great work！
I am learning and reproducing your ST3D++.
But the experimental results show that the performance of the models trained multiple times is different.
I trained 30 Epochs on KITTI using the model you provided pre-trained by ros on the nuScenes dataset. I used a single 1080Ti.  I used SECOND-IoU Detecter.
This result I reproduced fluctuates a lot, is it normal?

 The performance of nuscenes -> KITTI is not very stable due to its sparse point density, but its best performance should be avg+- 2 in most cases. As I typically use 4 gpu and a total batch size 16, I suspect that your case is caused by the smaller batch size.

May I know your batch size? The performance of nuscenes -> KITTI is not very stable due to its sparse point density, but its best performance should be avg+- 2 in most cases. As I typically use 4 gpu and a total batch size 16, I suspect that your case is caused by the smaller batch size.

I used batchsize 4 with a single 1080Ti GPU.

So, you use 4 times smaller batch size than default setting. I think the unstable could by caused by this change.

So, you use 4 times smaller batch size than default setting. I think the unstable could by caused by this change.

Thank you very much for your answer. I will try it on GPUs with more memory.


Fail to analyze ensemble model: "inference.ModelConfig" should not have multiple "scheduling_choice" oneof fields

When I use model-analyzer to analyze a ensemble model with local luanch mode, it always fails with following error:
root@dl:/inference# model-analyzer profile --checkpoint-directory checkpoints -m $PWD/model_repo --profile-models quartznet-ensemble --output-model-repository-path=/output_repo/temp --override-output-model-repository --client-protocol grpc --run-config-search-max-concurrency 800 --run-config-search-max-instance-count 2 --run-config-search-max-preferred-batch-size 64

Traceback (most recent call last):
  File "/usr/local/lib/python3.8/dist-packages/google/protobuf/json_format.py", line 538, in _ConvertFieldValuePair
    raise ParseError('Message type "{0}" should not have multiple '
google.protobuf.json_format.ParseError: Message type "inference.ModelConfig" should not have multiple "scheduling_choice" oneof fields.

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/local/bin/model-analyzer", line 8, in <module>
    sys.exit(main())
  File "/usr/local/lib/python3.8/dist-packages/model_analyzer/entrypoint.py", line 315, in main
    analyzer.profile(client=client)
  File "/usr/local/lib/python3.8/dist-packages/model_analyzer/analyzer.py", line 104, in profile
    self._model_manager.run_model(model=model)
  File "/usr/local/lib/python3.8/dist-packages/model_analyzer/model_manager.py", line 84, in run_model
    self._run_model_with_search(model)
  File "/usr/local/lib/python3.8/dist-packages/model_analyzer/model_manager.py", line 138, in _run_model_with_search
    self._run_model_config_sweep(model, search_model_config=True)
  File "/usr/local/lib/python3.8/dist-packages/model_analyzer/model_manager.py", line 167, in _run_model_config_sweep
    self._run_config_generator.generate_run_config_for_model_sweep(
  File "/usr/local/lib/python3.8/dist-packages/model_analyzer/config/run/run_config_generator.py", line 98, in generate_run_config_for_model_sweep
    model_config = ModelConfig.create_from_dictionary(
  File "/usr/local/lib/python3.8/dist-packages/model_analyzer/triton/model/model_config.py", line 117, in create_from_dictionary
    protobuf_message = json_format.ParseDict(model_dict,
  File "/usr/local/lib/python3.8/dist-packages/google/protobuf/json_format.py", line 454, in ParseDict
    parser.ConvertMessage(js_dict, message)
  File "/usr/local/lib/python3.8/dist-packages/google/protobuf/json_format.py", line 485, in ConvertMessage
    self._ConvertFieldValuePair(value, message)
  File "/usr/local/lib/python3.8/dist-packages/google/protobuf/json_format.py", line 599, in _ConvertFieldValuePair
    raise ParseError(str(e))
google.protobuf.json_format.ParseError: Message type "inference.ModelConfig" should not have multiple "scheduling_choice" oneof fields.

The model repository I used can be downloaded here.

@riverliuc Ensemble models are not supported yet in Model Analyzer. You can only use single-stage models in Model Analyzer.

Hey @Tabrizian,
Do you have any update on ensemble model support?

@okanlv We don't have any updates regarding the ensemble support. We'll update this issue as soon as more information is available.

Is ensemble models now supported for model-analyzer?

Not yet. We are working on it.

Do you have an expected timeline when this will be supported ?

@dhaval24  it's on our short term roadmap since it is a high priority feature. I can share a more granular timeline with you over the email thread we have

I see in 23:01 ensemble models are supported. However I am facing the same error.
google.protobuf.json_format.ParseError: Message type "inference.ModelConfig" should not have multiple "scheduling_choice" oneof fields.
I am using 23:01 model analyzer but the triton server is 22:07.

@jmuppave @riverliuc Do either of you have more recent repro? The original repro instructions are too old to use.

I see in 23:01 ensemble models are supported.

@jmuppave Ensemble model support is in the repository, but was not in 23.01 release. It will be in the 23.02 release which should come out in the next few days.

23.02 has been released. Ensemble support is available

Hi,
I have started using 23.02 still facing the same issue:
`
[Model Analyzer] Creating model config: ensemble_xyz_config_default
[Model Analyzer] Model ensemble_xyz_config_default load failed: [StatusCode.INTERNAL] failed to load
'ensemble_xyz_config_default', no version is available
[Model Analyzer] Found existing model config: ensemble_xyz_config_0
[Model Analyzer]   Setting instance_group to [{'count': 1, 'kind': 'KIND_GPU'}]
[Model Analyzer]   Setting max_batch_size to 1
[Model Analyzer]   Enabling dynamic_batching
[Model Analyzer] Saved checkpoint to /data/model_analyzer_output/checkpoints/3.ckpt
Traceback (most recent call last):
File "/usr/local/lib/python3.8/dist-packages/google/protobuf/json_format.py", line 559, in _ConvertFieldValuePair
raise ParseError('Message type "{0}" should not have multiple '
google.protobuf.json_format.ParseError: Message type "inference.ModelConfig" should not have multiple "scheduling_choice" oneof fields at "ModelConfig".
`

Please delete your checkpoint and start a fresh run. Hopefully that will resolve the issue you are seeing.

Thanks, it does the trick. However, I am facing another issue. My ensemble config is similar to ensemble example in architecture
Here is the entire log:
`[Model Analyzer] Starting a Triton Server using docker
[Model Analyzer] Pulling docker image nvcr.io/nvidia/tritonserver:22.07-py3
[Model Analyzer] No checkpoint file found, starting a fresh run.
[Model Analyzer] Profiling server only metrics...
[Model Analyzer]
[Model Analyzer] Starting quick mode search to find optimal configs
[Model Analyzer]
[Model Analyzer]
[Model Analyzer] Creating model config: xyz_preprocess_config_default
[Model Analyzer]
[Model Analyzer]
[Model Analyzer] Creating model config: xyz_config_default
[Model Analyzer]
[Model Analyzer]
[Model Analyzer] Creating model config: xyz_postprocess_config_default
[Model Analyzer]
[Model Analyzer] Model ensemble_xyz_config_default load failed: [StatusCode.INTERNAL] failed to load 'ensemble_xyz_config_default', no version is available
[Model Analyzer]
[Model Analyzer] Creating model config: xyz_preprocess_config_0
[Model Analyzer]   Setting instance_group to [{'count': 1, 'kind': 'KIND_GPU'}]
[Model Analyzer]   Setting max_batch_size to 1
[Model Analyzer]   Enabling dynamic_batching
[Model Analyzer]
[Model Analyzer]
[Model Analyzer] Creating model config: xyz_config_0
[Model Analyzer]   Setting instance_group to [{'count': 1, 'kind': 'KIND_GPU'}]
[Model Analyzer]   Setting max_batch_size to 1
[Model Analyzer]   Enabling dynamic_batching
[Model Analyzer]
[Model Analyzer] Creating model config: xyz_postprocess_config_0
[Model Analyzer]   Setting instance_group to [{'count': 1, 'kind': 'KIND_GPU'}]
[Model Analyzer]   Setting max_batch_size to 1
[Model Analyzer]   Enabling dynamic_batching
[Model Analyzer]
[Model Analyzer] Model ensemble_xyz_config_0 load failed: [StatusCode.INTERNAL] failed to load 'ensemble_xyz_config_0', no version is available
[Model Analyzer]
[Model Analyzer] Done with quick mode search. Gathering concurrency sweep measurements for reports
[Model Analyzer]
[Model Analyzer]
[Model Analyzer] Done gathering concurrency sweep measurements for reports
[Model Analyzer]
[Model Analyzer] Saved checkpoint to /data/model_analyzer_output/checkpoints/0.ckpt
[Model Analyzer] Profile complete. Profiled 0 configurations for models: []
[Model Analyzer]
[Model Analyzer] WARNING: Inference output field "perf_throughput", has no data
[Model Analyzer] WARNING: Inference output field "perf_latency_p99", has no data
[Model Analyzer] Exporting server only metrics to /data/model_analyzer_output/exports/results/metrics-server-only.csv
[Model Analyzer] WARNING: Requested top 3 configs, but none satisfied constraints. Showing available constraint failing configs for this model.
[Model Analyzer] WARNING: Requested top 3 configs, but found only 0. Showing all available configs for this model.
[Model Analyzer] WARNING: Requested top 3 configs, but none satisfied constraints. Showing available constraint failing configs for this model.
[Model Analyzer] WARNING: Requested top 3 configs, but found only 0. Showing all available configs for this model.
[Model Analyzer] WARNING: Requested top 3 configs, but none satisfied constraints. Showing available constraint failing configs for this model.
[Model Analyzer] WARNING: Requested top 3 configs, but found only 0. Showing all available configs for this model.
[Model Analyzer] WARNING: No data found for model ensemble_xyz, skipping export summary.
[Model Analyzer] WARNING: Requested top 3 configs, but none satisfied constraints. Showing available constraint failing configs for this model.
[Model Analyzer] WARNING: Requested top 3 configs, but found only 0. Showing all available configs for this model.
[Model Analyzer] WARNING: Requested top 3 configs, but none satisfied constraints. Showing available constraint failing configs for this model.
[Model Analyzer] WARNING: Requested top 3 configs, but found only 0. Showing all available configs for this model.
`

Thanks, logging helped. It was an issue with loading custom .so file.


Missing role required to unblock the user

The article doesn't provide what Azure AD Role is required to unblock users.

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

ID: 84b26750-880c-8f14-1628-436293de915e
Version Independent ID: 518b727f-79e8-b5c2-bcb3-ff4de3444c6f
Content: Configure Azure AD Multi-Factor Authentication - Microsoft Entra
Content Source: articles/active-directory/authentication/howto-mfa-mfasettings.md
Service: active-directory
Sub-service: authentication
GitHub Login: @Justinha
Microsoft Alias: justinha

@Michal-Ziemba
Thanks for your feedback! We will investigate and update as appropriate.

@Justinha
Can you please check and add your comments on this doc update request as applicable.

@Michal-Ziemba
Thanks for bringing this to our attention.
I'm going to assign this to the document author so they can take a look at it accordingly.

Thanks for your feedback. We need to close all open issues at this time so we can migrate to a new Entra repository. I created a work item to have an expert review this feedback and update docs accordingly. #please-close


Fix URL of 404 page

Change the fixed URL at the 404 page to .Site.BaseURL

Hi mvdnes,
thank you very much for your pull request!


adding back Artsy for galleries link to navigation menu

Per @katarinabatina request we need to add back Artsy for Galleries link into top nav menu. Here it is!


replace services with link

we should leverage the variables we have :)

Thanks, mastertinner!

Provisionally reverted change, to avoid using variable without strong rationale.
Hey mastertinner-- for this PR and similar https://github.com/cloudfoundry/docs-cloudfoundry-concepts/issues/26 we would like to have better understanding of your needs. We definitely want to help, and have been looking at how we and others working on docs need to use variables and partials. We are interested in use cases to help figure out best way for all. Thanks!

Hi @pspinrad, my motivation is that I'm a Cloud Foundry provider who wants to host his own docs but doesn't want to include the Cloud Foundry provider related stuff like "How to deploy CF" for example. If I don't include these, however, I get broken links from Bookbinder. That's why I would like to exclude the few links that point to these sections and replace them text that doesn't link to any of these sections. I'm guessing that other CF providers like Pivotal have similar issues.


Merge in main and update uri parsing in timeseries factory

Am I right in thinking this was just squashed merged? That's going to cause chaos with the commit history. Is there any chance you could add me as a collaborator on this repo so I can push to this branch?
I think the root of the problem is that you opened this PR from the main branch which means that contributors to sunpy can't push to the PR branch, which makes this a little easier.

Would it be easier if I merged #14 into its own branch on sunpy, and then you merge these subsequent PRs there? I think that would solve your permissions issue

The problem with changing branches is that it will close the existing PR which isn't really what I want.

If you can revert merging this then that will be a good start.

ok I reverted and add you to the org. Let me if you have access

Yes, thanks!


fix NettyCodecAdapter decoder memory leak

What is the purpose of the change
reproduce code
    @Test
    void testDecodeException() throws IOException {
        Codec2 codec2 = Mockito.mock(Codec2.class);
        doThrow(new IOException("testDecodeIllegalPacket")).when(codec2).decode(any(), any());

        URL url = Mockito.mock(URL.class);
        doReturn("default").when(url).getParameter(eq(Constants.CODEC_KEY));

        ChannelHandler handler = Mockito.mock(ChannelHandler.class);
        NettyCodecAdapter nettyCodecAdapter = new NettyCodecAdapter(codec2, url, handler);
        io.netty.channel.ChannelHandler decoder = nettyCodecAdapter.getDecoder();
        EmbeddedChannel embeddedChannel = new EmbeddedChannel();
        embeddedChannel.pipeline().addLast(decoder);

        // 模拟问题数据包
        ByteBuf input = AbstractByteBufAllocator.DEFAULT.buffer();
        input.writeBytes("testDecodeIllegalPacket".getBytes(StandardCharsets.UTF_8));

        DecoderException decoderException = Assertions.assertThrows(DecoderException.class, () -> {
            embeddedChannel.writeInbound(input);
        });
        Assertions.assertTrue( decoderException.getCause() instanceof IOException);

        Assertions.assertEquals(0, input.refCnt());
    }

when NettyCodecAdapter decoder decode exception param ByteBuf not released
expect refcnt 0 , actual 1
Solution
when decode excpetion . skip all readable bytes let ByteToMessageDecoder release ByteBuf
Brief changelog
Verifying this change

Checklist

[x] Make sure there is a GitHub_issue field for the change (usually before you start working on it). Trivial changes like typos do not require a GitHub issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[x] Each commit in the pull request should have a meaningful subject line and body.
[x] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[x] Check if is necessary to patch to Dubbo 3 if you are work on Dubbo 2.7
[x] Write necessary unit-test to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add sample in dubbo samples project.
[ ] Add some description to dubbo-website project if you are requesting to add a feature.
[x] GitHub Actions works fine on your own branch.
[ ] If this contribution is large, please follow the Software Donation Guide.

@AlbumenJ  PTAL


Clone and WithContext #471

client.Copy() method which will properly set client.cmdable.process = client.Process
And client.WithContext(c) that will do the same + set context.

Overall looks good. Left some minor comments to consider.

Comments are fixed.

Thanks! Merging.


fix broken tests and minor Check Entitlement improvements

Summary

[x] fix broken tests from https://github.com/finos/legend-engine/pull/1437
[x] minor Check Entitlement improvements

How did you test this change?

[ ] Test(s) added
[x] Manual testing (please provide screenshots/recordings)
[ ] No testing (please provide an explanation)

Thanks @YannanGao-gs for fixing this, would be interested to know if I can do something on my part to detect such failures. Thanks again


Lint/ShadowingOuterLocalVariable: a false positive case with case statement and block variable

Expected behavior
No offense with Lint/ShadowingOuterLocalVariable cop
Actual behavior
 W: Lint/ShadowingOuterLocalVariable: Shadowing outer local variable - item.
  array.each do |item|
                 ^^^^

Steps to reproduce the problem
def test(array)
  case array.length
  when 1
    item = array.first
    p item
  else
    array.each do |item|
      p item
    end
  end
end

RuboCop version
1.27.0 (using Parser <IP_ADDRESS>, rubocop-ast 1.16.0, running on ruby 2.7.5 x86_64-linux)
  - rubocop-performance 1.13.3
  - rubocop-rake 0.6.0
  - rubocop-rspec 2.9.0

I thought https://github.com/rubocop/rubocop/issues/10479 would fix this one, but it didn't.

I've confirmed that it does reproduce. Sorry. I'll attempt to fix it.

Oh, I see. I may have made a big mistake.
As has been mentioned at https://github.com/rubocop/rubocop/issues/3067

Ruby hoists local variables to the scope above the if statement.

@koic Do you think I need to revert https://github.com/rubocop/rubocop/pull/10484?


Added an example using actix-web, async_graphql and diesel

Pull Request
Related issue
Fixes #247
What does this PR do?

Adds an example which depivts using actix-web and async_graphql to query records in a postgres database

PR checklist
Please check if your PR fulfills the following requirements:

[x] Does this PR fix an existing issue, or have you listed the changes applied in the PR description (and why they are needed)?
[x] Have you read the contributing guidelines?
[x] Have you made sure that the title is accurate and descriptive of the changes?

Thank you so much for contributing to Meilisearch!

@curquiza can you take a look

Hey, thanks, codewise the example is impressive. That’s awesome; I’m not familiar myself with diesel or graph ql, so having such an example in the repository is awesome!
But when I tried to cargo run --release the example, I got the following error:
= note: ld: library not found for -lpq
          clang: error: linker command failed with exit code 1 (use -v to see invocation)

Any idea why that would be the format?

Hey, thanks, codewise the example is impressive. That’s awesome; I’m not familiar myself with diesel or graph ql, so having such an example in the repository is awesome!
But when I tried to cargo run --release the example, I got the following error:
= note: ld: library not found for -lpq
          clang: error: linker command failed with exit code 1 (use -v to see invocation)

Any idea why that would be the format?

Yes. libpq-dev is required when working with Postgresql. sudo apt install libpq-dev should fix that error


Password Reset

Details

Project URL: Password Reset
Build URL (if applicable):
Read the Docs username (if applicable): cannot remember it, I just have my mail address which has to be associated to an existing account because when I try to create a new account I am told that this mail address is already being used

Expected Result
I have attempted to reset my RTD password multiple times, but have actually received no mail after waiting more than 15 minutes.
Actual Result
At each attempt a web page displays this message:

We have sent you an e-mail. Please contact us if you do not receive it within a few minutes.

Yet No mail. Besides, how do I contact you? This ought to be explained, at least by providing a link.

Support is done on a community basis, an issue here is the best way to contact someone from the community. If you prefer another method of contacting the core team, happy to talk more about a support contract.
I see emails to your servers soft bouncing for whatever reason. Getting this response:
499 Unable to connect to MX servers

Strange. There are no known issues on our own MX servers as far as I know, I keep getting mail from other senders. Also:

I eventually received RTD mails sent to my dimitri.papadopoulos@ address hours later, so at least they hadn't been rejected as spam. Unfortunately the reset link was not valid anymore - if I recall correctly the reason was that subsequent password resets had been requested. Yet I still haven't seen the "last" mail that should have contained a valid link.
I ended up creating a new account associated to dimitri.papadopoulos-orfanos@ and it worked, including a password reset. Mails sent this address have been sent through rapidly.
I suspect the problem is not on my side.

I'll try a single last password reset of the account associated to the dimitri.papadopoulos@ mail address.

Made a new request a few minutes ago, around 2016-10-28 18:50 UTC.

I received the mail with the reset link shortly after. The link worked and I was able to reset the password. I now have access to the DimitriPapadopoulos account.

Still, I'm wondering why it failed before. Why the soft bounces, and specifically on the dimitri.papadopoulos@ (ends up into the same mail box as dimitri.papadopoulos-orfanos@).


Changes to text kerning -- auto-hinting compensation

[x] Has this change been discussed on the forum or in an issue before?
[x] Does the code follow the SFML Code Style Guide?
[ ] Have you provided some example/test code for your changes?
[x] If you have additional steps which need to be performed list them as tasks! -- N/A

Description
Ensures appropriate spacing between letters rendered, particularly at small fonts.
This PR is related to the issue #1642
Tasks

[ ] Tested on Linux
[x] Tested on Windows
[x] Tested on macOS
[ ] Tested on iOS
[ ] Tested on Android

Please correct me if I'm wrong, but I believe this was superseded by #1746, correct?

Yes. I had some criticisms of that PR related to optimization, but now that it is merged into master a new PR will be required if I ever have time to address those.

Feel free to also continue the original discussion and/or open a new issue if you think there's something that needs addressing.
Unfortunately some of the stuff went a bit over my head and I'd have to read more of SFML's code. 😅


Laravel 10 Mid-Shift

This pull request includes recent changes to Laravel skeleton files. Although some of these changes may be considered optional, they may be required to use recent additions - such as the hashed attribute cast and log processors. This also includes new configuration options and skeleton slimming.
Since Laravel has moved to an annual release cycle, changes since the initial release may reach a critical mass. When they do, you will receive this automated pull request as part of your Shifty Plan.
Before merging, you need to:

Checkout the shift-mid-laravel-10 branch
Review all pull request comments for additional changes
Run composer update (if the scripts fail, add --no-scripts)
Clear any config, route, or view cache
Thoroughly test your application (no tests?, no CI?)

If you would like more detail you may view the full changes. You may also stay informed of weekly Laravel release highlights through the Shifty Bits Newsletter.

:warning: Shift was not able to automate the updates to some of the core Laravel files. You should compare the following files against their latest Laravel 10 versions and merge any changes:

[ ] app/Models/User.php
[ ] lang/en/validation.php

:information_source: Shift updated your dependencies with the latest constraints from Laravel 10. While many of the popular packages are also reviewed, you may have to update some packages. Watch dealing with dependencies for tips on handling any package incompatibilities.

:information_source: Laravel added the VITE_APP_NAME environment variable to allow you to customize your application name in frontend components. By default, this uses the value from the APP_NAME environment variable.
Shift added this to your .env files, but you will need to update your frontend components to use this new variable. You may reference the original Breeze PR for examples.


feat: added the prisma dark and light theme

Hi @iamayushdas, thank you for proposing the changes, but React Native uses the custom Prism theme, and using only dark mode code blocks in both themes is a design decision.

Hi @iamayushdas, thank you for proposing the changes, but React Native uses the custom Prism theme, and using only dark mode code blocks in both themes is a design decision.

Okay


Packages list versus vignettes and Docker container

Should an effort be made to bring the packages list in synchrony with what is used in vignettes, and / or what gets installed into the Rpopgen Docker container?
Right now there are packages mentioned (such as LEA) that are neither used in any of the vignettes, nor in the Docker container. And we may well have more packages used in the vignettes and the container than in this list.

I think this is a good idea. Once it's done at this point, it shouldn't be too hard to keep up-to-date with the docker image.

This has been addressed in #163


Add top_k jvp and batching rules and tests

Can you add top_k to the lax docs also? It sounds like we are declaring this "supported" more or less.


Support for storage space data disks

This commit adds support in hcsshim to mount a virtual disk backed by storage spaces as a
data disk into a container. In container config the host_path in the mount entry
should use the format space://{storage_space_pool_guid}{storage_space_disk_guid} to
specify a storage space virtual disk.
Signed-off-by: Amit Barve<EMAIL_ADDRESS>

I think it would be more useful long-term if we can just make this feature be generic EVD support.
e.g. instead of space://{guid1}{guid2} can we just make the pattern something like evd://<type>/<path>? Then for spaces they would pass evd://space/{guid1}{guid2}, but other types would work as well without needing more work on our side.

@kevpar I like the idea of not having to make any other changes to support other types of EVDs. I just didn't do this because I didn't know if we wanted to keep the EVD part as an implementation detail.

This needs a rebase to grab the schema location changes (sorry 😞)


Support multiple messages in the same request

I would like to send multiple messages to the bot service in the same request.

Yes, your solution works perfectly (in my case), I wrote an post call to callbackUrl sending the notification (same as reminders or trivia plugins do).
I also implemented an simple function to do this more easier.
function notifyProcess(target, message){
    let body = {
        chat: { id: target.chat.id },
        text: `*[🕗 ${message}]*`
    };
    axios.post(target.callbackUrl, body)
    .then(async response => {
        if (response.status == 200) {
            console.log(`Received back: ${JSON.stringify(response.data)}`);
        }
    });
}

Thanks in advance, nice work!.

Yeah I should really add a nicer way to send a callback message, thanks for the suggestion. Glad it worked out for you!


Claim button should have animation

The claim button should have animation between the 2 states
maybe use
https://www.npmjs.com/package/react-native-card-flip

Verified as done on the Dev env v0.18.1-0
Devices:
iPhone 11 // 13.3
iPhone 6 // iOS 12.1.1
Redmi 7 // Android 9.0


Do not export FC/CC/CXX if already set

Currently the code will default to GNU.

For the moment I will not change this. This can have some strange side effects. Not so clear what to do
in this situation:
$ FC=foo ./setup --fc=bar


Lightbox with Mosaic

When I try to add the popular lightbox plugin, then jquery mosaic won't work.
This works, even with lazyload added:

But this won't work:

I tried to add the image as background of a div, but that also won't work.
Is there any way to make it work, since I really like jquery mosaic more than masonry.js

Hi @FullDome! I might need to do some tests, but unless the lightbox plugin is messing too much with the dom, it should work.
jQuery Mosaic needs to know either the width and the height, or the aspect ratio of each element, and I think the problem you're having might be related to that.
Indeed, jQuery Mosaic works fine also with  and  elements instead of , but you should be giving him a hint on their aspect ratio to let him do his thing. I think your example is not working because you're not giving the  element any implicit size, try this:

or this:

Of course you would need to know these sizes (or the aspect ratio) in advance, that would be the downside. But if you're building your mosaic with PHP or similar, it shouldn't be a problem.
Cheers!

Thank you for your help!
The image size or ratio is being handled by the lightbox. The smaller image is the one being handled with jquerymosaic.
Even if I add the size or resolution for the big image, jquerymosaic won't work.
I noticed that when I add a link, it doesn't work. So it probably doesn't concern the lightbox, but a href:

Hi, I think you're having conflicts by using two plugins that both might be trying to set the size of an element. It is true that jQuery Mosaic will read the width and height of each element if it has been set (either manually, or, say, by another plugin like Lightbox), but you'll need to be sure that jQuery Mosaic is called after Lightbox has done its thing.
I suggest the possible following scenarios for you to follow:

Find out whether Lightbox has some kind of a callback function that is called after the lightbox has been set up and try calling jQuery Mosaic in that function to ensure Lightbox has set the sizes before calling jQuery Mosaic.
Having two plugins messing with image sizes at the same time seems too overkill to me. I would try to use the Lightbox only for the purpose of showing a zoomed version of the image in a popup, not to set image sizes in my DOM.

Let me know if that works for you!

The padding is added on my page, but also yours. I thought it came from the lightbox plugin, but it appears when I add a link to the images. Also on your example page. See image attached, where I added a link and copied the image twice:

It's not a padding, it looks more like the plugin hasn't been executed, or it took the wrong sizes. I'm afraid I would need to take a look at your code, just having some description of the behavior and screenshoots won't do the trick.
Send me your code so I can reproduce the problem here and I might be taking a quick look at it whenever I can. The tin cat is very busy. Please check all my suggestions first.
Meow.

Dear tincat,
I used your suggestions. But adding a link to an image in my code or your code makes the same thing.
The screenshot is your code from your website. I just added a link to an image in the mosaic div.
If you add an url or image link to a photo inside the jquerymosaic, the plugin won't excecute.

The problem is that there are many thins that could cause this. For example, It seems to me the plugin is working well and the  has been resized to the adequate size, but you're expecting it to also resize the image you placed inside the , this is not what the plugin does. Take into account that the plugin resizes the container element, it doesn't touches anything inside.
There's another example in the source (https://jquery-mosaic.tin.cat/example_advanced.php) to exemplify what you're trying to do. You'll see in this example that each image in the mosaic is linkable (which is what you're trying to achieve), but in order for the image to be resized, it's being applied as a background image, with a background-size that allows for it to grow to the container's size. Since jQuery Mosaic resized the container and not the contents, this is one possible solution to achieve it.
Take a deep dive into this example and let me know if that solves your doubt.
Cheers!

Applying an image as background image made it possible to use a link!

Thank you very much!

Glad you solved it! Cheers!


UI display issue with Vietnamese language

Update: another bug here

It seems that the design of the UI has obviously not reached that stage yet.
Taking the "appearance" as an example, the web version can easily have no problems, but for the mobile phone buttons that can only accommodate two lines of text, the Vietnamese text and text in some other languages are obviously "too long".

It seems that the design of the UI has obviously not reached that stage yet.
Taking the "appearance" as an example, the web version can easily have no problems, but for the mobile phone buttons that can only accommodate two lines of text, the Vietnamese text and text in some other languages are obviously "too long".

Yeah, the dev team will try to fix them


Added Sissi to the cat name list

Our old cat was named Sissi so added it

Sorry, should have been clear about the list being locked :)
https://github.com/sindresorhus/cat-names/commit/df4a532e0336884727a54b6126a65355f2ece968


uv run --script - does not parse inline script metadata

I would expect that adding the --script flag didn't disable reading of inline script metadata from stdin, but it seems to be the case.
$ xsel -b
# /// script
# requires-python = ">=3.10"
# dependencies = [
#     "urllib3",
# ]
# ///

import urllib3
$ xsel -b | uv run -v -
DEBUG uv 0.5.10
Reading inline script metadata from `stdin`
DEBUG Using Python request Python >=3.10 from `requires-python` metadata
DEBUG Searching for Python >=3.10 in virtual environments, managed installations, or search path
DEBUG Searching for managed installations at `.local/share/uv/python`
DEBUG Found managed installation `cpython-3.11.9-linux-x86_64-gnu`
DEBUG Found `cpython-3.11.9-linux-x86_64-gnu` at `/home/zii/.local/share/uv/python/cpython-3.11.9-linux-x86_64-gnu/bin/python3.11` (managed installations)
DEBUG Caching via interpreter: `/home/zii/.local/share/uv/python/cpython-3.11.9-linux-x86_64-gnu/bin/python3.11`
DEBUG Using request timeout of 30s
DEBUG Solving with installed Python version: 3.11.9
DEBUG Solving with target Python version: >=3.11.9
DEBUG Adding direct dependency: urllib3*
DEBUG Found stale response for: https://pypi.org/simple/urllib3/
DEBUG Sending revalidation request for: https://pypi.org/simple/urllib3/
DEBUG Found not-modified response for: https://pypi.org/simple/urllib3/
DEBUG Searching for a compatible version of urllib3 (*)
DEBUG Selecting: urllib3==2.2.3 [compatible] (urllib3-2.2.3-py3-none-any.whl)
DEBUG Found fresh response for: https://files.pythonhosted.org/packages/ce/d9/5f4c13cecde62396b0d3fe530a50ccea91e7dfc1ccf0e09c228841bb5ba8/urllib3-2.2.3-py3-none-any.whl.metadata
DEBUG Tried 1 versions: urllib3 1
DEBUG marker environment resolution took 0.043s
Resolved 1 package in 43ms
DEBUG Assessing Python executable as base candidate: /home/zii/.local/share/uv/python/cpython-3.11.9-linux-x86_64-gnu/bin/python3.11
DEBUG Using base executable for virtual environment: /home/zii/.local/share/uv/python/cpython-3.11.9-linux-x86_64-gnu/bin/python3.11
DEBUG Ignoring empty directory
DEBUG Using request timeout of 30s
DEBUG Requirement already cached: urllib3==2.2.3
Installed 1 package in 7ms
 + urllib3==2.2.3
DEBUG Using Python 3.11.9 interpreter at: /home/zii/.cache/uv/archive-v0/BKxFXOwvV9ggZUwFrS0mg/bin/python3
DEBUG Running `python -c`
DEBUG Command exited with code: 0
$ xsel -b | uv run -v --script --isolated -
DEBUG uv 0.5.10
DEBUG No project found; searching for Python interpreter
DEBUG Searching for default Python interpreter in virtual environments, managed installations, or search path
DEBUG Searching for managed installations at `.local/share/uv/python`
DEBUG Found managed installation `cpython-3.11.9-linux-x86_64-gnu`
DEBUG Found `cpython-3.11.9-linux-x86_64-gnu` at `/home/zii/.local/share/uv/python/cpython-3.11.9-linux-x86_64-gnu/bin/python3.11` (managed installations)
DEBUG Creating isolated virtual environment
DEBUG Assessing Python executable as base candidate: /home/zii/.local/share/uv/python/cpython-3.11.9-linux-x86_64-gnu/bin/python3.11
DEBUG Using base executable for virtual environment: /home/zii/.local/share/uv/python/cpython-3.11.9-linux-x86_64-gnu/bin/python3.11
DEBUG Ignoring empty directory
DEBUG Using Python 3.11.9 interpreter at: /home/zii/.cache/uv/builds-v0/.tmp7lfpM3/bin/python
DEBUG Running `python -`
Traceback (most recent call last):
  File "<stdin>", line 9, in <module>
ModuleNotFoundError: No module named 'urllib3'
DEBUG Command exited with code: 1

Also, is it not supported to read a script from stdin and pass along command line arguments at the same time?
I can't get this to work:
$ echo '# /// script
     # requires-python = ">=3.10"
     # dependencies = [
     #     "urllib3",
     # ]
     # ///

     import sys; print(sys.argv)' | uv run - 1 2 3
Reading inline script metadata from `stdin`
['-c']

Placing a -- before or after - doesn't help either.

I'm guessing these are bugs.

Thanks! 🎉


support for DNS

When I put in an IP address it works perfect. When I put in a DNS address that resolves to it nothing works. The Smarthub is also on DNS and using the same DNS servers so I'm pretty sure it should be resolving the DNS name just find. I don't know if this a DH code issue simply expecting an IP and not supporting a DNS string or if it's a problem with the smartthings hub not being able to do that.

This is a ST hub limitation and work arounds are terrible.


indexer: exclude deleted/wrapped objects from snapshot & checkpoint batch fix

Description
https://mysten-labs.slack.com/archives/C0578KFD9D2/p1726765436840499
this will cut storage further down
Test plan
How did you test the new or updated feature?

Release notes
Check each box that your changes affect. If none of the boxes relate to your changes, release notes aren't required.
For each box you select, include information after the relevant heading that describes the impact of your changes that a user might notice and any actions they must take to implement updates.

[ ] Protocol:
[ ] Nodes (Validators and Full nodes):
[ ] Indexer:
[ ] JSON-RPC:
[ ] GraphQL:
[ ] CLI:
[ ] Rust SDK:
[ ] REST API:

this might break a bunch of graphql tests because we expect partial data (ObjectRef) to be available, and we map that to ObjectStatus::WrappedOrDeleted. Now that we are excluding wrapped/deleted objects from the snapshot, these will yield empty results

@wlmyng thx for the quick review! do I just need to look out and try to fix those tests, or there are real dependencies that we should hold on this change?

The dependency is real today, so the first step would be to remove the references to this kind of information from the GraphQL side.

@amnn sounds good, I also found a bug while writing this pr, I will split them then.

do we have linear issue tracking, can we make one if not?

I don't think so -- feel free to add one.

I am merging the other pre-requisite pr https://github.com/MystenLabs/sui/pull/19737 and this pr is now ready for another review @bmwill @wlmyng @amnn thanks!


[BUG] /i/ links dont embed and cause a broken pipe error

Describe the bug
Sending an uncached /i/ link through twxtter will cause it to not embed, and freeze on downloading, which will give broken pipe errors once it times out. Interestingly this doesnt block other downloads. On further testing it does block other downloads.
To Reproduce
Steps to reproduce the behavior:
Send an /i/ twxtter video link through discord
it wont embed
send the non /i/ version
it will embed
send the /i/ version again
It will embed
Expected behavior
/i/ links should embed
Screenshots

UPDATE: It seems like twitter's v2 RESTful api is having issues, and has for a month, this is most likely why tweets are on and off working. The same issue is present for other twitfix forks

UPDATE 2: Migrated server, seems to be fixed


[Enhancement] Insert seed with BIP-39 index numbers instead of words

There are many seed backup solutions around nowadays that use the BIP-39 words index (digits from 1 to 2048) to back up the seed. Recently I also reviewed the "Krux" device and they have the option to import the seed into the device using only the numbers index instead of writing the words. I think if might be a good idea to add this for easy input with numbers.
Example from Krux:
https://github.com/selfcustody/krux/blob/main/src/krux/pages/login.py
Look for "load_key_from_digits"

I am guessing this made sense for Krux because of hardware input limitations. The downside to this approach is that while BIP39 words have built-in context and meaning for users (humans think predominantly in words), numbers inherently have less context and would seemingly be more error prone (though of course the final word would still be an error flag). That said, I am not be opposed to incorporating this option -- I would be curious to get thoughts from @kdmukai on upside/downside and what workflow / menu options might look like.

Basically this is what the SeedQR does, which seems to evolve into something like a standard now with Sparrow Wallet supporting it too. I see three problems here for manual interaction:
a) Is it 1 to 2048 or 0 to 2047? Or implement both and choose?
b) Which language? English only?
c) Why would you do that? Defeats the purpose of the mnemonic. Here's what BIP-39 says:

A mnemonic code or sentence is superior for human interaction compared to the handling of raw binary or hexadecimal representations of a wallet seed. The sentence could be written on paper or spoken over the telephone.

This guide is meant to be a way to transport computer-generated randomness with a human-readable transcription.

They handle the word_num - 1 problem, but I hate that they assume the user is entering indices from a list that starts from 1.
BIP-39 has a trivial yet nightmarish problem with these index numbers. Exacerbated by Github displaying the wordlist file with usual line numbers (starting at 1) and even backup products using 1-based indices in the lists they distribute with their products.
I don't think it's a bad idea to add index number entry, but I would find it very hard to stomach incorrect 1-based index numbers. This is probably a losing battle but I'd rather keep educating people that the actual technical spec starts the indices at 0.
And now that I think about it, if this isn't implemented, there is already a simple workaround: the human looks up the wordlist on their own. I don't think the slight extra convenience is worth having this brought into the UI. Optionality is always good, but only if it solves an otherwise thorny or impossible problem for the user.
So I'd say NACK.

If you already have your index numbers, it doesn't seem like much work to reference the official BIP-39 wordlist online and enter the words corresponding to the index numbers in your seed documentation.
In a vacuum I like the option of being able to manually enter your seed word index numbers as proposed here (more options are better), but the potential user confusion around whether your numbers start from 0 or 1 is probably a bigger stumbling block than simply expecting that user to reference the word list to enter words.

Someone spoke of this today in the telegram group.
A thought on ux for this feature:

It matches the line numbers for the bip39 words list file in github.  (This used to make me cringe.  But recently I've flipped a bit in my brain that says: "Numbers" are for humans, and they start at 1.  "Index" or "Indexes" are for programmers they start at 0., and since then, it bothers me less.
A 10 digit number pad is easier to traverse than a 26 letter keyboard.
There are more chances to short-circuit to the correct word:  User hits enter as soon as the number is expressed (could be 1, 2, 3, only half the time will it be 4.  When the first digit is 3 or higher, it cannot be more than 3 digits total; when the first digit is 2 and the second digit is 1 or higher, same.
It's yet another way to backup a bip39 mnemonic that is arguably easier to conceal and/or to hide in plain sight?


fix(plugin-legacy): add invoke to modern detector to avoid terser treeshaking

Description
fixdetectModernBrowserDetector in @vite/plugin-legacy.Because terser will remove empty functions that are not being called which will cause detector different between html and entry file.For example:

Additional context

What is the purpose of this pull request?

[X] Bug fix
[ ] New Feature
[ ] Documentation update
[ ] Other

Before submitting the PR, please make sure you do the following

[X] Read the Contributing Guidelines, especially the Pull Request Guidelines.
[X] Check that there isn't already a PR that solves the problem the same way to avoid creating a duplicate.
[X] Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. fixes #123).
[X] Update the corresponding documentation if needed.
[ ] Ideally, include relevant tests that fail without this PR but pass with it.

done


the bot is clicking next to the river not center of the mid

as the title says how can i fix it?

Is your character running into the river?

Is your character running into the river?

yes

I will look into this, in the mean time, try making your minimap a smaller size and it should fix this issue.

Alright thanks, also would be extremely nice if you could add aram gamemode too in this because its still like the same thing, if it can click in the middle and just hit minions would be super nice


What's the situation? How to solve the limitation state?

ServerObject::isAutoStart tars.auth not allow to restart in limited state

A manual restart will remove it.

手动重启一次就解除了。


Manager.create_or_get does not respect composite keys

When create_or_get tries to get the row that prevented it from creating.  It does this by selecting on unique columns.  However, in a composite key the individual columns are not unique, only the combination is.  It appears that it does not include the composite key columns in the select.
Test that fails:
import asynctest
from peewee import CompositeKey, IntegerField, Model, TextField
from peewee_async import Manager

class CompKeyModel(Model):
    class Meta:
        table_name = 'generated_image'
        primary_key = CompositeKey('task_id', 'product_type')

    task_id = IntegerField(null=False)
    product_type = TextField(null=False)
    misc = TextField(null=True)

class AsyncPeeweeManagerTests(asynctest.TestCase):
    def setUp(self):
        self.db = # code to set up database...
        self.db_mgr = Manager(self.db)

    async def test_create_or_get_composite_key(self):
        await self.db_mgr.create(CompKeyModel, task_id=1, product_type='a', misc='13')
        await self.db_mgr.create(CompKeyModel, task_id=2, product_type='b', misc='15')
        entry, was_created = await self.db_mgr.create_or_get(CompKeyModel, task_id=2, product_type='b')
        self.assertFalse(was_created)

        # fails, entry can return (1, 'a', '13)
        self.assertEqual(2, entry.task_id)
        self.assertEqual('b', entry.product_type)
        self.assertEqual('15', entry.misc)

Here is suggested code for fixing the problem:
    async def create_or_get(self, model_, **kwargs):
        try:
            return (await self.create(model_, **kwargs)), True
        except IntegrityErrors:
            query = []
            composite_key_field_names = model_._meta.primary_key.field_names if model_._meta.composite_key else ()
            for field_name, value in kwargs.items():
                field = getattr(model_, field_name)
                if field.unique or field.primary_key or field_name in composite_key_field_names:
                    query.append(field == value)
            return (await self.get(model_, *query)), False

The method is deprecated. aio_get_or_create will work for you after release


Scene.load() failed to load in Edge with IPV6 loopback address

TANGRAM VERSION:
v0.14.2 (minified, tried using both web and local versions)
ENVIRONMENT:
Microsoft Windows 10 Home Version 10.0.16299 Build 16299
Microsoft Edge 41.16299.15.0
ISSUE:
When loading a page with a Tangram instance in Microsoft Edge when accessing page on local web server using IPV6 loopback address ([::1]), an error ensues and the map does not load:

This does not occur when using another browser or when accessing the page using IPV4 loopback address (<IP_ADDRESS>). Paths to YAML files show as follows:

OBSERVATIONS:
I also tested this using debug version, and I believe the issue is in the highlighted line, as Firefox appropriately returns brackets around the IP address for the IPV6 loopback address when displaying the output for base_info.host, while Edge does not. I am not familiar enough with how the host method works to suggest an alternate implementation that will not break in other use cases:

Hi, sorry for the delay. I've also been unable to find details on this issue, and I don't have experience working with IPv6 addresses. It's not surprising if there's a slight behavior difference in Edge, you can see from the screenshotted code above, it tries to account for variances in browsers' support of the origin field on an <a> element, and it would be consistent to see some difference there is how IPv6 is handled as well.
Do you have a minimal example of how one would set this up locally?


Add "help" feature to the tool which allows users to read more about features

as many user have trouble setting the tool up correctly and using it, a "help" should be added containing documents that allow quick insight about how to use a feature
should be able to support many topics, display a topic instantly (eg. when pressing help in the workspace dialog) and should allow for image display (maybe simple HTML view?)

partially will be fixed via https://github.com/ArmA-Studio/ArmA.Studio/pull/51 which introduces a help button to the Workspace Selector


Use pure OCaml library to show diffs

This uses the simple-diff library and my own code to render the diffs in the usual Unified format.
Closes #2
Current status:

[x] need to merge the fix https://github.com/gjaldon/simple-diff/pull/4
[x] need tests in Testo
[x] fix failing tests

PR checklist:

[ ] PRESERVE THE COMMITS WHEN MERGING
[x] Purpose of the code is evident to future readers
[x] Tests are included or a PR comment includes a reproducible test plan
[x] Documentation is up-to-date
[x] A changelog entry was added to CHANGES.md for any user-facing change

Check out CONTRIBUTING.md for more details.

Merging with a merge commit to preserve the history of simple-diff.


Introduce copy_from

Codecov Report

Merging #151 into master will decrease coverage by 0.06%.
The diff coverage is 87.17%.

@@            Coverage Diff             @@
##           master     #151      +/-   ##
==========================================
- Coverage   88.28%   88.22%   -0.07%
==========================================
  Files          12       12
  Lines         777      815      +38
==========================================
+ Hits          686      719      +33
- Misses         91       96       +5

Impacted Files
Coverage Δ

ssds/blobstore/__init__.py
72.46% <66.66%> (-0.68%)
:arrow_down:

ssds/blobstore/gs.py
90.55% <69.23%> (-2.44%)
:arrow_down:

ssds/blobstore/s3.py
98.05% <100.00%> (+0.34%)
: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 57d0870...85ec452. Read the comment docs.


GPUs for 5-way 50-shot

Hi, could you please tell us how many GPUs we need to train a few-shot model (Res-12+any classifier) in the setting of 5-way 50-shot. Also, is it necessary？ I mean, 50 samples may not be considered as "few-shot"

@christo4ferris One GPU with 12189MiB is enough.

OK. Thank you for letting me know that.


[SHORT FUSE] submit abstact for AMS Conference on Environmental Information Processing Technologies (due Sept. 8)

Sorry for the very late notice, but I realized tomorrow is the deadline for AMS2022 abstract submission
It would be really great to get a Pangeo Forge abstract submitted to https://annual.ametsoc.org/index.cfm/2022/program-events/conferences-and-symposia/38th-conference-on-environmental-information-processing-technologies/
Three potentially relevant sessions are

Cloud Computing for Big Data in Atmosphere, Ocean, and Climate (Joint with 21st Conference on Artificial Intelligence for Environmental Science, 12th Symposium on Advances in Modeling and Analysis Using Python, and  the Eighth Symposium on High Performance Computing for Weather, Water, and Climate)
Weather imposes constraints on human activity. As a consequence, most decision-makers/planners seek awareness to mitigate or eliminate weather impacts.  As datasets become larger and larger, new and improved tools to work with Big Data are critical.  This session welcomes contributions from research fields such as scientific visualization, information visualization or visual analytics that are applicable to large data sets from climatology, meteorology or related disciplines.  Presentations on using cloud computing for analyzing satellite and model data for weather, ocean, or climate relevant applications will also be welcomed.
This session will showcase solutions for analyzing and processing data from satellites such as GOES-16, GOES-17, Himawari-8, NOAA-20, and NPP while utilizing cloud computing. The NOAA Big Data Program has published years of satellite data into the cloud where it is co-located next to compute resources. Processing of this data can now happen in near real time with push notifications upon arrival of new data or over the entire period of record for satellite reanalysis.

Developing Cloud-based Tools for Data Analysis and Archiving  (Joint with 21st Conference on Artificial Intelligence for Environmental Science, 12th Symposium on Advances in Modeling and Analysis Using Python, and  the Eighth Symposium on High Performance Computing for Weather, Water, and Climate)
Cloud-based technologies continue to evolve and mature in their use and application. This session will delve into the growing use of cloud hosting solutions applied to benefit the environmental sciences and specifically applications for data analysis, visualization and archiving of  environmental information. This session encourages submissions on these topics, focusing on how submitters are incorporating cloud-based applications into their work, what they have done, and what advantages this approach has provided to their efforts.

FAIR and Open Data and Software within the Atmospheric and Ocean Sciences to Support  Replicable Research and Reusable Tools for Climate Analysis  (Joint with 25th Conference of  Atmospheric Librarians International and the 12th Symposium on Advances in Modeling and Analysis Using Python)
The need to understand linkages between climatic extremes and security outcomes such as displacement/migration, public health, and food/electricity production has never been higher. Analysis of these linkages requires integration of the Earth, social, economic, and political sciences.   In this session, we invite submissions that discuss projects, initiatives, and ideas that advance community understanding and build upon the principles of FAIR (Findable, Accessible, Interoperable, and Reusable) Open Data and Software within the atmospheric and related sciences. We welcome technical, policy, and community-focused submissions on a range of topics related to facilitating transparency and reproducibility of science, including: data and software discoverability, metadata creation and management, data and metadata formats, provenance tracking, distribution and storage, and other relevant topics.  Platforms for reproducible research and reusable tools will accelerate the analytics enterprise and build the salience, credibility, and legitimacy required to effectively inform policy. This session will highlight emerging open-science tools and platforms for climate-security analysis.

@cisaacstern - how would you feel about recycling your AGU abstract for an AMS submission?
@martindurant, @rsignell-usgs, @lsterzinger - have you made (or are planning) a submission on fsspec-reference-maker?

@rabernat I am not submitting a fsspec-reference-maker abstract to AMS but I have submitted one to AGU

I have a references abstract accepted for PyData Global. I don't know if you can view https://pydata.org/global2021/proposals/153/ , I can paste the MD form.

I can't see that, can you paste it?

"All you need is zarr.: Parallel access to remote HDF5, TIFF, grib2 and others."
Summary
We introduce ReferenceFileSystem, a virtual implementation for fsspec which views arbitrary byte chunks at specific keys, presenting chunks of HDF5, TIFF, grib2 and others at the appropriate paths conforming to zarr's model. Thus, you can use zarr to load data from potentially thousands of remote data files, selecting only what you need, and with parallelism and concurrency.
Outline

the standard formats used for massive data archiving and their shortcomings
brief introduction to zarr, designed for parallel access to remote data
the process by which we can expose archival chunks to zarr and aggregate datasets
examples of >>10TB datasets where we have done this

Description
fsspec's ReferenceFileSystem allows a file system like virtual view onto chunks of bytes stored in arbitrary locations elsewhere, e.g., cloud bucket storage. We can present each byte chunk as a particular path in the filesystem conforming to the zarr hierarchy model, such that the original set of chunks, potentially across many files, appears as a single zarr dataset. This brings the following advantages:

only zarr (plus the relevant codecs) is required to read the data, but the original data could be locked in HDF5, TIFF or grib2 files (and more to come)
the metadata is extracted once, so future opening of the dataset does not need to scan through the target files to find metadata, and so the process is much faster
you get a single logical view over potentially thousands of files, but due to zarr's access model, you only load the data you need
loading can happen chunk-wise and in parallel

The details of how to make such reference files is described at https://github.com/intake/fsspec-reference-maker , and the latest result for some 80TB/370k files of water flow modelling can be seen at https://nbviewer.jupyter.org/gist/rsignell-usgs/02da7d9257b4b26d84d053be1af2ceeb . Note that this is using xarray to process the data, but only zarr is needed to load it.

(NB: I am the only author on this proposal, but pydata allows you to add authors any time, so I was leaving that until later)

@rabernat, instead of the above-listed sessions, how about we submit to:

12th Symposium on Advances in Modeling and Analysis Using Python
New Python Tools in the Atmospheric and Oceanographic Sciences
Topic Description: Submissions are welcomed to new Python tools in the atmospheric and oceanographic sciences. Submissions must specifically either introduce new tools or present significant progress (feature additions) to existing tools. If submitters are wishing to present on tool use please submit to other sessions in the Python Symposium.

?

Given the cloud focus of Pangeo Forge, I think that the "Developing Cloud-based Tools for Data Analysis and Archiving" would probably be the most appropriate than the "New Python Tools." But totally your call as the abstract lead.

FWIW I submitted an abstract to

Cloud Computing for Big Data in Atmosphere, Ocean, and Climate (Joint with 21st Conference on Artificial Intelligence for Environmental Science, 12th Symposium on Advances in Modeling and Analysis Using Python, and the Eighth Symposium on High Performance Computing for Weather, Water, and Climate)
Weather imposes constraints on human activity. As a consequence, most decision-makers/planners seek awareness to mitigate or eliminate weather impacts. As datasets become larger and larger, new and improved tools to work with Big Data are critical. This session welcomes contributions from research fields such as scientific visualization, information visualization or visual analytics that are applicable to large data sets from climatology, meteorology or related disciplines. Presentations on using cloud computing for analyzing satellite and model data for weather, ocean, or climate relevant applications will also be welcomed.
This session will showcase solutions for analyzing and processing data from satellites such as GOES-16, GOES-17, Himawari-8, NOAA-20, and NPP while utilizing cloud computing. The NOAA Big Data Program has published years of satellite data into the cloud where it is co-located next to compute resources. Processing of this data can now happen in near real time with push notifications upon arrival of new data or over the entire period of record for satellite reanalysis.

which references pangeo-forge (see text below), but only as part of a more high level workflow. So I would tend to the other session (which @rabernat suggested) to give wide visibility to pangeo efforts across sessions? Happy to coordinate some of the actual presentation details.
Here is the submitted abstract:
CMIP6 in the cloud - Open, fast and accessible climate science with Pangeo
The Coupled Model Intercomparison Project (CMIP) provides the base on which much of modern climate/earth system science is built. The model output produced is crucial to inform current and future policy decisions about a changing climate due to anthropogenic greenhouse gas emissions.
Continued improvements in computing power,  model development, and scientific understanding of the earth system lead to more and more complex models and rapidly increasing size of output datasets.
Datasets of the current and future generation of global earth system models render the common 'download and analyze' workflow inefficient, blocking innovative analysis and fast scientific discoveries needed to deal with the dramatic changes happening to the planet.
To overcome these limitations, we present a workflow to analyze CMIP6 data in the cloud. Python tools and cloud infrastructure developed within the Pangeo community enables near-instantaneous access and fast data processing to much of the CMIP6 archive with only a web browser.
The tools presented here demonstrate a wide range of interoperable, efficient, community-based solutions. Starting with analysis-ready cloud optimized data provided by Pangeo-Forge, over labeled dataset representation with xarray, parallelized computing with dask to specialized solutions to enable easy and fast analysis specifically for CMIP6 data with cmip6_preprocessing.
We summarize by demonstrating a complete workflow in which we calculate global surface temperature time series.

Timing
The below-linked abstract is due by 8:59 PM PT / 11:59 PM ET  today. It is now 1:41 PM PT / 1:41 PM ET. To allow everyone a chance to review, but also not cut it too close,
I will submit at 6 PM PT / 9 PM ET today.
Any suggestions will be incorporated at that point and anyone who's @'ed below and has either made a suggestion or, alternatively, just read the draft and provided a  👍  on this comment, will be a co-author.
Draft link
@rabernat, @jbusecke, @rwegener2, @jhamman: here's a Google Doc with the AMS 2022 draft for your review. Please make edits in Suggesting mode. If you have no edits, just  👍 this comment.
I've provided our AGU abstract below the AMS draft, for reference. AMS allows 1000 words, so I had room to expand on what we'd written for AGU with more detail.
@martindurant, @rsignell-usgs, @lsterzinger: I mention reference maker, so would appreciate your eye to make sure I haven't misrepresented it. And even if you don't have any edits, but would like to be co-authors, please also reply to this comment with 👍 .
Finally (and apologies for the @'ing), if anyone from @pangeo-forge/dev-team would like to be a co-author, also just make sure you agree with the draft (or provide comments), and either way reply to this comment with 👍 . All of you have contributed in one way or another, so specifics edits are not expected for co-authorship.
Session choice
Based on discussion in this thread, we'll submit to

Developing Cloud-based Tools for Data Analysis and Archiving (Joint with 21st Conference on Artificial Intelligence for Environmental Science, 12th Symposium on Advances in Modeling and Analysis Using Python, and the Eighth Symposium on High Performance Computing for Weather, Water, and Climate)
Cloud-based technologies continue to evolve and mature in their use and application. This session will delve into the growing use of cloud hosting solutions applied to benefit the environmental sciences and specifically applications for data analysis, visualization and archiving of environmental information. This session encourages submissions on these topics, focusing on how submitters are incorporating cloud-based applications into their work, what they have done, and what advantages this approach has provided to their efforts.

For what it's worth, here is my abstract submitted to AGU ("Open Science in Action" Session) last month

@cisaacstern I worked on fsspec-reference-maker over the summer during an internship with NCAR. I'm not sure what should/shouldn't qualify me for an author spot on your abstract, but I'm happy to have my name on there if appropriate! @rsignell-usgs and @martindurant are the real experts though 😉

This abstract has been submitted! 🙌
All who provided inline comments on the Google Doc and/or 👍 'ed https://github.com/pangeo-forge/roadmap/issues/32#issuecomment-915439859 were listed as co-authors.
Thanks to everyone for rallying together to get this done on such short notice.

Amazing work Charles! Thanks for getting this done so well on a tight timeline.
It will be an awesome meeting, and I'm glad that our project will be well represented.

This abstract has been accepted to AMS 2022: https://ams.confex.com/ams/102ANNUAL/meetingapp.cgi/Paper/399850


Support 'next ticket' and 'previous ticket' in TicketShow

The TicketShow page comes from the results of a query in TicketList. There is value in being able to step through tickets in this list, without the cost of going back to the TicketList page.
Use 'n' and 'p' keys for this.

'n' is now taken. Use 'N' and 'P' instead.

fixed in 8719dc46ee444b515e53159b33f69632302a41ef


False positive unsafe_property_access warning on untyped field

Godot version:

3.2.2.stable.custom_build.bc8eea3e9
OS/device including version:

Linux 5.8.12-arch1-1
Issue description:

When an object field is declared without a type (because it is a Variant, and I don't know that you can explicitly specify Variant as a type?),
trying to access that field will warn you about unsafe_property_access, even though it should know that property exists.
Steps to reproduce:
extends Node2D

class Obj:
	var f: float
	var v # Variant

func _ready():
	var o := Obj.new()
	print(o.f)
	print(o.v) # The property 'v' is not present on the inferred type 'Obj' (but may be present on a subtype). [UNSAFE_PROPERTY_ACCESS]

Minimal reproduction project:

For context, my real life example is:
class Action:
	enum Kind {
		NONE, # returned when there is no action left for the current time
		GRAPPLE, # data is True (engage) or False (release)
		LOOK, # data is a Vector2(x_rotation, y_rotation) in radians
		JET, # data is a Vector3 with each coordinate in the range [0,1]
		MAX
	}

	var time: float
	var kind: int  # Kind
	var data

	func _init(t: float, k: int, d):
		time = t
		kind = k
		data = d

class Writer:
	var actions: Array  # Action[]

...snip...

	func save(path: String):
		var file := File.new()
		file.open(path, File.WRITE)
		for a in actions:
                        # arrays are untyped, so I have to do this cast
			var act: Action = a
                        # I was hoping to get warned if I typo a field in the next line
                        # unfortunately I get a false negative on .data
			file.store_line("%.4f %d %s" % [act.time, act.kind, var2str(act.data)])
		file.close()

I'm not quite sure why, but I can't seem to get warning-ignore working either:
	match act.kind:
		InputRecord.Action.Kind.GRAPPLE:
			# warning-ignore:UNSAFE_PROPERTY_ACCESS
			emit_signal("grapple", act.data)  # still warns me

This problem is still present in 3.2.3.stable

This problem is still present in 3.2.3.stable

OP's use case was with variants but (not surprisingly) this affects enums since they can't be explicitly typed. For my use case, having explicitly-typed enums would solve the problem.

OP's use case was with variants but (not surprisingly) this affects enums since they can't be explicitly typed. For my use case, having explicitly-typed enums would solve the problem.

I can confirm the issue in 3.x (7244ea7e20bdb5d0aa5fde61a82f958b1bc5bbec).
It's worth noting that the UNSAFE_PROPERTY_ACCESS warning is not enabled by default, so to reproduce the bug using the OP's script one needs to enable it in the Project Settings.


Document public API

Should mostly be lifted from the time standard library.

Formatting should also be fixed to render properly for web-based docs: https://pkg.go.dev/github.com/noodlebox/clock


Fix #92, LGTM exclude generated files

Describe the contribution
Fix #92 - Exclude generated Ui_* python files from LGTM analysis (2nd try, #94 didn't work)
Testing performed
Submit yml script on LGTM and it still functioned, but doesn't provide enough information to tell if the exclusion worked.  Requires an actual LGTM run.
Expected behavior changes
Exclude Ui*.py files from issue reporting
System(s) tested on
N/A
Additional context
#94 was the first attempt, didn't work.
Contributor Info - All information REQUIRED for consideration of pull request
Jacob Hageman - NASA/GSFC

CCB 2020-07-29, APPROVED

@skliper doesn't look like it worked. Maybe we can use the "file classifier" feature?
https://lgtm.com/help/lgtm/file-classification

You can see which files are evaluated by looking at the Files tab in the report

@astrogeco I did use the "file classifier" feature.  I suggest checking it into master to make sure it doesn't work.  LGTM doesn't seem to always do full analysis on PRs.

@astrogeco I did use the "file classifier" feature. I suggest checking it into master to make sure it doesn't work. LGTM doesn't seem to always do full analysis on PRs.

that's what I get for not looking at your file change :/

Just confirming this worked:


Fix npm audit failures

Nothing change on package.json? Will the change got overwritten if I run npm install?

I ran both npm ci and npm install and there are still zero vulnerabilities, so I think we're good.


PCL library import error

After installing the python-pcl follow readme to install pcl, I still can not import PCL
# install dependencies
sudo apt install libpcl-dev libvtk6-dev
# clone fork with fix for Ubuntu 18.04
git clone https://github.com/Tuebel/python-pcl
cd python-pcl
python3 setup.py install

When I import python3 and import pcl, it occurs error:
ModuleNotFoundError: No module named 'pcl._pcl'

Should follow the official install commands:
https://github.com/Tuebel/python-pcl
(In its Common setting)
# install dependencies
sudo apt install libpcl-dev libvtk6-dev
# clone fork with fix for Ubuntu 18.04
git clone https://github.com/Tuebel/python-pcl
cd python-pcl
python3 setup.py build_ext -i
sudo python3 setup.py install


Update @angular-devkit/build-angular

The previous version was not working in my computer.
The error was "Failed: EPIPE write EPIPE"

@ralph666 when you update package.json file you must update package-lock.json file. They must be synchronized.

Another Pull request has been created.


Adding custom cluster can only support one node

Describe your issue here
When adding a cluster with From my own existing nodes, I found that I can only create a cluster on one node, how can I create a cluster with multiple nodes?

Useful
Info

Versions
Rancher v2.6 UI: v2.6

Route
global-admin.clusters.new

The command is dynamically generated based on the options you select, you can run that command on 1 or 10 hosts, depending on how you want to setup your cluster.
For example, select only etcd and run the command on 3 separate nodes, select only controlplane and run the command on 2 separate nodes, select only worker and run the command on 2 workers to get a decent HA cluster.

Thanks @superseb , this helps ;-)


Bug: "cannot reshape array of size" after optimization is finished

Hi all,
sometimes I get randomly these errors when calling model.solve:
2024-11-13 15:05:34 [INFO]  Optimization successful:
Status: ok
Termination condition: optimal
Solution: 3017580 primals, 254750 duals
Objective: 3.52e+08
Solver model: available
Solver message: optimal
 in constants.info:239
2024-11-13 15:05:34 [ERROR] Attempt 1 failed with error: cannot reshape array of size 65160 into shape (1086,30) in model_builder.solve:276

rebuilding the model (aka initializing a new model class and re-adding constraints variables, etc without andy data changes) usually solves this problem. However, I run 24h simulations for up to 5-6 years in total and every like 20 simulations, I get this error. Also, its not traceable and not really reproducible. It really occurs randomly and happens with cplex and highs

Hey @tburandt, this is quite weird. linopy does not have a model_builder.solve, so I am wondering whether this occurs in one of the other libraries highspy or cplex.

@FabianHofmann hey, yeah sorry, model_builder.solve is my class and in essence only calls model.solve:
    def solve(self, [...]):
        attempt = 0
        while attempt < self.max_retries:
            try:
                logging.info(
                    f"Attempt {attempt + 1}/{self.max_retries} to solve the model."
                )

                model.solve(**solver_kwargs)
                logging.info("Model solved successfully.")
                return model
            except Exception as e:
                [...]


GCM Adapter bug

With the new 2.3.2 library version you change GCMAdapter line 148 from
$serviceMessage->setData($data);
to
$serviceMessage->setNotification($data);
but this is not generally correct because Data Message an Notification Message are quite different in android application (Google official documentation:
https://firebase.google.com/docs/cloud-messaging/concept-options)
Actually we rollbak to 2.3.1 release to fix this problem.
Can you restore old code or otherwise offer a setter methods to support both android notification mode?

Yes, both need to be provided.
@albovieira check this too.

@seyfer what do you suggest?
Maybe add to Sly\NotificationPusher\Model class a property that will define the notification type with a default value?

@albovieira, no, I will just return back an old setter. So, there would be two setters for the same $data.

@domenico-renna ,
the setter for Data returned back and the v2.3.2 tag had been updated.
https://github.com/Ph3nol/NotificationPusher/commit/a86b2be3aa2336ad03be5a5a7adb270144c7f003

Thanks for your quick fix.
I'll like to test 2.3.3 too but I can see that you are you using together
$serviceMessage->setData($data);
$serviceMessage->setNotification($data);

I don't know if your choice can be a problem but Google set a 4KB limit for DATA NOTIFICATION payload (you can find the specification following my origina link) and should be possible that using data+notification in json the limit will be reached before

Using together setter is wrong.
There must be an option that gives you the ability to set notification, data payloads, or both.
Google docs describes the difference:

App behavior when receiving messages that include both notification and data payloads depends on whether the app is in the background or the foreground—essentially, whether or not it is active at the time of receipt.

When in the background, apps receive the notification payload in the notification tray, and only handle the data payload when the user taps on the notification.
When in the foreground, your app receives a message object with both payloads available.

@domenico-renna, @VonSerpe, yes, it was a quick fix.
Definitely, it needs to be separated. When I will have time, I will add the setter. Feel free to do a PR if you will have time earlier.

Thx for the great job i want to ask about these setters will You have time for add ?

Please, somebody test this branch with real devices sends.
https://github.com/Ph3nol/NotificationPusher/pull/162
check documentation changes for an explanation. if it works - I will merge.

Have added setter via an optional class. See docs.
Please, somebody test the branch 157-fix


chore(Dependencies) : update zio-logging-slf4j from 0.5.16 to 2.3.0

About this PR
📦 Updates dev.zio:zio-logging-slf4j from 0.5.16 to 2.3.0 ⚠
📜 GitHub Release Notes - Version Diff
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "dev.zio", artifactId = "zio-logging-slf4j" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "dev.zio", artifactId = "zio-logging-slf4j" }
}]

labels: library-update, early-semver-major, semver-spec-major, commit-count:1

Superseded by #741.


Add doc changes related to MySQL8

Purpose
This PR contains doc changes related to configuring MySQL 8.0.x.
Related issue https://github.com/wso2/product-apim/issues/5501

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: vithu30:x: ruthryiYou have signed the CLA already but the status is still pending? Let us recheck it.


The command '/bin/sh -c apt-get upgrade -y' returned a non-zero code: 100

Trying to build the image from the included Docker file (using my fork), I keep getting this error:
$ docker build -t stevekm/docker-sge .
The command '/bin/sh -c apt-get upgrade -y' returned a non-zero code: 100

Any ideas? Did some Google'ing and included some tests to make sure network access was working inside the container, could figure out what is causing the error. Full output of docker build is here:
https://github.com/stevekm/docker-sge/blob/a30cfb3f687358a63d9f13faff31aa9992fedaa6/build_log.txt

also resembles this old issue: https://github.com/gawbul/docker-sge/issues/4


Audio output functionality

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

No matter what website I go to, I cannot get audio output out from Edge Dev Tools.
Describe the solution you'd like:

Being able to get audio output from Edge Dev Tools.
Describe alternatives you've considered:

I have checked that the same URLs produce audio output in other browsers. In Edge Dev Tools, they don't produce audio output.
Additional context:

This would be a huge feature for a lot of interactive audio and game developers!

Our VS Code extension works 2 ways:

either it displays the target webpage in a browser preview that's embedded in VS Code
or it attaches to a real browser target.

The embedded browser preview is useful when you want to have everything inside VS Code, but is quite limited. It's just a preview, not a real browser and a number of features can't work, such as audio streaming.
If you want audio to work, please use an external browser as described here: https://learn.microsoft.com/en-us/microsoft-edge/visual-studio-code/microsoft-edge-devtools-extension/external-browser-window
Closing this issue based on the above. But please let me know if this doesn't work for you.

Thank you. Switching to non-headless mode with the external automated browser window captures the functionality that I want!